Magnifier : Clicking on the voice combo box inside settings flyout often closes the settings flyout.

Describe the bug

Steps to reproduce the bug

Steps to reproduce the behavior:

Launch "Magnifier" application.
Activate settings button.
Click anywhere except on interactive elements i.e. just on the background
Now activate the voice combo box and observe.

Expected behavior

After clicking on the voice combo box, it should expand and user should be able to access it.
Screenshots

Additional context

Copied from internal bug
Comment in the original bug:

Doesn't seem like a Magnifier issue. Magnifier does not do anything when the combo box gets opened.

Also, the only path of reproing this is clicking on the "background" or label of the flyout and then on the combo box. If you clicked on the slider before interacting with the combo box (even if you clicked on the background before interacting with the slider), the combo box would not dismiss.

This is the same as #1467 I believe. Its turned into a real problem recently especially with custom controls. Currently there is no known work-around.

@kaiguo This is likely a dup. Can you please take a look at this repro when you have a fix for the other bug ? Thanks


chore: update solidity version

Changes

Update the Solidity version to 0.7.6.
Update linter to work with newer version of solidity

Notes
I couldn't push the solidity version up to 0.8.0 because OpenGSN has some issues with the latest version of OpenZeppelin and I had to fallback on OpenZeppelin v3.4.1.😕  Let me know if there are other ways to getting around this that I should also try.

✅
Totally forgotten about this PR already.🤣


Question: changing the order of validation for enums

Enum validation fails before applying custom keyword compilation in AJV v8.x
ajv.addKeyword({
  keyword: 'upperCase',
  modifying: true,
  type: 'string',
  compile(sch) {
    if (sch) {
      return (data, { parentData, parentDataProperty }) => {
        parentData[parentDataProperty] = data.toUpperCase();
        return true;
      };
    }
    return false;
  },
  errors: false,
  metaSchema: {
    type: 'boolean',
  },
});

const schema = {
    type: "object",
    properties: {
        "prop1": {
            type: 'string',
            enum: ['A', 'B', 'C'],
            upperCase: true, // custom keyword
        }
    }
}

const validator = ajv.compile(schema)
const isValid = validator.validate({ prop1: 'a' })
console.log(validator.errors)

output:
0: {
instancePath: ...,
schemaPath: ...,
keyword: 'enum',
params: { allowedValues: ['A', 'B', 'C'] },
message: 'must be equal to one of the allowed values'
}

There were no errors in AJV v6.x, this is related to change in order validation in AJV introduced in ajv v7.x
Custom keyword compilation/transformation is happening after enum validation that is why it is failing, any work arounds? Thank you.
What version of Ajv you are you using?
v8.x
What problem do you want to solve?
What do you think is the correct solution to problem?
Will you be able to implement it?

Hello @bios1337 ! If the 'enum' variable consists of upperCase 'A' and if we pass lowerCase 'a' in prop1 then it will give error. Hence the message is that prop1 value must be equal to one of allowed values in enum. I am not able to understand your exact issue.

to control validation order you need to use allOf - in this case your custom keyword will be in the first branch and enum in the second.


SAK-48324 Calendar: TypeError: this.i18n is undefined

https://sakaiproject.atlassian.net/browse/SAK-48324
Note: This fix should work in all browsers, the reason it shows an error in Firefox but not in Chrome because Firefox doesn't support template literals inside of ${...} expressions within template strings in some versions.

closing this one, we have a new pr: https://github.com/sakaiproject/sakai/pull/11895


[LLC] RequestOptions into Azure.Core

At some point RequestOptions will move from Experimental to Core, and we need to update for that.

=> Need to define what needs to happen to make this move
Any code that depends on RequestOptions will need this in order to GA
=> RequestOptions needs an arch board review
=> ResponseClassifier API needs to be resolved
=> Need to know there won't be breaking changes to this in order to release RC1.0


Cannot use oe-gdb to debug enclave code in simulation mode

Repro steps(Please not I could successfully use the same steps to debug on actual SGX hardware, issue only manifests in simulation mode)
1.
oe-gdb -args host/helloworldhost ./enc/helloworldenc.signed.so
(gdb) break enclave_helloworld
(gdb) run
Program runs without hitting breakpoint in enclave

oe-gdb -args host/helloworldhost ./enc/helloworldenc.signed.so
(gdb) break entersim.S:48
(gdb) break enclave_helloworld
(gdb) run
once break point is hit, start using next until you go to jmp *%rdx   in  .call_start. oe-gdb says "Cannot find bounds of current function"
0x00007ffff6002910 in ?? (
(gdb) n
Cannot find bounds of current function
Program completes execution without hitting breakpoint in enclave

Implemented by #1099


unicorn ignoring config when programmatically creating items or publishing

I have unicorn 2.0.4 set up in sitecore 7.2, the config is standard but i have added a few extra include and exclude rules. unicorn seems to ignore the exclude rules when i publish an item or create it programmatically. if i just create an item in content editor it doesnt get created

I've retested the exclusion part of this issue and am unable to reproduce it in several different scenario tries. It's worth noting that Sitecore 8 defaults to having schedule items in the database like @RaYell mentioned above that change their last run field. I've got those excluded in several production or in-development Sitecore 8 sites without issue.
I've also tried programmatically creating items in excluded locations:
    using (new SecurityDisabler())
    {
	  var db = Factory.GetDatabase("master");

	  var item = db.GetItem("/sitecore/system/publishing targets");

	  item.Add("cats", new TemplateID(TemplateIDs.Alias));
    }

No items were written either time.
I'll see if I can replicate the publishing piece.

Ok I tested using the exact publishing code from two comments ago, and I can't replicate that issue either. I tried publishing both with a root item that is excluded and one that is included, and no web database serialization occurred.
My best guess is that there's some kind of event handler, pipeline processor, etc that is doing something unexpected to the way Sitecore works. But without reproduction, I'm going to close this for now.


Not possible to be used with useAsyncData?

I use the exact same config as in the README, and attempted to fetch the API from useAsyncData using $fetch like so:
const { data } = useAsyncData('todos', () => $fetch(`${baseUrl}/api/todos`), { transform: ({ data }) => data })

But the page is just stuck at pending. Using it with useFetch seems to be working fine, though. It's particularly weird since the Nuxt's doc mentioned that useFetch is just a wrapper around useAsyncData and $fetch.
It also seems to be working fine when $fetch is used in CSR (i.e., onMounted).

same problem. i did some research and realized that this is the problem comes from nitro/h3 server or nuxt internal request handling. it somehow just stuck when i tried to proxy any request.
i tried http-proxy http-proxy-middleware @refactorjs/http-proxy @nuxtjs-alt/proxy nuxt-proxy and event pure realization.
all same: it stuck with await $fetch('/api/qwe') but works well with await $fetch('/api/qwe', {server: false}) (because this request invokes in browser), also just http://localhost:3000/api/qwe works good.
so, again, its only problem around SSR internal url/request handling

its possibly somewhere around creating and using h3App.nodeHandler and creating theirs own server side fetch which uses localCall to mimic regular http request to h3 server
https://github.com/unjs/nitro/blob/main/src/runtime/app.ts#L60-L61

useAsyncData works perfectly fine. Check the playground.


configure.ac: Fix implicit function declaration warning for -march=we…

gcc/clang will complain about an implicit declaration warning when checking if '-march=westmere' works:
conftest.c: In function 'main':
conftest.c:56:21: error: implicit declaration of function '_mm_popcnt_64'; did you mean '_mm_popcnt_u64'? [-Wimplicit-function-declaration]
   56 |   uint64_t popcnt = _mm_popcnt_64((uint64_t)argc);
      |                     ^~~~~~~~~~~~~
      |                     _mm_popcnt_u64
configure:3888: $? = 1

Replace '_mm_popcnt_64' with the correct '_mm_popcnt_u64' function name to resolve this.
Fixes: 380abdb96de4 ("Separate Autoconf and CMake compiler tests")

Thanks @Kumba42. Much appreciated :+1:


Set up IAM Draft Guide (Step #1)

Set up a very manual step-by-step guide, just like Matt did for EFS. These will make it really easy to write the CloudFormation templates later.
This guide will include stuff like creating a non-root main account, turning on MFA, creating other users, and creating the KMS keys.

w00t

Okay, just to get this rolling, can I use the following at the top of the guide:

## 🚴 Getting Started

To start things off, let’s create an AWS account and set you up as an administrative user.

1. Navigate to [https://aws.amazon.com/](https://aws.amazon.com/), and then choose **Create an AWS Account**.
2. Follow along with the signup wizard.
3. Now that you are logged in, click **Services** and then choose **IAM**.
4. In the left pane, click **Users**.
5. Click **Add user**.
6. Under **Set user details**, enter your username in the **User name** field.
7. Under **Select AWS access type**, select **Programmatic access** in the **Access type** area.
8. Click **Next: Permissions**.
9. Under **Set permissions for ...**, click the **Attach existing policies directly**  box.
10. With the table at the bottom of the page in view, select **AdministratorAccess** (will be the first row).
11. Click **Next: Review**.
12. Under **Review**, ensure all information reflects the above steps.
13. Click **Next: Create user**.

Ths work is covered in our new issue/task tracker at https://app.asana.com/0/346554951567668/list


For the trigger eventing to work new build only pipeline is required

These are the usage scenarios:

A pull request will do a build, but no push.  It will also update the github status check for the pull request. We will need:
i) a new build only pipeline.

Explanation of requirement:
Today we have a build and a deploy tekton task , and a build-deploy pipeline crd.
As a requirement of this ask from @mchenggit , we would need to create separate task and pipeline as below:

A build only task that will do just assembly and build of the git-source project.
A tekton pipeline build-only that will call the build task from point 1.

Steps I have tried:
I had to update build-task and create new one where i updated
Removed the `buildah-push` step that pushes the container image to container registry.
Removed the `outputs` section from the `build-task` yaml and created new build task called `build-only-task`.
Removed the `deploy-task` from the `build-deploy-pipeline` and created new `build-only-pipeline`.
Removed the `docker-image` from the resources from specs as well as from the tasks , under `build-only-pipeline`.
Removed the `docker-image` from the `manual-pipeline-run-template` file and created new one `manual-pipeline-run-build-only-template`.
Updated the `buildah bud` command in `build-only-task` where the input `-t`(imageName) was updated to any random image name instead of `${outputs.resources.docker-image.url}` which is not required and not provided anymore.

After updating with above changes, i ran the manual pipelinerun for nodejs-express and it was successful.

Pull request created, after @mchenggit confirmed that new pipeline works for this scenario
Pull request : https://github.com/kabanero-io/kabanero-pipelines/pull/103
We are using old Tekton 3. version of tekton for the files in this pull request. We are yet to move these pipelines to newest 4. version of the tekton.
We will do that once we test it with new tekton and ocp 4.2 cluster.

The changes for new tekton syntax is updatd and merged in pipelines repository after testing on openshift 4.2 cluster with updated tekton version . This can be marked as closed


Minor formatting of README.md.

thanks for this,
seems there are some issues with the travis build, though.

👍


Exec format error

ubuntu@hosting:~/docker/packwiz$ sudo docker-compose up --force-recreate
Recreating packwiz ... done
Attaching to packwiz
packwiz    | exec /app/packwiz: exec format error
docker-compose.yml:
version: '3'
services:
packwiz:
container_name: packwiz
networks:
- proxy
labels:
traefik.enable: true
traefik.http.routers.packwiz.entrypoints: https
image: getchoo/packwiz-serve:latest
volumes:
- ./packwiz:/data
ports:
- 2748:8080
restart: unless-stopped
networks:
proxy:
external: true

what is the architecture of the host? i'm assuming it's not x86_64, which this image is not compatible with currently

90ad9d652f009a53b57115c924446baf6f1d3b7b may have fixed this


Remove Meetup.com from Reach in News/Blog for Awario

What is the problem you are seeing? Please describe.
Meetup.com posts register as News/Blog when they are clearly not. This allows some to simply mention their app on an event and get the Reach score of the site.
How is this problem misaligned with goals of app mining?
The News/Blog category is meant to be true publications - Meetup.com is a great site, but should be considered a website. We should fix this until Awario adjusts their algorithms to classify it differently. Until then it would be an easy exploit for Reach that isn't really the Reach we want to measure as part of Blended Awareness.
What is the explicit recommendation you’re looking to propose?
Not count Meetup.com Reach
Describe your long term considerations in proposing this change. Please include the ways you can predict this recommendation could go wrong and possible ways mitigate.
Keeping Reach a score that is hard to game and doesn't have obvious holes to exploit
Additional context
This is a simple change that makes the Awario score ever tighter.

Moving forward with this, Meetup.com articles will not be counted in Reach for the next period (December run(


Need a way to combine independent generators

The usual way to compose generators is with flatMap and map, which covers most use cases, unless you're using something like Gen.listOfN to produce a series data.
Assuming a generator for tags:
val tagGen: Gen[Tag] =
  for {
    name <- Gen.identifier
    value <- Gen.alphaStr
  } yield Tag(name, value)

Or the equivalent:
val tagGen: Gen[Tag] = Gen.zip(Gen.identifier, Gen.alphaStr).map(Tag.tupled)

Generating a list using Gen.listOf(tagGen) tends to produce a high number of cases where all of the tag names are the same. As I understand it, this is because Gen.flatMap sequences the values and the full distribution is only really seen in the last Gen.
What I'm looking for, and have not seen how to do this is the docs or source, is something like this:
Gen.product(Gen.identifier, Gen.alphaStr).map(Tag.tupled)

Which would sample equally from Gens which are passed to it.

As I understand it, this is because Gen.flatMap sequences the values and the full distribution is only really seen in the last Gen.

I'm afraid you may be misunderstanding the semantics of generators.   You're confusion is probably caused by a defect in the underlying Gen.pick method of Scalacheck, see #333.  The fix isn't released, yet, though.  The Gen.identifier and Gen.alphaStr generators are implemented withGen.choose and that in turn uses Gen.pick.  That could explain the duplication and the poor distribution of values you're seeing.

That's possible, however the other noticeable place I've seen this behavior is with Gen.uuid, which uses Gen.choose[Long]. I've traced that one back and it doesn't look like it's using Gen.pick, though I could be wrong about that one.
I'll see if I can bang together a simplified test case for this over the weekend.

That is true that Gen.choose doesn't use Gen.pick, so it wouldn't suffer from the defect I mentioned earlier in this case.   I was mistaken.
It would be interesting to know if there was a distribution problem with of Gen.uuid or better Gen.choose[Long].  You should try to distill your issue in a test, for example:
    val pairGen: Gen[(Long, Char)] = for {
      x <- Gen.choose(0L, 1L)
      a <- Gen.choose('a', 'b')
    } yield {
      (x, a)
    }

    val pairsGen = Gen.listOfN(2, pairGen)

    property("pairsGen") = {
      Prop.forAll(pairsGen) { x =>
        Prop.collect(x.map(_.toString).mkString(" ")) {
          Prop.passed
        }
      }
    }

Would produce:
> Collected test data:
13% (0,a) (1,b)
8% (1,b) (1,a)
8% (0,b) (0,b)
7% (1,b) (0,a)
7% (0,b) (1,a)
7% (1,a) (0,a)
7% (0,a) (0,a)
6% (0,a) (1,a)
6% (0,b) (0,a)
6% (1,a) (1,a)
6% (0,b) (1,b)
5% (1,b) (1,b)
5% (0,a) (0,b)
4% (1,b) (0,b)
3% (1,a) (1,b)
2% (1,a) (0,b)

Sort of good news: I was unable to reproduce the issue, so Gen.uuid is likely fine.
If I see it again I'll try and pull out a minimal example and reopen, but for now I'm going to close.


upper/lower case handling in 'data-inputmask-*'

If trying to set inputmask property via html attribute 'data-inputmask-*' it does not work correctly if property name contains upper case characters e.g. insertMode.
See following jsfiddle: https://jsfiddle.net/wwLg6z35/
In my opinion it should work at least with lower case, i.e. 'data-inputmask-insertmode'.

@jmalkiewicz ,
All fixed ;-)

Super! :)

Capgemini Polska Sp. z o.o.,
Al. Jerozolimskie 96, 00-807 Warszawa,
Sąd Rejonowy dla M.ST. Warszawy, XII Wydział Gospodarczy Rejestrowy, nr KRS:<PHONE_NUMBER>,
NIP: 526-11-84-467,
Wysokość kapitału zakładowego: 16.403.320,00 zł.
This message contains information that may be privileged or confidential and is the property of the Capgemini Group. It is intended only for the person to whom it is addressed. If you are not the intended recipient, you are not authorized to read, print, retain, copy, disseminate, distribute, or use this message or any part thereof. If you receive this message in error, please notify the sender immediately and delete all copies of this message.

Super! :)


npm create tamagui@latest on select is printing list

Current Behavior

Expected Behavior
should be able to select item with out repeating content
Tamagui Version
latest

Platform (Web, iOS, Android)
cli

Reproduction
npm create tamagui@latest

System Info
No response

This happens when your terminal is too narrow, not sure how we fully fix but maybe other libraries are better? Open to input here.


Loading sessions with +workspace/load-session does not start from a clean session

Observed behavior
When loading a whole workspace session with +workspace/load-session, the loaded session is not loaded cleanly. That is, old workspaces remain. Is this intended behavior? Is there a way to open a new session and close all others?
Also, when the new loaded session shares names with any workspace in the old session, the order of the "tabs" different than what is in the session.
Expected behavior
When loading a new session, only workspaces of the new session should be shown. Or is this not intended behavior?

As of d979c46ea225fd3e68f351a41daf92b7bc9f74c7 and 8cb36afa21f81ffa9b86b3059b42798be681f6f7, this is resolved. Thanks for letting me know!


Add speed_scaling_factor msg and field in JointTrajectoryControllerState

This goes in line with https://github.com/ros-controls/ros2_controllers/pull/1191
Note: For backporting it is planned to not add the field in the JTCState message.

Note: For backporting it is planned to not add the field in the JTCState message.

you mean backporting for humble? we haven't branched off for iron here.

We can check if there is any change in the messages, if it's just adding a new field, I think it can be backported as it only causes ABI break


Set up blog posts with Jekyll

So we can write guides for being an awesome contributor and maintainer, as well as news and updates about Your First PR and it's events.

Hi! I'd love to work on this as my first PR.
From what I understand, you want to set up a blog for yourfirstpr.github.io right? Any Ideas of how you would like the posts to be displayed?
Also, I don't really know what needs-better-explination means. Can you shed a little light on this?
Thanks!

Hi brad!
So this website is built using Jekyll. What I want to know is how easy is it to do the following with Jekyll:

Keep the website as it is
When you go to youfirstpr.github.io/blog you see a list of posts
Posts take this format: youfirstpr.github.io//blog/name-of-post on the website
Blog posts are written in a /posts folder of this repository
Blog posts are written in markdown and converted to .html so they can be rendered on the website
Blog post can take the format 2015-12-10-your-first-pr.md

I know jekyll's strong point is to be a blogging platform, but write now all this repository does is serve static files.
I know that Hoodie manage to do this and would like to adopt a similar, if not simpler approach: https://github.com/hoodiehq/hood.ie
Does that make sense? Are you familiar with Jekyll?

It does make sense and I'm familiar enough with Jekyll to get started. I've cloned the project and will get started! I'll use the hoodie site for reference.

Awesome! Thank you for dedicating time to this little website :)

Hi Charlotte!
I've got a page with a list of the blog posts and a page for each post. I've hit everything on the list. The styling is minimal (read: basically non-existent).
I would love to keep going! What are your thoughts on perhaps using Sass?
Thanks for letting me work on this!

There's a little bug with the author name on the list of posts page. :-) I'm working on it.

@brad-j feel free to get a WIP PR in, can help out with the review if you do that :+1:
No need to use Sass here since it's so simple I think.

Wow @brad-j, this is phenomenal. Really and truly, thank you. I wish I could high five you.
What are your reasons for using Sass? I'm happy to consider it, I use Sass all the time, I just would like to check the WHY before we make any moves on that.
:sparkles: :sparkles: :sparkles: :sparkles: :sparkles: :sparkles: :sparkles: :sparkles:

And thank you @nickcolley for offering help too \o/

No real pragmatic reason I guess (if the site continues to grow I can see a pragmatic benefit). It's just easier for me to wrap my head around Sass (oddly enough). Being able to break things out into little 'component' type things makes sense to me.
With that said, this is the first project I've ever worked so I could very well be getting ahead of myself.

Hey @brad-j! I totally get what you mean. Right now, I'm going to say no as Sass is another barrier to entry for those who are unfamiliar with it. And due to the size of the project (so tiny!) we do not need it right now. I am totally going to revisit this in the future though :)

Hey @brad-j, hows it going? Absolutely no rush to get this done, but I'd love to see what you've come up with / help with anything.

Not much happening here, what's the status? Do you still need help? I use Jekyll for my site.

Quick update: Almost done with implementing this. Just tweaking styling a bit. Will write a quick first post to include in the PR.

This has been completed :)


[v8] Update to Stable version <IP_ADDRESS>

Update v8 to Stable version <IP_ADDRESS>

What does your PR fix?

Fixes #24510

Which triplets are supported/not supported? Have you updated the CI baseline?

Windows, Yes

Does your PR follow the maintainer guide?

Yes

If you are still working on the PR, open it as a Draft: https://github.blog/2019-02-14-introducing-draft-pull-requests/

I tested this on Windows 11, MSVC 2022, WinSDK 10.0.22000.194, seems the CI is still running on 10.0.19041.685.

I tested this on Windows 11, MSVC 2022, WinSDK 10.0.22000.194, seems the CI is still running on 10.0.19041.685.

Is this update requires WinSDK 10.0.22000.194?

I tested this on Windows 11, MSVC 2022, WinSDK 10.0.22000.194, seems the CI is still running on 10.0.19041.685.

Is this update requires WinSDK 10.0.22000.194?

I think so

cc @BillyONeal

Do you know if that SDK version needs to be installed to the exclusion of others or can we just add it?

Do you know if that SDK version needs to be installed to the exclusion of others or can we just add it?

You can just add it, I have 10.0.19x, 10.0.20x and 10.0.22x concurrently on my dev machine, the WindowsSDKVersion on the environment should be that of 10.0.22 though.

Ready for review now?

Ready for review now?

I think so.

Update v8 to Stable version <IP_ADDRESS>

What does your PR fix?

Fixes #24510

Which triplets are supported/not supported? Have you updated the CI baseline?

Windows, Yes

Does your PR follow the maintainer guide?

Yes

If you are still working on the PR, open it as a Draft: https://github.blog/2019-02-14-introducing-draft-pull-requests/

Convert this PR to draft since there is no progress.

Looks like the SDK installed is 10.0.22621.0, is it installed side by side with 10.0.2000.0? I installed 10.0.22621.0 side by side and I am not seeing these errors:
[1/1354] CXX obj/cppgc_base/object-size-trait.obj FAILED: obj/cppgc_base/object-size-trait.obj  ninja -t msvc -e environment.x64 -- "C:\Program Files\Microsoft Visual Studio\2022\Enterprise\VC\Tools\MSVC\14.32.31326\bin\Hostx64\x64/cl.exe" /c ../src/c77ccc7593-16c1b401a7.clean/src/heap/cppgc/object-size-trait.cc /Foobj/cppgc_base/object-size-trait.obj /nologo /showIncludes -DUSE_AURA=1 -D_CRT_NONSTDC_NO_WARNINGS -D_WINSOCK_DEPRECATED_NO_WARNINGS -D_HAS_EXCEPTIONS=0 -DCOMPONENT_BUILD -D__STD_C -D_CRT_RAND_S -D_CRT_SECURE_NO_DEPRECATE -D_SCL_SECURE_NO_DEPRECATE -D_ATL_NO_OPENGL -D_WINDOWS -DCERT_CHAIN_PARA_HAS_EXTRA_FIELDS -DPSAPI_VERSION=2 -DWIN32 -D_SECURE_ATL -DWINAPI_FAMILY=WINAPI_FAMILY_DESKTOP_APP -DWIN32_LEAN_AND_MEAN -DNOMINMAX -D_UNICODE -DUNICODE -DNTDDI_VERSION=NTDDI_WIN10_FE -D_WIN32_WINNT=0x0A00 -DWINVER=0x0A00 -D_DEBUG -DDYNAMIC_ANNOTATIONS_ENABLED=1 -DCPPGC_VERIFY_HEAP -DENABLE_DISASSEMBLER -DV8_TYPED_ARRAY_MAX_SIZE_IN_HEAP=64 -DOBJECT_PRINT -DV8_TRACE_MAPS -DV8_ENABLE_ALLOCATION_TIMEOUT -DV8_ENABLE_FORCE_SLOW_PATH -DV8_ENABLE_DOUBLE_CONST_STORE_CHECK -DV8_INTL_SUPPORT -DENABLE_HANDLE_ZAPPING -DV8_CODE_COMMENTS -DV8_ENABLE_DEBUG_CODE -DV8_ENABLE_HEAP_SNAPSHOT_VERIFY -DV8_SNAPSHOT_NATIVE_CODE_COUNTERS -DV8_USE_EXTERNAL_STARTUP_DATA -DV8_ATOMIC_OBJECT_FIELD_WRITES -DV8_ENABLE_LAZY_SOURCE_POSITIONS -DV8_SHARED_RO_HEAP -DV8_WIN64_UNWINDING_INFO -DV8_ENABLE_REGEXP_INTERPRETER_THREADED_DISPATCH -DV8_SHORT_BUILTIN_CALLS -DV8_EXTERNAL_CODE_SPACE -DV8_ENABLE_MAGLEV -DV8_ENABLE_SYSTEM_INSTRUMENTATION -DV8_ENABLE_WEBASSEMBLY -DV8_ALLOCATION_FOLDING -DV8_ALLOCATION_SITE_TRACKING -DV8_ADVANCED_BIGINT_ALGORITHMS -DV8_ENABLE_CHECKS -DV8_COMPRESS_POINTERS -DV8_COMPRESS_POINTERS_IN_SHARED_CAGE -DV8_31BIT_SMIS_ON_64BIT_ARCH -DV8_DEPRECATION_WARNINGS -DV8_IMMINENT_DEPRECATION_WARNINGS -DCPPGC_CAGED_HEAP -DV8_TARGET_ARCH_X64 -DV8_HAVE_TARGET_OS -DV8_TARGET_OS_WIN -DDEBUG -DBUILDING_V8_SHARED -DUSING_V8_BASE_SHARED -DUSING_V8_PLATFORM_SHARED -I../src/c77ccc7593-16c1b401a7.clean -Igen -I../src/c77ccc7593-16c1b401a7.clean/include -Igen/include /WX /wd4244 /Gy /FS /bigobj /utf-8 /Zc:sizedDealloc- /wd4117 /D__DATE__= /D__TIME__= /D__TIMESTAMP__= /Zi /MDd /wd4245 /wd4267 /wd4324 /wd4701 /wd4702 /wd4703 /wd4709 /wd4714 /wd4715 /wd4718 /wd4723 /wd4724 /wd4800 /wd4506 /wd4091 /wd4127 /wd4251 /wd4275 /wd4312 /wd4324 /wd4351 /wd4355 /wd4503 /wd4589 /wd4611 /wd4100 /wd4121 /wd4244 /wd4505 /wd4510 /wd4512 /wd4610 /wd4838 /wd4995 /wd4996 /wd4456 /wd4457 /wd4458 /wd4459 /wd4200 /wd4201 /wd4204 /wd4221 /wd4245 /wd4267 /wd4305 /wd4389 /wd4702 /wd4701 /wd4703 /wd4661 /wd4706 /wd4715 /O2 /Ob3 /Oy- /Zc:inline /Gw /std:c++17 /Zc:alignedNew- /TP /GR- /Fd"obj/cppgc_base_cc.pdb" C:\Program Files (x86)\Windows Kits\10\include\10.0.22621.0\ucrt\corecrt.h(19): fatal error C1083: Cannot open include file: 'winapifamily.h': No such file or directory

Looks like the SDK installed is 10.0.22621.0, is it installed side by side with 10.0.2000.0?

I don't think so based on:
https://github.com/microsoft/vcpkg/blob/678c0ea99b778c7871e648182d61e1db7cee8892/scripts/azure-pipelines/windows/deploy-visual-studio.ps1#L21

Looks like the SDK installed is 10.0.22621.0, is it installed side by side with 10.0.2000.0?

I don't think so based on:
https://github.com/microsoft/vcpkg/blob/678c0ea99b778c7871e648182d61e1db7cee8892/scripts/azure-pipelines/windows/deploy-visual-studio.ps1#L21

Ok, that's one problem, there is also a link issue on the release build.

Any progress here?

Closing this PR since it seems that no progress is being made. Please ping me to reopen if work is still being done.

Yeah, this is fine, the PR works but CI does not, may need to just let V8 catch up to the latest SDK.


Use a temp directory for the hot reloading pipe

This hides the hot reloading file by placing it in the temp directory.
This reverses #842. If the issue is just with @ in a path, this shouldn't revert that fix.
Closes https://github.com/DioxusLabs/dioxus/issues/1616

This will cause issues if two projects are being hotreloaded simultaneously, right? We should try one of three things:

put the pipe in the target directory
put the pipe in a folder in the temp dir
multiplex the pipe itself

Eventually we probably want 3 but either 1 or 2 would be fine.

It now puts the socket in the target folder


Add DataList component

Description
Adds a component for displaying a group of key/value pairs. The orientation prop controls the orientation between the key value pair, whether horizontally or vertically laid out.

Testing steps
See sink page for examples and also the example in explore components.
Todo

[x] Add changelog note
[x] Cleanup example code generation in sink and explore-components

Relates issues / PRs

#230
#231
#233

@vladmoroz Updated examples here, lmk if that's more aligned!

Looks great @kendallstrautman, I'll go ahead and merge to rebase my PRs earlier


Add access by column

Describe your changes
The proposed method ResultSet.column() iterates on a column given by name.
https://github.com/ploomber/jupysql/blob/17ab97c5307cfb2d8c8fc5cae0c672083cc8f583/src/sql/run/resultset.py#L249-L256
Additions:

a method column(name) in resultset.py (above);
a method test_resultset_column(result_set) in test_resultset.py with a successful test and a failed test;
a paragraph in doc_intro.md;
an entry in changelog.md.

Checklist before requesting a review

[x] Performed a self-review of my code
[x] Formatted my code with pkgmt format
[x] Added tests (when necessary).
[x] Added docstring documentation and update the changelog (when needed)

📚 Documentation preview 📚: https://jupysql--1002.org.readthedocs.build/en/1002/

@laowantong Hey is there a related issue? Please mention the issue number also

No existing issue. I have just tried to add a feature which I felt was missing for my usage. Not sure why some checks fail. Feel free to close this PR is the procedure is not OK or you think the feature is useless.

No existing issue. I have just tried to add a feature which I felt was missing for my usage. Not sure why some checks fail. Feel free to close this PR is the procedure is not OK or you think the feature is useless.

Sure will have a look. There are lint issues in the changes. Please refer to contributing guidelines
Also, do include a description of the changes in the PR description @laowantong

Sorry for spamming you with my failed attempts to make the checks pass. There are still lint issues, although pkgfmt succeeds on my machine:
$ pkgmt format
Running command:black .
All done! ✨ 🍰 ✨
115 files left unchanged.
Running command:nbqa black .
All done! ✨ 🍰 ✨
56 files left unchanged.
Finished formatting with black!

Sorry for spamming you with my failed attempts to make the checks pass. There are still lint issues, although pkgfmt succeeds on my machine:

The lints are passing but the testcase test_resultset_column is failing. @laowantong

Too much friction for me, sorry. I cannot run the tests on my machine. There are multiple failures everywhere. I give up. You can close this PR. Thanks.


Regarding audio stop

when i am just play the audio, before pause i pop in another view controller its already running ..................How to pause that audio

You can refer to ZHCAudioMediaItem to write a class that inherits from ZHCMediaItem and add a method to pause playback.
- (void)stopPlay
{
    if (self.audioPlayer && self.audioPlayer.playing) {
        self.playButton.selected = NO;
        [self stopProgressTimer];
        [self.audioPlayer stop];
    }
}

this error is showing when i pop to another controller with ur latest code also........

This error is the release of the ProgressView after the pop operation, but the audio callback function calls the freed ProgressView. So you can follow the custom item I said above and add a pause method to pause playback of the audio before the pop operation.

The problem has just been repaired. You can download the latest Objective-c version of the test.


Added decode ignore arg

To prevent errors like:
UnicodeDecodeError: 'utf-8' codec can't decode byte 0xdc in position 9: invalid continuation byte

To prevent errors like:
UnicodeDecodeError: 'utf-8' codec can't decode byte 0xdc in position 9: invalid continuation byte


Fix publish scripts

extract rook version
Updated rook.jar version to 0.1.224 [skip ci]
upadte makefile
remove rook jar
Updated rook.jar version to 0.1.224 [skip ci]
checkout master before updating rook jar
add version bump script
allow also to insert manual version
restore rook version
nit
add warning
refactor version_utils.py
uncomment
test
Fixing docker publish flow

/jira open task


length/2 segfault

v2.33.5
$ tpl
?- length(L,N).
   L = [], N = 0
; Segmentation fault (core dumped)
$

$ tpl
?- append(L,[],LL).
   L = [], LL = []
; Segmentation fault (core dumped)
$

Related?

Yes it is related. Gonna have to revert a previous top-level print fix.
On Tue, Jan 9, 2024 at 12:00 PM flexoron @.***> wrote:

Closed #458 https://github.com/trealla-prolog/trealla/issues/458 as
completed.
—
Reply to this email directly, view it on GitHub
https://github.com/trealla-prolog/trealla/issues/458#event-11424322395,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AFNKSEWQE3JUQV6TXV2KT2TYNSQEZAVCNFSM6AAAAABBSKOENOVHI2DSMVQWIX3LMV45UABCJFZXG5LFIV3GK3TUJZXXI2LGNFRWC5DJN5XDWMJRGQZDIMZSGIZTSNI
.
You are receiving this because you commented.Message ID:
@.***>


Merge MaxDepth and CircularReference exclusions

Since these two exclusion strategies walks through the metadata/data stack, it has an impact on performance and this is why there are disabled by default. But both do mostly the same things and cover similar features which are mostly enabled together, so, let's merge them...

Fixed by 35dc50339abb1b1990e16eaf30a0acab5b28c2d5


The package libyaml-dev in Ubuntu/apt does not have a config.cmake file

After installing "libyaml" with below command in Ubuntu 18.04 LTS and Ubuntu 20.04 LTS
sudo apt-get install -y libyaml-dev

We cannot find any config.cmake file for libyaml and then the CMake files of our project cannot use find_package(yaml)
Is the dpkg package libyaml-dev officially maintained by this repo ?
If so,  could you please tell us whether packaging config.cmake file to the dpkg package libyaml-dev is the task your planed ?
Thank you !

Package information lists Ubuntu Core Developers as maintainer for libyaml-dev package, both 18.04 and 20.04 versions. I would suggest to contact them.

Here is what I get on my Ubuntu 20.04
dpkg -L libyaml-dev
/.
/usr
/usr/include
/usr/include/yaml.h
/usr/lib
/usr/lib/x86_64-linux-gnu
/usr/lib/x86_64-linux-gnu/libyaml.a
/usr/lib/x86_64-linux-gnu/pkgconfig
/usr/lib/x86_64-linux-gnu/pkgconfig/yaml-0.1.pc
/usr/share
/usr/share/doc
/usr/share/doc/libyaml-dev
/usr/share/doc/libyaml-dev/copyright
/usr/lib/x86_64-linux-gnu/libyaml.so
/usr/share/doc/libyaml-dev/changelog.Debian.gz

The Ubuntu packages of libyaml are taken from Debian, where recently an equivalent bugreport has been filed in https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=1022825. I am the new maintainer of the Debian integration of libyaml.
I understand from the mentions above that it would be really beneficial to have yamlConfig.cmake available in Debian and thus eventually in Ubuntu as well.
It seems that the yamlConfig.cmake file currently will only be generated when using CMake to build libyaml. The standard build scheme via configure does not build it, though.
However, the yamlConfig.cmake.in appears simple enough as it is. Is there an easy way to generate a fully compliant yamlConfig.cmake from it without changing build systems?
Or could possibly the standard build scheme be adjusted to also just build this file?
Just for reference, #99 makes a case in favor of switching build systems to cmake. However, it seems #101, #138, and #141 would need to be addressed first, if they haven't already in the meantime.


SLIM-863, implemented GetConfigurationObject request code

SonarQube analysis reported 1 issue

 1 major

Watch the comments in this conversation to review them.

All is well.

All is well.

All is well.

All is well.

All is well.

All is well.

All is well.

All is well.

In the Platform pull request there are some remarks regarding the use of Orika which should make more use of the frameworks capacities instead of custom converters. This would require changes to GetConfigurationResponseDto here in Shared:

removing "Dto" from the end of the name of field configurationObjectDto;
removing "Dto" from the end of the name of constructor argument configurationObjectDto;
removing "Dto" from the end of the name of the getter method for the configuration object.

All is well.

All is well.

All is well.

All is well.


No dist folder

I cloned the project and ran the two commands，but I could not get the dis folder.

run the command npm i parcel and update  the version of @parcel/transformer-sass in the pakages. Then it works.


smuxsocket disabling

Hi
On CentOS 7.1.1503 snmpd listens on tcp port 199 by default. I tried to switch it off via startup flags but unfortunately these get ignored. Specifically "-I -smux" From the little reading I have done this does not seem to be widely used and is turned off on Ubuntu 14. The only way to disable it is by giving it the wrong configuration:
smuxsocket <IP_ADDRESS>
Now at the moment this is not surfaced as a parameter in the module. Before I add it as an option in a PR, I just wanted to get an opinion on whether it is worth adding. I did not see an option for adding arbitrary config, but it might be due to lack of coffee early in the morning.
Regards

There is a snmpd safety valve parameter to add in anything the module does not natively support.

closing


Link to Billing Model page missing

Hi,
the sentence For more information, see Billing model for Azure AD External Identities. feels like it should have pointed to a doc page, but it doesn't.

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

ID: 3a8a3416-ca49-e148-89ec-aa004914eaa5
Version Independent ID: 6e6831f4-40ed-4058-5a6f-56cd3900460a
Content: What are access reviews? - Azure Active Directory
Content Source: articles/active-directory/governance/access-reviews-overview.md
Service: active-directory
Sub-service: compliance
GitHub Login: @barclayn
Microsoft Alias: barclayn

@EricHindrichsen ,  Thanks for reaching out , I just created pull request to update Doc with [Billing model for Azure AD External Identities. Kindly allow sometime to reflect same changes public article. Once again thank you for bringing this to our attention.
#please-close


Update changelog for v0.1.2

New minor version to publish changes, which are necessary to release QuTiP-QOC

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.1%) to 69.461%

Totals

Change from base Build<PHONE_NUMBER>:
-0.1%

Covered Lines:
3298

Relevant Lines:
4748

💛  - Coveralls


Math Magicians: setup

Set up a React app.

Fixed linter error

Update README file

hello @ragangithub , please do not write PR summaries like Fixed linter error. Always write a brief summary that includes the project requirements rewarded as tasks you have done. forexample

Create react application using CRA command
Delete all starter files
Add index.js file ....
....

created React app using CRA
updated README file
fixed linter errors


add invoice_outside_freckle api calls

Mark entries invoiced outside freckle

Thanks! Somehow missed the notification for this.
Just waiting on the name change, then I'll get this added.

@timcraft I updated the api calls, seems they where changed with the rename.

Merged manually with different names and some edits, available in v1.5.0
Thanks for the contribution 👍


"Params" parameter not working

Not sure that this is a problem with the resource or if I'm doing something incorrectly, however I'm trying to maintain an offline, parameterized ADK 8.1 Installation.
Params in cChocoPackageInstaller contains
params = '/Features OptionID.WindowsPerformanceToolkit'
The package contains the following silentargs in the chocolateyinstall.ps1
silentargs = "$env:ChocolateyInstallArguments /norestart /quiet /ceip off"
The installation fails and exits with -2147023274.
Manual installation of the adk works via the adk GUI
Chocolatey Installation of the package works with
choco install adk81 --ia="/Features OptionID.WindowsPerformanceToolkit" -y
Attempted to change the resource to use --ia instead of the default --params, same error.
is this an issue or am I passing the params in incorrectly?

Do you have the underlying chocolatey log from the logs folder you can provide for the run?

chocolatey.txt
Thanks for taking a look.
From this line in the log -- 2016-08-01 18:12:53,123 [ERROR] - ERROR: Running ["c:\temp\ADK81\adksetup.exe" /Features OptionID.WindowsPerformanceToolkit /norestart /quiet /ceip off /Features OptionID.WindowsPerformanceToolkit] was not successful. Exit code was '-2147023274'
It appears that it's appending the $env:chocolateyInstallArguments at the beginning and end of the line?

Look at the arguments -
c:\temp\ADK81\adksetup.exe" /Features OptionID.WindowsPerformanceToolkit /norestart /quiet /ceip off /Features OptionID.WindowsPerformanceToolkit
I see what is wrong now. Chocolatey Install Arguments are already passed transparently to the native installer, and require no interaction from the package itself.
Change:
silentArgs    = "$env:chocolateyInstallArguments /norestart /quiet /ceip off"

to
silentArgs    = "/norestart /quiet /ceip off"

@lawrencegripper @ebekker please close as not an issue. Thanks. 👍

@dcuomo apologies, misread what you wrote in your original post.

My larger question, which I didn't make clear and mis-titled the "issue" (and apologize for throwing the troubleshooting of course) is that I don't believe there is a way to pass parameters to the native installer unless I'm doing something incorrectly (kind of a newbie to chocolatey - so totally possible).  In context of the ADK, on one system I'd like to install the performance toolkit.  On another system, I'd like to install the preinstallation environment and deployment tools.  I'd prefer if I could use one package and pass in the parameters via the DSC Resource to a specific node.
I'm receiving the same error whether $env:ChocolateyInstallArguments is added to the silent args or not using the cChoco module in a DSC Configuration.
Exit code was '-2147023274'
Chocolatey Log shows ["c:\temp\ADK81\adksetup.exe" /norestart /quiet /ceip off ] was not successful. Exit code was '-2147023274'
As a side note, during my troubleshooting I had changed the DSC Resource from using the --params to --ia.  I accidentally forwarded the chocolatey log from that attempt last time. - Again, apologies...
For reference, running the following from the command line works perfectly (with $env:ChocolateyInstallArguments in the ChocolateyInstall.ps1 line)
choco install adk81 --ia="/Features OptionID.WindowsPerformanceToolkit"
However this (which is how the DSC Resource implements it) installs the entire ADK
choco install adk81 --params="/Features OptionID.WindowsPerformanceToolkit"
First - Is there a way to pass items to a native installer such that I can use one package through the DSC Configuration, or is that something I should try to help solve ;-)
Second - Why do I get different results when running chocolatey from the cmd line vs via the DSC Configuration?

I second what dcuomo is saying. It would be really useful to be able to pass install arguments to chocolatey

As a side note, during my troubleshooting I had changed the DSC Resource from using the --params to --ia. I accidentally forwarded the chocolatey log from that attempt last time.

First - Is there a way to pass items to a native installer such that I can use one package through the DSC Configuration, or is that something I should try to help solve ;-)

This seems to make sense as an add unless someone else has already added this (which it could be)

Second - Why do I get different results when running chocolatey from the cmd line vs via the DSC Configuration?

The user context likely. When you run it, you are running it under your username. When DSC runs it, I believe it runs under LocalSystem (BUILTIN\SYSTEM). Not sure if that would cause any differences. I know that folks like @jpogran, @ebekker, and @lawrencegripper could probably provide more insight on the intricacies of DSC.

@dcuomo I don't have an answer to your first question.
As for the second, what @ferventcoder mentions is correct. DSC runs under the 'nt authority\system' context UNLESS you specify 'PsDscRunAsCredential'. This results in the fact that some installers might behave differently or even fail completely. An example of this would be Atom package.

But why don't you want to add ability to pass install arguments?

@4c74356b41 provided you are not a bot - we want both.

well, that's a nice way to talk to a stranger

@4c74356b41  we are working on that. Params is only to be used for Package params NOT chocolatey params. Refer to #52 for more details.

thanks, I got that already!

well, that's a nice way to talk to a stranger

@4c74356b41 Apologies, it's the username. It's similar to what we've seen in the past as bots (seemingly random characters for user name).

hah, never thought about that ;) its okay ;) ya, this name is a bit random, but I love it ;)

Closing this one off now as think we're sorted with the PR from @javydekoning


Implement save file functionality

Add a simple button and dropdown menu to save the contents of a card. Lots of screenshots:

We use the OS's native file picker and message box:

Future work

Generalize the dropdown so we can use it for other things too?
Add keyboard shortcut

will probably be a rabbit hole, haven't thought much about shortcuts yet

@rgwood Can't get it to show the save dialog on my Mac. The main windows goes dark like there is a popped up save dialog but it doesn't show up for me. I also get warnings on compile.
warning: unused imports: `MenuItem`, `Menu`, `Submenu`
  --> src/main.rs:21:13
   |
21 | use tauri::{Menu, MenuItem, Submenu};
   |             ^^^^  ^^^^^^^^  ^^^^^^^
   |
   = note: `#[warn(unused_imports)]` on by default

warning: `nana` (bin "nana") generated 1 warning

That's surprising that it doesn't work on macOS; might be a bug in Tauri, I'm using their file picker API.
I think those warnings are probably from another PR I did recently. I'll try to get cargo fmt/clippy working in CI to catch warnings like that.

I got it to work. It may be some weird edge case. If I stay on the virtual desktop while it's building and wait for it to pop up with just watch, it works.  If I switch to another virtual desktop while it's building and then it pops up on a different virtual desktop, the dialog doesn't pop up on any virtual desktop. It's just weird.
I also noticed that when I got it to work, it defaulted to a save folder that wasn't the folder that I was in. i.e. I ran just watch from the nana folder and the save dialog was in the nushell folder. I wonder if that's just where the save dialog was at last time from some other app?

I also noticed that when I got it to work, it defaulted to a save folder that wasn't the folder that I was in. i.e. I ran just watch from the nana folder and the save dialog was in the nushell folder. I wonder if that's just where the save dialog was at last time from some other app?

Good question, I'm not sure how Tauri (or the OS?) chooses the starting folder when you don't specify it. We might want to specify it eventually.


zero size structs with extern (C) attribute

Clear up documentation to correspond with https://github.com/dlang/dmd/pull/13009

ping @WalterBright


Please help me out?

Please make a video on how to generate QR code in termux!!
Please

Please make a video on how to generate QR code in termux!!
Please

Sorry but open-wa doesn't support Termux

Please make a video on how to generate QR code in termux!!
Please

Sorry but open-wa doesn't support Termux

Cant you change it from open wa to baileys so that handphone users can use it


Module Not Found: Cannot Resolve '@llamaindex/readers/obsidian' in Next.js Build

Describe the bug
When building my project, I encounter the error:
Module not found: Can't resolve '@llamaindex/readers/obsidian' in the file index.js in node_modules/llamaindex/dist/readers/. It appears that the @llamaindex/readers/obsidian module cannot be found.
Files
I encountered this issue with a specific setup, but the error is not related to a PDF file. The issue is module resolution related.
Job ID
Not applicable for this error as it pertains to a local build issue.
Client:

Frontend (cloud.llamaindex.ai)

Additional context

I am working in a Next.js environment (version 15.0.3 with Turbopack).
The error occurred when attempting to import @llamaindex/readers/obsidian, but that module seems to be missing or incorrectly referenced in the package.
No other issues with other readers like mongo, notion, or pdf.

Screenshots/code snippets:
I am attaching a screenshot of the error for reference:

Fixed in https://github.com/run-llama/LlamaIndexTS/commit/b8f0cbc4e056d9f0bcc3ec713e7faea72abfd1b3


Add ability to set cli flags with environment variables

Hey folks 👋🏼
Would be great if we could define the CLI flags also with Environment Variables, this is especially important for arguments that are somewhat sensitive (like --notify-url) where we mutate the value from Vault.
Thanks!

That sounds a great idea!

Is there a workaround for this? I see no way to specify notifyUrl from a secret (using the Helm chart), and not using a secret is not an option.

This is currently not possible. The problem is not the helm-chart I think (it supports extraEnvVars which can be populated from secrets), but the code from kured itself. Currently kured does not read flags from the command line. But this should be easily possible to implement in main.go with the existing viper-library.

Thanks @ckotzbauer!
I was looking into this earlier today and came across this issue: https://github.com/spf13/viper/issues/671
and it looks like integrating viper & cobra for what's needed here comes with a pretty big caveat.
but not all seem lost: https://github.com/spf13/viper/issues/671#issuecomment-671067523 suggests that this is indeed possible but requires some glue code to make it work right.

Thanks for your research.
Yes, it is possible with a bit of code, but that should not be a blocker 😉
Would you mind opening a PR for that?

Hey @ckotzbauer, my attempt to crack this in https://github.com/weaveworks/kured/pull/464
This seem to plumb nicely into the existing code and works as expected.
comments/suggestions are welcome 😄

I'll have a look today, thanks!! 🎉


The section How does Azure AD B2B differ from external sharing in SharePoint Online? is out of date

The section How does Azure AD B2B differ from external sharing in SharePoint Online? is out of date
This information on SharePoint external settings is out of date.
These are the options you see in the SharePoint Admin center for external sharing

Also the information of external user authentication needs updating as the Email one time passcodes started to roll out on 1 November 2021. This link has more information on the process https://docs.microsoft.com/en-us/sharepoint/sharepoint-azureb2b-integration  and https://docs.microsoft.com/en-us/azure/active-directory/external-identities/one-time-passcode
[Enter feedback here]

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

ID: b1ab5696-ee53-9104-95e8-9a55d3ec26c9
Version Independent ID: ed1092ea-de38-275d-5437-4c2267b75f36
Content: Microsoft 365 external sharing and B2B collaboration - Azure AD
Content Source: articles/active-directory/external-identities/o365-external-user.md
Service: active-directory
Sub-service: b2b
GitHub Login: @msmimart
Microsoft Alias: mimart

@nikki-c Thanks for the feedback! I have assigned the issue to the content author to investigate further and update the document as appropriate.

Hi @nikki-c, thanks so much for pointing this out. This article is now outdated and is being retired. A description of SharePoint/OneDrive integration is being added to the B2B collaboration overview with links to the relevant article in the SharePoint docs. Your feedback is much appreciated!


create unit tests

Each file under the R directory should have a matching unit test using the testthat package. What follows is a list of the R files that need unit tests with a person suggested to work on them in parentheses. Please let me know where to switch names around (or remove them), if I've gotten it wrong. Below the list of R files is a description of what the unit test might do, we can check files off the list when they're ready.

[ ] R/curatedMetagenomicData.R (@cmirzayi)
[ ] R/mergeData.R (@sdgamboa, @schifferl)
[ ] R/returnSamples.R (@sdgamboa)

R/curatedMetagenomicData.R
The unit test should check that results with dryrun both TRUE and FALSE are correct, it can check both cases of print argument and the counts arguments too. When checking the pattern argument, a golden test is possible with the inst/extdata/metadata.csv file and one should be implemented to check that all posibilites are there - these could also be downloaded and their classes could be checked (the matrices inside the objects too), but this might be prohibitive with the Bioconductor Build System. The latter might be implemented as a long test, but I've never done one so I can't say more than that.
R/mergeData.R
All data types should be check for correct merging (the two AsnicarF studies are particularly useful for this) and also differing data types should be checked for failed merging.

@sdgamboa, I just wanted to add a note to work on this one last. The package might change just a little more and I don't want us to have to redo the unit test when it happens.

Should transport as many of the tests from https://github.com/waldronlab/curatedMetagenomicData/tree/legacy/tests/testthat as possible.

Closed by the addition of 41 test in 7a2a2c3.


pkg/lock: Close the file descriptor on unsuccessful locking

NewLock() opens a new file descriptor to lock on but if file
is already locked, Try*Lock() methods return error without closing
the corresponding file descriptor.

doesn't this conflict with #3615?

Yes it does but created separate PR as these two PRs fix two different issues.

ok to test

I agree that NewLock should close the fd on error (good catch!). However I don't agree with other ancillary functions doing so, as it is not clear anymore who owns the fd (library or consumer) and because the operation may be re-tried on transitory errors (ENOLCK / EINTR). I would suggest instead to make explicit in costructor docstring that the consumers are responsible of calling Close on FileLock after they are done with it.

/cc @euank as he had opinions on the other similar-but-unrelated PR on the lock module.

@lucab Closing fd in other ancillary functions seems necessary as caller of these functions doesn't have any handle to the fd if just err is returned. So those fds are leaked.
I'll update NewLock docstring.

@shobhit85 you are right, I got confused by those functions acting on a *FileLock. Please disregard the first part of my comment above. Let's just tweak the docstring, and I'm ok with this.

@lucab Updated the NewLock docstring. If looks good I'll go ahead and squash the commits.

@shobhit85 yes please.

Done with rebase and squash. cc @lucab

@lucab Can this be merged?

@shobhit85 I just dismissed my stale review and re-triggered CI which was left in a confused state.

@lucab  That semaphoreci is still failing. Not sure why.


Fix cmdclass, allow passing in of base cmdclass

This commit makes it possible to use Versioneer with
packages that use a special cmdclass.
Fixes #128

looks good! landing momentarily.


[Home/Artist] Center spinner while loading. #trivial

Fixes #294

          1 Error

:no_entry_sign:
No CHANGELOG added.

          1 Warning

:warning:
The following files do not have tests: lib/components/home/artist_rails/artist_rail.js

  Generated by :no_entry_sign: danger

yayayayah, self merging.


VIM-1357+VIM-1566: Use OS shell to run filter command

This PR fixes a few issues with the filter (:!) command:

[x] Use OS shell to execute the given command. On Unix, this is $SHELL or sh, on Windows, this is cmd.exe. Fixes VIM-1566 and VIM-1357
[x] Add 'shellcmdflag', 'shellxescape' and 'shellxquote' options, required to properly run, quote and escape parameters
[x] Errors are displayed or substituted into text, like Vim
[x] Long running tasks can be cancelled
[x] Adds support for % inside a filter command, mapping to the current file name

That's great! Thank you for the fixes!

thanks for this PR, really love my shell filters in Vim!


NO-TICKET: Use GitHub Gradle Action

See https://github.com/gradle/gradle-build-action#why-use-the-gradle-build-action
Setting the executable permission for gradlew has been removed since the permission is set correctly by Git.

I like! Thanks for cleaning this up. Do you see improvements in terms of performance due to the new cache? Or any other noticable advantages so far?

In this job, the build was actually 1 min slower than one random other PR, but I assume the cache was completely cold for this run. We don't have enough data to answer the question yet.


Adds ability to adjust upload image quality

Finishes #137031547

@colinta, moving this to the 1.21 milestone. @caseydoran wants this in the next build, seems fine to me.


InsertionOrderSet performance issue

Hi,
I have a suite where I add dynamically tests, it is ~10k that I noticed heavy slowdown in running our tests.
Investigating what takes so long leads to:
  override def testNames: Set[String] = {
    InsertionOrderSet(atomic.get.testNamesList)
  }

When I have my Spec and for testing will do inside:
override def testNames: Set[String] = {
  log("start time")
  val result = super.testNames
  log("end time")
  result
}

It takes minutes to finish.
InsertionOrderSet is defined as:
  def apply[A](elements: List[A]): Set[A] =
    scala.collection.immutable.SortedSet(elements: _*)(new Ordering[A] {
      def compare(x: A, y: A): Int = elements.indexOf(x) compare elements.indexOf(y)
    })

Which if I understand correctly uses O(n) to check indexOf -> causes sorting of elements: List[] to be O(n^2*log(n)) ?
I know it is uncommon to have so many tests in a suite but could we somehow add optimization to also handle with reasonable performance suites with bigger amount of tests?
Also I tried to see if I could work around for now in my current version, but I can see it uses atomic which is private to spec, do you have any idea if it is possible to do something in my code using current scalatest to speed things up?
~Cheers
Krzysiek

@kpbochenek Sorry to hear about your case, unfortunately I can't think of a way you can adjust your code to speed things up.  I am working on improving the performance by using a implementation that wraps a scala.collection.mutable.LinkedHashSet underlying, I'll try to include that and release 3.2.18 this week, if the changes looks good to @bvenners .  I should be able to submit the PR today.

Thanks for a quick response 👍
For now I just copy&pasted FlatMapSpecLike and renamed to MyFlatMapSpec which I then used and changed InsertionOrderSet to simple Set and that fixed my performance issue. I don't care about order so its fine in my case but in general it cannot be used :/
Just a reference immutable SeqSet would have a perfect use here: https://github.com/scala/scala-library-next/issues/22

@kpbochenek Fyi I submitted the PR here:
https://github.com/scalatest/scalatest/pull/2308

@cheeseng awesome thanks for fixing it so fast! 👍

@kpbochenek fyi the PR is included in 3.2.18 that is already released.


Update profile data in create a channel doc

Resolves #4779
Finish the changes that were started in #4781 and #4788.

@solo-daemon @satota2 This should be the final change for main. Then we can backport to release-2.5.


Use Time objects in Ruby: Fix #970

@muga @sakama Can you have a look? It is trying to fix #970.

Adding more -- write to PageBuilder and test the writes.

Thanks!


searchChatMembers more than 200 users

Hello. I want to get more than 200 users from searchChatMembers response.
Is there something like offset?

You need to use getSupergroupMembers instead.

You need to use getSupergroupMembers instead.

With filter supergroupMembersFilterSearch, right?

But I don't get members more than 200 per search(
I want to get all members using searching by every writable ascii letter, but I cant get more than 200 results per letter(
What should I do?

https://core.telegram.org/tdlib/docs/classtd_1_1td__api_1_1get_supergroup_members.html
std::int32_t | limit_ The maximum number of users be returned; up to 200.

@pblackbird First of al,l you shouldn't abuse members search. You are supposed to be able to get up to 10000 members in supergroups and 200 members in channels using supergroupMembersFilterRecent and an appropriate offset for the method getSupergroupMembers.


Cookie not set in docker

Hello,
this is a fantastic cookie consent banner!
But I encountered two issues I do not know how to fix myself maybe anyone here has an idea.
When using the cookie banner in debug within Visual Studio (Edge browser) everything works as expected, the cookie is set inside the browser on my machine. On another MacOS based machine using Safari no cookie is set, while in chrome it worked.
My main problem is that when running my project inside docker, the cookie banner cookie is never set inside Edge (potentially others too).

Hi, thanks for using the library.
I've tried to think of what might be the reason for the behavior you're seeing, however I'm afraid with some debugging steps from your side I am unable to reproduce the issue on my end (since I don't experience this Docker specific issue and neither do I have a macOS based system to test on).
A few questions:

When speaking of "the cookie is not set on macOS with Safari", do you mean you're opening you're website that is hosted on a remote machine?
When talking about running inside docker and no cookie being set, do you host the Docker container locally or on a remote machine?
Do you use any kind of proxy server between your ASP.NET Core web server? NGINX Reverse Proxy, Cloudflare, etc.?
In any case where no cookie is set, have you tried checking the network calls your browser makes to see if the JS is potentially being blocked from being loaded?

The Safari issue might possibly be an issue with the JS (if the script is correctly loaded in the browser). The cookie not being set in Edge when hosting through Docker is something I'll investigate depending on whether it' a remote container or locally.

@RyanTT Thank you for your reply. I would be happy to assist to solve the issue, since this is a fantastic plugin for our project but also potentially others who might encounter a similar issue in the future.
To answer your questions:

MacOS is opening blazor in Visual Studio and debugging it locally. I haven't tried the docker setup on macOS, so far Windows 11 was used for the docker.
Currently only locally, but I can try to upload it to our development kubernetes server. Can a missing certificate be an issue potentially?
During debugging, I'm using the local IIS debug configuration. While on MacOS we have a Kestrel configuration.
When running inside docker: during local docker deployment, there is no proxy I believe. When our project is running in production, we use a Kubernetes cluster, but the blazor pod is directly accessible without any middleware.
I will give this a try, and check it. Correct me if Im wrong, when the cookie could not be set, there should be a log message if I understood the code correctly?

Thank you for your support, we are using blazorboilerplate if you are familiar with that blazor template.

Update:
I tried locally debugging with Kestrel and IIS, both set the cookie accordingly.
I checked the local docker version, and as far as I can see nothing is being blocked:
The cookieconsent.js script:

We are using the Blazor in Server based mode.

2. an a missing certificate be an issue potentially?

Not sure if the browser might interfere in writing a cookie if the website is accessed via HTTP. Maybe try setting this to false
https://github.com/BytexDigital/BytexDigital.Blazor.Components.CookieConsent/blob/master/BytexDigital.Blazor.Components.CookieConsent/CookieConsentOptions.cs#LL254C30-L254C30
Though I don't think it's the root cause of the problem. It would be interesting to know whether the JS to set the cookie is actually being ran in the scenarios where no cookie is successfully set so we can pin down where the issue might lie.
To do this, you could uncomment the include of the JS file and instead make a custom JS file you include with this code that will log the attempt to write the cookie:
export let CookieConsent =
{
    ReadCookie: function (name) {
        console.log("Read cookie called.");
        return document.cookie.match('(^|;)\\s*' + name + '\\s*=\\s*([^;]+)')?.pop() || '';
    },

    RemoveCookie: function (name) {
        console.log("Remove cookie called.");
        document.cookie = name + "=; expires=Thu, 01 Jan 1970 00:00:00 UTC; path=/;";
    },

    SetCookie: function (cookieString) {
        console.log("Set cookie called:" + cookieString);
        document.cookie = cookieString;
    },

    ApplyPreferences: function (categories, services) {
        const activatableScriptTags = document.querySelectorAll("script[type='text/plain']");

        activatableScriptTags.forEach(originalScriptElement => {
            const requiredCategory = originalScriptElement.getAttribute("data-consent-category");

            if (!requiredCategory) return;

            if (categories.includes(requiredCategory)) {
                originalScriptElement.type = "text/javascript";
                originalScriptElement.removeAttribute("data-consent-category");

                let sourceUri = originalScriptElement.getAttribute("data-src");

                if (sourceUri) {
                    originalScriptElement.removeAttribute("data-src");
                } else {
                    sourceUri = originalScriptElement.src;
                }

                const newScriptElement = document.createElement("script");
                newScriptElement.textContent = originalScriptElement.innerHTML;

                const sourceAttributes = originalScriptElement.attributes;

                for (let i = 0; i < sourceAttributes.length; i++) {
                    const attributeName = sourceAttributes[i].nodeName;

                    newScriptElement.setAttribute(
                        attributeName,
                        originalScriptElement[attributeName] || originalScriptElement.getAttribute(attributeName));
                }

                if (sourceUri) {
                    newScriptElement.src = sourceUri;
                }

                originalScriptElement.parentNode.replaceChild(newScriptElement, originalScriptElement);
            }
        });
    }
};

I placed breakpoints within the browser JS at the SetCookie function to see what happens:

Left half is debugging, right half is docker.
Both execute the JS and reach the breakpoints, it is just that within docker the cookie isn't assigned somehow.
I will try setting the flag to false next

i also love this plugins, and use it in an docker environment (wasm) with traefik as a reverse proxy.
It works for windows and mac as expected, and i cant remember that i need to set anything specific
if you want to check yourself: https://ejobs24.de/

If the cookie was the issue, no need to try the JS 👍

Alright, in that case, this issue is solved. Thank you very much @RyanTT


Create Toolbar

Create a toolbar at the top of the screen under the title bar.
This area should be non-dockable (so the dockable area will probably need to be shrunk)
Paranext POC toolbar https://github.com/tjcouch-sil/paranext-poc/blob/1b21609ff445468b4b211e1db877131466afe08d/react-electron-poc/src/renderer/components/layout/Layout.tsx#L392

I have a toolbar now. Is there anything to put on it by default? (There's a separate task to add the Scripture Reference Control.) I'm trying to add a menu. I think this is a useful learning exercise for me anyway, but is that what we want? Should I be thinking about how to make it possible to allow it to be customized (by extensions), or is the plan to just have it hold very general controls that will be the same for all users?


cape doesnt show

was wondering if you still work on this mod, but i found an issue sometimes the cape just dissapear when trying to match with other mod. but twhen trying to enter a world a few frame showed the cape.

was wondering if you still work on this mod, but i found an issue sometimes the cape just dissapear when trying to match with other mod. but twhen trying to enter a world a few frame showed the cape.

What other mod are you using. I can't guarantee it works with every mod

essential

essential

It works for me. Maybe you have some other mod installed

Same issue, I'm using waveycapes mod and its incompatible with this mod. Could you please fix it? Would be amazing, thank you!

mod

Ok


OGNL issues with java 16 and 17

WARNING: An illegal reflective access operation has occurred
WARNING: Illegal reflective access by ognl.OgnlRuntime (file:XXXXXXXXXXXXXXXXXXXXXX/WEB-INF/lib/ognl-3.0.21.jar) to method java.util.HashMap$Node.getValue()
WARNING: Please consider reporting this to the maintainers of ognl.OgnlRuntime
WARNING: Use --illegal-access=warn to enable warnings of further illegal reflective access operations
WARNING: All illegal access operations will be denied in a future release
This warning message happens while I was running java 15. But since java 16 on wards struts-jquery-chart-tags stop working because they rely on ognl to assign dynamic data value. Access operation to to method java.util.HashMap$Node.getValue() is denied.
Is there any solution or idea?
I am running ognl 3.2.1.
Higher releases shows us the error below:
java.lang.NoClassDefFoundError: ognl/DefaultMemberAccess

Just to clarify, your error message on the top shows ognl-3.0.21.jar which means you are using OGNL 3.0.21 (an very old version). Are you sure you are running OGNL 3.2.1?
I would suggest switching to the latest version 3.3.2 which should address the problem -> #144 - also the problem with missing DefaultMemberAccess has been resolved in 3.2.19.

Hi, Lukazlenart,
Yes, I am running ognl 3.2.1. And I tried 3.3.2 version... I read in a forum that DefaultMemberAccess was removed before 3.2.1 version. That is why I downgraded from 3.3.2 to 3.2.1.
The warning message while running versions 3.2.1 and 3.0.21 is the same using java 15. The reason it display 3.0.21 in the message is because I copied the message to report to you while running 3.0.21. I upgraded it later but I did not copy the message again.
But here is the message while running it with java 15:
WARNING: An illegal reflective access operation has occurred
WARNING: Illegal reflective access by ognl.OgnlRuntime (file:/C:/Users/xxxx/WEB-INF/lib/ognl-3.2.1.jar) to method java.util.HashMap$Node.getValue()
WARNING: Please consider reporting this to the maintainers of ognl.OgnlRuntime
WARNING: Use --illegal-access=warn to enable warnings of further illegal reflective access operations
WARNING: All illegal access operations will be denied in a future release

What version of Struts do you use?

I´m using struts2-core 2.5.29

I think you must wait for incoming Struts 6.0 (aka Struts 2.6) where a proper support for the latest Java has been already implemented.
https://cwiki.apache.org/confluence/display/WW/Struts+2.5+to+6.0.0+migration

That is good.
Is there any release date yet?

It should be soon, in 2-4 weeks

Thank you, Lukas. GJ.

Struts 6.0.0 is out


Add debouncing for setting read cursors

What?
Add debouncing for setting read cursors
Why?
So the service doesn't get hammered by loads of read cursor requests

CC @hamchapman

          1 Warning

:warning:
Any changes to library code should be reflected in the Changelog. Please consider adding a note there. \nYou can find it at CHANGELOG.md.

SwiftLint found issues
Warnings

File
Line
Reason

PCCurrentUser.swift
16
TODOs should be avoided (This should probably be [PCUse...).

PCCurrentUser.swift
368
TODOs should be avoided (Use the soon-to-be-created new...).

PCCurrentUser.swift
456
TODOs should be avoided (Do we need to fetch users in t...).

PCCurrentUser.swift
774
TODOs should be avoided (Do I need to add a Last-Event-...).

PCCurrentUser.swift
815
TODOs should be avoided (What happens if you provide bo...).

PCCurrentUser.swift
836
TODOs should be avoided (This should probably be a room...).

PCCurrentUser.swift
846
TODOs should be avoided (Should we be handling onError ...).

PCCurrentUser.swift
872
TODOs should be avoided (Only consider the room subscri...).

PCCurrentUser.swift
881
TODOs should be avoided (Should we be handling onError ...).

PCCurrentUser.swift
871
Unused parameter "err" in a closure should be replaced with _.

PCCurrentUser.swift
1015
Unused parameter "data" in a closure should be replaced with _.

PCCurrentUser.swift
887
Function body should span 40 lines or less excluding comments and whitespace: currently spans 49 lines

PCCurrentUser.swift
702
Lines should not have trailing whitespace.

PCCurrentUser.swift
707
Lines should not have trailing whitespace.

PCCurrentUser.swift
187
MARK comment should be in valid format. e.g. '// MARK: ...' or '// MARK: - ...'

PCCurrentUser.swift
246
MARK comment should be in valid format. e.g. '// MARK: ...' or '// MARK: - ...'

PCCurrentUser.swift
709
Arguments can be omitted when matching enums with associated types if they are not used.

PCCurrentUser.swift
709
Arguments can be omitted when matching enums with associated types if they are not used.

PCCurrentUser.swift
5
Variable name should be between 3 and 40 characters long: 'id'

PCCurrentUser.swift
56
Variable name should be between 3 and 40 characters long: 'id'

PCCurrentUser.swift
157
Variable name should be between 3 and 40 characters long: 'id'

PCCurrentUser.swift
174
Variable name should be between 3 and 40 characters long: 'id'

PCCurrentUser.swift
208
Variable name should be between 3 and 40 characters long: 'id'

PCCurrentUser.swift
263
Variable name should be between 3 and 40 characters long: 'id'

PCCurrentUser.swift
323
Variable name should be between 3 and 40 characters long: 'id'

PCCurrentUser.swift
1044
Enum element name should be between 3 and 40 characters long: 'messageIdKeyMissingInMessageCreationResponse'

PCCurrentUser.swift
104
Collection literals should not have trailing commas.

PCCurrentUser.swift
545
Collection literals should not have trailing commas.

PCCurrentUser.swift
821
Collection literals should not have trailing commas.

PCCurrentUser.swift
759
Opening braces should be preceded by a single space and on the same line as the declaration.

PCCurrentUser.swift
178
Line should be 120 characters or less: currently 122 characters

PCCurrentUser.swift
196
Line should be 120 characters or less: currently 144 characters

PCCurrentUser.swift
208
Line should be 120 characters or less: currently 137 characters

PCCurrentUser.swift
212
Line should be 120 characters or less: currently 134 characters

PCCurrentUser.swift
510
Line should be 120 characters or less: currently 127 characters

PCCurrentUser.swift
575
Line should be 120 characters or less: currently 121 characters

PCCurrentUser.swift
615
Line should be 120 characters or less: currently 121 characters

PCCurrentUser.swift
666
Line should be 120 characters or less: currently 133 characters

PCCurrentUser.swift
677
Line should be 120 characters or less: currently 132 characters

PCCurrentUser.swift
744
Line should be 120 characters or less: currently 123 characters

PCCurrentUser.swift
920
Line should be 120 characters or less: currently 123 characters

PCCurrentUser.swift
1020
Line should be 120 characters or less: currently 136 characters

PCReadCursorDebouncerManager.swift
28
Initializing an optional variable with nil is redundant.

PCReadCursorDebouncerManager.swift
78
Lines should not have trailing whitespace.

PCReadCursorDebouncerManager.swift
11
Line should be 120 characters or less: currently 121 characters

Errors

File
Line
Reason

PCCurrentUser.swift
4
Type body should span 200 lines or less excluding comments and whitespace: currently spans 617 lines

PCCurrentUser.swift
1063
File should contain 400 lines or less: currently contains 1063

  Generated by :no_entry_sign: Danger


Add setting to use a play button instead of autoplaying

If Parchment is loaded in an iframe then it should display a "play this" message instead of starting automatically.

Ideally with a progress meter

A progress indicator has been added! 🎉
But see this topic for some pushback on the idea of a play button: https://intfiction.org/t/should-parchment-autoplay-or-have-a-play-button/66685

This will now be an option, allowing people to manually change it. By default we'll autoplay unless:

[ ] The file's download size is >10MB
[ ] The game plays sounds before user events
[ ] Parchment is loaded in an iframe


Crash when using Material 3 version 1.2.*

Describe the bug
The app will crash when using either of the following dependencies with versions higher than 1.1.1:
dependencies {
// Material 3
implementation("androidx.compose.material3:material3:1.2.0-alpha07")
implementation("androidx.compose.material3:material3-window-size-class:1.2.0-alpha07")
}
Error message: java.lang.NoClassDefFoundError: Failed resolution of: Landroidx/compose/runtime/PrimitiveSnapshotStateKt;
at ...
...Caused by: java.lang.ClassNotFoundException: Didn't find class "androidx.compose.runtime.PrimitiveSnapshotStateKt"
Module

[x] compose
[ ] views
[ ] compose-m2
[x] compose-m3
[ ] core (select if none of the above apply)

To reproduce
To reproduce the issue, use either of these dependencies:
implementation("androidx.compose.material3:material3:1.2.0-alpha07")
implementation("androidx.compose.material3:material3-window-size-class:1.2.0-alpha07")
Expected behavior
I expect the app not to crash.
Android versions
Android 13 ("Tiramisu"), API Level 33

Hello! Thanks for the report. This isn’t a bug in Vico, but rather a result of breaking changes in Jetpack Compose. We use the latest stable releases of the Compose libraries. Unfortunately, we have no way of guaranteeing compatibility with future releases, and neither would it be optimal for stable Vico versions to depend on unstable library versions. For Vico to work, you’ll have to downgrade your dependencies. As new stable releases of the Compose libraries are made available, we’ll update these libraries in Vico.

Thanks for the answer.

You’re welcome!


Black screen on SysNAND boot after downgrading

So, I'm currently at this black screen and i can't seem to do anything. I can't go into recovery and with or without sd card works. During the downgrading process everything said successful until the one click setup and i encountered the, "TPHelper fails to verify my NAND" but i followed the instructions correctly step by step and it said success. I have no idea what went wrong and completely sad i might have to buy a new 3ds.
Specs: New 3ds USA with 9.0 fw
all help is appreciated thanks.

Repeat steps 1 through 6, this is not optional (in other words, you must TinyFormat your RedNAND twice - if you do not, your device may brick when you downgrade), ah shit. i didnt do this and read it as optional. is there anything i can do with this brick n3ds?

So with or without sd card "doesnt" work? It seems you need to hardmod your device. As long as you have nand back ups, you can try doing a hard mod or let some one do it for you to flash the working nand back into the 3ds.

thanks for the reply, i do have my previous nands' from before the brick, so hard modding still works even after bricking?

New guide is up and all currently open issues are being closed. If you feel your issue still applies to the new guide, you can request for it to be re-opened by replying to it.


UVI Request

--- UVI JSON ---
{
  "vendor_name": "Linux",
  "product_name": "Kernel",
  "product_version": "versions from  to before v5.10.51",
  "vulnerability_type": "unspecified",
  "affected_component": "unspecified",
  "attack_vector": "unspecified",
  "impact": "unspecified",
  "credit": "",
  "references": [
    "https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git/commit/?id=21bf1414580c36ffc8d8de043beb3508cf812238"
  ],
  "extended_references": [
    {
      "type": "commit",
      "value": "21bf1414580c36ffc8d8de043beb3508cf812238",
      "note": "fixed"
    }
  ],
  "reporter": "joshbressers",
  "reporter_id": 1692786,
  "notes": "",
  "description": "udf: Fix NULL pointer dereference in udf_symlink function\n\nThis is an automated ID intended to aid in discovery of potential security vulnerabilities. The actual impact and attack plausibility have not yet been proven.\nThis ID is fixed in Linux Kernel version v5.10.51 by commit 21bf1414580c36ffc8d8de043beb3508cf812238. For more details please see the references link."
}
--- UVI JSON ---

This issue has been assigned UVI-2021-1001300


bump nix to 0.14 to fix compilation issues

~Waiting on: https://github.com/nix-rust/nix/issues/1060#issuecomment-494627579~
See also: https://github.com/rust-embedded/linux-embedded-hal/pull/17
Patching to git source tested to be working

Thanks for the pull request, and welcome! The Rust team is excited to review your changes, and you should hear from @nastevens (or someone else) soon.
If any changes to this PR are deemed necessary, please add them as extra commits. This ensures that the reviewer can see what has changed since they last reviewed the code. Due to the way GitHub handles out-of-date commits, this should also make it reasonably obvious what issues have or haven't been addressed. Large or tricky changes may require several passes of review and changes.
Please see the contribution instructions for more information.

nix 0.14.x has landed and CI seems happy ^_^

Was going to comment on MSRV stuff again but the MSRV here is 1.26.0 and the MSRV for nix is 1.24.1 so we should be good to go.

bors r+


no video in custom videoSource

Before filing an issue please check that the issue is not already addressed by the following:

Video Guides
Changelog.

If this is an issue with the QuickStart itself, file it here. If this is an issue with the SDK or how to do something with the SDK please use twilio-video-ios instead.
Please ensure that you are not sharing any
Personally Identifiable Information(PII)
or sensitive account information (API keys, credentials, etc.) when reporting an issue.

Description
create a custom videoSource  not shown  video using the QuickStart screencapturin example,
Steps to Reproduce

create a custom viewSource  using a view for Sketch a pod for drawing on the screen
run the demo app and the remote screen is black
added in pod file pod 'Sketch'

Code
import Foundation
import Accelerate
import TwilioVideo
import Sketch

@available(iOS 11.0, *)
class DrawViewSource: NSObject {

    // TVIVideoSource
    public var isScreencast: Bool = true
    public weak var sink: VideoSink? = nil

    // Private variables
    weak var view: SketchView?
    var displayTimer: CADisplayLink?
    var willEnterForegroundObserver: NSObjectProtocol?
    var didEnterBackgroundObserver: NSObjectProtocol?

    // Constants
    static let kCaptureFrameRate = 5
    static let kCaptureScaleFactor: CGFloat = 1.0

    init(aView: SketchView) {
        sink = nil
        view = aView
    }

    func startCapture() {
        if (view == nil || view?.superview == nil) {
            print("Can't capture from a nil view, or one with no superview:", view as Any)
            return
        }

        print("Start capturing.")
        startTimer()
        registerNotificationObservers()
    }

    func stopCapture() {
        print("Stop capturing.")

        unregisterNotificationObservers()
        invalidateTimer()
    }

    private func startTimer() {
        invalidateTimer()

        // Use a CADisplayLink timer so that our drawing is synchronized to the display vsync.
        displayTimer = CADisplayLink(target: self, selector: #selector(self.captureView))
        displayTimer?.preferredFramesPerSecond = DrawViewSource.kCaptureFrameRate
        displayTimer?.add(to: RunLoop.main, forMode: RunLoop.Mode.common)
        displayTimer?.isPaused = UIApplication.shared.applicationState == UIApplication.State.background
    }

    private func invalidateTimer() {
        displayTimer?.invalidate()
        displayTimer = nil
    }

    private func registerNotificationObservers() {
        let notificationCenter = NotificationCenter.default;

        willEnterForegroundObserver = notificationCenter.addObserver(forName: UIApplication.willEnterForegroundNotification,object: nil,queue: OperationQueue.main,using: { (Notification) in
            self.displayTimer?.isPaused = false;
        })

        didEnterBackgroundObserver = notificationCenter.addObserver(forName: UIApplication.didEnterBackgroundNotification,object: nil,queue: OperationQueue.main,using: { (Notification) in
            self.displayTimer?.isPaused = true;
        })
    }

    private func unregisterNotificationObservers() {
        let notificationCenter = NotificationCenter.default

        notificationCenter.removeObserver(willEnterForegroundObserver!)
        notificationCenter.removeObserver(didEnterBackgroundObserver!)

        willEnterForegroundObserver = nil
        didEnterBackgroundObserver = nil
    }

    @objc func captureView( timer: CADisplayLink ) {
//        guard let webView = self.view else {
//            return
//        }
//        guard let window = webView.window else {
//            return
//        }
        let renderer = UIGraphicsImageRenderer(size: view?.bounds.size ?? CGSize(width: 0, height: 0))
        let image = renderer.image { ctx in
            view?.drawHierarchy(in: self.view!.bounds, afterScreenUpdates: false)
        }
        //let image = view?.takeScreenshot() ?? UIImage()
        print("image",image.size)
        self.deliverCapturedImage(image: image, orientation: VideoOrientation.up, timestamp: timer.timestamp)

//        let configuration = WKSnapshotConfiguration()
//        // Configure a width (in points) appropriate for our desired scale factor.
//        configuration.snapshotWidth = NSNumber(value: Double(webView.bounds.width * ExampleWebViewSource.kCaptureScaleFactor / window.screen.scale))
//        webView.takeSnapshot(with:configuration, completionHandler: { (image, error) in
//            if let deliverableImage = image {
//                self.deliverCapturedImage(image: deliverableImage,orientation: VideoOrientation.up,timestamp: timer.timestamp)
//            } else if let theError = error {
//                print("Snapshot error:", theError as Any)
//            }
//        })
    }

    private func deliverCapturedImage(image: UIImage,orientation: VideoOrientation,timestamp: CFTimeInterval) {
        /*
         * Make a (deep) copy of the UIImage's underlying data. We do this by getting the CGImage, and its CGDataProvider.
         * In some cases, the bitmap's pixel format is not compatible with CVPixelBuffer and we need to repack the pixels.
         */
        guard let cgImage = image.cgImage else {
            return
        }

        let alphaInfo = cgImage.alphaInfo
        let byteOrderInfo = CGBitmapInfo(rawValue: cgImage.bitmapInfo.rawValue & CGBitmapInfo.byteOrderMask.rawValue)
        let dataProvider = cgImage.dataProvider
        let data = dataProvider?.data
        let baseAddress = CFDataGetBytePtr(data!)!
        /*
         * The underlying data is marked as immutable, but we are the sole owner and can do as we please.
         * Also, the CVPixelBuffer constructor will only accept a mutable pointer.
         */
        let mutableBaseAddress = UnsafeMutablePointer<UInt8>(mutating: baseAddress)
        var pixelFormat = PixelFormat.format32BGRA

        var imageBuffer = vImage_Buffer(data: mutableBaseAddress,height: vImagePixelCount(cgImage.height),width: vImagePixelCount(cgImage.width),rowBytes: cgImage.bytesPerRow)

        switch byteOrderInfo {
        case .byteOrder32Little:
            /*
             * Pixel format encountered on iOS simulators. Note: We have observed that pre-multiplied images
             * do not contain any transparent alpha, but still appear to be too dim. This appears to be a simulator only bug.
             * Without proper alpha information it is impossible to un-premultiply the data.
             */
            assert(alphaInfo == .premultipliedFirst || alphaInfo == .noneSkipFirst)
        case .byteOrder32Big:
            // Never encountered with snapshots on iOS, but maybe on macOS?
            assert(alphaInfo == .premultipliedFirst || alphaInfo == .noneSkipFirst)
            pixelFormat = PixelFormat.format32ARGB
        case .byteOrder16Little:
            print("error")
            //assert(false)
        case .byteOrder16Big:
            print("error")
            //assert(false)
        default:
            /*
             * The pixels are formatted in the default order for CoreGraphics, which on iOS is kCVPixelFormatType_32RGBA.
             * This pixel format is defined by Core Video, but creating a buffer returns kCVReturnInvalidPixelFormat on an iOS device.
             * We will instead repack the memory from RGBA to BGRA, which is supported by Core Video (and Twilio Video).
             * Note: While UIImages captured on a device claim to have pre-multiplied alpha, the alpha channel is always opaque (0xFF).
             */
            assert(alphaInfo == .premultipliedLast || alphaInfo == .noneSkipLast)

            // Swap the red and blue channels.
            var permuteMap = [UInt8(2), UInt8(1), UInt8(0), UInt8(3)]
            vImagePermuteChannels_ARGB8888(&imageBuffer,&imageBuffer, &permuteMap,vImage_Flags(kvImageDoNotTile))
        }

        /*
         * We own the copied CFData which will back the CVPixelBuffer, thus the data's lifetime is bound to the buffer.
         * We will use a CVPixelBufferReleaseBytesCallback in order to release the CFData when the buffer dies.
         */
        let unmanagedData = Unmanaged<CFData>.passRetained(data!)
        var pixelBuffer: CVPixelBuffer?
        let status = CVPixelBufferCreateWithBytes(nil,cgImage.width,cgImage.height,pixelFormat.rawValue,mutableBaseAddress,cgImage.bytesPerRow,{ releaseContext, baseAddress in
            let contextData = Unmanaged<CFData>.fromOpaque(releaseContext!)
            contextData.release()
        },unmanagedData.toOpaque(),nil,&pixelBuffer)

        if let buffer = pixelBuffer {
            // Deliver a frame to the consumer.
            let frame = VideoFrame(timeInterval: timestamp, buffer: buffer,orientation: orientation)

            // The consumer retains the CVPixelBuffer and will own it as the buffer flows through the video pipeline.
            self.sink?.onVideoFrame(frame!)
        } else {
            print("Video source failed with status code: \(status).")
        }
    }
}

@available(iOS 11.0, *)
extension DrawViewSource: VideoSource {
    func requestOutputFormat(_ outputFormat: VideoFormat) {
        /*
         * This class doesn't explicitly support different scaling factors or frame rates.
         * That being said, we won't disallow cropping and/or scaling if its absolutely needed.
         */
        self.sink?.onVideoFormatRequest(outputFormat)
    }
}
extension UIView {

    func takeScreenshot() -> UIImage {

        // Begin context
        UIGraphicsBeginImageContextWithOptions(self.bounds.size, false, UIScreen.main.scale)

        // Draw view in that context
        drawHierarchy(in: self.bounds, afterScreenUpdates: true)

        // And finally, get image
        let image = UIGraphicsGetImageFromCurrentImageContext()
        UIGraphicsEndImageContext()

        if (image != nil)
        {
            return image!
        }
        return UIImage()
    }
}

changed webView = SketchView(frame: view.frame) in viewController viewDidLoad
removed the web view load code and url load code

Expected Behavior
see a white screen with the draw in the screen
Actual Behavior
draw is not shown remote screen is black
Reproduces How Often
always
Logs
// Log output when the issue occurs

Versions
All relevant version information for the issue.
Sketch Version
3.0 via CocoaPods
Video iOS SDK
3.1.0  via CocoaPods
Xcode
11.3
iOS Version
13.3
iOS Device
iPhone 7 Plus

Hi @kjoe07,
Sorry for the late response.
The technique of using UIGraphicsImageRenderer to draw the screen was extensively discussed in https://github.com/twilio/video-quickstart-ios/issues/303. We found that it was impossible to get reasonable performance in iOS 12+, which is what lead us to deprecating TVIScreenCapturer. If you are really interested in this technique you can probably borrow a working implementation from what I have posted in the issue in: https://github.com/twilio/video-quickstart-ios/issues/303#issuecomment-424489669.
Thanks,
Chris

ok thanks for response, I'm already trying replay kit to get it work

You are welcome. Let us know if you have any questions about ReplayKit.

i don't know is it ok to ask in here but I have some issues with replayKIt.

Hey @kjoe07 can you open a new issue for ReplayKit? Or if you notice an existing open issue which is similar you can post there.
Thanks!