Why psnr values differ so much when input Sony and Fuji rgb datasets

It's hard to ignore that in your paper Table 3, the psnr of Sony rgb input is 17.4 while another psnr is 25.11.  I couldn't understand why they differ so much when both are rgb images. I mean，if input rgb images,  they must have gone through the same process before put into the network.
And would you mind to explain how you convert your raw datasets to rgb images, and how to understand this difference?

I used rawpy to process the raw images to sRGB images. And then use these images as input to train the network for comparisons.

Could you please provide the code about how process the raw images to sRGB for experiment correctly?

Hi @KeqiWangSXuniversity,
This is from the author's code itself. Assuming raw is the RAW data, and rgb  is the processed RGB data you'd like to have, you can do -
import rawpy
rawpath = '' # Provide the path to the raw file here
raw = rawpy.imread(rawpath)
rgb = raw.postprocess(use_camera_wb=True, half_size=False, no_auto_bright=True, output_bps=16)


ConversationalChatAgent: Allow customizing TEMPLATE_TOOL_RESPONSE

It's currently not possible to change the TEMPLATE_TOOL_RESPONSE prompt for ConversationalChatAgent, this PR changes that.

When do you plan to merge?


Installing error sklearn

Hi,
I am trying to install this package using the recommended:
pip install CEEMDAN_LSTM
But I get the dependencies error:
The 'sklearn' PyPI package is deprecated, use 'scikit-learn'
      rather than 'sklearn' for pip commands.

      Here is how to fix this error in the main use cases:
      - use 'pip install scikit-learn' rather than 'pip install sklearn'
      - replace 'sklearn' by 'scikit-learn' in your pip requirements files
        (requirements.txt, setup.py, setup.cfg, Pipfile, etc ...)
      - if the 'sklearn' package is used by one of your dependencies,
        it would be great if you take some time to track which package uses
        'sklearn' instead of 'scikit-learn' and report it to their issue tracker
      - as a last resort, set the environment variable
        SKLEARN_ALLOW_DEPRECATED_SKLEARN_PACKAGE_INSTALL=True to avoid this error

I noticed that indeeed the setup.py call sklearn. Are there any plans to update this?
Thanks

Thanks for reporting the error. I will fix it ASAP.


Add API Access for Fields in Custom Array Component

We are trying to build an Array type component which comes with some default fields but can can be extended with some more fields the user chooses.
But we would also like to be able to remove the default fields the Array component comes with.
The new fields can be added and deleted from the right side properties UI. We tried our best to find a way to insert our default fields to this section, but we are yet to find a way to do that.
Any suggestions on how we can achieve this based on existing capabilities would also be highly appreciated!

Hi @ab5y - the Dynamic Fields API added in #280 is available in the canary build, and might help your use case. Let me know if that helps.


bug: white page/failed to load when session is innactive

When you leave the application running for a couple of days and session expire it doesn't seem to auto refresh and you get the white page "failed to load" (in Safari).

will fix https://github.com/dto-btn/ssc-assistant/issues/82 at the same time

I initially this was because of CORS settings in the App itself but even after adding them I still get the issue. Mind you this only happens when the app is deployed in Azure and when the tab stays innactive (in Chrome):
This might be due to resource caching and name chagning: https://github.com/facebook/create-react-app/issues/9145#issuecomment-1931751457


Switch to cypress-multi-reporters

Description
This fixes an issue we were seeing where the Cypress upgrade to 3.7.0 started failing due to an incompatibility between mocha 6 and mocha-multi-reporters that had been failing silently since that upgrade.

Since the build is broken, I'm going to close this since it isn't mergeable with the expectation that this work will be picked when we undertake this story: https://dp3.atlassian.net/browse/MB-724.


ReaderWriterLock in PropertyDictionary

Fix #4368 by using a ReaderWriterLockSlim instead of a full lock. This
will allow the case where a.Equals(b) and b.Equals(a) run concurrently,
since they both only want a reader lock on their _properties: to
enumerate their own properties, and on the other to index in.
@tmeschter suggested this approach offline, and it seems obviously correct (in hindsight!).

On a related note, is there any possibility that one of the PropertyDictionarys could be modified during an Equals call?

/azp run


Resolve issue 778

Description
This PR fixes issue #778 . If parallel state machines have the same name, and are in the same nested state machine, error "E073 Duplicate Parallel State Machine Name in Nested State Machine" is thrown. If parallel state machines have the same name and are in the same parallel region, error "E074 Duplicate Parallel State Machine Name in Parallel Region" is thrown.
Tests
Added tests "duplicateParallelStateMachineName_InSameNsm" and "duplicateParallelStateMachineName_InSameRegion" to UmpleParserStateMachineTest.java. Corresponding test files are 487_parallelStateMachines_*.ump

@vahdat-ab Following up to our meeting, I will now remove "E074 Duplicate Parallel State Machine Name in Parallel Region", the code associated with generating that error, and the test cases and manual examples associated with that error (listed below). I will also rename E073 to "E073 Duplicate Parallel State Machine Name". After updating this PR, it should be good to go, and the original issue #778 will be fixed.
I will create a separate issue which will address the errors in the generated Java code when there are two parallel state machines in different states that have the same name. The examples that show this issue are:

487_parallelStateMachines_sameRegionSameNames.ump
487_parallelStateMachines_sameRegionSameNames_2.ump
E074DuplicateParallelStateMachineNameInParallelRegion1.ump
E074DuplicateParallelStateMachineNameInParallelRegion2.ump


feat(node-bindings): add methods for returning instance urls

Motivation
The existing methods such as endpoint() and ws_endpoint() would return the URLs as a String. One would have to parse the String as a URL and then pass it to the provider.
Solution
Adds two methods endpoint_url() and ws_endpoint_url()  which return the HTTP and WS respectively for both anvil and geth instances.

PR Checklist

[ ] Added Tests
[x] Added Documentation
[ ] Breaking changes

cc @mattsse


fix: Implement workaround for the BackupUnits API bug

What does this fix or implement?
There is a problem with the BackupUnits API:

GET on all the BackupUnits works properly;
GET on BackupUnits/BackupUnitID doesn't work properly (the state in the response is BUSY and some fields are not populated properly)

Because of that, the client method BackupunitsFindById doesn't work properly.
I will open a bug for the API team, but until then I implemented a workaround: I created a method that simulates the BackupunitsFindById client method. The difference is that this method fetches all the backup units and then returns the searched one using ID matching.

Checklist

[ ] PR name added as appropriate (e.g. feat:/fix:/doc:/test:/refactor:)
[ ] Tests added or updated
[ ] Documentation updated
[ ] Changelog updated and version incremented (label: upcoming release)
[ ] Github Issue linked if any
[ ] Jira task updated

Changelog and link the issue with a  fixes #number


Passing a method as input argument

Hi,
I have a c# compiled dll method that receives a method and i don't know if it's possible to handle this. Thanks in advance.
Here is the example:
void SagaRegCalbackReqInput(ReqInputFunc func)
string ReqInputFunc(string title, string prompt, string strini)

Maybe you should use callback?

I have this example, but is not firing up the callback method, am i doing this right?
    const lib = koffi.load(config.SAGA.DLL);
    const saga_ini = lib.stdcall('saga_ini', 'int', ['string', 'string', 'string']);
    const saga_eval = lib.stdcall('saga_eval', 'string', ['string']);

    const ReqInputFunc = koffi.proto('ReqInputFunc', 'string', ['string', 'string', 'string']);
    const SagaRegCalbackReqInput = lib.stdcall('SagaRegCalbackReqInput', 'string', [koffi.pointer(ReqInputFunc)]);

    const exampleCallback = (title: string, prompt: string, strini: string) => {
        console.log("I'M RUNNING")
        console.log(title, prompt, strini);
    }

    const saga_ini_res = saga_ini(app.sagaini, username, password);
    console.log("saga_ini_res", saga_ini_res)

    SagaRegCalbackReqInput(exampleCallback);

    const input_res = saga_eval(`input('title', 'prompt', 'defaultvalue')`);

@neko-para can you help? Am I missing something?

Callback has two type: transient and registered.
What you are using is the transient type, which only allows the callback to be called in the before the function returned:
SagaRegCalbackReqInput(exampleCallback);
// From here the callback that passed into your library is invalid, as SagaRegCalbackReqInput returned.

As you wish the callback to be called after, you should use registered type. The document already shows how to do it.

If the DLL creates a new thread internally to handle the callback it works the same way?

To simplify i removed the DLL thread and o wrote this method in my application:
public registerSagaRegCalbackReqInput = () => {
    const ReqInputFunc = koffi.proto('const char *ReqInputFunc(const char *title, const char *prompt, const char *strini)');
    this.SagaRegCalbackReqInput = this.lib.stdcall('void SagaRegCalbackReqInput(ReqInputFunc *func)');

    const cb1 = koffi.register((title: string, prompt: string, strini: string) => {
      return `yes`;
    }, koffi.pointer(ReqInputFunc));

    this.SagaRegCalbackReqInput(cb1);
}

When i run the DLL method that fires this registered callback it does nothing... 🤷♂

Oops, probably c# compiled delegated methods are not compiled that way. Is this possible to work with koffi?
private string MyReqInput(string title, string prompt, string strini) {
    return 'yes';
}

delegate string ReqInputFunc(string title, string prompt, string strini);
static extern void SagaRegCalbackReqInput(ReqInputFunc func);

SagaRegCalbackReqInput(new ReqInputFunc(MyReqInput));


Experts bot doesn't use channel configured

When deploying this into a Team and selecting the channel it does not use this channel. The general channel is used for all messages.

Thanks for sharing the feedback. Currently the app only uses General channel to send notifications, which is configured from admin configuration web application.

Thanks for the answer! Shall look into changing via admin.


For variables named with ' prefix; ' is missed during suggestion

Description:

Sample code:
function rate(Currency 'from, Currency to) {
    string fromCurrency = 'from;
    string toCurrency = to;
    string responseKey = fr
}

Suggested Labels:

Suggested Assignees:

Affected Product Version:
OS, DB, other environment details and versions:
Steps to reproduce:
Related Issues:

As per a related discussion at https://github.com/ballerina-platform/ballerina-lang/issues/28567#issuecomment-779040893 and a suggestion made at https://github.com/ballerina-platform/ballerina-lang/issues/28567#issuecomment-779521353, will be updating the PR https://github.com/ballerina-platform/ballerina-lang/pull/29964 with an approach similar to https://github.com/ballerina-platform/ballerina-lang/pull/29701.


Query Complexity Analysis for GraphQL

Description:
As a part of adding security features, the GraphQL services should have a way to calculate the query complexity and stop executing the complexity of an incoming query is exceeding the allowed complexity. The developer should be able to add the complexity of a given resource, so the complexity of an operation can be calculated.

Closing as this is now completed.


caching concurrently on high volume requests

Hello,
I've implemented your code with some minor adjustments to my rest endpoint, which takes a url with parameters and returns json document. In that scenario, your code is perfect for middleware []byte storage. However, I've ran unto a problem where stress testing tool such as bombardier makes with the following parameters "/bombardier --method=GET --latencies --duration=10s --connections=150 <IP_ADDRESS>:3030/evs/1/at/153000,674000,1404000,1433926/" and hits the endpoint where I've enabled the caching support. I see that the endpoint, instead of caching a first request, it caches few more past the first one as depicted in the following log output:
INFO	2019/08/09 12:51:54 Starting server on :3030
New page cached: /evs/1/at/153000,674000,1404000,1433926 for 100s
New page cached: /evs/1/at/153000,674000,1404000,1433926 for 100s
New page cached: /evs/1/at/153000,674000,1404000,1433926 for 100s
New page cached: /evs/1/at/153000,674000,1404000,1433926 for 100s
New page cached: /evs/1/at/153000,674000,1404000,1433926 for 100s
New page cached: /evs/1/at/153000,674000,1404000,1433926 for 100s

Any thoughts?

That happens because there's a lot of concurrent requests which will MISS the first cache lookup, so they'll all execute your handler and then cache the response. Subsequent requests will then HIT the cache lookup.
To avoid that you'll need to use a Mutex to lock the execution to a single goroutine per time https://gobyexample.com/mutexes

I will try to make a change and post a gist for you to review, is that ok?

Following your suggestion, I've implemented mutex operation on the middleware, please see it here: https://gist.github.com/mlevkov/41094e7d536c720e2b09f617347eb1d2
Here are the performance numbers
--> Before:
Statistics        Avg      Stdev        Max   Reqs/sec      8028.24    1397.01   14279.64   Latency       18.67ms    30.76ms      1.99s   Latency Distribution      50%     8.37ms      75%    25.89ms      90%    49.43ms      95%    67.02ms      99%   107.95ms   HTTP codes:     1xx - 0, 2xx - 722736, 3xx - 0, 4xx - 0, 5xx - 0     others - 0   Throughput:    21.01MB/s
--> After:
Statistics        Avg      Stdev        Max   Reqs/sec     17715.84    6642.67   36610.48   Latency        8.46ms     2.80ms    99.89ms   Latency Distribution      50%     5.86ms      75%    10.31ms      90%    17.43ms      95%    21.20ms      99%    29.46ms   HTTP codes:     1xx - 0, 2xx - 1595959, 3xx - 0, 4xx - 0, 5xx - 0     others - 0   Throughput:    46.40MB/s
Please let me know if I've implemented the change correctly. Any other suggestions?

That's a good implementation, but I'd suggest using RLock as well.
It'd be like:

get a read lock and defer read unlock
try to get from cache
if found, respond and return
if not found, get a lock and defer unlock
try to get from cache
if found, respond and return
if not found, do handle the request
save to cache and respond

a read lock doesn't block other read locks, so you can have multiple read locks
you only need a "write lock" if you're going to write the cache
I expect this to also give you slightly higher throughput

in addition to the mutex or in place of the mutex?

You can use the same mutex, it should have a RLock func there

Oops. Minor mistake, your mutex should be this one https://golang.org/pkg/sync/#RWMutex

I made a quick change, followed your proposed change and testing it now.

I realized that such was the case. Here is the gist:
https://gist.github.com/mlevkov/9db36bed5b818e17f5e958a4dda52e3d

the above implementation performance is:
Statistics        Avg      Stdev        Max
  Reqs/sec      7767.36    1390.96   13622.35
  Latency       19.33ms    23.62ms   477.92ms
  Latency Distribution
     50%     6.29ms
     75%    11.59ms
     90%    76.47ms
     95%    94.72ms
     99%   125.68ms
  HTTP codes:
    1xx - 0, 2xx - 698412, 3xx - 0, 4xx - 0, 5xx - 0
    others - 0
  Throughput:    20.30MB/s

After that Lock, you’ll need an additional lookup to the cache. The reason being that two or more goroutines might try to acquire that lock, but only the first should do the work. The subsequent goroutines, after acquiring the lock, should then hit the cache.

Something like this ?
func (app *application) cached(duration string, handler func(w http.ResponseWriter, r *http.Request)) http.Handler {
	var mutex = &sync.RWMutex{}
	return http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
		mutex.RLock()
		defer mutex.RUnlock()
		content := app.cache.Get(r.RequestURI)
		if content != nil {
			//fmt.Print("Cache Hit!\n")
			w.Write(content)
		} else {
			mutex.RLock()
			defer mutex.RUnlock()
			if content = app.cache.Get(r.RequestURI); content != nil{
				//fmt.Print("Cache Hit!\n")
				w.Write(content)
				return
			} else {
				c := httptest.NewRecorder()
				handler(c, r)

				for k, v := range c.Result().Header { // c.HeaderMap (deprecated)
					w.Header()[k] = v
				}

				w.WriteHeader(c.Code)
				content := c.Body.Bytes()

				if d, err := time.ParseDuration(duration); err == nil {
					fmt.Printf("New page cached: %s for %s\n", r.RequestURI, duration)
					app.cache.Set(r.RequestURI, content, d)
				} else {
					fmt.Printf("Page not cached. err: %s\n", err)
				}
			}
			w.Write(content)
		}

	})
}

Yep, how does that perform? Also, that last write should be inside the brackets

ok, let me make that change. Here is the performance from that one:
Statistics        Avg      Stdev        Max
  Reqs/sec      7729.11    1396.59   13176.17
  Latency       19.42ms    23.34ms   475.55ms
  Latency Distribution
     50%     6.28ms
     75%    11.71ms
     90%    76.80ms
     95%    94.16ms
     99%   125.27ms
  HTTP codes:
    1xx - 0, 2xx - 695032, 3xx - 0, 4xx - 0, 5xx - 0
    others - 0
  Throughput:    20.21MB/s

here is the performance feedback from the last run with change of w.Write inside the brackets:
Statistics        Avg      Stdev        Max
  Reqs/sec      7715.70    1414.81   13246.45
  Latency       19.46ms    23.57ms   349.97ms
  Latency Distribution
     50%     6.27ms
     75%    11.56ms
     90%    77.64ms
     95%    94.57ms
     99%   125.92ms
  HTTP codes:
    1xx - 0, 2xx - 693625, 3xx - 0, 4xx - 0, 5xx - 0
    others - 0
  Throughput:    20.16MB/s

perhaps I'm not implementing this correctly. Would you be open to making a sample that you see fit the intent correctly?
Here is the code that I performs well so far:
{
	var mutex = &sync.Mutex{}
	return http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
		mutex.Lock()
		defer mutex.Unlock()
		content := app.cache.Get(r.RequestURI)
		if content != nil {
			//fmt.Print("Cache Hit!\n")
			w.Write(content)
			return
		} else {
			//mutex.RLock()
			//defer mutex.RUnlock()
			//if content = app.cache.Get(r.RequestURI); content != nil{
			//	//fmt.Print("Cache Hit!\n")
			//	w.Write(content)
			//	return
			//} else {
				c := httptest.NewRecorder()
				handler(c, r)

				for k, v := range c.Result().Header { // c.HeaderMap (deprecated)
					w.Header()[k] = v
				}

				w.WriteHeader(c.Code)
				content := c.Body.Bytes()

				if d, err := time.ParseDuration(duration); err == nil {
					fmt.Printf("New page cached: %s for %s\n", r.RequestURI, duration)
					app.cache.Set(r.RequestURI, content, d)
				} else {
					fmt.Printf("Page not cached. err: %s\n", err)
				}
				w.Write(content)
			}
		//}

	})
}

the performance outcome of this code is:
Statistics        Avg      Stdev        Max
  Reqs/sec     17194.11    6801.90   38327.80
  Latency        8.73ms     3.14ms   122.09ms
  Latency Distribution
     50%     5.92ms
     75%    10.81ms
     90%    18.09ms
     95%    21.99ms
     99%    30.59ms
  HTTP codes:
    1xx - 0, 2xx - 1546594, 3xx - 0, 4xx - 0, 5xx - 0
    others - 0
  Throughput:    44.96MB/s

I'm using bombardier for testing at 150 concurrent connections for 90 second period, just for reference.

That’s interesting,  I’d expect it to be better or equal in terms of perf. Will try to have a look at this later on.

That would be great. Thank you!

Hey @mlevkov, try this version.
The difference is that I'm using RLock first and only then a Lock (if value not found on cache).
func cached(duration string, handler func(w http.ResponseWriter, r *http.Request)) http.Handler {
	var mutex = &sync.RWMutex{}
	return http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
		mutex.RLock()
		content := storage.Get(r.RequestURI)
		mutex.RUnlock()
		if content != nil {
			//fmt.Print("Cache Hit!\n")
			w.Write(content)
		} else {
			mutex.Lock()
			defer mutex.Unlock()
			if content = storage.Get(r.RequestURI); content != nil {
				//fmt.Print("Cache Hit!\n")
				w.Write(content)
				return
			}

			c := httptest.NewRecorder()
			handler(c, r)

			for k, v := range c.Result().Header { // c.HeaderMap (deprecated)
				w.Header()[k] = v
			}

			w.WriteHeader(c.Code)
			content := c.Body.Bytes()

			if d, err := time.ParseDuration(duration); err == nil {
				fmt.Printf("New page cached: %s for %s\n", r.RequestURI, duration)
				storage.Set(r.RequestURI, content, d)
			} else {
				fmt.Printf("Page not cached. err: %s\n", err)
			}
			w.Write(content)
		}

	})
}


setup netlify for static site hosting of docs

https://netlify.com

done.
builds on every change.
https://drawio-intellij-plugin.netlify.app/


Use server's registry access in server audience component conversions

fixes #145 (kind of, consumers need to move from #asComponent to deprecated FabricAudiences#toAdventure where registry access is required)

Note to anyone watching this PR: you will need to suppress the deprecation warning for now, we are making further changes including removing the deprecation in 6.x: #147


Added new parameter: apiName

This will create a folder within reference as a parent folder to the api.
This is an optional parameter, and I have manually tested that it does not break backwards compatibility.
Also made one small change to how we display the version number for the API in presidium as the sizing used did not work nicely (changed from H1 to H3)

I added a small comment but besides that LGTM!

Yes! I was able to


adding of project in tier 1

Adding management system app like hotel management  would provide a great project ! i could contribute towards the code as i have made it ! let me known if you are interested in this plan . it would require guest reservation , checkout,dining , available rooms ,pricing from top rooms to medium level ones .

Interested

Interested
On Fri 12 Jun, 2020, 2:15 PM Augustine-source<EMAIL_ADDRESS>wrote:

Interested
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/florinpop17/app-ideas/issues/193#issuecomment-643152819,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AM6D5F72GFTNBPDXGCVAUALRWHTJPANCNFSM4NZGMFPA
.

Hi,
You want to contribute towards the different ideas for this project?
On Fri, 12 Jun 2020, 2:13 pm Augustine-source<EMAIL_ADDRESS>wrote:

Interested
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/florinpop17/app-ideas/issues/193#issuecomment-643152819,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ANEGEITM6YXLNVDU3Z6WBJ3RWHTEVANCNFSM4NZGMFPA
.

Oh yes

If you have modification on this project idea you could follow me and revert back there ! open to ideas


Update ocp-dasd timing

Tracking issue is #90 ...
The ocp-dasd.sh script does not currently have its own service, it is run at the tail-end of the ipconf.sh script.  As we move to these scripts/services not just being firstboot, and change the way that OCP guests are installed/implemented as part of Finna, the timing of ocp-dasd must become independent.

ocp-dasd.sh has been moved to elan-tools as part of the interface redesign, and was already updated to support being called as part of the new "ocp-define-cluster" workflow.  Closing this from here.


multi-versions-server-url-missing-version incorrectly detected for OAS v2

Identified with the example of the CyberVision API (see attached)
cybervision-download.json
The basepath is declared with a version: "3.0" BUT an error is raise byt the contract analyzer,
see  https://wwwin-github.cisco.com/stsfartz/oas_docs/blob/master/openapi_docs/cybervision/README.md#contract-completeness

note that this may be due to a corner case of Cyber Vision where the version is 3.0 instead of v3


CouchDB 2.0 support

I've heard from many folks that CouchDB 2.0 doesn't work with this module. At the very least we should add tests using CouchDB 2 in addition to 1.6 and see which tests are failing.

Hey @nolanlawson, are there still known issues with CouchDB 2 and this project?

Hi,
I am using pouchdb-auth. with CouchDB 2.0 (version 20 Dec. 2016) without
problems - have not encountered any at least.
Br,
Sinan
On 10 January 2017 at 23:15, Christopher Eyhorn<EMAIL_ADDRESS>wrote:

Hey @nolanlawson https://github.com/nolanlawson, are there still known
issues with CouchDB 2 and this project?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/nolanlawson/pouchdb-authentication/issues/128#issuecomment-271715407,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AFU80h406S4f_vYJMOv29JUtba8LINv5ks5rRAMLgaJpZM4Kp_Wq
.

Thanks @SinanGabel, I will give it a go.

they are using port 5986 instead of 5984 for the default _user database

Seems to work correctly. Closing.


Hype Machine support

Not the greatest code knock up (not to good at javascript yet), but it works. Available config options is user and type, where user is the username and type is either "history" or "loved". A working demo can be seen on my site with the small bug I unsure on how to fix appropriately.
P.S. I might get around to adding support for Twitter Intents and some of the other services I have seen in the issues. It seems to be a great way to practice my javascript knowledge.

Hey Blake,
Thanks for committing your time to this project.
It's really appreciated.

Hope they help you along to become an awesome JS developer.

Thanks Blake,
You're code should now be merged into this repository.


ci: Start testing 3.13 beta

Hit the first blocker: cffi support https://github.com/python-cffi/cffi/issues/71

Next blocker seems to be Pydantic, which will require updates to PyO3


Added multiple responses

For example if you have a 301 redirect then a 200 ok, the generator rendered responses in reverse order from history.


request: building a single file version

Is your feature request related to a problem? Please describe.
I'm building a screenshot feature on my Bot, using this package, building the webpage with Pug and rendering with Puppeteer, sadly I cannot import the package directly into the Puppeteer webpage, I can only make it work with the unpkg URL.
I can use the addScriptTag feature of Puppeteer, but relative import in the script won't work https://devdocs.io/puppeteer/index#pageaddscripttagoptions
Using unpkg is fine, but I rather not depend on them, especially when I can have the package locally, avoiding the extra 1-5 requests
Describe the solution you'd like

I was wondering if you could build a single file version of the package (minified or not) ?
Describe alternatives you've considered

Additional context
Using @skyra/discord-components-core package

Sorry but no can do. It's not something StencilJS supports: https://stenciljs.com/docs/distribution
An alternative solution to using unpkg and when using Sapphire framework is using our @sapphire/plugin-api and providing the proper JS file that way.

Alright, make sense, thanks for the idea of exposing the pacakge through a webserver


Release v1.8.0

Includes:

Funding a SOL Wallet via SOL topUpWithTokens via HexSolanaSigner (base58 secretKey constructed signer)

:tada: This PR is included in version 1.8.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Per-crate Cargo.lock checks?

Would it make sense to adapt cargo-hack for checking dependency version resolution per crate in a workspace? Especially with -Z minimal-versions.
If I have:

workspace

my_crate1

serde = "1.0.1"

my_crate2

serde = "1.0.123"

Then my_crate1 will only be tested with serde v1.0.123, but if I publish it outside of the workspace, someone else could use it with older dependencies I never tested with. The problem is not limited to direct dependencies in the workspace - the same thing can happen with transitive dependencies:

workspace

my_crate1

serde = "1.0.1"

my_crate2

wants_newer_serde = "7"

So I think the hack needed here is to remove crates from the workspace, re-link them as path dependencies if needed (or with [patch]? not sure), and test with individual Cargo.locks.

I this this can be done by completing https://github.com/taiki-e/cargo-minimal-versions/pull/23 and then porting it to cargo-hack (or just use it). (Or maybe by implementing https://github.com/taiki-e/cargo-hack/issues/216)
As for the minimal-versions check, AFAIK, the approach described here does not solve the problem of optional dependencies (https://github.com/taiki-e/cargo-minimal-versions/issues/6), so eventually, it would be best to implement the approach I described in https://github.com/tokio-rs/tracing/pull/2015#issuecomment-1092561025 in cargo-minimal-versions.
(In general, I would recommend using cargo-minimal-versions (https://github.com/taiki-e/cargo-minimal-versions) for minimal-versions check because it can automatically handle many of the things that can be problematic when doing minimal-versions check (and also supports the cargo-hack options because it calls cargo-hack internally). However, I'm not opposed to adding the ability to use cargo-hack to do that manually.)


Unexpected undo behaviour (cache issue?)

I apply a color token to a bunch of layers, and undo this using cmd+z in figma. This does remove the color fill, but does not remove the token. So now when I press update again, the fill will again be applied, which I think is causing a lot of frustration and is an unexpected behaviour. On further investigation, this seems to be an issue with our token cash, because when I restart the plugin, the tokens are actually not there on the layers after CMD+Z

Can we get a reproducible  screen cap of this?


🐛 [Bug report]: Create a play page not expanding to full height

Descrcibe the bug?
The form is not expanding to full height; instead, it covers a very small height, as shown below.

Steps to reproduce the bug?

yarn start to fire up the application locally
Click 'Create'
The form opens up

Expected behavior
It should open up as show in this screenshot posted here
Desktop (Please provide your system information)
OS: Apple M2 Ventura 13.1
Browser: Chrome
Mobile (Please provide your device information)
No response
Screenshot / Screenshare
No response
Relevant log output
No response
Code of Conduct

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

@ShreyaPrasad1209 is there a scroll bar appear that allow you to navigate some way?
Do you have a bigger monitor to test it if it opens better there?

I have to explicitly click on the scroll bar to navigate. This happened on my 24-inch monitor.

Got it. Thanks for reporting it. We will fix it soon but I hope you are able to make progress with play creation for now!

I can take up this issue if you want. Yes I'm filling up the form but the experience could be a lot better 🙈


When using the dev script env variables are not refreshed

I saw, that when you are using the dev script
"dev": "robo dev",

it does not refresh the env variables on reload.
Example:
I changed my api key for a service and he still used the old one after every reload first after exiting and rerunning the script it loaded the new key.

Resolved in #355


(FLK-W505) Doc line too long

Description
Docstring line lengths are recommended to be no greater than 79 characters.
Occurrences
There are 8 occurrences of this issue in the repository.
See all occurrences on DeepSource → deepsource.io/gh/mikeysan/retweet_anything/issue/FLK-W505/occurrences/

Hey, I have found all 8 occurrences where the number of characters are greater than 79.

What exactly is needed to be done with those lines?

I'm only seeing this now. I think the issues was resolved already, but we didn't attach this issue to it. I'll need to make a note about that later.
In the meantime, there are only two files left (if you click on the link above you will see them). The only thing that needs to be done is add the comment to multiple lines. Example:
This line:
# user.followers_count - no. of other users who are following this user (followers)

Could become:
# user.followers_count - no. of other users who are
# following this user (followers)

Resolved.


Move Cleanup.h from plugins/xdebug to include/tscpp/api .

PRs that are dependent on this PR:
https://github.com/apache/trafficserver/pull/9063
https://github.com/apache/trafficserver/pull/9044
https://github.com/apache/trafficserver/pull/8790

[approve ci autest]

[approve ubuntu]

[approve ci ubuntu]

Should Cleanup.h also be removed from /plugins/xdebug/Makefile.inc?
To confirm, this CPP API addition is to make these standard deleter/ptr definitions accessible for other plugins?

Should Cleanup.h also be removed from /plugins/xdebug/Makefile.inc?
yes
To confirm, this CPP API addition is to make these standard deleter/ptr definitions accessible for other plugins?
yes

[approve ci autest]

I haven't seen this proposed to the mailing list yet. Did I miss the email?

@randall there was an email titled "Proposed change to CPP API" a few weeks ago!


PHP Errors in Ajax Response warning when removing comments menu item

Query Monitor yelled at me after adding remove comments menu item to my array,
'wp-admin.{all}' => [
    ...
    'comments',
],

Here's what my console had to say,
component: "Core"
file: "wp-admin/includes/plugin.php"
key: "354ba22554f7809793fd8fbf0d259701"
line: 1769
message: "Invalid argument supplied for foreach()"
stack: (3) […]
  0: "remove_menu_page()"
  1: "Sober\\I\\A\\S\\Menu->Sober\\I\\A\\S\\{closure}()"
  2: "do_action('admin_init')"
  length: 3
  <prototype>: Array []
type: "warning"

Thanks @grahammtbr , added https://github.com/soberwp/intervention/blob/master/src/Admin/Support/Router.php#L60-L62, could you re-test from master? Should solve WP/AJAX related issues.

This fixed the warning, closing.


[WFCORE-3070] Found multiple secret keys sharing same CKA_LABEL

Synchronization issue when using FIPS enabled NSS DB as PKCS11 keystore.
https://issues.jboss.org/browse/WFCORE-3070
https://issues.jboss.org/browse/JBEAP-11693

@mchoma I was getting exception that in FIPS mode only standard trust manager can be used.

@pskopek , so probably you was hitting WFCORE-1135? Suficient solution to this was introduction of possibility to configure server fips compliant ssl context.
Havent't had that custom trust manager implementation its place? I mean that empty check* methods?
@darranl, can you have a look?

@mchoma the problematic commit has been reverted


AttributeError: 'Context' object has no attribute 'gi'

Hi everyone! I need to load annotations to the Apollo2.7 UcA track. I've previously used  add_transcripts_from_gff3_to_annotations.pl but it looks like the load_gff3 function from this library is the new way to go. I ran into the following error:
(apollo-arrow-env) ARS5APVN2WD42YY:Downloads mpoelchau$ arrow annotations load_gff3 --source https://apollo2-stage-node1-cbo.nal.usda.gov/apollo Galleria_mellonella test.gff
/Users/mpoelchau/Documents/programs/apollo-arrow-env/lib/python3.9/site-packages/urllib3/__init__.py:34: NotOpenSSLWarning: urllib3 v2.0 only supports OpenSSL 1.1.1+, currently the 'ssl' module is compiled with 'LibreSSL 2.8.3'. See: https://github.com/urllib3/urllib3/issues/3020
  warnings.warn(
Traceback (most recent call last):
  File "/Users/mpoelchau/Documents/programs/apollo-arrow-env/lib/python3.9/site-packages/arrow/decorators.py", line 16, in custom_exception
    return wrapped(*args, **kwargs)
  File "/Users/mpoelchau/Documents/programs/apollo-arrow-env/lib/python3.9/site-packages/arrow/decorators.py", line 46, in str_output
    print(wrapped(*args, **kwargs))
  File "/Users/mpoelchau/Documents/programs/apollo-arrow-env/lib/python3.9/site-packages/arrow/commands/annotations/load_gff3.py", line 51, in cli
    return ctx.gi.annotations.load_gff3(organism, gff3, source=source, batch_size=batch_size, test=test, use_name=use_name, disable_cds_recalculation=disable_cds_recalculation, timing=timing)
AttributeError: 'Context' object has no attribute 'gi'

'Context' object has no attribute 'gi'

I also get the same error when I use show_organism to make sure the library can find Galleria_mellonella.
The gff seems fine - I can load it onto the browser without a problem. I can also find the organism via curl:
(apollo-arrow-env) ARS5APVN2WD42YY:Downloads mpoelchau$ curl -i -X POST -H "Content-Type: application/json" -d '{ "username": "my-username", "password": "my-password", "organism": "Galleria_mellonella", "client_token": "ignore"}' https://apollo2-stage-node1-cbo.nal.usda.gov/apollo/organism/findAllOrganisms
HTTP/1.1 200
Server: nginx/1.25.1
Date: Fri, 11 Aug 2023 13:33:47 GMT
Content-Type: application/json;charset=UTF-8
Transfer-Encoding: chunked
Connection: keep-alive
Access-Control-Allow-Origin: *
Access-Control-Allow-Credentials: true
Access-Control-Allow-Methods: GET, POST, OPTIONS
Front-End-Https: on

[{"commonName":"Galleria_mellonella","blatdb":"/usr/local/blat/db/galmel/GCF_026898425.1_CSIRO_AGI_GalMel_v1_genomic.fna.2bi","metadata":"{\"creator\":\"1357111\"}","annotationCount":1,"currentOrganism":true,"obsolete":false,"sequences":221,"directory":"/app/data/other_species/galmel/CSIRO_AGI_GalMel_v1/jbrowse/data","publicMode":true,"valid":true,"genomeFastaIndex":null,"genus":"Galleria","species":"mellonella","id":4690912

Could someone point me in the right direction for debugging? Does this have something to do with the NotOpenSSLWarning?
Thanks!

Never mind, thanks to https://github.com/GMOD/Apollo/issues/2641 I realized that I needed to run arrow init first.

Hi Monica! Yep it's a typical message you get when there's a problem with the arrow config file, glad you found it, the message could be more explicit :thinking:


test commit

committing changes to Delora's article as a test

Docs Build status updates of commit 0d2eb15:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs-ref-conceptual/azure-services-the-azure-cli-can-manage.md
:bulb:Suggestion
View
Details

docs-ref-conceptual/azure-services-the-azure-cli-can-manage.md

Line 52, Column 4: [Suggestion-preserve-view-not-set] Did you mean to link to a specific version? If so, add &preserve-view=true to the URL: /azure/devops/pipelines/apps/cd/azure/build-data-pipeline?view=azure-devops
Line 146, Column 2: [Suggestion-preserve-view-not-set] Did you mean to link to a specific version? If so, add &preserve-view=true to the URL: /cli/azure/azure-cli-reference-for-hosted-apps?view=azure-cli-latest#azure-web-app-configuration
Line 151, Column 81: [Suggestion-preserve-view-not-set] Did you mean to link to a specific version? If so, add &preserve-view=true to the URL: /azure/devops/pipelines/process/variables?view=azure-devops&tabs=azure-devops-cli
Line 152, Column 4: [Suggestion-preserve-view-not-set] Did you mean to link to a specific version? If so, add &preserve-view=true to the URL: /azure/devops/pipelines/agents/agents?view=azure-devops&tabs=azure-devops-cli
Line 153, Column 4: [Suggestion-preserve-view-not-set] Did you mean to link to a specific version? If so, add &preserve-view=true to the URL: /azure/devops/pipelines/create-first-pipeline?view=azure-devops&tabs=azure-cli
Line 160, Column 2: [Suggestion-column-header-missing] Column headers are required for tables. Add appropriate header text, or remove any extra table cells that cause an extra column.
Line 162, Column 81: [Suggestion-preserve-view-not-set] Did you mean to link to a specific version? If so, add &preserve-view=true to the URL: /azure/devops/artifacts/quickstarts/universal-packages?view=azure-devops
Line 163, Column 75: [Suggestion-preserve-view-not-set] Did you mean to link to a specific version? If so, add &preserve-view=true to the URL: /azure/devops/organizations/settings/set-area-paths?view=azure-devops&tabs=azure-devops-cli#open-project-settings-list-project-areas
Line 168, Column 83: [Suggestion-preserve-view-not-set] Did you mean to link to a specific version? If so, add &preserve-view=true to the URL: /azure/devops/pipelines/process/variables?view=azure-devops&tabs=azure-devops-cli
Line 169, Column 4: [Suggestion-preserve-view-not-set] Did you mean to link to a specific version? If so, add &preserve-view=true to the URL: /azure/devops/pipelines/agents/agents?view=azure-devops&tabs=azure-devops-cli
Line 170, Column 4: [Suggestion-preserve-view-not-set] Did you mean to link to a specific version? If so, add &preserve-view=true to the URL: /azure/devops/pipelines/create-first-pipeline?view=azure-devops&tabs=azure-cli
Line 171, Column 73: [Suggestion-preserve-view-not-set] Did you mean to link to a specific version? If so, add &preserve-view=true to the URL: /azure/devops/repos/git/share-your-code-in-git-cmdline?view=azure-devops

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Docs Build status updates of commit c59ae6b:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs-ref-conceptual/azure-services-the-azure-cli-can-manage.md
:bulb:Suggestion
View
Details

docs-ref-conceptual/azure-services-the-azure-cli-can-manage.md

Line 52, Column 4: [Suggestion-preserve-view-not-set] Did you mean to link to a specific version? If so, add &preserve-view=true to the URL: /azure/devops/pipelines/apps/cd/azure/build-data-pipeline?view=azure-devops
Line 146, Column 2: [Suggestion-preserve-view-not-set] Did you mean to link to a specific version? If so, add &preserve-view=true to the URL: /cli/azure/azure-cli-reference-for-hosted-apps?view=azure-cli-latest#azure-web-app-configuration
Line 151, Column 81: [Suggestion-preserve-view-not-set] Did you mean to link to a specific version? If so, add &preserve-view=true to the URL: /azure/devops/pipelines/process/variables?view=azure-devops&tabs=azure-devops-cli
Line 152, Column 4: [Suggestion-preserve-view-not-set] Did you mean to link to a specific version? If so, add &preserve-view=true to the URL: /azure/devops/pipelines/agents/agents?view=azure-devops&tabs=azure-devops-cli
Line 153, Column 4: [Suggestion-preserve-view-not-set] Did you mean to link to a specific version? If so, add &preserve-view=true to the URL: /azure/devops/pipelines/create-first-pipeline?view=azure-devops&tabs=azure-cli
Line 162, Column 81: [Suggestion-preserve-view-not-set] Did you mean to link to a specific version? If so, add &preserve-view=true to the URL: /azure/devops/artifacts/quickstarts/universal-packages?view=azure-devops
Line 163, Column 75: [Suggestion-preserve-view-not-set] Did you mean to link to a specific version? If so, add &preserve-view=true to the URL: /azure/devops/organizations/settings/set-area-paths?view=azure-devops&tabs=azure-devops-cli#open-project-settings-list-project-areas
Line 168, Column 81: [Suggestion-preserve-view-not-set] Did you mean to link to a specific version? If so, add &preserve-view=true to the URL: /azure/devops/pipelines/process/variables?view=azure-devops&tabs=azure-devops-cli
Line 169, Column 4: [Suggestion-preserve-view-not-set] Did you mean to link to a specific version? If so, add &preserve-view=true to the URL: /azure/devops/pipelines/agents/agents?view=azure-devops&tabs=azure-devops-cli
Line 170, Column 4: [Suggestion-preserve-view-not-set] Did you mean to link to a specific version? If so, add &preserve-view=true to the URL: /azure/devops/pipelines/create-first-pipeline?view=azure-devops&tabs=azure-cli
Line 171, Column 73: [Suggestion-preserve-view-not-set] Did you mean to link to a specific version? If so, add &preserve-view=true to the URL: /azure/devops/repos/git/share-your-code-in-git-cmdline?view=azure-devops

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Docs Build status updates of commit dbb250b:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs-ref-conceptual/azure-services-the-azure-cli-can-manage.md
:bulb:Suggestion
View
Details

docs-ref-conceptual/azure-services-the-azure-cli-can-manage.md

Line 52, Column 4: [Suggestion-preserve-view-not-set] Did you mean to link to a specific version? If so, add &preserve-view=true to the URL: /azure/devops/pipelines/apps/cd/azure/build-data-pipeline?view=azure-devops
Line 146, Column 2: [Suggestion-preserve-view-not-set] Did you mean to link to a specific version? If so, add &preserve-view=true to the URL: /cli/azure/azure-cli-reference-for-hosted-apps?view=azure-cli-latest#azure-web-app-configuration
Line 151, Column 81: [Suggestion-preserve-view-not-set] Did you mean to link to a specific version? If so, add &preserve-view=true to the URL: /azure/devops/pipelines/process/variables?view=azure-devops&tabs=azure-devops-cli
Line 152, Column 4: [Suggestion-preserve-view-not-set] Did you mean to link to a specific version? If so, add &preserve-view=true to the URL: /azure/devops/pipelines/agents/agents?view=azure-devops&tabs=azure-devops-cli
Line 153, Column 4: [Suggestion-preserve-view-not-set] Did you mean to link to a specific version? If so, add &preserve-view=true to the URL: /azure/devops/pipelines/create-first-pipeline?view=azure-devops&tabs=azure-cli
Line 162, Column 81: [Suggestion-preserve-view-not-set] Did you mean to link to a specific version? If so, add &preserve-view=true to the URL: /azure/devops/artifacts/quickstarts/universal-packages?view=azure-devops
Line 163, Column 75: [Suggestion-preserve-view-not-set] Did you mean to link to a specific version? If so, add &preserve-view=true to the URL: /azure/devops/organizations/settings/set-area-paths?view=azure-devops&tabs=azure-devops-cli#open-project-settings-list-project-areas
Line 168, Column 81: [Suggestion-preserve-view-not-set] Did you mean to link to a specific version? If so, add &preserve-view=true to the URL: /azure/devops/pipelines/process/variables?view=azure-devops&tabs=azure-devops-cli
Line 169, Column 4: [Suggestion-preserve-view-not-set] Did you mean to link to a specific version? If so, add &preserve-view=true to the URL: /azure/devops/pipelines/agents/agents?view=azure-devops&tabs=azure-devops-cli
Line 170, Column 4: [Suggestion-preserve-view-not-set] Did you mean to link to a specific version? If so, add &preserve-view=true to the URL: /azure/devops/pipelines/create-first-pipeline?view=azure-devops&tabs=azure-cli
Line 171, Column 73: [Suggestion-preserve-view-not-set] Did you mean to link to a specific version? If so, add &preserve-view=true to the URL: /azure/devops/repos/git/share-your-code-in-git-cmdline?view=azure-devops

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Added and updated article links for all services except for notification hubs and static web apps services found under the web header.

Docs Build status updates of commit 0e74a7e:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs-ref-conceptual/azure-services-the-azure-cli-can-manage.md
:bulb:Suggestion
View
Details

docs-ref-conceptual/azure-services-the-azure-cli-can-manage.md

Line 52, Column 4: [Suggestion-preserve-view-not-set] Did you mean to link to a specific version? If so, add &preserve-view=true to the URL: /azure/devops/pipelines/apps/cd/azure/build-data-pipeline?view=azure-devops
Line 146, Column 2: [Suggestion-preserve-view-not-set] Did you mean to link to a specific version? If so, add &preserve-view=true to the URL: /cli/azure/azure-cli-reference-for-hosted-apps?view=azure-cli-latest#azure-web-app-configuration
Line 151, Column 81: [Suggestion-preserve-view-not-set] Did you mean to link to a specific version? If so, add &preserve-view=true to the URL: /azure/devops/pipelines/process/variables?view=azure-devops&tabs=azure-devops-cli
Line 152, Column 4: [Suggestion-preserve-view-not-set] Did you mean to link to a specific version? If so, add &preserve-view=true to the URL: /azure/devops/pipelines/agents/agents?view=azure-devops&tabs=azure-devops-cli
Line 153, Column 4: [Suggestion-preserve-view-not-set] Did you mean to link to a specific version? If so, add &preserve-view=true to the URL: /azure/devops/pipelines/create-first-pipeline?view=azure-devops&tabs=azure-cli
Line 162, Column 81: [Suggestion-preserve-view-not-set] Did you mean to link to a specific version? If so, add &preserve-view=true to the URL: /azure/devops/artifacts/quickstarts/universal-packages?view=azure-devops
Line 163, Column 75: [Suggestion-preserve-view-not-set] Did you mean to link to a specific version? If so, add &preserve-view=true to the URL: /azure/devops/organizations/settings/set-area-paths?view=azure-devops&tabs=azure-devops-cli#open-project-settings-list-project-areas
Line 168, Column 81: [Suggestion-preserve-view-not-set] Did you mean to link to a specific version? If so, add &preserve-view=true to the URL: /azure/devops/pipelines/process/variables?view=azure-devops&tabs=azure-devops-cli
Line 169, Column 4: [Suggestion-preserve-view-not-set] Did you mean to link to a specific version? If so, add &preserve-view=true to the URL: /azure/devops/pipelines/agents/agents?view=azure-devops&tabs=azure-devops-cli
Line 170, Column 4: [Suggestion-preserve-view-not-set] Did you mean to link to a specific version? If so, add &preserve-view=true to the URL: /azure/devops/pipelines/create-first-pipeline?view=azure-devops&tabs=azure-cli
Line 171, Column 73: [Suggestion-preserve-view-not-set] Did you mean to link to a specific version? If so, add &preserve-view=true to the URL: /azure/devops/repos/git/share-your-code-in-git-cmdline?view=azure-devops

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Docs Build status updates of commit ff2f62e:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs-ref-conceptual/azure-services-the-azure-cli-can-manage.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Docs Build status updates of commit 743d08e:
:warning: Validation status: warnings

File
Status
Preview URL
Details

docs-ref-conceptual/index.yml
:warning:Warning
View
Details

docs-ref-conceptual/get-started-with-azure-cli.md
:warning:Warning
View
Details

docs-ref-conceptual/TOC.yml
:white_check_mark:Succeeded
View

docs-ref-conceptual/azure-services-the-azure-cli-can-manage.md
:white_check_mark:Succeeded
View

docs-ref-conceptual/popular-articles-using-the-azure-cli.md
:white_check_mark:Succeeded

docs-ref-conceptual/index.yml

Line 45, Column 18: [Warning-file-not-found] Invalid file link: 'popular-articles-using-the-azure-cli.md'.

docs-ref-conceptual/get-started-with-azure-cli.md

Line 137, Column 3: [Warning-file-not-found] Invalid file link: 'popular-articles-using-the-azure-cli.md'.

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Docs Build status updates of commit 8651fbe:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs-ref-conceptual/TOC.yml
:white_check_mark:Succeeded
View

docs-ref-conceptual/azure-services-the-azure-cli-can-manage.md
:white_check_mark:Succeeded
View

docs-ref-conceptual/get-started-with-azure-cli.md
:white_check_mark:Succeeded
View

docs-ref-conceptual/index.yml
:white_check_mark:Succeeded
View

docs-ref-conceptual/popular-articles-using-the-azure-cli.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


Autocomplete Toolip has too small height

VSCode Version: 1.52.1
OS Version: Windows 10

Steps to Reproduce:

Whenever intelissense tooltip appears it is too small in height ( only one suggestion can fit you have to scroll otherwise)

Here There are much more suggestion and it appears only if you navigate via keyboard or scroll

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

/duplicate of #110554 (comment)


Regression in mcr.microsoft.com/iotedge/opc-plc:1.2.0

This issue is for a: (mark with an x)
- [ ] bug report -> please search issues before submitting
- [ ] feature request
- [ ] documentation issue or request
- [ x] regression (a behavior that used to work and stopped in a new release)

Minimal steps to reproduce
docker run -it --rm -p "50000:50000" mcr.microsoft.com/iotedge/opc-plc:1.2.0 --plchostname=opcplc --showpnjsonph --portnum=50000 --autoaccept --unsecuretransport

Any log messages given by the failure
Unable to find image 'mcr.microsoft.com/iotedge/opc-plc:1.2.0' locally
1.2.0: Pulling from iotedge/opc-plc
Digest: sha256:866d6e4b62ad24b4e84988a84d7c173cae24907074854942374f8b6b24173222
Status: Downloaded newer image for mcr.microsoft.com/iotedge/opc-plc:1.2.0
[14:27:40 INF] Current directory is: /app
[14:27:40 INF] Log file is: /app/dbf88900f3e5-plc.log
[14:27:40 INF] Log level is: info
[14:27:40 INF] OpcPlc V1.2.0 starting up...
info: Microsoft.Hosting.Lifetime[0]
      Now listening on: http://[::]:8080
info: Microsoft.Hosting.Lifetime[0]
      Application started. Press Ctrl+C to shut down.
info: Microsoft.Hosting.Lifetime[0]
      Hosting environment: Production
info: Microsoft.Hosting.Lifetime[0]
      Content root path: /app
[14:27:40 INF] Web server started on port 8080
[14:27:40 INF] opcstacktracemask set to: 0x0
[14:27:40 INF] OPC UA server base address: opc.tcp://opcplc:50000
[14:27:40 INF] Security policy http://opcfoundation.org/UA/SecurityPolicy#Basic256Sha256 with mode SignAndEncrypt added
[14:27:40 INF] Unsecure security policy http://opcfoundation.org/UA/SecurityPolicy#None with mode None added
[14:27:40 WRN] Note: This is a security risk and needs to be disabled for production use
[14:27:40 INF] Trusted Issuer store type is: Directory
[14:27:40 INF] Trusted Issuer Certificate store path is: pki/issuer
[14:27:40 INF] Trusted Peer Certificate store type is: Directory
[14:27:40 INF] Trusted Peer Certificate store path is: pki/trusted
[14:27:40 INF] Rejected certificate store type is: Directory
[14:27:40 INF] Rejected Certificate store path is: pki/rejected
[14:27:40 INF] Rejection of SHA1 signed certificates is disabled
[14:27:40 INF] Minimum certificate key size set to 1024
[14:27:40 INF] Application Certificate store type is: Directory
[14:27:40 INF] Application Certificate store path is: pki/own
[14:27:40 INF] Application Certificate subject name is: OpcPlc
[14:27:40 WRN] WARNING: Automatically accepting certificates. This is a security risk.
[14:27:40 INF] No existing Application certificate found. Create a self-signed Application certificate valid from yesterday for 12 months,
[14:27:40 INF] with a 2048 bit key and 256 bit hash.
[14:27:41 INF] Application certificate with thumbprint '9A9799313BD331342916BF04EC3196111A564A43' created.
[14:27:41 INF] Application certificate is for ApplicationUri 'urn:OpcPlc:opcplc', ApplicationName 'OpcPlc' and Subject is 'OpcPlc'
[14:27:41 INF] LDS(-ME) registration interval set to 0 ms (0 means no registration)
[14:27:41 INF] Trusted issuer store contains 0 certs
[14:27:41 INF] Trusted issuer store has 0 CRLs.
[14:27:41 INF] Trusted peer store contains 0 certs
[14:27:41 INF] Trusted peer store has 0 CRLs.
[14:27:41 INF] Rejected certificate store contains 0 certs
[14:27:41 INF] Starting server on endpoint opc.tcp://opcplc:50000 ...
[14:27:41 INF] Simulation settings are:
[14:27:41 INF] One simulation phase consists of 50 cycles
[14:27:41 INF] One cycle takes 100 milliseconds
[14:27:41 INF] Spike generation is enabled
[14:27:41 INF] Data generation is enabled
[14:27:41 INF] Anonymous authentication: enabled
[14:27:41 INF] Username/Password authentication: enabled
[14:27:41 INF] Certificate authentication: enabled
[14:27:41 INF] OPC UA Server started.
Unhandled exception.Unhandled exception.Unhandled exception.  Unhandled exception.  Unhandled exception. Unhandled exception. [14:27:41 INF] OPC Publisher configuration file: pn.json
[
  {
    "EndpointUrl": "opc.tcp://opcplc:50000",
    "UseSecurity": false,
    "OpcNodes": [
      { "Id": "ns=2;s=AlternatingBoolean" },
      { "Id": "ns=2;s=DipData" },
      { "Id": "ns=2;s=NegativeTrendData" },
      { "Id": "ns=2;s=PositiveTrendData" },
      { "Id": "ns=2;s=RandomSignedInt32" },
      { "Id": "ns=2;s=RandomUnsignedInt32" },
      { "Id": "ns=2;s=SpikeData" },
      { "Id": "ns=2;s=StepUp" }
    ]
  }
]

System.NullReferenceException: Object reference not set to an instance of an object.
   at OpcPlc.PlcNodeManager.SetValue[T](BaseDataVariableState variable, T value) in D:\a\1\s\src\PlcNodeManager.cs:line 777
   at OpcPlc.PlcNodeManager.set_DipData(Double value) in D:\a\1\s\src\PlcNodeManager.cs:line 36
   at OpcPlc.PlcSimulation.DipGenerator(Object state) in D:\a\1\s\src\PlcSimulation.cs:line 144
   at System.Threading.TimerQueueTimer.<>c.<.cctor>b__23_0(Object state)
   at System.Threading.ExecutionContext.RunFromThreadPoolDispatchLoop(Thread threadPoolThread, ExecutionContext executionContext, ContextCallback callback, Object state)
--- End of stack trace from previous location where exception was thrown ---
   at System.Threading.ExecutionContext.RunFromThreadPoolDispatchLoop(Thread threadPoolThread, ExecutionContext executionContext, ContextCallback callback, Object state)
   at System.Threading.TimerQueueTimer.CallCallback(Boolean isThreadPool)
   at System.Threading.TimerQueueTimer.Fire(Boolean isThreadPool)
   at System.Threading.TimerQueueTimer.System.Threading.IThreadPoolWorkItem.Execute()
   at System.Threading.ThreadPoolWorkQueue.Dispatch()
   at System.Threading._ThreadPoolWaitCallback.PerformWaitCallback()System.NullReferenceException: Object reference not set to an instance of an object.
   at OpcPlc.PlcNodeManager.SetValue[T](BaseDataVariableState variable, T value) in D:\a\1\s\src\PlcNodeManager.cs:line 777
   at OpcPlc.PlcNodeManager.set_AlternatingBoolean(Boolean value) in D:\a\1\s\src\PlcNodeManager.cs:line 54
   at OpcPlc.PlcSimulation.ValueGenerator(Object state) in D:\a\1\s\src\PlcSimulation.cs:line 216
   at System.Threading.TimerQueueTimer.<>c.<.cctor>b__23_0(Object state)
   at System.Threading.ExecutionContext.RunFromThreadPoolDispatchLoop(Thread threadPoolThread, ExecutionContext executionContext, ContextCallback callback, Object state)
--- End of stack trace from previous location where exception was thrown ---
   at System.Threading.ExecutionContext.RunFromThreadPoolDispatchLoop(Thread threadPoolThread, ExecutionContext executionContext, ContextCallback callback, Object state)
   at System.Threading.TimerQueueTimer.CallCallback(Boolean isThreadPool)
   at System.Threading.TimerQueueTimer.Fire(Boolean isThreadPool)
   at System.Threading.TimerQueueTimer.System.Threading.IThreadPoolWorkItem.Execute()
   at System.Threading.ThreadPoolWorkQueue.Dispatch()
   at System.Threading._ThreadPoolWaitCallback.PerformWaitCallback()System.NullReferenceException: Object reference not set to an instance of an object.
   at OpcPlc.PlcNodeManager.SetValue[T](BaseDataVariableState variable, T value) in D:\a\1\s\src\PlcNodeManager.cs:line 777
   at OpcPlc.PlcNodeManager.set_PosTrendData(Double value) in D:\a\1\s\src\PlcNodeManager.cs:line 42
   at OpcPlc.PlcSimulation.PosTrendGenerator(Object state) in D:\a\1\s\src\PlcSimulation.cs:line 168
   at System.Threading.TimerQueueTimer.<>c.<.cctor>b__23_0(Object state)
   at System.Threading.ExecutionContext.RunFromThreadPoolDispatchLoop(Thread threadPoolThread, ExecutionContext executionContext, ContextCallback callback, Object state)
--- End of stack trace from previous location where exception was thrown ---
   at System.Threading.ExecutionContext.RunFromThreadPoolDispatchLoop(Thread threadPoolThread, ExecutionContext executionContext, ContextCallback callback, Object state)
   at System.Threading.TimerQueueTimer.CallCallback(Boolean isThreadPool)
   at System.Threading.TimerQueueTimer.Fire(Boolean isThreadPool)
   at System.Threading.TimerQueueTimer.System.Threading.IThreadPoolWorkItem.Execute()
   at System.Threading.ThreadPoolWorkQueue.Dispatch()
   at System.Threading._ThreadPoolWaitCallback.PerformWaitCallback()

System.NullReferenceException: Object reference not set to an instance of an object.
   at OpcPlc.PlcNodeManager.SetValue[T](BaseDataVariableState variable, T value) in D:\a\1\s\src\PlcNodeManager.cs:line 777
   at OpcPlc.PlcNodeManager.set_NegTrendData(Double value) in D:\a\1\s\src\PlcNodeManager.cs:line 48
   at OpcPlc.PlcSimulation.NegTrendGenerator(Object state) in D:\a\1\s\src\PlcSimulation.cs:line 192
   at System.Threading.TimerQueueTimer.<>c.<.cctor>b__23_0(Object state)
   at System.Threading.ExecutionContext.RunFromThreadPoolDispatchLoop(Thread threadPoolThread, ExecutionContext executionContext, ContextCallback callback, Object state)
--- End of stack trace from previous location where exception was thrown ---
   at System.Threading.ExecutionContext.RunFromThreadPoolDispatchLoop(Thread threadPoolThread, ExecutionContext executionContext, ContextCallback callback, Object state)
   at System.Threading.TimerQueueTimer.CallCallback(Boolean isThreadPool)
   at System.Threading.TimerQueueTimer.Fire(Boolean isThreadPool)
   at System.Threading.TimerQueueTimer.System.Threading.IThreadPoolWorkItem.Execute()
   at System.Threading.ThreadPoolWorkQueue.Dispatch()
   at System.Threading._ThreadPoolWaitCallback.PerformWaitCallback()
System.NullReferenceException: Object reference not set to an instance of an object.
   at OpcPlc.PlcNodeManager.IncreaseNodes(BaseDataVariableState[] nodes, NodeType type) in D:\a\1\s\src\PlcNodeManager.cs:line 271
   at OpcPlc.PlcNodeManager.IncreaseSlowNodes(Object state) in D:\a\1\s\src\PlcNodeManager.cs:line 80
   at System.Threading.TimerQueueTimer.<>c.<.cctor>b__23_0(Object state)
   at System.Threading.ExecutionContext.RunFromThreadPoolDispatchLoop(Thread threadPoolThread, ExecutionContext executionContext, ContextCallback callback, Object state)
--- End of stack trace from previous location where exception was thrown ---
   at System.Threading.ExecutionContext.RunFromThreadPoolDispatchLoop(Thread threadPoolThread, ExecutionContext executionContext, ContextCallback callback, Object state)
   at System.Threading.TimerQueueTimer.CallCallback(Boolean isThreadPool)
   at System.Threading.TimerQueueTimer.Fire(Boolean isThreadPool)
   at System.Threading.TimerQueueTimer.System.Threading.IThreadPoolWorkItem.Execute()
   at System.Threading.ThreadPoolWorkQueue.Dispatch()
   at System.Threading._ThreadPoolWaitCallback.PerformWaitCallback()
System.NullReferenceException: Object reference not set to an instance of an object.
   at OpcPlc.PlcNodeManager.IncreaseNodes(BaseDataVariableState[] nodes, NodeType type) in D:\a\1\s\src\PlcNodeManager.cs:line 271
   at OpcPlc.PlcNodeManager.IncreaseFastNodes(Object state) in D:\a\1\s\src\PlcNodeManager.cs:line 87
   at System.Threading.TimerQueueTimer.<>c.<.cctor>b__23_0(Object state)
   at System.Threading.ExecutionContext.RunInternal(ExecutionContext executionContext, ContextCallback callback, Object state)
--- End of stack trace from previous location where exception was thrown ---
   at System.Threading.ExecutionContext.RunInternal(ExecutionContext executionContext, ContextCallback callback, Object state)
   at System.Threading.TimerQueueTimer.CallCallback(Boolean isThreadPool)
   at System.Threading.TimerQueueTimer.Fire(Boolean isThreadPool)
   at System.Threading.TimerQueue.FireNextTimers()
   at System.Threading.TimerQueue.AppDomainTimerCallback(Int32 id)

Expected/desired behavior
The previous version worked:
docker run -it --rm -p "50000:50000" mcr.microsoft.com/iotedge/opc-plc:1.1.6 --plchostname=opcplc --showpnjsonph --portnum=50000 --autoaccept --unsecuretransport

OS and Version?

macOS Catalina Version 10.15.7
docker desktop community Version <IP_ADDRESS> (49550)

Versions
mcr.microsoft.com/iotedge/opc-plc:1.2.0
Mention any other details that might be useful

@ajhewett please add the parameter --ctb for now, we are already deploying a fix.

Fixed with https://github.com/Azure-Samples/iot-edge-opc-plc/pull/31, container is now available in MCR


[Enhancement][opensearch] Allow user-defined labels on ingress resource

Is your feature request related to a problem? Please describe.
I am using external-dns with settings that makes it label-driven instead of annotation-driven. Because opensearch does not allow me to add user-defined labels to the ingress I cannot get external-dns to pick up the opensearch ingress and automatically create a DNS entry for the endpoint.
Describe the solution you'd like
I would like to be able to add user-defined labels in a similar fashion as annotations to the ingress resource created by the helm chart.
Describe alternatives you've considered
N/A

Hey @jwitko thanks for raising an issue, I see your problem, this should be an easy fix, just like how it was added to service labels, we should add the same way to ingress and then you can pass the external-dns label that can added to ingress file, are you open to raise a PR?
Thank you
@bbarani @rishabh6788

@prudhvigodithi I already created the PR.  It is attached to this Issue linked above.  #390

Thanks @jwitko I will take a look.


Revisit location or existence of NetworkStatus

by @tomaka from https://github.com/paritytech/substrate/pull/6986#issuecomment-687055563:

NetworkStatus was moved by someone not in the networking team as part of a big PR, and I didn't spot the change.

The entire point of NetworkStatus is to communicate some networking-related information internally between some components without having to depend on sc-network.
Moving it to sc-network entirely defeats this purpose.

It should either be moved back for example to sc-service, or be removed entirely and have everything depend on Arc<NetworkService>.

A little bit of context for this issue:

NetworkStatus reporting was added to the service in https://github.com/paritytech/substrate/pull/2916
NetworkStatus moved from service to network in https://github.com/paritytech/substrate/pull/6174

Dependent crates of struct NetworkStatus:

sc-informant
sc-network
sc-service

Cargo trees:

sc-informant -> sc-network
sc-service -> (other deps...) -> sc-network

So no wonder the struct is located in sc-network. Placing the struct inside sc-service create cyclic package dependency...


neon.exe: SIGTERM/SIGINT needs to terminate neon-cli subprocess too

I've noticed that when I use CRTL+C to kill the neon/kubectl tool, neon-cli.exe may continue running in the background,
Is it possible to somehow kill any subprocesses as well.?

This link describes some techniques to kill sub processes as including using Linux process groups and the Windows equivalent:
https://stackoverflow.com/questions/34730941/ensure-executables-called-in-go-process-get-killed-when-process-is-killed

After more research, it looks like we can use a process group to ensure that sub processes are killed when the parent process terminates.  The challenge with this is that process group is a Linux concept whereas Windows has job objects, so the implementation will be different.
Here's an example Windows implementation:
https://gist.github.com/hallazzang/76f3970bfc949831808bbebc8ca15209

DONE


Add support for deserializing list-encoded JSON structs [#6558]

Which issue does this PR close?
Closes #6558.
Rationale for this change
Currently, a StructArray can only be deserialized from a JSON object (e.g. {a: 1, b: "c"}), but some services (e.g. Presto and Trino) encode ROW types as JSON lists (e.g. [1, "c"]) because this is more compact, and the schema is known.
Arrow-json cannot currently deserialize these.
What changes are included in this PR?
This PR adds the ability to parse JSON lists into StructArrays, if the StructParseMode is set to ListOnly.  In ListOnly mode, object-encoded structs raise an error.  Setting to ObjectOnly (the default) has the original parsing behavior.
Are there any user-facing changes?
Users may set the StructParsingMode enum to ListOnly to parse list-style structs.  The associated enum,
variants, and method have been documented.  I'm happy to update any other documentation.
Discussion topics

I've made a JsonParseMode struct instead of a bool flag for two reasons.  One is that it's self-descriptive (what would true be?), and the other is that it allows a future Mixed mode that could deserialize either.  The latter isn't currently requested by anyone.
I kept the error messages as similar to the old messages as possible. I considered having more specific error messages (like "Encountered a '[' when parsing a Struct, but the StructParseMode is ObjectOnly" or similar), but wanted to hear opinions before I went that route.
I'm not attached to any name/code-style/etc, so happy to modify to fit local conventions.
One requirement was that benchmarks do not regress.  My running of benchmarks have been inconclusive (see https://gist.github.com/jagill/6749248171a1f12fb7c653ff70c5ed42).  There are often small regressions or improvements in the single-digit % range whenever I switch between master and this PR.  I suspect they are statistical but I wanted to note these.

Is there any changes I could make to this PR to make it easier to review?

Ok, I've hoisted the conditional outside the loop.  It also had the effect (as you predicted) that the gnarly match statement for the delimiters is a bit easier to understand.

Are you planning / willing to implement serializing support too (aka support in json_writer)? That would then allow us to verify that data can be round-tripped

I was not planning to, because there was no use-case requiring it.  But I'm certainly willing to, both for symmetry and for round-tripping.  Is that something you'd want in this PR, or a follow-up?

I believe this PR slows down performance (notes below)

When I check, I'm consistently seeing no difference between the main branch and either of the variants:
❯ critcmp main inner-conditional
group                    inner-conditional                      main
-----                    -----------------                      ----
large_bench_primitive    1.00      2.7±0.01ms        ? ?/sec    1.00      2.7±0.03ms        ? ?/sec
small_bench_list         1.01      8.4±0.09µs        ? ?/sec    1.00      8.2±0.06µs        ? ?/sec
small_bench_primitive    1.01      4.6±0.03µs        ? ?/sec    1.00      4.6±0.04µs        ? ?/sec

❯ critcmp main outer-conditional
group                    outer-conditional                      main
-----                    -----------------                      ----
large_bench_primitive    1.00      2.7±0.03ms        ? ?/sec    1.00      2.7±0.03ms        ? ?/sec
small_bench_list         1.00      8.3±0.11µs        ? ?/sec    1.00      8.2±0.06µs        ? ?/sec
small_bench_primitive    1.00      4.6±0.05µs        ? ?/sec    1.00      4.6±0.04µs        ? ?/sec

❯ critcmp inner-conditional outer-conditional
group                    outer-conditional                      inner-conditional
-----                    -----------------                      -----------------
large_bench_primitive    1.00      2.7±0.03ms        ? ?/sec    1.00      2.7±0.01ms        ? ?/sec
small_bench_list         1.00      8.3±0.11µs        ? ?/sec    1.01      8.4±0.09µs        ? ?/sec
small_bench_primitive    1.00      4.6±0.05µs        ? ?/sec    1.01      4.6±0.03µs        ? ?/sec

(btw, I didn't know about critcmp; thanks for the pointer!)
However, I'm not running in an isolated environment, so the differences I saw in the past seemed to be related to other processes.  I'm running on a M2 Mac and haven't tried on with x86 or other architectures, and haven't been able to isolate from the helpful random processes that Macos spins up, so I'll trust your benchmarks.

@alamb: With this update I've

included the hoisting of the conditional above the loop, which I missed last time due to atrophied git add muscles,
renamed and extracted StructParseMode to a top level StructMode in arrow_json, since I've...
enabled writing structs to JSON lists in the writer,
included some more examples in the with_struct_mode methods of ReaderBuilder and WriterBuilder, along with links to the scanty Presto/Trino documentation.

One thing that makes me sad is that the Writer has an explicit_nulls field which is not orthogonal to StructMode. In particular, writing to a JSON List must include nulls, so explicit_nulls == false and StructMode::ListOnly is not a consistent state.  I saw three main approaches here:

Split StructMode::ObjectOnly to ObjectOnlyWithImplictNulls and ObjectOnlyWithExplicitNulls (but with better names).  This means only valid states will exist, and the Reader can choose between accepting implicit nulls or not.  But this is a breaking change.
Error/panic in WriterBuilder if explicit_nulls == false and struct_mode == ListOnly.  This prevents users from using a non-consistent config, but adds failure paths.
Only skip nulls if explicit_nulls == false and struct_mode == ObjectOnly, and add that to the documentation for with_explicit_nulls and with_struct_mode.

I chose 3 but in theory I'm open to any of these (or other suggestions).  In practice, I think that 1 would involve too much yak shaving, and we can always do the breaking change combining the two options later.
I'm also happy to add more documentation for rational or examples, with the following caveats:

I wanted to add a doctest but constructing an expected StructArray was so verbose it obfuscated the purpose of the doctest.  Maybe there's a better way than I know?
Getting the doc update PR accepted/merged into Presto and Trino will take an unknown amount of time. Maybe a week, maybe 3 months.  I'm reticent to block this update for that.

Awesoe -- thank you @jagill  - I will try and review this PR later today or tomorrow


A visual separator needed for multiple relations on landing page

If a resource has multiple relations, the landing page (in view mode) blends them together.  Some sort of separator (perhaps similar to the line in edit mode) is needed.
A resource that illustrates this is https://www.hydroshare.org/resource/4104f344a1cc4a639a63164d0bf237c9/

@dtarb @Maurier  This can be closed after 1.14 goes out

Verified on candidate release beta.hydroshare.org and closing.  @aphelionz I feel it is OK to close issues that have been verified on a candidate release.

@dtarb copy that


improve mismatch install error situation

Prior to pacaur 4.4.0 (November 2015), the installation of built package was done though makepkg -i. As this flag installed all subpackages of a split package by default, the additional --pkg flag was used to selectively install the relevant subpackages of split packages.
However, this --pkg flag was removed in pacman 5.0 with the buggy ability to build individual packages, leaving no choice but to use pacman -U to install (split) packages correctly (#364) by passing the full path to the package file, including its full version for a robuster implementation. The version string is retrieved from the RPC interface, which itself retrieves it from the .SRCINFO file that is normally updated at the same time each PKGBUILD updates occur by packages maintainers.
Unfortunately, maintainers sometimes forget to update this .SRCINFO file, leaving the AUR interface and the RPC with obsolete information. Pacaur still relies on the retrieved RPC version string for installation. If a discrepancy is detected (expected built package not found), pacaur simply aborts and display the infamous "failed to install" message:
:: Installing X package(s)...
:: X package(s) failed to install.
:: ensure package version does not mismatch between .SRCINFO and PKGBUILD
:: ensure package name has a VCS suffix if this is a devel package

While some workaround are possible and even necessary for VCS packages (#428), the choice to error out with regular package is consciously done to incite users to report the issue to the maintainer in order to fix it.
In other words, I prefer to let AUR maintainers fix issues that should be fixed by them (in PKGBUILDs or outdated .SRCINFO) rather than implementing workarounds, and I am personally fine to bother and incite users to help maintain the AUR and its metadata. This seems to be, however, a controversial stance but I do consider it the lesser of two evils, considering the popularity of pacaur.
Now onto related issues and possible improvements:

The implementation as currently done is very lazy: it's a quick hack implemented at installation time in the very place where the old makepkg -i --pkg operation occured. It should be possible to move this version check earlier, compare the version of all packages on the build queue together and error out much earlier (ideally before any build occurs);
It should be possible to get --pkg or some variation of it back into makpekg. This was removed to prevent the ability to selectively build split packages, but the use case of selective installation does not seem to have been considered. There was a previous attempt to get it back, but it stayed unanswered.
It is difficult to properly detect all VCS packages and some would still be impacted by this issue (#507). Some packages behaves like VCS packages but actually aren't (see f.e. firefox-nightly or visual-studio-code-insiders, or #781 for a recent edge case). I am not sure what could be done here, apart from implementing the VCS trick (#428) for all packages. This could potentially effectively bypass the issue for all packages, but could leave the door open to a much less reliable RPC metadata on the longer term.
The ideal solution would be to find a way to prevent the .SRCINFO update mishap. This has been brought to the AUR dev team already (FS#46130), but only a very partial solution has been implemented so far.

This patch by eschwartz should alleviate the need to readd --pkg back into makepkg. It's not in master yet but shouldn't take long to be merged (#726).
Also, rework the error message. It's apparently "not honest enough", whatever that means.

Yeah, I've recently learned that two random guys fighting in a chat room with piss poor attitude apparently makes pacaur a political platform.

I like how you downplay the whole thing and reduce it to what happened one night on an IRC channel. In the end I don't care how you write your software or how people use it - but if I see this kind of harassement to AUR maintainers again, especially for issues which have little to no impact on people using the supported tools (being makepkg), accounts will start to magically become suspended. And just to be clear - the idiotic, page-long bikeshed from both sides equally counts on this regard.
Anyway, I just wanted to get that out of the way. Thanks for your initiatives on improving the AUR.

I consider it yet another high in the recurrent shitshow, but it is indeed a chain of events that might be even traced back to the time this project became somewhat popular.
I'm not sure what's the point of your message, but I am glad if you feel better now.

This said, it has been clear to me for quite some time that the hassle of maintaining a popular helper is not worth the time nor the hassle. Without entering into the ugly details, this has become an unpaid job I dislike more than enjoy while I have clearly way too many other problems irl to have the luxury to participate in that shitshow any longer - unless it magically becomes a job that I dislike but that is at least handsomely remunerated.

The above still holds true. Looking at what should ideally be done for a possible rewrite didn't take it out of my mind - on the contrary. Since the current code has run out its course, it sounds like the perfect time to end it.

The above still holds true. Looking at what should ideally be done for a possible rewrite didn't take it out of my mind - on the contrary. Since the current code has run out its course, it sounds like the perfect time to end it.

You've been saying that for a while. Personally I think all the "user friendly" helpers are a burden to the community, especially yaourt and [archlinuxfr]. Though removing them would probably just result in some crazy person forking the code.
I don't think there's anything left for me to say here, so I'll just leave it that.


Cannot import 'GradientDescentOptimizer'

I use train.GradientDescentOptimizer and tf.optimizers.GradientDescentOptimizer, which not work. So I read the doc but only search GradientDescentOptimizer  in tf.compat.v1.train.GradientDescentOptimizer

See the documentation for tf.train and tf.optimizers. According to these, we do not have a batch gradient descent optimizer in either. There is Stochastic Gradient Descent optimizer in tf.optimizers.SGD. For gradient descent optimizer, so the way to apply Gradient Descent is tf.compat.v1.train.GradientDescentOptimizer.

It works for me.Thanks.


Add dark theme

Very important.

I suggest using flatlaf, as the themes there look quite decent versus the default java LaFs.


fix: fall back to default value when publicPath is falsy

Types of changes

[x] Bug fix (a non-breaking change which fixes an issue)
[ ] New feature (a non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Description
Resolves #4364
Checklist:

[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly. (PR: #)
[x] I have added tests to cover my changes (if not applicable, please state why)
[x] All new and existing tests are passing.

Codecov Report

Merging #4365 into dev will increase coverage by <.01%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##              dev    #4365      +/-   ##
==========================================
+ Coverage   91.51%   91.52%   +<.01%
==========================================
  Files          56       56
  Lines        1885     1887       +2
  Branches      475      476       +1
==========================================
+ Hits         1725     1727       +2
  Misses        150      150
  Partials       10       10

Impacted Files
Coverage Δ

packages/config/src/options.js
94.81% <100%> (+0.07%)
:arrow_up:

Continue to review full report at Codecov.

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

Would suggest to release a patch (details in the linked issue above) after merging.

@manniL I have an alternative way to fix this: We can force add a trailing slash to the publicPath. This not only resolves this bug but also auto-fixes the situations that user forgot to add it.

@pi0 I'd combine both solutions 👍


Is there any way to pass key-values to DFP for targeting ADs?

Hi,
DFP allows targeting ADs: it could serve ADs based on defined key-values (as age, gender or preferences).
Reference here: https://support.google.com/dfp_premium/answer/188092
Android https://developers.google.com/mobile-ads-sdk/docs/dfp/android/targeting
IOS https://developers.google.com/mobile-ads-sdk/docs/dfp/ios/targeting
Does this plugin allow us to pass key-values to DFP for targeting AD?
Is there any way to achieve this?
Thank you,
Best regards

It's not able to do this yet, but it's possible to add this feature in near future.
What's your use case for this feature?

Hi,
this is needed when you profile users in apps (basically they have preferences and settings), and you need to target ADs for them, passing those preferences to DFP to let it choose the right AD to serve.
Let me explain with a simple use case: there are two different group of users, one tagged as "sailors" and the second one tagged as "phisician". They should see different ADs that match their interest. DFP can serve them different ADs if you pass the value for the key "job" (previously mapped in DFP console, serve AD X for job="sailors" while serve AD Y for job="phisician"). Plugin needs a way to pass key-values to DFP.
Android and IOS reference seems to show how native code should work.
Maybe it should exist a function named addCustomTargeting(key,value) where value can be string or array of string.
AdMob.addCustomTargeting("job","sailor");
AdMob.addCustomTargeting("age", ["24", "25", "26"]);
or a dedicated preference when creating banner
AdMob.createBanner( {
adId: admobid.banner,
position: AdMob.AD_POSITION.TOP_CENTER,
customTargeting: {
job: "sailor",
age: "23",
interest: ["boats","ports"]
}
} );

Thanks for explanation. I'd rather re-use the existing API:
https://github.com/floatinghotpot/cordova-admob-pro/wiki/1.2-Method:-AdMob.setOptions()
Like:
AdMob.setOptions({
  customTargeting: {
    job: "sailor",
    age: "23",
    interest: ["boats","ports"],
  },
});

As options can be passed to createBanner() and prepareInterstitial(), your second proposal is great.
Will find time to implement this, but I cannot give a schedule.

Cool,
I'll just wait for some news here (asap for you please!).
I think this is a must-have feature to use plugin with DFP, since everyone using DFP will search a way how to targeting users.
Thanks in advance for your time!

Is there any news about this?
Thank you.

I am also interested in this feature. In my case the app is the official application of a famous blogger and I want the ads to be relevant, otherwise I can get some ugly ones and it's a matter of reputation of the public figure.

See docs:
https://github.com/floatinghotpot/cordova-admob-pro/tree/master/docs#targetting-options

Great job!
Thank you!


Missing script: npm run render

At http://airbnb.io/react-sketchapp/ we are told to try this:
git clone https://github.com/airbnb/react-sketchapp.git
cd react-sketchapp/examples/basic-setup && npm install
npm run render

But the script doesn't exist
npm run render
npm ERR! missing script: render

@antoniobrandao Can you make sure you're in react-sketchapp/examples/basic-setup when you run npm run render?

I didn't realise I had to enter the folder, sorry. Would have had more luck if I had simply copy-pasted....
Thanks

Glad it worked!


Check: Back-deploy assertIsolation/assumeIsolation

Checking after rebase -- since cannot reproduce failure seen in https://github.com/apple/swift/pull/68378 🤔

@swift-ci please smoke test


Migrate Linked List to Swift 3.0

This is my first contribution to the repo, let me know if something should be added :)

Nothing makes me happier than Swift written with proper syntax! Thanks @ivanicspeter92.


Media Notification does not update content title on API Level > 29

Media3 Version
1.0.0-beta03
Devices that reproduce the issue
Emulator API Level > 29
Devices that do not reproduce the issue
Emulator API Level <= 29
Reproducible in the demo app?
Not tested
Reproduction steps

start playing a MediaItem with alternating MediaMetadata
observe MediaNotification

Expected result
Notification represents the current MediaMetadata:

content title to metadata title
content text to metadata artists

Actual result
Notification doesn't represent the current MediaMetadata only the content text is changing accordingly, the title is stuck to the initial value of metadata.title.
On Devices with API Level <= 29 everything is working as expected.
Media
Not applicable
Bug Report

[ ] You will email the zip file produced by adb bugreport to<EMAIL_ADDRESS>after filing this issue.

Hmmm, I'm actually surprised it is working on API levels below 30.  We have #33 about this and changing the metadata of the media item is not possible once it is added to the player. Afaik the only way the metadata is updated after adding the media item is if there are some metadata tags in the media itself (like ID3 or Icy).

start playing a MediaItem with alternating MediaMetadata

Can you give me some more details how you update the metdata of the media item? Because once you've added the MediaItem to the player with set/addMediaItem(mediaItem) you can't actually change the metadata of the instance anymore (it is immutable). You would have to replace the intitial media item and that's not possible as per #33.
If you have an URI to the media you are using and can post it here are send it by email (dev.exoplayer@gmail.com), I can test myself.

For our current use case, we need to fetch the metadata for the current track inside a stream via an external api.
We were aware that the metadata is immutable and can't be changed without setting a new MediaItem. So in order to use the external metadata api, we wrapped the ExoPlayer and implemeted some

Yes, so for our use case we need a way to update the current metadata from an external API, and we were aware that the metadata is immutable and can't be changed without a new MediaItem.
So in order to achieve this, we wrapped the Exoplayer and every added Listener with our own implementation of the Player and Player.Listener interface and added our custom implementation of Player.getMediaMetadata(), Listener.onMediaMetadataChanged(), Listener.onMetadata() and some other methods. This custom wrapped player is then used to create the media session.
And so far everything is working fine with our implementation, with the exception of the notification.
We used the DefaultMediaNotificationProvider and even wrote our custom implementation of the MediaNotification.Provider but in both cases the content title of the notification won't change for API Levels > 29, the title is stuck to the initial value of the MediaItem.Metadata.title.
I can't really wrap my head around why the title won't change, even though a new MediaNotification with the correct content title is created and the NotificationManager should replace the existing one, like it's done on devices with API <= 29.

Ah, ok. That makes sense. Quite nifty approach! :) And yeah sounds really odd that it is only partly updated. Thanks for reporting, we need to dig into this and try to repro.

I can repro this behaviour for all API levels above and including 30 and confirm your observation.
For context and as information for us when looking into this, there are three buckets of API levels for which the notification works differently.
API levels below and including 29 that only use assets from the NotificationCompat
For API level below and including 28 there is only the NotificationManagerCompat  that has an impact on the media style notification. The media session is not taken into account even when we set the media session token into MediaStyle. This means that the notification displays what we set to the notification as contentTitle and contentText. On this API levels everything works as intended.
API 29 then takes the duration that is send to the session as part of the MediaMetadataCompat. If METADATA_KEY_DURATION is available, the notification offers a seek bar for seeking within the notification. The text is still taken from the notification just as for API 28 and below. Again, on this API level everything works as intended.
For the above API levels, our update of the media metadata triggers an EVENT_MEDIA_METADATA_CHANGED, that is received by the MediaNotificationManager and triggers updating the notification. Also, DefaultMediaNotificationProvider correctly uses controller.getMediaMetadata() to build the notification.  That corresponds with your observation that it works on this API levels.
API levels 30 and higher that use MediaMetadataCompat
Starting with API 30, the contentTitle and contentText of the notification is not used anymore. Instead these are taken from the MediaMetadataCompat as documented here.
There are various problems in the Media3 library why this doesn't work.

The EVENT_MEDIA_METADATA_CHANGED is not propagated in MediaSessionLegacyStub, and hence the platform media session is not updated when you trigger this event with an update.

If the update would be propagated, the MediaMetadata (Media3) used to be converted to the MediaMetadataCompat is taken from the MediaItem.metadata, which as we know can't be changed once set. That's kind of consistently incorrect to the EVENT_MEDIA_METADATA_CHANGED not being propagated.

And lastly again consistently, the legacy session is only updated if mediaItem.metadata has changed. Which can never happen for the same media item for the reasons  stated in.

After changing 1 - 3 above it works according to my testing.
So given #33 I think we should change this accordingly to make a workaround like yours possible, or we would need to fix #33.
Thanks for reporting.

so what to do now?!
why setting content title like below not working?
val notificationBuilder = NotificationCompat.Builder(             context,             AppConstants.PLAYER_NOTIFICATION_CHANNEL_ID         )             .setSmallIcon(R.drawable.notification)             .setContentTitle("test title")             .setContentText("test text")             .setLargeIcon(BitmapFactory.decodeResource(context.resources, R.drawable.notification_large_icon))             .setStyle(MediaStyleNotificationHelper.MediaStyle(session))

so what to do now?!
why setting content title like below not working?
val notificationBuilder = NotificationCompat.Builder( context, AppConstants.PLAYER_NOTIFICATION_CHANNEL_ID ) .setSmallIcon(R.drawable.notification) .setContentTitle("test title") .setContentText("test text") .setLargeIcon(BitmapFactory.decodeResource(context.resources, R.drawable.notification_large_icon)) .setStyle(MediaStyleNotificationHelper.MediaStyle(session))
works only on android 10 and below

@marcbaechinger
by downgrading media3 from 1.0.1 to 1.0.0-beta03 my code works now!
struggling with this for about 3 days!

Having same issue here. setContentTitle() doesn't seem to do anything on API 33. As I understand it is used only as a fallback when title couldn't be retrieved from media metadata? Would it be possible to give priority to the title which is set when building MediaNotification, because sometimes there's need to provide custom title in notification for specific mediaItem?

Please read the comments above. From API 33 the System UI build the media notification from the media session. There is no way the library/app can make System UI use the data from the notification.

That's a bummer :/ Thanks for the answer anyways


[Bug-7319][MasterServer] fix taskNode NPE when switch else branch is empty

this pr close #7319

Codecov Report

Merging #7320 (685926c) into dev (3fadfca) will decrease coverage by 0.02%.
The diff coverage is 0.00%.

@@             Coverage Diff              @@
##                dev    #7320      +/-   ##
============================================
- Coverage     41.26%   41.24%   -0.03%
- Complexity     3662     3663       +1
============================================
  Files           634      634
  Lines         26363    26388      +25
  Branches       2956     2961       +5
============================================
+ Hits          10879    10883       +4
- Misses        14453    14476      +23
+ Partials       1031     1029       -2

Impacted Files
Coverage Δ

...g/apache/dolphinscheduler/dao/utils/DagHelper.java
47.38% <0.00%> (-0.58%)
:arrow_down:

...server/master/runner/task/SwitchTaskProcessor.java
0.00% <0.00%> (ø)

...he/dolphinscheduler/common/enums/SqoopJobType.java
0.00% <0.00%> (-88.89%)
:arrow_down:

.../org/apache/dolphinscheduler/api/enums/Status.java
100.00% <0.00%> (ø)

...inscheduler/common/task/sqoop/SqoopParameters.java
74.00% <0.00%> (ø)

...cheduler/api/controller/AccessTokenController.java
88.23% <0.00%> (+1.56%)
:arrow_up:

...duler/api/service/impl/AccessTokenServiceImpl.java
85.52% <0.00%> (+1.70%)
:arrow_up:

...er/master/dispatch/host/assign/RandomSelector.java
83.33% <0.00%> (+5.55%)
:arrow_up:

Continue to review full report at Codecov.

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


Error handling in WS code is broken

There are multiple issues with error handling in WebSocket handling code. Happy-path works fine, but in case of errors, code panics.

I'm going to prioritize this because it has bothered me for too long

First step can be to change https://github.com/celestiaorg/optimint/blob/main/rpc/json/ws.go#L68 from
h.logger.Error("expected text message")

to
h.logger.Debug("expected text message")