[NTR]: stromal cell of lamina propria of large intestine

Note - please check that the term does not already exist (check OLS: https://www.ebi.ac.uk/ols/ontologies/cl)
Preferred term label:
stromal cell of lamina propria of large intestine
Synonyms
Definition (free text, please give PubMed ID in format PMID:XXXXXX)
A connective tissue cell of an organ found in the loose connective tissue. This cell is found in the lamina propria of the large intestine.
Parent cell type term (check the hierarchy here https://www.ebi.ac.uk/ols/ontologies/cl):
stromal cell: http://purl.obolibrary.org/obo/CL_0000499
What is the anatomical structure that the cell is a part of? Please check Uberon: https://www.ebi.ac.uk/ols/ontologies/uberon
lamina propria of large intestine: http://purl.obolibrary.org/obo/UBERON_0011189
Your nano-attribution (ORCID)
0000-0003-4183-8865
Any additional notes or concerns

@ngeorgeebi
stromal cell of lamina propria of large intestine EFO_0010894
will be in the EFO release scheduled for Nov 16th.
I'll add the EFO IDs for your other terms in your CL tickets, as your GitHub handle doesn't seem to work in the EFO repo.
Thanks,
Paola


Add staffs screen

Issue

close #190

Overview (Required)

Implemented the transition from the about screen to the staff screen.

It has almost the same structure as the contributors screen.
Added feature-staff module.

staff screen and ui test

Added repository , api and response model to the core-data module.

Links

Figma

https://www.figma.com/file/NcSMs6dMsD88d4wOY0g3rK/Conference-App

Screenshot

Before
After (picture)
After (movie)

none

Thanks for contribution!!
Could you resolve conflicts??

@RyuNen344
Thanks for pointing out, I resolved the conflict:pray:

@t-nonomura
Could you run ./gradlew spotlessApply ? 🙏
It resolves the lint warnings.

@t-yusaku
I'm checking the error details, but ./gradlew spotlessApply could not be executed due to the following error.
An exception occurred applying plugin request [id: 'droidkaigi.primitive.dependencygraph']
> Could not find implementation class 'io.github.droidkaigi.confsched2022.primitive.DependencyGraphPlugin' for plugin 'droidkaigi.primitive.dependencygraph' specified in jar:file:/Users/username/.gradle/caches/jars-9/89627793f7570c1589b13b2439a6f06c/build-logic.jar!/META-INF/gradle-plugins/droidkaigi.primitive.dependencygraph.properties.

I found the problem from the build log, so I fixed it manually for the time being.
914e7c1

Sorry. Can you run spotlessKotlinApply?

@takahirom
Sorry, ./gradlew spotlessApply succeeded after deleting all gradle caches. (rm -rf  ~/.gradle/caches/)
Also the same for spotlessKotlinApply:bow:

Sorry for bothering you. Can you take a look at the conflict?

Let me see this tomorrow.

Please resolve conflict and move the function to core-designsystem 🙏
You don't have to move to core-designsystem, if you don't have time.


[Feature]: No option to close the rate us section

@param-code
There is no option to close the rate us section, if user dont want to share the rating, then there is no option to close, they have to reload the website only
So I would like add a close button
Could you please assign it to me

@param-code There is no option to close the rate us section, if user dont want to share the rating, then there is no option to close, they have to reload the website only So I would like add a close button Could you please assign it to me

Sure.


SRAM saving not working for ED64P

Unable to utilize game saves in SRAM on the ED64P stock OS or Alt64 OS.

Update - this is partially working, but not fully fleshed out.
With the stock ED64P firmware, you must first use the R trigger to 'Force SRAM', then in the NES ROM, use L+R and select save. Upon replays, remember to 'Force SRAM' and save.
For Alt64, there is no config option for .nes files to set the save type. It appears that between power cycles, you can use saves, but the saves are gone with powering off the console.

To the best of my knowledge there's no way for Neon64 to report that it needs SRAM, this is something the firmware needs to figure out for itself with a database of save typesor. Happy to reopen this is there's something that can change on my end, but for now I suggest asking the developer to add whatever version of Neon64 you want to run to their list, which may need to be updated with each new version. Unfortunately N64 ROMs don't have a header for info about cartridge hardware, so external databases are needed.

Hold my beer

i made a new build of altra64 with saves enabled.
ED64P.zip
this is save ram 128

as a temporary solution a buddy witten a program that pads roms in D
nes_padder.zip


Market list page Categorical Market type blends with the Tags

See screenshot:

@Chwy5 I believe this has been fixed - please confirm

Fixed - closing


feat: Enable anchor outputs

Anchor outputs are hidden by LDK behind a config flag.
We keep it consistent across all our targets to avoid unnecessary recompilation
for clippy.

Closing as it can't be finished until LDK 116 lands.


cannot find symbol "Detox" on Android RN 0.70

What happened?
$ ./gradlew assembleAndroidTest -DtestBuildType=debug

[...]

> Task :app:compileDebugAndroidTestJavaWithJavac FAILED
/Users/arthur/Code/Mobile/temp/Sample/android/app/src/androidTest/java/com/sample/DetoxTest.java:3: error: cannot find symbol
import com.wix.detox.Detox;
                    ^
  symbol:   class Detox
  location: package com.wix.detox
/Users/arthur/Code/Mobile/temp/Sample/android/app/src/androidTest/java/com/sample/DetoxTest.java:4: error: package com.wix.detox.config does not exist
import com.wix.detox.config.DetoxConfig;
                           ^
/Users/arthur/Code/Mobile/temp/Sample/android/app/src/androidTest/java/com/sample/DetoxTest.java:22: error: cannot find symbol
        DetoxConfig detoxConfig = new DetoxConfig();
        ^
  symbol:   class DetoxConfig
  location: class DetoxTest
/Users/arthur/Code/Mobile/temp/Sample/android/app/src/androidTest/java/com/sample/DetoxTest.java:22: error: cannot find symbol
        DetoxConfig detoxConfig = new DetoxConfig();
                                      ^
  symbol:   class DetoxConfig
  location: class DetoxTest
/Users/arthur/Code/Mobile/temp/Sample/android/app/src/androidTest/java/com/sample/DetoxTest.java:27: error: cannot find symbol
        Detox.runTests(mActivityRule, detoxConfig);
        ^
  symbol:   variable Detox
  location: class DetoxTest
5 errors

What was the expected behaviour?
The test APK builds successfully.
Was it tested on latest Detox?

[X] I have tested this issue on the latest Detox release and it still reproduces.

Did your test throw out a timeout?

[ ] I have followed the instructions under Identifying which synchronization mechanism causes us to wait too much.

Help us reproduce this issue!
# create 0.70.7 project / this also occurs on 0.71.3
npx<EMAIL_ADDRESS>init Sample --version 0.70.7
cd Sample

# add detox
npm i detox

# follow step 4 of https://wix.github.io/Detox/docs/introduction/project-setup#step-4-additional-android-configuration

# build test apk
cd android
./gradlew assembleAndroidTest -DtestBuildType=debug

Then you will run into #3867, which you can workaround using this solution.
After applying the workaround build again and notice the error above.
In what environment did this happen?
Detox version: 20.5.0
React Native version: 0.70.7 / 0.71.3
Has Fabric (React Native's new rendering system) enabled: (yes/no) no
Node version: v16.13.0
Device model:
Android version:
Test-runner (select one):
Detox logs
Detox logs
paste logs here!

Device logs
Device logs
paste your device.log here!

More data, please!
No response

node_modules/detox/Detox-android/com/wix/detox/20.5.0/detox-20.5.0.aar contains the specified symbols.
However node_modules/detox/Detox-android/com/wix/detox/20.5.0/detox-20.5.0-sources.jar does not.

Symbols

Archive:  /path/to/app/node_modules/detox/Detox-android/com/wix/detox/20.5.0/detox-20.5.0-sources.jar
   creating: META-INF/
  inflating: META-INF/MANIFEST.MF
   creating: org/
   creating: org/joor/
  inflating: org/joor/Reflect.java
  inflating: org/joor/ReflectException.java
   creating: com/
   creating: com/wix/
   creating: com/wix/detox/
   creating: com/wix/detox/common/
   creating: com/wix/detox/common/collect/
  inflating: com/wix/detox/common/collect/PairsIterator.kt
  inflating: com/wix/detox/common/UIThread.kt
  inflating: com/wix/detox/common/TextFileReader.kt
   creating: com/wix/detox/common/proxy/
  inflating: com/wix/detox/common/proxy/SpyingInvocationHandler.kt
  inflating: com/wix/detox/common/proxy/MethodsSpy.kt
  inflating: com/wix/detox/common/ErrorUtils.kt
  inflating: com/wix/detox/common/DetoxLog.kt
  inflating: com/wix/detox/common/DetoxErrors.java
  inflating: com/wix/detox/common/JsonConverter.kt
   creating: com/wix/detox/action/
   creating: com/wix/detox/action/common/
  inflating: com/wix/detox/action/common/MotionDefs.kt
   creating: com/wix/detox/espresso/
   creating: com/wix/detox/espresso/scroll/
  inflating: com/wix/detox/espresso/scroll/FlinglessSwiper.kt
  inflating: com/wix/detox/espresso/scroll/DetoxSwiper.kt
  inflating: com/wix/detox/espresso/scroll/ScrollHelper.java
  inflating: com/wix/detox/espresso/scroll/ScrollEdgeException.kt
  inflating: com/wix/detox/espresso/scroll/DetoxScrollActions.kt
  inflating: com/wix/detox/espresso/scroll/SwipeHelper.kt
  inflating: com/wix/detox/espresso/scroll/DetoxSwipe.kt
  inflating: com/wix/detox/espresso/scroll/ScrollProbes.kt
  inflating: com/wix/detox/espresso/ViewActionWithResult.kt
  inflating: com/wix/detox/espresso/UiControllerSpy.kt
  inflating: com/wix/detox/espresso/DeviceDisplay.kt
   creating: com/wix/detox/espresso/utils/
  inflating: com/wix/detox/espresso/utils/Vector2D.kt
   creating: com/wix/detox/espresso/action/
  inflating: com/wix/detox/espresso/action/TakeViewScreenshotAction.kt
  inflating: com/wix/detox/espresso/action/DetoxMultiTap.kt
  inflating: com/wix/detox/espresso/action/ScrollToIndexAction.kt
   creating: com/wix/detox/espresso/action/common/
  inflating: com/wix/detox/espresso/action/common/MotionEvents.kt
   creating: com/wix/detox/espresso/action/common/utils/
  inflating: com/wix/detox/espresso/action/common/utils/UiControllerUtils.kt
  inflating: com/wix/detox/espresso/action/common/DetoxViewConfiguration.kt
  inflating: com/wix/detox/espresso/action/common/TapEvents.kt
  inflating: com/wix/detox/espresso/action/common/ReflectUtils.kt
  inflating: com/wix/detox/espresso/action/DetoxSingleTap.kt
  inflating: com/wix/detox/espresso/action/ViewScreenshot.kt

I am not familiar with the java ecosystem, I don't know if that helps.

I have seen #1367 which looks very similar however my test is properly placed at android/app/src/androidTest/java/com/sample/DetoxTest.java.

My bad this was caused by wrong placement of the maven repository source.
I had to put it in "allprojects" section as stated in the docs, not "buildscript" section.
Change :
buildscript {
    repositories {
        maven { url("$rootDir/../node_modules/detox/Detox-android") }
    }
}

To :
allprojects {
    repositories {
        maven { url("$rootDir/../node_modules/detox/Detox-android") }
    }
}

New react native projects don't have the allprojects section which makes this easy to be missed. I think the docs should make it more explicit on step 4.
// Full android/build.gradle from a fresh react-native init
// Top-level build file where you can add configuration options common to all sub-projects/modules.

buildscript {
    ext {
        buildToolsVersion = "33.0.0"
        minSdkVersion = 21 // (1)
        compileSdkVersion = 33
        targetSdkVersion = 33
        kotlinVersion = '1.7.21' // (2)

        // We use NDK 23 which has both M1 support and is the side-by-side NDK version from AGP.
        ndkVersion = "23.1.7779620"
    }
    repositories {
        google()
        mavenCentral()

    }
    dependencies {
        classpath("com.android.tools.build:gradle:7.3.1")
        classpath("com.facebook.react:react-native-gradle-plugin")
        classpath("org.jetbrains.kotlin:kotlin-gradle-plugin:$kotlinVersion") // (3)
    }
}

allprojects { // (4)
    repositories {
        maven {
          url("$rootDir/../node_modules/detox/Detox-android")
        }
    }
}

thanks I also missed this step.

Thanks for this! the closing curly brace was collapsed on the docs that's why i missed it


Example app fails on generate-rewrites

Describe the bug
Example app doesn't run
Error: Cannot find module './dist/commonjs/utils/generate-rewrites'
Occurs in next-i18next version
it's the example app
Steps to reproduce
cd next-i18next/examples/simple
yarn
yarn add react react-dom
yarn dev
Expected behaviour
the app should start
Screenshots
If applicable, add screenshots or a GIF to help explain your problem.

OS (please complete the following information)

Device: Ubuntu 18.04

There are contributing docs here. The simple example is online at next-i18next.com, it definitely runs.


Dependency to iban is missing

Hello! Looks like dependency to iban is missing that result in build error

oh damn, good catch. will fix.


Enable more concrete rulesets on DelimitedList

Some delimited lists can have empty elements, and some delimited lists cannot. However, we don't specify concrete rulesets for each of these list types, and everything gets parsed in the most lenient way possible. This produces a valid tree, but also means that we're not providing errors where we should. We should revisit all of the classes that extend DelimitedList and define concrete rulesets for each.

Note: in terms of parsing this is probably a good thing.  A post-pass can check for lists that don't have the right number of elements and add errors for them.


cmd/compile: failed to parse floating point constant on buffer boundary

main.go.zip
Please answer these questions before submitting your issue. Thanks!
What version of Go are you using (go version)?
go version devel +c100a0f668 Tue Sep 19 14:56:08 2017 +0000 linux/amd64
Does this issue reproduce with the latest release?
Don't know.
What operating system and processor architecture are you using (go env)?
GOARCH="amd64"
GOBIN=""
GOEXE=""
GOHOSTARCH="amd64"
GOHOSTOS="linux"
GOOS="linux"
GOPATH="/home/elias/dev/go:/home/elias/git/ui"
GORACE=""
main.go.zip
GOROOT="/home/elias/dev/go-tip"
GOTOOLDIR="/home/elias/dev/go-tip/pkg/tool/linux_amd64"
GCCGO="gccgo"
CC="gcc"
GOGCCFLAGS="-fPIC -m64 -pthread -fmessage-length=0 -fdebug-prefix-map=/tmp/go-build857955865=/tmp/go-build -gno-record-gcc-switches"
CXX="g++"
CGO_ENABLED="1"
CGO_CFLAGS="-g -O2"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-g -O2"
CGO_FFLAGS="-g -O2"
CGO_LDFLAGS="-g -O2"
PKG_CONFIG="pkg-config"
What did you do?
go run main.go
What did you expect to see?
A successful run.
What did you see instead?
command-line-arguments
example/main.go:141:27: malformed constant: /5 (syntax error scanning number)

Nice find, thanks!

Reopening for 1.9.x backport per discussion with @rsc and @griesemer.

CL 64830 OK for Go 1.9.2.

go1.9.2 has been packaged and includes:

CL 64830

The release is posted at golang.org/dl.
— golang.org/x/build/cmd/releasebot, Oct 26 21:09:15 UTC


add digital marketplace redis metrics config

Please be gentle as I am unable to test this myself.
Attempted to follow steps in https://github.com/alphagov/paas-prometheus-endpoints/tree/main/src/redis#setup adding of credentials performed by @szd55gds in https://github.com/alphagov/re-secrets/pull/56 note, I was told the convention in the repository is to use hyphens in the credential filenames but our existing credential uses underscore so I am not sure if this is correct/will work.
CC paas peeps who have helped me with this @46bit @AP-Hunt @schmie
For @alphagov/digitalmarketplace this is ref: https://trello.com/c/vaR0HCCx/585-implement-redis-metrics-on-graphana

@Krenair I am unable to perform the task you mention. Please may someone with the appropriate permissions pick this up?
CC @szd55gds

I'm talking about changing the PR through Git, nothing privileged.

@Krenair OK sorry for the misunderstanding, this was not mentioned at all in the PaaS documentation, I assume I need another thing like https://github.com/alphagov/prometheus-aws-configuration-beta/blob/365fe9f86f60ab0df082b640eea3b9141843159d/terraform/projects/prom-ec2/paas-production/main.tf#L65 ?

@Krenair OK I think I have made the suggested changes now, please may you re-review?
Again; please be gentle as I am unable to test the changes myself nor am I able to view the PR https://github.com/alphagov/re-secrets/pull/56 to check it's content.


Issue with Inactive Records and Checkbox

Issue 1: Show Inactive checkbox should be available when a Configuration has Inactive Records (When a user adds an End Date and Time to a record, it is deemed Inactive)
Currently, the Inactive Checkbox is always greyed out and not available even when there are Inactive Records
Issue 2: When a record has an End Date and Time added, it should be displayed in the grid by default. It only shows up when a user selects 'Show Inactive'
Reference Records under Escalante (ORIS Code: 87)

Issue 3: For some Sections, the Show Inactive button is enabled even when there are no Inactive records available

Issue Fixed


chore(merge v4-dev): patched commit → 8ec32ee

That was an easy one 👯♂

Indeed, which is quite a good news I suppose, it means no breaking change was introduced in the current version :D


Set music volume

It's possible to set the volume of the current stream? Thanks in advance!

Thanks for the MR 👍


Accordion expand custom logic implement

Please always provide the GitHub issue(s) your PR is for, as well as test URLs where your change can be observed (before and after):
Fix #124
Test URLs:

Before: https://main--fonterra-fernleaf--hlxsites.hlx.page/en
After: https://124-Accordion--fonterra-fernleaf--hlxsites.hlx.page/en

@sselvara find likes
https://124-accordion--fonterra-fernleaf--hlxsites.hlx.page/drafts/gautham/products/milk-powder-for-whole-family
https://124-accordion--fonterra-fernleaf--hlxsites.hlx.page/drafts/gautham/products/test

fyi, @sselvara @ParthasharathiB
the links to pages impacted by the code change should actually be added test URLs section.
I just updated it and re-executed PSI check.
Before merging, please test the PSI scores, so they dont impact final LHS when merged in main.


rc/arduino

thank you so much for this great project
my issue is how to connect the rc with the arduino, I need four pins is that correct ?
I'd be grateful is you tell me how to connect them withe rc

Please see this image:
https://zhengludwig.files.wordpress.com/2015/07/img_9006.jpg?w=2000&h=&crop=1


Upgrade to version v1.0.1

[1.0.1] - 2023-10-26
Updated

Updated package versions to resolve security vulnerabilities.

Security scans of generative-ai-application-builder-on-aws < v1.1.0 will report a critical severity CVE due to use of the python package langchain-ai/langchain < v0.0.325
We can confirm that this version (v1.0.1) of Generative AI Application Builder on AWS is not directly impacted by the CVE because the solution does not use the LangChain Agents and the Python REPL capability identified in the CVE. However, we have released an update in v1.1.0 of the solution to incorporate the patch made available by the LangChain maintainers.


Give up on Cron

This PR reverts everything in https://github.com/AnalyticalGraphicsInc/cesium-concierge/pull/117, and replaces cron with node-schedule. I should have given up on cron far sooner. Reading the environment variables from disk is complicated by the fact that some of the variables have dashes in them (because repo names). It could have worked if the dumpEnvironment function dumped it into a config.json, but then updating the env variables on AWS after deployment would potentially cause a mismatch between the cron scripts and the actual concierge.
This just sets up the scheduled jobs in the application itself. No need to go out to cron, which also means it gets the same environment, so we avoid this issue altogether.

Thanks for the pull request @OmarShehata!

:heavy_check_mark: Signed CLA found.
:grey_question: CHANGES.md was not updated.

If this change updates the public API in any way, please add a bullet point to CHANGES.md.

:sparkles: :sparkles: :sparkles:I'm the Cesium Concierge and I'm here to test things. :sparkles: :sparkles: :sparkles:

@ggetz I opened an issue to remove supervisord at a later time. This should be good to go now.

Looks good, let's give it test. Thanks @OmarShehata!


Question. Displaying a message on the device.

How would I go about displaying a string or a message in game on the device containing data from the mitm proxy on my computer? Is there a built-in method or function in Pokemon Go already that I can utilize? If not, how would I go about doing so? I was maybe thinking about displaying the capture_probability for all the types of pokeballs on the device whenever I encounter a pokemon just so that I am aware of the odds that I am dealing with. Or maybe even possibly displaying the CatchPokemon status the moment the pokeball hits the pokemon so you know right away whether or not you caught the pokemon or if it has escaped or fled. Any help at all would be absolutely great! Thanks in advanced!

Uh that would be nice indeed, but the only thing you can do is modify an incoming response to trick the client to replace a string with another one (like the display_iv example it modify the response to append the IV% at the end of each pokemon's nickname).
You cannot yourself add any UI elements to the app. I don't remember how often the inventory is refreshed (haven't used the app since they started to check unknown6) but what you could do is once you're in an encounter save the probabilities in a global, and then append the probability of each pokeball to their name in the inventory when the inventory response comes in.
That way when you're in an encounter you would just have to wait for the inventory to be refreshed and when you check your bag you could see the probability of each pokeball in their name.
Maybe. I haven't tried, but that's how I would approach it.

Think about something like https://pushover.net/ - you install the app on your mobile and add the api call to their service into the MITM script. This would make it pretty easy to enable solid push notifications like IV of just caught pokemon. Cheers!

other idea:

use the username as temporary display after a catch! (it gets pulled after a catch to get the new stardust -> intercept!)
use the nickname append like in the display_iv

it's not as cool as prompting a user a message box with an own string though :)

Your first idea was what I was exactly just thinking about! I was thinking about maybe a separate app that will either send push notifications to the phone or maybe an overlay that would display data. Never knew there was already a service for what I was looking for haha. I just have no idea how to actually add the api to the MITM script and actually configure it to my liking since I'm not quite familiar with the CoffeeScript/JavaScript language. If anyone is willing to guide me a bit then that would be greatly appreciated! :) I'm just wondering how difficult it'd be actually code my own app for this on my android device. The overlayed message won't be the biggest problem for me, but how to actually send data from the MITM script to the app. Sorry if any of this is too much too ask for! But thanks yet again in advance!


Fix Button Alignment for Stories

Current Look:

this is my proposed layout, is this fine with you?

Go ahead @adin234

Hey @aashutoshrathi, if we are confirmed that we can use the design which @adin234 has suggested then I can implement that.

We need to do something like that only, but it's better if we can place it in line with three dots.

@lalitmee i think that PR was already merged before

@adin234 Instagram changed the div map for Stories, so it is not working now.

Hey @aashutoshrathi, I was looking for the buttons in the stories but for me, there is no button on the stories.

@lalitmee I have the same issue. I think it has to do with how the content on the page is loaded. The extension does not know when there is new content loaded and therefore doesn't load the buttons.
If you load some stories directly by reloading the storie url, the buttons appear (tried this one https://www.instagram.com/stories/alternate.de/). But it doesn't always work, at marvel's storie the buttons just wont appear (https://www.instagram.com/stories/marvel/)

@leventdeniz then we have to reload the inject button function each time page changes.

@lalitmee I'm already using onscroll

Oh, then  we have to debug it why its taking so much time for post other than the 1st one. I think there is something in the if and else condition which is looping again and again.

I tried onload and also tried to listen for url changes but it didn't work for some reason. Didn't had too much time to test this.


Adding other login option

I want to add other login options (eg. Google, Facebook) to the login page. Please assign me for this.

Assigned!

Thank you. I am working on it

I want to work on this .  kindly assign me this issue

Hello, I'm a GSSOC'24 contributor
I would like to work on the same. Can you please assign this to me.

Hello , I want to add one more option to the social media section i.e. Instagram. Can you please assign this to me?

Hello, I'm a Experienced Contributor, I have completed Multiple challenges like Hacktoberfest-2020 and 2023.
I would like to work on this issue. Can you please assign this to me.


Select box not disabling correctly when using closure and multiple

Package
filament/forms
Package Version
v3.0.69
Laravel Version
v10.26.2
Livewire Version
v3.0.5
PHP Version
PHP 8.2.5
Problem description
When using a Select input with the ability to select multiple on a form and having it disabled dinamically depending on another input (a toogle in this case) the input changes color as if it is disabled but it still allows the user to select a value. If I use the disabled property outright (->disabled()) on the select it does work as expected and it does not allow the user to select a value.

Expected behavior
Select box should become disabled and user is not allowed to select any value.
Steps to reproduce
Create a form with the following fields
Select::make('users')->options(User::orderBy('name')->pluck('name', 'id'))->multiple()->disabled(fn(Get $get) => $get('select_all')), Toggle::make('select_all')->live(onBlur: true)
Reproduction repository
https://github.com/davidabadi/filamentFormBug
Relevant log output
No response

Please search existing issues before creating a bug report.
This is a duplicate of https://github.com/filamentphp/filament/issues/3068 and a couple of other dupes that have been reported since.


RN Debug crashes the app in latest version

After updating to agora 3.1.2 debug mode crashes the app with error in console:
Cannot form weak reference to instance (0x6000003a9950) of class RCTAgoraRtcEngineModule.

how to recurrence problem?

by the way, which version do you use? 3.1.2 or 3.1.2-rc.1?

fixed in 3.1.2-rc.2

Only crash on iOS debug mode and reload the application.


Discuss the new approach with AK Trainers who conducted previous React Trainings

Regarding the exercises and content, a 100% like-for-like course is not feasible considering REACT is front end framework and Java is currently predominately used for the backend.
Hence we propose that we replace the exercise in the below manner:

TS --> Core Java
React --> SpringBoot

MOM of our discussion with other AK Engineers:

We have decided that we should replace the TS with Java and React with Springboot.
It's also been discovered that the intended audience typically has varied experiences so it's best to keep it simple and things for everyone.
A lot of current topics are already well established in Java so we have to see what we need to add, remove and skim (For example Dependency Injection is very common but how to use it for the purpose of TDD might be interesting to them).
Other engineers have shown interest in reviewing and having these kinds of conversations when and if required.


node@12 has been deprecated warnings

The GH actions that use node12 produce these warnings in our pipeline:
Node.js 12 actions are deprecated. For more information see: https://github.blog/changelog/2022-09-22-github-actions-all-actions-will-begin-running-on-node16-instead-of-node12/. Please update the following actions to use Node.js 16: fkirc/skip-duplicate-actions

Also similarly set-output was deprecated and this should be used instead: https://docs.github.com/en/actions/using-workflows/workflow-commands-for-github-actions#setting-an-environment-variable
So once/if those actions are updated to 16 or later we should also update those versions to get rid of those warnings.
Namely it's:

actions/checkout (3.1.0 should be ok)
fkirc/skip-duplicate-actions (master is on 16 so probably 5.2.0 should be ok)
actions/setup-go (https://github.com/actions/setup-go/blob/main/package.json#L37 anything 3.x.x should be ok)
docker/login-action (2.1.0 should be ok)

Also found that issue in coredns-crd-plugin.


Where does horizon=27000 come from in atari_env.py ?

Hi, in file atari_env.py:

class AtariEnv(Env):
def __init__(self,
             game="pong",
             frame_skip=4,  # Frames per step (>=1).
             num_img_obs=4,  # Number of (past) frames in observation (>=1).
             clip_reward=True,
             episodic_lives=True,
             max_start_noops=30,
             repeat_action_probability=0.,
             horizon=27000,
             ):

I want to ask that why it's 27000? I mean is this just an arbitrary value and can be 28000, 29000, or any value similar to it?
Thank U.

Oh yes that's sort of an informal research standard....many of the works using Atari benchmark used that number.  But you could change it without breaking anything. :)


Interdependent operations that depends on each other

Describe the bug
Operations are planned according to the facts gathered at the start of the run.
Operations usually changes the state of the host, this means that the preconditions that exists during the call to the operation is different that what is expected when declaring it.
To Reproduce
$ cat Dockerfile
FROM fedora:32

RUN dnf install -y git

$ docker build . -t my-fedora-image

$ cat deploy.py
from pyinfra import host
from pyinfra.operations import dnf

dnf.packages(
    packages=["git"],
    present=False,
    sudo=True,
)

dnf.packages(
    packages=["git"],
    present=True,
    sudo=True,
)

print(host.state.ops)
$ pyinfra --dry @docker/my-fedora-image deploy.py
--> Loading config...
--> Loading inventory...
    The @docker connector is in beta!

--> Connecting to hosts...
    [@docker/my-fedora-image] Connected

--> Preparing operations...
    Loading: deploy.py
{@docker/my-fedora-image: {'18996c2767e7142774328ef0b335b99342919a9e': {'commands': [StringCommand(dnf remove -y git)]}, '3b2f90227d43a40a2288506c2a99bdc5e71f32b2': {'commands': []}}}
    [@docker/my-fedora-image] Ready: deploy.py

--> Proposed changes:
    Groups: @docker
    [@docker/my-fedora-image]   Operations: 2   Commands: 1
    [@docker/my-fedora-image] docker build complete, image ID: 39b782da6859

Expected behavior
$ pyinfra --dry @docker/my-fedora-image deploy.py
--> Loading config...
--> Loading inventory...
    The @docker connector is in beta!

--> Connecting to hosts...
    [@docker/my-fedora-image] Connected

--> Preparing operations...
    Loading: deploy.py
{@docker/my-fedora-image: {'18996c2767e7142774328ef0b335b99342919a9e': {'commands': [StringCommand(dnf remove -y git)]}, '3b2f90227d43a40a2288506c2a99bdc5e71f32b2': {'commands': [StringCommand(dnf install -y git)]}}}

    [@docker/my-fedora-image] Ready: deploy.py

--> Proposed changes:
    Groups: @docker
    [@docker/my-fedora-image]   Operations: 2   Commands: 2
    [@docker/my-fedora-image] docker build complete, image ID: 39b782da6859

Meta

pyinfra --support output

--> Support information:

    If you are having issues with pyinfra or wish to make feature requests, please
    check out the GitHub issues at https://github.com/Fizzadar/pyinfra/issues .
    When adding an issue, be sure to include the following:

    System: Linux
      Platform: Linux-5.4.0-40-generic-x86_64-with-glibc2.29
      Release: 5.4.0-40-generic
      Machine: x86_64
    pyinfra: v1.0.3
    Executable: /home/FooBarQuaxx/Code/pyinfra/venv/bin/pyinfra
    Python: 3.8.2 (CPython, GCC 9.3.0)

installed from source
no pyinfra-debug.log

This is a tough one! It's a long-standing limitation of pyinfra's two phase deploy (see: https://docs.pyinfra.com/en/latest/deploy_process.html#interdependent-operations).
Fix wise I've got two approaches:

Hint based arguments (as documented above) - implemented for files/etc, could work for packages too (certainly the above example)
Have operations modify facts to match the state they expect to set - this is the ideal (IMO) but needs some serious thought. It also does not deal with things like scripts or server.shell commands/etc.

I hope we can do something about it because it's a fundamental limitation.

Agreed - I'd like to solve this.
For the docs example - I don't think this will ever be possible, certainly not with the two-phase execution model. Even if pyinfra was to update facts as operations executed, it'd miss stuff like package config files/etc being added to the filesystem. In the case of files one option could be to check at runtime, ie stat {path} && rm -f {path}, but that means always outputting a command for that operation, defeating a major advantage of the two phase execution!
Of note here is also the files.line operation, which explicitly handles the "unknown" situation by outputting a small script (https://github.com/Fizzadar/pyinfra/blob/master/pyinfra/operations/files.py#L262). It could be possible to replicate this same idea to other operations.
I'm keen to hear more about your use-case here as well  - particularly installing and later removing the same package. Normally I'd treat oprations like installing packages as things you'd only want to run once, but that's coming from my own experience only.
Sidenote: I played around a bit with a potential API for updating facts during phase 1, which if fully implemented would satisfy your original example and others: https://github.com/Fizzadar/pyinfra/compare/experiment-update-facts-in-operation.

For what its worth, I believe, I've just hit a similar case:
yum.packages(packages=['ssmtp'], present=False)
yum.packages(packages=['postfix', 'mailx']) #ssmtp has to be removed or postifix install will not succeed

Another example from #394 via @wookayin:
apt.packages(
    name='Install Docker via apt.',
    packages=['docker-ce', 'docker-ce-cli', 'containerd.io'],
    update=_add_apt.changed,
)

server.group(
    name='Create docker user group',
    group='docker',
)

will fail when docker gets installed, because
At the moment when facts are gathered, there is no docker installed and therefore no docker group
After docker installation is done, it tries to do add group, but fails with an error "the group already exists"

I've made some progress on this! Based on my experiment branch and some extra work, I now have support for the packaging modules (dnf.packages and yum.packages examples) implemented + tested in: https://github.com/Fizzadar/pyinfra/pull/397.
This approach works really well for calls to the same operation. The current model is roughly:

operations are idempotent when considered individually

this PR/approach would increase this to

single operations are idempotent across multiple calls

Unfortunately this goes nowhere to fixing the docker example which would be a full:

all operations are idempotent, both individually and with each other

So far I can't see any suitable way to achieve this without modifying the execution model to evaluate facts at runtime (loosing dry-runs and so on). For this situation, it could be possible to fix this by outputting scripts that first check to see if the state has changed, ie groups | grep <group> > /dev/null || groupadd <group>.

Ultimately I think there will always be some kind of edge cases here, no matter the implementation. I think it's super important to be really upfront where these remain and heavily document them, as I do not wish to change the execution method (I strongly believe the two-phase deploy has more advantages that disadvantages, esp. for future work ie APIs/servers/etc). Based on the above I can definitely see a path forward here that should support almost all use-cases, which is excellent :)

I am now confident that https://github.com/Fizzadar/pyinfra/pull/397 covers all of the cases listed here, which is excellent :)
Need to decide next steps in terms of documenting and keeping track of any similar future issues, as they can be extremely nuanced/difficult to identify.

For reference, i have a similar case:
server.user(
    name='Create user',
    user='nicolas',
)

server.shell(
    name='Hello',
    commands=['echo "Hello $USER"'],
    su_user='nicolas',
)

Here is the real usecase: https://www.reddit.com/r/pyinfra/comments/i50n0f/create_an_user_and_then_use_it_for_next_tasks/
I will test the PR to see it is fixed by it.

@nikaro I believe this is failing due to fact collection (which happens before the user is created) is trying to use that user. In effect it's not possible to create a user and then use them with the auth arguments sudo_user/su_user.
Just noticed your reddit reply which should indeed cover this, using the user/group arguments to files.* operations to avoid the need for su.
The auth arguments expect the users to already exist on the remote side. I need to a) document this better and b) replicate your issue and make it output something useful, because the error you're getting is an awful experience!

This has now been released in v1.1 🎉!


[ARM] az resource tag: Add parameter --is-incremental to support adding tags to resource incrementally (#12736)

Description of PR (Mandatory)
Feature request: #12736
Testing Guide
Run az resource tag --ids resource-id --tags new-key=new-value -i
History Notes:
(Fill in the following template if multiple notes are needed, otherwise PR title will be used for history note.)
[Component Name 1] (BREAKING CHANGE:) (az command:) make some customer-facing change.
[Component Name 2] (BREAKING CHANGE:) (az command:) make some customer-facing change.

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

[x] The PR title and description has followed the guideline in Submitting Pull Requests.

[x] I adhere to the Command Guidelines.

add to S168


As a player, I want to see the current scores of both players after each answered question to compare myself with the opponent

Acceptance Criteria:

[x] The scores contain the usernames, profile pictures and the points achieved in the current game
[x] The scores are only visible for ten seconds
[x] The player who gets to select the next topic is shown the topic selection at the same time

Development Tasks:

[x] sopra-fs23-group-10/sopra-fs23-group-10-client#22
[x] sopra-fs23-group-10/sopra-fs23-group-10-server#60
[x] sopra-fs23-group-10/sopra-fs23-group-10-client#23
[x] sopra-fs23-group-10/sopra-fs23-group-10-client#24

All dev tasks are completed and acceptance criterias are met. Issue is therefore closed.


Add Sheets v4 API discovery baseline

This will make it easier to see changes in the, adding optional params, CL

Current coverage is 83.86% (diff: 100%)

Merging #931 into master will not change coverage

@@             master       #931   diff @@
==========================================
  Files           337        337
  Lines         13088      13088
  Methods           0          0
  Messages          0          0
  Branches       1725       1725
==========================================
  Hits          10976      10976
  Misses         1674       1674
  Partials        438        438

Powered by Codecov. Last update d59d34e...744e713


Fixing missing default image bug

I had a problem when using the API from an external script, which was throwing this error:
Traceback (most recent call last):
  File "/usr/lib/python2.7/site-packages/comictagger-1.1.15b0-py2.7.egg/comicapi/comicarchive.py", line 648, in __init__
    with open(fname, 'rb') as fd:
TypeError: coercing to Unicode: need string or buffer, NoneType found

This fixes that error

Accidentally added other commits


claim  (MEGH-1257)

i got an error every time i do this step :
$ python rainmaker.py claim 'COM7'
[ERROR]:Failed to connect to Espressif device: Timed out waiting for packet header
Failed to connect to Espressif device: Timed out waiting for packet header
what should i do !!

@boujmilmaha , did you read my comment here? Is your COM port open somewhere else, probably in idf.py monitor? Is the COM port correct? Have you tried removing the quotes or adding double quotes to COM7?

Closing this issue as there has been no response on this, and the "Assisted Claiming" would also help mitigate it.


Add key type and duplicates verification with hashing

Closes #2230
There is currently no verification for the data type and the uniqueness of the keys yielded by the dataset_builder.
This PR is currently a work in progress with the following goals:

[x] Adding hash_salt to ArrowWriter so that the keys belonging to different splits have different hash
[x] Add key arrtibute to ArrowWriter.write() for hashing
[x] Add a hashing class which takes an input key of certain type (str/int/anything convertible to string) and produces a 128-bit hash using hashlib.md5
[x] Creating a function giving a custom error message when non-unique keys are found
[This will take care of type-checking for keys]
[x] Checking for duplicate keys in writer.write() for each batch

[NOTE: This PR is currently concerned with GeneratorBasedBuilder only, for simplification. A subsequent PR will be made in future for ArrowBasedBuilder]
@lhoestq Thank you for the feedback. It would be great to have your guidance on this!

@lhoestq The tests for key type and duplicate keys have been added and verified successfully.
After generating with an intentionally faulty mnist script, when there is an incompatible key type, it shows:
Downloading and preparing dataset mnist/mnist (download: 11.06 MiB, generated: 60.62 MiB, post-processed: Unknown size, total: 71.67 MiB) to C:\Users\nikhil\.cache\huggingface\datasets\mnist\mnist\1.0.0\5064c25e57a1678f700d2dc798ffe8a6d519405cca7d33670fffda477857a994...
0 examples [00:00, ? examples/s]2021-04-26 02:50:03.703836: I tensorflow/stream_executor/platform/default/dso_loader.cc:49] Successfully opened dynamic library cudart64_110.dll

FAILURE TO GENERATE DATASET: Invalid key type detected
Found Key [0, 0] of type <class 'list'>
Keys should be either str, int or bytes type

In the case of duplicate keys, it now gives:
Downloading and preparing dataset mnist/mnist (download: 11.06 MiB, generated: 60.62 MiB, post-processed: Unknown size, total: 71.67 MiB) to C:\Users\nikhil\.cache\huggingface\datasets\mnist\mnist\1.0.0\5064c25e57a1678f700d2dc798ffe8a6d519405cca7d33670fffda477857a994...
0 examples [00:00, ? examples/s]2021-04-26 02:53:13.498579: I tensorflow/stream_executor/platform/default/dso_loader.cc:49] Successfully opened dynamic library cudart64_110.dll
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "f:\datasets\datasets-1\src\datasets\load.py", line 746, in load_dataset
    builder_instance.download_and_prepare(
  File "f:\datasets\datasets-1\src\datasets\builder.py", line 587, in download_and_prepare
    self._download_and_prepare(
  File "f:\datasets\datasets-1\src\datasets\builder.py", line 665, in _download_and_prepare
    self._prepare_split(split_generator, **prepare_split_kwargs)
  File "f:\datasets\datasets-1\src\datasets\builder.py", line 1002, in _prepare_split
    writer.write(example, key)
  File "f:\datasets\datasets-1\src\datasets\arrow_writer.py", line 321, in write
    self.check_duplicates()
  File "f:\datasets\datasets-1\src\datasets\arrow_writer.py", line 331, in check_duplicates
    raise DuplicatedKeysError(key)
datasets.keyhash.DuplicatedKeysError: FAILURE TO GENERATE DATASET !
Found duplicate Key: 234467
Keys should be unique and deterministic in nature

Please let me know if this is what we wanted to implement. Thanks!

This looks pretty cool !
We can make focus on the GeneratorBasedBuilder for now yes.
Do you think we could make the ArrowWriter not look for duplicates by default ?
This way we can just enable duplicate detections when instantiating the writer in the GeneratorBasedBuilder for now.

Thank you @lhoestq

Do you think we could make the ArrowWriter not look for duplicates by default ?

We can definitely do that by including a check_duplicates argument while instantiating ArrowWriter().
However, since only GeneratorBasedBuilder uses the write() function (which includes the detection code) and the others like ArrowBasedBuilder use write_table() which remains as it was (without duplicate detection). I don't think it would be necessary.
Nonetheless, doing this would require just some small changes. Please let me know your thoughts on this. Thanks!

I like the idea of having the duplicate detection optional for other uses of the ArrowWriter.
This class is the main tool to write python data in arrow format so I'd expect it to be flexible.
That's why I think by default it shouldn't require users to provide keys or do any duplicates detection.
An alternative would be to subclass the writer to include duplicates detection in another class.
Both options are fine for me, let me know what you think !

This class is the main tool to write python data in arrow format so I'd expect it to be flexible.
That's why I think by default it shouldn't require users to provide keys or do any duplicates detection.

Well, that makes sense as the writer can indeed be used for other purposes as well.

We can definitely do that by including a check_duplicates argument while instantiating ArrowWriter().

I think that this would be the simplest and the more efficient option for achieving this as subclassing the writer only for this would lead to unnecessary complexity and code duplication (in case of writer()).
I will be adding the changes soon. Thanks for the feedback @lhoestq!

@lhoestq I have pushed the final changes just now.
Now, the keys and duplicate checking will be necessary only when the ArrowWriter is initialized with check_duplicates=True specifically (in this case, for GeneratorBasedBuilders)
Let me know if this is what was required. Thanks!

@lhoestq Thanks for the feedback! I will be adding the tests for the same very soon.
However, I'm not sure as to what exactly is causing the segmentation fault in the failing CI tests. It seems to be something from test_concatenation_table_cast from test_table.py, but I'm not sure as to what exactly. Would be great if you could help. Thanks!

You can merge master into your branch to fix this issue.
Basically pyarrow 4.0.0 has a segfault issue (which has now been resolved on the master branch of pyarrow).
So until 4.0.1 comes out we changed to using pyarrow<4.0.0 recently.

@lhoestq Thanks for the help with the CI failures. Apologies for the multiple merge commits. My local repo got messy while merging which led to this.
Will be pushing the commit for the tests soon!

Hey @lhoestq, I've just added the required tests for checking key duplicates and invalid key data types.
I think we have caught a nice little issue as 27 datasets are currently using non-unique keys (hence, the failing tests). I'd like to make changes to the faulty datasets' scripts.
However, I was wondering if I should do that in this PR itself or open a new PR as this might get messy in the same PR. Let me know your thoughts on this. Thanks!


feat: Introduce service worker adapter

I added adapter for Service Worker on browser.
I think that Hono is good for creating Service Worker.
In the PR, you can write code like that:
import { Hono } from 'hono'
import { handle } from 'hono/service-worker'

const app = new Hono()
app.get('/', c => c.text('Hello World'))

self.addEventListener('fetch', handle(app))

When app responses 404, adapter fetches the original server.
The author should do the following, if applicable

[x] Add tests
[x] Run tests
[x] bun run format:fix && bun run lint:fix to format the code
[x] Add TSDoc/JSDoc to document the code

Sounds good! I'll try it later.

Hi @nakasyou
I've noticed that this adapter is so important! If we have this adapter, we can obsolete the app.fire() method in the next major release. The app.fire() is this:
// src/hono-base.ts
fire = (): void => {
  // eslint-disable-next-line @typescript-eslint/ban-ts-comment
  // @ts-ignore
  addEventListener('fetch', (event: FetchEventLike): void => {
    event.respondWith(this.dispatch(event.request, event, undefined, event.request.method))
  })
}

I think it's okay to delete it completely because this service worker style is now used only for Fastlyu Compute and Service Worker.

@yusukebe It sounds good!
However, the adapter is for browsers. So if app returns 404, adapter fetches the original server. Currently replacing app.fire maybe not good.

However, the adapter is for browsers. So if app returns 404, adapter fetches the original server.

Yeah. I'll consider it.

Hi @nakasyou !
I've created the demo project to try this adapter: https://github.com/yusukebe/service-worker-adapter-demo
And I'll suggest the code, though I'm not sure this is 100% good:
diff --git a/src/adapter/service-worker/handler.ts b/src/adapter/service-worker/handler.ts
index 2c85343..b8801e3 100644
--- a/src/adapter/service-worker/handler.ts
+++ b/src/adapter/service-worker/handler.ts
@@ -1,12 +1,12 @@
+/// <reference lib="webworker" />
 /**
  * Handler for Service Worker
  * @module
  */

 import type { Hono } from '../../hono'
-import type { FetchEventLike } from '../../types'

-type Handler = (evt: FetchEventLike) => void
+type Handler = (evt: FetchEvent) => void

 /**
  * Adapter for Service Worker
@@ -14,24 +14,16 @@ type Handler = (evt: FetchEventLike) => void
 export const handle = (
   app: Hono,
   opts: {
-    fetch: (req: Request) => Promise<Response>
+    fetch: typeof fetch | undefined
   } = {
     fetch: fetch,
   }
 ): Handler => {
-  return (evt) => {
-    const fetched = app.fetch(evt.request)
-    if (fetched instanceof Response && fetched.status === 404) {
-      return
+  return async (evt) => {
+    let res = await app.fetch(evt.request)
+    if (res.status === 404 && opts.fetch) {
+      res = await opts.fetch(evt.request)
     }
-    evt.respondWith(
-      (async () => {
-        const res = await fetched
-        if (res.status === 404) {
-          return await opts.fetch(evt.request)
-        }
-        return res
-      })()
-    )
+    evt.respondWith(res)
   }
 }

There are two points:

To correct types on the user side, this uses FetchEvent in the WebWorker for evt.
Simplify the logic. I think this is enough. The current code can't handle the 404 well, but the error will be removed in the demo project with this code.

Plus, with this code, you can pass through the handling 404 by assigning undefined to the fetch option. This means it can support Fastly Compute.
addEventListener(
  'fetch',
  handle(app, {
    fetch: undefined
  })
)

What do you think of them?

Hi @yusukebe, I agree for your comment.
I'll change it.

Hi @yusukebe, I have an issue at TypeScript. It is made by webworker types.
Do you know how to resolve?

@nakasyou
Awesome work! I'll merge this into the next now. Thanks!


How to preview the items in the shulker box in the wireless terminal?

I have a lot of boxed items in the game, but I can't preview the items in the shulker box in the wireless terminal. How can I operate it to see the items in the box without taking out the shulker box

That doesn't sound like functionality this mod would provide. The network requires physical connection to the inventories being used, and you can't get that with a shulker in your inventory.

What mod are you using to preview? The terminal uses fake slots, so some mods may not detect it.


fix(getUsd()): Added function to get AVAX spot price

:tada: This PR is included in version 1.7.6 :tada:
The release is available on npm package (@latest dist-tag)
Your semantic-release bot :package::rocket:


Bug: Search do not return new created products

What are you trying to achieve?
I want to be able to search for products that I added a few minutes ago.
Steps to reproduce the problem

Create new product
Run query products with created product name in the search

Result:  product does not appear in search result

but it is visible when querying about all products

What did you expect to happen?
Product should be possible to find
Logs
No response
Environment
Saleor version: core v3.18.0-a.0
OS and version: …

Search vectors are updated asynchronously via a Celery worker. Is the Celery running on this env where the issue was seen? Also it may take some time (e.g. 20-30 seconds) to wait for the task to start, as it's running periodically with 20s intervals be default.

After 1 hour of the creation issue, the product still does not appear in the search result. I'm testing it on our cloud staging https://master.staging.saleor.cloud/ so celery should work there

I launched Saleor locally. I added 3 products and 1 sale, I don't see that the celery job saleor.product.tasks.update_products_search_vector_task started beside the job that started when I first started celery.
log_celery_18102023.txt

fixed here https://github.com/saleor/saleor-multitenant/issues/976


Quarantined tests, brought them back, added wait

Working thru getting e2e tests to pass.

@christian-bromann I believe these flakes are caused by the lack of GPU acceleration in xvfb. Relaxing timeouts and adding waits in the right places seem to increase reliability of the tests.


Passing POW session information to a LiveView in test?

Thanks for the great library! :)
I'm trying to pass an authenticated user to a LiveView so I can access current_user, with the end goal to authorize access to resources based on user.  I've read a few of the (very helpful and detailed) issues on integrating Pow and LiveViews, and am able to get access to the user inside of the LiveView during mount by verifying their auth key from the session.
My issues arises when I try to replicate this behavior in test. I want to be able to simulate a log so that I can test authenticated and unauthenticated behaviors, but I've been struggling a bit to get this piece in place.  In regular controllers, I've done this with a test callback:
  defp authenticate_user(%{conn: conn, user: user}) do
    auth_conn = Pow.Plug.assign_current_user(conn, user, otp_app: :euchre)

    %{auth_conn: auth_conn}
  end

I tried doing this in my LiveView tests but am having a bit of trouble.  I noticed that during my test runs  that session was always nil in my LiveView's mount.  I did some searching and it seems like that's pretty normal for LiveView tests, and that it seemed like most people solved it by either piggybacking off another request where there is already a session, or injecting session data through test callbacks.  I've been trying the "inject session data" approach.
How can I derive a user's auth key from their Pow user struct?  My secondary question is: does this seem like a good approach for my goals, or is there a better way to go about passing a user to a LiveView and testing it?  Thanks!

I just got done with this. I added this function to my setup macro:
`defp first_user(), do: Users.list_users() |> Enum.at(0)
defp setup_session(%{ conn: conn }) do
conn = Plug.Test.init_test_session(conn, %{})
opts = Pow.Plug.Session.init(otp_app: :userdocs_web)
conn =
conn
|> Plug.Test.init_test_session(%{})
|> Pow.Plug.Session.call(opts)
|> Pow.Plug.Session.do_create(first_user(), opts)
:timer.sleep(100)

%{ conn: conn }

end`

Using @johns10's example above I documented my full auth/test helper setup for Pow with LiveViews.  See it here.
I am piecing together various things I've found so it'd be worth consulting threads like this to ensure this follows best security practices before including in your app.

I'll have to check out your solution because I'm still having a hard time with my tests.


question code

Hello, thanks for your interesting paper. I would like to inquire about when your team plans to release the code.

Hello, thanks for your interesting paper. I would like to inquire about when your team plans to release the code.

Thanks for your interest. We will release the code after the corresponding paper is accepted.


⚠ Bofrost.de has degraded performance

In 12512bc, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 947 ms

Resolved: Bofrost.de performance has improved in f00de88 after 13 minutes.


pxt uploadtrg to staging results in a broken build

Uploading a clean build to staging results in a broken client application:

Example: https://arcade.staging.pxt.io/app/505fe8eea9f0575b798c37ebdc35588e9d98d2c8-aaa3f16783
Error from console may provide a clue:

Oh this could be a more interesting error:

Failing to load editor.js. Could it be CDN related?

Hopefully fixed now.


Add more German translations

Hi @ds5678,
I added some more German translations for the mod. Feel free to merge them.
If you have any questions, let me know.
And thanks for continuing the development on those mods.
Cheers
icreatethings

Thank you!


fixes to java backend for c semantics

@andreistefanescu please review

Based on conversation with @andreistefanescu I am going to close this because it looks like KSequence is handled differently in his upcoming PR. @andreistefanescu you should take a look at this diff and see if any of these fixes can be applied to your copy though.

Made obsolete by the A/AC feature of the java backend.


Node 6 support

Im getting this error after the new Node 6 update:
path.js:7
    throw new TypeError('Path must be a string. Received ' + inspect(path));
    ^

TypeError: Path must be a string. Received undefined
    at assertPath (path.js:7:11)
    at Object.dirname (path.js:1324:5)
    at /.../node_modules/clean-css/lib/utils/input-source-map-tracker.js:191:56
    at Array.forEach (native)
    at trackContentSources (/.../node_modules/clean-css/lib/utils/input-source-map-tracker.js:188:20)
    at InputSourceMapStore.trackLoaded (/.../node_modules/clean-css/lib/utils/input-source-map-tracker.js:255:3)
    at fromString (/.../node_modules/clean-css/lib/utils/input-source-map-tracker.js:32:8)
    at InputSourceMapStore.track (/.../node_modules/clean-css/lib/utils/input-source-map-tracker.js:236:5)
    at Object.%

@tcarlsen hmm, I can see the error is propagating from clean-css as you can see in the node_modules/clean-css folder. Can you let me know which version of this plugin you are using as well as posting your gulp task so I can take a peek? I'd like to get this reproducing on my end. I'll be able to look into this a bit later on today, thanks!

package.json:
"devDependencies": {
    "babel-preset-es2015": "^6.6.0",
    "gulp": "^3.9.1",
    "gulp-angular-templatecache": "^1.8.0",
    "gulp-autoprefixer": "^3.1.0",
    "gulp-babel": "^6.1.2",
    "gulp-clean-css": "^2.0.6",
    "gulp-concat": "^2.6.0",
    "gulp-jade": "^1.1.0",
    "gulp-ng-annotate": "^2.0.0",
    "gulp-rename": "^1.2.2",
    "gulp-sass": "^2.2.0",
    "gulp-sourcemaps": "^2.0.0-alpha",
    "gulp-stylelint": "^2.0.0",
    "gulp-uglify": "^1.5.3",
    "gulp-xo": "^0.11.0",
    "stylelint-config-standard": "^6.0.0",
    "xo": "^0.15.0"
  },

gulpfile.js:
/* Styles */
gulp.task('styles', () => {
  return gulp.src(`${build}src/css/app.scss`)
    .pipe(sourcemaps.init())
    .pipe(concat('app.css'))
    .pipe(sass())
    .pipe(autoprefixer())
    .pipe(cleancss())
    .pipe(sourcemaps.write('./sourcemaps'))
    .pipe(gulp.dest(build + dest));
});

Same thing here with the shiny new Node 6.

@tcarlsen @bgatellier I believed I tracked down the issue. Please pull 2.0.7 and verify this resolved?

@scniro upgrade gulp-clean-css to v2.0.7，no error.

nice job. THX!

Thx.Nice job!

 Try deleting node_modules and reinstalling for good measure? I fixed this and all seemed to work, so I can't reproduce anything based on what you have shared alone.

Stack trace:
Path must be a string. Received undefined in file undefined line no. undefined TypeError: Path must be a string. Received undefined in file undefined line no. undefined
at assertPath (path.js:7:11)
at Object.dirname (path.js:1324:5)
at /projects/websites/webkb/node_modules/clean-css/lib/utils/input-source-map-tracker.js:191:56
at Array.forEach (native)
at trackContentSources (/projects/websites/webkb/node_modules/clean-css/lib/utils/input-source-map-tracker.js:188:20)

at InputSourceMapStore.trackLoaded (/projects/websites/webkb/node_modules/clean-css/lib/utils/input-source-map-tracker.js:255:3)
at fromString (/projects/websites/webkb/node_modules/clean-css/lib/utils/input-source-map-tracker.js:32:8)
at InputSourceMapStore.track (/projects/websites/webkb/node_modules/clean-css/lib/utils/input-source-map-tracker.js:236:5)
at /projects/websites/webkb/node_modules/clean-css/lib/clean.js:145:44
at CleanCSS.minify (/projects/websites/webkb/node_modules/clean-css/lib/clean.js:127:42)

Gulp task:
gulp.task('styles', ['copy-fonts'], function() {
return gulp.src(path.join(stylesDirectory, 'main.less'))
	.pipe(gulpDebug())
	.pipe(
		plumber(
			function(err) {
				gutil.log(gutil.colors.red('LESS error:\n'), err.message, err.stack || '');
			}
		)
	)
	.pipe(sourcemaps.init())
	.pipe(less({
		modifyVars: {
			"@fa-font-path": '"../fonts"'
		},
		plugins: [cleancss, autoprefix]
	}))
	.pipe(rename({
		dirname: '',
		basename: 'build',
		extname: '.css'
	}))
	.pipe(sourcemaps.write())
	.pipe(gulp.dest(path.join(config.paths.static, 'dist', 'css/')));
});

The fix linked above seems to work, but doesn't seem to be included in the package when I npm install it.

@turnerhayes can you explain what you mean by the fix linked above? That change is published to npm. How were you able to verify that?

When I manually add the code linked:
if (typeof(sourceFile) == 'undefined')
   return;

it works, but when I install it from npm it's not there.

The fix was actually in the commit seen here, not that code snippet: f62380bcffe1b4b57b0c07927cb1ac9cf50e0f80
Unfortunately I am still struggling to identify the issue. Have you entirely deleted your node modules and tried reinstalling?

Ah, ok. Then that fix isn't working, I'm still getting the LESS error. I have deleted node_modules and reinstalled, still nothing.

@turnerhayes  Still looking into this and trying to get something reproducing. I know it's not ideal, but have you tried to just pipe the plugins after your less task? Does the following produce the same result?
.pipe(sourcemaps.init())
.pipe(less({
modifyVars: {
"@fa-font-path": '"../fonts"'
}
}))
.pipe(autoprefix())
.pipe(cleancss())
/.../
Was your original task working as is before the node upgrade? 6.0.0 or a bump higher (e.g. 6.2.0)?

@turnerhayes Looks like an issue was brought up with the underlying clean-css. Lets see where this pr will go jakubpawlowicz/clean-css#774

Great, I'll keep an eye on that. Thanks!

gulp-clean-css 2.0.8 has been published with the 3.4.13 dep https://github.com/scniro/gulp-clean-css/commit/32909e1eed30a536175afd2aa7d26727877a025a


Export not working

Hi, I am not able to export my resume into pdf.
When I am trying to convert it is giving below file
RxResume-60cjet (2).pdf
.

The print functionality in https://v2.rxresu.me should be fixed now, deployed a new version of the cloud function to print.

Thanks @AmruthPillai

You can now import JSON files exported through https://v2.rxresu.me, as of v3.2.3 🎉


ADVANCED_OPTIMIZATION renaming appears to behave inconsistently

This code
var TestObject = function(message, duration, dismissType) {
  this['message'] = message;
  this['duration'] = duration;
  this['dismissType'] = dismissType;
};
var DismissTypes = {
  "AUTO_DISMISS": 'AUTO_DISMISS',
  "MANUAL": 'SWIPE'
};

var instance = new TestObject('my message', 1000, DismissTypes['AUTO_DISMISS']);

console.log(instance.message);
console.log(instance.duration);
console.log(instance.dismissType);

produces this compiled output:
var a=new function(b,c,d){this.message=b;this.duration=c;this.dismissType=d}("my message",1E3,"AUTO_DISMISS");console.log(a.message);console.log(a.duration);console.log(a.a);

why are message and duration preserved, but dismissType is not?  Is this correct?  I was unable to find an explanation for this behavior in the documentation.

It's covered in the FAQ
TL;DR message and duration are probably properties in externs, where dismissType is not.


Remove extended public key from contacts if we are not yet supporting it

If the xpub in the Contacts doesn't do anything we should remove it as it is confusing

Agreed. When we're able to support it it'll be trivial to reinstate it on the UI. No need to change the protobufs.

I've removed the xpub from the UI (but not the protobuf nor the i18n).
Awaiting review and closing

Verified absence of xpub field.


✨ StateMachine with std::variant and eventFlags

All different tools used does not concluded quickly to a satisfying solution. Aborting this PR.


No syntax highlighting in code blocks

@mc-marcocheng, Nice that you noticed the issue regarding no syntax highlighting in blocks of code. I'll validate and see if it can improve.
If you want, feel free to collaborate. :D

Hi @Ymil, about this implementation, check if I'm ok, this coding, I implemented a default mode for the highlight and also a new configuration to include the file.css for importing another highlight style.

The configuration for a customizable style is highlight_css:
theme:
  name: dracula
  logo: assets/img/dracula.png
  favicon: assets/img/favicon.ico
  highlight_css: atom-one-dark.min.css

To test, just use this command:

pip install -i https://test.pypi.org/simple/ mkdocs-dracula-theme==1.0.7.dev3

mkdocs-dracula-theme 1.0.7.dev3

HI @FernandoCelmer, this ok for me.
Great solution, sorry, I didn't know the Dracula theme for Highlight.js exists.

@Ymil, Thanks!


Brooke-harvey-blog-international

Trello card
https://trello.com/c/aYTK18Dc/3066-international-teacher-case-study-interview
Context
Changes proposed in this pull request
Guidance to review

@peteryates I tried to add a thumbnail image but it didn't seem to like it and said it couldn't find it when I put it into the frontmatter. It has also chopped off the top of her head 😣 could you help pls? 😅

Yep will take a look this afternoon

@peteryates the picture on the blog page is chopping the top of her head off, can we fix it?

@EllieNodder - done!
The image will be placed in the centre of the box so things near the edge of the frame might get chopped. All I did to correct it here was to re-crop so her face is more central.

@gemmadallmandfe @valentine-carter Brooke has got back saying she's happy with the blog - as is Laura - so I'm happy for this to be merged if you two are


build failure when module if __name__ == "__main__"

The __main__ guarded code was calling a function defined above in the module, and for some reason this was leading to the MSVC compiler exiting with code -2 (using the -v flag did not show additional information)

Please provide an example code / python package that helps reproducing the error


Add Wire authz and challenges (OIDC+DPOP)

@beltram, at some point one of us needs to stomp on the merge button. I think that's long overdue and it would be nice to have a better separation of changes by PRs. Especially now that more eyes will have a look at the code.


Placing down Glasses terminal crashes client and dupes item

When placing down a Glasses Terminal my client will get kicked from server (Internal server error) server says http://pastebin.com/hnXHXNqs
I tried with forge, same error with a forge in place of cauldron error. Also says "[02:18:15 WARN]: Selector.select() returned prematurely 512 times in a row; rebuilding selector."
When you log back in you have the glasses terminal in your inventory along with the one you placed before crashing.
Running OpenGlasses v1.0.43 and OpenComputers v<IP_ADDRESS>
Side Note: I can access the Glasses terminal via computer just fine.

Try new build (45)[http://starchasers.pl:8080/job/OpenGlasses/45/] from jenkins.
We didn't and won't check Bukkit compatibility.

awesome! Thank you so much, will implement right now. Any simple examples for AR Glasses to put things on the screen 2D? I'll try to update wiki once I learn it and get more acquainted with it.

@shortybsd I have to redirect you to @marcin212.

Thanks again @Kubuxu! Works great, cauldron is ran more then forge servers. I would convert over to forge but due to instability, performance issues and have a few plugins I wrote that would have to be re-written. Thanks again for super quick fix and push.

Here is a quick demo I wrote for OpenGlasses, feel free to use it on the web site - https://gitlab.com/shortybsd/OpenComputers-Shared/blob/master/OGDemo.lua


[DO NOT MERGE] Remove sidekiq and update dependencies

Should not be merged until https://github.com/alphagov/govuk-puppet/pull/7293 is deployed to production.
This removes sidekiq as it's no longer used and does all around maintenance to the app by fixing deprecation warnings, making Gemfile consistent and updating dependencies.
/cc @h-lame

@h-lame Yeah - I've not written it down somewhere. I'm not really sure where to put it.
Currently it should be virtually impossible to trigger that error as Rails itself will raise it's own action dispatch param error (which converts to a 400) if we send malformed JSON in. So it looks like the controllers/routes need refactoring to a more modern rails param based approach and restrict non-json requests. This seems a somewhat tedious, low priority low value task so not sure where to even record it :-/

Fair enough, probably a more general case of an error that we might want to think about across all our apps.  I don't know where I'd write that down either.


提示要登录

Error:
UNAUTHORIZED
Please login at https://chat.openai.com/ first
And refresh this page or type you question again
Consider creating an api key at https://platform.openai.com/account/api-keys

需要登陆或者有key, 或者自建


Create 0000-01-02-Dankvardt.md

I created a new branch as told

I created a new branch as told

I created a new branch as told

Created a pull request

I created a new branch as told

Stil learning


Why your PCA components have 512 dimensions rather than 18*512 in Stylegan2 ?

I am really confused for this question. Is there anyone can answer it?

Because the authors did the PCA experiment on every one of those 18 groups of W, other than did them all once.


Training problem

When I want to train HQ_SAM by instruction,I get the error "HQ-SAM: error: unrecognized arguments: --local-rank=0
ERROR:torch.distributed.elastic.multiprocessing.api:failed (exitcode: 2) local_rank: 0 (pid: 1468) of binary: /usr/bin/python3" in colab.how to resolve it?

Hi, this issue might help you. It looks like a PyTorch version problem. You could use torchrun or install a lower version of torch.

I solved it by using torchrun instead of distributed

I solved it by using torchrun instead of distributed


Model & Extension update

The gitignore is to hide some of my tools.

I think this blocked because the .gitignore file in the main folder 🤔 thanks  ❤


Update docs for --namespace and --context-namespace

See https://github.com/dotnet/efcore/issues/9723

Also connection strings for update-database. https://github.com/dotnet/efcore/issues/10750


Trim command not working on multi part archive?

I just tried to trim some incomplete archive out of one multi part archive. However it always spawn "so sorry, this seems a multipart archive, abort" then quit. Does trim command currently not support multi part archive and does it gonna support in near future?

Trim does not support multi part archive and will not support anytime soon
Because you can directly delete the last part of the archive (with the incomplete transaction)
That's all

TRIM works with ONLY one single .zpaq file (no multipart)

Yes manual is one of solution. But generally incomplete transaction could happen without knowing (daily backup let say).
And pretty complex to determined which is broken batch (its not always last file but last batch of files, correct?), so I guess automatic cleanup is necessary. In this case is decided which batch of file is contain incomplete transaction, clean it.

In reality it essentially never happens, except by “treacherous” process interruption (if you press control-C it is intercepted and the last piece deleted). Deleting an incomplete transaction, in a sequence of files, is complex and fragile; there is no concept of a “chunk” in zpaq
Usually my advice is to use the backup (and testbackup) command in the case of multi-archives


Update composer.json

Pull Request Test Coverage Report for Build<PHONE_NUMBER>8
Warning: This coverage report may be inaccurate.
This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.

For more information on this, see Tracking coverage changes with pull request builds.
To avoid this issue with future PRs, see these Recommended CI Configurations.
For a quick fix, rebase this PR at GitHub. Your next report should be accurate.

Details

0 of 0   changed or added relevant lines in 0 files are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage decreased (-0.02%) to 78.016%

Files with Coverage Reduction
New Missed Lines
%

src/Tempest/Database/src/GenericDatabase.php
1
90.91%

Totals

Change from base Build<PHONE_NUMBER>6:
-0.02%

Covered Lines:
4372

Relevant Lines:
5604

💛  - Coveralls


Event handler props' types fail on custom component in .tsx file.

Version
3.0.5
Reproduction link
https://codesandbox.io/s/gifted-minsky-w5dxn?file=/src/index.tsx
with ts config
{
  "compilerOptions": {
    "strict": true,
    "module": "commonjs",
    "jsx": "react",
    "jsxFactory": "h",
    "jsxFragmentFactory": "Fragment",
    "esModuleInterop": true,
    "sourceMap": true,
    "allowJs": true,
    "lib": [
      "es6",
      "dom"
    ],
    "rootDir": "src",
    "moduleResolution": "node"
  }
}

Steps to reproduce
Follow the link.
What is expected?
<Comp onClick={() => {}} /> shouldn't have type error.
What is actually happening?
has type error

Vue denies unknown props by default, you can change this behavior through ComponentCustomProps:
declare module 'vue' {
  interface ComponentCustomProps {
    custom?: number
  }
}

Check this file: https://github.com/vuejs/vue-next/blob/master/test-dts/componentTypeExtensions.test-d.tsx , you can get more examples.

Vue denies unknown props by default, you can change this behavior through ComponentCustomProps:
declare module 'vue' {
  interface ComponentCustomProps {
    custom?: number
  }
}

Check this file: https://github.com/vuejs/vue-next/blob/master/test-dts/componentTypeExtensions.test-d.tsx , you can get more examples.

https://github.com/vuejs/vue-next/blob/master/test-dts/componentTypeExtensions.test-d.tsx

I wonder why it works in raw h function but not in tsx. @HcySunYang

https://github.com/vuejs/vue-next/blob/master/test-dts/componentTypeExtensions.test-d.tsx

I wonder why it works in raw h function but not in tsx. @HcySunYang

The reason is for type-safety, see https://github.com/vuejs/vue-next/issues/1519.

I wonder why it works in raw h function but not in tsx. Could you please explain what does the transformation do?

As I currently know, h doesn't deny unknown props by default, which does cause inconsistent behavior, so for this reason I reopen it for subsequent discussion.

The reason is for type-safety, see https://github.com/vuejs/vue-next/issues/1519.

I wonder why it works in raw h function but not in tsx. Could you please explain what does the transformation do?

As I currently know, h doesn't deny unknown props by default, which does cause inconsistent behavior, so for this reason I reopen it for subsequent discussion.

In my original recognition I thought tsx just do a raw transformation from
 to h('div', { onClick: null }) and do the identical type checking. However, currently it works differently in typescript (when building & editing). What's the reason of the behavior?

In my original recognition I thought tsx just do a raw transformation from
 to h('div', { onClick: null }) and do the identical type checking. However, currently it works differently in typescript (when building & editing). What's the reason of the behavior?

TSX has different type checking rules than h, here is about jsx type checking: https://www.typescriptlang.org/docs/handbook/jsx.html. For h, it's just a function call and has nothing to do with the type checking of jsx

TSX has different type checking rules than h, here is about jsx type checking: https://www.typescriptlang.org/docs/handbook/jsx.html. For h, it's just a function call and has nothing to do with the type checking of jsx

Thanks! And I find it's wired to implement v-model in tsx. To make props like onUpdate:value work, I need to write <Comp {...{ 'onUpdate:value': null }} />.

I think it make it hard to build a library for both tsx & template usage. Is there any better workaround to solve the problem?
Is it possible to treat onUpdateXxx as onUpdate:xxx  in vue? Just like xmlns:xlink in react.

Thanks! And I find it's wired to implement v-model in tsx. To make props like onUpdate:value work, I need to write <Comp {...{ 'onUpdate:value': null }} />.

I think it make it hard to build a library for both tsx & template usage. Is there any better workaround to solve the problem?
Is it possible to treat onUpdateXxx as onUpdate:xxx  in vue? Just like xmlns:xlink in react.

You can use https://github.com/vuejs/jsx-next
And, I close this issue. If you think the type checking between h and jsx still bothers you (inconsistent behavior), then you can open a new issue about that.

You can use https://github.com/vuejs/jsx-next
And, I close this issue. If you think the type checking between h and jsx still bothers you (inconsistent behavior), then you can open a new issue about that.

import * as RuntimeCore from "@vue/runtime-core";
import { number } from "echarts/lib/export";

type ReservedProps = {
  key?: string | number
  ref?:
    | string
    | RuntimeCore.Ref
    | ((ref: Element | RuntimeCore.ComponentInternalInstance | null) => void)
}

type ElementAttrs<T> = T & ReservedProps

type NativeElements = {
  [K in StringKeyOf<IntrinsicElementAttributes>]: ElementAttrs<IntrinsicElementAttributes[K]>
}

type JsxAttributes = {
  id?: string | number
} & ReservedProps & EventHandlers<Events>

declare global {
  namespace JSX {
    interface Element {
    }

    interface ElementClass {
      $props: {}
    }

    interface ElementAttributesProperty {
      $props: {}
    }

    interface IntrinsicElements extends NativeElements {
      // allow arbitrary elements
      // @ts-ignore suppress ts:2374 = Duplicate string index signature.
      [name: string]: any
    }

    interface IntrinsicAttributes extends JsxAttributes {
    }
  }
}

I submitted it before and it was rejected

import * as RuntimeCore from "@vue/runtime-core";
import { number } from "echarts/lib/export";

type ReservedProps = {
  key?: string | number
  ref?:
    | string
    | RuntimeCore.Ref
    | ((ref: Element | RuntimeCore.ComponentInternalInstance | null) => void)
}

type ElementAttrs<T> = T & ReservedProps

type NativeElements = {
  [K in StringKeyOf<IntrinsicElementAttributes>]: ElementAttrs<IntrinsicElementAttributes[K]>
}

type JsxAttributes = {
  id?: string | number
} & ReservedProps & EventHandlers<Events>

declare global {
  namespace JSX {
    interface Element {
    }

    interface ElementClass {
      $props: {}
    }

    interface ElementAttributesProperty {
      $props: {}
    }

    interface IntrinsicElements extends NativeElements {
      // allow arbitrary elements
      // @ts-ignore suppress ts:2374 = Duplicate string index signature.
      [name: string]: any
    }

    interface IntrinsicAttributes extends JsxAttributes {
    }
  }
}

I submitted it before and it was rejected


Cordova actions

Adds actions to retrieve the app name and identifier from the cordova config.xml file on a cordovoa project
some dicussion was done here https://github.com/fastlane/fastlane/pull/984

we are using this actions in our base configuration for cordova here https://github.com/platanus/fastlane-cordova

@ohwutup where is the PR we talked about

I've just rebased the PR against master.

Sorry for not coming back to your earlier. Thanks for updating the PR, this looks great. Could you just fix the merge conflicts, then we can merge the changes 👍

hey @KrauseFx no problem, I already fix the merge conflicts..

Hey @blackjid, first off, sorry for the delayed reply. We're currently working on a plugin system, so that you can maintain and distribute this action without us having to review and approve every change.
You can read more about why over here https://github.com/fastlane/fastlane/issues/4744
and here's the pull request with the code we currently have https://github.com/fastlane/fastlane/pull/4722
I'd love your action to be one of the first available fastlane plugins. It will be listed on the fastlane main repo, so you get the same discoverability 👍
I'll update this issue with more information about how to wrap your action into plugin over the next few days, it's really easy to do that 🚀
Let me know if that sounds good to you!

Hey @blackjid, great news, the plugin system is ready to be used, I'd love your feedback on how well it works for you and how you like the approach: Get started with plugins  🚀

Alright, plugins were published today, please migrate this into a plugin. Let us know if you need any help doing so 👍


I added my name to the list

Hello Oyin!
Just one more step to go! 🙏
Please open a new pull request. This time create a new branch with your changes before you do!
Also, do not forget to add your name anywhere in the middle. Doing so will significantly reduce the chances of a merge conflict happening!
We look forward to merging your next pull request 😊


Input variable base_type doesn't get renamed properly

Here's another problematic shader. I'm calling
glslCompiler.set_name(resources.stage_inputs[0].base_type_id, "InputStruct");

on this SPIRV, because the InputStruct has been renamed by the debug stripping pass and Mac GL needs it be the same to be able to link shaders. I call it on stage_inputs[0] because it only ever has one input, which is a struct shared by vertex and pixel shader. For some reason glslCompiler.compile() doesn't replace the base_type. I don't know what's going on but it works on most shaders, it's just a few that don't work. A minimal example works correctly, so I need to send you the spv as it's input to glsl compiler by email. Unless I'm using the API wrong and there's some expectation on my side that doesn't necessarily need to be met.

Fixed in https://github.com/KhronosGroup/SPIRV-Cross/pull/237. Please check if it fixed your issues.

Cool waiting for this to merge. Will let you know as soon as you have it.

Ok I copy pasted the diff into latest code and I confirm it works. Thanks for all your support (I'm removing a big chunk of hack I had to work around it which makes me happy)


VS Code 1.76 starts all onTaskType extensions when debugging

Type: Bug

Open the "Remote Extension Host" output window
Start Debugging (F5)
Note that every extension with an onTaskType:something activation event activates.

Pre 1.76, no extensions with onTaskType would activate, unless you were actually running that task.

VS Code version: Code 1.76.0 (92da9481c0904c6adfe372c12da3b7748d74bdcb, 2023-03-01T10:22:44.506Z)
OS version: Windows_NT x64 10.0.23405
Modes:
Sandboxed: No
Connection to 'SSH: one_click' could not be established  Connecting with SSH timed out
Remote OS version: Windows_NT x64 10.0.22000

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-8700K CPU @ 3.70GHz (12 x 3696)

GPU Status
2d_canvas: enabledcanvas_oop_rasterization: disabled_offdirect_rendering_display_compositor: disabled_off_okgpu_compositing: enabledmultiple_raster_threads: enabled_onopengl: enabled_onrasterization: enabledraw_draw: disabled_off_okskia_renderer: enabled_onvideo_decode: enabledvideo_encode: enabledvulkan: disabled_offwebgl: enabledwebgl2: enabledwebgpu: disabled_off

Load (avg)
undefined

Memory (System)
31.78GB (14.22GB free)

Process Argv
--crash-reporter-id d2ee1b87-344a-4fdd-bf7b-2ab4787239c4

Screen Reader
no

VM
0%

Connection to 'SSH: one_click' could not be established  Connecting with SSH timed out

Item
Value

Remote
SSH: wave_analysis

OS
Windows_NT x64 10.0.22000

CPUs
Intel(R) Xeon(R) Platinum 8272CL CPU @ 2.60GHz (8 x 2594)

Memory (System)
32.00GB (24.03GB free)

VM
0%

Extensions (27)

Extension
Author (truncated)
Version

vscode-toggle-quotes
Bri
0.3.6

remotehub
Git
0.52.0

wavework
Mic
1.2023.216002

blazorwasm-companion
ms-
1.1.4

vscode-edge-devtools
ms-
2.1.1

remote-containers
ms-
0.275.1

remote-ssh
ms-
0.96.0

remote-ssh-edit
ms-
0.84.0

vscode-remote-extensionpack
ms-
0.24.0

azure-repos
ms-
0.28.0

remote-explorer
ms-
0.2.0

remote-repositories
ms-
0.30.0

remote-server
ms-
1.0.0

material-theme
zhu
3.15.6

midl3-language-server
Ale
0.0.31

vscode-eslint
dba
2.4.0

vscode-research
dev
1.2022.1111005

vulnerability-extension
dev
1.2023.217002

copilot
Git
1.76.9071

vscode-azdo-codereview
Mic
1.2022.722003

vscode-nmake-tools
Mic
4.0.220809006

wavespace-remote-companion
Mic
1.2022.317002

azure-pipelines
ms-
1.208.0

vscode-dotnet-runtime
ms-
1.6.0

sarif-viewer
MS-
3.3.5

azure-account
ms-
0.11.3

cpptools
ms-
1.14.4

(2 theme extensions excluded)

A/B Experiments
vsliv368cf:30146710
vsreu685:30147344
python383:30185418
vspor879:30202332
vspor708:30202333
vspor363:30204092
vswsl492cf:30256860
vslsvsres303:30308271
pythonvspyl392:30443607
vserr242cf:30382550
pythontb:30283811
vsjup518:30340749
pythonptprofiler:30281270
vshan820:30294714
vstes263:30335439
vscorecescf:30445987
pythondataviewer:30285071
vscod805cf:30301675
binariesv615:30325510
bridge0708:30335490
bridge0723:30353136
cmake_vspar411:30581797
vsaa593:30376534
pythonvs932:30410667
cppdebug:30492333
vsclangdc:30486549
c4g48928:30535728
dsvsc012:30540252
pynewvextcfv2:30669238
azure-dev_surveyone:30548225
pyindex848:30662994
nodejswelcome1cf:30587006
2e4cg342:30602488
pyind779:30671433
f6dab269:30613381
pythonsymbol12:30671437
showlangstatbar:30672706

This also occurs when calling Run Build Task...

Probably caused by https://github.com/microsoft/vscode/issues/172065

Yeah, seems like #172065 is suspicious, because I don't see onTaskType: defined for the extension that made me discover this bug...
However, even if it was defined, VS Code still shouldn't be activating every single onTaskType extension on a normal debug session, right?

/duplicate https://github.com/microsoft/vscode/issues/176006

When will this be fixed? It's really annoying to have random extensions start up when I launch a Live Share session.

@meganrogge I'm not sure this is a duplicate. I'll use WAVE as the example here just because I happen to be on the same team as #176006.
Pre 1.76.0, debugging a TypeScript VS Code extension would not activate the WAVE extension.
On 1.76.0 and 1.76.1, debugging that same extension activates WAVE.
While #176248 and 1.76.1 fixed the hanging issue reported in #176006, I still think it's unexpected (and incorrect) behavior that debugging a TypeScript codebase activates an extension exclusively meant for Windows C++ developers.
The rust-analyzer extension also seems to be getting bug reports about this same unexpected behavior:

rust-lang/rust-analyzer#14231
rust-lang/rust-analyzer#14253

I'm seeing this in 1.76.2 and build tasks never seem to finish fetching:

When you debug, if there's a prelaunch task, we will getTask -> tryResolveTask -> _activateTaskProviders(type)
There, we wait for all extensions to be registered. In 1.76, we now implicitly activate onTaskType, thus what you are seeing.
https://github.com/microsoft/vscode/blob/6b85f1cfcf849e4a50185addc769a084a4c30eaf/src/vs/workbench/contrib/tasks/browser/abstractTaskService.ts#L575-L577
The fix would be to revert that change or to not wait for installed extensions to be registered, but that would break other things. @alexdima looks like you added that here. https://github.com/microsoft/vscode/commit/51d7dd1a7319cca932d2db8bb6560e2fc8dc3c03#diff-f77e08d48fa5a43f6cd385c0bac2c04ee2123565ead5f0f4b60e23c8af821b76R511-R514
do you have thoughts related to this?

still think it's unexpected (and incorrect) behavior that debugging a TypeScript codebase activates an extension exclusively meant for Windows C++ developers.

@meganrogge I don't remember the specifics, I'm just reading the code:
	private _getActivationEvents(type: string | undefined): string[] {
 		const result: string[] = [];
 		result.push('onCommand:workbench.action.tasks.runTask');
 		if (type) {
 			// send a specific activation event for this task type
 			result.push(`onTaskType:${type}`);
 		} else {
 			// send activation events for all task types
 			for (const definition of TaskDefinitionRegistry.all()) {
 				result.push(`onTaskType:${definition.taskType}`);
 			}
 		}
 		return result;
 	}

 	private async _activateTaskProviders(type: string | undefined): Promise<void> {
 		// We need to first wait for extensions to be registered because we might read
 		// the `TaskDefinitionRegistry` in case `type` is `undefined`
 		await this.extensionService.whenInstalledExtensionsRegistered();

 		await Promise.all(
 			this._getActivationEvents(type).map(activationEvent => this.extensionService.activateByEvent(activationEvent))
 		);
 	}

Is it possible that we call _activateTaskProviders(undefined)? That seems to result in all task providers getting activated. Do you know why type would be undefined?

Aha, here's a clue. It looks like this only happens when trying to debug something that depends on a default build task.
In safe mode (--disable-extensions) on 1.76.2, I see the following behavior:

Running a launch configuration with a preLaunch task of ${defaultBuildTask} does start all extensions that contribute tasks
Running a launch configuration that specifies an explicit preLaunch task does NOT start task extensions (other than the expected ones)

I can provide both my launch.json & tasks.json if necessary, though they're both pretty standard. (defaultBuildTask goes to a task of type npm and script compile:dev)
https://user-images.githubusercontent.com/2766036/225780360-3c60f107-72d5-490a-97e1-8e0cb5ec0eb3.mp4

Is it possible that we call _activateTaskProviders(undefined)?

The configuration resolver service for defaultBuildTask calls _getTasksForGroup(TaskGroup.Build) which calls _getGroupedTasks() (notice NO argument) which calls _activateTaskProviders(undefined) which activates all task provider extensions.
https://github.com/microsoft/vscode/blob/143d94ae0dfed656d43fe211ffb7c7d69a27f87f/src/vs/workbench/contrib/tasks/browser/abstractTaskService.ts#L305-L306 ⬇
https://github.com/microsoft/vscode/blob/143d94ae0dfed656d43fe211ffb7c7d69a27f87f/src/vs/workbench/contrib/tasks/browser/abstractTaskService.ts#L1374-L1375 ⬇
https://github.com/microsoft/vscode/blob/143d94ae0dfed656d43fe211ffb7c7d69a27f87f/src/vs/workbench/contrib/tasks/browser/abstractTaskService.ts#L1960-L1963
Thanks to this tip in your wiki that mentioned being able to debug release versions of VS Code using the builtin dev tools! :D

Any update on a fix coming? This bug makes Live Share basically unusable as it will launch unrelated extensions unless you uninstall every extension with an onTaskType.

Thanks for the clarification! I just have two more questions then:

Is it expected behavior that starting a launch configuration that has a preLaunchTask of ${defaultBuildTask} will activate all extensions with an (explicit or implicitly added) onTaskType activation event?
If it isn't expected, can this bug be reopened? If it is expected, would you be open to accepting a PR that tries to change the behavior back to pre-1.76 behavior without affecting the new implicit onTaskType events? (I don't know how feasible this is, but just putting it out there)

Yes, #1 is expected behavior

Digged in the sourcecode an managed to find the culprit and a workaround (for now).
The workaround is to set the setting task.saveBeforeRun to never.
The bug exists because AbstractTaskService._executeTask calls _saveBeforeRun,
and if editors where saved, it tries to reload the the task by calling AbstractTaskService.getTask:
https://github.com/Mai-Lapyst/vscode/blob/0d5e948d863153807cfe07ecd20dc10374eea6d2/src/vs/workbench/contrib/tasks/browser/abstractTaskService.ts#L1832-L1850
The getTask method, first trys to search the task inside the workspace and if that files calls _getGroupedTasks():
https://github.com/Mai-Lapyst/vscode/blob/0d5e948d863153807cfe07ecd20dc10374eea6d2/src/vs/workbench/contrib/tasks/browser/abstractTaskService.ts#L765
_getGroupedTasks normaly takes one parameter, a filter, to only activate a certain type of task providers:
https://github.com/Mai-Lapyst/vscode/blob/0d5e948d863153807cfe07ecd20dc10374eea6d2/src/vs/workbench/contrib/tasks/browser/abstractTaskService.ts#L1960-L1963

Opened a PR that fixes this issue.

Re

As far as I can tell, this is how it has always worked

While this is true, it would be good if we can improve upon it. Thanks for the PR

to verify, have an extension with activation event onTaskType:X set. Before, it would get activated immediately upon invoking Tasks: run task. Verify that now it only gets activated if you select a task of type X.

Appreciate the fix @meganrogge! Thank you. I've been confused for a few weeks why rust-analyzer kept trying to start in my frontend-only projects.


M3-207 - Checkbox Component UI Test

Added storybook webdriverIO configuration file (should be separate config from the standard wdio manager config since different spec folder paths, baseUrls, pre-test setup hooks, etc). Inherit everything from wdio.conf.js and override specific config values.
Added jsClick and jsClickAll browser custom commands, useful when attempting to click an element throws a selenium error like "other element would receive click"
Bumped chromedriver version to latest.
Added storybook homepage route to constants.js
Created test suite for Checkbox UI Component
Added data-qa attribute to checkbox component
Added storybook:e2e yarn script.
Updated docker scripts to remove containers after they exit, and to no longer hardcode the container names.

NOTE:  i made the component ui test specs live inside each component folder. I think this sort of test (especially since they can be run very quickly) should live longside the component in the code base. Since it is not typescript though, it will not be linted via yarn lint. I can configure eslint in a branch to lint e2e tests contained in here though
To Test:

Start storybook: yarn docker:storybook
(New shell) Start selenium: yarn selenium
(New shell) Run the tests: yarn storybook:e2e

Looks great, tests are easy to run. A few requested changes.

@patthiel is this still in progress?

@rjstires i made the changes @asauber requested and changed how to the check all enabled checkboxes test works.
I also added tests attempting to check disabled checkboxes per @asauber feedback as well. Otherwise this is good to go

Too @asauber for the final 👍then.


feat(app,robot-server): Subscribe to currentlyRecoveringFrom notifications

Overview
Closes EXEC-458.
Test plan

Push robot-server and ODD to a Flex
Induce a pickUpTip failure
Make sure the ODD goes into recovery mode when the pickUpTip failure happens
Make sure both recovery options work

And maybe check the app or server logs to make sure it doesn't spam network requests, especially when notifications are unavailable.
Changelog

Generalize the ...current_command MQTT topic, which was "the equivalent of GET /runs/:runId/commands?cursor=null&pageLength=1, to ...command_links, which is the equivalent of the links field of any GET /runs/:runId/commands query.
Update the frontend to take advantage of that. When it sees links update with a new currentlyRecoveringFrom command ID, it will do a separate GET /runs/{id}/commands/{id} query to fetch the full details of the command.

Review requests
Several frontend questions because I don't know what I'm doing there. See my GitHub comments below.
Risk assessment
Probably at least a medium risk of breaking recent unreleased work (#15170). Although we can cover these notifications hooks with tests, it doesn't look like those tests are in a good position to make sure the UI is actually doing the correct thing overall.

I'll probably test this on Monday and merge it then, assuming all goes well.


Request header too large

I create a request using the Dataverse ERD v2 prompt. The max token input is 4097 for GPT 4 1106 preview as per the Open AI documentation.
I believe the problem is to do with the large body coming back from Open AI API.

Don't use gpt-4-1106-preview for requests


How do I generate pure c bindings.

Brief Description
Is there a built in way to generate pure c bindings?
I found this issue #158 from back in 2014. Was a generator added?
I tried options.GeneratorKind = GeneratorKind.C but that crashes with a System.NullReferenceException.
I am using the nuget package at the moment so I can't easily debug it.
Thanks!

Hey @ChrisDill.
We never got around to adding C support to CppSharp.
You can try this fork https://github.com/InitialForce/CppSharp, which I think provides C support.


fix: Running Code coverage job

Test Plan

How do we know the code works?

Code coverage runs after making coverage report

@Mergifyio rebase


Framerate slow down with continuous GUI update on Android

Repro

Clone space Demo
npm install and npm run dev
open the demo on Android with Chrome
start a mission
when touching the thumb continuously, the framerate drops from 45FPS to 25FPs

Smartphone (please complete the following information):

Only Android (tested on Oppo Reno 2Z, Android 11). iOS is fine.

Additional context
I did a quick profiler and GUI update takes 8ms while touch event takes 2.5ms.

Not fixed. Reopening.