No rule to make target `/usr/lib/gcc/x86_64-linux-gnu/4.8/libgfortran.so'

I tried $ instant-clean but it did not solve this problem.
python poisson_convergence1.py
Calling DOLFIN just-in-time (JIT) compiler, this may take some time.
In instant.recompile: The module did not compile with command 'make VERBOSE=1', see '/home/sohail/.instant/error/dolfin_compile_code_7c28a3053842033e63e724baf19e43119eaa0567/compile.log'
Traceback (most recent call last):
File "poisson_convergence1.py", line 16, in <module>
 f = Expression("5*pi*pi*sin(pi*x[0])*sin(2*pi*x[1])")
 File "/home/sohail/anaconda/envs/fenics27/lib/python2.7/site-packages/dolfin/functions/expression.py", line 602, in __new__
mpi_comm=mpi_comm)
File "/home/sohail/anaconda/envs/fenics27/lib/python2.7/site-packages/dolfin/compilemodules/expressions.py", line 217, in compile_expressions
mpi_comm=mpi_comm)
File "/home/sohail/anaconda/envs/fenics27/lib/python2.7/site-packages/dolfin/compilemodules/expressions.py", line 145, in compile_expression_code
mpi_comm=mpi_comm)
File "/home/sohail/anaconda/envs/fenics27/lib/python2.7/site-packages/dolfin/compilemodules/jit.py", line 64, in mpi_jit
return local_jit(*args, **kwargs)
File "/home/sohail/anaconda/envs/fenics27/lib/python2.7/site-packages/dolfin/compilemodules/compilemodule.py", line 458, in compile_extension_module
**instant_kwargs)
 File "/home/sohail/anaconda/envs/fenics27/lib/python2.7/site-packages/instant/build.py", line 563, in build_module
 recompile(modulename, module_path, new_compilation_checksum, build_system)
 File "/home/sohail/anaconda/envs/fenics27/lib/python2.7/site-packages/instant/build.py", line 165, in recompile
instant_error(msg % (cmd, compile_log_filename_dest))
File "/home/sohail/anaconda/envs/fenics27/lib/python2.7/site-packages/instant/output.py", line 85, in instant_error
raise RuntimeError(text)
RuntimeError: In instant.recompile: The module did not compile with command 'make VERBOSE=1', see '/home/sohail/.instant/error/dolfin_compile_code_7c28a3053842033e63e724baf19e43119eaa0567/compile.log'

The compile.log, mentioned in the above error log is: http://pastebin.com/U7fhfEiX
 $ conda info
 Current conda install:

         platform : linux-64
    conda version : 3.16.0
   conda-build version : 1.11.0
   python version : 2.7.10.final.0
   requests version : 2.7.0
   root environment : /home/sohail/anaconda  (writable)
   default environment : /home/sohail/anaconda/envs/fenics27
   envs directories : /home/sohail/anaconda/envs
    package cache : /home/sohail/anaconda/pkgs
    channel URLs : https://repo.continuum.io/pkgs/free/linux-64/
                    https://repo.continuum.io/pkgs/free/noarch/
                    https://repo.continuum.io/pkgs/pro/linux-64/
                    https://repo.continuum.io/pkgs/pro/noarch/
      config file : /home/sohail/.condarc
    is foreign system : False

The conda list: http://pastebin.com/43KDy6K3

I am unable to reproduce this. Could you please paste conda info -a so we can see the environment variables too? In the meanwhile, please do conda update --all in the environment.

After doing conda update --all, the conda info -a returns: http://pastebin.com/Bj38KQEh

I don't think this will solve the issue, but just to check: would you try this?
$ bash
$ unset LD_LIBRARY_PATH
$ python poisson_convergence1.py

In the meanwhile, more questions:

I assume you're using Ubuntu 14.04, am I right?
Is libgfortran installed in your system? In other words, is the file /usr/lib/gcc/x86_64-linux-gnu/4.8/libgfortran.so present in your system?

If you don't have libgfortran, then you will need it. There might be a way to provide it in conda, but I would have to make sure first.
If you already have it, then I still won't know what's happening and I will request you to do the following:
$ instant-clean
$ echo 'from dolfin import *
> f = Expression("5*pi*pi*sin(pi*x[0])*sin(2*pi*x[1])")
' > test.py
$ python test.py

and upload to pastebin all the files that appear under ~/.instant/.

For the record: there's actually the option to install gfortran and libgfortran in conda: https://groups.google.com/a/continuum.io/d/msg/conda/HZlV2LXxWhs/UM_EgEeiYZIJ.

Here is the answer to every question I was asked: http://pastebin.com/0h7kN2NP
I don't think I need to install libgfortran, correct???
~.instant/ contained too many files, so here is the archive: https://www.sendspace.com/file/31ecqv

Actually, as a last step, try to remove libgfortran from conda (conda remove libgfortran). If that doesn't work, I will have to inspect your files...

conda remove libgfortran removed libgfortran as conda package.

Yes, I meant to try to rerun the script without the libgfortran conda package.

https://www.sendspace.com/file/molvi8

The above archive is when I run possion_convergence1.py and here is when I run test.py: https://www.sendspace.com/file/myowfl

The conflict with libgfortran dissapeared now, but apparently you mistyped the expression in the last case:
       values[0] = 5*pi*pi*sin(pi*x[0])*sin(2*pi*x[1];

Holy sh*t, now I found what's happening. You have an older version of gfortran than me! The program is looking for 4.8 and you have 4.6 (https://github.com/Juanlu001/fenics-recipes/issues/31#issuecomment-138124574). Can you confirm again what is your Linux distro and version? Now everything makes sense, but I'll have to figure out how to fix it.

Thanks for being so kind.
$ lsb_release -a
No LSB modules are available.
Distributor ID:	Ubuntu
Description:	Ubuntu 12.04.5 LTS
Release:	12.04
Codename:	precise

Moreover my gcc version is 4.8.1 and gfortran is 4.6.4. I always thought that gfortran comes with gcc.

On 2015-09-07 01:16, sakhan007 wrote:

Moreover my gcc version is 4.8.1 and gfortran is 4.6.4. I always
thought that gfortran comes with gcc.

Yeah, that looks like a strange mismatch for me too!
I assume that you cannot upgrade either gfortran or Ubuntu (FEniCS
developers dropped Ubuntu 12.04 testing this year), so there are two
options:

Try FEniCS 1.4.0, which in principle is compiled for older Linux
versions, or
Wait for me to compile FEniCS 1.6.0 with the conda gfortran and
libgfortran packages. Unfortunately I won't have time to do it until
next week probably.

I can upgrade gfortran and Ubuntu both as I have super user right.
What do you suggest?

Sorry, I automatically assumed you were in a shared server or something!
Well, if you find a way to install gfortran 4.8 in your Ubuntu version that would be perfect. Obviously upgrading to a newer version of Ubuntu also would work but I understand that this can be a total mess.
Anyway, if you are not in a hurry and wait for the new package I would use your help to try if everything works :)

Ok I will wait.

I'm getting some errors while trying to build instant with anaconda gcc and gfortran:
https://groups.google.com/a/continuum.io/d/msg/anaconda/lmJatGVqklU/QVIfYFkWCAAJ

There are some suggestions to improve portability here:
https://github.com/conda/conda-recipes/pull/279#issuecomment-98254389
I would need to see how to fit them into instant. I might need to submit an upstream patch, but definitely I will need to look at it with some care. @sakhan007 You should probably update gfortran because I will need some weeks to get this right.

Does just updating gfortran would do the trick? or I have to build the whole fenics afterwards??

In principle updating gfortran should be enough.
On Sep 14, 2015 10:16 AM, "sakhan007"<EMAIL_ADDRESS>wrote:

Does just updating gfortran would do the trick? or I have to build the
whole fenics afterwards??
—
Reply to this email directly or view it on GitHub
https://github.com/Juanlu001/fenics-recipes/issues/31#issuecomment-139993717
.

Hey guys, I didn't read the above thread too closely, but I noticed you referenced conda/conda-recipes#279.
Just FYI, for best results, you should use the latest version of the gcc package, which is only available via the asmeurer channel:  conda install -c asmeurer gcc=4.8.5.  That version also includes some important fixes from conda/conda-recipes#401.
(BTW, the gcc package includes gfortran, and the libgcc package includes libgfortran.)
Theoretically, the conda guys will eventually push that gcc package over to the main anaconda channel.  Until then, you'll need to specifically reference the asmeuer channel (or copy that package to your own channel, of course.)  Just to repeat: Don't use the gcc 4.8.2 package that conda provides by default. It has portability problems.

Hi @stuarteberg, do you know if the Anaconda guys already pushed the latest fixes to the main channel? I'm waiting for that to fix my own packages. Thanks!

do you know if the Anaconda guys already pushed the latest fixes to the main channel?

It looks like they finally did:
https://anaconda.org/anaconda/gcc/files
$ conda search gcc
Fetching package metadata: ....
gcc                       .  4.8.2                         5  defaults
                          .  4.8.5                         3  defaults
libgcc                    .  4.8.4                         1  defaults
                          .  4.8.5                         1  defaults

Thanks a lot for your answer, I will try to fix our issues now.

I pushed some changes that should fix these issues, but now I'm getting build errors related to PETSc. This is the neverending story :)

New packages uploaded. @e-aranda @sakhan007 Could you try updating DOLFIN to see if the errors are gone already? In principle the packages should be much more portable now.

Juan Luis,
thanks for your effort, but...
I installed gcc-4.8 in my system and everything was fine. In order to try the new packages, I uninstalled gcc-4.8 and update dolfin from your channel. Then, running demo_poisson.py gives the following:
$ python demo_poisson.py
Vendor:  Continuum Analytics, Inc.
Package: mkl
Message: trial mode expires in 30 days
Calling DOLFIN just-in-time (JIT) compiler, this may take some time.
Calling DOLFIN just-in-time (JIT) compiler, this may take some time.
Calling FFC just-in-time (JIT) compiler, this may take some time.
Calling FFC just-in-time (JIT) compiler, this may take some time.
Solving linear variational problem.
libGL error: unable to load driver: nouveau_dri.so
libGL error: driver pointer missing
libGL error: failed to load driver: nouveau
libGL error: unable to load driver: swrast_dri.so
libGL error: failed to load driver: swrast
X Error of failed request:  BadValue (integer parameter out of range for operation)
  Major opcode of failed request:  156 (GLX)
  Minor opcode of failed request:  3 (X_GLXCreateContext)
 Value in failed request:  0x0
 Serial number of failed request:  58
 Current serial number in output stream:  59
(fenics27)$

How can I undo the changes I made?
Ernesto

Hello @e-aranda, sorry for the inconvenience! Apparently I introduced some problem with the graphics which I have yet to understand. I will open a new issue, in the meanwhile you can manually install the previous build and run $ conda install /tmp/linux-64_dolfin-1.6.0-py27_2.tar.bz2.


UnityDemo not working on iOS devices and Mac

I have forked ZXing.Net for supporting iOS/OSX/Android under Unity.
https://github.com/cedricpoon/ZXing.Net
However, all encoding functions (e.g. creating QR code) are removed due to heavy dependent coding on System.Drawing. I hope it may helps you.


[FEAT]Card context-automatic based on parent-child relationship under Outliner mode

Discussed in https://github.com/st3v3nmw/obsidian-spaced-repetition/discussions/193

Originally posted by zhixiangCai July 15, 2021
Many obsidian users prefer using Outliner mode by Oultiner plugin to write notes like RoamResearch and RemNote.
If the flash cards can automatic have the parent items context of the answers will highly improve the efficiency of flascards making and memory, because the parent items can directly and easily provide memo recall clues.
The flashcard is like the following:

a
b

c

d

Question:: Answer

The flashcard in preview mode:

b (note: no "a" above)

c

d

Question::

Answer

Sorry, this is a repeated issues, I do not know how to delete it.

You could just close the duplicate.

Closing dup...


Intel Realsense viewer: Deepth stream start failure

win10 system, visualstudio2015, cuda8.0, 1080ti graphics card, Intel realsense viewer v2.25.0, camera D435, D435 FW vertion <IP_ADDRESS>

And then I re-clicked the Stereo Module on-off button, it  worked...

I don't know what's going on... （´Д`）

And the next day I ran it in my nootbook, it worked well:

never wrong!
My nootbook information:
win10 64bit 2018 version, vs2015, cuda9.0 with the same cudnn, MX250 graphic card, Intel Realsense viewer2.25.0, camera D435, FW version<IP_ADDRESS>...

@Dontla, I'm closing this now as there is no follow up to investigate.
Feel free to reopen if the issue resurfaces.


WSL support

This PR fixes #133 by adding a new setting: WSL mode. When WSL mode is on, it runs all commands in WSL via wsl -e <command> <args...>. On non-Windows installations, the WSL setting will not appear.
TODO:

[x] Test on non-clean-install WSL
[x] Is this the most ergonomic option? Would users prefer to have a per-language toggle?
[ ] Implement in non-basic executors (waiting on #140)

The other PR is merged. I'm looking forward to trying out your solution in notebook mode!

Complete & tested with Python and NodeJS!
There was a minor bug where, if the language is not installed, WSL will simply exit without any error or output. To make sure the user understands what's happened, I've added a notice when runtimes exit.

Looks great, thank you!


fixing mastervert to use parallel cl

Replacing the snafu invocation with the original parallel invocation of cluster loader, as snafu's cl doesn't support the parallel invocation yet.

closing as we should now switch to kubeburner.


added some changes in DenyServiceExternalIPs

This pull request is to resolve the unclearness of the topic DenyServiceExternalIPs, as mentioned in the issue https://github.com/kubernetes/website/issues/39579.

Hi @Shivam164, please sign Easy CLA first which help reviewers to review your code.

Hey @NitishKumar06, Please have a look at it. I think it should be fine now.

Hey @davidopp, please have a look at it.

/lgtm

@kubernetes/sig-auth-pr-reviews : PTAL

Hey @Shivam164 , please would you be able to address the changes suggested by @tengqm ?

/assign @thockin
Could you please take a look.

@Shivam164 , I'm closing this PR because the issue (https://github.com/kubernetes/website/issues/39579) this PR fixes is resolved and it's been a few months and a comment/recommendation has not been addressed.
If you feel the changes proposed in this PR is an improvement, feel free to reopen this PR with /reopen in a comment
/close


Can't get RSS feed from https://alexvermeer.com/feed

Can't get RSS feed from https://alexvermeer.com/feed/. Getting "We haven't found any articles for this RSS feed yet" error message.
Winds version: 3.1.11

@kirillzh is this the feed? https://winds.getstream.io/rss/5ca6e1cfbdb0002d64b30b04


Remove unneeded surefire configuration for hadoop-file-system

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

[ ] Make sure there is a JIRA issue filed for the change (usually before you start working on it).  Trivial changes like typos do not require a JIRA issue.  Your pull request should address just this issue, without pulling in other changes.
[ ] Each commit in the pull request should have a meaningful subject line and body.
[ ] Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue.
[ ] Write a pull request description that is detailed enough to understand what the pull request does, how, and why.
[x] Run mvn clean verify to make sure basic checks pass. A more thorough check will be performed on your pull request automatically.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

Coverage decreased (-0.007%) to 70.41% when pulling 490cf7e875be6e7de4766dd6c7f13a0b65622164 on iemejia:remove-unneeded-surefire-config into 6ca53055be7049ecb9c93d8ff89652a1880f0c9b on apache:master.


tabView error

Hello,
I am writing to report an error that I encountered while trying to use the demo test application in the "test" folder. Specifically, I downloaded the application and I am using it in a Windows environment. Every time I click on a tab to switch pages, I receive four error messages that read as follows:
"Binding: 'IsSelected' property not found on 'UraniumUI.Material.Controls.TabItem', target property: 'Microsoft.Maui.Controls.Grid.Bound'"
I am not sure how to resolve this issue, and I would greatly appreciate any guidance or assistance that you can provide. Thank you for your help.

Possibly the same error with https://github.com/enisn/UraniumUI/issues/302#issuecomment-1537167082
There should be x:DataType when Compiled Bindings feature is used.

Hi @t3377806 , may i know whether you have resolved the issue? I am hitting the same error.
@enisn Sorry, i am not really understand

Possibly the same error with #302 (comment)
There should be x:DataType when Compiled Bindings feature is used.

The solution showed in the other post does not solve the problem and it's also a different error, as this happens without adding any custom binding or such.
The error happens, even if you use the first example in "Usage" here: https://enisn-projects.io/docs/en/uranium/latest/themes/material/components/TabView
<material:TabView>
    <material:TabItem Title="Tab One">
        <material:TabItem.ContentTemplate>
            <DataTemplate>
                    <Label Text="First Tab" />
            </DataTemplate>
        </material:TabItem.ContentTemplate>
    </material:TabItem>

    <material:TabItem Title="Tab Two">
        <material:TabItem.ContentTemplate>
            <DataTemplate>
                    <Label Text="Second Tab" />
            </DataTemplate>
        </material:TabItem.ContentTemplate>
    </material:TabItem>

    <material:TabItem Title="Tab Three">
        <material:TabItem.ContentTemplate>
            <DataTemplate>
                    <Label Text="Third Tab" />
            </DataTemplate>
        </material:TabItem.ContentTemplate>
    </material:TabItem>
</material:TabView>

I am currently having the same issue. I am also seeing a performance drop when trying to click on each individual tab. The small catch with my setup is that I am using MacCatalyst and .NET 8.0, which is still in the preview stage (I decided to use the .NET 8.0 instead of the finalized .NET 7.0, because all of the .NET 7.0 bug fixes were being pushed to .NET 8.0 ONLY).

It seeems similar problem with https://github.com/enisn/UraniumUI/issues/141#issuecomment-1401547853
You can try using xmlns:material="clr-namespace:UraniumUI.Material.Resources;assembly=UraniumUI.Material" namespace instead

Shouldn't the IsSelected property be bound in OneWay mode?

Nope. Doesn't work. I tried using the suggestion above and then I tried to use the built-in autocomplete feature to populate the xmlns declaration and both returned the error below.

I think I found the source of the error. At the time of this writing, on line number 492, 'IsSelected' is not a property - it is a field/variable.
You'd need to change it to something like this (written manually, not tested):
public bool IsSelected { get => TabView.SelectedTab == this || (TabView.CurrentItem != null && TabView.CurrentItem == Data); }
A few notes about this suggestion:

It does not have a setter because when looking through the code, I have not seen a single instance where 'IsSelected' has been set aside from the initial declaration.
Converting it to a property should hopefully solve the issue, but may potentially have a small side effect. The demo XAML pages that use this property may potentially throw an error if it attempts to set the property via binding. To solve this issue, you can change the binding to {Binding Path=IsSelected, Mode=OneWay}.


Use the GitHub API to decide between Issues and PRs

This has been a papercut since I first joined #servo, and it's not that hard to fix.

Thanks!

Fixed by #66.


Update dockerfile versions

Pull request
Description
This also removes the installation step of installing libcurl and libssl :)
Related
Checklist

[x] The RFC, if required, has been submitted and approved
[x] Any user-facing impact of the changes is reflected in docs.tremor.rs
[x] The code is tested
[x] Use of unsafe code is reasoned about in a comment
[x] Update CHANGELOG.md appropriately, recording any changes, bug fixes, or other observable changes in behavior
[x] The performance impact of the change is measured (see below)

Performance

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
3 unchanged lines in 3 files lost coverage.
Overall coverage increased (+0.01%) to 85.629%

Files with Coverage Reduction
New Missed Lines
%

src/connectors/source/channel_source.rs
1
78.33%

src/connectors/source.rs
1
80.48%

src/system/flow_supervisor.rs
1
85.59%

Totals

Change from base Build<PHONE_NUMBER>:
0.01%

Covered Lines:
40339

Relevant Lines:
47109

💛  - Coveralls


installation error

hi i follow the procedure and i get this error when i run this command
go get github.com/kahing/goofys
i am using go version 1.8
ccloud@amitkertes-goofys:~$ go get github.com/kahing/goofys
github.com/kahing/goofys/vendor/google.golang.org/grpc/credentials/internal
work/src/github.com/kahing/goofys/vendor/google.golang.org/grpc/credentials/internal/syscallconn.go:29: syntax error: unexpected = in type declaration
github.com/kahing/goofys/vendor/google.golang.org/grpc/internal/syscall
work/src/github.com/kahing/goofys/vendor/google.golang.org/grpc/internal/syscall/syscall_linux.go:79: tcpconn.SyscallConn undefined (type *net.TCPConn has no field or method SyscallConn)
work/src/github.com/kahing/goofys/vendor/google.golang.org/grpc/internal/syscall/syscall_linux.go:100: tcpconn.SyscallConn undefined (type *net.TCPConn has no field or method SyscallConn)
github.com/kahing/goofys/vendor/google.golang.org/grpc/resolver/dns
work/src/github.com/kahing/goofys/vendor/google.golang.org/grpc/resolver/dns/dns_resolver.go:91: unknown field 'Dial' in struct literal of type net.Resolver
github.com/kahing/goofys/vendor/golang.org/x/net/http2
work/src/github.com/kahing/goofys/vendor/golang.org/x/net/http2/server.go:220: s.RegisterOnShutdown undefined (type *http.Server has no field or method RegisterOnShutdown)

you need to upgrade go


Tectonic authentication erro

Issue Report Template
Tectonic Version
1.7.9
Environment
What hardware/cloud provider/hypervisor is being used with Tectonic?
vmware esxi 5.5
Expected Behavior
login to Tectonic account
Actual Behavior
brouser URL flashed to
https://tectonic.maydomain/error?error=invalid_code&error_type=auth
and then changed to
https://tectonic.mydomain/identity/auth/local?req=aenmaaau4iws6v45kfgykkfua
Reproduction Steps

...
...

Other Information
Esterday everything work fine. Today i'am can't login to Tectonic GUI. The password is correct. If enter wrong pass or login - there is red window appears, but in my case i enter valid login and pass? but page reload after that and Tectonic Identity page reappear.
Rebooting worker nodes does not solve problem. I still able to communicate via ssh but not GUI.
After rebooting worker nodes i see this in console (maybe it is not the issue but maybe helpful):
https://ibb.co/bSeHCm
Feature Request
Environment
What hardware/cloud provider/hypervisor is being used with Tectonic?
Desired Feature
Other Information

I have the same issue, it persist from version 1.7.5(cluster was installed with this version) and now on 1.7.9.
kubectl working fine and any other login as well, issue only with GUI access.
Issue could be solved with rebooting master and appear again after a day or something. I'm using Google Chrome browser. I would assume that it is due to some auth token expiration or like that.

I used firefox and try on chrome too. Clear browser cache not solve problem.

When i rebooted last 3 master node i can auth Tectonic but there is a message "Oh No! There are something wrong!" or something like that. And there is a screenshot
https://ibb.co/j6xOJR

Now after some time everything is fine
https://ibb.co/e4wdk6
but who gives guarantee that this not happen in the future? Everytime reboot master nodes is not a solution )

two weeks gone - flying normal.

Sorry, so issue just gone without no action? as I still need to reboot master each time, to connect to gui.

Maybe some DNS issue here. I'am using simple DNS round-robin and after shutting down some nodes cannot connect to tectonic console. I'am open another issue on that about best practices on how to make HA for tectonic in case there some nodes are down.

This happens

I fixed it for my cluster. I did 2 things:

In POD logs for tectonic-console there were errors for name resolution of console hostname, so I fixed DNS for all cluster nodes/pods.
I removed round-robin for DNS record tectonic-console and left only one A record (as in test environment I'm not using load balancer in front of the cluster and probably auth issue related to not persistent connections coming to different nodes with the same token).


Organize “item”-related files in a folder per ng style guide

Nice :)  @vchimev just curious, I just now tried ng create project-name --ng and it did not pull down the latest, do you know why?

@NathanWalker, because we haven't published it yet. :)
We are going to publish<EMAIL_ADDRESS>along with the core packages for the {N} 2.5.0 release.
In the meantime, if you want to use it you could:
tns create project-name --template https://github.com/NativeScript/template-hello-world-ng.git#master

Merged with aa8ac9625473bfc123b112a89c438bd418340938. Thanks!


Fixed pyright errors inside /memory

Fixed pyright errors in the memory directory

pyright errors are in the unit tests not in /memory


SCA of golang

Hello,
I love this approach of polyglot coding. Great Idea!
I 've looked a bit in the go sources.
Not sure, but GoReportCard are not such keen:

92 Issues in 378 Files. Can you have look?
Yours, CF

This is an interesting tool thanks for bringing it to our attention. A lot of these issues are things like cyclomatic complexity and comments which may be a little too nitpicky for our needs. We will have to take a closer look. Things like the formatting though this is very helpful in uncovering.
92 issues are a lot for a single GitHub issue! Perhaps you can break them down into more specific areas?

I'm going to close this, its an interesting tool, but until we can isolate it in a more granular fashion the report is overwhelming and in many cases do not apply.


Revert "Remove obsolete member_label usage"

Reverts codevise/pageflow#1272
Selected share providers were no longer displayed as checked when opening the account edit form.
REDMINE-17257

Coverage remained the same at 99.053% when pulling 9f1a07387e4e677553f418f08e423c69aeac5d58 on revert-1272-remove-member-label into 24b7ff9328bace95b2c46dcc9634ead0d1ab0bb0 on master.


Remove reva https config from chart

See https://github.com/pondersource/sciencemesh-nextcloud/issues/80#issuecomment-1236889821

https://github.com/pondersource/charts/blob/addbdb4/revad/values.yaml#L241-L243 and also change the port number from 443 to whatever the ingress expects it to be (port 80?)

this issue has been closed with this commit


Acceptance test fails in PhantomJS

Hi, using mirage 0.3.0 beta 4 with ember-cli 2.10.0.
My test pass in Chrome but not in PhantomJS.  This is my acceptance test:
import { test } from 'qunit';
import moduleForAcceptance from 'strrr/tests/helpers/module-for-acceptance';

moduleForAcceptance('Acceptance | shows');

test('I can view the shows', function(assert) {
	assert.expect(2);
	let shows = server.createList('show', 10, 'timestamps');
	visit('/shows');
	andThen(function() {
		assert.equal(currentURL(), '/shows');
		assert.equal(find('.ListGroup-item:first h3').text().trim(), shows[0].attrs.title, 'the data matches mirage');
	});
});

It seems to be the createList method that causes the error, which looks like this:
Acceptance | shows: I can view the shows
    ✘ Died on test #1 http://localhost:7357/assets/tests.js:7:18
    exports@http://localhost:7357/assets/vendor.js:140:37
    requireModule@http://localhost:7357/assets/vendor.js:32:25
    require@http://localhost:7357/assets/test-support.js:7107:14
    loadModules@http://localhost:7357/assets/test-support.js:7099:21
    load@http://localhost:7357/assets/test-support.js:7129:33
    http://localhost:7357/assets/test-support.js:7013:22: undefined is not a constructor (evaluating 'Object.assign({}, modelClass.be
longsToAssociations, modelClass.hasManyAssociations)')
        associationsFor@http://localhost:7357/assets/vendor.js:131413:29
        http://localhost:7357/assets/vendor.js:129579:63
        inverse@http://localhost:7357/assets/vendor.js:129591:13
        _disassociateFromHasManyInverses@http://localhost:7357/assets/vendor.js:130926:42
        _disassociateFromOldInverses@http://localhost:7357/assets/vendor.js:130913:48
        http://localhost:7357/assets/vendor.js:130904:46
        forEach@[native code]
        _saveHasManyAssociations@http://localhost:7357/assets/vendor.js:130903:78
        _saveAssociations@http://localhost:7357/assets/vendor.js:130885:38
        save@http://localhost:7357/assets/vendor.js:130532:31
        create@http://localhost:7357/assets/vendor.js:131311:45
        create@http://localhost:7357/assets/vendor.js:131248:33
        create@http://localhost:7357/assets/vendor.js:133965:44
        createList@http://localhost:7357/assets/vendor.js:134002:38
        http://localhost:7357/assets/tests.js:10:32
        runTest@http://localhost:7357/assets/test-support.js:3494:34
        run@http://localhost:7357/assets/test-support.js:3480:13
        http://localhost:7357/assets/test-support.js:3654:15
        process@http://localhost:7357/assets/test-support.js:4716:26
        begin@http://localhost:7357/assets/test-support.js:4699:11
        http://localhost:7357/assets/test-support.js:4660:11

My mirage/models/show.js has a belongsTo defined with a hasMany on the other model. If I can provide any more information let me know.

First time using Mirage and the benefits are now obvious, so thank you!
I'm using Phantomjs 2.1.1. I tried to debug a bit more and now the culprit seems to be one of my two models.
import { Model, hasMany } from 'ember-cli-mirage';
export default Model.extend({
	videos: hasMany()
});

If I remove the hasMany relationship the test passes again.

Is there an error? What is the response to the web request, and what do you expect it to be?

The error appears only through my acceptance test in Phantomjs. Pasted the error in the issue above. There are some hasMany methods hidden in there...
In the front-end (and chrome ci tests) the relationships and mirage all work.

Ok so if it's still that error it's Object.assign is not a function, it's because of the version of JS that Phantom is using.
I really don't like PHantom because it uses an outdated version of V8, all hte major browsers support Object.Assign right now (I actually thought this was transpiled by Babel). In the meantime can you switch your test runner to Chrome only? Though I do think we should change this to just use _assign from Lodash, since so many people do use Phantom.
Anyway, that's what's going on and I can update it when I get back to my computer.

Aha, that makes sense in a JavaScript kind-of way. As long as Phantom is enabled by default in ember-cli I assume more people will run into it.
Also https://github.com/ember-cli/rfcs/pull/86

If you really want to use Phantom you can also enable babel polyfill in your ember-cli-build.

Thanks, this was helpful.  I'm going to try switching to Firefox from phantom.

d6373945220bac85a23e570fc9ce23af0661fcf0 closes


Using elevation with animation prop on Card causes the production app to render empty screen

Current Behavior
Hi!
Whenever I render <Card /> compontent with both animation and elevation prop like this:
<Card
          elevation="$4"
          height={100}
          width={100}
          backgroundColor="green"
          animation="medium"
          onPress={() => {
            console.log('press');
          }}
          pressTheme
          pressStyle={{
            scale: 0.9,
          }}
          enterStyle={{
            opacity: 0.5,
            y: 20,
          }}
        >
          <Text>Contents</Text>
        </Card>

on production build, it causes the app to render an empty screen instead. It works as expected on debug build.
Expected Behavior
The app should render correctly on the production build.
Tamagui Version
1.94.3

Platform (Web, iOS, Android)
iOS

Reproduction
1. Clone following repo: https://github.com/TheUnderScorer/TamaguiIssue
2. Run `pnpm install` and `expo prebuild ios`
3. Run `expo run:ios --configuration Release`
4. Notice that app render empty, white screen.
5. Update `App.js` by removing `elevation` prop from `<Card />` component
6. Build app again via `expo run:ios --configuration Release`
7. Notice that app renders correctly

System Info
System:
    OS: macOS 14.4.1
    CPU: (8) arm64 Apple M1 Pro
    Memory: 462.63 MB / 32.00 GB
    Shell: 5.9 - /bin/zsh
  Binaries:
    Node: 21.6.2 - ~/.nvm/versions/node/v21.6.2/bin/node
    npm: 10.2.4 - ~/.nvm/versions/node/v21.6.2/bin/npm
    pnpm: 8.15.4 - ~/.nvm/versions/node/v21.6.2/bin/pnpm
  Browsers:
    Chrome: 123.0.6312.107
    Safari: 17.4.1
  npmPackages:
    @babel/core: ^7.20.0 => 7.24.4
    @tamagui/animations-moti: ^1.94.3 => 1.94.3
    @tamagui/config: ^1.94.3 => 1.94.3
    @tamagui/themes: ^1.94.3 => 1.94.3
    expo: ~50.0.14 => 50.0.14
    expo-status-bar: ~1.11.1 => 1.11.1
    prettier: ^3.2.5 => 3.2.5
    react: 18.2.0 => 18.2.0
    react-native: 0.73.6 => 0.73.6
    react-native-reanimated: ~3.8.1 => 3.8.1
    tamagui: ^1.94.3 => 1.94.3

Thanks @TheEhsanSarshar, that works, but I feel like it's just a workaround for underlying issue 😁

yeah I'll check that, having animateOnly is a good practice for performance as well

ok this PR will fix this issue


five-bells-conditions -> js-crypto-conditions

Are we planing to rename form js-crypto-conditions  to crypto-conditions or  cryptoconditions ?


Keyboard.meta should be renamed to Keyboard.super

The documentation for Keyboard.meta states that
{-| The meta key is the Windows key on Windows and the Command key on Mac.
-}

That doesn't match the usage I've seen. In my experience, Meta has referred to "Alt" on Windows/Linux and "Option" on Mac, whereas the "Windows"/"Command" key has always been referred to as "Super".

See also: http://askubuntu.com/a/19565

In contrast, see https://developer.mozilla.org/en-US/docs/Web/API/KeyboardEvent/metaKey. This is how it is in Browser/JavaScript land. Elm is not going to change that. So the behavior and documentation of Keyboard.meta is exactly as web devs will expect. Keyboard.super would be an alien concept to them.

@jvoigtlaender is exactly right about why it is named that way. I think it makes sense to stick with the browser versions of the terminology.

0_o well there you go. TIL. Thanks for the explanation, @jvoigtlaender , and @evancz  for a cool language :)


Add JDK 21 to build in CI

Each LTS version should be tested, or at least the active ones :)

Are you referring to the name of the docker image? Currently, I'm (normally) already checking java 21 :

But I have to admit, I wrongly named my docker image and I was too lazy to change its name : https://github.com/kbuntrock/docker-mvn-jdk-8-11-17

Oh, my bad!
Indeed I've been confused by the docker image name :D


[#2886] Minikube on Docker, with custom cpu and memory

Defaulting for Docker when using Minikube.
Allow overrides for cpu, memory and driver when using Minikube.

lgtm, but small doc update would be great


Getting errors with Snell TCP server

Hello,
I'm getting,
[2023-09-07T17:18:52Z ERROR shoes::tcp_server] (MYIP):5448 finished with error: Custom { kind: InvalidData, error: "failed to setup server stream: open failed for length" }

error when starting snell server with

aes-256-gcm
chacha20-ietf-poly1305

ciphers.
Server works fine with aes-128-gcm cipher.
Please look into this. I'm looking forward for a solution.
Thank you.

what client are you using?

Sorry about delayed reply. I'm using Clash as a client. I tried with all clash cores (premium, meta, etc..), getting same error.


"Brănești" is listed in the drop-down menu

Description:
"Brănești" is listed in the drop-down menu alongside its corresponding county, according to the law
Precondition:
The website is up an running.

Step 1
Type "Brănești" in the search bar.

Expected results
The location is listed in the drop-down menu as "Comuna Brănești, GORJ".


fixed

По умолчанию:
scope_width =-1 - не сокращаем
scope_width = N- сокращаем от N символов
и накинула на эти комбинации тесты

Codecov Report
Patch coverage: 100.00% and project coverage change: +0.02 :tada:

Comparison is base (bf6cc18) 94.60% compared to head (c12cfb3) 94.63%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
Additional details and impacted files
@@            Coverage Diff             @@
##           master      #49      +/-   ##
==========================================
+ Coverage   94.60%   94.63%   +0.02%
==========================================
  Files         112      112
  Lines        2690     2703      +13
  Branches      266      268       +2
==========================================
+ Hits         2545     2558      +13
  Misses        126      126
  Partials       19       19

Impacted Files
Coverage Δ

vedro/plugins/director/rich/_rich_printer.py
91.01% <100.00%> (+0.63%)
:arrow_up:

vedro/plugins/director/rich/_rich_reporter.py
97.41% <100.00%> (+0.03%)
:arrow_up:

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

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

LGTM


Print partition ids when logging Put and Delete requests

Codecov Report

Merging #732 into master will decrease coverage by 0.08%.
The diff coverage is 100%.

@@             Coverage Diff              @@
##             master     #732      +/-   ##
============================================
- Coverage     65.73%   65.65%   -0.09%
+ Complexity     3811     3808       -3
============================================
  Files           330      330
  Lines         25240    25242       +2
  Branches       3159     3159
============================================
- Hits          16592    16573      -19
- Misses         7771     7787      +16
- Partials        877      882       +5

Impacted Files
Coverage Δ
Complexity Δ

...ain/java/com.github.ambry.protocol/PutRequest.java
73.22% <100%> (+0.21%)
14 <0> (ø)
:arrow_down:

.../java/com.github.ambry.protocol/DeleteRequest.java
93.1% <100%> (+0.12%)
18 <0> (ø)
:arrow_down:

...github.ambry.rest/AsyncRequestResponseHandler.java
88.52% <0%> (-2.3%)
23% <0%> (ø)

...b.ambry.network/BlockingChannelConnectionPool.java
64.03% <0%> (-2.2%)
8% <0%> (ø)

.../src/main/java/com.github.ambry.store/Journal.java
89.65% <0%> (-1.73%)
19% <0%> (-1%)

...in/java/com.github.ambry.store/BlobStoreStats.java
85.85% <0%> (-1.44%)
93% <0%> (-1%)

...java/com.github.ambry.store/CompactionManager.java
91.04% <0%> (-0.75%)
17% <0%> (ø)

...in/java/com.github.ambry.network/SocketServer.java
82.78% <0%> (-0.41%)
13% <0%> (ø)

Continue to review full report at Codecov.

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


[NEW MODULE] Tadukoo File Format

Describe the new module you're proposing
Create a module for a generic file format that can be used for various kinds of data.
Why should we add this new module?
It'll be helpful for generic data storage.
Describe alternatives you've considered
None
Additional context
Tadukoo File Format was technically started a while back and is already partially released, this issue is mainly just about finishing it.
Changes TODO

[x] Improve classes and write JUnit tests for them

[x] Node

[x] Make a builder for Node

[x] FormatNode
[x] FormatNodeVerification
[x] FileFormatSchema
[x] FileFormat
[x] FileFormatSchemaVerification
[x] TadFormatNodeHeader

[ ] Write a class/interface to convert between MappedPojos and Nodes
[x] Remove GHDR stuff (was a FileFormat for a private project that I was using for JUnit testing in the past)

Writing a generic conversion between Nodes and MappedPojo is something that could get complex, as the idea with Tadukoo File Format is to be able to have the data in whatever form you like (so long as it obeys the rules for the Nodes), so for now it will not be done. In the future, that may change, but right now it seems too complex for a generic solution


[Feature] Added Standard Deviation Calculator

This PR adds Standard Deviation Calculator to the repository.
Workflow :
I have added the function that will calculate the standard deviation of the given population using the following formula :

This PR closes #271
Screenshots/Video :
demo22.webm
Would be glad if you'd merge this PR. Thanks @Srijita-Mandal 😄 👍

@pooranjoyb Please add placeholder in the input field how the samples should be entered.

Fixed bugs @Srijita-Mandal  😃

Please label the PR @Srijita-Mandal


How to create shortcut for the specific characters which can't be input from keyboard?

Hi,
I would like to create shortcuts for arrow symbol (e.g. ← ↑ → ↓), but those characters can't be input from keyboard. How can I create shortcuts for them?
Thanks.

Please use Unicode Hex Input Hack.
https://pqrs.org/osx/karabiner/faq.html.en#unicode-hex-input

Awesome!!
Thank you so much and have a nice day :smile:

Hello, I have the exact same question, and the solution seems to not apply any longer. What's the correct way to do this now?


Adds ProviderID

Signed-off-by: Chuck Ha<EMAIL_ADDRESS>What this PR does / why we need it:
This PR adds a ProviderID as is required by ClusterAPI v1alpha2.
Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Related to #127
Special notes for your reviewer:
NONE

/assign @ncdc

/lgtm

/test pull-cluster-api-provider-docker-verify

/lgtm


test: improve test coverage

Add a few error state tests to improve test coverage on config/rotate

path_config_rotate.go

master: 77.1% (overall 62.4%)
this branch: 88.6% (overall 65%)

We have all the "positive" test cases covered, but I am not sure how to create some of the errors. If you have ideas, let me know. I am going to tackle /config/admin next.

path_config.go

master: 68.1% (62.4% overall)
this branch: 91.2% (68.2% overall)

Learned something...
We don't need this case statement to check for bool, because no matter what we submit, it comes out as a string? although this feels like a bug to me :)

Most likely because the attribute is defined as TypeString (https://github.com/jfrog/artifactory-secrets-plugin/blob/master/path_config.go#L33) 😄
Instead we should use TypeBool.

Learned something...
We don't need this case statement to check for bool, because no matter what we submit, it comes out as a string? although this feels like a bug to me :)

Most likely because the attribute is defined as TypeString (https://github.com/jfrog/artifactory-secrets-plugin/blob/master/path_config.go#L33) 😄
Instead we should use TypeBool.

Created a new issue for this.


Unbreak Circle CI, and other fixes

Fix the Circle CI:

Update Fedora kernel URL
Use recent Fedora with recent QEMU
Few fixes to avoid the script looping for hours on CI

Due to Tianocore BZ#1747 change in EnrollDefaultKeys,
generate a temporary X509 certificate to run the test.

I think your changes look good to me.  Thanks, Phil.  Now that Laszlo's patch is merged; we can take it take this a bit less hurried :-)  It looks good to me, I'll let Laszlo / Patrick also have a further look at this.

In patch Generate temporary X509 certificate to be enrolled as Platform Key, passing the --oem-string ... option should not be necessary in the run testing-only case (i.e. when --skip-enrollment is passed). Namely, if we don't enroll, then we don't have to pass down the certificate to enroll.
But that's a minor wart only; I'm happy with this series going in. If Phil chooses to drop the one --oem-string ... instance in question, then my R-b can be carried forward. For the series ending at commit 8052109:

Thanks for the review, I did the requested change (Kashyap approving and yours got reset).


Switch from react-simple-dnd to react-dnd.

- Summary
react-simple-dnd is just a wrapper for react-dnd, and it doesn't support React 16. This PR changes to using the underlying library directly.
- Test plan
Manually tested unpublished entries drag-and-drop cards.
- Description for the changelog
Switch from react-simple-dnd to react-dnd.
- A picture of a cute animal (not mandatory but encouraged)

Codecov Report

Merging #705 into master will increase coverage by 0.13%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #705      +/-   ##
==========================================
+ Coverage   16.75%   16.88%   +0.13%
==========================================
  Files         146      145       -1
  Lines        2650     2629      -21
==========================================
  Hits          444      444
+ Misses       2206     2185      -21

Impacted Files
Coverage Δ

src/index.js
0% <0%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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

Going to go ahead and merge since this is mostly a port and fairly straightforward (and blocking the React 16 update).


[runbook-sync-downstream] Close outdated PRs

What this PR does / why we need it:
When new changes are added to the runbooks, a new PR is created upstream updating all diffs between runbooks. But an old PR that has not been merged yet would remain open with only a portion of the diffs. This change makes the tool close those outdated PRs.
Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #
Special notes for your reviewer:
Checklist
This checklist is not enforcing, but it's a reminder of items that could be relevant to every PR.
Approvers are expected to review this list.

[ ] Design: A design document was considered and is present (link) or not required
[ ] PR: The PR description is expressive enough and will help future contributors
[ ] Code: Write code that humans can understand and Keep it simple
[ ] Refactor: You have left the code cleaner than you found it (Boy Scout Rule)
[ ] Upgrade: Impact of this change on upgrade flows was considered and addressed if required
[ ] Testing: New code requires new unit tests. New features and bug fixes require at least on e2e test
[ ] Documentation: A user-guide update was considered and is present (link) or not required. You want a user-guide update if it's a user facing feature / API change.
[ ] Community: Announcement to kubevirt-dev was considered

Release note:

none

Why is the existing PR not updated with the new changes?

Why is the existing PR not updated with the new changes?

We would have to easy way in the next runs to know if those changes are already applied or not.
Currently we track the branch dates to know which changes are already proposed downstream, if we updated old PRs, we would need to come up with a more complicate mechanism to iterate all the PRs (open and closed) and check each commit to see if new changes are present there


Fix a little bit of "Por Grammer"

Typo fix to lib.rs documentation (This showed up on the first page of the documentation.)

Thank you.


Write-RsCatalogItem.ps1 suppresses $warnings, even in -Verbose mode

$warnings is assigned via a [ref] but is not output, even in -Verbose mode.
As I explain in my pull request, this appears to be a much bigger problem than I first thought, as apparently the 2016 Reporting Services endpoint I am testing against silently fails to actually create the report.

Thank you very much for submitting the fix for this issue! I am going to close it now.


Finemap runner

WIP

Probably also going to be really useful to add this while you're debugging :smile:
cat $temp_file


[add] man pages

TODO: add more info in man pages

I think I'll merge this along with scdoc and build.rs once I get that done.

@Idlidev https://github.com/waycrate/swhkd/pull/54#issuecomment-1054151037 can you add signal handler data once this pr is merged?


i ran it multiple times and it made my mouse drifty

i literally just ran it and my mouse started moving around the screen
windows 10 2004

If you meant to say that the mouse started lagging, then that might be because it currently writes every input event to the database immediately.
The next version will handle database writes in bigger batches.

If you meant to say that the mouse started lagging, then that might be because it currently writes every input event to the database immediately.
The next version will handle database writes in bigger batches.

alright! just making sure you knew.

alright! just making sure you knew.

Reopening the issue, so that I can verify that the problem has been solved once the next version is released.

Reopening the issue, so that I can verify that the problem has been solved once the next version is released.

Database writes should no longer cause notable lag in the new v1.3 release.
Can you confirm?

Database writes should no longer cause notable lag in the new v1.3 release.
Can you confirm?

it works!
On Sun, Jan 31, 2021 at 12:27 PM Erki Suurjaak<EMAIL_ADDRESS>wrote:

Database writes should no longer cause notable lag in the new v1.3 release.
Can you confirm?
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/suurjaak/InputScope/issues/6#issuecomment-770417636,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AE4IUAM7V3ZOK37XAXIGVRDS4WHHLANCNFSM4VTQHK7A
.

it works

fart


[AutoPR eventhub/resource-manager] Event Hub: Added missing API to respective API version Swaggers

Created to sync https://github.com/Azure/azure-rest-api-specs/pull/2457

Codecov Report

Merging #1944 into restapi_auto_eventhub/resource-manager will decrease coverage by <.01%.
The diff coverage is 53.39%.

@@                            Coverage Diff                             @@
##           restapi_auto_eventhub/resource-manager    #1944      +/-   ##
==========================================================================
- Coverage                                   53.15%   53.15%   -0.01%
==========================================================================
  Files                                        4724     4730       +6
  Lines                                      117181   117283     +102
==========================================================================
+ Hits                                        62293    62347      +54
- Misses                                      54888    54936      +48

Impacted Files
Coverage Δ

...venthub/azure/mgmt/eventhub/operations/__init__.py
100% <100%> (ø)
:arrow_up:

...azure/mgmt/eventhub/event_hub_management_client.py
100% <100%> (ø)
:arrow_up:

azure-mgmt-eventhub/azure/mgmt/eventhub/version.py
100% <100%> (ø)
:arrow_up:

...mt-eventhub/azure/mgmt/eventhub/models/__init__.py
100% <100%> (ø)
:arrow_up:

...ure/mgmt/eventhub/operations/regions_operations.py
37.5% <37.5%> (ø)

...t/eventhub/operations/messaging_plan_operations.py
43.33% <43.33%> (ø)

...nthub/azure/mgmt/eventhub/models/messaging_plan.py
50% <50%> (ø)

...mt/eventhub/models/messaging_regions_properties.py
62.5% <62.5%> (ø)

...ub/azure/mgmt/eventhub/models/messaging_regions.py
71.42% <71.42%> (ø)

...re/mgmt/eventhub/models/messaging_regions_paged.py
80% <80%> (ø)

... and 6 more

Continue to review full report at Codecov.

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

Broken PR


Add a length seed_type

If an array, its length. If it a string, it's length. If an object, its number of properties

This is unnecessary because property works as expected. Thanks, javascript!


Why is it that material-ui calls their dialogs twice?

[ ] The issue is present in the latest release.
[x] I have searched the issues of this repository and believe that this is not a duplicate.

Current Behavior 😯
I tried to set a modal first and I noticed that behaviour, then I changed it for a dialog and it is the same.
I tried this:
import Dialog, { DialogProps } from '@material-ui/core/Dialog';
import { useStores } from './stores';
import { useObserver } from 'mobx-react';

export const MyDialog: React.FC<Partial<DialogProps>> = (props) => useObserver(() => {
  const { uiStore: { dialogConfig: { open, body } } } = useStores();

  if (!body) { return null; }

  return (
    <Dialog {...{ ...props, open, maxWidth: false }}>
      { body }
    </Dialog>
  );
});

I call it in the app.tsx: <MyDialog />
And when I call the function to show the dialog, it shows twice in the DOM, what makes me curious is that one of them has the aria-hidden=true attribute.

But they have the exact same content inside if I inspect them.
I noticed something similar in the docs page https://material-ui.com/components/dialogs/, if you inspect the DOM, there is always one dialog but the difference is that it has visibility: hidden.
Any ideas about this?

Expected Behavior 🤔
Not showing repeated componentes in the DOM even if they are not present in the UI.
Steps to Reproduce 🕹
The issue is present in your docs: https://material-ui.com/components/dialogs/
Steps:

Open https://material-ui.com/components/dialogs/
Open the devtools Inspect the DOM
Check the first div/component after the list of scripts
Voilá

Context 🔦
I am trying to avoid not having duplicate components I don't need in the DOM.
Your Environment 🌎
  System:
    OS: macOS 10.15.6
  Binaries:
    Node: 12.13.0 - /usr/local/bin/node
    Yarn: 1.22.4 - /usr/local/bin/yarn
    npm: 6.14.8 - ~/Work/mira-editor-plugins/node_modules/.bin/npm
  Browsers:
    Chrome: 88.0.4324.96
    Edge: 88.0.705.53
    Firefox: Not Found
    Safari: 13.1.2
  npmPackages:
    @emotion/styled: ^10.0.27 => 10.0.27
    @material-ui/core: ^4.11.0 => 4.11.0
    @material-ui/icons: ^4.9.1 => 4.9.1
    @material-ui/lab: ^4.0.0-alpha.56 => 4.0.0-alpha.56
    @material-ui/styles:  4.10.0
    @material-ui/system:  4.9.14
    @material-ui/types:  5.1.0
    @material-ui/utils:  4.10.2
    @types/react: ^16.9.49 => 16.9.49
    react: ^16.13.1 => 16.13.1
    react-dom: ^16.13.1 => 16.13.1
    typescript: ^3.9.7 => 3.9.7

These two demos have keepMounted, the behavior is correct

These two demos have keepMounted, the behavior is correct

Ok good to know that the behaviour is correct.
This causes accessibility problems for our implementation of the MUI modals though, because the duplicate code causes <label> elements NOT to send focus to the <input>'s for which they have for attribute matching to the <input>s id attribute, because the duplicate modal HTML means that the id fields are in fact NOT unique. If I delete one of the duplicate modal HTML root elements, then focusing of the inputs works correctly / as expected.

i got the same problem, a workaround i came up with was to set the visibility of the second occurrence to hidden
.css-79ws1d-MuiModal-root:nth-of-type(2) {   visibility: hidden !important; }


Issues with importing Vuelidate Library

Node version: v14.17.0
Sails version (sails): 1.3.1
ORM hook version (sails-hook-orm): 2.1.1
Sockets hook version (sails-hook-sockets): 2.0.0
Organics hook version (sails-hook-organics): 2.2.0
Grunt hook version (sails-hook-grunt): 4.0.1
DB adapter & version (e.g. sails-mysql@5.55.5<EMAIL_ADDRESS>
I want to use Vuelidate for Field Validation for a form on a Sails.js page. But, I am having issues with importing Vuelidate library using import/require().
You can get more info in this Link to my Question. Does this question require more detail or different labels for better categorization/visibility?

Hi @shreyas-mallya, The best way for you to add Vuelidate would be downloading and adding vuelidate.min.js to assets/dependencies and using it with the Vuelidate global

Tried it out, worked like a charm...


ENG-7133 add EC2 endpoint

Description

Adds the EC2 endpoint /2009-04-04.
Why is this needed

There has been an effort to have Hegel succeed Kant, and one of the endpoints that Kant has (and Hegel doesn't) is the /2009-04-04 endpoint.
Fixes: #
How Has This Been Tested?

Integration tests
Manual test with tinkerbell setup

How are existing users impacted? What migration steps/scripts do we need?

No migration steps needed.
Checklist:
I have:

[ ] updated the documentation and/or roadmap (if required)
[x] added unit or e2e tests
[ ] provided instructions on how to upgrade

@DavidHwu you mentioned:
DavidHwu 5 days ago Member

IMO I am use to constructing the path dynamically and do not find that hard to read.
I see this as a fine/easier way to service and have the flexibility.

I'm not a fan of too much dynamic stuff and the string/map[string]interface{}` nature of the filter code wasn't sitting right with me. @kdeng3849 and I DM'd and I think we landed on something that will cut down on a lot of the complexity from this PR. A new PR will be opened so we can have a look at it separately.

this can be closed right @kdeng3849 ?

this can be closed right @kdeng3849 ?

Yeah


Support for ExpressRoute Gateway sku type

Is there an existing issue for this?

[X] I have searched the existing issues

Description
Hi, I'd like to setup erg with this module but noticed that the sku types won't pass the validation. I've proposed a PR #14 to address this.
New or Affected Resource(s)/Data Source(s)
azurerm_virtual_network_gateway
Potential Terraform Configuration
Included in the PR #14

variable "sku" {
  type        = string
  description = "The SKU (size) of the Virtual Network Gateway."

  validation {
    condition     = contains(["Basic", "HighPerformance", "Standard", "UltraPerformance", "VpnGw1", "VpnGw2", "VpnGw3", "VpnGw4", "VpnGw5", "VpnGw1AZ", "VpnGw2AZ", "VpnGw3AZ", "VpnGw4AZ", "VpnGw5AZ", "ErGw3AZ"], var.sku)
    error_message = "sku possible values are Basic, HighPerformance, Standard, UltraPerformance, VpnGw1, VpnGw2, VpnGw3, VpnGw4, VpnGw5, VpnGw1AZ, VpnGw2AZ, VpnGw3AZ, VpnGw4AZ, VpnGw5AZ, ErGw3AZ."
  }
}

References

No response

Thanks for raising this and the subsequent PR, I'll look to see this merged.


selecting element by shortcut key

there needs to be an option to start the element selection process with customizable keyboard shortcut.
I have a page that utilized  :active  listeners to change css on the fly. I can never select the element I want because:

I click on icon (onclick event) which initialized the js behind the scenes to changes the css to display a text field which is normally hidden. This is the field I want to edit with stylebot.

I click on "click to enable selection of element" button from stylebot window... which is also onclick event and triggers js behind the scenes.

the js behind scenes acts as a toggle. if the field i'm trying to work with is hidden, it will be displayed. If it is displayed, it will be hidden. It is driven off of onclick events. so there is no possible way for me to select this element with your "click to enable selection of element" button.

Please fix.

there is a shortcut already for this, the s key


Unnecessary statement setImmediate require

While doing a react-native project I realized that will not compile because it was requiring a native module, which is timers. Maybe I am wrong but these are suppose to be global variables and thus, that statement is unnecessary and reduces compatibility across platforms.
Moreover sax uses native streams modules, so it requires a little further transformation to be 100% compatible.
Thanks for your work :)

Please do not submit patches to the generated JS source, they will be overwritten by the next run of the compiler.

Oh sorry, did not realize.


module declares its path as urfave/cli  but was required as codegangsta/cli

Need to update the files from codegangsta/cli to urfave/cli.
Need to update the go.mod file:
Also update imports on go.mod , utils\flags.go
I also ran:
go get -u github.com/urfave/cli

I updated the Godeps.json file:
		{
			"ImportPath": "github.com/urfave/cli",
			"Comment": "v1.22.5",
			"Rev": "af7fa3de115a4f9772a7758c51f92a1e3b30a010"
		},

It fixed the issue
When file a bug report (see below for feature requests)
Please answer these quesions for a bug report. Thanks!
What version of GoTTY are you using (gotty --version)?
What operating system and browser are you using?
What did you do?
If possible, please provide the command you ran.
What did you expect to see?
What did you see instead?
If possible, please provide the output of the command and your browser's console output.
root@ubuntu:~/go/src/gotty# go get -u github.com/codegangsta/cli
go: downloading github.com/codegangsta/cli v1.22.5
go get<EMAIL_ADDRESS>updating to
<EMAIL_ADDRESS>parsing go.mod:
        module declares its path as: github.com/urfave/cli
                but was required as: github.com/codegangsta/cli

I updated the go.mod file and tried to build but received:
root@ubuntu:~/go/src/gotty# go build main.go
go: inconsistent vendoring in /home/jiren/go/src/gotty:
<EMAIL_ADDRESS>is explicitly required in go.mod, but not marked as explicit in vendor/modules.txt
<EMAIL_ADDRESS>is marked as explicit in vendor/modules.txt, but not explicitly required in go.mod

        To ignore the vendor directory, use -mod=readonly or -mod=mod.
        To sync the vendor directory, run:
                go mod vendor

When file a new feature proposal
Please provide an actual usecase that requires your new feature.

I fixed it in my repository:
https://github.com/g3rzi/gotty2/commit/2487168a67080a2515712c300fc0fe679ecf7876


formatting .Digest in docker images results in  unless --digests is passed on the command line

The documentation specifies that .Digest is a formattable field in docker images output, but this is only true if the --digests flag is passed.
Steps to reproduce the issue:
$ docker images --digests postgres
REPOSITORY          TAG                 DIGEST                                                                    IMAGE ID            CREATED             SIZE
postgres            latest              sha256:6ac0fbeddde3bb7b0003a2ace8c126f742f8bdd90695801337d3edaaf1fcc478   4e18b2c30f8d        3 weeks ago         266 MB
$ docker images --format '{{ .Tag }} {{ .Digest }} '  postgres
latest <none>
$ docker images --digests --format '{{ .Tag }} {{ .Digest }} '  postgres
latest sha256:6ac0fbeddde3bb7b0003a2ace8c126f742f8bdd90695801337d3edaaf1fcc478

Output of docker version:
$ docker version
Client:
 Version:      17.03.0-ce
 API version:  1.26
 Go version:   go1.7.5
 Git commit:   60ccb22
 Built:        Thu Feb 23 10:40:59 2017
 OS/Arch:      darwin/amd64

Server:
 Version:      17.03.0-ce
 API version:  1.26 (minimum version 1.12)
 Go version:   go1.7.5
 Git commit:   3a232c8
 Built:        Tue Feb 28 07:52:04 2017
 OS/Arch:      linux/amd64
 Experimental: true

Output of docker info:
Containers: 4
 Running: 0
 Paused: 0
 Stopped: 4
Images: 44
Server Version: 17.03.0-ce
Storage Driver: overlay2
 Backing Filesystem: extfs
 Supports d_type: true
 Native Overlay Diff: true
Logging Driver: json-file
Cgroup Driver: cgroupfs
Plugins:
 Volume: local
 Network: bridge host ipvlan macvlan null overlay
Swarm: inactive
Runtimes: runc
Default Runtime: runc
Init Binary: docker-init
containerd version: 977c511eda0925a723debdc94d09459af49d082a
runc version: a01dafd48bc1c7cc12bdb01206f9fea7dd6feb70
init version: 949e6fa
Security Options:
 seccomp
  Profile: default
Kernel Version: 4.9.12-moby
Operating System: Alpine Linux v3.5
OSType: linux
Architecture: x86_64
CPUs: 4
Total Memory: 1.952 GiB
Name: moby
ID: IKSC:KVBD:PQ2P:YTHR:KFSK:I5M2:CIQE:XE3J:AQS5:BW45:OCXX:F5S7
Docker Root Dir: /var/lib/docker
Debug Mode (client): false
Debug Mode (server): true
 File Descriptors: 16
 Goroutines: 27
 System Time: 2017-03-27T19:47:43.186941816Z
 EventsListeners: 1
No Proxy: *.local, 169.254/16
Registry: https://index.docker.io/v1/
Experimental: true
Insecure Registries:
 <IP_ADDRESS>/8
Live Restore Enabled: false

This environment is Docker for Mac, but the behavior appears consistent on a few Linux hosts I tried on as well.

@thaJeztah is it a bug or a problem in the docs? I believe is a missing detail in the documents.

I had a fix for this, as part of another pull request, but should dig it up, because I didn't have time to work on it at the time.

Do you have time to work on it or you don't mind if I try my luck? 👼

@thaJeztah OK caught it. Do you want to submit a PR or you don't mind I will?
Thanks.

@thaJeztah what I did to fix this is changing these lines to:
if !ctx.Digest && len(digests) > 0 {
    digests = digests[:1]
}


Links should be open in new tab

It can be a better user experience to open links in new tabs when clicked such as issue title, repository, view button, and issue creator.

This is now released, thanks for making the PR! 😄


What is this libopus-0.x64.dll ? Can you share the SOURCE CODE of this ?

libopus-0.x64.dll ?
What is this exactly why obfuscate this DLL
What there to hide..
Or Do you guys Receive all our hits. ? So if not why not include this DLL source.
I want to see what inside the .DLL
I ain't running or setting this unless I see, INSPECT , that libopus-0.x64.dll file.
Care you explain me me any ADMIN .,

@whois-hoeless
@mategol @ZorroMundoYT  @neek8044  @AWeirDKiD @HorridModz
I need answers guys , Why did you guys obfuscate this DLL file ?
What's inside ?
If there is nothing wrong , please upload the source of the dll also.
libopus-0.x64.dll  = hitlogger  or what
Or do you infect us too with the malware or when we get all the  victim stuff, you guys also gets the same.
Don't obfuscate DLL lol

The what now?
It's an open source library, it's used by many big platforms like Discord or Skype, you can search it up on the internet.

What is this exactly why obfuscate this DLL. I want to see what inside the .DLL

First of all, no one obfuscated the file. A dll is not readable, it contains machine code and stuff. Try to open any dll from your C:/Windows folder, same thing will happen.
Secondly, as Zorro said, it is an open source library and you can search it up.

Or do you infect us too with the malware or when we get all the victim stuff, you guys also gets the same.

Unless you run the file which you have to compile in the builder (it will be under the dist folder), you won't get infected by any chance.
You can easily launch a virtual machine and do everything in there if you still don't trust the dll and want to feel safer.

@whois-hoeless
@mategol @ZorroMundoYT  @neek8044  @AWeirDKiD @HorridModz
I need answers guys , Why did you guys obfuscate this DLL file ?
What's inside ?
If there is nothing wrong , please upload the source of the dll also.
libopus-0.x64.dll  = hitlogger  or what
Or do you infect us too with the malware or when we get all the  victim stuff, you guys also gets the same.
Don't obfuscate DLL lol

@ArabCrew786 everything neek and zorro said is correct. Also if you want to verify that our one is not a malicious payload which is injected into some process then go ahead and compare the checksum using certutil or try to decompile it and read the asm ( since you know so much about obfuscation you probably are able to understand asm ).
In the future please refrain from accusing us from those kind of things if you didn't do any research. Just because you see a dll somewhere doesn't mean it will hack you like your cod cheats..


TRE Cli missing dependencies

Resolves #3446
What is being addressed
This PR contains changes required for release - resolving issues on the deployment repo .

/test

/test


Python GA code generation

The changes here look mostly good.  My one issue is more with the GA API.  GenericRpcHandler feels like cruft that could be replaced by a map from /servicename/method to an RpcMethodHandler.
You could argue that GenericRpcHandler adds the ability to use the metadata and some other special logic to route a call to a specific method, but I think that logic belongs in the method itself.  This might be outside the scope of the PR, but I just wanted to give my 2 cents.

@soltanmm, once this change goes in, you should be unblocked on #6749 for the GA API.

The use in this changelist were the only foreseen kind of use for GenericRpcHandler, I'd agree. The reason GenericRpcHandler has the shape that it has is so that applications can handle methods about which they don't know in advance - a map from /servicename/method to an RpcMethodHandler has all the RpcMethodHandlers it will ever have, but a GenericRpcHandler can create new RpcMethodHandlers when called to service an RPC. Proxies are the most obvious use case for generic handling, but I'm pretty sure there are others.

Back over to you, @kpayson64.

LGTM on tests.


Need help reviewing and packaging : Tamazight Large Pack

Hi @menny
I need reviewing and packaging, if you have time.

Link to repo : https://github.com/BoFFire/LanguagePack
Link to F-Droid : https://gitlab.com/BoFFire/fdroiddata/blob/master/metadata/com.anysoftkeyboard.languagepack.tamazight.txt

Thank you :)

I see a few issues. I want to suggest that you create a PR, to merge this back into AnySoftKeyboard/LanguagePack and I can comment there. Sounds good?
What is the name of the pack? I see you have a few languages packed together

PR open : https://github.com/AnySoftKeyboard/LanguagePack/pull/133

Next version, Arabic dictionary will be based on the excellent work of @linuxscout Ayaspell
https://github.com/linuxscout/ayaspell/tree/master/dict

My ASK "ar" localizations are here : https://crowdin.com/project/anysoftkeyboard/activity_stream

thanks.
I'm closing this issue. We'll continue the discussion on the PR page.


dask-gateway: bump from 2023.9.0 to 2024.1.0

This makes latest tag not come with IfNotPreset pull policies, which easily could cause issues if a latest tag was used. That in turn is something I want before suggesting use of latest tag in https://2i2c.freshdesk.com/a/tickets/794 as a compromise.

Thank you @sgibson91!!


Handle the unsigned shift operator in DDC

This is the DDC issue for #30886.
This is a non-breaking change and is not required to be in Dart 2.0.
Probably blocked by #30889 .

I believe this will be mostly handled by the CFE and core libraries, so I mostly expect this issue to be validation that it all works properly.

@nshahan is this addressed by https://github.com/dart-lang/sdk/commit/b250a58d8e81e4b58c42e83523808fceeb33e7e9?

Yes the implementation is in and this seems more or less done. I'll validate with the test suites and file bugs if there are any failures.


Share replicator connections by proxy

Prior to these changes, proxied and unproxied replication connections would be shared based on their source or target. This caused a bug where a proxied connection could function as an unproxied connection if an idle unproxied connection was shared with it.

+1 Nice job.
Tested with mitmproxy

Saw connection pool ets table has connections to the proxy (localhost:8080) as expected.
_scheduler/docs shows new proxy field:
{
            "database": "_replicator",
            "doc_id": "cdyno-0000002-0000017",
            "error_count": 0,
            "id": "e44a2caf271fb2c21d5da3ed7658ac42+continuous",
            "info": null,
            "last_updated": "2016-11-23T20-23-52Z",
            "node"<EMAIL_ADDRESS>            "proxy": "http://localhost:8080",
            "source": "http://adm:*****@localhost:15984/cdyno-0000002/",
            "start_time": "2016-11-23T20-23-52Z",
            "state": "running",
            "target": "http://adm:*****@localhost:15984/cdyno-0000017/"
}

That was the rebase onto 63012-scheduler.


Can tasks be created solely based on the author's original name instead of their ID?

Sometimes, if fetching the username fails, it defaults to using the ID for naming.
which might lead to redundant downloads if you're using "ktoolbox sync-creator" to retry downloading previously failed files, as the naming based on ID might differ.
It's hoped to include a feature to retry reading the author's name when it fails, instead of using the ID for naming.

By default, all Kemono APIs will retry up to 3 times. You can modify this by setting KTOOLBOX_API__RETRY_TIMES.
# .env.prod
KTOOLBOX_API__RETRY_TIMES=10


Nesting of django calls in Trace Details is incorrect

Description
Trace details are nested for django calls but they shouldn't be
Expected Behavior
The trace should be linear/non-nested so it can be viewed and used as expected
Steps to Reproduce

Connect newrelic to django app
Upload data from web requests
View that Trace Details are nested when they shouldn't be

Your Environment
django 3.2.16 running with python 3.9.16 on Debian Bullseye

@daveisfera Do you have any images to help us further understand this issue, or documentation you are referencing on the expectations?

Here's a screenshot. You can see the middleware calls that are "siblings" and not nested, so it's incorrectly attributing the time to the "nested functions" when they're actually running really fast and not the issue

This is still an issue


Roll Flutter Engine from d13479968b8f to 8fb248479af0 (1 revision)

https://github.com/flutter/engine/compare/d13479968b8f...8fb248479af0
2024-01-09<EMAIL_ADDRESS>Roll Skia from f4626a2e3293 to e3af4a11007d (1 revision) (flutter/engine#49613)
If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Please CC rmistry<EMAIL_ADDRESS>on the revert to ensure that a human
is aware of the problem.
To file a bug in Flutter: https://github.com/flutter/flutter/issues/new/choose
To report a problem with the AutoRoller itself, please file a bug:
https://issues.skia.org/issues/new?component=1389291&template=1850622
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md

Commit queue failed; closing this roll.


merge package from arlac77/npm-package-template

package.json

chore(scripts): cover@#overwrite c8 --temp-directory build/coverage ava && c8 report -r lcov --temp-directory build/coverage
chore(package): add nyc from template
chore(package): set $.ava.require='esm' as in template
chore(package): set $.ava.files='tests/-test.js,tests/-test.mjs' as in template
chore(package): set $.ava.extensions='js,mjs' as in template

LICENSE

chore(license): add current year 2019

Coverage remained the same at 83.951% when pulling 5c269d86937920fd8404ef41344a89a28db9d8c5 on template-sync-1 into 8b156585624a2c2a973e88dcba9c975cb162c1f5 on master.


Fix 2D debug navigation flickering with tile maps

Both the tile map layers and the debug navigation canvas items did fight for the same z order causing a lot of flickering in certain situations.
Before the debug rendering was changed to server use in https://github.com/godotengine/godot/pull/92372 this was kinda hidden due to node order. This change now sets the z index one less the navigation agent debug paths index so the paths can render on top of the navigation mesh debug.

Thanks!

Cherry-picked for 4.3.1.


Support for *UrlParams

It is currently not possible to have custom *UrlParams because merge functions transform "conflicting arrays" into objects which have no "forEach" support. It causes an exception in Oauth2Service.buildQueryString() when trying to iterate over this.defaults[params]

Can you give an example input? and where exactly it fails?

I have used "defaultUrlParams" instead of "optionalUrlParams"

So is there still an issue?

Bug occurs on "defaultUrlParams". I update the title but let you decide if not customizing "defaultUrlParams" is bug or not.

removed ability to change defaultUrlParams in version 7.0.1


Extract Non-Racing Specific Code

Extract non-racing specific code to an external package so that it can be reused by other projects.
A quick glance at the code base suggests that the following modules should be extracted:

events
profiling
threaded_queues

It's possible that the Entity base class should also be extracted, although this deserves further consideration.

It has been decided that the Entity base class should remain part of pyracing rather than extracted... for now! It is possible that in future, the common code in Entity could be extracted while the small amount of racing-specific code could be moved to a racing-specific sub-class of the extracted Entity class.


Feedback/contact us form

Is there an existing issue for this?

[X] I have searched the existing issues

Feature Description
I would like to recommend above the social link section their should be a contact us/ feedback form which allows user to have direct interaction with the admin gaining confidence on the site. @ashutosh-rath02 sir i would love to do so, i would suggest to have a minimal but professional pop up form.
Use Case

User- interaction

Benefits
No response
Add ScreenShots
No response
Priority
Medium
Record

[X] I have read the Contributing Guidelines
[X] I'm a GSSOC'24 contributor
[X] I have starred the repository

Thank you for creating this issue! 🎉 We'll look into it as soon as possible. In the meantime, please make sure to provide all the necessary details and context. If you have any questions or additional information, feel free to add them here. Your contributions are highly appreciated! 😊
You can also check our CONTRIBUTING.md for guidelines on contributing to this project.

@ashutosh-rath02 yes sure sir, will be waiting for your response.

Duplicate issue

Hello @ananyag309! Your issue #249 has been closed. Thank you for your contribution!


Routes in Modules..?

Can / should we have module specific routes?    Routes that potentially come with a module.
Really like this new modules approach. great job :+1:

I was in for that idea initially. So just to be clear, nothing stops you from putting routes inside the module, but if you consider that your module might be redistributed between different apps than you will most likely look look to define routes in application wide config file.

Can i load my routes as part of an
app.loadModule(someModule);

Here is what my client/main.js file looks like
import {initContext} from '/imports/configs/context';
import routes from '/imports/configs/routes.jsx';
import {createApp} from '/imports/libs/mantra';

import coreModule from '/imports/modules/core';
import commentsModule from '/imports/modules/comments';

import _homeModule from '/imports/modules/_home';
import _homeRoutes from '/imports/modules/_home/configs/routes.jsx';

import _usersModule from '/imports/modules/_users';
import _usersRoutes from '/imports/modules/_users/configs/routes.jsx';

import _colorsModule from '/imports/modules/_colors';
import _colorsRoutes from '/imports/modules/_colors/configs/routes.jsx';

const context = initContext();

const app = createApp(context);
app.loadModule(coreModule);
app.loadModule(commentsModule);

app.loadModule(_homeModule);
app.loadModule(_usersModule);
app.loadModule(_colorsModule);

app.loadRoutes(routes);
app.loadRoutes(_homeRoutes);
app.loadRoutes(_usersRoutes);
app.loadRoutes(_colorsRoutes);

Ok.. thanks !

I like to place routes into a common place rather loading them from modules. I'm not disagreeing with it.
Then it's easy to visualize the whole app at a single place and modules does all the UI/action stuff.
Can I get some counter arguments and you reason on putting routes next to the modules?

My thinking is around the CMS side of things.  if i can add a module that represents "Events" and it will come with the backend routes to create / manage / update / delete an Event item.
I'm playing with this concept of a modular CMS build-pack.  Where a module can represent:

widgets that can be used to craft the UI experience
admin side features (including specific module admin routes)

a common place for routes then works for the main app.
I think for this to work.. the 'admin' routes would need to load up in the Module main.js file. And come loaded with the app.loadModule() call.
thoughts?

I don't mean to bring this back to life if it was finished, but personally I think separating the routes by their modules makes the most sense and means the modules are easier to maintain. From a CMS perspective, or any project in my mind, each module should be self contained. A "Users" module for example: EVERYTHING related to the users is put inside it, including the routes used for accessing them.
In the current setup you could have the users module, make a change, and need to go to the "Core" to update the routes associated to your change, but if you are using that same module in multiple projects then you have to do it for each individual project. This means that I have 3 apps all using the "Users" module I created, I have to open all 3, modify the core of all 3, before I can run my E2E and other tests, finally leading to push the update to staging/production. By putting the routes inside the module it cuts out the copy & pasting someone needs to do to use the module in different projects. It also makes it fully self contained, meaning you don't need to go outside of the module to update any aspect of it, meaning it's easier to maintain and expand.
As a general design rule I seperate everything down to core modules. I did this previously with packages in Meteor, but with the ability to use imports in 1.3, I will likely move away from that. This limits the amount of time spent hunting for where the component is that is having an issue. "Oh? There's a problem on 'example.com/user/ID/edit' CLIENT 1? I'll get right on that!." If you have multiple modules handling routes that start with /user then in my mind you didn't design things out well. Seeing that URL I should be able to tell what module it's related to. In the same thought process if I don't know where things are I can easily figure out what module I need to look at, and from there use it's route list to find the problem component if it didn't become evident. This helps immensely in the maintenance of an app, as well as in on boarding new team members who need to pick up and add new features, or help support older ones.
That's my thoughts on this. I'd be interested in hearing more. personally it comes down to ease of maintenance, expansion, and logical design.

im using dedicated routes in each of my modules..  no problem now.
The routes are defined in my configs folder as normal..
then my index.js sets them up
import methodStubs from './configs/method_stubs';
import actions from './actions';
import routes from './configs/routes.jsx';

export default {
  routes,
  actions,
  load(context) {
    methodStubs(context);
  }
};

I moved to the same concept @payner35, I just saw @arunoda request opinions and I'm an opinionated person at time on certain topics :stuck_out_tongue:


fix bug where appliedTo wasn't initialized for MPVariantAction for in…

…itWithCoder. stop variant actions on the main thread which makes them take more consistently

LGTM


Design improvements

Interesting. It certainly tells the story better for the stream gages - which is a good thing. From a design POV: the green-blue color adds another color scheme to the current 'blue, grey, orange'. I hadn't mentioned that before bc the lines and dots kept blue-green from being too prominent. Now that there is a fill though, it plays a larger role. One way to soften it is to play with the transparency of the fill. <- that is my suggestion. I like it otherwise.


Warn, instead of exiting on error

Signed-off-by: Alex Ellis (OpenFaaS Ltd<EMAIL_ADDRESS>
Description

Warn, instead of exiting on error
Motivation and Context
Fixes: https://github.com/zeerorg/cron-connector/issues/25
How Has This Been Tested?
Testing pending, it should stop the code from hanging when the list functions endpoint returns an error though.

Did a quick testing for the above MR code. It looks working fine

Install  faasd

git clone https://github.com/openfaas/faasd --depth=1
cd faasd
./hack/install.sh

Login to faasd

sudo cat /var/lib/faasd/secrets/basic-auth-password | faas-cli login -s
faas-cli ls

Export Gateway URL and secret path for cron-connector

export gateway_url=http://<IP_ADDRESS>:8081
export basic_auth=true
export secret_mount_path=/var/lib/faasd/secrets

Add current user to root

sudo usermod -a -G groupname username

Build Cron Connector and start

go build --ldflags "-s -w  -X github.com/openfaas/cron-connector/version.GitCommit=${GIT_COMMIT} -X github.com/openfaas/cron-connector/version.Version=${VERSION}" -a -installsuffix cgo -o cron-connector .

sudo -E bash -c './cron-connector'

deploy sleep and nodeinfo function from store

faas-cli store deploy sleep --annotation topic=cron-function --annotation schedule="*/1 * * * *"
faas-cli store deploy nodeinfo --annotation schedule="*/5 * * * *" --annotation topic=cron-function

Remove nodeinfo function after first run then sleep function. The first time remove command causes failure, a quick second one removes it.

faas-cli remove nodeinfo
faas-cli remove sleep

Logs for cron-connector

2022/08/17 22:27:56 Version:    Commit: dev
2022/08/17 22:27:56 Gateway URL: http://<IP_ADDRESS>:8080
2022/08/17 22:27:56 Async Invocation: false
2022/08/17 22:27:56 Rebuild interval: 10s       Rebuild timeout: 5s
2022/08/17 22:31:46 Added: nodeinfo.openfaas-fn [*/5 * * * *]
2022/08/17 22:32:26 Added: sleep.openfaas-fn [*/1 * * * *]
2022/08/17 22:33:00 Invoking: sleep.openfaas-fn [*/1 * * * *]
2022/08/17 22:33:02 Response: sleep [200] (2.06s)
2022/08/17 22:34:00 Invoking: sleep.openfaas-fn [*/1 * * * *]
2022/08/17 22:34:02 Response: sleep [200] (2.04s)
2022/08/17 22:35:00 Invoking: nodeinfo.openfaas-fn [*/5 * * * *]
2022/08/17 22:35:00 Invoking: sleep.openfaas-fn [*/1 * * * *]
2022/08/17 22:35:00 Response: nodeinfo [200] (107ms)
2022/08/17 22:35:02 Response: sleep [200] (2.09s)
2022/08/17 22:35:26 Removed: nodeinfo.openfaas-fn [*/5 * * * *]
2022/08/17 22:36:00 Invoking: sleep.openfaas-fn [*/1 * * * *]
2022/08/17 22:36:02 Response: sleep [200] (2.02s)
2022/08/17 22:36:36 error listing functions in openfaas-fn: server returned unexpected status code: 500 - Metrics hander: unexpected status code retrieving functions from backend
2022/08/17 22:36:46 error listing functions in openfaas-fn: server returned unexpected status code: 500 - Metrics hander: unexpected status code retrieving functions from backend
2022/08/17 22:36:56 error listing functions in openfaas-fn: server returned unexpected status code: 500 - Metrics hander: unexpected status code retrieving functions from backend
2022/08/17 22:37:00 Invoking: sleep.openfaas-fn [*/1 * * * *]
2022/08/17 22:37:02 Response: sleep [200] (2.28s)
2022/08/17 22:37:06 error listing functions in openfaas-fn: server returned unexpected status code: 500 - Metrics hander: unexpected status code retrieving functions from backend
2022/08/17 22:37:16 error listing functions in openfaas-fn: server returned unexpected status code: 500 - Metrics hander: unexpected status code retrieving functions from backend
2022/08/17 22:37:26 error listing functions in openfaas-fn: server returned unexpected status code: 500 - Metrics hander: unexpected status code retrieving functions from backend
2022/08/17 22:38:00 Invoking: sleep.openfaas-fn [*/1 * * * *]
2022/08/17 22:38:02 Response: sleep [200] (2.30s)
2022/08/17 22:38:56 Removed: sleep.openfaas-fn [*/1 * * * *]

Thank you

I've removed the sleep statements and am merging the fix for a release.


Authentication Method related special groups are put in claim set even if a different authentication method is used

Describe the bug
Currently, the set of special groups, to be put in context, during login flow, is always added in context and put into JWT claims set. The only constraint is that the authentication method for which they are configured is enabled. This flow makes sense for those methods which are not "actual" authentication methods, but are put in the stack (e.g. the IPAuthentication). But there are cases when repository administrators want those groups add to context only when an user logs in using a particular method, e.g. users logged in via Shibboleth might have permissions not granted to those logged in via PasswordAuthentication and vice versa.
To Reproduce
Steps to reproduce the behavior:

Enable Shibboleth authentication
Configure a special group to be put in context during Shibboleth Authentication flow, using authentication-shibboleth.default-roles and authentication-shibboleth.role.shibusers properties.
Login via Password Authentication and go to "Profile" Page
"Authorization special groups you belong to" will list the special group set at point 2

Expected behavior
System should distinguish when special groups are to be put in context only when an authentication method is enabled from those to be put in context only when an authentication method is used. We, at 4Science, can look into it and propose a solution in a PR.

Thanks @corrad82-4s !  Assigning to you and pulling over to the 7.6.x maintenance board.
NOTE: this also seems to be related to or the same issue as #9074

Thanks @corrad82-4s !  Assigning to you and pulling over to the 7.6.x maintenance board.
NOTE: this also seems to be related to or the same issue as #9074


Maintainership

As @bastibe seems to have been busy/away for a few months now and there are quite a few bugs and PRs, I would like to ask how to move on here.
Would you be open to adding co-maintainers to the repo?
Should we move to an organization such as https://github.com/emacsorphanage or https://github.com/toemacs ?
CC @tarsius @purcell

I would absolutely be open to adding maintainers. Life is indeed very busy at the moment and leaves little room for active maintenance on this project.

So I would consider @dppdppd @jmay and myself due to recent contributions :)
@dalanicolai also has two PRs up, idk if he might still be interested in joining again

Happy to help out in any way I can, I do have some time available. But my single contribution to the code was pretty minimal.

Gladly! I'll grant commit rights to @jmay and @xeruf, and to @dppdppd as well if they so desire.
Since I've been in a situation like this before, it is a good idea at this time to think about how we'd like to collaborate.
I think it's generally a good idea to never commit to master, and to only merge pull requests if at least a second person has accepted it (barring time-critical bug fixes etc.). I also think it's a generally good idea to not have too many fixed rules to follow and keep things informal and personal 😁.
What do you think?

Yes, totally agree, if there are more than two maintainers this is not a problem.
I do think it is sensible to enable branch protection on master, preventing force pushes and requiring a PR review, all this can be added in the settings.
I guess we all will only contribute sporadically, but the point is that we do not have a single point of failure anymore :)
I have written https://kull.jfischer.org, maybe you find that useful :)

Personally, I prefer social conventions over technological restrictions. As such, I am not generally a fan of git hooks or protected branches to enforce policy. I've found it more of a hindrance than a useful protection for professionals (although it is a good safety net for novice contributors).
Regardless, if you'd prefer a more rigid workflow, I won't stand in the way.

Agree with @bastibe. Until I know the codebase really well, I would not merge anything without discussion, but I don't like to set up elaborate control structures & systems before they are needed.

Feel free to merge as you see fit! Of course you can always ping me if you'd like my input.
Most important of all, however, please do not feel pressured to "work" on org-journal. We're doing this for fun, and there is absolutely no obligation to respond to things quickly or at all. I've had my brush with burnout, and it's not worth it for anybody. When in doubt, it is perfectly acceptable to tell people to fix things on their own. Github is a platform for friendly collaboration, not work allocation.

@bastibe I'm happy to test & merge the most recent few PRs, but I don't have permissions yet.

@jmay, Github listed you as "invite expired". I've sent you another one.

I had and still have to resolve personal things, but I think I`m fit enough to contribute again. A big thanks to all the contributor for helping to improve org-journal. @bastibe also reached out around 2021 in person, but at that time a wasn't fit enough to continue, sorry for not being honest -- Does that mean I have passed the "I'm not a robot" test?

I think it's generally a good idea to never commit to master, and to only merge pull requests if at least a second person has accepted it (barring time-critical bug fixes etc.). I also think it's a generally good idea to not have too many fixed rules to follow and keep things informal and personal 😁.

The CI is running again 👍🏽 So no commits at all from now on to master "(barring time-critical bug fixes etc.)" etc. loophole :-D
What about time constraints? What if nobody is answering for N days?

I'm glad to hear that you're feeling better, and I'm glad to have you back in action!
Nevertheless, always remember that we're doing all of this for fun only! Our personal health and wellbeing are way more important than this here project. Stay safe everybody.


i  manual  instal the MMProgressHUD ,I like it

My question:   I import the lib in my new project ,buy has  very error ,

Property 'height' not found on object of type '__strong id'
2.: Implicit declaration of function 'CGSizeMake' is invalid in C99

3 Use of undeclared identifier 'CGFloat'

It sounds like you have not imported UIKit somewhere.


Addressing AOAI.Inference outstanding comments on documentation

Closes #33789

Added missing links to DefaultAzureCredential
Rephrased some parts of the README

API change check
API changes are not detected in this pull request.


fix: check clustering attribute type before calling partition tree shuffle

some maskers, such as text and composite, have a clustering attribute, but attribute is not a numpy ndarray representing a partition tree as partition tree shuffle expects.
this change ensures that if clustering is not the expected numpy ndarray type, it will gracefully fallback to non-partitioned shuffle.
this is what partition shuffle expects
https://github.com/slundberg/shap/blob/6e04be3f4a9c7d61ada301366e4273a096e5c1f5/shap/utils/_clustering.py#L17-L30
this is what text provides:
https://github.com/slundberg/shap/blob/6e04be3f4a9c7d61ada301366e4273a096e5c1f5/shap/maskers/_text.py#L163-L168
and what composite provides:
https://github.com/slundberg/shap/blob/6e04be3f4a9c7d61ada301366e4273a096e5c1f5/shap/maskers/_composite.py#L20-L50

Thanks! I think this might be something we don't want to just "fall back" for though, since if the masker has a (non-None) clustering attribute it means it needs to be respected. The Composite masker still needs some work, but do you mind sharing the case where this fails with the text masker? Thanks again.

Thanks! I think this might be something we don't want to just "fall back" for though, since if the masker has a (non-None) clustering attribute it means it needs to be respected. The Composite masker still needs some work, but do you mind sharing the case where this fails with the text masker? Thanks again.

do you mind sharing the case where this fails with the text masker?

Sure, I encountered it using a HuggingFace transformer with PermutationExplainer
Roughly following https://shap.readthedocs.io/en/latest/example_notebooks/api_examples/plots/text.html
from sentence_transformers import SentenceTransformers # wrapper around HuggingFace transformers
from sklearn.neural_network import MPLClassifer
from shap import PermutationExplainer

model = SentenceTransformer("quora-distilbert-base")
classifier = MPLClassifer()

# train classifier on encoded sentences

def f(x):
    return classifier.predict_proba()

explainer = PermutationExplainer(f, model.tokenizer)
shap_values = explainer(values_to_explain)

do you mind sharing the case where this fails with the text masker?

Sure, I encountered it using a HuggingFace transformer with PermutationExplainer
Roughly following https://shap.readthedocs.io/en/latest/example_notebooks/api_examples/plots/text.html
from sentence_transformers import SentenceTransformers # wrapper around HuggingFace transformers
from sklearn.neural_network import MPLClassifer
from shap import PermutationExplainer

model = SentenceTransformer("quora-distilbert-base")
classifier = MPLClassifer()

# train classifier on encoded sentences

def f(x):
    return classifier.predict_proba()

explainer = PermutationExplainer(f, model.tokenizer)
shap_values = explainer(values_to_explain)

Thanks for the example. I see now that the Permutation explainer did not handle maskers where the clustering is dynamic (it changes for each sample (like text). I just added some support for that to this PR. See if it works for you.

Apologies for the delay in responding.
Yes, trying this out locally a900937  works with the tokenizer swimmingly.
Thanks @slundberg!


Support for S3 transfer acceleration

It would be great to not have to use this escape hatch:
(uploadBucket.s3Bucket.node.defaultChild as s3.CfnBucket).accelerateConfiguration = {
    accelerationStatus: 'true',
};

And instead do:
  const uploadBucket = new Bucket(scope, 'UploadBucket', {
    s3Bucket: {
      bucketName: `${process.env.APP_NAME}-uploads-${scope.stage}`,
      transferAcceleration: true,
    },
  });

Yup.
I also voted on the CDK's issue in case they get to this first.

We are transition this repo to https://github.com/sst/v2 and are closing this issue for now. If it is still relevant please reopen it there - sorry for the trouble


Add an option for flair in specific subreddit

Add this option

i don't think there is any hope for an update as this script was made 5 years ago 💀