Issue importing SharePoint search schema

Target SharePoint environment
SharePoint Online
What SharePoint development model, framework, SDK or API is this about?
SharePoint CSOM
Developer environment
Windows
What browser(s) / client(s) have you tested

[ ] 💥 Internet Explorer
[ ] 💥 Microsoft Edge
[ ] 💥 Google Chrome
[ ] 💥 FireFox
[ ] 💥 Safari
[ ] mobile (iOS/iPadOS)
[ ] mobile (Android)
[X] not applicable
[ ] other (enter in the "Additional environment details" area below)

Additional environment details
No response
Describe the bug / error
We were importing the search schema without any issue for months. Last 2 weeks ago, we got an error trying to use the same search schema without any change applied. The error was "Bad request: Please export a new search configuration firstly and then use it to import if you are improting a stale exported search configuration file". Even using the UI was failing, not just with the code. It happened just in one tenant, but now happening in 3-4 tenants.
Steps to reproduce

Import and old search schema

Expected behavior
We were expecting the search schema to be imported without issue.

I have more information, finally we managed to import the search schema, we exported from the tenant and we made some amends into our search schema that we are provisioning with code. After that, the error "Bad request: Please export a new search configuration firstly and then use it to import if you are improting a stale exported search configuration file" disappeared but a new one was coming "Microsoft.SharePoint.Client.ServerException : Bad request while trying to reach endpoint POST Call to Setting Type Bulk, Please try again, SSMS request".
After reviewing it, looks like it's something about the result sources, if we put the same id of the result source from the tenant in the xml, no error is coming but this is not good, as we were deploying the search schema in a lot of clients and the result sources ids were created without taking into account what is in the XML. I guess, that the easy option is to remove the result sources from the tenant and provision them again, but I'm not even allowed to remove the result source from there, the page is refreshed without removing it and getting 302 status error from the Network.

My client is encountering this issue, too. This code has been running fine for years.
Here is the CSOM:
Microsoft.SharePoint.Client.SearchExtensions.SetSearchConfigurationImplementation(ClientRuntimeContext context, SearchObjectLevel searchObjectLevel, String searchConfiguration)

Hello, I have encountered this issue as well on multiple tenants. Same error from CSOM as above.

FYI: Strange behaviour on our side as well :-(.
We have an issue with Result Sources (started on Tuesday last week). The settings that has been exported months ago does not work anymore. Newly exported search settings from on site cannot be imported to another if there are Results sources defined already (update between environments, error message "Source with nam  already exists").
However, the import works for fresh sites or after we removed existing Result Sources (it was required to remove existing Result Sources before first import only)

@VesaJuvonen can provide an update and ETA on a fix? Thanks.

For us is happening in a fresh tenant as well, the problem is with the result sources, if we remove them from the schema imported, it works. We managed to import it with the result sources if the result source id is the same from result source deployed in the tenant, but this was not required in the past, even if you deployed a result source with an id, it was provisioned with a different one.

@wobba are you aware about this issue about the provisioning of search schema? we have raised a ticket with Microsoft support but there is no luck in providing a fix or providing the steps to remediate it.

I think I've run into a similar issue now. My use case would be to update 50 result sources with a breaking change in the search query syntax.
Exporting the config from tenant A works.
Doing changes to result sources and reimporting to tenant A doesn't update anything.
Importing the config to tenant B works. And I am also able to do additional changes within that search config xml and reimport to tenant B.
I've managed to track down the issue to to the resultsource Id. If I'm creating the result source programmatically from another tenants XML, as long as I use the same Result Source Id I used when first importing the result source is updated. Even though the generated resultsource Id is different from the one in the imported configuration xml.
So technically I'm able to update result sources programmatically within a tenant if I create the configuration first on tenant A, and then keep importing that configuration XML to tenant B while doing changes on the XML.

Currently having the same issue with importing an XML search configurations. I have tried this both manually and utilising PnP PowerShell I get the same issue regardless with the error message appearing as:
Set-PnPSearchConfiguration: Bad request: Please export a new search configuration firstly and then use it to import if you are improting a stale exported search configuration file
I have tried exporting search schemas from two different tenants and importing these in to seperate tenants and the same issue occurs. When I utilise PnP PowerShell as follows:
Set-PnPSearchConfiguration -Path ".\SearchConfiguration.xml" -Scope Site
Additionally if I try to import a PnP XML site template I get the same error in the console when the search configuration is imported.
I am trying to create an automated process within Azure Functions whereby a new site is created and the search is configured, any ideas on a work around for this would be great.

@adam13hylo I suggest you log this as a support ticket with Microsoft, and when repro'ing the issue use the import/export UX in SharePoint and not the PnP cmdlet's. Anything API/PowerShell takes longer to get thru.

Getting this now as well. This is a pretty new site collection, and the first time I'm making Search schema changes, just in case it helps.

Hello everyone,
I seem to be getting the same issue in a relatively fresh tenant. No result sources to import, just property mappings and failing every time.

Same issues on our side. We have problems on 2 out of 3 tenants. Some problems started around March/April but we managed to workaround them (it was about change in default value for Result Types processing parameter and the issue with Query Rules conditions - specifically condition to fire on give result source that is no longer working).
This week huge problems with importing/exporting search configuration started on those 2 same tenants. Basically things that have been working for years without problems suddenly stopped. Exporting search configuration with CSOM (on fresh site with one single result source created manually in GUI) do not work since this week - and the same was fine for years. We get meaningless 'something went wrong' without any details while trying to access Query Rules view in Site Settings.
Interestingly all those issues thankfully not reached our third tenant (we have situation in which tenants 'differ 3 months' in terms of some 'search flavors' and damaged functionalities).

@dev-a11y it would help if you could open a support ticket for each affected tenant.

@ronaldvanhelden you should log a support ticket.

Any updates?

@rodasasamoila please log a support ticket with Microsoft for this issue.

In my case, I have seen some issue related to searchconfiguration.xml file.
Exported searchconfiguration.xml from "A" tenant.
Before importing the searchconfiguration.xml to "B" tenant. I made changes to the values in the SchemaId for two xml tags mentioned below. The below two xml tags will be repeated multiple times and should have same value consistent through out the entire "searchconfiguration.xml" file.
Tag1:
d3p1:SchemaId 0 </d3p1:SchemaId>
Tag2:
d5p1:SchemaId 0 </d5p1:SchemaId>
The value may not be 0 always  but it should have any value that should be consistent in the entire file.
This one resolved the issue for me : "Bad request: Please export a new search configuration firstly and then use it to import if you are importing a stale exported search configuration file".

We are getting an internal server error now when trying to import a search configuration into the tenant. This seems to happen on all tenants and there is no difference if you are doing it via UI or code...

Getting an internal server error too..

We managed to fix the error "Bad request: Please export a new search configuration firstly and then use it to import if you are improting a stale exported search configuration file" changing the schema, now getting Internal Server Error without any clarifications.

This workaround does not work. You can export the search config from a brand new tenant and import it into the same tenant again and it will fail.
@wobba can you have a look at this and let us know here if you can confirm this issue and if there is work beeing done to get this resolved?

@wobba are there any plans to issue a fix? What does logging a support ticket accomplish?

We have opened tickets with Microsoft from different tenants to elevate the issue urgency, if you are opening tickets as well, please share your ids if you can to pack them to show Microsoft that this is affecting everyone.

I have created a ticket with microsoft [Case #:32656650] still work in progress on their side

I have news about this, we can import now the schema without issues, new result sources are imported fine, existing result sources  are updated fine, but if we delete an existing result source and we try to deploy it again using the schema, they are deployed but we are getting Sorry, something went wrong from the UI when we try to access to it

Should be fixed as of 9/12.


[Hammerhead - Cortex Prime - New Character Sheet]

Changes / Comments
New Character Sheet - Supporting the Cortex Prime Spotlight Hammerhead setting.
Roll20 Requests
Comments are very helpful for reviewing the code changes. Please answer the relevant questions below in your comment.

[ ] Does the pull request title have the sheet name(s)? Include each sheet name.
[ ] Is this a bug fix?
[ ] Does this add functional enhancements (new features or extending existing features) ?
[ ] Does this add or change functional aesthetics (such as layout or color scheme) ?
[ ] If changing or removing attributes, what steps have you taken, if any, to preserve player data ?
[ ] If this is a new sheet, did you follow Building Character Sheets standards ?

If you do not know English. Please leave a comment in your native language.

Character Sheet Info Roll20 Internal Use only.


Enable retrieving user status string from presence for online users

Housekeeping job for pending requests.
Requested by dillysfriend on Apr 4th, 2022, here.
Reference: none?

Since this can be done at the same time as #99, it's better to close this issue and handle everything there instead. We can get the data, it's only a matter of writing the PR for it now.


Support customizing client accessibility

Fixes https://github.com/microsoft/typespec/issues/4602

API change check
APIView has identified API level changes in this PR and created following API reviews.
Microsoft.Generator.CSharp


fix compile agent error

This PR is for:

Documents

Fixes <bug description, issue number or issue link>
the command to compile agent in build.md is wrong.
Steps to reproduce the bug
/deepflow]#docker run --privileged --rm -it -v     $(pwd):/deepflow -v ~/.cargo:/usr/local/cargo hub.deepflow.yunshan.net/public/rust-build bash -c     "cd /deepflow/agent && cargo build"
bash: cargo: command not found
Changes to fix the bug
rm -v ~/.cargo:/usr/local/cargo param
Affected branches

main

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


Product Searching

Description:
Implement functionality to allow users to search for products by name, category, or price range.
SP: 0
Acceptance Criteria:
Users can search for products using various criteria and receive accurate results.

Testing phase

Bug in search by price range
Search by name needs to be more lenient


[app/job]: change secure parameter to a custom type

Check for previous/existing GitHub issues

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

Issue Type?
Feature Request
Module Name
avm/res/app/job
(Optional) Module Version
No response
Description
The secrets parameter is defined as an object and contains a secureList property. This should be changed to a custom type to improve syntax highlighting and get rid of the workaround introduced earlier.
#2070
(Optional) Correlation Id
No response

Waiting for merge of #2074

Removing the Needs: Triage label to avoid a flood of comments ;)


[WIP] New Benchpark command: "system-create"

Add command that can be used to instantiate a new system definition, for example:
$ benchpark system-create --from=HPECray-zen3-MI250X-Slingshot cray1
$ ls configs/cray1
auxiliary_software_files  spack.yaml  system_definition.yaml  variables.yaml

$ benchpark system-create cray2 --template=templates/scheduler/slurm/ --template=templates/generic-system/cray/
$ ls configs/cray2
spack.yaml  variables.yaml

This will often require an intermediate editing step, in particular to define auxiliary_software_files; after that, you could e.g. do:
$ benchpark setup amg2023/rocm cray2...

Broadly:

You can copy an existing config with --from
Or you can assembly a generic configuration using a series of --template options

Current issues:

[ ] Right now, if two templates define the same file name, the last one will overwrite all prior
[ ] I've added some initial entries to templates/ to give a notion of how they are supposed to be used in combination with the system-create command, but there's not enough there to be actually useful yet

@slabasan to migrate boilerplate system_definition.yaml from https://github.com/LLNL/benchpark/pull/93 to this PR


Not able to upload Teams Package -Step 5

I get the following error when I try to upload the teams package Into Microsoft Teams-
Error while extracting package: Please ensure there are no folders in your app package and that your app package contains only the manifest.json and the two required icon files at its root level.
I have ensured that there are no folders or subfolders in the package. Any Ideas on how can this be resolved?

This was resolved. When I extracted the folder, I actually found out that there was a sub folder. So, in order to resolve this, I followed these steps-

Select all the files that need to be zipped
Right Click
Send To->Compressed Folder


Sinope Thermostat

I have a Sinope Thermostat (for baseboard heaters) installed in SmartThings.  I am trying to change the time format on the device from 24h to 12h AM/PM. In the 'Settings' I can change the time format from 24h to 12h AM/PM, but it has no effect on the display. It stays at the 24h format.   I downloaded your 'Zigbee Thermostat MC' driver, but there is no listing for 'Settings'.   Can you do an update on your Driver to accommodate this?
Thank you for all the work you have put into these Drivers.   I, and many others appreciate your commitment.
TIA

Hi,
I don't know which model you are referring to, what fingerprints?.
In the driver there are 3 profiles of Sinope thermostats, which have the time format setting in the preference settings.


WebP plugin doesn't work in M1-updated Photoshop

I followed the instructions for installing the WebP plugin. It is in the plugin folder of Photoshop on my M1 Mac mini. I ran the terminal command you suggested.
Though the plugin is in the correct folder, it does not show up in the About > Plugins menu. Only Camera Raw does.
I attempted to install the day after the Photoshop update for M1 Macs was released. I assume the plugin isn't compatible with the newly updated version?

May I know which command you ran? The one described in the README.md was for Photoshop 2020, so you may need to replace "2020" by "2021":
sudo xattr -r -d com.apple.quarantine /Applications/Adobe\ Photoshop\ 2021/Plug-ins/WebPShop.plugin

Also it might help to know the exact Photoshop version.

The plugin is not working natively on Apple M1 silicon yet, it seems. If you open the Photoshop info with Command+I and tick the "open with Rosetta" checkbox it works. Let's hope there will be a universal binary or Apple version soon…

@choegl - that worked for me, thanks!

If all you're needing WebPShop for is to convert it to a JPEG or PNG, I found that Apple's Preview app will recognize the WebP format and allow you to convert it to a JPEG or PNG by converting it to a TIFF.

The plugin is not working natively on Apple M1 silicon yet, it seems. If you open the Photoshop info with Command+I and tick the "open with Rosetta" checkbox it works. Let's hope there will be a universal binary or Apple version soon…

The latest 22.3 version with native M1 support doesn't have the checkbox for opening with Rosetta anymore.

We've added a candidate "universal" build with M1 support on the latest release  https://github.com/webmproject/WebPShop/releases/tag/v0.3.3
Could you try it and let me know if it works?

Just tested it. Works like a charm! (once I was able to get around the pesky Apple safe keeper and allowing access to the plugin file in the system settings security section)

Thank you, glad to know it works! I'm closing the issue then.


Add missing injections

Added missing injections on FederatedCatalogCacheExtension that caused NPE on extension initialization.
Removed a duplicated publishing block on the extension build file (rebase typo?)

Closing as these changes were included in #486.


Problem with max_pods

Hi team, when using EKS clusters, makes no sense to set max_pods because the size of the ec2 instance implies the maximum number of ENI that can be attached to a node, see:
https://github.com/awslabs/amazon-eks-ami/blob/master/files/eni-max-pods.txt
We should use this list, maybe create a map and use the value from the map to set the max pods for the nodes

Does it also affect if you changed the default cni to canico?

AFAIK we don't change CNI to calico on EKS

No, we stick to VPC CNI. On EKS there is no need to expose such variable.

Ye, the idea to expose that variable is to enable changing the CNI from VPC CNI to any other (calico mostly).
Then, what we can do is to make that variable somehow optional. What do you think?

Ye, the idea to expose that variable is to enable changing the CNI from VPC CNI to any other (calico mostly).
Then, what we can do is to make that variable somehow optional. What do you think?

This is a problem using EKS. We should use the official values https://github.com/awslabs/amazon-eks-ami/blob/master/files/eni-max-pods.txt or, omit the max_pods completely if possible.
We use calico on EKS only to add networkpolicy capability.

Addressed on v1.8.1 in:

AKS
EKS
GKE

SIGHUP cloud installers :)


Update installation-options.md

Add a note to the effect that Admin Center is non-operational using the default browser (IE). Whilst this needs to get fixed in due course, in the meantime a note to let users know that IE doe\s NOT work with AC.

Thanks for adding this clarification! I'm going to run the language through the WAC team and will reply back here with any changes. Thanks, Liza

That's fine Lisa. If you or they can improve the language, then please do so.

Hi - the team have updated this article with the following information:
Important
You can't use Internet Explorer to manage Windows Admin Center - instead you need to use a supported browser. If you are installing Windows Admin Center on Windows Server, we recommend managing by connecting remotely with Windows 10 and Edge. Alternatively, you can manage locally on Windows Server if you have installed a supported browser.
Because it addresses the same thing as your PR, I'm going to close out this PR. Thank you SO much for raising the issue!


fix issue with updating external file located on mixed-case path

Because the GBFR takes lowercase filepath as the input of hash function, not original filepath (may mixed-case).
Hence,
current tool-chain will add new external record, not update existing record what we expected in index-file.

Good spot, thanks!


Support interruptible sections inside .uninterruptible

A solution for #574 that adds the ability to have interruptible blocks of code inside of.uninterruptible.
If you look at ZIO's implementation of bracket
final def bracket[E, A, B](
    acquire: IO[E, A]
  )(release: A => IO[Nothing, _])(use: A => IO[E, B]): IO[E, B] =
    Ref.make[IO[Nothing, Any]](IO.unit).flatMap { m =>
      (for {
        a <- acquire.flatMap(a => m.set(release(a)).const(a)).uninterruptible
        b <- use(a)
      } yield b).ensuring(IO.flatten(m.get))
}

you will see that .uninterruptible is applied too conservatively, to both flatMap, which we absolutely need for the releasing logic to work, and to acquire, which might actually contain some interruptible steps. This leads to Fibers locked in acquiring resources no longer needed by the program.
This PR adds .interruptible function that

Inside a single .uninterruptible, makes the action it is applied to interruptible, but guarantees that no interruption will happen right after action produces a result. For example, infoo.interruptible.flatMap(result => bar(result)).uninterruptible

foo will be interruptible, but as soon as it produces a result (using Point, SyncEffect, AsyncEffect, etc), the fiber will become uninterruptible.
Otherwise behaves as the action itself, e.g. io.interruptible is the same as io when the fiber is already interruptible, and io.interruptible.uninterruptible.uninterruptible is the same as io.uninterruptible.uninterruptible.

Unfortunately, the current AsyncEffect implementation contains a hidden flatMap that can be interrupted inside an .interruptible without any user control over it. I've baked in a hack that prevents such interrupts from happening when async0's callback returns a Point (e.g. in asyncInterrupt), but it is not a good solution. I would propose changing async* functions callback signatures to work with Either instead of IO. The old behavior could be when recovered, but with actual user control over what happens with that flatMap vs interruptibility:
object Async {
  def now[E, A](result: Either[A, E]): Async[E, A]
}
def async0[A, E](register: (Either[A, E] => Unit) => Async[E, A]): IO[E, A]
def asyncReturningIO =
  async0(k => ...)[.interruptible].flatMap { io =>
    io[.interruptible]
  }[.uninterruptible]

Wow, Promise.done is unsafe to interrupt! Will submit a PR with just that fix.

@jdegoes Any chance of giving this PR a thought before 1.0? I feel that it will be harder to land afterwards.

@mvv Sure. The problem is that it's not very sound.
To understand this, let's take a look at this:

Inside a single .uninterruptible, makes the action it is applied to interruptible, but guarantees that no interruption will happen right after action produces a result.

OK, let's imagine that inside acquisition, it has already acquired the resource, but has not yet produced it (i.e. it is still "inside" acquisition). In this case, acquisition can be interrupted, without running the release; the release cannot be run, because even though the acquisition has produced it, it hasn't yet returned it. So the acquisition will be interrupted, the release will not be run, and the resource will be leaked.
There is a subset of problems that can support interruptible acquisition. Managed models this quite well—it's a generalization of bracket.
Have you looked to see if Managed can solve your current problem?

@jdegoes

OK, let's imagine that inside acquisition, it has already acquired the resource, but has not yet produced it (i.e. it is still "inside" acquisition).

Yes, when the monad has the "interruptible by default" semantics, developers have to think carefully about appropriate interruption points in the code they write (consider, for example, the bug in Promise.done). That's the price for the feature. The status quo is to have "special" IO subroutines that are not interruption-safe, but work when used with bracket through the brute force of .uninterruptible. I would argue that every subroutine should be interruption-safe, and the .interruptible/.uninterruptible pair is a tool to achieve that. For example, if you are writing an acquisition function that does some post-processing of the acquired resource:
def acquire(a): IO[R] = preProcessing(a).flatMap { b =>
  allocate(a, b).interruptible
    .flatMap { r =>
      postProcessing(r)
    }
    .uniterruptible
}

Here we have reasoned about appropriate interruption points locally, knowing what preProcessing, allocate and postProcessing do. As the result, acquire is now safe to use in any context.
One corollary to this approach is that combinators should almost never use .uninterruptible on their arguments, instead they should focus on interruptibility of the code that combines arguments into the result.
Managed would still have its place as a delayed version of nested bracket, but without overuse of .uninterruptible. The current form of Managed.make is as bad as the bracket itself.
Also, resource allocation is not the only type of tasks that are sensitive to interruptions. We could have, for example, a multi-stage state transitions with designated interruption/rollback points.

Just to avoid confusion: postProcessing in the example above could also contain interruptible sections (guarded with .onInterrupt(release(r))) and points of failure (again, guarded).

@mvv
Right now, ZManaged does not have great documentation. All the documentation is written in terms of ZManaged.make.
But the constructor can be used directly:
case class ZManaged[R, E, A](reserve: ZIO[R, E, ZManaged.Reservation[R, E, A]])

In this form, you provide an effect that, when run, produces a Reservation.
The Reservation has an acquire effect, and a release effect.
The interesting thing about this formulation is that acquire can be interruptible.
Can you try this and see if it solves this problem? We could always support this API directly in ZIO, e.g:
ZIO.reserve(reservation)(resource =>
  ...
)

bracket, after all, is not primitive in ZIO, it's just a helper function. Could be the same with reserve, above.

@mvv By the way, I am still interested in the possibility of interruptible. It does introduce potential for space leaks though, which is why I'd first like to explore if reserve can work.

Closing for now in favor of #696, but still quite open to this idea!

@jdegoes
I think we got a bit sidetracked by the issue of changing bracket semantics, which in fact is a separate issue - you can have .interruptible and keep bracket fully uninterruptible in acquire. I think that .interruptible has merits that go beyond resource allocation:
Firstly, it can be used to simplify bracket, bracketExit and effectAsyncM implementations. For example, the current bracket can be rewritten as a much more readable:
final def bracket[R >: LowerR, E <: UpperE, A, A1 >: A, A2 >: A, B](
    acquire: ZIO[R, E, A],
    release: A => ZIO[R, Nothing, _],
    use: A => ZIO[R, E, B]
  ): ZIO[R, E, B] =
     for {
       r <- environment[R]
       b <- acquire.flatMap(a => use(a).interruptible.ensuring(release(a).provide(r))).uninterruptible
     } yield b

Look ma no Refs!
Secondly, it can be used in situations that do not follow the resource allocation pattern. For example, I've recently found myself writing
def serveReqs(reqs: Queue[Req]): IO[E, Unit] = {
  def loop: IO[E, Unit] = {
    reqs.take.interruptible.flatMap {
      case Req1(...) =>
        processReq1
      case Req2(..) =>
        for {
          _ <- startProcessingReq2
          _ <- longOperation.interruptible.onInterruptOrError(abortReq2)
          _ <- finishProcessingReq2
       } yield ()
    }.uninterruptible *> loop
  }
  loop
}

I don't see how it can be expressed via Managed (surely not without sacrificing readability), it is a different pattern, but it is still interruption-sensitive and requires some combinators to control it.

@mvv You might be onto something, I need to see if the semantics permit the simpler definition of bracket you've sketched. Anyway, I probably won't have time until this weekend to take a longer look.


Updated ConfigurationFile.md file

Fixes issue #613
Added the following paragraph in the ConfigurationFile.md file:
**Note**: It's important to note the order in which DeepForest searches for the deepforest_config.yml file. First it looks for the file in your current working directory. If it can't find the file there, it defaults to using the default config.

Thanks @Om-Doiphode!


Enable vcpus number for virtual provisioning step

Hi, currently it seems that virtual provisioning step has only 2 vCPUs. That is quite low on the current HW and more demanding tests might be quite slow thanks to this.
Could we add a possibility to specify number of vCPUs?

I believe we can.


Bug running NodeJS API on Windows and WSL

Verify canary release

[X] I verified that the issue exists in the latest Turborepo canary release.

Link to code that reproduces this issue
https://github.com/gtoro98/turbo_issue
What package manager are you using / does the bug impact?
npm
What operating system are you using?
Windows
Which canary version will you have in your reproduction?
1.13.1-canary.2
Describe the Bug
NodeJS Fastify server fails to run locally from turborepo on windows systems
Expected Behavior
Server to start
To Reproduce
run npm run dev on issue repo
Additional context
Version 1.12 works fine, this issue only appears to happen on Windows, no problem on M chip macs. Tested with WSL with Ubuntu distro and issue also happens

Can you include how it is failing? The repro you provided seems to be working fine on my Windows machine:

Information such as which console you are running from and Windows build version could also help.

Can you include how it is failing? The repro you provided seems to be working fine on my Windows machine:
Information such as which console you are running from and Windows build version could also help.

It looks like it is running but none of the commands are excecuting and the server isn't running

@gtoro98 could you also provide the console you're using as well as your windows build version?
Thanks!

@tknickman OS Build 22631.3296 but have reproduced in various others on windows powershell, command prompt and terminal

@chris-olszewski running 'npm run dev' from within the apps/api directory results in succesful starting of the server

Closing in favor of #7834 as I believe this is an bad interaction with turbo and tsx on Windows.


Release training codes

It would be great if you could release the training codes as well.

Sorry that I cannot upload my training code since the privacy of the company.
I am pleased to answer any questions about the training problems in email.
You can use the pre-trained models for any evaluation, too.


Updates UseCensusData component - ticket#416

What
Updates the way the <UseCensusData /> component displays on the map screen to match the V4 Design.

signed off by design


Would CollabVM count as a web desktop?

Hi!
I run a site called CollabVM, and I was wondering if it would count as a "web desktop", as the website runs several collaborative desktop virtual machines that anyone can control.
Here's the link: https://computernewb.com/collab-vm/
Do let me know, cause I'd love to add it here.

Hi there!
I came across your fantastic project a while ago, it's really impressive! Unfortunately though, I prefer adding projects which don't involve Virtual Machines but more web technologies for building web desktops.
I really admire your project and good luck with the development!


[WIP] Tests Progress

DO NOT MERGE THIS REQUEST, IT SERVES PURPOSE OF TRACKING PROGRESS, PLEASE
Preparation

[x] Setup Jest and add tests scripts & coverage report
[x] Enable continuous integration

Tests

[x] Library exports
[x] TARGETS
[x] PROPERTIES
[x] PROPERTY PARAMETERS
[x] ANIMATION PARAMETERS
[x] VALUES
[x] KEYFRAMES
[x] STAGGERING
[x] TIMELINE
[x] CONTROLS
[x] CALLBACKS & PROMISES
[x] SVG
EASINGS
[x] HELPERS

Coverage Report
https://codecov.io/gh/mubaidr/anime/branch/tests

@juliangarnier Currently this issue blocks any advancement for svg tests, since the method used is not standard and deprecated it is not available in jsdom, also it will be removed in future browser versions and will break svg features: https://github.com/juliangarnier/anime/issues/275
We have currently ~81% coverage. Which will be improved over the time.
I am re-basing this pull-request to fix merge issues.

Any update?

No love for this effort? :(

No love for this effort? :(

Not at all, but will wait for the next major release to merge the tests.

Cool. Let me know how can these tests can be further improved.


Created a separate integration test suite

Why?

avoid running integration tests locally

What?

defined an intTest sourceset
added an integrationTest task
configured it to only run in GitHub PRs

closes #45

How?

Checklist
Please ensure the following conditions are met:

[x] I have added tests that prove my fix is effective or that my feature works
[ ] I have added any necessary documentation (if appropriate)

Thank you!

This PR does whitebox testing on an integration test, which is not ideal.
TODO: move to black-box testing in #14.


Update HTML.php

Thanks. This needs a reasonable commit summary and update of the unit tests so they pass.


Very slow performance on git clone to a mounted folder

Very slow performance on git clone with a thousands of small files.

Mount a folder using AWS MemoryDB Redis as metadata database on EC2 instance on same subnet
Enter into the mounted folder and execute time git clone https://github.com/thiagoalvescosta/jose123.git
Result:
real	2m12.502s
user	0m3.184s
sys	0m3.836s
Enter into /tmp/ folder and execute time git clone https://github.com/thiagoalvescosta/jose123.git
Result:
real	0m2.848s
user	0m2.348s
sys	0m0.912s

The difference is very big. I don't know why. I also tried with --writeback and was the same
I executed: juicefs bench /home/ubuntu/juicefs -p 4 and got:
  Write big blocks count: 4096 / 4096 [==============================================================]  done    Read big blocks count: 4096 / 4096 [==============================================================]  done Write small blocks count: 400 / 400 [==============================================================]  done  Read small blocks count: 400 / 400 [==============================================================]  done   Stat small files count: 400 / 400 [==============================================================]  done Benchmark finished! BlockSize: 1 MiB, BigFileSize: 1024 MiB, SmallFileSize: 128 KiB, SmallFileCount: 100, NumThreads: 4 Time used: 37.5 s, CPU: 93.9%, Memory: 532.3 MiB +------------------+------------------+---------------+ |       ITEM       |       VALUE      |      COST     | +------------------+------------------+---------------+ |   Write big file |     355.08 MiB/s |  11.54 s/file | |    Read big file |     243.04 MiB/s |  16.85 s/file | | Write small file |    140.7 files/s | 28.43 ms/file | |  Read small file |    594.6 files/s |  6.73 ms/file | |        Stat file |   1646.3 files/s |  2.43 ms/file | |   FUSE operation | 75643 operations |    1.96 ms/op | |      Update meta |  1277 operations |    7.64 ms/op | |       Put object |  1424 operations |  152.35 ms/op | |       Get object |     0 operations |    0.00 ms/op | |    Delete object |     0 operations |    0.00 ms/op | | Write into cache |  1424 operations |  218.87 ms/op | |  Read from cache |  1430 operations |  290.81 ms/op | +------------------+------------------+---------------+
Environment:

JuiceFS version: 1.0.4+2023-04-06.f1c475d
Cloud provider or hardware configuration running JuiceFS: AWS EC2 Instance m5a.xlarge
OS (e.g cat /etc/os-release):

NAME="Ubuntu"
VERSION="16.04.3 LTS (Xenial Xerus)"
ID=ubuntu
ID_LIKE=debian
PRETTY_NAME="Ubuntu 16.04.3 LTS"
VERSION_ID="16.04"
HOME_URL="http://www.ubuntu.com/"
SUPPORT_URL="http://help.ubuntu.com/"
BUG_REPORT_URL="http://bugs.launchpad.net/ubuntu/"
VERSION_CODENAME=xenial
UBUNTU_CODENAME=xenial

Kernel: Linux ip-10-20-19-33 4.4.0-131-generic #157-Ubuntu SMP Thu Jul 12 15:51:36 UTC 2018 x86_64 x86_64 x86_64 GNU/Linux
Object storage: memoryDB Redis 6.2 db.t4g.medium same subnet of EC2 instance
Metadata engine info: Redis 6.2

Basically JuiceFS (and most of distributed systems) is not suitable for workloads like git as it has too many "small" operations. However, if you'd like to take a try, there are two suggestions that may make it smoother:

For git clone, clone the repo locally first and then mv it into a JuiceFS mount point
For daily git use, aggressively cache the metadata by increasing attr-cache, entry-cache and dir-entry-cache in mount command (note this may lead to inconsistent view on multiple hosts, see JuiceFS metadata cache)

Thanks for the answer. For the use I need, I can't clone and then move. And the options did not help. It's weird that after the clone everything works really fast. I currently use s3backer and it appears to perform the same as a local disk both in the clone and in the rest of the use. What makes juicefs slow in clone?

Did you try --writeback --upload-delay=30 if this helps?

git issue many operations on metadata, which require fast meta engine. The MemoryDB is VERY slow to write, because it persistents log to a distributed storage for every transaction.
So, it's expected to be VERY slow for git when MemoryDB is used as meta engine.


Broken when used form an addon

⠋ Buildingapp.import is not a function
TypeError: app.import is not a function
    at Class.module.exports.included (/Users/tchak/git/github/ember-cli/ember-ajax/node_modules/ember-network/index.js:17:15)
    at /Users/tchak/git/github/ember-cli/ember-ajax/node_modules/ember-cli/lib/models/addon.js:244:32

Steps to reproduce:

create a new addon
ember install ember-network
ember test
:bomb:

PR that fixes this forthcoming.

From the referenced PRs, I believe this to be fixed. Please let us know if otherwise!


Mobile: too many windows to close

First of all: changes for mobile are all in all ok. But...if you change the map under "Mapping" and with the new layout on mobile you have to close two windows if you change the loaded map. It's annoying enough to always click ok when you change a map. Now that you have to close two windows makes that double. It would be better if there were no window to close.

Ok, I think use map, start task, load task could be done without confirmation over modal. But for rename, delete or copy things I will stay with confirmation over modal window.


Eager glob expansion

Currently the code calls expand aka globby each time unwrap is called. It seems like it would be beneficial to eagerly evaluate in source, so it is done once for each source, rather than multiple times per wrap/target.

Yeah definitely, been thinking about this too.
Would need a getter method that asks for a saved "stash", before unwrapping.
Probably live at this._.files

I don't think you need a getter method. I think all you need is this:
Fly.prototype.source = function (globs) {
	globs = arrify(globs);
        var flattenedGlobs = flatten(globs);

	assign(this, {
		_: {
			filters: [],
			globs: flattenedGlobs,
                        files: Promise.all(flattenedGlobs.map(expand))
		})
		}
	})

	this._.cat = undefined
	_('source %o', this._.globs)
	return this
}

Yeah, true -- by the time any filter/plugin is called, source() has already been run... early morning haze 😴

Closed by #189


[FLINK-29025][docs] add docs for Hive dialect

What is the purpose of the change
Add docs for Hive dialect
Brief change log
The docs for Hive dialect contains:

Overview (how to use Hive dialect)
Queries (The queries using Hive syntax )
DDL and other statement

Also, it remove the old page for Hive dialect, and copy most of contents to overview page for Hive dialect.
Verifying this change
build the doc locally
Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): (yes / no)
The public API, i.e., is any changed class annotated with @Public(Evolving): (yes / no)
The serializers: (yes / no / don't know)
The runtime per-record code paths (performance sensitive): (yes / no / don't know)
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Kubernetes/Yarn, ZooKeeper: (yes / no / don't know)
The S3 file system connector: (yes / no / don't know)

Documentation

Does this pull request introduce a new feature? (yes / no)
If yes, how is the feature documented? (not applicable / docs / JavaDocs / not documented)

@flinkbot run azure

LGTM. I added 4 commits to adjust the page order anduse dash-case ("hive-compatibility") instead of camelCase ("hiveComptibility") in URL and remove "alias" front matter.


Modify AMI to use a Debian Bullseye base AMI

🗣 Description
This pull request modifies the packer configuration to start from a Debian Bullseye base AMI.
💭 Motivation and context
Debian Bullseye has been out for a few months now, so it makes sense to move away from the previous release.
🧪 Testing
All pre-commit hooks pass.  I also built a staging AMI and deployed it to our staging COOL environment and verified that it functions as expected.
✅ Pre-approval checklist

[x] This PR has an informative and human-readable title.
[x] Changes are limited to a single goal - eschew scope creep!
[x] All relevant type-of-change labels have been added.
[x] I have read the CONTRIBUTING document.
[x] These code changes follow cisagov code standards.
[ ] All relevant repo and/or project documentation has been updated to reflect the changes in this PR.
[ ] All new and existing tests pass.
[ ] Create and test a staging AMI with these changes.

✅ Pre-merge checklist

[ ] Finalize version.

✅ Post-merge checklist

[ ] Create a production AMI with these changes.

Closed in favor of #61.  We can't currently upgrade to Debian Bullseye since we lack Bullseye packages for the CDM tools.


Fix unrelated release notes being added to 0.15 entry (Cherry-pick of #1486)

This is a better fix for https://github.com/Qiskit/documentation/issues/978 than https://github.com/Qiskit/qiskit-ibm-runtime/pull/1479 was.
The issue with #1479 is release notes for 0.16+ were being put into the 0.15 entry.
This new approach also allows us to move the hacky code we needed in qiskit/documentation. This new file was tested with our API generation script and does the right thing.

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

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

Details

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
5764

Relevant Lines:
7005

💛  - Coveralls


No need to link pdf related libraries in component build.

depends on atom/libchromiumcontent#115

:+1:


Edge definition not being held

ArangoDB: v3.5.1
ArangoJS v6.10.0
Apologies if this issue is due to a misunderstanding of mine, but is there a way to constraint which vertex collections can be used for _in and _out in a certain edge definition?  After creating a graph with a specific Edge Definition, I can still create through ArangoJS edges that are not compliant with the Edge Definition.
db.useDatabase('arangoDB2');

e1Collection = db.collection('E1_CRM_Entity');
e1Collection.create().then(
  () => console.log('Collection created'),
  err => console.error('Failed to create collection:', err.response.body.errorMessage)
);

e55Collection = db.collection('E55_Type');
e55Collection.create().then(
  () => console.log('Collection created'),
  err => console.error('Failed to create collection:', err.response.body.errorMessage)
);

e2Collection = db.collection('E2');
e2Collection.create().then(
  () => console.log('Collection created'),
  err => console.error('Failed to create collection:', err.response.body.errorMessage)
);

const p2Collection = db.edgeCollection('P2_has_type');

p2Collection.create({
	waitForSync: true
}).then(
  () => console.log('Edge Collection created'),
  err => console.error('Failed to create edge collection:', err.response.body.errorMessage)
);

const graph = db.graph("graph");

graph.create({
	edgeDefinitions: [{
		collection: 'P2_has_type',
		from: ['E1_CRM_Entity'],
		to: ['E55_Type']
	}]
}).then(
  () => console.log('Graph created'),
  err => console.error('Failed to create Graph:', err.response.body.errorMessage)
);

graph.addVertexCollection('E2');

const col1 = graph.vertexCollection("E1_CRM_Entity");
const col2 = graph.vertexCollection("E55_Type");
const col3 = graph.vertexCollection("E2");
const edge1 = graph.edgeCollection("P2_has_type");

const crmEntity = await col1.save({ name: "E1" });

const crmType = await col2.save({ name: "E55" });

const twoTy = await col3.save({ name : "E2"})

const edge = edge1.save(
	{name: "P2"},
	crmEntity.vertex._id,
	crmType.vertex._id
).then(
  () => console.log('Edge P2 document created'),
  err => console.error('Failed to create Edge P2 from ' + crmEntity._id + ' '+ crmType._id +  ':', err.response.body.errorMessage)
);

const edge2 = edge1.save(
  {name: "P2"},
  crmEntity.vertex._id,
  twoTy.vertex._id
).then(
  () => console.log('Edge P2 document created'),
  err => console.error('Failed to create Edge P2 from ' + crmEntity._id + ' '+ crmType._id +  ':', err.response.body.errorMessage)
);

This still creates two edges in the graph (one of which the destination vertex is from a collection not defined in the edge definition), despite the Edge definition stating that from must be E1_CRM_Entity and to must be E55_Type. Is there a way to force the edge creation to follow the edge definition like the creation of edges in the web framework?

Hey @nunofreitas96, sorry about only seeing this issue just now.
This sounds more like an issue with ArangoDB rather than the driver: https://github.com/arangodb/arangodb/issues
For more direct support also see the ArangoDB Community Slack: https://slack.arangodb.com
Thanks and apologies for the delayed response.


feat: OpenId4VC support

Codecov Report

Merging #1604 (bee0438) into main (a0458fe) will decrease coverage by 1.04%.
The diff coverage is 43.39%.

@@            Coverage Diff             @@
##             main    #1604      +/-   ##
==========================================
- Coverage   85.76%   84.73%   -1.04%
==========================================
  Files         951      962      +11
  Lines       22854    23331     +477
  Branches     4017     4178     +161
==========================================
+ Hits        19601    19769     +168
- Misses       3069     3348     +279
- Partials      184      214      +30

Files
Coverage Δ

packages/core/src/index.ts
100.00% <100.00%> (ø)

packages/core/src/modules/vc/index.ts
100.00% <100.00%> (ø)

...ackages/openid4vc-client/src/OpenId4VcClientApi.ts
80.00% <100.00%> (-20.00%)
:arrow_down:

...ages/openid4vc-client/src/OpenId4VcClientModule.ts
100.00% <100.00%> (ø)

...nid4vc-client/src/OpenId4VcClientServiceOptions.ts
100.00% <100.00%> (ø)

packages/openid4vc-client/src/index.ts
100.00% <100.00%> (ø)

...ckages/openid4vc-client/src/presentations/index.ts
100.00% <100.00%> (ø)

...nid4vc-client/src/presentations/selection/index.ts
100.00% <100.00%> (ø)

...s/openid4vc-client/src/utils/claimFormatMapping.ts
100.00% <100.00%> (ø)

packages/openid4vc-client/src/utils/index.ts
100.00% <100.00%> (ø)

... and 9 more


Support for input from sources other than files

It would be great if this project supported input from sources other than files. For example, we use the JDBC and JDBC static plugins to load data from relational databases into Elasticsearch. We want to write tests that check all aspects of our pipeline config, including:

Did the configured SQL query/queries select the expected data from the raw data set?
Did it connect to the right database with the right credentials and connection options?
Did it filter and transform the raw data correctly before loading it into Elasticsearch?

That sounds awesome. This can probably be handled with stdin. We already pass the input logs via stdin here https://github.com/agolo/logstash-test-runner/blob/master/test.sh#L73
PRs are welcome to allow configurable input streams


Fix metric fetches when parsing from FB

Coverage remained the same at 100.0% when pulling 837db449ba8ddc0d3a8445091fb9ea3486a0f523 on fix-metrics into 66a0263e14b473ba3acc5f40e57b5a34523043ee on master.

Coverage remained the same at 100.0% when pulling ec8552d695822609eae74bf1120f15b67aeb264d on fix-metrics into 66a0263e14b473ba3acc5f40e57b5a34523043ee on master.

Closed by #67


Publishing Docker image to Docker Hub

New Feature
It would be desirable to publish a Docker image to Docker Hub during the release process. It would allow users to use the image directly in a Kubernetes or container service.

@martinm82 @decyjphr any update on this? really interested so we don't need to checkout and build the image ourselves.

@quike There is a process where the latest release is published to Docker. See the latest docker images...

I think it is time to change this to GH Container registry. I'll look at doing that.
Bit for your reference, the workflows that publish to Docker are:
https://github.com/github/safe-settings/blob/main-enterprise/.github/workflows/create-pre-release.yml
and
https://github.com/github/safe-settings/blob/main-enterprise/.github/workflows/create-release.yml


cleanup(build): Use a single cmake module for driver_config.h

Instead of manually generating driver_config.h when needed (approximately, since the libscap engines do not generate it, even though they rely on it), encapsulate all the logic in a single cmake module.
Note: this removes the driver_config directory from LIBSCAP_INCLUDE_DIRS. If you actually need driver_config.h, add this to your CMakeLists.txt:
include(driver_config)
include_directories(${DRIVER_CONFIG_OUTPUT_DIR})

What type of PR is this?
/kind cleanup
Any specific area of the project related to this PR?
/area build
/area driver-kmod
/area driver-bpf
/area driver-modern-bpf
What this PR does / why we need it:
driver_config.h is generated in three different places, in different ways, into two target locations (including the source directory). Clean this up and use just one well-defined place we can refer to later.
Also, (as it turned out to be a prerequisite for the fix), the BPF build now no longer touches the source directory and does not depend on any files outside the (generated) source directory.
Does this PR introduce a user-facing change?:
(I'm not sure about user-facing, but it is a change for libscap consumers, even though driver_config.h is generally not needed)
action required: driver_config.h is no longer available in ${LIBSCAP_INCLUDE_DIRS}

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: gnosek
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [gnosek]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

NOTE: since driverkit jobs are not required checks, we can go on with this one.

/milestone 0.12.0

@gnosek can you fix other tests ?

/milestone 0.13.0

Moving to
/milestone 0.14.0
since I believe it's to late to make this work by the end of this month.

Also checking in on this PR, do we have concrete plans?

Hi @gnosek just kindly checking in on the status of this PR. When do you all think we should work on it? Thank you!

Issues go stale after 90d of inactivity.
Mark the issue as fresh with /remove-lifecycle stale.
Stale issues rot after an additional 30d of inactivity and eventually close.
If this issue is safe to close now please do so with /close.
Provide feedback via https://github.com/falcosecurity/community.
/lifecycle stale

@gnosek any update on this?
/remove-lifecycle stale

Issues go stale after 90d of inactivity.
Mark the issue as fresh with /remove-lifecycle stale.
Stale issues rot after an additional 30d of inactivity and eventually close.
If this issue is safe to close now please do so with /close.
Provide feedback via https://github.com/falcosecurity/community.
/lifecycle stale

@gnosek any update on this?
do we still want to merge this PR? :thinking:

As you might imagine, this has fallen behind other things in my stack of yaks to shave. Would be nice to have it but I guess I'll redo it from scratch some day.

Stale issues rot after 30d of inactivity.
Mark the issue as fresh with /remove-lifecycle rotten.
Rotten issues close after an additional 30d of inactivity.
If this issue is safe to close now please do so with /close.
Provide feedback via https://github.com/falcosecurity/community.
/lifecycle rotten

Issues go stale after 90d of inactivity.
Mark the issue as fresh with /remove-lifecycle stale.
Stale issues rot after an additional 30d of inactivity and eventually close.
If this issue is safe to close now please do so with /close.
Provide feedback via https://github.com/falcosecurity/community.
/lifecycle stale


Bound queue for batch reporter

Make the backing queue bounded by default, and allow configuring what to
do when it runs full (either drop or block).
Revives #40

/cc @alexbiehl

Cool, thanks! Btw. we are using this library in Production quite successfully -- it might make sense to remove the Experimental label in the description of the repository.

Now that I merged this, I'm wondering how smart the choice of defaults was. It is certainly a better default to drop data instead of affecting the instrumented application, but since this doesn't preserve the previous semantics I think I would need to release as 0.3?


Improve MPC modeling accuracy

Checklist

[X] I've read the contribution guidelines.
[X] I've searched other issues and no duplicate issues were found.
[X] I've agreed with the maintainers that I can plan this task.

Description
The current mpc model uses the "error kinematics model in Frenet coordinate". This model obtains the reference path information from only curvature and indirectly regenerates the reference path information by integrating curvature in the MPC process. However, curvature calculation could have non-small errors, and the reference path information given by integration could also have errors.
In this video, the green path is the trajectory given by the planning module, and the white path is the trajectory re-generated in the MPC model from curvature. There is a large difference when driving in a curve. (branch for visualization)
https://user-images.githubusercontent.com/21360593/236382083-79c2a5ce-58d0-4958-8d9e-5499385f420d.mp4
Purpose
To improve the MPC tracking accuracy
Possible approaches
improve model
Definition of done
mpc performance is measured and improved

In the video above, the white line is generated by the linearized model, while in the video below, the nonlinear path model is calculated.

orange-line: nonlinear path model generated from curvature
white-line: predicted ego-path of the MPC

The accuracy is much better in the nonlinear model.
https://user-images.githubusercontent.com/21360593/236626809-1f7cff17-f66b-46a0-9f73-c452045d05ec.mp4

Silly question: why does MPC needs to regenerate the reference path?

Hello,I have a question why there some difference when calculate discrete dynamic model .


Pattern Matching is broken for Array

Describe the bug
Pattern matching doesn't work for Success([1]) in Success([a]), hoverer it works for Success[1] in Success[a]
To Reproduce
[1] pry(main)> Success([1]) in Success([a])
NoMatchingPatternError: Success([1])
from (pry):41:in `<main>'

# The working examples:
[2] pry(main)> Success[1] in Success[a]
=> nil
[3] pry(main)> Success([1]) in Success[a]
=> nil

Expected behavior
Success([a]) should work exactly the same as Success[a], otherwise examples should be updated.
My environment

Affects my production application: No, since there is a workaround (eg Success[1])
Ruby version: ruby 2.7.2p137
OS: MacOS Big Sur

Could you point which example should be updated?

Here

Thank you! I'll look into that but I think the current behavior is intentional. I know the API is confusing in this part but there were reasons why it happened. Constructing values with Success.[]/Failure.[] came earlier than PM, then everything messed up. I'll try to improve things in 2.0 since it has to be a breaking change.

@flash-gordon Hey, is it possible to update the documentation in the mean time. Had me totally stumped today. :-)


Update Functions.lua

performance fixes, deprecation upgrades, a lot of other stuff

dis better be pushed 💢 💨

@Sceleratis This would be nice to have pushed right about now, since it does help with the client's performance quite a bit.


clean up documentation slightly

Fixes minor grammar/punctuation discrepancies
Adds a basic README.md that points to the Github Pages site
minor restructuring/re-wording.

Feel free to propose any changes if you feel like anything needs to be edited.

Also, I'm not quite sure that replacing the music with your own is a great idea, you could instead provide a guide on how to make your own .cue and .wav files into a 'disc' for Vib-Ribbon instead. (I can draft a PR if needed.)

Also, I'm not quite sure that replacing the music with your own is a great idea, you could instead provide a guide on how to make your own .cue and .wav files into a 'disc' for Vib-Ribbon instead. (I can draft a PR if needed.)

It does sound weird to replace tracks 2-7! But it is actually needed to play the hard-coded obstacle courses inside SCRIPT/SYSTEM3.FSL. You can still draft one if you want. This does give me the idea of turning it into when the replace game tracks and when to insert a disc the old fashioned way.


[WIP] Closes #44: Abstract spongy castle

Implements walleth/kethereum#44
This is a wip PR to provide update on the progress
Current status

Created a crypto_api module that defines interfaces for all underlying crypto functions
Created a crypto_api_spongycastle module that uses SpongyCastle to implement the interfaces defined in the crypto_api module
Create a crypto_api_wrapper module that provides an implementation for each abstraction defined in the crypto_api. This allows us to switch over gradually and at some point the wrapper could be removed completely.

Next

Add interfaces for javax.crypto.* classes to crypto_api
Use spongy castle to implement javax.crypto.* interfaces
(If wanted) Create crypto_api_provider to use javax.crypto providers to implement some of the interfaces

I used objects since most of the implementations don't need a state and therefore I avoid that they need to be recreated each time you access them.
Any specific reason why you prefer classes (I am just curious 😄 )

:+1: thanks - great work!
I prefer classes to objects as I hate global state - and objects tend to introduce global state


Add option to hide heading checkbox in data table

A boolean option showHeadingCheckBox has been added to the data table configuration. This allows the user to choose whether to display the heading checkbox or not. In cases where the checkbox is not needed, a sized box takes its place instead.

Could you please added a test for this feature?

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 95.86%. Comparing base (2e204cf) to head (b081743).

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #270      +/-   ##
==========================================
- Coverage   95.86%   95.86%   -0.01%
==========================================
  Files           3        3
  Lines        1138     1137       -1
==========================================
- Hits         1091     1090       -1
  Misses         47       47

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

Could you please added a test for this feature?

Test added in cb100c4ba31cdcb568566b2769ca8e70da8b344f commit

The tricky cases of varying number of fixed cols/rows is not handled properly. Please check the /example forlder, screens/data_table2_fixed_nm.dart, try disabling the checkbox in the header and you'll see that the layout is broken:

The tricky cases of varying number of fixed cols/rows is not handled properly. Please check the /example forlder, screens/data_table2_fixed_nm.dart, try disabling the checkbox in the header and you'll see that the layout is broken:

fixed in 5ff65f5a2b70fbfa4b92dfcdcdce8d55d7d63b44

Looks good. Thanks for the contribution!


Manifest merger failed : uses-sdk:minSdkVersion 16 cannot be smaller than version 17 declared in library [[]:react-native-get-music-files:unspecified]

Got this error after installing library:
:app:processDebugManifest FAILED FAILURE: Build failed with an exception. * What went wrong: Execution failed for task ':app:processDebugManifest'. > Manifest merger failed : uses-sdk:minSdkVersion 16 cannot be smaller than version 17 declared in library [[]:react-native-get-music-files:unspecified] C:\Users\[]\Desktop\[]\[]\[]\[]\MusicPlayer\android\app\build\intermediates\exploded-aar\MusicPlayer\react-native-get-music-files\unspecified\AndroidManifest.xml Suggestion: use tools:overrideLibrary="com.cinder92.musicfiles" to force usage * Try: Run with --stacktrace option to get the stack trace. Run with --info or --debug option to get more log output. BUILD FAILED

How did you resolve this issue?

just increase minSdkVersion to 16

just incease minSdkVrsion to 17


Feature Proposal: up/down votes on WSFS advisory requests

Hi Chris,
The WSFS team wants to have an up/down vote on a new Hugo Category to be
run after the Hugo voting has closed (in July).
Basic requirements are:

voting limited to WSFS members with vote status in Glasgow Reg
vote is a straight yes/no to a simple question

Nicholas can provide move information on the text etc.
I suggested that it would make sense to add this to NomNom since it
already does the member checking and a simple HTML form with data
collection should be possible (and it is related to Hugos). What do you
think and would you have time to implement something?
cheers
Henry @balen

My reply:
My first reaction was "nope" but ... it probably isn't a huge lift to put in there. I'm cautious about scope creep in NomNom -- it really is intended to be quite minimal -- but it's plausible that this could be done in a relatively tidy way.
I have a concern about the future of this feature; it's possible that NomNom becomes the system of record for recording votes for WSFS activity in the future. If these advisory votes play out successfully, or WSFS leans into a remote voting constitution, then it's not a huge stretch to think of a future where votes need to happen closer to realtime, which is going to take the application in directions that I am not up for, technically, and would require someone with experience with realtime web applications to develop and integrate. I realize I'm looking around a corner or two here, so ... maybe it's not a concern yet.
I would want fully fleshed out requirements; what needs to be displayed in a vote, what is the user interaction flow for them, etc, what are the admin requirements for the votes. I have some loose ideas but I'd hesitate to implement them without a bit more detail. If the amount of work is reasonable, I'll take it on (or accept PRs to add it, possibly).

So...
Part of the reason for this exercise is precisely to establish what a shift to more votes like this would mean in terms of resources and logistics. I'm certainly not excluding the possibility that we come out at the end with a recommendation to never do this again. But we have to try it first.
If we ever did realtime voting on matters before the Business Meeting as it happened, that would I think need to be bundled into the online meeting platform, live at the convention, rather than NomNom. So I don't think we need to plan for that option.
Indeed it could lead to a future situation where we have a lot of up-down votes on WSFS matters happening simultaneously, though current proposals are that this would take place over a more relaxed rather than a more intense timeframe, ie over the months rather than days before the convention.
Q1: What needs to be displayed in a vote: text explaining the vote, buttons to vote for or against, all on the same page. (I think not including an abstain option, if people can't be bothered to vote one way or the other then we don't count them at all).
What is the user interaction flow for them: Log in, read the arguments, vote.
What are the admin requirements for the votes: Check that each vote is cast by someone with voting rights, and that nobody has voted twice; be able to enter paper ballots if there are any; count the results. I'm not going to die on a hill on paper ballots but it would be useful to have.
Does that make sense?
N

Actually on reflection (and thanks to Henry for helping to clarify this), all we're doing is a vote for WSFS members which is like a Hugo vote but with only two options instead of 140. Does that sound manageable?

A few clarifications:

How many yes/no questions?
Do questions need to have a "voting closes" time?
Do all members get to vote, only Hugo voters? Nominators? Another set? Can this be different per-question?
Can that status change over time?
What kind of reporting is needed?
Can votes be invalidated?
Who should be allowed to see the results?

Thanks, Chris.

How many yes/no questions?
For the moment, just one. Based on your feedback, and many other factors,
there could be more in future years.

Do questions need to have a "voting closes" time?
We need to terminate the voting process the week before the convention. I'm
thinking of having the voting period from 26 July to 4 or 5 August.

Do all members get to vote, only Hugo voters? Nominators? Another set?
Can this be different per-question?
It should be only Glasgow members with WSFS memberships, ie Hugo voters -
but will also include any who buy WSFS memberships after the Hugos deadline
has passed. And it should be the same for every question, if we ever have
more than one question.

Can that status change over time?
What status? I think you mean, will the voting set be different at the end
than the start - and the answer yes, anyone who buys a WSFS membership in
the interim gets to vote.

What kind of reporting is needed?
For external purposes, just the numbers voting each way. For internal
purposes obviously we'd need the usual precautions against fraud, ie seeing
name, IP address, timing of voting and how the membership was paid for.

Can votes be invalidated?
Yes. Incidentally, I hope that we also have the ability to disqualify Hugo
votes in NomNom!

Who should be allowed to see the results?
WSFS DH/DDH and a small number of designated staff members (probably just
one).

N
On Thu, Jun 13, 2024 at 4:20 PM Chris Rose @.***> wrote:

A few clarifications:

How many yes/no questions?
Do questions need to have a "voting closes" time?
Do all members get to vote, only Hugo voters? Nominators? Another
set? Can this be different per-question?
Can that status change over time?
What kind of reporting is needed?
Can votes be invalidated?
Who should be allowed to see the results?

—
Reply to this email directly, view it on GitHub
https://github.com/WorldconVotingSystems/nomnom/issues/150#issuecomment-2165816495,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BJDIQH6KXBS44ZUAJVTYJ23ZHGTCHAVCNFSM6AAAAABJCSPGLWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCNRVHAYTMNBZGU
.
You are receiving this because you commented.Message ID:
@.***>

--
Nicholas Whyte (he/him/his)
+32 489 805 514
Twitter: @nwbrux
Blog: http://fromtheheartofeurope.eu

Thanks, good questions.

I would not want the final vote to be visible to anyone except
administrators.

OK, we can always check that independently but it may be something
to consider for future iterations.

I don’t see why not, if they are eligible voters and they do it before
the deadline, unless you tell me it is technically impossible.

N
Nicholas Whyte (he/him/his)
+32 489 805 514
Twitter: @nwbrux
Blog: http://fromtheheartofeurope.eu
On Thu, Jun 13, 2024 at 19:17 Chris Rose @.***> wrote:

I need a couple more clarifications.

Should the final vote be visible to voters during, after voting,
after the vote closes, another time, or never? What about to non members?
nomnom cannot see how members paid; that's not going to change. Hope
that's okay.
Can members change their vote after it is cast?

—
Reply to this email directly, view it on GitHub
https://github.com/WorldconVotingSystems/nomnom/issues/150#issuecomment-2166371126,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BJDIQH7IMIR6ZN6ME5Q7QC3ZHHH2TAVCNFSM6AAAAABJCSPGLWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCNRWGM3TCMJSGY
.
You are receiving this because you commented.Message ID:
@.***>

For (2) NomNom will not ever have access to membership payment information. That is by design and will not change; integrating those pieces of data is possible from the registration side where there is a material need for that information, but retaining that from the NomNom side is a nonstarter.


refactor(streaming): use table catalog for materialize executor

I hereby agree to the terms of the Singularity Data, Inc. Contributor License Agreement.
What's changed and what's your intention?
As title, part of https://github.com/singularity-data/risingwave/issues/3657.
The next PR will try to construct state table in materialize executor by from_table_catalog.
Checklist

[x] I have written necessary rustdoc comments
[x] I have added necessary unit tests and integration tests
[x] All checks passed in ./risedev check (or alias, ./risedev c)

Documentation
If your pull request contains user-facing changes, please specify the types of the changes, and create a release note. Otherwise, please feel free to remove this section.
Types of user-facing changes
Please keep the types that apply to your changes, and remove those that do not apply.

Installation and deployment
Connector (sources & sinks)
SQL commands, functions, and operators
RisingWave cluster configuration changes
Other (please specify in the release note below)

Release note
Please create a release note for your changes. In the release note, focus on the impact on users, and mention the environment or conditions where the impact may occur.
Refer to a related PR or issue link (optional)

Oh, I realize it's your next PR. so LGTM

Oh, I realize it's your next PR. so LGTM

Yes, constructing state table in materialize executor by from_table_catalog need to take arrange_columns into consideration, maybe it won't be easy, I will refactor this later.


Form::VisibleFields() and FieldList::VisibleFields()

I added these when working on a form when Form.ss is too heavyweight.  It let me structure a form like this:
	<% with Form %>
	<form $FormAttributes>
		<% loop VisibleFields %>$Field<% end_loop %>
		$HiddenFields
		$Actions
	</form>
	<% end_with %>

OK, @stojg I've added some publicity.

all good, merging.


tests: kernel: timer_api: fix formatting specifiers in diagnostic

Several of the values passed to the conversion failure diagnostic are
unsigned and/or 32-bit values, while all format specifiers are for
signed 64-bit integers.  Make the specifiers consistent with the
argument.
This eliminates bogus diagnostic output like:
    Assertion failed at WEST_TOPDIR/zephyr/tests/kernel/timer/timer_api/src/timer_convert.c:157: test_conversion: (diff <= maxdiff && diff >= mindiff && 0 i
s false)
Convert 0 from ERRhz to 0hz (= 0) failed. diff 0 should be in [999:32768]

as demonstrated in https://github.com/zephyrproject-rtos/zephyr/issues/25750#issuecomment-635868467

@andyross take a look please?


Filtering articles whose publishedOn is after today

There is a way to put a page like a table of contents at the top of the article list by setting "pubDate" to a future date. For example, this feed has a "pubDate" of Fri, 31 Dec 9999 23:59:00 +0900.
The current v1 code accepts all such articles, but this PR have changed it to reject them.
If this is not in line with your project policy, please feel free to close it.

This issue is fixed and released in v1.12.1 Thank you again for contributing.


docs on show constraints statement

This PR adds docs on the SHOW CONSTRAINT statement. Fixes #521.
HTML version here: http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/ac93df39a27eed858da4f66cb1379724cd98ff2d/show-constraints.html

This change is

@dt, while documenting this, I found that NOT NULL and DEFAULT constraints are missing from the response. See https://github.com/cockroachdb/cockroach/pull/7584.

It should include any named NOT NULL/DEFAULT constraints. I started it since I wanted a way to list the named constraints (those that I could refer to in an ALTER or DROP, etc). I'm starting to think we should just generate names for every constrain all the time so that all constraints have names, but until then, I think that's what you're seeing.

Review status: 0 of 6 files reviewed at latest revision, 1 unresolved discussion.

show-constraints.md, line 31 [r1] (raw file):

## Response

The following fields are returned for each constraint.

let's throw a little banner in here saying that this is under development and the exact output will continue change? I don't want anyone relying on anything in here having a particular value or form just yet.

Comments from Reviewable

 mod putting a warning about the specific output of SHOW CONSTRAINTS not being something to rely on.

Review status: 0 of 6 files reviewed at latest revision, 1 unresolved discussion.

Comments from Reviewable

Thanks, @dt. I updated the intro to clarify that the statement returns all named constraints and any unnamed check constraints. Let me know if something's still missing.

Review status: 0 of 6 files reviewed at latest revision, 1 unresolved discussion.

show-constraints.md, line 31 [r1] (raw file):
Previously, dt (David Taylor) wrote…
> let's throw a little banner in here saying that this is under development and the exact output will continue change? I don't want anyone relying on anything in here having a particular value or form just yet.

Done.

Comments from Reviewable


readinto function hangs while in progress

Package Name: azure-storage-blob
Package Version: 12.12.0
Operating System: Amazon AL2
Python Version: 3.7

Describe the bug
when calling the below piece of code - the download hangs indefinitely while in progress
    with open(local_file, "wb") as f:         download_stream = blob_client.download_blob(         )         download_stream.readinto(f)
To Reproduce
Steps to reproduce the behavior:

Run the above piece of code on a file > 1gb

Expected behavior
The download should complete and the data should be stored in the file
Additional context
output below, hangs just after the last statement for 12+ hours

23:20:26.705 __main__.log_progress INFO<PHONE_NUMBER> bytes have been completed out of<PHONE_NUMBER>
23:20:27.13 __main__.log_progress INFO<PHONE_NUMBER> bytes have been completed out of<PHONE_NUMBER>
23:20:27.78 __main__.log_progress INFO<PHONE_NUMBER> bytes have been completed out of<PHONE_NUMBER>
23:20:27.135 __main__.log_progress INFO<PHONE_NUMBER> bytes have been completed out of<PHONE_NUMBER>
23:20:27.188 __main__.log_progress INFO<PHONE_NUMBER> bytes have been completed out of<PHONE_NUMBER>
23:20:27.273 __main__.log_progress INFO<PHONE_NUMBER> bytes have been completed out of<PHONE_NUMBER>
23:20:27.328 __main__.log_progress INFO<PHONE_NUMBER> bytes have been completed out of<PHONE_NUMBER>
23:20:27.345 __main__.log_progress INFO<PHONE_NUMBER> bytes have been completed out of<PHONE_NUMBER>
23:23:39.847 __main__.log_progress INFO<PHONE_NUMBER> bytes have been completed out of<PHONE_NUMBER>

Label prediction was below confidence level 0.6 for Model:ServiceLabels: 'Azure.Identity:0.39250913,Storage:0.2970738,Service Bus:0.07714585'

@affanv14 Thanks for your feedback, we'll investigate asap.

Hi @affanv14 Mohammed, thanks for the report. Can you share a bit about how you are tracking the progress of the download? Also are you setting the max_concurrency keyword in your call to dnowload_blob? There is a known issue with the previously recommended way to track progress (if you are using the SDK to do so) when performing a concurrent download where the progress would never reach 100%.

Hi @jalauzon-msft we originally had it running without any keywords and it was hanging. We tried a bunch of things including max_concurrency. the last thing we tried was
with open(local_file, "wb") as f:
        download_stream = blob_client.download_blob(
            progress_hook=log_progress,
            timeout=MAX_DOWNLOAD_WAIT_TIME,
            max_concurrency=NUM_DOWNLOAD_THREADS,
            read_timeout=MAX_DOWNLOAD_WAIT_TIME,
        )
        LOG.info("Download stream received from blob client")
        download_stream.readinto(f)

Hi again @affanv14 Mohammed, thanks for sharing the sample. Can you also share a sample of how you are tracking the progress to show the output in your original post? This progress tracking output is not something I recognize so it must be custom on your end.
I'm trying to determine if the download itself is hanging or possibly the progress reporting is just never reporting that it's done. Thanks!

def log_progress(completed: int, total: int) -> None:
    LOG.info("{} bytes have been completed out of {}".format(completed, total))

here is what we used to log progress

There are statements after this(including a log statement right after) which is not reached. I am certain that it hangs on this. Furthermore, we previously had no params passed in the download_blob call - it was still having the issue

Hi @affanv14 Mohammed, sorry for the long delay and thanks for confirming that readinto is indeed hanging.
The time the client will wait if there is no response from the server while reading data is configured via the read_timeout keyword and our default value is 80000 seconds (about 22 hours) (which is probably too high, and we are thinking of updating). I see you tried setting read_timeout for yourself to MAX_DOWNLOAD_WAIT_TIME? What is the value you used here? Did you wait for this length of time to see if some error eventually came back?
I think a good next step would be to enable debug logging so we can see the requests/responses happening and see if it is indeed hanging while waiting for a response of if something else is happening. Can you try enabling debug logging? Some documentation on how to do that is here but here is a code snippet from that documentation of all you should need (there are other ways to configure in article if needed):
import sys
import logging
from azure.storage.blob import BlobClient

# Set the logging level for the azure.storage.blob library
logger = logging.getLogger('azure.storage.blob')
logger.setLevel(logging.DEBUG)

# Direct logging output to stdout. Without adding a handler,
# no logging output is visible.
handler = logging.StreamHandler(stream=sys.stdout)
logger.addHandler(handler)

blob_client = BlobClient(..., logging_enable=True)

Thanks for your help and patience.


chore: use the full blown Renovate image

Description
Instead of using the slim image, this PR activates the full blown image with all managers, .... to have them available if needed.
Verification
Check Renovate job in this feature branch to use the full image.
Checklist

[x] My code follows the style guidelines of the project
[x] I have performed a self-review of my own code
[ ] I have made corresponding changes to the documentation

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


Project: Teams tab to convert numerous supported source file types to several target file types (PDF, HTML, JPG)

Project name
tab-sso-graph-file-conversion
Description
This Teams Tab enables a user to upload a supported (depending on the target) file type via drag and drop while the uploaded file will be converted as PDF, HTML, JPG depending on user's selection. In Teams context (as configurable Tab) it uses the current Teams' site and the default drive. In case of a personal tab the SharePoint root site's default drive' It uses the following capabilities (mostly) on behalf of Microsoft Graph:

Use HTML5 drag and drop event handling
Teams SSO and O-B-O flow access token generation in backend
Teams Toolkit for Visual Studio
Writing normal files smaller 4MB
Writing files bigger than 4MB with a resumable upload
Retrieving files with format=pdf/html/jpg conversion which is currently supported only with Graph beta endpoint
Use Fluent UI Web Components with Blazor

Hack-together requirements

Used Microsoft Graph .NET SDK 4.54.0 due to blocking issue with TeamsFx and issue with used beta-endpoint)
Used HttpClient with REST for one endpoint due to v1.0 / beta mix and above issue with used beta-endpoint)

Extend sample

Convert from Teams Tab to stand-alone SPA e.g.
Once above issues solved convert to Microsoft Graph .NET SDK >=5.0.0
Extend to GLB conversion (was simply not familiar with that file type(s) and had no samples to test)

Repo URL
https://github.com/mmsharepoint/tab-sso-graph-file-conversion
Team members
mmsharepoint

Woot! Awesome to see this hack! Thank you @mmsharepoint!


auto complete incorrectly word to uppercases even within quotes

while the uppercase of keywords like where is very neat.  I actually don't want columns within quotes like "column1" to be automatically uppercased to "COLUMN1" because it results in incorrect column name.

DBeaver doesn't convert identifiers case. Most likely your identifier conflicts with reserved keyword (and this explains why it should be quoted).
DBeaver shouldn't convert keyword case if keyword is in quotes. This is a bug, it will be fixed in the next version, thanks for report.
Workaround: disable keywords case conversion (Preferences->General->Editors->SQL Editor->Auto format).


1.9.0

[x] logging with discordpy builtin (logging.INFO level)
[x] emojitts (emoji: Optional[str])
[x] more /playnoise cmds (five nights at freddys scream, thomas the train earrape)
[x] member commands! (randomly move member around voicechannels?)
[x] merge /legal into /info

bugs:

[x] refactor /settings, /serversettings to use bool typehint instead of Literal['on', 'off']
[x] remove legacy logging #4
[x] clarify dmblocker meaning
[x] overflow err for /copypasta
[x] /settings dmblocker none type assignment -> occurs during new user assignment
[x] untrack db
[x] send backups daily to discord webhook - logs, db
[x] update privacy policy to reflect data storage requirements
[x] rename "asyncCmds.py" -> "utilities.py"
[x] refactor $restart (declutter)
[x] changeff properly affects /copypasta
[x] check for range on /textwall
[x] tips have been updated to slash commands, can't believe people didn't notice lol
[x] /meme fails on deepfriedmemes
[x] dmthreaten fails if response takes too long
[x] memorygame fails if some other random dude interacts with it...
[x] autoresponse fails if channel perms are incorrect

replit doesnt support jdk 17

draft patch notes:

emojitts
more /playnoise cmds (five nights at freddys scream, thomas the train earrape)
member commands! (randomly move member around voicechannels?)
/legal has been removed. /info now contains the information previously found in /legal

bugs:

dmblocker meaning is more clear, hopefully.
overflow error for /copypasta has been fixed
/settings dmblocker none type assignment
backups of your user data are now sent to the support server in a private channel
the privacy policy and terms of use has been updated to reflect this
/copypasta censors words correctly if family friendly mode is turned on
/textwall throws an error at you if you try to generate a 0-length text wall
tips have been updated to slash commands, can't believe people didn't notice lol

remaining chores:

[ ] add extra playnoise optioned mentioned in patch notes
[ ] fix qa bugs
[ ] add contributors txt and show in /info
[ ] update /help (emojitts, gtfo from vc, playnoise options, legal rm)
[ ] channeltroll archiving fails if thread DNE
[ ] dmtroll takes too long to create dm (defer + followup)

[ ] update server join message
[ ] on failure dm to owner


Deprecate org.apache.pulsar.io.core.KeyValue

Fixes #15027
Motivation
We have org.apache.pulsar.common.schema.KeyValue and org.apache.pulsar.io.core.KeyValue.
io.core.KeyValue is limited to pulsar-io and can be safely replaced by common.schema.KeyValue or some Pair implementation.
Modifications
Marked the class as deprecated (can't just delete it since it is a Public/Stable).
Replaced all its usages with common.schema.KeyValue (except for the pulsar-io/kafka-connect-adaptor - https://github.com/apache/pulsar/pull/15025 will take care of that)
Verifying this change

[ ] Make sure that the change passes the CI checks.

Does this pull request potentially affect one of the following parts:
If yes was chosen, please highlight the changes

Dependencies (does it add or upgrade a dependency): (yes / no)
The public API: YES, org.apache.pulsar.io.core.KeyValue is deprecated
The schema: (yes / no / don't know)
The default values of configurations: (yes / no)
The wire protocol: (yes / no)
The rest endpoints: (yes / no)
The admin cli options: (yes / no)
Anything that affects deployment: (yes / no / don't know)

Documentation
Check the box below or label this PR directly (if you have committer privilege).
Need to update docs?

[ ] doc-required
(If you need help on updating docs, create a doc issue)

[x] no-need-doc

Javadoc / release notes should be enough

[ ] doc
(If this PR contains doc changes)

Closed as stale, conflict, and no consensus. Please rebase and resubmit the patch if it's still relevant.


PostCSS 8 issue still present

Hey, I tried this starter on a fresh install, and I still get
Error: PostCSS plugin autoprefixer requires PostCSS 8.
Any idea about what can be happening?

Seems to come from the global install.
If install packer locally it works.


OIDC: Support Implicit Flow

In #141, the current recommendation for using code flow and PKCE is described. Unfortunately, not all IdPs support this (apparently ORCID is one such).
So, the next best thing would be to support the implicit flow, where the IdP access token is returned directly to the browser. Here is a working example:

Browser application redirects the user to the IdP with this URL. response_type=token says "give me a token directly"

https://orcid.org/oauth/authorize?
response_type=token
&redirect_uri=<browser url>
&client_id=APP-0ROS9DU5F717F7XN
&scope=openid

On successful login, the IdP redirects the user with the access_token in the url, like:

<browser_url>#
access_token=1cab6215-3eed-45a6-a742-029fc8a94ca7
&token_type=bearer
&expires_in=599
&tokenVersion=1
&persistent=true
&id_token=eyJraWQiO <trimmed>

Now what does the browser client do? This is an ORCID token, not a Tiled token. I think we need to send the id_token to Tiled to validate the id of the user (and use signature validation to assure that it came from ORCID) and return a tiled access_token and refresh_token. But I don't think there is an endpoint for this yet.
Thoughts @danielballan, @kleinhenz?

It's entirely possible that I'm over-engineering this idea.
With implicit flow, the ORCID access_token and id_token are leaked  to the user, and the client secret is never used.
Wtih code flow and no PKCE the code is leaked but it's extremely short lived...and the client secret is still used and hidden from the user (directly asked for by tiled). I think that despite the current recommendation, that code flow without PKCE is arguably better than implicit flow. I am going to close this issue...happy to reopen if needed.


[gRPC] Is it possible to have environment variables in paths?

I need to do something like the following:
proto < $GOPATH/my-service/my-service.proto
enums: String,

###

GRPC http://localhost:9090/myservice.MyService/GetUser
{
  "userId": "123456",
}

How can I achieve this?

Good idea. I had not thought of that at all.

With the new version it is now possible to use variables.
@assetsDir=./

proto < {{assetsDir}}hello.proto
includeDirs: ["../request"]

GRPC grpcb.in:9000/hello.HelloService/sayHello
{
  "greeting": "world"
}

System environment variables can be used with process.env.GOPATH
Thanks for the idea.

Thank you for the super quick answer!
I'm testing your suggestion, but unfortunately it is not working.
It works when I do the following:

Thank you for the super quick answer!
I'm testing your suggestion, but unfortunately it isn't working.
It works when I run the following code:
@assetsDir=/Users/user.name/go

proto < {{assetsDir}}/my-service/my-service.proto

###

GRPC http://localhost:9090/myservice.MyService/GetUser
{
  "userId": "123456",
}

But it does NOT work when I run this:
@assetsDir=process.env.GOPATH

proto < {{assetsDir}}/my-service/my-service.proto

###

GRPC http://localhost:9090/myservice.MyService/GetUser
{
  "userId": "123456",
}

I also tried with a pre-script running this:
{{
  //pre request script
  console.log(`my GOPATH: ${process.env.GOPATH}`)
  exports.assetsDir = process.env.GOPATH
}}

proto < {{assetsDir}}/my-service/my-service.proto

###

GRPC http://localhost:9090/myservice.MyService/GetUser
{
  "userId": "123456",
}

And it still doesn't work. The log is printing INFO: my GOPATH: undefined.
Is my code wrong?

All expression between {{...}} are executed as javascript. Because of this following is invalid (process.env.GOPATH is interpreted as string)
// invalid
@assetsDir=process.env.GOPATH
proto < {{assetsDir}}/my-service/my-service.proto
// valid
proto < {{process.env.GOPATH}}/my-service/my-service.proto

Your next example would also be valid, but apparently the environment variable GOPATH is not defined? I have tested it and should work

It is working with the following piece of code, but only if I run httpyac from the shell:
proto < {{process.env.GOPATH}}/my-service/my-service.proto

###

GRPC localhost:9090/myservice.MyService/GetUser
{
  "userId": "123456",
}

When I run it within VSCode, I still get undefined. Please note that I'm on macOS and I'm using the default zsh shell.
If I run echo $GOPATH in my zsh, I correctly get /Users/user.name/go.
I also defined the GOPATH env variable for bash so that it also works when I run httpyac from bash.
On the other hand, it never works when I use process.env.$GOPATH.
Is something happening with the VSCode extension?

Also with $GOPATH it works correctly under Windows. I will test it tomorrow with Ubuntu. Can you try the following commands to see if they give the right result.
printenv
node -e "console.info(process.env)"

If both also output $GOPATH, you can output all variables once in httpyac.
{{
console.info(process.env)
}}

How VSCode accesses the environment variables under MacOSX or how they are set, I unfortunately don't remember. My last Mac was too long ago. But if bash also has to be configured extra, maybe this one is missing the corresponding configuration (/etc/launchd.conf?).

When I run printenv and node -e "console.info(process.env)" in my shell I get all the environment variables I expected and GOPATH among them.
I also added console.info(process.env) to a pre request script and I get different output depending on how I run the request. When I run it via shell, I get the expected environment variables and GOPATH among them. When I run it via VSCode extension, I get the following env variables:
{
  "ELECTRON_RUN_AS_NODE": "1",
  "USER": "user.name",
  "COMMAND_MODE": "unix2003",
  "__CFBundleIdentifier": "com.microsoft.VSCode",
  "PATH": "/usr/bin:/bin:/usr/sbin:/sbin",
  "HOME": "/Users/user.name",
  "SHELL": "/bin/zsh",
  "XPC_FLAGS": "0x0",
  "LOGNAME": "user.name",
  "ORIGINAL_XDG_CURRENT_DESKTOP": "undefined",
  "VSCODE_CWD": "/",
  "VSCODE_NLS_CONFIG": "{\"locale\":\"en-gb\",\"availableLanguages\":{},\"_languagePackSupport\":true}",
  "VSCODE_IPC_HOOK": "/Users/user.name/Library/Application Support/Code/1.60.2-main.sock",
  "VSCODE_PID": "591",
  "PWD": "/Users/user.name",
  "SHLVL": "0",
  "_": "/usr/bin/open",
  "OLDPWD": "/Users/user.name",
  "VSCODE_AMD_ENTRYPOINT": "vs/workbench/services/extensions/node/extensionHostProcess",
  "VSCODE_PIPE_LOGGING": "true",
  "VSCODE_VERBOSE_LOGGING": "true",
  "VSCODE_LOG_NATIVE": "false",
  "VSCODE_HANDLES_UNCAUGHT_ERRORS": "true",
  "VSCODE_LOG_STACK": "false"
}

Apparently this is something strictly related to how VSCode executes the extension.

Can you start VSCode from your zsh Shell using command code? I think, if you start it with Spotlight you get not the same env variables as with zsh.

That's right, the env variables depends on how you launch VSCode. If I launch it using the code command, it works.
I found an issue related to this behaviour and apparently this is expected. I think I will avoid shell env variables and rely on constants defined within the httpyac environment.
Thank you for the support and the great tool!


how to get all friends of a user

user = FbGraph2::User.new('user_id').authenticate('access_token')
user = user.fetch
user.friends   # get only one or two users not all the users

You can't do that because FB closed the feature.

thanks for reply

Is there any alternatives I can use to be able to invite the user's friends to use my app ??


[VS Code] Cannot read property 'isAnonymous' of undefined

Error:
Cannot read property 'isAnonymous' of undefined
Steps to Reproduce:

Version Data

extensionName
VSLS

extensionVersion
0.3.954

protocolVersion
2.2

applicationName
VSCode

applicationVersion
1.29.0

platformName
Windows

platformVersion
10.0.16299

Dup of https://github.com/MicrosoftDocs/live-share/issues/1282.