go: show EOF error on last line

When the following program is tested:
package main

import "fmt"

func main() {
  a := 1
  b := 2
  if a + b > 6 {
    fmt.Println(a + b)  // note - no closing brace here
}                       // this is the last line of the file

The error produced, go: syntax error: unexpected EOF, expecting } (E) is beyond the last line of the file and therefore doesn't appear (in the gutter or on a hover).
When a blank line follows the last closing brace, the error appears on that line, but only until the file is saved.
Expected behavior
It would be nice if any errors that were attributed to nonexistent line numbers were associated with the last line of the program.
Steps to reproduce
See above. Use the standard go maker.
Output from :NeomakeInfo
Neomake debug information
Async support: 1
Current filetype: go
Windows: 0
[shell, shellcmdflag, shellslash]: ['/bin/bash', '-c', 0]
makeprg=make
Enabled makers
For the current filetype ("go", used with :Neomake):

go

append_file: 0

args: ['test', '-c', '-o', '/dev/null']

auto_enabled: 1

cwd: '%:h'

errorformat: '%W%f:%l: warning: %m,%E%f:%l:%c:%m,%E%f:%l:%m,%C%\s%+%m,%-G%.%#\[no test files],%-G#%.%#'

exe: 'go'

postprocess: function('neomake#postprocess#compress_whitespace')

serialize: 1

serialize_abort_on_error: 1

version_arg: 'version'

version information (/home/seth/.local/go/bin/go version): go version go1.13.5 linux/amd64

golangci_lint

append_file: 0
args: ['run', '--out-format=line-number', '--print-issued-lines=false', '--enable', 'golint']
auto_enabled: 1
cwd: '%:h'
errorformat: '%f:%l:%c: %m,%f:%l: %m'
exe: 'golangci-lint'
output_stream: 'stdout'
version information (/home/seth/dev/go/bin/golangci-lint --version): golangci-lint has version 1.21.0 built from 645e794 on 2019-10-15T1
8:16:56Z
NOTE: you can define g:neomake_go_enabled_makers to configure it (or b:neomake_go_enabled_makers).
For the project (used with :Neomake!):

makeprg

append_file: 0
args: []
auto_enabled: 1
errorformat: '%[^"]"%f"%\D%l: %m,"%f"%\D%l: %m,%-G%f:%l: (Each undeclared identifier is reported only once,%-G%f:%l: for each functio
n it appears in.),%-GIn file included from %f:%l:%c:,%-GIn file included from %f:%l:%c,,%-GIn file included from %f:%l:%c,%-GIn file include
d from %f:%l,%-G%[ ]from %f:%l:%c,%-G%[ ]from %f:%l:,%-G%[ ]from %f:%l,,%-G%[ ]from %f:%l,%f:%l:%c:%m,%f(%l):%m,%f:%l:%m,"%f", line %l%
\D%c%[^ ] %m,%D%\a[%\d]: Entering directory %['']%f'',%X%*\a[%*\d]: Leaving directory %*['']%f'',%D%\a: Entering directory %[`'']%f'

mode
',%X%\a: Leaving directory %[`'']%f'',%DMaking %*\a in %f,%f|%l| %m'

exe: 'make'
version information (/usr/bin/make --version): GNU Make 4.2.1
Built for x86_64-pc-linux-gnu
Copyright (C) 1988-2016 Free Software Foundation, Inc.
License GPLv3+: GNU GPL version 3 or later http://gnu.org/licenses/gpl.html
This is free software: you are free to change and redistribute it.
There is NO WARRANTY, to the extent permitted by law.
NOTE: you can define g:neomake_enabled_makers to configure it.
Default maker settings:
output_stream: 'both'
remove_invalid_entries: 0
buffer_output: 1

Settings
New-style (dict, overrides old-style)
g:neomake: {
  'automake': {

    'ignore_filetypes': [
      'startify',
    ],
    'events': {
      'BufWritePost': {
        'delay': 0,
      },
      'FileChangedShellPost': {},
      'FileType': {},
      'BufWinEnter': {},
      'InsertLeave': {},
      'TextChanged': {},
    },
  },
  'automake_delay': 750,
}
b:neomake: unset

Old-style

g:neomake_error_sign = {'texthl': 'NeomakeErrorSign', 'text': '✖'}
g:neomake_go_golangci_lint_args = ['run', '--out-format=line-number', '--print-issued-lines=false', '--enable', 'golint']
g:neomake_info_sign = {'texthl': 'NeomakeInfoSign', 'text': 'i'}
g:neomake_logfile = '/tmp/neomake.log'
g:neomake_message_sign = {'texthl': 'NeomakeMessageSign', 'text': '➤'}
g:neomake_place_signs = 1
g:neomake_warning_sign = {'texthl': 'NeomakeWarningSign', 'text': '‼'}

:version
NVIM v0.5.0-291-g0c436559d
Build type: RelWithDebInfo
LuaJIT 2.1.0-beta3
Compilation: /usr/bin/cc -U_FORTIFY_SOURCE -D_FORTIFY_SOURCE=1 -O2 -g -Og -g -Wall -Wextra -pedantic -Wno-unused-parameter -Wstrict-prototype
s -std=gnu99 -Wshadow -Wconversion -Wmissing-prototypes -Wimplicit-fallthrough -Wvla -fstack-protector-strong -fdiagnostics-color=always -DIN
CLUDE_GENERATED_DECLARATIONS -D_GNU_SOURCE -DNVIM_MSGPACK_HAS_FLOAT32 -DNVIM_UNIBI_HAS_VAR_FROM -DMIN_LOG_LEVEL=3 -I/home/seth/build/neovim/b
uild/config -I/home/seth/build/neovim/src -I/home/seth/build/neovim/.deps/usr/include -I/usr/include -I/home/seth/build/neovim/build/src/nvim
/auto -I/home/seth/build/neovim/build/include

LuaJIT 2.1.0-beta3
Compilation: /usr/bin/cc -U_FORTIFY_SOURCE -D_FORTIFY_SOURCE=1 -O2 -g -Og -g -Wall -Wextra -pedantic -Wno-unused-parameter -Wstrict-prototype
s -std=gnu99 -Wshadow -Wconversion -Wmissing-prototypes -Wimplicit-fallthrough -Wvla -fstack-protector-strong -fdiagnostics-color=always -DIN
CLUDE_GENERATED_DECLARATIONS -D_GNU_SOURCE -DNVIM_MSGPACK_HAS_FLOAT32 -DNVIM_UNIBI_HAS_VAR_FROM -DMIN_LOG_LEVEL=3 -I/home/seth/build/neovim/b
uild/config -I/home/seth/build/neovim/src -I/home/seth/build/neovim/.deps/usr/include -I/usr/include -I/home/seth/build/neovim/build/src/nvim
/auto -I/home/seth/build/neovim/build/include
Compiled by seth@hydrogen
Features: +acl +iconv +tui
See ":help feature-compile"
   system vimrc file: "$VIM/sysinit.vim"
  fall-back for $VIM: "/home/seth/.local/share/nvim"
Run :checkhealth for more info

:messages
"main.go" [New] 10L, 135C written

It would be nice if any errors that were attributed to nonexistent line numbers were associated with the last line of the program.

Makes sense.
Except for < 1, which should be attributed to line 1 then probably.

The error produced, go: syntax error: unexpected EOF, expecting } (E)

What's the raw output here?  Does it have a line number after all?

Does this help?
11:49:30 2046 [V      ] [2.2:1:1] Starting async job: go test -c -o /dev/null.
11:49:30 2046 [D      ] [2.2:1:1] cwd: /home/seth/dev/go/src/github.com/sbromberger/vt1.
11:49:30 2046 [D      ] [-.-:1:1] automake: started jobs: [2].
11:49:30 2046 [D +0.07] [2.2:1:1] output on stderr: ['# github.com/sbromberger/vt1', '../../src/github.com/sbromberger/vt1/main.go:13:1: synt
ax error: unexpected EOF, expecting }', ''].
11:49:30 2046 [D      ] [2.2:1:1] exit: go: 2.
11:49:30 2046 [D      ] [2.2:1:1] Processing 2 lines of output.
11:49:30 2046 [D      ] [-.-:1:1] Using setting cwd='%:h' from 'maker'.
11:49:30 2046 [D      ] [-.-:1:1] Using setting cwd='%:h' from 'maker'.
11:49:30 2046 [D      ] [-.-:1:1] Using setting postprocess=function('neomake#postprocess#compress_whitespace') from 'maker'.
11:49:30 2046 [D      ] [2.2:1:1] Modified list entry 1 (postprocess): {'changed': {'text': [' syntax error: unexpected EOF, expecting }', 's
yntax error: unexpected EOF, expecting }']}}.
11:49:30 2046 [D      ] WARN: seen entries with bufnr different from jobinfo.bufnr (1): {'2': 1}, current bufnr: 1.
11:49:30 2046 [D      ] [2.-:1:1] Adding 1 list entries.
11:49:30 2046 [D      ] [2.-:1:1] Reusing location list for entries.
11:49:30 2046 [D      ] list: call: set: [0, [], 'r', {'context': {'neomake': {'make_info': 'make_id=2'}}, 'items': [{'lnum': 13, 'bufnr': 2,
 'col': 1, 'valid': 1, 'vcol': 0, 'nr': -1, 'module': '', 'type': 'E', 'pattern': '', 'text': 'syntax error: unexpected EOF, expecting }'}],
'title': 'Neomake[auto]: main.go (go..., golangci_lint?)'}].
11:49:30 2046 [D      ] list: got qfid (action=set): 2.

Yes, thanks.

11:49:30 2046 [D +0.07] [2.2:1:1] output on stderr: ['# github.com/sbromberger/vt1', '../../src/github.com/sbromberger/vt1/main.go:13:1: syntax error: unexpected EOF, expecting }', ''].


conda package manager

users need to manage packages, and we need to verify that the packages that we require are installed.
this is the next instance of https://github.com/twosigma/beaker-notebook/issues/1350
depends on https://github.com/twosigma/beaker-notebook/issues/4395
questions: are dependencies in the language manager (and hence per notebook)?  we use the conda version specs right?  how do multiple notebooks with different specs run at the same time?  how does this work with lab where there is a readonly virtualenv with many packages, can additions to that be stored locally?

the abstraction provided by conda is the virtualenv. our MVP should just expose this.
create/name/delete/addlib/removelib, with autocomplete.  this would be a java wrapper around the std conda API (https://github.com/conda/conda-api), exposing it to a dialog on the front end.

are dependencies in the language manager (and hence per notebook)?

When you say "dependencies" do you just mean conda dependencies, or does Beaker define this word too?

how do multiple notebooks with different specs run at the same time?

You need separate processes for each notebook, I'm not sure what the Beaker noun here is but it jupyter it's "kernel".  Once the conda env is created, running the python binary inside it will automatically have the right PYTHONPATH and stuff.

Sidenote, I think managing the lifetimes of conda envs is probably an interesting/hard problem here, especially in Lab.  You don't want to have conda compute the dependency resolution during startup each time, it can be quite slow.  So you want to cache envs.  But, they should expire at some point, probably, and you need a mechanism for users to update them.

the abstraction provided by conda is the virtualenv. our MVP should just expose this.

When you say "virtualenv", do you mean virtualenv the product or the abstract concept which conda mimics?  Conda envs are similar to virtualenv the product environments, but not exactly the same.

std conda API (https://github.com/conda/conda-api)

Protip, conda_api is kind of weak from an API standpoint, it's a relatively thin and incomplete wrapper around shelling out to the conda binary and parsing the results.  You would probably do better to interact with the conda binary yourself, as conda_api doesn't add much and might introduce its own idiosyncracies.

this would be a java wrapper around the std conda

I think this approach makes sense, for python/conda at least.  Expose a list of envs, with the ability to show the packages and versions in one, update all packages in one, add/remove packages from one, delete an env, and create new env.

One feature request I'd add, and this shouldn't be in the first milestone, is "clone an existing env with additional packages added".  I have seen conda sort of corrupt envs by adding new packages after creation, I prefer to treat my envs as immutable and I think encouraging that behavior will be helpful.  One issue with that is that in order to do it, you need to know more than conda stores about the env (it only knows the currently installed packages): you need the user's original creation intent for an env, if you want to clone it and add some stuff.  That is, if I create an env with "conda create -n myenv python numpy pandas" and then realize I forgot sklearn, I'd rather run "conda create -n myenv2 python numpy pandas scikit-learn" than "conda install -n myenv scikit-learn".


Type definition for the React style property

Is it possible to add a flowtype support for the style property? For example, if I type 'padding-left': '10px' it would warn me that I should use paddingLeft: '10px'.
I found this 3rd party definition: https://github.com/Polyconseil/flow-inlinestyle/blob/master/index.js, but it is not perfect and it would be great to have an official support for this.

there is also https://github.com/frenic/csstype


Version number

Hi Guys, Thanks a lot for this grat plug-in.
About v2.4.3, even after a plug-in upgrade (2.4.1 to .3 or .2 to .3), the version number is stuck to 2.4.2.
This lead sthe update manager to prompt for a plug-in update.

Merci !

Already in the new commit list :'( Sorry !


Filename not visible with option styleButtonRemoveItemPosition: 'right' for local files

Describe the bug
When using the option styleButtonRemoveItemPosition: 'right' as described in the documentation, the labels for file name and size are not visible anymore on local files. The labels seem to be pushed to the right outside the filepond--item
I fixed it temporarily like this
.filepond--file-info {
  margin-right: 2em;
  transform: translate3d(0px, 0px, 0px) !important;
}

To Reproduce
https://codesandbox.io/s/vanilla-filepond-forked-jhx52?file=/src/index.js
Try to uncomment the styles in index.html there
Expected behavior
File labels should be visible and displayed inside filepond--item div
Information about your project:

Your browser and the version: latest Chrome, Firefox, Safari
Your operating system: MacOS 10.14
FilePond version: latest 4.23.1

Context
It is also mentioned here, although the issue is already closed: https://github.com/pqina/filepond/issues/366#issuecomment-708156604

Should be fixed in https://github.com/pqina/filepond/releases/tag/4.25.2
Please close when confirmed.

Yep. It's fixed now. Thanks.


Example code in the literate programming section

The shown CWEB is not the one developed by Knuth, so it would be weird to show an example of it. The CWEB by Knuth looks unorganised, however, and is therefore not really fit for demonstration purposes. This means that another example needs to be found. It is removed for now, so that #14  can be merged.
Old code:

After `WEB', another language called `CWEB' was developed to introduce the same
literate programming style to the C programming language.
An illustrative `CWEB' code snippet can be found in \cref{code:cweb}.

\begin{figure}[htb]
  \begin{minipage}{.4\textwidth}
    \begin{verbatim}
@* Maths Functions
We define a  function \$exp(x, y)\$
to return \$x\$ to the power \$y\$.
@c
long exp(x, y)
{ long z;
  @<Check \$y\$ is positive@>
  ...
  return z;
}
@ @<Check \$y\$ is positive@>=
  if( y < 0 )
    error ...
    \end{verbatim}
    \subcaption{Specification}
  \end{minipage}
  \hspace{1.5cm}
  \begin{minipage}{.4\textwidth}
    \begin{tabbing}
      12. \= \textbf{Maths Functions.} We define a function $exp(x, y)$ \\
          \> to return $x$ to the power $y$. \\
          \> \textbf{long} $exp(x, y)$ \\
          \> \{ \hspace{40pt} \= \textbf{long} $z;$ \\
          \>                  \> $\langle$Check $y$ is positive \textbf{13}$\rangle$ \\
          \>                  \> ... \\
          \>                  \> \textbf{return} $z$; \\
          \> \} \\
      13. \> $\langle$Check $y$ is positive \textbf{13}$\rangle\equiv$ \\
          \> \textbf{if} $( y < 0 )$ \\
          \> \hspace{10pt} $error...$ \\
      \textbf{This code is used in section 12.}
    \end{tabbing}
    \subcaption{Formatted output}
  \end{minipage}
  \caption{`CWEB' code snippet outlining a literate program}
  \label{code:cweb}
\end{figure}

Closing, as we decided not to show any example code.


fstat call fails for non-altered streams

Using PHP-VCR and FPDI together in a test, when an FPDI tries to load a PDF file it crashes. The reason for this is it tries to get the size of an PDF file in one of the methods, but the file stream is wrapped in the PHP-VCR StreamProcessor which always returns false when asked for stream_stat even if the stream is not actually modified by any transformers. I prepared a PR to fix this issue: https://github.com/php-vcr/php-vcr/pull/321

Not to self: a PR has been made: https://github.com/php-vcr/php-vcr/pull/321

Is fixed in 1.5


always pass const DispatchArgs & to dispatchCall-related functions

Motivation

Consistency and epsilon performance improvements (we were copying DispatchArgs on every. single. dispatchCall.).
We already did something like this in #3640, but it didn't stick.  Let's try a little harder this time, with deleting the copy constructor for DispatchArgs.
Test plan

See included automated tests.

We have a policy of testing changes to Sorbet against Stripe's codebase before
merging them. I've kicked off a test run for the current PR.  When the build
finishes, I'll share with you whether or how it failed. Thanks!
Stripe employees can see the build results here:
→ https://go/builds/bui_LQbZ84r8FkAa3i
→ https://go/builds/bui_LQbZDNFX6SpoPj


Add support for local repositories

Related to #30
Add support for local directories in the UI and backend.

UI Changes:

Add a file input field for selecting local directories in src/templates/components/github_form.jinja.
Add checkboxes to select between "local directory" and "remote repo" options in src/templates/components/github_form.jinja.

Backend Changes:

Update ingest function in src/gitingest/ingest.py to handle local directories by checking if the input is a directory path.
Modify parse_query function in src/gitingest/parse_query.py to parse local directory paths and set appropriate query parameters.
Ensure the scan_directory function in src/gitingest/ingest_from_query.py includes safety limits such as MAX_DIRECTORY_DEPTH, MAX_FILES, and MAX_TOTAL_SIZE_BYTES to prevent excessive processing.

Documentation:

Update README.md to include instructions for using local directories with the CLI and Python package.

User Feedback:

Provide feedback to the user if the directory exceeds the safety limits in src/templates/components/result.jinja.

Very interesting work, thank you!
I'll take some time to look more into it, but I'm willing to merge this feature at some point.
One thing I can tell you already:
Since I host gitingest.com straight from this repo we need at least an option to disable this feature from and env_var and maybe make it enabled by default
But if you don't know/don't have time to add this just tell me I can do it myself

@cyclotruc Absolutely please task from here.

@cyclotruc Absolutely please take the task from here.

To be honest I think I'm going to close this branch from now
There's been many refactors in the code and I since I have to finish this it's probably going to be easier to start from scratch
If you want you can have another try at it on the updated codebase


According to the latest code provided, retrain, can't fully reproduce the metrics in the paper, please tell me the results in the paper, are there any settings that need extra attention?

DrealSR in the paper:
27.9
0.7628
0.3299
0.2363
151.88
6.4893
0.6005
65.11
0.6708
reproduce:
28.15
0.7832
0.3175
0.2332
164.45
6.867
0.4413
61.38
0.6167
It seems that the reproduced model has higher fidelity and much less generative power.


Strange output

Hello,
Today I tried to update go version. And I see a lot empty lines.
❯ gobrew use latest
[Info]: Fetching remote versions

	[Info] Downloading version: 1.19.4
[Info] Downloading from: https://go.dev/dl/go1.19.4.darwin-arm64.tar.gz
[Info] Downloading to: /Users/deevsyukov/.gobrew/downloads
Downloading 100% |████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████| (140/140 MB, 27.660 MB/s)
[Info] Extracting from: /Users/deevsyukov/.gobrew/downloads/go1.19.4.darwin-arm64.tar.gz
[Info] Extracting to: /Users/deevsyukov/.gobrew/versions/1.19.4
[Success] Untar to /Users/deevsyukov/.gobrew/versions/1.19.4
[Success] Downloaded version: 1.19.4
[Info]: Fetching remote versions

	[Info] Changing go version to: 1.19.4
[Success] Changed go version to: 1.19.4

We need to investigate it. It not first time.

Yea.


Verify all the packages are restored before running 'nuget.exe update'

🐛 Bug Report
NuKeeper throws an exception when it tries to update packages. Before this, there are several mentions of "Cannot determine the packages folder to restore NuGet packages. Please specify either -PackagesDirectory or -SolutionDirectory.". See https://github.com/NuKeeperDotNet/NuKeeper/issues/72 ?
Here are the logs when run using v -D
nukeeper-1.log
The repository uses the new PackagesReference format, but does not have the new SKD-style csproj format. Maybe that has something to do with it. I will try to confirm this when I have some time.
It also has the following nuget.config:
<?xml version="1.0" encoding="utf-8"?>
<configuration>
  <config>
    <add key="DependencyVersion" value="HighestMinor" />
  </config>
  <packageSources>
    <add key="nuget.org" value="https://api.nuget.org/v3/index.json" />
    <add key="particular myget.org" value="https://www.myget.org/F/particular/" />
  </packageSources>
</configuration>

Expected behavior
Packages are restored, and updated.
Reproduction steps

have a non-SDK style repo with PackageReferences (?)
run the nukeeper repo <url> <pat> -v D command
Nuget restore and subsequently nuget update fails

Configuration
Version: 0.33.0
Platform if applicable:

[X] :hammer_and_wrench: NuKeeper CLI
[ ] :sparkles: GitHub
[ ] :robot: AzureDevops
[ ] :checkered_flag: Bitbucket
[ ] :earth_americas: Gitlab
[ ] :tv: Gitea
[ ] :whale: Docker

I'm experiencing this problem for another repository. This second repository has a custom nuget.config file that changes the location of the packages folder using the repositoryPath configuration key.
However, I tried to set the variable NUGET_PACKAGES  (see https://docs.microsoft.com/en-us/nuget/reference/nuget-config-file) to a shared location, but still receive the same exception.

Also, I can confirm that issue I described in the original post is related to nuget update not yet working for projects using PackageReferences. See https://github.com/NuGet/Home/issues/4103

Bump.

Same here. I am not cleaning sources at all. Also I can run the DevOps pipeline a second time. But then the restore worked and I stuck with:
Git checkout new branch 'nukeeper-update-3-packages-C9E5BCFE867B87305EED1B0A7B080850'
Updates failed NameConflictException : failed to write reference 'refs/heads/nukeeper-update-3-packages-C9E5BCFE867B87305EED1B0A7B080850': a reference with that name already exists.
So on the first run, it was not waiting for the restore and on the following run it blames the repository already exist. But it is not there.


Playgrounds

Github Issue or Trello Card
This PR addresses this issue:
What type of change is this?
Please delete options that are not relevant.

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[X] Updating documentation.
[ ] Updating deployment/build pipeline.

Checklist:

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

UI Checklist

[X] I have conducted visual UAT on my changes/features.
[X] My solution works well on desktop, tablet, and mobile browsers.

@juanfabrega here's how to disable controls for specific properties
https://storybook.js.org/docs/react/essentials/controls#disable-controls-for-specific-properties


BUG: Extend scipy.stats.arcsine.pdf to endpoints 0 and 1 #7427

Do not set the arcsine._support_mask to _open_support_mask.
The arcsine distribution is theoretically the same as the
beta(0.5, 0.5) distribution,  but the arcsine code was restricting
itself to the interior points of the interval [0, 1] and the _pdf()
method was returning values different from
scipy.stats.beta(0.5, 0.5)._pdf().
By not setting arcsine._support_mask, the endpoints are no-longer
special-cased and the _pdf()s for the two distributions agree at the
endpoints.
Updated the testing so that arcsine is no longer tested in test_support().
See #7421.

Test error is unrelated. The change itseld seems OK, merging. Thank you @pvanmulbregt , kep them coming!


Fetch session info from specific user

I have another enanchment feature to request. I have my own Plex server where i invited many friends. If any of them is watching something on my server the script detects the watching status as if I was watching it.
It would be cool if we could choose for which user fetching session to.

This is a limitation that I'm aware of, but I have yet to find a comfortable solution for it. The Movie and Episode classes in pkkid/python-plexapi do not appear to have any ability to discern which user (name or id) is responsible for the session.
For anybody looking to contribute this Issue, there are two theoretical solutions - the first of which would be preferred:

Add an array to config.json, similar to the servers array, and prioritize sessions from users in this list.
First, prompt the user to choose one of the active sessions if there are multiple. Then save a unique identifier of the user responsible for the session and use it for subsequent updates.

I don't know if this can help but there's an old script like this which had an option called 'listen for user' https://github.com/Phineas05/discord-rich-presence-plex

Nice! It seems that pkkid/python-plexapi doesn't explicitly expose the usernames array anywhere, but it does exist.
I've added a users field to the config, just enter Plex usernames in there and Perplex use the first one that it finds active.


added capability to add interfaces to running containers

This is a draft implementation for #252.
Issue the following to add a veth pair between wan1 and wan2.
./containerlab tools connect containers  -t lab-examples/srl03/srl03.yml -a wan1:intf1 -b wan2:intf2 -m 1500
Please have a look and provide feedback.
Cheers
close #252

Hey @steiler, thanks for looking into that one

I had the following command name in my mind: containerlab tools veth create -a $cont-name:$if-name -b ... -m
do not depend on topo file to be available, make use of the functions to find the container by its name

Alright, removed the references to the topology file.
And also changed the command to the following:
containerlab tools veth create -a clab-srl03-wan1:if1 -b clab-srl03-wan2:if2 -m 5000

added support for host attachment as well
I will now add the docs and will merge this
Thank you for kickstarting and prototyping!

Hello, I'm trying to create a new connection between 2 containers (2 Arista ceos 4.29.0) running in the same host but I doesn't work as expected. I don't get errors but after create the connetion, I access to devices and I new interfaces doesn't exist:
root@containerlab:/etc/containerlab/lab-examples/multi_fabric# containerlab tools veth create -a clab-backbone-backbone_11:Eth4 -b clab-fabric_pro_1-border_leaf_f1_11:Eth4 -m 9000
INFO[0000] Creating virtual wire: clab-backbone-backbone_11:Eth4 <--> clab-fabric_pro_1-border_leaf_f1_11:Eth4
INFO[0000] veth interface successfully created!
root@containerlab:/etc/containerlab/lab-examples/multi_fabric# docker exec -it clab-backbone-backbone_11 Cli
backbone_11>ena
backbone_11#show interfaces status
Port       Name                           Status       Vlan     Duplex Speed  Type            Flags Encapsulation
Et1        P2P_LINK_TO_SPINE1_Ethernet1/1 connected    routed   full   1G     EbraTestPhyPort
Et2        P2P_LINK_TO_SPINE2_Ethernet1/1 connected    routed   full   1G     EbraTestPhyPort
Et3                                       connected    routed   full   1G     EbraTestPhyPort
Ma0                                       connected    routed   a-full a-1G   10/100/1000
Can anybody help me. I may be doing something wrong.....
Thanks!

@alfonsomoneo It looks like arista might not support live interface attachments. probably you want to ask arista people if it is supported

Thanks Helt, I will do it!


[Dajngo 4.0 Warning] RemovedInDjango40Warning: django.utils.translati…

…on.ugettext_lazy() is deprecated in favor of django.utils.translation.gettext_lazy()
What changes were proposed in this pull request?

ugettext_lazy -> gettext_lazy and ugettext -> gettext

How was this patch tested?

https://docs.djangoproject.com/en/1.11/_modules/django/utils/translation/
https://docs.djangoproject.com/en/3.1/internals/deprecation/
https://docs.djangoproject.com/en/3.0/_modules/django/utils/translation/

If with Django 1 it is not equal, we need to keep it the same, and import the new function only with Python 3 so

I have added the flag
if sys.version_info[0] > 2:
  from django.utils.translation import gettext as _
else:
  from django.utils.translation import ugettext as _

and added import sys in approximately all the modified files.
Please go through the changes once if possible.


Update various mentions of SELECT FOR UPDATE

Fixes #6779.
Summary of changes:

Update 'Migrate from Oracle' to note that we also support SELECT FOR
UPDATE (SFU) in v20.1+

Update 'Postgres Compatibility' to note that we support SFU

Update the locking clause description of the 'Selection Queries' page
to note our SFU support

Update 'SQL Feature Support' to add a "new in v20.1" tag

Update 'SQL Statements' to add a "new in v20.1" tag

Update various uses of SELECT ... FOR UPDATE to say SELECT FOR UPDATE instead, to aid the site search engine

This change is

@lnhsingh going straight to you for docs review since there is nothing in here that needs eng review.
Mostly a "cleanup" PR that brings every mention of that statement up to date and uses a consistent name for it everywhere as well.

Online preview: http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/b7700e02abf795e157078621c0fdc2c37fb645a6/
Edited pages:

v20.1/migrate-from-oracle.md
v20.1/postgresql-compatibility.md
v20.1/select-for-update.md
v20.1/selection-queries.md
v20.1/sql-feature-support.md
v20.1/sql-statements.md

Thanks Lauren!!!!


toOpenarray missing

It is rather simple:
template toOa(s: String; first, last: int): untyped =
  toOpenArray(toCStr(addr s), first, last)

template toOa(s: String): untyped =
  toOpenArray(toCStr(addr s), 0, len(s)-1)

Done.


Strip quotation marks from query pasted from a package.json file.

Thanks for building this site!
This partially fixes #55 to allow people to paste their browserslist config directly from a package.json file.
For instance, my package.json looks like this:
"browserslist": [
    ">0.19%",
    "not dead",
    "not ie <= 10",
    "not op_mini all",
    "ios_saf >= 9.0"
],

This PR allows me to copy/paste the following directly into the form and get the desired results:
     ">0.19%",     "not dead",     "not ie <= 10",     "not op_mini all",     "ios_saf >= 9.0"

Thanks for considering this PR, and please let me know if any questions/comments/suggestions!

Can you rebase this @YPCrumble ?

Looks like it's working 🚀


fix: Bazel version checks for dev versions

@sgammon One more missing piece, without this change Bazel's bootstrap test (and really any dev version) fails when it uses the ruleset.


Autoware source build error with nlopt

Hello everyone,
I am trying to install autoware from source build, But unfortunaltely i am getting following error .
CMakeFiles/calibrationtoolkit.dir/CalibrationToolkit/calibrationtoolkit.cpp.o: In function nlopt::opt::get_errmsg() const': /usr/local/include/nlopt.hpp:500: undefined reference to nlopt_get_errmsg'
collect2: error: ld returned 1 exit status
sensing/fusion/packages/calibration_camera_lidar/CMakeFiles/calibrationtoolkit.dir/build.make:424: recipe for target '/home/ajay/Autoware/ros/devel/lib/libcalibrationtoolkit.so' failed
make[2]: *** [/home/ajay/Autoware/ros/devel/lib/libcalibrationtoolkit.so] Error 1
CMakeFiles/Makefile2:26968: recipe for target 'sensing/fusion/packages/calibration_camera_lidar/CMakeFiles/calibrationtoolkit.dir/all' failed
make[1]: *** [sensing/fusion/packages/calibration_camera_lidar/CMakeFiles/calibrationtoolkit.dir/all] Error 2
make[1]: *** Waiting for unfinished jobs....
Makefile:138: recipe for target 'all' failed
make: *** [all] Error 2
Invoking "make -j12 -l12" failed
Steps to Reproduce the Problem
$ cd ~/Autoware/ros/src
$ catkin_init_workspace
$ cd ../
$ rosdep install -y --from-paths src --ignore-src --rosdistro $ROS_DISTRO
$ ./catkin_make_release
Specifications

Ubuntu version: 16.04
ROS version: Kinetic
Autoware branch: git clone https://github.com/CPFL/Autoware.git --recurse-submodules

Could anyone please help me to fix this issue?
Kind regards,
Ajay

@ajay1606 have you checked if nlopt is installed in your system?

Hello sgermanserrano,
Thank for your response.
yes i have installed nlopt already, i can find the files in :
/usr/local/include/nlopt.hpp
/usr/local/include/nlopt.h
/usr/local/include/nlopt.f
But still am getting error.

@ajay1606 can you change the installation directory of nlopt to /usr/include as per https://nlopt.readthedocs.io/en/latest/NLopt_Installation/#changing-the-installation-directory and try installing Autoware again?

@sgermanserrano The error i am getting is I"n function nlopt::opt::get_errmsg() const': /usr/local/include/nlopt.hpp:500: undefined reference to nlopt_get_errmsg'"  look like compiler error rather than linker error.
Do you think issues related to the installation of a library?

@ajay1606 I'm not sure if it is due to the installation of the library. Have you installed it manually from source? When I have compiled Autoware from source it gets installed automatically in /usr/include that's why I suggested changing its installation path

@sgermanserrano Yes i did  installed nlopt manually (https://nlopt.readthedocs.io/en/latest/NLopt_Installation/)
I tried installing the library in the path /usr/include and i am getting the same error in that case also.
But I didn't understand the reason.

It's probably due to linking error.
Do you have nlopt installed from repository? (i.e. sudo apt install libnlopt-dev)
The linker could be linking different .so file if that's the case.
Try sudo apt remove libnlopt-dev and compile again.
You might need to sudo ldconfig after apt remove command.
Do you have any reason to use manually installed nlopt?
Otherwise, I suggest using nlopt installed from apt command.
In that case, you would have to uninstall nlopt that is installed from source.

@mitsudome-r Than you so much.. Yes it is problem with wrong .so file, Its now working after reinstalling nlopt library by removing the old one.


[Package Issue]: Facebook.Messenger has incorrect hash

Please confirm these before moving forward

[X] I have searched for my issue and not found a work-in-progress/duplicate/resolved issue.
[X] I have not been informed if the issue is resolved in a preview version of the winget client.

Category of the issue
Installer hash mismatch.
Brief description of your issue
winget install facebook.messenger gives hash mismatch
Steps to reproduce
winget install facebook.messenger
Actual behavior
Gives hash mismatch and does not install the package
Expected behavior
Facebook.messenger should be installed
Environment
Windows Package Manager: 1.5.2201
Windows: Windows.Desktop v10.0.22621.2283
Architettura di sistema: X64
Pacchetto: Microsoft.DesktopAppInstaller v1.20.2201.0

Screenshots and Logs

Hi @numerunix
This software has published new version. I've opened Pull Request to resolve it.
Have a good day!


Little HotFix

Removed DB_URL from nodemon.json as conflicting with .env (DB_URL)
The app was loading "DB_URL" environment variable from "nodemon.json".

@charlie1404 Thank you for PR. One of the other moderators will get back to you since @danivijay  and @skyshader are handling backend.

@charlie1404 let's remove nodemon.json because it's not needed. We will stick with .env from now on.


[Service Fabric] fix 2020-01 preview version add AddOnFeature, remove clusterUpgradeMode and description

Fixing version 2020-01-01-preview.
Add: ManagedCluster AddOnFeature (missing property)
Remove: clusterUpgradeMode and clusterUpgradeDescription as they are not available yet
And fix examples
MSFT employees can try out our new experience at OpenAPI Hub  - one location for using our validation tools and finding your workflow.

Changelog
Please ensure to add changelog with this PR by answering the following questions.

What's the purpose of the update?

[ ] new service onboarding
[ ] new API version
[ ] update existing version for new feature
[ ] update existing version to fix swagger quality issue in s360
[x] Other, please clarify

When you are targeting to deploy new service/feature to public regions? Please provide date, or month to public if date is not available yet.
When you expect to publish swagger? Please provide date, or month to public if date is not available yet.
If it's an update to existing version,  please select SDKs of specific language and CLIs that require refresh after swagger is published.

[ ] SDK of .NET (need service team to ensure code readiness)
[ ] SDK of Python
[ ] SDK of Java
[ ] SDK of Js
[ ] SDK of Go
[ ] PowerShell
[ ] CLI
[ ] Terraform
[ ] No, no need to refresh for updates in this PR

Contribution checklist:

[ ] I commit to follow the Breaking Change Policy of “no breaking changes
[ ] I have reviewed the documentation for the workflow.
[ ] Validation tools were run on swagger spec(s) and errors have all been fixed in this PR. How to fix?

If any further question about AME onboarding or validation tools, please view the FAQ.
ARM API Review Checklist

[ ] Ensure to check this box if one of the following scenarios meet updates in the PR, so that label “WaitForARMFeedback” will be added automatically to involve ARM API Review. Failure to comply may result in delays for manifest application. Note this does not apply to data plane APIs, all “removals” and “adding a new property” no more require ARM API review.

Adding new API(s)
Adding a new API version
Adding a new service

[ ] Please ensure you've reviewed following guidelines including ARM resource provider contract and REST guidelines. Estimated time (4 hours). This is required before you can request review from ARM API Review board.

[ ] If you are blocked on ARM review and want to get the PR merged with urgency, please get the ARM oncall for reviews (RP Manifest Approvers team under Azure Resource Manager service) from IcM and reach out to them.

Breaking Change Review Checklist
If there are following updates in the PR, ensure to request an approval from API Review Board as defined in the Breaking Change Policy.

[ ] Removing API(s) in stable version
[ ] Removing properties in stable version
[ ] Removing API version(s) in stable version
[ ] Updating API in stable version with Breaking Change Validation errors
[ ] Updating API(s) in preview over 1 year

Action: to initiate an evaluation of the breaking change, create a new intake using the template for breaking changes. Addition details on the process and office hours are on the Breaking change Wiki.
Please follow the link to find more details on PR review process.

Breaking changes:
Removed clusterUpgradeDescription and clusterUpgradeMode from managed cluster definition. These operations are not available, They were added by mistake in this preview version

Hi @a-santamaria , would you follow the breaking change process?

@leni-msft do we need to do this? this is a preview version and those parameters were never there (they were added to the swagger by mistake) we want to remove them to have the correct documentation

hi @a-santamaria , per current rule, yes. Please provide business justification in the intake, I think they will approve it soon.

created the intake to get approved: https://msazure.visualstudio.com/One/_workitems/edit/9193186

created the intake to get approved: https://msazure.visualstudio.com/One/_workitems/edit/9193186

@leni-msft thanks, I followed the breaking change process and it has been approved. Can you merge the pr, please?

@leni-msft thanks, I followed the breaking change process and it has been approved. Can you merge the pr, please?


Actually use hy-blue in links.

@andyngo Can you complete this PR ? In particular take care of the "correct" way to have the "Tachyons" link blue ?

This looks about right. Thanks for adding in .hy-hover-blue. Also, if you don't want the subtle transition when you hover on .link, we can remove that. It seems that .link only adds a transition effect with nothing else on the side.

Also, note to myself:

Add more border-colors like so .hy-b--${color}
Add more hover-colors .hy-hover-${color}

I'll create a PR for the above.

Hi Andy,
Yes, you can remove that transition. And make the links always blue in the
paragraph (i.e. not just on hover).
Thanks
Le lun. 30 déc. 2019 à 05:37, Andy Ngo<EMAIL_ADDRESS>a écrit :

Also, note to myself:

Add more border-colors like so .hy-b--${color}
Add more hover-colors .hy-hover-${color}

I'll create a PR for the above.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/hypered/design-system/pull/44?email_source=notifications&email_token=AAAMILHJW4COSPWD6DXYSM3Q3F3H7A5CNFSM4KAL3UW2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEHZRZ5Y#issuecomment-569580791,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAAMILCML4DHEVEOKD5Z353Q3F3H7ANCNFSM4KAL3UWQ
.


CSS fixes

-Change "border-collapse: always" to "border-collapse: collapse"
-Remove all "overflow: stretch" (Doesn't exist. "Stretching" is the default behavior of divs.)
-Replace instances of "line-spacing" with "line-height"
-Remove "7: 1.5" property from #wait (Seems like a typo, but not sure what it was meant to be originally.)

Strange. Can't get it to break on any of my devices, but I'll take a deeper look myself  later.


SOLR-16852: Let backups have custom key/values

Currently backups have some properties attached such as configName, collectionAlias, indexVersion, etc. It would be useful to allow custom properties/annotations to add some information coming from the process that triggers the backup. For example, say you have a process that consumes from a queue, adds documents to Solr and then triggers a backup command, the backup could include the queue offset at the time of triggering the backup so that the application could potentially continue processing from such offset upon restore.

I'm working on the docs for this change. Will update this PR.

@dsmiley, Any concerns with this approach, given the discussion in the Jira issue?


chore(actions) avoid code duplication for report's PipelineURL generation

Follow up of #1737 (see https://github.com/updatecli/updatecli/pull/1737#discussion_r1382388606)
This PR only refactor the code to avoid duplication, by introducing a new ci package.
Allows extensibility with custom  behaviors in the future.
Please note the validation of each CI has been decreased: we can add more validation but it looks like a lot of code for something not really probable.
Test
To test this pull request, you can run the following commands:
cd pkg/plugins/utils/ci
go test

Additional Information
Tradeoff

Potential improvement

Thanks @dduportal for the improvement, I just added an interface comment

@dduportal Just in case you missed it, test are not passing :)

@dduportal Just in case you missed it, test are not passing :)

Weird, this test passes locally. I wonder if it could be related to the test SetEnv variable not having the same behavio. I'm putting the PR as draft and I'll send a debug commit.
If I don't get any result, I'll remove the test itself.

Fixed: checks are passing except the updatecli one which fails for the same reason as the main branch check @olblak


Neo-cli=>transfer

/// <summary>
        /// Process "transfer" command
        /// </summary>
        /// <param name="tokenHash">Script hash</param>
        /// <param name="to">To</param>
        /// <param name="amount">Ammount</param>
        /// <param name="from">From</param>
        /// <param name="signersAccounts">Signer's accounts</param>
        [ConsoleCommand("transfer", Category = "NEP17 Commands")]
        private void OnTransferCommand(UInt160 tokenHash, UInt160 to, decimal amount, string data = null, UInt160 from = null, UInt160[] signersAccounts = null)
        {
               ...
        }

It looks like the 4th argument is data.


Update image building system to blue build

Moving to blue build provides better control over system files, flatpaks and installed packages. The docker  package repository and engine have been added as part of the update but not x11. Will install x11 again later down the line for hardware that doesn't function under wayland but this will be done after testing of the ISO. Before merge the ISO will be tested on a laptop to ensure everything works from install to daily usage.

Removed another reference to secureblue and added more important information as a replacement.


⚠ TorrentLeech has degraded performance

In 5319011, TorrentLeech ($TL) experienced degraded performance:

HTTP code: 200
Response time: 3990 ms

Resolved: TorrentLeech performance has improved in cf1b045 after 7 minutes.


Create Software for CODEC Parametrization via I2C

The team should make a software tool for configuring WM8731 codec to stream audio samples according to defined set of parameters. As I2C bus is used for sending configuration to the codec, it is allowed to use an embedded platform (i.e., Raspberry Pi) which contains an I2C controller and low-level software support for this interface.
Acceptance criteria (new)

Able to demonstrate that clock generation circuit in the design operate correctly by showing the generated clock on a GPIO pin on the development board using oscilloscope.
Able to demonstrate that the clock is correctly propagated to CODEC. Use oscilloscope to confirm this.
Able to demonstrate that I2C SCK signal is correctly propagated to corresponding CODEC pin from RPi (via a GPIO pin on the development board). Use oscilloscope to confirm this.

Acceptance criteria (old)

Able to demonstrate that embedded platform which runs the configuration tool is correctly connected to the codec on the DE1-SoC board.
Able to demonstrate that the configuration tool successfully communicates with the codec and send the defined parameters via I2C without errors
A document which describes configuration tool usage and available options

Closing this issue, but it needs further confirmation for SCL signal propagation to the CODEC on the board.


docs: how to run prover

Current guide: https://hackmd.io/@aztec-network/epoch-proving-integration-guide
Will need to convert to docs page.


fix(versions): update Activiti/example-runtime-bundle versions into master

UpdateBot pushed maven dependency: org.activiti.cloud.rb:activiti-cloud-runtime-bundle-dependencies to: 7.1.198

UpdateBot commands:
updatebot push-version --kind maven org.activiti.cloud.rb:activiti-cloud-runtime-bundle-dependencies 7.1.198


Make tag helper binding more granular

This splits tag helper binding into two independent phases: in scope discovery, and actual re-writing. We also record which tag helpers were used during the re-writing.
This will allow the source generator to know which tag helpers were actually used by the rewriting phase, and only re-write the document if the input tag helpers have actually changed.

@jaredpar Any more feedback here?


drivers: i2c: flexcomm: Add i2c slave support

Add i2c slave support to mcux_flexcomm driver.
Signed-off-by: Oliver Stäbler<EMAIL_ADDRESS>Tested on an lpcxpresso55s28 with the eeprom slave driver.

@ostaebler Thank you for your contribution. How are you testing this feature?

@ostaebler Thank you for your contribution. How are you testing this feature?

@mmahadevan108 I used the eeprom slave driver in Zephyr (drivers/i2c/slave/eeprom_slave.c) on a lpcxpresso55s28. On the master side, I used a Linux system from which I accessed the EEPROM device via the at24 (drivers/misc/eeprom/at24.c) driver to write and read from the EEPROM slave.
This was the easiest test that came to mind. Happy to test it in a different way if requested.


PyO3 Fails to Import DLL in Python Virtual Environment on Windows

Bug Description
When running cargo test within a python virtual environment on Windows, PyO3 fails to import DLLs, resulting in a STATUS_DLL_NOT_FOUND error. This occurs even after attempting to use --no-default-features to exclude potential issues caused by default feature sets.
Additional Attempt to Resolve:
Following the suggestion from the PyO3 FAQ, adding the directory containing the Python DLL (from the existing Python environment) to the PATH environment variable was attempted. However, this leads to importing modules from the existing Python environment instead of the virtual environment.
Steps to Reproduce

Set up a python virtual environment on Windows.

Create a minimal Rust project using PyO3.

Add a simple test case in src/lib.rs, for example:

    #[test]
    fn test() {
        pyo3::prepare_freethreaded_python();
        Python::with_gil(|py| {
            let _numpy = PyModule::import(py, "numpy").unwrap();
        });
    }

Run the project using cargo test.

Observe the error output.

Backtrace
No response
Your operating system and version
Win 10
Your Python version (python --version)
Python 3.11.0
Your Rust version (rustc --version)
rustc 1.71.0-nightly (4a59ba4d5 2023-05-12)
Your PyO3 version
0.19.2
How did you install python? Did you use a virtualenv?
python.org install
python -v venv venv
Additional Info
No response

I found that this issue is similar to #1896. I also found that the above steps work normally on Linux (with Ubuntu 22.1 as the system) when running tests with --no-default-features as a parameter. Could this possibly be an issue unique to Windows systems and Python?

This is almost certainly a duplicate of #1896. It's on my list to solve eventually, I think it'll get easier when we can drop 3.7 support and adopt the new PyConfig APIs. @alex - how is the cryptography situation with 3.7 looking? In pydantic we will be dropping 3.7 in our 2.6 release (so probably by end of year).
@luoshuijs, for now you should be able to workaround this by using PYTHONHOME and PYTHONPATH environment variables.

Quite bad :-( It's still something like 1/4 of all of our downloads. From the metrics, a huge portion of the downloads comes from Amazon Linux 2.

Thank you very much, @davidhewitt, for your response. However, after trying to set the PYTHONHOME and PYTHONPATH environment variables as you suggested, I'm still facing issues. Here's what I did in PowerShell:.
PS K:\PycharmProjects\python_genshin_artifact> venv/Scripts/activate.ps1
(venv) PS K:\PycharmProjects\python_genshin_artifact>
(venv) PS K:\PycharmProjects\python_genshin_artifact> $env:PYTHONHOME="C:\Users\luoshuijs\AppData\Local\Programs\Python\Python311"
(venv) PS K:\PycharmProjects\python_genshin_artifact> $env:PYTHONPATH="C:\Users\luoshuijs\AppData\Local\Programs\Python\Python311"
(venv) PS K:\PycharmProjects\python_genshin_artifact> $env:PYTHONHOME
C:\Users\luoshuijs\AppData\Local\Programs\Python\Python311
(venv) PS K:\PycharmProjects\python_genshin_artifact> $env:PYTHONPATH
C:\Users\luoshuijs\AppData\Local\Programs\Python\Python311
(venv) PS K:\PycharmProjects\python_genshin_artifact> cargo test
    Finished test [unoptimized + debuginfo] target(s) in 0.27s
     Running unittests src\lib.rs (target\debug\deps\_python_genshin_artifact-9ae2f0247525dfea.exe)
error: test failed, to rerun pass `--lib`

Caused by:
  process didn't exit successfully: `K:\PycharmProjects\python_genshin_artifact\target\debug\deps\_python_genshin_artifact-9ae2f0247525dfea.exe` (exit code: 0xc0000135, STATUS_DLL_NOT_FOUND)

@luoshuijs did you sat PATH too? I think you'll still need that also.

@luoshuijs did you sat PATH too? I think you'll still need that also.

Indeed, I forgot to set the Path. After making the modification, it did have an effect. However, the corresponding modules were imported from the existing Python environment (not a virtual environment), but it's not a big issue. I plan to write a script to resolve the above issue. I am very grateful for your response. I hope to completely fix this problem by the end of this year.

Hello,
Any news on this subject?
I found the same problem on Windows with a venv.
I got the problem when I tried to compile Torch code using the Rust binding  (https://github.com/LaurentMazare/tch-rs) .
The compilation is fine, but when I tried to import the pyd file (just a copy of the compiled DLL) directly into Python, I got a DLL loading error.
I used (https://github.com/lucasg/Dependencies) to debug my DLL and discovered that some DLLs were not found (torch_python.dll, torch_cpu.dll, c10.dll) which are in fact the DLLs of the locally installed "torch" Python library. I could find these weirdos files in my venv under the folder ".venv/Lib/site-packages/torch/lib".
A really  ugly solution was to use this line before importing the .pyd file into Python.
import os
os.add_dll_directory(Path(os.getcwd()).joinpath(".venv/Lib/site-packages/torch/lib").resolve())

Now all's well.
It seems that PyO3 doesn't link the right DLL directories on Windows from the package installed in the venv. Maybe it's looking for DLLs globally or something else.
A friend of mine had the same issue with Pybind and C++, so maybe it's not PyO3 related, I really don't know.

https://github.com/PyO3/pyo3/issues/3589#issuecomment-1820456435 still applies. I'd love to solve this soon, but my own priorities for time have been elsewhere.

#3589 (comment) still applies. I'd love to solve this soon, but my own priorities for time have been elsewhere.

Okey thank you I completely understand, the best workaround I found was to use Linux directly and avoid this crappy Windows.


Correcting String.charCodeAt signature

The index is optional, as per the spec, and defaults to 0.

This PR has been proposed and rejected multiple times; see #15018. The PR template specifically asks for an issue in the Backlog milestone; please do not send PRs with no associated issues. Thanks!


GUUI 3334 modify review status to show pending uwg review

Description
An approach for modifying request status when it is in uwg_review step. Please let me know your thoughts or if there are any changes needed.
This is a PR to support the GES DISC workflow, specifically to modify a "Request Status" when workflow is in UWG review step. It should show "Pending UWG Review" instead of Pending Review when workflow is in uwg review step.
The change supports adding step_status_label to the step table and passing it with value to the uwg review step in GES DISC workflow.
Spec
See Ticket: https://bugs.earthdata.nasa.gov/browse/GUUI-3334

Validation

Make sure all merge request checks have passed (CI/CD).
Pull related branches locally.

Change Log
(copy/paste-able change log notes. check the box when the change is also in CHANGELOG.md)

[Add _____](link to commitid)
[Fix _____](link to commitid)

@KrupaRami @DavisGauntUAH Have the requested changes of this PR been resolved? Is this PR ready to be validated for merging?

@KrupaRami @DavisGauntUAH Have the requested changes of this PR been resolved? Is this PR ready to be validated for merging?
I have pushed the requested changes.

Ready for testing once https://github.com/eosdis-nasa/earthdata-pub-dashboard/pull/50 is ready

@KrupaRami @joncarlson Is the step label working for you locally? In validating, I'm still just seeing the default label with these branch changes.

Yes it is working for me locally. step label only appears when step is in uwg review step.


Add CircleCI

See https://circleci.com/gh/codysoyland/pdk for example build with this configuration.

The linter actually passes as configured. I started with what we had enabled on pilosa/pilosa, and disabled the ones that are broken. The logger fix was actually caught by one of them.

oh nice, that's awesome - can you make a ticket listing the disabled ones?


Module not found - react/lib/Object.assign

I'm getting this error (with React 15).
Error: ./config/../~/react-canvas/lib/Image.js
Module not found: Error: Cannot resolve module 'react/lib/Object.assign' in

I resolved this by following https://github.com/react-ga/react-ga/issues/53#issuecomment-207703590

+1


Change DHT resolve Test Vectors to be byte array as input

Change the input to be the actual bytes of the dht resolver
https://github.com/TBD54566975/web5-spec/blob/main/test-vectors/did_dht/resolve.json

@nitro-neal can we close as a dupe of #167 ? please make sure this case is covered in there


Nested inline models

Given the following situation: We have a supplier that has products, and we have translations of the product.
I want to be able to add or edit these all in one form using nested inline models.
Supplier, Product and ProductTranslation all have their own models.
I'm able to create nested inline models using the following (simplified) code:
class ProductView(ModelView):
    form_columns = ['id', 'internal_name']

    inline_models = [
        (ProductTranslation, dict(form_columns=['id', 'lang', 'name', 'description']), ),
    ]

class SupplierView(ModelView):
    inline_models = [
        ProductView(Product, db.session),
    ]

I'm required to pass the id in the form_columns for the ProductView, else I will get an "object has no attribute 'id'" error during save.
But this will also show id as a field in the form. But people should not fill in the id field, it should be automatically generated by the database when saving.
If I use an InlineFormAdmin instead of an ModelView for ProductView I can't use the inline_models property.
If I just disable the id field it will throw required errors.
How can I hide the id field without getting an error during save?
Or is there a better way to have nested inline models?

Kind of fixed it by doing this:
class ProductView(ModelView):
    form_columns = ['id', 'internal_name']
    form_widget_args = {
        'id': {'placeholder': 'new id will be automatically created', 'readonly': True},
    }

    inline_models = [
        (ProductTranslation, dict(form_columns=['id', 'lang', 'name', 'description']), ),
    ]

class SupplierView(ModelView):
    inline_models = [
        ProductView(Product, db.session),
    ]

And I set the id property in the Product model to nullable=True, so that I won't get a required error.
Not really happy that I had to set the primary key to nullable=True.
class Product(db.Model):
    id = db.Column(db.Integer, primary_key=True, nullable=True)
    internal_name = db.Column(db.String(200))

Hi @ikkev , I tried you solution here, it works well until I edit the one Supplier, after I click the save button and check it again, I found the Products turns double, say the Supplier had 2 Products before I edit it, after I save it without change any thing, it duplicated 2 more Products for that Supplier. Do you have any solution for this issue? Thanks very much.

@KyleXie
I'm not getting that situation. So I can't reproduce it.
When you check the 2 products, do they show a value in the readonly id fields?
Are you sure there is no other code duplicating these products?

Hi @ikkev,
Thanks very much for you reply. It turns out to be an issue of flask-admin. You can see it from here: https://github.com/flask-admin/flask-admin/issues/1536. I also made a simple fix for this issue: https://github.com/flask-admin/flask-admin/pull/1535.

7 years later for anyone interested in this:
The way to prevent the nullable pk is to override the view's scaffold_form() function and add hidden_pk=True to form.get_form() call. here's the full code that needs to be added to the nesting view:
    def scaffold_form(self):
        """
            Create form from the model.
        """
        converter = self.model_form_converter(self.session, self)
        form_class = form.get_form(self.model, converter,
                                   base_class=self.form_base_class,
                                   only=self.form_columns,
                                   exclude=self.form_excluded_columns,
                                   field_args=self.form_args,
                                   hidden_pk=True,
                                   ignore_hidden=self.ignore_hidden,
                                   extra_fields=self.form_extra_fields)

        if self.inline_models:
            form_class = self.scaffold_inline_form_models(form_class)

        return form_class

The function was copied from flask-admin's sqla source code


[NT-1269, NT-1270] Deprecate BackingViewController

📲 What

Deprecates all old references to BackingViewController, which is now replaced by ManagePledgeViewController.
Allows a creator to view a backing from a message thread via the dashboard.

🤔 Why
BackingViewController was still left over from the work that we did to re-build our checkout flow natively. This PR removes it and updates the callsites that configured and presented it.
🛠 How

Refactored ManagePledgeViewController to support only being configured by a Project param (slug or id) and a Backing param (id).

ManagePledgeViewModel will now always fetch the relevant Project and Backing instead of only sometimes fetching the Project as before. This just helped to simplify this view model as it was becoming unwieldy trying to handle all of the configuration scenarios.

Updated callsites and configuration in the following view controllers:

ActivitiesViewController.
MessagesViewController.
ProjectActivitiesViewController.
ProjectPamphletContentViewController.
ProjectPamphletViewController.

Updated MessagesViewModel to retrieve the backing ID from the MessageThread.

Note: I wasn't able to find a scenario in which the MessageThread did not have a Backing (unless the thread was between a creator and a user that hadn't backed of course). I would just want to be sure there isn't a use case that I'm unaware of in which the thread might not have a Backing.

👀 See

Before 🐛
After 🦋

✅ Acceptance criteria

[ ] Test navigating to manage/view a backing from ActivitiesViewController.
[ ] Test navigating to manage/view a backing from MessagesViewController.

This is accessible via the dashboard logged in as a creator that has a message thread with a backer.

[ ] Test navigating to manage/view a backing from ProjectActivitiesViewController.
[ ] Test navigating to manage/view a backing from ProjectPamphletContentViewController.
[ ] Test navigating to manage/view a backing from ProjectPamphletViewController.

          1 Warning

:warning:
Big PR

  Generated by :no_entry_sign: Danger


Pass louderSpeaker to initialize the speaker status

Closed in favour of https://github.com/membraneframework/react-native-membrane-webrtc/pull/32/files


Adjust subscription locking

Fixes the number of critical sections in SubscriptionFuture from 2 to 1, helping out with the hangs in exchange_block which have been recorded in #134. Also backports the related_subs cleanup from #264.

couldn't see a way to write a test case even for the previous behaviour, but this was found using tracing (koivunej/rust-ipfs?branch=tracing-hack or #270); it seemed that in the hanging cases there were no subscriptions to wake up, and that was because the ordering of:

first critical section of subscriptionfuture (removed related_subs)
finish_subscription from another task
second critical section of subscriptionfuture (inserted back related_subs)

Now that I think of it, I should at least add a test case for polling multiple subscriptions at the same time, unless there was one. I think that should had failed with the previous code.

I am in fact hoping there would be a timeout :)

oh well, there's always the next PR :laughing:

Argh, no failures.


User needs to be able to opt out of custom editor experiment if they ever were part of it.

@DonJayamanne I'm not fully sure what specific issue this bug was addressing (there is no associated merge). But is seems like this has been covered by you recent changes. I'm able to test out the next interface without getting stuck in it like what happened early on. Validated.


chore(release): prepare for 0.27.0

🧀
Highlights: https://github.com/ratatui-org/ratatui-website/pull/644

This PR seems to be introducing windows style newlines for some reason - can you please change these to unix style newlines so it doesn't show up as a huge edit?


Fixing the issue django.contrib.admin.sites.NotRegistered: The model User is not registered.

To fix this https://github.com/stephenmcd/mezzanine/issues/1542

Thanks Vladir


Conflicts with latest A1111 extra networks implementation

On latest automatic11111 master, with their rework of the extra networks. I noticed if you run a generation with a lycoris, and then add a lora, the lycoris does not function anymore. If you remove lyco generate again, then add it back, the lora does not function anymore. You can only get them both on there if you run an empty prompt, and then add all the networks together

Sorry wrong repo


Add lazy loading and async decoding to images

This improves page responsiveness on first load, especially on pages with many images (eg images list in Wagtail admin)
Please check the following:

[x] Do the tests still pass?[^1]
[x] Does the code comply with the style guide?

[x] Run make lint from the Wagtail root.

[x] For Python changes: Have you added tests to cover the new/fixed behaviour?
[ ] For front-end changes: Did you test on all of Wagtail’s supported environments?[^2]

[ ] Please list the exact browser and operating system versions you tested:
[ ] Please list which assistive technologies [^3] you tested:

[x] For new features: Has the documentation been updated accordingly?

Please describe additional details for testing this change.
[^1]: Development Testing
[^2]: Browser and device support
[^3]: Accessibility Target

Can you share data that substantiates it’s a real-world improvement to decode images asynchronously? I have some awareness of how this is meant to work, but no sense of the browser/OS support, what real-world perf looks like, and whether this is desirable for all images or just some.

Following https://twitter.com/zcorpan/status/1537067661452054529 circulating internally, I think it's best not to default to lazy loading. I've removed that default, and instead added the ability for the user to control the default attributes.

Adding a 👍🏻 to our need for this functionality as well.

@RealOrangeOne thanks for updating this PR, would you be up for rebasing on main (a few conflicts) and adding docs for this.
If possible we would love to see this in 4.0 but fully understand if you do not have time for that.
The hard stop for the release candidate is this Friday.
cc @jacobtoppm

Discussed at core team meeting today - the 'opt in' approach for this would be a great feature to have in core and hopefully we can help get this reviewed & merged in.
If not for 4.0 - then for the next release.


[Feat]: Enlisting all the contributors

What feature?
I have an idea

In about section, enlist all the contributors.
Adding the carousel effect into the cards.
Enlisting contributors in decreasing order of the contributions, they done.

Add screenshots
Add screenshots

or may be at the bottom of that section
Code of Conduct

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

@JasonDsouza212 @VenomFate-619 sir, please assign this task to me

https://free-hit.vercel.app/community
we have this.


Editable cell?

This might just be a misunderstanding on my part, but I had expected :replite: provide an editable cell, instead it acts more like a repl, which means copy & pasting the code. That makes sense in that it prevents the user from messing with the example, but it doesn't feel very notebook-y to me and it was a bit confusing.
Did I understand the intention correctly, and is there a plan to have embedded executable cells as well?

I guess this relates to https://github.com/jupyterlite/jupyterlite/issues/148

I guess this relates to jupyterlite/jupyterlite#148

Yes having some kind of editable cell like the mockups and examples in https://github.com/jupyterlite/jupyterlite/issues/148 are still on the radar.


[$500] Language - Error message on New contact method page remains on Spanish

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Version Number: 1.4.26-1
Reproducible in staging?: Y
Reproducible in production?: Y
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Expensify/Expensify Issue URL:
Issue reported by: Applause - Internal Team
Slack conversation:
Action Performed:

Login same user on two devices
Navigate to New dot on mobile device
Go to Staging> Preferences> Language> Spanish
Go to Staging> Profile> Contact method> New contact method
Add same email as login email so error message will trigger
Navigate to other device and change the language to English

Expected Result:
Contact method with error message should appear on English language
Actual Result:
Error message on New contact method page remains on Spanish when other device change the language to English
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[x] Android: Native
[x] Android: mWeb Chrome
[ ] iOS: Native
[ ] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[ ] MacOS: Desktop

Screenshots/Videos
Add any screenshot/video evidence

https://github.com/Expensify/App/assets/78819774/2575a16e-15df-4507-ac4b-32fb3ea7d907
View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~01926d8327e3320b3a
Upwork Job ID:<PHONE_NUMBER>450725376
Last Price Increase: 2024-01-24

Proposal
Please re-state the problem that we are trying to solve in this issue.
Language - Error message on New contact method page remains on Spanish
What is the root cause of that problem?
We do not translate the error messages.
What changes do you think we should make in order to solve the problem?

Use translate in the error messages.

Proposal
Please re-state the problem that we are trying to solve in this issue
Error message on New contact method page remains on Spanish when other device change the language to English.
What is the root cause of that problem?
This happens because when we called the validate memozied function by clicking the Add button while the translation was in Spanish but then we changed the language back to English from the other device, the validate function didn't get called again in order to update the translation to English.
What changes do you think we should make in order to solve the problem?
Add logic that calls the validate function when preferredLocale changes.

Proposal
Please re-state the problem that we are trying to solve in this issue.
Error messages in forms doesn't get updated on locale change
What is the root cause of that problem?
Error messages for form inputs gets set by the validate function prop provided to the FormProvider.
https://github.com/Expensify/App/blob/4b7b8f2480866200c64a8b614a1f485c0112aa11/src/pages/settings/Profile/Contacts/NewContactMethodPage.js#L121
https://github.com/Expensify/App/blob/4b7b8f2480866200c64a8b614a1f485c0112aa11/src/pages/settings/Profile/Contacts/NewContactMethodPage.js#L66-L86
The validate method gets invoked on submit, onBlur and onValue change events. A locale change event doesn't retrigger the validate function. So the error messages continue to be in the same locale as it was previously set.
What changes do you think we should make in order to solve the problem?
As this happens to all Forms in the app, fix should be global. Listen to locale changes in FormProvider and invoke validate method if there are any existing errors in the form.

const FormProvider = forwardRef(
    ({validate, formID, shouldValidateOnBlur, shouldValidateOnChange, children, formState, network, enabledWhenOffline, draftValues, onSubmit, ...rest}, forwardedRef) => {
        const inputRefs = useRef({});
        const {preferredLocale} = useLocalize();

useEffect(() => {
  if (_.keys(errors).length === 0) {
    return;
  }
  const trimmedStringValues = ValidationUtils.prepareValues(inputValues);
  onValidate(trimmedStringValues);
  // Run this effect only on locale change
  // eslint-disable-next-line react-hooks/exhaustive-deps
}, [preferredLocale]);

Result
Before
https://github.com/Expensify/App/assets/6880914/964dda4c-7353-4e69-8858-9c578d5b4f1e
After
https://github.com/Expensify/App/assets/6880914/eb1f1f5f-c06a-4c5d-a0e6-cf34ac14e174
What alternative solutions did you explore? (Optional)
None (edited)

Proposal
Please re-state the problem that we are trying to solve in this issue.
Language - Error message on New contact method page remains on Spanish
What is the root cause of that problem?
We are not running the validate method when we detect the language was changed in NewContactMethodPage.
What changes do you think we should make in order to solve the problem?
The simplest solution is adding a reference to submit button to call validate function when preferredLocale change, we can get this with useEffect:
useEffect(() => (submitButtonRef.current ? submitButtonRef.current.click() : null), [preferredLocale]);
We get this sending a submit button ref in FormProvider like (submitButtonRef={submitButtonRef}):
const submitButtonRef = useRef(null);
.
.
.
<FormProvider
                formID={ONYXKEYS.FORMS.NEW_CONTACT_METHOD_FORM}
                validate={validate}
                onSubmit={addNewContactMethod}
                submitButtonText={props.translate('common.add')}
                style={[styles.flexGrow1, styles.mh5]}
                submitButtonRef={submitButtonRef}
                enabledWhenOffline
            >

Of course we should add the ref to reach  the submit button, like ( ref={submitButtonRef}):
                   <Button
                        ref={submitButtonRef}
                        success
                        pressOnEnter={!disablePressOnEnter}
                        text={buttonText}
                        style={style}
                        onPress={onSubmit}
                        isDisabled={isDisabled}
                        isLoading={isLoading}
                        danger={isSubmitActionDangerous}
                        medium={useSmallerSubmitButtonSize}
                    />

Result:
https://github.com/Expensify/App/assets/5216036/c1c7e180-35f3-4809-a78b-7f0e29916666

Proposal
Please re-state the problem that we are trying to solve in this issue.
Language - Error message on New contact method page remains on Spanish
What is the root cause of that problem?
Error messages are generated by the validate function , which is not called after changing the language.
What changes do you think we should make in order to solve the problem?
We can jus return  translation keys instead translated messages in addErrorMessage  function. and use this keys  to display text. To do that we can make below changes

Change this line with below.

const translatedMessage = message;

Change this line with below

 errorText: Localize.translateIfPhraseKey(errors[inputID]) || fieldErrorMessage,

What alternative solutions did you explore? (Optional)

@rushatgabhane let's get review on these proposal when you have a chance

useEffect directly within FormProvider.js where we check prevPreferredLocale !== preferredLocale and if true we call onValidate(inputValues)

I like @ikevin127's alternative proposal https://github.com/Expensify/App/issues/34684#issuecomment-1897457007
🎀 👀 🎀

@rushatgabhane What do you think about my proposal here?

@rushatgabhane @mountiny  I mentioned the RCA in my proposal. This bug does not appear on all the pages that use FormProvider (you can try the Address page, ...), just appears on the pages that use addErrorMessage

@rushatgabhane interesting points from @DylanDylann if this is not happening across different forms. What do you think?

@DylanDylann makes sense
We are already passing the translation key, right?
https://github.com/Expensify/App/blob/0d0b0a8c9b073c30a9b54de15066789d17f7c718/src/pages/settings/Profile/Contacts/NewContactMethodPage.js#L82

So the issue here is that the validate func isn't run again.

@rushatgabhane Yes. We are passing the translation key, but the addErrorMessage will return something like:
{
    "phoneOrEmail": [
        "The Entered Contact Method already exists.",
        {
            "isTranslated": true
        }
    ]
}

As you can see, when the language changes, we do not have translation key to render the correct text

@rushatgabhane Do you have any feedback about my comment here?

Can I get assigned so I can start working on a PR as per reviewer decision here #34684 (comment) ?
It's been 6 days since the reviewer's decision without any change or re-assignment.
cc @mountiny @michaelhaxhiu

Still waiting for @rushatgabhane feedback

This bug does not appear on all the pages that use FormProvider (you can try the Address page, ...), just appears on the pages that use addErrorMessage

@DylanDylann yeah i agree. your rootcause it correct.
But i don't like the solution. Some forms in the app have validate() that uses addErrorMessage().
We are passing the translation key but it's not being rerendered on language change.
So a solution that comes to my mind is to rerender when the language changes. This is what @ikevin127 is proposing
@mountiny what do you think?

@rushatgabhane

We are passing the translation key but it's not being rerendered on language change.

That is my solution. We should rerender the text error component rather than re-validate once language changes

My solution is calling validate again when the language changes.
It DOES NOT call form submit - it just calls the validate function so that the translations are updated accordingly once the language changes from another instance than the current user with spanish.

My solution is calling validate again when the language changes.
It DOES NOT call form submit - it just calls the validate function so that the translations are updated accordingly once the language changes from another instance than the current user with spanish.
There's no need to change the addErrorMessage and make it more complicated when the function already works as expected.
Reminder: the RCA for this issue is that the form errors do not update when language is changed from another instance of the same account, becausr the validate is not being called again.

@rushatgabhane @mountiny Please check the edit history of @ikevin127 's alternate proposal. It was modified after I posted my proposal here. The original proposal didn't had an alternate solution.

@aswin-s My main solution which suggested calling validate when the language changes was posted before yours.
So who's to say that you did not inspire your solution from my main ? This applies to all other proposals which suggested the validate method.

@aswin-s My main solution which suggested calling validate when the language changes was posted before yours.
So who's to say that you did not inspire your solution from my main ? This applies to all other proposals which suggested the validate method.
Here are the timestamps (in cron order):
(these can be seen by hovering on the time of posts)
ikevin127: Jan 18, 2024, 2:33 AM (GMT+2) - initial proposal timestamp
ikevin127: Jan 18, 2024, 2:33 AM (GMT+2) - 1st edit of proposal timestamp
aswin-s: Jan 18, 2024, 2:36 AM (GMT+2) - initial proposal timestamp
ikevin127: Jan 18, 2024, 2:41 AM (GMT+2) - 2nd edit proposal timestamp [here I added the alternative after his initial proposal]

@aswin-s My main solution which suggested calling validate when the language changes was posted before yours.
So who's to say that you did not inspire your solution from my main ? You had about 1 hour from 1:32 to 2:36 to do so.
This applies to all other proposals which suggested the validate method.
Here are the timestamps (in cron order):
(these can be seen by hovering on the time of posts)
ikevin127: Jan 18, 2024, 1:32 AM (GMT+2) - initial proposal timestamp
ikevin127: Jan 18, 2024, 2:33 AM (GMT+2) - 1st edit of proposal timestamp
aswin-s: Jan 18, 2024, 2:36 AM (GMT+2) - initial proposal timestamp
ikevin127: Jan 18, 2024, 2:41 AM (GMT+2) - 2nd edit proposal timestamp - here I added the alternative after his initial proposal

@rushatgabhane I updated comment

I will go ahead with @ikevin127 proposal given C+ recommendation. I appreciate the discussion and I am sorry that this can only be worked on by one person.
@ikevin127 please try to keep the conversation without any personal comments, take this as a piece of feedback. Such comments are not necessary

@mountiny Do you have any feedback about my comment here?

please try to keep the conversation without any personal comments, take this as a piece of feedback. Such comments are not necessary

My apologies to those affected by my comments ✌, will avoid personal comments in the future.
Will have a PR ready by EOD today (UTC+2)! 🚀

Looks like the automation might've failed to set the 7-day hold for payment since the PR https://github.com/Expensify/App/pull/35509#issuecomment-1945392359 was deployed to production ~4 hours ago 🧐

requested $500 on newdot here - https://staging.new.expensify.com/r/6728438891608723

@laurenreidexpensify could you please attach payment summary, thanks 🙇

Payment Summary:
Contributor: $500 @ikevin127 to be paid in Upwork on 22 Feb
C+: $500 @rushatgabhane to be paid in newdot on 22 Feb

Payment Summary:
Contributor: $500 @ikevin127 paid in Upwork on 22 Feb
C+: $500 @rushatgabhane pls request payment in newdot
Outstanding action: checklist / regression test confirmation from @rushatgabhane

WIP ⚠

The PR that introduced the bug has been identified. Link to the PR:

The offending PR has been commented on, pointing out the bug it caused and why, so the author and reviewers can learn from the mistake. Link to comment:

A discussion in #expensify-bugs has been started about whether any other steps should be taken (e.g. updating the PR review checklist) in order to catch this type of bug sooner. Link to discussion:

Determine if we should create a regression test for this bug.

If we decide to create a regression test for the bug, please propose the regression test steps to ensure the same bug will not reach production again

@rushatgabhane bump to wrap up thanks

@rushatgabhane can we get this one wrapped in next day pls :) Thanks!

@laurenreidexpensify sorry for the delay. updated the checklist : )


Code Organized

-Added all images in a separate folder
-updated the code format

Do change the location of images in index page


Add optional argcomplete support

For users with argcomplete installed, they can have some form of tab-completion, especially for the --type option.
Limitations: Without needing to optimize imports to only import modules when necessary, argcomplete may feel sluggish due to it needing to evaluate the script up until the parser is created. This seems to be a problem with global completion, and there's not much to fix it besides refactoring to move the parser to invoked as quickly as possible.
Follow-ups: Rewriting README.md to address argcomplete support in a better fashion.
Example terminal output (argcomplete installed):

Default terminal output (argcomplete uninstalled):

Being sluggish is fine for now, I was planning on doing a bit of refactoring anyways to decouple the CLI and setting up the deobf, so deferring imports after that is done should be easy.


[Dev] Apply upstream patches

I think this should be ready to go in.
Failure on the Linux test is due to the fact that this was not properly tested yet after changes to main forbidding the "MyColumn:1" syntax (should be probably changed to "tbl_with_case_sensitive_columns.MyColumn")

Yolo


Bug: Fix Webpack plugin registration

Doing what's being recommended:
  plugins: [
    new HtmlWebpackPlugin({
      template: "index.html",
    }),

    new MiniCssExtractPlugin(),

    require('unplugin-skypin/webpack')({
      packages: [
        'hueman'
      ]
    })

    // Add your plugins here
    // Learn more about plugins from https://webpack.js.org/configuration/plugins/
  ],

Results in this:
[webpack-cli] Failed to load 'C:\test\webpack.config.js' config
[webpack-cli] TypeError: require(...) is not a function

While doing this:
const SkypinPlugin = require('unplugin-skypin/webpack');

// ...

  plugins: [
    new HtmlWebpackPlugin({
      template: "index.html",
    }),

    new MiniCssExtractPlugin(),

    new SkypinPlugin({
      packages: [
        'hueman'
      ]
    })

    // Add your plugins here
    // Learn more about plugins from https://webpack.js.org/configuration/plugins/
  ],

Creates this error:
[webpack-cli] Failed to load 'C:\test\webpack.config.js' config
[webpack-cli] TypeError: SkypinPlugin is not a constructor

Maybe unplugin-skypin/webpack has a default export? And you can try something like this:
plugins: [
    require('unplugin-skypin/webpack').default({
      packages: [
        'hueman'
      ]
    })
  ],

I'll assume that fixes the issue and close this for now.


feat: add information for about libraries support

Include additional string resources for use by AboutLibraries to display
information about this library when included as a project dependency.
This change follows the guidelines here and has been tested by including the project as a submodule in AboutLibraries.

Build failure looks to be
`Ensure required Android SDK components
Retrying...
Failed to ensure android components, error: output: > Configure project :lifecyklelog
fatal: No names found, cannot describe anything.
FAILURE: Build failed with an exception.

Where:
Build file '/bitrise/src/lifecyklelog/build.gradle' line: 11
What went wrong:
A problem occurred evaluating project ':lifecyklelog'.

Process 'command 'git'' finished with non-zero exit value 128
`

Weird, project evaluation is passing for me locally but the 'build' task still fails because of a no class def error when running the unit tests. I've made a speculative change and moved the property declaration to the project root.

Would you mind sharing the bitrise yaml?

Bitrise yml
format_version: '7'
default_step_lib_source: https://github.com/bitrise-io/bitrise-steplib.git
project_type: android
trigger_map:
- pull_request_source_branch: "*"
  workflow: primary
workflows:
  primary:
    steps:
    -<EMAIL_ADDRESS>        run_if: '{{getenv "SSH_RSA_PRIVATE_KEY" | ne ""}}'
    -<EMAIL_ADDRESS>{}
    -<EMAIL_ADDRESS>{}
    -<EMAIL_ADDRESS>        inputs:
        - gradlew_path: "$PROJECT_LOCATION/gradlew"
    -<EMAIL_ADDRESS>        inputs:
        - project_location: "$PROJECT_LOCATION"
        - module: "$MODULE"
        - variant: "$VARIANT"
    -<EMAIL_ADDRESS>        inputs:
        - project_location: "$PROJECT_LOCATION"
        - module: "$MODULE"
        - variant: "$VARIANT"
    -<EMAIL_ADDRESS>        title: jacocoTestReport
        inputs:
        - gradle_task: "$MODULE:jacocoTestReport"
    -<EMAIL_ADDRESS>{}
    -<EMAIL_ADDRESS>        title: ktlint
        inputs:
        - gradle_task: ktlint
    -<EMAIL_ADDRESS>        inputs:
        - CODECOV_TOKEN: "$CODECOV_TOKEN"
    -<EMAIL_ADDRESS>{}
    -<EMAIL_ADDRESS>{}
    -<EMAIL_ADDRESS>        inputs:
        - channel: "#lifecyklelog"
        - fields: |-
            App|${BITRISE_APP_TITLE}
            Branch|${BITRISE_GIT_BRANCH}
            Workflow|${BITRISE_TRIGGERED_WORKFLOW_ID}
            Build|#${BITRISE_BUILD_NUMBER}
        - channel_on_error: "#lifecyklelog"
        - api_token: "$SLACK_BOT_TOKEN"
app:
  envs:
  - opts:
      is_expand: false
    PROJECT_LOCATION: "."
  - opts:
      is_expand: false
    MODULE: lifecyklelog
  - opts:
      is_expand: false
    VARIANT: ''
  - opts:
      is_expand: false
    GRADLEW_PATH: gradlew

Doesn't look like its doing anything special when its trying to build

Looks to be because it shallow clones the repository, the git describe --tags invocation is failing. Try the below block, I think that will work
ext.getVersionTag = {
    def stdout = new ByteArrayOutputStream()
    exec {
        commandLine 'git', 'describe', '--tags', '--always'
        standardOutput = stdout
    }
    def longTag = stdout.toString().trim()

    return longTag.substring(longTag.indexOf('/') + 1, longTag.indexOf('-'))
}

I see the tags aren't being cloned along with the repo, so the following line returns the commit hash, not the tag. commandLine 'git', 'describe', '--tags', '--always'
The recommendation from Bitrise is to add a setup script running something like git fetch <remote> 'refs/tags/*:refs/tags/*' to fetch only tags without associated commits
Currently bitrise is performing:
git "init"
git "remote" "add" "origin" "git@github.com:ShaunPlummer/LifecykleLog.git"
git "fetch" "origin" "testing"
git "checkout" "testing"
git "merge" "origin/testing"
git "submodule" "update" "--init" "--recursive"

I've tested this added to the bitrise.yml and it appears to be working
    -<EMAIL_ADDRESS>        inputs:
        - content: |-
            #!/usr/bin/env bash
            # fail if any commands fails
            set -e
            # debug log
            set -x

            git fetch origin 'refs/tags/*:refs/tags/*'

Looks like that added script fixed it 👍 merged in


BatchReceivePolicy implements Serializable.

Motivation
BatchReceivePolicy implements Serializable.
Does this pull request potentially affect one of the following parts:
If yes was chosen, please highlight the changes

Dependencies (does it add or upgrade a dependency): (no)
The public API: (yes)
The schema: (no)
The default values of configurations: (no)
The wire protocol: (no)
The rest endpoints: (no)
The admin cli options: (no)
Anything that affects deployment: (no)

Documentation

Does this pull request introduce a new feature? (no)

cherry-picked into branch-2.5


Force Deletion

The article doesn't discuss how to force a deletion when PowerShell returns "We cannot remove policy '<policyname>' since it is already in pending deletion state."

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

ID: a7b41d5a-c85c-e5d9-90c8-90c1277a223f
Version Independent ID: 8baa76bc-2f5b-fb0c-ba2b-2d8b5d491763
Content: Remove-DlpPolicy (exchange)
Content Source: exchange/exchange-ps/exchange/policy-and-compliance-dlp/Remove-DlpPolicy.md
Service: exchange-powershell
GitHub Login: @chrisda
Microsoft Alias: chrisda

@officedocsbot assign @yogkumgit

Hi @terrogin, thank you for your feedback.
Almost the same than https://github.com/MicrosoftDocs/office-docs-powershell/issues/5461. Please let´s follow it there.

Hi @terrogin,
If you consider this parameter important, I need to kindly redirect you to this resource https://office365.uservoice.com/.
There, your suggestion will reach the product team.

@terrogin  Thank you very much for your kind feedback. Considering your valuable feedback, as advised please log this issue as a User Voice. Concerning team will evaluate and take the appropriate action, as per their schedule. We proceed here to close it. Thanks again for taking out some time and sharing your feedback.


[DOCUMENTATION] fix bold syntax error

Description
fix bold syntax error @qiaojiali

Thanks for your contributions!
I think there are more "bold syntax errors", you can keep finding them and creating more PRs :D

Do my best to fix same error in one PR :)


Build failed while running mvn clean install due to ro/isdc/wro/maven/plugin/Wro4jMojo has been compiled by a more recent version of the Java Runtime (class file version 53.0), this version of the Java Runtime only recognizes class file versions up to 52.0

For java 8 build we have to use <wro4j.version>1.8.0</wro4j.version> not <wro4j.version>1.9.0</wro4j.version> as mentioned in master

added pull request 707

should we close @rstoyanchev  @scothis

@Arka-Bandyopadhyay there is no need to mention individual team members. Issues are triaged when time permits by a member of the team.

Closing in favour of PR #707


updated merge with open-liberty/dev/wlp-jakartaee-transform/rules/jakarta-direct.properties with some reordering to allow easy future merging as needed

Signed-off-by: Scott Marlow<EMAIL_ADDRESS>Moved some entries around, sorted some and synced up again with pull request that I will create for OpenLiberty

Just opened https://github.com/OpenLiberty/open-liberty/issues/14995

Since the license terms of the jakarta-direct.properties in Batavia is Apache but OpenLiberty is using EPL, we could consider a multiple license approach similar to https://github.com/jboss/jboss-jakarta-el-api_spec/blob/master/LICENSE.md where users can decide the license to use (Apache which has more freedom versus EPL which is more restrictive).  We may need some advice on this.

@bstansberry Batavia is using the ASL 2.0 and OpenLiberty uses Eclipse Public License 1.0.  I'm thinking of adding EPL 1.0 as a second (ASL 2,0 or EPL 1.0) license to Batavia.  Not sure if that is really needed though for sharing these properties with OpenLiberty.

@scottmarlow if you and Richard want to add EPL that's fine with me.

I'm fine with dual licensing @scottmarlow & @bstansberry .


feature resolver is required

Following on from https://github.com/substrate-developer-hub/cumulus-workshop/issues/54#issuecomment-851689063 I'm having another go at the workshop.
I'm currently at https://substrate.dev/cumulus-workshop/#/en/1-prep/1-compiling and
# Switch into the Polkadot directory
cd polkadot

# Checkout the proper commit
git checkout v0.9.8

# Build the relay chain Node
cargo build --release

I'm getting this error:
HEAD is now at 3a10ee63c Staking Miner (#3141)
❯ cargo build --release
    Updating git repository `https://github.com/paritytech/substrate`
    Updating crates.io index
    Updating git repository `https://github.com/paritytech/grandpa-bridge-gadget`
  Downloaded static_init v0.5.2
  Downloaded target-lexicon v0.12.0
  Downloaded socket2 v0.4.0
  Downloaded trie-db v0.22.5
  Downloaded security-framework v2.2.0
  Downloaded serde v1.0.126
  Downloaded arrayvec v0.7.0
  Downloaded tracing-core v0.1.18
  Downloaded parity-db v0.2.4
  Downloaded wasmi-validation v0.4.0
  Downloaded futures v0.3.15
  Downloaded async-executor v1.4.1
  Downloaded libp2p-core v0.28.2
error: failed to parse manifest at `/Users/samueljoseph/.cargo/registry/src/github.com-1ecc6299db9ec823/libp2p-core-0.28.2/Cargo.toml`

Caused by:
  feature `resolver` is required

  this Cargo does not support nightly features, but if you
  switch to nightly channel you can add
  `cargo-features = ["resolver"]` to enable this feature

this is my cargo version:
❯ cargo --version
cargo 1.49.0 (d00d64df9 2020-12-05)

I'm on OSX 11.5.1 (20G80)
I can't see a specification of which version of cargo we should be using for the workshop - is that something to add maybe?

I just upgraded
❯ cargo --version
cargo 1.54.0 (5ae8d74b3 2021-06-22)

and it all worked.  Is there some way to require a minimum version of cargo?  Or perhaps just mention one in the workshop?

It is generally assumed that people are building with the latest toolchain, unless explicitly needing to use an older nightly for some reason. (There have been times thus was needed historically)
A PR to remind of this and how to update would be great, sure!

here you go https://github.com/substrate-developer-hub/cumulus-workshop/pull/95 :-)


Azure Media Services 2020-02-01-preview update

Latest improvements:
MSFT employees can try out our new experience at OpenAPI Hub  - one location for using our validation tools and finding your workflow.

Contribution checklist:

[x] I have reviewed the documentation for the workflow.
[x] Validation tools were run on swagger spec(s) and have all been fixed in this PR.
[ ] The OpenAPI Hub was used for checking validation status and next steps.

ARM API Review Checklist

[ ] Service team MUST add the "WaitForARMFeedback" label if the management plane API changes fall into one of the below categories.
adding/removing APIs.
adding/removing properties.
adding/removing API-version.
adding a new service in Azure.

Failure to comply may result in delays for manifest application. Note this does not apply to data plane APIs.

[ ] If you are blocked on ARM review and want to get the PR merged urgently, please get the ARM oncall for reviews (RP Manifest Approvers team under Azure Resource Manager service) from IcM and reach out to them.
Please follow the link to find more details on API review process.

/azp run automation - sdk

/azp run automation - sdk

/azp run automation - sdk

Can one of the admins verify this patch?

@microsoft-rin please fix the code style CI error

/azp run automation - sdk


Replace PC_tree_t (paraconf) by Expression

In GitLab by @ksiero on Jul 30, 2021, 11:22
We are close to be able store YAML tree structure in Expression. The change is colossal, so the idea should be divided to sub-issues, carefully designed and prototyped.

In GitLab by @ksiero on Sep 17, 2021, 11:23
mentioned in commit ac80b3784d1b27f0cdaa1d88e7bfec94904b840e

In GitLab by @ksiero on Oct 5, 2021, 13:02
mentioned in commit c7b4bce0a0e60735278a5febb006e1b4bca5b0e4

In GitLab by @ksiero on Oct 6, 2021, 14:05
mentioned in commit ecb97d69520759a5a48b9dca7336e2fb55617bfb

In GitLab by @ksiero on Sep 1, 2021, 11:32
First step should be to get PC_tree_t from user, pass it to Expression and create Reference from it.
We should replace PC_tree_t by Expression.
To do that we need:

[x] Expression should support template value method
[x] Expression should support to_map(), to_vector(), to_ordered_map()
[x] Ref::scalar_value should try to implicitly get read access
[x] Ref::scalar_value should support std::string type
[x] Ref::operator[] should support not only size_t, but also int to not always use [0UL]
[ ] Ref iterator, begin() and end() to easily iterate over subrefs

In GitLab by @ksiero on Sep 17, 2021, 11:35
mentioned in commit 81d72e3de2a377f223cd6884343439c600f01cb8

In GitLab by @ksiero on Nov 18, 2021, 12:00
mentioned in commit 1c3acd789f8609703bdca74e9f473fe2afc1c1ec

In GitLab by @ksiero on Oct 11, 2021, 14:40
mentioned in commit f94c74c9d867b95e20198d92b55f19e450f1e2de

In GitLab by @ksiero on Nov 17, 2021, 13:50
mentioned in commit 5e5ffe96ef6fd6d85789a6c880631bdc2edf1104

unassigned @ksiero

In GitLab by @ksiero on Nov 16, 2021, 12:47
mentioned in commit 2938d8dfbe6603c94bf7811df1914686198e211b

In GitLab by @ksiero on Nov 16, 2021, 13:16
mentioned in commit 0256f127ed1bf26c2c6f3959f3712f468acee625

In GitLab by @ksiero on Nov 24, 2021, 12:23
mentioned in commit 1785de217bc3d78c068290f2df5794bb025345c1

In GitLab by @ksiero on Oct 7, 2021, 08:54
mentioned in commit a7e4417eb20b880c2616cf64d86e4e4247d44636

In GitLab by @ksiero on Nov 24, 2021, 12:52
mentioned in commit a4c3576738588dd468ac7683837c626f1b525494


Gallery is showing display none elements

Hey, i'm having troubles in a project where i need to use magnific-popup in combo with mixitup.
I have a grid of clickable items that starts a magnific-popup gallery if u click on them, the problem pop out when i filter those items using mixitup (mixitup adds css display property in line on the elements).
Substantially, after i've filtered some items, magnific-popup gallery still include the display none elements and i do not want this to happen.
Here's an example: http://codepen.io/anon/pen/zrzgpN
Sorry for my bad english, hope someone can help me. Thanks!
Cheers,
Michele

Solved