Web API quickstart guide no longer works

Following the quickstart in https://developer.spotify.com/documentation/web-api/quick-start/ using the example authorization code no longer works.
After filling in client id and secret as needed and setting the redirect uri to 'http://localhost:8888/callback' as indicated by the guide,  clicking the "Log in with Spotify" button, produces the following error:
INVALID_CLIENT: Invalid redirect URI.
I've tried replacing it with https://example.com/callback from https://developer.spotify.com/documentation/general/guides/authorization-guide/#authorization-code-flow to test that the actual authorization code works if a correct redirect URI is supplied, and it appears that it does, but obviously the rest of the quickstart guide (displaying the /me info) doesn't work.
Please update the quickstart guide to use a different redirect URI (or update the code to ship with a working example uri) or update the guide itself if this type of redirect is no longer allowed.

Hi, did you add your redirect_uri(http://localhost:8888/callback) inside the spotify dashboard to your application?
https://developer.spotify.com/dashboard/applications

Hmm, can't say for sure 🤔 . I definitely tried adding different redirect uris in the settings during this process (and even created a whole new application just in case my original had accidentally been flagged as malicious or something). But as a final experiment, I tried deleting all redirect uris I had, in case having a whitelist at all was messing with whatever the default behavior might be. There doesn't seem to be any history/changelog in the settings and now it's been two weeks, so... maybe? I think I would have tried that, but it's possible the idea of putting a localhost address that the web api would have no way to resolve in the public application settings may have seemed too crazy to work. However, having tried it right now, I am able to log in and be redirected correctly 👍 , so it's likely that I did not.
It might be worth adding that you have to do that as a step in the tutorial. At least for me, it wasn't obvious that that would be required, especially when using a localhost address in a quickstart that doesn't mention it.

Following the quickstart in https://developer.spotify.com/documentation/web-api/quick-start/ using the example authorization code no longer works.
After filling in client id and secret as needed and setting the redirect uri to 'http://localhost:8888/callback' as indicated by the guide, clicking the "Log in with Spotify" button, produces the following error:
INVALID_CLIENT: Invalid redirect URI.
I've tried replacing it with https://example.com/callback from https://developer.spotify.com/documentation/general/guides/authorization-guide/#authorization-code-flow to test that the actual authorization code works if a correct redirect URI is supplied, and it appears that it does, but obviously the rest of the quickstart guide (displaying the /me info) doesn't work.
Please update the quickstart guide to use a different redirect URI (or update the code to ship with a working example uri) or update the guide itself if this type of redirect is no longer allowed.

This, exactly. I didn't have any issues with the authorization code flow once I whitelisted my callback API, but the implicit grant flow didn't even come close to working until I whitelisted the example .com callback. It would seem that that flow doesn't allow redirects from localhosts, which doesn't fill me with confidence since that will make it a much bigger pain to test..

@dylanesque this is very strange, it makes no sense to whitelist a random callback url that is not used in your project.
Are you sure the redirect url you have whitelisted is the one you are using in your example, there are also differences between <IP_ADDRESS> and localhost and of course http https.
Maybe you don't use the http://localhost:8888/callback endpoint but at least in the implicit flow example it is only http://localhost:8888 without callback in the end. Only put callback if you really have an /callback endpoint in your project.
That said I tested several times different auth flows via localhost which seems to work fine.
Did a little example repo for the PKCE flow: https://github.com/tobika/spotify-auth-PKCE-example

@dylanesque this is very strange, it makes no sense to whitelist a random callback url that is not used in your project.
Are you sure the redirect url you have whitelisted is the one you are using in your example, there are also differences between <IP_ADDRESS> and localhost and of course http https.
Maybe you don't use the http://localhost:8888/callback endpoint but at least in the implicit flow example it is only http://localhost:8888 without callback in the end. Only put callback if you really have an /callback endpoint in your project.
That said I tested several times different auth flows via localhost which seems to work fine.
Did a little example repo for the PKCE flow: https://github.com/tobika/spotify-auth-PKCE-example
PS: #72 (comment)

I mean, it's not a random callback uri, it's the one I'm using. At any rate, I tried adding localhost without the callback appended at the end, and that doesn't work either.


[Feature] Is there a way to get actual frame Frame/FrameImpl object from FrameLocator? How to get childFrames from FrameLocator

Is there a way to get child frames from FrameLocator?
Is there a way to get actual FrameImpl object from FrameLocator?   (asFrame()?)
ex (suggested syntax):
page.frameLocator("[name='myFrameName']).asFrame().waitForLoadState(LoadState.DOMCONTENTLOADED)
page.frameLocator("[name='myFrameName']).first().asFrame().waitForLoadState(LoadState.DOMCONTENTLOADED); // make sure get first one if multiple

page.frameLocator("[name='myFrameName']).childFrames()
// can be done like this?:  page.frameLocator("[name='myFrameName']).locator("iframe,frame")

Is there a way to get child frames from FrameLocator?

Something like this will give you locator for a child frame:
page.frameLocator("[name='myFrameName']").frameLocaltor(".childFrame")

Is there a way to get actual FrameImpl object from FrameLocator? (asFrame()?)

No, the idea is that you should not need one and stick with locators. The Locator already provides methods for performing actions in the page, but looks like you need waitForLoadState.

See also relevant discussion in the upstream repo: https://github.com/microsoft/playwright/issues/10191. Can you tell us a bit more about your use case? Perhaps it can be addressed without waitForLoadState similar to the example in the bug.

I need to do more tests to see if other event dispatch from the frame after it's been opened.   I just want to make sure if the frame is completed rendering, because, visually, even when I see a popup screen (just div contain iframe inside), it still take a few seconds to complete.  Depend on the application state, if application just start up, it will take longer time to render (due to backend slow response - class loader).

FrameLocator interface does not expose functions that Frame has (such as url, title, textContent, evaluate, screenshot, and waitForLoadState)

I need to do more tests to see if other event dispatch from the frame after it's been opened. I just want to make sure if the frame is completed rendering, because, visually, even when I see a popup screen (just div contain iframe inside), it still take a few seconds to complete. Depend on the application state, if application just start up, it will take longer time to render (due to backend slow response - class loader).

Do you expect a particular element to be loaded in the iframe (in that case you should be able to use locator.waitFor or perform a user action using element locator which will auto-wait) or you want to wait for the frame to load for some other reason?

Do you expect a particular element to be loaded in the iframe (in that case you should be able to use locator.waitFor or perform a user action using element locator which will auto-wait) or you want to wait for the frame to load for some other reason?

I think it would work if I can use locator waitFor.
If I use Locator to locate an iframe(or frame) would it be any different from using FrameLocator to locate the iframe(frame)?
internally, Does Playwright have any distinction?  Base on the code look like FrameLocator and Locator just patch CSS Selector to the object.    What if there are multiple levels iframe and middle one is refresh for some reason (could be timer), would that affect?
ex:
FrameLocator parentFame = page.frameLocator("[name='parentFrame']")
FrameLocator childFrame = parentFrame.frameLocator("[name='childFrame']");  // can combine above line into one, but I want to show the different
childFrame.locator("input[name='login']").fill("abc");
childFrame.locator("input[name='password']").fill("def");
.... parent frame has been refresh for some reason....
childFrame.locator("button[name='submit']").click();   // <-- would this get affect by parent frame refresh?

If I use Locator to locate an iframe(or frame) would it be any different from using FrameLocator to locate the iframe(frame)?

Yes, Locator will resolve to an IFRAME element whereas you want to dive into the content of the iframe which is what FrameLocator is for.

internally, Does Playwright have any distinction?

Yes, they are handled differently in the driver.

Base on the code look like FrameLocator and Locator just patch CSS Selector to the object.

Correct, but the devil is in the details, look for 'control=enter-frame' in the frame locator implementation.

What if there are multiple levels iframe and middle one is refresh for some reason (could be timer), would that affect?

That's OK, the entire selector is reevaluated every time you perform an action and if the previous frame was refreshed/removed it will resolve to the new one. This is one of the key advantages over the element handles and one of the main reasons we recommend locators over handles.

Hello @yury-s,
Is there a way to convert Locator to FrameLocator?   I want to build something generic in my framework which I can find an element by Locator and detect if it is a frame/iframe then I would like to convert it into FrameLocator.   Is it possible?   Since CSS Selector cannot select self (ex:  myLocator.frameLocator(":self")), I cannot convert it this way.    Can we do asFrameLocator() in LocatorImpl (new Locator method) and just create mew instance of FrameLocatorImpl - with the same frame and css selector object).
public FrameLocatorImpl asFrameLocator() {
    return new FrameLocatorImpl(this.frame, this.selector);
}

Use ":scope" CSS selector, it will work:
myLocator.frameLocator(":scope")


Is it possible to parallelize training?

Each sample is taking ~1 second to process. To train 100K samples it takes lot of time. If ocropus supports mutli threading/processing, training time will reduce drastically.

I cannot say anything about parallelization, but...
A faster training should be available with CLSTM. Moreover, implementations relying more on GPU power (e.g. based on Tensorflow or Torch) are appearing as well.

To benefit from the GPU speed with TF you need very expensive Nvidia card.

Add a new FAQ for this https://github.com/tmbdev/ocropy/wiki/FAQ#how-can-the-training-been-speed-up-is-it-possible-to-parallelize-the-trainig-or-to-use-gpu-power-for-training


Tray

Minimize to tray after closing any window. Add hotkey option for showing from tray.

I ended up merging your code manually and adding support for other actions.


fix(Templates) rename Template to View

Rename concepts due to https://github.com/angular/angular/commit/bf7933714a884e10124ce43aa27d63791fc37e6a

Good catches. Thanks!

/cc @davideast & @alexwolfe


Update COC to latest Contributor Covenant version

Hello! :wave:
I wanted to update the github.com/rspec/rspec-support Code of Conduct but when I checked out the file it said not to do so as it is automatically generated and taken from this repository. So, this PR updates your code of conduct to the latest version of the Contributor Covenant.
I don't know which email to add into [INSERT EMAIL ADDRESS HERE], but if you can let me know which address to use I am happy to update it.
Have a fantastic day :rainbow:.

Thanks @Charlotteis I'll take over from here to publish this out.


Add ability to create gin indexes on specific jsonb fields

For an object in my database, I need to use a containment query like this:
var matches = new[] {"abc", "xyz"};
context.Query<T>().Where(x => x.MemberIds.Any(m => matches.Contains(m)));

Marten has specific support for this query, and produces a Postgres search like this:
data -> 'MemberIds' ?| array['abc', 'xyz']`

The problem is that this type of query will not be able to use either a ComputedIndex on this field via For<T>.Index(x => x.MemberIds), nor the complete gin index on data that can be added using For<T>.GinIndexJsonData().
The computed index does not work because it selects using data ->> MemberIds and the gin index does not work because the query needs data -> MemberIds instead of just data.
It seems like the best solution here would be the ability to create an IndexDefinition instead of a ComputedIndex for this field, which could be setup as gin with jsonb_ops to support the query.
Right now the only way I've found to hack this is to directly modify the Expression on the gin index of data, so the specific field is also available with jsonb_ops instead of jsonb_path_ops to support the query:
// ugly, ugly hack
For<T>().GinIndexJsonData(def =>
 def.Expression = "data jsonb_path_ops, ((data -> 'MemberIds'::text))"
);

This hack is very fragile because the formatting of the Expression is used to when doing a SchemaUpdate check.  Those extra parens around the field and spacing is critical or it will try to rebuild the index on each schema check.
Ideally I'd like to have something like this which would create an index that supports ?| for the specific Any().Contains() query on MemberIds
public class DocumentMappingExpression<T>
{
    public DocumentMappingExpression<T> GinIndex(Expression<Func<T, object>> expression, Action<IndexDefinition> configure = null);
}

@barryhagan  I think that what you're trying to achieve is already supported. See Customizing a Calculated Index section of documentation: http://jasperfx.github.io/marten/documentation/documents/customizing/computed_index/
So you could do:
var store = DocumentStore.For(_ =>
{
    _.Connection(ConnectionSource.ConnectionString);

    // The second, optional argument to Index()
    // allows you to customize the calculated index
    _.Schema.For<YourTypes>().Index(x => x.MemberIds, x =>
    {
        // Change the index method to "gin"
        x.Method = IndexMethod.gin;
    });
});

@oskardudycz  that won't work because of how the locator is built between index and query time.
If I setup the index like you suggest, it will create an index with this DDL:
CREATE INDEX mt_doc_type_idx_member_ids ON public.mt_doc_type USING gin (  (CAST(data ->> 'MemberIds' as jsonb)));

But the query is always performed using:
data -> 'MemberIds' ?| array['abc', 'xyz']`

Those will not match each other and so it drops to a sequential scan.
I need the index to be created with the same locator syntax as the query, or alternatively it would work if the query was changed to use the same CAST locator as the index:
where (CAST(data->>'MemberIds' as jsonb) ?| array['xyz'] )

So maybe the actual request here is to change the .Any().Contains() query to match the index definition. But I'm not sure why the selection as string and cast back to jsonb is required.

Closed via #1334


Start/end frame index about youtube data.

Thanks for your great work!
You mentioned that the human subject in the clip should shows most of body regions. However, in the  invisible data, the back region is unseen during the video in project page.
Could you please provide more details about the youtube data, such as the start/end frame index and the sampling interval etc. ;)

Hello,
For invisible, I used the clip roughly between 00:01:16~00:01:23 without skipping any frames.
However, if possible, I hope you do not use these YouTube videos as a benchmark dataset. The dancing artist grants me to access his videos only for this work. They are not intended to be a public dataset for the community. Please respect his copyright and privacy.
If you want to make an algorithm comparison, please consider using ZJU-MoCap, other public datasets, or self-captured data. Thank you for taking care of this issue seriously.

Thanks for your reply and reminder! I just want to run a local test, and I will respect this copyright and privacy.


has_many as a nested object example?

https://www.elastic.co/blog/managing-relations-inside-elasticsearch <-- the nested object example here. How would that be implemented in a Chewy index?
i.e.
{
  "name" : "Zach",
  "car" : [
    {
      "make" : "Saturn",
      "model" : "SL"
    },
    {
      "make" : "Subaru",
      "model" : "Imprezza"
    }
  ]
}

Which is mapped in ES like so:
{
  "person":{
    "properties":{
      "name" : {
        "type" : "string"
      },
      "car":{
        "type" : "nested"
      }
    }
  }
}

Is it something like this? https://github.com/toptal/chewy/blob/master/lib/chewy/type/mapping.rb#L94

Hey, try this: https://github.com/toptal/chewy/#multi-nested-and-object-field-types

Thanks!


Can't build with SPM

Hey, Im trying to build the project with SPM, but getting
error: noManifest(baseURL: "https://github.com/robbiehanson/CocoaAsyncSocket.git", version: Optional("7.6.2"))

this is my Package.swift, and Im just doing swift package resolve
// swift-tools-version:4.0
import PackageDescription

let package = Package(
    name: "Heos",
    products: [
        .executable(name: "heos", targets: ["Heos"]),
    ],
    dependencies: [
        .package(url: "https://github.com/pryomoax/SwiftSSDP.git", .branch("master"))
    ],
    targets: [
        .target(
            name: "Heos",
            dependencies: ["SwiftSSDP"]),
    ]
)

sorry, i will close this as is duplicate of #1


RE3 Crash if the dogs are damage by a explosive Barrel

Version
3.0.2 (10d7467)
Seed
R731-G4NT-XXLG0FNZ2Y300N05PR80N7K7N5TURZZZ6NZ
Scenario
RE 3 (Jill)
Player Character
Annette
What happened?
In room 21A  there is 2 dogs, if you shoot the explosive barrel and the dogs are damage by the explotion the game crash
I tested it and it's the dogs being damage by the explotion what cause the crash, since i don't crash if the dogs are dead or far enough of the explotion

Debug Save here, because it didn't let me before (My own mistake)
Debug Save.zip

I confirmed that I also crash. It's actually kind of hilarious.

thanks, dogs are now banned from this room


Add user database to CRM

As an admin, I would like the CRM to hold our user database, so that we can begin linking our user database to other automated actions.
At the moment, we have users lists in partial, and in a number of different areas. We need to get our whole user list in Airtable under a single base and table, and then begin building systems off of that. Since we can't refer to multiple bases, this is going to have to suffice.

Wouldn't this be handled on operationcode.org?


Add jobs for automatically building test builds for PRs

Summary
This PR adds some new CircleCI jobs that will get triggered only when adding the Build Apps for PR label NONE
NONE

@devinbinnie just a small request, to rebase and squash the commits in one :)

@devinbinnie just a small request, to rebase and squash the commits in one :)

This will happen on merge :D


Fix #6816 - Add supportsQueryComment & Fix #7078 - Athena Session Token

Describe your changes :

Fix #6816
Fix #7078

Type of change :

[x] Bug fix
[x] Improvement
[x] New feature
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Documentation

Frontend Preview (Screenshots) :
For frontend related change, please link screenshots of your changes preview! Optional for backend related changes.

Checklist:

[x] I have read the CONTRIBUTING document.
[ ] I have commented on my code, particularly in hard-to-understand areas.
[ ] I have added tests that prove my fix is effective or that my feature works.
[x] All new and existing tests passed.

Reviewers

need to remove the property form the UI. Will push here


Spend some attention on checking the ugrid.sel method

It looks like the renumbering is not going right in all cases, in particular an example of merged partitions.

The issue here is that renumbering goes wrong, as it doesn't take fill_values into account.
Hence, a fill_value of -1 will be renumbered into a 0, thereby shifting the numbering and including a node that shouldn't be included. Best to implement a renumber function that understands a grid.fill_value.


Support --[no-]recur flags

https://github.com/petdance/ack/pull/200

I don't see why we would need to do this.


(Take 2) PF-2985: [Terra Resource Janitor] - Upgrade PGDJBC Postgresql-Jre7

https://broadworkbench.atlassian.net/browse/PF-2985
Updating TCL had some issues -- try just updating postgresql.

Connected tests pass and Janitor works locally. So I think this seems okay.


sm make_db

Which software do I need to install for running this command?
I tried sudo apt-get install sm
But this software just prints "make_db" on my screen in huge letters when I run sm make_db
Appreciate any help.

sm is just a bash-function wrapper around snakemake https://bitbucket.org/johanneskoester/snakemake/wiki/Home . See https://github.com/mpicbg-scicomp/tissue_miner/blob/7a6bb1ffe2da42a17750e6ab83c8d98e635af24c/installation/ubuntu/tm_profile.txt#L6
But actually there should no need to install it manually, since the snakemake installation is covered by the installer of TIssueMiner, see https://github.com/mpicbg-scicomp/tissue_miner/blob/master/installation/local_installation.md#ubuntu
To avoid all the installation hassle you could also run it via docker (recommended). See https://github.com/mpicbg-scicomp/tissue_miner#installation-inside-docker

Actually I'm new to docker. Could you help me out with how exactly to "run it via docker"?
Thanks

Sure, I'd suggest get started by working through our tutorials on https://github.com/mpicbg-scicomp/tissue_miner#documentation . Feel welcome to come back to us if something not clear.

When I run the command tm sm make_db (the very first command in the tutorials, after downloading the sample data), I get this:
sudo: tm: command not found

tm is an alias. See https://github.com/mpicbg-scicomp/tissue_miner/blob/gh-pages/quickstart_tutorial/other_os/tm_qs_example_data.md#2-download-the-example-data-set-100-mb

I added the alias, but still I got the error message, I think it could be due to the fact that I couldn't get the docker quickstart terminal running.

You don't need a quickstart terminal when using ubuntu.

Just pull the image (see https://github.com/mpicbg-scicomp/tissue_miner#installation-inside-docker)
fetch demo data (or prepare your own data, and configure your environment (see https://github.com/mpicbg-scicomp/tissue_miner/blob/gh-pages/quickstart_tutorial/other_os/tm_qs_example_data.md#2-download-the-example-data-set-100-mb)
run it (see https://github.com/mpicbg-scicomp/tissue_miner/blob/gh-pages/quickstart_tutorial/other_os/tm_qs_example_data.md#3-select-the-analysis-you-are-interested-in). Don't forget quotes around the arguments given to tm!

I followed this (edited .bashrc), and got this output:
jarvis@jarvis-Inspiron-5558:~/example_data/demo$ tm "sm shear_calculate topo_countt1 polygon_class tri_categorize; analyze_movie.R . output_analysis"
docker: Cannot connect to the Docker daemon. Is the docker daemon running on this host?.
See 'docker run --help'.
jarvis@jarvis-Inspiron-5558:~/example_data/demo$ sudo tm "sm shear_calculate topo_countt1 polygon_class tri_categorize; analyze_movie.R . output_analysis"
[sudo] password for jarvis:
sudo: tm: command not found

This is a docker setup issue, which is out of the scope of TissueMiner. Just search the web, which contains different solutions to overcome this problem.

"CAUTION for Linux Users: this only works if your user ID is 1000. We are working on a more generic solution for any user"
Could the problem I mentioned be due to this? How do I check my user ID on a linux machine?

The problem is probably due to the alias not being created properly when I update the .bashrc, which is not a TissueMiner problem. Thanks a lot @holgerbrandl for all the help.


NIFI-9850 Add support for multiple expressions to GrokReader

Description of PR
NIFI-9850 Adds support for multiple expressions to GrokReader.
The implementation updates the existing Grok Expression property with a new display name of Grok Expressions and enhances the property descriptor to support reading multiple expressions from a resource reference. The resource reference enables reading Grok Expressions from the property value itself as well as an external file path or URL. This approach maintains compatibility with existing property values that define a single Grok Expression.
The updated Grok Expressions property descriptor enables GrokReader to load multiple expressions, separated by newlines. The GrokRecordReader iterates over the configured expressions and returns on the first match found.
Changes include unit test optimization as well as a new unit test that configures multiple patterns to parse different log lines.
In order to streamline the review of the contribution we ask you
to ensure the following steps have been taken:
For all changes:

[X] Is there a JIRA ticket associated with this PR? Is it referenced
in the commit message?

[X] Does your PR title start with NIFI-XXXX where XXXX is the JIRA number you are trying to resolve? Pay particular attention to the hyphen "-" character.

[X] Has your PR been rebased against the latest commit within the target branch (typically main)?

[X] Is your initial contribution a single, squashed commit? Additional commits in response to PR reviewer feedback should be made on this branch and pushed to allow change tracking. Do not squash or use --force when pushing to allow for clean monitoring of changes.

For code changes:

[ ] Have you ensured that the full suite of tests is executed via mvn -Pcontrib-check clean install at the root nifi folder?
[X] Have you written or updated unit tests to verify your changes?
[X] Have you verified that the full build is successful on JDK 8?
[ ] Have you verified that the full build is successful on JDK 11?
[ ] If adding new dependencies to the code, are these dependencies licensed in a way that is compatible for inclusion under ASF 2.0?
[ ] If applicable, have you updated the LICENSE file, including the main LICENSE file under nifi-assembly?
[ ] If applicable, have you updated the NOTICE file, including the main NOTICE file found under nifi-assembly?
[ ] If adding new Properties, have you added .displayName in addition to .name (programmatic access) for each of the new properties?

For documentation related changes:

[ ] Have you ensured that format looks appropriate for the output in which it is rendered?

Note:
Please ensure that once the PR is submitted, you check GitHub Actions CI for build issues and submit an update to your PR as soon as possible.

for:

I think that should be mentioned in the property documentation explicitly
I think that it is fine to do this as a follow on
Maybe we can log the matching pattern or something to start?  At some debug level

Also, should it be an error if any of the grok patterns do not produce something matching the schema during validation?

Thanks for the reply! The current documentation for the STRING_FIELDS_FROM_GROK_EXPRESSION strategy notes that the fields will be derived from the Grok Expression, so I will update it to mention that this includes fields from all configured expressions.
As far as the matching pattern, logging the match could be verbose given the potential volume of information processed. Although this might make sense at the trace level, it could still result in hundreds or thousands of log per FlowFile. It seems better to avoid logging for now, and consider a separate task to optionally include the matching pattern index on the record.
Given the different schema strategies and the ability to configure different behavior when there are no matching expressions, it does not seem like it should be an error if the patterns do not produce something matching the schema.

I updated the schema strategy @ottobackwards, please let me know if you have any additional feedback.

@exceptionfactory if you could spare some time to look at https://issues.apache.org/jira/browse/NIFI-9863 and maybe suggest any ideas or corrects, that would be super cool


Add prop to autocomplete to place labels below or above input

I have updated the Autocomplete component to take a selectedPosition prop which allows you to choose if the selected labels should go above or below the input.
Defaults to above.
Accepts above or below.
Also updated documentation  (Am I missing somewhere else?).
Happy to make changes if there is a better way to do this, or if I need to update extra documentation or updated something wrong (Not sure how react-toolbox.d.ts works?)

I think this is ok but as something temporary thought. I think the best solution would be to remove the autocomplete selection from the Autocomplete component. The selected fields should probably be outside the component so that a parent would be able to render Chips or other component.
I have no time to do this right now, but I think it would be a great improvement. I'm going to merge this anyway while the split is coming.
Thanks for your contribution!

This seems to conflict with the direction prop but in no case does what it's supposed to do (played with it on react-toolbox.io). Am I understanding its intended behaviour wrong or did it bug out?


update FP ref, drop 10646

Closes Issue #426.
Note that "IEEE" is no longer part of the official title. (IEEE has its own equivalent standard.)

Yes, the whole world knows and uses the term "IEEE FP", while very few know about the ISO/IEC equivalent spec. However, the politically correct thing for an internationally distributed spec is to reference the ISO/IEC spec, and that's what all language specs I've worked on do.
We have a similar situation with Unicode and ISO 10646. The 10646 folks always want acknowledgement for their contributions when most people have no idea what 10646 is, but they all know Unicode! (And we are in the processing of removing the 10646 reference.)


 tags are not interpreted as whitespace when converting HTML to plaintext

Similar to https://github.com/microformats/mf2py/issues/51 and https://github.com/indieweb/php-mf2/issues/69, the Ruby parser is stripping <br> tags rather than converting them to newlines when converting HTML to plaintext.
This is very apparent on Tantek's autoformatted posts. Compare the name and content.value for one of his posts:

php parser
python parser
ruby parser

this is pretty tricky as this is how nokogiri does it, so it basically means rewriting html to text conversion :(
maybe another library does this better

@aaronpk @dissolve I have a possible solution to this issue, but I don't know enough about the codebase to know where to make the changes.
Here's a bit of code that might be useful and/or spark some ideas. It uses the aforementioned page on Tantek's website and assumes we're only interested in .e-content. That's a narrowed use case for demonstration purposes, of course.
Save the following to a file (e.g. ~/to_plaintext.rb) and run ruby ~/to_plaintext.rb in a Terminal:
require 'net/http'
require 'nokogiri'

@doc = Nokogiri::XML(Net::HTTP.get(URI('http://tantek.com/2018/061/t2/improving-test-suite-home-pages')))

@doc.css('.e-content br').each do |node|
  node.replace(Nokogiri::XML::Text.new("\n", @doc))
end

puts @doc.css('.e-content').text

The output should look like:
Appreciate the explanation and link to the source file; makes sense.

However there is still a fundamental usability problem of the discoverability of how to file issues and suggested improvements for CSS module test suites.

I would like to suggest improving the generated test suite home pages themselves (e.g. http://test.csswg.org/suites/css-cascade-3_dev/nightly-unstable/) to link directly to https://github.com/w3c/web-platform-tests/ and suggest searching it for any source files one might want to file issues (or contribute patches) for, as you demonstrated in your comment (which I will do shortly for the cascade-import-002.htm source file specifically, thanks for the pointer. Update, done: https://github.com/w3c/web-platform-tests/issues/9910).

Note: the existing text of "More information on the contribution process and test guidelines is available on the wiki page." is not really useful, as the "wiki page" that is linked to (http://wiki.csswg.org/test) has A TON of links (a maze of passages that all appear alike if you will), none of which contain the precisely useful advice that you gave in your comment! Nor is it readily obvious how to fix http://wiki.csswg.org/test as it seems to serve many purposes, and the two likely links "How to Contribute" and "Reviewing Tests" both say on their pages: "This page has been deprecated and is no longer being maintained." with a top-level link to http://web-platform-tests.org/ which is also not useful, and that's already three clicks deep (if you guessed right which links to click) with still no answer as to how to contribute to this specific module's test suite.

Where should I file an issue and/or patch for the template or generation of the home pages of CSS module test suites like the specific page http://test.csswg.org/suites/css-cascade-3_dev/nightly-unstable/?

Thanks!

☝  Note the line breaks which under-the-hood are \n characters. Success!
That's a pretty gnarly bit of code to drop in a bunch of places, so it might be worth adding a to_plaintext method (private or not) in one of the gem's classes (FormatParser, maybe?) so it can be used more frequently (akin to to_hash, to_json, etc.).
What do you think? Seem like a workable solution?


[8.x] Allow queued Artisan commands to be batched

Pretty straightforward: add simple batch support to the queue Artisan command job.

I think it would actually be better to queue your own batchable jobs which run the Artisan command you want within themselves. What if someone wants custom behavior if the batch has been cancelled? Etc.


Coda Discovery Document page

Discovery docs:

Use the Discovery main table and create a link within Caltrans / other projects: https://coda.io/d/Data-Services-Engineering_dY5Oul8-jdi/Document-Repository_suZpG#_lugRb
Make the “Documents” Page the top level and “Discovery
be below it. Rename “Discovery” to “Systems” or similar

Doc type - keep loose for now but categorizing the type of document
If related to a system or milestone include a link reference to that object so the team can easily get to it from where they are doing their work

Add a subpage for important terms
Look through Discovery subfolder in GDrive for anything else we should add

Start with Caltrans - and ask the team if they have any other items to be added

Needs discussion if we really need to do this - documents are already available in the GDrive folder to everyone on the team so is this necessary?


Improve IMAP response handling

Usually every command send to an imap server has one or several responses. Currently they don't always get treated and checked the way they should be.
This was discovered during a discussion on issue #263
For example the command APPEND:
RFC references:

https://www.rfc-editor.org/rfc/rfc3501#section-6.3.11
https://datatracker.ietf.org/doc/html/rfc9051#section-6.3.12

Possible responses:
TAG3 OK APPEND completed

..or:
TAG3 OK [APPENDUID 38505 3955] APPEND completed

There are also intermediate responses like:
+ Ready for literal data

..which indicates the server is ready to accept additional data.
Bad commands or server errors are currently only partially handled and should be handled for every command send. Every command can fail in two ways. Either with a NO or BAD alongside some additional and provider specififc error message.
To improve the response handling, every command send to the server, should return an ImapResponse::class instance which holds information such as the send command and received responses / errors. This response class might also include "magic" accessors to retrieve named data from the response (like the Attribute::class class).

@szymekjanaczek suggested to use something similar to Laravel Resources:

Like Resources in Laravel - but firstly we have to check the compatibility between responses of each operation. In the ImapResponse::class we can parse responses and return them as specific params (like eg. UIDs of newly created messages - in case of  copying or appending: OK [COPYUID <some number> <copied uids> <uids in the new folder>] Copied UIDs)

I like the idea. Using a known schema or creating one which behaves similar to a known one has definitely some benefits. But I have to admit, that I haven't knowingly worked with Laravel Resources before.
Regarding potential dependencies
This library already uses third party dependencies:
https://github.com/Webklex/php-imap/blob/cc3e72f8517b0bf169a91d9fe5831ebd5740874d/composer.json#L21-L31
But at the same time I would like to keep them at a minimum.
One additional thought. I'm a bit worried about the overhead we might create if we do integrate an response interface in the first place. How much longer will it take to fetch, lets say 1000 messages?

Okey, some thoughts:
Divide and conquer
I think we can (and maybe should) divide this project into two parts:

refactor current code base to return raw array response from IMAP command - instead of booleans.
prepare universal ImapResponse::class

Why?

If we release a version with raw data, probably it would help with gathering more people to work on this feature - more observators of possible IMAP responses. So then, with more experience, it would be easier to create ImapResponse::class
Also it could be one of the methods: ->getResponse() and ->getRawResponse() (or sth similar). So (more demanding) users of this lib will be able to work on their own on the response (good for unique individual usages or requirements).

About Laravel Resources and additional packages
There is no need to install any additional lib. We can create our own DTO as well. I mentioned Laravel's resources only as an example of data structure.

I'm not sure what do you mean at this:

One additional thought. I'm a bit worried about the overhead we might create if we do integrate an response interface in the first place. How much longer will it take to fetch, lets say 1000 messages?

Sounds good to me :+1:
I've meant that every class / object you create during the runtime will need some additional system resources. At some point this adds up and could slow down the operation.

Hmm, you're right - optimization is really important.
Additionally IMAP has its own limits - eg. query length - as I've read somewhere in docs. It's another aspect to think over.
About fetching messages - I think we can leave it at the current version - and instead focus on the following operations (and their responses):

append
copy
move
delete
mark

Because the response of fetching messages operation is already done - and works fine (but I'm a bit worried about this: performance of version 4.0.0 #443)

So for now, I suggest:

Change operations mentioned above return types to array - and as response give back raw data from IMAP.
Add custom exceptions for these operations - as the following docs says[a]:

/**
 * @return array|bool|null tokens if success, false if error, null if bad request
 */

[a] - instead of dividing into "error" or "bad request", we can temporarily treat both of them as errors and throw them as exceptions with eg. imap_last_error(). And more precise error handling build with the new ImapResponse::class.

Hi @Webklex
Can you take a look at my PR, please? It's a kinda draft of changes in the ImapProtocol.php. Changes in legacy are still needed. But I'd prefer to consult the current schema/idea with you.
Additionally - do we have any unit/feature tests?

Hi @szymekjanaczek ,
yes I agree on both 1.) and 2.) however I'm having some trouble with [a]. This would add a php-module dependency in the mix, which is currently optional (php*-imap).
There are currently no unit tests available. I gave it a try a few times, but had trouble to work with faked resources.
Also thanks for your pull request! Ignore my two questions from there - I now understand why you've required the php-imap module. I'm just not sure how to proceed - and if it's worth to try to keep php*-imap out of it.
Best regards,

Is there a reason to keep the legacy protocol around if theres going to be as major of a rewrite as the PR? From what I can see, legacy protocol was to work around not having php*-imap installed.
If a majority of the operations are going to change their return values, then this would jump the project to v5 as its a pretty breaking change. I think it's worth dropping the legacy protocol and the optional uncoupling from php*-imap  and just making it a requirement.
If pr #277 does go through, I intend to follow up with some tests if they're not included by either of you two fine individuals.

@ainesophaur ,
sounds reasonable to me. I wouldn't mind kicking php*-imap out entirely.
What are possible downsides? Folks which rely on the legacy mode are stuck with v4.x, header decoding quality could for some under certain circumstances deteriorate (currently the php*-imap module can be used as a fallback to support header value decoding - for some reason some users had reported problems with my implementation. I never truly found out why, so it could be a "ghost" problem as well). Am I missing something / someone?
Yes we are talking about a v5. I believe the proposed changes are worth to introduce some breaking changes. After all, those changes are there to help us all in the long run :)

By kicking out php*-imap we also reduce the amount of code that has to be refactored and tested. Win / Win ?

Hi @szymekjanaczek @ainesophaur ,
I've created a v5 branch: https://github.com/Webklex/php-imap/tree/v5
Please create a pull request to this branch instead of the master / main. By doing so we can play with it isolated from the main branch. Once we are happy with v5 I'm going to merge it into main.
Otherwise the main branch is locked until we've finished v5.
If you think this is a bad idea, please feel free to let me know - I'm happy to learn more :)
Best regards,

Hi!
It looks like a good idea to me - gonna start coding off-the-clock ;).
Greetings!

@szymekjanaczek how similar is the work in #277 to what you're working on?  @Webklex do you have any preferred way of discussing/planning/communicating regarding v5 work?

@ainesophaur it's a kinda draft - as I mentioned. Instead of returning less-informative booleans as a response, I switched it to give back the raw response from the IMAP server after the operation. It's quite connected with this issue - because is focussed on more detailed output. But in v5 we would like to return it like a special class (eg. ImapResponse::class) with an organized structure, similar in each operation (to work with eg. PHPStan).
About the communication way - firstly we have to gain interested people. @Webklex  what do you think about the small announcement in the README section?
@ainesophaur, can I assume, you would like to help?

Absolutely. I'm working a project that heavily depends on this library so
I'd love to contribute back.
On Fri, Oct 7, 2022, 2:50 AM Szymon Janaczek @.***>
wrote:

@ainesophaur https://github.com/ainesophaur it's a kinda draft - as I
mentioned. Instead of returning less-informative booleans as a response, I
switched it to give back the raw response from the IMAP server after the
operation. It's quite connected with this issue - because is focussed on
more detailed output. But in v5 we would like to return it like a special
class (eg. ImapResponse::class) with an organized structure, similar in
each operation (to work with eg. PHPStan).
About the communication way - firstly we have to gain interested people.
@Webklex https://github.com/Webklex what do you think about the small
announcement in the README section?
@ainesophaur https://github.com/ainesophaur, can I assume, you would
like to help?
—
Reply to this email directly, view it on GitHub
https://github.com/Webklex/php-imap/issues/270#issuecomment-1271180837,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABDYC5SUWCJVUYBFU6OP44TWB7BZXANCNFSM57S54HGA
.
You are receiving this because you were mentioned.Message ID:
@.***>

Okay, @ainesophaur so we have a similar situation. Nice - nothing motivates more than need.
I think we can start on our own discord - just DM me at szymekjanaczek#4390

Hi @szymekjanaczek ,
I appreciate your enthusiasm and time your are giving this project!  I know I'm lagging behind, sorry for that.
You are right, a discussion is probably easier via discord or something similar, compared to this issue section :)
https://discord.gg/rd4cN9h6
For example, should this library stay compatible with php >= v7/v7.2 or is >= v8 enough?
eg.: https://github.com/Webklex/php-imap/pull/277/files#diff-fa4f22d9f79d3e4dd1e5352f5e967291c02a7b3a39e32a0282d9b7d85c707c5aR132 this function came with php 8.
Given all the "new" and great features php 8 provides, it's perhaps a big blocker if this library continues supporting php < v8.


fix(lint): ignore .lock files in copyright checks

See discussion at #3982. The linter currently checks .lock files for
copyright headers, which do not need to have one.

cc @kythe


Querying for cyrillic words is case sensitive

Searching for english words is not case sensitive, but searching for russian words is case sensitive, i.e when the search term is "москва" nothing's found, while the search term "Москва" returns the correct result.

Closing since this is unmaintained. Please use soulheart instead!


Google Sheet API Import Error

Sample Name
Spreadsheet Quickstart
Expected Behavior
Run Successfully
Actual Behavior
Build Failed
Specifications

1.8.0_191
Windows

I use Eclipse Neon for JavaFX and when I copy and paste the test code from the Quickstart Guide it gives me error for all com.google imports saying they can not be resolved.
Exception in thread "main" java.lang.Error: Unresolved compilation problems:  	NetHttpTransport cannot be resolved to a type 	GoogleNetHttpTransport cannot be resolved 	Sheets cannot be resolved to a type 	Builder cannot be resolved to a type 	Sheets cannot be resolved to a type 	JsonFactory cannot be resolved to a type 	ValueRange cannot be resolved to a type

Please follow the steps in the quickstart to install the libraries:
https://developers.google.com/sheets/api/quickstart/java


Dr. Mario: Miracle Cure

Game Name (and version if necessary)
Dr. Mario: Miracle Cure
Hardware Specs (If necessary)
No response
Operating System
Windows 11 22H2
Game State
Title screen has broken shaders. Menus appear to work fine.
When starting up a level, the viruses do not appear, Dr Mario never throws the first pill, and the level never starts. Also, Dr. Mario's shading is incorrect.
Screenshots

Additional information
The console states "OpenFile failed" upon startup.

Try LLE DSP

Try LLE DSP

That fixed it! Here's a screenshot showing the game working in LLE. Looks like it requires it.


554 Flow API for one-time and periodic tasks

IMPORTANT: All PRs must be linked to an issue (except for extremely trivial and straightforward changes).
Fixes #554
Description
Implementation of FlowAPI for managing sync states and last sync date for one-time and periodic sync
Alternative(s) considered
Have you considered any alternatives? And if so, why have you chosen the approach in this PR? WIP
Type
Feature
Screenshots (if applicable)
Checklist

[x] I have read and acknowledged the Code of conduct
[x] I have read How to Contribute
[x] I have read the Developer's guide
[x] I have signed the Google Individual CLA, or I am covered by my company's Corporate CLA
[x] I have discussed my proposed solution with code owners in the linked issue(s) and we have agreed upon the general approach
[x] I have run ./gradlew spotlessApply and ./gradlew spotlessCheck to check my code follows the style guide of this project
[x] I have run ./gradlew check and ./gradlew connectedCheck to test my changes locally
[x] I have built and run the reference app(s) to verify my change fixes the issue and/or does not break the reference app(s)

@f-odhiambo Here is the PR for feedback on last sync date issue

Codecov Report

Merging #648 (25af544) into master (e1b48f2) will decrease coverage by 4.03%.
The diff coverage is 1.44%.

:exclamation: Current head 25af544 differs from pull request most recent head a1663ed.

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

@kevinmost @yigit Thanks alot for the detailed and through feedback. It helped and made the code much more readable and precise. I have resolved all of the comments and this PR is ready for review. Aside from code and general design approach please review the approach being used in FhirSyncWorker and WorkManager usage in SyncJobImpl.

@maimoonak , could you please advise a date by when the changes suggested by Kevin in his last review get incorporated?

@kevinmost Please review. I have updated it and also added the Sync status to UI where for now it sends the updated state.


@extends inside an if - blade

Hello,
When there is a @extends within a @if the compiler translates extends out of the condition.
Sample:
@if(Request::wantsJson())
{!!json_encode(['status'=>0, 'msg'=>'lorem ipsum'])!!}
@else
@extends('layouts.admin')
@section('content')
lorem ipsum inside content...
@endsection
@endif
In translated view at storage/app/framework/views:

<?php echo json_encode(['status'=>0, 'msg'=>'lorem ipsum']); ?>

<?php $__env->startSection('content'); ?>
    lorem ipsum inside content...
<?php $__env->stopSection(); ?>

make('layouts.admin', array_except(get_defined_vars(), array('__data', '__path')))->render(); ?>
This makes my return will be the json the "if" and the text "else"
Sorry for my english. Thanks!

Why do you have such kind of logic in View layer?

This is a view for when the user is not allowed to perform an action. The return can be a json (for ajax requests) or html (when a simple get).
However, this problem can occur with any condition, not only in this case.
Thank you!

I don't think you have such conditions inside views. This fails the MVC architecture. Why don't you have the condition in the controller itself, and if it's json :  return response()->json(['status' => 0, 'msg' => 'lorem ipsum']);, otherwise you return view('...');?

Definitely. Views should have nothing to do with the http layer.

I agree that the layers are mixed, but I discovered this bug accidentally. I modified the view to give a clearer perspective of the situation now without involving another layer of logic.
@if(true)
It's true
@else
This should not appear
@extends('layouts.admin')
@endif
The result was "It's true" + the contents of layouts.admin: https://s31.postimg.org/ffmjvffpn/Captura_de_tela_de_2016_06_27_11_22_14.png
This is the compiled view in storage/framework/views:

It's true

This should not appear

make('layouts.admin', array_except(get_defined_vars(), array('__data', '__path')))->render(); ?>
This behavior was expected?
Sry for my english.

@extends() must be on the first line of a blade template, which is a little weird, but whatever. Used ternary operator like this and it works!
@extends($var ? 'account.base' : 'base.base')


Added recipes for raw iron, gold and copper.

Description
Adds recipes for raw iron, gold and copper. 100% chance of 1 dust, 25% chance of a second dust (matches thermals output) + misc chance of a related dust.
Also changes ore recipes to give 1 raw ore and 33% chance of another ore. This is the same as mining the ore block with Fortune 1. I think this is a fair benefit for using silk touch then funneling it into a ore processor.
Closes #327

Checklist

[x] My code follows the style guidelines of this project (.editorconfig, most IDEs will use this for you).
[x] I have made corresponding changes to the documentation.
[x] My changes are ready for review from a contributor.

Does different tiers of grinding balls affect this ?

Yeah, it uses the default option of MULTIPLY_OUTPUT currently.

Gonna merge this, we can refine further through player feedback after release :)


I can't see the service under services.msc

Good day,
I have installed the service using below command, but i couldn't see the service on service.msc (windows service)
D:\Temp\PeterKottas.DotNetCore.WindowsService.Example.exe action:install

you have to start the console as administrator

@PeterKottas yes, I'm on as administrator.

I didn't find my service at first. Although my code was using SetName, the Services list uses the display name. I found it listed under the full namespace of my class. Once I used SetDisplayName then it showed up with the name I expected.


batch inference vs fps

@jkjung-avt
I converted my darknet weights to single batch inference  and 4 batch image inference trt detector respectively. The input resolution for my yolov4 is 1920 vs 1024 for both networks. The resulting inference time I am obtaining on the single batch inference is 0.14 seconds per image on my nvidia 1660 ti gpu. Whereas for the 4 batch inference engine I was obtaining an inference time of 0.53 seconds which is almost equal to the single inference network * 4. Is there very little performance gain in batch inference and if this is normal what use case would serve well for batch inference?

Please refer to: https://github.com/jkjung-avt/tensorrt_demos/issues/56
I don't have more insight into this issue.


Embed external resources

What this PR does / why we need it:
Add support for embedding external resources in the rendered HTML so theme developers could place their styles and other external resources in external files

Overall it looks good. Should finish the full review tomorrow.
Some notes already that most likely should be morphed into issues, if you agree to handle longer term:
Make embedding default, but able to switch it off? That would allow someone to easily host it as a website and also make it easier for devs. By default the single output file is great for users and later for us to transform it into a PDF or?
Should we cache images? Say by default anything that was fetched should be cached locally and used in subsequent renders. Question is when to invalidate? Only on url change? On url change and a flag to disable the cache?
Local file support instead of a full url?
Such as /image.png, which looks for a image.png file within the current directory?
That would make it easy to manually store image files etc.
Thoughts?


Fixes some typing issues in tensor flow adapter

Hi @Tomaz-Vieira Thanks for the PR, I have requested @FynnBe to review your PRs!


Moar Lcd

Little fixes that make it now gold to receive the form/selector PR (#867).

+1

Fixed and merged, thanks.


1.17 - Glow Vines

1.17 adds glow vines to caves, OTG needs a resource for this (generic cave wall resource?)

Added in https://github.com/PG85/OpenTerrainGenerator/commit/a0add21fee8868ab40bc8fe7206e2fc3a7c87502


Update requirement.txt

Today I tried to pip install and failed on both macOS 11.15 and Ubuntu 20.
Updated requirement.txt to the latest packages and they installed clearly on those platforms.
Also cleaned up some FutureWarning

Thank you, looks good

You're welcome.  Could you immediately publish 0.1.9 with those changes so it can be installed with pip?


error: [wrong # args] error when irule is being uploaded using AS3 template

Environment

Application Services Version:  f5-appsvcs-3.22.1-1.noarch
BIG-IP Version: BIG-IP 15.1.0 Build 0.0.31 Final

Summary
iRule upload issue when using AS3. This iRule wroks when we configure it using the GUI or other tmsh means
error: [wrong # args][switch -glob -- [string tolower \\\"[getfield [HTTP::host] : 1]
Steps To Reproduce
Steps to reproduce the behavior:

Submit the following declaration:

"irulename": {
      "class": "iRule",
            "iRule": {
                  "base64": "d2hlbiBIVFRQX1JFUVVFU1QgcHJpb3JpdHkgNTAwIHsKICBzd2l0Y2ggLWdsb2IgLS0gW3N0cmluZyB0b2xvd2VyICJbZ2V0ZmllbGQgW0hUVFA6Omhvc3RdIDogMV0iXSBcCiAgICAiZm9vLmV4YW1wbGUuY29tIiAtIFwKICAgICJiYXIuZXhhbXBsZS5jb20iIHsKICAgICAgIHBvb2wgcG9vbGZvb2JhciAKICAgIH0gXAogICAgIm90aGVyLmV4YW1wbGUuY29tIiB7CiAgICAgICBwb29sIHBvb2xvdGhlciAKICAgIH0gXAogICAgZGVmYXVsdCB7CiAgICAgICMgQmxvY2sgYWxsIG90aGVyIHJlcXVlc3RzCiAgICAgIEhUVFA6OnJlc3BvbmQgNDAzCiAgICAgIGV2ZW50IGRpc2FibGUKICAgICAgcmV0dXJuCiAgICB9Cn0K"
            }
      },

Observe the following error response:

fatal: [f5host]: FAILED! => changed=false
  cache_control: no-store, no-cache, must-revalidate
  connection: close
  content: '{"code":422,"message":"request failed with null exception","originalRequestBody":"{\"results\":[{\"code\":422,\"message\":\"declaration failed\",\"response\":\"01070151:3: Rule [/EH/Shared/irulename] error: /EH/Shared/wildcardVip-UP.ext.ehealth.fgov.be_443.tcl:2: error: [wrong # args][switch -glob -- [string tolower \\\\\\\"[getfield [HTTP::host] : 1]\\\\\\\"] ]\\n/EH/Shared/wildcardVip-UP.ext.ehealth.fgov.be_443.tcl:3: error: [undefined procedure: foo.example.com][\\\\\\\"foo.example.com\\\\\\\" - ]\\n/EH/Shared/wildcardVip-UP.ext.ehealth.fgov.be_443.tcl:4: error: [undefined procedure: bar.example.com][\\\\\\\"bar.example.com\\\\\\\" {\\n       pool poolfoobar\\n    } ]\\n/EH/Shared/wildcardVip-UP.ext.ehealth.fgov.be_443.tcl:7: error: [undefined procedure: other.example.com][\\\\\\\"other.example.com\\\\\\\" {\\n       pool poolother\\n    } ]\\n/EH/Shared/wildcardVip-UP.ext.ehealth.fgov.be_443.tcl:10: error: [undefined procedure: default][default
    {\\n      # Block all other requests\\n      HTTP::respond 403\\n      event disable\\n      return\\n    }]\\n\",\"host\":\"localhost

  content_length: '2048'
  content_security_policy: 'default-src ''self''  ''unsafe-inline'' ''unsafe-eval'' data: blob:; img-src ''self'' data:  http://<IP_ADDRESS> http://<IP_ADDRESS>'
  content_type: application/json; charset=UTF-8
  date: Fri, 11 Sep 2020 12:37:11 GMT
  elapsed: 13
  expires: '-1'
  msg: 'Status code was 422 and not [200]: HTTP Error 422: Unprocessable Entity'
  pragma: no-cache
  redirected: false
  server: Jetty(9.2.22.v20170606)
  set_cookie: BIGIPAuthCookie=5CD28FE05909FA73A79D39A52E9D23F485566CF0; path=/; Secure; HttpOnly, BIGIPAuthUsernameCookie=middleware; path=/; Secure; HttpOnly
  status: 422
  strict_transport_security: max-age=16070400; includeSubDomains
  url: https://f5host.be/mgmt/shared/appsvcs/declare
  x_content_type_options: nosniff
  x_frame_options: SAMEORIGIN
  x_xss_protection: 1; mode=block

Expected Behavior
should upload fine
Actual Behavior
syntax error while uploaded using AS3

It looks like a problem with the line continuations. I have added this to our internal product backlog as AUTOTOOL-1879.
As a work-around you can remove the line continuations. I understand that this is far from ideal, but it might get you unstuck.
when HTTP_REQUEST priority 500 {
  switch -glob -- [string tolower "[getfield [HTTP::host] : 1]"]     "foo.example.com" -     "bar.example.com" {
       pool poolfoobar
    }     "other.example.com" {
       pool poolother
    }     default {
      # Block all other requests
      HTTP::respond 403
      event disable
      return
    }
}

d2hlbiBIVFRQX1JFUVVFU1QgcHJpb3JpdHkgNTAwIHsKICBzd2l0Y2ggLWdsb2IgLS0gW3N0cmluZyB0b2xvd2VyICJbZ2V0ZmllbGQgW0hUVFA6Omhvc3RdIDogMV0iXSAgICAgImZvby5leGFtcGxlLmNvbSIgLSAgICAgImJhci5leGFtcGxlLmNvbSIgewogICAgICAgcG9vbCBwb29sZm9vYmFyIAogICAgfSAgICAgIm90aGVyLmV4YW1wbGUuY29tIiB7CiAgICAgICBwb29sIHBvb2xvdGhlciAKICAgIH0gICAgIGRlZmF1bHQgewogICAgICAjIEJsb2NrIGFsbCBvdGhlciByZXF1ZXN0cwogICAgICBIVFRQOjpyZXNwb25kIDQwMwogICAgICBldmVudCBkaXNhYmxlCiAgICAgIHJldHVybgogICAgfQp9Cg==

workaround seems to work .
as we have 230 tcl files containing irules where we previous used the line continuations mechanism this will be some extra work to make them AS3 supported.

If you are still looking for a fix, please reach out to us at<EMAIL_ADDRESS>


chore: refactor KeymanHosts

KeymanHosts is currently overly complex. However there are a number of challenges around determining which URLs to be using by various parts of the sites -- for example handling overrides for development (as we may not want to be running e.g. api.keyman.com locally).

Use a table-based structure rather than constructing URLs, so that we have explicit references to each tier's possible URLs.
Allow for overrides on DEVELOPMENT tier from a localenv.php? file
Use the same table to generate javascript constants for scripts -- and always use the constants and don't guess
Ensure TIER_TEST overrides.

Also, refactor environment getenv() calls into module so we have a single global KeymanSiteEnvironment class

Future: let users choose which sites they want to host locally, and munge internal links accordingly

Future: let users choose which sites they want to host locally, and munge internal links accordingly

See impact to https://github.com/keymanapp/website-local-proxy

From A18S14 sprint plan, we'll defer refactoring till 19.0


Cuda out of memory

Hi, I couldn't find any information on what kind of setup you used but my GTX 1080 with 8GB of Memory can only run Stage 0. Trying to run Stage 1 results in the following error message:
Check failed: error == cudaSuccess (2 vs. 0)  out of memory
Did you use it with cuDNN? I tried to use it (both v6 and v7) but make failed when using cuDNN. The error message was "caffe cudnn.hpp  error: too few arguments in function call"
so my current setup is:
Ubuntu 16.04
CUDA 8
NO cuDNN
GTX 1080 - 8GB
and I am running out of memory running the following commands:
python run_segmentation.py 1 4
sh train_fold.sh 1
any suggestions?

Hi, I used K80 and Titan Pascal Xp cards for these experiments. I think, both of these cards have 12GB of memory. For training, you can try reducing the batch size to fit into memory (line-177 in input_data_layer.py). For testing, it is surprising that 8GB of memory is not enough as I used a batch size of 3 for training with 12GB card. So, I expect that a single batch would fit into 8GB of memory.  I do not have GTX1080 to test this issue. Regarding cuDNN, I think, I didn't use cuDNN in my experiments.


refactor: add check for unsaved changes to modEditSubmissionForm

Related issue: #771
🔧 What changed
This PR refactors the modEditSubmissionForm component so that the modal only shows if a user hits the back button and has unsaved changes.
🧪 Testing instructions
In your local environment:

Make a change to a submission and hit the back button. You should see the modal.
Make a change to a submission and hit the "Save and exit" button. You shouldn't see the modal.
Click a submission and hit the back button without making a change. You shouldn't see the modal.

See the steps in the README for how to log into the moderation panel.
📸 Screenshots

Before

After

Looks like the tests are failing, can you take a look? @theyokohamalife


Google Maps Directions

Add Google Maps directions
Pokemons :

Gyms :

Pokestops :

Spawns :

Compatible with Iphones and androids.

sorry for more or less stealing your PR, I've opend a new PR #163 which does the same + human readable despawn times

My friend we are working  together on monocle dont be worry about my pr :) I'am a litle away from git and discod and some stuff code,but i have some nice ideas and maybe i  gonna do the next weekend.

Thanks. I've merged #163 which was a bit cleaner.


Improved remote data provider support

prerequisites:

[ ] branch is up-to-date with the branch to be merged with, i.e. develop
[ ] build is successful
[ ] code is cleaned up and formatted
[ ] tested with Firefox 52, Firefox 57+, Chrome 64+, MS Edge 16+

Summary
improved remote data provider support as a basis for the reference implementation in https://github.com/lineupjs/lineup_remote based on Postgres stored procedures

@sgratzl Did you finish the work on this PR? If yes, could you please describe how to use the remote data provider and what might change for current implementations (that use the local data provider)? I could not find any information in the reference implementation https://github.com/lineupjs/lineup_remote neither. Thanks.

@sgratzl Did you finish the work on this PR? If yes, could you please describe how to use (and test) the remote data provider and what might change for current implementations (that use the local data provider)? I could not find any information in the reference implementation https://github.com/lineupjs/lineup_remote neither. Thanks.

this PR re-enables the remote data provider functionality for LineUp.js (3/4). A remote data provider has the idea in the nutshell to outsource all relevant data operations (sort, filter, aggregate) to an external entity. Since LineUp mostly deals with indices and only need the real data for rendering certain rows this is possible without loading all rows. Rows are loaded only on demand.
A usage example is given in https://github.com/lineupjs/lineup_remote/blob/master/src/index.ts which implements the needed methods by performing them on the server side in python / postgres. Note: this lineup_remote repo also has some pretty neat postgres stored procedures to compute statistics (histogram, boxplots, ...) directly in Postgres.
In my opinion this branch is must a minor version change since its adds features but the API remains the same since the remote didn't work before anyhow.

closing for now, nice experiment but probably will be removed


MIG backed vGPU

Hello, I recently learnt from the Nvidia doc that MIG-backed vGPU is not supported by Kubevirt. I was wondering if you could share some context why it's not supported. Thank you.

@rthallisey can you shed some light here

Right it's not currently supported.   However it's something that we can look to support in the future.
I haven't tested it but it's possible MIG mode may already work with the device plugin.  If you follow the docs in https://docs.nvidia.com/datacenter/tesla/mig-user-guide/index.html#enable-mig-mode to setup a MIG device, the device plugin might play along.  Try it and let us know :).

Also @pomodorox, please file an issue under https://github.com/NVIDIA/kubevirt-gpu-device-plugin/issues and we can look into this.

Hi @rthallisey, thanks for the update. I have tried the Nvidia K8s device plugin and the MIG manager and it works -- I was able to create MIG devices using MIG manager and create GPU workloads consuming the MIG devices. I guess the thing unclear to me is why we do not support vGPU over MIG for Kubevirt. Was that a limitation from Kubevirt or Nvidia?

I think the only barrier was that it hadn't been tested/validated on kubevirt.  File an issue in the kubevirt gpu device plugin and I can look into getting this addressed.


Use weblate for managing UI translations

Is your feature request related to a problem? Please describe.
The way TcMenu app and plugin UI localization works is non-systematic. It is hard to track changes in English strings as the changes occurr in regular pull requests. Also, the barrier for the translators is higher as they have to be familiar with git, github, creating pull requests, etc.
Describe the solution you'd like
All the translations should be managed through the professional tools with which we will integrate our libraries. Weblate seems to be our best bet as it's completely open-source but likely other tools like Transifex are also free to use for the open-source projects like ours. We will have to do some research.

There seems to be a free weblate option for open-source projects at https://hosted.weblate.org/


Unauthorized Access

When try to deploy app in my domain, after OAuth getting error:
Unauthorized Access

User is not permitted to access this instance of PagerDuty Live. Please contact the associated site owner for access.

My Dockerfile:
FROM node:20.14.0

WORKDIR /app

COPY . .

RUN yarn install --frozen-lockfile

CMD ["yarn", "start"]

EXPOSE 3000

My .env:
VITE_PD_OAUTH_CLIENT_ID=***
VITE_PD_OAUTH_CLIENT_SECRET=***
VITE_PD_SUBDOMAIN_ALLOW_LIST=my.domain.com
VITE_PD_REQUIRED_ABILITY=teams

I try it with Classic User OAuth and Scoped OAuth. Always getting same error message.
Try OAuth from my own account and from account owner, nothing changed.
Is I doing something wrong?
Or something changes in PagerDuty OAuth? Because in all forks of this repo, only this have working "demo".
Original repo have same error, other available installations have same error.

Sorry, that was ENV problem with img. Fix it.


Feature request: Firewall, port redirection

Security
Normally, mobile devices use some NAT'ed networks or are firewalled by their ISPs.
But if we create true-p2p network where every user gets "white" IP-address we may have a problem in terms of unsolicited actions by any bad actors in network.
I propose we implement some kind of filtering that will let through only outgoing and established connections as it is advised everywhere where we talk about configuring firewalls.
I think that Android OS has some kind of firewall itself, but I'm not sure about decent configuration of it.
Features
Imagine, if you could host some website on your old mobile device? Just install some HTTP-server and use it to to serve some pages and files?
Current state of Android app enables you to do this already, but (without root) not on low ports < 1024. But if your users want to get to your site you will need to make all links include port, like my-awesome-ygg-page.com:8080 or so. This is not aesthetic and not memorable.
Also, if we want to use DNS-server on local mobile device we can't bind to port 53 (again without root), and that is the only port that OS will use to make requests. But we can configure our own Ygg-address as DNS-server in already available UI and "patch" packets on TUN to change the ports.

Would love to see this happen.

I don’t believe this to be Android-specific, but instead a general Yggdrasil consideration. We might want to move this as a result, as it is true on all platforms.

I don’t believe this to be Android-specific, but instead a general Yggdrasil consideration. We might want to move this as a result, as it is true on all platforms.

I agree.

Now we have yggstack for this purpose. If we integrate necessary parts into native library and adapt interface similar to Orbot (SOCKS/VPN mode, app selection, port forwarding view), we can solve this issue and maybe several others.


SSL ERROR

Hello Guys,
This is Milav Soni.
I download opessl-master from this link.
and i am able to cross-compilation openssl.
after this i am attach openssl include path and library path to Mosquitto library.
But when i run mosuitto_pub command on my custom board than it gives following error.
"OpenSSL Error: error:14000086:SSL routines::certificate verify failed
Error : A TLS error occurred."
please help me to sort out this error.

What certificate is it that's being verified?

Hello levitte,
Thank you very much for your response.
Actually, I linked Openssl library in mosquitto lib.
and when i run mosquitto_pub cmd with AWS Certificated to connect with AWS IoT server, i face following error.
I used TLS communication in mosquitto .
Certificate catagory Like --cafile , --cert, --keyfile ...
so please help me to sort out this example.

Hi @Milav-teq
Please try with below option
--proxy
Specify a SOCKS5 proxy to connect through. "None" and "username" authentication types are supported. The socks-url must be of the form socks5h://[username[:password]@]host[:port]. The protocol prefix socks5h means that hostnames are resolved by the proxy. The symbols %25, %3A and %40 are URL decoded into %, : and @ respectively, if present in the username or password.
Example : jram@edison:~# mosquitto_sub --cafile rootCA.pem --cert tomato.cert.pem --key tomato.private.key -h (Imnotgiving myend point).iot.us-east-1.amazonaws.com -p 8883 -q 1 -d -t topic/test -i clientid2
If it's works fine then Please close this issue.
Thanks
Jayaram Matta

Hello  Jayaram Matta,
It is worked.
thank you for your response.


Temurin Java downloads says x32 instead of x86 but x32 is something else

Changed x32 to match x86

Fixes https://github.com/adoptium/adoptium.net/issues/1948
cc @gdams @hendrikebbers
Checklist

[ ] npm test passes
[ ] documentation is changed or added (if applicable)
[ ] permission has been obtained to add new logo (if applicable)
[ ] contribution guidelines followed here

You'll have to apply the same logic on the archive and nightly pages too


Job stuck @ created state

From https://argus.scylladb.com/tests/scylla-cluster-tests/3da61c09-c93f-45c4-95c1-1c4ef2cf9887 :

I'm not sure why it's in this state.

Looks like a problem with the builder:


Update joda-time to 2.12.4

About this PR
📦 Updates joda-time:joda-time from 2.12.0 to 2.12.4
📜 GitHub Release Notes - Version Diff
Usage
✅ Please merge!
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

⚙ Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "joda-time", artifactId = "joda-time" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "30 days" },
  dependency = { groupId = "joda-time", artifactId = "joda-time" }
}]

labels: library-update, early-semver-patch, semver-spec-patch, commit-count:1

Superseded by #1636.


Fix(Examples): MSDK-1203 Update READMEs to remove outdated BOARD info

Solved the issue captured in MSDK-1203

/clang-format-run


SC allows requests even if incorrect project name is provided

Describe the bug
Suppose the client requests to connect to "projectA" on port X.
If SC is running on port X and doesnt have the project "projectA" (but has project "projectB"), in some cases, the client is given an error, but in other cases, it connects to "projectB"
Expected behaviour
An error should be returned to the client
Steps to reproduce
Use "allow" as the rule for file man or for services&functions
Your environment

Space Cloud version: v10.0.1
OS: Windows
The client api version (if applicable):
Browser (if applicable):

If this bug restricts your use of space-cloud, give it a thumbs up reaction, so that we can determine which bugs need to be fixed immediately.
👍

Resolved in 0.10.1


Submitting Xunlei version <IP_ADDRESS>0.yaml

Add Xunlei
Microsoft Reviewers: Open in CodeFlow

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

uninstall is not clean.  leaves a number of files behind


#86dtfv7aw - Error when compiling smart contract with custom class th…

…at has a method that returns an explicitly typed dict
Summary or solution description
An error was being thrown when using an attribute inside a dict and typing the dict in the return.
How to Reproduce
https://github.com/CityOfZion/neo3-boa/blob/a2e9ac581e53b58118a879eea326edb11090a104/boa3_test/test_sc/class_test/ReturnDictWithClassAttributes.py#L1-L53
Tests
https://github.com/CityOfZion/neo3-boa/blob/a2e9ac581e53b58118a879eea326edb11090a104/boa3_test/tests/compiler_tests/test_class.py#L463-L491
Platform:

OS: Windows 11 x64
Python version: Python 3.11

Task linked: CU-86dtfv7aw Error when compiling smart contract with custom class that has a method that returns an explicitly typed dict

coverage: 91.195% (+0.001%) from 91.194%
when pulling a2e9ac581e53b58118a879eea326edb11090a104 on CU-86dtfv7aw
into 504b102c8e8273f767a03314ecc939ddb3fcac04 on development.


incorrect left margin for day event

After update to 0.3.6 i have another issue with displaying day events.
It seems to JS concat strings, and position of event is huge :)  So this mean that  event properties are treated  as strings

I'll do an investigation. In demo 0.3.6 everything is fine. I did diff with my custom version, but couldn't see nothing.  I'm using Polish locale  import 'intl/locale-data/jsonp/pl-PL';  and that's only difference with official version.
Till this time, maybe someone has na idea why is JS  treat day event properties as strings.

I think the issue is with your day view, the option eventWidth="150" is treated as a string, and should be [eventWidth]="150"

it's not this :(
 template:
<div
class="cal-event mdl-list"
[style.marginTop.px]="dayEvent.top"
[style.marginLeft.px]="dayEvent.left + 70"
[style.height.px]="dayEvent.height"
[style.width.px]="dayEvent.width - 1"
[style.backgroundColor]="dayEvent.event.color.secondary"
[style.borderColor]="dayEvent.event.color.primary"
[class.cal-starts-within-day]="!dayEvent.startsBeforeDay"
[class.cal-ends-within-day]="!dayEvent.endsAfterDay"
[ngClass]="dayEvent.event.cssClass">

<mwl-calendar-event-title
[event]="dayEvent.event"
view="day"
(click)="eventClicked.emit()"
class="mdl-list__item-primary-content">

<mwl-calendar-event-actions
[event]="dayEvent.event"
class="mdl-list__item-secondary-action">

``
but i'll investigate it  :)

If you can make a runnable repo that reproduces the issue I'm happy to dig in and try and see what's going wrong :)

ok,  if I can not find fault, i'll setup my own demo version for you

I Found!
When i saw your last changes  in calendar-utils
https://github.com/mattlewis92/calendar-utils/commit/d0f4ce020b959edd36834c75a0ce3bc5f511748b
i was wonder what will it produce after conversion to JS  so i took a look at
C:\DEV\calendar.localhost\node_modules\calendar-utils\dist\src\calendarUtils.js
and i saw
        var left = 0;
        while (overlappingPreviousEvents.some(function (previousEvent) { return previousEvent.left === left; })) {
            left += eventWidth;
        }

after change to  (parseInit)
        var left = 0;
        while (overlappingPreviousEvents.some(function (previousEvent) { return previousEvent.left === left; })) {
            left += parseInt(eventWidth);
        }

my calendar looks fine

I thought that might be why, eventWidth should always be a number, how are you passing it through?

<mwl-calendar-day-view
          *ngSwitchCase="'day'"
          [viewDate]="viewDate"
          [events]="events"
          eventWidth="250"
          locale="pl-PL"
          hourSegments="2"
          dayStartHour="6"
          dayEndHour="21">
        </mwl-calendar-day-view>

eventWidth="250" should be [eventWidth]="250", otherwise angular treats it as a string. Probably the same for the other numeric bindings as well

so True!


Parses the remaining events from the spreadsheet

This code reads the following information from the spreadsheet: Socials, Plenary Sessions, Workshops, Tutorials, Findings, Industry Track, Demo Sessions, Coffee Breaks, Diversity and Inclusion, Student Research Workshop and Birds of a Feather.
Note that the spreadsheet doesn't have information about the specific papers shown in each one of these events. Therefore, some of this code will have to be replaced for code that actually reads the papers.
Finally, this code also modifies the template for the daily overview to remove empty Presentation types. That is, the user no longer sees (for instance) "Oral Presentations" if there are no Oral Presentations in that slot.

FYI, for changes to appear on the website, need to run python acl_miniconf/import_acl2023.py then commit conference.json. I'll make a private GDrive folder to have current versions of the source files.


Use it with NextJs

Hi, thank you for the plugin. I tried with CRA and it is working. But, now I am doing a project with NextJS and I do not know what should I do to make it work.
Any suggestions?
Thank you.

@Alfrex92 as far as I know nextjs allows you to setup custom .babelrc file. So everything you need is just to add this plugin to .babelrc.

I tried this plugin with Nextjs and got the error from babel. It said that current plugin crashes during converting a template.
The problem is that new babel@7 made some breaking changes. It was fixed within #48 by @shirohana.
@Alfrex92 please, try new version 6.0.1 which contains the necessary fix. Feel free to re-open the issue if my answer is not helpful ;).


Documentation for Workers / MailChannels integration

Which Cloudflare product(s) does this pertain to?
Workers
Subject Matter
The MailChannels/Workers integration (as described on the Cloudflare Blog)
Content Location
Although not a fully-realized runtime API, it would make sense to have it in the Runtime APIs section
Otherwise, a Tutorial or Example in their respective sections of the Workers API would make sense.
Additional information
I spent a bit of time today putting together what I would consider to be a good overview. Although it relates to more than specifically workers, I believe it would be useful to go into some depth about the DNS requirements, good practices, etc.
https://github.com/zaccomode/better-cfw-mailer-docs
I wouldn't consider this documentation to be the golden example and there is definitely more depth that could be explored in proper docs, but it covers more than any individual resource I could find online at the moment.

@codewithkristian @rickyrobinett Bringing your attention to this tutorial! Tagged as dev education for your review.

@JacobMGEvans / @admah, also not sure if you'd be able to help with this one.

Howdy @zaccomode, apologies for the delay here.... I believe this might be mostly addressed by the Pages documentation.
Closing this out as likely completed since it covers most of the points hit in your docs, but feel free to re-open another issue if this doesn't meet the spirit of your issue.


Status bar text behaviour

From Cocoaforge forums: Often I hover over a link in an article to see where it goes, only to have the status bar overwritten with the string, "Article load completed." It would be nice if this advisory were to be suppressed if the status bar is in actual use at the time. Thanks.

Perhaps it would be an idea to get rid of the status bar eventually. The fetch status is all over the place: in the toolbar, in the sidebar next to each feed, a notification and the status bar. I think it would be most useful in the sidebar. Alternatively, there could be a dialog at the bottom of the sidebar, like Mail has.

The URL description could be shown in an overlaying status bar, like in Safari.

Related to PR #936


WK in DS ult will spawn 100s of illusions

https://cdn.discordapp.com/attachments/317271861966929921/319091045667438593/unknown.png

Should be fixed now. (indirectly caused by Armlet's issue)


Gist routing is interfering with caching

Our Gist redirector is meaning that we're not serving our static content as static. This means that we're currently preventing any of it being cached - and we don't get the benefit of CDN provision for any of our assets.
Lets consider strategies for fixing this.

We should set up some regexs in our web/app.yaml so that real files get served directly - only things which are (or look like) gist redirects should get served by the python code.


Transfer probabilistic_occupancy_grid package from sensing to perception directory

Description

Transfer a package from sensing/ directory to perception directory.
See disscussion in #583.

No need to change message name and folders
Verified with rosbag sample simulation

Pre-review checklist for the PR author
The PR author must check the checkboxes below when creating the PR.

[x] I've confirmed the contribution guidelines.
[x] The PR follows the pull request guidelines.

In-review checklist for the PR reviewers
The PR reviewers must check the checkboxes below before approval.

[ ] The PR follows the pull request guidelines.

Post-review checklist for the PR author
The PR author must check the checkboxes below before merging.

[ ] There are no open discussions or they are tracked via tickets.

After all checkboxes are checked, anyone who has write access can merge the PR.

DCO must be resolved before merging. (I could be due to me clicking updating branch button in PR.)


occm: Fix LoadBalancer deletion when the underlying LoadBalancer does not exist

/kind bug
Which issue this PR fixes(if applicable):
Fixes https://github.com/kubernetes/cloud-provider-openstack/issues/1912
Release note:

[openstack-cloud-controller-manager] An issue preventing Service of type LoadBalancer to be deleted when the corresponding OpenStack LoadBalancer does no longer exists is now fixed.

/lgtm
/approve


stickyHeaderIndices not support nested child

env:
"react": "16.0.0",
"react-native": "0.51.0",
iOS:8.0
I set the prop stickyHeaderIndice on ScrollView
stickyHeaderIndices={[0]}
<View               key={'rightscrollviewheader'}               style={{ flexDirection: 'row', alignItems: 'center', backgroundColor: 'white', height: 40, }}>               {this.state.rightHeader.map((title, idx) => this._favHeader(title))}             </View>
the children will not layout by flexDirection Row

sorry
i have fiex the issue.
i use component TouchableHighlight wrappers the View. but i do not know why


[Dubbo-issue1054][Baiji-02]Support to send the Consumer application name to the Provider.

Support to send the Consumer application name to the Provider.
Get the application name of the Consumer by calling the getConsumerAppName method of RpcContext.

I think it can work, but only for DubboProtocol, it would be better if consider other protocols, such as HessianProtocol, etc.
No need of extra app properties.


Font display

Hi! 😃 , This PR has a workaround the problem for loading fonts in the Curve Editor.
This is a bug "Test font loading in Edge doesn't load sometimes" #8478
It appears that the problem is caused by trying to copy the styles. We get a CSS load error.

DOMException: Failed to read the 'cssRules' property from 'CSSStyleSheet': Cannot access rules
    at Function../inspector.ts.Inspector._CopyStyles (http://localhost:1338/.temp/localDevUMD/inspector/babylon.inspector.bundle.js:77426:32)
    at Function../inspector.ts.Inspector._CreatePopup (http://localhost:1338/.temp/localDevUMD/inspector/babylon.inspector.bundle.js:77632:14)
    at PopupComponent../components/popupComponent.tsx.PopupComponent.openPopup (http://localhost:1338/.temp/localDevUMD/inspector/babylon.inspector.bundle.js:75089:80)
    at PopupComponent../components/popupComponent.tsx.PopupComponent.componentDidMount (http://localhost:1338/.temp/localDevUMD/inspector/babylon.inspector.bundle.js:75082:14)
    at commitLifeCycles (http://localhost:1338/.temp/localDevUMD/inspector/babylon.inspector.bundle.js:47995:22)
    at commitLayoutEffects (http://localhost:1338/.temp/localDevUMD/inspector/babylon.inspector.bundle.js:50984:7)
    at HTMLUnknownElement.callCallback (http://localhost:1338/.temp/localDevUMD/inspector/babylon.inspector.bundle.js:28369:14)
    at Object.invokeGuardedCallbackDev (http://localhost:1338/.temp/localDevUMD/inspector/babylon.inspector.bundle.js:28418:16)
    at invokeGuardedCallback (http://localhost:1338/.temp/localDevUMD/inspector/babylon.inspector.bundle.js:28473:31)
    at commitRootImpl (http://localhost:1338/.temp/localDevUMD/inspector/babylon.inspector.bundle.js:50722:9) {stack: <accessor>, code: 18, name: 'SecurityError', message: 'Failed to read the 'cssRules' property from 'CSSStyleSheet': Cannot access rules'}

By adding the fonts on the Curve Editor stylesheet we are able to load the fonts correctly.

This is weird because we do not have the issue with NME for instance (which is also pop out) or the texture inspector

I'll look into NME implementation, to see what is the difference there... A few weeks ago the curve editor was able to load the fonts correctly though...

I think I found what to do. In NME there is this fix for the fonts (link) and late binding. This is not in the Inspector.ts file, so I might add a Popup.ts class.

deal!


[helm/prometheus] Rename extra rules

If having a release that contains prometheus then those extra rules ConfigMap names will be something like prometheus-prometheus-rules
Having the rules suffix clashes with the ConfigMap name for the default prometheus rules.

Timing out issues again?
--- FAIL: TestPrometheusMultiplePrometheusRulesDifferentNS (318.10s)
	prometheus_test.go:623: waiting for alert 'first-alert' to fire: timed out waiting for the condition: expected 1 query result but got 0
=== CONT  TestPrometheusMultiplePrometheusRulesSameNS
--- FAIL: TestPrometheusRulesExceedingConfigMapLimit (330.48s)
	prometheus_test.go:665: waiting for alert 'my-alert-0' to fire: timed out waiting for the condition: expected 1 query result but got 0

These Tests just got merged. Please ignore them if they are flaking. cc @mxinden

Rebased on master


Added background parameter

I added the newly added background parameter from the API.

@LukiLeu Really sorry, I didn't see your PR here until I already merged my own PR (#99) that adds a --background flag. Hope that's OK.

No problem, it is probably the better approach to work with a flag.


Adding NSDC code

Initial pull request for the NSDC reading code. Please check the function names, whether they are clear and not confusing in the big picture of tudatpy. Make sure to load the necessary kernels, in this case 'jup344.bsp'. You can verify with the following file:
ji0003.txt
You should get the following results:
Means: [ 4.82986612e-07 -4.81475772e-07] Stddevs: [1.54415899e-06 1.06032470e-06]

@FDahmani  Could you add the script that should be run, from which I would get this output?


Fix __valueKeyWithHeaderIndex() machingIndexes

machingIndexes were getting as key the value of the values parameter, and for dict. value it got index if possition in header. Keys in dict were in conflict so it return less elements in dict as it was suppose to.

Coverage remained the same at 97.96% when pulling 3d9b809302e7a08f9d37ac9568bff2bb70fea5b9 on development into ca3719943f0dd07e714c701ea071b4d2e8b2aee7 on master.

Coverage remained the same at 97.96% when pulling 3d9b809302e7a08f9d37ac9568bff2bb70fea5b9 on development into ca3719943f0dd07e714c701ea071b4d2e8b2aee7 on master.


Fixed #24690 -- Added a warning about mutable defaults for ArrayField/JSONField.

https://code.djangoproject.com/ticket/24690

I'm not sure which is not what you want is useful. At best it sweeps the mutable-default part of python under the rug and implicitly says "just don't", at worst it comes off as condescending. Leaving it out doesn't ruin the sentence, and sets the user up to at least google/ask why mutable defaults are a no-no if they care to find out.

Thanks for the suggestion.


Allow generic reference type for subimage

As a proposed solution to #762 , this allows any reference type as the underlying storage to SubImage. This makes it possible to use the non-mutable &ImageBuffer. Only mutable reference types implement the GenericImage trait.

I like this implementation a lot, but I think we could do better here. The problem I see is that I'm basically unable to call any non mutating GenericImage functions on SubImage because we can only provide an implementation for &mut I since GenericImage also requires get_pixel_mut and put_pixel. This is a bit inconvenient imho because we'd be forcing a call to to_image to copy the whole view just to call some non mutating methods.
My proposal would be to create a new GenericImageView trait that contains only the non mutating methods and then GenericImage would also require GenericImageView. This way we could implement GenericImageView safely for SubImage<&Self> and in addition we could still implement GenericImage for SubImage<&mut Self>. I think that this shouldn't break any code, but I'm not totally sure.
We could also implement this in another PR if you think it's a bit out of scope in this PR since it requires changing GenericImage.
@HeroicKatora @bvssvni what do you think?

@d-dorazio Sounds like a good idea. If this is a weakness in the current API then a breaking change is reasonable.
We won't publish yet, so this could be implemented in another PR.

Merging.

Thanks!


fix: pass attributes through in BDropdownItemButton

Describe the PR
Currently, attributes passed to BDropdownItemButton are swallowed. But we want to add them to the list element like it was in BootstrapVue.
PR checklist

What kind of change does this PR introduce? (check at least one)

[x] Bugfix :bug: - fix(...)
[ ] Feature - feat(...)
[ ] ARIA accessibility - fix(...)
[ ] Documentation update - docs(...)
[ ] Other (please describe)

The PR fulfills these requirements:

[x] Pull request title and all commits follow the Conventional Commits convention or has an override in this pull request body This is very important, as the CHANGELOG is generated from these messages, and determines the next version type. Pull requests that do not follow conventional commits or do not have an override will be denied

@VividLemon I did the changes as suggested 👍.

@VividLemon Can we have a version published since this is blocking us shipping sth.? 😊


Using color picker with Angular 2.1.0 causes problems

After including the ColorPickerModule in my AppModule class.  Any attempt to use the Color Picker with a  webpack compiled app produces the following message in console.
Uncaught Error: Unexpected value 'e' imported by the module 'AppModule'(…)
Not particularly useful I know.  Can you give me a rough guide as to when the Color Picker will be moving towards Angular 2.1.0?
The problem does not occur with SystemJS, but that is expected, as SystemJS does not package up the requisite parts from the angular node modules.
Also strangely, when not including the color picker and not uglyfying the code, the app.js chunk is 34kb.  When including colorpicker and using Angular 2.1.0, the app.js is 900kb.  It appears as if the the entire of the Angular 2.0.0 system that the color picker requires is included in app.js.

It fails with Angular 2.0.0 with error "Uncaught Error: Unexpected value 'e' imported by the module 'AppModule'(…)" while the previous version failed with "Uncaught Error: Unexpected value 't' imported by the module 'AppModule'(…)".
I use webpack to build an app.

For some reason, if I use Angular 2.0.0 and webpack it does work, but from Angular 2.0,1 onwards it fails. I can only guess as to what has changed, but the Angular 2 did introduce AOT, though that could be a red herring in this case.

There is a example with webpack in the project. I've compiled with Angular 2.0.0 and Angular 2.1.0 and it works

Thanks, I have just updated my project to use version 1.3.0 and it now loads the color picker correctly.

It's started to fail again as of angular 2.2.3 with the same error in the first post , here are my versions:
"@angular/common": "^2.2.3",
"@angular/compiler": "^2.2.3",
"@angular/core": "^2.2.3",
"@angular/forms": "^2.2.3",
"@angular/http": "^2.2.3",
"@angular/platform-browser": "^2.2.3",
"@angular/platform-browser-dynamic": "^2.2.3",
"@angular/router": "^3.2.3",
"angular2-color-picker": "^1.3.0",

In version 2.2.0 , I get this error again.

I'm getting this error too. I'm using the latest version of angular which as of right now is 5.0.2.
I'm assuming this library hasn't been updated for angular 4


test: Local URL should pass (currently doesn't)

https://github.com/adil192/saber/issues/247

Done
https://github.com/agungnursatria/regexpattern/blob/master/test/pattern/url_test.dart