Firebase_Auth_CSharp_UserInfoInterface_ProviderId_get line 28

[REQUIRED] Please fill in the following fields:

Unity editor version: 2020.3.21f1
Firebase Unity SDK version: 8.6.2
Source you installed the SDK: .unitypackage
Problematic Firebase Component: Auth
Other Firebase Components in use: Database, Analytics, Messaging, Crashlytics, Dynamic Links, Installations
Additional SDKs you are using: Ironsource (Facebook, Admob,
Platform you are using the Unity editor on: Windows
Platform you are targeting: Android & iOS
Scripting Runtime: il2cpp

[REQUIRED] Please describe the issue here:
We did not change anything related to Auth. The crash started happening out of nowhere. After firebase token is fetched, we get the provider ID as a string. Fetching of provider ID does not happen if this process is faulted or canceled. Here is the stack trace of the crash recorded :
at libFirebaseCppApp-8_6_2.Firebase_Auth_CSharp_UserInfoInterface_ProviderId_get(Firebase_Auth_CSharp_UserInfoInterface_ProviderId_get:28)
at libil2cpp.0x1d0087c()
at libil2cpp.0x1d05da8()
at libil2cpp.0xd6e8ac()
at libil2cpp.0xd6e328()
at libil2cpp.0x21df624()
at libil2cpp.0x1613350()
at libil2cpp.0xf807f0()
at libil2cpp.0x19826f4()
at libil2cpp.0x1613a80()
at libil2cpp.0x1613884()
at libil2cpp.0x1595020()
at libil2cpp.0x98b098()
at libil2cpp.0x8e93a8()
at libil2cpp.0x8b2838()
at libil2cpp.0x87ea94()
at libil2cpp.0x87d418()
at libc.__pthread_start(void*)(__pthread_start:64)
at libc.__start_thread(__start_thread:64)
This crash was also noticed in firebase version 8.6.1. I was not able to reproduce this crash. We've recorded in total of 100 instances of this crash since 2 weeks.

Hi @Originator99,
Thanks for reporting this. In order to identify what's causing this behavior, could you verify if this issue persists using the quickstart? If so, please provide the complete steps to reproduce the issue.

Hi @paulinon , thank you for the quick response.
I was not able to re-produce this using quickstart.

Hi @Originator99,
Version 8.7.0 of the SDK has been released recently. Could you try this out and see if it resolves the issue? If it doesn't, kindly provide a sample code that I could try on my end along with the complete steps to replicate this issue.

Hi @Originator99,
Version 8.7.0 of the SDK has been released recently. Could you try this out and see if it resolves the issue? If it doesn't, kindly provide a sample code that I could try on my end along with the complete steps to replicate this issue.

Hi ,
I'll do the upgrade and get back to you with results in 1.5-2 weeks at Max.
Thank you

Hi @paulinon
Thank you for your patience.
We had made a release 2 days ago with firebase upgraded to version 8.7.0.
Unfortunately, the crash seems to be happening in this version too.  However there was a new crash trace that was logged.
backtrace:
  at __memcpy
  at std::__ndk1::vector<firebase::auth::UserInfoInterface*, std::__ndk1::allocator<firebase::auth::UserInfoInterface*> >::__swap_out_circular_buffer(std::__ndk1::__split_buffer<firebase::auth::UserInfoInterface*, std::__ndk1::allocator<firebase::auth::UserInfoInterface*>&>&)
  at std::__ndk1::vector<firebase::auth::UserInfoInterface*, std::__ndk1::allocator<firebase::auth::UserInfoInterface*> >::__append(unsigned long)
  at firebase::auth::User::provider_data() const
  at Firebase_Auth_CSharp_FirebaseUser_ProviderData_get
  at
  at
  at
  at
  at
  at
  at
  at
  at
  at
  at
  at
  at
  at
  at
  at
  at __pthread_start(void*)
  at __start_thread
  at libc.__memcpy
  at libFirebaseCppApp
  at libFirebaseCppApp-8_7_0.std::__ndk1::vector<firebase::auth::UserInfoInterface*, std::__ndk1::allocator<firebase::auth::UserInfoInterface*> >::__append (unsigned long)(__append:152)
  at libFirebaseCppApp-8_7_0.firebase::auth::User::provider_data () const(provider_data:112)
  at libFirebaseCppApp-8_7_0.Firebase_Auth_CSharp_FirebaseUser_ProviderData_get (Firebase_Auth_CSharp_FirebaseUser_ProviderData_get:8)
  at libil2cpp.0x1db3f58 (Native Method)
  at libil2cpp.0x1db7344 (Native Method)
  at libil2cpp.0xc8afac (Native Method)
  at libil2cpp.0xc8ac50 (Native Method)
  at libil2cpp.0x21fa0c8 (Native Method)
  at libil2cpp.0x167cc70 (Native Method)
  at libil2cpp.0x1052724 (Native Method)
  at libil2cpp.0x1a4c358 (Native Method)
  at libil2cpp.0x167d3a0 (Native Method)
  at libil2cpp.0x167d1a4 (Native Method)
  at libil2cpp.0x15fb940 (Native Method)
  at libil2cpp.0x9c4bdc (Native Method)
  at libil2cpp.0x922bd8 (Native Method)
  at libil2cpp.0x8ec068 (Native Method)
  at libil2cpp.0x8b82c4 (Native Method)
  at libil2cpp.0x8b6c48 (Native Method)
  at libc.__pthread_start
  at libc.__start_thread (__start_thread:64)

Hi , thought I'd drop an update with a comment of the code snippet where the crash seems to be happening in our game. I am also including a better stack trace of the crash recorded.
public void UpdateAuthProvidersData() {
        Firebase.Auth.FirebaseUser user = auth.CurrentUser;
        Debug.Log(user);
        if(user != null) {
            IEnumerable<Firebase.Auth.IUserInfo> user_profiles = user.ProviderData; //at this line where crash seems to be happening
            foreach(Firebase.Auth.IUserInfo info in user_profiles) {
                if(info.ProviderId == "facebook.com") {
                    isFacebookConnected = true;
                }
                if(info.ProviderId == "playgames.google.com") {
                    isGooglePlayConnected = true;

                }

                if(info.ProviderId == "apple.com") {
                    isAppleConnected = true;
                }
                Debug.Log($"{info.ProviderId} is linked");
            }
        } else {
            Debug.LogError("Firebase User is null");
        }
    }

I have marked the line where the crash seems to be happening According to this stack trace :

Thread 0 (crashed)
0   libc.so                            0x000000764adbe808 <symbols missing for uuid: 554cb674fad07588ff08040bb89924c9>
1   libFirebaseCppApp-8_7_0.so         0x00000073a7b89a68 <symbols missing for uuid: 3104bb0105330426b5c947b4131b049c>
2   libFirebaseCppApp-8_7_0.so         0x00000073a7b94dfc <symbols missing for uuid: 3104bb0105330426b5c947b4131b049c>
3   libFirebaseCppApp-8_7_0.so         0x00000073a7b93f84 <symbols missing for uuid: 3104bb0105330426b5c947b4131b049c>
4   libFirebaseCppApp-8_7_0.so         0x00000073a7b86a0c <symbols missing for uuid: 3104bb0105330426b5c947b4131b049c>
5   libil2cpp.so                       0x0000007537a01f58 AuthUtilPINVOKE_FirebaseUser_ProviderData_get_m1B89485E8AF0AFEDFE4E34690516E15A9CFF79CE (D:\HitwicketUnity2\Library\Il2cppBuildCache\Android\armeabi-v7a\il2cppOutput\Firebase.Auth.cpp:6000)
6   libil2cpp.so                       0x0000007537a05344 FirebaseUser_get_ProviderData_mC41C8C4DC928DEA05201E099E541185693483607 (D:\HitwicketUnity2\Library\Il2cppBuildCache\Android\armeabi-v7a\il2cppOutput\Firebase.Auth.cpp:8829)
7   libil2cpp.so                       0x00000075368d8fac FirebaseLinkController_UpdateAuthProvidersData_m695238F4CA598C1016C49BC67F25628F3D6C3FB5 (D:\HitwicketUnity2\Library\Il2cppBuildCache\Android\armeabi-v7a\il2cppOutput\Assembly-CSharp11.cpp:50415)
8   libil2cpp.so                       0x00000075368d8c50 FirebaseLinkController_InitializeFirebaseExternal_m22159447855DF3975A95C956DF913528B793E75E (D:\HitwicketUnity2\Library\Il2cppBuildCache\Android\armeabi-v7a\il2cppOutput\Assembly-CSharp11.cpp:49381)
9   libil2cpp.so                       0x0000007537e480c8 Action_1_Invoke_m587509C88BB83721D7918D89DF07606BB752D744_gshared (D:\HitwicketUnity2\Library\Il2cppBuildCache\Android\armeabi-v7a\il2cppOutput\Generics1.cpp:0)
10  libil2cpp.so                       0x00000075372cac70 Task_Execute_m35673A0487248C5577449A4A3FBDF7FD3B6BA086 (D:\HitwicketUnity2\Library\Il2cppBuildCache\Android\armeabi-v7a\il2cppOutput\mscorlib16.cpp:76)
11  libil2cpp.so                       0x0000007536ca0724 ContextCallback_Invoke_mF4F8496213E8F0925947DD8994A477AE2E54EFDF (C:\Program Files\Unity\Hub\Editor\2020.3.21f1\Editor\Data\il2cpp\libil2cpp\vm\ClassInlines.h:0)
12  libil2cpp.so                       0x000000753769a358 ExecutionContext_RunInternal_mC5D58D6EDE270B4CDA05181E9064E040D6692B2B (D:\HitwicketUnity2\Library\Il2cppBuildCache\Android\armeabi-v7a\il2cppOutput\mscorlib6.cpp:27034)
13  libil2cpp.so                       0x00000075372cb3a0 Task_ExecuteWithThreadLocal_m25476BDF5AAFF102A95E92CB1E0989DFF72408A8 (D:\HitwicketUnity2\Library\Il2cppBuildCache\Android\armeabi-v7a\il2cppOutput\mscorlib16.cpp:28867)
14  libil2cpp.so                       0x00000075372cb1a4 Task_ExecuteEntry_m55634616576BB085EF3CEDB04801C4183FE1B053 (D:\HitwicketUnity2\Library\Il2cppBuildCache\Android\armeabi-v7a\il2cppOutput\mscorlib16.cpp:28731)
15  libil2cpp.so                       0x0000007537249940 ThreadPoolWorkQueue_Dispatch_mCC5743D99870EC7844CAB3FF3FA7D23A44A4A984 (D:\HitwicketUnity2\Library\Il2cppBuildCache\Android\armeabi-v7a\il2cppOutput\mscorlib17.cpp:110)
16  libil2cpp.so                       0x0000007536612bdc RuntimeInvoker_FalseSByte_t928712DD662DC29BA4FAAE8CE2230AFB23447F0B(void (*)(), MethodInfo const*, void*, void**) (D:\HitwicketUnity2\Library\Il2cppBuildCache\Android\armeabi-v7a\il2cppOutput\Il2CppInvokerTable.cpp:110398)
17  libil2cpp.so                       0x0000007536570bd8 worker_thread(void*) (C:\Program Files\Unity\Hub\Editor\2020.3.21f1\Editor\Data\il2cpp\libil2cpp\mono\ThreadPool\ThreadPoolWorkerThread.cpp:250)
18  libil2cpp.so                       0x000000753653a068 il2cpp::vm::ThreadStart(void*) (C:\Program Files\Unity\Hub\Editor\2020.3.21f1\Editor\Data\il2cpp\libil2cpp\vm\Thread.cpp:708)
19  libil2cpp.so                       0x00000075365062c4 il2cpp::os::Thread::RunWrapper(void*) (C:\Program Files\Unity\Hub\Editor\2020.3.21f1\Editor\Data\il2cpp\libil2cpp\os\Thread.cpp:183)
20  libil2cpp.so                       0x0000007536504c48 il2cpp::os::ThreadImpl::ThreadStartWrapper(void*) (C:\Program Files\Unity\Hub\Editor\2020.3.21f1\Editor\Data\il2cpp\libil2cpp\os\Posix\ThreadImpl.cpp:125)
21  libc.so                            0x000000764ae25188 <symbols missing for uuid: 554cb674fad07588ff08040bb89924c9>
22  libc.so                            0x000000764adc49d0 <symbols missing for uuid: 554cb674fad07588ff08040bb89924c9>

Thanks for the update, @Originator99. Could you provide a sample project that contains this function so that we can identify what's causing the behavior?

Thanks for the update, @Originator99. Could you provide a sample project that contains this function so that we can identify what's causing the behavior?

Sure @paulinon , I will provide you with a sample project which contains this function.


Fixed MixMaterial.isReadyForSubMesh to remove WebGL warnings

Hi,
I took over this PR which corrected the TerrainMaterial and had the same problem with MixMaterial.
So I copy the patches on MixMaterial hoping it's good for you @julien-moreau
https://github.com/BabylonJS/Babylon.js/commit/1a29f504d33f5031b4ec3aae90fadfd67c814e91

There is a lint issue (some trailing whitespaces): https://travis-ci.com/BabylonJS/Babylon.js/jobs/176531346

Ok, I to correct the extra spaces and I add a test if the mixMap2 is null to not test diffuse textures 5,6,7 and 8.

Added a last review, then everything looks good to me :)

Should be good now.

Ok.

Like this ?

Good to me !

No, I did not test it, but it should remove the WebGL error as you did on the TerrainEditor.

Anyway looks ok
Can you fix the lint issue and update the what’s new? Thanks!

It is add.

we still have lint issues:
https://travis-ci.com/BabylonJS/Babylon.js/jobs/176649995

Decidedly ... I find it annoying all its spaces that adds itself Why can not it be removed automatically by compiling it?
Should be good now.

All checks passed, you rox :)

Cool. Ready to be validated so. :+1:

Thanks a lot!!!


cysec_header_profile is missing as config value in cfgressources file

the "cysec_header_profile" config variable is missing as config sensible value in cfgressources file.

The header profile is there now I need to write a sensible processing for the config value.
Proposed something like https://{host}:{port}/{context}/app/users.jsp


Store is empty after deleting all cookies

I though store.js is not using cookies ?
So why is my storage away after deleting all cookies ?

Your browser probably did it, by flushing all cookies AND local storages

@lightsnow0: @thomasgodart is right, removing cookies also clears localstorage in almost all browsers.


"The TypeScript language service died 5 times right after it got started. The service will not be restarted"

VSCode Version: Code 1.19.3 (7c4205b5c6e52a53b81c69d2b2dc8a627abaa0ba, 2018-01-25T10:25:01.248Z)
OS Version: Darwin x64 17.4.0
Extensions:

Extension
Author (truncated)
Version

mithril-emmet
Fal
0.6.1

code-settings-sync
Sha
2.8.7

vs-code-css-comments
ash
1.0.3

vscode-intelephense-client
bme
0.8.5

sublime-keybindings
ms-
3.0.3

vetur
oct
0.11.7

vim
vsc
0.10.13

(5 theme extensions excluded)

Steps to Reproduce:

Reproduces without extensions: Yes/No

A few questions:

Does this happen if you disable all extensions?
Does this happen for all projects you open, or just one specifically?
Have you set a custom typescript.tsdk value in your workspace or user settings?

Can you also please try collecting the TypeScript server logs:

Add "typescript.tsserver.log": "verbose" to your user settings and restart VSCode.
Reproduce the issue
Back in the TypeScript section of the output panel, look for a TSServer exited message. Right after that message there should now be the path to a log printed. Open that log directory and look in the tsserver.log file

Look for errors, stack traces, or other suspicious info near the end of the logs. If you can share the logs I can try looking through them to see what is going on

⚠Warning: The TypeScript log may include information from your workspace, including file paths and source code. If you have any concerns about posting this publicly on Github, just let me know and we can arrange something else. On our side, we only use these logs to investigate issues like this


DPE-2217 premptively switchover

Issue
On scale down of primary, MySQL will reelect the primary after some delay.
Solution
Minimize delay by preemptively setting primary to unit 0 (last one to be torn down) if primary is being removed.

What happens if unit 0 is offline?

it would go boom. fixed on df0cd64

will this change remain k8s-only? if not, is it safe on vm—where having a unit 0 is not guaranteed?

VM need's another rule - commonly one would choose the leader unit. Eventually we merge the implementations differing the chosen unit as only difference


Show debug prompt when tail is cancelled (ctrl+c)

Also not sure how I feel about ctrl-c not aborting the CLI but only aborting part of the command. It might not be what's expected. We could decide to unhide defang debug and show that in a hint after abort.

This is a misunderstanding from our meeting yesterday, I was under the impression this is intended solution for quicker jump to debug when user sees error and wants to stop. If that is not the intended behaviour, i'll remove this behavior.

already in main


Licence is incorrectly rejected

I have trouble validating my crate due to the second term at https://github.com/ResearchObject/ro-crate-validator-py/blob/c3c5c18509c00ff51a96d430ed5ba5dc3e50b86c/src/rocrateValidator/semanticCheck.py#L581 It excludes the root data entity but shouldn’t only ro-crate-metadata.json be excluded?

@bronger I think that is correct.  I'm attaching a valid URL license to the root data entity and always getting an IDError result from upd_licenseRlt.  I would like to understand why that second part of the if statement is necessary.

I'm not clear on why you can't have a license on ro-crate-metadata.json either. I'm following the pattern here to attach a separate CC0 license for the metadata itself: https://www.researchobject.org/ro-crate/1.1/contextual-entities.html#metadata-license


add a method set_common_block

Sometimes people need to add some common block (such as headers) for a 'desc' method in a given class which inherits from class Grape::API. So this method may be useful.
By this method:
You can set a common block for a specified class.
It works only when you do not pass a block for the desc method.
  This can reduce many repeated code when you need to add a same header for many requests.
  You can just put them in the same class which inherits from class Grape::API.

  @example
  By the following code, you can add a common header 'Token' for the first two requests.
  But the third one is not added the header 'Token'.

      init_common_block do
        headers Token: { description: 'Token', required: true }
      end

      desc 'create a user'
      post '/users' do
        # ...
      end

      desc 'find a user'
      get '/user/:id' do
        # ...
      end

      desc 'update a user' do
      end
      put '/user/:id' do
        # ...
      end

I am not a big fan of automatically reused blocks. We already have reusable params via use. Maybe we want a similar approach for the rest of the things?

Thanks for your reply!
I'm sorry but I've not known exactly what you mean by saying  "We already have reusable params via use.".
However, there are two reasons that make me want to add this method:
1,"reusable params via use "  will add something in the url rather than in the HTTP Header. Am I right?
In my project, I'd like to add something in the Header, such as "Token" which is for security. And this is something I refer to:
https://www.owasp.org/index.php/Cross-Site_Request_Forgery_(CSRF)_Prevention_Cheat_Sheet#General_Recommendation:_Synchronizer_Token_Pattern
2,I want to only add common headers for specific class which inherits form Grape::API, not all classes under a version. So in my opinion, this new method may make things flexible.
Thanks again!

I see the need for shared headers (or shared anything). What I am saying is that I'd rather see the DSL be the same for headers as for params, aka:
helpers do
   headers :owasp do
       header ...
   end
end

headers do
  use :owasp
end

Cool. Lets close this for now.

ok.


ability to use cd in the before hook execute command

I would like to create a global plugins directory but do not want to commit the plugins to git. So everytime terragrunt | init |  apply | plan are run they have a before hook that cd's into the global directory and does a terraform init.
terraform {
before_hook "before_hook" {
    commands     = ["init", "apply", "plan"]
    execute      = ["cd", "${get_parent_terragrunt_dir()}/global", "&&", "terraform init"]
  }
  extra_arguments "init_args" {
    commands = [
      "init"
    ]

    arguments = [
      "-get-plugins=true",
      "-plugin-dir=../global/.terraform/plugins/linux_amd64/",
    ]
  }
  }

cd is not an executable so it does not work! any other possibilities to accomplish this?
terragrunt version v0.23.8

To answer your direct question, Terragrunt expects a binary and arguments to be passed, whereas  cd and && are shell built-ins. If you want to rely on those, you need to execute a shell explicitly: something along the lines of:
execute = ["bash", "-c", "cd ${get_parent_terragrunt_dir()}/global && terraform init"]

Also, Terraform has a way to configure the provider cache globally.

thankyou for your response. Yes, I eventually used bash.


Using runtime locale?

Is there a way to load locales based on a runtime variable?
I'd rather not import every package prior to knowing the locale. :)

Hey @sharpner
I did use to have code generated that would accept the local string eg. en_US I did end up removing it however as having all the locales imported dramatically increases you're compile time and binary size; because the locale is found at runtime that means ALL locales will be compiled and included in your binary, even if you only use one.
however I have had a few people ask about it, so as long as the tradeoffs aren't a big deal I'll add it back next time I regenerate.

I understand. Thanks for keeping it in mind :)

https://github.com/bep/gotranslators


also remove ca.key

See issue Distribute CA cert without key #1031
What's in this PR?
When distributing the ca cert to other namespaces, also remove the ca.key. This leaves the ca.crt as the only item in the secret.
Checklist

[x] Code meets the Developer Guide

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


Add manual coordinate constraints to partition_pdf().

Is your feature request related to a problem? Please describe.
I'm using hi_res strategy for my PDF files because I need to extract all images and tables etc. and the PDF files have headers and footers that I wish to remove.
Normally it wouldn't be a problem because I'll just use the element.metadata.coordinates.points to filter them out but in my documents the headers and footers have logos in it which makes it an Image and partition_pdf() will run OCR on it. And OCR is slow.
By specifying start/end coordinates, elements that are considered "outside" shall not go through OCR and thus saving time.
Describe the solution you'd like
Ability to specify start/end coordinates in partition_pdf() function. Maybe it can be added to other partition functions as well.
Describe alternatives you've considered
I came across https://github.com/Unstructured-IO/unstructured/pull/2455 but it's for fast strategy and it doens't allow manual constraints too.
Right now I need to wait for the partition_pdf() to finish and filter the header and footers out using element.metadata.coordinates.points.
cleaned_elements = [
    element
    for element in elements
    # Nuke the element even if 1 point is outside the cutcoord.
    if all(
        cutcoord_top < coord[1] < cutcoord_bottom
        for coord in element.metadata.coordinates.points
    )
    and (start_page_number < element.metadata.page_number < stop_page_number)
]

Additional context
Nope.

Hi @ChiNoel-osu ! Thanks for submitting this. Would filtering on the coordinates after partitioning solve your use case?

cutcoord_top

Hi,  @ChiNoel-osu
the “cutcoord_top"  "cutcoord_bottom" come from where？and how to get the value of stop_page_number? could you help me?

Hi, @ChiNoel-osu the “cutcoord_top" "cutcoord_bottom" come from where？and how to get the value of stop_page_number? could you help me?

Hi @huangpan2507 they're just custom variables.

Hi, @ChiNoel-osu the “cutcoord_top" "cutcoord_bottom" come from where？and how to get the value of stop_page_number? could you help me?

Hi @huangpan2507 they're just custom variables, change them to get different results.

@ChiNoel-osu  Thanks for your reply, but I still wonder these custom variables, how to get these variables? I had met a problem about partition_pdf, I want to filter the Header and footer, but oops, it extract something like
"Page2of 29
2024-1-1
Leave Policy xxx in China
Human Resources"

these thing is about the Header, it is in the element .category :CompositeElement.
That is to say, when I want to use partition_pdf to deal with pdf file(text, table, pic inside),
I print the element.category, the result shows like this:
element .category :CompositeElement element .category :Table element .category :Table element .category :CompositeElement element .category :Table element .category :CompositeElement ....
then,
"Page2of 29
2024-1-1
Leave Policy xxx in China
Human Resources" (these words are about Header)，they are in the element .category :CompositeElement,  how can I filter the Header and footer?  Can you help me?

@huangpan2507 When you do partition_pdf(), each element will have a coordinate that you can find by going into element.metadata.coordinates.points. Then you can easily check those numbers and filter out elements you don't need.
coord[1] in my code is its y coordinate, cutcoord_top and cutcoord_bottom will be based on the location of header and footer in your PDF.

@huangpan2507 When you do partition_pdf(), each element will have a coordinate that you can find by going into element.metadata.coordinates.points. Then you can easily check those numbers and filter out elements you don't need. coord[1] in my code is its y coordinate, cutcoord_top and cutcoord_bottom will be based on the location of header and footer in your PDF.

Great! Thanks for your kindly help, I will try it in my case.

@huangpan2507 When you do partition_pdf(), each element will have a coordinate that you can find by going into element.metadata.coordinates.points. Then you can easily check those numbers and filter out elements you don't need. coord[1] in my code is its y coordinate, cutcoord_top and cutcoord_bottom will be based on the location of header and footer in your PDF.

Hi, @ChiNoel-osu By the way, I had try to use the y coordinate to filter the Header and footer, but , the question is if I had a lot of different PDF files(the location of Header and footer are not the same)，So, I need to check the y coordinate of every pdf?

Hi, @ChiNoel-osu By the way, I had try to use the y coordinate to filter the Header and footer, but , the question is if I had a lot of different PDF files(the location of Header and footer are not the same)，So, I need to check the y coordinate of every pdf?

Unfortunately yes.
However if your header and footers all have the same content then you could just leave them alone and replace their text with empty string later.


Linux-2  failing to install/remove applications

Phone is failing to install/remove applications. This is most likely a test bed problem:
[flutter_gallery__back_button_memory] [STDOUT] stdout: [+3169 ms] Performing Streamed Install
[flutter_gallery__back_button_memory] [STDOUT] stdout:
[flutter_gallery__back_button_memory] [STDOUT] stdout:            adb: failed to install /opt/s/w/ir/k/recipe_cleanup/tmp_JnhbO/flutter sdk/dev/integration_tests/flutter_gallery/build/app/outputs/flutter-apk/app.apk:
[flutter_gallery__back_button_memory] [STDOUT] stdout: [        ] Installing build/app/outputs/flutter-apk/app.apk... (completed in 3.2s)
[flutter_gallery__back_button_memory] [STDOUT] stdout: [        ] Warning: Failed to install APK.
[flutter_gallery__back_button_memory] [STDOUT] stdout: [        ] Uninstalling old version...
[flutter_gallery__back_button_memory] [STDOUT] stdout: [   +1 ms] executing: /opt/s/w/ir/cache/android/platform-tools/adb -s ZY223TDZ45 uninstall io.flutter.demo.gallery
[flutter_gallery__back_button_memory] [STDOUT] stderr: [   +1 ms] Error: ADB exited with exit code 1
[flutter_gallery__back_button_memory] [STDOUT] stderr: [   +1 ms] Performing Streamed Install

Logs:
https://ci.chromium.org/p/flutter/builders/prod/Linux flutter_gallery__back_button_memory/642?

Looks like this is working now? https://ci.chromium.org/p/flutter/builders/prod/Linux flutter_gallery__back_button_memory

Closing for now, please reopen if this is still as issue.


Mouse move when locked

mouseMove doesn't work when the mouse is locked. I tried mouseMoveSmooth but that didn't work either.

Hey @Kerndog73! What do you mean by locked?

If you open a first person shooter game. The mouse stays in the centre of the screen and is invisible but the moving the mouse changes the view. This state is called "locked".
I'm making a wireless game controller that turns any phone or tablet into a wireless game controller that can be used to control a laptop.
I want to move the mouse while its in this locked state

If you get and print the mouse position, what is it? Does it change or is it fixed?

After calling moveMouse the mouse is in a new position but the game doesn't react

Maybe you need the PointerLock API? I think this could be incorporated into RobotJS API. Could help to have this. :+1:  for that feature request

@sushruth I don't think that will help here. The Pointer Lock API is for the browser, RobotJS is on the "server" or desktop. But thanks!

I'm going to close this in favor of #159. I was able to replicate and started to work on this but I don't have a resolution yet. I'll update the other ticket with more information!

I don't think that will help here. The Pointer Lock API is for the browser, RobotJS is on the "server" or desktop. But thanks!

There is, however, an equivalent for desktop -- at least on Windows: https://msdn.microsoft.com/en-us/library/windows/desktop/ms648383(v=vs.85).aspx


Add String.to_{camel,pascal,snake}_case methods

Fixed bug with extra underscores.
Rewritten the main part of the camelcase_to_underscore function.
Added new test cases.
Added to_{camel,pascal,snake}_case methods, documentation.
Replaced with new methods in places of use.

Closes godotengine/godot-proposals#719.

Why is milestone 4.1 assigned? Most other PRs are assigned 4.0, whether it's a bugfix or an enhancement.

Why is milestone 4.1 assigned? Most other PRs are assigned 4.0, whether it's a bugfix or an enhancement.

Technically, all new PRs at this point should be assigned to 4.1 or 4.x due to the roadmap freeze on Aug 3rd. We can allow smaller PRs that don't break compatibility back into 4.0 on the case by case basis. Or any PRs that have been pre-approved for 4.0.
This one seems okay for 4.0, though not critical either. If it is reviewed and approved soon, then it can be merged now.

Tests should be added for the new methods to validate their output on a wide range of inputs.

Done.

Thanks!


Timeout after nickname change

While in the reply example you are listening to the incoming messages and someone changes his nickname via the web interface the connection will time out and the bot will shortly disconnect.
It only happens in this scenario.

this should be fixed with 9b8743b68827e88faa248bc432197c570871117d


check loopbody codegen in advance to avoid locking done function body counters

when ETW rundown kicks in the functionBody may have not have bytecode generated yet before registering to utf8SourceInfo
accessing MapLoopHeaders in background thread can causes the functionBody counters locked for updating
and cause assertion when bytecode generation is done and updating bytecodeCount counter on functionBody
so check if the functionBody has done loopbody codegen in advance to not call into Map function in case
loopbody codegen is not done yet

:shipit:


Project: GUI Framework

Hi, I'm currently experimenting with writing a GUI Abstraction framework (meaning I want the OS to render buttons and stuff) and
I made some nice progress with how I want the API to look and feel and how to make it work in rust. Yesterday I finally started on how to deal with user events (e.g. mouse clicks).
My background is actually C/C++ but I recently did a few Typescript/AngularJS Frontend Projects. I really liked how Angular uses Observables for dealing with events and that's why I would really like to do something similar in Rust. I'm currently looking into and playing around with Streams and it looks promising.
I tried to keep the project described in the PR a generic GUI frameworks with the only assumption that it uses Streams for the user event handling.
I did add some example code on how such an API could be used. It's meant to give an idea and is not meant to compile. I hope this makes things a bit more clear.
Character
I do believe a new character might be useful. Someone with a background in Javascript or similar with a focus on Frontend-Applications. I feel the current character are more Backend/Low-Level and not Front-End Developers. I know this is not a common scenario with Rust missing a UI Framework but that hopefully changes in the future. Let me know and I can write something up.
Note:
I probably mixed up some stuff. Some parts are written from the view of someone Developing a UI framework in rust while other aspects come from someone using a UI framework in rust.
I'm not sure if it's worth splitting those two up.

@Fice Alan has a background in JavaScript, I imagine he would be a good fit for your purposes.

Alan looked like a Backend Developer to me.

Ruby on Rails, node.js, and used Django

Altough Frontend stuff is quite common for node.js guys so I'm happy with him if you are :)

@Fice When it comes to JS or what have you, I guess I was imagining more "full stack". Maybe we can tweak the bio to make that clear. :)

@Fice

Not sure why it didn't cross my mind, but it makes absolute sense to use a GUI Application as a Project. It's more generic and I'm sure I can bring across the points I wanted to make. Will rewrite tomorrow.

Did you still plan to rewrite this to be a specific project?
One thing I was thinking is that it might be nice to pair the new project with a status quo story that leans on it, or at least the description of one. Do you have any thoughts on that?

I actually wrote a status quo story on Saturday before life got in the way. It's now commited so you can have a read.
I did not have any time on writing the actual GUI Project , but #71 seems like a good start with only minor adjustments needed.
I do have an idea for one more status quo idea and then quite a few "How i wished it would work shiny future ones". I hope everything settles down the day after tomorrow so I can write up the project and the second story.

@Fice do you think you'll have time to revisit some of the comments and questions here?

If it helps, we could try to schedule a writing session perhaps.

I've still got various questions, but I'm inclined to merge this since @Fice seems to be busy. @Fice -- feel free to open a fresh PR with edits or amendments, or to clarify some of the questions I raised here.


Power went out & Quicksilver needed to be reinstalled

Just a note of an unusual occurance last night when the power went out for a few hours.
My iMac shut off without going through a normal shutdown and when the power came back on, and I rebooted the computer, Quicksilver was unresponsive and wouldn't launch. No message. Nada.
I reinstalled the app and now everything works fine again. Thought you might find it worth noting.

There could be several explanations for this. If it happens again, the first thing I’d try is removing ~/Library/Caches/Quicksilver. Thanks for the report.


Cache problem if I change max image size after use other one

First I used 700x700 image size then I changed it to 1000x1000 but Glide read image from before cache which is can be onlly max 700x700.I think It was really big issue because if someone wants to increase or decrease max image size it will not work which some companies do more often

This is intended behaviour with Glide, since it wouldn't be very performant if Glide checked the source file on every request. The recommended workaround for this is to simply clear the cached images for that image when it is updated. For example:
$server->deleteCache('kayaks.jpg');

You can read more about that here.

Hi @reinink
WRT refreshing the cache, are you no longer recommending adding a dummy parameter to URL?
I ask because, while that method seems to be the easiest, I'm wondering if there might be a more automated way to do this. That is, if I drop a newer version of an image into an assets folder, I'd like Glide to realize that the new image is newer than any of the manipulations stored in cache.
Is this possible with Glide alone?
Thanks.

sorry about bumping this closed thread but I think I just ran into this problem when using Glide to produce a responsive image set. IE:
<img sizes="100vw"
     srcset="/img/analog-camera-photography-vintage-1844.jpg?w=175 175w ,
       /img/analog-camera-photography-vintage-1844.jpg?w=350 350w ,
       /img/analog-camera-photography-vintage-1844.jpg?w=700 700w ,
       /img/analog-camera-photography-vintage-1844.jpg?w=1400 1400w "
     src="/analog-camera-photography-vintage-1844.jpg?w=175"
     alt="analog-camera-photography-vintage-1844" >

In a vagrant box running ubuntu 16.04 the image regardless of which one is the source candidate will be whichever was created first. That is if you are on a desktop with a ~1300px wide screen then that would be the 700px image.
So a user on a phone would get the image /img/analog-camera-photography-vintage-1844.jpg?w=350 but it would still be 700px wide. This breaks the purpose of responsive images.
If you're on windows then for some reason each image candidate would be generated (opposite of what happens in the above mentioned vagrant box)
If you add in signatures this causes other havoc as well, that being the image requested wouldn't match the signature of the saved (cached) image .. note: unless I'm horribly misunderstanding or doing something wrong .. if so pleeeeeease tell me haha I've been pulling my hair out for two days now :)
So should we delete in our controller each time a new image is requested with the same name? like you ( @reinink ) mentioned?
/** Example using a symfony style app */
public function connect(Application $app)
{
  $ctr = $app['controllers_factory'];
  $ctr->get('/{path}', [$this, 'glideImage'])
      ->assert('path', '.+')
      ->bind('glide_thumbs');
  return $ctr;
}

public function glideImage( $path, $request )
{
  // other things
  $server->deleteCache($path');
  return $server->getImageResponse($path, $request->query->all());
}

Which would negate Glides cache for someone else requesting a different dimension on a different device.
Or should we skip the Image Response part and pull say the width part of the query string, place that in the the file name then save to the cache? Similar to this sitepoint tutorial on glide

@cdowdy Have you tried hitting each individual image endpoint first to see Glide is properly loading each image size? I mean outside of just testing this via the srcset property?

hey thanks @reinink !
I did what you suggested and hit each image endpoint separately - images with widths of 200, 350, and 800 - and each image was generated at the correct size.
Then I used srcset and got the correct size for all of them.
I don't know if the size/cached size mismatch happens when there are multiple sizes requested from one image in a session (like a stampede) or if it was just a quirk I came across with this vagrant box and a windows 10 box.
It's super inconsistent. so you might be able to close this again haha! Sorry about this... I'll try to get some better testing done (between a few different environments) eventually and if I come across it again I'll post back here!
Thanks for your time!

That's good at least. You should be able to request 10 different image sizes with Glide at the exact same time and get all the properly sized images. One request should have no bearing on the other request whatsoever.
Good luck! If you do figure out what's wrong in your situation, please report back!


Update status codes for better monitoring

What changed? Why?
Updating to something more standardized.
409 means user has already claimed
500 means server or unexpected error
This will help us have better monitoring and alerts
Notes to reviewers
How has it been tested?
local testing directly to API calls

🟡 Heimdall Review Status

Requirement
Status
More Info

Reviews

🟡
0/1

Denominator calculation

Show calculation

1 if user is bot
0

1 if user is external
0

From .codeflow.yml

1

Additional review requirements

Show calculation

Max
0

0

From CODEOWNERS

0

Global minimum
0

Max

1

1

1 if commit is unverified
0

Sum
1


[Do-No-Merge][WIP] Running TestCleaner tests repeatedly

Tips

Thank you very much for contributing to Apache Hudi.
Please review https://hudi.apache.org/contributing.html before opening a pull request.

What is the purpose of the pull request
(For example: This pull request adds quick-start document.)
Brief change log
(for example:)

Modify AnnotationLocation checkstyle rule in checkstyle.xml

Verify this pull request
(Please pick either of the following options)
This pull request is a trivial rework / code cleanup without any test coverage.
(or)
This pull request is already covered by existing tests, such as (please describe tests).
(or)
This change added tests and can be verified as follows:
(example:)

Added integration tests for end-to-end.
Added HoodieClientWriteTest to verify the change.
Manually verified the change by running a job locally.

Committer checklist

[ ] Has a corresponding JIRA in PR title & commit

[ ] Commit message is descriptive of the change

[ ] CI is green

[ ] Necessary doc changes done or have another open PR

[ ] For large changes, please consider breaking it into sub-tasks under an umbrella JIRA.

Codecov Report

Merging #3155 (9ce57d1) into master (380518e) will decrease coverage by 41.07%.
The diff coverage is 0.00%.

@@             Coverage Diff              @@
##             master   #3155       +/-   ##
============================================
- Coverage     44.15%   3.07%   -41.08%
+ Complexity     4537      82     -4455
============================================
  Files           819     279      -540
  Lines         36171   10856    -25315
  Branches       3920    1107     -2813
============================================
- Hits          15970     334    -15636
+ Misses        18467   10496     -7971
+ Partials       1734      26     -1708

Flag
Coverage Δ

hudicli
?

hudiclient
0.00% <0.00%> (-16.46%)
:arrow_down:

hudicommon
?

hudiflink
?

hudihadoopmr
?

hudisparkdatasource
?

hudisync
5.38% <ø> (-46.35%)
:arrow_down:

huditimelineservice
?

hudiutilities
9.52% <ø> (-48.89%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

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

closing this for now


[Flink] Disable statistics for source if the predications are all value filters

Purpose

Linked issue: close #2185
Tests
DataTableSourceTest#testCheckAllValuePredication
FileStoreTableStatisticsTestBase#testTableFilterValueDisableStatistics
Below is the tpch test result:
Env: JM 64 core; TM 16 core，8 slot per tm; Paimon 0.5 x Flink 1.17
Table options: No partitions; The left is under configuration: source.value-filter-statistics.disable = true; The right is under configuration: source.value-filter-statistics.disable = false

query / lantency （ms）
source.value-filter-statistics.disable = true
source.value-filter-statistics.disable = false

1
7300
14745

2
13768
14179

3
9323
9490

4
5957
6091

5
13720
133810

6
3849
4664

7
25553
31348

8
22970
23417

9
26187
26653

10
10630
33056

11
9491
9062

12
7741
40244

13
8082
8291

14
5829
9153

15
6515
6922

16
8650
8515

17
13995
38186

18
15374
17667

19
7780
3305

20
11588
11195

21
14273
15423

22
7920
8090

As we can see, most of the queries are more efficient with source.value-filter-statistics.disable = true than the opposite, especially the query 5, with the help of statistics disabled, the query can be run into 2 mins and never timeout.

@JingsongLi We add some benchmark for this issue, please help to review it when you're free, thanks

Thanks @schnappi17 and @FangYongs for the benchmark, can you also test Flink config table.optimizer.source.report-statistics-enabled to false?

Thanks @schnappi17 and @FangYongs for the benchmark, can you also test Flink config table.optimizer.source.report-statistics-enabled to false?

Sure, we can do more test on this.


Use babylon estree and ranges

Fixes #440

we might be able to remove some other stuff in babylon-to-estree

Wasn't sure, a lot of other stuff is unrelated to estree, tokens, types. But yeah maybe there is some more.

maybe
like https://github.com/babel/babel-eslint/blob/master/babylon-to-espree/index.js#L32-L33
https://github.com/babel/babel-eslint/blob/master/babylon-to-espree/convertComments.js#L14

can land when it passes, can iterate

lol nvm? we can land what you have

Seems we don't add ranges to comments🤔

would be nice to have a perf check for this change (or this vs regular parser)


Validate multicluster + ipsec + tunnel

We should figure out some automation around testing this combination and validate the functionality against a test suite for encryption in conjunction with tunneling mode, and ensure this works across clusters.

Two broad options:

Refactor existing tests to make them cluster-oblivious. This could be a lot of work but would allow high coverage for existing tests. Increases development burden for writing and maintaining tests.
Add a new set of multicluster-aware tests. Test the pieces that are different in multicluster:

Connectivity
LB (across clusters)
Policy (identity is different across clusters, including cluster labels)
Visibility (identities / labels show up as expected in monitor output)

I think @seanmwinn is or was recently testing this?

I believe so, but manually. We will need to integrate this into some form of regular automated tests to prevent regressions.

One potential direction: Make use of kind per #11157 instructions.


Feature ideas

Hi bkis,
I still like Sprudel and following your changes! Please find a list with feature ideas for the future:

Possibility to delete a comment
If "Admin Link" option is used, it should not be possible to update an existing entry by creating a new entry with the same name.
If "Admin Link" option is used, provide option to lock a poll ("read only" view).
Receive email after new comment
I intent to use Sprudel for a "online staff roster" for my sports club, so that people can register for certain tasks/events. For this it would be good to add the possibility to store contact information (e.g. email/phone number/...) with the entry. For sure this information shall not be visible in the public view.
Possibility to limit the number of "Yes" entries for all/each options e.g. if maximum X people are needed for certain event/task.

As you know from the past, I'm willing to help to improve Sprudel. But before start coding/creating pull requests, I want to double check with you, if you are interested in (some of) the above features. Feel free to split the feature ideas in separate issue for individual follow up.

Hi there,
thanks for these suggestions! Let me quickly comment with my first thoughts...

* Possibility to delete a comment

Would be a nice feature and easy to implement, I agree. It's not very critical, but nice.

* If "Admin Link" option is used, it should not be possible to update an existing entry by creating a new entry with the same name.

You are right. I got used to use this as a "feature" ;) But it shouldn't work this way, no. BTW here you mean if "Admin Link" option is not used, right?

* If "Admin Link" option is used, provide option to lock a poll ("read only" view).

Nice, I didn't think of that, yet. Why not!

* Receive email after new comment

... or after new poll entries! That would be possible. We could offer a config option for using the php mail command (local server settings) or a remote SMTP with credentials. But that would be some work to do and it has to be designed rather carefully.

* I intent to use Sprudel for a "online staff roster" for my sports club, so that people can register for certain tasks/events. For this it would be good to add the possibility to store contact information (e.g. email/phone number/...) with the entry. For sure this information shall not be visible in the public view.

You mean so the admin could collect the contact information? Hm... that would expand the functionality in a very new direction. I don't know if it's too much of a corner case to become a standard feature. But yeah, it could be useful. I think it's important to not confuse users who don't need this feature, though. It should be as easy and self-explanatory as possible to open and use a poll. A feature like this would have to come with a lot of options/settings to read through, which raises the barrier to use the tool. So it would have to be kind of "hidden", you know?

* Possibility to limit the number of "Yes" entries for all/each options e.g. if maximum X people are needed for certain event/task.

I think in most (if not almost all) of the use cases this is not what you want. In my experience when polling a large group of people, you'll always have last minute cancellers and would be happy to have more people who said "yes". I don't say the cases where you could use this feature don't exist at all, though. This and some of the other features could be in a collapsible "Advanced features" component on the start page.

As you know from the past, I'm willing to help to improve Sprudel. But before start coding/creating pull requests, I want to double check with you, if you are interested in (some of) the above features. Feel free to split the feature ideas in separate issue for individual follow up.

Thank you for your ideas and for reaching out to me before putting work into this! I would love to see this project grow and I am certain I will add to it and improve it in the future when time comes. But there are two things I want to say: Firstly, I have a full-time job and a family, so my spare time is limited. I won't be very quick working on any of those things. If you like to implement stuff (and write own issues for features) you are very welcome! But it's possible that I need some time to review your PRs, for the abovementioned reasons :)
Secondly, php is not my first love. I like php for it's availability on almost any web server system, though. That's why I chose php for this. If it was only about what I like to work with, I would have built a Java (or maybe Python) backend and a nice JS SPA for a frontend. Not that it's php and I didn't want to spend time learning any real php frameworks, the code of Sprudel is very messy and far from well-designed!  You may have recognized I tried to separate stuff better, but it's not good enough, yet.
What I want to say is: As long as this is a small application with only few features, this messy dump of code is still manageble enough. But if we want to add more and more features (which would be cool), maybe it would be clever to do a complete re-design and re-write of the whole thing, first. It should at least have completely separated frontend and backend. So, a server API and a client app that communicates with this API instead of ugly server php composing HTML pages and JS snippets. This is far from state of the art :disappointed:

Hi again,
many thanks for your detailed answer!
Especially due to your last paragraph, I had a look on existing PHP frameworks. During the last days I played around with CakePHP and "ported" sprudel to that framework, see sprudel-ng. I really enjoyed your lightweight code approach, so I hope that using an additional framework isn't an overkill for this little project!
I don't want to "steal" your baby, so I hope, that you might have a look in future and join the port. Php is not my first love as well :), so I'm pretty sure, that my port needs some polish as well (or could use some CakePHP features in much more elegant way).
Currently I named the port sprudel-ng ("next generation" ;)), but if you wish, that I should change the name/brand, just let me know!

Hi,
no worries! I'm happy you want to work on this!
I am currently on my holidays so i'll look at your new approach in two weeks.
My first thought on the name sprudel-ng was that "ng" is much used as a kind of namespace by Angular.js. It makes it look like an Angular project.
At first I wanted to call Sprudel "Poodle" (like the dog). Maybe you like the idea ;)
Cheers!
Am 10. September 2020 16:34:25 MESZ schrieb El Thoro<EMAIL_ADDRESS>
Hi again,
many thanks for your detailed answer!
Especially due to your last paragraph, I had a look on existing PHP
frameworks. During the last days I played around with CakePHP and
"ported" sprudel to that framework, see
sprudel-ng. I really enjoyed
your lightweight code approach, so I hope that using an additional
framework isn't an overkill for this little project!
I don't want to "steal" your baby, so I hope, that you might have a
look in future and join the port. Php is not my first love as well :),
so I'm pretty sure, that my port needs some polish as well (or could
use some CakePHP features in much more elegant way).
Currently I named the port sprudel-ng ("next generation" ;)), but if
you wish, that I should change the name/brand, just let me know!
--
You are receiving this because you commented.
Reply to this email directly or view it on GitHub:
https://github.com/bkis/sprudel/issues/10#issuecomment-690330530

@ElTh0r0 Do you have a working live instance running anywhere?

Hi @bkis
yes, I have. Do you need help with the setup or do you just want to have a look?
If you want to have a look: sprudel-ng.bplaced.net
This instance might be deleted in few weeks and some features are disabled (send email after new entry/comment; add admin / change admin password). The instance is using the default admin user/password as mentioned in the readme.

I was just curious to see if you already have a runnable version. Glad to see you do :+1:
When time comes I'll look into opportunities to contribute, as I think it makes more sense to follow this new path (using the framework) than to keep working on the DIY mess this little project was before :wink:
If you indent to make this more than a hobby project for yourself (and possibly attract real world users), I'd still advise you to change the project's name to something else, though. It's not only that "-ng" smells like Angular (which it doesn't have anything to do with), but it's also quite unwieldy, doesn't catch the attention, you now? People will be like "What's the name again? Sprud- something...".
BTW I hereby give this project idea to you, with the opportunity for me to contribute later. Your implementation (if it works as expected) makes the original "sprudel" very, very redundant - if not superfluous :shrug:
Just tell me when you feel confident with it, I'll add a hint to you project to the original README. Cheers!

Many thanks for your comments @bkis!
I decided to rename the project to PREFERendum since I think, that the wordplay fits for a polling app. I appreciate if you could add a link to this repo in your README file.
I really hope to hear from you in future and that you are joining the ported app!

You will, I promise.
I added said hint with a link to the Sprudel README and will archive the repo at some point in the future.
BTW I think it was a good decision rename the project. Good choice of a name, too. I am looking forward to see the project evolve!

End of maintenance. See README!


Code scroll doesn't work

Hello,
The code scroll doesn't work on all challenges, I have to use page up and down to work through the code plus the undo(not the reset) doesn't work.
Timothy J.
Challenge http://www.freecodecamp.com/challenges/waypoint-use-an-id-attribute-to-style-an-element has an issue. Please describe how to reproduce it, and include links to screenshots if possible.

Hi @Toytradeadman what browser are you using. Challenges currently wont function on IE, EDGE and Safari

@benmcmahon100  I'm using Google Chrome

This is quite odd you should get a scroll bar just to the left of the code editor
Can you also post a screenshot for me?

it worked before they did all these updates to freecodecamp the only thing that didn't was the undo. it does have the scroll bar but it doesn't move

Then you should be able to see all the code! A screenshot would be really useful so that i can figure out what's going on

I get the picture of a cell phone with a circle showing it can't be move d

in full screen mode I almost see all the code

So it's the result window that's broken? The code there looks fine

correct

So the code scroll works perfectly and the preview does not
Okay so is it getting hidden behind the footer?
Or is the html overflowing past the end of the phone?

the code scroll is not working. If I hit the arrows or click and hold the scroll bar it doesn't move I don't why a picture of a cell phone is there when I do that.

Oooooooooooooh. That's not good... Thanks so much for finding this I know whats wrong now! :p

welcome


Css styling thread Annihilate

You do
Terminate
Free
but thread may be freed after a pause, not just after Terminate. Terminate only sets the flag.

Oh you're right. I'll fix it. Thanks!


/createRoom is not clear about what should happen if you specify yourself as an invitee

Link to problem area: https://spec.matrix.org/v1.1/client-server-api/#post_matrixclientv3createroom
Issue
Current Synapse behaviour is to raise a 403 much like it would do if you tried to invite yourself (this is specced behaviour for /invite).
My suggestion would be to just specify that retroactively so that other homeservers can benefit from having the clarity and to prevent divergence in behaviour across other implementations.
Of course, if it'd be preferable to change, then I don't see why it couldn't be done — it just seems like sending a 403 as you would for an /invite is sensible enough.
Related: https://github.com/matrix-org/synapse/issues/10740

As I said over on the synapse issue; Why is this a 403? Shouldn't it be a 406 (Not Acceptable) or 422 (Unprocessable Entity)?

As I said over on the synapse issue; Why is this a 403? Shouldn't it be a 406 (Not Acceptable) or 422 (Unprocessable Entity)?

That might be a separate question worthy of a separate thread (the 403 behaviour is specced for /invite).
I suspect it's because some other violations here (e.g. join → invite for another user; that'd essentially kick them) are 'you're not allowed to change that person's membership' a.k.a. 'you're not allowed to send that state event' — I think it's an auth rule failure?
In a way, the precise code doesn't matter and it'd be nice not to have to get into a big debate about it, however having agreement on what the code is for all clients and servers is at least somewhat useful to avoid differences between e.g. Synapse and Conduit.


Module not found Error showing

Hi Everyone,
I just stumbled upon Flet project and can't wait to get going with it.
I installed Python 3.11.0 and Flet 0.3.2 (see screenshot)
But I still could'nt run the first "Hello World" demo code I found in the flet.dev documentation.
The Error in the terminal:
AttributeError: partially initialized module 'flet' has no attribute 'page' (most likely due to a circular import)
I also run:
pip install flet --upgrade
But still same issue.
If anyone has an idea, I'd greatly appreciate it!
Thank you so much.

Thanks for trying out Flet!

Please install the latest Flet 0.10.3.

Most probably your program is called flet.py - hence the error.

Yeah! Thanks alot


Create JIRA client bundle

Ticket 104424786

@rjkip @DRvanR

@rjkip :poultry_leg:

Seem to be missing the actual configuration, the kernel update and the parameters.yml.dist update?

@DRvanR @rjkip

Bar the failed test and missing scalar config null test, :+1: from me.

Agree witth @rjkip

Fixed the test.
Null already fails the cannotBeEmpty() test, so it is not part of any custom validation.


Add namespace quota per zone policy

Checklist

Adds policy check-namespace-quota with 2 rules

Check that kubectl create ns $ doesn't exceed allocated count
Check that oc new-project $ doesn't exceed allocated count

Each organization can have defined overrides. Each override is stored as a config map in the appuio-cloud namespace.
For example: kubectl -n appuio-cloud create cm override-foo --from-literal=namespaceQuota=4

[x] Update the documentation.
[x] Categorize the PR by setting a good title and adding one of the labels:
bug, enhancement, documentation, change, breaking, dependency
as they show up in the changelog

Documented a how-to page in https://github.com/appuio/appuio-io-docs/pull/68

@corvus-ch Currently, one has to manually create a configmap to set overrides for specific organizations. Should the component be able to manage those overrides via parameters? e.g.
namespaceQuotaOverrides:
  orgName: 50

where each Key/value pair creates a config map in appuio-cloud ns.

[…] Should the component be able to manage those overrides via parameters? […]

Yes, I would prefer to have it that way. The documentation should suggest to do this on a zone level (so a customer can create 42 ns on one zone but only the default 25 on another).

I have not really an idea why the PR checks fail:
 docker run --rm -u "$(id -u)" -w /appuio-cloud -v "${PWD}:/appuio-cloud" projectsyn/commodore:latest component compile . --search-paths ./dependencies --search-paths . -f tests/defaults.yml
docker: Error response from daemon: Head "https://registry-1.docker.io/v2/projectsyn/commodore/manifests/latest": unknown: unable to scan repos service account row from query: context deadline exceeded.

The bitnami/jsonnet:latest image is also failling. Maybe we hit Docker rate limit?

Should the component be able to manage those overrides via parameters?

We should think about it, if we really should manage this via Commodore. It brings us a disconnect between self-service and managing stuff with Commodore. For the time being I think we can go this way, but on the long run we should manage such specifics via APPUiO Cloud mechanisms. We should track this in a tech debt issue.

@tobru

It brings us a disconnect between self-service and managing stuff with Commodore.

AFAIK this override feature is left to administrators only, there shouldn't be self-service for users. Or did I understood it wrong?
What other mechanism did you have in mind?

What other mechanism did you have in mind?

Configuration on an organizational level (e.g. annotation on Organization), only available to admins. But as written, not something we have to do now but something we need to track and not forget about.
The reason: The source of truth about organizations isn't in Commodore and I see a disconnect of where we configure such things here.

I've created internal issue APUB-177 for this feature.


cumulatively, evaginate

This pull request was generated by the 'mq' tool
[test]
flake rate: 0.4
logical conflict every: 5
sleep for: 300s
[pullrequest]
requests per hour: 30
deps=[c,e]

/trunk merge


Port config fix to 2.2

@ajcvickers

There's no reload for user secrets in 2.2 so those tests had to be removed

Original PR: https://github.com/aspnet/Extensions/pull/1202
Issue: https://github.com/aspnet/Extensions/issues/1189
Description
There is a race condition that can be hit if a file is being updated while the configuration system is refreshing configuration from that file.
Customer Impact
Customer reported. Application crashes with exception.
Regression?
No; same behavior in 2.1. At this time, we're bring for 2.2 only.
Risk
Low. Configuration system is relatively small and code change is simple.

For my own curiosity, how often does "Shiproom" meet to discuss such proposed changes?

@martincostello Usually twice per week.

Thanks - I was just wondering as I was reviewing our production logs from over the weekend and we had a handful of the exception occur, so I just wanted to what the status was.

This fix is incomplete without an update to eng/PatchConfig.props.

Approved for 2.2.4
Checkin after branding update @dougbu

@dougbu what am I supposed to do to patchconfig.props?

I think Microsoft.Extensions.Configuration.KeyPerFile and Microsoft.Extensions.Configuration.EnvironmentVariables need adding to the 2.2.4 node.

Alright, @dougbu updated patchconfig

Hrm build checks are red after updating the patch config...  any ideas @dougbu ?
WARNING: 0 error(s) and 2 warning(s) found with package Microsoft.Extensions.Configuration.EnvironmentVariables (2.2.4-servicing-19164-12)
WARNING: ASSEMBLY_FILE_VERSION_MISMATCH @ lib/netstandard2.0/Microsoft.Extensions.Configuration.EnvironmentVariables.dll: The managed assembly file version '<IP_ADDRESS>73' does not match the package version '2.2.4-servicing-19164-12' for package Microsoft.Extensions.Configuration.EnvironmentVariables.
WARNING: ASSEMBLY_VERSION_MISMATCH @ lib/netstandard2.0/Microsoft.Extensions.Configuration.EnvironmentVariables.dll: The managed assembly version '<IP_ADDRESS>' does not match the package version '<IP_ADDRESS>' for package Microsoft.Extensions.Configuration.EnvironmentVariables.
INFO: Analyzing Microsoft.Extensions.Configuration.KeyPerFile (2.2.4-servicing-19164-12)
WARNING: 0 error(s) and 2 warning(s) found with package Microsoft.Extensions.Configuration.KeyPerFile (2.2.4-servicing-19164-12)
WARNING: ASSEMBLY_FILE_VERSION_MISMATCH @ lib/netstandard2.0/Microsoft.Extensions.Configuration.KeyPerFile.dll: The managed assembly file version '<IP_ADDRESS>73' does not match the package version '2.2.4-servicing-19164-12' for package Microsoft.Extensions.Configuration.KeyPerFile.
WARNING: ASSEMBLY_VERSION_MISMATCH @ lib/netstandard2.0/Microsoft.Extensions.Configuration.KeyPerFile.dll: The managed assembly version '<IP_ADDRESS>' does not match the package version '<IP_ADDRESS>' for package Microsoft.Extensions.Configuration.KeyPerFile.
WARNING: Unnecessary exclusion in Microsoft.Extensions.Logging.Testing
Issue: BUILD_ITEMS_FRAMEWORK
Instance: *

Showing JSON for ignore content:
{
dotnet.exe failed with exit code: 1
  "Microsoft.Extensions.Configuration.EnvironmentVariables": {
At D:\a\.dotnet\buildtools\korebuild\2.2.1-build-20190104.5\scripts\common.psm1:11 char:9
+         throw "$cmdName failed with exit code: $exitCode"
+         ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    + CategoryInfo          : OperationStopped: (dotnet.exe failed with exit code: 1:String) [], RuntimeException
    + FullyQualifiedErrorId : dotnet.exe failed with exit code: 1

    "Exclusions": {
      "ASSEMBLY_FILE_VERSION_MISMATCH": {
        "lib/netstandard2.0/Microsoft.Extensions.Configuration.EnvironmentVariables.dll": "Enter justification"
      },
      "ASSEMBLY_VERSION_MISMATCH": {
        "lib/netstandard2.0/Microsoft.Extensions.Configuration.EnvironmentVariables.dll": "Enter justification"
      }
    }
  },
  "Microsoft.Extensions.Configuration.KeyPerFile": {
    "Exclusions": {
      "ASSEMBLY_FILE_VERSION_MISMATCH": {
        "lib/netstandard2.0/Microsoft.Extensions.Configuration.KeyPerFile.dll": "Enter justification"
      },
      "ASSEMBLY_VERSION_MISMATCH": {
        "lib/netstandard2.0/Microsoft.Extensions.Configuration.KeyPerFile.dll": "Enter justification"
      }
    }
  }
}

Oh well, this isn't quite ready to go

@HaoK will get back to you on the version mismatches once I get home. Things aren't working as they should…

@HaoK didn't find similar changes elsewhere,. But, I'm guessing the package verifier is correct.
Add the following to the Default/packages array in eng/NuGetPackageVerifier.json:
  "Microsoft.Extensions.Configuration.EnvironmentVariables": {
    "Exclusions": {
      "ASSEMBLY_FILE_VERSION_MISMATCH": {
        "lib/netstandard2.0/Microsoft.Extensions.Configuration.EnvironmentVariables.dll": "Enter justification"
      },
      "ASSEMBLY_VERSION_MISMATCH": {
        "lib/netstandard2.0/Microsoft.Extensions.Configuration.EnvironmentVariables.dll": "Enter justification"
      }
    }
  },
  "Microsoft.Extensions.Configuration.KeyPerFile": {
    "Exclusions": {
      "ASSEMBLY_FILE_VERSION_MISMATCH": {
        "lib/netstandard2.0/Microsoft.Extensions.Configuration.KeyPerFile.dll": "Enter justification"
      },
      "ASSEMBLY_VERSION_MISMATCH": {
        "lib/netstandard2.0/Microsoft.Extensions.Configuration.KeyPerFile.dll": "Enter justification"
      }
    }
  }

Cool that was it, green now @dougbu


Installing 6.1.0 via composer fails

Hi, thanks a lot for this tool, it is solving our problem very neatly!
Up to v6.0.0, I had been using command $ composer global require gordalina/cachetool to make cachetool available for my webserver user.
Since v6.1.0, the installation fails with:
$ composer global require gordalina/cachetool==6.1.0
Changed current directory to /home/vagrant/.config/composer
./composer.json has been updated
Loading composer repositories with package information
Updating dependencies (including require-dev)
Your requirements could not be resolved to an installable set of packages.

  Problem 1
    - Installation request for gordalina/cachetool =6.1.0 -> satisfiable by gordalina/cachetool[6.1.0].
    - gordalina/cachetool 6.1.0 requires consolidation/self-update 1.x-dev -> satisfiable by consolidation/self-update[1.x-dev] but these conflict with your requirements or minimum-stability.

Installation failed, reverting ./composer.json to its original content.

Fix released in v6.1.1

Fix released in v6.1.1


support repeat loops with constant repeat counts outside of constant functions

I changed the sat/counters test to use repeat instead of for, and it seems to work. I noticed that the test ran much slower (about ~0.2s vs. ~1.3s), so I imagine I'm missing something important here.
Feedback is greatly appreciated!

I noticed that the test ran much slower (about ~0.2s vs. ~1.3s), so I imagine I'm missing something important here.

That's fine. The issue is that in the for-loop case the values of i are resolved by the front-end when unrolling the loop, whereas in the repeat case the front-end doesn't know what the values of i are for the individual loop iterations. Therefore the front-end generates a much more complicated circuit and is takes a second or so to optimize to what the for-loop case generated right away.
The diff looks good to me. Merging it now.


Prevent @enum from begin deleted

Is your feature request related to a problem? Please describe.
When using the table approach to ENUMs (as described in https://www.graphile.org/postgraphile/enums/) the plugin will remove the key out because recognized as a foreign key. If a column (or table) is marked with @enum, it should be skipped.
Describe the solution you'd like
If a column (or table) is marked with @enum, it should be skipped.

Any news? I've seen you did some changes on dev :)

Any news? I've seen you did some changes on dev :)

Hi @vabatta. I think I have a solution but just haven't had time to test it. I'll try to get to it today or tomorrow. Please feel free to try it from the development branch if you want.

@vabatta Please let me know if that fixes it for you, and I'll close this issue. 👍🏼

@jarvisuser90 Nope. The issue here is that @enum is marked on the referenced table, not the actual table that goes through the hook as GraphQLObjectType. Also, note that @enum is a smart comment rather than a smart tag, as it runs in an earlier stage introspection, you won't be going through enum tables with that hook. This means the condition https://github.com/jarvisuser90/postgraphile-remove-foreign-key-fields-plugin/blob/main/src/index.ts#L28-L31 is unnecessary.
From the docs

IMPORTANT: this is one of the few places where a smart tag and a smart comment is not equivalent, this must be achieved with a smart comment since smart tags have not yet loaded at this stage in introspection.

The actual fix I can see straight away (which I have locally tested and works :) ) is to check if the referenced table in the keyAttribute is marked as @enum instead of the actual constraint.
From (https://github.com/jarvisuser90/postgraphile-remove-foreign-key-fields-plugin/blob/main/src/index.ts#L76-L79)
// Check if the foreign key is marked as an enum reference.
const isEnum = attribute.comment
  ? attribute.comment.includes("@enum")
  : false;

To
const { foreignClass } = constraint;
// Check if the foreign key is marked as an enum reference.
const isEnum =
  foreignClass && foreignClass.comment
  ? foreignClass.comment.includes("@enum")
  : false;

As a final note, we might be missing other use cases where @enum could be excluded; I haven't had a deeper look into it, might be worth to have a look with a proper use cases.

@vabatta ah, I see. I can make the changes or if you'd like, feel free to submit a pull request.

@jarvisuser90 I'll fork, do the changes and some more testing. A PR will come afterwards :)

@jarvisuser90 I'll fork, do the changes and some more testing. A PR will come afterwards :)

@vabatta PR done and merged. A new NPM package has been pushed.

@jarvisuser90 Nice, thanks!


feat: scheduled tests for installer Action

Signed-off-by: laurentsimon<EMAIL_ADDRESS>This scheduled test lists all the versions of the verifier and installs its action. It dynamically lists the versions and runs the installer as a local Action.
To complement this, we also run the latest version of Action as a remote Action in https://github.com/slsa-framework/example-package/blob/main/.github/workflows/e2e.installer-action.yml

BTW, check-dist wasn't added as a required pre-submit so the installer action's distribution got out of sync. This PR fixes it but we still didn't require check-dist to pass on PRs so I added it as a required check.


Mic 4670/process shard metadata

Mic 4670/process shard metadata
Processes shard metadata files in to one metadata proportions file to pass to pseudopeople for noise level validation.

Category: Post-processing
JIRA issue: MIC-4670
Research reference:

Changes and notes
-adds feature to the end of post-processing which aggregate and calculates metadata proportions for all dataset, location, and year combinations.
Verification and Testing
Successfully ran make results and metadata file was written with correct outputs.

One thing I think needs to be in here that is not is aggregate for all locations. This seems a bit tricky because with the sample data we would already have just a US location. Not sure what your thoughts on this are @rmudambi

One thing I think needs to be in here that is not is aggregate for all locations. This seems a bit tricky because with the sample data we would already have just a US location. Should the full data (at least for each year) be calculated in here as well? Not sure what your thoughts on this are @rmudambi

Since it is possible for a user to specify "all years" we do need to have that aggregated value as well.

Sorry, I accidentally approved this thinking it was the other PR


Add components nuxtServerInit hooks on components matched by the route

This hook will be executed when the server is loading the application
and will not be executed when routing in the browser.

Types of changes

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

Description

This hook is required to perform actions in the component when the application is first loaded and does not require a call when routing on the client side
Checklist:

[x] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly. (PR: #)
[ ] I have added tests to cover my changes (if not applicable, please state why)
[ ] All new and existing tests are passing.

Codecov Report

Merging #4485 into dev will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##              dev    #4485   +/-   ##
=======================================
  Coverage   91.36%   91.36%
=======================================
  Files          60       60
  Lines        2108     2108
  Branches      508      508
=======================================
  Hits         1926     1926
  Misses        167      167
  Partials       15       15

Continue to review full report at Codecov.

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

Hey @Pavel-Mishalov 👋
Thanks for the PR. I'd probably wait for https://github.com/vuejs/vue/pull/9017 to get merged into Vue so we have that functionality out of the box.
For now I'd go with:

nuxtServerInit in the store
asyncData + process.server for page components

Thank @manniL for considering my proposal, I didn’t immediately check option 2 for the idea it should work out accordingly

@Atinux @pi0 also agree with @manniL here. Should we close?
@Pavel-Mishalov the problem is that we're likely introducing similar enhancements based on the upcoming ssrPrefetch option, so introducing an isolated, stored-specific API for this right now might create a conflict in the near future.

@manniL @galvez Thanks for your insights.
@Pavel-Mishalov You contribution is more than appreciated by us. To prevent doing the same thing in a different way I suggest waiting for Vue.js ssrPrefetch.


libXCTest.so on Linux has no RUNPATH

When building on Linux, the libXCTest.so which is installed in /usr/lib/swift/linux/ ends up with no RUNPATHs whatsoever, neither $ORIGIN nor the toolchain's lib install directory. This has the effect of enforcing an entirely artificial link-ordering dependency when building Linux xctest executables due to its dependency on libswift_RegexParser. Control over link ordering is not generally available via SwiftPM, and is in fact actively discouraged by the default usage of swift-autolink-extract. Add to this that there is no attempt even by XCTest itself to guarantee its own build ordering, having relied for some time now on accident and luck to hide the problem, and its seem fairly obvious that the ordering dependency is both unintentional and undesirable.
I'm not clear enough on how the Linux toolchain is set up to understand whether the fix is to add a $ORIGIN rpath, to remove the -no-toolchain-stdlib-rpath flag from the XCTest CMake build (both of which successfully solved the linkage problem when I tried them), or some other approach.
This issue was revealed by apple/swift#64312; specific details are given by https://github.com/apple/swift/pull/64312#issuecomment-1466935354 and https://github.com/apple/swift/pull/64312#issuecomment-1467017450.

I argued about adding $ORIGIN to this library with @compnerd a couple years ago. While he was open to it, he felt it shouldn't be done here, but in the Swift build script. Removing -no-toolchain-stdlib-rpath will only help you with local builds and won't work with the official toolchain, as it adds an absolute path from the build host.
@compnerd, just pick one place to add it for ELF platforms like linux, either build-script-impl or here, and let's finally get this rpath in. I'm not sure if this is precisely the fix for @gwynne's link issues, but we should've done it before anyway.

@buttaface I think that build-script is the right place for this.  We can do it in build-script-impl or even better would be build-script as that has been where things are migrating (albeit slowly).

@gwynne, if you submit a pull adding that, we can finally get it in.


Transporter transfer failed. When uploading to iTunes connect

This seems to be related to this issue: https://github.com/fastlane/fastlane/issues/6151#issuecomment-249993354
When I run fastlane appStore which is my appStore lane - once the compiling is done and at the point of uploading to iTunes connect, I get the following error:

Return status of iTunes Transporter was 1:
The call to the iTMSTransporter completed with a non-zero exit status: 1. This indicates a failure.
Could not download/upload from iTunes Connect! It's probably related to your password or your internet connection.

I then tried: FASTLANE_ITUNES_TRANSPORTER_USE_SHELL_SCRIPT=1 fastlane appStore
And it too gave the same error.
Using the latest version of Fastlane and Xcode 8
Any suggestions?
Thanks

Sorry about that. Here we go:
`🚫 fastlane environment 🚫
Stack

Key
Value

OS
10.12.2

Ruby
2.4.0

Bundler?
false

Git
git version 2.10.1 (Apple Git-78)

Installation Source
/usr/local/bin/fastlane

Host
Mac OS X 10.12.2 (16C67)

Ruby Lib Dir
/usr/local/Cellar/ruby/2.4.0/lib

OpenSSL Version
OpenSSL 1.0.2j  26 Sep 2016

Is contained
false

Xcode Path
/Applications/Xcode.app/Contents/Developer/

Xcode Version
8.2.1

System Locale

Error

No Locale with UTF8 found 🚫

fastlane files:
`./fastlane/Fastfile`
default_platform :ios

platform :ios do
  before_all do
    increment_build_number
    sh("bash ./scripts/bidorbuy_web_fonts.sh")
  end

  desc "Submits a new Beta Build to HockeyApp"
  lane :beta do
    match(type: "adhoc")
	 gym(
		  scheme: "MyApp Ad-Hoc",
		  configuration: "Demo -Ad-Hoc",
		  clean: true,
		  include_bitcode: true,
		  include_symbols: true,
		  workspace: MyApp.xcworkspace",
		  output_directory: "./builds/",
		  output_name: MyApp.ipa",
		)

		hockey(
			  api_token: "OUR_TOKEN",
			  notes_type: "1",
			  notes: changelog_from_git_commits
    )
  end

  desc "Deploys a new version to the App Store"
  lane :deploy do
     match(type: "appstore")
	 gym(
		  scheme: "MyApp Release",
		  configuration: "Release",
		  clean: true,
		  include_bitcode: true,
		  workspace: "MyApp.xcworkspace",
		  output_directory: "./builds/",
		  output_name: "MyApp.ipa",
		)
    deliver(force: true)
  end

end```
</details>

<details><summary>`./fastlane/Appfile`</summary>

```ruby
app_identifier "com.MyApp.MyApp" # The bundle identifier of your app
apple_id<EMAIL_ADDRESS># Your Apple email address

team_id "[[DEV_PORTAL_TEAM_ID]]"  # Developer Portal Team ID

# you can even provide different app identifiers, Apple IDs and team names per lane:
# More information: https://github.com/fastlane/fastlane/blob/master/fastlane/docs/Appfile.md

fastlane gems

Gem
Version
Update-Status

fastlane
2.12.0
✅ Up-To-Date

Loaded fastlane plugins:
No plugins Loaded
Loaded gems

Gem
Version

did_you_mean
1.1.0

slack-notifier
1.5.1

i18n
0.7.0

json
1.8.6

thread_safe
0.3.5

tzinfo
1.2.2

minitest
5.10.1

activesupport
<IP_ADDRESS>

CFPropertyList
2.3.4

claide
1.0.1

colored
1.2

nanaimo
0.2.3

xcodeproj
1.4.2

rouge
1.11.1

xcpretty
0.2.4

terminal-notifier
1.7.1

unicode-display_width
1.1.3

terminal-table
1.7.3

plist
3.2.0

public_suffix
2.0.5

addressable
2.5.0

multipart-post
2.0.0

word_wrap
1.0.0

babosa
1.0.2

highline
1.7.8

commander
4.4.3

excon
0.54.0

faraday
0.11.0

unf_ext
<IP_ADDRESS>

unf
0.1.4

domain_name
0.5.20161129

http-cookie
1.0.3

faraday-cookie_jar
0.0.6

fastimage
2.0.1

gh_inspector
1.0.3

uber
0.0.15

representable
2.3.0

retriable
2.1.0

mime-types-data
3.2016.0521

mime-types
3.1

hurley
0.2

little-plugger
1.1.4

multi_json
1.12.1

logging
2.1.0

jwt
1.5.6

memoist
0.15.0

os
0.9.6

signet
0.7.3

googleauth
0.5.1

httpclient
2.8.3

google-api-client
0.9.24

mini_magick
4.5.1

multi_xml
0.6.0

rubyzip
1.2.0

security
0.1.3

xcpretty-travis-formatter
0.0.4

dotenv
2.1.2

bundler
1.14.3

faraday_middleware
<IP_ADDRESS>

io-console
0.4.6

openssl
2.0.2

generated on: 2017-01-27
`

Could you share the complete output when running fastlane?

@KrauseFx Thanks for the response. I think there might be an issue with the network at my work place. Since I was able to successfully deploy the app to iTunes Connect without using anything more than the simple fast lane command- using my home network. I'll revert to the I.T department next week.
Closing this as it isnt an issue with fastlane.


[now-next] Update routes for new check: true behavior

As discussed this moves the handle: filesystem usage to the right location now that we have check: true for the rewrites

The 07-custom-routes and 08-custom-routes-catchall integration tests should cover this right?

@dav-is I'm not sure we should add the redirect for *.html* since we have to account for them adding HTML files in their public or static directories. If we decide this is a behavior we want to expose for those directories as well we can do it in a follow up PR

Codecov Report

Merging #3383 into master will decrease coverage by 1.13%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #3383      +/-   ##
==========================================
- Coverage   13.87%   12.73%   -1.14%
==========================================
  Files         267      267
  Lines       10445    10299     -146
  Branches     1256     1319      +63
==========================================
- Hits         1449     1312     -137
+ Misses       8926     8893      -33
- Partials       70       94      +24

Impacted Files
Coverage Δ

src/util/get-update-command.ts
4.87% <0%> (-6.89%)
:arrow_down:

src/util/dev/server.ts
58.84% <0%> (-5.78%)
:arrow_down:

src/util/dev/builder.ts
66.88% <0%> (-5.75%)
:arrow_down:

src/util/dev/yarn-installer.ts
83.72% <0%> (-5.37%)
:arrow_down:

Continue to review full report at Codecov.

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


Implement logs and stats from langfuse

⚠ Please check that this feature request hasn't been suggested before.

[X] I searched previous Ideas in Discussions didn't find any similar feature requests.
[X] I searched previous Issues didn't find any similar feature requests.

🔖 Feature description
A user should be able to see logs and and stats from langfuse inside superoagent. Thare should be a button for each log item to deep link to langfuse.
Get trace now includes latency, cost, and the path of the trace in the langfuse ui
https://api.reference.langfuse.com/#get-/api/public/traces

New metrics/daily endpoint
Changelog: https://langfuse.com/changelog/2024-02-19-metrics-api-endpoint
Reference: https://api.reference.langfuse.com/#get-/api/public/metrics/daily

Acknowledgements

[X] My issue title is concise and descriptive.
[X] I have searched the existing issues to make sure this feature has not been requested yet.
[X] I have provided enough information for the maintainers to understand and evaluate this request.

Attaching #877


feat(app): add ethernet connection screen

Overview
Add ethernet connection screen
Close RCORE-377
Note: ConnectViaEthernet component's header will be exported as a TouchscreenHeader component in another PR.
https://opentrons.atlassian.net/browse/RCORE-434
[no network found]
This screen can be reviewed with make -C app dev OT_APP_IS_ON_DEVICE=1
https://www.figma.com/file/AoTLAYuWawlaWItB1umOjr/OT-3-On-Device-Display?node-id=1776%3A30238&t=F2ZPLV4UXIjoeJdx-0
[connected]
For this, need an OT-3/a dev kit that is connected to a network via ethernet.
https://www.figma.com/file/AoTLAYuWawlaWItB1umOjr/OT-3-On-Device-Display?node-id=1776%3A30238&t=F2ZPLV4UXIjoeJdx-0

Changelog

add ConnectViaEthernet component
add test for ConnectViaEthernet component

Review requests

plug the lan cable into a dev kit and tap Ethernet on network setup menu -> connection status: connected and show the ip address, subnet mask and mac address
unplug the lan cable and the app show no network found but still display mac address

Risk assessment
low

[ ] merge edge


Add support for logging to AWS Kinesis (basically a hosted Kafka)

https://aws.amazon.com/kinesis/streams/

Looks good, thanks for the contribution!

Any idea about these test failures? They seem unrelated to the patch.
Also, is it ok to just add failed records back to the queue this way when the buffer has sync=true? No deadlock risk? Or should I only add them back to the queue when sync=false?

The test failures are due to a development dependency issue with the latest release of Nokogiri. It's resolved in master, so if you rebase the build failures should go away.
It should be safe to add records back to the buffer this way. Even with sync=true, the exact same buffering mechanism is in use. The only difference is that a flush is forced every time a message is written. While a flush is in progress, messages can still come into the buffer and get queued up. They will get wait there and get dequeued during the next round of flushing.
I think this is good to go, and hope to merge/release it this week.

Ok, rebased and updated PR.

Current coverage is 91.45% (diff: 92.42%)

Merging #111 into master will increase coverage by 0.11%

@@             master       #111   diff @@
==========================================
  Files            59         61     +2
  Lines          1408       1474    +66
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
+ Hits           1286       1348    +62
- Misses          122        126     +4
  Partials          0          0

Powered by Codecov. Last update 5874e07...987a2ff

I noticed something missing from this device which is present in other devices. If there is an unrecoverable connection error, other devices will call close(flush: false) to clean up the IO object without flushing any messages. This prevents connections being leaked if there is extended downtime or perhaps a configuration issue. This is the only change I would make. I can totally do it after merging unless you would like to do it.

It would also be a good idea to override close! unless Aws::Kinesis::Client responds to close. (It's hard to tell from the aws-sdk documentation whether it does or not.)

I'll take a look and update the PR first thing in the morning.

Released in 0.22.0.

Hey guys, @aianus, @dwbutler If I set sync: true, is this supposed to start pushing all logging to my kinesis stream? If so, I'm not able to get it working locally on my dev machine. I can create a LogStashLogger::Device::Kinesis object in rails console and call :write_one to my stream, and that works fine. Wondering what Im doing wrong.
Thanks,
Ash

@ashramsey That's unusual. Yes, the expected behavior when setting sync: true is that every log message is immediately written to the device. For buffered devices, the buffer is still used, but is autoflushed after each write. For debugging, it will help if you set the buffer_logger option. If you're still having trouble with this, feel free to open a new ticket.


*fix bug with deserialization

Bug with deserialization.
Reproduce in  "Don't Keep Activities" mode if try close-open app

@josinSbazin, May I ask why have you added @JvmField annotation?

Because on some devices deserialization causes crash.
Reproduce in "Don't Keep Activities" mode if try close-open app.
Reason - https://stackoverflow.com/a/35252507/8308233


add overflow checks for MT_UNSIGNED, MT_NEGATIVE and MT_TAG

__builtin_add_overflow is not supported by my MSVC compiler.  I would not be in favor of this.

@jimsch It would be interesting to know: how do you do overflow checks? Does MSVC have similar builtins? Do they have library functions doing integer overflow checking?

I am not aware of any such function that is part of MSVC.  The few times that I have cared about it I have always written specific code for that addition.  Usually by adding in a longer int and checking the result.  The need to make sure that things run on all the different architectures targeted.


Add Akshita

@akshitagupta15june , please can you make an issue with the image as the comment, so when we vote we can use the issue? Thanks.

yes yes

i have added it please check @CodingSpecies

@akshitagupta15june thank you I saw the issue.

@all-contributors please add @akshitagupta15june  for content


Implement snap ghosts for grab widgets.

This matches the implementation for original snap functionality except now SelectionWidgets also have a snap ghost (a simple bounding cube)

Closing. Going to split this into two PRs


Fix Broken Link in README

The DNSCrypt Windows Service Manager GUI link is broken, relink to github project.

Merged, thanks!

Not a Problem


Service offline page

What's included?
Service offline page improved.
GDS/JAC Styling used.
Advised wording added.
Who should test?
✅ Product owner
✅ Developers
How to test?

Switch to branch locally
Open apply/service-offline/index.html with any browser

Risk - how likely is this to impact other areas?
🟢 No risk - this is a self-contained piece of work
Additional context

PREVIEW:DEVELOP
can be OFF, DEVELOP or STAGING

@tomlovesgithub  - Changes from comms team

@tomlovesgithub  - Can we have a new preview URL please? We would like to review this and push it out


chore(charts/paperless-ngx): remove line breaks from comments

What this PR does / why we need it
Just removes some whitespace / line breaks which will be generated from comments in charts/paperless-ngx/templates/configmap.yaml.
Also updating helpers for same comment convention.
Which issue this PR fixes
no issue created
Special notes for your reviewer
no special note
Checklist

[ ] Chart Version bumped -> waiting for other open PR #20 before bumping version
[x] User name added to AUTHORS -> file added via other PR
[x] Title of the PR starts with a valid commit scope as detailed in the CONTRIBUTING (e.g.
fix(charts/linkwarden): ...)

Please do not close PR - still valid.

Hello @FMJdev I've bumped the chart version - after this is merged I also will bump the version of #22 if this is fine for you!
I added some fix to complete ct lint (new linke, spaces) - but still struggling with the following:
 ✖ paperless-ngx => (version: "0.2.5", path: "charts/paperless-ngx") > failed validating maintainer "FMJ Studios Helm Authors": 404 Not Found

How would you link to fix that?
If you want I can move the fix commits into another PR (I added theme here to get sucessful checks).

It seems the name must match a GitHub username / organization.
diff --git a/charts/paperless-ngx/Chart.yaml b/charts/paperless-ngx/Chart.yaml
index cbc6df7..4123eed 100644
--- a/charts/paperless-ngx/Chart.yaml
+++ b/charts/paperless-ngx/Chart.yaml
@@ -36,7 +36,7 @@ keywords:
 sources:
   - "https://github.com/fmjstudios/helm"
 maintainers:
-  - name: FMJ Studios Helm Authors
+  - name: fmjstudios
     email<EMAIL_ADDRESS>   - name: FMJdev
     email<EMAIL_ADDRESS>
When adding this change the linting will be sucessful.
see: helm/chart-testing#659

Hey, first off thank you for the research into the issue. Very surprising to see this pop up now. I wanted to use the Authors rather than the GitHub Org to credit everybody in the AUTHORS file, but nvm I guess.
I took a closer look at the issue and like you noted and commited ct will then start complaining about bracket spacing and missing ending newlines. I wanted to add pre-commit anyways to aid with consistency and took the time to erase all of these issues across all charts as well as add a new chart-testing target for make for more regular linting. Sorry for voiding your work somewhat there. I published 0.2.5 already as a result of this so I'd kindly ask you to edit the PR a final time 🤞🏼
Feel free to edit #22 once we have this sorted out. I'll merge when you've had the time. Thanks a lot once again! 🥇

Solved the merge conflict and bumped the chart version.
Now creating the kind cluster in Testing failed:

docker: Error response from daemon: driver failed programming external connectivity on endpoint chart-testing-control-plane (00b7cdf4af30278583d36d498b18c9c4576dc29d2a88e921150f312a1e88bfd3): Error starting userland proxy: listen tcp4 <IP_ADDRESS>:22: bind: address already in use.

How fitting for all of this to pop up during your PR, not during my hundreds of commits 😆 The Testing workflow is quite finicky due to the changed checks 👈🏼  I'm going to forego CI here since it's a bogus issue and I'm going to take a closer look at it on the weekend. Seems like GitHub's CI machines already have port 22 bound, obviously most likely for SSH but I would've expected CI runners to use custom ports.
Nonetheless, thank you for all your work so far! 🥇


Clarify the balance referred to in eps-event-roles.rst

Describe the change being requested
From @JacalynLaird : Clarify what 'the balance of the event' means
State the reason for the change
Clarifying
Describe the impact of the change on other parts of the SSI-EOG
N/A
Describe any alternatives you have considered (if applicable)
N/A
Additional context

clarified as the type and number of sessions


Revise method for requesting MOSRS access

Closes issue #21

Yet more issues are to be addressed. Merge the current changes and tag as v0.4.6 on the master branch.


Clarify reservations can be paid for monthly

This PR clarifies that reservations can be paid for monthly. This was always the case, but isn't stated explicitly today (although it is for other services like VMs and App Service).

@johndowns : Thanks for your contribution! The author(s) have been notified to review your proposed change.


adding database class and test_connection method

Added the database class as well as the test_connection method.
Let me know if you need anything.
Best regards.

Also added finishing touches for working over https and extra configurations needed for remote chart creation

Hello, @nogreintel thank you for your contribution, this seems fine for me. I will merge it and release a new version ! 👏


[WFCORE-1802] Ensure the providers are registered first so they have priority.

https://issues.jboss.org/browse/WFCORE-1802
https://issues.jboss.org/browse/JBEAP-8673

I've got #2408 on master-ignore and expect to merge it once CI is done; once I do this will need a rebase as it conflicts in the subsystem xml template.

@bstansberry ready to give this one another go.


Fix dependecy vulnerability

Summary
Fix dependency vulnerability with svgo -> css-select -> nth-check.  https://github.com/advisories/GHSA-rp65-9cf3-cjxr
Test plan
N/A

Hello @MelonBall, thanks but I will take care of upgrading deps before releasing v6.


Combination of blockwise and observe not working

Observing a resource whose payload needs more than 1 block to be transferred is not working. The sequence of messages is as follows:

Client performs first GET, with observe flag set
Server responds back with block #0. Due to the check in https://github.com/plgd-dev/go-coap/blob/master/net/blockwise/blockwise.go#L543, this execution does not reach to store anything in sendingMessagesCache
Client perfoms second GET of block #1
Check at https://github.com/plgd-dev/go-coap/blob/master/net/blockwise/blockwise.go#L361, will return that sendingMessageExist is false. Final effect is that server responds block #0 again. Which breaks the logic in the client

Removing check at https://github.com/plgd-dev/go-coap/blob/master/net/blockwise/blockwise.go#L543 solves the issue.

@azambranogalbis Hi, do you have an example for blockwise transfer from client to server?
Thank you.


Remove declaration from SOAP request

First of all, the library is very good. Thanks..!!
My desired SOAP request:
<x:Envelope xmlns:x="http://schemas.xmlsoap.org/soap/envelope/" xmlns:cif="http://www.mawarid.ae/linkedCardsSummary/CRM/CIF.xsd">
<x:Header />
<x:Body>
cif:CIFReq
cif:CIFNo000000196</cif:CIFNo>
</cif:CIFReq>
</x:Body>
</x:Envelope>
Below is my code to create SOAP request:
let soapRequest = AEXMLDocument()
let attributes = ["xmlns:x" : "http://schemas.xmlsoap.org/soap/envelope/", "xmlns:cif" : "http://www.mawarid.ae/linkedCardsSummary/CRM/CIF.xsd"]
let envelope = soapRequest.addChild(name: "x:Envelope", attributes: attributes)
let header = envelope.addChild(name: "x:Header")
let body = envelope.addChild(name: "x:Body")
let getStockPrice = body.addChild(name: "cif:CIFReq")
getStockPrice.addChild(name: "cif:CIFNo", value: "000000196")
And by Output is:

<x:Envelope xmlns:x="http://schemas.xmlsoap.org/soap/envelope/" xmlns:cif="http://www.mawarid.ae/linkedCardsSummary/CRM/CIF.xsd">
<x:Header />
<x:Body>
cif:CIFReq
cif:CIFNo000000196</cif:CIFNo>
</cif:CIFReq>
</x:Body>
</x:Envelope>
But i want to remove declaration i.e. "" .
Please help me

Hi,
I'm not sure how (or why) could you do that,
but it seems to me from looking at the desired and output requests here that those are identical,
so because of that I don't quite understand the problem behind?
Cheers