How to move session_with_store

Hi, I am using this library with async-graphql and I would like to access session_with_store inside my handler.
So I have a code looks like this
.and_then(
            move |session_with_store: SessionWithStore<MemoryStore>,
                  (schema, mut request): (SchemaRoot, async_graphql::Request)| async move {
                request = request.data(session_with_store);

                let response = Response::from(schema.execute(request).await).into_response();

                Ok::<_, Infallible>((response, session_with_store))
            },
        )
        .untuple_one()
        .and_then(warp_sessions::reply::with_session);

The issue is that I cannot move session_with_store into request.data as it looks like that it is being use later in warp_sessions::reply::with_session.
I wonder if there is a way or best practice to achieve this? Thanks,

Might be related to https://github.com/http-rs/async-session/issues/20

It would be nice to make session inside SessionWithStore Arc<Mutex> or something, it is very common that we need to pass session around to mutate it.

@phungtranminh I think the solution here is to defer to the consuming code how it would like to pass around a shared, potentially mutable reference to the session, as enforcing this within SessionWithStore itself would restrict said code. Here is a potential solution for you to extract the session, wrap it an Arc<RwLock> for shared mutability, and then re-inserting it into SessionWithStore for processing by with_session:
                        let shared_session = Arc::new(RwLock::new(session_with_store.session));
                        shared_session
                            .write()
                            .unwrap()
                            .insert("test", "test")
                            .unwrap();
                        session_with_store.session = Arc::try_unwrap(shared_session)
                            .unwrap()
                            .into_inner()
                            .unwrap();

@phungtranminh I think the solution here is to defer to the consuming code how it would like to pass around a shared, potentially mutable reference to the session, as enforcing this within SessionWithStore itself would restrict said code. Here is a potential solution for you to extract the session, wrap it an Arc<RwLock> for shared mutability, and then re-inserting it into SessionWithStore for processing by with_session:
let shared_session = Arc::new(RwLock::new(session_with_store.session));
shared_session
    .write()
    .unwrap()
    .insert("test", "test")
    .unwrap();
session_with_store.session = Arc::try_unwrap(shared_session)
    .unwrap()
    .into_inner()
    .unwrap();

@phungtranminh If you check the examples/ folder you'll find a new example under shared_mutable_session.rs :)

Thanks for the suggestion @ajpauwels it works well.
Just to be clear, I can clone shared_session with relatively cheaper cost?.

Yes, cloning shared_session in this case is cheap as it simply clones a reference to the session struct and increases Arc's internal ref counter.
If this solution works for you I will go ahead and close this issue. If I missed something for any reason, please feel free to re-open or create a new issue :)

Thanks again @ajpauwels


Fix: PDF loader breaks pinecone upsert

The latest PDF loader creates Documents with a pdf property in the metadata that is an object. However, this throws an error error [Error: PineconeClient: Error calling upsert: Error: value must be a string, number, boolean or list of strings
Pinecone expects the metadata properties to not be objects as is the case with the current PDF loader.
To fix this, I flattened the pdf object and stringified its values.

Not sure if related but facing a similar issue:
error - unhandledRejection: [Error: PineconeClient: Error calling query: Error: PineconeClient: Error calling queryRaw: FetchError: The request failed and the interceptors did not return an alternative resp onse]
      const blob: Blob = new Blob([contents], { type: "application/pdf"})
      const loader = new PDFLoader(blob);
      const rawDocs = await loader.load();

      /* Split text into chunks */
      const textSplitter = new RecursiveCharacterTextSplitter({
        chunkSize: 1000,
        chunkOverlap: 200,
      });

      const docs = await textSplitter.splitDocuments(rawDocs);
      // metadata: { source: 'blob', blobType: 'application/pdf', pdf: [Object] }

There's a lib that will do the flattening for us: https://www.npmjs.com/package/flat
There's these three lines (that I can see) where we'd need to wrap metadata in flatten

https://github.com/hwchase17/langchainjs/blob/main/langchain/src/vectorstores/pinecone.ts#L56

https://github.com/hwchase17/langchainjs/blob/main/langchain/src/vectorstores/pinecone.ts#L86

https://github.com/hwchase17/langchainjs/blob/main/langchain/src/vectorstores/pinecone.ts#L113

Then once https://github.com/hwchase17/langchainjs/pull/424 is in, you could filter by { "pdf.foo": "bar" }

@mayooear did you want to pick this up, or I can?

@mayooear I think using something like that flat library suggested above should work, something like

any questions let me know

@mayooear I think using something like that flat library suggested above should work, something like

any questions let me know

Thanks, I'll work on this shortly and send the updated commit.

@mayooear I think using something like that flat library suggested above should work, something like

any questions let me know

@nfcampos  please review the changes.

@mayooear I tried to push a commit to your branch fixing the merge conflicts but looks like I'm not able to push your branch. Can you fix the merge conflicts

@mayooear I tried to push a commit to your branch fixing the merge conflicts but looks like I'm not able to push your branch. Can you fix the merge conflicts

It should work now.

thanks @mayooear I've merged this in now


Howto create Minecraft worlds

A minecraft world is created using a seed:
https://www.makeuseof.com/tag/best-minecraft-seeds/

https://minecraft.gamepedia.com/Mods/Installing_Forge_mods
http://files.minecraftforge.net/

https://www.instructables.com/id/Python-coding-for-Minecraft/

https://www.curseforge.com/minecraft/mc-mods

It seems they idea is to crate mods for Minecraft:
https://mcmodhelp.fandom.com/wiki/Minecraft_Modding_Wiki
https://minecraft.gamepedia.com/Mods
The ideas is to became a modder

Mods are changes in the Java classes of Minecraft. There is no API to extend gracefully Minecraft, so it is tricky and can break things. It is a risky process.

https://www.pinterest.co.uk/loumcl/ideas-for-our-minecraft-worlds/ Inspiration to build things

https://github.com/EngineHub/WorldEdit
WorldEdit is Minecraft mod that turns Minecraft into an in-game map editor (sorta, kinda).

The mods world is impressive. Take a look to mods like:
https://ftb.gamepedia.com/BuildCraft
There are mods based on other mods. Compatibility is tricky.


move all validation into Param.validateValue

@DanCech is that like what you had in mind in your recent comment?

should the option checking be done after "None is ok for optional params" ?

right, it should! Updated it


Implement Tinkoff broker

Codecov Report

Merging #3 (f798b0f) into master (0f4b111) will decrease coverage by 1.58%.
The diff coverage is 38.46%.

@@            Coverage Diff             @@
##           master       #3      +/-   ##
==========================================
- Coverage   63.76%   62.17%   -1.59%
==========================================
  Files           1        1
  Lines         218      230      +12
==========================================
+ Hits          139      143       +4
- Misses         76       84       +8
  Partials        3        3

Impacted Files
Coverage Δ

trengin.go
62.17% <38.46%> (-1.59%)
:arrow_down:

Continue to review full report at Codecov.

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


Correct /about route for GH Pages deployment

/about route works locally but not on GH Pages
This presumably extends to all other non-default react-router-dom endpoints but needs to be fixed for the app to be useful to most

Closed with #14


MAC: Number of retansmissions

Actually 802.15.4 MAC layer performs up to 15 retransmissions. The value is defined in mac.hpp:72 as kMaxFrameRetries.
Thread Specification mandates to set macMaxFrameRetries PIB (retries allowed after a
transmission failure) to be 3.
Is there any good reason to perform that huge number of retransmissions?

The Thread Spec defines FAILED_CHILD_TRANSMISSIONS and FAILED_ROUTER_TRANSMISSIONS in a way that allows "up to 16 unacknowledged individual frame transmissions and retries".  Currently, OpenThread does not differentiate between MAC and higher layer retransmissions so they are combined as you noticed.  Do you feel that implementing the retransmission logic at both the MAC and higher layer is beneficial?

I think it might be worth while. For the Windows implementation, when running OpenThread on host, we are investigating making minor changes to the Mac layer to push all backoff and retry logic to the radio layer itself, since Windows cannot guarantee the timer responsiveness that layer requires. If that happens, then it would require the separation of MAC and higher layer retransmission logic, correct?

Thanks for comments!
How I understand the Thread Specification is that definition of FAILED_CHILD_TRANSMISSIONS and FAILED_ROUTER_TRANSMISSIONS are completely independent from macMaxFrameRetries. Those parameters i would use only when MCPS-DATA.conf for some message returns NO_ACK status, and only for checking if some node has timed-out.
Also I don't see a reason to care about retransmissions of application layer protocol on MAC layer, e.g. MLE specifies optional retransmission pattern (4.9) also CoAP uses its own different one.

Okay, I understand the distinction now.  OpenThread's current implementation will transmit a given packet up to 16 times and, if no ack is received, the link will be invalidated.  The Thread spec implies that it should take 4 distinct packets (each with 4 MAC transmissions) before the link is invalidated.  Concretely, if an application is sending packets with a period of X seconds, then OpenThread will invalidate a link on the 1st application packet whereas the Thread specified behavior would invalidate the link after the 4th application packet (3 * X seconds).  Yet another behavior that is not explicitly tested in the cert test plan.  I'm fine with updating kMaxFrameRetries to 3 to be inline with the spec.


Release the GIL in core transformation code

See discussion in https://github.com/pyproj4/pyproj/pull/380#issuecomment-516365850.
By using nogil in cython we can ensure to release the GIL in the hot spots of the transformation code, which can ensure that other applications can use pyproj code in parallel (eg using dask)

I think before we do this, we will need to check in with the PROJ devs to see if this is safe. Maybe do some testing with it as well.
Are you currently able to use dask with multithreading? Or is this blocking anything?

I think it is certainly interesting to discuss with PROJ devs about thread safety using single PJ objects (to do parallel work in pyproj as you were trying), as I mentioned also on the PR.
But releasing the GIL would already allow to 1) do other calculations that release the GIL in parallel or 2) to do transformations in parallel if you ensure to create the Transformer object in each thread.
Eg to_crs in geopandas creates a Transformer object within that method, so you could see a case where this method is called on different chunks of the dataframe in parallel.

threading related issue: https://github.com/OSGeo/PROJ/issues/1047

Looks like you need a new context per thread. This conflicts with this issue #374

Based on current master for reference with future changes:

import concurrent.futures
from pyproj import Transformer

TRANSFORMER = Transformer.from_crs(4326, 3857)

def transform_point(aa):
    assert TRANSFORMER.transform((12, 11), (13, 12)) == ((1447153.3803125564, 1335833.8895192828), (1345708.4084091093, 1232106.80189676))

def transform_point__recreate(aa):
    assert Transformer.from_crs(4326, 3857).transform((12, 11), (13, 12)) == ((1447153.3803125564, 1335833.8895192828), (1345708.4084091093, 1232106.80189676))

def transform_threaded_recreate():
    with concurrent.futures.ThreadPoolExecutor(max_workers=10) as executor:
        executor.map(transform_point__recreate, range(20))

def transform_threaded():
    with concurrent.futures.ThreadPoolExecutor(max_workers=10) as executor:
        executor.map(transform_point, range(20))

def transform():
    [transform_point(ii) for ii in range(20)]

def transform_recreate():
    [transform_point__recreate(ii) for ii in range(20)]

Based on this, I think you would have to have a way to prevent re-starting threads and re-creating transformers each time a thread runs a job. You would have to have a each thread do a batch of transforms based on a Transformer created at the start of the thread job.

Based on this, I think you would have to have a way to prevent re-starting threads and re-creating transformers each time a thread runs a job. You would have to have a each thread do a batch of transforms based on a Transformer created at the start of the thread job.

I don't fully follow what you are saying here.
The use case I have in mind is to_crs on a dask-based GeoDataFrame. You could transform each chunk in parallel, and indeed in the function you apply in parallel you need to recreate the Transformer object (as otherwise it cannot be threadsafe).
I did a quick experiment, taking the code base from a few weeks ago (before the global PROJ_CTX was introduced), and added a with nogil: statement around the proj_trans_generic call in _Transformer._transform. And with 4 cores, it gives me a 2x speed-up (so not really great).

Interesting, so a 2x speedup from not releasing the Gil? Does shapely release the Gil?

No, not with shapely. I should have been clearer, it was not a direct timing of to_crs (which indeed currently goes through shapely's transform), but just focusing on the pyproj transform part. So what I timed was running this function on 10 million points:
def _geopandas_to_crs(x, y, crs_in, crs_out):
    # specific optimized version for points
    proj_in = pyproj.Proj(crs_in, preserve_units=True)
    proj_out = pyproj.Proj(crs_out, preserve_units=True)
    transformer = pyproj.Transformer.from_proj(proj_in, proj_out, always_xy=True)
    x2, y2 = transformer.transform(x, y)
    return x2, y2

Which is certainly not the only part involved in to_crs. Because if we want to go this way for geopandas in general, we need functionality to first extract all coordinates of the geometries and then reconstruct geometries (something which needs to happen anyway (and currently happens with shapely's transform), but would be interesting to time if doing all proj transformations in one go by passing an array is faster)


[WinUI3 1.4 Regression] Incorrect open direction in CommandBar

Describe the bug
Set CommandBar.ClosedDisplayMode = "Minimal"
Closed:

Opened (Incorrect):

Steps to reproduce the bug

Create a new CommandBar with multiple PrimaryCommands in XAML
Set ClosedDisplayMode to "Minimal"

Expected behavior
Open direction should be "Top" instead of "Bottom"
Screenshots
No response
NuGet package version
WinUI 3 - Windows App SDK 1.4.0: 1.4.230822000
Windows version
Windows 11 (22H2): Build 22621
Additional context
This issue has never seen in Windows App Sdk 1.3.X

@bpulliam Still exists in Windows App SDK 1.4.2. Could share any plans about fixing this bug?

@bpulliam @Scottj1s Please make it more transparent but just keeping silent.

@llongley Not fixed in the latest WindowsAppSdk release?

We have the fix in WinAppSDK 1.5.


Master testing broken

Seems like it is due to the Ruby 2.4 PR changes:
spec/spec_helper.rb:13:in `remove_const': constant ActiveRecord::TestFixtures not defined (NameError)
cc @juanignaciosl

I'd love to know how come tests passed back then >_<

:|

Confirmed 🍏


Not Disposing SqlCommand or SqlDataReader on Caching.SqlServer

Describe the bug
We are being hit at least weekly with this issue filed in corefx.
Going through the source code of the Caching.SqlServer package I notice that some classes that implement IDisposable are not wrapped in a using block.
https://github.com/aspnet/Extensions/blob/f2ef1b55d65feac6ad4e3dd8b04daa21f464631b/src/Caching/SqlServer/src/DatabaseOperations.cs#L215-L227
In this snippet both SqlCommand and the SqlDataReader implement IDisposable but they are not disposed, and looking at their implementation it looks like they are actually doing some clean up work:
https://github.com/dotnet/corefx/blob/a53ada190e2e0bd229e76ae4a6d45c13a986ab18/src/System.Data.SqlClient/src/System/Data/SqlClient/SqlCommand.cs#L777-L785
https://github.com/dotnet/corefx/blob/a53ada190e2e0bd229e76ae4a6d45c13a986ab18/src/System.Data.SqlClient/src/System/Data/SqlClient/SqlDataReader.cs#L773-L793
Is there a particular reason why this is not done?

Is there any progress with this investigation?
We are about to put this library into production is this a stopper, or we should proceed?
Thanks in advance

I submitted a PR that was merged, but it will not be available until 3.0.
The truth is I don’t know if this fix will actually resolve the issues we’ve been having in production.
Also, the fact that no one else have reported this makes it look it’s a problem on my side, rather than the framework, but at the same time we interact with other DBs and we have not run into this problem (following the recommendation of disposing of all ADO.NET objects).
I would really like to hear from you if this issue comes up. Maybe then, @tratcher will consider it for a servicing release.
I’m going to close this issue because the PR was accepted.


Added Static File Handler

Added Static File Handling
If a public directory is found then it will create a public endpoint by default
If we want to added any other folders can use AddStaticFiles endpoint and then if endpoint and directory is provided then it will check whether the particular endpoint is available otherwise throws an error
Closes: #502.

Let me know if any changes can be done to accommodate more functionality.

@KedarisettiSreeVamsi Thankyou for your contribution.
Please add consider adding tests for any new code added as not doing it will decrease the overall code coverage.
Also, in my opinion we don't actually need an example of how to serve static files. Your example is small and simple so a better way is to add a documentation for this.
You can add it in the advanced-guide directory under the docs directory of the repository.

@KedarisettiSreeVamsi please add unit tests for the feature you've implemented!

Hi @vipul-rawat,
Can you please mention how I can write unit tests without a static folder in the same folder as test file? Should I use a file within the folder for the testing purpose?

I am pretty new to this so was the reason I am asking this question.

@KedarisettiSreeVamsi the pipeline for your PR is failing: https://github.com/gofr-dev/gofr/actions/runs/9414675135
Please have a look and fix

Poor @KedarisettiSreeVamsi this code review is a long journey

Now the unit tests have been fixed, can someone please merge it.

I hope you will my review useful even I'm doing a machine gun review 😅

Please resolve the comments one by one, so I can see what is fixed. Thanks

Facing a weird issue when making the static folder as default as the workflow pipeline is failing for favicon handler and when public is made the default folder then I am not facing the issue. Can someone help in this case as to why it is failing? Is it due to embedding of static in static/files.go for favicon testing?

Currently I made public as the default endpoint to check if tests are still failing.

@KedarisettiSreeVamsi - thanks for creating a separate ticket for codeclimate fix. Although we would need to fix linter errors in this PR itself, so please check on that.

Hi @srijan-27,
I do understand that I need to fix the linter issues but I need some help to fix those. Also as I mentioned above, if someone can help me why when keeping static as default folder the favicon test is failing and when public is made static the test is passing.

@KedarisettiSreeVamsi I have fixed the linters, you may see the commit to understand it more.

Hi @aryanmehrotra,
Thanks for the help. Can we merge the pull request then?

Currently I made public as the default endpoint to check if tests are still failing.

@KedarisettiSreeVamsi please make the default dir as static as we already have static files in the same dir. As for the failing tests, it is failing because of the os.RemoveAll() on line 614 clean up, instead of testing you can name the HTML file as test.html and remove the specific one! This is a small change.

@vipul-rawat @srijan-27 @aryanmehrotra
What are your thoughts about this ?
https://github.com/gofr-dev/gofr/pull/674#discussion_r1646130482
A side topic that I raised by reviewing the PR.
You could reply in the thread for keeping the conversation in one place

Hi @ccoVeille,
I think we already have api's with .well-known is it really necessary to expose .well-known folder?

Hi @ccoVeille,
I think we already have api's with .well-known is it really necessary to expose .well-known folder?

.well-known file has to been publicly exposed
Lucky there is a bug I spotted here, so it works
https://github.com/gofr-dev/gofr/pull/674#discussion_r1652005236

I might be wrong, it needs further checks

Can someone resolve the conflict of docs/reference/configs/page.md as it is blocking all the tests? I tried to fix it but unfortunately it Is not getting fixed.

ExcludeExtensions, HideDotFiles can be removed for now, I dont find a valid usecase. (IMO user can just not keep that file if they dont want it to be rendered) (fixed)

Seem safer and avoid potential breaking changes 👍


[typespec-csharp] Remove dependency "typespec-autorest"

Removing this dependency causes one spec to fail generation in the net repo:
 npx tsp compile /mnt/vss/_work/1/s/azure-sdk-for-net/sdk/healthinsights/
   Azure.Health.Insights.ClinicalMatching/TempTypeSpecFiles/healthinsights.trialmatcher/client.tsp
   --emit @azure-tools/typespec-csharp
   --option @azure-tools/typespec-csharp.emitter-output-dir=
   /mnt/vss/_work/1/s/azure-sdk-for-net/sdk/healthinsights/
   Azure.Health.Insights.ClinicalMatching/src
 39>_GenerateCodeForTypeSpec:
 TypeSpec compiler v0.45.2

 41>_GenerateCodeForTypeSpec:
 TypeSpec compiler v0.45.2

 39>_GenerateCodeForTypeSpec:
 Diagnostics were reported during compilation:

 /mnt/vss/_work/1/s/azure-sdk-for-net/sdk/healthinsights/Azure.Health.Insights.CancerProfiling/TempTypeSpecFiles/healthinsights.oncophenotype/route.oncophenotype.tsp:2:1 - error import-not-found: Couldn't resolve import "@azure-tools/typespec-autorest"
 > 2 | import "@azure-tools/typespec-autorest";
 | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
 /mnt/vss/_work/1/s/azure-sdk-for-net/sdk/healthinsights/Azure.Health.Insights.CancerProfiling/TempTypeSpecFiles/healthinsights.oncophenotype/route.oncophenotype.tsp:12:7 - error unknown-identifier: Unknown identifier Autorest
 > 12 | using Autorest;
 | ^^^^^^^^

https://dev.azure.com/azure-sdk/internal/_build/results?buildId=2867478&view=logs&j=a59e8b2b-145b-5c08-bac3-c8d41476a686&t=b360b5d6-342a-59db-d57c-25f0340a37f5&l=4663
The spec is here:
https://github.com/Azure/azure-rest-api-specs/blob/bdc89afc8a56a7268828efa4cd81a50bf7ebde2f/specification/cognitiveservices/HealthInsights/healthinsights.oncophenotype/client.tsp
client.tsp imports route.oncophenotype.tsp, which does import @azure-tools/typespec-autorest.
It appears this spec was implicitly relying on the C# emitter to pull in @azure-tools/typespec-autorest as a transitive dep.
How should this be resolved?

Add @azure-tools/typespec-autorest to the root package.json in the azure-sdk-for-net repo?
Change the spec to not import @azure-tools/typespec-autorest when generating the C# client?
Something else?

The one thing I don't think we should do, is keep the dependency on @azure-tools/typespec-autorest in @azure-tools/typespec-csharp, since the emitter doesn't actually need this dep in general.

Removing this dependency causes one spec to fail generation in the net repo:
 npx tsp compile /mnt/vss/_work/1/s/azure-sdk-for-net/sdk/healthinsights/
   Azure.Health.Insights.ClinicalMatching/TempTypeSpecFiles/healthinsights.trialmatcher/client.tsp
   --emit @azure-tools/typespec-csharp
   --option @azure-tools/typespec-csharp.emitter-output-dir=
   /mnt/vss/_work/1/s/azure-sdk-for-net/sdk/healthinsights/
   Azure.Health.Insights.ClinicalMatching/src
 39>_GenerateCodeForTypeSpec:
 TypeSpec compiler v0.45.2

 41>_GenerateCodeForTypeSpec:
 TypeSpec compiler v0.45.2

 39>_GenerateCodeForTypeSpec:
 Diagnostics were reported during compilation:

 /mnt/vss/_work/1/s/azure-sdk-for-net/sdk/healthinsights/Azure.Health.Insights.CancerProfiling/TempTypeSpecFiles/healthinsights.oncophenotype/route.oncophenotype.tsp:2:1 - error import-not-found: Couldn't resolve import "@azure-tools/typespec-autorest"
 > 2 | import "@azure-tools/typespec-autorest";
 | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
 /mnt/vss/_work/1/s/azure-sdk-for-net/sdk/healthinsights/Azure.Health.Insights.CancerProfiling/TempTypeSpecFiles/healthinsights.oncophenotype/route.oncophenotype.tsp:12:7 - error unknown-identifier: Unknown identifier Autorest
 > 12 | using Autorest;
 | ^^^^^^^^

dev.azure.com/azure-sdk/internal/_build/results?buildId=2867478&view=logs&j=a59e8b2b-145b-5c08-bac3-c8d41476a686&t=b360b5d6-342a-59db-d57c-25f0340a37f5&l=4663
The spec is here:
<EMAIL_ADDRESS>client.tsp imports route.oncophenotype.tsp, which does import @azure-tools/typespec-autorest.
It appears this spec was implicitly relying on the C# emitter to pull in @azure-tools/typespec-autorest as a transitive dep.
How should this be resolved?

Add @azure-tools/typespec-autorest to the root package.json in the azure-sdk-for-net repo?
Change the spec to not import @azure-tools/typespec-autorest when generating the C# client?
Something else?

The one thing I don't think we should do, is keep the dependency on @azure-tools/typespec-autorest in @azure-tools/typespec-csharp, since the emitter doesn't actually need this dep in general.

I also met this problem during the upgrade of typespec 0.45.0. Since the dependency is defined in the specification, I chose to keep the typespec/autorest. I'm curious how other language emitters handle this.

I also met this problem during the upgrade of typespec 0.45.0. Since the dependency is defined in the specification, I chose to keep the typespec/autorest. I'm curious how other language emitters handle this.

According to the Typespec team, most specs should not be importing typespec-autorest directly.  I'm working to see if I can remove it from the specs (https://github.com/Azure/azure-rest-api-specs/issues/24565).

This PR should fix the spec that is failing to generate code in the net repo:
https://github.com/Azure/azure-rest-api-specs/pull/24598

This PR should fix the spec that is failing to generate code in the net repo:
Azure/azure-rest-api-specs#24598

@mikeharder, not fully understand that PR Azure/azure-rest-api-specs#24598, if we remove @example, how can we link the payload with the operation?

@markcowl: Can you answer this question?

@m-nash: I think this is ready to merge now.  It's passing checks "autorest.csharp - ci" and "Autorest Regen Preview (typespec-current)".  It's failing "Autorest Regen Preview (typespec-next)", but the error is the same as "feature/v3", so I think this is unrelated.

azure-sdk-for-net pipeline is restored. Fell free to merge it.


Separating files it doesn't work

Hi everyone !!
I have created a project with vue-cli and I would like to separate the js and css from the * .vue file but when I do this error appears to me.
[Vue warn]: Failed to mount component: template or render function not defined.
By default the project has been created in runtime-only mode but I have added the alias ('vue $': 'vue / dist / vue.js') to switch to standalone.
What am I doing wrong?
my App.vue
<template>

		<router-view></router-view>

</template>

my Hello.vue
<template>

{{ msg }}

Essential Links

				Core Docs

				Forum

				Community Chat

				Twitter

				Docs for This Template

Ecosystem

				vue-router

				vuex

				vue-loader

				awesome-vue

</template>

my Hello.js
export default {
  name: 'hello',
  data () {
    return {
      msg: 'Welcome to Your Vue.js App'
    }
  }
}

my package.json
{
  "name": "edrive",
  "version": "1.0.0",
  "description": "Expert Drive",
  "author": "Antonio Pérez",
  "private": true,
  "scripts": {
    "dev": "node build/dev-server.js",
    "start": "node build/dev-server.js",
    "build": "node build/build.js",
    "lint": "eslint --ext .js,.vue src"
  },
  "dependencies": {
    "vue": "^2.4.4",
    "vue-router": "^2.7.0"
  },
  "devDependencies": {
    "autoprefixer": "^7.1.4",
    "babel-core": "^6.26.0",
    "babel-eslint": "^8.0.1",
    "babel-loader": "^7.1.2",
    "babel-plugin-transform-runtime": "^6.23.0",
    "babel-preset-env": "^1.6.0",
    "babel-preset-stage-2": "^6.24.1",
    "babel-register": "^6.26.0",
    "chalk": "^2.1.0",
    "connect-history-api-fallback": "^1.3.0",
    "copy-webpack-plugin": "^4.1.0",
    "css-loader": "^0.28.7",
    "cssnano": "^3.10.0",
    "eslint": "^4.8.0",
    "eslint-config-standard": "^10.2.1",
    "eslint-friendly-formatter": "^3.0.0",
    "eslint-loader": "^1.9.0",
    "eslint-plugin-html": "^3.2.2",
    "eslint-plugin-import": "^2.7.0",
    "eslint-plugin-node": "^5.2.0",
    "eslint-plugin-promise": "^3.5.0",
    "eslint-plugin-standard": "^3.0.1",
    "eventsource-polyfill": "^0.9.6",
    "express": "^4.16.1",
    "extract-text-webpack-plugin": "^3.0.1",
    "file-loader": "^1.1.4",
    "friendly-errors-webpack-plugin": "^1.6.1",
    "html-webpack-plugin": "^2.30.1",
    "http-proxy-middleware": "^0.17.4",
    "opn": "^5.1.0",
    "optimize-css-assets-webpack-plugin": "^3.2.0",
    "ora": "^1.3.0",
    "rimraf": "^2.6.2",
    "semver": "^5.4.1",
    "shelljs": "^0.7.8",
    "url-loader": "^0.6.1",
    "vue-loader": "^13.0.5",
    "vue-style-loader": "^3.0.3",
    "vue-template-compiler": "^2.4.4",
    "webpack": "^3.6.0",
    "webpack-bundle-analyzer": "^2.9.0",
    "webpack-dev-middleware": "^1.12.0",
    "webpack-hot-middleware": "^2.19.1",
    "webpack-merge": "^4.1.0"
  },
  "engines": {
    "node": ">= 4.0.0",
    "npm": ">= 3.0.0"
  },
  "browserslist": [
    "> 1%",
    "last 2 versions",
    "not ie <= 8"
  ]
}

Thanks in advance.

You should probably rather ask on forum.vuejs.org and/or chat.vuejs.org
Also, from what you showed, it's not clear what your problem is. You say you want to remove the CSS from the .vue file, yet all your code still has it in them.
What did you try to do to make this error appear?

@LinusBorg thanks for your fast reply and I will ask the question in the indicated forum.
In the published example I just removed the javascript from the Hello.vue file and put it in the Hello.js file. Once the change is made, I execute "npm run dev" opens the browser (Chrome) and the indicated error appears in the console.
I want to have javascript files in order to use esLint.
Thanks.


Import and export of the functions

I love the AFE, but after the GUI had changed from free-text editor to a new compartmentalized look where each function definition is inside it's own section, the implementation is inline and can't be edited unless the edit button is hit makes it much less useful for me suddenly. When I started to use it I loved the possibility to simply copy the whole thing at once, keep it in a separate text file inside GIT and work with it as a normal source code - diffs, branches, refactorrings etc. Is there a way how I could keep the copy of the function list in a local plain text file and import/export it as needed? I could use a stronger text editor/ide to work with the code and then simply test it after importing back (or synchronizing somehow) to the Workbook.
Thank you in advance for any suggestions.

Does the module's tab help here? You can copy and paste the content between a file? Was there a specific feature in the previous version you were missing?
I think previously there was a "import from text" which was slightly different to what we have now.

Hi Jack, it partially does. I can migrate it all there, but it would help a lot if I can simply export the whole thing to a text file.
Additionally, what is the future of the Module tab? Will it stay in form of a free text editor, or will it suffer a similar redesigns in the foreseeable future?
I don't really mind how is the built-in editor done, but I would appreciate if I can keep and edit it outside of AFE.
Thank you in advance!


[FeatureRequest] Support container-level output binding for blob storage

There are some scenarios when we need to output several files from Azure function into blob storage. In this case, it would be convenient to be able to bind to output blob container declaratively, and then write multiple files to it. Similar to the way multiple rows of table can we written when dealing with Azure tables output binding in C# using IWriter.
Related to #652.
/cc @stefanushinardi @dariagrigoriu

@eamonoreilly fyi

@cachai2 Let's see if we can make some progress on this next semester. This would help other bindings and languages too.
Related: #654

This can be addressed by using SDK Type Bindings for Blob. For this particular situation, binding to a ContainerClient type would be useful. You can dynamically define the container through an HTTP trigger or other method and use the client type to upload multiple files to the container.
For more information on how to use SDK Type Bindings:

SDK type bindings for Azure Blob Storage
Python Developer Reference - SDK Type Bindings


RFC: move decision-making of desired VM size to VM monitor

See also draft implementation of this in:

https://github.com/neondatabase/neon/pull/8113 and
https://github.com/neondatabase/autoscaling/pull/980

+1 on the high-level idea that the workload should request the compute size, not an external observer.
I'm missing details on the ScaleRequest semantics. Is it a synchronous call? Is it just a "would be nice to have but until you give it to me, I will work with existing resources"? Is the response to the ScaleRequest an estimate for how long it's going to take until the upscaling is complete?

It's "would be nice to have but until you give it to me, I will work with existing resources". The agent doesn't send any response to the ScaleRequest. If the ScaleRequest results in upscaling or downscaling, however, the agent will send a DownScaleRequest or UpscaleNotification to the VM monitor, just like it does today when it decides to perform an upscale or downscale.
I'm not sure that's the best protocol, but I think it's the path of least resistence, because it's very close to how the current UpscaleRequest mesage works.


Please fix for the new update, nothing works.

The modded lobbies dont show, bark mod menu doesnt work, and yizzi doesnt work. please help.

use new bx

bark is broken and the person who made it isn't going to fix it 😭😭😭😭


Replace bent by internal fetch API + Refactor

I made a few more optimizations :slightly_smiling_face:  Because of the internal fetch API the minimum requirement is now node version 18, that is also the requirement for the current MagicMirror² version.

Yeah, the new internal Fetch API makes things a bit easier :slightly_smiling_face:


[FLINK-24577][table-planner] Implement Cast from BINARY to RAW

What is the purpose of the change
Implement CAST from BINARY/VARBINARY/BYTES to RAW(<class>).
Brief change log

Implement BinaryToRawCastRule
Add support for explicit cast in LogicalTypeCasts
Make changes to RexSimplify and RexLiteral to enable the cast

Verifying this change
This change added tests and can be verified as follows:

Added tests in CastRulesTest
Added tests in LogicalTypeCastsTest
Added tests in CastFunctionMiscITCase

Does this pull request potentially affect one of the following parts:

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

Documentation

Does this pull request introduce a new feature? yes
If yes, how is the feature documented? docs

To allow this cast we need to make changes to RexSimplify and RexLiteral.
If you agree with these, I'll update the javadocs of those classes to reference the changes we need.


open menu on hover

how can i open menu on mouse hover ??
example: when user move to menu i don't want to click mouse on menu. i need to open menu on Hover

Hi @mdnafeed ,
you can use ControlledMenu component to manage what triggers menu to open, here is a very basic example: https://szhsin.github.io/react-menu#managing-state
If you want to enable opening and closing transitions and make them smooth, a timer can be used to bridge the mouse enter and leave events, https://codesandbox.io/s/react-menu-hover-open-i7isjv

Thanks You @szhsin


LUCENE-10234: Add automatic module name to JAR manifests.

This is what I'd like to add to branch_9_0 (and branch_9x), @jpountz . Can I?

Please do!


Revert "[Diagnostics] Removing FailureDiagnosis::visitCoerceExpr from CSDiag"

Reverts apple/swift#29064
@hamishknight Reports this this might be causing problems with source compatibility suite.

@swift-ci please smoke test

@swift-ci please test source compatibility


'Invalid argument, errno = 22' when calling .open()

I'm receiving the following error when trying to open a box. I haven't been able to find any indication on why this would fail (it's happening to about 5% of users).

Uncaught Platform Error: FileSystemException: read failed, path = '/var/mobile/Containers/Data/Application/D2B812D0-C5B0-45CB-9789-4D0D614B61A2/Documents/graphqlclientstore.hive' (OS Error: Invalid argument, errno = 22)

Full stack trace:

FileSystemException: FileSystemException: read failed, path = '/var/mobile/Containers/Data/Application/D2B812D0-C5B0-45CB-9789-4D0D614B61A2/Documents/graphqlclientstore.hive' (OS Error: Invalid argument, errno = 22)
File "common.dart", line 42, in _checkForErrorResponse
File "file_impl.dart", line 740, in _RandomAccessFile.read.
File "zone.dart", line 1406, in _rootRunUnary
File ""
File "frame_io_helper.dart", line 41, in FrameIoHelper.framesFromFile
File ""
File "storage_backend_vm.dart", line 86, in StorageBackendVm.initialize
File ""
File "hive_impl.dart", line 111, in HiveImpl._openBox
File ""
File "hive_impl.dart", line 142, in HiveImpl.openBox
File ""
File "hive_store.dart", line 41, in HiveStore.openBox
File ""
File "hive_store.dart", line 52, in HiveStore.open

Potentially relevant information. We're seeing another Hive error:

(OS Error: Cannot allocate memory, errno = 12)

I'm facing the same issue, any solution to this?

Nope. We fixed the memory error and are still seeing this one :(


Docs: AQT not included in API documentation

Not sure if this is intentional or not, but the AQT device is entirely unmentioned in the v0.6 (stable) docs.
This could be fixed with something like sphinx-apidoc if this project uses that.
Related:

Several of the functions/classes in the AQT file are either not fully-documented or malformatted.

Examples:
https://github.com/quantumlib/Cirq/blob/9b51176ed42ddc1832a218436903b1f02f0bd844/cirq/aqt/aqt_sampler.py#L52-L56
https://github.com/quantumlib/Cirq/blob/9b51176ed42ddc1832a218436903b1f02f0bd844/cirq/aqt/aqt_sampler.py#L84-L91

Thanks for the report.
We need to gradually expand docs/docs_coverage_test.py to cover all packages (instead of a few whitelisted ones). The aqt package is one of the packages that is not tested for doc coverage yet.

I improved the aqt device documentation. See PR #2578

Closing for inactivity


Remove name for rabbitmq to fix validation error

Related to #962
r? @mstriemer
Seems docker-compose 1.5.1 has remove name from the JSON schema for the docker-compose.yml. It doesn't appear to be changing the container name - so I'm not sure what name used to do or does currently.
This seems to work. But @mstriemer if you have a good test I can run to make sure the queue is functioning I'm happy to test this further.
I did have some issues with error messages relating to volumes but I suspect that's unrelated since my oympia env hadn't been updated for a while. I am however now running the latest of docker-machine (which includes docker-compose 1.5.1 on OSX) and the docker server is updated to match the client.

That should be fine. It was supposed to set the name of the container which the container would use to set something else but just the hostname might be enough. This works for me on docker-compose version: 1.4.2.

r+


Moved data structures and algorithms into core

Moved the RingBuffer data structure to core
Moved the SlidingMax algorithm to core
Moved the smoothed value classes to the new Algorithms folder

This is a non-critical PR that moves files from realtime graph to core.
@jselstad
@protodeep

I don't suppose there's anything we can do about

   if (average_dt * 10 < newValue - value)
     Debug.Log("Average dt was excessive!");

in SmoothedFloat.cs while we're here, is there?   Normal devs probably won't appreciate the stunning insights that this log provides...

Please delete.  We should be robust against long pauses anyhow.  It is not clear this indicates a problem.

I'll change to
if (average_dt != newValue - value)
  Debug.Log("Imperfect dt was imperfect!");

instead then.


Get flutter version 0.0.1

What to do if the build fails?
At the moment contributors do not have access to the CI workflow triggered by StepLib PRs. In case of a failed build, we ask for your patience. Maintainers of Bitrise Steplib will sort it out for you or inform you if any further action is needed.
New Pull Request Checklist
Please mark the points which you did / accept.

[X] I will not move an already shared step version's tag to another commit
[X] I read the Step Development Guideline
[X] I have a test for my Step, which can be run with bitrise run test (in the step's repository)
[X] I did run bitrise run audit-this-step (in the step's repository - note, if you don't have this workflow in your bitrise.yml, you can copy it from the step template.)
[X] I read and accept the Abandoned Step policy

Following up on https://github.com/bitrise-io/bitrise-steplib/pull/2494
New Step
Thank you for the new Step share! The CI check might will fail due to our extended validation engine. Nothing to worry about yet, we will get back to you shortly.

 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.


Removed code and tests depreciated in 0.21

It fixes #10748

Summary
Details and comments

tox test fails on test\benchmarks\scheduling_passes.py as it  only contains check for TimeUnitConversion . Should we keep test\benchmarks\scheduling_passes.py or not ? also i have removed most of the  tests for depreciated or removed classes. Please let me know if i am missing something.

I am unsure about optimizers/spsa - mainly in regards of whatever upgrade note you do for the release note (which has yet to be included here). The optimizers are part of algorithms - I did have a prior issue #9866 that included addressing spsa but when the whole of algorithms was deprecated in the last release I simply closed it. I know in other similar issues that @1ucian0 created any deprecation removal for algorithms was dropped - maybe that's the simplest course here @1ucian0? - not to change spsa here given the whole module its part of is already deprecated - that will avoid any issues around including it or not in the release note

I'm with @woodsp-ibm, I think it's best to leave algorithms-related code untouched until its removal.

I agree that leaving out qiskit.algorithms from this removal is probably the best option (the list was created by a program, my apologies for including it originally.

Take a look to the conflicts! When ready for review, hit Ready for review.

@1ucian0 These conflicts are due to  recent merge of PR #10753  as in the PR #10753  classes ALAPSchedule, AlignMeasures, DynamicalDecoupling are changed which are deprecated in 0.21 (removed with merge of this PR).  I think we can merge this PR. Please let me know what do you think on this ?

These conflicts are due to recent merge of PR #10753 as in the [...] Please let me know what do you think on this ?

I mean, by "take a look" I wanted to say "resolve the conflicts". Sometimes my word choice is not the best. My bad.

@1ucian0 It seems these conflicts can not be resolved by me (refer: How do I fix a merge conflict due to removal of a file in a branch?) Maintainers should remove qiskit/transpiler/passes/scheduling/alap.py, qiskit/transpiler/passes/scheduling/alignments/align_measures.py and qiskit/transpiler/passes/scheduling/dynamical_decoupling.py while merging .  If that is not the case please help me to resolve the conflicts .

Following test is failing on windows , let me check the fix and resolve the conflict:
https://github.com/Qiskit/qiskit/blob/e62c86bbc2720883270641f1c154743a657cfca8/test/python/algorithms/test_measure_error_mitigation.py#L56

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

1 of 1 (100.0%)  changed or added relevant line in 1 file are covered.
99 unchanged lines in 6 files lost coverage.
Overall coverage decreased (-0.1%) to 87.167%

Files with Coverage Reduction
New Missed Lines
%

crates/qasm2/src/expr.rs
1
93.76%

qiskit/transpiler/passes/scheduling/alignments/check_durations.py
2
72.0%

crates/qasm2/src/lex.rs
3
92.17%

qiskit/pulse/library/waveform.py
3
93.75%

qiskit/transpiler/passes/scheduling/base_scheduler.py
33
0.0%

qiskit/transpiler/passes/scheduling/alignments/reschedule.py
57
24.05%

Totals

Change from base Build<PHONE_NUMBER>:
-0.1%

Covered Lines:
73897

Relevant Lines:
84776

💛  - Coveralls


bug in PolygonArea

There is some kind of bug with the masking here:
https://github.com/PyxisInt/GeographicLib/blob/master/PyxisInt.GeographicLib/PolygonArea.cs#L100
I was able to hack enough of a debug session to tell that the g.GeodesicScale12 value was not getting computed inside the Inverse function. This is turn makes any subsequent calls to PolygonArea.Compute fails, i.e. sends back NaN for the area.
I was unable to dig deep enough to find the real culprit. But I did go back to the Java implementation and do my own C# port and it is giving back area answers as expected. I'd be happy to share that port but it doesn't diff well against these sources so I'm kinda at my limit for debugging here.

Thanks! Will take a look


minor fix

Added a line for installing peft library for LoRA training and added clarification about loading the dataset from local files.
Some of the things I am confused about:

What is the use of image_token? If it was missed, where should it be used?
How do we assign padded tokens and the image_token with the value -100? Where do we utilize it?

Hello and sorry for the late response. -100 is ignore_index value of the model where things with -100 aren't calculated in loss, i.e. if we included image tokens and pad tokens in loss calculation, model would see them a lot and repetitively generate them for an easy win.

@arkajyotimitra I think I mistakenly left it there. I started developing this notebook way before PaliGemma was released during integration and the final API wasn't shaped back then, initially we were converting to -100 ourselves but in the final API the processor handles it.


Can't select a Public variable from another flowchart

Reported by Patrick via email

Once you select the public variable the inspector stops displaying the dropdown list.


build(dependencies): 🧱 update crates

Description
Update dependencies:
Bumps serde_json from 1.0.132 to 1.0.133.
Bumps tower-http from 0.6.1 to 0.6.2.
Updating aws-lc-rs v1.10.0 -> v1.11.0
Updating aws-lc-sys v0.22.0 -> v0.23.0
Updating axum v0.7.7 -> v0.7.9
Updating bstr v1.10.0 -> v1.11.0
Updating cc v1.1.37 -> v1.2.1
Updating flate2 v1.0.34 -> v1.0.35
Updating h2 v0.4.6 -> v0.4.7
Updating hyper v1.5.0 -> v1.5.1
Updating itoa v1.0.11 -> v1.0.12
Updating libc v0.2.162 -> v0.2.164
Updating rustix v0.38.40 -> v0.38.41
Updating rustls v0.23.16 -> v0.23.17
Updating schannel v0.1.26 -> v0.1.27
Updating unicode-ident v1.0.13 -> v1.0.14
Type of change

[X] Dependency update

How Has This Been Tested?

[X] cargo test run with all tests passing

Checklist:

[X] My code follows the style guidelines of this project
[X] I have performed a self-review of my own code
[X] I have commented my code, particularly in hard-to-understand areas
[X] I have made corresponding changes to the documentation
[X] My changes generate no new warnings
[X] I have added tests that prove my fix is effective or that my feature works
[X] New and existing unit tests pass locally with my changes
[X] Any dependent changes have been merged and published in downstream modules

:warning: Please install the  to ensure uploads and comments are reliably processed by Codecov.
Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 40.57%. Comparing base (2f487fb) to head (5f8e3d1).

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##             main      #36      +/-   ##
==========================================
+ Coverage   40.54%   40.57%   +0.03%
==========================================
  Files          12       12
  Lines        6312     6307       -5
==========================================
  Hits         2559     2559
+ Misses       3753     3748       -5

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

🚨 Try these New Features:

Flaky Tests Detection - Detect and resolve failed and flaky tests


Use OrderedHashMap for enum_values

Fixes https://github.com/godotengine/godot/issues/55534

Thanks!


Update smartcar.ino

Instead of only using the left odometer to send the speed, the right one was also added to give a more accurate speed display.
:ballot_box_with_check: Definition of Done checklist

[x] Meaningful title and valuable description in pull request
[x] Code change tested manually if not covered by automated tests
[x] Pull request linked to one or more issues/stories
[x] Acceptance criteria of the linked stories satisfied
[x] Code change comprehended by those who approved it
[x] State of target branch maintained or improved after the code change

We did not know that there was a getSpeed() that worked like that, we might implement it later and thank you for telling us.


remove comparison for boolean

page.comments is enough

@junhan-z Actually there is a small detail you're not seeing though.
Currently, the comments are rendered by default. i.e. even if you don't define a comments key in the front matter. One can disable the comments for a particular page by explicitly having comments: false in the front matter.
With your change, the behavior changes.
The comments would not be rendered by default. One has to explicitly have comments: true in all pages and documents to render comments.

@ashmaroli
I see, I didn't consider that, now it makes sense to me, thanks!

Maybe we should add some comments near it?


Don't import the file ~ /.xonsh/add_variable.xsh if it doesn't exist

https://github.com/ErickTucto/xontrib-add-variable/blob/d1310be0a80b1884dc044c2d8a0367cbac364b86/xontrib/add_variable/__init__.py#L9-L17
If it isn't yet created, the following error:

This issue was resolved in the commit fb4a9dd


automatic-timezoned: 1.0.108 -> 1.0.110

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/maxbrunet/automatic-timezoned/releases.
meta.description for automatic-timezoned is: Automatically update system timezone based on location
meta.homepage for automatic-timezoned is: https://github.com/maxbrunet/automatic-timezoned
meta.changelog for automatic-timezoned is: https://github.com/maxbrunet/automatic-timezoned/blob/v1.0.110/CHANGELOG.md
Updates performed

Rust version update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 1.0.110 with grep in /nix/store/dy24f9kfp396rx61vwv08pvb5ay5fava-automatic-timezoned-1.0.110
found 1.0.110 in filename of file in /nix/store/dy24f9kfp396rx61vwv08pvb5ay5fava-automatic-timezoned-1.0.110

Rebuild report (if merged into master) (click to expand)

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
automatic-timezoned

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/dy24f9kfp396rx61vwv08pvb5ay5fava-automatic-timezoned-1.0.110 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A automatic-timezoned https://github.com/r-ryantm/nixpkgs/archive/5f5a37cd7e9949af1bf9f8d02016ec4545e13759.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/dy24f9kfp396rx61vwv08pvb5ay5fava-automatic-timezoned-1.0.110
ls -la /nix/store/dy24f9kfp396rx61vwv08pvb5ay5fava-automatic-timezoned-1.0.110/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

automatic-timezoned

Maintainer pings
cc @maxbrunet for testing.

@ofborg eval


Revised credential refresh rate

Proposing that we should refresh access token every 12 hours.

Force merging since this contains urgent bug fixes.


Move the Karma Car token to examples

And update it to new schema

@JamesSmartCell bump, you can give the file to me and I can upload it if you want

@JamesSmartCell bump, you can give the file to me and I can upload it if you want

I think it is here, but I don't think it is under the new schema
https://github.com/JamesSmartCell/Karma-Car-Owner-Token

@JamesSmartCell I made a signed version here: https://github.com/AlphaWallet/TokenScript-Repo/blob/add-karma/aw.app/2019/10/karma.tsml
But you need to publish the un-canonicalized version here as your repo doesn't have it


Delegation list link broken

The link to the Delegation list is broken
The following URL produced a 404:
https://docs.microsoft.com/en-us/power-platform/power-fx/maker/canvas-apps/delegation-overview.md#delegable-data-sources

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

ID: 9b784f58-59eb-fb06-da8f-4747f6900db3
Version Independent ID: 9b784f58-59eb-fb06-da8f-4747f6900db3
Content: Filter, Search, and LookUp functions in Power Apps (contains video) - Power Platform
Content Source: power-platform/power-fx/reference/function-filter-lookup.md
Service: power-platform
Sub-service: power-fx
GitHub Login: @gregli-msft
Microsoft Alias: gregli

@jebalert Apologize for the delayed response. I checked the topic and all the links are working now.
Thanks,
Mints


Possible performance issue

Currently, supervisor runs a single goroutine to handle various tasks including checkpoint containers and it does checkpoint synchronously. If checkpointing a single container takes minutes which it can possibly be, then containerd can do nothing during this time?

True, we can move this to happen in the background

@crosbymichael Isn't this is deigned to be sync? I mean, tasks order sequence should be guaranteed?

The supervisor has been removed in the 1.0 containerd release.  This should not be an issue anymore.
Thanks!


[UX] Associate anomaly detector with a visualization on Dashboards

Is your feature request related to a problem?
As an unsure user who is troubleshooting a problem, I want to augment my visualization with anomaly detection, so I can quickly identify if something is problematic or not.
What solution would you like?
https://github.com/opensearch-project/anomaly-detection/issues/521
https://github.com/opensearch-project/anomaly-detection-dashboards-plugin/issues/400
Meta UX issue
Initial UX proposal on Anomaly Detection on Dashboards https://github.com/opensearch-project/OpenSearch-Dashboards/issues/1798
See also Create anomaly detector from visualizations
User story: User wants to associate existing anomaly detector to a time series visualization to view anomalies on a dashboard.

Associate existing anomaly detector with a visualization
User wants to create an association between a time series visualization and an existing anomaly detector (which could look at the same data source or another one) in order to overlay the events from the detector on the visualization's timeline.
https://user-images.githubusercontent.com/105245012/224620026-b24fcd72-b50e-4a7b-8889-1c6f8082686a.mp4
Screen 1: User selects a visualization on a dashboard and selects "Anomaly detection" from the visualization context menu OuiContextMenu:

Screen 2: User selects "Add anomaly detector" from the context menu:

Screen 3: "Add anomaly detector" option opens a flyout OuiFlyout with "Create anomaly detector" option prefilled with the information from the visualization:

Screen 4: User selects "Associate existing detector" (OuiCheckableCard, OuiRadio):

Screen 5: An alternative path to open "Associate existing detector" flyout is to select Anomaly Detection -> Associated detectors -> Associated detectors flyout -> click on "Associate detector" button.
Note on the flyouts behavior - in order to open a new flyout ("Associate detector"), the previous one should be closed ("Associated detectors")

Screen 6: User selects a detector from the list (Single item OuiComboBox):

Screen 7: After selecting a detector user sees a summary of detector's details to review the choice.

Screen 8: After selecting anomaly detector user clicks on "Associate detector" button. The flyout closes, user returns to the dashboard.

Done:
https://github.com/opensearch-project/anomaly-detection-dashboards-plugin/issues/400
https://github.com/opensearch-project/OpenSearch-Dashboards/issues/2880
https://github.com/opensearch-project/alerting-dashboards-plugin/issues/457


Happy eyeballs crashes on request, rather than NACKs, invalid configuration

Example config:
- clusterName: outbound|80||echo.default.svc.cluster.local
  endpoints:
  - lbEndpoints:
    - endpoint:
        additionalAddresses:
        - address:
            envoyInternalAddress:
              endpointId: '[fd00:10:244::10]:80'
              serverListenerName: connect_originate
        address:
          envoyInternalAddress:
            endpointId: <IP_ADDRESS>:80
            serverListenerName: connect_originate
        healthCheckConfig: {}
      healthStatus: HEALTHY
      loadBalancingWeight: 1
      metadata:
        filterMetadata:
          envoy.filters.listener.original_dst:
            local: <IP_ADDRESS>:80
          envoy.transport_socket_match:
            tunnel: http
          istio:
            workload: echo;default;echo;;Kubernetes
    loadBalancingWeight: 1
    locality: {}
  policy:
    overprovisioningFactor: 140

This line crashes on requests:
https://github.com/envoyproxy/envoy/blob/a65765e00f4dda2bd92aee7b263ffcdbee08775a/source/common/network/happy_eyeballs_connection_impl.cc#L121.
The problem here is we try to access the ip field but there is none.
If happy eyeballs does not support non-IP fields it would be preferred for it to NACK rather than crash at runtime.
Note: for our specific case here, it doesn't really make sense to use additional_addresses anyways; we are doing HTTP CONNECT tunneling and need to put the target address in the metadata field. Since metadata is not tied to an address, you can see we always set the IPv4 there which makes it kind of obsolete anyways. I was mostly trying to see what would happen if I did this.

Not sure if EDS is the best area for this bug. Howard can you suggest a better category/


Remove Discord option

I think we should remove it as is-a.dev will soon be removing it as a social option due to the username changes by Discord.
https://dis.gd/usernames

#5


Update protoc-java-resource-names-plugin

java-resources-names-plugin just got released to v0.0.13.

PTAL


Add project updates to adminbro

Some feedback from Ashley and Nikola.
Currently Verification team cant see the project updates.

This adds the project update to the project view and links it to its show page, also adds the sidebar project updates (I limit it for now only to show and list).

Sure! Yeah we don't really have an issue.
@maryjaf This is a minor issue, but whenever you can test it, it would be good. Quality of life for adminbro.
Basically the projectUpdates should be visible in the project Show view (when you click a specific project in adminbro).
It should have working links to the user and to the projectUpdate show view.
By default this added a link in the sidebar to the projectUpdate (listing all). Right now its not posible to hide. But I disabled from that view all actions, leaving only the option to filter and show page of a project update.

Because projectUpdate should not be modified by admins directly, thats why I disabled actions on them.
(there is another issue not yet implemented to review them, but that's not yet implemented) However this set the ground work so when we do that, it's much much easier.

I think I did not understand the problem correctly @CarlosQ96
But currently, the following links are displayed on the project details page in the "Project Updates" section
Adminbro User Link
https://serve.giveth.io/admin/resources/User/records/379/show
Project Update Adminbro Edit Link
https://serve.giveth.io/admin/resources/ProjectUpdate/records/2474/show(As you said, now there is only a show and there is no editing feature)

@maryjaf The problem is:
Ashley told me they'd cant read project updates in the adminbro. So I added this section so they could.
This also was half the work for the next feature that will be allow admins to review project updates.
But I had to do this part first.
So think of it as the previous steps to the next feature haha.


Feature Request: Need flexibility in the dimension.

I have seen that the program works fine when only 2D data or 2 inputs are given. But there s no option for adding more features if needed.
I have 5 features as input. But I could not understand how I can add the 3D data or more dimension data set in the Playground. Hence, I would like to know how I can add features in the Playground.
If there is a possibility please add this feature in the list. It will be awesomely helpful to the newbie like me in learning Tensorflow and other algorithms.

@JafferWilson I'm afraid I can't offer technical support here. Please post your problem on StackOverflow using the tensorflow.js tag - that's where the team is paying attention. https://stackoverflow.com/questions/tagged/tensorflow.js

@ewilderj I tried on Stackoverflow but no one is there to support, Sir. I do not understand what I am suppose to do. I have posted number of questions there. I get donwvotes but no answers. I don't know whether the team is paying any attention to my requirements and understanding. Sir, please help me. I am stuck with Tensorflow playground as it is not giving a multi dimensional input feature.


fix npm start compilation errors due to name migration

fix yarn start & yarn test in compile stage when migrate kraken name to webf.

Duplicated with https://github.com/openwebf/webf/pull/1

close due Duplicated with https://github.com/openwebf/webf/pull/1


Error getting devices/logging in. Error code 403

[04/02/2024, 07:24:38] [Wyze] Error, logging in and trying again
[04/02/2024, 07:24:38] [Wyze] Request failed: AxiosError: Request failed with status code 403

[04/02/2024, 07:26:40] [Wyze] Response PerformRequest ([object Object]): "\n<META HTTP-EQUIV="Content-Type" CONTENT="text/html; charset=iso-8859-1">\nERROR: The request could not be satisfied\n\n403 ERROR\nThe request could not be satisfied.\n
\nRequest blocked.\nWe can't connect to the server for this app or website at this time. There might be too much traffic or a configuration error. Try again later, or contact the app or website owner.\n
\nIf you provide content to customers through CloudFront, you can find steps to troubleshoot and help prevent this error by reviewing the CloudFront documentation.\n
\n
\n\nGenerated by cloudfront (CloudFront)\nRequest ID: sE0kmORdwIJQNyakRBoCI0WLlwk1t5NFwz0gib4p_-v3qqIFMTrwtw==\n\n\n\n"


Question on deep sorting/filtering

Hi,
Would it be possible to sort/filter on nested data where the nested data comes from a different database?
For example, lets say we want to sort/filter the publicFeed based on the authors. For example sort the publicFeed by author.firstname of filter by author.firstname = Elissa and so on.
Would this be possible?

Would it be possible to sort/filter on nested data where the nested data comes from a different database?

I don't think there's an efficient way. I would store the relevant data in the same db.

Ok. Thank.


[docs-infra] Allow {{"component": “}} to import from same markdown folder

Summary
Having to the component source outside of the markdown folder like in https://github.com/mui/material-ui/blob/master/docs/data/material/all-components/all-components.md is not great. You want to be able to delete a page of the docs and all its related content all at once (closer folders).
Examples
No response
Motivation
No response
Search keywords: -

Follow up on #https://github.com/mui/material-ui/pull/34243

To provide more details on the problem, the goal is to have the .js file and the markdown in the same folder but:

The use of {{"demo": resets the theme to use the default of Material UI, which isn't the theme of the docs. So ❌
The use of bg:"inline" doesn't include a padding on mobile, this needs to be added back ⚠
The use of {{"component": doesn't allow to import the components from the same folder as ``{{"demo":` would So ❌

We could solve this in different ways:

a. support import path from a higher level, by fixing 3
b. have a demo flag to keep the default theme and another one for the layout. But I get the feeling that it's what {{"component": vs. {{"demo":  is all about. So option a. might make the most sense.


return learning rate (float) value as documented

lr_find() does not return a float value as documented.
I'm not sure if its more appropriate to return an array of learning rates for the caller to choose from or to just grab this one.

Alternatively it may preferable to refactor LRFinder::find and have these values come directly from the class method.

Thanks!  I merged your commits, but then ended up refactoring as you suggested.  Also, I'm uneasy about returning the estimated lr by default in lr_find, as there are instances when the estimate is terrible and this will be confusing to beginners.  Instead, there is now an lr_estimate method that will return a tuple of two float values representing estimates from the two different methods: 1) minimum numerical gradient and 2) minimum loss divided by 10.    If desired, this can be called to automatically set learning rate in liu of visual inspection of plot from lr_plot:
learner.lr_find()
lr_estimate1, lr_estimate2 = learner.lr_estimate()

This change will be included in v0.17.0.
Thanks again.

there are instances when the estimate is terrible and this will be confusing to beginners

Great point. I just ran into a few cases where the suggested learning rates were both bad. I like the inclusion of lr_estimate() to make it more clear. Perhaps including a link to https://sgugger.github.io/how-do-you-find-a-good-learning-rate.html could help onboard beginners by providing a little more context for Fast AI's methods and motivation?
From the more advanced side maybe documenting that after calling lr_find() on a learner object the learning rates and respective losses are available for further analysis via learner_object.lr_finder.lrs and learner_object.lr_finder.losses?


PlatformException (PlatformException(channel-error, Unable to establish connection on channel., null, null)) on Android devices

Steps to reproduce
After upgrading to flutter 3.19.6, after sometime when I try to run flutter run on Android emulator API 34, I suddenly run into problems.
Expected results
Run smoothly like before, the app should open instead of hanging with async suspensions.
Actual results
Packages like path_provider and shared_preferences always report exception:
[ERROR:flutter/runtime/dart_vm_initializer.cc(41)] Unhandled Exception: PlatformException(channel-error, Unable to establish connection on channel., null, null)

Rebuilding dependencies with flutter upgrade outdated_package, flutter clean, flutter pub cache clean do not work. App development has been suspended because of this wired one.
Code sample
Code sample
Future<void> main() async {
  WidgetsFlutterBinding.ensureInitialized();

  await DeviceService().init();
  // Connect to local database.
  await ConfigService().initConfigurations();
  await DataStore().initDB();
  // Set applications paths.
  AppPaths().getPaths();
  // Media support.
  MediaKit.ensureInitialized();

  runApp(const ProviderScope(child: MyApp()));
}

  Future<void> initConfigurations() async {
    prefs = await SharedPreferences.getInstance();
  }

 Future<void> initDB() async {
    final dir = await getApplicationDocumentsDirectory();
}

Screenshots or Video

Screenshots / Video demonstration
[Upload media here]

Logs
Logs
Launching lib\main.dart on sdk gphone64 x86 64 in debug mode...
INFO: Precompiled binaries are disabled
INFO: Building irondash_engine_context_native for x86_64-linux-android
INFO: Building irondash_engine_context_native for i686-linux-android
INFO: Building irondash_engine_context_native for x86_64-linux-android
INFO: Precompiled binaries are disabled
INFO: Building super_native_extensions for x86_64-linux-android
INFO: Building super_native_extensions for i686-linux-android
INFO: Building super_native_extensions for x86_64-linux-android
√  Built build\app\outputs\flutter-apk\app-debug.apk.
Connecting to VM Service at ws://<IP_ADDRESS>:3979/aYLZeAx5_-I=/ws
I/flutter ( 4719): Running on sdk_gphone64_x86_64unknown

Flutter Doctor output
Doctor output
Doctor summary (to see all details, run flutter doctor -v):
[✓] Flutter (Channel stable, 3.19.6, on Microsoft Windows [版本 10.0.22631.3527], locale zh-CN)
[✓] Windows Version (Installed version of Windows is version 10 or higher)
[✓] Android toolchain - develop for Android devices (Android SDK version 34.0.0)
[✓] Chrome - develop for the web
[✓] Visual Studio - develop Windows apps (Visual Studio Community 2022 17.9.6)
[✓] Android Studio (version 2023.3)
[✓] IntelliJ IDEA Community Edition (version 2024.1)
[✓] VS Code (version 1.89.0)
[✓] Proxy Configuration
[✓] Connected device (3 available)
[✓] Network resources

• No issues found!

Hi @EMUNES
Your sample code above is production code and it's unclear which plugin is causing the issue. You should narrow down the issue by removing the usage of 3rd party plugins/packages one by one, unrelated/unaffected pages, and isolating the affected page into a minimal sample code that can produce the issue. And please share it here when you have such an example. Thanks!

Thanks @huycoz. I never thought a plugin can cause this big trouble to the project. Taking your suggestion, I can reproduce the exception if I had flutter_quill in my project. The exception persists from flutter_quill 9.0.0 to newest 9.3.10 version. The minimal example is as follows:
the minimal dependency in pubspec.yaml
name: fix
description: "A new Flutter project."
publish_to: "none"
version: 1.0.0+1

environment:
  sdk: ">=3.3.4 <4.0.0"

dependencies:
  flutter:
    sdk: flutter
  shared_preferences: ^2.2.3 # Persistent settings.
  flutter_quill: "9.0.0"

dev_dependencies:
  flutter_test:
    sdk: flutter

flutter:
  uses-material-design: true

the minimal production code to run
import 'dart:async';

import 'package:flutter/material.dart';
import 'package:shared_preferences/shared_preferences.dart';

Future<void> main() async {
  WidgetsFlutterBinding.ensureInitialized();

  final prefs = await SharedPreferences.getInstance();

  runApp(const MyApp());
}

class MyApp extends StatelessWidget {
  const MyApp({super.key});

  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      title: 'Flutter Demo',
      theme: ThemeData(
        colorScheme: ColorScheme.fromSeed(seedColor: Colors.deepPurple),
        useMaterial3: true,
      ),
      home: const MyHomePage(title: 'Flutter Demo Home Page'),
    );
  }
}

class MyHomePage extends StatefulWidget {
  const MyHomePage({super.key, required this.title});

  final String title;

  @override
  State<MyHomePage> createState() => _MyHomePageState();
}

class _MyHomePageState extends State<MyHomePage> {
  int _counter = 0;

  void _incrementCounter() {
    setState(() {
      _counter++;
    });
  }

  @override
  Widget build(BuildContext context) {
    return Scaffold(
      appBar: AppBar(
        backgroundColor: Theme.of(context).colorScheme.inversePrimary,
        title: Text(widget.title),
      ),
      body: Center(
        child: Column(
          mainAxisAlignment: MainAxisAlignment.center,
          children: <Widget>[
            const Text(
              'You have pushed the button this many times:',
            ),
            Text(
              '$_counter',
              style: Theme.of(context).textTheme.headlineMedium,
            ),
          ],
        ),
      ),
      floatingActionButton: FloatingActionButton(
        onPressed: _incrementCounter,
        tooltip: 'Increment',
        child: const Icon(Icons.add),
      ),
    );
  }
}

It looks like I have found the problem. Everything was fine before, maybe I upgraded certain dependencies under flutter_quill and caused this trouble. Maybe I should raise a question there.

Well, since you left shared_preferences in the sample code above, I re-checked this using its package example but couldn't reproduce the issue.
I think it may not issue of flutter_quill because I don't see it has platform implementation (it's a normal package as I can see) but it could be caused by a plugin that it's depending on, see its Platform Specific Configurations.
So, if the issue appears when you only import and use flutter_quill package, you should report the issue on its repository for better support there. In case you can narrow it down and find something new that you think this is a Flutter 1st party package bug, please write in comments.

Sure. I found that this issue only occur if shared_preferences is used with flutter_quill. Using shared_preferences with other dozens of packages in our project all works well. I will first check super_clipboard that flutter_quill depends on, which uses Platform Specific Configurations. If I found other problems, I will report.

It's a package issue related to rust language recent changes. Fixed by https://github.com/irondash/irondash/pull/51.


Image transparency?

Is there a way to change the background image transparency? I love this module idea but would like it to be a more subtle sort of background indication of weather conditions. The current view makes it very hard to make out my modules' text, even with the recommended changes to my css file.

If your targetDom is fullscreen_below; add this into your css/custom.css
.fullscreen.below {
  opacity: 0.5;
}


Slow animation on iOS

I use the following code
https://github.com/xotahal/react-native-material-ui-demo-app/blob/master/src/ActionButton/ActionButtonSpeedDial.react.js
When I click on ActionButton in iOS , I see very slow fade in animation , but in android there is no animation and items show immediately
Where is the problem , How can I have a fast fade in animation both android and iOS ?

How can I solve issue?

You may have to activate animation for that, ```if (Platform.OS === "android") {
UIManager.setLayoutAnimationEnabledExperimental &&
UIManager.setLayoutAnimationEnabledExperimental(true);
}````


Unexpected Identifier error fix when importing YAML files.

jsymal.load returns a Javascript Object. Object.prototype.toString returns [Object Object] string that causes Unexpected Identifier error when evaluated.

This should be merged. It's pure laziness that it hasn't been merged. This entire plugin is broken without this!!

@tb ?

@tb Would you mind transferring ownership or adding as contributors more interested in maintaining this library people?

@ArmorDarks I made transfert of git repo to you. What is your npm user?

@tb Finally merged, thanks! :)
https://www.npmjs.com/~armordarks

@tb Accepts repository transfer, thanks. Still waiting for NPM addition or transfer.

@ArmorDarks Npm transfered too. Happy coding :)

@tb thanks! I will code those 9 lines to the death :)


WAMP 2.0 no supported

The wamp socket 2.0 no is supported so the example no work.
Some idea to fix this problem
Thanks

Ratchet supports v1 of the WAMP protocol which is compatible with Autobahn.js version 0.8.*. If you want to use WAMPv2 check out Thruway

Hello @cboden ! Thanks for this hint. Just read about it. One question. Thruway is "just" a client and router. Does this mean, the websocket-server is set up with ratchet and only the connection to it is done via thruway? If not, which websocket-server has to be used with thruway? (Sorry I don't wanted to post this question on both repos. But while I'm using ratchet right now and also posted here in the past, I decided to ask this question here.)
Thanks

Hi @TheMiller2015 ,
A websocket server is not necessarily called a router. "Router" is a term that the WAMP2 protocol uses. The Thruway router does have a websocket server built into it (which is Ratchet) that is used to talk WAMP2.
WAMP2 systems are built with a router that allows the clients to offer services (through RPC) and the ability to publish and subscribe. The application logic resides mostly in the clients themselves - whether it is a browser that is calling RPCs or backend client that is providing an RPC.
Different WAMP2 components can be used together (Thruway, crossbar, autobahn, etc.) to build distributed systems out of application components which are loosely coupled and communicate in (soft) real-time.

hey. Thanks for your great answer. I read the docs of crossbar and your answer. Things became clearer now. And I also do have my first ws-connection (within the new setup).
Thanks.


Update CHANGELOG date in azure-json

API change check
API changes are not detected in this pull request.


vSphere node template ui shows variable tokens and not field labels. Not functional.

Rancher versions:
rancher/server:v2.0.0-alpha21
rancher/agent:v2.0.3
Infrastructure Stack versions:
healthcheck:
ipsec:
network-services:
scheduler:
kubernetes (if applicable):
Docker version: (docker version,docker info preferred)
Client:
Version:       18.02.0-ce
API version:   1.36
Go version:    go1.9.4
Git commit:    fc4de447b5
Built: Tue Feb 13 15:28:01 2018
OS/Arch:       linux/amd64
Experimental:  false
Orchestrator:  swarm
Server:
Engine:
Version:      18.02.0-ce
API version:  1.36 (minimum version 1.12)
Go version:   go1.9.4
Git commit:   fc4de447b5
Built:        Tue Feb 13 15:28:34 2018
OS/Arch:      linux/amd64
Experimental: false
Operating system and kernel: (cat /etc/os-release, uname -r preferred)
Linux haruko 4.15.6-1-ARCH #1 SMP PREEMPT Sun Feb 25 12:53:23 UTC 2018 x86_64 GNU/Linux
Type/provider of hosts: (VirtualBox/Bare-metal/AWS/GCE/DO)
Base-metal & vSphere
Setup details: (single node rancher vs. HA rancher, internal DB vs. external DB)
single node tancher
Environment Template: (Cattle/Kubernetes/Swarm/Mesos)
Kubernetes
Steps to Reproduce:
Run the current preview and attempt to setup a node template for vSphere. It looks like a ui template is not being processed and tokens are being left on the page.
Results:
All fields show tokens like **%nodeDriver.vmwarevspheree.vcenter.label%**
Attempting to fill out the fields based on the token names and submitting results in Validation failed in API: vmwarevsphereConfig which may be due to incorrect input on my part but I have no idea what field is wrong.

Version - 2.0 master 5/11
Verified fixed


picEdit submits only the original image

Hi,
i found out about this tool today and added in one of my projects. My problem is, that although i can modify the picture, he always submits the original image. He is not taking any of the changes. Here ist my code:
	 $("#image").picEdit({
	         redirectUrl: baseURL + "/userinfo",
		 defaultImage: false
	 });

<form method="POST" enctype="multipart/form-data" action="user/addimage">
<input id="image" accept="image/*" type="file" name="userimage">
<button type="submit" class="btn btn-primary btn-lg"><spring:message code="common.submit.save"/></button>
</form>

What am i doing wrong? Is it not working with multipart/form-data or something like that?

There is a second image which has your changes, it is submitted at the same time as the first. I ended up changing the script to empty the first image myself.

@simonpapworth6742  You've mentioned at edit to blank the first image in order to get the editted image, can you post a sample or open a fork or something.

@KokoRyuKai, if you haven't figured it out yet, what you can do is set the file name in the html form to an array e.g. name="image[]". Then you can remove the index 0 attributes or use the index 1 attributes. It saves it as image( 'name' => array('original_filename','altered_filename'), 'tmp_name' => array('original_tmp_name','altered_tmp_name') ...

@desnudopenguino , I get the concept, but don't understand your code.  Can you please elaborate?
Thanks!

This is all going to be in PHP on the server side, sorry if that won't work for you, it's what I've been using most of the time recently, so I can give a complete answer in it only.
Step one, making the server able to accept the altered image:
use <input type="file" name="image[]" /> instead of <input type="file" name="image" /> in your html. Otherwise the original image is all the server sees. It sets it up to make $_POST['image'] an array.
Step two, now that you can parse the data, you need to make sure you are pulling the right image into your server code.
The data sent to the server will look like this:
[_FILES] => Array
        (
            [image] => Array
                (
                    [name] => Array
                        (
                            [0] => filename.jpg
                            [1] => filename.jpg
                    [type] => Array
                      (
                           [0] => ...
                           [1] => ...
                    [tmp_name] => Array
                      (
                          [0] => /tmp/aoeu897u
                          [1] => /tmp/uau4pyao4

                    [error] => Array
                        (
                             [0] => 0
                             [1] => 0
                            ...
                    [size] => Array ...
                )
        )

So you have to get the [1] thing in each of the sub-items in the image array. If you already have some elaborate code written and don't care about the original image you can write a quick foreach that jumps through the list of fields like this...
foreach($_FILES['image'] as $attribute_name => $attribute_value) {
    $_FILES['image'][$attribute_name] = $attribute_value[1];
}

That should parse everything back out to the way it would be without dealing with multiple images. You can do that or access the [1]st things it the arrays. Whichever is easier for you. I hope this helps.

:+1: @desnudopenguino

Hi everyone,
You can also send always the edited image through AJAX by using the Blob object. For example:
var thereIsBlob = false;
var uploaded_photo;
$("#myInputFileElement").picEdit({
    imageUpdated: function(img) {
        if ((typeof img.file === "undefined") && (typeof img.src !== "undefined")) {
            uploaded_photo = dataURItoBlob(img.src);
            thereIsBlob = true;
        }
        else {
            uploaded_photo = img.file;
            thereIsBlob = false;
        }
    }
});
// (...)
var data = new FormData();
if ((typeof uploaded_photo !== "undefined")) {
    if (thereIsBlob) data.append("photo", uploaded_photo, "photo.png");
    else data.append("photo", uploaded_photo);
}
// AJAX call code with the FormData and so on

The dataURItoBlob function converts the Data URI obtained from img.src to a Blob object that can be sent like a file to a PHP script (or another kind of file) through AJAX. You can find how to implement that JS function on the Internet.
I hope this helps!

I want to bring some kind of clarification to this thread.
Due to limitations in HTML5 Form API its impossible to actually update the file-upload in the original form. Instead, what picEdit does is duplicates your original html form in javascript, substitutes the file-upload on the fly and sends the whole form thru Ajax. Your original form is not being sent.
However, if your original form has a custom "submit" event handler attached, it can cause your original form submitting to the server along with the Ajax POST request from picEdit. You'll see it as the "double submission", when the edited image is being sent along with the original one.
Depending on your code you might want to tweak your custom js code to make it work with picEdit.

Meanwhile, I'm working on the alternative way to submit the image data to simplify the upload process and overcome the above issues.


[OTel plugin] Add new schema version

Using the stable 1.23.1 semantic conventions version. Attribute names are also remapped.
Ref: https://github.com/Azure/azure-sdk-for-python/issues/32803

API change check
API changes are not detected in this pull request.


Remove DocumentHelpers

This is another of the remaining classes/modules including the word "document" in its name.
There are already methods in ManualHelpers which relate to both manuals and sections, so it doesn't seem worth keeping DocumentHelpers around given how few useful methods it contains.
My preference would be to make this change and then to extract the relevant methods out of ManualHelpers into a new SectionHelpers module if appropriate.

Force-pushing to encourage Jenkins to pick up this branch which was pushed earlier this morning before the problems with Jenkins.

Rebasing against master and force-pushing in preparation for merging.

@chrisroos: Thanks.
Rebasing against master and force-pushing in preparation for merging.


addenda99: test reading without a file header/control

Codecov Report

Merging #719 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #719   +/-   ##
=======================================
  Coverage   92.56%   92.56%
=======================================
  Files          65       65
  Lines        7448     7448
=======================================
  Hits         6894     6894
  Misses        395      395
  Partials      159      159


Allow overwriting internal types to improve result coercion

The 0.11.x update seem to have improved variable coercion and some of the scalar types as well. The coercion adheres the definition in http://facebook.github.io/graphql/October2016/#sec-Scalars.
Now, we've tried to extend BooleanType to make it more strict, so that it throws if the passed value isn't a boolean in order to avoid false positives within our app:
class ExtendedBooleanType extends BooleanType {
    /**
     * @param mixed $value
     * @return bool
     */
    public function serialize($value) {
        if (!is_bool($value)) {
            throw new InvariantViolation("Boolean must be represented by strict value. " . Utils::printSafe($value) . " given.");
        }
        return $value;
    }
}

The need for this relates to running in weak mode, where even methods with return types :bool end up in coercion. Therefore defining an error inside serialize is important for us.
However, the Instrospection class (among others) uses static calls, including calls to Type::boolean(), so building our schema throws once again:
Schema must contain unique named types but contains multiple types named

What are your thoughts on strictly checking for boolean to prevent errors?
Is there a way we could point to our boolean definition from the different calling contexts?

I don't think that replacing standard types with custom types is a good idea because their parsing behavior is defined in spec and clients rely on it (except maybe ID type).
On the other hand, I do agree that serializing process is a bit different and could be customizable to some extent. Two options I see:

Introduce strict/weak mode for serialization
Allow defining optional custom serialization function

I'd like to keep this issue open for a while for discussion (maybe others will post their thoughts as well). Maybe I'll reconsider this and just allow defining own standard types.
In the meantime, the only workaround for you is to use Reflection to override GraphQL\Definition\Type::$internalTypes.

@vladar thanks for the very quick answer. I agree let's keep this opened for a bit. Personally I do like option 1. because it plays nice when configuring PHP to run in either weak or strong typing mode.

Reading the spec for Boolean closer and given than PHP does have a bool type (unlike other langs), we're not limited to returning a coerced type. Therefore, even though is not coercing, I believe the next would be safe:
class ExtendedBooleanType extends BooleanType {
    /**
     * @param mixed $value
     * @return bool
     */
    public function serialize($value) {
        $value ? true : false;
    }
}

"The Boolean scalar type represents true or false. Response formats should use a built‐in boolean type if supported; otherwise, they should use their representation of the integers 1 and 0."
Again, the point I'm trying to make is that PHP is already loose enough coercing types, and strict mode makes no difference to this behavior, leaving us with either tackling it here or using strict type with method return types: serialize($value): boolean { ... }  which we do not use. As I understand, GraphQL philosophy is to avoid returning typing errors and instead coerce.

Sorry, missed your last comment somehow. But I am not sure I am following it. $value ? true : false is a current behavior. But then what's your problem with it (again, I feel that I am missing something)?

Sure...
return !!$value; // returns coerced value, e.g.: $value = 'foo'; return !!$value; will equal (int) 1

return $value ? true : false; // returns boolean value

It's the same:
$foo = 'foo';
$bar = !!$foo;
var_dump($bar);

// outputs: bool(true)

Oh my... echo VS var_dump, echo is obviously stringifying the real output. My bad.


PK means Platform Key

Example 1 on this page describes PK as a private key and that PK.cer contains a private key.  Both descriptions are incorrect and has misled at least one reader.
In this context, PK stands for 'Platform Key', and while the key being described has both a public and private key associated with it, only the public part of the key is held in the variable and provided in the PK.cer file (as an X.509 certificate).
The private key exists elsewhere (it is used to sign things like the certificate) but normally would never appear on the platform being set up.

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

ID: 3f849d0b-e013-8248-85c2-a51ae26a1933
Version Independent ID: fbe31ff2-671c-d506-b107-dbe5b8180f46
Content: Format-SecureBootUEFI (SecureBoot)
Content Source: docset/winserver2022-ps/secureboot/Format-SecureBootUEFI.md
Product: w10
Technology: windows
GitHub Login: @JasonGerend
Microsoft Alias: jgerend

To make it easier for you to submit feedback on articles on learn.microsoft.com, we're transitioning our feedback system from GitHub Issues to a new experience.
As part of the transition, this GitHub Issue will be moved to a private repository. We're moving Issues to another repository so we can continue working on Issues that were open at the time of the transition. When this Issue is moved, you'll no longer be able to access it.
If you want to provide additional information before this Issue is moved, please update this Issue before December 15th, 2023.
With the new experience, you no longer need to sign in to GitHub to enter and submit your feedback. Instead, you can choose directly on each article's page whether the article was helpful. Then you can then choose one or more reasons for your feedback and optionally provide additional context before you select Submit.
Here's what the new experience looks like.
Note: The new experience is being rolled out across learn.microsoft.com in phases. If you don't see the new experience on an article, please check back later.
First, select whether the article was helpful:

Then, choose at least one reason for your feedback and optionally provide additional details about your feedback:

Article was helpful
Article was unhelpful

Finally, select Submit and you're done!


Modified date label to be nostalgic-friendly (i.e. "Friday, 8 days ag…

…o", "December, 4 years ago"), and moved stylign to css

I can't maintain this module any more.
Have you interest to take new ownership of this module?
https://forum.magicmirror.builders/topic/12779/wanted-new-ownership-of-modules?page=1

Wasn't MMM-GooglePhotos was just totally re-written?
On Sun, May 10, 2020 at 5:45 PM Seongnoh Sean Yi<EMAIL_ADDRESS>wrote:

I can't maintain this module any more.
Have you interest to take new ownership of this module?
https://forum.magicmirror.builders/topic/12779/wanted-new-ownership-of-modules?page=1
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/eouia/MMM-GooglePhotos/pull/49#issuecomment-626339194,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AEIVW2B37ZOBHLYLPZZGFG3RQ24YHANCNFSM4KI23YKA
.

You are right. But if you have an interest, take this over and change or modify as you wish.


@sanity/vision/lib/index.esm.js not found

Using sanity-plugin-asset-source-unsplash with sanity@dev-preview and<EMAIL_ADDRESS>results in the following issue:
Error: ENOENT: no such file or directory, open<EMAIL_ADDRESS>    at Object.openSync (node:fs:599:3)
    at Object.readFileSync (node:fs:467:35)
    at extractExportsData (file:///Users/shaun/Desktop/work/bitfo/node_modules/vite/dist/node/chunks/dep-c842e491.js:42970:31)
    at async runOptimizeDeps (file:///Users/shaun/Desktop/work/bitfo/node_modules/vite/dist/node/chunks/dep-c842e491.js:42647:29)
    at async optimizeNewDeps (file:///Users/shaun/Desktop/work/bitfo/node_modules/vite/dist/node/chunks/dep-c842e491.js:42050:16)
    at async runOptimizer (file:///Users/shaun/Desktop/work/bitfo/node_modules/vite/dist/node/chunks/dep-c842e491.js:42079:55)

It looks like a depedency within the plugin is expecting @sanity/vision to export from a index.esm.js file but inspecting the package, it only exports from an index.cjs file.
I'm on a Mac M1 // OSX Ventura if that's relevant to the issue.

Running npm i sanity@latest @sanity/vision@latest sanity-plugin-asset-source-unsplash@latest should resolve this issue 🙂