ValueError: Graph has cycles.

Hello, and thank you so much for this huge undertaking! I'm working on trying to convert Mozilla's DeepSpeech pre-trained model, and I get the error: "ValueError: Graph has cycles."
Any ideas about what I could do for this? Are graphs with cycles simply not going to work?

CoreML does not support graphs with cycles at the moment. However, it does support recurrent nodes (RNN, LSTM, GRU etc). When these cells are implemented in TF, they get broken down into cyclic graphs in the frozen .pb file. tfcoreml cannot handle such graphs at the moment. This is a known issue currently. Handling such graphs would require going from the broken down graph structure to the higher level structure and mapping the nodes to an operation that CoreML supports (RNN, LSTM etc).
I think the simplest way for to deal with such graphs now, is to abstract the weight matrices and bias vectors. And then use them to build a CoreML model directly using the neural network builder API provided by coremltools.

Yes, YOLO model does not have cycles, so that should work.

what about Faster-rcnn?

When we can use tf.nn.bidirectional_dynamic_rnn?

@aseemw Any plan to support BLSTM?

@aseemw Could you please tell me about a frozen_inference_graph.pb created with the help of Tensorflow.
I haven't converted it to be used for Mobile Apps. But I directly tried it to be converted into .MlModel with TensorFlowtoCoreML.
Gave me the same error.

+1 for BiRNN support

@aseemw

And then use them to build a CoreML model directly using the neural network builder API provided by coremltools.

Your provided API link doesn't work (404)

This is link: https://apple.github.io/coremltools/generated/coremltools.models.neural_network.builder.html

I wonder where ssd model contains cycles? it look like just feedforward network.
WARNING:root:TensorFlow version 1.12.0 detected. Last version known to be fully compatible is 1.5.0 .

Loading the TF graph...
2019-03-11 12:34:45.237621: I tensorflow/core/platform/cpu_feature_guard.cc:141] Your CPU supports instructions that this TensorFlow binary was not compiled to use: AVX2 FMA
Graph Loaded.
Traceback (most recent call last):
  File "ssd_sandbox/convert_from_tensorflow_to_coreml.py", line 20, in <module>
    convert_from_tensorflow_to_coreml(is_post_training_quantize=False)
  File "ssd_sandbox/convert_from_tensorflow_to_coreml.py", line 10, in convert_from_tensorflow_to_coreml
    'detection_classes:0', 'num_detections:0'])
  File "/usr/local/lib/python3.6/site-packages/tfcoreml/_tf_coreml_converter.py", line 586, in convert
    custom_conversion_functions=custom_conversion_functions)
  File "/usr/local/lib/python3.6/site-packages/tfcoreml/_tf_coreml_converter.py", line 167, in _convert_pb_to_mlmodel
    OPS = _topological_sort_ops(OPS)
  File "/usr/local/lib/python3.6/site-packages/tfcoreml/_tf_graph_transform.py", line 194, in _topological_sort_ops
    _push_stack(stack, node, in_stack)
  File "/usr/local/lib/python3.6/site-packages/tfcoreml/_tf_graph_transform.py", line 38, in _push_stack
    raise ValueError('Graph has cycles.')
ValueError: Graph has cycles.

This issue is being closed due to inactivity preceeding the request for an update. Please feel free to re-open. Thanks!

Me too got same error as mentioned by @Mstronach
We used  SSD mobilenet version and generate protobuf file and tried to convert ML model we get same issue - PFA

Any updates on this issue ?


Unable to run basic example

Hi,
I'm not able to run the basic example provided by the documentation I'm under the ^0.1.4 and I tested with different versions.
I am a beginner with Dart may I have done something wrong ?
here the basic example
class MapPage extends StatefulWidget {
  @override
  State createState() => new MapPageState();
}

class MapPageState extends State<MapPage> {
  Widget build(BuildContext context) {
    return new FlutterMap(
      options: new MapOptions(
        center: new LatLng(50.5, 30.51),
        zoom: 13.0,
      ),
      layers: [
        new TileLayerOptions(
          urlTemplate: "https://api.tiles.mapbox.com/v4/"
              "{id}/{z}/{x}/{y}@2x.png?access_token={accessToken}",
          additionalOptions: {
            'accessToken': 'MYTOKENxxxxxxx',
            'id': 'mapbox.streets',
          },
        ),
        new MarkerLayerOptions(
          markers: [
            new Marker(
              width: 80.0,
              height: 80.0,
              point: new LatLng(51.5, -0.09),
              builder: (ctx) =>
              new Container(
                child: new FlutterLogo(),
              ),
            ),
          ],
        ),
      ],
    );
  }
}```

`error: The constructor returns type 'dynamic' that isn't of expected type 'LatLng'. (strong_mode_invalid_cast_new_expr at ...)`

fixed by adding in import:
import 'package:latlong/latlong.dart';


feat: Add kv-store connection check to readiness probe

Motivation
When the modelmesh is not able to connect to the kv store to update its instance recording or sync with the other instances it can not reliably serve inference requests. For a short time a disconnect can be tolerated and the cached values can be used to serve requests. However after some time the data may be stale and the routing of requests may result in errors. For example with instance A and B, if A has connection issues while B leaves the mesh, A still have the outdated instance record and will still route inference requests to B, which fail. To prevent this, A should be marked unready if it can not connect to the kv store, to inform upstream proxies to not route traffic to A.
Modifications
Add existing verifyKvStoreConnection check to isReady check.
Result
isReady will return false when the model mesh instance lost connection to the kv store. Allowing systems such as kubernetes to react to this condition.

Some further comments gathered from @tjohnson31415:

The routing reporting NOT READY here doesn't impact how MM routes inference traffic internally. K8s routing deals with external connections to MM pods while internal MM pods connect directly to others using LiteLinks.

If etcd was down, pods would no longer receive traffic although MM could still serve inference requests, and there isn't an automatic resolution for this scenario but we also don't want to prevent MM from serving any requests at all (serving 50% of requests is better than responding to none at all).

If 1+ MM pods have connection problems, but etcd is still working, we may want to restart the affected MM pods. A failed readiness probe would not trigger that, it'd require a liveliness probe failure.`

So, we don't want to change default behaviour, but possibly the best path forward is to add a new environment variable to enable an additional check for this case (i.e. not set by default), and could add an etcd check tot he initial startup probe too.

Hmm, I'm not sure about this behavior being the default.
As stated, ModelMesh is intended to handle downtime in etcd by continuing to serve requests for existing models (adding models and scaling would fail). If all pods reported as Unready when etcd goes down, then no requests would succeed. Even if the downtime extends over a longer period, I think it would be better to serve the requests it can until the system recovers.
Where this change would help is in the situation described in the issue where one or two of the pods are disconnected from the KV store, but this situation is not easily distinguishable from a complete etcd downtime. Merging this PR would prevent partial service in the case that etcd does go down. 🤔
That said, depending on the failure mode, perhaps reporting as NOT READY would be useful in some circumstances. If you do want this behavior for your use-case, I think we can add an experimental feature-flag via environment variable to control this behavior (off by default).

Thanks for the feedback, I will add a feature flag for it.

How should I implement the feature flag, as command line argument or as environment variable? I had a quick look at the code and I don't understand how to add a new command line argument or if there is any util for using features flags from the environment variables.


Update ex13_08.h

There is no need to allocate new memory in copy-assignment operator. Just copy the string directly.

Well, I guess your way is more generic.


Extremely slow loading time with Rails 3.1

This is unscientific but I was noticing extremely slow loading times on my app with Rails 3.1 so I decided to just run "thin start".
Load time on localhost:3000 8sec
Load time on Pow 2 minutes. (asset pipeline?)

@ddengler wow, that worked great. I came here looking for a way to solve the slow reloading of pages (especially annoying when messing with css files and making small changes).
This solution should be put on the front page in Futura 60px!

hi guys, how about in ubuntu 11.04 CSS, Javascripts and Images are not correctly loading.

dscacheutil -flushcache didn't work for me, but clearing the development.log worked

@choxi is right, deleting log/development.log does the job.

This worked great:
dscacheutil -flushcache
> log/development.log

I have the same problem, I gave up using pow

same problem like aribeiro.
This works only for 5 min on a mac:
dscacheutil -flushcache
rm log/development.log

Since I do "dscacheutil -flushcache" once a week I don't have any
problems anymore...
On Wed, Feb 29, 2012 at 4:09 PM, Ferhat Ziba
<EMAIL_ADDRESS>wrote:

same problem like aribeiro.
This works only for 5 min on a mac:
dscacheutil -flushcache
rm log/development.log

Reply to this email directly or view it on GitHub:
https://github.com/37signals/pow/issues/181#issuecomment-4239077

Nothing on this thread solves it for me.

Rails 4.2, pow and powder gem. It takes for me around 30 seconds to load the page which is horrible. Nothing here helped me too.

Ok guys I don't know what exactly helped me but I did few things:

I added bypassing proxy for .dev (Go to System Preferences -> Network -> Advanced -> Proxies -> Add *.dev to the field Bypass proxy settings for Hosts & Domains)
I added localhost to the top of DNS (Go to System Preferences -> Network -> Advanced -> and add <IP_ADDRESS> to the top, cleck Ok and after that Apply)
I updated the pow (since I use powder I just used powder update).
I truncated log/development.log
I used $ dscacheutil -flushcache
I restarted pow in the end (using powder: $ powder down and $ powder up

It works well now.

2 seems odd. Does pow actually run a local DNS?  If not, then you're
causing a delay on every single host lookup.
On Tue, Jul 19, 2016, 11:00 Serj L<EMAIL_ADDRESS>wrote:

Ok guys I don't know what exactly helped me but I did few things:

I added bypassing proxy for .dev (Go to System Preferences ->
Network -> Advanced -> Proxies -> Add *.dev to the field Bypass proxy
settings for Hosts & Domains)
I added localhost to the top of DNS (Go to System Preferences ->
Network -> Advanced -> and add <IP_ADDRESS> to the top, cleck Ok and
after that Apply)
I updated the pow (since I use powder I just used powder update).
I truncated log/development.log
I used $ dscacheutil -flushcache
I restarted pow in the end (using powder: $ powder down and $
powder up

It works well now.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/basecamp/pow/issues/181#issuecomment-233572067, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAEGD5RqebYqNpyXdRbl8QcxkPi_Ka1Fks5qXJITgaJpZM4Aqe6a
.

--
Wojtek

@gamut This step was based on @samgreen solution above in this thread. It helped some people.


Add scheduler versions to third_party not a as module

How to categorize this PR?

/area dev-productivity
/kind cleanup
/priority normal
What this PR does / why we need it:
Having modules makes vendoring gardener repository hard.
Which issue(s) this PR fixes:
Fixes #
Special notes for your reviewer:
Release note:

NONE

@mvladev
I'm currently working on #3109 and was therefore also vendoring k/*.
TBH, this was quite a pain with the scheduler stuff under third_party.
What am I supposed to do with those copied files when vendoring?
Can we please automate this when running make generate or revendor?

@mvladev
I'm currently working on #3109 and was therefore also vendoring k/*.
TBH, this was quite a pain with the scheduler stuff under third_party.
What am I supposed to do with those copied files when vendoring?
Can we please automate this when running make generate or revendor?


Add 5.x - de.unihalle WFs

In future, I will have 2 commits: The base WF and our amendments for more transparency.

Hi @Rillke, do you want me to wait for the second commit, or should this be merged as is?

Merging as is, you can file another PR if need be.


Add caveat for opApply and exceptions.

As discussed on the forum, add explanation about exception handling inside opApply, and their interaction with exceptions thrown from within the opApply delelgate.

Link to discussion<EMAIL_ADDRESS>

@Shachar  if you update the wording a bit from the comments, i'll pull this.

Hi @Shachar, could you amend the suggested fixes? If you do not have the time we can take over. Thanks!

I'll pull now and make the changes as a fixup.

https://github.com/dlang/dlang.org/commit/ca83d45b1d8e7576639faac50f67d5bce0737277


remove redundant function for edmfx boundary condition

Purpose

Combine sgs_h_tot_first_interior_bc and sgs_q_tot_first_interior_bc into one function as they are the same.
To-do

Content

[ ] I have read and checked the items on the review checklist.

bors r+


VM Size Standard_E64ds_v4 cached MBPS doubt

[Enter feedback here]
Hi.
looking at the below chart, E64s_v4 has 615000 IOPS/3872 MBps limit and 1600 MB temp drive cache, but since I found different info on ASC: VM Limit - Max Cached  MBps | 1024
I would like to know which of the 2 values is accurate
thanks!

Size
vCPU
Memory: GiB
Temp storage (SSD) GiB
Max data disks
** Max cached and temp storage throughput: IOPS/MBps (cache size in GiB)
Max uncached disk throughput: IOPS/MBps
Max NICs
Expected Network bandwidth (Mbps)

Standard_E80ids_v4 2
80
504
2400
32
615000/3872(1600)
80000/1200
8
30000

ASC info:
VM Limit - Max Cached  IOPS | 615000
VM Limit - Max Cached MBps | 1024

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 2b709e36-46ce-0a49-d7e4-7f87f4f36bbe
Version Independent ID: dc8675b5-4a8c-be16-146e-5688e08dd5c1
Content: Edv4 and Edsv4-series - Azure Virtual Machines
Content Source: articles/virtual-machines/edv4-edsv4-series.md
Service: virtual-machines
Sub-service: vm-sizes-memory
GitHub Login: @brbell
Microsoft Alias: brbell

Hello @jose-l-carvalho , thanks for the question! We are taking a look into this and will get back to you soon.

@jose-l-carvalho  Thanks for raising this question! Firstly, apologies for the delay in responding here and any inconvenience this issue may have caused.  Thanks for the feedback! I have assigned the issue to the content author to investigate further and update the document as appropriate.

@brbell Can you clarify the max cache?
@mimckitt

@jose-l-carvalho I will add you to the email thread offline.
#please-close


NEI CRASH only when Mekanism v8 is installed.

Ikeep crashing whenever i use NEI and click a recipe or usage. It
doesn't matter if i use the keyboard shortcuts or if i search and click,
any item-i crash every single time.
Single Player
ATLAUNCHER
just updated mekanism to <IP_ADDRESS>
When Mekanism isn't installed nei works fine

Log please

Closing, as this most definitely is some form of interaction issue. I'll continue to provide support.

I know it's already closed but here's the crash
http://pastebin.com/FgGZzKRQ


Run filter is not working for the new reports at the Product overview statistics

Run filter is not working for the New reports at the Product overview statistics page.

#3175 fixed this problem.


Equalizer

Is there a way to get the Equalizer not showing over all keys?
It would be awesome if I could leave the F1 - F12 and the Numpad blank. :3
thanks for the help :)

This will be possible in the next version through the 'Key/Region exclusion' option available to all layers

Oh sweet! I figured out how i can make keys black when not used so the equalizer wont be on them :)
Keep up the good work!


Unable to activate conda environment on Colab

I've been trying to get LMFlow setup on Colab, but I'm not able to activate the conda environment using the following code -
!git clone -b v0.0.7 https://github.com/OptimalScale/LMFlow.git %cd LMFlow !conda create -n lmflow python=3.9 -y !conda activate lmflow !conda install mpi4py !bash install.sh
It tells me to use conda init before using conda activate, but it doesn't work either way.
Has anyone been able to setup LMFlow on Colab? If so, how did you activate the conda env. Thank you so much!

Thanks for your interest in LMFlow! There is an old version of Colab demo in LMFlow v0.0.2 (https://colab.research.google.com/drive/1P9Hf6_mLE7WHH92pw73j9D5kz6GTdkow?usp=sharing), which maybe helpful in the conda environment initialization :smile:

Thanks for your interest in LMFlow! There is an old version of Colab demo in LMFlow v0.0.2 (https://colab.research.google.com/drive/1P9Hf6_mLE7WHH92pw73j9D5kz6GTdkow?usp=sharing), which maybe helpful in the conda environment initialization 😄

I checked out the demo, but it has the same problem of not being able to activate the conda environment -
CommandNotFoundError: Your shell has not been properly configured to use 'conda activate'. To initialize your shell, run
In the demo, you proceeded to install LMFlow dependencies even though the conda environment wasn't activate, so does that mean it's not necessary?

Thanks! This command works about 1 year ago. But you are right, conda environment is just for environment separation. If you are in Colab, direct pip install -e . should work as well

Thanks for the help!
But when I type in pip install - e . , I get another error -

Hi, seems like this problem can be caused by flash attention. Maybe we can try removing it from requirements.txt to see if it works.

Ya I removed flash-attn and it worked, thanks!

You are welcome 😄 Please let us know if there are any other problems encountered


Unsupported cookie spec: ignoreCookies

From<EMAIL_ADDRESS>on February 27, 2015 17:11:33
What steps will reproduce the problem? 1. Write a simple case, say testing 404
@Test
public void testStatusNotFound() {
given().log().all().with().authentication().basic("Blah","BlahhalB").
expect().statusCode(404).when().
get("/notfound");
}
baseURI is already set. My service is up and running

Run this in Eclipse as JUnit test
Get the following exception :- java.lang.IllegalStateException: Unsupported cookie spec: ignoreCookies.

Full stacktrace attached. Below is just an excerpt :
java.lang.IllegalStateException: Unsupported cookie spec: ignoreCookies
at org.apache.http.cookie.CookieSpecRegistry.getCookieSpec(CookieSpecRegistry.java:114)
at org.apache.http.client.protocol.RequestAddCookies.process(RequestAddCookies.java:168) What version of the product are you using? On what operating system? rest-assured 2.4.0 /Windows 8.1 /Eclipse Kepler with Maven/ Tomcat 8.0.15/ Jersey Please provide any additional information below. I get the same exception with POST as well, and also when I disable basic auth.
Attachment: RestAssuredCookieIssue.txt
Original issue: http://code.google.com/p/rest-assured/issues/detail?id=387

If anyone is still looking for causes for this check to see if you have clashing versions of org.apache.http.* in your classpath. I shaded something upstream of me (I forked that upstream thing) and everything went back to working (tests pass) again.


theme.mstab_dont_resize_flaves = true doesnt seem to work

I still see a small window behind a transparent terminal in mstab

You can find my configs here
Btw a side question. Maybe im dumb but i couldnt find out how to use the keybind tabbed.pick mentioned in ToDo and #13
can you clarify what function i need to bind for that?
Thank you in advance

Hello, try to type slaves not flaves I hope it should work since every non master is named slave

Hello, try to type slaves not flaves I hope it should work since every non master is named slave

@ninelore Bran is right. It was a typo in the template file. Sorry for that. Does it work with the actual variable name?

@ninelore Bran is right. It was a typo in the template file. Sorry for that. Does it work with the actual variable name?

yes it works thanks. Whats with my side question?

yes it works thanks. Whats with my side question?

@ninelore This works for me, put it in rc.lua or whatever you are storing keys and edit for your needs
 -- Bling tabbar keybinds
    awful.key({ modkey }, "t", function () bling.module.tabbed.pick() end,
        {description = "Enable tabbar for focused client", group = "Bling"}),
    awful.key({ modkey, "Shift" }, "t", function () bling.module.tabbed.pop() end,
        {description = "Remove focused client from tabbar", group = "Bling"}),
    awful.key({ modkey }, "Tab", function () bling.module.tabbed.iter() end,
        {description = "Iter through all tabs", group = "Bling"}),

@ninelore This works for me, put it in rc.lua or whatever you are storing keys and edit for your needs
 -- Bling tabbar keybinds
    awful.key({ modkey }, "t", function () bling.module.tabbed.pick() end,
        {description = "Enable tabbar for focused client", group = "Bling"}),
    awful.key({ modkey, "Shift" }, "t", function () bling.module.tabbed.pop() end,
        {description = "Remove focused client from tabbar", group = "Bling"}),
    awful.key({ modkey }, "Tab", function () bling.module.tabbed.iter() end,
        {description = "Iter through all tabs", group = "Bling"}),

isnt bling.module.tabbed.pick for the mouse? it got mentioned it it possible to select with the keyboard, but i cant find anything on how exactly

isnt bling.module.tabbed.pick for the mouse? it got mentioned it it possible to select with the keyboard, but i cant find anything on how exactly

@ninelore The only other option to add clients is with the bling.module.tabbed.pick_with_dmenu function. It defaults to rofi as the external dmenu application but you can also pass another command as a string to make it work with dmenu and what not (for example with bling.module.tabbed.pick_with_dmenu("dmenu")). It is bind the same way that the other functions are. And you are right, I should have added that to the readme.

@ninelore The only other option to add clients is with the bling.module.tabbed.pick_with_dmenu function. It defaults to rofi as the external dmenu application but you can also pass another command as a string to make it work with dmenu and what not (for example with bling.module.tabbed.pick_with_dmenu("dmenu")). It is bind the same way that the other functions are. And you are right, I should have added that to the readme.

Fair enough. Thank you

Fair enough. Thank you


TypeError of import operation of run_eval.py

Traceback (most recent call last):
File "/opt/conda/lib/python3.8/runpy.py", line 194, in _run_module_as_main
return _run_code(code, main_globals, None,
File "/opt/conda/lib/python3.8/runpy.py", line 87, in _run_code
exec(code, run_globals)
File "open-instruct/eval/mmlu/run_eval.py", line 11, in
from eval.utils import get_next_word_predictions, load_hf_lm_and_tokenizer, query_openai_chat_model
File "open-instruct/eval/utils.py", line 10, in
from eval.dispatch_openai_requests import dispatch_openai_chat_requesets, dispatch_openai_prompt_requesets
File "open-instruct/eval/dispatch_openai_requests.py", line 11, in
messages_list: list[list[dict[str,Any]]],
TypeError: 'type' object is not subscriptable
async def dispatch_openai_chat_requesets(
    messages_list: list[list[dict[str,Any]]],
    model: str,
    **completion_kwargs: Any,
) -> list[str]:

    async_responses = [
        openai.ChatCompletion.acreate(
            model=model,
            messages=x,
            **completion_kwargs,
        )
        for x in messages_list
    ]
    return await asyncio.gather(*async_responses)

why the typeError occur when importing?

Ah, this is because list[str] is only a valid typehint in Python 3.9 and older, but you're using python 3.8. Changing the typehints to use List instead, or using a newer version of python, should fix things (afaik we used python 3.9). I'll make a PR to fix this up, since using List is more consistent with the rest of the codebase.


Run Object in Cloud Sandbox

Only works on local server, not on the cloud sandbox, because the launch.json is totally different.

example:
https://businesscentral.dynamics.com/sandbox?page=22
so, it's quite easy - always that url.


Use with asyncio?

Is this compatible with one of the async libraries, like aiobotocore or aioboto3?

... and the answer is no.
I played around with it a bit and here is a simple solution for async; see https://github.com/boto/botocore/issues/761#issuecomment-1937535330. Not as full featured as this, but should do the job for the basic use case.


Add virtual card request model

Related: https://github.com/opencollective/opencollective/issues/4312
Related: https://github.com/opencollective/opencollective/issues/6835
Add virtual card requests model and record virtual card request

Updated with tests and a collection query for the host dashboard.


/oidc/userinfo endpoint is not explained in the Graph API docs

The OpenID Connect Metadata Document for Azure AD app registrations declare that https://graph.microsoft.com/oidc/userinfo is the user info endpoint. However the Graph API documentation does not explain this endpoint at all or what operations are available upon it.
I would like to programmatically determine the sub value for a number of users in my organisation for integration with an OIDC app. The /users Graph API endpoint allows me to specify a userPrincipalName as a path element but I cannot find any similar functionality for the /oidc/userinfo endpoint or even whether it is possible.
This issue is closely related to: https://github.com/MicrosoftDocs/azure-docs/issues/28317

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 628db02d-345a-5dc4-17f0-3d6ff76d7fe5
Version Independent ID: 524b460d-47b7-72c9-e994-ea94b3a9dd2b
Content: OpenID Connect protocol - Microsoft identity platform
Content Source: articles/active-directory/develop/v2-protocols-oidc.md
Service: active-directory
Sub-service: develop
GitHub Login: @rwike77
Microsoft Alias: ryanwi

Hey @g-dx - are you looking to get the sub claim for multiple apps? That likely won't be possible - we do not allow app A to see what the sub claim for a user is if they were to sign into app B.

@hpsin - Thanks for your quick reply!
I’m an Azure AD administrator & I need to configure an app with OIDC authentication for my users. To do this the app needs to know, up front, the sub & email addresses of the users who are going to sign in to it. As an admin I need to provide this information to the app vendor.

Does the sub claim vary from app to app in Azure AD? That is, only consistently represents the same user for a single app registration?

Yup - AAD uses a pairwise claim (hash(userID, tenant, appid)), rather than a consistent claim across apps.  We use the "oid" claim for this, which remains the same across apps.

@hpsin - again, thanks for your reply. Two other questions:

Is what I’ve outlined here actually possible: https://github.com/MicrosoftDocs/azure-docs/issues/50810#issuecomment-603410472
Given the “pairwise claim” you outlined in your last comment, does that mean that the requests in issue #28317 to add the sub claim to the /me endpoint are not possible?

It will be helpful to know what claims are returned by the user info endpoint. And more helpful if could be configured

Userinfo is now documented here:
https://docs.microsoft.com/en-us/azure/active-directory/develop/userinfo
#please-close
Do let us know if the docs could be updated to further help you use the platform.  Thanks!


IP Restricted additional information

Is there an existing issue for this?

[X] I have searched the existing issues

Describe the problem to be solved
I want to only enable SHOW_DETAILS when the request comes from a specific ip range.
Ex. Customers don't really need to know where the request is going inside the k8s cluster. But for debugging, it would be beneficial.
Suggest a solution
Add an environment variable to allow source-ip ranges for SHOW_DETAILS.
Additional context
No response

Hello there, and thank you for raising this issue! May I ask why you prefer not to enable detailed debugging information in development or staging environments while keeping it disabled in production? This approach seems more correct, doesn't it?


.insert() only returns int even when using Uint8List for primary keys.

With the new clientDefault setting added for #294, an entity can be inserted into the database without knowing the primary key in advance however the insert() function only returns an int making it difficult to obtain the primary key of the new entity when it isn't an integer.
insert() could return a Future<dynamic> or Future<T> with T being the type of the primary key.
It might even be better for insert() to return the new entity that was added to the database.

Some background: In sqlite, pretty much every table has an implicit column called rowid that serves as the native primary key. When you declare an INTEGER PRIMARY KEY, that column becomes an alias for rowid. Quite confusingly, any other primary key is just a UNIQUE constraint in disguise. See sqlite docs for details.
Since the real primary key is almost always an int, sqlite exposes the sqlite3_last_insert_rowid function, which is what moor returns for insert statements we well. To my knowledge, there's no builtin api to retrieve the user-defined primary key for the last insert.
A potential solution to this could be to select a primary key like this:
SELECT pk_column FROM table WHERE rowid = last_insert_rowid();

Obviously, changing insert would be a breaking change. We could add another method to load the "real" primary key of the last insert, which could return a correctly-typed future. For now, putting the above query in a moor file or into UseMoor(queries: should do the trick.

Thanks for the details. For now, the best choice for me seems to be avoiding clientDefault and instead, generating my primary keys(UUIDs) before running insert().
Is Dart support for the WITHOUT ROWID phrase when creating tables be possible?
Thanks.

Is Dart support for the WITHOUT ROWID phrase when creating tables be possible?

Adding this to your table class should work:
@override
bool get withoutRowId => true;

Is Dart support for the WITHOUT ROWID phrase when creating tables be possible?

Adding this to your table class should work:
@override
bool get withoutRowId => true;

Is there any way to verify that the change took effect or any other schema change for that matter? Running code generation didn't change anything.
Thanks.

Running code generation didn't change anything.

The migrator will take a look at that field at runtime and append the WITHOUT ROWID to the created statement. https://github.com/simolus3/moor/blob/0cdfc9a4abb803fdb6414af880961a26be952207/moor/lib/src/runtime/query_builder/migration.dart#L134-L136

Is there any way to verify that the change took effect or any other schema change for that matter?

You can verify which CREATE SCHEMA statements are sent by setting logStatements: true on the FlutterQueryExecutor or VmDatabase, respectively.
I'm also working on a CLI tool that would dump the schema into a json file (part of #240).


using MD_DIALOG_DATA testbed provider throws an error

Bug, feature request, or proposal:
Bug
What is the expected behavior?
Able to inject MD_DIALOG_DATA provider into tests.
What is the current behavior?
node_modules/@angular/material/material"' has no exported member 'MdDialogData'.
What are the steps to reproduce?
import { MdDialogRef, MD_DIALOG_DATA } from '@angular/material';

...

    TestBed.configureTestingModule({
      imports: []
      declarations: [
        MyComponent
      ],
      providers: [
        { provide: MD_DIALOG_DATA, useValue: 'wtf' },
        { provide: MdDialogRef, useValue: { close: () => null } }
      ]
    });

Running the test throws
.../node_modules/@angular/material/material"' has no exported member 'MdDialogData'.

Which versions of Angular, Material, OS, TypeScript, browsers are affected?
Angular 4.3.6
Material 2.0.0-beta8

Nevermind, I wasn't importing MdDialogModule


Onboarding active tooltips

[ ] votes
[ ] Progress bar
[ ] New section
[ ] New version

Add popups/active tooltips for onboarding new users.
Pop up option -
3-4 cards with editable text (from strings files) and screenshots images (upload directly or using some images storage and display service)
Active tooltip option -
Something like "walkme", tooltips will be displayed on this components, in this order:

[ ] votes
[ ] Progress bar
[ ] New section
[ ] New version


Option to disable cursor blink

Is your feature request related to a problem? Please describe.
I really dislike cursor blink.
Describe the solution you'd like
Let me disable cursor blink.
Describe alternatives you've considered
Use Sublime text, which allows me to disable cursor blink.

I would also appreciate this, it's the first change I make in every editor or terminal. I find it very distracting and it makes it difficult to focus on reading. When the blink draws my attention away it feels like a mini context switch.

This seems like an easy fix. Every editor has this option.

Can we also have settings like making the cursor an underline, bold, thin etc.? Maybe a few different animations for blinks as well? 🙈
Not sure if I should create a new ticket for this^

any update on this 👀?

Cursor blinking can be disabled by adding "cursor_blink": false in your local settings. This has been in Zed for a few versions at this point, but we forgot to close this issue when the setting was released. Thanks so much for the report!
@yashank09: it'd be great if you could create another issue that just captures the ability to change cursor style. Thanks!


Serialize times with milliseconds

Describe the bug
Timestamps cannot be used to generate stable cursors since they are serialized with precision of seconds
https://github.com/bibendi/graphql-connections/blob/c979406fc407216384fbe80351c1cf749794eae8/lib/graphql/connections/base.rb#L42
To Reproduce

Install the graphql gem.
Create a User ActiveRecord model with timestamps, such as created_at.
Create a type for the User model
Create a query to fetch users which returns a stable connection from this gem

class User < ApplicationRecord
end

module Types
  class UserType < Types::BaseObject
    field :id, ID, null: false
  end
end

module Types
  class QueryType < Types::BaseObject
    field :users, Types::UserType.connection_type, null: false
    def users
      GraphQL::Connections::Stable.new(User.all, primary_key: :created_at)
    end
  end
end

The generated SQL looks like this:
SELECT "users".* FROM "users"
WHERE "users"."created_at" > '2021-11-10 14:26:19'
ORDER BY "users"."created_at" ASC LIMIT 10

Expected behavior
The SQL query should include milliseconds to avoid skipping records that were created in the same second:
SELECT "users".* FROM "users"
WHERE "users"."created_at" > '2021-11-10 14:26:19.152283'
ORDER BY "users"."created_at" ASC LIMIT 10

To achieve it, serialize times with higher precision
# lib/graphql/connections/base.rb
def serialize(cursor)
  case cursor
  when Time, DateTime, Date
    cursor.iso8601(9)
  else
    cursor.to_s
  end
end

Context (please complete the following information):

Ruby: 2.7.4
graphql gem: 1.14
rails: 6.1

Hi!
Makes sense!


install via kwindrem SetupHelper

Is your feature request related to a problem? Please describe.
dbus-serialbattery does not install via https://github.com/kwindrem/SetupHelper It would be nice to have it do so. I checked, and there is no archive in the proper location for SetupHelper to work, namely:
https://github.com/Louisvdw/dbus-serialbattery/archive/latest.tar.gz  (when using 'latest' as the release tag in SetupHelper)
This archive exists for other packages that do work in their repos. with SetupHelper. Of course, there may be other reasons for it not to work, (script locations/existence,/function etc).
Describe the solution you'd like
Be able to select your repo and release in SetupHelper and have it download and install dbus-serialbattery
Describe alternatives you've considered
I can pull down the software and have it install locally.
Additional context
You did provide the following comment a few years ago over on the victron community:
"I have not created a setup to work with SetupHelper. There is a ticket open to do that."
https://community.victronenergy.com/questions/76159/victron-venusos-driver-for-serial-connected-bms-av.html
Any updates? Thanks for this software, it is appreciated.

See https://github.com/Louisvdw/dbus-serialbattery/issues/67.


chore: Fix copying of variables into Docker

Fixes set -u errors, and prevents variables being set to undefined values
Change-type: patch

Neat!


What type of update are you requesting?
A new version of an existing package
Current Package Identifier
TortoiseSVN.TortoiseSVN
Package Version
1.14.7
Please describe the changes you would like to see
There is a new version (1.14.7) available.
Downloadlink: https://tortoisesvn.net/downloads.html

Double request. This one can be closed.

@Regtur1983, it should be fixed but I think there is another issue
@stephengillie would the two package version in the following screenshot override the latest update since they are a bigger number? (1.14.29387, 1.14.29465)

@stephengillie already did pull requests just in case

@Regtur1983, this issue has been successfully resolved, kindly close this ticket as complete please.


feat(k8s): delete from gobox, it has been moved to services

What this PR does / why we need it
k8s folder has been migrated to the services repo here: https://github.com/getoutreach/services/tree/main/pkg/k8s (cause gobox is public). This PR deletes the old copy.

JIRA ID: [QSS-00]

Notes for your reviewers

:tada: This PR is included in version 1.29.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Everyone as a days off requires an iteration selection

While good for planning purposes, it may be a holiday, in which case it should be allowed to be empty simply to record as it may or may not impact velocity.
It also doesn't show on the calendar if the days off is beyond my current iterations.

We're closing old issues so we can get back to actively maintaining the extension. Feel free to reopen if this is still an issue for you.


terraform: support provider aliases

This PR fixes a bug due to which resources wouldn't be correctly estimated if using provider aliases. Closes #31

This made me think on https://github.com/cycloidio/tfdocs/blob/master/aliases/aws/aliases.json this aliases, WDYT it'll happen if they are used?

We could use them though ATM there's not that many of them (e.g. currently using only aws_lb~aws_alb) so not sure about adding complexity :man_shrugging:


Add tooltips to buttons in graphing calculator

Fixes part of #338.
Description of the changes:

Added tooltips to buttons in graphing calculator

How changes were validated:
manually

I believe there's a small state icon too that just got checked in today.

I believe there's a small state icon too that just got checked in today.

Rudy handled tooltips in his commit for small state already.


Messages won't send through in builds exported without debug

Operating system: Windows 10 and Windows 11
Godot version: 3.5.3 stable
I've encountered an issue where messages won't send through in builds exported without debug, without any error messages. Exporting with debug enabled resolved the issue, but now we are working on a Mac build for our game. It seems that GodotSteam won't work for Mac if exported with debug, so we have to uncheck that option.
Below are my codes for the reporter node, modified from your original codes. Thank you for your excellent work.
extends NinePatchRect

export var cfg_path := "res://addons/BugReporter/config.cfg"
export var hide_after_send := true
export var clear_after_send := true

const url="https://discord.com/api/webhooks/1216567474804232273/eYyqYPamYPyMW9sOFM8xC0tDr7H9vLMyqYPs0jEi3gONnGbNtlY5CexjxP2l6xgIhtze"
var _cfg : ConfigFile

onready var _screenshot := $"%Screenshot"
onready var _screenshot_check = $MarginContainer/VBox/CheckBox
onready var _mail : LineEdit = $MarginContainer/VBox/Mail/LineEdit
onready var _http := $HTTPRequest
onready var _send_button = $MarginContainer/VBox/SendButton
onready var message_window = $"%Message"
onready var timer = $Timer
onready var notice = $"%Notice"

func _ready():
	_cfg = ConfigFile.new()
	var err := _cfg.load(cfg_path)
	if err != OK:
		push_error("Bugreporter couldn't load config. Reason: %s" % err)
	var img := get_viewport().get_texture().get_data()
	img.flip_y()
	var text := ImageTexture.new()
	text.create_from_image(img)
	_screenshot.texture = text

func _on_SendButton_pressed():
	timer.start()
	if _http.get_http_client_status() != HTTPClient.STATUS_DISCONNECTED:
		return

	var message : String = message_window.text.replace("```", "")
	var player_id := "playerid: %s" % _unique_user_id()
	if _cfg.get_value("webhook", "anonymous_players", false):
		player_id = "anonymous"
	var contact_info := _mail.text.dedent()

	var request_body := []# 1st place is reserved for json_payload
	var json_payload := {
		"username" : "%s:" % _get_game_name(),
		"tts" : _cfg.get_value("webhook", "tts", false),
	}
	var embed = {
			"title": "Bug reported by %s" % [player_id],
			"color": _cfg.get_value("webhook", "color", 15258703),
		}
	var fields := []

	if !contact_info.empty():
		fields.push_back({
				"name" : "Contact Info:",
				"value" : contact_info
		})

	if !message.empty():
		fields.push_back({
				"name" : "Message:",
				"value" : "```\n%s\n```" % message,
			}
	)

	if _screenshot_check.pressed:
		embed["image"] = {
					"url" : "attachment://screenshot0.png",
				}

		request_body.push_back(_screenshot.texture)

	embed["fields"] = fields
	json_payload["embeds"] = [embed]

	request_body.push_front(json_payload)
	request_body.push_back("user://logs/godot.log")
	var payload := _array_to_form_data(request_body)

	if fields.empty():
		return

	_http.request(url,#_cfg.get_value("webhook", "url", ""),
			PoolStringArray(["connection: keep-alive", "Content-type: multipart/form-data; boundary=boundary"]),
			true,
			HTTPClient.METHOD_POST,
			payload
	)
	_send_button.disabled = true
func _on_HTTPRequest_request_completed(result, response_code, headers, body):
	_send_button.disabled = false
	if clear_after_send:
		_mail.clear()
		message_window.text = ""
	if hide_after_send:
		queue_free()
func _unique_user_id() -> String:
	return str(hash(str(OS.get_unique_id(), "|", _get_game_name())))
func _get_game_name():
	return _cfg.get_value("webhook", "game_name")
func _texture_to_png_bytes(texture : Texture, max_size:=8000000)->PoolByteArray:
	var img := texture.get_data()
	var bytes : PoolByteArray = img.save_png_to_buffer()

	while bytes.size() > max_size:
		img.resize(img.get_width()/2, img.get_height()/2)
		bytes = img.save_png_to_buffer()
	return bytes
func _array_to_form_data(array:Array)->String:
	var file_counter := 0
	var output = ""
	for element in array:
		output += "--boundary\n"
		if element is Dictionary:
			output += 'Content-Disposition: form-data; name="payload_json"\nContent-Type: application/json\n\n'
			output += to_json(element) + "\n"
		elif element is Texture:
			output += 'Content-Type: image/png; name="files[%s]"\n' % file_counter
			output += 'Content-Disposition: attachment; filename="screenshot%s.png"\n' % file_counter
			output += 'Content-Transfer-Encoding: base64\nX-Attachment-Id: f_ljiz6nfz0\nContent-ID: <f_ljiz6nfz0>'
			output += "\n\n"
			output += Marshalls.raw_to_base64(_texture_to_png_bytes(element)) + "\n"
			file_counter += 1
		elif element is String:
			var f = File.new()
			if f.open(element, File.READ) == OK:
				var file = f.get_as_text()
				f.close()
				if !file.empty():
					output += 'Content-Type: plain/text"\n'
					output += 'Content-Disposition: attachment; filename="log"; name="files[%s]";\n' %file_counter
					output += "\n"
					output += file
			else:
				printerr("BugReporter could not attach File %s to Message" % element)
			file_counter+=1
	output += "--boundary--"
	return output
func _on_Close_pressed():
	queue_free()
func _on_Timer_timeout():
	print("http connecting result:",_http.get_http_client_status())

Don't share your webhook url it can lead to data leaks. For testing it is fine to use it but for release, you should use a proxy.
It works for me with debug and without. Did you maybe forget to include the config.cfg in release builds?

I tested it in a new project. Didn't work if exported without debug...I also tried to turn off the embed PCK option, but it didn't seem to matter. Here's my export settings:

Please let me know if you need more information.

Still can't reproduce it.
Maybe your firewall blocks the release build but not debug builds?
If that is the case it would fail without error message.
If you look at result and response_code in this function
func _on_HTTPRequest_request_completed(result, response_code, headers, body):
it might tell you what exactly is wrong.

I just backported the new features and added the missing error message so if you update you should get an error.

Thanks for that! I tried the new version and I got:
BugReporter message send
BugReporter Error sending Report. Result: 0 Responsecode: 400 Body: {"message": "Cannot send an empty message", "code": 50006}
But I did add a message though. "build test" specifically. And in the editor, even if I don't write anything, the message can be sent through.

It seems that the request payload is somehow getting messed up.
Can you add print(payload) on line 175 in webhook_builder.gd and tell me what it prints?
When you only send a message it should look something like this:
--b4174803096
Content-Disposition: form-data; name="payload_json"
Content-Type: application/json

{"attachments":[],"embeds":[{"color":15258703,"fields":[{"inline":false,"name":"Message:","value":"```\ntest\n```"}],"title":"Bugreport by playerid: 149521579"}],"tts":false,"username":"BugReporter:"}
--b4174803096--

If it looks like that then I have no idea what else it could be.
You can change the URL to https://discord.com/api/v10/webhooks/... to get a better error message from Discord. That might help as well.

I found out what's wrong! When I send a message with no attachment, it'll send through. But not the ones with analytics. I get:
--b1299572883
Content-Disposition: form-data; name="payload_json"
Content-Type: application/json

{"attachments":[{"id":0}],"embeds":[{"color":15258703,"fields":[{"inline":false,"name":"Message:","value":"```\nbuild test with log\n```"}],"title":"Bugreport by playerid:<PHONE_NUMBER>"}],"tts":false,"username":"From Soil to Bottle:"}
--b1299572883
--b1299572883--
BugReporter message send
BugReporter Error sending Report. Result: 0 Responsecode: 400 Body: {"message": "Cannot send an empty message", "code": 50006}

It seems there is some invalid data in _request_body.
That way there are 2 boundaries right next to each other making the request invalid.
Can you replace the print with print(_request_body) and tell me what it prints?

Ok. I found and fixed a bug that caused this exact problem when you tried to send the log file (Or any file after the first send).
But You didn't send log files at the start so I'm not sure if that will fix your problem.

Ohhhh it works now. Thank you very much!


Throw an exception on Pod- or ReplicationController-Update

As updates of Pods and ReplicationControllers are not supported, an exception should be thrown and no updates at all should be executed.

Implemented.


MARMOTTA-639: switched to 'red' as default style

MARMOTTA-639:

I don't get the idea of color change. Why? IMHO red is very aggressiv style ;)


firestore.cmake: point to the HEAD of the NanopbNoPackedStructsCmakeFix branch in the iOS repository

DO NOT MERGE
This is a temporary branch that picks up https://github.com/firebase/firebase-ios-sdk/pull/8923 (CMakeLists.txt: add -DPB_NO_PACKED_STRUCTS=1 to the nanopb compiler flags) from the iOS SDK. With this, I'm going to create a custom build of the C++ SDK so that the customer can test to see if it fixes https://github.com/firebase/firebase-cpp-sdk/issues/712 ([Bug] Firestore fails to link for desktop on macOS Monterey because of unaligned pointers).

I've started a "CPP binary SDK packaging" action at d74c415fb53f0e91dd318f2c7cf71194c685fae5:
https://github.com/firebase/firebase-cpp-sdk/actions/runs/1420069216

I've merged in the latest from the main branch and started a "CPP binary SDK packaging" action at 0f77498f30e260aabda2b2c700928744b4144964:
https://github.com/firebase/firebase-cpp-sdk/actions/runs/1421892145
Fingers are cross that this will fix the "merge_analytics.sh: No such file or directory" errors.

Turns out, the merge_analytics.sh build error was actually a symptom of an earlier build error that was silently ignored:
CMake Error at /home/runner/work/firebase-cpp-sdk/firebase-cpp-sdk/out-sdk/external/tmp/firestore-gitclone.cmake:40 (message):
  Failed to checkout tag: '9aa0c85b1817684a601c21dcabe8bfe8e74c3692'
make[2]: *** [CMakeFiles/firestore.dir/build.make:99: src/firestore-stamp/firestore-download] Error 1
make[1]: *** [CMakeFiles/Makefile2:307: CMakeFiles/firestore.dir/all] Error 2

I'll double-check that I used the correct commit hash and try again.

Ahh, the problem is that https://github.com/firebase/firebase-ios-sdk/pull/8923 was merged into master and the branch was deleted. So the commit is no longer valid. I'm just going to use the head commit from right now: https://github.com/firebase/firebase-ios-sdk/commit/03e174c4b75fb127a81fde78692cc0d963c75069

Okay, I've updated the pinned commit to the current head of the iOS SDK and started a a "CPP binary SDK packaging" action at 44e348ac227c2d93ef25805a75e9098a00da22c0:
https://github.com/firebase/firebase-cpp-sdk/actions/runs/1426820775

Closing since the C++ SDK v8.8.0 (released yesterday) contains the fix hacked in by this PR.


Tello does not give back telemetry data via UDP 8890

Hi John
first of all thanks for this great tutorial :-)
I was able to fly the drone by the flow/control buttons on the dashboard.
Unfortunately I don't receive data back from the drone.
Commands are transfered and I receive the "OK" back - so basic coversation works on Windows 10 and Mac OS-X. On both machines I switched of firewall and checked with TCPView or netstat for a listening software at UDP 8890 towards <IP_ADDRESS> - but could not find anything listening.
Any Idea?
I did not check with linux yet because I wanted to use it under Windows and I am pretty sure, that ports are open.
Best Regards,
Axel

I am getting the same issue, any views, please advise,
Below is the node-red trace shows some error,
:08 - [info] [udp in:890bfeab.dba5] udp: port 52955 already in use
10 Sep 17:55:08 - [info] Started flows
10 Sep 17:55:08 - [info] [udp in:b588c26f.cc38e] udp listener at <IP_ADDRESS>:52955
10 Sep 17:55:08 - [info] [udp in:890bfeab.dba5] udp listener at <IP_ADDRESS>:52955
10 Sep 17:55:08 - [error] [udp in:bd7e5975.45fad8] error: EACCES
10 Sep 17:55:08 - [info] Server now running at http://<IP_ADDRESS>:1880/
10 Sep 17:55:08 - [info] [udp out:9d63d734.6c6db] udp re-use socket: 52955 -> <IP_ADDRESS>:8889
10 Sep 17:55:08 - [info] [udp out:50a17ab9.0520a4] udp re-use socket: 52955 -> <IP_ADDRESS>:8889

@VipinNa  - Are you certain you have opened the UDP ports in your firewall?
See the Linux / MAC OSX instructions on this page:
https://github.com/johnwalicki/Node-RED-Tello-Control/blob/main/docs/PART5.md

Yes, I am using Windows 10.
See firewall settings & UDP status on CLI

hi @johnwalicki,
I check also on wireshark, i am getting data from drone but a i can't still see any data on node-red.
Attached is the screen shot.

Hi @johnwalicki,,
The issue is resolved.
I installed Ubuntu 20.04 and then opened the ports.
Now it is working fine.
Thank you for you time.

Good news.  I have no experience with Windows firewall configuration. Glad you were able to set up Ubuntu.


Event callbacks don't work after navigation

Describe the bug
I have two pages:

Home: there is a button to go to the "Agora page"
Agora: where can I enter and leave a channel

The first time I enter a channel, everything works.
Then I try to go back to the home page, I click on the button to go to the Agora page.
When I try to click the "Join" button, the callback event on JoinChannelSuccess doesn't fire.
So I notice that after browsing the event callbacks no longer fire.
I used the example of docs.
To Reproduce
Steps to reproduce the behavior:

Launch the app
Click on the button 'Go to Agora Call'
Click on the button 'Join'
The text change from "Join a channel" to "Waiting for a remote user to join..."
Click on the button 'Leave'
The text change from "Waiting for a remote user to join..." to "Join a channel"
Click on back arrow
Click on the button 'Go to Agora Call'
Click on the button 'Join'
The text doesn't change, the event JoinChannelSuccess was not fired

Expected behavior
I expect that the event callbacks work always also with navigation
Additional context
Here there is the project
Please fix the bug

Hello,
I am also facing this same issue in my application.
In my application there is a home page and I move into the details of user page and click on the video button and request for the call It works well at the first time. Also, I entered into the channel. At the first time I get the callback into the into the event, but at the second time I can not get the callback into the event.
But In my case audio is working fine, but I can not get the video stream in the event.
To Reproduce
Steps to reproduce the behavior:

Launch the app
Get the list of user the home page
Navigate to the user detail page
Open one dialog for the video request when I clicked on the request I joined the agora channel.
Disconnect the call and leave to the channel.
Agian folow the process from step 4.
Audio is working but video stream not work.

Please look into this and help me for same.

I have found a fix and it's working fine for me.
try adding following line while you are disposing the widget.
@override
void dispose() {
_engine.release();
super.dispose();
}

As @devhardikbeladiya stated, the RtcEngine needs to be released on dispose callback.

This is the doc issue, I have submitted an issue to the doc repo.
https://github.com/AgoraIO/Docs-Source/issues/876

Now, everything works.
Thanks


Add auth gate service interface

Issue number of the reported bug or feature request: RDSIRABBIT-1962
Introducing auth gate service interface
Auth gate service will be used to authn/authz each client connection during initial handshake. This service can be any external http service. This PR only contains interface to communicate with that external http service and default implementation of the interface. The default implementation will always allow all the clients to connect without any authn/authz. The PR also contains unit tests for default implementation of the interface. Currently we are going to target similar authn/authz mechanism for entire server level. The second PR will contain the http implementation of the interface, in which all the clients will be authn/authz using external http service and control command to configure http service parameters like hostname, port, target. The auth service will get vhost, auth mechanism and credentials information from amqpprox for connecting AMQP client.  And the service will return allow/deny with reason, auth mechanism and credentials information after processing the request.  The returned credential information will be forwarded to server/broker to finally connect.
In case of, unauthorized connection, the proxy will close client connection based on authentication_failure_close capabilities defined in client properties, received from client. If that value is true, then proxy will send the close connection method before snapping socket connection. For example, pika client will receive an error like pika.exceptions.ProbableAccessDeniedError: ConnectionClosedByBroker: (403) 'ERROR: Not authorized by amqpprox proxy'. But if that value is false, then proxy will snap the socket connection without sending any close connection method. For example, pika client will receive an error like pika.exceptions.ProbableAccessDeniedError: StreamLostError: ('Transport indicated EOF',)
** What testing happened
This PR introduces unit tests for the default implementation of the auth interface component. The PR also modifies session tests to incorporate usage of auth interface and change the behaviour based on allowed or denied auth request.

otherwise LGTM


Swift 4 Update

Looks like there are more discrepancies between UIKit and AppKit.  I modified all of the places in the codebase where NSLayout and UILayout differ. There are also general formatting changes.  I also updated the unit tests.

I don't think this is necessary after merging #446 which I believe simplified this a lot and is a cleaner solution but I do appreciate your PR.

Ah yep! Totally agree. I'll reset my remote back to this repo.  Thanks for keeping this maintained.


add default metadata attributes

Changes
basically the simplest way I could think of to always serialize metadata attributes of any class (if it exists)
ToDos

[x] wrap from_tree to read metadata field
[ ] remove additionalProperties from all schemas by default
[ ] I have a feeling it would be safer to wrap to_tree_tagged instead of to_tree since this is the final function that produces the TaggedObject
I think we can skip this for now but may have to be careful for classes that implement their own to/from_tree_tagged

Related Issues
Closes #205
Checks

[ ] updated CHANGELOG.md
[x] updated tests
[ ] updated doc/
[ ] update example/tutorial notebooks

A note regarding the TimeSeries schema:
It's a bit tricky to remove additionalProperties the way the schema properties are defined right now because they all use the same key values and unit right now
We could use different keys for values depending on the TimeSeries type, e.g. const, expression and values
%YAML 1.1
---
$schema: "http://stsci.edu/schemas/yaml-schema/draft-01"
id: "http://weldx.bam.de/weldx/core/time_series-1.0.0"
tag: "tag:weldx.bam.de:weldx/core/time_series-1.0.0"

title: |
  Schema that describes a time series.
description: |
  Describes a time dependent quantity.
oneOf:
  - type: object
    description: |
      Implementation for constant values.
    properties:
      values:
        description: |
          Number or n-dimensional array that is constant in time.
        oneOf:
          - type: number
          - tag: "tag:stsci.edu:asdf/core/ndarray-1.0.0"
      unit:
        description: |
          Unit of the data.
        type: string
    required: [values, unit]
    additionalProperties: false

  - type: object
    description: |
      Implementation for expressions.
    properties:
      values:
        description: |
          A mathematical expression that describes the time dependent behaviour.
        tag: "tag:weldx.bam.de:weldx/core/mathematical_expression-1.0.0"
      unit:
        description: |
          Resulting unit of the data when the expression is evaluated.
        type: string
      shape:
        description: |
          (optional) Resulting shape of the data when the expression is evaluated.
        type: array
    required: [values, unit]
    additionalProperties: false

  - type: object
    description: |
      Implementation for discrete data.
    properties:
    ....
      unit:
        description: |
          Unit of the data.
        type: string
     ....
      values:
        description: |
          Set of discrete n-dimensional data.
        tag: "tag:stsci.edu:asdf/core/ndarray-1.0.0"
    required: [time, unit, shape, interpolation, values]
    additionalProperties: false

propertyOrder: [time, unit, shape, interpolation, values]
flowStyle: block
...

Hello @CagtayFabry! Thanks for updating this PR.

In the file weldx/asdf/tags/weldx/core/time_series.py:

Line 8:1: F401 'weldx.core.MathematicalExpression' imported but unused


Proposal: A website for the SoulBoundNFT community on Klaytn (minters, airdrops and more great stuff)

I am embarking on a journey to create a groundbreaking decentralized application (dapp) that enables users to mint SoulBound Non-Fungible Tokens (NFTs) on the Klaytn blockchain network. The concept of SoulBoundNFTs goes beyond mere digital ownership; it encapsulates the emotional connection and uniqueness of digital assets, fostering a deep bond between creators and collectors.
The SoulBoundNFT minter dapp will leverage the robust infrastructure of the Klaytn blockchain, renowned for its scalability, security, and developer-friendly environment. Through this dapp, users will have the power to immortalize their digital creations, whether it be artwork, music, or any other form of digital content, as SoulBoundNFTs, imbued with a sense of authenticity and exclusivity.
By integrating with Klaytn, a leading blockchain platform with a strong community and ecosystem support, the SoulBoundNFT minter dapp aims to democratize the creation and ownership of NFTs, offering creators and collectors a seamless and efficient way to participate in the burgeoning digital asset space.
Furthermore, the development process will be open and transparent, as the source code will be hosted on the official GitHub repository of Klaytn. This not only ensures accountability but also invites collaboration and feedback from the wider developer community, fostering innovation and continuous improvement.
In summary, the SoulBoundNFT minter dapp on Klaytn represents a pioneering endeavor to redefine digital ownership and empower creators and collectors alike. With its user-centric design, seamless integration with Klaytn, and commitment to transparency, this dapp promises to revolutionize the way we perceive and interact with digital assets in the blockchain era.

Cool. Write and deploy the DApp on testnet and comment the repo link + testnet address here once you are done.

Yeah!!! Thank you for your waiting.
This is my repo: https://github.com/FucktheKingcode/Mint-SoulBound-NFT-Dapp.git
You can test my dapp at the website: https://dragon-soul-bound-nft-dapp.vercel.app/

Approved. DApp works well. Minted some fun stuffs.

I will be making an upcoming update to add token and NFT airdrop functions to the SoulBoundNFT address on the site itself

Cool. Let me know when you make an update. Will try it out.

Hi, please find the link to my website here:
https://dragon-soul-bound-nft-dapp.vercel.app/
Deployed address on Baobab: https://baobab.klaytnfinder.io/account/0xe6a6c69AEf46c72bF437dD7d90cDF6bEa9823d94
Deployed address on Cypress: https://klaytnfinder.io/account/0xFa593B4783e788B0F10cAc661b6dbc15E2165EF8
UI repo: https://github.com/FucktheKingcode/Mint-SoulBound-NFT-Dapp
Contracts repo: https://github.com/FucktheKingcode/Contract_SoulBoundNFT_Community

Looks ok overall. I would need a slight UI improvements to clearly demonstrate the ability to airdrop tokens to the addresses with the SoulBoundNFT
Approved for additional bounty after making the UI improvements

Yeah!!
I have improved the UI of the website and you can check it out.
My website here:
https://dragon-soul-bound-nft-dapp.vercel.app/
Deployed address on Baobab: https://baobab.klaytnfinder.io/account/0xe6a6c69AEf46c72bF437dD7d90cDF6bEa9823d94
Deployed address on Cypress: https://klaytnfinder.io/account/0xFa593B4783e788B0F10cAc661b6dbc15E2165EF8
UI repo: https://github.com/FucktheKingcode/Mint-SoulBound-NFT-Dapp
Contracts repo: https://github.com/FucktheKingcode/Contract_SoulBoundNFT_Community

approved for 2nd bounty payout. But for 3rd bounty payout, you would substantial UX improvements

Update proposal: I will add the user authentication functionality already in my discord and telegram communities to allow NFT minting and improve the UI/UX of the website.


simplify app-bridge initialization code

WHY are these changes introduced?
Simplify starter code (originally attempted in #743 )
WHAT is this pull request doing?
Removing a redundant line of code.

@paulomarg , I saw your comment here.
From what I'm seeing, our docs are a tiny bit misleading - The forceRedirect option is true by default in production environment, but it is false by default in development environment.
It was a nice attempt to try and simplify it, but I feel like it's better to forget about it for now.
Thanks for 🎩 ing and letting me know about the redirect issue 🙏
cc @vividviolet , @henrytao-me

Agreed that we should keep it as is for consistency across the environments in the generated app!

Maybe as a compromise we should add a comment above the forceRedirect: true to say that this can be turned off if you want to allow opening your app without Admin for testing? I just realized we have no docs on this config property. We only have docs for the React config: https://shopify.dev/apps/tools/app-bridge/react-components/provider#appconfig
I've seen lots of questions from developers about this. Sometimes they just want to see their app UI and test it without needing to open it in Admin.

I've seen lots of questions from developers about this. Sometimes they just want to see their app UI and test it without needing to open it in Admin.

Do you think this will continue to be a problem, now that we have HMR working out of the box for the UI?
That being said, I agree that the comment is probably helpful in either case!

we should update our docs and add type annotations

💯
It came up (specifically, around the config object) in the chat that Elana and I had about docs - Our shopify.dev content isn't consistent (some features have API docs, some only have code examples). We will fix it.


How can I use my custom font?

How can I use my custom font from an URL.
I added font-face to style tag at head tag, but it didn't work.
My template header like that:

why this issue was closed?

Quite possibly because this isn't a support forum.

Yes, that's not an issue with the templates. That falls more into the realm of further customization. For these base templates, our primary goals are simplicity, ease of use, and compatibility. While we may dabble at custom external fonts in the future, it's not currently something that they're designed to support without heavy customization.


Project does not have an explicit license

Hi @mendhak, thanks for this neat script! I'm interested in using it for Stripe's Java library, however your project currently does not have an explicit license. Would you consider releasing your code under an open-source license?

Oops that's a mistake, thanks for spotting it. I should have added one.  I'll try to add one soon.

OK - how is this does this look fine?
https://github.com/mendhak/Gradle-Travis-Colored-Output/blob/master/LICENSE

Awesome! Thanks for the quick response :) Keep up the good work!


Factories

According to #180 i know how should factory works, but it doesn't as I want to.
Laravel 5.3 with laravel-translatable:^6.0 (actual 6.1).
Model City:
class City extends Model
{
    use Translatable;
    protected $table = 'cities';
    protected $fillable = ['latitude', 'longitude'];
    public $translatedAttributes = ['name'];
    public $timestamps = false;
    public $translationModel = CityTranslation::class;

    public static function boot()
    {
        parent::boot();

        self::creating(function (self $city) {
            $city->user_id = Auth::id();
        });
    }

    public function user()
    {
        return $this->belongsTo(User::class);
    }
}

I've City factory wity City Translatable:
$factory->define(\App\Models\Common\City::class, function (Faker\Generator $faker) {
    return [
        'name:en' => $faker->city,
        'user_id' => 1,
        'approved' => true,
    ];
});

And always just FIRST test fails with error
Argument 1 passed to Illuminate\Database\Eloquent\Model::fill() must be of the type array, string given, called in /home/me/Code/app/vendor/dimsav/laravel-translatable/src/Translatable/Translatable.php on line 255

This first test have heper same as next few tests:
    protected function prepareClientAndProject()
    {
        factory(City::class)->create(['id' => 1, 'name:en' => 'London', 'approved' => true]);
        factory(Address::class)->create(['id' => 1, 'city_id' => 1]);
        factory(Client::class)->create(['id' => 1, 'address_id' => 1]);
        factory(Project::class)->create(['id' => 1, 'client_id' => 1]);
    }

What should I do now?

Sorry, it was my fault in config/translatable.php
<?php

$config = require __DIR__.'/app.php';

return [
    'locales' => $config['locales'],
    'locale_separator' => '_',
    'locale' => null,
    'use_fallback' => true,
    'fallback_locale' => $config['fallback_locale'],
    'translation_suffix' => 'Translation',
    'locale_key' => 'locale',
    'to_array_always_loads_translations' => true,
];

I had 'locales' => [$config['locales']], and system shows languages as "0_en" not just "en"


[Joint State Broadcaster] Add option to support only specific interfaces on specific joints

Joint state broadcaster support for specific joint_names and interface_names.
Extend documentation.

@bmagyar I updated the code to current state and according to your comments. Please have a look at this again.


Fix narrow threshold

This PR fixes an issue with the miller columns where columns are not shifting to the left after the 3rd level.
Before

After

Note
The update dependency commit is prevent this from failing CI but will be dropped before merge as it's captured in https://github.com/alphagov/miller-columns-element/pull/11/commits/4ecaab49fefcdb8fb8b6ea99064ec0027dbd8ad8.
Trello card

Nope, was meant to be dropped, same as dependencies. I dropped it now, will commit /dist on release.


AIX openj9 build failure: File libjvm.so is not an archive file

https://ci.adoptopenjdk.net/view/Failing Builds/job/build-scripts/job/jobs/job/jdk11u/job/jdk11u-aix-ppc64-openj9/926/console
20:11:32  ar: 0707-108 File libjvm.so is not an archive file.
20:11:32  gmake[6]: *** [runtime/redirector/CMakeFiles/jvm_redirect.dir/build.make:114: runtime/redirector/libjvm.so] Error 1
20:11:32  gmake[6]: *** Deleting file 'runtime/redirector/libjvm.so'
20:11:32  gmake[5]: *** [CMakeFiles/Makefile2:8295: runtime/redirector/CMakeFiles/jvm_redirect.dir/all] Error 2
20:11:32  gmake[5]: *** Waiting for unfinished jobs....

cmake is at 3.16 on the machine. The last passing build was done with cmake 3.14.3, however the AIX Toolbox repository seems to not be allowing me to properly downgrade back to that level :-(

OK managed to downgrade it by manually installing cmake-rpm-macros and cmake-filesystem then using yum install cmake-3.14.3. I've kicked off a new build at
Potentially we could also use the later version available from http://download.aixtools.net/tools/aixtools.cmake.<IP_ADDRESS>.I but I want to at least verify if the problems we're seeing is related to 3.16 by trying the version that previously completed the build successfully or if we have another problem to fix.

I found the same issue during AIX build.  How can i get cmake-3.14.3 package. Seems I can't access the link here. Thanks.

FYI. download the cmake-3.14.3 Thanks.


Success markers: write summary statistics

Overview of solution:
Implement a function which takes the summary statistics from compute_summary_stats and writes the output to a JSON file.
See implementation proposal for project overview.
Definition of done:

[ ] function implemented
[ ] function unit tested

effort: 1


Cox PH Regression Baseline Hazard - Categorical Variables

Hi,
I'm performing survival analysis on a data set, and getting some strange results.  I've created a reprex using the gbsb dataset from the lifelines datasets.
I'm new to survival analysis, so it's quite possible that my interpretation is at fault.  The issue is when I am looking at the coefficients of categorical variables in tandem with plotting covariate groups.  Here is the reprex:
# stats
from lifelines import *
from patsy import ModelDesc, dmatrix

# utility
import pandas as pd
import numpy as np

from lifelines.datasets import load_gbsg2
gbsb = load_gbsg2()

# select a few columns, including one categorical
gbsb_regr = gbsb[["tgrade", "time", "cens", "age", "tsize", "pnodes", "progrec"]]

# specify group III as the reference group
formula = ModelDesc.from_formula("""C(tgrade, Treatment('III')) + time + cens + age + tsize + pnodes + progrec""")

# get the dataframe using the formula - no intercept for CoxPH
gbsb_regr_encode = dmatrix(formula, data = gbsb_regr, return_type = "dataframe").drop(columns = ["Intercept"])

# Cox model - no regularization as we are trying to estimate coefficients
gbsb_cph = CoxPHFitter()
gbsb_cph.fit(gbsb_regr_encode, duration_col = "time", event_col = "cens")

gbsb_cph.print_summary()

print_summary() results:

gbsb_cph.plot_covariate_groups(["C(tgrade, Treatment('III'))[T.I]", "C(tgrade, Treatment('III'))[T.II]"], np.eye(2))

What I am confused about are the survival curves for the different encoded categorical variables.  The orange curve (level TII) has a higher hazard function (or lower survival) compared to baseline, which seems contrary to the coefficient estimate of -0.18.  From the coefficient estimate, I would expect a lowering of the hazard function.
I've read the issue #543 that states in lifelines the baseline hazard function is equal to the mean of the covariates - what does that mean for the reference-treatment coding I applied using patsy?
Thanks for writing a great package - the docs are excellent.

Hi @EvanYathon,
The baseline hazard function is a bit misleading here. Because we use the mean of the covariates, the baseline is evaluated at:
C(tgrade, Treatment('III'))[T.I]        0.118076
C(tgrade, Treatment('III'))[T.II]       0.647230
time                                 1124.489796
cens                                    0.435860
age                                    53.052478
tsize                                  29.329446
pnodes                                  5.010204
progrec                               109.995627
dtype: float64

The other two curves are evaluated at:
C(tgrade, Treatment('III'))[T.I]        0
C(tgrade, Treatment('III'))[T.II]       1
time                                 1124.489796
cens                                    0.435860
age                                    53.052478
tsize                                  29.329446
pnodes                                  5.010204
progrec                               109.995627
dtype: float64

and
C(tgrade, Treatment('III'))[T.I]        1
C(tgrade, Treatment('III'))[T.II]       0
time                                 1124.489796
cens                                    0.435860
age                                    53.052478
tsize                                  29.329446
pnodes                                  5.010204
progrec                               109.995627
dtype: float64

So the baseline should be inbetween the other two curves.
The problem is not your interpretation - it's how lifelines handles categorical variables for the baseline estimation. We do the naive thing (mean), but we should set categorical variables to all zeros (to be part of a refactor including #1060). I think you could hack this:
values = np.array([
    [1, 0],
    [0, 1],
    [0, 0],
])
gbsb_cph.plot_covariate_groups(["C(tgrade, Treatment('III'))[T.I]", "C(tgrade, Treatment('III'))[T.II]"], values, plot_baseline=False)

produces:

That makes sense.  Thank you for the quick reply!


ARC: Add the missing pieces for HS6x support

With the changes in this patch, one can use the latest GNU
toolchain to build for HS6x targets. Things that have changed are:

Pass "-mcpu=hs6x" instead of "-mcpu=arc64" to gcc
The updated output format is "elf64-littlearc64" (vs. "elf64-littlearc")
The BFD format (used by objcopy) was missing for HS6x and now is added.

This means that the SDK must be updated as well to reflect the latest, and
the greatest, toolchain.
Signed-off-by: Shahab Vahedi<EMAIL_ADDRESS>

Hi Shahab,
The mcpu change can be done right now, but unfortunately elf-output format change in include/linker/linker-tool-gcc.h break compatibility with the old SDK.
@abrodkin I guess we want to support both old & new SDKs at least for the transition period?

Closed in favor of 43760.


ADR 007 C4 & PlantUML

Couldn't agree more.

Can someone from @morganstanley/compose-developers provide a final review before merge?


Create dataplane services in dataplane_kuttl_prep

We want to get rid of the reconciliation of services in nodeset controller.

recheck
fatal: [localhost]: FAILED! => {"changed": false, "msg": "Failure downloading https://golang.org/dl/go1.19.10.linux-amd64.tar.gz, The read operation timed out"}",


notifu/balloon doesn't work on 32bit windows

Originally brought to our attention in #76, i've decided this needs it's own seperate issue, to track progress and everyone's thoughts.
Original message:

Hey @kurisubrooks just an update on the Win7 Balloon problems I was having. Finally got around to get a debug of the code and was receiving the error

"Error: ENOENT, node_modules/node-notifier/vendor/notifu/notifu not found in ..MyApp/resources/app.asar"

>So I took a look in the balloon.js and notice it was looking for ".../notifu/notifu" which I assume is to handle the regular .exe and the 64.exe. For this we are only building a 32 bit version, so I changed the .js to notifu.exe and it fixed the problem.

> Not sure if this will help fix the problem if you need x32 & x64, but just wanted to let you guys know. Thanks!
> – @sean6bucks

+1

+1

I don't have access to Windows 7 any more. Could someone try out the branch and see if this issue is fixed?
Install by doing:
npm install mikaelbr/node-notifier#notifuArch

@mikaelbr — Tested against Windows 7 32-bit. Works :+1:


Perf optimizations

This PR is in support of https://github.com/FormidableLabs/victory-chart/pull/93.  It include one hot-path performance improvement, as well as a multi-arg memoization function that is consumed by victory-chart.
It looks like I have to update a couple of tests for getStyles.  Also, this PR will probably have to wait until after the victory-core merge.
/cc @boygirl @coopy

I'm moving this PR over to victory-core, will check back here in case you reply @ryan-roemer.


Fix a race condition in AppState that prevents listeners from being notified

If someone has setup a subscription on AppState and we correct AppState via getCurrentAppState call, we need to notify all the subscribers of AppState.
1 ) Initial AppState.currentState = 'active'
2-start) Subscribe to AppState Changes
3-start) Fetch Current AppState
4 ) App Code subscribes to AppState module
5 ) App becomes backgrounded
2-finish) AppState listeners are setup (missing background event)
3-finish) AppState.currentState updated to background
At this point the subscription setup in 4) will never be called with the change.
Motivation
AppState should always call subscribers on change
Test Plan
This is very difficult to formally test since it's due to a race condition. We've seen this condition via bug reports but have had no local repro.
Release Notes
[GENERAL][BUGFIX][AppState] - Fix a race condition that could prevent AppState subscription change listener from firing on initial launch

@facebook-github-bot shipit


prevent html injection in cockpit

Could we also apply this to packages/embark-ui/src/components/Console.js line 94 and line 100 to prevent console command usage containing < and > from injecting elements in to the cockpit?

This might not be necessary, in favour of #1385. @emizzle WDYT?

Yep, looks good!


[magento] Fix ChangelogTemplate and remove link tags

We might need to refix changelogTemplate if they decide to use double digit version numbers
But until that day come this is a good solution

@captn3m0 Looks working without any problem , could you also make a test/review for me


Use Circuit Breaker (Issue #53)

Coverage decreased (-0.3%) to 99.2% when pulling 8702fec4a37b8d7cbd6581b51b27d681541c3233 on issue-53 into eb95a3f61dac7fff262a9fa3e47d09464dedd7f2 on master.


The underscore symbol is not recognized as part of the url

The resource can't update the status of the pull request if the name of the repo contains underscore. It gives the following error:
put failed: failed to set status: POST https://api.github.com/repos/Test/test-test_test/statuses/73a21b9b4d39386b85c765d117fc3af98f35a82a: 404 Not Found []

That may be an issue related to GitHub not supporting underscore characters _ in repo names, don't you think?
Anyway, there is no hope that you get any further support here.
But recent news is that the Cloud Foundry community is now maintaining a fork of this resource at cloudfoundry-community/github-pr-resource.
On our Concourse installation at Gstack, that one is working fine. Could you give a try and provide feedback there?
(the new resource is a drop-in replacement; you only have to switch telia-ossto cfcommunity in in the resource_types: declaration, i.e. specify repository: cfcommunity/github-pr-resource)


disable variable tracking for tests file on Linux

this adds the compile option -fno-variable-tracking when
compiling the catch test
this slightly speeds up compilation with newer g++ versions and prevents
the "variable tracking size limit exceeded" warnings from occurring.
additionally, removed an unused file from the lists of files to
compile for testing, and removed a test file to actually include "Test"
in its filename

http://jenkins01.arangodb.biz:8080/view/PR/job/arangodb-matrix-pr-linux/753/

http://jenkins01.arangodb.biz:8080/view/PR/job/arangodb-matrix-pr-linux/762/


Svelte 5 Support

Would it be possible to support Svelte 5? The stores API is unchanged, so in theory it should just be a matter of allowing Svelte 5 as a peer dependency

I just released the version 2.4.2 that allow Svelte 5 :slightly_smiling_face:

maybe this solution will solve your problem link

Thank you!


Orientation.addOrientationListener keep listening to device orientation change after Orientation.lockTo...

Hi,
I have a video element and I want to implement the following scenario:

device in portrait orientation.
click full screen - > video is full screen and in landscape orientation while device orientation is portrait.
user change device orientation to landscape -> video remain in full screen and in landscape orientation
user change device orientation to portrait  -> video exit full screen and change orientation to portrait.

in step 3 my onRotate method is never called, I think its because I use Orientation.lockToLandscape() in step 2 which, I think, prevent Orientation.addOrientationListener to listen to device orientation change.
I am using:
react native v0.52
react-native-orientation v3.1.0

hello @assafb81. how did you solved ( if you solved ) this problem ?

y workaround is:
In the end in step 2 after 2.5 seconds I unlocked  the orientation lock:
setTimeout(() => {
    Orientation.unlockAllOrientations();
}, 2500);

that period of time is good for my business logic

Thank you

This repo dead , use the fork https://github.com/wonday/react-native-orientation-locker


Update ResourceIdentity reference and remove ManagedInstanceMetric

Most of the changes in this PR are  auto-generated by SQL DB DSMainDev Repo:

change the reference of ResourceIdentity in https://github.com/ericshape/azure-rest-api-specs/blob/2382a3b1c11562eeba18480e7b8fd5490ae0129f/specification/sql/resource-manager/Microsoft.Sql/preview/2019-06-01-preview/servers.json#L349-L376 auto-generated
add None and UserAssigned to ResourceIdentity in SQL specific common type file: https://github.com/ericshape/azure-rest-api-specs/blob/33b5733cc82028c9ce6e2c9ec9f28e7f763098f7/specification/sql/resource-manager/common/v1/types.json#L100-L129 manually change

It would lead to breaking change: Add new values to the enum.

update all the references in 2020-08-01-preview spec files.  auto-generated
remove all the ManagedInstanceMetric from 2020-08-01-preview, which is requested by the owner and not ready to publish in Public API.  auto-generated

It would lead to breaking change: removing paths. But we have not published any doc and SDK for 2020-08-01-preview. It is fine for us.

MSFT employees can try out our new experience at OpenAPI Hub  - one location for using our validation tools and finding your workflow.

Changelog
Please ensure to add changelog with this PR by answering the following questions.

What's the purpose of the update?

[ ] new service onboarding
[ ] new API version
[ ] update existing version for new feature
[x] update existing version to fix swagger quality issue in s360
[ ] Other, please clarify

When you are targeting to deploy new service/feature to public regions? Please provide date, or month to public if date is not available yet.
When you expect to publish swagger? Please provide date, or month to public if date is not available yet.
If it's an update to existing version,  please select SDKs of specific language and CLIs that require refresh after swagger is published.

[ ] SDK of .NET (need service team to ensure code readiness)
[ ] SDK of Python
[ ] SDK of Java
[ ] SDK of Js
[ ] SDK of Go
[ ] PowerShell
[ ] CLI
[ ] Terraform
[ ] No, no need to refresh for updates in this PR

Contribution checklist:

[ ] I commit to follow the Breaking Change Policy of “no breaking changes
[ ] I have reviewed the documentation for the workflow.
[ ] Validation tools were run on swagger spec(s) and errors have all been fixed in this PR. How to fix?

If any further question about AME onboarding or validation tools, please view the FAQ.
ARM API Review Checklist

[ ] Ensure to check this box if one of the following scenarios meet updates in the PR, so that label “WaitForARMFeedback” will be added automatically to involve ARM API Review. Failure to comply may result in delays for manifest application. Note this does not apply to data plane APIs, all “removals” and “adding a new property” no more require ARM API review.

Adding new API(s)
Adding a new API version
Adding a new service

[x] Please ensure you've reviewed following guidelines including ARM resource provider contract and REST guidelines. Estimated time (4 hours). This is required before you can request review from ARM API Review board.

[ ] If you are blocked on ARM review and want to get the PR merged with urgency, please get the ARM oncall for reviews (RP Manifest Approvers team under Azure Resource Manager service) from IcM and reach out to them.

Breaking Change Review Checklist
If there are following updates in the PR, ensure to request an approval from API Review Board as defined in the Breaking Change Policy.

[ ] Removing API(s) in stable version
[ ] Removing properties in stable version
[ ] Removing API version(s) in stable version
[ ] Updating API in stable version with Breaking Change Validation errors
[ ] Updating API(s) in preview over 1 year

Action: to initiate an evaluation of the breaking change, create a new intake using the template for breaking changes. Addition details on the process and office hours are on the Breaking change Wiki.
Please follow the link to find more details on PR review process.

@PhoenixHe-msft  could you have a look at the failure of SDK azure-sdk-for-java

/azp run

The Java automation is not blocking the PR merge. And it shows in log that the Java generation is successful.

Hi @ericschung, can I know when ManagedInstanceMetric is added? in which PR?

Hi @ericschung, can I know when ManagedInstanceMetric is added? in which PR?

it was just added in 2020-08-01-preview two weeks ago. And we have not added it into any SDK and public doc. Thus, we are fine with it.
BTW, you might tag the wrong ID LOL

@akning-ms
I have addressed your comments and @jamestao also approved this PR. For the breaking changes, I have explained them in the beginning of the PR description.
Can you help us to merge this PR? Thanks!
BTW, the upcoming PR is ready: #11840 and it passed all the checks. Can you help to take a look?
In #11840, I cherry-picked the changes from this PR #11960  to make it through. I will rebase it once #11960 is merged.


Increase the size of the property_value column in the history store from TEXT to MEDIUMTEXT

Certain properties such as source.filebased.fs.snapshot can be too large to fit in a TEXT column.

@liyinan926 You mind chiming in?

I think it's ok to merge and I agree we should de-dup task properties that are already in job properties, which can be in a separate PR.

@liyinan926 I will provide a PR for the de-dupe.
Also, I find that saving history is extremely slow.  I was looking into replacing all the EXISTS->INSERT/UPDATE code with an UPSERT.  To do this, I believe the indexes on the metric and property tables would need to change.  In my local tests, it seems much faster.  Additionally, I was looking at making the insert of metrics/property be done in batches of up to 1000.  What do you think?

BTW, do you guys use the history store in prod?

Using batches and upserts sounds like a good idea to me.

@abti any other comments? I think this should be good to merge.

@abti @sahilTakiar I'd like to close this one in favor of #699


SyntaxHighlighter Not Working in Chrome on Mac OS X (El Capitan)

I have a Wordpress.com blog where I post code snippets using SyntaxHighlighter.  However, the code is not highlighted when viewing my blog in Google Chrome on a Mac.  All other browsers / platforms seem to show the code correctly highlighted.
Here is a blog post that shows the behavior: http://blog.tonysneed.com/2016/02/27/visual-studio-code-typescript-part-2/
Here is a screenshot of the post in Chrome on a Mac.

Thanks,
Tony Sneed

Which chrome version? Could you check the console for errors?
On Thu, Mar 3, 2016, 12:20 AM Anthony Sneed<EMAIL_ADDRESS>wrote:

I have a Wordpress.com blog where I post code snippets using
SyntaxHighlighter. However, the code is not highlighted when viewing my
blog in Google Chrome on a Mac. All other browsers / platforms seem to show
the code correctly highlighted.
Here is a blog post that shows the behavior:
http://blog.tonysneed.com/2016/02/27/visual-studio-code-typescript-part-2/
Here is a screenshot of the post in Chrome on a Mac.
[image: wp-code-chrome]
https://cloud.githubusercontent.com/assets/2836367/13488381/22fb3b08-e121-11e5-87ae-1e6f09ee893b.png
Thanks,
Tony Sneed
—
Reply to this email directly or view it on GitHub
https://github.com/syntaxhighlighter/syntaxhighlighter/issues/379.

You can close this now.  I re-installed Google Chrome on my Mac, and the syntax highlighting works now.  Thanks!

On Mar 6, 2016, at 3:25 PM, Alex Gorbatchev<EMAIL_ADDRESS>wrote:
Which chrome version? Could you check the console for errors?
On Thu, Mar 3, 2016, 12:20 AM Anthony Sneed<EMAIL_ADDRESS>wrote:

I have a Wordpress.com blog where I post code snippets using
SyntaxHighlighter. However, the code is not highlighted when viewing my
blog in Google Chrome on a Mac. All other browsers / platforms seem to show
the code correctly highlighted.
Here is a blog post that shows the behavior:
http://blog.tonysneed.com/2016/02/27/visual-studio-code-typescript-part-2/
Here is a screenshot of the post in Chrome on a Mac.
[image: wp-code-chrome]
https://cloud.githubusercontent.com/assets/2836367/13488381/22fb3b08-e121-11e5-87ae-1e6f09ee893b.png
Thanks,
Tony Sneed
—
Reply to this email directly or view it on GitHub
https://github.com/syntaxhighlighter/syntaxhighlighter/issues/379.

—
Reply to this email directly or view it on GitHub https://github.com/syntaxhighlighter/syntaxhighlighter/issues/379#issuecomment-192895739.


Datetime attribute resets to 1970 when saving a product.

A datetime attribute (popup calender) gets the wrong date format if the browser language is set to for example Swedish.
For example 9/3/2016 12:00 PM is changed to 9/3/2016 12:00 em which causes the backend to set the date to 1970 (epoch zero)

After some more detailed digging into this I found the source of this problem. It is a bug in the Magento Adminhtml module.
If you set your brower language to for example swedish, the following http header will be sent to the server:
Accept-Language: sv, en-US;q=0.8, en;q=0.6, nb;q=0.4, fr;q=0.2

When Magento is rendering the datetime element, using Varien_Data_Form_Element_Datetime the value is converted from the internal date format in Varien_Data_Form_Element_Date::setValue, row 106
$this->_value = new Zend_Date($value, $format, $locale);

Since the $locale variable is null in the above code, Zend_Date will use Zend_Locale to find the correct locale, which will fallback and get the locale from the http header, sv in this case.
When the product is saved, the same conversion is done but reversed in the datetime backend model, Mage_Eav_Model_Entity_Attribute_Backend_Datetime:: formatDate , row 86
However, here the locale from the system configuration is used (en_US) instead of the http header.
My guess is that the intention from Magento is to use the system configuration setting in all places, and the fix would be to set the correct locale to Varien_Data_Form_Element_Datetime before it is rendered.
This could be done in Mage_Adminhtml_Block_Widget_Form:: _setFieldset , row 212
    ... more code ...
                } else if ($inputType == 'datetime') {
                    $element->setImage($this->getSkinUrl('images/grid-cal.gif'));
                    $element->setTime(true);
                    $element->setStyle('width:50%;');
                    $element->setFormat(
                        Mage::app()->getLocale()->getDateTimeFormat(Mage_Core_Model_Locale::FORMAT_TYPE_SHORT)
                    );
    ... more code ...

by adding $element->setLocale(Mage::app()->getLocale()->getLocale());
This would fixes the issue for all locales, not just Swedish. I tried Norwegian, and it has a similar issue by converting AM to a.m.. I think it is safe to assume there are more locales that has a similar issue.


fix: change buildpack, builder and stack ids

Instead of using com.redhat.faas, these ids should be prefixed dev.boson.
Fixes: https://github.com/boson-project/buildpacks/issues/42
Signed-off-by: Lance Ball<EMAIL_ADDRESS>

@zroubalik thanks for the review. I've addressed your points.


HHH-9787 Remove outdated Infinispan configuration elements

@galderz Please checkout this PR.

Looks good, integrated


Update get_EAR_reviewer.py

Fix current implementation that is prioritizing older reviews before considering the number of total reviews

@arash77 I think this will fix the issue that I found the other day, do you agree? I'm also looking at the justification messages below the table...

Yes, this will lead to better ordering by checking Total Reviews before Parsed Last Review.

Great, thanks for look at it. I'll merge it :)


Make gasMultiplier take floats.

I think it would be a lot more flexible to allow for floats as gasMultiplier.

Added in https://github.com/tellor-io/TellorMiner/commit/a061e72a8072166de8984e0f3c9f8bca2145e542.


HTML

HTML
User Story
As a bootcamp student
I want the prework notes to be structured on a webpage
so that I can easily find and read information
##Acceptance Criteria
GIVEN a Prework Study Guide website
When I visit he website in my browser
Then I see four boxes titled HTML, CSS, Git, and JavaScript with associated notes listed

Done


Fix for llvm identifiers

(Original issue 442 created by erickt on 2009-09-28T02:06:38+00:00)
According to the [http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/AsmParser/LLLexer.cpp?revision=82784&view=markup llvm asm lexer], it's a little more liberal with identifiers than what pygments expects. Here's the relevent comments:
{{{
/// LexAt - Lex all tokens that start with an @ character:
///   GlobalVar   @\"[^\"]\"
///   GlobalVar   @[-a-zA-Z$._][-a-zA-Z$._0-9]
///   GlobalVarID @[0-9]+
/// LexPercent - Lex all tokens that start with a % character:
///   LocalVar   ::= %\"[^\"]\"
///   LocalVar   ::= %[-a-zA-Z$._][-a-zA-Z$._0-9]
///   LocalVarID ::= %[0-9]+
/// LexQuote - Lex all tokens that start with a " character:
///   QuoteLabel        "[^"]+":
///   StringConstant    "[^"]*"
/// LexIdentifier: Handle several related productions:
///    Label           [-a-zA-Z$._0-9]+:
///    IntegerType     i[0-9]+
///    Keyword         sdiv, float, ...
///    HexIntConstant  [us]0x[0-9A-Fa-f]+
}}}
I've attached a patch that fixes this.

(Original issue was assigned to gbrandl)
(Original issue 442 last updated on 2010-01-01T18:40:38+00:00)
(Issue automaticaly closed due to status in Bitbucket: resolved)

(Original comment by gbrandl on 2010-01-01T18:40:38+00:00)
Thanks for the patch, applied in [bacdff38f9bb].

(Original comment by thatch on 2009-10-08T16:26:20+00:00)
Thanks for the patch. Can you provide an example file that includes the newly-highlighted keywords?