a player must receive one push notification even if he has been registered more than once

currently there will be send a push for every event a player is registered for

the push have to contain every information necessary to join the match like:
Welcome to your minigolf-friday.
at 20:00 you play in "BOOGEY15" on "Shangrila hard" in a game with 5 players.
at 22:00 you play in "LALA15" on "Sweetopia easy" in a game with 3 players.
Have fun!


[BUG] UnicodeDecodeError in line 313 of app.py

Describe the bug
When I try to run the program, it briefly opens the main window and then closes, presenting the following message:
  File "e:\OneDrive\Programas\PyDebloatX-1.9.0\pydebloatx\app.py", line 313, in run
    names_list = x.communicate()[0].decode().split()
UnicodeDecodeError: 'utf-8' codec can't decode byte 0xc6 in position 74: invalid continuation byte

To Reproduce

Downloaded the 1.9.0 version folder from GitHub
Opened the folder "/pydebloatx/" and tried to run app.py.
I used VS Code with a proper environment setup that has pyqt5 and Python 3.8.6.

Expected behavior
Just hoped the main window would at least open
Desktop (please complete the following information):

Windows 10 Pro 20H2 in Brazilian Portuguese
Version 1.9.0

Additional context
When I run the command in Powershell, I get the following:
pydebloatbug.txt

Could you please test the latest master branch if there is any difference? I am not able to reproduce the issue.

Ok, the error changed, now I get:
  File "e:\OneDrive\Programas\PyDebloatX\pydebloatx\app.py", line 340, in run
    names_list = json.loads(names_str)
  File "C:\Users\Guilherme\anaconda3\envs\QWSim\lib\json\__init__.py", line 357, in loads
    return _default_decoder.decode(s)
  File "C:\Users\Guilherme\anaconda3\envs\QWSim\lib\json\decoder.py", line 337, in decode
    obj, end = self.raw_decode(s, idx=_w(s, 0).end())
  File "C:\Users\Guilherme\anaconda3\envs\QWSim\lib\json\decoder.py", line 355, in raw_decode
    raise JSONDecodeError("Expecting value", s, err.value) from None
json.decoder.JSONDecodeError: Expecting value: line 1 column 1 (char 0)

So I put print(names_list) in the code, and it printed the following (I removed the lines that list each app, because I think that the problem is in the first line):
. : O arquivo C:\Users\Guilherme\Documents\WindowsPowerShell\profile.ps1 nÆo
pode ser carregado porque a execu‡Æo de scripts foi desabilitada neste
sistema. Para obter mais informa‡äes, consulte about_Execution_Policies em
https://go.microsoft.com/fwlink/?LinkID=135170.
No linha:1 caractere:3
+ . 'C:\Users\Guilherme\Documents\WindowsPowerShell\profile.ps1'
+   ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    + CategoryInfo          : ErrodeSeguran‡a: (:) [], PSSecurityException
    + FullyQualifiedErrorId : UnauthorizedAccess
[
    {
        "Name":  "Microsoft.NET.Native.Runtime.1.4",
        "InstallLocation":  "C:\\Program Files\\Windo
...
...
        "InstallLocation":  "C:\\Windows\\SystemApps\\Microsoft.Windows.SecureAssessmentBrowser_cw5n1h2txyewy"
    },
    {
        "Name":  "Microsoft.Windows.SecHealthUI",
        "InstallLo
Traceback (most recent call last):
  File "e:\OneDrive\Programas\PyDebloatX\pydebloatx\app.py", line 341, in run
    names_list = json.loads(names_str)
  File "C:\Users\Guilherme\anaconda3\envs\QWSim\lib\json\__init__.py", line 357, in loads
    return _default_decoder.decode(s)
  File "C:\Users\Guilherme\anaconda3\envs\QWSim\lib\json\decoder.py", line 337, in decode
    obj, end = self.raw_decode(s, idx=_w(s, 0).end())
  File "C:\Users\Guilherme\anaconda3\envs\QWSim\lib\json\decoder.py", line 355, in raw_decode
    raise JSONDecodeError("Expecting value", s, err.value) from None
json.decoder.JSONDecodeError: Expecting value: line 1 column 1 (char 0)

There seems to be an issue with the script execution, because it was "disabled in my system". It can't load a powershell profile and throws an error. Maybe the unicode error is in the symbol "nÆo" which should actually be "não". Beyond that, I think that the "json.loads" was expecting a list without this phrase before, but I'm not sure, never used this function.
Running in an elevated command prompt doesn't help with the issue.

What if you remove the following line from line 338:
universal_newlines=True

And set the line 339 to:
names_str = x.communicate()[0].decode('utf-8', errors='ignore')

I get the same error, but the output of "print(names_str)" doesn't show "nÆo", now it's "no".

First, you try to run with the packaged installation file to see if there are any exceptions.
If the packaged file works properly, the bug has nothing to do with the code.
Solutions in VS Code:

Running PowerShell in administrator mode
View system execution policy status Get-ExecutionPolicy
Modify execution policy status Set-ExecutionPolicy -ExecutionPolicy RemoteSigned
Finally, close PowerShell and VS Code, and then reopen VS Code

First, you try to run with the packaged installation file to see if there are any exceptions.
If the packaged file works properly, the bug has nothing to do with the code.
Solutions in VS Code:

Running PowerShell in administrator mode
View system execution policy status Get-ExecutionPolicy
Modify execution policy status Set-ExecutionPolicy -ExecutionPolicy RemoteSigned
Finally, close PowerShell and VS Code, and then reopen VS Code

Yea, that might be a permission issue. Most users are not expected to run those commands in Powershell. And without them, we don't have access to the app folders without admin rights.

Yea, that might be a permission issue. Most users are not expected to run those commands in Powershell. And without them, we don't have access to the app folders without admin rights.

Ok, I will try the steps proposed by @space9bug .

Ok, I will try the steps proposed by @space9bug .

I installed version 1.9.0 and it worked correctly, so I guess it really is a permission issue. Thanks for your help and sorry for the trouble.

I installed version 1.9.0 and it worked correctly, so I guess it really is a permission issue. Thanks for your help and sorry for the trouble.


Update lgtm.yml

Hmmm ... it was not supposed to be merged. Didn't ask for review yet.
(was not set as work in progress cause seems LGTM does not evaluate those)

Hmmm ... it was not supposed to be merged. Didn't ask for review yet. (was not set as work in progress cause seems LGTM does not evaluate those)

Oh, sorry, I thought it was ready.


Audio: move from old MAME YM core to ymfm

The existing YM2151 core has a license that could potentially be problematic.  ymfm is licensed under BSD, which is compatible with the rest of the project being under MIT.
This introduces a C++ lib into a C project. For now the glue interface replicates the old function calls so that the emu side didn't have to change how it called into the lib.

I don't think there are any technical barriers here, but since there might be philosophical arguments against adding a C++ lib into what is otherwise a pure C project. I want to ensure the top contributors are okay with this before it gets merged.  @indigodarkwolf @stefan-b-jakobsson @jestin @irmen if you have any objections, please raise them.
x16-emulator is still a C project, and is a functional interface (ymglue.cpp) that the C code will call to reach the YM2151 C++ code through a shim.

Personally I like to avoid C++, but if there is a great benefit I think we can use such a library.

I don't have any concerns about adopting C++ into x16emu, but I would take care to check that the ymfm library is outputting data at an appropriate sample rate for replacing the previous ym2151 code. Box16 needed to resample from the ymfm library's sample rate, which turned into an adventure. That said, all of Box16's contributions should be license-compatible with x16emu, so if needs be, things should be portable.

but I would take care to check that the ymfm library is outputting data at an appropriate sample rate for replacing the previous ym2151 code.

It looks like this is already the case. It appears ymfm is true chip emulation so it's that funky ~55.9KHz rate is coming out as x16emu is clocking the chip at 3.579545MHz.  Natt's code already handles resampling and mixing with that expectation.

I remember box16 had some issues being able to be compiled on certain platforms when it upgraded to a newer C++ standard. I  am FAR from an expert here, but I see in this makefile you set the c++ standard to c++17.   Is this potentially problematic? Is it required? Can we select an older standard this library has to conform to, to avoid such issues?

I remember box16 had some issues being able to be compiled on certain platforms when it upgraded to a newer C++ standard. I am FAR from an expert here, but I see in this makefile you set the c++ standard to c++17. Is this potentially problematic? Is it required? Can we select an older standard this library has to conform to, to avoid such issues?

C++17 is what ymfm needs, according to its docs.  I think Stephen was trying to use C++20 in Box16 which was probably a bit more bleeding edge at the time.


1809 version 437 prevents multiusers in RDP wrap

Using 292 listening and fully supported allowed three concurrent sessions on 4/6/20 within the LAN to my Windows 10 Pro.  Setup WAN port forwarding on 4/10/20 and connected with success.  However, now I can only connect to the same computer with one user at a time.  Every time I try to login I get the message that another session is already in progress and I have to terminate one to start another.  Yes I have unchecked the single session box in RDPconf.  This problem did not exist until I tried from a WAN location.  Now even LAN connections are one at a time.  Another thing I just noticed is that over the weekend I got updated to 1809 437.  Is that the problem?  Still listening and fully supported...  Would I be better off going to Windows 7 Pro for stability?

Fixed issue by rolling back windows\system32\termsrv.dll to 292 which thankfully I had saved.  Had to change the ownership of termsrv.dll to allow replacement.  For the future I created a batch file that  run in administrative mode which stop terminalservice and then copies the file and then restarts terminalservice.    Just like that it started working again.


🚀 Feature: Render a single template file similar to fetch:plain:file

🔖 Feature description
The fetch:template action currently supports rendering template files as an entire directory.  I would like the ability to selectively render an individual template file (and/or filter a subset of directory files) with variables.
🎤 Context
This feature could be very useful when attempting to reuse template files (avoiding duplication) and combine them with other files, where necessary.  Multiple fetch:template steps could be used to prepare the workspace files before publishing, some rendering an entire directory, and others rendering individual files.
✌ Possible Implementation
This request seems to have parallels with the fetch:plain:file action that was implemented from Issue 15538.  Perhaps creating a new fetch:template:file action would address similar concerns.
👀 Have you spent some time to check if this feature request has been raised before?

[X] I checked and didn't find similar issue

🏢 Have you read the Code of Conduct?

[X] I have read the Code of Conduct

Are you willing to submit PR?
No, but I'm happy to collaborate on a PR with someone else

Sounds like a good idea!

This sounds awesome. I am looking for exactly this atm.

@Ivaylo-Korakov do you want to contribute this functionality? :pray:

waiting for this

We use backstage templates a lot, and we would really like to see this feature coming to backstage.

I'm working on this. I've created the action, just working on tests now. Should have a PR out for it soon.


Result section

Show the latest election result using chart

Chart library to use
https://github.com/AnyChart/AnyChart-Android


source: plumb sym link base through to syft CLI

Continuing the work of b81c980, change the behavior of the syft command: the root for symbol link resolution of directory scans is now the directory root, not "/".
Do this pedantically, by separating the directory root to the scanned from the directory sym link base.  As of this commit, these two values will always be the same, but this strategy leaves the door open for them to be different someday, if that would ever be useful.
Also remove some unused functions and consolidate down to just the NewFromDirectoryBaseWithName() function.
Refs #1485, #1359

Looks like some of those "unused" functions I removed are used by tests.  I'll investigate...

Hi @jsquyres and thanks for the contribution here -- we have a bit of a different change that does this in PR https://github.com/anchore/syft/pull/1867. It should be noted the referenced PR has a prerequisite of some other changes and we're looking at getting these done hopefully in a week-ish timeframe. That said, I'm going to close this PR due to the overlap. Please let me know if I've misunderstood something and would be happy to reopen it!

@kzantow Sorry for my long silence on this -- I got distracted by vacation, holidays, and other shiny objects.  If #1867 implements effectively the same kind of functionality as I did in this PR, we can certainly close this PR.  I'd just like the functionality, regardless of whether it's my PR or someone else's. 😄

Thanks @jsquyres! I'm going to close this as noted -- we're making progress


Filled map and API

[X] Map item variants
[ ] Packet management
[ ] API to register maps
[X] API to update maps

Save modes

[X] No saving
[ ] Save map ID and some other properties
[ ] Save all map data including pixels

Compiler exit with code 2??

https://discord.com/channels/623638955262345216/623917901409419274/967355750089371668

I am not sure how I should indicate a map as persisted. Using map ID 0? Map ID is an int64.

Change MapDataViewer to MapViewer?

I am not sure how I should indicate a map as persisted. Using map ID 0? Map ID is an int64. It is not as easy as adding another field. Since this requires saving another unnecessary and non-official NBT tag in disk.

I am going to use the map_is_init NBT field for this. Map data provider will only give access to player whose map has true for that field.

I am not sure how I should indicate a map as persisted. Using map ID 0? Map ID is an int64. It is not as easy as adding another field. Since this requires saving another unnecessary and non-official NBT tag in disk.

I am going to use the map_is_init NBT field for this. Map data provider will only give access to player whose map has true for that field.

I decide to not mess with persistence in this PR.

Locked map item should be added. But I can not find the item ID on Fandom.


Fails if a track's image is 'None'

ERROR    2023-08-08 13:30:15,290 [27475:NotifyFrontend-18 (_actor_loop)] mopidy.listener
  Triggering event failed: track_playback_started(tl_track)
Traceback (most recent call last):
  File "/home/abadger/.local/lib/python3.11/site-packages/mopidy/listener.py", line 44, in on_event
    getattr(self, event)(**kwargs)
  File "/home/abadger/.local/lib/python3.11/site-packages/mopidy_notify/frontend.py", line 43, in track_playback_started
    self.show_notification(tl_track)
  File "/home/abadger/.local/lib/python3.11/site-packages/mopidy_notify/frontend.py", line 75, in show_notification
    icon = self.fetch_icon(track.uri)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/home/abadger/.local/lib/python3.11/site-packages/mopidy_notify/frontend.py", line 127, in fetch_icon
    preferred = self.find_preferred_image(images)
                ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/home/abadger/.local/lib/python3.11/site-packages/mopidy_notify/frontend.py", line 107, in find_preferred_image
    acceptable = [i for i in images if i.width <= self.ext_config["max_icon_size"]]
                 ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/home/abadger/.local/lib/python3.11/site-packages/mopidy_notify/frontend.py", line 107, in <listcomp>
    acceptable = [i for i in images if i.width <= self.ext_config["max_icon_size"]]
                                       ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
TypeError: '<=' not supported between instances of 'NoneType' and 'int'

It seems that some Mopidy backends will supply an icon with each track, but the icons are all 'None' objects. mopidy-notify does not expect this and throws an exception instead of ignoring the invalid image.
So far I've had this happen with every track from the following backends:

mopidy-youtube
mopidy-podcast
mopidy-somafm

Hi, thanks for reporting the issue! I've looked into it, and it doesn't seem like the backends are doing anything wrong. In fact, Mopidy verifies the images returned by backends.
What I think happens is that some backends return images where width = None. I'll try to fix that.


Fix #4648: Let the combiner combine with things other than cobblestone

This lets us combine netherrack with quartz to make nether quartz ore now, and can be extended to combine any two arbitrary items. Note the following two (rather minor) breaking changes:

Cobblestone used to be consumed at the beginning of work. It's now consumed at the end. Any combiners that were running prior to this update will lose the single piece of cobblestone that they had consumed but not yet finished using.
Any CraftTweaker scripts that used the three-argument form of mods.mekanism.combiner.addRecipe (passing <gas:liquidStone> as the second argument) will break.

Iis there a particular reason behind a new recipe/TE/container/gui type that exists to only be extended by the combiner? The combiner already has its own RecipeType

I imagine it could be useful in future machines, and there's already precedent for it: ChanceMachineRecipe is only extended by SawmillRecipe.

I also wonder whether we should make each ”lane” of the factories have their own cobblestone/netherrack/whatever slot, instead of all sharing one, so the same factory could do more than one kind of combining at once (although I don't think the infusion factory supports this either, so I'm not overly concerned right now).

It would probably get too messy if you added a slot per lane. You will need to account for that specially if you're consuming the item upon completion of course

Okay, I think it's good for now with just the one slot. (And we can always change that later if we really want.)


move recordId and contextId out of descriptor as peer level properties

Implemented by:
https://github.com/TBD54566975/dwn-sdk-js/commit/5c5f1812a13a9a0afc5ae30b4cf5b31290f25948
https://github.com/TBD54566975/dwn-sdk-js/commit/ec115077c0d85d60f59f113c77af88e363453f18
https://github.com/TBD54566975/dwn-sdk-js/commit/79ce876e62c4c1c72968bf83325c6cac87c7abc1


Fix labels and service selector

Updates service selector labels to fix the document-index-service and applies common labels to all resources.
Currently Values.podLabels and Values.selectorLabels = app: vespa, but the service template is using vespa.selectorLabels
from _helpers.tpl causing the labels to mismatch and the service to not forward traffic to the pods.
helm template .
# Source: vespa/templates/service.yaml
apiVersion: v1
kind: Service
metadata:
  name: document-index-service
...
  selector:
    app.kubernetes.io/name: vespa
    app.kubernetes.io/instance: release-name

# Source: vespa/templates/statefulset.yaml
apiVersion: apps/v1
kind: StatefulSet
metadata:
  name: vespa
  labels:
    app: vespa

Thanks for making this helm chart by the way!

thankyou for making these changes. was the helm chart locally test post the changes?

understood the point and tested the changes. merging.


AddInMemoryUsers inconsistent (List vs IEnumerable)

AddInMemoryUsers takes a List whereas others take IEnumerable.
It's argued that input args should be of the least specific type.
CLA (thinktecture.IdentityServer) signed
PR to follow

In IdSvr3 it was a list because it needed to be mutable. I don't know about IdSvr4 yet if we still need it that way.

Hmmm, ok...
It looks like the LoginService (in Host) does need to add to the collection (shouldn't it be InMemoryLoginService?). But the services in IS4 do not (at the moment)
I bumped into this because I tried to use a different collection type. List can be a little constraining.
It looks like the direction is for IS4 to be "readonly" in relation to the "stores". It's another parts responsibility to maintain the stores.
It could be argued that AddInMemoryUsers should take IEnumerable and LoginService should take ICollection. It's then up to the dev to pass the right instance into each to achieve the desired effect.

I abstracted the methods from LoginController using interfaces. One for Cookie Handling and one for in memory / database operations. It uses IEnumerable. What I haven't figured out is the best way to implement AutoProvisionUser using AspIdentity as part of my solution. Any suggestions? IUserService

things always look simple when you first start. Then the detail devil sticks its head up.
Using the interfaces is "better" as it better expresses the needs of the dependent class. But, in this case, the same collection instance needs to be used by two dependents with different requirements.
So you either

Just use List (or ICollection) and accept that some classes (ie InMemoryProfileService) take a dependency that is over featured
register the collection twice (IEnumerable and ICollection) but this may seem a little weird/awkward.

However, you could also argue that this shouldn't be a concern of IS4 at all. It only needs abstractions like IProfileService. This only seems like an issue because there are implementations like InMemoryXYZ within IS4.
Maybe the "solution" is to remove the InMemory implementations from IS4 and boil it down to its essentials. Would you ever use InMemory for any reason other than testing? Maybe they belong in the Host project which is a demo/sample/debugging construct. Maybe in a separate proj, "IndentiServer4.InMemory". Then it would sit as a peer with IndentityServer4.EF/TableStorage/DocumentDB/whatever.
(not sure what to think about the DefaultXYZ implementations)
So, from the library developers pov, you're just saying "here's what the library needs" (ie IProfileService and friends). Implementations of these services are a different concern (ie storage and admin).
Thoughts?

See also #179.
I'm not sure, but InMemory may be a valid option for transient data (e.g. something like a RefreshToken), while for persistent data (Clients, Scopes, etc.) it would only be used for testing. So could be good to keep these apart (#208)

Yea, just hold off on this for now, please. This month we're doing a lot of work on IdSvr, and for this particular issue we're not yet ready on a direction or what's needed. What you see now is just a snapshot in time of where we're going.

closing for now. please open a new issue for the RC


Implement collision

Every cell consists of 4 sides. We can subdivide it once to have 4 squares per cell. Now we can drop the center vertex to have just 8 edges.
+-------+    +---+---+    +---+---+
|       |    |   |   |    |       |
|       | -> +---+---+ -> +       +
|       |    |   |   |    |       |
+-------+    +---+---+    +---+---+

Now we're going in the clockwise motion with bits starting from the top left corner.
    T   t
  +---+---+
l |       | R     TtRrBbLl
  +       +     0b00000000
L |       | r
  +---+---+
    b   B

And now the collision checking which is done every tick per entity. So each entity has some x and y velocity which is one of those three values: {-SPEED, 0, SPEED}, where SPEED is a constant.
Each entity is of size exactly the same as one tile and there may be a problem with the entity fitting into the normal door.
There are 2 options:

preprocess the collision map and apply some rules to achieve:
[0b11111111, 0b00000000, 0b11111111] -> [0b10000111, 0b00000000, 0b01111000]
and simply check collision by corners
[x, y] [x + TILE_SIZE, y], [x + TILE_SIZE, y + TILE_SIZE], [x, y + TILE_SIZE]
check collision by corners:

const a = TILE_SIZE // square side length
const r = a / 2 // circle radius
const d = r - r / Math.SQRT2 // r - r * sin(45deg), sketch at the bottom of the comment
const c = a - d

const cornersToCheck = [
  [x + d, y + d],
  [x + c, y + d],
  [x + c, y + c],
  [x + d, y + c]
]

And don't think about preprocessing the map

And I've found a great resource that removes all of the issues with both approaches from the outdated comment.
https://www.youtube.com/watch?v=ZDhPRKWYNKI


You can clip through blocks infront of portals

You can clip through blocks infront of portals if youre fast enough, walking speed is usually enough.
https://user-images.githubusercontent.com/81494543/112929577-c9c8d180-9129-11eb-85df-5945c4ebf4ec.mp4

Probably fixed.


Fix an error in _app.tsx.

Fix an issue in the _app.tsx where it would give this error. This is because the curly braces aren't needed, it is already getting it from "next/app", so you wouldn't need the curly braces anymore.
Before changes:

After changes:

The base app is created using create-next-app, see _app file here: https://github.com/vercel/next.js/blob/canary/packages/create-next-app/templates/typescript/pages/_app.tsx
I usually do:
import type { AppType } from "next/dist/shared/lib/utils";

const MyApp: AppType = ({ Component, PageProps }) => { ... }
export default MyApp

and this is how we have in all other _app.tsx so maybe migrate that pattern here too?

Ohhh wait I get what you said now.

Alright I made the PR to the main branch.


[HELP] Auth permission denied

Describe the bug
I set up firebase and Github Oauth, but authentication fails because of permission denied.
How can I fix this issue?
To Reproduce
Steps to reproduce the behavior:

Go to homepage
Click on Github
See error: PERMISSION DENIED

Expected behavior
Login successfully.
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

OS: macOS 10.15.3
Browser Chrome
Version 80.0.3987.122

Additional context
Add any other context about the problem here.

Are you running the repo locally or are you on the live site https://postwoman.io? More info is appreciated @mzch thanks!

Sorry, I'm running locally, https://postfellow.com

I see you are running MacOS and I can confirm there is an issue when I just tried to login on a Mac w/ Chrome. (I was logging in with 2Factor auth as well; Not sure if that matters)

@liyasthomas @NBTX @jamesgeorge007 @AndrewBastin

I tried to login via Postwoman (Postwoman is running on Debian 10.3)...

I have already added postfellow.com to authorized domains in the console of Firebase.

@liyasthomas

Auth + sync is only tailored to postwoman.io

@liyasthomas
Is it impossible to auth Google/Github by modifing functions/fb.js?

Oh, when rerunning npm run build, then fixed. Thanks!


"0_0 (: /-\ :) 0-0" is not palindrome or otherwise we should also consider _ and -

Challenge Check for Palindromes has an issue.
User Agent is: Mozilla/5.0 (Macintosh; Intel Mac OS X 10_11_3) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/48.0.2564.116 Safari/537.36.
Please describe how to reproduce this issue, and include links to screenshots if possible.
My code:
function palindrome(str) {
  // Good luck!
  var sanitized = str.replace(/\W+/gi, "" ).toLowerCase();
  return sanitized === sanitized.split("").reverse().join("");
}

console.log(palindrome("0_0 (: /-\ :) 0-0"));

Ok, I should correct the code like this \W|_

But this string contains a escape symbol--"", this string actually just is "0_0 (:/-:) 0-0".
So, the bug is also exists.


Why the Custom Loading Component may not work?

Version
v2.3.4
Reproduction link
https://codesandbox.io/s/github/nuxt/nuxt.js/tree/dev/examples/custom-loading?from-embed
Steps to reproduce
I configured my custom loading component as described in documentation:
  /*
  ** Customize the progress-bar color
  */
  loading<EMAIL_ADDRESS>
and created my new component
<template>

    Loading..

</template>

What is expected ?
My custom loading component showed during page load.
What is actually happening?
Nothing, my custom component and even the default one are never showed up.
Additional comments?
Related issued closed but not solved:
#1454
#1463
I notice that even the example provided in the documentation is not working, I just notice a little flicker caused by browser load.
I try to modify the template like this:
<template>

    Loading..

</template>

and I correctly see the component, so the component have the correct dimension/position/z-index.
I try to add some console log output:
export default {
  data: () => ({
    loading: false
  }),
  methods: {
    start() {
      console.log('loading..')
      this.loading = true
    },
    finish() {
      console.log('loaded!')
      this.loading = false
    }
  }
}

and I never see that output inside the console, so those methods are never called.

This bug report is available on Nuxt community (#c8458)

mounted() {
this.$nextTick(() => {
this.$nuxt.$loading.start()
setTimeout(() => this.$nuxt.$loading.finish(), 2000)
})
}

it does not work for me !!


Update dependencies

This updates all dependencies to latest versions as of February 8th, 2023.

Thanks @jrdalenberg! Just tested your updated version.  Runs great.

@SiggiSmara thanks for testing!

@jrdalenberg Thanks for this! Everything still seems to run great. Always disheartening when the tutorial breaks after you do exactly what it says.

Thanks for the PR, @jrdalenberg. Since the code is tied to a tutorial, they both need to be updated at the same time. Doing that now in https://github.com/testdrivenio/fastapi-celery/pull/13.


Support Incremental Backups

Feature Description
It would be great if there was a way to support incremental backups so you aren't constantly uploading a giant zip to Amazon S3 every day.
Backing up only changes and then once a month doing a full backup would be ideal.
Screenshots
No response

Probably for some data incremental backup could be doable, for example backing up only changed repository git data. But for some data (like db and some other assets, not so easy)

A possible method: do regular backup and use rsnapshot to get incremental data and upload it to S3, then you can control the whole backup progress.
Incremental backup is too complex and I am afraid it's not suitable to be a built-in feature for Gitea (neither for most applications)

Closing per @wxiaoguang comment. This is what I would suggest as well.


✨ Add CAPI patch to support in-place upgrade in KCP

Description of changes:
This PR adds support for InPlace upgrade strategy in KCP. When the deployment strategy is set to InPlace, the kubeadm-control-plane-controller adds a controlplane.clusters.x-k8s.io/in-place-upgrade-needed annotation to the KCP object when an upgrade is needed. The external upgrader should watch for this annotation and trigger an in-place upgrade when it's added.
After the upgrade is done, the external upgrader is expected to update the statuses and remove this annotation.
Corresponding PR on our CAPI fork https://github.com/abhay-krishna/cluster-api/pull/27
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

/unhold

/approve

/lgtm


[ESP32] Fixed server initialisation order

Problem
What is being fixed?  Examples:

LogError E (1624) chip[ZCL]: GeneralDiagnostics: Failed to record BootReason event: 3.
On bootup matter plugins is being initialised before EventManagementState changes from ShutDown to Idel.

Change overview
Moved InitDataModelHandler after  EventManagement initialisation.
Testing
How was this tested? (at least one bullet point required)

Manually tested commissioning and cluster control

PTAL @bzbarsky-apple @tcarmelveilleux


Masters- Navbar

Navbar for the app, separates the real life tourny and fantast tourny

Needs update to be an offcanvas navbar


FloatField should accept comma as decimal separator

just convert "," to "." on assign. half of the world uses comma.

I think this should be done on some localization layer instead.
Workaround:
number = FloatField('Number', filters=[lambda x: x.replace(',', '.')])

As @kblomqvist pointed out this can be handled using filters. I would be hesitant to include something like this in the library as it depends on locale and using a comma as a separator isn't technically a valid floating point number by most definitions.

using 2.2.1 this workaround doesnt seem to apply for float fields or decimal fields. perhaps it's trying to cast the input as a float / decimal before applying the filter?

Could it be that processform runs before the filter operation? Using version 2.2.1 I cannot get the filter option above to work in either DecimalField or FloatField


Begging to share review code

Hello, I am honored to follow your recent excellent work on talking face in ICCV2023: "Efficient Region-Aware Neural Radiance Fields for High-Fidelity Talking Portrait Synthesis"
I don't know if it is convenient, can you share your metric evaluation code? Greatful! Anyway, thanks a lot!

+1


Sentry error SMOOTHIE-37V with rive

https://openfoodfacts.sentry.io/issues/4673526817/?notification_uuid=6056c55f-69b7-4871-8b0f-f27443f8f7c0&project=5376745&referrer=weekly_report
Could not find an elegant way to copy the sentry issue in github.

There is a dedicated button for that 😉

Opened https://github.com/openfoodfacts/smooth-app/issues/4929

Thank you @M123-dev, now I know where I have to click - on a 5-inch screen it takes a lot of vertical scrolling ;)


[Update]: Update Sieve exercise scripts

This PR updates the Sieve exercise scripts to match the other exercises

That raindrops commit shouldn't be there


Move control byte to separate array. Add functions to modify dynamically

Per issue #821, control flags have been moved from the map to a separate array for the button matrix and functions to modify the flags have been implemented.

Looks good, thank you very much!
I'm missing only one thing. It should be a lv_btnm_set_ctrl_map(btnm, ctrl_map) function. It would make possible to decale the the whole layout in compile time. For example, it is used by the keyboard obejct

That seems like a  good step on the road to removing the control chars from the string map.
The control bits can still currently be defined at compile time as they always have by using the control chars, but I agree that having a lv_btnm_set_ctrl_map(btnm, ctrl_map) function helps move things towards removing them completely.
One potential issue is that the current lv_btnm_set_map function was built with the expectation of knowing the control bits when it is called (since they are in the map). Specifically the relative widths.
Calling lv_btnm_set_map and then calling lv_btnm_set_ctrl_map necessitates calling  lv_btnm_set_map again (presumably from lv_btnm_set_ctrl_map). lv_btnm_set_map is also currently called from lv_btnm_create with the default map.
lv_btnm_create
    lv_btnm_set_map // with default map
lv_btnm_set_map // with new map
lv_btnm_set_ctrl_map
    lv_btnm_set_map // with new map

So lv_btnm_set_map is called a minimum of 3 times, with the full layout performed each time. I'm not sure exactly how expensive it is, but it seems a bit wasteful.
I'm not sure I have any solid ideas on how to avoid this with the code as it currently stands, although I would definitely suggest suggest changing lv_btnm_def_map to perhaps a single button to reduce the amount of time wasted laying out a map that won't be used.
A lv_btnm_set_map(btnm, map, ctrl_map) function would avoid the second call and is closer to the original idea of having the control bits in the map, but I'm not sure if it's in line with the current API which seems to restrict all set functions to setting a single parameter.
So I guess the choice is lv_btnm_set_ctrl_map(btnm, ctrl_map), lv_btnm_set_map(btnm, map, ctrl_map), or both.

@canardos

I would definitely suggest changing lv_btnm_def_map to perhaps a single button to reduce the amount of time wasted laying out a map that won't be used.

I think one good reason for having a default map is so that the user can get a feel for what the widget will look like on their screen. It also allows them to quickly experiment with styles, etc. without having to create a button map first.

I agree that having a default map seems fine the reasons you mentioned. I was just suggesting that loading, parsing, allocating memory for it etc. prior to loading/parsing the real map for every control created seems a bit wasteful.
I would have expected it to be defined in lv_btnm.h to be used like lv_btnm_set_map(btnm, lv_btnm_sample_map) for example.

I agree that having a default map seems fine the reasons you mentioned. I was just suggesting that loading, parsing, allocating memory for it etc. prior to loading/parsing the real map for every control created seems a bit wasteful.
I would have expected it to be defined in lv_btnm.h to be used like lv_btnm_set_map(btnm, lv_btnm_sample_map) for example.

Fair enough. The only issue with that is that the user would have to find the sample map themselves, instead of it being automatically set up for them.

@canardos Applying the map several times is really wasteful and can have cause some performance loss. However, nobody complained about "slow object creation" so far so I think we can leave the default map. As @embeddedt has highlighted it's useful for the users to see how the object looks like without doing anything.
With your update, we have a nice API to control the buttons one-by-one. I assume most of the people will use it because it's very clear and readable. So the last parameter of lv_btnm_set_map(btnm, map, ctrl_map) might be used rarely. Therefore I vote for having a lv_btnm_set_ctrl_map(btnm, ctrl_map) which can be used in some special cases if required.

@kisvegabor No problem. I'll implement lv_btnm_set_ctrl_map(btnm, ctrl_map) in the next few days and update the PR.

Note that there are still a couple of issues to resolve post the above commits. Will let you know when it's ready for review.

@kisvegabor I think that should be all good. Please take a look.

As there is a plan to update lv_btm (https://github.com/littlevgl/lvgl/issues/901) I merge it now.
@canardos
Please clarify the comment above.
Thank you very much for this update! It made lv_btnm more flexible and will help a lot to add new features.

As there is a plan to update lv_btm (#901) I merge it now.
@canardos
Please clarify the comment above.
Thank you very much for this update! It made lv_btnm more flexible and will help a lot to add new features.
@kisvegabor Thanks for reviewing and merging. Which comment did you want clarified?

@canardos
I commented on the commit but it seems people don't get notified if someone comments on their commit. Nexttime  I'll use the "mentioning".
So I ment:
Is this comment correct?
https://github.com/littlevgl/lvgl/blob/688cd79a550e058aee0c9ad95cf5043bb7575887/lv_objx/lv_btnm.c#L272-L287
It says "one element for each button AND new line".

@kisvegabor  Ah, missed that one. Per my earlier reply, the array size/index is equal to that of the buttons and there are no entries for newlines. You'll see in commit https://github.com/littlevgl/lvgl/pull/825/commits/688cd79a550e058aee0c9ad95cf5043bb7575887 that I updated the comments in all of the other functions to reflect this, but it seems I missed that one. It should read:
  * @param ctrl_map pointer to an array of `lv_btn_ctrl_t` control bytes. The
  *                 length of the array and position of the elements must match
  *                 the number and order of the individual buttons (i.e. excludes
  *                 newline entries).
  *                 The control bits are:
  *                 - bit 5   : 1 = inactive (disabled)
  *                 - bit 4   : 1 = no repeat (on long press)
  *                 - bit 3   : 1 = hidden
  *                 - bit 2..0: Relative width compared to the buttons in the
  *                             same row. [1..7]

Thank you! I'll update it accordingly.


Adding Memair to Official APIs list

Adding Memair (a Quantified self / extended mind platform) to Official APIs list

@gregology Thanks for PR 👍 Merged 🎉
P.S. It's sad that you need to login in order to play with GraphiQL 😞

Yes, sorry about that. All the end points are scoped to the the user so to
avoid frustration I've made everyone log in. I could add some other generic
non scoped endpoints. Is there an established pattern for instances like
this? I appreciate your input.
On Mon, Jul 23, 2018, 08:16 Ivan Goncharov<EMAIL_ADDRESS>wrote:

@gregology https://github.com/gregology Thanks for PR 👍 Merged 🎉
P.S. It's sad that you need to login in order to play with GraphiQL 😞
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/APIs-guru/graphql-apis/pull/22#issuecomment-407037977,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABhYOONAoQHDX7bLB2A4or6ce-35__q9ks5uJb60gaJpZM4VZ9T6
.

On Jul 23, 2018 08:16, "Ivan Goncharov"<EMAIL_ADDRESS>wrote:
@gregology https://github.com/gregology Thanks for PR 👍 Merged 🎉
P.S. It's sad that you need to login in order to play with GraphiQL 😞
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/APIs-guru/graphql-apis/pull/22#issuecomment-407037977,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABhYOONAoQHDX7bLB2A4or6ce-35__q9ks5uJb60gaJpZM4VZ9T6
.

@gregology Yes you're right, it totally makes sense for your domain.
Maybe you can't support GitHub or Google login button.

👍 That's a great idea @IvanGoncharov, I'll implement GitHub and Google logins. Thanks for the suggestion

Thanks again for the suggestion, I've added Google and GitHub login options so it will be 2 click access to Memair's GraphiQL


Enhance the "Model" Section

The declaration of classes in model is incomplete. We need to provide a comprehensive list before releasing the protocol.

@sebbader-sap please add more details

e.g. Dataspace Registry is not explained in text

I will check the the terminology for data space authority and its registry and add a note that this is not covered in this spec

registry is in the first figure

Split section 2.1 in non normative first part and normative part starting with the second figure

AssetEntry needs to be removed from figure 2


Pick Dark/Light mode based on light sensor

To really crank up the laziness of devs have the modes be determined by the ambient light.
From a quick Google I found
https://stackoverflow.com/questions/17625495/how-do-you-programmatically-access-the-ambient-light-sensor-on-mac-os-x-10-5
But thanks to Apple that may or may not work.

Interesting idea, but I'm not sure if the implementation could be practical enough that people would actually want to use it.

Depending on how good the sensor data is, I could see dark mode flipping on and off annoyingly based on fluctuations in light level
I could see things getting pretty weird if this feature tries to interact with the system preference to activate dark mode at sundown

I would love to try this out. I'm working outside a lot. I think there's enough sensor information to see that it's quite bright out here... You could also set a threshold, like, it has to be very bright for more than a minute, before it switches to light mode.
Anyway, a shortcut is already great, but I think this would be a good addition!

It will be activate in a percentage range (customizable) of darkness

Closed as not planned


Cannot use full features of handsontable with the component

I found that I cannot use the full features of handsontable.
The component just references handsontable.js, but not handsontable.full.js.
Is there any way to use the full features?

Hi @popo1221
all needed dependencies are downloaded each time you run npm i and are build to the end file. Do you get any errors or are not able to use some functionalities?

Hi @AMBudnik
My issue is as same as https://github.com/handsontable/react-handsontable/issues/51.
Thanks.

Thank you for connecting the issues.


Component wont transpile during build

the shortand functions inside the .js file imported as a mixin (BaseModal.js):
es. data()
wont be transpiled ( in data: function()) when i build the app with webpack
this gives error on ie 11 or lower

Have you traced to where the issue is ? I had similar issue when using webpacker in Rails. Babel plugin wouldn't transpile.

I'm having the same problem using Laravel Mix.

sis you checked your Webpack.base.conf ?   is the babel-loader excluding /node_modules from compiling.?  in which case , add the node_module/vue-bulma-modal file path into the include...


Unable to cast object of type 'generatedProxy_3' to type 'Microsoft.PowerPlatform.Dataverse.Client.IOrganizationServiceAsync'

Using Microsoft.PowerPlatform.Dataverse.Client 0.5.10
I got an Azure function with a Blob trigger that parses the blobs and writes to Dynamics 365 CE. Was working fine until the frequency of Blobs uploaded increased drastically. First I was seeing the error

Unable to cast object of type 'generatedProxy_2' to type 'Microsoft.PowerPlatform.Dataverse.Client.IOrganizationServiceAsync'

I read through #48 and added

RequireNewInstance=true

to the connectionString.
Now it looks like more Blobs get processed but the majority now produces the very similar exception

Unable to cast object of type 'generatedProxy_3' to type 'Microsoft.PowerPlatform.Dataverse.Client.IOrganizationServiceAsync'

Setting UseWebApi = false seemingly solved the issue,

@Dashulol can you link a repro of your use case?  that would help us understand what is happening as it sounds like a threading issue.

We have refactored some of the client code in this upcoming release to see if we can lock this down. or at least give us a better understanding of where the type is getting lost.
should be in the next update.

I still occasionally got the error "Unable to cast object of type 'generatedProxy_2' to type 'Microsoft.PowerPlatform.Dataverse.Client.IOrganizationServiceAsync'", The version of Microsoft.powerplatform.dataverse.client is 0.6.1


Port over SQL reports from Git to .sql

A new branch needs to be created with a .sql file with your commands.
e.g
#reportID xx
SELECT * FROM x

Reviewed with team, merge complete with updated SQL files on Feb 19/22.


Update Team Name instructions

Vercel API expects Team ID (shown in Team URL field), not Team Name.

Thanks @gpxl! How do you feel about using the Team ID instead of the Team URL slug? That seems to be the primary identifier for Vercel API calls now, and feels more secure.


Lower the priority of looking up the rank of scope

In the previous change of #5060, we propose a way to resolve the ambiguous call when considering the scope of a function.
But this rule should be considered as a low priority than "specialized candidate", aka. we should consider more "specialized candiate" first.

In the latest change, I update the algorithm to compare the candidates by calculating distance
from reference site to declaration site via nearest common prefix
in the scope tree.
This will involve finding the common parent node of two child nodes
and how sum the distance from the common parent to the two child nodes.

@csyonghe, this is not the same as we discussed offline. So can you please look at that to see if would agree with the rule like this?

Again, we hit the issue that we should not put this logic here.
When looking up for the generic functions, it doesn't gaurantee to provide the valid candidates to function CompareOverloadCandidates, so it breaks the assumption again.
To be more specific, I can provide the repro shader here:
namespace A
{
    struct HPackedArray<let SIZE : uint>
    {
        uint data[SIZE];
    };

    static HPackedArray<Z1> mad<let Z0 : uint, let Z1 : uint>(
       HPackedArray<Z0> ip)
    {
        HPackedArray<Z1> res;
        return res;
    }
};

namespace A
{
    struct HCoopVector<let SIZE : uint>
    {
        HPackedArray<SIZE> vec_simt;
    }

    static HCoopVector<Z1> mad<let Z0 : uint, let Z1 : uint>(
        HCoopVector<Z0> ip)
    {
        HCoopVector<Z1> res;
        res.vec_simt = mad<Z0, Z1>(ip.vec_simt);  // it will fails here, it should choose above "mad"
         return res;
    }
};

In previous implementation, CompareOverloadCandidates won't find the best match, so it will leave the job to the following logic, while this PR gives the wrong answer because we assume both candidates are valid.

followup PR to issue #4476.


Font size and customization

Hello
Can you explain how to customize nteract, which file, where is it located etc. I have a big screen and font seems to be very big. I tried to search but there is no clue.
Thanks

Is this for the nteract desktop app or hydrogen the atom plugin?

Thank you for response. I noticed that I wrote to Hydrogen issues. I asked for desktop application.

The browser style zoom-in/zoom-out is available (⌘++, ⌘+-) for basic needs. There isn't configurable zoom that will persist across sessions though.
Feel free to open an issue on nteract/nteract for this feature request. If you've got a common pattern that you like for this UX, please detail it. At least for us, it could be as simple as us tracking the last zoom level in our config.


Changed definitions of overridden functions to match the original

In https://github.com/singlestore-labs/singlestoredb-laravel-driver/pull/46/ I overrode 4 functions of the MySqlGrammar and added a return type to the definition.
Occurred that this is a breaking change as all users that extended from our class and overridden the same functions - also should add the return type.

I'll land and ship this now. But in parallel is it possible to add some tests to ensure we don't break this in the future?


Attempt non-blocking io

Rather than using sockets, this pr implements a small C module wrapping around poll(2) to check if stdin has any data. This is preferential to sockets since a suprising amount of clients don't actually support using a socket and only use stdio.
Additionally this implements a very rough outline of an event-loop/async work-queue where documents can cancel type checking on themselves if they've been updated before the check completes. This gives a massive speed boost to textDocument/didChange so I've also reenabled it. (Previously we only checked on textDocument/didSave since typechecking is quite expensive)
closes #3
also a windows implementation will have to be figured out since poll(2) is a unix thing

I'd like to use this, but am unclear on how poll.c is being compiled.  How is the tealls.poll module being added to lua here?

luarocks make should build the c module and copy the already built lua files to the correct place (to do an in tree build, use luarocks init to make a local ./luarocks that works on a local ./lua_modules)
A couple notes: I've been pretty busy and haven't looked at this for a bit. So keep in mind this is a pretty rough draft, and if I recall correctly the event loop is pretty eager to just spin the cpu.

Thanks, I have it building and running now  :)
I was getting errors of the form:
no type information for required module: 'dkjson'
no type information for required module: 'lfs'

But was able to avoid these by adding dkjson.d.tl and lfs.d.tl files to the project.  Do you have a better way of providing type definitions for dependencies?

You can find type definitions for some lua modules in the teal-types repo here. We don't have a great way of distributing them so for now you'll just have to manually download them and add them somewhere to your LUA_PATH

Just want to mention that for some reason I am finding that some RPCs are not received when using this MR.  I can clearly see in the neovim LSP log that they are being sent, but they do not appear in the teal-language-server log anywhere

Ok figured out my issue!  Turns out you need to explicitly set stdin and stdout to be unbuffered like this:
io.stdin:setvbuf  'no'
io.stdout:setvbuf 'no'

Otherwise, poll.c will think that there is no data available when there is


[TF-TRT] Base Test Split to avoid OOM on GPUs with limited memory

This PR aims to fix some OOM issues that may occur specifically during unittesting.
Consequently splitting a unittest in 3.
Depends on #56497

@bixia1 for review

@bixia1 all comments have been addressed. Please review

this is currently waiting on https://github.com/tensorflow/tensorflow/pull/56497

Hi @DEKHTIARJonathan Can you please resolve conflicts? Thank you!

Hi @DEKHTIARJonathan Can you please resolve conflicts? Thank you!

Hi @DEKHTIARJonathan Can you please resolve conflicts? Thank you!

Hi @DEKHTIARJonathan Can you please resolve conflicts? Thank you!

Rebased on latest: #56497

Hi @DEKHTIARJonathan Can you please resolve conflicts? Thank you!

Hi @DEKHTIARJonathan Any update on this PR? Please. Thank you!

Hi @DEKHTIARJonathan Any update on this PR? Please. Thank you!

Hi @DEKHTIARJonathan Any update on this PR? Please. Thank you!

Hi @DEKHTIARJonathan Any update on this PR? Please. Thank you!


fix: floating window lead causes no problems

This PR is to be primarily understood as a suggestion for solving issue #37.
It aims at making loading a session in presence of a floating windows more robust by additionally closing the last window left over from the current session after splitting a fresh one off of it in _init.lua_s close_everything's function. This gives a clean slate to load the desired session from.

Decided to go with 31938d818f11924da712918cb066937c557ee741 as the fix


No Ca in kubeconfig file from UI 2.3.1

What kind of request is this (question/bug/enhancement/feature request):
After creation of AKS Cluster, when we download kubeconfig, in the config file is not present the certificate-authority-data
Steps to reproduce (least amount of steps as possible):
Click on button kubeconfig file
Result:
we can't execute any command with kubectl, we receive:
Unable to connect to the server: x509: certificate signed by unknown authority
Other details that may be helpful:
In the cluster page we don't see in green the "Controll MAnager" and "scheduler"; they are both in grey.
Environment information

Rancher version (rancher/rancher/rancher/server image tag or shown bottom left in the UI):
Installation option (single install/HA):

Rancher 2.3.1 HA
We use our certificate signed by Recognized CA

Cluster information

Cluster type (Hosted/Infrastructure Provider/Custom/Imported):
Machine type (cloud/VM/metal) and specifications (CPU/memory):
Kubernetes version (use kubectl version):

Cluster AKS
Kubernetes version --> 1.14.7
VM --> 8 core / 64 gb RAM for 3 nodes

Docker version (use docker version):

(paste the output here)

The kubeconfig that is generated includes the CA cert that is configured when setting up certificates for Rancher, as it connects to Rancher. When you specify that you are using a recognized CA, you don't need the CA cert as it should already be present on the device (as its recognized). What CA is the certificate from that you are using? Usually this means that you didn't supply the needed intermediate certificate from the CA.
The status for controller manager and scheduler are hidden for AKS as they don't work (and componentstatuses get deprecated at some point), see https://github.com/rancher/rancher/issues/17344

Hi,
we create the secret for the Rancher ingress with this command:
kubectl -n cattle-system create secret tls tls-rancher-ingress   --cert=tls.crt   --key=tls.key
we don't specify any ca.
Do we have to specify the ca? Which is the parameter? In the ca file , do we put the intermediate and the ca file?
Thanks,
Alessio

I have re-created the secret for the ingress of rancher adding in one file with cert, intermediate and ca.
After that, the kubectl command works fine.
Just one question to close the issue, AKS Api Server doesn't support a CA ?

Great. What does AKS Api Server doesn't support a CA mean? What are you trying to do or what have you read that doesn't work?


Weekly Notes confusing week number

Its Sat Feb 19. When I create a weekly note:

The note file name is W06
The Calendar highlights it on last week which is W07
This week in the calendar is actually W08

This is my note name:
YYYY/[Weekly]/gggg-[W]WW

From what I can see periodic notes plugin gets confused only when calendar plugin is also installed (or enabled). Was noticing this problem as well. Everything went away after removal of calendar plugin.
Maybe fiddle with locales in the calendar plugin to setup first day of week.

I tried Locale default - Sunday, Sunday, and Monday but got the same results.

@mchwalisz @liamcain any ideas?

I removed the calendar plugin and it started to work as expected. I, however, understand this is not really solution you are looking for.
Maybe one should also raise awareness in the calendar plugin. One needs to debug where the behavior change is coming from. I don't have time and expertise to look into it.

It seems like the the en-us locale is not using ISO week numbers. It works for me when I override the locale to en-ie for Ireland.
Quick unrelated question: What do I need to do to link the previous and next weekly note in my template? There's nothing in the docs about that. Thanks!

I rewrite your code to work even when creating a weekly note for a future or past week:
[[<% tp.date.weekday("gggg-[W]WW", -7, tp.file.title, "gggg-[W]WW") %>]] <== This Week ==> [[<% tp.date.weekday("gggg-[W]WW", 7, tp.file.title, "gggg-[W]WW") %>]]

Requires that the note is named accordingly of course, e.g. 2022-W15.

This issue has been discussed in several issues in obsidian-calendar, ie. in liamcain/obsidian-calendar-plugin#222
Thanks for the Templater template, @stefandesu. That's really helpful, it should be included in the official documentation.


numerical instability in MDF make_j3c

Running the test pbc/df/test/test_mdf_builder.py with multiple threads can fail. The failing tests are those involving MDF make_j3c with aosym='s1'. It seems to me that some code may be not thread-safe.

Has PR #1622 fixed this?

Please feel free to reopen.


Frontend-Project "SIGN UP FORM" Form Design Improvement

Describe the Feature you want to end
The design of the form is nice but it can be improved by making some design adjustments. Through some styling changes, I can make it look even better which will effect overall website.
Expected behavior
Attractive form design
Additional context (optional )
Styling on the form

Hello i want to work in this project. Can you please assign me recently i make the login form hope you like this.


(maint) Pin to a released beaker version

Previously we were tracking on the master branch of beaker, which makes
it hard to know which version of beaker was compatible with puppet 4
when it was released. For example, as we continue to support puppet
4.0.z releases while beaker development moves forward.
This commit pins us to beaker ~> 2.8, which includes 2.8.1 and 2.9.z
releases, but not 3.0.

/cc @kylog @johnduarte @haus

:+1: I'm in.

Since only beaker 2.8.0 has been released, this will cause puppet to test against an earlier beaker version: https://github.com/puppetlabs/beaker/compare/beaker2.8.0...master.
However, the new 2.8.1 beaker is supposed to be tagged end of day tomorrow (where master currently is), so worst case, we'll be back where we started tomorrow.

Also I sanity tested rhel7 and win2012r2, made sure the presuite passed and a handful of with_puppet_running_on acceptance tests passed, and agent commands, e.g. puppet agent -td, etc

+1 for me.


remove VNFD from NSD

It seems that the dashboard offers the possibility to remove the VNFD from the NSD, however when trying to do that, I get an error message: undefined. Error code: 422
I assume is a bug. no records was active using that descriptor.

Can we close this? @marcellom @gc4rella

depends if you want to merge #239


If .bom file name differs from branch/tag name, not file found exception is thrown

Caught: : Warning: Could not find file /home/mvalle/buildoop/recipes/openbus-v1.0-dev/openbus-v1.0-dev.bom to copy.

Closed due not buildoop bug


Fix dependent option sets

Motivation for this change
As I learned from a report from @arcnmx, 15c873b486347e7861c64fb0b5a7852be9fc82e4 from https://github.com/NixOS/nixpkgs/pull/82751 gives infinite recursion for option-dependent configuration with optionalAttrs, but only with nested option sets. https://github.com/NixOS/nixpkgs/pull/82802 only added tests for the non-nested case, which is why currently all tests still pass.
In this PR I revert the problematic commit and extend @roberth's tests to check for nested options as well, which would fail if the commit wasn't reverted.
Things done

[x] Ran ./modules.sh successfully

Can confirm my nixos config evaluates again with this PR, thanks!

Thanks!


Support 'hostname' and 'ip' when communicating with ingressgateway.

When E2E tests talk to the revisions under test, they use the knative-ingressgateway to talk to them. Given there's no resolvable domain, the IP of said ingressgateway is taken and the hostname of the revision to talk to is spoofed in via the Host header. Under some circumstances the ingressgateway loadbalancer does not get an IP but a hostname though (namely when using Docker for Mac's Kubernetes support).
Falling back to the hostname value if no IP is set supports this use-case.

How does this change relate to usage of the resolvable flag? It looks like the case this covers is when the ingress does not get assigned a specific IP but instead assigned a resolvable hostname.
Is there an example of where this logic would be useful but setting the resolvable flag would not? Or, does this change render the resolvable flag no longer needed?

@bbrowning I might be mistaken here but I think the resolvable hostname needs to be the hostname of the service itself whereas I'm technically spoofing a host to another host. I could make my test-setup bind against "localhost" I suppose, I haven't actually tried that to be honest.
The curl needed for my service is something like: curl -H "Host: something.example.com" localhost

/approve
/lgtm


Add retention period to prometheus

By default, the retention will be set to 7 days (168h).
A new command line will allow to override with the -r command line flag.
Fixes #124
Signed-off-by: Amnon Heiman<EMAIL_ADDRESS>

then close the original issue #124
On Mon, Feb 5, 2018 at 11:47 AM, Tzach Livyatan<EMAIL_ADDRESS>wrote:

@amnonh https://github.com/amnonh one can already set the retention with
-b storage.local.retention=1000h
Its already documented in the README here
https://github.com/scylladb/scylla-grafana-monitoring/blob/master/README.md#prometheus-command-line-options
I do not see much value in yet one more option to start_all. There are
many Prometheus and Grafana options, rewriting each would be redundant
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/scylladb/scylla-grafana-monitoring/pull/278#issuecomment-363032471,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ACBRv--ugBFV6RLwn74dmTirKGWUwbV1ks5tRs4ygaJpZM4R5KME
.

closing as retention option already avilable


let homebridge control a bluetooth projector, streamer?

hi,
Is it somehow possible to let homebridge turn of my (xiaomi mi laser projecter with mi tv) streamer via a bluetooth turn off command?
I see a lot of questions on the internet on how to use a bluetooth keyboard with a PI, but I want my PI to send a turn off/on, next etc command to media devices.
thanks a lot!

You can connect a device to the projector as a bluetooth keyboard. For instance I've achieved this using the logitech harmony hub and connected the projector as a "Mac" computer profile. I've then found using the sleep keyboard button it will activate the power off, restart window on the Mi Laser Projector - then you need an 'enter' command to turn it off.


1565 winfroms events

What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)
Add ReactiveUI.Events.Winforms, Builds on #1352
What is the current behavior? (You can also link to an open issue here)
ReactiveUI.Events.Winforms doesn't exist in solution. #1565
What is the new behavior (if this is a feature change)?
Adds a new package to the build ReactiveUI.Events.WinForms
What might this PR break?
N/A
Please check if the PR fulfills these requirements

[ ] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been added / updated (for bug fixes / features)

Other information:

Closing this PR as #1640 supercedes it


Deleting a Post always deletes the item of the first clicked delete button in the list

Laravel Version: 5.3
Voyager Version: 0.10.13
PHP Version: 5.6

The JS in the browse.blade template replaces the __id string with the id of the item of the clicked delete button, if I click cancel, the delete_form action still contains the id of that item even if I click the delete button of an another item, the delete_form is still containing the old id and its action is not updated with the new id.

I'm assigning myself to this :D I'll send the commit later

I added this as a temporary solution :D
$('button:contains("Cancel")').on('click', function () {
        window.location.reload();
});

Good luck.

This was fixed inside /tcg/voyager/resources/views/bread/browse.blade.php but wasn't copied to /resources/views/vendor/voyager/posts/browse.blade.php
Simply replace the click function in posts/browse.blade.php to:
$('td').on('click', '.delete', function (e){
    var form = $('#delete_form')[0];

    form.action = parseActionUrl(form.action, $(this).data('id'));

    $('#delete_modal').modal('show');
});

function parseActionUrl(action, id) {
    return action.match(/\/[0-9]+$/)
        ? action.replace(/([0-9]+$)/, id)
        : action + '/' + id;
}

I won't make a PR since, you have assigned it to yourself.

We are getting rid of those views (posts/* and users/*) in next version, so they use default BREAD views.
And the posts BREAD will probably be moved to a plugin/hook


Fix atari dep

Codecov Report

Merging #1343 (38297d8) into master (e56340f) will not change coverage.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##           master    #1343   +/-   ##
=======================================
  Coverage   82.33%   82.33%
=======================================
  Files          65       65
  Lines        5316     5316
=======================================
  Hits         4377     4377
  Misses        939      939

Impacted Files
Coverage Δ

flax/errors.py
88.13% <ø> (ø)

flax/training/checkpoints.py
98.37% <100.00%> (ø)

Continue to review full report at Codecov.

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


Insufficient account privileges - how to workaround

Is your feature request related to a problem? Please describe.
Trying to deploy this where I'm not allowed to have the following privileges:

microsoft.directory/applications/create
microsoft.directory/servicePrincipals/create

Describe the solution you'd like
I'd like to know which steps could be performed by someone else with such privileges so that the deployment script "make deploy" won't fail. I guess something could require changes in the scripts that I'm willing to perform.
Describe alternatives you've considered
Getting what is missing is, unfortunately, not an option. Asking to run the deployment script to someone else also may be quite hard.
Thanks!

You will need to have these prerequisite fullfilled in order to deploy IA successfuly
Administrative rights on the Azure Subscription
a.	Tenant level: Application Administrator role
b.	Subscription level: User Access Administrator and Contributor Role.
Deployment will create two service principals. One called "infoasst_web_access" which manages access for the web app. The second is "infoasst_mgmt_access"This App Reg and SP is used by the app to make Azure CLI AI Management API calls to get information about the AOAI model deployment like model name, version, etc.
If you don't have permission to deploy, I would suggest to either 1) get above permission highlighted in bold or 2) have someone else deploy who has above permissions.
There is no other workaround.

Thank you for your response.
What I meant with "workaround" is, would it be possible to ask someone which has such privileges to create the two service principals separately, according to specs, and then run the script pointing to those?

Thank you for your response. What I meant with "workaround" is, would it be possible to ask someone which has such privileges to create the two service principals separately, according to specs, and then run the script pointing to those?

Not individual pieces but someone who has privileges can follow the deployment process. It is a straight forward deployment from start to end with instructions step by step and it takes about 30-40 minutes. At the end of it you will hjave all your resooiurces, application deployed with all of them working together.

I have provide a potential workaround solution in discussion https://github.com/microsoft/PubSec-Info-Assistant/discussions/457. Please give it a try and let us know if it unblocks you.

This issue is marked for closure due to inactivity for 2 weeks. It will be closed in 5 days.

Closing due to inactivity.


[nnx] pure API

What does this PR do?
Docs: preview.
nnx.split now returns a Pure NamedTuple when a single output state is returned (when either zero or one filters are passed). Pure can call any methods from the underlying node type but returns a tuple with output and a new pure instance with the new state.
from flax import nnx
import jax
import jax.numpy as jnp

class StatefulLinear(nnx.Module):
  def __init__(self, din, dout, rngs):
    self.w = nnx.Param(jax.random.uniform(rngs(), (din, dout)))
    self.b = nnx.Param(jnp.zeros((dout,)))
    self.count = nnx.Variable(jnp.array(0, dtype=jnp.uint32))

  def __call__(self, x):
    self.count.value += 1
    return x @ self.w + self.b

linear = StatefulLinear(3, 2, nnx.Rngs(0))
pure_linear = nnx.split(linear)

@jax.jit
def forward(x, pure_linear):
  y, pure_linear = pure_linear.call(x)
  return y, pure_linear

x = jnp.ones((1, 3))
y, pure_linear = forward(x, pure_linear)
y, pure_linear = forward(x, pure_linear)

linear = nnx.merge(*pure_linear)
assert linear.count.value == 2

Hey @cgarciae ! I'd love this to be merged because I'd like to play with it... Is there any reason this is being left behind?

Hey @PhilipVinc, I was wondering if there was a way to avoid increasing the API surface. If you check the updated notes this new version, Pure is now a NamedTuple and nnx.split now returns Pure instances when no filters are passed.


revert changes for a single file doesnt work on private repos

in a private repos I cannot 'revert changes of a single file'. I am getting a 'Revert failed - see console for details' message and the following error logs:

after adding the 'personal token' in the extensions settings it works as expected.
would be great to have a cleaner error message that I missed to configure the token
using 19.8.20 in google chrome 78

I am aware that ‚some features‘ require the token.
A simple console log error telling that the token is missing would be great for UX

If it's for great UX then we should use a snackbar or popup instead of console messages.
There are already messages telling you that token may be needed. But your screenshot didn't contain the error message for revert-file feature.

Indeed. The token error is already present in the console. We generally expect it to be there and don’t provide extra UI for such errors.


Using vai_q_tensorflow2 API quantize .h5 file error: list index out of range.

We are working on a FPGA project. Our target is to put a Yolov4 .weight file on KV260.
We had already convert our .weight(yolov4) file into .h5 file but now there is a problem.
When we use the vai_q_tensorflow2 API quantize our .h5 file as the following picture:

There is always an error: list index out of range.

We want to know how to solve this problem by using vitis-AI 2.5.

have you got the solution of this problem?

hi @ChuDavid0624 ,
Could you please provide the complete code, '2022_no_mish.h5' model file, and '2022_dataset' for us to better assist you?

Hi @ChuDavid0624,
Thank you for bringing this to our attention. However, due to the lack of detailed information in the initial report and no response to our follow-up questions, it's challenging for us to pinpoint and address the issue you're experiencing.
In order to maintain an organized issue tracker, we will be closing this issue for now. If you can provide more details or answer the previous questions, please feel free to reopen the issue or create a new one with the additional information.
Thank you for understanding.


Use real paths of symlinks when linking castle into home

The problem is the following: one may want to use selected files from external git modules in castle structure. Good example is vim colour schemes. Consider the following structure of the castle repo:
castle
|-- home
|   `-- .vim
|       `-- colors
|            `-- solarized.vim -> ../../modules/vim-solarized/colors/solarized.vim
`-- modules
    `-- colors
        `-- solarized.vim

Currently homesick link of such castle will result in getting the broken links in ~/.vim/colors:
~
`-- .vim
    `-- colors
         `-- solarized.vim -> ../../modules/vim-solarized/colors/solarized.vim

Proposed trivial patch will link original files from castle structure, resulting in following:
~
`-- .vim
    `-- colors
         `-- solarized.vim -> ~/.homesick/repos/castle/modules/vim-solarized/colors/solarized.vim

Coverage remained the same at 88.303% when pulling 5614b6b8b3b11b31e488c15890e9b36f3f967970 on singular0:symlink_realpath into 570b0636325c0b7fab67e54659645b32a772d214 on technicalpickles:master.

Coverage remained the same at 88.303% when pulling 5614b6b8b3b11b31e488c15890e9b36f3f967970 on singular0:symlink_realpath into 570b0636325c0b7fab67e54659645b32a772d214 on technicalpickles:master.

Looks good to me. Are there any extra tests you can add to verify this behaviour? I'm not sure how much value this would have, what do you think?


Use esp-idf vfs

This branch by @rnplus seems like a good way to bolt the esp-idf vfs under micropython in stead of its default vfs.

Erm... all the code in there is dummy'ed out, is that intentional?

Sorry for lack of intro from irc .. this is not yet ready code
Just a start . .
So yes, it seems it is intentional 😉

previous comments mentioned stubs. there shouldn't be any stubs anymore.


optimize zoom bounds

Minor optimization - no point to compute bounds on every calculation - enough to keep them around with const pre-generation.

Looks good!
As I'm not an expert in this GIS world I have a question: Is this constant expected to be correct for all tile-based renderers?
I was already wondering whether my calculations follow a globally accepted standard for vector tiles, or whether there are different tile-based map renderers which use different bounds.

Looks good!
As I'm not an expert in this GIS world I have a question: Is this constant expected to be correct for all tile-based renderers?
I was already wondering whether my calculations follow a globally accepted standard for vector tiles, or whether there are different tile-based map renderers which use different bounds.

Will merge after main is fixed


Add cloud config structs

What this PR does / why we need it:
This PR will introduce the structs used by the cloud-provider implementations in Kubernetes.
Also it will introduce functions to convert those structs to a string. This is required as the lib to parse ini-files in kubernetes can only read but not write. There are only a few libraries to write ini files, and all implement a different style of escaping - this i used a library which does not escape at all & added the escaping manually.
All introduced structs are exported, so they can be reused.

One thing to note on this PR:
It will drop support for kubernetes v1.8.
The reason is that the vSphere provider changed the format of the cloud-config from v1.8 to v1.9


Overlay scrollbars on top of all content

I'm testing this using v0.31.2-32bit on Windows 8.1 64 bit.
This might be a bug in the chrome engine, I'm not sure, but the overlay-scrollbars stay on top of any content:

Also, it would be quite useful if the scrollbar changed from a dark overlay to a light one if the background colour itself is dark (or having a thin white border would also work great). As it is really hard to tell there is another scrollbar on the bottom right corner.

I'm not sure whether it is a bug of your HTML or Chrome engine, but I suggest dumping your page into an independent HTML page and testing it in different browsers, and then report it to Chromium's issues tracker. Electron renders the page exactly the same with Chrome browser, so even if it is a bug of Chrome browser we are not able to fix it on our side.

Just tested in chrome 45.0.2454.85 and canary 47.0.2505.0 setting the chrome://flags and experienced the same issue, so I'll try to create a minimum example and report it to the Chromium tracker.
One thing I've noticed though is that Chrome does indeed have a lighter border around the overlay scrollbars (and they are slightly thicker) making them more visible, is there any way to force electron to do the same?

One thing I've noticed though is that Chrome does indeed have a lighter border around the overlay scrollbars (and they are slightly thicker) making them more visible, is there any way to force electron to do the same?

Is it a new behavior in Chrome 47? The drawing of scrollbars should be the same in Electron v0.32.x and Chrome 45.

No, both 45.0.2454.85 (32 bit) and 47.0.2505.0 (64 bit) have the same scrollbars. I'm also using electron 0.32.1, so it should have the v45 engine.
This might be a very long shot, but could it be that Chrome detects the touch screen in my laptop and increases the scrollbars for touch, and somehow electron does not?

This might be a very long shot, but could it be that Chrome detects the touch screen in my laptop and increases the scrollbars for touch, and somehow electron does not?

Nope, just found a flag to disable touch support in Chrome and the scrollbars are still rendered larger and with the lighter border.


V1.10 release

To do/decide:

[x] #2557 - merged
[x] #2745 - needs some tweaks, but we need to decide if it's a good idea before V2
[x] #2190 - deferred
[x] cherry pick stuff from v1.9 branch, maybe just history #4350
[x] #3346
[x] #3593 -  deferred
[x] #3946
[x] #4028 - API will change in v2
[x] #4354
[x] #4216
[x] #4191
[x] #3941 - revert or fix
[x] #4339
[x] #4356

Ideally I'd like to get a pre-release of V1.10 out early next week.
If you have a change you'd like to be included in V1.10, please make sure your PRs are ready to merge and use the magic comment "please review" to assign us.

#4356 is ready to go i think

#4165 is ready to go i think

#3674 would be amazing 🙏

#3674 would be amazing pray

this is already merged, so will be included.

so are #4165 and #4356.


[Bug]: _helpers.tpl message gives wrong advice when creating passwordFromSecret

Contact Details
No response
What happened?
Hi,
I was just installing the chart locally and testing the passwordFromSecret option.
It appears the error message is giving the wrong advice. It currently says Secret examplepassword must contain key NEO4J_DATA  Think that needs to be Secret examplepassword must contain key NEO4J_AUTH.
Not a biggie obviously, but might save someone a bit of head scratching.
Thanks
Chart Name
Cluster
Chart Version
5.5.0
Environment
Local Machine
Relevant log output
{{- define "neo4j.secretName" -}}
    {{- if .Values.neo4j.passwordFromSecret -}}
        {{- if not .Values.disableLookups -}}
            {{- $secret := (lookup "v1" "Secret" .Release.Namespace .Values.neo4j.passwordFromSecret) }}
            {{- $secretExists := $secret | all }}
            {{- if not ( $secretExists ) -}}
                {{ fail (printf "Secret %s configured in 'neo4j.passwordFromSecret' not found" .Values.neo4j.passwordFromSecret) }}
            {{- else if not (hasKey $secret.data "NEO4J_AUTH") -}}
                {{ fail (printf "Secret %s must contain key NEO4J_DATA" .Values.neo4j.passwordFromSecret) }}
            {{/*The secret must start with characters 'neo4j/`*/}}
            {{- else if not (index $secret.data "NEO4J_AUTH" | b64dec | regexFind "^neo4j\\/\\w*") -}}
                {{ fail (printf "Password in secret %s must start with the characters 'neo4j/'" .Values.neo4j.passwordFromSecret) }}
            {{- end -}}
        {{- end -}}
        {{- printf "%s" (tpl .Values.neo4j.passwordFromSecret $) -}}
    {{- else -}}
        {{- include "neo4j.name" . | printf "%s-auth" -}}
    {{- end -}}
{{- end -}}

Code of Conduct

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

thanks @cychainey
You are absolutely right here. I will fix this quickly.
Thanks once again for reporting this.

No worries, happy to help

@cychainey this is fix in https://github.com/neo4j/helm-charts/pull/173


Switching events is "jumpy"

According to bug report over email:
When switching an event, after having been on another, it jumps between the two events while it loads in the attendees.

Can confirm, happens when you select another event while one is in the process of loading.


Using history.back() inside of an embedded  causes full page refresh on parent

I have an application that embeds user content within an <iframe>. Calling history.back, history.forward, or history.go will always cause a full page refresh in the parent window.
Expected Behavior
When I navigate around within that <iframe> calling window.history.back() (inside of the <iframe>) should only change the window.location.href of the <iframe>. It should not affect the parent window at all.
Actual Behavior
Calling window.history.back() inside of an <iframe> causes a full page refresh in the parent window.
Addition Tidbits
In real Chrome and other browsers, when history entries are created by <iframes> going back / forward between those entries will only ever affect the actual <iframe> and will not leak across to parents.
Also, I have nodeIntegration disabled in the BrowserWindow instance.

Also it would be nice if it were publicly accessible from webContents to access the actual history array on the NavigationController instance. I read through the source and saw the array is never exposed.
I'm currently patching the navigation controller object to do this:
require("navigation-controller").prototype.getHistory = ->
 {history: @history, index: @currentIndex}

As long as this is added prior to instantiating new BrowserWindow it'll get picked up as a function on the prototype.
Now we can access that object:
mainWindow.webContents.getHistory() ## => { ... }

As a temporary fix until upgrading to a fixed version, one can replace the iframe's history.back() with the one of the containing window:
iframeElement.contentWindow.history.back = window.history.back;


Add Get all extension points method

Added Get all extension points method. To be used from aio-cli-pulin-app for extension point listing.
Description

Related Issue

Motivation and Context

How Has This Been Tested?

Screenshots (if appropriate):
Types of changes

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

Checklist:

[x] I have signed the Adobe Open Source CLA.
[x] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[x] I have read the CONTRIBUTING document.
[ ] I have added tests to cover my changes.
[ ] All new and existing tests passed.

Closing this PR as we need to merge changes to release-ext-reg.


Push tags with protected branches

How do I go about setting permissions to update tags when I have protected branches enabled?
Here is the error I receive.
remote: FATAL: W refs/tags/test-tag repositories/redmine/rms-ctp-tessitura redmine_user_name_6 DENIED by fallthru
remote: error: hook declined to update refs/tags/test-tag

As discussed in https://github.com/jbox-web/redmine_git_hosting/issues/732#issuecomment-532362560, issues related to Redmine < 4.0 or severely outdated issues are being closed to help clean up the issue tracker.
If this issue is still relevant to you and you are running Redmine >= 4.0, please open a new issue including all new relevant information.


Centralized GA calls/queueing and made logging consistent

Added logAllCalls configuration to do an in-memory store of all calls made to GA for debugging purposes.
Created method _gaq to wrap calls to $window._gaq.push in order to facilitate logging and ensure existence in a single location.
Created method _ga to wrap calls to $window.ga in order to facilitate logging and warn if ga method doesn't exist.
Cleaned up the usage of me, that, and this. 'me' has been replaced with 'that'.
Removed numerous 'var that = this, args = arguments;' statements that no longer serve any purpose.
Updated all tests to handle the standardized way logging is now performed. Tests are 100% passing with improved verifications added to some methods.
Simplified numerous tests to use toEqual for arrays instead of multiple toBe statements.

really cool update, was needed, thanks !


Create schema for COVID that could be repurposed for generic ELR

Discuss with COVID/CELR team on how to better represent the data for their ELR messages.

https://github.com/CDCgov/data-exchange-hl7/pull/974

COVID.txt


Teams: Update RSC concept doc

Docs Build status updates of commit 18350b5:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

msteams-platform/graph-api/rsc/resource-specific-consent.md
:white_check_mark:Succeeded
View

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

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

File
Status
Preview URL
Details

msteams-platform/graph-api/rsc/resource-specific-consent.md
:white_check_mark:Succeeded
View

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

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

File
Status
Preview URL
Details

msteams-platform/graph-api/rsc/resource-specific-consent.md
:white_check_mark:Succeeded
View

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


Add example notebooks to docs

For rendering on readthedocs

They render OK, but sphinx parses each notebook heading as a full heading; I'd like to move them each to their own level so that notebook sub-headings get parsed sensibly. Alternatively maybe a thumbnail gallery is the way to go

I like the gallery much better, but am having trouble getting the default thumbnail to register

Good enough. Just need to be able to see it on the white background now.


Null shift_state published via MQTT

I noticed that Teslamate is intermittently reporting a null shift_state via MQTT:
Client mosq-4HVU4zyo8u049OF7QP received PUBLISH (d0, q0, r0, m0, 'teslamate/cars/1/shift_state', ... (0 bytes))
Client mosq-4HVU4zyo8u049OF7QP received PUBLISH (d0, q0, r0, m0, 'teslamate/cars/1/inside_temp', ... (4 bytes))
18.7
Client mosq-4HVU4zyo8u049OF7QP received PUBLISH (d0, q0, r0, m0, 'teslamate/cars/1/charge_energy_added', ... (5 bytes))
15.18
Client mosq-4HVU4zyo8u049OF7QP received PUBLISH (d0, q0, r0, m0, 'teslamate/cars/1/geofence', ... (4 bytes))
Home
Client mosq-4HVU4zyo8u049OF7QP received PUBLISH (d0, q0, r0, m0, 'teslamate/cars/1/charger_voltage', ... (3 bytes))
232
Client mosq-4HVU4zyo8u049OF7QP received PUBLISH (d0, q0, r0, m0, 'teslamate/cars/1/rated_battery_range_km', ... (6 bytes))
244.54
Client mosq-4HVU4zyo8u049OF7QP received PUBLISH (d0, q0, r0, m0, 'teslamate/cars/1/shift_state', ... (1 bytes))
P

first and second line - with a few seconds in between. This seems to happen repeatedly, and I find it a bit odd? I don't mind the duplicate notifications with data (the consumer can just filter those), but the null-data gives some really weird history data, so just suppressing those would be better, I think...

This is by design. It basically reflects what the Tesla API reports. At a certain point null is reported as shitft_state.
Say the shift state was "D" and you drive into an underground garage with poor reception. The car will then eventually fall asleep and the shift_state will become null.  If null values were not published, the shift state would appear to be "D" for the entire time the car is parked.

This is by design. It basically reflects what the Tesla API reports. At a certain point null is reported as shitft_state.
Say the shift state was "D" and you drive into an underground garage with poor reception. The car will then eventually fall asleep and the shift_state will become null.  If null values were not published, the shift state would appear to be "D" for the entire time the car is parked.

Hmm, yeah, okay, I can see why changing this may not be obviously better... Fair enough, I'll handle this on the consumer side :)

Hmm, yeah, okay, I can see why changing this may not be obviously better... Fair enough, I'll handle this on the consumer side :)


Allow to lint issues

It would be super cool if Danger could assist linting issues. Even though GitHub introduced the ISSUE_TEMPLATE.md, I see overall on projects I work on, that these are not fully understood or filled correctly.
Some of them could use some heuristics, indicators or measures to figure out whether an issue is meaningful or not. Feedback from Danger is likely easier to handle for all involved than from humans.
Also Danger could help classifying issues, which need revision by their author by tagging them as such to streamline the workflow for contributors and maintainers who triage issues.

To pull this off, we need to be able to support danger as a service - https://github.com/danger/danger/issues/42
As there would be no CI process to run

Tagging could happen within CI right? It'd only run when commits are pushed though, not when comments are placed (unless you manually change the CirceCI/Travis hook settings in GitHub, but then all would run when a comment is placed)

@mrackwitz means actual issues like this one, thing that don't or won't have related commits and thus no connection to CI

Oh, actual issues would be no-go until we introduce 'DaaS', yeah


The number of target animals increases using env.reset()

Hello,
In harvest_milk and harvest_wool tasks, when calling reset(), the environment produces a new cow/sheep and keeps the old animals. To train an RL agent, it makes the task easier to solve when calling more reset(), because the animals are easier to find.
figure: during training, the number of cows increases after many environment episodes.

My environment is made with env = minedojo.make(task_id='harvest_milk_with_empty_bucket_and_cow', image_size=(256,256))
Could you check for this issue?

Hey, thanks for reporting this. Please refer to this to eliminate spawned mobs during reset. At this point users need to call this manually. We will integrate it at some point. Sorry for the inconvenience.


extension: Add initial openvscode-server support

Document everything that is currently known about this introduction, in README as usual for this extension.

Main goal being, to be able to remotely deploy this extension, alongside still locally too.

Downgrade the vscode version from 1.78.0 to 1.77.3, the latter being the currently latest stable (Verified, [1]) one for this server integration.

This is because [1]'s version and this vscode version have to align, in order for openvscode-server to work.

Contributes to fixing the foreign (companion's) Issue [2], below.
[1]https://github.com/gitpod-io/openvscode-server/tags
[2]https://github.com/eclipse-cdt-cloud/vscode-trace-extension/issues/15
Signed-off-by: Marco Miller<EMAIL_ADDRESS>

Wrong branch/fork; about to redo, WIP.


Override From Youtube Failing

Overrides from youtube are failing.
I've tried several different songs but all keep failing.
Also says to check logs but i cannot find the logs anywhere

Is this still happening for you? Because it was broken at least from yesterday until a few hours ago but it should work again now.
To find the logs, you can do one of the following:

If your client failed to open, click the Open logs folder button.
Open the screenshot directory by right-clicking 📷 Camera button, navigate 1 directory up, then open logs folder.
Navigate to %userprofile%.runelite\logs on Windows or $HOME/.runelite/logs on Linux and macOS.

at nl.alowaniak.runelite.musicreplacer.YouTubeSearcher.getSearchResults(YouTubeSearcher.java:46)
at nl.alowaniak.runelite.musicreplacer.YouTubeSearcher.lambda$search$0(YouTubeSearcher.java:40)
at java.base/java.util.concurrent.Executors$RunnableAdapter.call(Unknown Source)
at java.base/java.util.concurrent.FutureTask.run(Unknown Source)
at java.base/java.util.concurrent.ThreadPoolExecutor.runWorker(Unknown Source)
at java.base/java.util.concurrent.ThreadPoolExecutor$Worker.run(Unknown Source)
at java.base/java.lang.Thread.run(Unknown Source)

2022-09-01 22:15:08 [Client] ERROR n.r.c.p.t.farming.FarmingTracker - Offset error! Observed new offset of 25, previous observed offset was 15 (15) Player Loc:WorldPoint(x=2947, y=3368, plane=0)
this is what i found its giving an error

I think you're missing part of the error message, there should be some more above it.
(Alternatively you could just drop the whole client log file.)

client_2022-09-01.0.log

No problem :)
This looks like it's the problem that existed up until a few hours ago (my server was down).
Are you sure you're still having this issue?

currently still having this issue. i just tried it just now and said failed

Could you perhaps also send the new client log?
And check if you can access  (it's what the plugin tries to access).
Only thing I can imagine that might be happening is if you're searching for the same thing and the error is cached for some reason.

client.log

yeah i clicked that link and its taking me to some gibberish coding line lmao

Ah seems to be something wrong when it tries to dl the audio from youtube.
I'm afraid this might take a while to get fixed (I use another library for the youtube stuff, so they have to fix it first).
So for the time being you probably won't be able to search for new songs.
(Any songs already cached on my server should work though, so the osrsbeatz and rs3 (presets) should work.)

yeah, they're working fine just overriding seems to be the issue. but thank you! sorry it dragged out a bit but glad i was able to help you find the error for the little bit i did lmao

I've done a work-around which works for now, I'm not entirely sure if this work-around will work in all cases though.
But AFAICT it should be fixed now, please let me know if it isn't for you.
And definitely no problem "it dragged out a bit", always happy to help and thank you for letting me know of the issue :)


Change V1/V2 references to high/low level

Rename methods that contain references to V1 and V2 to mention the high
level and low level consumers, to make the code more readable.

Otherwise, LGTM

[Current coverage][cc-pull] is 38.55%

No coverage report found for master at ef78c3f.

Powered by Codecov. Last updated by [ef78c3f...8c533ab][cc-compare]
[cc-base-branch]: https://codecov.io/gh/linkedin/pinot/branch/master?src=pr
[cc-compare]: https://codecov.io/gh/linkedin/pinot/compare/ef78c3f0acb287a7238c08dfaa102d4641c1ae00...8c533ab48e66d50d37f41dad85f2649703e72beb
[cc-pull]: https://codecov.io/gh/linkedin/pinot/pull/199?src=pr

Discarding this PR as it is obsoleted by #216 and #219.