Going back to the main menu from a lobby kills matches

Hey!
I've been using Totsugeki with no issues but one, whenever I return to the main menu and join another lobby/park/floor every duel station starts saying "Failed to connect to the Duel Station" or something along those lines.
I can still access lobbies, rankings and everything else, I just can't connect to matches after leaving another lobby.
I don't have this issue without Totsugeki and rebooting the game seems to fix it (luckily it's super fast now).
Not sure what the issue can be or if it is even on the mod's end at all, I don't have any special message in the command prompt either so not sure where it could come from.
Anyways thanks for the awesome mod, you're making a godlike job!

I return to the main menu and join another lobby/park/floor every duel station starts saying "Failed to connect to the Duel Station" or something along those lines.

I've actually encountered this prior to writing Totsugeki, but haven't run into it recently. If you can screenshot or paste the logs from Totsugeki right after you see that message it will help me see what's going on.
I'm somewhat doubtful it's due to Totsugeki, but the logs will help confirm what's going on. If it's a server-side issue, the logs will help me see if there's way for Totsugeki to work around it.

Anyways thanks for the awesome mod, you're making a godlike job!

Glad you find it useful!

Well apparently the issue only pops up when I leave the "News" tab open after joining a lobby, when I close it the stations start working again and the issue disappears entirely... :|
Otherwise whenever I run into that issue I get a neverending passive stream of this in the logs https://b.catgirlsare.sexy/r-rP_3CIWceV.png no idea if that's even useful but they don't seem to appear at all when lobbies are working as intended.
I guess it might just be some Strive server fuckery but who knows, at least if someone also runs into that issue just know that pressing a single button fixes it

That's interesting. Looks like maybe an endless loop in GGST. Maybe Totsugeki can work around this problem if /api/tus/read is a cacheable response.
I'll see if I can reproduce it on my side. To confirm, what are the steps you take for the issue to happen?

1/ Launch GGST with Totsugeki
2/ Network -> Online Match
3/ Select a lobby/floor
4/ Everything works
5/ Leave back to main menu
6/ Network -> Online Match
7/ Select a lobby/floor
8/ "Failed to obtain Duel Station information."
I think that's about it, thank you for looking into it!

I just tried the above steps and did not have this issue.

Might just be on my end then, I'm able to reproduce this consistently.

I just tried this tonight and could not reproduce. When this happens does this happen on all duel stations? Sometimes the ones near the entrance bug out.

It happens on all duel stations and never fixes itself, it always happens when the news tab in lobbies doesn't load any information, also leaves the "wifi" loading indefinitely.
I also get this issue when I join a lobby after checking the rankings in the main menu.
Weird but I guess I'm just unlucky, I'll try it out on a different PC :|

Let me try blocking the news API to simulate your situation. I suspect that GGST is only allowed to make one API request at a time, so the duel station ones are backed up behind waiting for news to load.

Try the latest release I'm hoping the timeouts may help here if a connection is getting stuck.

Closing for now. Let me know if this is still happening.


Issue/428 ttn logger

Coverage remained the same at 74.347% when pulling 42f64f4775715ae595f351f3ad1e86802e04ce9c on issue/428-ttn-logger into b42225737e92f5f7ffba7c4f9d63b39e4e7018b6 on develop.


fix: Fix default_mode casing

Should be defaultMode as written in the readme. I tried with default_mode just to check and it doesn't do anything

Thanks!


Missing line in  cfDNA/data/Ensembl_v75/README

Hi,
It seems there is line 4 missing in the README file which leads to an absent file that is needed for consecutive processing.
My guess is that it would be something like:
./extractGeneBody.py | bgzip -c > ensemblv75_canonicalTranscriptIDs.protein_coding.lst.gz
Another issue:  cfDNA/WPS_overlays/Maurano_et_al_TFclusters/README seems to refer to data/Maurano_et_al_func_var/hg19.taipale.recluster.counts in line 18. How do I get this file?
Thanks,
Jan

I added the missing Encode file and added a line to DNA/WPS_overlays/Maurano_et_al_TFclusters/README for generating the count file. I hope this resolves your problems.


Drop support for Puppet v2

Since this module is no longer tested against v2 and that v3 is almost out of support, we should drop support for v2.

closed in favor of #53


Getting Cartesian position and velocity vectors for propagated orbit

I am trying to get the cartesian position and velocity vectors for each propagation step in my orbit. I  generated the orbit with Keplerian elements.
According to the documentation I should be able to get the state vectors (both position and velocity) from class orbital.utilities.StateVector , but I get a Type Error: new() takes exactly 3 arguments (2 given)
Here is the code:
from scipy.constants import kilo
import orbital
from orbital import earth, KeplerianElements, Maneuver, plot, utilities
from orbital.utilities import Position, Velocity
import matplotlib.pyplot as plt
import numpy as np

#Orbit Setup
orbitPineapple = KeplerianElements.with_period(96 * 60, body=earth, i=(np.deg2rad(51.6)))
plot(orbitPineapple)
plt.show()
orbitPineapple

Out[23]: KeplerianElements(a=6945033.343911132,
e=0,
i=0.90058989402907408,
raan=0,
arg_pe=0,
M0=0.0,
body=orbital.bodies.earth,
ref_epoch=)
prop1 = orbital.maneuver.PropagateAnomalyTo(M=1.00)
orbitX = orbitPineapple.apply_maneuver(prop1)
plot(orbitPineapple, title='Go Pineapple!')
plt.show()

orbital.utilities.StateVector(orbitPineapple)

TypeError                                 Traceback (most recent call last)
 in ()
4 #print(orbital.utilities.StateVector.velocity(orbitPineapple))
5
----> 6 orbital.utilities.StateVector(orbitPineapple)
7 #orbital.utilities.StateVector.position(orbitPineapple())
8
TypeError: new() takes exactly 3 arguments (2 given)

Use orbitX.r and orbitX.v.

I tried it before and again now and it resulted in an attribute error.
Code:
<print(orbitX.r)
print(orbitX.v)>
<AttributeError                            Traceback (most recent call last)
 in ()
----> 1 print(orbitX.r)
2 print(orbitX.v)
AttributeError: 'NoneType' object has no attribute 'r'>

It turns out that for the state vectors .r and .v you can only use the original orbit name. The original name will automatically be updated after each maneuver or propagation.
So, print(orbitPineapple.r) and print(orbitPineapple.v) will give me the state vectors, position and velocity, in Earth centric inertial coordinates (ECI).

Yes, apply_maneuver mutates the orbit as shown in the documentation.


fixed issue with precision

Fix to stop HTTP 400 responses from influxdb due to the use of ms rather than m when posting precision

Crap, it's not. Haha, fooled myself.


Render inconsistent for bottom line vs other lines above

Describe the bug
I'm totally newbie for lipgloss, firstly thank you for this wonderful library! tldr; I have the following simplified code:
func main() {
	lines := []string{
		" 1/2        ✅      update    -                     azurerm_resource_gr…  4s                       ",
		"\x1b[38;5;229;48;5;57m 2/2        ✅      update    -                     azurerm_resource_gr…  6s                       \x1b[0m",
	}
	contents := lipgloss.NewStyle().
		Width(90).
		Height(2).
		MaxHeight(2).
		MaxWidth(90).
		Render(strings.Join(lines, "\n"))

	fmt.Println(contents)
}

This results into:

While, if I swap the elements in lines, I get:

The difference is that former one, the focused line extends over the visible text content. If drilling down, it seems somehow the rendered content of the second line (in the original version) will be:
\x1b[38;5;229;48;5;57m 2/2        ✅      update    -                     azurerm_resource_gr…  6s              " (Note that the ending ANSI escapes are missing).
Additionally, with the initial version, if I add a third line (will be truncated as the height is 2), it's all good:

Setup
Please complete the following information along with version numbers, if applicable.

OS [Archlinux]
Shell [zsh]
Terminal Emulator [kitty]
Terminal Multiplexer [tmux]
Locale [en_US.UTF-8]

To Reproduce
As mentioned above
Expected behavior

Additional context
I'm trying to use the table (from bubbles). Especially, I'm dynamically setting the Width and Height for the table in response of WindowSizeMsg. Especially, I'm dynamically setting the width of each column like below:
func TableColumn(width int) []table.Column {
	const statusWidth = 6
	const actionWidth = 8
	const timeWidth = 24

	dynamicWidth := width - statusWidth - actionWidth - timeWidth

	indexWidth := dynamicWidth / 5
	moduleWidth := dynamicWidth / 5 * 2
	resourceWidth := dynamicWidth / 5 * 2

	// indexWidth := 10
	// moduleWidth := 10
	// resourceWidth := 10

	return []table.Column{
		{Title: "Index", Width: indexWidth},
		{Title: "Status", Width: statusWidth},
		{Title: "Action", Width: actionWidth},
		{Title: "Module", Width: moduleWidth},
		{Title: "Resource", Width: resourceWidth},
		{Title: "Time", Width: timeWidth},
	}
}

This causes 3 of the columns have the fixed width, while the other 3 are related to the width of the current window.
With this setup, I ends up with the bottom line always rendered to the max width of the window. Whilst, if I uncomment the line in above snippet, i.e. making those 3 dynamic widths to be fixed width, all come back to normal (bottom line is just like the other lines that rendered to the end of visible text).
However, I've tried to do the same thing for the bumbletea table example, but it is all good.

Fixed by updating from  github.com/charmbracelet/bubbletea v1.1.0 to v1.2.4 solved the issue, have no idea why I was at v1.1.0...


How to update cache after mutation without pollInterval?

Hello, Query gives old data after mutation - need new, how can I do this without pollInterval (because I can't to do much requests to the server)?

Hey, @Goolpe I would say the cache is not as fully functional as one might hope. I am writing the cache and some react apollo sugars for the next release. You are also more than welcome to contribute. I had to move to Norway early this week so I didn't have enough time to contribute. But I'm more settled in now. Should get some update this weekend.

Thank you, it would be awesome!


Multiple app instances in standalone mode?

Sorry if this is a dumb question but ...
Does the standalone mode support running multiple instances of an app? I have scoured the docs and can't find that.
If supported, should probably add these details to the docs.  I'll be happy to add given more details.

@rschiefer - Are you asking if you run, for example, 3 instances of an app in standalone mode using a single invocation of dapr run?  If so, no.
However, it can be achieved by running dapr run in different terminals with different -app-ids and different ports.
Is there a particular scenario you're interested in?

If we run the same app in different app-ids & ports in standalone mode, can we make a cluster like in Orleans?
I am more interested in such a setup where the Dapr sidecar of the same app running in multiple machines, forming a cluster. Is that possible with Dapr?

If we run the same app in different app-ids & ports in standalone mode, can we make a cluster like in Orleans?
I am more interested in such a setup where the Dapr sidecar of the same app running in multiple machines, forming a cluster. Is that possible with Dapr?

Yes, you can. Dapr is designed to run in clustered mode.

Yes, you can. Dapr is designed to run in clustered mode.

Do we have any docs/how-to's explaining the same?

This is covered in the overview. https://github.com/dapr/docs/blob/master/overview.md#running-dapr-on-a-local-developer-machine-in-self-hosted-mode
If there is more that you would like to see open a new issue.

Link is broken? Can you update?

https://github.com/dapr/docs/tree/master/overview#running-dapr-on-a-local-developer-machine-in-self-hosted-mode

The link is still broken and there's no mention on how to run multiple instances locally.

@yaron2 do you have multiple instances of app in standalone mode

Months I've been back here, still looking.


Question: Compile micropython to MIR for a simple Python JIT

Hi,
I am playing around a bit with MIR (mostly for fun) to see what is possible, and to learn more about JIT's.
I was wondering if you'd think it would be possible (and worthwhile) to compile the C implementation of something like Micropython to MIR and then use that to create some sort of (Micro)Python-JIT?
And later on, maybe try the "real" CPython?

The original MIR project goal is to create a MRuby/CRuby light-weight JIT.  CPython is not different from CRuby with the point of view of  MIR usage for implementing a good JIT.
I did not start a real work on implementing a MIR-based JIT for Ruby yet and I expect some changes will be needed to MIR project to be successful with this goal.  It is an iterative process.
Although some people were already successful with using MIR for JIT implementations.  The best example is Ravi implementation (https://github.com/dibyendumajumdar/ravi).
As for micropython JIT, usage of MIR might be too heavy for this.  Micropython site advertises 256KB code space and 16KB RAM.  It is too big constraint for MIR usage.
I encourage anyone to try MIR as a JIT compiler and provide me some feedback.  This feedback would help MIR development and JIT implementations based on it.

Sure, I was referring to MicroPython only as a simpler target that should be easier to compile and experiment with (not for serious usage). Thanks for pointing me at Ravi, I'll have a look. I'll report back if I get something interesting to work!


Issue with handling of keys in text inputs

Verify

[X] I understand that if I ignore these instructions, my issue may be closed without review.
[X] I have made sure that there are no similar opened issues.
[X] I have made sure that the mod version matches the latest release.
[X] I have reproduced this issue when other mods have been disabled.

Describe the bug
Related to https://github.com/dstmodders/mod-dev-tools/issues/14.
Even though the main branch should already handle UI inputs correctly in most cases, the no-sdk branch doesn't. After introducing the new crafting menu, our key bindings don't respect its text input and trigger our actions. Since we are not planning to release v1.0.0 anytime soon, we should consider fixing this one in v0.8.0 instead.
Steps to reproduce

Open console
Press ] during input
See opened Dev Tools menu

Expected behaviour

Open console
Press ] during input
Just add "]" in the text input and ignore opening Dev Tools menu

Console output
Click to expand
If applicable, replace this line with the console output.

Screenshot
Click to expand
If applicable, replace this line with an embedded screenshot to help explain your problem.

Merged manually into no-sdk branch.


Pause audio player on incoming calls or other apps play audio | ios and android

Current behavior
The audio does not pause on incoming calls or other apps play audios
Also onAudioFocusChanged does not get called
I tried setting ignoreSilentSwitch to ignore and obey with no luck
Reproduction steps
      <Video
          ref={(ref) => { _PlayerData.player = ref; }}
          source={{ uri }}
          style={styles.player}
          rate={1}
          volume={1}
          paused={!isAudioPlaying()}
          muted={muted}
          repeat={repeat}
          onLoad={this.onLoad}
          onProgress={this.onProgress}
          onError={this.onError}
          onEnd={this.onEnd}
          onAudioBecomingNoisy={this.onAudioBecomingNoisy}
          onAudioFocusChanged={this.onAudioFocusChanged}
          audioOnly
          playInBackground
          playWhenInactive
        />

Expected behavior
My app to pause the audio that is being played on incoming calls or other apps play an audio such as Instagram.
Platform
Which player are you experiencing the problem on:

iOS
Android ExoPlayer

We have the same problem. Any ideas?

We have the same problem.

We have the same problem. Observed on Android Pixel 7 API level 34, RNV 6.0.0-beta.6. RN 0.73.6.

playInBackground
ignoreSilentSwitch='ignore'

And Info.Plist contains:
<key>UIBackgroundModes</key>
	<array>
		<string>audio</string>
	</array>

audio does not continue when backgrounding the sample basic app while it is playing video.

Making a session manager / exposing some abstraction of the iOS category settings may be helpful here.

@paulrinaldi I think this could help https://github.com/TheWidlarzGroup/react-native-video/pull/3850


Issue #8986 fixed.

Issue Reference

Issue #8986 fixed.

@suraj-webkul
As I have checked and found that the issue has been fixed. Please check the video for reference.
Video
https://github.com/bagisto/bagisto/assets/92721837/496d2659-bca4-4f61-a930-14efeaa2a187


Swipe more than one slide per once when clicking arrows

Hello again,
I'm using the swiper in carousel mode and I have next and previous arrows. Anytime I click either, it advances or rewinds one slider. Is there a way to make it more than one?
Thanks.

You may try the slidesPerGroup parameter or use .swipeTo method by passing custom index there

The slidesPerGroup didn't work (I set it to 2) and the swipeTo method I can't find it in the API :\

@coldpumpkin i guess you use Swiper 2.x, it is API here https://github.com/nolimits4web/Swiper#swiper-2xx

Alright, I think I found it: mySwiper.swipeTo(index, speed, runCallbacks)
I should put it in my arrow function, right? Only thing is, I don't understand what I should put in the index parameter :O


Adjust isEnabled method

Issue
I added a way to be able to use a config yaml file to be able to disable certain, or all, tasks with a simple config.
There also is a new config to set the default behaviour. This has been added to the dev.yml
SilverStripe\Dev\BuildTask:
    default_enabled: true

To use these changes, you can create your own yml config, example:
SilverStripe\Dev\BuildTask:
  default_enabled: false
DevTasks\ExampleTask:
  enabled: true

This would disable all tasks, besides the ExampleTask.
I also added the suggestion from this comment to add a deprecation notice.
Let me know if something needs to be adjusted.

Thanks a lot for reviewing!
First time contributing to such a large project, sorry for the smaller mistakes I made ($is_enabled vs. $enabled, woops :D , unclear comments) I'm learning :)

Removed the default_enabled flag
adjusted the $enabled to $is_enabled
adjusted the comments as suggested
removed the deprecation notice
added a fallback value (to true) for tasks that do not have the $enabled property set
removed the method signature changes (force of habit 😇 )

Adjusted the isEnabled method to not use static::$enabled, and instead use $this->enabled
This should be all :)
Thanks a ton again for the massive help and the reviews. Really appreciate it!


Added capability to auto force merge as much as possible to reduce the number of segments

Description
In our product, for small index (about 20g-), the frequency of writing/updating is not high, we have to frequently execute forceMerge segment to reduce the segments, to improve query performance. As lucene support TieredMergePolicy, which just merges segments of approximately equal size.
If we can support another MergePolicy, which can auto merge as much as possible to reduce the number of segments.

Have you tried writing a custom merge policy? This is a supported API and designed for users to customize/extend


[Compile Time Constant Extraction] Fix test failure

Fix test failure introduced in https://github.com/apple/swift/pull/62465
Test change conflicts with https://github.com/apple/swift/pull/62461

@swift-ci test


Fixes, builds and remote installs with new go tool against weekly/tip

Thanks for your fixes and sorry for taking so long to respond.
I haven't actually merged your pull request, but about the same fixes went in.


Prompts not showing in the status

If the script is required to get some input from the user,
in that case using input(prompt) will not show any message in the status of the program.
Only when you stop the program, then the prompt is shown in the status.

[x] OS - Windows 10
[x] Python Version - 3.6.5
[x] Gooey Version - 1.0.8
[x] Thorough description of problem

[x] Expected Behavior - when input(prompt) is used the Gooey should show some window for taking the user input
[x] Actual Behavior - No message/prompt is shown

[x] A minimal code example -- preferably copy/pastable in the issue itself (less time figuring out how to run your code == more time debugging!)

import argparse
from gooey import Gooey

@Gooey
def main():
    """main function"""
    description = "Description"
    parser = argparse.ArgumentParser(
        description=description, conflict_handler="resolve"
    )
    parser.add_argument(
        "required_arg1",
        help="Required argument 1",
        type=str,
    )
    general = parser.add_argument_group("General")
    general.add_argument("-h", "--help", action="help", help="Shows the help.")
    general.add_argument(
        "-v", "--version", action="version", version="v1.0.0", help="Shows the version."
    )

    args = parser.parse_args()
    input("Please enter something: ")

if __name__ == "__main__":
    main()

[ ] Screenshot (if visual quirk)
[x] Anything else you may think will be helpful - it may be helpful if somehow gooey can patch the input() function so that it shows a dialogue with a text that can be passed back to the code, but patching inbuilt functions imho is not the preferred way, so providing any function like gooeyinput() can be helpful.

Thanks! ^_^
And feeling awesomely happy with @Gooey

Hey @toddatctera, thanks for the answer. You're right: Gooey's working as intended in this case. Your example is spot on. 👍


Add support for docker build --ssh in docker_build command

Requested in the #tilt channel, and separately in this issue request: https://github.com/windmilleng/tilt/issues/2437
I suspect it's straightforward, will add it now

may be closed by https://github.com/windmilleng/tilt/commit/e1f020ef163398d56fba5dc65da12450697bb868

oops! yes. i tagged the wrong issue. fixed in https://github.com/windmilleng/tilt/pull/2750


#bug; received null value for amountUSD in convex-finance at block number 17577075

Describe the bug
A clear and concise description of what the bug is.
Reproduction
Where can we see this bug in action?
git checkout c9a03c8052ffa6b90758097d571a10d3e59e90fe
messari build convex-finance-ethereum
export thegraph_admin=http://<IP_ADDRESS>:8020

# create the subgraph in postgresql
graph create --node $thegraph_admin convex-finance-ethereum
graph deploy --node $thegraph_admin \
--ipfs http://<IP_ADDRESS>:5001 convex-finance-ethereum

By starting indexing at block number 17577075, such error will occur.
Expected behavior
No error occurs, to should not be a null value
Screenshots

Hey @chenzongxiong thank you for this issue! I think what is being described was fixed in this PR: https://github.com/messari/subgraphs/pull/2247
Do you know what subgraphVersion you are using in this example? If it is not fixed still please re-open this issue or make another referencing this. THanks!


Dubbo zookeeper registry failed to re-subscribe

[x] I have searched the issues of this repository and believe that this is not a duplicate.
[x] I have checked the FAQ of this repository and believe that this is not a duplicate.

Environment

Dubbo version: 2.7.5
Operating System version: Ubuntu 16.04
Java version: jdk-1.8

Steps to reproduce this issue

zookeeper cluster hang because of backend storage issue(aliyun nas in our case)
dubbo registry received a state change of RECONNECTED event and invoke fetchLatestAddresses method, but the zookeeper server has not recovered yet, so the 3 times re-subscribe all failed and the failed retry task exist in failedSubscribed hashmap
zookeeper server finally recovered, dubbo registry received a state change of NEW_SESSION_CREATED event, it triggered the recover process, but the FailedSubscribedTask will not be created since there was a previous failed task, see
https://github.com/apache/dubbo/blob/6a6eb0b85c428e9def9bd36bcc5a6dd0b3a955ab/dubbo-registry/dubbo-registry-api/src/main/java/org/apache/dubbo/registry/support/FailbackRegistry.java#L133-L145
when the provider re-deployed in k8s cluster and changed the provider ip, the consumer would broken

Pls. provide [GitHub address] to reproduce this issue.
Expected Result
dubbo zookeeper registry re-subscribe the provider successfully after the zookeeper server recovered
Actual Result
dubbo zookeeper registry did not trigger a fresh recover process due to a previous failed re-subscribe
What actually happens?
If there is an exception, please attach the exception trace:
Just put your stack trace here!

let me have a look


update aks-preview version

This checklist is used to make sure that common guidelines for a pull request are followed.

For new extensions:

[x] My extension description/summary conforms to the Extension Summary Guidelines.

About Extension Publish
There is a pipeline to automatically build, upload and publish extension wheels.
Once your PR is merged into master branch, a new PR will be created to update src/index.json automatically.
The precondition is to put your code inside this repo and upgrade the version in the PR but do not modify src/index.json.

If this PR is for a new extension or change to an existing extension, use the following to try out the changes in this PR:
docker run -it microsoft/azure-cli:latest
export EXT=<NAME>
pip install --upgrade --target ~/.azure/cliextensions/$EXT "git+https://github.com/zqingqing1/azure-cli-extensions.git@update-version#subdirectory=src/$EXT&egg=$EXT"

Lgtm

aks-preview


TypeError: 'class Meta' got invalid attribute(s): translate

I'm on Django 2.1.
This error appears on trying makemigrations with a basic model:
from django.db import models
from django.utils.translation import ugettext as _
# 2nd party
from linguo.models import MultilingualModel # used for internationalization of models
from linguo.managers import MultilingualManager # used for internationalization of models

# Create your models here.
class Chapter(MultilingualModel):
    number = models.IntegerField(verbose_name=_('number'))
    title = models.CharField(max_length=255, verbose_name=_('title'))

    class Meta:
        translate = ('number', 'title')

How did you solve this issue. I have the same one

I've given up this app because I haven't found any solutions. Sorry

Can you give a suggestion for django multilingual model like linguo?

No. I've even given up on this idea eventually.

Any answer for this issue?


Gradle task fails when installing Android Support Repository v47.0.0

Trying to set up a CI build of an Android app and as the gradle task works through its chores, it fails trying to install the Android Support Repository (revision 47.0.0).
Here is the error from the gradle build step:
`
FAILURE: Build failed with an exception.

What went wrong:
A problem occurred configuring project ':app'.

Could not resolve all dependencies for configuration ':app:_debugApk'.
A problem occurred configuring project ':components'.
Could not resolve all dependencies for configuration ':components:_debugPublish'.
A problem occurred configuring project ':models'.
Failed to install the following SDK components:
[Android Support Repository]
Please install the missing components using the SDK manager in Android Studio.
`
Where do I tell my project to not require this version of the support repository?

I have other Xamarin.Android projects that build just fine. I know it's like apples and oranges, but ultimately the xamarin build needs the same SDK files as a native android app does, right?

@mrbelk: What android SDK version are you targeting?

The project was targeting 25, but we just changed it to 24. We are using the hosted build agent, FWIW.

@mrbelk: Thanks, does 24 work or do you see the same issue? The hosted build agents don't have all the build tools versions for SDK 25. Please check the build-tools version being used in your project.

no, targeting 24 does not work, either. the gradle task fails with the same error trying to load the support repository v47.0.0. what version of the build tools should the project use?

25.0.2 and 25.0.3 are not installed on the hosted build machines, so targeting those build tools will give an error. We have an update planned to include those. I checked the "Android Support Repository" version included on the image - we have version 46, I see the update to version 47 but we don't have a plan to update that as of now.
The software update process on the hosted build agents is slower due to several reasons - one of them being we have one VM image for all accounts in Team Services. We try to install most of the components  needed to build common projects but it is not guaranteed to include all components or the latest components.
We are moving towards a different model that will allow us to add new software faster and give users options in what software components they require. Till that is available the alternate option would be to setup your own build server and install the components required for your build.
Thanks,
Madhuri

I just looked over the dev's shoulder and he's specifying the build tools at v25.0.0. He doesn't see anywhere to explicitly set the support repository version.

@madhurig How does one use Android support repository 46 instead of 47?

@mrbelk Have you found a workaround?

@mrbelk how did you change android support repository version to 46?

@niiiksh
I managed to get my build running with the help of @leantk.
There's a couple of things that I did:

I set up the following build steps

In the powershell script for "Type" select "Inline Script". In the "Inline Script" text input enter the following:
`New-Item -ItemType Directory -Path "C:\java\androidsdk\android-sdk\licenses"
New-Item -Name 'android-sdk-license' -ItemType File -Value '8933bad161af4178b1185d1a37fbf41ea5269c55' -Path "C:\java\androidsdk\android-sdk\licenses"
Get-Content "C:\java\androidsdk\android-sdk\licenses\android-sdk-license"`

Then navigate to "Project Settings" -> "Agent queues"

Select hosted VS2017. Then click on "Capabilities", add java ("C:\Program Files (x86)\Java\jdk1.8.0_112") and then click save changes. It should look like this.

Then when you build it select "Hosted VS2017" from the dropdown.

@sahilNaran I'm using Android Studio, does hosted VS2017 will work for it?

@niiiksh In my honest opinion, it need not matter what IDE you are using.
I am using Android Studio as well, however whenever I commit to the master branch my build (using the Hosted VS2017 build agent) is automatically triggered,it builds successfully and generates the artifacts (.APK files).
I do hope someone VSTS replies and provides you with clear instructions as I could be sending you on a wild goose chase :-)

@sahilNaran, @niiiksh: VS 2017 image should work for native android builds (i.e. even when you are using Android Studio as your IDE). With the next update of VS 2017 image you will not need to setup the capabilities manually. The script you are using to add the licenses should not be required as well.

@madhurig Seems like script for licenses is not required already. But I followed @sahilNaran steps with no luck. This is what I get in console:
C:\a\1\s\gradlew.bat :application:assembleRelease
Observed package id 'add-ons;addon-unknown-unknown-19' in inconsistent location 'C:\Program Files (x86)\Android\android-sdk\add-ons\addon-google_apis-google-19' (Expected 'C:\Program Files (x86)\Android\android-sdk\add-ons\addon-unknown-unknown-19')
Observed package id 'add-ons;addon-unknown-unknown-23' in inconsistent location 'C:\Program Files (x86)\Android\android-sdk\add-ons\addon-google_apis-google-23' (Expected 'C:\Program Files (x86)\Android\android-sdk\add-ons\addon-unknown-unknown-23')
Incremental java compilation is an incubating feature.

FAILURE: Build failed with an exception.

* What went wrong:
Project 'application' not found in root project 's'.

And at the end:
Error: C:\a\1\s\gradlew.bat failed with return code: 1

@niiiksh instead of :application:assembleRelease use :app:assembleRelease
On a side note could you please upload your module structure and your gradle file

@sahilNaran Thank you, that helped!


client registering wrong ipv6 address

Not sure if this is my fault or a code issue, my interface (enp0s4) has 3 v6 addresses, an fe80: address, a public address starting with 2600:, and an fdeb: address. Is there a way to tell it to ignore the fdeb address, because it's registering that in my zone instead of the 2600 address.
Thanks

The tool uses the first global unicast IP (as it is implemented in IP.IsGlobalUnicast()) on the specified interface.
There is no way to influence that now, but we can add something like excluding or including regex, subnet prefixes, or something like that.

I was just doing some tcpdump snooping on my lan and it looks like Apple TV units are actually the culprits here, handing out ULA addressing for whatever reason via RAs, possibly for "Thread"? I don't even know what that is.

Please, check with https://github.com/Zebradil/cloudflare-dynamic-dns/releases/tag/2.1.0 or https://github.com/Zebradil/cloudflare-dynamic-dns/releases/tag/2.1.1
With the new releases, you can now specify preferred subnets.


Adds collections integrity unit tests

Also adds updated - added resources and made other small adjustments - collections to collections/ folder.

Also, I've already removed the test collection in https://github.com/olmps/memo/pull/276. Will merge both #275 and #276 so you can rebase on that.

Will these collections properly run at the same time that the workflow is merged into main?
Like, we're developing in server, then, when merging into main, will the .github/ changes work effectively with the script itself?

Good catch. Updated update-collections workflow to run the integrity tests before running the collections sync.

Good catch. Updated update-collections workflow to run the integrity tests before running the collections sync.

But does it change the fact that all collections are going to be merged before the workflow is available?

But does it change the fact that all collections are going to be merged before the workflow is available?

I didn't understood your question. The update-collections is the one who merges the collections in Firestore. If it fails in the tests lane, it doesn't continue with the merging process.

This workflow will only work when changes are pushed to main branch. All collections here in this pull will be available at the same time as the workflow in the main branch, which, in my naive perspective, wouldn't run the sync, as it will only detect such changes, in that particular folder, after it has been available in main.
TLDR: shouldn't we split merging the collections after the server is available in main?

Actually I believe they'll be synched, because the workflow runs on every push on main detecting the changes since the last merge commit, which includes all added collections. It'll compare the merge commit in main with b15d97, which is the last commit, and in this difference these added collections will be considered.


Update to 2.6.0

cookiecutter 2.6.0
Destination channel: {defaults}
Links

PKG-4281
Upstream repository
Upstream changelog/diff

Thanks for the reviews, guys!


Small change to allow for strictNullChecks

Re: #5
Just changed the flow (if condition for popperOptions.modifiers potentially being undefined and a slight work around for querySelector returning Element | Null and boundariesElement being typed to string | Element | undefined in popper.js types.
Didn't want to make too radical a change, as bit unfamiliar with code base, but the show method could be a bit cleaner.
Also fixed a typo (node_module -> node_modules) in .gitignore

@MrFrankel would you be able to bump to version 1.0.4 to include this?

@ryanhyslop bumped and published


Lua indent mishandles "elseif"

As of c22996, Lua indent fails to outdent the line containing elseif. The code formerly relied on else being a substring of elseif to match.
Fix: add an elseif\> alternate to the pattern, i.e. '^\s*\%(end\>\|else\>\|elseif\>\|until\>\|}\)'
Maintainer could not be contacted (email bounced).

diff --git a/runtime/indent/lua.vim b/runtime/indent/lua.vim
index 5f049d4..393994c 100644
--- a/runtime/indent/lua.vim
+++ b/runtime/indent/lua.vim
@@ -54,7 +54,7 @@ function! GetLuaIndent()

   " Subtract a 'shiftwidth' on end, else (and elseif), until and '}'
   " This is the part that requires 'indentkeys'.
-  let midx = match(getline(v:lnum), '^\s*\%(end\>\|else\>\|until\>\|}\)')
+  let midx = match(getline(v:lnum), '^\s*\%(end\>\|else\>\|elseif\>\|until\>\|}\)')
   if midx != -1 && synIDattr(synID(v:lnum, midx + 1, 1), "name") != "luaComment"
     let ind = ind - &shiftwidth
   endif

@muhmuhten
Thank you for reminding us.
[The issue] (https://groups.google.com/forum/#!topic/vim_dev/uXHoW2eXdgs/discussion) was already raised in our mailing list with a patch, which is the same as yours.
To the list: On this opportunity, I do think now is the time to include the patch, resolving another issue on maintainer.

Are you the maintainer of indent/lua.vim? Then please send the complete and updated indent script to Bram for inclusion. Thanks.

@chrisbra
No, I'm not the maintainer.
As can be seen in the thread I quoted above, about five months ago, Akira Sekiya, who initiated the thread and proposed a patch there, sent a Cc to marc.. at bol.com.br, the email address of the maintainer of indent/lua.vim.
As Akira hasn't sent us any followup so far, we don't know what happened to his patch at all.
That said, if the maintainer accepted Akiraa''s patch and did something with it properly, we would have seen the patch included in the repo.
And this time, @muhmuhten wrote:

Maintainer could not be contacted (email bounced).

Obviously, for some reason, the maintainer fails to fulfill his commitment as maintainer.
When a situation like this happens, you usually ask someone (often a person who reports an issue and proposes a patch for fixing it) for taking over maintenance work, don't you?
P.S.  I quoted that old thread, but I didn't mean by that @muhmuhten must not be qualified as candidate; I simply wanted to emphasize the issue is such a long standing one that shouldn't be left behind.

I'll include the change.  I do hope we can find another maintainer.


Merge branch 'master' of https://github.com/tinygrad/tinygrad

Refactor Tensor.no_grad and Tensor.wino class variables to use idiomatic with statements and context manager for auto save/restore blocks.

Way too big of a change for this! Terrible PR name


How To enable VSS snapshots using powershell

Hi,
I have script that takes snapshot perodically and it working good. When i try to enable VSS using this reference http://googlecloudplatform.github.io/google-cloud-powershell/#/google-compute-engine/GceSnapshot/Add-GceSnapshot
by adding parameter ‑GuestFlush or ‑VSS is not working. Can anyone give me an example how the VSS is working in the powershell command.
Thanks

@ihabhady, thanks for using the module. Can you provide the exact command that you use?

foreach ($Zone in $Zones) {
$DisksInZone = Get-GceDisk -Project $Project -zone $Zone | foreach { $_.Name }
foreach ($Disk in $DisksInZone) {
    Write-Host "=========================================="
    Write-Host "$Zone "-" $Disk"
    Write-Host "=========================================="
    Add-GceSnapshot -project $Project -zone $Zone $Disk -VSS
    }

}
I used -VSS and ‑GuestFlush but not working.

@ihabhady Do you happen to have the error message when you run the command? Also, I believe VSS snapshots work only on Windows instances with image version v20160810 and newer.


docs: add integrations section

Would suggest that we add a new top-level section to the docs called "Integrations", where we document how to get set up with GitHub, GitLab, Azure, Bitbucket, and any other that might come 😅
Could be something like an overview section that explains the basics, where config goes, which parts of Backstage interacts with integrations, etc. Then perhaps one page per integration with more in-depth documentation on configuration of each one, the various flavors, as well as interaction and further configuration for the different core features.

@timbonicus You seem to be working on this one, but the root ask from @Rugvip here is already addressed: there is an integrations section.  Not sure if you're working on an internal story or not, but wondering if we want to close this and pivot it to say separate stories, one per integration, that could possibly allow different teams in the community who may have experience with one or another contribute them back into this section?  Thoughts?

@adamdmharvey Yeah, seems like this can be closed. I'm working on another PR with a similar treatment for auth providers. Maybe some cross-references between the docs will be sufficient? Or maybe there is a need for "BitBucket all the things" type tutorials/pages that link out to integrations, auth, etc docs.

I think the "all the things" page could be integrations/bitbucket/index.

I'd say this is done, and is now just an ongoing improvement thing.


format StmtBreak

Summary
format StmtBreak
trying to learn how to help out with the formatter. starting simple
Test Plan
new snapshot test

do i need any custom comment handling like in https://github.com/astral-sh/ruff/blob/30574e1a196912baaa94620fc217f53b647c41b5/crates/ruff_python_formatter/src/statement/stmt_while.rs#L21  ? if not, how come it's needed for while?

FormatNodeRule has default implementations for fmt_leading_comments, fmt_trailing_comments and fmt_dangling_comments which do the right thing for break (a comment can be on its own line before the break or trailing the break either on the same line or on its own line line after).
For while it's more complicated, e.g. you can have
while x: # trailing condition comment, do not indent
    body # trailing body comment, keep with body
    # trailing body comment, keep with body
# leading else comment, keep with else
else:  # trailing else comment, do not indent either
    print("else")

Since our AST is often lacking nodes for parts of the syntax, we often have to set comments as "dangling" in placements.rs and handle them manually in the node formatting. For the while case there is e.g. no node for the else, we get the or body as a Vec<Stmt>. That means we get comments once for the entire while-else statement and have to parse out the leading else comments. Similarly the : doesn't have any representation, so we mark trailing comments after a colon here.

thank you, i sort of followed that. will hopefully become more clear as i dig in more

Thanks @davidszotten!


Links in traceback for paths with spaces are broken

This is essentially a duplicate of unresolved issue microsoft/vscode-python#7491 (closed and conversation locked) which was moved to microsoft/vscode/issues/81190 (closed and conversation locked) which is marked as a duplicate of microsoft/vscode/issues/21125 which is marked as fixed (closed and conversation locked).  But the original issue for this extension is still unresolved.  At https://github.com/microsoft/vscode/issues/97941#issuecomment-685828441 it seems to be indicated that there is now a path for this extension to address the behavior, so maybe the original issue can be re-opened, and this issue marked as a duplicate.
Environment data

VS Code version: 1.49.0
Extension version (available under the Extensions sidebar): v2020.8.109390
OS and version: Ubuntu 20.04
Python version (& distribution if applicable, e.g. Anaconda): 3.7
Type of virtual environment used (N/A | venv | virtualenv | conda | ...): pyenv
Relevant/affected Python packages and their versions: NA
Relevant/affected Python-related VS Code extensions and their versions: NA
Value of the python.languageServer setting: Pylance

Expected behaviour
When i click a link in traceback which has a space in the path, i expect the corresponding file to open in the editor (ideally at the line number indicated in the traceback).
Actual behaviour
The UI shows a link for the first and last parts of the path, but the link(s) don't trigger any behavior at all.
Steps to reproduce
See https://github.com/microsoft/vscode-python/issues/7491

@karrtikr please see https://github.com/microsoft/vscode/issues/97941#issuecomment-685828441
While paths with space detection would be lovely in core, we have not yet gotten to a nice solution that doesn't compromise performance. We also have the issue of slowing down every terminal when we add yet another regex pattern to match, even if they never use Python.
However, we have adding a link provider API recently that allows extensions to detect and handle links in the terminal. This would allow the Python extension to handle all links of this form however it wants to:
File "<file>", line <line>, in <symbol>

It would even provide a separate link to <symbol>. Since this is a Python link, it makes sense for its detection to live in the Python extension which the majority of Python users should have installed.

Duplicate of #13938


Help with combine() function

Hello. I am trying to use the combine() function to add a rectangle to a complex model. I have not bee able to configure the options of the function correctly to get the effect I am looking for. Specifically, I would like to extend the "pixel heart" model of mine by adding a rectangle. I tried using combine() and combineUnion() but haven't been able to get it to work. I did notice that the location where the rectangle is supposed to intersect my model has been broken into 3 paths by combine() (see screenshot). Any ideas as to what I am doing wrong? Thanks :1st_place_medal:
Here is my code that runs in the playground:
var makerjs 			= require('makerjs')
var model = {"paths":{"ShapeLine1":{"type":"line","origin":[0,9],"end":[1,9]},"ShapeLine2":{"type":"line","origin":[1,9],"end":[1,10]},"ShapeLine3":{"type":"line","origin":[1,10],"end":[2,10]},"ShapeLine4":{"type":"line","origin":[2,10],"end":[2,11]},"ShapeLine5":{"type":"line","origin":[2,11],"end":[5,11]},"ShapeLine6":{"type":"line","origin":[5,11],"end":[5,10]},"ShapeLine7":{"type":"line","origin":[5,10],"end":[8,10]},"ShapeLine8":{"type":"line","origin":[8,10],"end":[8,11]},"ShapeLine9":{"type":"line","origin":[8,11],"end":[11,11]},"ShapeLine10":{"type":"line","origin":[11,11],"end":[11,10]},"ShapeLine11":{"type":"line","origin":[11,10],"end":[12,10]},"ShapeLine12":{"type":"line","origin":[12,10],"end":[12,9]},"ShapeLine13":{"type":"line","origin":[12,9],"end":[13,9]},"ShapeLine14":{"type":"line","origin":[13,9],"end":[13,6]},"ShapeLine15":{"type":"line","origin":[13,6],"end":[12,6]},"ShapeLine16":{"type":"line","origin":[12,6],"end":[12,5]},"ShapeLine17":{"type":"line","origin":[12,5],"end":[11,5]},"ShapeLine18":{"type":"line","origin":[11,5],"end":[11,4]},"ShapeLine19":{"type":"line","origin":[11,4],"end":[10,4]},"ShapeLine20":{"type":"line","origin":[10,4],"end":[10,3]},"ShapeLine21":{"type":"line","origin":[10,3],"end":[9,3]},"ShapeLine22":{"type":"line","origin":[9,3],"end":[9,2]},"ShapeLine23":{"type":"line","origin":[9,2],"end":[8,2]},"ShapeLine24":{"type":"line","origin":[8,2],"end":[8,1]},"ShapeLine25":{"type":"line","origin":[8,1],"end":[7,1]},"ShapeLine26":{"type":"line","origin":[7,1],"end":[7,0]},"ShapeLine27":{"type":"line","origin":[7,0],"end":[6,0]},"ShapeLine28":{"type":"line","origin":[6,0],"end":[6,1]},"ShapeLine29":{"type":"line","origin":[6,1],"end":[5,1]},"ShapeLine30":{"type":"line","origin":[5,1],"end":[5,2]},"ShapeLine31":{"type":"line","origin":[5,2],"end":[4,2]},"ShapeLine32":{"type":"line","origin":[4,2],"end":[4,3]},"ShapeLine33":{"type":"line","origin":[4,3],"end":[3,3]},"ShapeLine34":{"type":"line","origin":[3,3],"end":[3,4]},"ShapeLine35":{"type":"line","origin":[3,4],"end":[2,4]},"ShapeLine36":{"type":"line","origin":[2,4],"end":[2,5]},"ShapeLine37":{"type":"line","origin":[2,5],"end":[1,5]},"ShapeLine38":{"type":"line","origin":[1,5],"end":[1,6]},"ShapeLine39":{"type":"line","origin":[1,6],"end":[0,6]},"ShapeLine40":{"type":"line","origin":[0,6],"end":[0,9]}}}

var rectangle = new makerjs.models.Rectangle(1, 1)

makerjs.model.move(rectangle, [-.5,7]) // position so it overlaps off the left side by .5
makerjs.model.combine(model, rectangle,true,true,false,true)

module.exports = model

and the screenshot:

Thanks for the code snippet, that really helps. You just need to add the rectangle to your output model:
model.models = {
  myRectangle: rectangle
};

That worked! thanks for the quick reply. I came across combine() in the docs and realized it was going to do the box notching mentioned in story #246 much better than the solution I had. It is interesting that rectangle is also mutated by the combine operation.

Yes the mutations are an interesting side effect. Let me know your feedback on that. It's primarily to keep the modeling in the same structure that you create.

Check out the first few sections in http://microsoft.github.io/maker.js/docs/advanced-drawing/#content

After using combine() you might end up with a mix of paths and models within your main model. I could see that being annoying for some people. In that case however, would calling simplify() clear out the sub-models and simplify the model into just a set of paths?

I tried to mimick Adobe Illustrator in the combine() function. Not sure if you use AI, but when you select 2 shapes, you can choose "combine" from their menu. This mutates both shapes into one. So that's how the mutation came to be.
It would be fairly simple to write a non-mutating wrapper. Here's the pseudo-code:

Clone modelA and modelB.
Perform combine on the clones.
Return a new model, which has 2 child models, cloneA and cloneB.

BTW combine does not create sub-models, but it does create more paths. Every path is broken into two, at the point where it intersects a path in the other model. These broken bits exist in the same place in the tree; so they need to have a unique but similar name.
Now everything is completely broken to the point where there are no intersections at all. Then the unwanted bits are removed, whether you wanted the inside or outside ones.
The simplify() function does the opposite of the above, it takes broken bits and reunites them. Well, it will unite any line on the same slope, or any arc on the same origin/radius. Perhaps it should be renamed...
But it does not do tree pruning, which may be what you are thinking of. After a combine, you may have removed entire swaths of a drawing leaving no renderable paths, but the tree structures remain. I was thinking of implementing a prune function to clean up trees after a combine, but this introduces other problems. There may be code which expected a certain structure to a given model. Typically it is ok to have a null at the end of a tree route, like model.models.foo.paths.bar - where bar is undefined, but if the route had a null along the way the code would fail and terminate.

After reading your question again, you might be referring to a concept I call "flattening" - so there is not much of a tree, just one model with all paths. This is possible but there's also downsides. Having models within models is desirable because each sub-model can be measured as a group, and when doing further combine() operations it is helpful to cache the bounding box of each sub-model. Then, some intersections can be avoided altogether if we know they are outside of a bounding box.

Thanks for the clarification Dan. I think flattening for me would be nice as it simplifies the data schema, though I see your reason for not doing it by default. For my project I am all set currently. I was able to get everything working and am looking to move to the laser cutter this week for some prototypes.


[WIP] Fix the IntuneAppProtectionPolicyRequiredException for token response

In this case, the browser cookie persistent after Company Portal installed if broker enabled the browser access. Thus, the authorization request got succeeded. And the token endpoint returned the AADSTS53005 error. I checked the MSAL code. We did not catch the IntuneAppProtectionPolicyRequiredException exception from the token result. Instead, it returns a general MSALServiceException.

This was specifically removed from generic ExceptionAdapter as this only needs to thrown in `Joined account cases.
See bug : https://github.com/AzureAD/ad-accounts-for-android/issues/1000
PR's to fix the bug : https://github.com/AzureAD/ad-accounts-for-android/pull/1019
https://github.com/AzureAD/microsoft-authentication-library-common-for-android/pull/521
This change will regress the behavior. I don't follow the use case here, why would MSAL make a request if broker is installed and has a joined account in this case ?

This was specifically removed from generic ExceptionAdapter as this only needs to thrown in `Joined account cases.
See bug : AzureAD/ad-accounts-for-android#1000
PR's to fix the bug : AzureAD/ad-accounts-for-android#1019
#521
This change will regress the behavior. I don't follow the use case here, why would MSAL make a request if broker is installed and has a joined account in this case ?

I'm trying to understand the reason we ran into the issue in #521 in the first place.

Why are we treating the same error code differently based on the WPJ state? If we are not expecting them in non-joined flow, why are we getting back the error in non-joined flow?
Did the MAMSDK issue happened because we didn't have enough information to fill the error object? (Specifically, UPN). If we expand ExceptionAdapter to take all of those necessary values, would that fix the issue?

This was specifically removed from generic ExceptionAdapter as this only needs to thrown in `Joined account cases.
See bug : AzureAD/ad-accounts-for-android#1000
PR's to fix the bug : AzureAD/ad-accounts-for-android#1019
#521
This change will regress the behavior. I don't follow the use case here, why would MSAL make a request if broker is installed and has a joined account in this case ?

I'm trying to understand the reason we ran into the issue in #521 in the first place.

Why are we treating the same error code differently based on the WPJ state? If we are not expecting them in non-joined flow, why are we getting back the error in non-joined flow?
Did the MAMSDK issue happened because we didn't have enough information to fill the error object? (Specifically, UPN). If we expand ExceptionAdapter to take all of those necessary values, would that fix the issue?

This is because the service thinks the device is still registered (Device registration is not tied to user) and throws this IntunePolicyRequiredException for non-joined case. This is the same issue Heidi is trying to fix (where service thinks broker is installed and registered due to a browser cookie, even though it's not)

We might still get the upn from login hint, but we  would never have the uid in the case which is necessary for MAM to resolve this. Unless it's registered we can't get the uid.

@kreedula @rpdome
The scenario of this issue.

The true mam user joined the device with the broker installed and enable the browser access to the certificate inside the broker settings.
The user uninstalled the broker app. (The certificate is not removed when the broker uninstalled. Because on most OEM Android devices, we do not have a way to clear the WPJ state in the system keychain when CP app was uninstalled. In this case, System browser can theoretically continue to use the WPJ state when it should not.)
The true mam user opens an MSAL app and trying to login.
As the broker is not installed, msal will start the login with the browser.
After entering the credentials, as the WPJ state is still true, it continues to the token request.
At this point, the server returns the "protection_policy_required" suberror from the token endpoint to the native MSAL app.
It is expected to throw the IntunePolicyRequiredException at this point to the MSAL app.
The MSAL app calls the MAM sdk and make a silent call to MSAL to get the token.

For this issue, the broker is not installed on the device. And the IntunePolicyRequiredException will only be thrown when getting the "protection_policy_required" suberror.
As to the required fields, like uid and tid, they are included in the ClientInfo from the authorization response returned in step 5. To get the client info, the authorization request needs to pass the clientinfo=1.

This issue needs more discussion with server dev. Two ways  to fix the issue.
Fix from the server side. Send the user-agent in the token request where the token endpoint will redirect the uri to the broker installation page when getting the IntuneAppProtectionPolicyRequiredException sub error.
Fix from the client side. When getting the IntuneAppProtectionPolicyRequiredException from token endpoint and a browser interactive flow is happening, the client side need to return the token error response saying the broker is not installed and restart with an embedded web view interactive flow which will trigger the broker installation page at the :auth endpoint.

I'm having same problem on Android where IntuneAppProtectionPolicyRequiredException is been thrown after the call to acquireToken. Is there any way to resolve it ? I'm not clear on "embedded web view interactive flow"
Thanks.


How to fix this error

http://i.imgur.com/W1KDWH7.png

Do npm install quad-node

read installation instructions on main page.
The following command needs to be executed from server folder:
npm install

THX :)


Update an order Completed

update an order completes and testing requires implementation.

It failed to pass the test: <E   It should Update an Order within 24 hours of placing it.>. With error: AttributeError: 'str' object has no attribute 'name'.


dont merge. for bot testing purposes!!

retest this please

retest this please

retest this please

retest this please

retest this please

retest this please

retest this please

retest this please

retest this please

retest this please

retest this please

retest this please

retest this please

retest this please


Allow passing implementation address to factory

A single factory with a frozen implementation address was chosen to

Ensure users could get the same address across as many chains as possible
Ensure users would opt into all upgrades and never be forced to upgrade via a factory on some chain having a new implementation.

However, the downside of our current approach is that totally new users in the future will create an account and then immediately need to call to upgrade their account to the last version. It would be better if new users could pass the implementation they want to use. They then would need to pass this same implementation on all future chains where they want the same address. This PR enables that.
Supersedes #50

Decided against this:

The EmptyUUPS is moot if we need implementation determining address for security
Users need to remember first implementation passed and pass again on every new chain to get same address. Easier to solve with factories.


Handle count before commands

Hello,
Thanks for this great plugin !
Sill, how can we use a count before a command ? (ex: 42G goes right at the bottom even if there are more than 42 lines, where G is mapped this way: keymap('n', 'G', "<Cmd>lua Cinnamon.Scroll('G', 0, 0, 3)<CR>", opts))
Regards.

Thank you for liking it!!
The third argument for the function enables using a count so 'G' should be mapped to:
keymap('n', 'G', "<Cmd>lua Cinnamon.Scroll('G', 0, 1, 3)<CR>", opts)

I updated this for the README and the 'extra keymaps' which will be generated to fix this issue.
Could you please try updating the plugin and let me know if you have still have any issues :).
Thanks again!


[Foundation] Modernize hashing in Foundation's Swift-only types

This PR upgrades the Foundation overlay to use the new hashing API introduced in SE-0206. This gets rid of the deprecation warnings for hashValue implementations and brings the overlay in sync with current Swift best practices.
Background
SE-0206 radically changed how hashing works in Swift 4.2+. It is now the stdlib's responsibility to choose a hash function; types implementing Hashable only need to worry about selecting what they want to feed to the hasher. To make this possible, Hashable now requires hashing to be implemented through the hash(into:) method.
The old hashValue property is deprecated as a Hashable requirement, and it's only kept to support source-level compatibility with existing code. Set and Dictionary have switched over to using hash(into:) as their entry point for hashing; they do not directly call hashValue at all.
Beyond simplifying hashing, the intent of SE-0206 is to enable Swift to provide certain guarantees about its quality. In particular: as long as hash(into:) feeds enough data the hasher to unambiguously decide equality, Swift attempts to guarantee that collision attacks won't be possible. For this to work, it is critically important for Hashable implementations to include everything that Equatable.== looks at; and this is especially the case for the basic boundary types that come built-in with Swift, like Data.
The exact hash algorithm used is a private implementation detail of the stdlib; the Hasher API was carefully designed to leak as little information about it as possible, enabling future versions of the stdlib to switch to other algorithms without worrying about ABI compatibility issues.
While random seeding makes the hash algorithm opaque, it is technically possible for Swift code to rely on the particular hash encoding of a stdlib- (or SDK-)provided type. This is highly unlikely to happen in practice, but the longer we wait to fix hashing issues, the more likely it will be that someone starts relying on hashing particulars. For example, I could imagine this to happen through a well-meaning desire to work around a broken hash implementation. It is therefore important to do this sooner rather than later.
Notes
I'll highlight points specific to a particular changes as self-review comments. Here is a list of general remarks:

It is neither required nor desirable for Swift's hashValue to return reproducible values.

We don't need to ever match hash values between Swift types and the Cocoa classes that they bridge with. The act of bridging involves a full rehashing of all Set/Dictionary values.

The only hard requirement is that hashing needs to be consistent with the equivalence classes implemented by == within the same type.

Hashing is meaningless without equality. Equatable only defines equality for values of a single type. Therefore, Hashable makes no requirements or expectations about the hash values produced by values of different types.

It is perfectly fine for hash values produced by different types to not match. (0 as Int8).hashValue and (0 as Int16).hashValue will (typically) not be equal in Swift.

It is also perfectly fine for values of two distinct types to produce matching hash values. Measurement<UnitLength>.hash(into:) and Measurement<UnitDuration>.hash(into:) are allowed to use the same hash encoding, since there is no way they can ever produce large-scale collisions within the same hash table.

While this is not a hard requirement for user code, for boundary types provided in the stdlib/SDK, we require that hashing isn't just consistent with equality, but that it's equivalent to it. The Swift test suite has checks to actively enforce this -- this is possible through repeatedly salting the hash function.
"Optimizing" hashing by omitting some of the data compared by == is generally a mistake in Swift, because it completely breaks all guarantees about the strength of hashing, and opens the door to (accidental or deliberate) collision attacks.
It's perfectly acceptable to hash a gigabyte of data if someone inserts some large value (such as a big collection) as a key in a hash table. Multi-megabyte String keys are easy to protect against; hidden hashing weaknesses aren't.

It would be nice if we could somehow improve hashing in Foundation's Objective-C classes, but this is outside the scope of this PR (or indeed, this repository). However, we should at least make sure Swift programmers can rely on the quality of hashes produced by native Swift value types.

https://github.com/apple/swift-corelibs-foundation/pull/2118 is the corresponding PR for swift-corelibs-foundation. These PRs should land at the same time.

Replacing hashValue implementations on types with hash(into:) has no effect on the symbols exposed as ABI -- the compiler automatically synthesizes hashValue from hash(into:), and vice versa. (This is not the case in protocol extensions, but we've fixed those in the 4.2/5.0 timeframe.) In the Swift 5.0 ABI, all of these types include definitions for both hashValue and `hash(into:) so we don't need to worry about availability issues.
The sole exception is URL, which implements hashValue but isn't Hashable. Unfortunately we can't add the conformance in 5.1, but we should add a (versioned) hash(into:) definition to discourage people from trying to implement it on their own. (Edit: URL is not a special case; it's Hashable like the rest. I got mislead by an unrelated error.)

rdar://problem/43394032

@itaiferber @millenomi can y’all give this a run through? There are a number of changes here but the general concept seems well founded. I am worried that I am missing any hidden gotchas to binary comparability or other developer ergonomics etc.

My main concern, which is just top-of-my-head this very moment, is: what about AnyHashable equality? While all the points about Hashable hold, IIRC currently we hold AnyHashable(x) == AnyHashable(x as AnyObject) for any object that implements _ObjectiveCBridgeable.

I want to highlight the IIRC; I'm not immediately sure and I have to research this, but I do know the ergonomics can be complex there esp. wrt newly bridged dictionaries of [AnyHashable: Any] type.

Overall, most of the changes here look reasonable (and very welcome!) to me.

@millenomi The bridgeable types I touched should all implement _toCustomAnyHashable() by bridging to Swift, so AnyHashable behavior wouldn't regress by these changes.
Possible weird edge case: I can't figure out where Decimal/NSDecimalNumber get their _HasCustomAnyHashableRepresentation conformances. They seem to do the right thing, though.

I submitted a PR for swift-corelibs-foundation with matching changes: https://github.com/apple/swift-corelibs-foundation/pull/2118

@swift-ci please test

Build failed
Swift Test OS X Platform
Git Sha - f859dd1316131cd049b29712be60fc95ee7fff41

Build failed
Swift Test Linux Platform
Git Sha - b711ed9f90222302cebc6ae6e36ba7eb755a7d83

@swift-ci please test macOS platform

Build failed
Swift Test OS X Platform
Git Sha - 13bc5670854483e8021f576a1f643cdd4b19b27d

@swift-ci please test macOS platform

@swift-ci please test linux platform

Build failed
Swift Test OS X Platform
Git Sha - 13bc5670854483e8021f576a1f643cdd4b19b27d

Build failed
Swift Test Linux Platform
Git Sha - 13bc5670854483e8021f576a1f643cdd4b19b27d

@swift-ci please test

Build failed
Swift Test OS X Platform
Git Sha - 71833f495e7c7b262af732d7f2afbdfb56937ca3

Build failed
Swift Test Linux Platform
Git Sha - 71833f495e7c7b262af732d7f2afbdfb56937ca3

02:13:44 ******************** TEST 'Swift(iphonesimulator-i386) :: stdlib/TestPersonNameComponents.swift' FAILED ********************
...
02:13:44 [ RUN      ] TestPersonNameComponents.test_Hashing
02:13:44 stdout>>> check failed at /Users/buildnode/jenkins/workspace/swift-PR-osx/branch-master/swift/test/stdlib/TestPersonNameComponents.swift, line 54
02:13:44 stdout>>> stacktrace:
02:13:44 stdout>>>   #0: /Users/buildnode/jenkins/workspace/swift-PR-osx/branch-master/swift/stdlib/private/StdlibUnittest/StdlibUnittest.swift:2530
02:13:44 stdout>>> expected: 230665695 (of type Swift.Int)
02:13:44 stdout>>> actual: -422011617 (of type Swift.Int)
02:13:44 stdout>>> hash(into:) expected to match, found to differ
02:13:44 stdout>>> lhs (at index 0): givenName: Kevin familyName: Frank
02:13:44 stdout>>> rhs (at index 1): givenName: Kevin familyName: Frank

Hm; NSPersonNameComponents.hash seems not to work correctly on iphonesimulator-i386.

@swift-ci please test

Build failed
Swift Test OS X Platform
Git Sha - 6647c50d410e4f6344782dcee6f14a3b4c35a97b

@parkera @millenomi @itaiferber are we good to go ahead with this (with the Data part subsetted out for further discussion)

Yup


Simplify viewing deployments, use Node.js v20

Tested in byu-oit/hw-fargate-api#1217, as seen here.


Bug in HydroDyn External Buoyancy Force Reported in Summary File

The vertical component of the buoyancy force (BuoyFzi) reported in the HydroDyn summary file is based on the total structure volume rather than the submerged volume.  It sounds like @ghaymanNREL was aware of this bug and offered a fix a couple years ago, but this fix doesn't seem to have been implemented in the current OpenFAST code base.  More information is available in the following topic on the forum: https://wind.nrel.gov/forum/wind/viewtopic.php?f=4&t=1592&p=7576.

@jjonkman is this fix included in FAST v8 or somewhere else?

From the forum post linked above, @ghaymanNREL stated that he had a fix, so, the solution is probably in one of Greg's branches.  Hopefully Greg can respond.


Updated citation to DGBid 4.0

Closes #580

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 37.931%

Totals

Change from base Build 745250924:
0.0%

Covered Lines:
2420

Relevant Lines:
6380

💛  - Coveralls


Colors don't update

i changed the waxed exposed cut copper stairs to white and re rendered it but nothing changed

Unable to reproduce.


Nightly build is broken

I'm looking forward for a version of a recent merged pull request but I've noticed nightly builds are no longer published for quite a while because the build fails: https://dev.azure.com/AvaloniaUI/AvaloniaUI/_release?_a=releases&view=mine&definitionId=2
cc @grokys

Something is wrong with publishing to MyGet, you can use the "all" feed for now. The only downside about it is that it contains builds from PRs.

@kekekeks Thank you. Do you have a nuget link for me I can use in VS?

I've tried the URL from the wiki but it shows me no releases at all: https://github.com/AvaloniaUI/Avalonia/wiki/Using-nightly-build-feed#all-builds-feed

I think this is fixed now thanks to @danwalmsley
https://github.com/AvaloniaUI/Avalonia/wiki/Using-nightly-build-feed
If this works for you I think this issue can be closed.
Happy coding
Tim


Create needed repos during host deployment

Fixes #137
Have the agent deployment mechanism create the required repos for
installing the agent's dependencies instead of requiring that the
storage servers have these repos pre-configured.

One thing to keep in mind is that we check for repos before even deploying the agent. With this model, it would seem that we want to remove that check. https://github.com/intel-hpdd/intel-manager-for-lustre/blob/51cd704fb41c988fd92b4a86cc82492ee32accbe/chroma-manager/chroma_core/models/host_jobs/test_host_contact.py#L96

@jgrund That is indeed interesting because IIRC I have not removed that check but my local SSI for this patch is passing, way beyond having deployed storage servers.  I will look into why it is passing.


Adding support for BlackBerry Launcher

Is there any way I can modify LauncherHelper.java? If so how? Do I have to import library into my app or can I just add a LauncherHelper.java class into my own app?
I'm trying to add BlackBerry Launcher : https://play.google.com/store/apps/details?id=com.blackberry.blackberrylauncher
I looked into LauncherHelper.java and found out that it works perfectly with "applyManual", If I can't modify it is there any way you could add it please?
I tested my Icon Packs with the BlackBerry Launcher and they work perfectly
Thank you in advance
daLight

QUOTE "You should open a new issue instead. Blackberry launcher doesn't  support direct or manual apply from the dashboard. The only way to apply  icon pack is go to settings and apply icon pack from there."
Thanks for your response, isn't that what the applyManual inside LauncherHelper.java is for? It does a popup saying to go to launcher settings? Because that is what I want to do.

Yes, but blackberry launcher settings can't be accessed from the dashboard.

Yes, evie settings can't be accessed.
I'll considering about this.

Thank you very much!
Just so you know, in the BlackBerry Launcher, to apply the pack it is exactly "tap and hold homescreen - settings - iconpack - iconpackname"
So just applying the same popup as Evie would make it perfect.
Thank you very much for your great work!
daLight


Fix freeze when parsing array tables

It seems i was right to worry about accidentally breaking something with that last minor change, neovim started freezing whenever i tried to type in an array table because I fucked up the logic of parsing the table headers. I've done it much more carefully this time, so that shouldn't happen anymore

Huh weird I couldn't really reproduce the freeze but it turns out that only happens on 0.9 not on 0.10.


Cyclic dependency Error while installing Driver or Consumer  Pods in iOS

facing cyclic dependency error after installing Driver or Consumer pod in swift project. how to solve this problem

If you would like to upvote the priority of this issue, please comment below or react with :+1: so we can see what is popular when we triage.
@amresh1 Thank you for opening this issue. 🙏
Please check out these other resources that might help you get to a resolution in the meantime:

Check the issue tracker - bugs and feature requests for Google Maps Platform APIs and SDKs
Open a support case - Get 1:1 support in Cloud Console.
Discord - chat with other developers
StackOverflow - use the google-maps tag

This is an automated message, feel free to ignore.

@mikkin @ShirwaM


Some block properties are missing always

I am currently running

SpongeVanilla version: 1.12.2-7.1.7 and 1.12.2-7.1.6
Java version: OpenJDK 1.8.0_222
Operating System: Kubuntu 18.04 LTS
Plugins/Mods: None

Issue Description
List of properties that I can't get (sorted):
FlammableProperty
GroundLuminanceProperty
HardnessProperty
IndirectlyPoweredProperty
PoweredProperty
SkyLuminanceProperty
TemperatureProperty
UnbreakableProperty

Test code (also tested with BlockSnapshots and BlockStates):
@Listener
public void onInteractEvent(InteractBlockEvent.Secondary.MainHand event) {
    Objects.requireNonNull(event, "event");

    BlockType blockType = event.getTargetBlock().getState().getType();
    Logger logger = getLogger();

    for (Property<?, ?> property: blockType.getApplicableProperties()) {
        logger.info(property.getClass().getSimpleName() + " -> " + property.getValue());
    }
}

A sample output for a bedrock block with the present properties (sorted):
BlastResistanceProperty -> 3600000.0
FullBlockSelectionBoxProperty -> true
GravityAffectedProperty -> false
HeldItemProperty -> ItemBlock{Name=null}
InstrumentProperty -> SpongeInstrumentType{id=minecraft:bass_drum, name=Bass Drum}
LightEmissionProperty -> 0
MatterProperty -> SOLID
PassableProperty -> false
ReplaceableProperty -> false
SolidCubeProperty -> true
StatisticsTrackedProperty -> false
SurrogateBlockProperty -> false

UPD:
I also tested item properties. All properties are available when they are applicable to an item item, but in the case of blocks, in addition to the properties of the item, the list contains the properties of the block, but not all.
Here is a sample output for a bedrock item stack (sorted):
FullBlockSelectionBoxProperty -> true
HeldItemProperty -> ItemBlock{Name=null}
InstrumentProperty -> SpongeInstrumentType{id=minecraft:bass_drum, name=Bass Drum}
LightEmissionProperty -> 0
MatterProperty -> SOLID
SmeltableProperty -> false
SolidCubeProperty -> true
StatisticsTrackedProperty -> false

It seems that AbstractBlockPropertyStore is passing null locations to the handlers as the blockstate/block doesnt contain location info, which is what is breaking this in part

The properties show up as expected when the block location is queried for the data,
This data can't be provided with just the block or the blockstate by itself as the location data is needed for these to be gathered.
package org.spongepowered.test;

import com.google.inject.Inject;
import org.spongepowered.api.Sponge;
import org.spongepowered.api.block.BlockState;
import org.spongepowered.api.command.CommandSource;
import org.spongepowered.api.data.Property;
import org.spongepowered.api.entity.living.player.Player;
import org.spongepowered.api.event.Listener;
import org.spongepowered.api.event.block.InteractBlockEvent;
import org.spongepowered.api.event.filter.cause.First;
import org.spongepowered.api.plugin.Plugin;
import org.spongepowered.api.plugin.PluginContainer;

import java.util.Objects;

@Plugin(id = "blockpropertiestest", name = "Block Properties Test", description = "Testing block properties.", version = "0.0.0")
public class BlockPropertiesTest implements LoadableModule {

    @Inject
    private PluginContainer container;

    private final BlockPropertiesListener listener = new BlockPropertiesListener();

    @Override
    public void enable(CommandSource src) {
        Sponge.getEventManager().registerListeners(this.container, this.listener);
    }

    public static class BlockPropertiesListener {
        @Listener
        public void onInteractEvent(InteractBlockEvent.Secondary.MainHand event, @First Player player) {
            Objects.requireNonNull(event, "event");

            System.out.println("Properties at location");
            for (Property<?, ?> property : event.getTargetBlock().getLocation().get().getApplicableProperties()) {
                System.out.println(property.getClass().getSimpleName()+" - > "+ property.getValue());
            }
            System.out.println();

            System.out.println("Properties on blockstate");
            BlockState state = event.getTargetBlock().getState();
            for (Property<?, ?> property: state.getApplicableProperties()) {
                System.out.println(property.getClass().getSimpleName()+" - > "+ property.getValue());
            }
        }
    }

}


Align with 1.0.0

The final stable release of Property Graph Exchange Format is going to be published. See https://github.com/pg-format/specification/issues/9 for a summary of open issues to do so. With the release pg-formatter should be fully compliant to the specification.

Some test cases with edge cases

a -> b | b -> a
x:: a -> b
node a:b: c

Looks good. More unit tests required to be sure.


DEBUG=* should cause all queries caused by CLI to be logged

C:\Users\Jan\Documents\throwaway\planetscale>npx prisma migrate dev --create-only
  prisma:loadEnv project root found at C:\Users\Jan\Documents\throwaway\planetscale\package.json +0ms
  prisma:tryLoadEnv Environment variables loaded from C:\Users\Jan\Documents\throwaway\planetscale\.env +0ms
Environment variables loaded from .env
  prisma:engines using NAPI: false +0ms
  prisma:engines binaries to download query-engine, migration-engine, introspection-engine, prisma-fmt +0ms
Prisma schema loaded from prisma\schema.prisma
Datasource "db": MySQL database "fk-test2" at "<IP_ADDRESS>:3306"

Error: Error: undefined: Database error
Error querying the database: Server error: `ERROR HY000 (1105): target: fk-test2.-.master: no healthy tablet available for 'keyspace:"fk-test2" shard:"-" tablet_type:MASTER ''

C:\Users\Jan\Documents\throwaway\planetscale>set DEBUG
DEBUG=*

Although DEBUG=* is set, I have no indication which query was sent that caused this error to be output.
(Might actually be on creating a connection - that should optimally also be logged)

Hmm, I don't think we log any queries in migrate cli so this would be feature?

That could be the case.
Right now I have no idea how to find out what query is causing this error message and why.

For me RUST_LOG=debug was useful, If I remember right, for db push, so it should work for migrate as well. Though, it didn't so anything for db pull.

Actually, at least for sqlite provider (and probably other providers) this works
DEBUG=prisma:introspectionEngine:stderr RUST_LOG=debug npx prisma db pull

DEBUG=* is also valid but RUST_LOG=debug must be set.


multiparts: health check

[ ] check if all multiparts migrated
[ ] check if there is no empty multiparts
[ ] check if all documents created
[ ] check if documents of multiparts have attached correct items
[ ] check if all fields are correct

No item migrated:
https://sandbox-cds-ils.web.cern.ch/backoffice/documents/qjk9v-zyy59
https://cds.cern.ch/record/570932?ln=en


[AVM Module Issue]: Support autorotation for customer managed key in storage account

Check for previous/existing GitHub issues

[X] I have checked for previous/existing GitHub issues

Issue Type?
Feature Request
Module Name
avm/res/storage/storage-account
(Optional) Module Version
No response
Description
Currently if specifying a customer managed key it defaults to an explicit version (either specified in the AVM-module or using latest if not specified). From /res/storage/storage-account/main.bicep
  keyvaultproperties: !empty(customerManagedKey)
          ? {
              keyname: customerManagedKey!.keyName
              keyvaulturi: cMKKeyVault.properties.vaultUri
              keyversion: !empty(customerManagedKey.?keyVersion ?? '')
                ? customerManagedKey!.keyVersion
                : last(split(cMKKeyVault::cMKKey.properties.keyUriWithVersion, '/'))
            }
          : null

This leads to autorotation of the encryption key being disabled on the created storage account:

It should be possible to not specify the keyversion to have automated key rotation work as desired. I believe when you use the resource directly and don't specify a keyversion automated rotation is supported.
(Optional) Correlation Id
No response

Hey @thasimongyldendal,
thanks for raising this issue. Have you been able to confirm that autorotation is enabled when not specifying a version? If so, we should update the AVM spec and update this as well as other modules to allow users to specify that they want auto-rotation enabled (e.g., by specifying as such in the interface / as a value).

Hello @AlexanderSehr
I have confirmed that not specifying a keyversion gives you automatic key rotation.
I used this bicep code to confirm it:
resource storageAccount 'Microsoft.Storage/storageAccounts@2023-05-01' = {
  kind: 'StorageV2'
  name: name
  location: resourceGroup().location
  sku: {
    name: sku
  }
  identity: {
    type: 'UserAssigned'
    userAssignedIdentities: {
      '${managedIdentity.id}' : {}
    }
  }
  properties: {
    encryption: {
      keyvaultproperties: { //Here I did not set the keyversion property
        keyname: keyName
        keyvaulturi: keyVaultUri
      }
      identity: {
        userAssignedIdentity: managedIdentity.id
      }
      keySource: 'Microsoft.Keyvault'
    }
  }
}

And the result in the portal:

I'll see to create a PR once the new RBAC schema was roled out to the storage account module via #2008 to avoid conflicts

Is there any updates on this? We've just spotted this issue and need to make a decision on whether to move to a custom module for storage accounts depending on time scales.

Hey @Dekor86,
I took a closer look just recently. As this may have an impact on the spec (which in turn would affect all modules with CMK) I want to have collect the feedback from the core team before going ahead and changing the implementation.
The alternatives would be to either

Remove the KeyVersion autofetch completely, leaving it up to the module owner to either require that parameter (if it is required by the Resource Provider), or to default to null (as asked for above)
Or to add another boolean switch that enables/disables the functionality that would automatically fetch the latest version at deployment time for the user

There's a call happening this Thursday that I hope will bring a decision after which we can then go ahead and implement the change. It may take until next week to release the update though.

The decision is unfortunately still pending as we did not come to an agreement yet and are still discussing how to fix the issue

Closing as per #3853


As ruby, crystal has open class?

Im studying this lang, and I'm wondering the possibility to do something like this:
class Foo
end

bar = Foo.new
def bar.bar foo_bar
  puts "this is a '#{foo_bar}'"
end

bar.bar "Foo Bar"
=> this is a 'Foo Bar'

is this possible?

This is a duplicate of #238. You can read/continue the discussion there :-)

But the short answer is that for now we have no plans to implement this.

Note the related concept in Ruby is called singleton class. The open class model is what allows
class Foo
  def foo
  end
end

class Foo
  def bar
  end
end

Yeah, I was wondering about this feature because i was studying the Crystal active_record.cr, with this feature, work with schema is going to be more clean then define field on class as MongoID on ruby.
But looking for this feature as I said https://github.com/manastech/crystal/issues/238#issuecomment-138600185, Singleton Class Concept and Open Class is very useful, but i don't know if is viable to sacrifice performance for it.


Incompatibility with PopOut Resizer

If PopOut Resizer is active, the chat window will be placed near the middle of the screen and have the wrong size. Using the settings to change its position will also have no effect.
I haven't tried it with any other windows (like the combat  tracker), but I suspect the behavior may be similar.

I also posted about this issue at PopOut Resizer's github

I don't believe there's anything sensible I can do fix this from my end. I'll leave a note for Cardagon on how to avoid messing with the stream user's popouts, in case they're willing to add compatibility.


Bike Segements

Please add Bike Segments
Error in $: When parsing the record AthleteDetailed of type Strive.Types.Athletes.AthleteDetailed the key bikes was not present.

Oh, there are bike segments, it just doesn't work now. I need to update the Strava library and rebuild. :-)

Should be fixed now.


CTRL-SHIFT-J for dev tools

How about CTRL-SHIFT-J for dev tools?
That's the Chrome default. Any objection to changing it from ALT-CTRL-I?

I am used to f12. But CTRL-SHIFT-J is fine for me.


Internationalization - AppLocalizations.of(context) null using VMaterialApp

Hello,
I followed the documentation for implementing internationalization https://flutter.dev/docs/development/accessibility-and-localization/internationalization. While doing so I found out that AppLocalizations.of(context) returns null when using VMaterialApp. Is this a known issue, or is there a workaround?

I realize that VMaterialApp is badly named and badly documented so that's my bad as well


Fix issues in TypeScript getting started example code

Which problem is this PR solving?
Problems in the TypeScript getting started example:

The use of a non-generic type as generic
Gaps in the TypeScript type coverage

Short description of the changes

Change the use of Metric<BoundCounter> to Counter (Metric is no longer generic, so this didn't compile)
Add @types/node so TypeScript can compile app.ts which uses process.env
Add @types/express so the express API is no longer any
Define the the result type of the countAllRequests function as an Express RequestHandler so that req, res, and next are no longer any

I noticed 2 more things while I was following the guide:

typescript is not a (dev)dependency, so the npm scripts don't work (well, they might inside the opentelemetry-js repo due to phantom dependency).
the README.md of the guide explains step by step how to arrive at the state in the folder and includes code samples and npm install commands. We'll want to update the readme to keep it in sync.

I'll try to update this PR soon, probably tomorrow.

I am going to merge this, with a follow up to lint the getting started guide


How to import audio file?

How could I import audio file (such as .wav files) just like 'PyFbxFactory'?
Or there's anyother way?

Hi
import unreal_engine as ue

from unreal_engine.classes import SoundFactory

factory = SoundFactory()

sound_asset = factory.factory_import_object('D:/foo.wav', '/Game/FooSound')

Here you find the list of all the factories:
https://api.unrealengine.com/INT/API/Editor/UnrealEd/Factories/UFactory/index.html

Thank you
But when I add a MovieSceneAudioSection to my sequencer,UE crashed

Do you have a traceback to post for the crash ?

You can select a sequencer run that script I post,Engine crash directly

Ok, basically in 4.20 the CreateNewSection for audio tracks crashes ue.
If you update to latest plugin code you can force the creation of a specific section (like you were doing with your code by passing the section type).
If you cannot upgrade you can use this workaround:
import unreal_engine as ue
from unreal_engine.classes import MovieSceneAudioTrack , MovieSceneAudioSection

selSeqs = ue.get_selected_assets()

if selSeqs == []:
	ue.message_dialog_open(ue.APP_MSG_TYPE_OK, "Please Select One Sequencer At Least ！ ")
else:
    thisSeq = selSeqs[0]
    track = thisSeq.sequencer_add_master_track(MovieSceneAudioTrack)
    section = MovieSceneAudioSection('', track)
    track.AudioSections = [section]
    print(section)

Ok,Tnang you


Backport to v3

Backport PRs #258 and #260 to the v3 branch.

Note: these return types are not in the 2.2 branch code, so there is nothing to backport for 2.2.*

THX


bundle command fails, attempts to load from non-existent path

It appears that rubygems 2.7.5 (released on 6-Feb-2018) introduces an issue with a newly built instance of rails-dev-box. The bundle command fails, referencing a non-existent path:
vagrant@rails-dev-box:~$ bundle --version
/usr/local/bin/bundle:23:in `load': cannot load such file -- /usr/lib/ruby/gems/2.4.0/gems/bundler-1.16.1/exe/bundle (LoadError)
	from /usr/local/bin/bundle:23:in `<main>'
vagrant@rails-dev-box:~$ which bundle
/usr/local/bin/bundle

My environment:
Vagrant 2.0.2
Virtualbox 5.1.28
rails-dev-box: 83efeddf8adfe17ee250e5a03a0d489267bf444e
I have done limited research, and see the following:

If I access /usr/bin/bundle2.4 directly, bundle executes as expected:

vagrant@rails-dev-box:~$ /usr/bin/bundle2.4 --version
Bundler version 1.16.1

If I modify bootstrap.sh to use rubygems version 2.7.4 instead of the latest (gem update --system 2.7.4 -N >/dev/null 2>&1) I am able to access bundle as expected.

If I add export DONT_USE_BUNDLER_FOR_GEMDEPS=1 to the bootstrap script (prior to the gem update and gem install), I am able to access bundle as expected. However, support for this environment variable was added in rubygems 2.7.5, and the standard behavior prior to this version was for this to be set to false, so I would not expect to need to set this variable.

At this point I am unsure if the bootstrap script should be modified or if this points to an issue with rubygems.

Similar issue has been opened at RubyGems https://github.com/rubygems/rubygems/issues/2180

Provisioning now installs Ruby 2.5 and does not upgrade RubyGems, which is 2.7.3.
Thanks for investigating this!


Put initialisers and events in a condition

What?
Analytics are currently initialised on first load. If a user doesn't give us permission for those, that step shouldn't even happen. We should wrap initialisation (and event helpers while we're at it) in a conditional that checks if we have consent to do it. I'd do it at the helper level so we only do it once and not bloat the code across.
Something like:
<Head>
+     {consentForAnalytics && (
          {/* Adding toggles etc. to the datalayer so they are available to events in Google Tag Manager */}
             <Ga4DataLayer
              data={{
                toggles: this.props.toggles,
              }}
            />
           <GoogleTagManager />

          <script
            dangerouslySetInnerHTML={{ __html: renderSegmentSnippet() }}
          />
 +    )}
</Head>

function track(conversion: Conversion) {
+ if (!consentForAnalytics) return;

  const debug = Boolean(getCookie(cookies.analyticsDebug));

  // We make toggles available of the dataLayer for GTM, so can use them here too
  const togglesString = window?.dataLayer?.find(data => data.toggles)?.toggles;
  const toggles = togglesString ? togglesString.split(',') : [];
  const sessionId = getSessionId();
  const session: Session = {
    id: sessionId,
    timeout: sessionTimeout,
  };
  const { eventGroup, properties, ...restConversion } = conversion;

  localStorage.setItem(lastTrackedLocalStorageKey, Date.now().toString());

  if (debug) {
    console.info(eventGroup, {
      session,
      properties: { ...properties, toggles },
      ...restConversion,
    });
  }

  window.analytics.track(eventGroup, {
    session,
    properties: { ...properties, toggles },
    ...restConversion,
  });
}

I'm not sure exactly what it would look like at this stage as I don't know yet how we'll be checking for consent, but we could do it with a mock variable that is currently always true and then just change that bit with the bulk of the work done.
Why?
To be in-line with consent laws.
Background
Preparatory work to move us to a compliant cookie implementation which doesn't place non-essential cookies until active consent has been granted by user.
Done when

[ ] No cookies are added and no events are sent over to our tracking platforms should the consent value be false.

@rcantin-w Created this on the back of your comment from cookies kick-off

I've added some information in the ticket, maybe @kenoir could review when he's back to confirm that's what he was agreeing with when I mentioned it 👍

This makes sense, we also need to think about what happens when a user revokes consent after the page has been loaded after we record consent choice outside of (i.e. they change their mind).
There is also something for another issue about understanding when re-consent is required and being able to trigger that process, for example if we were to add another analytics cookie or the purpose for which we are using the data collected changes we must be able to ask all users to re-consent. From an engineering point of view we need to be able to trigger a re-consent of users somehow outside of cookie expiration.

This makes sense, we also need to think about what happens when a user revokes consent after the page has been loaded after we record consent choice initially (i.e. they change their mind).

Absolutely. I might leave it as part of the "integrate with 3rd party" part of the work though, as that would affect how we set the condition/detect the change. In the past I'd that action/function on the "Save" button of the "Manage cookie preferences" modal, so we'll have to figure out how it works with whatever solution we end up with?
I believe it'll mostly be about removing the cookies (we use cookies-next and they offer a deleteCookie() helper) and force a page refresh so the conditionals take effect, which should be easy enough (famous last words). It'll also depend on which cookies we need to remove, some offer some ideas.

There is also something for another issue about understanding when re-consent is required and being able to trigger that process, for example if we were to add another analytics cookie or the purpose for which we are using the data collected changes we must be able to ask all users to re-consent. From an engineering point of view we need to be able to trigger a re-consent of users somehow outside of cookie expiration.

Very true again. Last time I did this our consent cookies was actually an stringified object, allowing us to know which cookies had been accepted. I wonder if we could do a check on first load, if there's a change in what the shape should look like, the banner comes up again? Again, I'm not sure how we could utilise the 3rd party to help with this, or if the stringified object idea would be useful still.

I wonder if we could do a check on first load, if there's a change in what the shape should look like, the banner comes up again?
I like this idea.

Doesn't seem to work in staging. It doesn't BLOCK tracking when it should, it always tracks, so I don't see danger in the code making it to production while I debug.

We found out it should've worked but it wasn't due to a condition in our Cloudfront policy that ignored all cookies without the WC_ or toggle prefix, so we've change it to be WC_consentCookie.
Next steps are to change the getConsent function to work for both client and server side, so moving back to in progress.


Unhandled error in oauth authorize call made for CORS request

The example worked at first but for the last two days I have been getting this error with the call to AAD which is made to get the auth token for CORS request to the ToGoAPI:
"AADSTS90033: Service is temporarily unavailable. Please retry later"
Although this appears to be a server-side issue with AAD, the response code is 200 and it doesn't seem to be triggering the authentication failed event. I think this type of response should trigger the authentication failed event so that it can be handled by the application.
Also, I wanted to share here to see if maybe it's related to compatibility between the example code or adal-js library and the azure AD service. Azure status page reports no problem with Azure AD and the error seems to be confined to this specific scenario. I also have a support ticket open with Azure AD so I will post back here if they find anything on their end.

I imagine this is on the Azure side, but I just saw this error 2016-04-29 10:45:50Z which was the first time for me. Other calls to my AAD instance for tokens immediately before and after succeeded.

Closing this as we are archiving this sample. Please check out the new sample at ms-identity-javascript-angular-spa-aspnetcore-webapi.