ARC: Add multcore (in SMP arrangement) ARC HS in nSIM

NOTE: There's a prerequisite - https://github.com/zephyrproject-rtos/zephyr/pull/17747 and so only the last commit in this fake series to be reviewed & discussed as after https://github.com/zephyrproject-rtos/zephyr/pull/17747 is merged (hopefully anytime now) I'll rebase on top of it and those commits will be gone.
Now when SMP support for ARC is available we may introduce a simulation platform which might be used for testing & app development for SMP setups.
One important note is stand-alone nSIM (as well as its "Free" flavour) doesn't support SMP simulation so we have to switch to use of nSIM via proprietary MetaWare debugger (https://www.synopsys.com/dw/ipdir.php?ds=sw_metaware) and so:

We introduce new emulation target "mdb"
It's only possible to run that platform for those who have MetaWare tools installed and valid license.

Though QEMU port for ARC is in work at the moment and once we open that port and it has SMP support we'll switch to it and everybody will be able to try ARC HS with SMP.
If one has MDB license to try how SMP works just:
# mkdir build
# cd build
# cmake -DBOARD=nsim_hs_smp ../tests/kernel/smp/
# make run

System Architecture Configuration
--------------------------------------------------------------------------------
+ SYSTEM:'__TOP__' { RAM: [ latency:5, clock_freq:250[MHz], clock_divider:1, bus_width:32 ] }
  + MODULE:'MOD0'
    + CONNECT:'MOD0_CONNECT' {
          V:2 - ASI:0, ICI:2, ICS:1, ICM:1, PMU:1, ICD:2, GFRC:1, LLM:0, IDU:2, PDM:0
        , aap_num:0, ics_num_semas:16, pmu_dvfs:0, icm_sram_size:512, icm_sram_ecc:0, ici_irq:19
        , llm_size:131072, llm_base:2, llm_ecc:0, idu_cirqnum:64, idu_irq_base:24
      }
    + SCU:'SHARED COHERENCY UNIT'
    + CORE:'MOD0_CPU0' { UARCH:HS }
      + CACHE:'CPU0_ICACHE' { size:64KB, line:64B, sets:256, ways:4, repl:Pseudo-Random }
      + CACHE:'CPU0_DCACHE' { size:64KB, line:64B, sets:512, ways:2, repl:Pseudo-Random }
      + BPU:'CPU0_BPU' { HS - ways:1, sets:2048, pt_entries:16384, ras_entries:8 }
    + CORE:'MOD0_CPU1' { UARCH:HS }
      + CACHE:'CPU1_ICACHE' { size:64KB, line:64B, sets:256, ways:4, repl:Pseudo-Random }
      + CACHE:'CPU1_DCACHE' { size:64KB, line:64B, sets:512, ways:2, repl:Pseudo-Random }
      + BPU:'CPU1_BPU' { HS - ways:1, sets:2048, pt_entries:16384, ras_entries:8 }
--------------------------------------------------------------------------------
Console now belongs to UART, hit CRTL-] to return to simulator.
***** Booting Zephyr OS build zephyr-v1.14.0-2835-g36f26ad68ab5 *****
Running test suite smp
===================================================================
starting test - test_smp_coop_threads
PASS - test_smp_coop_threads
===================================================================
starting test - test_cpu_id_threads
PASS - test_cpu_id_threads
===================================================================
starting test - test_coop_resched_threads
PASS - test_coop_resched_threads
===================================================================
starting test - test_preempt_resched_threads
PASS - test_preempt_resched_threads
===================================================================
starting test - test_yield_threads
PASS - test_yield_threads
===================================================================
starting test - test_sleep_threads
PASS - test_sleep_threads
===================================================================
starting test - test_wakeup_threads
PASS - test_wakeup_threads
===================================================================
Test suite smp succeeded
===================================================================
PROJECT EXECUTION SUCCESSFUL

Re-based on the current master so that only one patch that matters is left.
Should be ready for merge.

@ruuddw could you please look if my comment addresses your concern?

Shippable complaints, looks like openocd Zephyr awareness doesn't work with SMP (tries to get offset of the 'current' field in struct z_kernel, but that doesn't exist in SMP.
It's not really related to this patch, but would be good to fix. Maybe disable openocd debugging support for now, and create a new issue?

@ruuddw indeed I completely forgot to run sanity check for nsim_hs_smp and treated Shippable build failure as yet another false positive one (seeing mention of "openocd" which looked completely unrelataed to me).
Reported https://github.com/zephyrproject-rtos/zephyr/issues/18124 and fixed with https://github.com/zephyrproject-rtos/zephyr/pull/18125.
So once https://github.com/zephyrproject-rtos/zephyr/pull/18125 gets merged we may rebase this one or merge this one first if that's OK as now sanity checker is happy:
./scripts/sanitycheck -p nsim_hs_smp -b
JOBS: 96
Cleaning output directory .../zephyr/sanity-out
Building testcase defconfigs...
Filtering test cases...
140 tests selected, 119003 tests discarded due to filters
total complete:  138/ 138  100%  failed:    0
138 of 138 tests passed with 0 warnings in 112 seconds

Thanks for trying to do a quick fix for openocd - not sure though if that is the right fix and gets merged soon. Why not disabling the (experimental) OPENOCD_SUPPORT config for this target to not be dependent?

@nashif @andrewboie any thoughts on what should be done here? Do we need to fix https://github.com/zephyrproject-rtos/zephyr/issues/18124 first given it's a generic problem and should have been caught long ago on either SMP platform?

@nashif @andrewboie any thoughts on what should be done here? Do we need to fix #18124 first given it's a generic problem and should have been caught long ago on either SMP platform?

IMO put a workaround in this PR to disable the test. And leave the bug open so someone still fixes it, with a note saying to revert the workaround once fixed.

@nashif @andrewboie any thoughts on what should be done here? Do we need to fix #18124 first given it's a generic problem and should have been caught long ago on either SMP platform?

IMO put a workaround in this PR to disable the test. And leave the bug open so someone still fixes it, with a note saying to revert the workaround once fixed.

You mean disable OPENOCD_SUPPORT for this particular platform?

You mean disable OPENOCD_SUPPORT for this particular platform?

I mean, do whatever is least invasive to get CI to stop rejecting this PR.

You mean disable OPENOCD_SUPPORT for this particular platform?

I mean, do whatever is least invasive to get CI to stop rejecting this PR.

I'm not sure in Kconfig it's possible to black-list an option from a board's defconfig, so if I'm not missing anything we may add some dependency for OPENOCD_SUPPORT.
Either:
diff --git a/subsys/debug/Kconfig b/subsys/debug/Kconfig
index a55d0f3350..a5dff0ad4f 100644
--- a/subsys/debug/Kconfig
+++ b/subsys/debug/Kconfig
@@ -223,6 +223,7 @@ config EXCEPTION_STACK_TRACE

 config OPENOCD_SUPPORT
        bool "OpenOCD support [EXPERIMENTAL]"
+       depends on !SMP
        select THREAD_MONITOR
        select THREAD_NAME
        help

or
diff --git a/subsys/debug/Kconfig b/subsys/debug/Kconfig
index a55d0f3350..a5dff0ad4f 100644
--- a/subsys/debug/Kconfig
+++ b/subsys/debug/Kconfig
@@ -223,6 +223,7 @@ config EXCEPTION_STACK_TRACE

 config OPENOCD_SUPPORT
        bool "OpenOCD support [EXPERIMENTAL]"
+       depends on !SOCNSIM_HS_SMP
        select THREAD_MONITOR
        select THREAD_NAME
        help

latter is only applicable to this particular PR but makes not that much sense as the problem is for all SMP platforms, former is IMHO better but has nothing to do with this particular PR, thought I may submit it as a temporary work-around for https://github.com/zephyrproject-rtos/zephyr/issues/18124.

I don't think anything is required for this PR. First, I thought OPENOCD_SUPPORT was explicitly enabled somewhere in this PR. However, it isn't. The shippable fail was unrelated, just a random config that happened to fail (zephyr/samples/synchronization/sample.yaml enables it - now triggered by ARC HS SMP, but could be triggered by any SMP target).

@vonhust can you check the CI error?

@vonhust can you check the CI error?

I made some fixes, except the following one.

I don't think anything is required for this PR. First, I thought OPENOCD_SUPPORT was explicitly enabled somewhere in this PR. However, it isn't. The shippable fail was unrelated, just a random config that happened to fail (zephyr/samples/synchronization/sample.yaml enables it - now triggered by ARC HS SMP, but could be triggered by any SMP target).

So it should be ok to merge

@vonhust can you check the CI error?

I made some fixes, except the following one.

I don't think anything is required for this PR. First, I thought OPENOCD_SUPPORT was explicitly enabled somewhere in this PR. However, it isn't. The shippable fail was unrelated, just a random config that happened to fail (zephyr/samples/synchronization/sample.yaml enables it - now triggered by ARC HS SMP, but could be triggered by any SMP target).

So it should be ok to merge

@vonhust thanks for the update, let's wait for CI. However, your fix-bug commit needs to be (formally) approved before we can merge that PR; is it possible for @ruuddw @abrodkin to give a +1 today? If not, I would move this to 2.1.0 release, is this OK with you?

@vonhust can you check the CI error?

I made some fixes, except the following one.

I don't think anything is required for this PR. First, I thought OPENOCD_SUPPORT was explicitly enabled somewhere in this PR. However, it isn't. The shippable fail was unrelated, just a random config that happened to fail (zephyr/samples/synchronization/sample.yaml enables it - now triggered by ARC HS SMP, but could be triggered by any SMP target).

So it should be ok to merge

@vonhust thanks for the update, let's wait for CI. However, your fix-bug commit needs to be (formally) approved before we can merge that PR; is it possible for @ruuddw @abrodkin to give a +1 today? If not, I would move this to 2.1.0 release, is this OK with you?

+1 from @ruudw @abrodkin may be a litte difficult as it's weekend now.
My two commits are not only for this PR,  but also a generci fix， so i still hope this PR can be merged for 2.0.
Now there is a still failure related to OPENOCD support.  the correct fix is #18125 .
I can make a workaround to let PR pass CI, leave #18125 to trace this issue

@vonhust your last commits are bug fix patches and can be merged for 2.0 during stabilization period, no problem. But the board support is not, so it needs to go in now or wait for 2.1. Now, I can't really review the fault.S patch myself, need someone with Arc knowledge


Docs on builtin service load balancer; NodeAffinity for host ports feature request

Would be nice to have more docs on the builtin LB and traefik.
Is it possible right now to "pinpoint" a LB for a service to a specific node?
In my particular case I have a 3 nodes k3s cluster, 1 master and 2 workers.
I have 2 different DNS services, which I want to expose to the world: first one listening on 53 port of the master node, and second one on 53 port of both worker nodes.

Yeah, svc lb needs explains in much greater detail. Current docs basically hand wave the feature.

For reference, is what we currently say about the default service load balancer:

K3s includes a basic service load balancer that uses available host ports. If you try to create a load balancer that listens on port 80, for example, it will try to find a free host in the cluster for port 80. If no port is available, the load balancer will stay in Pending.
To disable the embedded load balancer, run the server with the --disable servicelb option. This is necessary if you wish to run a different load balancer, such as MetalLB.

@ShylajaDevadiga Do you have any information on the configuration options for this load balancer?


Can't override default prefixes in TurtleSerializer

Here's a minimal code:
import qualified Data.Map as Map
import Data.RDF (RDF)
import qualified Data.RDF as RDF
import qualified Data.RDFimport qualified Data.RDF.Namespace as RDF
import Text.RDF.RDF4H.TurtleSerializer

main :: IO ()
main = do
  let emptyRdf = RDF.empty :: RDF RDF.TList
  let mappings = RDF.PrefixMappings $ Map.fromList [ ("mo", "http://purl.org/ontology/mo/") ]
  let graph1 = RDF.addPrefixMappings emptyRdf mappings True
  RDF.writeRdf (TurtleSerializer Nothing mappings) graph1

I'm expecting to only have my defined mo prefix. However, the following default prefixes are added in my output:
@prefix dc: <http://purl.org/dc/elements/1.1/> .
@prefix dct: <http://purl.org/dc/terms/> .
@prefix ex: <http://www.example.org/> .
@prefix ex2: <http://www2.example.org/> .
@prefix foaf: <http://xmlns.com/foaf/0.1/> .
@prefix mo: <http://purl.org/ontology/mo/> .
@prefix owl: <http://www.w3.org/2002/07/owl#> .
@prefix rdf: <http://www.w3.org/1999/02/22-rdf-syntax-ns#> .
@prefix rdfs: <http://www.w3.org/2000/01/rdf-schema#> .
@prefix skos: <http://www.w3.org/2004/02/skos/core#> .
@prefix xsd: <http://www.w3.org/2001/XMLSchema#> .

Thanks for reporting @koslambrou . Fixed in https://github.com/robstewart57/rdf4h/commit/9658c9c361fb175f614a127c96da8ec42665334a


Fix: Typography mixin $base argument

Add support to the $base argument so that it works regardless if the $type argument is "title" or "text". Right now it does not work if the $type is title.

Related issues

https://github.com/patternfly/patternfly-elements/issues/1384

Preview

Link(s) to demo page(s) where this element can be viewed:

Link

What has changed and why

Modified pfe-typography mixin to universally apply $base argument to title & text types

Testing instructions
Nothing should regress, because the PFE project is not setting base to false (its more useful for other projects like webrh). You can verify a few pages like pfe-band - they should look the same
If you are using the pfe-typography function, you should be able to opt out of the extra properties (beyond font-family, font-size) if desired, via the $base argument set to false:
Test by throwing this sass into some .scss file in pfe-styles:
.foo {
  @extend %pfe-title--lg;
}
.bar {
 @include pfe-typography(lg, $type: "title");
}
.baz {
 @include pfe-typography(lg, $type: "title", $base: false);
}

the expected output should return:
.foo {
  font-family: "Red Hat Display", "RedHatDisplay", "Overpass", Overpass, Arial, sans-serif;
  font-family: var(--pfe-theme--font-family--heading, "Red Hat Display", "RedHatDisplay", "Overpass", Overpass, Arial, sans-serif);
  font-size: 18px;
  font-size: var(--pf-c-title--m-lg--FontSize, var(--pf-global--FontSize--lg, 18px));
  line-height: 1.5;
  line-height: var(--pfe-theme--line-height, 1.5);
  font-weight: 400;
  font-weight: var(--pfe-theme--font-weight--normal, 400);
}

.foo:not(:last-child) {
  margin-bottom: 16px;
  margin-bottom: var(--pfe-theme--content-spacer--heading--sm, 16px);
}

.bar {
  font-family: "Red Hat Display", "RedHatDisplay", "Overpass", Overpass, Arial, sans-serif;
  font-family: var(--pfe-theme--font-family--heading, "Red Hat Display", "RedHatDisplay", "Overpass", Overpass, Arial, sans-serif);
  font-size: 18px;
  font-size: var(--pf-c-title--m-lg--FontSize, var(--pf-global--FontSize--lg, 18px));
  line-height: 1.5;
  line-height: var(--pfe-theme--line-height, 1.5);
  font-weight: 400;
  font-weight: var(--pfe-theme--font-weight--normal, 400);
}

.bar:not(:last-child) {
  margin-bottom: 16px;
  margin-bottom: var(--pfe-theme--content-spacer--heading--sm, 16px);
}

.baz {
  font-family: "Red Hat Display", "RedHatDisplay", "Overpass", Overpass, Arial, sans-serif;
  font-family: var(--pfe-theme--font-family--heading, "Red Hat Display", "RedHatDisplay", "Overpass", Overpass, Arial, sans-serif);
  font-size: 18px;
  font-size: var(--pf-c-title--m-lg--FontSize, var(--pf-global--FontSize--lg, 18px));
  line-height: 1.5;  /* This shouldn't be here */
  line-height: var(--pfe-theme--line-height, 1.5); /* This shouldn't be here */
  font-weight: 400; /* This shouldn't be here */
  font-weight: var(--pfe-theme--font-weight--normal, 400); /* This shouldn't be here */
}

.baz:not(:last-child) {
  margin-bottom: 16px; /* This shouldn't be here */
  margin-bottom: var(--pfe-theme--content-spacer--heading--sm, 16px); /* This shouldn't be here */
}

Full list of things you can throw into the mixin
http://pastebin.test.redhat.com/947129

Ready-for-merge Checklist

[x] Expected files: all files in this pull request are related to one request or issue (no stragglers or scope-creep).
n/a Tests have been updated to cover these changes.
n/a Browser testing passed.
[ ] Repository compiles and tests pass.
[x] Changelog updated.
[x] Documentation (README.md, WHY.md, etc.) updated or added:  https://github.com/patternfly/patternfly-elements/pull/1395
n/a Approved by designer.

Merging
Please squash when merging and ensure your commit message uses conventional commit formatting.
Be sure to share your updates with the<EMAIL_ADDRESS>mailing list!

Note: the $base: false argument is still printing margins

@heyMP this has been updated; you could copy/paste the contents of _copy-mixins.scss from here into the file in your branch to test w/ your components


librtmp: spec refactoring

librtmp provides API and useful functions for streaming multimedia content which using real-time messaging protocols. I prepared three packages: librtmp (libraries), librtmp-devel (headers) and rtmpdump (toolkit for RTMP streams). I hope that it works.

Thanks!


Feature request: include web3-react in default template

Requested by Prodoggy on Reddit.

Closing in favour of #28.


This adds a new recipe

A cheap, easy, and quick salad to make.

@DrewSansom  Could you fix the word "vinaigrette", please?

@igorsteinmacher good catch! Fixed.


Add support for CoinSpot exchange

It would be great to have support for CoinSpot https://www.coinspot.com.au/
Specification: https://www.coinspot.com.au/api

I started work on that a while ago but didn’t end up needing that so didn’t finish it. Might be good start for someone else..
https://github.com/nielsdraaisma/XChange/tree/develop/xchange-coinspot


Add the Minecraft Earth protocol support

Minecraft Earth (genoa) is based on v361 (from what we can tell) with protocol number 5000425. It has a decent amount of new packets and the odd other new thing but is very similar to bedrock since it's based on the same codebase.
This is WIP as part of Project Earth and will be marked ready when we are happy with it and feel like it has all the parts needed.
https://projectearth.dev/

I think it's safe to say this PR will never be finished so I will be closing it. 😞


docs(sphinx): Fix a few autodoc vertical spacing issues

@jmvrbanac Thanks!

Not sure what's up with codecov... I'm just going to merge this.


[TEST] Fix the Jenkins space issue

What's the test to develop? Please describe
We are running multiple jobs on Jenkins and always run issue of space available.
Describe the items of the test development (DoD, definition of done) you'd like

[ ] Consider increasing the disk size.
[ ] Build a cron job to clean up the space regularly.


Issue 45122: Default filesystem couldn't parse S3 based URI

Rationale
Issue 45122: Default filesystem couldn't parse S3 based URI
Changes

Use the PageFlowUtil to parse the path

Looks like this allows the s3CloudWatcher.listReloadTest to pass: https://teamcity.labkey.org/buildConfiguration/LabKey_223Release_Premium_ModulesSuites_CloudSqlserver/1760843?showRootCauses=false&expandBuildChangesSection=false


Left tab cut off

Hello,
It seems like the left most tab isn't displayed correctly, I've reinstalled and such, and it seems to be a persistent issue. Adding a spacer shows that that part of the UI is "loading" but it just is cut off somehow

Fresh profile in Firefox beta, base, and nightly, still all doing the same thing for some weird reason

I had this problem aswell. I reinstalled MaterialFox and then the problem
was fixed.
On Sun, Feb 3, 2019 at 2:39 PM Kastilo<EMAIL_ADDRESS>wrote:

Fresh profile in Firefox beta, base, and nightly, still all doing the same
thing for some weird reason
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/muckSponge/MaterialFox/issues/60#issuecomment-460052453,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AFtquizt-xCdkjBDFdiWfzv3Bp3YTFl1ks5vJuZzgaJpZM4aUh4W
.

Same issue on MacOS 10.14.2 with Firefox Developer Edition 66.0b5

Fixed it by changing line 85 on tabbar.css from
#tabbrowser-tabs:not([overflow]) .arrowscrollbox-scrollbox { 	margin: 0 -8px !important; }
to
#tabbrowser-tabs:not([overflow]) .arrowscrollbox-scrollbox { 	margin: 0 8px !important; }
Result

Should hopefully be fixed now. While I was able to reproduce on macOS eventually, I couldn't get it to occur on Win10, so I've been unable to test that.

Should hopefully be fixed now. While I was able to reproduce on macOS eventually, I couldn't get it to occur on Win10, so I've been unable to test that.

@muckSponge confirming that is fixed on MacOS. Thank you

Closing this due to inactivity so I assume it is fixed for Windows users too. If not, let me know and I'll reopen.


CRD validation for federated type primitives

A goal in transitioning to CRDs (depends on #74) is to maintain the same quality of validation for the resources embedded in templates as is currently provided by first-class API types.  Potential strategies:

Validate via decoding of object from JSON
Validate via OpenAPI spec:

For a given template resource (e.g. FederatedSecret), find the FederatedTypeConfig that specifies the template type.
For the target type of the FederatedTypeConfig, retrieve the OpenAPI spec.
Validate the target resource embedded in the template with the OpenAPI spec.
Note that the OpenAPI spec generated may lack the ability to validate string values as part of an enumerated string type e.g. imagePullPolicy being either Always, Never, or IfNotPresent. This may require updating the OpenAPI generation utility.

This shouldn't block release of v1alpha1.

Looks like dry-run has been implemented server-side in kube 1.12, and that would seem to offer higher fidelity validation than could be achieved with openapi.  The only catch is that a generic webhook validating an arbitrary type would need to have permission/quota/etc to actually create the target resource.

Each crd generated for a federated type with kubefed2 enable includes a validation schema (sourced from openapi or crd schema).  Dry-run validation is still suggested to provide more exact validation, and as mentioned perviously a generic webhook is probably the best way to accomplish this.

See https://github.com/kubernetes-sigs/kubefed/pull/1063#discussion_r308701486 for additional validations that would be useful.

/remove-lifecycle stale


podcast:id needs URL

Service slugs historically in open source are exclusionist.
Here is some context
https://wordpress.org/support/article/embeds/
It is really hard for a video or podcast service with a cool player to get on that list.
For oEmbed to work, you have to either get a blog owner to reduce security or install a custom plugin.
Even embedly (now owned by Medium) hasn't managed to get on that list
The open-source project becomes a gatekeeper for what is acceptable use
You can have a list of common slugs, but anyone should be able to just use a new one for their service
You need to have a URL
In that way even if the gatekeepers of the common list don't include you, it can still work.
Thus podcast:id needs to follow the same format as podcast:funding & podcast:social
It could be argued that if you have the URL, then the service slug is actually redundant and adds maintenance overhead
Exploding a URL for a service name isn't rocket science.
podcast:funding you are going to end up with people defining 100 different options... just wait. Affiliate links to everything they can think of, links to their sales page for various products etc.
But open standards are better than exclusion

@andybeard I see what you mean and I agree that we shouldn't be excluding any services that may not even exist today. I think we should avoid defining the universe of "services" and corresponding IDs and stick with URLs whenever possible.
In Phase 1:
I can see the change for funding. As you said, with the URL, the service slug is additional overhead that adds a trivial amount of functionality.
<podcast:funding title="[user provided note (string)]">[url for the show at the platform]</podcast:funding>
The platform one is harder. @daveajones currently Buzzsprout integrates with 17 different directories -- and we add more all the time. Is the proposal that each of these would be provided in the format below?
<podcast:id platform="[service slug]">[the id string]</podcast:id>

Agreed. Will change the id tag to take a url also.
The slugs file here is very open. Anyone can submit a PR to add to it. As long as what they submit isn’t pure craziness it’ll get merged.  What we see in the wild will surely differ anyway.  I’ll probably even automate something on our side to show us what’s in use and add periodically.

@andybeard - with regard to <podcast:id platform="[service slug]">[the id string]</podcast:id>
I think keeping the slugs as open as possible for people to add their own is as sensible as we can be. There are good technical reasons why the Wordpress example you give is a little more difficult to get into - embedding of other peoples' code in web pages is a horrid privacy nightmare, and that's - thankfully - nothing we have to worry about here.
I think it might be helpful for the list of acceptable slugs to also have URL expansion rules and marketing names (and, in time, icons) to allow for automated links to them. That would make it even easier for them to be used.
So, I guess I'm expecting a (?) JSON (?) file, showing the slug of apple-podcasts, the URL expansion of https://podcasts.apple.com/us/podcast/feed/id%%id%%, a marketing name of Apple Podcasts, and ideally an SVG file, to aid directories to link. Or even the current breed of "hear a random podcast" apps - none of which are trying to be a full podcast app, and where a link to your full podcast app would help considerably.
This world is full of edge cases, however, and as one example, Apple Podcasts lists my podcast at https://podcasts.apple.com/us/podcast/feed/id1325018583 but for me, a better URL would be https://podcasts.apple.com/au/podcast/feed/id1325018583 since I live in Australia, and this gives me better alternative shows to listen to.
URLs also change - Google Podcasts is only two years old, but has changed its URL structure entirely already. So I'd be keen to consider avoiding a full URL, if only because it adds complication to the podcast hosts to keep all these things updated.

URL changes can happen easily at a number of touchpoints

The podcast host can build a URL based on a slug at their end
A site such as a directory can add logic to modify URLs if needed
The Player host such as Google Podcasts can implement redirects

By keeping it open, I can expect there to even be a service need for there to be blacklists for things like NSFW, Age, Religion, Politically incorrect, etc.
I don't want to go into the ramifications of 2nd and 3rd tier linking and bad neighbourhoods here.
You are defining a standard that will be used by political regimes you might not agree with, hardcore p___, cults, sects and many will have their own players, because western platforms would ban them.
But that is the nature of open standards

I get your drift.  And, perhaps those things will happen.  This ultimately comes down to writing a spec and then seeing what people do with it in the real world.  Then changing the spec to match what's going on in practice.  I'm committed to maintaining this spec in an open, flexible way.  We'll see what happens.


Feature Request: Auto header level

When you right-click on the block to set the header level, there is an option to set it to "auto". This is useful for moving around blocks as they will automatically maintain the proper heading level regardless of where they're located.
Screenshot:

Yeah! That would be great!

I found a nice workaround:

To do this:

Create a command in config.edn, e.g.

 :commands
 [
 ;; I use orgmode; you can adapt it to markdown as well
 ["heading-orgmode" ":PROPERTIES:\n:heading: true\n:END:"]
 ]

Restart logseq
Type your new, e.g. "test", heading
Hit shift + enter for a line break
Use your command: /heading-orgmode (or whatever you called it in step 1)

you can use this plugin: logseq-plugin-smart-typing to automatically add heading:: true property.
my setting is as follows.

Thus, when you enter @h at the end of a block, it will add this property automatically.
Note that there is a Shift+Enter before heading:: true.


Question about why requests for an MSNBC extractor have been ignored for more than a year

My extractor request posted yesterday (8/12/15) for MSNBC.com, one of the largest news sites on the internet was closed without comment, because it was a duplicate of an earlier request.
I see now that there is an open request https://github.com/rg3/youtube-dl/issues/2371 from February, 2014, a year and a half ago that received no replies or action.
YouTube-dl supports NBC.com, and NBC shares resources with MSNBC as they are the same company, but the URLs NBC uses are mostly MSNBC.com URLs.
I don't understand why these requests have been completely ignored while much smaller sites receive extractors.
Don't get me wrong. I am grateful for the outstanding work done by the YouTube-dl volunteer coders, but I'd think an important site like NBC News deserves at least a post describing why it won't work with YouTube-dl.
Thanks for all the work you guys do.

Your issue has been closed as a duplicate of another since we do the same for every duplicate we find to keep open issues as distinct as possible.
Lack of support for particular extractor may be due to lots of reasons. Generally, no responses in extractor support request issue means that no one of developers/contributors have interest/time implementing it. As you already mentioned, as volunteers we are working on things and features we prefer most.


⚠ Bofrost.nl has degraded performance

In d4bb507, Bofrost.nl (https://www.bofrost.nl) experienced degraded performance:

HTTP code: 200
Response time: 1429 ms

Resolved: Bofrost.nl performance has improved in 3c010fa after 6 minutes.


Tab autocompletion doesn't seem to be working in iTerm 2/Terminal

Describe the bug
I'm trying the text suggestions in 0.3.0-alpha.2 in a terminal on macOS; it tells me I can use "tab to auto-complete", but it doesn't seem to work.
Reproduction project
This is my Cargo.toml:
[package]
name = "example"
version = "0.1.0"
edition = "2021"

[dependencies]
inquire = "0.3.0-alpha.2"

and this is my main.rs, which is a simplified version of examples/text_options.rs:
use inquire::{error::CustomUserError, Text};

fn suggester(val: &str) -> Result<Vec<String>, CustomUserError> {
    let suggestions = ["andrew", "charles", "christopher"];

    Ok(suggestions
        .iter()
        .filter(|s| s.contains(&val))
        .map(|s| String::from(*s))
        .collect())
}

fn main() {
    let answer = Text::new("What's your name?")
        .with_suggester(&suggester)
        .prompt()
        .unwrap();

    println!("Hello {}", answer);
}

Expected/actual behaviour
If I run the code, I get a prompt with three suggestions:

If I then type cha for charles, my selection reduces to a single item:

At this point, I expect that pressing tab will complete to charles, but as far as I can tell nothing happens.
Desktop

OS: macOS Catalina 10.15.7
Terminal: iTerm 2 3.4.16 and Terminal 2.10; I get the same behaviour in both
Version: 0.3.0-alpha.2

Additional context
I can reproduce this error if I swap out the back-ends, in particular either of the following lines in my Cargo.toml have the same behaviour:
inquire = { version = "0.3.0-alpha.2", features = ["console"] }

inquire = { version = "0.3.0-alpha.2", features = ["termion"] }

I get similar behaviour in 0.2.1.
Cargo.toml:
[package]
name = "example"
version = "0.1.0"
edition = "2021"

[dependencies]
inquire = "0.2.1"

main.rs:
use inquire::Text;

fn suggester(val: &str) -> Vec<String> {
    let suggestions = ["andrew", "charles", "christopher"];

    suggestions
        .iter()
        .filter(|s| s.contains(&val))
        .map(|s| String::from(*s))
        .collect()
}

fn main() {
    let answer = Text::new("What's your name?")
        .with_suggester(&suggester)
        .prompt()
        .unwrap();

    println!("Hello {}", answer);
}

I was trying the alpha version because I want to play with the shiny autocomplete stuff! :D

I've done a bit of digging, and it's not clear to me if this was ever wired up:

In 0.2.1, the Input.handle_key() method doesn't seem to support tabs at all: https://github.com/mikaelmello/inquire/blob/v0.2.1/src/input.rs#L70-L89
In 0.3.0-alpha.2, the handle_tab_key() method only looks at the completer, not the suggester: https://github.com/mikaelmello/inquire/blob/v0.3.0-alpha.2/src/prompts/text.rs#L339-L351

I think that gives me a way forward for what I'm trying, but I wanted to highlight that the current behaviour was quite non-obvious to me.

hey so I realize the behavior is very unintuitive and why it looks like there's a bug or missing implementation.
What tab does is: once you have a highlighted suggestion, you can press tab to update your current text input to the value in that suggestion. The worst of this approach is that there is no visual indication that tab did anything, apart from other suggestions that might vanish.
e.g. You type "an" and it suggests "ANdrew" and "dANiel". You highlight Andrew and press tab. Since your input text is now Andre the Daniel suggestions vanishes.
With your attention to this, I am thinking of the following change to pressing tab:

If there are no suggestions, nothing happens.
If there are suggestions but none of them are highlighted (the user didnt navigate with arrow keys), the text input is replaced by the text of the first suggestion.
If there are suggestions and one of them is highlighted, nothing happens.

Sounds reasonable to you?

Little update, I had completely forgotten about the existence of the Completer attribute.
Yeah so Completer and Suggester seem to be solutions that don't work well together. Besides what I said, the Completer is meant to be a solution for the kind of blind tab auto-completion we see when traversing paths for example.

I'll build a better solution tomorrow that has clear defined behaviors for each autocompletion method. Will keep you posted :)

The proposed behaviour sounds reasonable, yeah – I was thinking tab would only autocomplete if there was an unambiguous suggestion, but the first suggestion sounds good too.
I look forward to it, thanks!
For what it’s worth, I was able to build approximately the functionality I want with 0.3.0-alpha.2, in a small prototype here: https://github.com/alexwlchan/inquire_tag_autocomplete

I really liked the way you went about it! I opened a new PR with a kind of revamped approach to autocompletion, trying to unify both use-cases, let me know what you think before I commit to it: #69

I opened a new PR with a kind of revamped approach to autocompletion, trying to unify both use-cases, let me know what you think before I commit to it: https://github.com/mikaelmello/inquire/pull/69

Ooh, looks interesting! I don't have a lot of time for code review right now, but I’ve stuck it on my todo list and I’ll get to it when I can.

Turning this into a discussion as it isn't quite a bug!

Actually, I'll close the issue and we can either create a discussion or continue on the PR.


add validation for list length

It would be great to get validation on list lengths.
eg.
ListField(Field(...), max_length=10)

Please see #27


oer.model.impl.AbstractModel.hashCode() violates contract

How to reproduce
Run the attached test.
Expected behaviour
Test passes with equal hash codes.
Actual behaviour
Test fails with different hash codes, violating the general Object.equals()/hashCode() contract, which requires that x.equals(y) => x.hashCode() == y.hashCode()
Notes

the issue lies with the usage if the Models.isomorphic() method in the AbstractModel.equals() method, which causes bnode ids to be ignored in equality tests while they are being considered in hash code computation
in a real-world context, the issue is not systematic, as it depends on specific combinations of randomly generated bnode ids
the issue is probably not critical, as models are unlikely to be used as map keys

AbstractModelTest.java.zip

Yeah that's not right. We should not be using the isomorphic comparison as part of the object equality check. Good find.

Btw. If we stop using isomorphic as part of the model equality code we could break existing code. I recommend we release this as part of a minor release rather than a patch release.

I agree. I think we can treat this as a bug and therefore not backwards-incompatible (so it doesn't require a major release) but it's been wrong sufficiently long that I think fixing this as a patch is going to catch people out. We'll need to add something relevant to the release notes as well.

The simplest fix for this is to just remove the use of Models.isomorphic from the equals method (and in fact just not overriding the equals method at all and just relying on the superclass equals method). With the test case @knoan provided above, that will result in the equals method no longer returning true for the two models with different bnodes.


Troubleshooting helm or docker authorization

This is worth writing up as it is a nasty gotcha.
If you see an error like that at the bottom if this issue then it could be because helm has cached a personal access token that has expired.
When you do authenticate to the GHCR using docker login or helm registry login you would usually supply a Personal Access Token. At least in the case of helm this will then fail even for public packages when that PAT expires.
In the case of helm you can see the entry in `$HOME/.config/helm/registry.json' - deleting it will fix the issue (for public registries - if you do want to access a private then you will need to make a new PAT and login with it again)
anecdotally docker suffers from the same issue - need to look into where docker caches credentails.
hgv27681@pc0116 work/bl47p (main) » ec deploy bl47p-ea-dcam-02 2024.4.1

Command Failed:
helm dependency update /tmp/tmp8xcw7ge0/services/bl47p-ea-dcam-02/../../helm/shared
Hang tight while we grab the latest from your chart repositories...
...Successfully got an update from the "gitlab" chart repository
Update Complete. ⎈Happy Helming!⎈
Saving 1 charts
Downloading ioc-instance from repo oci://ghcr.io/epics-containers
Save error occurred:  could not download oci://ghcr.io/epics-containers/ioc-instance: failed to authorize: failed to fetch oauth token: unexpected status from GET request to https://ghcr.io/token?scope=repository%3Aepics-containers%2Fioc-instance%3Apull&service=ghcr.io: 403 Forbidden

Error: could not download oci://ghcr.io/epics-containers/ioc-instance: failed to authorize: failed to fetch oauth token: unexpected status from GET request to https://ghcr.io/token?scope=repository%3Aepics-containers%2Fioc-instance%3Apull&service=ghcr.io: 403 Forbidden

Not seeing this issue any more - particularly with Argo CD that only uses helm for templating but also with raw helm deployments.


Winecfg window is buggy when virtual desktop is turned on.

As title says :1234:

Needs clear reproduction steps and better description.


[Bug]: Outline around active tab

What happened?
After the 130 update, there is now a white outline around the active tab.

Steps to reproduce

Open Firefox (updated to 130)
Observe outline on active tab.

Additional info (Optional)
Firefox version: 130
Operating System: Windows 11

Did you use the latest Material Fox version? I can't reproduce this issue because it was already fixed in #62

That's likely one of your normal firefox themes causing that.  Had a similar problem so I decided to just use custom.css to theme rather than using userChrome.theme-default with random themes.

Did you use the latest Material Fox version? I can't reproduce this issue because it was already fixed in #62

You're totally right! Sorry, I must have copied my old chrome folder when I thought I was "pasting" my new one. Issue is good to close.


HHH-11257 - Connection leak when SQLGrammarException is thrown

https://hibernate.atlassian.net/browse/HHH-11257

Superseded by this Pull Request.


Changes for sharded proxies

Sharded proxies means >1 proxy running at the same time pinned to the same database, sticky based on the user ID (to reuse user caches). This needs several changes:

On startup, the proxy populates its GlobalCache using bulk SELECT queries. These queries need to be done in a DB-wide lock, otherwise joins/leaves in the startup process received by other instances of the proxy will be missing in the proxy starting up. Specifically this affects:

AllJoinedMembers: specifically the call s.accumulator.snapshotTable.CurrentSnapshots(), selecting events can be done without the lock.
MetadataForAllRooms: this does bulk select/join ops across the entire DB to pull out RoomMetadata. It all needs to be in a DB lock realistically.

We need to make each proxy be responsible for a subset of users. This means we should only service requests from those users, and only start the v2 pollers for those users. This needs changes to:

AllDevices: this needs to be scoped to whatever chunking algorithm we use to ensure we only pull out the right devices. Needs no additional locks so long as we guarantee that we get all devices for a given user (we should do so we can maximise the usage of the UserCache).
We need whatever load balancer we use to know how to route traffic. This is Hard as it cannot be stateless. The only distinguishing data we know about is the access_token which there is no guarantee that it bakes in the user ID. We need to hit /whoami with this token to know where to route to, then keep a cache of that.

nginx.ingress.kubernetes.io/upstream-hash-by is the magic k8s haproxy ingress annotation to use to route based on an http header.

The proxy needs to be split into two to support scaling in order to satisfy:

AllDevices: this needs to be scoped to whatever chunking algorithm we use to ensure we only pull out the right devices.

As we cannot mimic the chunking algorithm atomically as it relies on knowing which shards exist (it's a hash ring).
As a result, we want to end up with an architecture like this:
New events are inserted into the DB and they a pg_notify informs the v3 API.
  +--------+              .===.               +-----------+
  | V3 API | ---LISTEN-> | DB | <---NOTIFY--- | V2 Poller |
  +--------+              `---`  new events   +-----------+

When new /sync requests are made, we check the DB to see if there is a poller for this device. If not,
it is signalled to the v2 poller:
  +--------+              .===.               +-----------+
  | V3 API | ---NOTIFY-> | DB | <---LISTEN--- | V2 Poller |
  +--------+  new user    `---`               +-----------+

Scale down events will move a user to a new proxy which is unaware of the connection and ?pos= value, resulting in an HTTP 400 connection expired error. We need to make sure that clients transparently reconnect when this happens, else scale down events will be noticeable to clients.


add traffic direction to http connection manager stat prefix

Currently Http connection manager's stat prefix is set to the listener name which is of the form _. From this it is difficult to identify whether this is an inbound or outbound connection manager. This PR appends traffic direction to the stat prefix.
And to help us figure out who should review this PR, please
put an X in all the areas that this PR affects.
[ ] Configuration Infrastructure
[ ] Docs
[ ] Installation
[X ] Networking
[ ] Performance and Scalability
[ ] Policies and Telemetry
[ ] Security
[ ] Test and Release
[ ] User Experience
[ ] Developer Infrastructure

@ramaraochavali: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

integ-new-install-k8s-tests_istio
c3d0561053a1552bff485dad83ece0af30ed73f9
link
/test integ-new-install-k8s-tests

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

/test integ-new-install-k8s-tests

@ramaraochavali: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

integ-new-install-k8s-tests_istio
c3d0561053a1552bff485dad83ece0af30ed73f9
link
/test integ-new-install-k8s-tests

integ-mixer-k8s-tests_istio
c3d0561053a1552bff485dad83ece0af30ed73f9
link
/test integ-mixer-k8s-tests

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

@ramaraochavali: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

integ-mixer-k8s-tests_istio
c3d0561053a1552bff485dad83ece0af30ed73f9
link
/test integ-mixer-k8s-tests

integ-new-install-k8s-tests_istio
c3d0561053a1552bff485dad83ece0af30ed73f9
link
/test integ-new-install-k8s-tests

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

Yes it would be set to http connection manager filter inside a listener - which would be selected by filter chain match.

Ask Doug Reid

@douglas-reid @mandarjog Can you please check if this is stat prefix change would have any backward compatible issues?

My only concern would be for anyone that is using direct Envoy stats in their monitoring. We aren't using those stats in our example dashboards, etc., but others may. This feels like something that should be called out in the release notes.


Failed to integrate and compile

In file included from /root/work/include/Eigen/Core:297:0,
                 from /root/work/include/Eigen/Dense:1,
                 from /root/work/include/Eigen/Eigen:1,
                 from /root/work/include/omni/frontalizer_cpu.h:4,
                 from /root/work/src/main.cpp:28:
/root/work/include/Eigen/src/Core/DenseBase.h: In instantiation of ‘void Eigen::DenseBase<Derived>::swap(const Eigen::DenseBase<OtherDerived>&) [with OtherDerived = Eigen::Matrix<double, -1, -1>; Derived = Eigen::Matrix<double, -1, -1>]’:
/root/work/include/Eigen/src/Core/PlainObjectBase.h:852:33:   required from ‘void Eigen::PlainObjectBase<Derived>::swap(const Eigen::DenseBase<OtherDerived>&) [with OtherDerived = Eigen::Matrix<double, -1, -1>; Derived = Eigen::Matrix<double, -1, -1>]’
/root/work/include/tapkee/defines.hpp:52:20:   required from here
/root/work/include/Eigen/src/Core/util/StaticAssert.h:32:40: error: static assertion failed: THIS_EXPRESSION_IS_NOT_A_LVALUE__IT_IS_READ_ONLY
     #define EIGEN_STATIC_ASSERT(X,MSG) static_assert(X,#MSG);
                                        ^
/root/work/include/Eigen/src/Core/DenseBase.h:427:7: note: in expansion of macro ‘EIGEN_STATIC_ASSERT’
       EIGEN_STATIC_ASSERT(!OtherDerived::IsPlainObjectBase,THIS_EXPRESSION_IS_NOT_A_LVALUE__IT_IS_READ_ONLY);
       ^
make[2]: *** [src/CMakeFiles/omni.dir/main.cpp.o] Error 1
make[1]: *** [src/CMakeFiles/omni.dir/all] Error 2
make: *** [all] Error 2

@cysin this looks like some discrepancy with eigen, what is the version you use?

The error is of 3.3 alpha1 version. I also tried 3.25 and 3.26 it also reports errors but with different error info. My project has already been using Eigen. I added the following lines to my code:
#define TAPKEE_EIGEN_INCLUDE_FILE <Eigen/Eigen>
#include <tapkee/tapkee.hpp>

I am on CentOS 7 with tapkee code checked out from trunk

@cysin could you please try to replace the following line in tapkee/defines.hpp:52
        embedding.swap(e);

to
        embedding = e;

It seems the api of swap has been slightly changed at some point.

I did the modification to both line 52 and line 57. And now it can be compiled. Thanks a lot!

@cysin ok cool! I'll try to reproduce it locally and commit the fix asap

Maybe we should fix the version of Eigen3 we are depending on. In other words, say that we know that Tapkee is stable with Eigen 3.2.x, with x <= 4. then we fire a CMake error if we are trying to configure the build with Eigen 3.2.y with y > 5.
What do you think, @lisitsyn?

@iglesias yeah, but some ifdef should suffice. Swap is here to avoid copying, not that major thing


Ability to bundle metadata and templates inside the SQLite file

One of the nicest qualities of SQLite as a data format is that you get a single file which you can then backup or share with other people.
Datasette breaks this a little once you start including custom metadata.json or template files and CSS.
It would be cool if there was an optional mechanism for baking that extra configuration into the SQLite file itself. That way entire datasette mini-applications (including canned queries and custom HTML and CSS) could be constructed as single .db files.
Since datasette configuration is all file-based, one way to achieve that would be to support a "datasette_files" table which, if present is used to search for file contents by path.
This is inline with the philosophy described by https://www.sqlite.org/appfileformat.html

One point of complexity: datasette can be used to bundle multiple .db files into a single "app".
I think that's OK. We could require that the datasette_config table is present in the first database file passed on the command-line. Or we could even construct a search path and consult multiple versions of the table spread across multiple files.
That said... any configuration that corresponds to a specific table should live in the same database file as that table.
Ditto for general metadata: if we have license/source information for a specific table or database that information should be able to live in the same .db file as the data.

This would be a great feature to have!

Potential problem: the existing metadata.json format looks like this:
{
    "title": "Custom title for your index page",
    "description": "Some description text can go here",
    "license": "ODbL",
    "license_url": "https://opendatacommons.org/licenses/odbl/",
    "databases": {
        "database1": {
            "source": "Alternative source",
            "source_url": "http://example.com/",
            "tables": {
                "example_table": {
                    "description_html": "Custom table description",
                    "license": "CC BY 3.0 US",
                    "license_url": "https://creativecommons.org/licenses/by/3.0/us/"
                }
            }
        }
    }
}

This doesn't make sense for metadata that is bundled with a specific database - there's no point in having the databases key, instead the content of database1 in the above example should be at the top level. This also means that if you rename the *.db file you won't have to edit its metadata at the same time.
Calling such an embedded file metadata.json when the shape is different could be confusing. Maybe call it database-metadata.json instead.

This is likely to be covered by plugin hooks: #860 for the metadata and after investigating in #1042 it looks like the existing prepare_jinja2_environment hook may already be enough to load templates from the database.


create-react-app

Description
frontend branch created - create-react-app ran
Fixes # (issue)
Type of change
Please delete options that are not relevant.

[x] New feature (non-breaking change which adds functionality)
[x] This change requires a documentation update

How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration

[ ] Test A
[ ] Test B

Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[ ] My code has been reviewed by at least one peer
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes

create-react-app initialization for frontend branch.


[4.2 EARLY] [Serialization] Track whether a cross-reference came from Clang

Explanation: Declaration cross-references in swiftmodules are disambiguated by the target declaration's containing module, with the assumption that two modules will never have the same name. However, an overlay has the same name as its underlying Clang module, which means that there can be two declarations with the same name, the same type, and the same module name. This is the underlying cause of the 'UIEdgeInsetsZero' problem, but it also affects the CloudKit overlay. By tracking a bit that just says "this came from Clang", we're able to resolve otherwise ambiguous cross-references.

Scope: Affects how all declaration cross-references are resolved during deserialization of a swiftmodule.

Issue: rdar://problem/40839486

Risk: Medium-low. This should only affect cases that were previously "ambiguous"; that is, where a declaration defined in Swift conflicted with a declaration imported from Clang. But it is a change that exists for all cross-reference resolution.

Testing: Added compiler regression tests, verified that the original project now builds successfully.

Reviewed by: @DougGregor, @rjmccall

@swift-ci Please test

@swift-ci Please nominate


Add logging options

It seems like notarizing can take quite a while, it would be nice to be able to pass in an options.logger to get a little bit of feedback as it completes some of the steps, to know it's not hung.

@ianstormtaylor as with most other electron ecosystem modules we use the debug module.  Setting DEBUG=electron-notarize* will spit out some helpful information every now and again


[BEAM-2143]: Fix default temp location for DataflowRunner

Be sure to do all of the following to help us incorporate your contribution
quickly and easily:

[ ] Make sure the PR title is formatted like:
[BEAM-<Jira issue #>] Description of pull request
[ ] Make sure tests pass via mvn clean verify.
[ ] Replace <Jira issue #> in the title with the actual Jira issue
number, if there is one.
[ ] If this contribution is large, please file an Apache
Individual Contributor License Agreement.

https://github.com/apache/beam/pull/2602 made gs://some-bucket path invalid and expects it to be of the form gs://some-bucket/basename. So adding /temp as the basename for the default tempLocation.
Also fix log messages.

R: @bjchambers @davorbonaci

Coverage increased (+0.002%) to 70.491% when pulling ea165bfba449cd5572f9342bf8107018dc584a30 on vikkyrk:df_temp_loc into b130d7aac466860ed5a7abec0bef33d9e0dd3c6d on apache:master.

Coverage decreased (-0.005%) to 70.484% when pulling e8421d8e62c5e292bab73f1265f74b7840e997aa on vikkyrk:df_temp_loc into 3bffe0e0014bdd6ae73dc2e8ecfc2b61d066120c on apache:master.

Coverage increased (+0.006%) to 70.495% when pulling e8421d8e62c5e292bab73f1265f74b7840e997aa on vikkyrk:df_temp_loc into 3bffe0e0014bdd6ae73dc2e8ecfc2b61d066120c on apache:master.

If tempLocation is not specified:
INFO: No tempLocation specified, attempting to use default bucket: dataflow-staging-us-central1-996210899482
If both tempLocation and stagingLocation is not specified
INFO: No tempLocation specified, attempting to use default bucket: dataflow-staging-us-central1-996210899482
INFO: No stagingLocation provided, falling back to gcpTempLocation

Done

Coverage increased (+0.01%) to 70.5% when pulling 02b50fdf234f19580fdc3d9373361d70fd5f10ad on vikkyrk:df_temp_loc into 3bffe0e0014bdd6ae73dc2e8ecfc2b61d066120c on apache:master.

The exception I get from this is still not particularly useful. It says:
...
Caused by: java.lang.IllegalArgumentException: Missing object or bucket in path: 'gs://dataflow-staging-us-central1-927334603519/', did you mean: 'gs://some-bucket/dataflow-staging-us-central1-927334603519'?
...

What command did you run? You shouldn't see an exception after this change for missing either of tempLocation or stagingLocation

Ah -- it looks like it wasn't using the patched version of the SDK. Looks good now.

@bjchambers @davorbonaci Looks good to be merged.


feat: fix pom.xml to fix CI for publishing to https://central.sonatype.com

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

GitHub release
v1.2.0

Your semantic-release bot :package::rocket:


remove font from library but added to docs

Coverage increased (+0.03%) to 48.632% when pulling 5dd5a28e498ab5c395fe4ffe5319ee5b9e268371 on muhshahabipour:master into 23d269016f2695fc9a56956f8af9507d8c7bfcf3 on mberneti:master.

@muhshahabipour thank you for contribution.
So after that every one should add this style:
.calendarContainer *, .datepicker-input, .calendarContainer .heading .title, .calendarContainer .daysOfWeek>div, .time-picker-container, .rc-time-picker-input, .rc-time-picker-panel-inner {     font-family: $fontFamily; }
I know loading default fonts is not good at all, I'm looking for a way to add new feature for adding new custom font or disabling default fonts loading dynamically.

So after that every one should add this style:
.calendarContainer *, .datepicker-input, .calendarContainer .heading .title, .calendarContainer .daysOfWeek>div, .time-picker-container, .rc-time-picker-input, .rc-time-picker-panel-inner { font-family: $fontFamily; }

okey. in next pull request add below code that users should use their default fonts.
users code:
body {
    // font-family: $EVERY_CUSTOM_FONT_NAME;
}

library code:
.calendarContainer *, .datepicker-input{
    font-family: inherit;
}

Great, I'll check it ASAP.

@all-contributors please add @dadwic for design and code


Full Screen exoplayer

Is full screen exoplayer coming?

Not yet, I haven't done much work on this project recently. Let me know if it's something of interest then I'll look into it over the weekend.

Full screen exoplayer is available now
https://github.com/rubenquadros/Jetpack-Compose-Video-Games-Example/pull/11


Plex added something like this

Just wanted to say that plex just added something like this but I personally will continue to use plex auto languages as this is a more streamlined approach imo. Thank you for making this tool!

Thank you for the kind words, I really appreciate it !
Are you talking about this line in the recent changelog:

(Subtitles) Attempt to automatically detect improperly-tagged forced streams based on their titles (#14096)

I think he means this

I think he means this

Yes this

I imagine it will be pretty good too but the problem I have with it is, is that it's not exactly per user. So as long as you don't change Seasons or TV shows themselves than it is. But if I go in and I force a series under edit and then advanced and I change the language, that is then forced upon all of my users. But as long as I don't do that then it is per user because they can go into their client settings and change the language there. But with Plex Auto languages it is truly per user regardless of if I change a subtitle or not. Now Plex may change all this in the future but managed accounts or non-admin accounts in general don't have the power to edit items inside of plex

Plex and Plex-Auto-Languages (or PAL for short) handle audio and subtitles in different ways. With Plex, you can set a default language, but it will automatically select the first available audio track in that language. PAL, on the other hand, allows you to choose the exact audio and subtitle tracks that you prefer.
Take Dragon Ball Z as an example. The show has an original Japanese audio track but also comes with three different styles of English dubs due to its international airings. With PAL, you can specifically select which style of English dub you want to hear. Once you've made your choice, PAL saves it. This gives you the audio you actually want, instead of Plex's more generalized approach of picking the first available English track.


Add 374 in c language

File(s) Modified: 374-guess-number-higher-or-lower.c
Language(s) Used: c
Submission URL: _https://leetcode.com/submissions/detail/831469827/_

Thanks, @julienChemillier!


LoAF Summary attribution information

Fixes #559
Includes object like this:
{
    "numScripts": 3,
    "slowestScript": {
        "name": "script",
        "entryType": "script",
        "startTime": 10458259,
        "duration": 692,
        "invoker": "LABEL.onclick",
        "invokerType": "event-listener",
        "windowAttribution": "self",
        "executionStart": 10458259,
        "forcedStyleAndLayoutDuration": 233,
        "pauseDuration": 0,
        "sourceURL": "",
        "sourceFunctionName": "",
        "sourceCharPosition": -1
    },
    "slowestScriptPhase": "processingDuration",
    "totalDurationsPerPhase": {
        "processingDuration": {
            "event-listener": 710
        }
    },
    "totalForcedStyleAndLayoutDuration": 251,
    "totalNonForcedStyleAndLayoutDuration": 40.29999999701977,
    "totalScriptDuration": 710
}

incidentally, it seems like we're leaning subparts, not phases for LCP, so we might want to do the same thing here (or object quickly on the LCP side of things :)

I'm cool with subparts so switched to that throughout.


Add "queue" support

Much like cron tasks, there can also be a "task master" type facility that leverages SQS and then just a single cron/hook that gets called on a set schedule (configured by up ... maybe every 5 minutes for example or even configure that in the YAML) and handled by the Go app code.
This is a little different than cron. While a scheduled job invokes the Lambda to process the queue in SQS, it still means that a certain number of messages will be read and processed at once. So there may be a few configuration options to consider here. At any rate, it's a queue that can even support retries unlike a cron task. Though it's still very helpful.

Going to implement scheduled tasks and allow a separate package to handle a "queue" or "task master" of the sorts. It will be highly subjective I think and before Aegis gets too opinionated...It's best to leave just a scheduled task facility in place and let the end user decide if there's SQS involved, etc.
It also leaves Aegis a bit more open to potentially running on other clouds in the future. Aegis is opinionated a bit still in that it looks to AWS Lambda + API Gateway, but the local dev mode kinda proves it can run in Docker anywhere...So there's potential to broaden in an elegant way.


Add runtyping as a related library

I recently wrote a little package to automate the creation of runtypes from JSON schema and static types. Wondered if you fancied adding it to your README.

Thanks! 🤩


Path problem including graphql files

Hello,
When I try to simply import a gql file, I always get the error :
error: bundling failed: Error: Cannot find module './myQuery.gql'
    at Function.Module._resolveFilename (internal/modules/cjs/loader.js:580:15)
    at Function.resolve (internal/modules/cjs/helpers.js:30:19)
    at PluginPass.exit (C:\Project\importGQL\node_modules\babel-plugin-import-graphql\build\index.js:72:66)
    at newFn (C:\Project\importGQL\node_modules\@babel\traverse\lib\visitors.js:193:21)
    at NodePath._call (C:\Project\importGQL\node_modules\@babel\traverse\lib\path\context.js:53:20)
    at NodePath.call (C:\Project\importGQL\node_modules\@babel\traverse\lib\path\context.js:40:17)
    at NodePath.visit (C:\Project\importGQL\node_modules\@babel\traverse\lib\path\context.js:97:8)
    at TraversalContext.visitQueue (C:\Project\importGQL\node_modules\@babel\traverse\lib\context.js:118:16)
    at TraversalContext.visitMultiple (C:\Project\importGQL\node_modules\@babel\traverse\lib\context.js:85:17)

My code :
import Query from "./myQuery.gql";

The only way to make it working is to use the absolute path to the graphql path. ex : C:\Project\importGQL\components\myQuery.gql
Reproduction
I reproduce it on a simple project : https://github.com/Kisepro/importGQLBug.git
Investigation
When I check in babel-plugin-import-graphql\build\index.js, it calls the function with the following parameters  :

importPath ./Query.gql

jsFilename C:\Project\importGQL\node_modules\react-native\scripts\components\ComponentA.js

The ComponentA.js file is obviously not in under node_modules

This is also a problem for me.

Were any of you who experienced this problem not using react-native? I've got too much going on to set up and mess with react-native, but if anyone can offer a reproduction of this problem that doesn't involve react-native I can take a look. Pull requests by anyone who uses react-native are also welcome. :)

After seeing this mentioned in #48 as well, I believe @stephencaldwell is correct. I'm closing this. If anyone experiences this issue with the settings mentioned above, let me know and I'll reopen.


[BugFix] Fix pk concurrent apply issue

Why I'm doing:
There's an issue with pk tables applying at the same time:

BE receive a increment clone request, and tablet1(tablet id is x) execute apply in thread1
BE receive drop tablet1 request and erase it from tablet_manager,  but drop is not finished because need to wait for apply finished.
BE receive a new clone request and create a new tablet2(tablet id is also x).
BE receive a new incremental clone request and tablet2 execute apply in thread2
tablet1 and tablet2 are doing apply at the same time. These two tablets are different tablet but has the same tablet id, so they will get the same primary index in memory and update at the same time. It will cause unexpected issue.

What I'm doing:

remove tablet from tablet_manager at the last step of dropping tablet.
reject clone request when we find tablet is under dropping but not finished.

Fixes #issue
What type of PR is this:

[x] BugFix
[ ] Feature
[ ] Enhancement
[ ] Refactor
[ ] UT
[ ] Doc
[ ] Tool

Does this PR entail a change in behavior?

[ ] Yes, this PR will result in a change in behavior.
[x] No, this PR will not result in a change in behavior.

If yes, please specify the type of change:

[ ] Interface/UI changes: syntax, type conversion, expression evaluation, display information
[ ] Parameter changes: default values, similar parameters but with different default values
[ ] Policy changes: use new policy to replace old one, functionality automatically enabled
[ ] Feature removed
[ ] Miscellaneous: upgrade & downgrade compatibility, etc.

Checklist:

[x] I have added test cases for my bug fix or my new feature
[ ] This pr needs user documentation (for new or modified features or behaviors)

[ ] I have added documentation for my new feature or new function

[ ] This is a backport pr

Bugfix cherry-pick branch check:

[x] I have checked the version labels which the pr will be auto-backported to the target branch

[x] 3.3
[x] 3.2
[x] 3.1
[ ] 3.0
[ ] 2.5

 These two tablets are different tablet but has the same tablet id
->
. These two tablet instances of one tablet  has the same tablet id


Drop down box small

The arrow drop down box on https://papermc.vercel.app/downloads/paper seems real small and you have to be fairly close to the arrow for the button to activate

I don't know the proper way to fix this but in https://github.com/PaperMC/face-lift/blob/main/src/components/input/SoftwareDownloadButton.tsx if you remove the px-5, py-3, and gap-4 from the parent div, add padding: .75rem 0 .75rem 1.25rem; to the a tag, and add padding: .75rem 1.25rem; to the button tag you get the same visual appearance but the clickable areas cover the entire blue button area.


Grpc.Core.Internal.PlatformApis throws FileNotFoundException

What version of gRPC and what language are you using?
C# Grpc.Core 1.22.0
What operating system (Linux, Windows,...) and version?
Windows 10
What runtime / compiler are you using (e.g. python version or version of gcc)
netstandard2.0
netcoreapp2.2
What did you do?
just using google cloud firestore c# library
Bug
3 exception every time i start my app:
FileNotFoundExeption for Xamarin.IOs, UnityEngine and Mono
the part in Grpc.Core.Internal.PlatformApis which identify the platform is wrong and inefficent.
it just try to get the type:
var unityApplicationClass = Type.GetType(UnityEngineApplicationClassName);
without checking if the assembly is present and this throw a FileNotFoundException
this will always throw 3 exception in every platform

Do you have any suggestions for how to improve the Unity platform check?
Btw, according to the documentation https://docs.microsoft.com/en-us/dotnet/api/system.type.gettype?view=netframework-4.8#System_Type_GetType_System_String_,
the Type.GetType(UnityEngineApplicationClassName);  doesn't throw when the type is not found - it only returns a null (which is consistent with the behavior I'm seeing = no exception). Can you provide better instructions for how to reproduce the FileNotFoundException?

We were facing issue too for one of our application - the exact stack trace was
The type initializer for 'Grpc.Core.Internal.PlatformApis' threw an exception. --->
System.TypeInitializationException: The type initializer for 'Grpc.Core.Internal.PlatformApis' threw an exception. --->
System.IO.FileLoadException: Could not load file or assembly 'UnityEngine' or one of its dependencies. An argument was out of its legal range. (Exception from HRESULT: 0x80131502) ---> System.ArgumentOutOfRangeException: Length cannot be less than zero. Parameter name: length at System.String.Substring(Int32 startIndex, Int32 length)
at Common.OnResolveAssembly(Object sender, ResolveEventArgs args)
at System.AppDomain.OnAssemblyResolveEvent(RuntimeAssembly assembly, String assemblyFullName)
at System.RuntimeTypeHandle.GetTypeByName(String name, Boolean throwOnError, Boolean ignoreCase, Boolean reflectionOnly, StackCrawlMarkHandle stackMark, IntPtr pPrivHostBinder, Boolean loadTypeFromPartialName, ObjectHandleOnStack type)
at System.RuntimeTypeHandle.GetTypeByName(String name, Boolean throwOnError, Boolean ignoreCase, Boolean reflectionOnly, StackCrawlMark& stackMark, IntPtr pPrivHostBinder, Boolean loadTypeFromPartialName)
at System.RuntimeType.GetType(String typeName, Boolean throwOnError, Boolean ignoreCase, Boolean reflectionOnly, StackCrawlMark& stackMark) at System.Type.GetType(String typeName)
at Grpc.Core.Internal.PlatformApis..cctor()
Found out that our app had a delegate registered on
AppDomain.CurrentDomain.AssemblyResolve += CurrentDomain_AssemblyResolve.
So, when PlatformApis invoke
Type.GetType(UnityEngineApplicationClassName);
The delegate fires as this type could not be resolved and this delegate attempted to load from another location and the assembly was missing there too, and this exception was thrown.
Looking at stack trace from @edvher I see the same stack there as well -
at ManagedLoader.AssemblyResolveHandler(Object sender, ResolveEventArgs args)
at System.AppDomain.OnAssemblyResolveEvent(RuntimeAssembly assembly, String

We were facing this issue too for one of our application - the exact stack trace was
The type initializer for 'Grpc.Core.Internal.PlatformApis' threw an exception. --->
System.TypeInitializationException: The type initializer for 'Grpc.Core.Internal.PlatformApis' threw an exception. --->
System.IO.FileLoadException: Could not load file or assembly 'UnityEngine' or one of its dependencies. An argument was out of its legal range. (Exception from HRESULT: 0x80131502) ---> System.ArgumentOutOfRangeException: Length cannot be less than zero. Parameter name: length at System.String.Substring(Int32 startIndex, Int32 length)
at Common.OnResolveAssembly(Object sender, ResolveEventArgs args)
at System.AppDomain.OnAssemblyResolveEvent(RuntimeAssembly assembly, String assemblyFullName)
at System.RuntimeTypeHandle.GetTypeByName(String name, Boolean throwOnError, Boolean ignoreCase, Boolean reflectionOnly, StackCrawlMarkHandle stackMark, IntPtr pPrivHostBinder, Boolean loadTypeFromPartialName, ObjectHandleOnStack type)
at System.RuntimeTypeHandle.GetTypeByName(String name, Boolean throwOnError, Boolean ignoreCase, Boolean reflectionOnly, StackCrawlMark& stackMark, IntPtr pPrivHostBinder, Boolean loadTypeFromPartialName)
at System.RuntimeType.GetType(String typeName, Boolean throwOnError, Boolean ignoreCase, Boolean reflectionOnly, StackCrawlMark& stackMark) at System.Type.GetType(String typeName)
at Grpc.Core.Internal.PlatformApis..cctor()
Found out that our app had a delegate registered on
AppDomain.CurrentDomain.AssemblyResolve += CurrentDomain_AssemblyResolve.
So, when PlatformApis invoke
Type.GetType(UnityEngineApplicationClassName);
The delegate fires as this type could not be resolved and this delegate attempted to load from another location and the assembly was missing there too, and this exception was thrown.
Looking at stack trace from @edvher I see the same stack there as well -
at ManagedLoader.AssemblyResolveHandler(Object sender, ResolveEventArgs args)
at System.AppDomain.OnAssemblyResolveEvent(RuntimeAssembly assembly, String

hi sreejit,
how did you solve the problem?

@sreejit-a

We were facing this issue too for one of our application - the exact stack trace was
The type initializer for 'Grpc.Core.Internal.PlatformApis' threw an exception. --->
System.TypeInitializationException: The type initializer for 'Grpc.Core.Internal.PlatformApis' threw an exception. --->
System.IO.FileLoadException: Could not load file or assembly 'UnityEngine' or one of its dependencies. An argument was out of its legal range. (Exception from HRESULT: 0x80131502) ---> System.ArgumentOutOfRangeException: Length cannot be less than zero. Parameter name: length at System.String.Substring(Int32 startIndex, Int32 length)
at Common.OnResolveAssembly(Object sender, ResolveEventArgs args)
at System.AppDomain.OnAssemblyResolveEvent(RuntimeAssembly assembly, String assemblyFullName)
at System.RuntimeTypeHandle.GetTypeByName(String name, Boolean throwOnError, Boolean ignoreCase, Boolean reflectionOnly, StackCrawlMarkHandle stackMark, IntPtr pPrivHostBinder, Boolean loadTypeFromPartialName, ObjectHandleOnStack type)
at System.RuntimeTypeHandle.GetTypeByName(String name, Boolean throwOnError, Boolean ignoreCase, Boolean reflectionOnly, StackCrawlMark& stackMark, IntPtr pPrivHostBinder, Boolean loadTypeFromPartialName)
at System.RuntimeType.GetType(String typeName, Boolean throwOnError, Boolean ignoreCase, Boolean reflectionOnly, StackCrawlMark& stackMark) at System.Type.GetType(String typeName)
at Grpc.Core.Internal.PlatformApis..cctor()
Found out that our app had a delegate registered on
AppDomain.CurrentDomain.AssemblyResolve += CurrentDomain_AssemblyResolve.
So, when PlatformApis invoke
Type.GetType(UnityEngineApplicationClassName);
The delegate fires as this type could not be resolved and this delegate attempted to load from another location and the assembly was missing there too, and this exception was thrown.
Looking at stack trace from @edvher I see the same stack there as well -
at ManagedLoader.AssemblyResolveHandler(Object sender, ResolveEventArgs args)
at System.AppDomain.OnAssemblyResolveEvent(RuntimeAssembly assembly, String

hi @sreejit-a
how did you solve the problem?

@sreejit-a

We were facing this issue too for one of our application - the exact stack trace was
The type initializer for 'Grpc.Core.Internal.PlatformApis' threw an exception. --->
System.TypeInitializationException: The type initializer for 'Grpc.Core.Internal.PlatformApis' threw an exception. --->
System.IO.FileLoadException: Could not load file or assembly 'UnityEngine' or one of its dependencies. An argument was out of its legal range. (Exception from HRESULT: 0x80131502) ---> System.ArgumentOutOfRangeException: Length cannot be less than zero. Parameter name: length at System.String.Substring(Int32 startIndex, Int32 length)
at Common.OnResolveAssembly(Object sender, ResolveEventArgs args)
at System.AppDomain.OnAssemblyResolveEvent(RuntimeAssembly assembly, String assemblyFullName)
at System.RuntimeTypeHandle.GetTypeByName(String name, Boolean throwOnError, Boolean ignoreCase, Boolean reflectionOnly, StackCrawlMarkHandle stackMark, IntPtr pPrivHostBinder, Boolean loadTypeFromPartialName, ObjectHandleOnStack type)
at System.RuntimeTypeHandle.GetTypeByName(String name, Boolean throwOnError, Boolean ignoreCase, Boolean reflectionOnly, StackCrawlMark& stackMark, IntPtr pPrivHostBinder, Boolean loadTypeFromPartialName)
at System.RuntimeType.GetType(String typeName, Boolean throwOnError, Boolean ignoreCase, Boolean reflectionOnly, StackCrawlMark& stackMark) at System.Type.GetType(String typeName)
at Grpc.Core.Internal.PlatformApis..cctor()
Found out that our app had a delegate registered on
AppDomain.CurrentDomain.AssemblyResolve += CurrentDomain_AssemblyResolve.
So, when PlatformApis invoke
Type.GetType(UnityEngineApplicationClassName);
The delegate fires as this type could not be resolved and this delegate attempted to load from another location and the assembly was missing there too, and this exception was thrown.
Looking at stack trace from @edvher I see the same stack there as well -
at ManagedLoader.AssemblyResolveHandler(Object sender, ResolveEventArgs args)
at System.AppDomain.OnAssemblyResolveEvent(RuntimeAssembly assembly, String

hi @sreejit-a
how did you solve the problem?

We fixed by changing the assembly resolve delegate to not throw an exception and return a null type.

Ok so it looks like that the root cause for the problem was:

a custom assembly resolver was registered in the affected application
when grpc code checks whether or not it's running on unity, it does var unityApplicationClass = Type.GetType(UnityEngineApplicationClassName);, which with combination with the custom assembly resolver has thrown an exception.

Looks like this is not really a problem in gRPC code, but in the custom assembly resolver you used. Closing.


list dtype in from_pandas() is casted as string[pyarrow]

Describe the issue:
I cannot load lists into Dataframes in the most recent version of Dask because they are type casted.

Environment:

Dask version: 2023.9.1
Python version: 3.11
Install method (conda, pip, source): pip

It's a feature apparently. Can be disabled with
dask.config.set({"dataframe.convert-string": False})


Appstudio update test-component-pac-rnym

Pipelines as Code configuration proposal

Pipelines as Code CI/test-component-pac-rnym-on-pull-request has successfully validated your commit.

StatusDurationName

✅ Succeeded
16 seconds
init

✅ Succeeded
19 seconds
clone-repository

✅ Succeeded
59 seconds
build-container

✅ Succeeded
22 seconds
sanity-inspect-image

✅ Succeeded
22 seconds
deprecated-base-image-check

✅ Succeeded
50 seconds
clamav-scan

✅ Succeeded
3 minutes
clair-scan

✅ Succeeded
17 seconds
sbom-json-check

✅ Succeeded
10 seconds
sanity-optional-label-check

✅ Succeeded
10 seconds
sanity-label-check

✅ Succeeded
16 seconds
show-summary


Specification of stimuli locations

Allow more flexible and compact specification of stimulus locations

Addressed by https://github.com/lookit/exp-addons/pull/76


Create kernels with the environment variables related to activated environment

We should scrape environment variables of the current environment (using existing services) and place those variables into the kernel spec (that we create).
This makes it unnecessary to start jupyter within the same environment.
Making this simple change now paves the way to starting jupyter from another environment.

Validated


Improve LDA formula interface behavior

Use temporary names for feature transformer columns to avoid clashes.
Use regex tokenizer instead of tokenizer, which is more robust (e.g. splits on \\s+ instead of  ).
Add a stop words remover stage.

In case this helps, looks like tests need to be updated:
Context: broom-lda
lda.tidy() works: failure: nrow(o) not equal to `exp.row`.
1/1 mismatches
[1] 18 - 27 == -9
lda.tidy() works: failure: td1$beta[1:3] not equal to c(0.879, 0.946, 1.15).

Yep, @samuelmacedo83 has https://github.com/rstudio/sparklyr/pull/1886 fixing these, closing.


Add files via upload

Your checklist for this pull request
Author name
Author:
NumberBasher
About your game
What is your game about?
Sliding Puzzle Game Without Friction
How do you play your game?
WASD.

How do I resubmit?

@GalaxyGamingBoy also, while we're here, how much should i worry about in terms of optimization (will sprig require optimization)
my current code is not worrying about optimization at all (a lot of redundant checks, etc.)

How do I resubmit?

Hi! Just commit the updated file and I'll rereview it! No need to close this PR, GitHub will automatically update it based on your repo

@GalaxyGamingBoy also, while we're here, how much should i worry about in terms of optimization (will sprig require optimization)
my current code is not worrying about optimization at all (a lot of redundant checks, etc.)

You don't have to worry about that for now. You can always do further optimizations after getting your sprig so you can test it on device

@GalaxyGamingBoy Updated!

hello?

Unfortunately, this game is too similar to others in the gallery (i.e. much of the code is identical).  Closing this PR as it hasn't moved in some time.


modify batchproducer to take an interface for logging

Hi, I'm using this library and the only way I could get it to integrate with our custom logging library was to make batchproducer take an interface instead of the log.Logger instance. The interface is only one method Printf(format string, ...args interface{}) which log.Logger satisfies. This allows me to add some custom formatting around the logs the library already outputs.

Hello. So, in this case we should not import log package, should we?

No you can continue to import the log package because you will use it as your default logger.

Thanks


Update README.md

The webserver is running on the device's IP, not your computers

Hi @harikb thanks for the PR. This change was made already in another PR, so I am going to close it. Thanks again.


Update docs site and add release notes for v1.1.5

For https://github.com/cockroachdb/cockroach/issues/21890.

This change is

http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/aa4732d25f0a91a5920b704608ab7e2d7a5f241a/

http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/fbda7407572d4818ffe4b84e334a9139d44b9276/

http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/fbda7407572d4818ffe4b84e334a9139d44b9276/


Replace lib/messages with a @ldapjs/messages module

This work will be very similar to #797.

https://github.com/ldapjs/messages has been created and I'm working through migrating all of the message type implementations to it. Unit tests in CI are failing at the time of writing because the code depends on an unpublished version of @ldapjs/asn1. I do not plan to publish that version until I am certain no further changes need to be made as I implement the remaining message types.

⚠ This issue has been locked due to age. If you have encountered a recent
problem that seems to be covered by this issue, please open a new issue.
Please include a minimal reproducible example
when opening a new issue.


Args in labels and menus of RakuScript

Inspired by #235 and it shows that we need #53.
Is just way to make menu and labels be as useful as funcs in other langues.
Example:
func _ready():
  var door_script = Rakugo.parse_script("res://door.rk")

  door_a.pressed.connect(Rakugo. execute_script(door_script, "door_menu", ["door_a"]))
  door_b.pressed.connect(Rakugo. execute_script(door_script, "door_menu", ["door_b"]))

menu explore:
  "Go to door A" > door_menu("door_a")
  "Go to door b" > door_menu("door_b")
  "Go to next corridor"

menu door_menu(door_id):
  "Lock pick" show if (has_lockpics > 1) > lockpick_minigame(door)
  "Open Door" if door_id in keys > open_door(door_id)
  "Open Door" hide if door_id in closed_doors > missing_key
  "Go in" show if door_id in open_doors > door_id
  "Go Back to exploring" > explore

@theludovyc I updated and fixed example and description in inital comment of this issue.


Update README.md

https://github.com/all-contributors please add @agupta15k for infrastructure, tests and code

https://github.com/all-contributors please add @nagarajumadamshetti  for infrastructure, tests and code

@all-contributors

@all-contributors please add @nagarajumadamshetti for infrastructure, tests and code

https://github.com/all-contributors please add @agupta15k  for infrastructure, tests and code

@all-contributors please add @agupta15k for  infrastructure, tests and code

@all-contributors please add @nitesh31mishra for infrastructure, tests and code

@allcontributors please add @Arunsp2000

@all-contributors please add @Arunsp2000 for his code


Change the names for form fields "first name" and "last name", to match the mapping in User model.

At sign up, first name and last name are not stored by Redis because the form fields' names don't match the expected values. Instead of firstname and lastname, givenName and lastName are expected.

Thanks @ovi-tamasan-3pg, this will be in the next release :)


Invalid aria in Introduction to ARIA page

What's changed, or what was fixed?

Fix invalid aria in Introduction to ARIA page

Fixes: #8642
Target Live Date: YYYY-MM-DD

[ ] This has been reviewed and approved by (NAME)
[ ] I have run npm test locally and all tests pass.
[ ] I have added the appropriate type-something label.
[ ] I've staged the site and manually verified that my content displays correctly.

CC: @petele

@robdodson - can you confirm and merge if its correct? thanks!

good catch @IgorWilbert, thank you!


CSS rule with content and CSS entity reference results to an undefined stylesheet

Description
When using the styles getter inside a LitElement and having a CSS rule that includes some special character escaping results in the whole StyleSheet to be undefined.
Removing the special character rule makes it work again.
Example css:
 static get styles() {
    return [css`
      div {background-color:#ff0}
      div::before {content:"\2014"}
    `];
  }

Is it possible to get that supported inside LitElements?
Live Demo
https://stackblitz.com/edit/lit-element-example-5fv9er
Steps to Reproduce

Run  the app
Remove the div::before{content:"\2014"} CSS rule
StyleSheet works again

Expected Results
It should show the content before every div.
Actual Results
StyleSheet is undefined. Actually stored the css rules in a variable to see what happens.
Output:
{
  "_styleSheet": {}
}

Browsers Affected

[x] Chrome

Versions

"lit-element": "^2.0.0-rc.5",

You need to escape the backslash content:"\\2014".

Which is

cumbersome for designers that are used to write correct CSS
not that easy when loading the CSS from a file which doesn't escape backslashes
a workaround not a solution (other tools can handle unescaped CSS just fine)

I have tried to adopt a webpack-loader that grabs the CSS from a file and generate the needed lit css syntax. Although, search-replacing backslashes is not easy in JS, at least i haven't found a way to do that programmatically.

I am facing this problem with 3rd party css... so sadly I have no chance to escape anything.

I tried to figure out what is going on on lit-element side and it looks like it goes deeper and this behaviour is caused by tagged templated string. More over, it looks like this is default behaviour of javascript engine and templated string behaves the same way. So the only solution for now is escape slashes in source code.

I belive this is a side effect.
The documentation needs to be spelled out on this "problem". I also think that there should maybe be a special template literal for a raw file (e.g. fileCSS) where you process the .raw (post-interpolated) template literals.
I don't know how common this is (https://tc39.github.io/proposal-template-literal-revision/) but it seems to be present in the browsers I am testing.
I was able to solve this problem with a patch to lit-element and using the unsafeCSS, you could similarly check the cssText for undefined in the css template tag and apply the raw in that case as well
--
index cea2875..7920009 100644
--- a/node_modules/lit-element/lib/css-tag.js
+++ b/node_modules/lit-element/lib/css-tag.js
@@ -46,7 +46,7 @@ export class CSSResult {
  * this with trusted input.
  */
 export const unsafeCSS = (value) => {
-    return new CSSResult(String(value), constructionToken);
+    return new CSSResult(String(value) || String(value.raw), constructionToken);
 };
 const textFromCSSResult = (value) => {
     if (value instanceof CSSResult) {
diff --git a/node_modules/lit-element/src/lib/css-tag.ts b/node_modules/lit-element/src/lib/css-tag.ts
index e508be3..5502c78 100644
--- a/node_modules/lit-element/src/lib/css-tag.ts
+++ b/node_modules/lit-element/src/lib/css-tag.ts
@@ -57,7 +57,7 @@ export class CSSResult {
  * this with trusted input.
  */
 export const unsafeCSS = (value: unknown) => {
-  return new CSSResult(String(value), constructionToken);
+  return new CSSResult(String(value) || String(value.raw), constructionToken);
 };

 const textFromCSSResult = (value: CSSResult) => {

This most likely a failure on updating the lit-element style doc. There is
PR awaiting merge at:
https://github.com/Polymer/lit-element/pull/600/commits/64985b86d359ce6b668adf00bd6586bef9e977a7
Please refer to this for guidance for now.

sort of...the issue is in the real world you want a build process where you can do something like this:
import css from '/some/commerical/library/not/under/my/control.css'
as part of your build process but it is buggered up with css unicode escape sequences like
ul::before
{
content: \2022
}

and you can't really do much about preprocessing it before it becomes a template literal, when it does it breaks because the template processing function (like css in css-tag.js) gets "undefined" as the template result.  This same function gets strings.raw as well. The right way might be to check if strings[0] is undefined and if strings.raw is defined it should process that instead.
The author of the ticket's example is also valid and is solved equally where the css template function will accept either the template interpolation OR the raw value if the template interpolation is not available.
If you don't accept a code change, then I would at least document that unicode escape sequences can break ES2016 template strings and the result will import correctly as a css template literal .
I think my example (above) probably should involve css and not unsafeCSS, but essentially the same fix

Looks like this is just a gotcha we need to add to the docs.

Caveats around escaping CSS in template literals is now covered in https://lit.dev/docs/components/styles/#using-unicode-escapes-in-styles.


Nil-Value with embedded_in and field-type Array

I've created a Rails3 Repo with a rspec-test to illustrate the problem here: https://github.com/QEDio/mongoid_error.
Clone, bundle install, rake.
In short
class Computer
  include Mongoid::Document
  embeds_many :parts
end

class Part
  include Mongoid::Document
  embedded_in :computer

  field :description1, type: Array
  field :description2
end

This fails:
computer = Computer.new
part = Part.new

computer.parts << part
computer.save

part.description1 = PART_DATA1
part.save

part.description1 = PART_DATA2
part.save

==> now part.description1 is null (in the db)

Can you test against 2.4.0-stable branch, I think I may have just fixed this with this commit: 4be33fef2daed3bd0070abbd3ca0c39c5c58244a

Sorry, no such luck :(.
Bundler tells me 'Using mongoid (2.4.0) from git://github.com/mongoid/mongoid.git (at 2.4.0-stable)' and if at the source at 'bundler show mongoid' I see your changes from https://github.com/mongoid/mongoid/commit/4be33fef2daed3bd0070abbd3ca0c39c5c58244a

Ok thanks... I'll have a better look in the morning.

I can confirm this bug on master 6ff71c18a2cb0a863478a6f2db4e46cdd222b8be.

Issue is trying to do document.attributes[key] for embedded keys: "relation.4.attribute" instead of just the name of the field.


chore(main): release sandworm-mocha 1.2.0

:robot: I have created a release beep boop
1.2.0 (2022-09-27)
Features

extract common utils to separate module (010027c)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/sandworm-hq/sandworm-mocha/releases/tag/sandworm-mocha-v1.2.0 :sunflower:


Use senderTimestampMs instead of messageTimestamp

message.messageTimestamp contains timestamp in seconds, not in milliseconds as content.pollUpdateMessage.senderTimestampMs
It's important to get exact milliseconds for multi options poll, because we can get two message.update events with the same senderTimestampMs (because it's in seconds, not in milliseconds) so we can understand which vote is the latest one
Here's an event example:
{
    "messages.upsert": {
        "messages": [
            {
                "key": {...},
                "messageTimestamp":<PHONE_NUMBER>,
                "message": {
                    "messageContextInfo": {...},
                    },
                    "pollUpdateMessage": {
                        "pollCreationMessageKey": {...},
                        "vote": {...},
                        "senderTimestampMs": "1692774684709"
                    }
                }
            }
        ],
        "type": "notify"
    },

There's .toNumber() because senderTimestampMs has Long type
{
  "low": 739148892,
  "high": 394,
  "unsigned": false
}

If we want to keep Long - let me know and I'll adjust the changes!

Let me know if I need to fix the "check-lint" type issues, I'll handle type it!


v0.3.1 with pf2e 6.0.3 and foundry v12.327 recall knowledge still non-functional

Describe the bug
The recall knowledge buttons are not added correctly to NPC Sheets, nor are the default ones removed.
To Reproduce

Create a world with pf2e 6.0.3, foundry v12.327
Activate pf2e-reaction module
Configure settings to enable the recall knowledge feature of the module
Open NPC sheets and see that no recall knowledge buttons are present

Expected behavior
The recall knowledge buttons to be added to NPC sheets and, if applicable, the default ones to be removed.
Screenshots

Please check console, maybe some error exists. Because as I see options like hide default and add lore no work

This seems to be happening with the newest version of the system 6.4.0

Settings is on ? Any errors in console?

Settings is on ? Any errors in console?

Setting is on - no console errors.

Okay further testing shows this is a problem specifically with pf2e bestiary tracker. I'm gonna see if it can be fixed on their end first.

Okay further further testing - I think its a weird conflict with how each thing is looking at what. It seems like PF2e Reaction Checker is looking at actors, not the compendium version of an NPC, but bestiary tracker is looking at the compendium. So for example, your recall knowledge edits show up on the actor sheet of a monster, but not the compendium sheet, which probably shouldnt be the case (they should be the same imo), but bestiary tracking is looking at the compendium thing which confuses me.

This module read info of actor. Actor is editable and could have difference with compendium

(PF2E Bestiary Tracking dev) Heya,
It's not really an error on either module. I am simply rendering the base actor's sheet, not the sheet of a token's synthetic actor.
So the rendering logic of this module isn't performed, as expected.(sheet.token = null) if (game.user?.isGM && isNPC(sheet.actor) && sheet.token && Settings.recallKnowledge) {`

sheet.token - null, only if you open actor sheet from compendium
this module works with "real" actor, but I could delete this check and also show same for compendium

What I'm doing on my end is that I have the actor's UUID, fetching the actor and then rendering the sheet.

I'm fetching an actor from within the world: game.actors.find(actor => actor.uuid === <savedUUID>).
Render actor.sheet.render(true).
In this situation, sheet.token = null

Removing that check should allow it to work in the bestiary if that's a possibility for you.


Does not work for new comment added anymore

After the payload updated, new comments to PR do not get their dependencies found

might have been resolved by my fix here
not sure about the scenario behind this

It is fixed indeed. I've just checked.

fixed by #35


Does amazon plan to provide a C/C++ based AuthServer?

IMPORTANT: Before you create an issue, please take a look at our Issue Reporting Guide.
Python is a little heavy on an embedded device which has limited flash . Does Amazon have a plan to develop a C/C++ based AuthServer to replace the current Python based AuthServer?
Briefly summarize your issue:

What is the expected behavior?

What behavior are you observing?

Provide the steps to reproduce the issue, if applicable:

Tell us about your environment:

What version of the AVS Device SDK are you using?
  <x.y.z>

Tell us what hardware you're using:

[ ] Desktop / Laptop
[ ] Raspberry Pi
[ ] Other - tell us more:

Tell us about your OS (Type & version):

[ ] Linux
[ ] MacOS
[ ] Raspbian Stretch
[ ] Raspbian Jessy
[ ] Other - tell us more:

Hi Subjectxbj,
No, there are no plans to create a C/C++ version of AuthServer.py.  AuthServer.py does not model any of the device side behaviors of the various methods for authorizing your device or application to access AVS (see [Authorization](https://developer.amazon.com/docs/alexa-voice-service/api-overview.html#authorization for details on those methods)).  We are considering adding code to the SDK for the device side of those behaviors as time allows and as it becomes clear which ones would provide the most benefit to developers.
Thank you,
-SWH

Closing for now, but please re-open if you have any further questions.
Thanks,
Sanjay


Can access to the latitude and longitude and the time of shooting？Thanks

If you are creating an issue for a BUG please fill out this information. If you are asking a question or requesting a feature you can delete the sections below.
Failure to fill out this information will result in this issue being closed. If you post a full stack trace in a bug it will be closed, please post it to http://gist.github.com and then post the link here.
Bug Information
Version Number of Plugin:
Device Tested On:
Simulator Tested On:
Version of VS:
Version of Xamarin:
Versions of other things you are using:
Steps to reproduce the Behavior
Expected Behavior
Actual Behavior
Code snippet
Screenshotst

Has been resolved，Thank you

Has been resolved，Thank you