Hide and show main toolbar

Is there a way to programmatically hide and show the main toolbar in the editor?

I'm able to do it via:
$('.te-toolbar-section').css('display', 'none');
but is there an easier way?

@SteveSchreiner Sorry for late reply. Currently there is no easy way to hide and show tool bar. We did not consider this feature, so if you need this feature please make a new issue for supporting api for visibility of toolbar. I will close this issue because this issue so old. Again sorry for late reply.


Question for indexing and slicing tutorial

I've been reading the indexing and slices tutorial, which is very helpful.  Would it be easy to to add a comment to the tutorial on relative efficiency of slicing as opposed to doing one's own indexing into the original matrix (for example)?
Does slicing copy parts of the matrix into a new data structure, or is it in effect just a convenient way to let Owl manage complex indexing schemes, so that the cost of using slicing would be minimal?  Or maybe slicing is potentially more efficient than indexing by hand because more of the process will stay on the OpenBLAS side?  (I don't know whether that question makes sense.)
(I am thinking about ways to optimize some very slow code, and started looking at slicing as one tool that might help me to restructure it.)
Thanks-

@mars0i  thanks for suggesting this. I can add some tips and discussions about the performance in the tutorial. In case you encounter any performance issues or bugs in using Owl's slicing and indexing functions, please always let me know so I can improve it.
Indexing, slicing, and broadcasting are fundamental functions of the core ndarray module. Currently, taking a slice out (get_slice) creates a copy from the original variable; setting a slice (set_slice) on the other hand modifies the the original variable directly. In the future, I might consider to introduce view into Owl to avoid copying data when slicing data.
The performance of slicing really depends on the actual slice definition, because Owl sticks to the c layout, so slicing at low dimension is always much faster than at high dimension. Owl first calculates the minimum continuous block size then tries to use block copying in CBLAS to speed up the operation. Further optimisation on slicing and broadcasting is on my to-do. As you might have noticed, broadcasting has been significantly optimised these two days (opt branch will be merged to master either today or tomorrow).
But yes, please do give it a try to see if slicing can help in improving the performance of your code, and let me know the result. That would be very helpful for me to further improve the slicing function.

Owl sticks to the c layout, so slicing at low dimension is always much faster than at high dimension.
I want to make sure that I understand.  For example, with matrices, if I extract a row using get_slice (or get_slice_simple, that will be faster than if I extract a column, because the internal layout is [row1 row2 ... rowN].  If I extract one column, that means copying N blocks of size 1.

However, with get_slice, the copy of the row (or column) that I operate will be made inside OpenBLAS, so the copying itself will be relatively fast.
I'll try out slicing.  Knowing how the underlying layout affects speed will be very helpful.
I had heard of broadcasting but didn't understand what it was.  I just read about it on the web and in Owl's README and tried it out.  Very cool.  I don't think that the project I'm currently working on can make use of broadcasting, but I will look for ways to use it and see if it speeds things up.

Yes, your understanding is correct. The code below: f slices a row out whilst g slices a column out. On my machine (not very fast), f takes about 0.46ms whilst g takes 1.82ms
let x = Mat.uniform 1000 1000;;
let f () = Mat.get_slice_simple [[99]; []] x;;
let g () = Mat.get_slice_simple [[]; [99]] x;;
Utils.time f;;
Utils.time g;;

But if you mostly operate on matrices, you can just use Mat.row and Mat.col, they are even fasters. For example, in the code below, h does the same thing as f and j does the same as g, but h only takes 0.005ms and j takes 0.04ms.
let h () = Mat.row x 99;;
let i () = Mat.col x 99;;
Utils.time h;;
Utils.time i;;

But note, Mat.row does not make a copy of data, so if you modify the returned row, the corresponding elements in x will also change.
In general, slicing function is meant for more complicated slice definition of ndarray. I hope this is helpful.

For broadcast, this page seems helpful? It has very simple rules and I have documented in the code here.
It is a quite useful operation, used a lot in Owl's neural network module.

Oh, that difference between row/col and get_slice is very useful.  I had assumed that get_slice would be faster, but that makes sense.  I didn't realize that I could set the underlying array using row or col.  That actually might be very useful to me, although I can see that it could be dangerous.
Yes, thanks for the broadcast info link. That expands my understanding.


Docs: amdgpu-pro-vulkan on Gentoo.

When running Bevy on Gentoo using an AMD Radeon GPU, it panics unless amdgpu-pro-vulkan has been installed (and it took quite a bit of experimentation to find this information). This PR adds a mention of this to the linux dependencies documentation.

bors r+


 Retracting
should be very inexpensive as already defined in the configuration.
Publishing should be more expensive but still less expensive than tasks
like encoding.

Hi @lkiesow All the job load PRs are fine. I just wait with merging so that others have a change to comment the new job loads.


[Edit observation] Changes not reflected on View Observation after saving changes

To Reproduce
Steps to reproduce the behavior:

Edit an existing observation and click "Save".
View observation screen doesn't reflect changes. Changes appear in observation list and after returning to view observation screen.

@shobhitagarwal1612 would you be able to take a look? 🙏

I followed the same steps as mentioned in the description but wasn't able to repro the error.
Can you please check the exact steps again?

Can we close this?

Yes, it seems this issue hasn't recurred so closing now. Thanks!


Python 3.7 Build For Ray >=2.1.0

Comment:
Hi,
A Python 3.7 build is no longer published for ray version >=2.1.0

https://github.com/conda-forge/ray-packages-feedstock/tree/main/.ci_support ) -
https://anaconda.org/conda-forge/ray-all/files

I wanted to ask why, and if it would be possible to re-add the relevant CI support to publish py37 builds.
I'm asking because:

I'm using Ray in a Python 3.7 code base
Ray project still support Python 3.7, and publish PyPi builds accordingly ( https://pypi.org/project/ray/#files )

Thank you,
Harel

Python 3.7 is no longer supported by the conda-forge build system. See the announcement

Thank you @mattip , I missed that.


stop the loop when listener is closed

Fix 258.

Thanks Weidi! As usual you beat me to it. Do you know why this is needed though? Like why doesn't closing the listener bump out of the Accept loop? (Only when SSH connections are active...?)

The listener wrapper is used in my own listener plugin, which is only used on linux. I guess because listeners on linux use port reuse by default, the error there was temporary. On other platforms, it's implemented via a deadline, which is temporary.

It's not ssh though, the listener loop is still running, and the error here is never received, because this listener silently discard it.

Should I test this again or is it ready?

I'll circle back to this soon! Been busy

@mholt No stress just wanted to ask <3

CI failure seems unrelated


Provide signature help in template argument lists

👍
Boris Staletic @bstaletic 15:02
Right, so that's fine.
With clangd, my problem is this:

std::basic_string<|
With cursor at | you don't get signature help.

I have to rely on the scratch buffer.

Ben Jackson @puremourning 15:06
trigger characters for sig help don't include <
so i guess clangd doesn't do it


Support running the model without GPU

There should be an option for the user to run the model on CPU

It should fall back to CPU if you don't have a GPU


AFN URLSession:dataTask:didReceiveResponse:completionHandler: delegate method not callback.

ℹ Please fill out this template when filing an issue.
Per our *CONTRIBUTING guidelines, we use GitHub for
bugs and feature requests, not general support. Other issues should be opened on Stack Overflow with the tag afnetworking.

[x] I've read, understood, and done my best to follow the *CONTRIBUTING guidelines.

What did you do?
i use AFN send dataTask and want URLSession:dataTask:didReceiveResponse:completionHandler: delegate method callback.
What did you expect to happen?
i want URLSession:dataTask:didReceiveResponse:completionHandler: delegate method callback.
What happened instead?
the delegate method"URLSession:dataTask:didReceiveResponse:completionHandler:" do not callback at all.
but, i use NSURLSession doing dataTask.
the delegate method"URLSession:dataTask:didReceiveResponse:completionHandler:" can do callback and works fine.
the key class is:TestViewController
AFNetworking Environment
**AFNetworking version:**4.0.0
**Xcode version:**Version 12.1 (12A7403)
**Swift version:**0.0
**Platform(s) running AFNetworking:**iOS 14
**macOS version running Xcode:**10.15.6
Demo Project
https://github.com/debugmm/Zoomed

i find invoking setDataTaskDidReceiveResponseBlock,will works fine.


Nick: Watch laprimeraperu.pe

Nick requests the watch of the watch_keyword laprimeraperu\.pe. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

laprimeraperu\.pe has been seen in 1 true positive, 0 false positives, and 0 NAAs.

Approved by Jeff Schaller in Charcoal HQ


Aborting an XHR still results in a timeout callback

The issue seems to be around here: https://github.com/Raynos/xhr/blob/master/index.js#L184
When XHRs are aborted, the timeout isn't cleared. When the timer function is invoked, it doesn't first check the status of the XHR before executing the callback.
A possible solution is to listen for the abort event and clear the timeout.

Confirmed, nice catch. Thanks.
I can't promise I'll fix it this week, so let me know if you'd like to PR this with a test ;)

What's the correct behavior?  Should the callback be called with a timeout error right away?
I started to write a test and then realized I didn't know what I should be asserting.
Not calling the callback at all sounds like a way to tempt Zalgo.

Based on issue #93, I propose that

the callback should be called immediately when the abort is fired
the error object should be augmented with err.code = 'ECONNRESET'

Does that sound reasonable?  I don't like the idea of no longer calling the callback at all, if only for the reason that it would be a breaking API change.

We want to mimic how request/request is reacting to .abort()
I checked quickly and it does not call the callback. Nothing else happens after abort.
This issue is about the internal timeout we use instead of the one on XMLHttpRequest object, which doesn't work in some browsers.
Here's a solution https://github.com/Raynos/xhr/tree/abort-timeout, but it'd require extensive cross-browser testing before merging in.

Looking for volunteers to test in IEs, Safari etc.


Were these contracts deployed on MainNet?

Hi,
Is it possible to call this smart contract on main net? Where can we get the hash of it? Also, where are you guys planning to store these non native contracts script hashes?
What about TestNet? What is the hash? Thanks!

Duplicate of #9


Bigcache short description

We are working on an article about the current state of concurrent cache implementations in Go. We are planning to include bigcache with following short description. Please let me know if my understanding is correct, or you would like to modify it.
BigCache divides the data into shards based on the hash of the key. Each shard
contains a map and a ring buffer. Whenever a new element is set, it appends the
element in the ring buffer of the corresponding shard and the offset into the
buffer is stored in the map. If the same element is set more than once, the
previous entries in the buffer are marked invalid. If the buffer is too small,
it is expanded until the maximum capacity is reached.

The map stores data from uint32 to uint32. Each key is hashed into a uint32
that becomes the key of the map. The value of the map points to the offset
into the buffer where value is stored along with metadata information. If
there are hash collisions, BigCache ignores the previous key and stores the
current one into the map.

LGTM, but pinging original authors to clarify this @druminski @janisz @mxplusb

LGTM :+1:

LGTM 👍

I would add more information regarding the collisions. I had to dig in the code to understand how the collisions are handled.

@janisz ? :)

@larytet what it the question? I don't get it?
We do not handle collisions at all see: https://github.com/allegro/bigcache/blob/c00e2be7ad487b8fe06fc1da3d62edd19fb54519/bigcache_test.go#L605
Your aproach is valid and popular in hashmaps implementations (see wiki). Let's imagine we get the same hash for every entry (the worst case scenario) then our complexity is O(n) when n is the number of existing entries, we need to iterate over whole map to find matching element. We can improve it by using tree instead of map but then we need to have a way to compare elements.
Does it answer your question?

@larytet what it the question? I don't get it?
We do not handle collisions at all see:
Does it answer your question?

I find lack of any support surprising. It is not hard to provide some mechanism which reduces significantly the probability of collisions. The complexity will remain O(1) if the linear probing is limited by a small number of slots.
In my naivety I expected the Set() to fail if there is a different key with the same hash already stored in the cache. This is not the case.

@larytet Oh, thanks for calrifiction. Let's create new task to fix it. I updated readme for now https://github.com/allegro/bigcache/pull/151


CI Changes and expose types via overload

pylint<2.0 due to python2.7 requirements
last mypy adoption
tox mypy target & expose types detail using typing.overloaded
Fix gevent & greenlet version for CI due to binary compatibility
issues between them

Pull Request Test Coverage Report for Build 85

6 of 10 (60.0%)  changed or added relevant lines in 6 files are covered.
44 unchanged lines in 4 files lost coverage.
Overall coverage decreased (-0.1%) to 98.644%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

threaded/_gthreadpooled2.py
0
2
0.0%

threaded/_threaded2.py
0
2
0.0%

Files with Coverage Reduction
New Missed Lines
%

threaded/_base_gthreadpooled.py
3
100.0%

threaded/_gthreadpooled2.py
10
100.0%

threaded/_base_threaded.py
13
97.33%

threaded/_threaded2.py
18
100.0%

Totals

Change from base Build 83:
-0.1%

Covered Lines:
291

Relevant Lines:
295

💛  - Coveralls

Pull Request Test Coverage Report for Build 85

6 of 10 (60.0%)  changed or added relevant lines in 6 files are covered.
44 unchanged lines in 4 files lost coverage.
Overall coverage decreased (-0.1%) to 98.644%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

threaded/_gthreadpooled2.py
0
2
0.0%

threaded/_threaded2.py
0
2
0.0%

Files with Coverage Reduction
New Missed Lines
%

threaded/_base_gthreadpooled.py
3
100.0%

threaded/_gthreadpooled2.py
10
100.0%

threaded/_base_threaded.py
13
97.33%

threaded/_threaded2.py
18
100.0%

Totals

Change from base Build 83:
-0.1%

Covered Lines:
291

Relevant Lines:
295

💛  - Coveralls


migrate shelljs into just a plain shell script

Its probably safe to just eliminate shelljs and replace it with a shell script.

The downside to consider is that it will no longer work on Windows, probably not a problem though

Bash actually runs on windows these days, so this gets a +1 from me!

npm-run-all is also very windows friendly and lets you do complex things without too many incompatible bash tricks.
Sent from my mobile

On Apr 28, 2017, at 5:57 PM, Feross Aboukhadijeh<EMAIL_ADDRESS>wrote:
Bash actually runs on windows these days, so this gets a +1 from me!
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.


[PATCH v2] Removes potential '//' from email links (#420, #519, #523)

Re-submission of a non-regressing patch for a #420 detail, after #523.
The one-liner has been broken for readability purposes.

Hey over here, could this one be merged ?
Thanks ! Bye :wave:

I've tested it, it is working for me.
@blatinier @jelmer

Up @jelmer 👍

Sorry for the delay in reviewing, I finally spend some time trying to understand what's been broken in the various pull requests.


Fixed #29943 -- Document that admin changelist adds pk to ordering.

ticket_29943

Is this version sufficient?
.. admonition:: Performance considerations with ordering and sorting

    To ensure a deterministic ordering of results, the changelist adds
    ``pk`` to the ordering if it can't find a single or unique together set
    of fields that provide total ordering.

    For example, if you have a model with ``Meta.ordering = ['name']``
    (where ``name`` isn't unique), then the changelist is sorted by
    ``name`` and ``pk``. This could perform poorly if you have a lot of
    rows and don't have an index on ``name`` and ``pk``.

The other question is why is the note in ModelAdmin.ordering? I guess the same consideration from Meta.ordering apply to ModelAdmin.ordering.

@timgraham thank you for the comment.
I replaced your note. it seems better. and moved them to Meta.ordering

I didn't mean to suggest that. Documentation for contrib apps like admin doesn't belong in "core" documentation. if we had some other ideas about performance related admin concerns, perhaps a new section or document in ref/admin could be created.

I am confused :)). I don't know what should I do :))
If this PR doesn't make sense it is better to close it.
Thank you again for the reply


Incorrectly formatted FQDN's to googleapis.com

I've noticed all of the references to the Fully Qualified Domain Name "googleapis.com" are incorrectly formatted. Per RFC 1034 FQDN's should have a trailing dot in order to remove any ambiguity and unnecessary lookups.
Explanatory documentation HERE
We were pointed to this doc by a Kubernetes network engineer at Google after having many DNS lookup issues inside of our GKE clusters. After running some tests we noticed a minimum 100ms improvement per request after adding the trailing dot.
I have a PR ready that I will post for review.

Hi @ericuldall,
Thanks for reporting.
I am not going to make the change suggested in #2214 for the reasons I covered there. However, you can continue to use this in your own implementation by specifying your baseUrl directly:
new BigTable({ baseUrl: '...' });

I understand your concerns. I would highly suggest putting a warning for people using this library in a kubernetes cluster with a large number of http requests as the internal kube-dns will drop lots of requests without the trailing dot. This is because in the kubernetes resolve.conf they use a large number of ndots to check for internal services.
resolv.conf sample:
search default.svc.cluster.local svc.cluster.local cluster.local google.internal c.project.internal
nameserver <IP_ADDRESS>
options ndots:5

One of the main problems is the setting: sysctl -w net.netfilter.nf_conntrack_max=65535
That table is of a finite size per host machine and if it overflows requests get dropped. Using the trailing dot seems to be an easy remedy to avoid all the internal lookups and bypass the problem completely.
Another option is to set the pod dnsPolicy, DnsPolicy: Default in the pod spec.
We will move forward creating instances with the overridden baseUrl for our needs. Thanks for looking into this.

Hmm, I understand. Let me think on this more and discuss it and see if my position changes. (It might.)

Due to an issue brought up in #2249, we're going to revert this change.


[Release Radar Request] CLI for Microsoft 365

Open Source Project name
CLI for Microsoft 365
What is your project?
Using the CLI for Microsoft 365, you can manage your Microsoft 365 tenant and SharePoint Framework projects on any platform. No matter if you are on Windows, macOS or Linux, using Bash, Cmder or PowerShell, using the CLI for Microsoft 365 you can configure Microsoft 365, manage SharePoint Framework projects and build automation scripts.
Version
6.0
Date
November 30 2022
Description of breaking changes

Extended documentation with commands output
Configure how you want to display commands help
Create list items in batch
Extended support for managing Power Platform (Manage Power Platform cards, Manage Dataverse tables, Manage Power Platform solutions and publishers)
Manage versions of files stored in SharePoint

GitHub Repo
https://github.com/pnp/cli-microsoft365
Website
https://pnp.github.io/cli-microsoft365/
Link to changelog
https://pnp.github.io/cli-microsoft365/about/release-notes/#v600
Social media
https://twitter.com/climicrosoft365
Anything else to add?

CLI for Microsoft 365 just had its 5th birthday 🎂🥳
we reached over 670 stars 🌟
we broke the 100 contributors milestone 🤩 (thank you all for the awesome work on creating this open source tool 💪)

Congrats on being featured in the November 2022 Release Radar 🥳

Read the blog here
Check out our GitHub Tweet


Not sure if it's a bug or not about background channel

Hi sir,
Thank you for providing this repo. I observed in the file telephony.dart the following on lines 81-84:
  static final Telephony _instance = Telephony._newInstance(
      const MethodChannel(_FOREGROUND_CHANNEL), const LocalPlatform());
  static final Telephony _backgroundInstance = Telephony._newInstance(
      const MethodChannel(_FOREGROUND_CHANNEL), const LocalPlatform());

Is this ok to have the same _FOREGROUND_CHANNEL for both the foreground and background?
I was trying to send sms from the background but it is not working. It is only working from foreground,
Thank you sir,
Arkan

Also my intention is to send sms from a background (when my app is not active), my use case is to receive push notification from firebase, then in the background I want to send sms

I tried a similar setup to send SMS within firebase messaging background handler and got error
MissingPluginException (MissingPluginException(No implementation found for method sendSms on channel plugins.shounakmulay.com/foreground_sms_channel))

flutter 2.0.4
firebase_messaging  9.1.1
telephony 0.1.2
Other plugins like shared preference work fine in firebase background handler except for this one.
I think this PR broke the expected behavior https://github.com/shounakmulay/Telephony/commit/29ffc3381c2b576a9c502f331cf13fc6be4e7565#diff-d8470fb64214119fb5a3045932e214b0be49e6852ef472b778f1e1334873d789R35
I guess the reason for this PR is to allow the incoming sms handler to calls telephony's methods but it made the foreground channel unavailable for another plugin usage.

In my case, and because I needed to only send sms from (foreground and background), I've created a plugin on my local machine that does this (only on android), and I've tested it with FCM background function (from flutter), and it worked.
Maybe someone will make some fixes for the Telephony plugin.

I've got the same error "No implementation found for method sendSms on channel plugins.shounakmulay.com/foreground_sms_channel". Im using Android Alarm Manager to trigger it in the background but unable for it to get working.

Checkout version v0.1.3


How to use precomputed_knn in a GridsearchCV pipeline

I can run several of the examples in Precomputed k-nn of the user docs, but none of the examples show how to use precomputed_knn within a GridSearchCV pipeline.  Has anyone gotten this to work?
When I run this:
import pandas as pd
from sklearn.pipeline import Pipeline
from sklearn.svm import LinearSVC
from umap import UMAP
from umap.umap_ import nearest_neighbors
from sklearn.utils import check_random_state
from sklearn.model_selection import GridSearchCV
from sklearn.datasets import make_classification
from sklearn.model_selection import train_test_split

X, y = make_classification(n_samples=1000, n_features=10, n_informative=5,
                           n_redundant=2, n_classes=3, n_clusters_per_class=2)
X_train, X_test, y_train, y_test = train_test_split(X, y, test_size=0.2)

params_grid_pipeline = {
    "umap__n_neighbors": [5, 10, 20],
    "umap__n_components": [15, 25, 50],
    "svc__C": [10 ** k for k in range(-3, 5)],
}

random_seed_shared = 10 # KNN and UMAP must use same random stae to get the same answer
random_state_shared = check_random_state(random_seed_shared)

n_neighbors_max = max(params_grid_pipeline["umap__n_neighbors"])

knn_pre = nearest_neighbors(X,
                        n_neighbors=n_neighbors_max, # UMAP() reorgs to get smaller KNNS
                        metric='euclidean',
                        metric_kwds=None,
                        angular=False,
                        n_jobs=-1, # not necessary, is default already
                        random_state=random_state_shared)

umap = UMAP(precomputed_knn=knn_pre, random_state=random_state_shared)
pipeline_knn_pre = Pipeline([("umap", umap), ("svc", svc)])

clf_pipeline_knn_pre = GridSearchCV(pipeline_knn_pre, **params_grid_pipeline)
clf_pipeline_knn_pre.fit(X_train, y_train)
print("Accuracy on the test set with preKNN UMAP transformation: {:.3f}".format(
      clf_pipeline_knn_pre.score(X_test, y_test)))

I get the error message:
---------------------------------------------------------------------------
TypeError                                 Traceback (most recent call last)
Cell In[34], line 25
     22 umap = UMAP(precomputed_knn=knn_pre, random_state=random_state_shared)
     23 pipeline_knn_pre = Pipeline([("umap", umap), ("svc", svc)])
---> 25 clf_pipeline_knn_pre = GridSearchCV(pipeline_knn_pre, **params_grid_pipeline)
     26 clf_pipeline_knn_pre.fit(X_train, y_train)
     27 print("Accuracy on the test set with preKNN UMAP transformation: {:.3f}".format(
     28       clf_pipeline_knn_pre.score(X_test, y_test)))

TypeError: __init__() got an unexpected keyword argument 'umap__n_neighbors'

First, your example needs a bit of tweaking because svc is undefined in the Pipeline (presumably it's a LinearSVM()?).
Second, the reported error isn't a UMAP problem, but the use of the GridSearchCV constructor, I recommend replacing **params_grid_pipeline with param_grid=params_grid_pipeline.
With that out of the way, I think pre-computing the knn has a lot of implementation issues that will be tough to solve. First, computing the knn should really be seen as part of the training, so only done on X_train, not X as a whole, or you have data leakage occuring where some level of knowledge of the test set is encoded in the knn.
You have a more immediate problem if you don't do that anyway because when you pass the knn for the full dataset to UMAP, when you ask it to fit with X_train, it will notice that the knn has more entries than X_train and ignore the knn data. You will get a warning to that effect.
Unfortunately, using nearest_neighbors with X_train instead of X isn't going to solve your problems. The CV bit of GridSearchCV means that it will hold out some of the data for internal validation. This in turns that what is passed to UMAP for fitting has fewer items than X_train and again UMAP will spot that the knn is the wrong size, ignore it and warn you about it.
As it stands I don't think there is an easy way to workaround any of that.


Use Hugo Extended

Seeing that we're using sass, we'll need to use Hugo extended. That's what I use by default 😊.

That is Chinese to me unfortunately :p
@AnXh3L0 ?

Sounds good to me

We are now using Hugo Extended. Closing.


Improve Html Formatting

I've got the following unstructured paragraphs in the recent report:
Last week achievements:
opeo-maven-plugin:

Fixed the bug related to 'super' invocation [#229]
Added Short Spring Integration Test [#229]
Decompile Only Files With Supported Opcodes [#226]
Make 'spring-fat' Integration Test Pass Successfully [#229]
Optimize Memory Consumption [#229]jeo-maven-plugin:

Added support of method parameters annotations [#596]

Added 'InnerClasses' Attribute to Bytecode [#589]

Optimized Memory Consumption [#578]

As you can see, jee-maven-plugin header doesn't have a leading space.

I've fixed this bug in this PR: https://github.com/volodya-lombrozo/newsman/pull/33


Click to edit readme from the main repo page

@hkdobrev suggested in https://github.com/sindresorhus/refined-github/pull/33#issuecomment-189283145.

Usually you have to click into the readme from the file browser first.
I think we could have an edit button on the top right. Maybe only shows on hover?

I think @stephenplusplus wanted something like this in the past too if I remember correctly.

Yes (good memory, sheesh), this would be awesome! The on-hover sounds good to me, maybe an icon: :memo: or :pencil2:, or just a button is cool too.

Cool. Yeah, we'll use the pencil icon GitHub provides.
<svg class="octicon octicon-pencil" height="16" version="1.1" viewBox="0 0 14 16" width="14"><path d="M0 12v3h3l8-8-3-3L0 12z m3 2H1V12h1v1h1v1z m10.3-9.3l-1.3 1.3-3-3 1.3-1.3c0.39-0.39 1.02-0.39 1.41 0l1.59 1.59c0.39 0.39 0.39 1.02 0 1.41z"></path></svg>

PR coming in 3... 2...


CB-6952 Generate list about UMS right - CB API method pairs

generate permission needed for API in /info endpoint of the given service
move classes related to default resource permission checks into a package
move classes related to filtering list APIs based on permissions into a package

calling generate method once
mock/spring boot test for info page


Error: Function called outside component initialization when using createEventDispatcher()

Describe the bug
When using
    import { createEventDispatcher } from 'svelte';
    const dispatch = createEventDispatcher();

the preview does not render.
Error: Function called outside component initialization
    at get_current_component (/.vscode/extensions/rafaelmartinez.svelte-preview-2.7.1/node_modules/svelte/internal/index.mjs:986:15)
    at createEventDispatcher (/.vscode/extensions/rafaelmartinez.svelte-preview-2.7.1/node_modules/svelte/internal/index.mjs:1043:23)
    at instance (/test.svelte:154:19)
    at init (/.vscode/extensions/rafaelmartinez.svelte-preview-2.7.1/media/svelte.js:2111:7)
    at new Component (/test.svelte:164:3)
    at /test.svelte:168:5

To Reproduce
Steps to reproduce the behavior:
Add the following two lines to any svelte file:
    import { createEventDispatcher } from 'svelte';
    const dispatch = createEventDispatcher();

Expected behavior
A normal render.
Version:
2.7.1
Additional context
I read it is a 'known bug' in the extension's docs. Maybe it will get fixed, or there is a way to circumvent the error message.

I found a temporary solution with code from here: https://stackoverflow.com/a/66985113
    // Hack for svelte-preview.
    // Uncomment the first line and remove the 'import get_current_component' line and function in production.
    //import { createEventDispatcher } from 'svelte'
    import { get_current_component } from "svelte/internal";
    function createEventDispatcher() {
        const component = get_current_component();
        return (type, target, detail) => {
            const callbacks = component.$$.callbacks[type];
            if (callbacks) {
                if (target.dispatchEvent) {
                    const event = new CustomEvent(type, { detail });
                    // the key is to call `dispatchEvent` manually to set `event.target`
                    target.dispatchEvent(event);
                    callbacks.slice().forEach((fn) => {
                        fn.call(component, event);
                    });
                }
            }
        };
    }

Don't know exactly why, but it works now without the hack directly with
    import { createEventDispatcher } from 'svelte';

Strange...


stress test docker baseimage updates

updating the base images for stress test examples and stress watcher, already triggered the test-resource-deployer docker build here

The test csproj also needs to be updated to 7 for build/test CI to pass.


ContextMenu for the map does not work properly

Hello,
I've searched through the issues to make sure it's not yet reported.
I am using Leaflet.contextmenu to attach a contextmenu to:
-the map
-the nodes (markers)
-the edges (polylines)
This is how the contextmenu setup looks like for the map but its the same structure for nodes and edges.
map = L.map('map', {
contextmenu: true,
contextmenuWidth: 140,
contextmenuItems: [{
text: 'Show coordinates',
callback: showCoordinates
}]);
I did not noticed this issue (maybe bug) at first.
The problem is that after I open the contextmenu of a node or an edge the contextmenu of the map will never show up again.
It seems like the contextmenu of the node and the edge overwrite the one of the map.
After I tried to find a solution myself for quite some time, my next step was to look for alternatives.
So I used the Leaflet.Dialog Plugin to create my own contextmenu for the map. It worked pretty well,  the only problem is that when I open the contextmenu of an edge, the contextmenu of the map opens in additon to the regular contextmenu.
It looks like the polyline is treated like the map.
To open the contextmenu of the map I wrote :
map.on("contextmenu", function f () {
dialog.setContent(content)
.addTo(map);
dialog.open();
});
I also increased the weight, to make sure I dont accidentally hit the map.
To sum up I have two questions.

Why is it not possible to use the Leaflet.Contextmenu for the map, polylines and markers?
2.Why does map.on("contextmenu") gets fired when the user right clicks a polyline?

I hope you understand what my problem is, if not I am happy to explain it in more detail.
Best regards!

Hi there!
Regarding Leaflet.Contextmenu, you should file an issue with that plugin. It's not maintained by the Leaflet team, so we have no detailed insight as to what the problem might be.
Regarding your second question, events on path's (like polylines) bubble to their container, which is the map. This works much like other DOM events, where a click on a div will also bubble to its container unless you stop it (see stopPropagation).


include polygons of subdivisions from "subregionDistribution" on distribution maps

Adding subdivisions of the countries would be a great next step -- we actually have a 'subregionDistribution' field in the CSV, but it is only populated for certain countries + the USA.  Could you implement this for the USA states with an option for it to populate as we curate additional data in the column?
Originally posted by @n8upham in https://github.com/mammaldiversity/mammaldiversity.github.io/issues/13#issuecomment-1918117021

Thanks, good idea to add an issue for this.


Improved usage messages

Prospero without arguments:

prospero -h (no intro):

prospero install -h:

prospero update -h:

@spyrkob @marekkopecky I posted couple of screenshots of improved usage messages. Any feedback welcome.

Looks good, I like the parameter groups heading, it makes it much easier to understand the list of options

+1

Removing the draft label, I think it's ready for merging.

Thanks @TomasHofman!


 These (minor) changes allow Invoker to be started with a config file defining such processes. In this case, no processes get spawn by Invoker but the DNS / port-proxying part works as expected.
Example of a now valid config file:
[ruby-app]
port = 3000

[some-node-app]
port = 9001

You may also add a default config which will redirect any domain.dev to the specified port:
[default]
port = 3000

@rchampourlier @gnufied Any updates on this? I have an application that configures themes based on hostname and this would provide a way to make invoker start a single rails process plus multiple "hostnames" for that same process in the following way:
[myapp]
command = bundle exec rails server -b <IP_ADDRESS> --port 8081
port = 8081

[customer1]
port = 8081

[customer2]
port = 8081

I can provide an updated Pull Request if there's interest in this feature

Couldn't you just use invoker add_http <process_name> <port> syntax to add a service which was started externally to invoker DNS/proxy?
But then may be this feature provides a way of doing this without using invoker add_http command?  If there is enough demand for this - we can implement this feature. @rchampourlier  can you rebase this branch against master?

Hi,
@gnufied:
Yes, using invoker add_http <process_name> <port> was an option, but it requires you have an Invoker process started. And this requires to have at least one process to start through Invoker.
This PR allows to have no process started by Invoker at all, only using it for proxying.
I've rebased the PR from master 😄
@juanger: I'm not sure what needs to be changed on this PR for your use case. Feel free to complete it!
Sadly, I'm not using Invoker anymore, I've switched to manual /etc/hosts and Nginx, because Invoker stopped working some times ago (maybe due to my Docker native install). I just tested it while rebasing, it seems to work again, maybe I'll come back!
Best,

@rchampourlier cool. Which OS you use it on btw? Linux or OSX ? Please file a bug report, I would love to debug and fix.
Also, looking into the PR. thank you for opening this.

Any chance this will get merged. Seems handy!

I'd love to have that feature too

Closing this PR which has been inactive for a loooong time 😅


Validator error - specimen

The validator returns a 500 error while checking the specimen resource. We need to understand the reason for the error.
The checking of the specimen will be temporarily removed.

Fixed. Will close this issue right after deploy


Main compatibilty - http api workaround when server forgotten access on newer engines

too much newer or too much older engines do not allow http access
check also issue https://github.com/SkyBuilder1717/essentials/issues/3
workaround complete info is on readme of pull request
from https://github.com/SkyBuilder1717/essentials/pull/4
provide backaguard falback on older servers, biome data functions missing
will work with 0.4 servers like just test and famous subnasa
make it work with older versions of minetest

provide backaguard falback on older servers, biome data functions missing…
missign sfinv for menu support falback
http api workaround when server forgotten access on newer engines

working on it.


Have concourse worker use any locally configured proxies.

If a proxy is configured, unfortunately Upstart will not start processes
with /etc/environment variables available. This means that if you have
an external proxy setup (e.g. via vagrant-proxyconf) then things will
not work.
Use the workaround listed at: https://serverfault.com/a/717057
This will explicity set the three parameters:
--http-proxy
--https-proxy
--no-proxy
But if there is no value specified in /etc/environment
this has no ill effects.
The configuration will be passed down to the containers and they will
also utilise the passed through proxy as well.

@akumria Please sign the Contributor License Agreement!
Click here to manually synchronize the status of this Pull Request.
See the FAQ for frequently asked questions.

@akumria Thank you for signing the Contributor License Agreement!

There is a typo. My apologies I never updated this once I found it.
basically the sed pattern should be: s/^export //
Should I update this PR, or prepare another one, or ... ?
Again, apologies - I hadn't heard anything so wasn't sure if you were taking this.

@akumria it was already merged so if it was incorrect please create a new PR with the fix.


Add a recipe lookup

Currently, recipe returns all recipes. Users should have the ability to type recipe [keyword] to retrieve matching recipes.

Fixed with 339c0bd.


Ensure atomic experiment creation.

The transaction wrapping the creation of experiment keys in
Experiment.save did not watch any of those keys.  Consequently
parallel writers could cause duplicate writes against an
experiment's mutable values, such as its alternatives list.
This adds a watch guard to the experiment's key. Note that
Experiment.is_new_record now checks the existence of the
experiment's key rather than its membership in the experiments
set; this is so that the code watches a value unique to the
experiment and not one shared by all.
On conflict the experiment's traffic fraction is persisted.

Coverage increased (+0.3%) to 80.298% when pulling 000721cafea29be8bfc10bc2734e34601d9273f0 on ShopRunner:atomic-experiment-creation into 0755f1f8c46837dce33d37e28b6f03461bfa5bed on seatgeek:master.

Looks good! Thanks.

Hi @zackkitzmiller - I just wanted to thank you for sixpack.  I know managing an open source project is a ton of largely thankless work.  Just so you know, I really appreciate your efforts :)


Default credentials to connect to Horizon dashboard

We just deployed OpenStack using Ursula (using Vagrant). What are the credentials to connect to the Horizon dashboard ?

You should find the credentials on each of the hypervisors in hosts in /root/stackrc

Thanks for the info. We are able to get access to the BlueBox dashboard but when we enter the credentials we got a 'Connection Reset'. When we enter a wrong password, we have the expected error message 'Invalid userid/password'. Any idea ?

No ideas. This are working okay on the master branch.


Add color adjustment to all RBG channels

1.Tell us something about your changes.
With this piece of code we are able to define which combination of R, G and B channels are our TV Red, Green and Blue.
This modification makes the 'color correction' modification useless, as we can modify the whitelevel within the 'color adjustment' config file.
To achieve a perfect color matching between led strip and TV we should follow these steps:

Use a pure white (FFFFFF) image/video and take a look at your led strip. If the led strip color is a bit red, green or blue modify the max channel level until you get a perfect white

Use different grey (3F3F3F, 7F7F7F, BFBFBF...) images/videos and adjust gamma of each R,G,B colors. You should change gamma levels of each channel until you get a perfect grey on every step.

Once you get a perfect gamma level, you can adjust R, G, B channels. Use pure red, green and blue (FF0000, 00FF00, 0000FF) images/videos and play with channel adjustments until you achieve a perfect red, blue and green. For example, my perfect green (according to TV) is 20,255,15.

Increase Saturation and/or Luminance if you want.

After the four steps you should have achieved a perfect calibration of the led strip, matching your TV colors.
2. If this changes affect the .conf file. Please provide the changed section.
Inside "color", there is a new configuration block:
"color" :
{
.
.
.
	"channelAdjustment" :
	[
		{
			"id"   : "default",
			"leds" : "*",
			"pureRed" :
			{
				"redChannel"	: 255,
				"greenChannel"	: 0,
				"blueChannel"	: 0
			},
			"pureGreen" :
			{
				"redChannel"	: 0,
				"greenChannel"	: 255,
				"blueChannel"	: 0
			},
			"pureBlue" :
			{
				"redChannel"	: 0,
				"greenChannel"	: 0,
				"blueChannel"	: 255
			}
		}
	],
.
.
.
},

3. Comments.
This modification makes "color correction" step not needed at all, as we can use the main channel of each color to limit output. I mean, the "redChannel" of "pureRed" perform the same operation of the red channel correction value. To avoid performing the same operation twice, would be good to implement this mod to HyperCon and delete the colorCorrection from it.

I said don´t hurry! complainabouttohavenotimeforeverything
Hehe, thank you! very very great addition even with hyperion-remote.
Do you know how long i need to extend HyperCon with this grml
But dreams come true!

As always, the addition is fully tested, even hyperion-remote (and yes, you can send also those values at the same time as transform/correction/temperature ones) ;)

One point, cause this is a new, we push Hyperion to V1.02.0. The point is now, could you have a look on that index thing too (hyperion-remote)? tpmodding and i thought about to publish not too often new feature bumps (To prevent that people have to update every week or every x days)
So we could push all in one

It would be better to have more concrete commit names and not only the single file editions. Otherwise we end up with a very confusing git history.

@brindosch
I'm sorry about that, but as hyperion-remote is not intended as a remote configuration tool, there are no current implementations of led index. In fact, you can only apply transforms/corrections/adjustments using the already loaded qualifiers, not new ones. We may think about creating some other JSON functions that allow not only giving a led index. but modifing everything. I'm not a JSON expert tho, I can take a look but I don't know what we should do here.
@NicoHood
I was coding directly on github while I was at work, I don't have git there. I know is a bit messy, but I didn't want to redo all the code. You can always commit to your fork and commit back here while keeping my work as is ;)

No problem! We could drop the idea and save it for later.
I will start modify HyperCon soon.
@NicoHood
there is a command to merge all single commits to one?

Sure there is (its called squash i think, never used it though). Developing with git would make the thing still better, since single commits make it more readable. So 1000 and also 1 is still a bit bad. But yeah I know, that sometimes you need to test etc and cannot invent the solution straight on. I'd make a single commit out of all those.

@AEtHeLsYn
I will remove the complete color correction part from hypercon
Could hyperion handle that as long as it is in hyperion code? If people mix up versions config<>hyperion
Maybe we should remove the code inside hyperion and hyperion-remote too. If it is now useless.
and it is new, new enough that we may kick it better now than later
Will people get an error if they use a new hyperion (without colorC code) and a old config (with colorC config)
Your way of setting up colors i never used.
One big appeal to you: You have knowledge about this, we not. Could you provide a proper wiki article that descibes how to setting up colors the right way? People very often request this but my knowledge is limited. You could add pictures with the grey values and a white picute too, if you want.

Use your new commited settings with a white picture

Adjust Gamma with Gamma spinner
3.Use your new commited settings with  grey pictures (?!)

When do we use temperature values and for what?

Blacklvl - Checked
6 Whitelvl - What do we do with this setting now? You already said it. But what should we suggest to do with this?
Should i remove it from the HyperCon Panel? (Just the Panel, not the configfile. As long as i don´t know if something breaks- have to check code but minor)

I would really appreciate a good wiki article that explains everything. ColorAdjustment is a big part of hyperion and we now lacks a little for information - at least for the public out there.
Thank you for feedback!

Guys, what would you say if I develop KODI plugin for color calibration? I think it would be more convenient to adjust the settings while looking at TV :)
If you think it's a good idea I will take care of that within few weeks.

Well, the way I adjust leds is by using color patterns on Kodi and sending hyperion-remote commands til I get a perfect match.
Using a Kodi plugin to do that would be easier ;)

Yes, a kodi plugin is a good addition

Perfect. Who can help me understand (or when I can find information) how to communicate with hyperion? Shall I use json interface?

@brindosch
Working on a wiki article atm. And now about unused settings... we could leave hyperion as-is at the moment to keep backwards compatibility, but after all the color adjustments I made we only need this configuration to achieve a perfect control of color:
"color" :
{
	"temperature" :
	[
		{
			"id"   : "default",
			"leds" : "*",
			"temperatureValues" :
			{
				"red"   : 255,
				"green" : 255,
				"blue"  : 255
			}
		}
	],
	"channelAdjustment" :
	[
		{
			"id"   : "default",
			"leds" : "*",
			"pureRed" :
			{
				"redChannel"	: 255,
				"greenChannel"	: 0,
				"blueChannel"	: 0
			},
			"pureGreen" :
			{
				"redChannel"	: 0,
				"greenChannel"	: 255,
				"blueChannel"	: 0
			},
			"pureBlue" :
			{
				"redChannel"	: 0,
				"greenChannel"	: 0,
				"blueChannel"	: 255
			}
		}
	],
	"transform" :
	[
		{
			"id"   : "default",
			"leds" : "*",
			"hsl" :
			{
				"saturationGain" : 1.0000,
				"luminanceGain"  : 1.0000
			},
			"red" :
			{
				"threshold"  : 0.0000,
				"gamma"      : 1.0000,
			},
			"green" :
			{
				"threshold"  : 0.0000,
				"gamma"      : 1.0000,
			},
			"blue" :
			{
				"threshold"  : 0.0000,
				"gamma"      : 1.0000,
			}
		}
	],
	"smoothing" :
	{
		"type"            : "linear",
		"time_ms"         : 200,
		"updateFrequency" : 20.0000,
		"updateDelay"     : 0
	}
},

Note that there is not any blacklevel/whitelevel at the transform, nor a hsv part (only hsl). Color correction isn't there too.
@tociek
hyperion-remote uses json interface to send values to hyperion. I'm not an expert using json, but it wasn't too difficult to create new messages. Take a look at source code of hyperion-remote (json cpp) and you will find what you need

This is awesome! Thank you. You said you could keep cpu usage low, but will it work on Raspberry A without falling behind? (If not tested yet, I may can do it once HyperCon was updated)
I wouldn't have any problem upgrading to a new Raspberry though :P

@segelkma
Don't worry, the cpu usage is the same as before, maybe a 0,5-1% more, no need to upgrade to a new raspberry ;)
@brindosch
Wiki page created, take a look .

Great work on the code and the wiki!
Imagine if your wiki was built Ito a wizard in the iOS and android apps...

@penfold42
That would be awesome... any volunteers?

@AEtHeLsYn
What a great article, to be honest. I never understood everything but now i does.
Regarding the Blacklvl, i use it to keep the lights always on (0,03, 0,03, 0,01) on "dark" scenes
Is there a another possibility to keep that "feature" if we kick blacklvl? Or scew that setting something other up?
Thank you for all efforts you invested! I translate it later to german, a recalibration with the new HyperCon is not avoidable!

@brindosch
What you do with blacklvl is reduce dynamic range of leds, You can achieve the same using threshold, but without changing range.
Let's say you have a R range from 0.0 to 1.0 (0 - 255). Applying blacklvl config what you are doing is adding some amount of color (0.03) to every value and they multiply everything back to keep color in range.
If you use threshold instead, the only thing you are going to do is keep lower values than 0.03 zeroed, but you will not affect the color range.

 I will test this and report back.
Thank you!

awesome work! I take my hat off to you!

sorry for OT, but how can I update my hyperion installation on openelec to test this new features

Please be patient until we tested and hypercon is ready.

Or compile from source

HyperCon is as near as complete. We could start testing
/Merged to Beta branch

No Idea how you did this via webgui, but thats better now. Good :)
https://github.com/tvdzwan/hyperion/commit/8ab465e59834f12a21709a6f4546bd6808a2a495


ERROR: request returned Internal Server Error for API route and version http://%2F%2F.%2Fpipe%2Fdocker_engine/_ping, check if the server supports the requested API version

Bug description
Getting this error:
ERROR: request returned Internal Server Error for API route and version http://%2F%2F.%2Fpipe%2Fdocker_engine/_ping, check if the server supports the requested API version
when attempting to run the docker build command for version 11.315.0 using the command here: https://github.com/felddy/foundryvtt-docker#pre-installed-distribution-builds
Steps to reproduce
Run Docker Desktop v4.25.0
Open windows powershell
Run the command
Returns this error
Expected behavior
docker builds the 11.315.0 version of the image
Container metadata
I can't even run the above command due to a similar API error:

request returned Internal Server Error for API route and version http://%2F%2F.%2Fpipe%2Fdocker_engine/v1.24/containers/%7B%7Brange%20$k,%20$v%20:=%20.Config.Labels%7D%7D%7B%7B-%20printf%20%25s/json, check if the server supports the requested API version
request returned Internal Server Error for API route and version http://%2F%2F.%2Fpipe%2Fdocker_engine/v1.24/containers/=/json, check if the server supports the requested API version
request returned Internal Server Error for API route and version http://%2F%2F.%2Fpipe%2Fdocker_engine/v1.24/containers/%22%25s%22%5Cn%20$k%20$v%20-%7D%7D%7B%7Bend%7D%7D/json, check if the server supports the requested API version
request returned Internal Server Error for API route and version http://%2F%2F.%2Fpipe%2Fdocker_engine/v1.24/containers/felddy/foundryvtt:release/json, check if the server supports the requested API version

Relevant log output
No response
Code of Conduct

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

This is most likely not an issue with this container.
The error would suggest that your Docker daemon had died.
Are you able to communicate with the Docker daemon in other ways?  Please try running the hello-world container with docker run hello-world:
➤ docker run hello-world
Unable to find image 'hello-world:latest' locally
latest: Pulling from library/hello-world
70f5ac315c5a: Pull complete
Digest: sha256:88ec0acaa3ec199d3b7eaf73588f4518c25f9d34f58ce9a0df68429c5af48e8d
Status: Downloaded newer image for hello-world:latest

Hello from Docker!
This message shows that your installation appears to be working correctly.

To generate this message, Docker took the following steps:
 1. The Docker client contacted the Docker daemon.
 2. The Docker daemon pulled the "hello-world" image from the Docker Hub.
    (arm64v8)
 3. The Docker daemon created a new container from that image which runs the
    executable that produces the output you are currently reading.
 4. The Docker daemon streamed that output to the Docker client, which sent it
    to your terminal.

To try something more ambitious, you can run an Ubuntu container with:
 $ docker run -it ubuntu bash

Share images, automate workflows, and more with a free Docker ID:
 https://hub.docker.com/

For more examples and ideas, visit:
 https://docs.docker.com/get-started/


Remove AirGoogleMaps from AirMaps.xcodeproj

Does any other open PR do the same thing?

No
What issue(s) is this PR fixing?
#2159, #2270
The non-Cocoapods manual install instructions appear to be gone, but IIRC it was previously required to manually link the Google Maps libraries in your Xcode project if you wished to use Google Maps on iOS - this patch restores that behaviour.
How did you test this PR?

npm link with the patch applied on master into a project where I was reproducing #2270, after using git bisect to identify the likely first bad commit.

Nice! it working for me :)

+1 I think google maps shouldn't be auto-included if the build breaks by default.

+1
I have the same issue

I drafted this up for non-cocoapoders: https://gist.github.com/theonetheycallneo/bea215ebfbc7d7ade422458f60760df1

This should be the default behaviour of this library, thanks @mikemorris.

+1. Putting
"react-native-maps": "https://github.com/mikemorris/react-native-maps.git#rm-airgooglemaps",

in my package.json avoids the problem.

@alvaromb @alvelig can you guys have a look at this as it seems to be related to https://github.com/react-community/react-native-maps/issues/1954#issuecomment-403093113 🐽

Well, makes sense. So just to declare: we won't include GMaps by default. Developers are responsible for adding it manually themselves. Is it ok?

That's ok @alvelig, thanks.

@alvelig @alvaromb so should we merge this?

@alvelig In the spirit of https://github.com/react-community/react-native-maps/issues/1954#issuecomment-357644912 I wasn't attempting to add a documented or "endorsed" installation method, just removing an obstacle to upgrading for users who have chosen to install in an undocumented manner.
I'd be happy to document a non-Cocoapods installation in a separate PR if that would be desirable.

@mikemorris the problem is that it breaks current installation instruction process, doesn't it? Do we need to modify current installation instructions?

@mikemorris the problem is that it breaks current installation instruction process, doesn't it? Do we need to modify current installation instructions?

It shouldn't, but would be good to confirm. The existing Cocoapods installation instructions should still work just fine, pulling down the Google Maps SDK as a pod instead of expecting it to already exist in the project.

@rborn could you check please, as you made the actual installation instructions, if they still work as they are. To have them up to date with the changes we make. TY!

@alvelig I'll do it after 5th of August, I'll be offline starting tomorrow :)

Thanks! Happy holiday :D

@rborn we're testing this installation tomorrow. I'll ping you if it works ok.

After installing via yarn add mikemorris/react-native-maps.git#rm-airgooglemaps, I started from the instructions here and am successfully running on iOS without CocoaPods & using react-native link. I know that this is not the point of the issue, but I strongly believe that using react-native link should be an acceptable installation option.
I'm not quite sure where to suggest it, but here's the changes that I made to the installation instructions:
25,40c25,34
< Make sure your GoogleMapsBase.framework, GoogleMapsCore.framdwork, and GoogleMaps.framework are all listed in the Compiled Sources.
< ** You might need to drag them into Embedded Frameworks depending your Xcode project setup too.
< - Search Xcode Build Settings for "Header Paths" and add your new framwork folder from above:
< ```
< $(SRCROOT)/Frameworks/Google-Maps-iOS-Utils
< ```
< 4. Install AirMaps React Native Wrapper
< - Open
< ```
< $(GITROOT)/node_modules/react-native-maps/lib/ios
< ```
< - Drag both folders into your Xcode project and ignore the Airmaps.xcproject
< ```
< $(GITROOT)/node_modules/react-native-maps/lib/ios/AirGoogleMaps
< $(GITROOT)/node_modules/react-native-maps/lib/ios/AirMaps
< ```
---
> Make sure your GoogleMapsBase.framework, GoogleMapsCore.framdwork, and GoogleMaps.framework are all listed in the _Link Binary With Libraries_ section.
>
> 4. Link required frameworks
>
> In the _Link Binary With Libraries_ section, click the plus button to link the following frameworks:
>
> - `GLKit.framework`
> - `CoreData.framework`
>
> Note that there are quite a few other frameworks that Google suggests linking that may be required for other uses of the framework and/or `react-native-maps`, but these should get you started. Reference the start guide from above for additional frameworks that should be linked.
42c36
< 5. Now link
---
> 4. Now link
44c38
< react-native link
---
> react-native link react-native-maps
46d39
< We run this command to link the Android side (which works fine as-is) and make sure once we can still successfully link npm updates.

@mikemorris @alvelig @rbor I've opened #2396 which allows react-native link to be used to properly link and integrate the library without much work on the user's part and in a way that hopefully will not result in too much additional overhead in terms of support.

I can confirm this installation made by @mikemorris works perfect under iOS with react-native link /cc @rborn @alvelig.
Haven't tested @wbyoung PR, but it looks nice in order to ease the support of Google Maps for iOS.

I was also able to use this library without cocoapods by using commit #ef5c839252e427268ae2ebbf4f0cced1cb51fed6. I now only run react-native link to get everything up and running. I did also have to run pod deintegrate in my /ios folder, though.

Are there plans to merge this PR?

@mikemorris Do you have an eta for when this PR will be merged?

@jamesone Nope, as I don't have commit access. It's working for myself, and a few others per the comments above though, so I'd very much like to see it merged and released soon!

@jamesone if you're trying out a PR to resolve this issue, please consider trying #2396 which is a solution that handles all use cases a little, but should still require no extra work if you're not planning on using Google Maps on iOS via react-native link (defaulting to not using Google Maps).

@jamesone did you install the google maps manually to the project and just replace the node module reference? wondering if I should clear cache too

@icemancast I didn't setup Google maps. I used the default (Apple maps)
I would suggest removing node_modules, clearing the cache, reinstalling, and then restarting the dev server.
rm -rf node_modules
watchman watch-del-all
rm -rf ~/Library/Developer/Xcode/DerivedData/
rm -rf $TMPDIR/react-*
rm -rf ~/.node-gyp
yarn

Thanks @jamesone , I finally got it working. I used the pr as my library until they merge it in and then installed google maps manually per google sdk docs. That and a couple back flips made it work!

Any update on getting this merged @rborn @alvelig?

@mikemorris have you had a chance to try #2396 which should solve the same problem?
There’s been some discussion over there and it may help to get your feedback to see if it also resolves your problems as well. I think/hope it likely will.

@wbyoung @mikemorris did't have the chance to test the last push to #2396 that fixes the markers (as far as I remember) so maybe if somebody could do it we'll push for it

If anybody needs something tested, please let me know 👍Happy to help

@jamesone if you could test the example to work with #2396  (and maybe fix it if needed 🤪) would be amazing 🤗

@rborn I'm kinda philosophically opposed to the approach of leaving a vendored library committed in this repo. The changes in #2396 look good, but I'd kinda like to see that happen in addition to removing this dependency and recommending Cocoapods to manage the dependency if there's a desire to use Google Maps on iOS.

@mikemorris the changes in #2396 don’t add the vendored Google frameworks/libraries to the repository. They leave that as the user’s responsibility during install (only if not using CocoaPods). The choice of committing those files or managing them in another way is entirely deferred to the user.
What #2396 does is basically remove all of the AirGoogleMaps files but with preprocessor conditionals. So it’s essentially exactly the same as this PR in the default case. The subtle difference is that compiler will still see the files, but won’t produce any instructions as a result. And when desired, that conditional can be flipped on so the files do get compiled.
Personally, I have the desire to avoid using CocoaPods for the project that I’m integrating this into while needing to use Google Maps as well. I’d imagine others will at some point have the same preferences and/or needs. While I understand the superiority of CocoaPods for this specific case (especially since some dependencies aren’t pre-built & since many in the RN community don’t know Xcode well), I think it’s still important to have the option.

@wbyoung Ack, yea okay seeing that now - for some reason I was thinking that GoogleMaps/GoogleMaps.h itself got committed at some point.
Your approach looks good, I'll test it this weekend.

Closing in favor of #2396


Updating generic-table-dev branch with p4lang:p4runtime:main

Hi @saynb I am a bit puzzled why this PR isn't just "sync main to generic-table-dev." It would make the lineage clearer.

Understood.

Merging. @saynb please confirm it resolves to be same as main, otherwise we can fall back to a rebase and force-merge.

@saynb got CI build error, even after retry: https://github.com/p4lang/p4runtime/actions/runs/7671762843/job/20910840903#step:5:42

@chrispsommers I can confirm that the generic-table-dev branch has no diff with main even if changes show up in a PR if you try and create one from main->generic-table-dev.
Also, this CI job has somehow been failing on main branch itself since last month
https://github.com/p4lang/p4runtime/actions/workflows/ci-build-proto.yml

Hi @antoninbas do you have any ideas about the CI failures cited above?

It's a bazel build issue that I haven't had time to look into.
Looks like the failure is because https://mirror.bazel.build/github.com/bazelbuild/rules_proto/archive/9e4c622ba8c2178b71420ed3d14fb8874beee3a5.tar.gz no longer exists. Maybe @smolkaj knows what we should be using instead?

Interesting!
Looks like a regression on the main branch.
Last passing run (10 Dec 2023): https://github.com/p4lang/p4runtime/actions/runs/7161170241
First failing run (11 Dec 2023): https://github.com/p4lang/p4runtime/actions/runs/7174991901

Here is what changed:
2023/12/11 00:10:44 Downloading https://releases.bazel.build/6.4.0/release/bazel-6.4.0-linux-x86_64...

vs
2023/12/12 00:10:26 Downloading https://releases.bazel.build/7.0.0/release/bazel-7.0.0-linux-x86_64...

Looks like we're not yet ready for Bazel 7.x. This is easy to fix by adding a .bazelversion file. Let me send a PR.

@saynb Could you do another PR to sync to main and gain the benefit of the Bazel fix? Thx.


Bug - when transaction fails, spinner spins forever

https://github.com/w3f/Grant-Milestone-Delivery/pull/713#issuecomment-1426085300
Steps:

Scan QR code, make transaction that fails (ex: low balance)

Expected: Shows error message, user can exit back to home
Actual: On failed transaction, forever spinning spinner

Fixed


GVRKit plugin metadata generation issue

I have a GVRKit plugin that I have been developing for a project, and the runtime metadata is incorrect, and I need to find a solution to get around this in NativeScript.
In some ways slightly related to #826 (Like him I've got one that is working).
Partially related to this issue #710...
Error is:
Error: Actual arguments count: "0". Expected: "1".
      - Name:            handleTrigger
        JsName:          handleTrigger
        Filename:        GVRRenderer.h
        Module:
          FullName:        GVRKit.GVRRenderer
          IsPartOfFramework: true
          IsSystemModule:  false
          Libraries:
        Flags:           [ IsIosAppExtensionAvailable ]
        Type:            Method
        Signature:
          - Type:            Bool

      - Name:            'handleTrigger:'
        JsName:          handleTrigger
        Filename:        GVRRenderer.h
        Module:
          FullName:        GVRKit.GVRRenderer
          IsPartOfFramework: true
          IsSystemModule:  false
          Libraries:
        Flags:           [ IsIosAppExtensionAvailable ]
        Type:            Method
        Signature:
          - Type:            Bool
          - Type:            Interface
            JsName:          GVRHeadPose
            WithProtocols:

Do you see that the metadata JS name is the SAME, but the core name is different.   One has a prototype of:
bool handleTrigger()
and the other  is
bool handleTrigger(GVRHeadPose *headpose)
I really want to call the first option.  I can't seem to because NativeScript has decided the only valid one is the second one.  ;-(
Solutions?

Hi, I have the same issues with StarIO-framework.
  - Name:            'appendLineFeed:'
    JsName:          appendLineFeed
    Filename:        /Users/anders/Apps/nativescript-star-io/demo/node_modules/nativescript-star-io/platforms/ios/StarIO_Extension.framework/Headers/ISCBBuilder.h
    Module:
      FullName:        StarIO_Extension.ISCBBuilder
      IsPartOfFramework: true
      IsSystemModule:  false
      Libraries:
    Flags:           [ IsIosAppExtensionAvailable ]
    Type:            Method
    Signature:
      - Type:            Void
      - Type:            Int
  - Name:            appendLineFeed
    JsName:          appendLineFeed
    Filename:        /Users/anders/Apps/nativescript-star-io/demo/node_modules/nativescript-star-io/platforms/ios/StarIO_Extension.framework/Headers/ISCBBuilder.h
    Module:
      FullName:        StarIO_Extension.ISCBBuilder
      IsPartOfFramework: true
      IsSystemModule:  false
      Libraries:
    Flags:           [ IsIosAppExtensionAvailable ]
    Type:            Method
    Signature:
      - Type:            Void

This is not the only place I have discovered it. It is the same with appendMultiple:height: vs appendMultipleHeight: that both is recognised as appendMultipleHeight in Nativescript.
Is there a plan to fix this?

Hi @ulvesked. Thank you for the ideas presented. I like approach number 3 the most, but it has a small drawback - there could still be cases where the number of arguments will be the same. Consider the following hypothetical example:
- (void)appendMultiple:(NSInteger)width height:(NSInteger)height;
- (void)append:(NSInteger)height multipleHeight: (NSInteger)multipleHeight;

Both methods would end up with the same name (appendMultipleHeight) and the same number of arguments (two). That's why I think that besides the logic that you proposed, we could make the metadata generator fallback to appending a number suffix to the JS name of the method whenever it detects such a collision, which I suppose will be an extremely rare situation.
We will try to fix the issue for the upcoming 5.0 release of {N}.


Error building v2.1.2

Building v2.1.2:
[2/79] Building CXX object lib/Refureku/Refureku/Generator/CMakeFiles/RefurekuGenerator.dir/Source/main.cpp.o
FAILED: lib/Refureku/Refureku/Generator/CMakeFiles/RefurekuGenerator.dir/Source/main.cpp.o
/usr/bin/c++  -I/home/exuvo/code/Aurora-C/lib/Refureku/Refureku/Generator/Include -I/home/exuvo/code/Aurora-C/lib/Refureku/Refureku/Generator/ThirdParty/Kodgen/Kodgen/Include -I/home/exuvo/code/Aurora-C/lib/Refureku/Refureku/Generator/ThirdParty/Kodgen/Kodgen/ThirdParty/Include -g -fdiagnostics-color=always -Wall -Wextra -Wpedantic -MD -MT lib/Refureku/Refureku/Generator/CMakeFiles/RefurekuGenerator.dir/Source/main.cpp.o -MF lib/Refureku/Refureku/Generator/CMakeFiles/RefurekuGenerator.dir/Source/main.cpp.o.d -o lib/Refureku/Refureku/Generator/CMakeFiles/RefurekuGenerator.dir/Source/main.cpp.o -c /home/exuvo/code/Aurora-C/lib/Refureku/Refureku/Generator/Source/main.cpp
In file included from /home/exuvo/code/Aurora-C/lib/Refureku/Refureku/Generator/Include/RefurekuGenerator/CodeGen/ReflectionCodeGenModule.h:602,
                 from /home/exuvo/code/Aurora-C/lib/Refureku/Refureku/Generator/Source/main.cpp:9:
/home/exuvo/code/Aurora-C/lib/Refureku/Refureku/Generator/Include/RefurekuGenerator/CodeGen/ReflectionCodeGenModule.inl: In lambda function:
/home/exuvo/code/Aurora-C/lib/Refureku/Refureku/Generator/Include/RefurekuGenerator/CodeGen/ReflectionCodeGenModule.inl:419:114: error: ‘class kodgen::TypeInfo’ has no member named ‘computeTemplateSignature’
  419 |                                                 inout_result += "template <template <" + nestedStructClass->type.computeTemplateSignature(true) + "> typename>";
      |                                                                                ^~~~~~~~~~~~~~~~~~~~~~~~

With kodgen v2.2.0
Tried with both your release tags and my branches which have minor changes.
v2.0.3, v2.1.0, v2.1.1 all build and work for me.

Looks like the Kodgen submodule inside of the Refureku repository is not up-to-date.
Try updating Kodgen submodule (see git submodule update command).

I did that manually with git fetch upstream and git merge while in Refureku/Refureku/Generator/ThirdParty/Kodgen. That is the same right? Ah no you pushed some commits after the tag, i see now.


Test against Elasticsearch 6.5.2

Verify that 6.5.2 works, per #52

Coverage remained the same at 98.0% when pulling 8a16dd795c30cb4ee36d609389bdaa208a8b6e4c on add-latest-elasticsearch into 5c7c2dd186217cb7d3c6252275e4ae47b193fe4b on master.


Add guidance for local Jekyll preview with custom baseurl

Code of Conduct

[X] I have read and agree to the GitHub Docs project's Code of Conduct

What article on docs.github.com is affected?
https://docs.github.com/en/pages/setting-up-a-github-pages-site-with-jekyll/testing-your-github-pages-site-locally-with-jekyll
What part(s) of the article would you like to see updated?
The "Testing your site locally" section needs updating. Add information about running Jekyll locally when _config.yml has a baseurl set to the GitHub repository's link. This change supports users who encounter local preview issues due to baseurl conflicts. The expected outcome is clearer guidance on using bundle exec jekyll serve --baseurl="" to run Jekyll locally without modifying _config.yml, improving the local development experience for GitHub Pages sites.
Additional information
This issue affects users who set their baseurl in _config.yml to match their GitHub Pages URL structure, which is a common practice. The problem occurs consistently for these users when trying to preview their site locally. While the exact number of affected users is unknown, it's a recurring issue for developers working with GitHub Pages. No screenshots seemed necessary, as the change involves adding a command and explanation to the documentation.

UQAutOwc-EVANIwf1MG2m_M0wsubYsZnwUvPMMCkI8a3fkQx

UQAutOwc-EVANIwf1MG2m_M0wsubYsZnwUvPMMCkI8a3fkQx

UQAutOwc-EVANIwf1MG2m_M0wsubYsZnwUvPMMCkI8a3fkQx

UQAutOwc-EVANIwf1MG2m_M0wsubYsZnwUvPMMCkI8a3fkQx

UQAutOwc-EVANIwf1MG2m_M0wsubYsZnwUvPMMCkI8a3fkQx

UQAutOwc-EVANIwf1MG2m_M0wsubYsZnwUvPMMCkI8a3fkQx

UQAutOwc-EVANIwf1MG2m_M0wsubYsZnwUvPMMCkI8a3fkQx

UQAutOwc-EVANIwf1MG2m_M0wsubYsZnwUvPMMCkI8a3fkQx

UQAutOwc-EVANIwf1MG2m_M0wsubYsZnwUvPMMCkI8a3fkQx

UQAutOwc-EVANIwf1MG2m_M0wsubYsZnwUvPMMCkI8a3fkQx

UQAutOwc-EVANIwf1MG2m_M0wsubYsZnwUvPMMCkI8a3fkQx

UQAutOwc-EVANIwf1MG2m_M0wsubYsZnwUvPMMCkI8a3fkQx

UQAutOwc-EVANIwf1MG2m_M0wsubYsZnwUvPMMCkI8a3fkQx

UQAutOwc-EVANIwf1MG2m_M0wsubYsZnwUvPMMCkI8a3fkQx

Is this issue open?

Is this issue open?

@rajneesh-18 Hello! 👋 There is currently a PR waiting for review that will resolve this issue - https://github.com/github/docs/pull/34182
If you're looking an issue to pick up take a look at our help wanted section to find open issues you can work on! ✨


Personal and Company Address input fields have a label that is too long

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

Action Performed:

Head over to http://localhost:8080/bank-account/new
Click Connect Manually
Enter 123123123 for both fields
Observe the label on the Company Address input.

Expected Result:
Describe what you think should've happened
The label inside the input should not be two lines long. Instead, the bit in the parentheses should be on its own line.
Actual Result:
Describe what actually happened
The label is two lines long.
Workaround:
Can the user still use Expensify without this being fixed? Have you informed them of the workaround?
Platform:

Where is this issue occurring?

Web
iOS
Android
Desktop App
Mobile Web

Version Number:
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
Expensify/Expensify Issue URL:
View all open jobs on Upwork

Triggered auto assignment to @conorpendergrast (AutoAssignerTriage), see https://stackoverflow.com/c/expensify/questions/4749 for more details.

@conorpendergrast you can ignore this, I'm just going to do it myself real quick.

:wave: Friendly reminder that deploy blockers are time-sensitive ⏱ issues! Check out the open StagingDeployCash deploy checklist to see the list of PRs included in this release, then work quickly to do one of the following:

Identify the pull request that introduced this issue and revert it.
Find someone who can quickly fix the issue.
Fix the issue yourself.


Update _collapsible.scss

Display the body of an active collapsible region.

This appeared to just be missing. Without this, the current code depends on the click handler to expand the collapsible body.
Setting
 would not expand the body until it is clicked on twice (the first click handler detects the active class and removes it. The second detects that there is no active class and adds it, but it also adds the additional display: flex; CSS directly to the .collapsible-body element.

You can just add class="active" to the collapsible-header.

@Dogfalo - No - setting this on the collapsible-header has no effect on the collapsible-body. The effect when setting class="active" on the parent element of the collapsible-header also doesn't work.
Take the following code:

title

content

title 2

content

title 3

content

I would expect this to render the accordion with the first region expanded. It does not, because the collapsible-body display is still hidden. Furthermore, if you click on the header (which is still collapsed), the jQuery removes the active class as  though the panel is being collapsed. You then have to click on it again and it expands by setting the display:flex on the collapsible-body.
The following code DOES work as expected:

title

content

      ...

If you look at the diff of the PR I submitted, you'll see there is just an empty block with no CSS attributes in the original file. It just needs display: flex to allow devs to just add class="active" to the
. By merging this, the following code works as expected:

title

content

      ...

For reference, I am only using the SASS with the following imports:
@import "components/prefixer"
@import "components/mixins"
@import "components/color"
@import "components/variables"
@import "components/global"
@import "components/typography"
@import "components/collapsible"

The JS code is as follows:
    $(document).ready(function(){
      $('.collapsible').collapsible({
        accordion : false
      });
    });

I can reference this as a bug if you like.

There is no need to initialize the collapsible.

First

Lorem ipsum dolor sit amet.

Second

Lorem ipsum dolor sit amet.

Third

Lorem ipsum dolor sit amet.

I don't think the issue has been explained clearly. The goal is to get one or more of the options to expand by default so the user doesn't have to click on anything. The goal is to make the initial screen render as shown below:

The code you provided above does not do this. You can see a copy/paste of your code rendered below:
Notice the display body is none

Using your code snippet, you can click on the "Second" header to make it expand like the first screenshot, but the goal is to do this by default so the user isn't forced to click on anything when the page initially loads.
The original code (lines 45-47) in _collapsible.sass responsible for displaying the body is empty, which is why display is defaulting to none (from the CSS declaration on line 35):

My PR simply fills in the missing piece:

You can set a collapsible element to active, as the original CSS clearly intended, and it will auto-expand. Here's a screenshot with this PR applied:
Notice the li.active .collapsible-body class containing display: flex;

I hope that clears things up.

By adding active to collapsible-header, this makes the collapsible expand without the user having to do anything. The reason this is not working for you is because you initialized the component twice.

Sorry to be dense... in reviewing the documentation again I see where you're coming from now.


Create ponyc options

Refactor the option processing code in main to libponyc/options. The hope
is that this will allow easier testing of standard ponyc options in the
future.

Can you check my review from the last time you posted this?
https://github.com/ponylang/ponyc/pull/2432#discussion_r157906140


Add cs-engine 1.12.6-cs6 release notes

ping @joaofnfernandes

@aduermael how can I retry the Jenkins job? It seems that the problem was an HTTP 502 when doing an svn checkout during the build. I've seen Github having problems with svn checkouts in the past, and usually retrying the build would fix it.


[Feedback] Thank you for the recent enhancement in temporal web 1.9.0 where workflow Input now shows in JSON format.

Your feedback here!

Thank you temporal web for the recent enhancement in temporal web 1.9.0 where workflow Input now shows in JSON format.
Since earlier Input was just a plain text it was difficult to use this input in case you want to resubmit workflow from tctl.
Now with JSON we can quickly copy the input & resubmit a workflow from tctl.
Also it's easier to read each Input property more clearly.
Thank you team and keep up the good work. :relaxed:
PS: Raised a small bug as well here:
https://github.com/temporalio/web/issues/332
:grin:

i think this one is all credit to @feedmeapples  :) https://github.com/temporalio/web/pull/309

@ShubhSingh wow we appreciate the feedback, thank you!


Fix certs in K8s 1.19

This change incorporates
https://github.com/open-policy-agent/gatekeeper/pull/813 and
https://github.com/open-policy-agent/gatekeeper/pull/811 to allow HNC's
internal cert generator to work on K8s 1.19, which uses Go 1.15.
Tested: HNC cannot be installed on KIND 1.19 without this change, and
can successfully be installed and have all e2e test run with it.

/assign @yiqigao217
/assign @rjbez17

/hold

This doesn't require two approvals IMO. You only need to hold if you're not
confident
On Mon, Sep 21, 2020 at 9:54 AM Yiqi Gao<EMAIL_ADDRESS>wrote:

/hold
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/kubernetes-sigs/multi-tenancy/pull/1125#issuecomment-696129812,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AE43PZFIX6QKPRPDO3LRBGDSG5LJNANCNFSM4RUBY55Q
.

Thanks for the explanation!
/hold cancel


HDDS-8864. Remove redundant checkAcls() when caller is volume owner during key or prefix access

What changes were proposed in this pull request?
It is unnecessary to call checkAcls() twice when caller is volume owner in OzoneAclUtils#checkAllAcls.
Because the reason we had to split that into two calls in HDDS-5903 is because Ranger only has one {OWNER} tag, and that we want {OWNER} tag used in bucket/key level policies to be filled in with the bucket owner during ACL check when the caller is NOT the volume owner.
In the case where the caller is the Ozone volume owner, this hierarchical check is already enforced by the authorizer (OzoneNativeAuthorizer or RangerOzoneAuthorizer) internally. Thus it is unnecessary.
What is the link to the Apache JIRA
https://issues.apache.org/jira/browse/HDDS-8864
How was this patch tested?

All existing tests should pass.

Thanks @aswinshakil for the review.
I will delay the merge of this PR by a few more days.

@smengcl can we just merge this?

@smengcl can we just merge this?

Yes. Merged. Thanks for the reminder @arp7 .


[FEATURE REQUEST] Sunburst graph

Any plans to have support for sunburst graphs?
Similar to this
http://bl.ocks.org/mbostock/4348373

No plan to implement this kind of chart built-in but we will be happy to reference an external plugin if one exists or if somebody have time to create / maintain it.


Update Docs for JSON File Input

Inform users that JSON file content can be provided when passing parameters for actions, packages, and triggers

@csantanapr, ready for review.

cc @bjustin-ibm

@csantanapr, anything else?

rebase and merged ansible/roles/nginx/files/openwhisk-request.csr


Remove unused maven dependencies

Partial completion of https://github.com/deeplearning4j/deeplearning4j/issues/3470
What changes were proposed in this pull request?
remove unused maven dependencies
How was this patch tested?
Ran test suite.
Ongoing upstream tests

/cc @saudet

If we remove any of the ones that are related to JavaCPP, a whole lot of things are going to break. Those are not "unused"...

@agibsonccc @AlexDBlack, @saudet I have removed the controversial elements, then I haven't found a way to make this one fail for a while.

Thanks! Looks good to me now.


Strange pipeline behavior 'dev' public frontend does not show public-notices test data for 'Years of Operation' fields

Details of this bug is mentioned in #203.

While Anne-Marie was testing on new PR for the minor label change that deploy from pipeline to 'dev' she found that the existing public-notices (4 current test data) on the public frontend does not have the new fields show up (for Years of Operations) while she checked the admin frontend with the same FOM record and the Years of Operations appear in admin.
There was no code change for public frontend and only admin that time, nothing is wrong for public code and I checked with that as well after discussion with Anne-Marie.
However, while looking into frontend code, I also changed one more label at frontend although it was not mentioned from business, but just need to keep it consistent with the labeling change at admin. The pipeline redeployed with 'public' end change this time, and after deployment, the fields (for Years of Operation) shows up.
The strange behaviors of the public frontend described above need some investigation. It feels like the admin connects to the backend that has the database with test data migrated correctly while public connects to the backend that has the database with test data migration incorrectly, but how could that happen?
This probably can be isolated only on 'dev' environment before PR merged into TEST as we were looking into 'test data', if TEST are being tested manually properly by Anne-Marie and Julius.

Details of this bug is mentioned in #203.

While Anne-Marie was testing on new PR for the minor label change that deploy from pipeline to 'dev' she found that the existing public-notices (4 current test data) on the public frontend does not have the new fields show up (for Years of Operations) while she checked the admin frontend with the same FOM record and the Years of Operations appear in admin.
There was no code change for public frontend and only admin that time, nothing is wrong for public code and I checked with that as well after discussion with Anne-Marie.
However, while looking into frontend code, I also changed one more label at frontend although it was not mentioned from business, but just need to keep it consistent with the labeling change at admin. The pipeline redeployed with 'public' end change this time, and after deployment, the fields (for Years of Operation) shows up.
The strange behaviors of the public frontend described above need some investigation. It feels like the admin connects to the backend that has the database with test data migrated correctly while public connects to the backend that has the database with test data migration incorrectly, but how could that happen?
This probably can be isolated only on 'dev' environment before PR merged into TEST as we were looking into 'test data', if TEST are being tested manually properly by Anne-Marie and Julius.

I'm pretty sure this is due to caching - FOMs updated in Admin not yet shown in public due to the cache for the online public notice data, so not an actual issue. Problem resolves when redeploying API (forces cache refresh).


KIP-429: cleanup PartitionAssignor & friends POC

Main changes of this PR

Deprecate old consumer.internal.PartitionAssignor and add public consumer.PartitionAssignor
Refactor assignment/subscription data related classes for cleaner separation of user and consumer-provided state, plus easier to evolve API

Other previously-discussed cleanup included in this PR:

Remove Assignment.error added in pt 1
Remove ConsumerCoordinator#adjustAssignment added in pt 2

Just @hachikuji 's comment about naming of joinMetadata. I'm a bit in favor of the current name as it is consistent with assignment, whereas now we have:
Subscription -> joinMetadata
Assignment -> userData

clients:checkstyleTest failed.

Closing in favor of PR #7108


Documented the settings variable for WAGTAIL_USAGE_COUNT_ENABLED

Fixes #1723

Thanks @jjanssen!
If I'm not mistaken counter doesn't show images that were used in rich text editor or in stream field. How do you think, it's worth mentioning?

Thanks for the feedback @m1kola . I think it's worth mentioning in a note just beneath the description. I'll look into it and send an update with the note.

Thanks!

Last I checked, this counter also doesn't take usage by snippets into account, although I think it should. Snippets could be a major component of your Wagtail site; for example, Student models are snippets that have foreign keys to profile photos but those photos appear to be unused in the image browser UI.


Add unit test for NetworkConnect

I was time for NetworkConnect to be tested 🐒.
🐸
Signed-off-by: Vincent Demeester<EMAIL_ADDRESS>

LGTM

Updated :wind_chime:

@vdemeester thanks - LGTM


cpu mode host-passthrough - Unsupported argument

System Information
Linux distribution
Fedora 36
Terraform version
Terraform v1.2.7

Provider and libvirt versions
0.6.14

Description of Issue/Question
Hi, i'm trying to provision my vm with cpu  mode = "host-passthrough", i added the block according to the documentation, but i get this error:
terraform plan
╷
│ Error: Unsupported argument
│
│   on vm-create.tf line 22, in resource "libvirt_domain" "PROV-TST":
│   22:   cpu = {
│
│ An argument named "cpu" is not expected here. Did you mean to define a block of type "cpu"?
This is my terraform code:
`terraform {
required_providers {
libvirt = {
source = "dmacvicar/libvirt"
}
ansiblevault = {
source = "MeilleursAgents/ansiblevault"
}
}
}
provider "libvirt" {
uri   = "qemu+ssh://root@xxx.xxx.xxx.xxx/system?keyfile=/home/xxxxx/.ssh/id_rsa&sshauth=privkey"
}
provider "ansiblevault" {
vault_path  = "/xxx/.xxxxxx.txt"
root_folder = "/xxx/"
}
resource "libvirt_volume" "volume-PROV-TST_01_BOOT" {
name = "PROV-TST_01_BOOT.qcow2"
pool = "vms"
format = "qcow2"
size =<PHONE_NUMBER>
}
resource "libvirt_volume" "volume-PROV-TST_02_OS" {
name = "PROV-TST_02_OS.qcow2"
pool = "vms"
format = "qcow2"
size =<PHONE_NUMBER>
}
resource "libvirt_domain" "PROV-TST" {
name   = "PROV-TST"
memory = "2048"
vcpu   = 2
cpu = {
mode = "host-passthrough"
}
network_interface {
network_name = "lan"
}
network_interface {
network_name = "priv-iscsi1"
}
network_interface {
network_name = "priv-iscsi2"
}
disk {
volume_id = "${libvirt_volume.volume-PROV-TST_01_BOOT.id}"
}
disk {
volume_id = "${libvirt_volume.volume-PROV-TST_02_OS.id}"
}
video {
type = "virtio"
}
console {
type = "pty"
target_type = "serial"
target_port = "0"
}
graphics {
type = "spice"
listen_type = "address"
autoport = true
}
}
`
Is this a bug? Or cpu mode is not supported anymore in the provider?

Hi,
the host-passthrough mode should be supported.
Can you try without equals sign (=) after cpu?
cpu { # <--
  mode = "host-passthrough"
}

Yup, that is it. Can't believe i didn't noticed that.
Thank you.

Since I stumbled over the same: Could you please also correct the documentation?
https://registry.terraform.io/providers/multani/libvirt/latest/docs/resources/domain#cpu-mode
That's why at least I tapped into that issue...

I believe this is fixed right now. @mvrk69 could you close the issue?


legacy: split attestation handling phases

(WIP; I need to add some coverage.)
Only persist attestations once the file is created.

@woodruffw FYI, this still needs some work.


Doesn't render collapsible headers

Use rothgar/awesome-tuis as an example:

likewise, clicking on the links (anchors) it does not scroll to the correct section, and instead re-fetches the page

If I recall correctly, collapsing sections aren't part of Markdown, but more of an inline HTML thing; Frogmouth is of course a Markdown viewer.
However, if a plugin happens to exist for the parser we use, we can look at using that (I've done some work in Textual to make it easier to use plugins, and so some work on Frogmouth to make use of this will be happening soon).

aah, you're right. thank you for being so on top of all these issues! lemme take a look.


Added logSmartSnippetSuggestionShowMore and logSmartSnippetSuggestionShowLess

https://coveord.atlassian.net/browse/KIT-1595

For the SmartSnippet, the equivalent action is called expand/collapse, but SmartSnippetSuggestions already has an expand/collapse action for when we expand/collapse the accordion.

👍 Maybe some doc review is required though

Meh, not strictly necessary.


@qualweb/qw-element: fix unit test

This PR addresses #12 and #26
The package doesn't actually have any unit tests right now. This PR only fixes the actual execution/compile errors.

The lack of a changeset is intentional. There are no changes to the function of the package, so no release should be made.


wrong shape after using concat when using HybridBlock

import mxnet as mx
from mxnet import gluon
from mxnet.gluon import nn

class TestNet(gluon.HybridBlock):
    def __init__(self, **kwargs):
        super(TestNet, self).__init__(**kwargs)
        with self.name_scope():
            self.fc = nn.Dense(128)

    def hybrid_forward(self, F, x):
        print "hybrid: origin shape of x is {}".format(x.shape)
        y = F.concat(self.fc(x), self.fc(x))
        print "hybrid: change shape of x to {}".format(y.shape)
        return self.fc(x)

net = TestNet()
net.collect_params().initialize()
x = mx.nd.normal(shape=(10,103))
net(x)

print "ndarray: shape of x is {}".format(mx.nd.concat(x,x).shape)

result:
hybrid: origin shape of x is (10L, 103L)
hybrid: change shape of x to (10L, 256L)
ndarray: shape of x is (10L, 206L)

why the shape changed is (10, 256) and not (10, 206)?  @ZiyueHuang
git rev-parse HEAD
7232b1292221adf3ed19169540f8813dcd63bb71

@tornadomeet It seems right. self.fc will output shape (10, 128), and concat(fc_out, fc_out, dim=1) will output shape (10L, 256L).

oh.. got it , thanks @ZiyueHuang


Compilation error under Angular 16: Cannot find module 'src/regions' or its corresponding type declarations.

Full error:
Error: node_modules/@linode/api-v4/lib/account/types.d.ts:2:24 - error TS2307: Cannot find module 'src/regions' or its corresponding type declarations.

2 import { Region } from 'src/regions';
                         ~~~~~~~~~~~~~

This is a regression of issue #7686 and was introduced in this commit.

Thank you for opening this issue! We released the fix in<EMAIL_ADDRESS>Please reach out to us if you have anymore issues!


MGMT: Added Signing Credential Type to SAML Client screen

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


error configure header-only

When i try to create the Header-only Library i get this error: "error: unrecognized linear algebra library". Eigen is installed.
I installed the eigen library 3.4.0 via cmake/make install on debian. Eigen exist in folder /usr/local/include/eigen3/
Any idea to solve the problem? Thanks.

I cannot replicate this error on my end. Running:
./configure --header-only-version

does not require you to specify a backend linear algebra library.


side application error

hi my friend i have this error

run update script and then

service npanel start

it should fix most problems causing server not being able to start

i try update and start again have this problem

run these commands and show me the output

service npanel stop
cd /opt/Npanel/
./Npanel

root@V2rayserver:~# service npanel stop
root@V2rayserver:~# cd /opt/Npanel/
root@V2rayserver:/opt/Npanel# ./Npanel
network checks...
free port: 80
free port: 443
free port: 2060
free port: 2061
BBR ...
(output: "", exitCode: 0)
(output: "\e[H\e[J\e[3J---------- System Information ----------\n OS      : Ubuntu 20.04.5 LTS\n Arch    : x86_64 (64 Bit)\n Kernel  : 5.4.0-139-generic\n----------------------------------------\n Automatically enable TCP BBR script\n\n URL: https://teddysun.com/489.html\n----------------------------------------\n\nPress any key to start...or Press Ctrl+C to cancel\n\n\e[1;31;32m[Info] \e[0mTCP BBR has already been enabled. nothing to do...\n", exitCode: 0)
disabling ipv6 ...
net.ipv6.conf.all.disable_ipv6 = 1
net.ipv6.conf.default.disable_ipv6 = 1
net.core.default_qdisc = fq
net.ipv4.tcp_congestion_control = bbr
Firewall stopped and disabled on system startup
could not parse users.db; did you corrupt it? , loaded no users.
input(1, 0) Error: { expected

so the users.db file is somehow corrupt. if you dont mind the users you have created , you sohuld delet users.db (in /opt/Npanel/)
otherwise you should resotre your backup , i already told in backup section that users.db and panel.json are the 2 files that you should backup before you update.
since i tested my own vps with update script, it is not a corruption made by the update script, it may be you that edited the file or other reasons.

Why did this problem occur?

can you tell me how to reproduce this problem? , what exactly did you do before this happens?

this error happens because the file holding your users data and settings has invalid data.

I didn't do anything on the server, one of the users called me and said that he couldn't connect, and I checked the server and found this error.

Yes, I have a backup copy, but unfortunately it is not available to me now

the first thing you have to do in this case is to restore your backup of users.db.
if you dont have a backup. then delete users.db file and start npanel . then re-create the users you had before.

service npanel stop
cd /opt/Npanel/
rm users.db
service npnael start

please try to take a backup of users.db every day untill i address this problem and fix it .

Thank you my friend for your attention, I hope this problem will be solved soon

if you haven't already deleted users.db
please send it to me here so i can investigate

Unfortunately, I deleted it a few moments ago and now I am creating new users.  But I have the backup copy and I can send it to you in the next few hours

ok that would be fine, Thanks

Update to 0.1.5 since this version is more stable.

Hello, my friend, I just got access to the system. This user file was corrupted
users.txt

Thank you, i investigate and will release new version when i feagure out the reason behind it.

Thank you very much, my friend, I hope you can successfully complete this project

This problem is fixed after the auto-backup and restore implementation (v 02.2) , the cause of it was restarting the system when the users.db file was being overwritten.


Add support for swimlanes

Is your feature request related to a problem? Please describe.
Swimlanes are a common concept in Kanban implementations.
Describe the solution you'd like
Swimlanes have a name and are presented horizontally across Lanes. A card belongs to the intersection of a Lane and a Swimlane. Like this : https://help.rallydev.com/sites/default/files/multimedia/swimlanes_expedite.gif

I see, thanks.