Restrict another hub to prevent accidents

Please provide a description of this PR:

@howardjohn: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

gencheck_istio
a10b47a74067852a07d084313815c1c4dfcbf409
link
unknown
/test gencheck_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/test gencheck_istio

@howardjohn: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

gencheck_istio
a10b47a74067852a07d084313815c1c4dfcbf409
link
unknown
/test gencheck_istio

integ-security-multicluster_istio
a10b47a74067852a07d084313815c1c4dfcbf409
link
unknown
/test integ-security-multicluster_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/retest

@howardjohn: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

integ-pilot-istiodremote-mc_istio
a10b47a74067852a07d084313815c1c4dfcbf409
link
unknown
/test integ-pilot-istiodremote-mc_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.


Add PKITS to unit tests

For compliance with RFC 3280.
The existing script test/pkits-test.pl attempts to run the tests but needs integration with the test framework and some of the tests are currently failing.

Scope:

re-run the test
filter out failing tests
report real bugs, if any


Remove text when spinner is present

Hi, thanks for creating this loader button - it's great!
Just wanted to mention that when I use it, I remove the text when the spinner is present. I personally like that approach vs. moving the text to the side when it's active. The only thing I have to consider is the width of the button. It would be nice if this were somehow built into the module with an option. Here's an example of how I use it (grey-spinner class modifies the color of the spinner graphic):
<Button bsStyle="default" loading={state == 'busy'} className="grey-spinner" style={{minWidth: '90px'}} onClick={() => this.props.dispatch(getLog())}>{state !== 'busy' ? 'View Log' : ''}</Button>

Hi @jgentes,
First of all, thank you for the nice feedback. If I understood correctly, you want to hide the text but keep width of the button. Currently, I am not able to check, what happens if button is in the loading state but there is no text as I will have my laptop back only next week. If it renders ugly, than it is a bug obviously. If everything is fine but you want a different behavior, I would recommend to define your own component based on this one, which overrides custom behavior in a way you want.

I was able to get it work perfectly by just specifying the minimum width of
the button. Then when the text disappears and the spinner appears, there's
no change to the button width. I was just thinking that if there's a more
programmatic way to do it, that would be nice.
Thanks again!
On Thu, Feb 23, 2017 at 1:49 PM Yury Dymov<EMAIL_ADDRESS>wrote:

Hi @jgentes https://github.com/jgentes,
First of all, thank you for the nice feedback. If I understood correctly,
you want to hide the text but keep width of the button. Currently, I am not
able to check, what happens if button is in the loading state but there is
no text as I will have my laptop back only next week. If it renders ugly,
than it is a bug obviously. If everything is fine but you want a different
behavior, I would recommend to define your own component based on this one,
which overrides custom behavior in a way you want.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/yury-dymov/react-bootstrap-button-loader/issues/1#issuecomment-282132420,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABGKHjxgLsvUOp2qw-zk3zXFhfNkJwxaks5rff8EgaJpZM4MKhRr
.

Hello @jgentes, finally got my laptop back.
Give me couple of days to think, what can be done to implement this feature without evaluating width during runtime or doing something strange

Oh, completely forgot about this one. I would say it works fine for me and I don't face such use-case, so I am not going to implement it but PRs are welcome :)


[WIP] Ground work to enable rav1e to access memory for blocks straddling on frame boundaries

The goal of this work is to establish the ground so that rav1e can encode blocks that straddle on bottom or right frame boundaries (i.e. #2396).
I thought it would better approach to first land this groundwork then land the open-partition feature,  #2396.
Because, that way we can ensure the groundwork itself does not affect the integrity of current master without any open-partition code mixed.
With this PR only, it is expected to not change coding behavior of current master.
Until it is finished, it will fail many of rav1e's internal cargol tests, mostly when comparing encoder and decoder outputs, not because the enc and dec's outputs really unmatched but more likely the dimension of plane when it is declared is not correctly changed.
When compare the two reconstructed outputs from rav1e and decoder that run on command line and not via cargo test , I see that they are matched.
Also, at the moment I create this PR, I don't observe rust panic caused by memory access violation (i.e. accessing outside the bounding box defined by PlaneRegion (+ subregion) ).

One tip for me: To test one failed test,
env RUST_BACKTRACE=full RAV1E_CPU_TARGET=rust cargo test --features=decode_test_dav1d dimension_32x32_dav1d

Hmm, after I see compare_pic( ) (used during test, comparing encoder and decoder output frames),  I start to think that the idea of rounding up input and output frame (i.e. their planes) size to SB might be bad.

In Irc #daala,
xiphmont, Another thing I am seeing now that is surely related to work you did before/do later is the use of bounding box in cdef_filter_tile( ) via rect( ).width|height.
 This makes me look again what I think so far is reasonable. I.e. TileRect is rounded up to SB size (thus its input and output (i.e. rec) tiled planes (i.e. PlaneRegion) ) but ts.width|height is not rounded up and keep coded w and h of tile.)  With my approach, In cdef_filter_tile( ),  "rec_w = rec.planes[p].rect().width" cause w and h include outside of coded frame w and h, which is wrong with current code of the  function.
 So, I am basically needing more time on pondering which way is more correct, tiled plane following coded w and h as current master or extend it include the block crossing frame boundaries. The most basic clue to me is that, as with the case how CFL works with block straddle on frame boundaries, if a block being encoded for any prediction mode requires to read/write pixels outside coded frame, the area should be included  as a plane.

Coverage increased (+0.003%) to 81.859% when pulling 2bafb76f1ef0628470c2d71b1401638a9ced37e2 on ycho:access_beyond_frame_area into bea1bdea80ab10979316ca94facb5ff0a05dd0ec on xiph:master.

YAY~~~~~~~!!!

There is no change in coding efficiency or encoding speed.
awcy, -s 6

With my most recent commit(s), I have focused on correctly replace the existing bounding 'rectangle' under PlaneRegion (either from input or output tile planes) with TileState.input_tile|res.plane[ ].plane_cfg.width|height or TIleState.width|height if possible. In some cases, correct bounding rectangle requires to apply the origin of rect, i.e. rect.x|y, because the origin of rectangle can be off the (0,0), as in two cases I see, 1) via subregion(), 2) in cargo test, 'tile_encoding_with_stretched_restoration_units' .

As soon as this PR has landed, I will rebase my open-partition PR https://github.com/xiph/rav1e/pull/2396 on it then ask reviewers, esp. @lu_zero, to review again.

@xiphmont, @lu_zero, Your opinions are greatly appreciately as well!

If I'm reading the code correctly, this has no effect whatsoever except at the right and bottom visible frame boundaries.  Tile access still does not overlap.
So... Not the way I'm chasing it in my Rectangle Reform patches, but this is simple, tiny, and works.  I see no reason not to land it ASAP.
r+me


fix: new artifact streaming containerd configuration and contract

Fixes #
Description
Adds support for artifact streaming. The change is gated on the ACR Side and karpenter will just enable artifact streaming by default. We want to get this in now so that for GA we have some testing for it.
How was this change tested?
e2e tests
Does this change impact docs?

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

Release Note

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

0 of 0   changed or added relevant lines in 0 files are covered.
37 unchanged lines in 4 files lost coverage.
Overall coverage decreased (-0.04%) to 97.736%

Files with Coverage Reduction
New Missed Lines
%

pkg/providers/launchtemplate/launchtemplate.go
2
86.67%

pkg/apis/v1alpha2/aksnodeclass.go
3
0.0%

pkg/providers/imagefamily/bootstrap/aksbootstrap.go
6
87.1%

pkg/providers/instance/instance.go
26
90.5%

Totals

Change from base Build<PHONE_NUMBER>:
-0.04%

Covered Lines:
35665

Relevant Lines:
36491

💛  - Coveralls

closing this in favor of enabling when we have contract


pointing referenced variable groups to the latest documentation

All Submissions:

[YES] Have you followed the guidelines in our Contributing document?
[YES] Have you added an explanation of what your changes do and why you'd like us to include them?
[YES] I have updated the documentation accordingly.
[NA] I have added tests to cover my changes.
[YES] All new and existing tests passed.
[YES] My code follows the code style of this project.
[YES] I ran lint checks locally prior to submission.
[YES] Have you checked to ensure there aren't other open Pull Requests for the same update/change?

What is the current behavior?

The documentation for the getting started flow is out of date based on the updated pipelines. I removed some duplicate docs and linked to the new documentation instead.
Issue Number: #377

Glad to see the pointers back to a single, more 'authoritative' var group reference. That sort of thing helps documentation to stay more 'evergreen' longer.
+1


implement SklearnClassifier.batch_prob_classify

Alright, here's scikit-learn classification with probabilistic outputs. Is there anything else I need to do for this module? Documentation?

Thanks!  Some docstrings with doctests would be a great way to show users how to use the code, cf:
https://github.com/nltk/nltk/blob/master/nltk/tokenize/regexp.py
If you want to include any formal regression tests, they would go here:
https://github.com/nltk/nltk/blob/master/nltk/test/classify.doctest

Alright, but tests would require importing scikit-learn. Is that a problem? Is there an established way to do a conditional import?

We assume tests are run on machines which have all dependencies installed.  You can see how we do conditional imports of submodules in nltk/tag/init.py (tag.hmm is only imported if numpy is installed -- could you do the same for numpy in classify/init.py please?).  This means users get some functionality on a bare-bones install.
Also, I propose that you try to import the sklearn package and give a useful error message if its not there, cf. classify/megam.py:134.  This means users get a hint about how to get the code working, without needing to read it.


Add Saurabh-Raj to Contributors list

Hi  @B2-Raj09, I'm quite elated about your pull request. I wanna evolve this project to addresses various problems faced by first-time contributors. I'd love to learn about your journey in open source community, the problems, pain points you had etc.
Could you explain how you felt when you went through the tutorial, made a pull request and learned that I merged it?
We’ve recently added social share  to our web app. Could you please go to https://roshanjossey.github.io/first-contributions/#social-share and share your first contribution to open source?
Also, check out projects with easy issues while you’re there.


TypeError: _write_metrics() missing 1 required positional argument: 'data_time'

File "/home/a/.local/lib/python3.8/site-packages/detectron2/engine/train_loop.py", line 133, in train
self.run_step()
File "/home/a/code/centerX-master/engine/defaults.py", line 353, in run_step
self._write_metrics(metrics_dict)
TypeError: _write_metrics() missing 1 required positional argument: 'data_time'

Maybe the your detectron2 version is not match.
refer to https://github.com/JDAI-CV/centerX/issues/17#issue-772594795

Maybe the your detectron2 version is not match.
refer to #17 (comment)

学习率每设置好，导致FloatingPointError: Loss became infinite or NaN at iteration=227!，进而导致了这个错误。


When an executable is dragged into debugview is it not correctly logging a terminate message

is the stdout/stderr pipes are broken, we could check if the process has actually ended or the pipes were closed and log a message accordingly

as it turns out it does not really matter if the pipes are broken, the ProcessReader is not detecting it any way. We could add the detection, but then we would have to ignore it to solve this issue.
We've solved this issue in 5d4425e03c97d3b4d042eb2741483f8ad1a1e368 by correctly adding the process handle and logging the terminate message


Work with remte data

how to fetch data from remote

You should fetch data in options.source: function (request, response) {/* fetch data and response(data)*/}


SAKIII-5271 Don't double-escape group titles in my memberships

They've already been escaped when applying three dots in the groups api

https://jira.sakaiproject.org/browse/SAKIII-5271

Verified and merged.


Add webpack's new cache loader

https://github.com/webpack-contrib/cache-loader

never try it before, will it replace happy pack usage?

Possibly.  It's a new kid on the block.  We'll have to do some testing.

I dont think it will be a 1:1 replacement for HappyPack. Main reason is HappyPack allows bundling to happen on different threads. Would be curious to see if running them in tandem gives a benefit.

Yeah, definitely not a 1:1 replacement, however if the benefits of happypack ontop of cache-loader are negligible I would vote in favour of removing happypack to reduce complexity.

The problem with HappyPack is that even though the workload is distributed between threads, the work itself is not separate enough to get the benefits I'd like.
The main webpack script gets a thread, and each of the loaders (new HappyPack()) get a thread (as I understand it). But almost all the work lies with the babel-loader, which only gets a single thread. Imagine if we could split that into multiple threads.
But its still faster than normal, both with and without the HappyPack cache.

@birkir if you change the HappyPack plugin to look like this
import os from 'os';
import { execSync } from 'child_process';
import appRootDir from 'app-root-dir';

import HappyPack from 'happypack';

export function happyPackPlugin({ name, loaders }) {
   const compilerThreadPool = HappyPack.ThreadPool({
    size: os.cpus().length,
  });
  return new HappyPack({
    id: name,
    verbose: false,
    threadPool: compilerThreadPool,
    loaders,
  });
}

It will delegate the most resource intensive plugins, like babel-loader to use more threads than say sass-loader

Got it! Now seeing my CPU on full blast!


remove "did you mean" feature

Dev rel team said this doesn't work

Are you sure you didn't miss updating the lock file? BTW that's why CI should use npm ci

This is working - if there's a bug or proposal to remove, let's open an issue first.


Create a separate tox environment for integration tests.

Move integration tests to a separate directory.

The code LGTM, but I have a question about the interface: will the integration tests run automatically if I just run tox? I think that would be a bad behavior.
Also, sorry to make you jump through so many hoops, but I think that given that tox -e integration sets PERFKIT_INTEGRATION for you, we should just drop the environment variable requirement. I can do that in a separate PR if you want.

Running just tox will use the default envlist, which just runs flake8 and the tests in the tests directory (resursively). But because PERFKIT_INTEGRATION is neither forwarded nor set, the integration tests will not run.
If you want to remove the PERFKIT_INTEGRATION variable, then all unit tests will probably need to be in a separate directory, similar to how I've split out the integration tests in this PR.

I see. I think it might be nice to have separate folders, but that can be a separate PR from this one. LGTM.


Garden fieldsettings should have a lastGroupingStep option

Expected Behavior

A garden's fieldsettings should have an optional property called lastStep which should be set to true if this specific grouping should be the last, and no other grouping is allowed after it.
Current Behavior

A user can now group on everything without any constraint which might cause errors. For example in the app Heat Trace, the user should not be allowed to add any more groupings after HT Cable is chosen.
Possible Solution

Here

Closed due to serverside migration


⚠ Bitwarden has degraded performance

In bcb359b, Bitwarden (https://bitwarden.underthere.xyz) experienced degraded performance:

HTTP code: 502
Response time: 18880 ms

Resolved: Bitwarden performance has improved in cdf1aa0 after 1 minute.


BUG: special: silent precision loss in kv near (1.5, -1)

For n = 1.5 and z = -1 + 1j*e for real e small (less than 1e-5), kv returns inaccurate values.  Example:
>>> scipy.special.kv(1.5, -1 - 1j*1e-20)
(-5.6498675426425281e-17+2.2204460492503131e-16j)
>>> mpmath.besselk(1.5, -1 - 1j*1e-20)
mpc(real='-3.4068610448155485e-20', imag='-1.7034305224077743e-40')

There may be other points in (n, z) space with similar issues, but this is the only one I've been able to find.

FWIW, the issue is that kv has a zero at n = 1.5 and z = -1 and at the zero the continuation formula it's using (DLMF 10.34.2 with m = 1) suffers from catastrophic cancellation. For orders of the form n = 2*i + 1.5 there's also a zero on the negative real axis (it moves further out the axis as n increases) which will cause a similar loss in precision. For example, at n = 3.5 there's a zero at approximately z = -2.322185354626086 + 0j and we have
>>> kv(3.5, z)
(-3.734975229435317e-17-1.1102230246251565e-15j)
>>> mp.besselk(3.5, z)
mpc(real='0.0', imag='-8.9342613861852027e-16')

I'm closing this-loss of relative accuracy around zeros is just a fact of life. We could special-case the n = 1.5 case to solve this particular problem, but as there are similar problems for other orders I don't see much of an advantage in that.


Setting "titleBar.position" to nonzero x or y makes panel jump on click

When I set these options (sorry for ClojureScript syntax):
{:titleBar
   {:drag true
    :position {:x 10 :y 10}}}

Then, whenever I click on the panel, it jumps to those coordinates.

If I only set y to a non-zero position, the panel only jumps to the new y position, same with only setting x.
I don't think that this is related to the options object getting redefined, because the Leva component is not re-rendering on click (I'm pretty sure?)
I wasn't sure where we came out on the Discord discussion, but this behavior definitely seems like a bug.

This also took me by surprise, but the way it currently works is that once you supply a position to Leva, it is then expecting you to control it's position all the time, even during dragging. What this means in practice is that you must supply an onDrag handler as well.
I think that there is probably a more elegant default solution which should be added to Leva, but to get it to work for now:
// in component
const [controllerPosition, setControllerPosition] = useState({ x: 100, y: 100 })

// in return
<Leva
  titleBar={{
    position: controllerPosition,
    onDrag: (position) => setControllerPosition(position),
  }}
/>

Ah, that's great. Yes, I see how to stick with React idioms we'd also want a defaultPosition which would act the way I was hoping.


chore(doc): documentation reindexing

Reindex the documentation as soon as you release a new version of the
website. You need to export the associated ALGOLIA_API_KEY env variable.
Also move to the world-wildly replicated latency account.

Ok I will add the admin key to travis as a secured env

What I will do: enhance the travis build so that the index is rebuilt when gh-pages is pushed. You will also be able to do it manually of course by running a task


unknown option: 'c'

When executing :set +c in Intero 0.1.15, I get an error saying that isn't recognized.
λ ~/Projects/lol/ stack exec -- intero --version
0.1.15
λ ~/Projects/lol/ stack ghci --with-ghc intero
Using main module: 1. Package `lol' component exe:lol with main-is file: /home/matt/Projects/lol/src/Main.hs
Configuring GHCi with the following packages: lol
Intero 0.1.15 (GHC 7.10.3)
Type :intro and press enter for an introduction of the standard commands.

*** WARNING: /home/matt/.dotfiles/haskell/ghci.symlink is writable by someone else, IGNORING!
[1 of 1] Compiling Foo              ( /home/matt/Projects/lol/src/Foo.hs, interpreted )
Ok, modules loaded: Foo.
[2 of 2] Compiling Main             ( /home/matt/Projects/lol/src/Main.hs, interpreted )
Ok, modules loaded: Foo, Main.
*Main Foo> :set +c
unknown option: 'c'
*Main Foo>

Oh, that's awesome :smile: Unfortunately, it's still showing up in the :help output:
...
  Options for ':set' and ':unset':

    +m            allow multiline commands
    +r            revert top-level expressions after each evaluation
    +s            print timing/memory stats after each evaluation
    +t            print type after evaluation
    +c            collect type/location info after loading modules
    -<flags>      most GHC command line flags can also be set here
                         (eg. -v2, -XFlexibleInstances, etc.)
                    for GHCi-specific flags, see User's Guide,
                    Flag reference, Interactive-mode options

Thanks for the pointer, I'll remove it.


[Bug] Can't acsess mounted drives

It seems that the file browser is not able to enter mounted drives.
E.g. if i use Open Repository... and go to my root folder i can't even see the /mnt or /media folders where my other drives are mounted.
It also cant follow symlinks, they will be simply displayed as a file of the type symboliclink.
version: 1.2.2
os: Ubuntu 18.04

Hi @RainerBayr are you using the Flatpak version? Currently this version only allows to access /home


ENOTFOUND error when using in Firebase Cloud Function

I'm trying to call balance and openOrders from a Firebase Cloud Function and getting an error:
{ Error: getaddrinfo ENOTFOUND api.binance.com api.binance.com:443
    at errnoException (dns.js:28:10)
    at GetAddrInfoReqWrap.onlookup [as oncomplete] (dns.js:76:26)
  code: 'ENOTFOUND',
  errno: 'ENOTFOUND',
  syscall: 'getaddrinfo',
  hostname: 'api.binance.com',
  host: 'api.binance.com',
  port: 443 }

Similar error for balance.
When I run it locally I get the data back that I would expect.
Inside of my cloud function it looks like:
exports.syncBalanceData = functions.https.onRequest((req, res) => {
  // sync balance data to firebase for evaluation
  binance.balance((error, balances) => {
    if (error) {
      console.log('balance error: ', error);
    } else {
      for (const index in balances) {
        if (balances.hasOwnProperty(index) && balances[index].available > 0.5) {
          const element = balances[index];
          console.log(index + ' balance: ', element.available);
        }
      }
    }
  });
});

I'm not too familiar with the Firebase platform, but I believe you need to be on a paid plan to make external API calls. Either way, this is an issue you'll have to take up with Firebase.

I didn't even think about that, thanks.


Test

VSCode Version:
OS Version:

Steps to Reproduce:

Does this issue occur when all extensions are disabled?: Yes/No

\gifPlease

\gifPlease


operator project-quay (3.9.0-nightly.20230320)

Add 3.9.0-nightly.20230320 to community operators

/merge possible

/merge possible


Build Error TargetMachine.h

Hi,
I am trying to build the compiler on linux. It failed with the message
hylo/.build/checkouts/Swifty-LLVM/Sources/llvmshims/src/shim.cc:1:10: fatal error: 'llvm-c/TargetMachine.h' file not found
Ok, so I tried on Darwin.
also failed with the same message.

You have most likely not followed the instructions regarding the llvm.pc file in the README.

I'm closing as the problem appears to have been fixed. Feel free to re-open if necessary.

One caveat to this is that if llvm is updated by package manager, eg apt or homebrew, this needs to be done again. At least if using the installation of llvm.pc file instead of using environment variable.

Well, it depends on what your update is doing to your system and how your pc file has been configured.
If you only use paths that do not mention a specific version of LLVM, I would expect that an "update" would work just fine as long as LLVM API doesn't change (but then I guess there's nothing we can do about it).
If you use paths mentioning a specific version and you have a package manager able to host multiple versions of LLVM on your system, you should also be fine.

Expectations aside, it is a fact that I have had to regenerate my .pc files several times for this reason.
I think we should tell people it might happen to them. Not everyone knows how this stuff works.

Yes, at least for those using homebrew this is an issue. I have had to redo this setup several times. Eg my current llvm.pc file looks like this:
Name: LLVM
Description: Low-level Virtual Machine compiler framework
Version: 17.0.3
URL: http://www.llvm.org/
Libs: -L/opt/homebrew/Cellar/llvm/17.0.3/lib -L/opt/homebrew/Cellar/zstd/1.5.5/lib -lc++ -lLLVM-17
Cflags: -I/opt/homebrew/Cellar/llvm/17.0.3/include

And now after brew upgrade I have llvm 17.0.5
It should be possible to setup a llvm.pc with persistent paths, using the provided homebrew version symlinks, but this would require a modified setup compared to what is described in readme.
ls -l /opt/homebrew/opt/llvm
lrwxr-xr-x@ 1 nils  admin  21 Nov 17 08:55 /opt/homebrew/opt/llvm@ -> ../Cellar/llvm/17.0.5

Using the /opt/homebrew/opt/llvm symlink in the llvm.pc would be useful for homebrew.

Yup this llvm.pc seem to work for me, the Version field is not up-to-date, but does not seem to be used by the compilation.
Name: LLVM
Description: Low-level Virtual Machine compiler framework
Version: 17.0.3
URL: http://www.llvm.org/
Libs: -L/opt/homebrew/opt/llvm/lib -L/opt/homebrew/opt/zstd/lib -lc++ -lLLVM-17
Cflags: -I/opt/homebrew/opt/llvm/include

I sympathize with the difficulty to set things up but things are getting beyond our control I think. We didn't choose how SPM uses pkg-config files and we didn't choose how homebrew decides to store its packages. Similarly we can't tell where people decided to install clang and that might have a impact when they move things around.
We can certainly better document how these tools work and/or point to their docs but there will always be some level of customization we can't foresee because everyone can setup their system differently.


When kong is deployed to Google Cloud Kubernetes , konga does not display the list of services properly

HI:
kong version: 0.14.1, konga version: 0.13.0,  I use key_auth for certification.
After I have created a connection rule, I click ACTIVATE button to show that I can connect to the service normally, But when I click on the list of services, it appears empty.
And then I manually accessed this url. curl -X GET https://xxx.xxx.com/api/admin-api/services?apikey=xxxxx, It returns complete service list information.
And what I need to say is that our domain name was created through ingress-gcp.
So I want to ask next, what do I need to do?

On the other hand, I use LoadBalancer to directly open port 80 of kong, The konga can connect to the kong properly and manage the kong properly. So I also need to know whether it's konga or ingress-gcp, so I want to know how to tell if Konga is normal.

@linydquantil , I haven't had the chance to use Konga with Kong on a k8 cluster using ingress controller.
I'll tag this issue in case someone else has experiences something like that.


Doesn't work in NextJS 13.2.4

Adding this swc plugin causes the next dev server to fail to start.
// next.config.js
  experimental: {
    swcPlugins: [['@lingui/swc-plugin', {}]],
  },

It is worked fine before upgrading Next.js

i believe they simply bumped a bundled version of SWC, so now layout of AST is not compatible with current compiled version.

@andrii-bodnar The nextjs-13 example doesn't upgrade to the latest next.
https://github.com/lingui/swc-plugin/blob/main/examples/nextjs-13/package.json#L20
 "next": "13.1.2",

Yep, the plugin doesn't work with @swc/core >= 1.3.39.

I may be missing something but it seems that NextJS 13.2.4 is using @swc/core 1.2.203 (https://github.com/vercel/next.js/blob/v13.2.4/package.json) and the @lingui/swc-plugin is breaking it, so it is not only for @swc/core >= 1.3.39

Current stable version of the plugin compiled with swc_core "0.56.1" that according to Selecting swc_core

Do you guys know any custom SWC plugins which works with latest nextjs? I want to take it as an example.

FYI https://github.com/vercel/next.js/issues/46989

I'm getting this in my project 🙇🏼 any ideas how to fix
https://github.com/lensterxyz/lenster

https://github.com/vercel/next.js/issues/46989#issuecomment-1493911169

I'm getting this in my project 🙇🏼 any ideas how to fix

Downgrade to nextjs < 13.2.4
Latest next-swc has a bug, waiting this to be merged

Looks like v13.3.1-canary.12 fixes this issue.

@TimFL did you try the current version of a plugin with canary release, or just notices that the fix was merged by nextjs team?  I'm asking because whont to know should i rebuild plugin with latest swc-core to support v13.3.1-canary.12 or it works as is.

@TimFL did you try the current version of a plugin with canary release, or just noticed that the fix was merged by nextjs team? I'm asking because whont to know should i rebuild plugin with latest swc-core to support v13.3.1-canary.12 or it works as is.

I'm sorry for the confusion, I meant: that release fixes the swc problems, but plugin devs have to update their libraries so they work again. This needs updating now.

I've tested the latest version of swc_core (0.75) with the newest next.js canary release, and I can confirm that the wasm file can be executed without any issues.

https://github.com/lingui/swc-plugin/pull/54/files

v4 is relased officially and it's already built with latest swc-core. Please see compatibility table in readme

I am also trying to use  Typewind with tailwind config. but it is not working. It seems we can't use any custom plugin with nextjs 13.x.
Also If i am trying to use babel. then its working fine but not able to use most of the latest support from nextjs 13 like fonts giving error. it is asking SWC environment.
Nextjs team please help fix this or give me a workaround to move further. it is extremely required for us.
I also raise some query on Typewind but it is related to Nextjs 13.x

@pawanrana1992 please read compatibility table carefully. The plugins indeed work with nextjs 13.*. You need to choose the right version of both. SWC plugins is experimental feature, they DO BREAK backward compatibility in patch and minor releases, so you need to choose an exact version of nextjs


Clean up labelling

Labeling stuff that has nothing to do with the actual tool:

Change "extra income/savings" to "other income/savings". Same with "extra spending" -- "other spending" is clearer.
Label the amount fields consistently and with a temporal component for clarity -- something like "Annual Amount ($)".

https://www.reddit.com/r/financialindependence/comments/fvsowu/updating_cfiresim/fml2nzd/?context=3

If you think of anything else here you'd want to see cleaned up before closing this issue, please comment here. I'll be happy to go in here and do any more cleanup as needed.

No, I am happy to close out. Thanks.


When I am zooming, it's capturing black and white image

Describe the bug
I am using Camera in my app and have enabled the zooming feature using the default provided prop, when I zoom in and capture the camera, it captures a black-and-white image
To Reproduce
Below is my code:
<Camera
ref={cameraRef}
torchMode={flashMode}
style={{
aspectRatio: 3 / 4,
}}
focusMode="on"
resetFocusWhenMotionDetected={true}
hideControls={true} // (default false) optional, hides camera controls
showCapturedImageCount={false} // (default false) optional, show count for photos taken during that capture session
/>
// Capture method
const onCapture = async () => {
let {uri} = await cameraRef.current.capture(true);
if (uri.startsWith('file://')) {
let path = uri;
setImageToCrop(path);
}
};
Expected behavior
It should keep the colours as it is and not make it black and white
Screenshots

Version [e.g. 22]

Additional context
Mainly facing this issue with iPhone 14 pro and iOS 17.0.2

Can you reproduce this with the example project?


Details about port range used for coturn and dialog services

I couldn't easy find reading the README and the chart the port range used for coturn and dialog services.
FYI in previous aws CloudFormation template, we had those settings and I'm using those settings in my own helm chart for a project using aframe and previous janus sfu.
coturn turnserver.conf
min-port=49152
max-port=51609
and janus
rtp_port_range = "51610-65535"
now hubs is using dialog instead of janus.
In https://github.com/hubs-community/mozilla-hubs-ce-chart it says 35000 - 60000 to be open.
For dialog I think this is the configuration https://github.com/mozilla/dialog/blob/257315f8a418c756fc466c26a8bcc43d6c4d568e/config.js#L62
rtcMinPort : process.env.MEDIASOUP_MIN_PORT || 40000,
rtcMaxPort : process.env.MEDIASOUP_MAX_PORT || 49999
I don't see any MEDIASOUP_MIN_PORT MEDIASOUP_MAX_PORT variables in the helm chart
so I think it's using 40000-49999 range.
For coturn, I see it's using https://hub.docker.com/r/doginal/mozilla-hubs-coturn with no github repo I could find.
Looking directly inside the image:
docker run -it doginal/mozilla-hubs-coturn bash
cat entrypoint.sh
min-port=49152
max-port=51609

so using 49152-51609 range
Isn't it an issue to have overlapping port range here for the two services if the two pods ends up on the same instance?

Hey,
With hostNetwork set to true only a single instance/pod can run of that type because only a single port exists. The LB is not a part of this as it's using the host's network (public ip) to do the peering.
I had issues with the preview version of Hubs ce which is why I built the image. I may swap it back to Mozilla's version but it still wasn't working the last time I tried it.

With hostNetwork set to true only a single instance/pod can run of that type because only a single port exists. The LB is not a part of this as it's using the host's network (public ip) to do the peering.

Yes I know that.
But two pods using hostNetwork: true (here one pod dialog and one pod coturn) can be on the same instance/vm/worker of the pool, using the worker public ip. I'm not sure what the behavior of coturn allocating a port in the configured port range and mediasoup already allocated the same port, or vice versa. It's a hard to see issue unless you are under heavy loads and exhausted most of the ports in the range.
So it would probably best to set MEDIASOUP_MAX_PORT=49151 instead of the default 49999 to be sure this is not an issue.

It's not an issue. They use different ports. (I have them running on the same node no issues)
Dialog uses 4443.
CoTurn uses port 5349 TCP and UDP 35000-60000.

Dialog is a sfu, 4443 is only used for the signaling part, and a udp port is allocated for each webrtc connection, in the range 40000-49999 from the config I see in the dialog repo. For users that don't need a TURN server, no udp port is used for coturn.

For the coturn image, for maintenance and trust, it would be probably best if we can use the official coturn image that is built automatically on new release and when alpine version change
https://github.com/coturn/coturn/releases

@vincentfretin I believe the base Moz image was based on the official coturn image.
I have no issues with swapping it to a default. The hubs one is creating the config at build time and i would like it as a configmap.


Update view width of documentation

What this PR does / why we need it:
Which issue this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged): fixes #
Special notes for your reviewer:
Screenshots

Fixed

Looks good.


chore: add license header to files and github action workflow to check license

Resolves #149

Codecov Report

Merging #154 (a5e97d6) into main (02fc54a) will not change coverage.
The diff coverage is n/a.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
@@           Coverage Diff           @@
##             main     #154   +/-   ##
=======================================
  Coverage   89.13%   89.13%
=======================================
  Files          21       21
  Lines        1693     1693
=======================================
  Hits         1509     1509
  Misses        149      149
  Partials       35       35

Impacted Files
Coverage Δ

revocation/ocsp/errors.go
100.00% <ø> (ø)

revocation/ocsp/ocsp.go
81.39% <ø> (ø)

revocation/result/errors.go
100.00% <ø> (ø)

revocation/result/results.go
100.00% <ø> (ø)

revocation/revocation.go
100.00% <ø> (ø)

signature/algorithm.go
100.00% <ø> (ø)

signature/cose/envelope.go
93.24% <ø> (ø)

signature/envelope.go
100.00% <ø> (ø)

signature/errors.go
94.73% <ø> (ø)

signature/internal/base/envelope.go
97.72% <ø> (ø)

... and 11 more

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


(non-critical) (1.19.2) Weird server error

Hi!
This does not cause server crashes or visual malfunctioning (at least not observed yet), so it's not a top priority but...
My server log is cluttered with a strange error (followed by a stack trace, which is of no interest as it is related to MC world generation):
Unknown registry key in ResourceKey[minecraft:root / minecraft:worldgen/structure_pool_element]: minecraft:tree_pool_element
All traces lead to this mod, where I see "tree_pool_element" is registered in the sources. The first suspicious thing is the "minecraft" namespace. The second - where it is used or defined as a structure? Do you build something in this mod as structures? It looks to me as either some leftover code or missing namespace in the definition.
Please, fix it.
Thanks!

Unfortunately, there are not any traces or crashes to identify it. Just tons of above messages. Right now, it does not appear because I turned world generation off. Possibly you are right and some other mod modifies something in a way that it appears.
Just as a precaution, I would simply recommend reviewing the code around the "tree_pool_element" (or possibly operating it) and re-check is the namespace is always included.
No other ideas...


Upgrade apimachinery in subscription-cleanup-job

Description
Changes proposed in this pull request:

Upgrade apimachinery in subscription-cleanup-job

Related issue(s)

/test pre-master-kcp-development-artifacts


Download images/gif from DM

I am getting following TwitterDirectMessage
{:id=>1429037175354572806, :created_at=>2021-08-21 11:06:27 UTC, :sender=>{:id=>1151323431555551057}, :sender_id=>1151323431555551057, :recipient=>{:id=>44115796855455}, :recipient_id=>44115796855455, :text=>"https://twitter.com/messages/media/1431598232581386244"}
and i need to download that image. How i can fetch this image

@sferik  Can you help me on this?


Nuxt 3 compatibility

Hello is this compatible with Nuxt 3?
I tried to create a nuxt plugin like this, but without results:
import { defineNuxtPlugin } from "#app";

export default defineNuxtPlugin(async (nuxtApp) => {
  if (process.env.NODE_ENV === "development") {
    const VueAxe = await import("vue-axe");
    nuxtApp.vueApp.use(VueAxe.default);
  }
});

Any update on this? Would be helpful if there is a guide/example similar to this - https://axe.vue-a11y.com/guide/#nuxt-js

Create a client side plugin, vue-axe.client.ts
import vueAxe from 'vue-axe';

export default defineNuxtPlugin(nuxtApp => {
    nuxtApp.vueApp.use(vueAxe);
});

Then you need to render the popup component somewhere in your app for it to display, in a root component or a layout perhaps?
<script setup lang="ts">
import { VueAxePopup } from 'vue-axe';
const config = useAppConfig();
const showAxePopup = ref(false);

onMounted(() => {
    showAxePopup.value = config.isDevelopment;
});
</script>

<template>
  /* Other markup excluded... */
  <VueAxePopup v-if="showAxePopup" />
</template>

Make sure you only show it after the component has been mounted, the VueAxePopup causes an error when server-side rendered.

When using Nuxt with Vite, this can be further simplefied inside the component:
<script setup lang="ts">
  import { VueAxePopup } from 'vue-axe';
  const showAxePopup = import.meta.env.DEV
</script>

<template>
  <VueAxePopup v-if="showAxePopup" />
</template>

I was able to get it to load, but I was not able to get nuxt to ignore it for production builds. Anyone got any ideas for that? I've tried adding it as a dev-only component, and it didn't load the component, but something is still requiring that the package is added to the bundle.

@Djeisen642
For me it's really just defining the Plugin and then adding it as described in my last comment (which I just edited, since I needed to move it into the onMounted()).
// plugins/axe.client.ts
export default defineNuxtPlugin((nuxtApp) => {
  nuxtApp.vueApp.use(VueAxe);
});

If that does not work for you, try logging your import.meta.env.MODE - I would expect it to be different on a nuxt dev vs nuxt build (=productive environment).

The following setup works fine for me for only included it in development:
nuxt.config.ts :
export default defineNuxtConfig({
  vite: {
    optimizeDeps: { include: [...(process.env.NODE_ENV === 'development' ? ['axe-core'] : [])] },
  },
  runtimeConfig: {
    public: {
      env: process.env.NODE_ENV || 'development',
    }
  },
})

plugins/a11y.client.ts :
import { useRawConfig } from '~/frontend/composables/useConfig'

export default defineNuxtPlugin(async (nuxtApp) => {
  const config = useRuntimeConfig()
  if (config.public.env === 'development') {
    // @ts-ignore import has no types
    const VueAxe = await import('vue-axe')
    nuxtApp.vueApp.component('VueAxePopup', VueAxe.VueAxePopup)
    nuxtApp.vueApp.use(VueAxe.default, {
      auto: false,
    })
  } else {
    nuxtApp.vueApp.component('VueAxePopup', h('div'))
  }
})

package.json :
{
  "devDependencies": {
    "axe-core": "4.7.2",
    "nuxt": "3.6.5",
    "vue-axe": "3.1.2",
  },
}


Research of Google Extensions:  Integration Custom Tab to Developer Tools

Research of Google Extensions:

Integration custom tab to dev tools in Google Chrome Browser

Could I ask to provide a link to repo with your experiments ?


Markdown-it customization

Hi there,
Is your feature request related to a problem? Please describe.
I want to add the "breaks": true rule to markdown-it, but can't find a proper way to customize the configuration. Am I missing something?
I found https://github.com/slidevjs/slidev/blob/e165df0889e030225a2ff065f8d2c8f7006ef0c2/packages/slidev/node/plugins/markdown.ts#L84 but it seems that mdOptions are directly herited from the cli call, and be passed at this point.
Describe the solution you'd like
In case we can't easily customize markdown-it, and as it's a preset built-in plugin, I suggest we rely on a dedicated file: setup/mardown-it.ts  (i.e. like setup/windicss.ts) that allow to add custom configuration to makrdown-it, like plugins to use or custom setup in options.
Thanks!

Yeah, we could have it. Open for PR, thanks

https://sli.dev/custom/config-vite.html#configure-internal-plugins

Woo-hoo, awesome ✨! Thanks 🙏


Pokestops missing

I'm noticing a lot of stops that are still there in the game to be suddenly missing from the app. A delete and reinstall did nothing.

Nah, there is one right by my work and I have the app up on my computer all day. Whether I reload it or let it sit there for an hour, the stop never shows up. But it's there on Pokemon GO.

Also to note that when I first started using this program, the stop showed up. It has just disappeared some time over the last few updates. Same goes for the 15 or so stops in a park I frequent.

Downloaded an older version of the app and the stops are still there. Not sure how that happens in the new versions.


show all schools in the my school menu

I think you will need some additional changes, due to how the wider logic works.
If an existing school admin is added to a new school, then we call add_cluster_school via ClusterAdminsController which adds that new school. But the users existing school won't be in the list initially.
If a user switches school then we call add_cluster_school via SchoolSwitcherController which adds their existing school to the list, then updates their direct association.
So the list of schools returned via .cluster_schools won't necessarily be complete. Also need to make sure that the users original school is in there.
It works like this because the menu was originally "Switch schools" and showed the list of schools you could switch to, rather than all those that you are an admin of.
(Longer term the whole auth mechanism here needs updating)
Also need some revisions to existing specs around this.


Make Pregel work on OneShard databases without issues

Scope & Purpose
Pregel can work without issues on a OneShard database, so don't error out in that case.
Enterprise part with a test: https://github.com/arangodb/enterprise/pull/1350

For this PR to pass tests, a fix for OneShard databases developed by @hkernbach is currently necessary


How do you tackle "RequestsLimitError" ?

Hi guys !
I need to make a huge number of request to instagram but I am tackling the issue "RequestsLimitError".
I have used some proxy referenced on https://free-proxy-list.net/ but it doesn't seem to change anything, I even think it's worse because those proxy are quite slow...
I still have this limitation "RequestsLimitError". Is there any restriction regarding instagram account you use ? Should I create multiple account and rotate between them ?
Looking for your recommendations 😊
Thanks

Hey could you say how you use proxy? I try like:
var session = new Client.Session(device, storage, 'username', 'password', 'http://<IP_ADDRESS>:40973/');
but here is mistake:
/Users/Sparrow/Documents/mamp/igparserjs/node_modules/instagram-private-api/client/v1/session.js:65
            throw new Error("`proxyUrl` argument is not an valid url")
            ^

Error: `proxyUrl` argument is not an valid url
    at Session.set (/Users/Sparrow/Documents/mamp/igparserjs/node_modules/instagram-private-api/client/v1/session.js:65:19)
    at new Session (/Users/Sparrow/Documents/mamp/igparserjs/node_modules/instagram-private-api/client/v1/session.js:13:23)
    at Object.<anonymous> (/Users/Sparrow/Documents/mamp/igparserjs/node_modules/instagram-private-api/index.js:18:15)
    at Module._compile (internal/modules/cjs/loader.js:722:30)
    at Object.Module._extensions..js (internal/modules/cjs/loader.js:733:10)
    at Module.load (internal/modules/cjs/loader.js:620:32)
    at tryModuleLoad (internal/modules/cjs/loader.js:560:12)
    at Function.Module._load (internal/modules/cjs/loader.js:552:3)
    at Function.Module.runMain (internal/modules/cjs/loader.js:775:12)
    at startup (internal/bootstrap/node.js:300:19)
    at bootstrapNodeJSCore (internal/bootstrap/node.js:826:3)
MacBook-Pro-Sparrow:instagram-private-api Sparrow$

Same here. Anyone have a solution?

@Miracle-Inc I use it like this :
Client.Session.create(device, storage, auth.username, auth.password, currentProxy)
Note the "Client.Session.create" use of create method 😉

@Nisthar did you figure out anything ?

@charlyBerthet Not yet, I am currently using proxy. But its not working for me well.

@Nisthar yes same, I use public proxy found dynamically on "https://free-proxy-list.net" but guess many people do the same so they are kinda overwhelmed and give directly Requestlimitation. What proxy do you use ? Private or public ?


NRE when changing planet in the drop down menu.

Whenever a new planet is selected, this happens:
_[EXC 18:41:33.852] NullReferenceException: Object reference not set to an instance of an object
TransferWindowPlanner.TWPWindow.HideAngles ()
TransferWindowPlanner.TWPWindow.ddlDestination_OnSelectionChanged (KSPPluginFramework.DropDownList sender, Int32 OldIndex, Int32 NewIndex)
KSPPluginFramework.MonoBehaviourWindowPlus+DropDownList.DrawBlockingSelector ()
KSPPluginFramework.MonoBehaviourWindowPlus+DropDownListManager.DrawBlockingSelectors ()
KSPPluginFramework.MonoBehaviourWindowPlus.DrawWindowPre (Int32 id)
KSPPluginFramework.MonoBehaviourWindow.DrawWindowInternal (Int32 id)
UnityEngine.GUILayout+LayoutedWindow.DoWindow (Int32 windowID)
UnityEngine.GUI.CallWindowDelegate (UnityEngine.WindowFunction func, Int32 id, Int32 instanceID, UnityEngine.GUISkin skin, Int32 forceRect, Single width, Single height, UnityEngine.GUIStyle style)
[LOG 18:41:39.487] 6/12/2019 6:41:39 PM,TransferWindowPlanner,Generating DeltaV Values
[LOG 18:41:39.751] 6/12/2019 6:41:39 PM,TransferWindowPlanner,Working out Log Values to determine DeltaV->Color Mapping
[LOG 18:41:39.753] 6/12/2019 6:41:39 PM,TransferWindowPlanner,Generating DeltaV Color Palette
[LOG 18:41:39.753] 6/12/2019 6:41:39 PM,TransferWindowPlanner,Placing ColorIndexes in array
[LOG 18:41:39.756] 6/12/2019 6:41:39 PM,TransferWindowPlanner,5.7695243002966
[LOG 18:41:39.756] 6/12/2019 6:41:39 PM,TransferWindowPlanner,ERROR: Background Worker Failed
Object reference not set to an instance of an object
at TransferWindowPlanner.TWPWindow.SetTransferDetails () [0x00000] in :0
at TransferWindowPlanner.TWPWindow.bw_GeneratePorkchop (System.Object sender, System.ComponentModel.DoWorkEventArgs e) [0x00000] in :0
[LOG 18:41:39.818] 6/12/2019 6:41:39 PM,TransferWindowPlanner,Placing Colors on texture-292x292
KSP.log

Ive been unable to repro this one or see where it could be dgoing wrong in the logged methods, if you can give me some more detail on the steps, or specific planets your changing from to I could have a look


Inconsitency of inclusion of ModelicaUtilities.h

In the Library include files we find includes like this
#include "ModelicaUtilities.h"
#include "../src/include/CompatibilityDefs.h"

in cas of MSVC. But ModelicaUtilities.h is located in same directory as CompatibilityDefs.h is. Thus ModelicaUtilities.h is not found without explicitly adding src/include to the include path. Thus I propose to get either rid of the extra src/include directory by adding all files to the main Include directory. Or alternetively the include directives should be made consistent.

The idea with ModelicaUtilities.h was following:

If simulated using a tool (in this case Dymola) it should take the ModelicaUtilities.h that ships with the tool
if using the file for C-only unit tests without Modelica it should use the file in ../src/include/ which is achieved by adding that path to the include paths when compiling for C-only unit tests.

I was not sure to that time  whether one may always assume that they are equal  and wanted to be on the "safe" side. However, I guess they should be and agree that it should be moved as suggested.

Yes, ModelicaUtlities.h should be tool-independent.


Merchant Grid View

Reason:
This is for local merchants to accept Seeds and to act like a point of sales system.
With this first iteration merchants can edit items and create invoices.
It should allow add, edit,

Please provide proper information on what is going on here.
Why do we need this, Who is this for, How do we test it, ETC

Its for POS system. Should eventually work on iPad and also store data on firebase.
This is experimental to get some experience with it at Parq and in Tulum - so it's leading to a full merchant feature


nn

Summary

Motivation

Describe alternatives you've considered

Additional context

Thanks for reaching out!
We require the template to be filled out on all new issues and pull requests. We do this so that we can be certain we have all the information we need to address your submission efficiently. This allows the maintainers to spend more time fixing bugs, implementing enhancements, and reviewing and merging pull requests.
Thanks for understanding and meeting us half way :grinning:

Username for 'https://github.com':


Create a video walkthrough of the Website

The mentee should have good video editing skills and can add stock music

Can I work on it under JWOC

Ok

I have done almost part made video walkthrough for pc and mobile.
Can I submit it by tomorrow because exam .

Sure ATB

done

have sent a pr right now


ARCHIVE FAILED: RNPermissionHandlerPhotoLibrary.m

Bug report
After migration from 2.x.x to 3.0.0 we cannot archive the application
Summary
** ARCHIVE FAILED **
The following build commands failed:
	CompileC /Users/vagrant/Library/Developer/Xcode/DerivedData/PandaDoc-flxcxbsvmgvolqfjkaxiltkutahi/Build/Intermediates.noindex/ArchiveIntermediates/PandaDoc/IntermediateBuildFilesPath/Pods.build/Release-iphoneos/Permission-PhotoLibrary.build/Objects-normal/armv7/RNPermissionHandlerPhotoLibrary.o /Users/vagrant/git/node_modules/react-native-permissions/ios/PhotoLibrary/RNPermissionHandlerPhotoLibrary.m normal armv7 objective-c com.apple.compilers.llvm.clang.1_0.compiler
(1 failure)
Archive failed, error: exit status 65

Environment info
react-native info output:
System:
    OS: macOS 11.0.1
    CPU: (8) x64 Intel(R) Core(TM) i5-8257U CPU @ 1.40GHz
    Memory: 386.74 MB / 16.00 GB
    Shell: 3.1.2 - /usr/local/bin/fish
  Binaries:
    Node: 12.16.2 - /usr/local/bin/node
    Yarn: 1.22.10 - /usr/local/bin/yarn
    npm: 6.14.8 - /usr/local/bin/npm
    Watchman: 4.9.0 - /usr/local/bin/watchman
  Managers:
    CocoaPods: 1.10.1 - /usr/local/bin/pod
  SDKs:
    iOS SDK:
      Platforms: iOS 14.2, DriverKit 20.0, macOS 11.0, tvOS 14.2, watchOS 7.1
    Android SDK: Not Found
  IDEs:
    Android Studio: Not Found
    Xcode: 12.2/12B45b - /usr/bin/xcodebuild
  Languages:
    Java: Not Found
    Python: 2.7.16 - /usr/bin/python
  npmPackages:
    @react-native-community/cli: Not Found
    react: 16.14.0 => 16.14.0
    react-native: 0.63.4 => 0.63.4
    react-native-macos: Not Found
  npmGlobalPackages:
    *react-native*: Not Found

Library version: 3.0.0
Steps to reproduce

Install module
Update pods

+permissions_path = '../node_modules/react-native-permissions/ios'
+pod 'Permission-Camera', :path => "#{permissions_path}/Camera"
+pod 'Permission-PhotoLibrary', :path => "#{permissions_path}/PhotoLibrary"

update Info.plist
Try to archive app

Describe what you expected to happen:

build without errors

Reproducible sample code
...

Sorry (
That issue fixed, (just changed xCode to 12.x on CI)

Sorry (
That issue fixed, (just changed xCode to 12.x on CI)


Angular 8

IMPORTANT: Please provide a sample using: http://plnkr.co/edit/lV7zsw7Yqossgs9JOfQU?p=preview

I'm submitting a ...
[ x] bug report
[ ] feature request
[ ] question about the decisions made in the repository

Do you want to request a feature or report a bug?
Both. so i'm wondering if ngx-modialog still works with angular 8. i attempted to get it working, everything seems to be working find except one issue that i'm facing.

What is the current behavior?
NullInjectorError: No provider for Modal$$1!
a NullInjectorError due to the dependency injection not working properly in the constructor declaration of modal (public modal: Modal)

If the current behavior is a bug, please provide the steps to reproduce and if possible a minimal demo of the problem via
https://plnkr.co or similar (you can use one of the templates above as a starting point).

set up ngx-modialog
declare public modal: Modal in the constructor of the desired component
ng serve

What is the expected behavior?
the code will compile with no errors

Please tell us about your environment:

Angular version: angular 8
Browser: [Chrome 78.0.3904.108]

It works with Angular 8.

I have the same problem, what is the solution?


odd turns when attempting to generate dubins paths

I am working with some custom swath generation to work with a field shape that is of interest to us.

And when generating curves, I am getting really odd curves from these sorted swaths at the top, but not at the bottom.

I know I am not providing much detail right now, but are there some general things I might want to look at in terms of this behavior?
I am just not really sure why the turns at one end double back on themselves yet.

Hi @vinnnyr,
Check the size of your F2CSwaths. I have the feeling that for each swath you are generating 2 F2CSwath in opposite directions. So the robot goes to one end and turns there to come back.

Yes I was :face_exhaling:  Thanks so much for helping me catch that!

Btw, cool job. That seems impressive :)


Replace ERB with Liquid for templating

Want to back this issue? Place a bounty on it! We accept bounties via Bountysource.

https://github.com/factor-io/factor/commit/8c97d73fef1d4cdea47263f55afed207ba8bcb17


You don't think this is better?

name: Stats Backend Deploy
on: [push]
jobs:
  build-and-deploy:
    runs-on: ubuntu-latest

    strategy:
      matrix:
        node-version: [18.x]

    steps:
      - name: Check out repository code
        uses: actions/checkout@v4

      - name: Use Node.js ${{ matrix.node-version }}
        uses: actions/setup-node@v3
        with:
          node-version: ${{ matrix.node-version }}
          cache: yarn
          cache-dependency-path: ./yarn.lock

      - run: yarn global add turbo

      - run: yarn prune:back-stats

      - run: |
          cp ./captain-definition ./out
          cp ./Dockerfile ./out

      - run: |
          cd out
          tar -czvf ./deploy.tar ./*
          mv ./deploy.tar ../

      - uses: caprover/deploy-from-github@main
        with:
          server: '${{ secrets.CAP_SERVER }}'
          app: '${{ secrets.BACK_STATS_APP }}'
          token: '${{ secrets.BACK_STATS_TOKEN }}'

Could you be more specific?
I see you are using the "out" directory name, but I don't think that's necessary.
I also see you are using Yarn, but that's really just a matter of developer preference.
Also, why deviate from the nomenclature of the secrets?
So, without any other context, I am going to say no, this is absolutely in no way better than what was already provided in the example.

I was just talking about using the tar from linux cli rather then using that action, there is any performance improvement if use the tar action?
everything you said was about using yarn and out dir!

@mohitxskull The tar GH action is cross-platform


StyleGuide - Detach components and directives which are not visible from the view

*To be discussed ... *
I copied several of these from early discussions and will list them as issues for future discussion after ngConf
cc // @mgechev @wardbell

Detach components and directives which are not visible from the view in order to prevent the change detection running over them.
/* RECOMMENDED */
@Directive({
  selector: '[pane]'
})
class Pane {
  @Input() title;
  view: EmbeddedViewRef;
  constructor(private _changeDetectorRef: ChangeDetectorRef) {}
  // Attaches the change detector
  show() {
    this._changeDetectorRef.reattach();
  }
  // Detaches the change detector
  hide() {
    this._changeDetectorRef.detach();
  }
}

Nice one.
I saw it mentioned several times that OnPush should be favored over the default anyway. If this is the case this should be promoted more in the doc examples as well.
Would it make sense to detach when OnPush is set?

Bad idea. Use other techniques. If temporary, turn change detection off. If you wanted to do what is recommended here, *ngIf would do it. I don't think performance optimizations are a matter of style.


ML 3.5: Hierarchical Clustering (V)

Description
📌  Issues for Week 3
Welcome to 'ML' Team, good to see you here
▶ This issue will helps readers in acquiring all the knowledge that one needs to know about ---- Hierarchical Clustering
Implement Support Vector Machine without using any standard ML library like scikit-learn or more.
🔴 To get assigned to this issue, add your Batch Numbers mentioned in the spreadsheet of "Machine Learning", the approach one would follow and choice you prefer (Documentation, Audio, Video). You can go with all three or any number of options you're interested to work on.
✔ Domain : Machine Learning
🔴🟡 Points to Note :

The issues will be assigned on a first come first serve basis, 1 Issue == 1 PR.

"Issue Title" and "PR Title should be the same. Include issue number along with it.

Changes should be made inside the Machine Learning/Machine_Learning/Unsupervised_Machine_Learning Branch.

Follow Contributing Guidelines & Code of Conduct before start Contributing.

This issue is only for 'GWOC' contributors of 'Machine Learning' domain.

**
✅ To be Mentioned while taking the issue :

Full name
Batch Number
GitHub Profile Link
Which type of Contribution you want to make :

[ ] Video
**
Happy Contributing 🚀

All the best. Enjoy your open source journey ahead. 😎
Domain
Machine Learning
Type of Contribution
Video
Code of Conduct

[X] I follow Contributing Guidelines & Code of conduct of this project.

@Isha307 , Can you please assign me this issue.
Batch Number: 2
Serial Number: 260
GitHub Profile Link: https://github.com/ashish-reddy-20-08

one issue at a time @ashish-reddy-20-08

@abhisheks008 , I have completed the PR for the first issue
PR LINK: https://github.com/girlscript/winter-of-contributing/pull/4133

@abhisheks008 , Can you please assign sir , As i have done PR also sir.

okay

@ashish-reddy-20-08 Please wait until we review the PR.

@geeythree @Isha307 @abhisheks008 , Mam and sir others will take up the good issues if we dont comment soon  , right mam and sir, that is the reason i have made the PR soon by completing the first issue and now asking for the second one .
Can you please make it soon mam and sor , as i want to work more and learn new things , if the team helps me by making the merging soon then it would be help me mam and sir.

@ashish-reddy-20-08 issue has already been assigned to you. Also, you need not address us as Sir or Ma'am 😄

@Isha307 , Mam can you please evaluate my PR soon. So that I can work on other issues.
Name: M. Ashish Reddy
Batch - 2
PR LINK : https://github.com/girlscript/winter-of-contributing/pull/4151


Fixing Regex to work with multiple Media Query rules

Fixing Regex to work with multiple Media Query rules, like: "(min-height: 900px) and (min-width: 992px) 150px"

I found this limitation while trying to execute the following code:
<img src="assets/img/ef-logo-200.png" alt="EF Education First"
                sizes="(min-height: 900px) and (min-width: 992px) 150px,(min-width:768px) 100px, 60px"
                srcset="assets/img/ef-logo-60.png 60w,
                      assets/img/ef-logo-100.png 100w,
                      assets/img/ef-logo-120.png 120w,
                      assets/img/ef-logo-200.png 200w" />

I fixed the regular expression but I wasn't sure which branch I should request a pull to, did it for Master but let me know if I should do it for a different branch.
On Chrome (which has HTMLPictureElement) it works fine, but on Firefox for example, the rule (min-height: 900px) and (min-width: 992px) 150px was never working. Now it seems to be fine, I tried to test some other variations and it all seems to be ok now. Please let me know if you find any problem.
Thanks,
Paulo

Yeah, much needed. But... my parser rewrite in the PF3 proposal handles all these cases correctly, and more, and much closer to spec.  So +1 but we ought to think about when we're going to shift energy towards PF3 proposal.

+1 from me as well.
@baloneysandwiches: you, me, @aFarkas, and the rest of the team should block out a little time next week to talk about where things stand on the PF3 proposal. Once you guys are feeling pretty good about a 3.0.0, I want to get some dedicated beta testing volunteers.

We already have something like this in our 2.4 branch. Can you check whether this solves your problem. See: https://github.com/scottjehl/picturefill/blob/2.4/src/picturefill.js#L183

@paulomunoz If you get some time, can you checkout our 2.4 branch, or 3.0-beta to see if the updates are what you're looking for?

I think we can close this.


[NetworkControl] Fix DHCP NAK returining success

Success response was returned on DHCP Not Acknowledge response

https://jenkins.code.rdkcentral.com/job/github-triggered/1308/ started

https://jenkins.code.rdkcentral.com/job/github-triggered/1318/ started


How to properly analyze data and protect it

Answering your question, you have two options:

you may pass the data set through contract invocations, as part of the requests. Then the contract may store it in the contract state, or a different one.
you may build a state offline with your dataset, and then pass the state root through a contract invocation. Then the contract can use the state root to access the state -- provided that it's available locally, usually through the storage service (sservice).

Originally posted by @bvavala in https://github.com/hyperledger-labs/private-data-objects/issues/337#issuecomment-958109694

Hello, my original vision is: based on PDO and smart contracts running in the Contract Enclave interpreter, data can be mined by executing algorithms to generate value, but at the same time, data can be guaranteed not to be leaked to algorithm creators .
I originally planned to imitate the example of fibonacci to write the task of pulling data (from sservice via HTTP/GET) and the algorithm into the contract, but it is quite a pity that I have tried many solutions but still have not succeeded: including
(1) Use c++ to call python, but the cross-compilation fails.
(2) Link libcurl.a, but failed to compile libcur into wasm library.
(3) And use the net/sys interface provided in wasi-sdk to implement HTTP/GET. It can be linked to wasm, but it crashes when it runs. It either reports "failed to create the wasm execution environment" or "wasm exception = Exception: failed to call unlinked import function (wasi_snapshot_preview1, fd_prestat_get)".
I think it would be difficult for me to continue trying. Unless there are new and better ideas worth trying. Thank you for your help during your busy schedule.

I solved it suddenly, incredible.


No icon + color in "Show All Tasks" picker

Testing https://github.com/microsoft/vscode/issues/153377
I added a color to the "Core - Build" task in this repo. The color shows correctly for the "Run Task" picker, but when I choose "Show All Tasks..." then no icons show.

do you mean the Show running tasks quickpick? did not know that existed


Add missing actions from desktop app to the task's contextual menu

:speaking_head: Foreword
In a desktop app there are four buttons in the right hand side of every task: Start tracking time, Remove from My Day, Mark as done/undone, Show/Hide additional info. In a mobile app there is no space for these buttons. "Mark as done" action is controlled by swiping the task to the right direction.
:person_frowning: Problem Statement
Swiping is a bit tricky: user needs to guess this hidden feature, because almost nobody reads the documentation. In addition, it doesn't always work as expected. Sometimes swiping doesn't do anything and the user needs to swipe again and again to trigger the expected action (I will create a bug report for this issue when I am able to record a screencast). It would be great to have also an alternative way of triggering at least for "mark as done" action.
:grey_question: Possible Solution
Adding the mentioned actions to the task's contextual menu would be a good solution I think.

:arrow_heading_up: Describe alternatives you've considered
I considered adding a hint to the app header, but I think it could be visually distracting.
Maybe double tapping could work?
:heavy_plus_sign: Additional context
Function to complete tasks missing? #2576

This should be easy to implement. We need to check if there is enough space though for all the actions in extreme cases with lots of entries (e.g. issue tasks).


Map API step 4

This is the completed map API for week 4. I canceled the merges so I'm not sure why they are commits.

Yeah, there's some funkiness with the merge commits. We want to start a new branch from a brand new synced master branch, so ideally your feature branch would have no merge commits.


Preview of changes, for discussion

This is work in progress, needs BIG cleanup. It's here only for discussion
Rough overview of changes:

use struct printf_state to pass around % specifier info.
Mostly it seems to work fine and reduces register pressure, but sometimes it's a bit cumbersome. I'll look into it further.

use struct out_base to handle out callback.
Personally I prefer this way of handling callbacks - parameters can be easily added, out_base can be included in any other object (for example serial driver struct) etc.
In case of out function it is a bit overkill - it is undesirable to include printf_state in out_base (a lot of compiler optimizations would be disabled) and it is possible to store both function pointer and argument in state.

change idx to be uintptr_t and allow starting from any value. idx is incremented on each output, so it is desirable to have it in register. With uintptr_t it is possible to store output buffer pointer directly, saving some instruction, with almost zero penalty (for example it is impossible to write last byte of memory)

test idx < maxidx before output function call
better caching of values in register, simpler output functions

add FLAGS_ZERO
I hope it's consistent with rest of flags

unify _ntoa_long and _ntoa_long_long using macro (only parameter type differs)
Removes code duplication at cost of more complicated debugging

implement (almost?) correct rounding for %f
fmsub is relatively simple, some FPUs (Cortex-M4) support equivalent instruction

fix %g trailing zeroes
The implementation isn't as elegant as I'd like, it may be conditional or maybe there is better way to rewrite it

I encountered interesting unspecified  case:
(unsigned int)-INT_MIN (and (unsigned int)(0 - value) in current code)  is causing integer overflow, which is unspecified behavior in C standard. gcc was able to use this to sign-extend long first, producing incorrect result for printf("%d", INT_MIN);

move zero-termination from _vsnprintf to snprintf
fixes printf("%c", 0);, is cleaner (only string needs termination), simplifies code a little

uintptr_t idx removes need to special-case sprintf(NULL, ...) case, just pass 0 as limit

added new test cases from other sources (mainly libsanity), discovering some  bugs (mostly opened as issues).
modified tests are just temporary hack to get testing quickly, I'll clean it up

Hello @ledvinap ,
this is really a bunch of bugs you found.
I'll review them and create a next version branch to get your updates merged in.
Thanks a lot for your outstanding work!

@mpaland : Hi!
Most of bugs are from libinsanity tests. Mostly easy to fix. The rest of the code needs some polishing, still thinking about how to do it correctly.
I'll prepare some parts as separate PRs, fixing bugs and sometimes improving existing code. Then it can be decided about more philosophical changes ...
Can you please overview list of changes and do a quick comment about them (OK/don't like/not sure/needs further work)?

When I can expect to pull these commits into main branch and have new release?


TablePager: fix showing wrong info-text when filter returns no rows

changed Info to show '0-0 of 0' instead of '1-0 of 0' when filter does not match anything
added Viewer and bUnit-tests

As it can be tested here: https://dev.mudblazor.com/components/table#table-with-pagination-and-filtering
When filtering for something not matching anything, the info-label will show 1-0 of 0 which should be 0-0 of 0.

Workaround

create a ref for the table
set the InfoFormat-parameter of the MudTablePager component
check whether the GetFilteredItemsCount() of _mudTable equals 0 or not and use string depending on that

<MudTable T="SomeDto" @ref="@(_mudTable)" @* some more params *@ >
  @* some more markup *@
  <PagerContent>
    <MudTablePager InfoFormat="@(_mudTable.GetFilteredItemsCount() == 0 ? "0-{last_item} of {all_items}" : "{first_item}-{last_item} of {all_items}")" />
  </PagerContent>
</MudTable>

@code{
  privat class SomeDto { }

  privat MudTable<SomeDto> _mudTable;
}

Thank you!


Incorrect behavior of in-place operation between ndarrays share same memory space

Incorrect behavior of in-place operation between ndarrays which

share same memory space and
have different strides.

Test code:
import numpy
import cupy

for xp in (numpy, cupy):
    x = xp.arange(10000).reshape(100, 100)
    x += x.T
    print(xp.__name__, ':', x.sum())  # Expected: 99990000 = 10000 * (10000 - 1)

Output:
numpy : 99990000
cupy : 100026222

Looks like this is a different manifestation of the same problem #2405?

To resolve this issue, we need to support cupy.shares_memory or cupy.may_share_memory and copy inputs before the operation.


Remove unused SQL query ListProfilesByProjectID

Also delete tests and custom methods around its return type.
Summary
Provide a brief overview of the changes and the issue being addressed.
Explain the rationale and any background necessary for understanding the changes.
List dependencies required by this change, if any.
Fixes #(related issue)
Change Type
Mark the type of change your PR introduces:

[ ] Bug fix (resolves an issue without affecting existing features)
[ ] Feature (adds new functionality without breaking changes)
[ ] Breaking change (may impact existing functionalities or require documentation updates)
[ ] Documentation (updates or additions to documentation)
[x] Refactoring or test improvements (no bug fixes or new functionality)

Testing
Outline how the changes were tested, including steps to reproduce and any relevant configurations.
Attach screenshots if helpful.
Review Checklist:

[x] Reviewed my own code for quality and clarity.
[ ] Added comments to complex or tricky code sections.
[ ] Updated any affected documentation.
[ ] Included tests that validate the fix or feature.
[ ] Checked that related changes are merged.

coverage: 54.299%. remained the same
when pulling 2107f8953bb1511317bf48c1f15dcaa9a799b943 on delete-unused-profile-query
into 9d5b603a78d105c3672e0f9b495654ea6df9f6fc on main.


[master] fixes #1831 - replace update with lateupdate in FixedAngularSize

Overview
FixedAngularSize was previously using Update to adjust the element's visible size. This caused a flicker when applied to a cursor and this update came before the update performed by the GazeManager (as reported in #1831).
Changes

Replaced Update with LateUpdate to ensure the scaling occurs after other modifications.
Encapsulated the SizeRatio field per the pending code style guide and applied a range check in the setter.
Add a range to SizeRatio (was defined as 0 - 1 in SetSizeRatio())
Add summary tag comments throughout.

Fixes: #1831 .

Should we be fixing this in dev as well?

@StephenHodgson, yes. I plan on fixing this and other updates taken to master in dev this week.


Remove jQuery dependency

This pull request removes the jQuery dependency from Cornerstone.

It introduces a pubSub function to do all the event handling
Uses ES6 Promise instead of jQuery Deferred
Fixes some tests that were useless before (cause of some never called asserts)

All tests succeed and all examples run without errors.
Examples still use jQuery for DOM stuff.
A little breaking change (just thought that it makes more sense):

CornerstoneImageCacheFull is now fired before the image cache was cleaned up and is full
CornerstoneImageCacheCleaned is a new event fired after the image cache was cleaned up when full

Upcoming soon ... make CornerstoneTools jQuery free :-)

Coverage increased (+1.9%) to 55.236% when pulling c71bd23f096e20315703b09e8acd3e4d8efe005c on medihack:remove-jquery-dep into 126040a20fd31e73fa757bc1f3c17897327b29d3 on chafey:master.

Coverage increased (+2.009%) to 55.312% when pulling 80ab9541347cdb65fbe130f6d0fe968dfa74997e on medihack:remove-jquery-dep into 126040a20fd31e73fa757bc1f3c17897327b29d3 on chafey:master.

Ouch, CornerstoneTools is so hardly bound to jQuery stuff, it will take a while to figure things out. I am not even completely sure if this makes sense to free Cornerstone from its jQuery dependency. Any opinions?

One step at a time, right? I think I remember seeing a discussion a while back where @swederik mentioned cornerstone would need to be updated to be free of the dependency before cornerstoneTools could be the slow process of converting tools over one-by-one.
An intermediary solution could be simply duplicating the events emitted using a more vanilla approach, migrating the tools to use the new events, and then yank jQuery support from cornerstone?
I'd argue it's worth doing. jQuery is a large library that is a pain to keep as a dependency. Especially in the global scope.

The problem I see is that CornerstoneTools is so strongly event driven. I really like the architecture of Cornerstone and dicomParser, but in my opinion CornerstoneTools is really hard to test and maintain. That's why I started a discussion about the future of CornerstoneTools at Google groups and I am really interested in your opinion: https://groups.google.com/forum/#!topic/cornerstone-platform/fqxZTKiUUHo


enha: canary miner: log txs before executing

PR Type
Enhancement

Description

Enhanced logging in the fake miner mode of the Importer
Added a tracing::info! log statement to record transaction details before execution
Improves visibility and debugging capabilities for transactions processed by the fake miner
This change helps in tracking and analyzing transaction flow in the fake miner scenario

Changes walkthrough 📝
Relevant filesEnhancement

importer.rsAdd transaction logging in fake miner mode

src/eth/follower/importer/importer.rs
Added logging for transaction execution in fake miner mode  Logs transaction details before execution using tracing::info!

+2/-0

💡 PR-Agent usage: Comment /help "your question" on any pull request to receive relevant information

Final benchmark:
RPS Stats: Max: 983.00, Min: 614.00, Avg: 818.47, StdDev: 67.82
TPS Stats: Max: 983.00, Min: 589.00, Avg: 818.14, StdDev: 68.89
Plot: View Plot

Final benchmark:
RPS Stats: Max: 1020.00, Min: 523.00, Avg: 893.63, StdDev: 55.01
TPS Stats: Max: 1002.00, Min: 598.00, Avg: 799.92, StdDev: 106.74
Plot: View Plot

Final benchmark:
Run ID: bench-2293173580
Git Info:

Branch: main
Commit: d6227aa
PR: #1858

Configuration:

Target Account Strategy: Default (Random)

RPS Stats: Max: 1097.00, Min: 612.00, Avg: 875.67, StdDev: 59.38
TPS Stats: Max: 1037.00, Min: 734.00, Avg: 849.00, StdDev: 71.34
Plot: View Plot


CI: Add ci scripts that execute tests on SemaphoreCI

This commit adds the CI scripts that will execute the functional
and integration tests from the clearcontainers/tests repository.
Signed-off-by: Salvador Fuentes<EMAIL_ADDRESS>

Coverage decreased (-0.3%) to 74.445% when pulling d9525c1a264260528fb21fde8748aa045b31d334 on chavafg:topic/add_ci into 0aa922e0bc8d9cb0c1558f39767350cba6ffec2c on clearcontainers:master.

Coverage decreased (-0.3%) to 74.445% when pulling d9525c1a264260528fb21fde8748aa045b31d334 on chavafg:topic/add_ci into 0aa922e0bc8d9cb0c1558f39767350cba6ffec2c on clearcontainers:master.

lgtm

lgtm


Schema missing from URL field

https://github.com/Azure/Azure-Sentinel/blob/156b3e8bb2abdcc554f0cf7df205c93aa3e8fb8f/Parsers/ASimWebSession/Parsers/vimWebSessionPaloAltoCEF.yaml#L217
Hi,
According to HTTP session fields specification the schema (http, https, ...) is mandatory but seems to be not logged on Palo Alto logs.
That's an issue that can be resolved on the ASIM parser level?
What I am trying to do is use the parse_url function to break down the URL to path, query, ... but the function does not work without the schema.
Thanks,
Christos.

Hi @cgiampoulakis : the documentation is not consistent on this issue: if you go through the web session schema doc, you will find that there are opportunities in which you will not get a schema, and the schema is therefore not mandatory.  The reason is that in many cases it is not provided, and not even deducible from other fields, apart from guessing based on the port number. We therefore decided not to make it mandatory. I will update the docs accordingly.


Provides support to use custom jql

Currently we can only run custom jql from commandline with no support to save and run it anytime. With this PR we can now save this jql in config.json and use it by calling "jira jql -c keyname" where keyname is the keyname we've used to store our jql against in config.json.

Hi @palashkulsh! Thanks for your PR. I will take a look about how this works first :)

Hi @germanrcuriel I've also added support to conditionally check the resolution while marking issues as done. This was the problem being faced by many people here. Thanks for your time :)

@palashkulsh I just saw the new commits! Thank you very much, I will test everything this weekend :)

seems like I messed up creating PR from master, now all changes which I have in pipeline to raise a PR would be available. may be I'll re raise the PR with required changes.


What's needed for new release

Can you please make a list so I can create pull requests and we get a new release on pypi ?

Sure, I am going to put in a bit of work on this project over the weekend; it's been a bit neglected by my lately.  I want to put in something to make working with json.facet easier and fix SolrResponse a bit.
Can you add unit tests to verify the routing is actually working correctly? I haven't looked at all the code you submitted, especially the stuff going over in the aware router. This is needed so when someone else (or me) is working on changing the transport, we can test to make sure we didn't break any of the routing functionality. Some documentation so people can figure out how to use it would be good too.
Also, on SolrVagrant I am running into issue with indexq tests on solr 5.2.1, it looks like it is a solr 5.2.1 config issue.

I updated dev / master and pushed a new release (0.2.0) to pypi because of the previous bug with index method on the client, however I disabled your routing module.
I would really like to include it next time, but I want unit tests for it to prevent breakage in the future. Having even a single unit test for the index method would definitely have identified the infinite loop and would have prevented it from being merged into the code and pushed out to pypi. I added that unit test now.
Please add the tests for the router functionality, they can be totally independent from the rest of the tests to not mix things up. One thing you can do is to set up an extra devel argument that you can pass to the object with you instantiate it, then have it check if the flag is on and store some extra information about usage that you can later verify. Not a perfect solution, but something that would be a decent workaround.
Let me know what you think.
Thanks


 Can you fetch them from a browser and via wget? Could be a network / proxy issue?
Can you please post questions on Stack Overflow and tag them react-native and only create a Github issue in case there's a bug? http://stackoverflow.com/questions/tagged/react-native


visibleCells is empty in iOS 12

visibleCells is empty in iOS 12
I guess you didn't override visibleCells to cause this situation.
When used IBPCollectionViewCompositionalLayout in iOS12, it will create some IBUICollectionView as section to my collectionView, by this time the cell is not belong to my collectionView, but belong to IBUICollectionView.

Experiencing the exact same issue as well!

Is this also causing the crash on i.e. viewWillTransition?


How to redefine buildEditor method?

I write Trumbowyg component for vue.js. And i need redefine method buildEditor for attach vue model for two way binding.
How best way to do it?

Why do you need to change buildEditor exactly?

Need for add two way binding for textarea.

So base your code on the textarea, then launch Trumbowyg on this textarea and you will be able to have 2 way binding work like a charm :)
If you want, there are Angular and Angular 2 directives:
https://github.com/lizardK/trumbowyg-ng
https://github.com/MuhammedHasan/ng2-trumbowyg

I stay here but want to close to keep issues clean :)


[firebase_auth] add custom claims to example app

Description
Adds custom claims example in example app. Also it uses the claims to for the IdTokenResult fields.
Checklist
Before you create this PR confirm that it meets all requirements listed below by checking the relevant checkboxes ([x]). This will ensure a smooth and quick review process.

[X] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[X] My PR includes unit or integration tests for all changed/updated/fixed behaviors (See Contributor Guide).
[X] All existing and new tests are passing.
[X] I updated/added relevant documentation (doc comments with ///).
[X] The analyzer (flutter analyze) does not report any problems on my PR.
[X] I read and followed the Flutter Style Guide.
[X] The title of the PR starts with the name of the plugin surrounded by square brackets, e.g. [shared_preferences]
[x] I updated pubspec.yaml with an appropriate new version according to the pub versioning philosophy.
[x] I updated CHANGELOG.md to add a description of the change.
[x] I signed the CLA.
[x] I am willing to follow-up on review comments in a timely manner.

Breaking Change
Does your PR require plugin users to manually update their apps to accommodate your change?

[ ] Yes, this is a breaking change (please indicate a breaking change in CHANGELOG.md and increment major revision).
[x] No, this is not a breaking change.

The approach take here is actually the approach taken by the SDK itself.  Why pass the data twice since we already have it there? Bellow you can see the Android implementation.
@PublicApi
public class GetTokenResult {
  private String zza;
  private Map<String, Object> zzb;

  @KeepForSdk
  public GetTokenResult(String var1, Map<String, Object> var2) {
    this.zza = var1;
    this.zzb = var2;
  }

  @Nullable
  @PublicApi
  public String getToken() {
    return this.zza;
  }

  @PublicApi
  public long getExpirationTimestamp() {
    return this.zza("exp");
  }

  @PublicApi
  public long getAuthTimestamp() {
    return this.zza("auth_time");
  }

  @PublicApi
  public long getIssuedAtTimestamp() {
    return this.zza("iat");
  }

  @Nullable
  @PublicApi
  public String getSignInProvider() {
    Map var1;
    return (var1 = (Map)this.zzb.get("firebase")) != null ? (String)var1.get("sign_in_provider") : null;
  }

  @PublicApi
  public Map<String, Object> getClaims() {
    return this.zzb;
  }

  private final long zza(String var1) {
    Integer var2;
    return (var2 = (Integer)this.zzb.get(var1)) == null ? 0L : var2.longValue();
  }
}

flutterfire has been re-homed to https://github.com/FirebaseExtended/FlutterFire
Please open a PR in https://github.com/FirebaseExtended/FlutterFire
I'm closing this PR.


cleanup(storage): fix IWYU warnings

There are so many IWYU warnings that I cannot see the useful warnings.
It is easier to just add the #includes than to fight the tooling.

This change is

just curious, did you go through these manually (e.g. on upkeep)? or did some tool make the changes?

upkeep, and then used grep + comm to find the files missing things.  The most interesting command was
comm -23 <(git grep -l std::move -- google/cloud/storage/**_test.cc | sort) <(git grep -l '#include <utility>' -- google/cloud/storage/**_test.cc | sort) | xargs sed -i -e '/#include <gmock/a\' -e '#include <utility>'


Document "Unstar all messages in topic" feature

We should document the "Unstar all messages in topic" feature (accessible from the topic menu) on https://zulip.com/help/star-a-message.
Also, this tab should be Desktop/Web, not Desktop:

@zulipbot claim


[BUG:] /Spawn

the /spawn command is closing the server, can you fix it?

I don't have this problem. Please open a bug-report with the bug-report template.
-> Closed due too few information.


Change cancelled workflow icon color

Description
In GitHub Workflows, a cancelled workflow is indicated by a grey circled exclamation mark. So I changed it to be grey instead of red.
Checklist

[x] I read the extension guidelines
[x] I read the documentation about publishing
[x] I ran npm run build and tested this distribution build in Raycast
[x] I checked that files in the assets folder are used by the extension itself
[x] I checked that assets used by the README are placed outside of the metadata folder

FYI: @thomaslombart is on vacation until next week; he will look at this PR when he is back 🙂


rollup: use nodePolyfills

This is necessary to address the following warning from rollup:
(!) Missing shims for Node.js built-ins
Creating a browser bundle that depends on "crypto". You might need to include https://github.com/FredKSchott/rollup-plugin-polyfill-node

Interestingly, this fails E2E tests.
I guess we'll live with the warning then - although it would be nice to suppress it.

we should not need to import crypto at all. Let's find out why that didn't work instead?

I'll keep trying in the meantime, but currently I am out of ideas.


Declarative installation instructions

Thanks for your hard work on these derivations.
Currently, the installation instructions involve an imperative installation (nix-env -iA). I think it would be useful to illustrate how to install these emacs variants in some declarative way (eg. for a nix-darwin configuration, since that's mentioned in the README).
For example, in the past, I have been able to use "The Emacs Overlay" like this: https://github.com/lancelet/dotfiles/blob/e352d1e0e8a81b60514431b29d2fba38efcf3792/nixpkgs/darwin-configuration.nix#L10-L18
That allows me to:

pin a version (so I don't have unexpected breakages, and can revert to a specific earlier version if necessary)
share config between my work and home machines in a single place, without having to remember to do any extra manual steps.

I tried a similar thing with this repository and it failed. Unfortunately, I don't know enough Nix to suggest what it should look like.

Hi @lancelet, you are not the first to request this :)
I will make time for this next week and build a declarative version.
and thanks for the dotfile reference

circling back on this, I'm trying to move my homebrew config to nix Darwin and was excited to see the mention of it in the readme, but I'm clueless as to how I would actually get this installed via nix-darwin and home-manager.


Auth.JS error.

When attempting to run "node auth.js" I got a PKCE warning when accessing the URL.
Error: You must use PKCE when generating the authorization URL to not include a client secret
at DropboxAuth.getAccessTokenFromCode (/home/pi/MagicMirror/modules/MMM-DropboxPictures/node_modules/dropbox/cjs/src/auth.js:379:17)
at /home/pi/MagicMirror/modules/MMM-DropboxPictures/auth.js:33:12
at Layer.handle [as handle_request] (/home/pi/MagicMirror/modules/MMM-DropboxPictures/node_modules/express/lib/router/layer.js:95:5)
at next (/home/pi/MagicMirror/modules/MMM-DropboxPictures/node_modules/express/lib/router/route.js:144:13)
at Route.dispatch (/home/pi/MagicMirror/modules/MMM-DropboxPictures/node_modules/express/lib/router/route.js:114:3)
at Layer.handle [as handle_request] (/home/pi/MagicMirror/modules/MMM-DropboxPictures/node_modules/express/lib/router/layer.js:95:5)
at /home/pi/MagicMirror/modules/MMM-DropboxPictures/node_modules/express/lib/router/index.js:284:15
at Function.process_params (/home/pi/MagicMirror/modules/MMM-DropboxPictures/node_modules/express/lib/router/index.js:346:12)
at next (/home/pi/MagicMirror/modules/MMM-DropboxPictures/node_modules/express/lib/router/index.js:280:10)
at expressInit (/home/pi/MagicMirror/modules/MMM-DropboxPictures/node_modules/express/lib/middleware/init.js:40:5)
Running node.js 22.2.0

Hmmm.. I made some mistake on README about installation. Could you try reinstall?

I got it! I was trying to run this via SSH then accessing it via IP/hostname from another workstation, instead of localhost. Logged in via VNC then ran from terminal, and it worked.
User error.
Also, I copied the template in the readme and modified as needed. I had to remove the ... near the end as it showed a syntax error.
{
module: 'MMM-DropboxPictures',
position: 'top_left', // fullscreen_below is more usual.
//header: "DROPBOX",
config: {
verbose: true,
imageLife: 1000 * 60 * 30, // 30 minutes
directory: "/Photos",
fileNames: [ "DSC", /^IMG/ ],
width: "400px", // For fullscreen, "100%" would be fit.
height: "400px",
...
}
},
After this, I'm back up and running.


Fixed the size

Finally, we got a full-width and full-height terminal in the browser page (specifically in the DOM element) which works smoothly. Before, it was a hack to make it almost full width & height, but now the experience is much, much cooler!
Big thanks to @lcristianiim for letting me know how to do that! :cake:
Bonus, I generated documentation, contributing guides and more using Blah.

@lcristianiim Yeah, that made my day! :smile:


Define a generic is_directed(::Type{<:AbstractDataGraph})

Define a generic is_directed(G::Type{<:AbstractDataGraph}) = is_directed(underlying_graph_type(G)). This would then rely on concrete subtypes implementing underlying_graph_type.
Inspired by https://github.com/mtfishman/ITensorNetworks.jl/pull/9, since it seems like in general we should be able to deduce whether a DataGraph is directed or not from other information.

Closed by #4.


Adjust e2e test run script and fixes

Signed-off-by: ShylajaDevadiga<EMAIL_ADDRESS>Proposed Changes

Update run_tests.sh script to include more tests.
Adjust script to run the tests locally

Verification
NA as it is part of nightly runs

Will this affect the work @dereknola and @nikolaishields are doing on running these tests as part of PR runs?

Will this affect the work @dereknola and @nikolaishields are doing on running these tests as part of PR runs?

No it will not affect running these as part of PR runs

Codecov Report
Base: 9.66% // Head: 9.66% // No change to project coverage :thumbsup:

Coverage data is based on head (884bf2f) compared to base (f0ec6a4).
 Consider uploading reports for the commit cee8d56 to get more accurate results

Additional details and impacted files
@@          Coverage Diff           @@
##           master   #6718   +/-   ##
======================================
  Coverage    9.66%   9.66%
======================================
  Files         139     139
  Lines       10225   10225
======================================
  Hits          988     988
  Misses       9033    9033
  Partials      204     204

Flag
Coverage Δ

unittests
9.66% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

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


[Enhancement] Support metadata query for task

What type of PR is this：

[ ] bug
[ ] feature
[x] enhancement
[ ] others

Which issues of this PR fixes ：

Fixes #6182
Problem Summary(Required) ：

We provide an INFORMATION_SCHEMA.tasks to query the Task created by the user, and another INFORMATION_SCHEMA.task_runs to query the execution status of the task run, including history and just created or normally executed task runs.

run starrocks_fe_unittest

[FE PR Coverage check]
:disappointed: fail : 22 / 62 (35.48%)
file detail

path
covered line
new line
coverage

:large_blue_circle:
com/starrocks/analysis/SchemaTableType.java
0
3
00.00%

:large_blue_circle:
com/starrocks/service/FrontendServiceImpl.java
0
37
00.00%

:large_blue_circle:
com/starrocks/catalog/SchemaTable.java
22
22
100.00%

[FE PR Coverage check]
:disappointed: fail : 22 / 66 (33.33%)
file detail

path
covered line
new line
coverage

:large_blue_circle:
com/starrocks/analysis/SchemaTableType.java
0
3
00.00%

:large_blue_circle:
com/starrocks/service/FrontendServiceImpl.java
0
37
00.00%

:large_blue_circle:
com/starrocks/scheduler/TaskRunStatus.java
0
2
00.00%

:large_blue_circle:
com/starrocks/scheduler/TaskRunExecutor.java
0
1
00.00%

:large_blue_circle:
com/starrocks/scheduler/TaskRun.java
0
1
00.00%

:large_blue_circle:
com/starrocks/catalog/SchemaTable.java
22
22
100.00%

run starrocks_fe_unittest

[FE PR Coverage check]
:disappointed: fail : 24 / 82 (29.27%)
file detail

path
covered line
new line
coverage

:large_blue_circle:
com/starrocks/analysis/SchemaTableType.java
0
2
00.00%

:large_blue_circle:
com/starrocks/service/FrontendServiceImpl.java
0
51
00.00%

:large_blue_circle:
com/starrocks/scheduler/TaskRun.java
0
1
00.00%

:large_blue_circle:
com/starrocks/scheduler/TaskRunStatus.java
1
4
25.00%

:large_blue_circle:
com/starrocks/scheduler/TaskRunExecutor.java
1
2
50.00%

:large_blue_circle:
com/starrocks/catalog/SchemaTable.java
22
22
100.00%

run starrocks_fe_unittest

[FE PR Coverage check]
:disappointed: fail : 39 / 112 (34.82%)
file detail

path
covered line
new line
coverage

:large_blue_circle:
com/starrocks/analysis/SchemaTableType.java
0
2
00.00%

:large_blue_circle:
com/starrocks/service/FrontendServiceImpl.java
0
49
00.00%

:large_blue_circle:
com/starrocks/scheduler/TaskRun.java
0
1
00.00%

:large_blue_circle:
com/starrocks/scheduler/TaskRunStatus.java
1
4
25.00%

:large_blue_circle:
com/starrocks/scheduler/TaskRunManager.java
4
14
28.57%

:large_blue_circle:
com/starrocks/scheduler/TaskRunExecutor.java
1
2
50.00%

:large_blue_circle:
com/starrocks/scheduler/TaskManager.java
10
17
58.82%

:large_blue_circle:
com/starrocks/server/GlobalStateMgr.java
1
1
100.00%

:large_blue_circle:
com/starrocks/catalog/SchemaTable.java
22
22
100.00%