Remove push Travis Trigger

Thanks to comment: https://github.com/ProcedureKit/ProcedureKit/pull/757#issuecomment-314947291

@jshier err, not sure about this. Seems like it doesn't build anything adding in your suggestion to travis.yml. We still want to run the travis job on PRs. Doesn't seem to be a way to get it to build only PRs.
I don't know, I'm not all that familiar with TravisCI.... any thoughts?

Odd. We use this setting on Alamofire just fine. It's also the accepted answer. Perhaps the stages feature is incompatible?

I can't see the repo settings, but perhaps make sure the Travis integration is through the actual Travis integration and not a manual webhook?

Yeah, I think it might be the stages.

[x] Saw this "accepted answer" - copied settings exactly.
[x] Setup via GitHub "Integration" not a manual web hook.

Hmm

Stages and this setting seem like they should be orthogonal but perhaps not. I read through most of the stages issue for Travis and no one mentioned an issue like this, but I did skim towards the end.


Network protocol error when accessing Paper 1.20 server (using ViaVersion)

Modloader
NeoForge
Minecraft Version
1.21.3
Modloader Version
Neoforge 21.3.7-beta
No Chat Reports Version
1.21.3-3.0.0
Modpack Info
No response
The latest.log File
https://mclo.gs/sAOg3HH
Bug Description
When connecting to a Paper 1.20.1 server, using ViaVersion 5.1.1, I get an error message stating network protocol error, and the above disclosed disconnect log. When disabling NoChatReports it connects fine, leading me to believe the mod is the issue.
Steps to Reproduce
No response
Other Information
I wasn't sure if the issue would be a duplicate of #492 but given some significant differences in versions I thought it'd be better to report separately rather than cause noise in the other issue

Most likely a bug similar to #492.


Add on_opcode event

This is my first ever PR on github so it might be a bit weird but this adds an on_opcode event in response to #26

I think I'll close this PR in favor of something more official from the maintainers since i'm relatively new to crystal.


test_ffmpeg seems to stuck on compile (?)

The test_ffmpeg seems to stuck on compilation. CPU shows 100% and processes (8) are RUNNING.
License: GPL version 3 or later
Creating config.mak, config.h, and doc/config.texi...
config.h is unchanged
config.asm is unchanged
libavutil/avconfig.h is unchanged
CC	libavdevice/alsa.o
CC	libavdevice/alldevices.o
CC	libavdevice/alsa_dec.o
CC	libavdevice/alsa_enc.o
CC	libavdevice/avdevice.o
CC	libavdevice/dv1394.o
CC	libavdevice/fbdev_common.o
CC	libavdevice/fbdev_dec.o

See top screenshot:

Let me see if I can reproduce.

Worked for me with a fresh download and build. What were the circumstances in which this occurred? Did you install the necessary ffmpeg deps and makedeps?

For dependencies I just compiled https://aur.archlinux.org/packages/ffmpeg-full/ so I assume I have them. How can I clear the build and try again?
My process was:

clone repo
setup
test xz
test unix
test ffmpeg

See the README, you need the deps and makedeps for ffmpeg.

I have the deps and makedeps for ffmpeg.

this happens if you have other gcc wrappers installed like colorgcc or ccache. please remove those from your path when executing those tests

@anthraxx indeed I have ccache - I will try without. Thanks

I'm having this issue as well. Could you add the conflicting software to the readme?

@klusark We can add a hint. there may be an arbitrary amount of things that wrap gcc

@anthraxx Looks good! Thanks for fixing it.


Limit state count in StateContextCache

Is your feature request related to a problem? Please describe.
Memory exhaustion occurs in periods w/o finality, mostly from many states / epoch ctxs stored in the StateContextCache.
Describe the solution you'd like
We should prune the StateContextCache when it gets past a certain size. (pruning the oldest states first, perhaps with a few exceptions)
This likely must be done in tandem with two additional features:

state recreation - when pruned states are requested, they must be recreated from most recent available state. We can walk the fork choice block dag to retrieve ancestor state roots / block roots.
checkpointed states - replaying many blocks across many epochs to recreate a state may be "too slow". We should keep 1 state per N epochs per fork either in memory or on disk.

closed with #1494


Introduce credo

I believe it would be a good idea to start using credo to lint the Elixir code. It should also be added to the CI.
What do you think?

Sounds good!

Sounds good!

Cool, I'm looking into this.


Incremented version to 1.3.2.

Signed-off-by: Sayali Rohidas Gaikawad<EMAIL_ADDRESS>Description
Coming from https://github.com/opensearch-project/opensearch-build/issues/1805 post release activity.
Check List

[x] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Codecov Report

:exclamation: No coverage uploaded for pull request base (1.3@e124827). Click here to learn what that means.
The diff coverage is n/a.

:exclamation: Current head 0958ab3 differs from pull request most recent head 9117232. Consider uploading reports for the commit 9117232 to get more accurate results

@@          Coverage Diff           @@
##             1.3     #124   +/-   ##
======================================
  Coverage       ?   83.47%
  Complexity     ?      553
======================================
  Files          ?       61
  Lines          ?     2046
  Branches       ?      145
======================================
  Hits           ?     1708
  Misses         ?      233
  Partials       ?      105

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 e124827...9117232. Read the comment docs.


externalTrafficPolicy=Local seems to be always needed

For the Kubernetes-Nginx controller in AKS:
There's a difference in the config for the Helm chart config vs the method that the Kubernetes Nginx controller project recommends for Azure, namely the using this template. In short the Microsoft doc recommends Helm, while the project itself recommends another approach.  Odd.  (I don't know if the Kubernetes nginx project would agree that those two should yield the same result?)
Anyway, it is actually the latter approach which works, not the one described in the Microsoft docs.  The reason is that
externalTrafficPolicy=Local
seems to be absolutely required. The template-based method for Azure does this by default, but Helm chart method doesn't by default.
There seems to be others who have had the same problem as me.
In short, for my use-case using AKS Kubernetes v 1.24.6, if was definitely a requirement to set externalTrafficPolicy=Local. If not, the external IP will not reply (it will timeout).

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

ID: fd0e7d5a-37e9-07dc-f395-9cb5bb580e45
Version Independent ID: 7e4faf70-9724-7e7b-832b-1cd99a974920
Content: Create an ingress controller in Azure Kubernetes Service (AKS) - Azure Kubernetes Service
Content Source: articles/aks/ingress-basic.md
Service: container-service
GitHub Login: @rayoef
Microsoft Alias: rayoflores

@lbruun
Thanks for your feedback! We will investigate and update as appropriate.

Hi @lbruun
I'd recommend working closer with our support team via an [Azure support request] (https://docs.microsoft.com/en-us/azure/azure-portal/supportability/how-to-create-azure-support-request). Or you can leverage our Q&A forum by posting your issue there so our community, and MVPs can further assist you in troubleshooting this issue or finding potential workarounds.
[Teams Q&A forum] (https://docs.microsoft.com/en-us/answers/topics/46488/office-teams-windows-itpro.html) for technical questions about the configuration and administration of Microsoft Teams on Windows.

Ran into the same issue as @Ibruun. Setting externalTrafficPolicy=Local seems to be missing in the document. Without this Traffic to External IP times out.

Ran into the same issue few minutes ago on AKS. I installed newest version of nginx ingress via helm install. Have exactly the same issue.  To resolve this, the externalTrafficPolicy=cluster has to be change to externalTrafficPolicy=local.
For production it might be disaster.

I've got a similar issue, too; I need to set it to 'Local' to preserve source IPs on x-forwarded-for header. Neither 'kubectl patch' or 'kubectl edit' lets me modify it, either (the platform reverts the changes I've just made to its default value even if I see the changes reflected for a time)

@AjayBathini-MSFT Why is this marked as closed when it is still a relevant issue? This impacts both the public and private LB

@atmask
you can leverage our Q&A forum by posting your issue there so our community, and MVPs can further assist you in troubleshooting this issue or finding potential workarounds.
[Teams Q&A forum] (https://docs.microsoft.com/en-us/answers/topics/46488/office-teams-windows-itpro.html)


Missing Step (Settings)

Between steps 1 and 2 you need to add a step to select Settings.  The Device Onboarding is under settings while this page implies it is at the top level which it is not.

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

ID: 1e0ac0e1-6f08-2f8b-10c7-a967e53886f8
Version Independent ID: f510237f-0ec9-d266-3ec6-0ad6b49e2522
Content: Get started with Microsoft 365 Endpoint data loss prevention - Microsoft 365 Compliance
Content Source: microsoft-365/compliance/endpoint-dlp-getting-started.md
Service: o365-seccomp
GitHub Login: @chrfox
Microsoft Alias: chrfox

@msbemba Thank you very much for the contribution and updating the documentation with PR. @PortableTech Hope this update is helpful for you. Thanks for taking out some time to open the issue. Appreciate and encourage you to do the same in future also.


Doesn't build on Linux

Installed the latest yarn, stuff seems broken.
[13:34:44] मanishearth@aperture ~/mozilla/playpen/ui/frontend ^_^
$ yarn
yarn install v1.3.2
[1/4] Resolving packages...
[2/4] Fetching packages...
[########################################################################################-----------------------------------------------------] 477/[##########################################################################################---------------------------------------------------] 489/info<EMAIL_ADDRESS>The platform "linux" is incompatible with this module.
info<EMAIL_ADDRESS>is an optional dependency and failed compatibility check. Excluding it from installation.
[3/4] Linking dependencies...
[4/4] Building fresh packages...
Done in 19.06s.
[13:35:06] मanishearth@aperture ~/mozilla/playpen/ui/frontend ^_^
$ yarn run watch
yarn run v1.3.2

/home/manishearth/mozilla/playpen/ui/frontend/webpack.config.js:115
  );
  ^

SyntaxError: Unexpected token )
    at createScript (vm.js:56:10)
    at Object.runInThisContext (vm.js:97:10)
    at Module._compile (module.js:542:28)
    at Object.Module._extensions..js (module.js:579:10)
    at Module.load (module.js:487:32)
    at tryModuleLoad (module.js:446:12)
    at Function.Module._load (module.js:438:3)
    at Module.require (module.js:497:17)
    at require (internal/module.js:20:19)
    at requireConfig (/home/manishearth/mozilla/playpen/ui/frontend/node_modules/webpack/bin/convert-argv.js:97:18)
error Command failed with exit code 1.
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.

"works for me"
It also successfully builds in Travis, so I'd be highly inclined to think this is a local issue.
Can you make sure the files appear to be correct? I'm at git commit 2def0d8aef0cee3a0ce041de656ddb8ccc60e441:
$ md5sum package.json webpack.config.js
09cd5dcfd22eb4c9ff22c8158974c77c  package.json
84c94b4e9fe4be522e7032d35d9f566f  webpack.config.js

Does the file actually look like malformed JS to you?

The checksums work.
I suspect it's a missing package. I'm using yarn 1.3.2.

Aha, installed the latest node and now it works.
(Please list node as a dependency ; as someone unaware of the Node ecosystem I assumed yarn would install the appropriate compiler as well)

You had Node installed, otherwise you couldn't have gotten a stack trace of any kind. What version of Node did you have?

probably the default distro one


Fixes scaling for the kube-controller-manager during shoot restoration

How to categorize this PR?

/area control-plane-migration
/kind bug
What this PR does / why we need it:
During the restore phase of control plane migration the kube-controller-manager will now be properly created with 1 replica if the shoot is not hibernated.
Which issue(s) this PR fixes:
Fixes #4773
Special notes for your reviewer:
Release note:

During the restore phase of control plane migration, the kube-controller-manager deployment will be properly created with 1 replica if the shoot is not hibernated.

/invite @stoyanr
although this isn't as critical it would be good to get it in the new release
/milestone v1.33

@stoyanr noticed that with the new logic when the shoot is being restored and hibernated at the same time the KCM would be created with 0 replicas.
The restoration finishes successfully, but there is actually a different problem (not related to the one fixed in this PR) with the mentioned scenario (hibernation + restoration at the same time).
Since in the new seed the kube-apiserver is not scaled up (it's deployment doesn't exist from a previous reconciliation and the etcd is also not scaled) the shootClient will not get initialised here: https://github.com/gardener/gardener/blob/b12f41152e867e6c0678311c9a82400ea7f0e5e6/pkg/operation/operation.go#L358-L367
which means that later all of the checks will be skipped in this code:
https://github.com/gardener/gardener/blob/b12f41152e867e6c0678311c9a82400ea7f0e5e6/pkg/operation/botanist/controlplane.go#L137-L172
This basically means that there will be leftover machines with actually running pods for a shoot that is flagged as hibernated.
I could tackle this problem in a different PR later on, as it would also require additional changes to his function: https://github.com/gardener/gardener/blob/b12f41152e867e6c0678311c9a82400ea7f0e5e6/pkg/operation/botanist/kubeapiserver.go#L350
which is already modified by #4779 and most likely other places in the reconciliation flow that would require special handling for hibernation + restoration.
As an alternative we could disable the hibernation of a shoot cluster while it is being migrated, similar to what we did for deletion.

@voelzmo yep I'll do the cherry pick in a few mins


Possible Uri Exception if MahApps is running in a test environment (or non wpf app)

after some discussion in this so
we should just tell the user that we need a wpf environment and throw an exception if MahApps is located in a test (without a wpf app) or console app etc.
so here is a simple line that causes the exception
new Uri("pack://application:,,,/...")
in MahApps
_accents = new List<Accent>(colors.Length);

foreach (var color in colors)
{
    _accents.Add(new Accent(color, new Uri(string.Format("pack://application:,,,/MahApps.Metro;component/Styles/Accents/{0}.xaml", color))));
}

return _accents;

Skimmed the SO thread so I might be missing some details, but I'm concerned about making the library "too clever" to detect this - which feels like an outlier issue. Did you find an appropriate hook to verify the context? Themes is a likely place where users might try and run things in a funny context to verify behaviour, not sure if there's other areas you've seen...

@shiftkey the idea was only to fire an exception with a better text info what happens...

:thumbsup:


Enable RSS autodiscovery

Think we need something like <link rel="alternate" type="application/rss+xml" title="Thunderbird Blog" href="<?php bloginfo(‘rss_url’); ?>" /> in the header.php

Alternatively there's also https://codex.wordpress.org/Automatic_Feed_Links if you use wp_head() I think. To get the actual URL there's also the specialized https://developer.wordpress.org/reference/functions/get_feed_link/

Yes, I would go with @freaktechnik's approach. Since we already use wp_head() adding
add_theme_support( 'automatic-feed-links' );

would automatically add the posts feed and the comments feed to the header:
<link rel="alternate" type="application/rss+xml" title="Thunderbird Blog » Feed" href="https://blog.thunderbird.net/feed/" />
<link rel="alternate" type="application/rss+xml" title="Thunderbird Blog » Comments Feed" href="https://blog.thunderbird.net/comments/feed/" />


feat: Add cross-platform build docker images and use quay.io/opendevin as image registry and org.

feat: Add workflow for building and pushing multi-arch images to ghcr.io.
I have successfully tested the workflow in my repo, see the results here:
https://github.com/iFurySt/OpenDevin/actions/runs/8415626483/job/23041038547
screenshot in package:

I'm not in the OpenDevin org, so you need to confirm the workflow permission was already set to RW.
Your Org -> OpenDevin -> Actions -> Workflow permissions:

@huybery can you help take a look? I did not see this option on my Action panel

@iFurySt @xingyaoww I just opened it. How about now?

@huybery just review pr and merge it, the workflow will auto-trigger to build and push to OpenDevin's package, and maybe you need to public the packages after the images are pushed.

@xingyaoww @huybery I've just noticed the workflow triggered when pr is opened, I removed the trigger of pull_request in case the wrong pr will trigger the workflow to push the wrong images to ghcr


Cqrs

Refactor of Commands and Query interfaces and implementations
Moved to CQRS package.
Extend 'Manager' Interfaces to factory objects (Using interface methods hiding implementation)
Testing to 100% classes and > 80% lines including assumed common negative cases.
Testing on MewbaseTestBase removing all references to Server

 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.


Asset loading progress

I noticed this was removed when the asset loader was merged, what is the alternative when showing or notifying progress?

Unfortunately Promises no longer have a notify function (the angular promise polyfill I was using did). So there's not really a way to notify though the functions when an asset has loaded other than calling some global function.
Now that I think about it, I could add a notify event using the recently implemented events api.

I see and that would be great if loading progress can be tied to events. Looking forward to it.
btw, something that I found recently without the notify function that might be of use as well..
https://codepen.io/potch/details/yereOM


Dartium fails on non-typed List in gl.uniform*v

According to WebGL IDL, uniform*fv(location, v) should accept both Float32Array and sequence<float> for v.
This call:
gl.uniform2fv(location, <num>[1.0, 0.0]);

fails in Dartium (1.18.1) with
Uncaught Unhandled exception:
TypeError: Failed to execute 'uniform2fv' on 'WebGLRenderingContext': No function was found that matched the signature provided.
#0      Blink_JsNative_DomException.callMethod (dart:_blink:23416)
#1      BlinkWebGLRenderingContext.uniform2fv_Callback_2_ (dart:_blink:20794)
#2      RenderingContext.uniform2fv (dart:web_gl:3168)

but works in Chrome after dart2js compiling.
Call with typed array works in both cases:
gl.uniform2fv(location, new Float32List.fromList(<num>[1.0, 0.0]));

That error is coming from JavaScript. Dartium passes both cases through to exactly the same place. See https://api.dartlang.org/stable/1.18.1/dart-web_gl/RenderingContext/uniform2fv.html
The reason is probably that in the version Dartium is based on (46) the IDL says
    void uniform2fv(WebGLUniformLocation? location, Float32Array v);
    void uniform2fv(WebGLUniformLocation? location, sequence<GLfloat> v);

Also, that doesn't look like it's generally valid. See
https://developer.mozilla.org/en-US/docs/Web/API/WebGLRenderingContext/uniform
and in particular the line listing the types
A Float32Array for floating point vector methods (methods with "fv").

Just tested: Chrome & Firefox actually allow non-typed arrays. IDL too.
Moreover, dart2js-compiled call with non-typed array works in Dartium.

Hmm. That's peculiar. It suggests that the problem might be with passing the List through JS interop.

It suggests that the problem might be with passing the List through JS interop.

Very likely: same bug with vertexAttrib*fv functions.

Any ongoing fixes for this issue?

Dartium is no longer being maintained, and is being phased out in 2.x.
https://webdev.dartlang.org/tools/dartdevc

This isn't an issue with Dartium, happens when code is dart2js'd too.


Fix couple of typos in README

Great work!
I edited some parts of the documentation to make it better fix some typos.

Pata Rack ub


Reduce memory of Zinc by de-duplicating Strings, etc

Using Java Mission control, take a heap dump of a running SBT build. The resulting UI includes the JOverflow analysis of opportunities to reduce footprint.
ClassLike.savedAnnotations contains duplicate strings for scala.native scala.deprecated etc.

UsedName / NameHash

My work on RelationsBuilder in https://github.com/sbt/zinc/pull/764 is a step in the right direction.
Related fix for SBT itself: https://github.com/sbt/sbt/pull/5535

Closed by #1326


Remove tick mark & add padding to left of message

Looks good. Assume this won't affect the e2e tests

Thanks. No, it shouldn't as we're not checking the message


feat: Add it naming rector

Adds a rector for changing tests to using the it naming schema if the name of the test suggest that was the purpose of it.
Ported from Drift

Looks good, I've added a little feedback, but we can merge this as it is to iterate and have it in the core :+1:
Thanks :clap:

@TomasVotruba awesome, yeah I also think we will quickly come in a situation where we need to extract things into util classes for getting test names etc.


Removing Molecule Parameters

I have an issue with removing molecule parameters using the 'tools>Delete Molecule Parameters' function.

When putting a specified parameter in and pressing OK, the parameter is not removed.
When removing all parameters, all parameters are removed.

Hi @NadiaHuisjes,
I can reproduce the issue. When I attempt to delete one parameter using the top part it doesn't nothing. But all parameters are removed. I will investigate why this is happening.

This fixed the issue for me. Can you confirm?

Thank you! It works indeed.


Binary code coverage for Linux

Per https://github.com/MicrosoftDocs/azure-devops-docs/pull/11835, updating the docs in the Yaml pipelines

#sign-off


GraphQL Enhancements

Fetchye currently creates a new cache for every GraphQL query fired (assuming the queries have minor differences). Consider the following scenarios:

If a query requests a subset of fields already requested by a previous query, an entirely new api call is made and stored in cache.
If a query only requests one or two new fields and many already requested fields, an new api call is again made with the full set of query fields.

This change aims to prevent duplication in the way Fetchye requests data from GraphQL APIs. The goals of this change are as follows:

A new call is made to a GraphQL API only if the query requests new fields. Otherwise, it pulls existing data from cache.
If a query only requests a few new fields, all already existing fields are stripped from the request.
If the query includes different arguments, naming, etc that could impact the response of the GraphQL API, a new cache is created for that request.
This change should have an opt-in / opt-out strategy so that it does not impact existing Fetchye users, and is not mandatory to use if the assumptions (listed at the bottom) do not apply to your use case.

The following goals are accomplished by introducing these changes:

A "query" field has been added to the store (alongside isLoading, data, and error).
A new field can be passed to options named "isGraphQL". Adding this field as true will opt-in to these changes.
Whenever a GraphQL call is made to Fetchye, the query in the body is trimmed down to only the special GraphQL fields before the hash key is created. This way, the hash key is always the same for all GraphQL apis using the same special fields.
Whenever a GraphQL call is sent to Fetchye, the query is pulled from the body and dispatched to the store. Subsequent queries are merged and compared against the query already present. If any new fields have been added, an API call is made. Otherwise, Fetchye responds with data from the cache.
Before the API call is made, any existing query fields are stripped from the body (with the exception of required special fields).
GraphQL responses are merged with the data in the cache rather than replacing it. This allows already existing fields to be served.

The following assumptions are made to ensure this process can function:

A "query" field is present in the request body.
The "query" field is formatted using json-to-graphql-query. (Necessary for object comparison.)
The response from the API is stable, and can be safely merged.

Potential issues identified with this approach:

Because data is merged, Fetchye will respond with the full list of fields requested by all GraphQL calls. If developers blindly use fields that were not part of a specific request, they may introduce dependency issues.
Comparisons and merging use deep compare and deep merge functions which can be costly. These functions are currently only running when data changes (not on every render), so the impact may not be significant.
An error given by any of the GraphQL requests would reflect to all of them. This PR is open to suggestions on how to handle those scenarios.

Thank you for your time in reviewing this! If you have any concerns or questions please leave a comment. I also have a working demo of this change if you would like to see it. Please reach out to me if so.

 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.lpsmit seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


[Azure Digital Twins ] API Review

New API Review meeting has been requested.
Service Name: Azure Digital Twins
Review Created By: Rachel Teitelbaum
Review Date: 09/05/2023 01:00 PM PT
Onboarding Record:
PR:
Hero Scenarios Link: Not Provided
Core Concepts Doc Link: Not Provided
Description: Would like to take our private previewed Bulk Delete API to GA on an accelerated timeline
Detailed meeting information and documents provided can be accessed here
For more information that will help prepare you for this review, the requirements, and office hours, visit the documentation here

Meeting updated by Rachel Teitelbaum
Service Name: Azure Digital Twins
Review Created By: Rachel Teitelbaum
Review Date: 09/13/2023 09:00 AM PT
Onboarding Record:
PR:
Hero Scenarios Link: Not Provided
Core Concepts Doc Link: Not Provided
Description: Would like to take our private previewed Bulk Delete API to GA on an accelerated timeline
Detailed meeting information and documents provided can be accessed here
For more information that will help prepare you for this review, the requirements, and office hours, visit the documentation here

Meeting updated by Rachel Teitelbaum
Service Name: Azure Digital Twins
Review Created By: Rachel Teitelbaum
Review Date: 09/13/2023 09:00 AM PT
Onboarding Record:
PR: https://github.com/Azure/azure-rest-api-specs/pull/25746
Hero Scenarios Link: Not Provided
Core Concepts Doc Link: Not Provided
Description: Would like to take our private previewed Bulk Delete API to General Availability. No major API changes from private preview to GA at the moment.
Detailed meeting information and documents provided can be accessed here
For more information that will help prepare you for this review, the requirements, and office hours, visit the documentation here

Notes from API Review meeting, 9/13/23
API is essentially unchanged from private preview. [PR 12803]
Discussed during the meeting:

Why use POST rather than PUT to create the delete job?
What customer feedback did you get from private preview?
If we have /jobs/imports collection, should the delete collection then also be called /jobs/deletesin plural?
Have any customers used SDKs for this new feature?
Why delete on /jobs/imports but not on /jobs/delete ?
Need better documentation! When to use delete vs cancel.
Using the same API pattern for imports and deletes will make it easier on users

PUT will be easier for idempotency as well

"deletions" is a better option than "delete"

"/jobs/deletions"


Fix empty transaction execution save log

Fix empty transaction execution save log. Currently it is not saving if empty, but it should.

https://github.com/cloudwalk/stratus/issues/1270


Using react-figma

Hi! It's possible to use react-figma for creating universal react components that can be used on Figma (rendering icons to document through plugin) and on the web. I can make PR if are you interested.

What would the benefit of rendering from React to Figma be?
We already have the icons in a Figma file, that acts as the source of truth. When the icons are updated in Figma, we run a script which converts them into React components.
I can't see any benefit in going the other way around?

It's could be useful only if you want to use a different approach - code as a source of truth. So, I see that you not need it and close the issue. 🙌


Export DecorationPosition, use it in ListItem

Borders added with ListItem.divideItems() wouldn't show up if the list items were completely opaque because DecoratedBox draws in the "background" by default.


Initial installation fails

Just running npx @openinterface/cofounder runs into this error. Any help wpuld be appreciated.

same issue here

run command in /cofounder/api folder.

Try this:
Clone the repository directly
git clone https://github.com/raidendotai/cofounder.git
Change into the project directory
cd cofounder
Install dependencies
npm install
Run the package (assuming it has a start script or main entry point)
npm start


performance of hanzi-writer

I'm testing hanzi-writer on mobile platform, and with large displays I have real issue with general performance of animations (probably below 10fps) and it is really ugly (I'm on a google galaxy note 4 with a Android 4.4). Inkstone is based on the same data and same langage and technique here (javascript in webview and tween, the only difference is that they use Cordova, but to my limited knowledge, it shouldn't have any impact) and achieve much better on the same terminals.
I'm still quite new to all this, and I suspect that Inkstone is actually upscaling the resolution rather than drawing more pixel precise animation. I'll post updates if I have time about this topic here for any person finding Hanzi-Writer a little slow and possible technical solutions. Any thoughts on that subject are welcome.
One question I had related to that is : are the CSS animation faster than the javascript ones ? (I'd guess so), and if so, it seems that some actions can be transfered to CSS animation (using transition and keyframes ...).

Another possible difference is using SVG clip-path instead of SVG mask to control the way the stroke looks while animating. I think clip-path should have better performance, but the reason I didn't switch to it fully is because using clip-path doesn't allow adding stroke width, and currently there's small gaps between overlapping strokes in the data from make me a hanzi (https://github.com/skishore/makemeahanzi/issues/28). I want to try working on adding caps to the strokes from make me a hanzi when I have some time. If that issue can be resolved we can switch to using clip-paths for everything.
Could you try setting usePolygonMasks: true in Hanzi Writer? This uses clip-path instead of mask but does more work in javascript to try to calculate a wider clipping path to avoid the stroke gap issue. It might result in smoother animations if mobile is slow due to SVG performance.

Just seen in the code that Inkstone is using createjs which uses canvas and enable all WebGL niceties and use the GPU mostly.

oooh interesting I hadn't considered using canvas. That could be worth investigating - it would certainly have better performance than SVG, but probably requires a fair bit of extra work to translate the SVG path strings from Make me a Hanzi into canvas pixels.

setting usePolygonMasks to true didn't change anything on my test mobile phone.

A valuable comparison on SVG vs Canvas: https://www.sitepoint.com/canvas-vs-svg-choosing-the-right-tool-for-the-job/

I wouldn't be surprised if we could have most of both world by off-loading a lot to CSS animation that will be able to use hardware acceleration and keep SVG and crisp rendering. That would at least worth a try in my opinion.

I wonder if it's just that SVG performance isn't great on mobile. How do some of the SVG images from Make me a Hanzi look on mobile? They're using CSS to generate the animations: https://github.com/skishore/makemeahanzi/tree/master/svgs

Another possibility is that maybe having to calculate opacity for the highlight layer and the other hidden layers is straining on mobile. Maybe if we set display: none on elements after their opacity is 0 that might boost performance too?

They are not animated at all. I guess these CSS animation are not supported. I'll tinker a little more, but this is very possible that only a few CSS operations are supported... as I found out that CSS scale is not supported on my end. Android 4.4 is now fairly old, but I guess it is still an important target.

Ah darn, it might be the case that SVG css animations aren't supported in older browsers. I think canvas should actually be pretty easy to implement though - it looks like there's a path element in canvas that can accept SVG path strings, and canvas supports clipping and transforms too. That should be everything we need for Hanzi Writer to work with canvas instead.

I made a proof-of-concept PR using canvas rendering here: https://github.com/chanind/hanzi-writer/pull/57. Does this perform better on your device? When I was testing in Chrome it appeared to have worse performance than using pure SVG, but it could be that Chrome has really optimized SVG rendering. It's also possible it needs more tweaking to optimize canvas rendering fully.

Hello @chanind
I used your POC on a heavy page, that I open Hanzi Writer as a popup.
For my page, the Canvas version had a huge performance impact.
I recorded 2 videos in the same page, to demonstrate it.
SVG Version:
https://youtu.be/4M7Lo2Gt7eE
Canvas Version:
https://youtu.be/QF2mwjMPw-k
Thank you for your work.
What do you think is missing on the Canvas version?

The Canvas version should work, but when I tested it using Chrome developer tools it didn't seem to perform as well as the SVG version. It could be that Chrome is just really optimized for SVG rendering. Which browser is that in the video? It looks like a webview on Android?

It's a PWA running on Google Chrome 70, Android 8.0, Moto Z2 Play

It probably makes sense to polish the canvas PR and allow it to be enabled as a config option, something like { renderer: 'canvas' }. I'm worried about switching over fully since there are likely users relying on CSS to style the SVG output like in https://github.com/chanind/hanzi-writer/issues/87. I'll put together a PR adding the canvas renderer as a config option.

Great! Thank you!
I just found a bug on the Canvas version.
When you change the ideogram, it doesn't clean the previous ideogram loaded:
https://youtu.be/LYp-1yzM3Ms
To change, i'm using the setCharacter method.
That does't happen on the SVG version.


ci : added default shell in create-release-pull-request

This issue solves the problem in https://github.com/build-trust/ockam/issues/3995 the problem was , in every job or run command there was a need for mentioning which shell do we need to run the command in .

To solve this issue , I used a default to create a map of the settings that will apply to every job in the workflow and gave a default value for shell as bash since every run command in the workflow runs in bash

Checks

[x] All commits in this Pull Request are signed and Verified by Github.
[x] All commits in this Pull Request follow the Ockam commit message convention.
[x] I accept the Ockam Community Code of Conduct.
[x] I have accepted the Ockam Contributor License Agreement by adding my Git/Github details in a row at the end of the CONTRIBUTORS.csv file in a separate pull request to the build-trust/ockam-contributors repository. The easiest way to do this is to edit the CONTRIBUTORS.csv file in the github web UI and create a PR, this will mark the commit as verified.

Nice. One little thing, @rghdrizzle we generate changelogs using users commits, so its very important they follow set of rules, can you check your commit message ci : added default shell in create-release-pull-request
Also, we lint all commit messages in this workflow and this PR currently fails in this CI due to the commit message

I'm sorry , I always end up messing it up with a whitespace

Ok phew a lot of re-basing was done , Hope that its fine now :)


site-packages folder of virtualenv is not correctly read

on Windows when trying to execute the script, the site-packages folder of the calling python is used and not of the python executable specified in the -p argument.
Debugging showed that in _run_subprocess for windows when using sys.executable in subprocess.Popen, the sys.path has the value of sys.executable and not of the venv. I have tested using args=command_line.split() without executable=sys.executable and it worked as expected

Fixed by https://github.com/ftpsolutions/python-third-party-license-file-generator/pull/9 which is now merged in- release 2020.7.18 is up on PyPI at https://pypi.org/manage/project/third-party-license-file-generator/release/2020.7.18/
Thanks all involved- will close this.


Smart Audio Source Selection

Easier way of changing between audio sources.
Suggestion: Add to the Custom Mouse Binding tabs the option "Next/PreviousAudioSource" in the CommandType column.
I'd bind it to mouse wheel up or down, to change between Spotify and AudioBee, depending of what i'm listening.
An auto detection of the audio source would be preferable, but I don't know if it would require monitoring of all sources all the time, may not be viable.

Your first suggestion is something that svr333 or I could probably do.
In issue #64 I do mention automatically switching AudioSources automatically but I don't really know how to do that yet :cry:

First thing is quite easy to implement and is a really good idea, I'll try to get on it when I have some time if TechSupport hasn't already.
The second, on paper it sounds quite easy but I'm not really sure how to effectively approach this new feature.
Currently there are only 4 popular audiosources, but the Windows 10 audiosource can be used with 15+ different apps (probably even hundreds if you count small apps) so it can be quite a struggle to manually assign apps to audiosources.
I also don't want to add code to AudioBand that actively tracks or monitors processes that are currently open on the device, I don't want this app to "spy" on its users.
If anyone has an idea on how to effectively and non-invasively do this, I'm all ears and open for suggestions - but right now I don't see a good way to handle it.


misc: Merge develop .github dir into stable

Is there an easy way to add a description when you make merges like this which includes the information as to what's in the merge?

Is there an easy way to add a description when you make merges like this which includes the information as to what's in the merge?

Not really, no. I just copy the directory. Literally cp -r .github from one branch to another. I don't think there's a better way. It's an unconventional way to use git and github, but we do development on our non-main branch which is just, annoyingly, unsupported in github when it comes to testing so we need to do it this way.


implement sorting PRs and Issues

Allow to sort the PRs and Issues within each section - either newest_first (default) or oldest_first.

Codecov Report
Attention: 7 lines in your changes are missing coverage. Please review.

Comparison is base (1fd2f86) 50.64% compared to head (5f2de20) 50.25%.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #104      +/-   ##
==========================================
- Coverage   50.64%   50.25%   -0.40%
==========================================
  Files           7        7
  Lines         389      396       +7
==========================================
+ Hits          197      199       +2
- Misses        192      197       +5

Files
Coverage Δ

github_changelog_md/config/settings.py
100.00% <100.00%> (ø)

github_changelog_md/main.py
47.82% <ø> (ø)

github_changelog_md/changelog/changelog.py
28.79% <12.50%> (-0.29%)
:arrow_down:

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


Orphaned articles in the 'articles' folder

Bug 4419083 is tracking these 3 related issues.
Summary
The following files in the articles folder are orphaned, meaning that they do not appear in any TOC:

bing-maps-on-azure-marketplace.md
creating-a-store-locator-using-bing-maps-and-azure.md
open-maps-understanding-odbl.md

1. bing-maps-on-azure-marketplace.md

Move file from the articles folder to the archive directory.
Add to .openpublishing.redirection.json.  Redirect to article folder.

2. creating-a-store-locator-using-bing-maps-and-azure.md
This file currently has only a "coming soon" message. Work with Clemens and Justine to complete article.
NOTE: This will be tracked in a separate PR and done at a later date.
3. open-maps-understanding-odbl.md
Add open-maps-understanding-odbl.md to the TOC. It is already referenced in the landing page (index).

Docs Build status updates of commit 0eba2f4:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

.openpublishing.redirection.json
:white_check_mark:Succeeded

BingMaps/articles/TOC.yml
:white_check_mark:Succeeded
View

BingMaps/retired/bing-maps-on-azure-marketplace.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


404

Getting 404 on this
https://rawgit.com/personality-insights/sunburst-chart/tree/master/lib/index.js

Use<EMAIL_ADDRESS>


ipfs: fix fusermount: executable file not found

Motivation for this change
A proposal to fix #27414
Things done

[ ] Tested using sandboxing
(nix.useSandbox on NixOS,
or option build-use-sandbox in nix.conf
on non-NixOS)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] Linux

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[x] Tested execution of all binary files ($SRC/result-bin/bin/ipfs mount -f $HOME/ipfs/ -n $HOME/ipns/)
[ ] Fits CONTRIBUTING.md.

Note that fusermount is a SUID executable, so stdenv.lib.makeBinPath [fusermount] will probably not work. Or does ipfs always run as root?

In the pre fixup hook, it does not find the ipfs binary. Is this the right hook to use or I missed something?
nativeBuildInputs = [ makeWrapper ];
preFixup = ''
  wrapProgram $out/bin/ipfs --prefix PATH : "${stdenv.lib.makeBinPath [ fuse ]}"
'';

post-installation fixup
mv: cannot stat '/nix/store/hcvhgkp6zvkydzj2ssx0a2myxjzp5dr0-ipfs-0.4.10/bin/ipfs': No such file or directory

@bennofs Oh, yeah, right, so this should rather be solved by the NixOS module?
@apeyroux Sorry, you would have to use $bin instead of $out for wrapProgram. But as @bennofs this is indeed no the correct solution.
I'm wondering now why you even have this problem as the security wrappers module at nixos/modules/security/wrappers/default.nix already unconditionally adds the fusermount wrapper and thus makes it available on every NixOS system.

With :
nativeBuildInputs = [ makeWrapper ];

preFixup = ''
  wrapProgram $bin/bin/ipfs --prefix PATH : "${stdenv.lib.makeBinPath [ fuse ]}"
'';

build is ok but I have this error :
nix-shell -I nixpkgs=/home/alex/src/nixpkgs --pure -p ipfs
ipfs mount -f /tmp/ipfs -n /tmp/ipns
Error: fuse failed to access mountpoint /tmp/ipfs

In strace "ipfs mount", there are futex(0x1460b10, FUTEX_WAIT, 0, NULL)   = -1 EAGAIN (Resource temporarily unavailable) ...

[...]
[pid 10209] getsockname(4,  <unfinished ...>
[pid 10215] futex(0xc4201d8510, FUTEX_WAIT, 0, NULL <unfinished ...>
[pid 10209] <... getsockname resumed> {sa_family=AF_INET, sin_port=htons(45100), sin_addr=inet_addr("<IP_ADDRESS>")}, [112->16]) = 0
[pid 10209] getpeername(4, {sa_family=AF_INET, sin_port=htons(5001), sin_addr=inet_addr("<IP_ADDRESS>")}, [112->16]) = 0
[pid 10209] setsockopt(4, SOL_TCP, TCP_NODELAY, [1], 4) = 0
[pid 10209] setsockopt(4, SOL_SOCKET, SO_KEEPALIVE, [1], 4) = 0
[pid 10209] setsockopt(4, SOL_TCP, TCP_KEEPINTVL, [30], 4) = 0
[pid 10209] setsockopt(4, SOL_TCP, TCP_KEEPIDLE, [30], 4) = 0
[pid 10209] futex(0xc4201d8510, FUTEX_WAKE, 1) = 1
[pid 10215] <... futex resumed> )       = 0
[pid 10215] futex(0xc4204fe910, FUTEX_WAKE, 1) = 1
[pid 10209] write(4, "POST /api/v0/mount?encoding=json"..., 251 <unfinished ...>
[pid 10214] <... futex resumed> )       = 0
[...]
[pid 10209] futex(0xc42002e910, FUTEX_WAIT, 0, NULL <unfinished ...>
[pid 10215] <... read resumed> 0xc42029c000, 4096) = -1 EAGAIN (Resource temporarily unavailable)
[pid 10215] futex(0xc4201d8510, FUTEX_WAIT, 0, NULL <unfinished ...>
[pid 10214] <... pselect6 resumed> )    = 0 (Timeout)
[...]
Error: fuse failed to access mountpoint /tmp/ipfs
[...]

and tcpdump return ipfs daemon return 500 with a POST on <IP_ADDRESS>:5001

G...G...POST /api/v0/mount?encoding=json&f=%2Ftmp%2Fipfs&n=%2Ftmp%2Fipns&stream-channels=true HTTP/1.1
Host: <IP_ADDRESS>:5001
User-Agent: /go-ipfs/0.4.10/
Connection: close
Content-Length: 0
Content-Type: application/octet-stream
Accept-Encoding: gzip
09:44:32.756757 IP localhost.commplex-link > localhost.45832: Flags [P.], seq 1:457, ack 252, win 350, options [nop,nop,TS val1191458217 ecr<PHONE_NUMBER>], length 456
E...c<EMAIL_ADDRESS>G.5.G...HTTP/1.1 500 Internal Server Error
Access-Control-Allow-Headers: X-Stream-Output, X-Chunked-Output, X-Content-Length
Access-Control-Expose-Headers: X-Stream-Output, X-Chunked-Output, X-Content-Length
Content-Type: application/json
Server: go-ipfs/0.4.10
Trailer: X-Stream-Error
Vary: Origin
Date: Tue, 18 Jul 2017 07:44:32 GMT
Connection: close
Transfer-Encoding: chunked
42
{"Message":"fuse failed to access mountpoint /tmp/ipfs","Code":0}

ipfs daemon return this log :

09:51:54.739 ERROR       node: error mounting: fusermount: exit status 1 mount_unix.go:87
09:51:54.739 ERROR       node: error mounting: fusermount: exit status 1 mount_unix.go:91

https://github.com/ipfs/go-ipfs/blob/master/fuse/node/mount_unix.go#L87
https://github.com/ipfs/go-ipfs/blob/master/fuse/node/mount_unix.go#L91
The err1 and err2 refer to the two files :
https://github.com/ipfs/go-ipfs/blob/master/fuse/ipns/mount_unix.go
https://github.com/ipfs/go-ipfs/blob/master/fuse/readonly/mount_unix.go
Maybe the "cmd = exec.Command("fusermount", "-u", point)" problem? - https://github.com/ipfs/go-ipfs/blob/master/fuse/mount/mount.go#L46

ping @mguentner

Fixed in #29133


brackets.rb did not remove the older version when updating from 1.5 to 1.6

Today I updated my Brackets from 1.5 to 1.6 but noticed that the formula did not remove the old version (1.5)

It’s not supposed to. You’re supposed to uninstall it yourself.
Also, we call them casks, not formulas. They’re very different in every aspect.

Even if I installed 1.5 with cask? Do I still need to uninstall it myself outside of cask?
Clearly I am new to casks and just want to learn it better and make sure I am not going to break something by deleting an app outside of cask.

Yep, anything inside an old version subfolder is safe to delete.
Alternatively, brew cask uninstall brackets --force && brew cask install brackets

Short answer: we don’t implicitly uninstall, you need to do so explicitly. You don’t need to uninstall outside of cask, though. If you do something like brew cask uninstall --force brackets, you’ll uninstall all versions.
This will all be handled better in the future.

That is good to know.
One last question:
If I am to run brew cask uninstall --force brackets will that also remove my brackets extensions? If it will what would be the best way to remove the older version without removing my brackets extensions?

If I am to run brew cask uninstall --force brackets will that also remove my brackets extensions?

No. The --force just means that all installed versions of brackets will be uninstalled.

So, I should do:
brew cask uninstall --force brackets
brew cask install brackets
and that should keep all my brackets extensions and remove older versions of brackets?

Yes.

It worked! :smile:
That brings me to my last question :innocent:
What should I do if I would like to remove all brackets versions with all expansions and leave no trace of it?

brew cask zap brackets
We have plenty of helpful documentation along with man brew-cask so that you don't have to ask these questions in issues (and we don't have to answer them again and again :wink:)


add trace support

add trace support across event flow
food for thought at this point

add optional util and connector support to initialize x-ray
include trace id in events as a tag or special header
define standard field to store trace id in data stores like dynamodb and s3 records so the from functions can make them available to pipelines
update sinks and queries to pass trace id to sdk calls per pipeline/uow

useful info here: https://docs.aws.amazon.com/xray/latest/devguide/xray-services-sqs.html#xray-services-sqs-retrieving


Personal information

[ ]  Does the code work?
[ ]  Are functions, methods, and variables adequately named?
[ ]  Any test on the new component?
[ ]  Is it well tested? All components could be rendered and pass the test.
[ ]  Is the code clear and easy to understand?
[ ]  Does it meet the team guidelines and style guides?
[ ]  Is there any duplicated code?
[ ]  Is documentation on functions, methods, classes, contexts, and behaviours adequate?
[ ]  Is the code as modular as possible?
[ ]  Are there better or simpler solutions?
[ ]  Is there any performance issue?

Display personal information.
The code could adapt the multi fields code in manual input component and add record component


Dependency on tzinfo

Hey @jmettraux , here is some issue I am facing -
I am using rescue-scheduler v4.0.0
I am using rufus-scheduler v3.0.0
This gem page specifies that with rufus-scheduler v3.0.0, you can use tzinfo >=0
But, going through the source code of rufus-scheduler for tag 3.0.0, I see that you are using classes present only in tzinfo v1.x.x and not in tzinfo 0.x.x
The class which bothers me is TZInfo::TransitionDataTimezoneInfo
Can you please provide some clarification? Correct me if I am wrong.

Hello,
you're probably using Bundler, so you can easily ensure that you have tzinfo 1.x.x installed and you will thus have no problems.
I hope this will help.
Best regards.
P.S. it's resque-scheduler, not rescue-scheduler.


Change date and time format

The format of the date and time is not the best and gives unnecessary information to the user.
I believe I can change this :)
Also, the time display should be from the location searched, it is always on the user's location time. I can also fix this.

Go ahead! Contribute well🚀


Implement ConceptMap $translate

Our current use case is to be able to Map SNOMED codes to codes from other healthcare system inputs.


please use HDF reader for matlab v7.3 files

When I evaluate the experiment results, it always shows "please use HDF reader for matlab v7.3 files", how can I evaluate the mat files? Thank you!

Hello, can you provide the link to download the dataset, I applied to participate in the competition on codalab, no news yet, thank you

@Dave-SUN you are saving results as Mat v7.3 instead of v7.2, please see example code.
@liuyulvv  Please see https://github.com/boazarad/NTIRE2022_spectral


windows could not boot and endpoint not responding

I tried local deployment and stuck at step 3.2 ./run-local.sh --prepare-image true
Here is terminal output
Invoking Docker Container with the command:
docker run -it --rm -p 8006:8006 -p 3390:3389 --name winarena --platform linux/amd64 -e KVM=N -e RAM_SIZE=8G -e CPU_CORES=8 --mount type=bind,source=/Users/xyz/Documents/projects/WindowsAgentArena/src/win-arena-container/vm/image/setup.iso,target=/custom.iso -v /Users/xyz/Documents/projects/WindowsAgentArena/src/win-arena-container/vm/storage/.:/storage -v /Users/xyz/Documents/projects/WindowsAgentArena/src/win-arena-container/vm/setup/.:/shared -v /Users/xyz/Documents/projects/WindowsAgentArena/src/win-arena-container/client/.:/client --cap-add NET_ADMIN --stop-timeout 120 --entrypoint /bin/bash -e OPENAI_API_KEY=xyz windowsarena/winarena:latest -c './entry.sh --prepare-image true --start-client true --agent navi --model gpt-4-vision-preview --som-origin oss --a11y-backend uia'
Starting WinArena...
Starting VM...
Starting WinArena VM...
❯ Starting Windows for Docker v0.00...
❯ For support visit https://github.com/dockur/windows
❯ CPU: 0 | RAM: 12/13 GB | DISK: 670 GB (/0x6a656a63) | KERNEL: 6.6.26-linuxkit...

❯ Extracting local ISO image...
❯ Detecting version from ISO image...
❯ Detected: Windows 11 Enterprise (Evaluation)
❯ Adding drivers to image...
❯ Adding OEM folder to image...
❯ Adding win11x64-enterprise-eval.xml for automatic installation...
❯ Building Windows 11 image...
❯ Creating a 30G growable disk image in raw format...
Cannot find device "dockerbridge"
❯ Booting Windows using QEMU v9.1.1...

BdsDxe: failed to load Boot0002 "UEFI QEMU QEMU HARDDISK " from PciRoot(0x0)/Pci(0xA,0x0)/Scsi(0x0,0x0): Not Found
BdsDxe: loading Boot0001 "UEFI QEMU DVD-ROM QM00013 " from PciRoot(0x0)/Pci(0x5,0x0)/Sata(0x0,0xFFFF,0x0)
BdsDxe: starting Boot0001 "UEFI QEMU DVD-ROM QM00013 " from PciRoot(0x0)/Pci(0x5,0x0)/Sata(0x0,0xFFFF,0x0)
Waiting for a response from the windows server. This might take a while...
❯ Windows has started successfully. You can connect using either a web browser or RDP, both accessible at localhost on the specified ports...
Waiting for a response from the windows server. This might take a while...
Waiting for a response from the windows server. This might take a while...
Waiting for a response from the windows server. This might take a while...

the message Waiting for a response from the windows server. This might take a while... will not end and here is how the browser looks like

Here is what I have tried:
tried https://github.com/microsoft/WindowsAgentArena/issues/22
docker image ran fine and I verified the browser ran windows just fine.
docker run -it --rm -p 8006:8006 -e KVM=N --cap-add NET_ADMIN --stop-timeout 120 dockurr/windows
tried https://github.com/microsoft/WindowsAgentArena/issues/45 but windows.boot is still missing under WindowsAgentArena/src/win-arena-container/vm/storage folder
also tried the branch of win10e-support branch https://github.com/microsoft/WindowsAgentArena/issues/22
Here is my setup:
Mac OS with m3 chip, windows 11 ISO – Enterprise download (64 bit)
Really appreciate any help in fixing this issue!

I have the same problem with you , have you solved this problem? Really appreciate any help.

not really, I think this issue is related to mac, I switched to linux and code ran just fine there.


[BI-1057] Fix plot sorting for Field Book in brapi v1 & v2 endpoints

Adds a fix for plot sorting by plot number in the brapi v1 & v2 endpoints.
Accounts for the following cases:

Plot number is NULL, works fine, gets put last
Plot number is an empty string, gets converted to NULL
Plot number is a string with all letters, gets converted to NULL
Plot number is a string with letters and numbers, letters get removed and numbers remain
Plot number is all numbers, all numbers remain.

Fixes: https://github.com/Breeding-Insight/sgn/issues/25

Do you know why it was apparently working in one of the cases in #25 and for me with blueberry when it wasn't doing any int conversion before? Also strange that it seems to have been missing the order_by in the v1 case.

If you don't specify an order_by value, it does ORDER BY trial_name, observationunit_uniquename, so maybe your names happen to align properly with your plot numbers?


Prevent breaking blocks beneath pipes or the flagpole

People commonly use bricks below exit pipes from underground levels, which may be destroyed, rendering the level uncompletable. This is no fun and really doesn't need to happen at all. Additionally, flagpoles are commonly placed on hard blocks, which can also be destroyed, leaving the pole floating. This is wonky and similarly doesn't need to be allowed.

The hard block can be used as a solid block, or a custom property of the flagpole can be used by adding properties that prevent the block from being destroyed. It would be feasible for the flagpole to be an entity.


Override wrong graphql types automatically

I would like to be able to override my types I have defined in my queries with the right ones.
When trying to generate types and a type in a gql tag is wrong I get an error like this one:
GraphQLDocumentError: Variable "$name" of type "StringFieldUpdateOperationsInput!" used in position expecting type "String".

I would like to pass a flag like --override so that graphql-code-generator just automatically replaces those with the right, expected one

Could you share a reproduction on CodeSandbox?

@BjoernRave We aim not to do manipulations on the GraphQL schema or operations.
If you need to manipulate your schema or operations, you can use custom schema loader or custom documents loader

Okay, I will look into that, but I think a plugin for this wouldn't hurt anyone and could save a lot of time.


Support Maven options

Support Maven options. For example, we'll able to pass in settings file and m2 repo path.
Improve command line options.
Small fixes.

@mayankgarg1990 @aweisberg Ready for review, thanks!


PiShrink giving me an error on WSL Ubuntu.

I'm trying to shrink a Raspberry Pi OS 64-bit image, but when I attempt to shrink it, this happens
pishrink.sh v0.1.2
pishrink.sh: Gathering data ...
Kernel not configured for semaphores (System V IPC). Not using udev synchronisation code.
Kernel not configured for semaphores (System V IPC). Not using udev synchronisation code.
losetup: cannot find an unused loop device
tune2fs: No such file or directory while trying to open
Couldn't find valid filesystem superblock.
tune2fs 1.45.5 (07-Jan-2020)
pishrink.sh: ERROR occurred in line 289: tune2fs failed. Unable to shrink this type of image
How do I fix this? I'm running Ubuntu WSL on Win10 Pro 64-bit 20H2, the image is Raspberry Pi OS 64-bit img created using Win32DiskImager.

Looks like WSL is approx 50% compatible to Linux ... Just use a real Linux and you will be fine :wink:

Looks like WSL is approx 50% compatible to Linux ... Just use a real Linux and you will be fine 😉

Would using an Ubuntu VM work fine? I'd rather do something like that instead of running Linux on an actual computer.

A VM would work perfectly fine. Some people had issues using the Shared Folder feature in VirtualBox and VMware so if you run into issues make sure you're not using that.


Deploy to IBM Cloud Foundry not working - no cloudant !

fails to provision CloudantDB
fails to include manifest.yml - deployment fails due to exceeded memory - 1.25GB instead of 256MB, no dependency on cloudantNoSQLDB service
fixing memory in Cloud Foundry app view results in deployment, but fails to use Cloudant
APP/PROC/WEB | 0 | 13 Apr 14:46:14 - Failed to find the Cloudant URL | Apr 13, 2021, 3:46:14 PM
-- | -- | -- | --
APP/PROC/WEB | 0 | 13 Apr 14:46:14 - Loading application settings | Apr 13, 2021, 3:46:14 PM
APP/PROC/WEB | 0 | 13 Apr 14:46:14 - Starting first-use setup | Apr 13, 2021, 3:46:14 PM
APP/PROC/WEB | 0 | 13 Apr 14:46:14 - Waiting for first-use setup to complete

followed by

APP/PROC/WEB | 0 | 13 Apr 14:47:31 - [info] Flows file     : /home/vcap/app/.node-red/flows.json | Apr 13, 2021, 3:47:31 PM
-- | -- | -- | --
APP/PROC/WEB | 0 | 13 Apr 14:47:31 - [info] Creating new flow file

stop the app once launched/configured and start again -- all config lost (not  a surprise since it was written to ephemeral local file in node.js runtime instance)

This is working for me. Do you have repro steps?
2021-04-13T15:06:31.98+0000 [APP/PROC/WEB/0] OUT > node --max-old-space-size=160 index.js --settings ./bluemix-settings.js -v
   2021-04-13T15:06:33.36+0000 [APP/PROC/WEB/0] OUT 13 Apr 15:06:33 - Starting Node-RED on IBM Cloud bootstrap
   2021-04-13T15:06:33.36+0000 [APP/PROC/WEB/0] OUT 13 Apr 15:06:33 - Loading bluemix-settings.js
   2021-04-13T15:06:33.48+0000 [APP/PROC/WEB/0] OUT [2021-04-13T15:06:33.483] [INFO] ibm-cloud-env - Initializing with /home/vcap/app/server/config/mappings.json
   2021-04-13T15:06:33.49+0000 [APP/PROC/WEB/0] OUT [2021-04-13T15:06:33.498] [ERROR] ibm-cloud-env - File does not exist /home/vcap/app/server/localdev-config.json
   2021-04-13T15:06:33.49+0000 [APP/PROC/WEB/0] OUT 13 Apr 15:06:33 - Using Cloudant service: cloudant db:prodnoderedcftest prefix:nodered
   2021-04-13T15:06:33.71+0000 [APP/PROC/WEB/0] OUT 13 Apr 15:06:33 - Loading application settings
   2021-04-13T15:06:35.57+0000 [APP/PROC/WEB/0] OUT 13 Apr 15:06:35 - Starting first-use setup
   2021-04-13T15:06:35.59+0000 [APP/PROC/WEB/0] OUT 13 Apr 15:06:35 - Waiting for first-use setup to complete
   2021-04-13T15:06:36.27+0000 [CELL/0] OUT Container became healthy

perhaps I wasn't clear -- This behaviour results when using the Deploy to IBM Cloud button in the README.md  of this repo, not the Node-RED app starter kit mechanism from  IBM Cloud catalog.
. initial deployment of toolchain goes OK - but with no prompt for Cloudant initialisation/sharing
. deploy stage fails with out of memory error
. application console shows deployment attempted with 1.25GB -- reset to 256MB
. manually rerun deploy stage
. after (very long) deployment process, application loads -- logs indicate NO Cloudant connected - any changes will be lost after a restart

looks like standalone Deploy to IBM Cloud no longer works as expected
can this be fixed,  or removed if the inttent is to no longer maintain direct deployment from this repo?

@jmeis We don't support direct deployments anymore, right? Since mapping.json is not generated in this scenario.

I think we can either remove the button from the repo or create a link to the Starter Kit on public cloud.

Replaced the button with a new one that aligns with the other starter kits.

thanks Joseph!


Bug/Enhancement? - Georeference incompatibility too sensitive - does not allow for borders

When I am adding a polygon as a georeference, and the borders of the polygon are the same as the borders of the geographic area, then I cannot assign those borders.
See example of Du Bois Ill and its geographic area below:
Use case:

Furthermore, the polygon tool automatically snaps the polygon points to the edges of geographic areas, which automatically makes that georeference "Outside the boundaries".
Solutions: make it so that exact matches with edges do not cause "is not contained" error to appear. Is the model for georeferencing incompatible with this?
@mjy @jlpereira thoughts? I'm guessing this is a difficult situation to model?

Closing as outdated.


Sudden memory spike on downloading lots of files

Hello! I've been chasing down the source of some rather large memory spikes I've been seeing in a project I'm working on where I'm downloading large lists of files, and I think I've narrowed it down to this particular crate, with a reproducer:
use reqwest::blocking;
use std::path::Path;
use std::fs::File;
use std::io::{self, BufRead, BufReader};
use memory_stats::memory_stats;
use std::time::Instant;

fn main() {
    let filelist = "/home/user/filelist.txt";
    let download_path = "/home/user/file-downloads";
    let client = blocking::Client::new();
    let reader = BufReader::new(File::open(&filelist).unwrap());
    let mut counter = 1;
    for file in reader.lines() {
        let now = Instant::now();
        let file_url = &file.unwrap();
        let path = Path::new(&file_url);
        let filename = path.file_name().unwrap().to_str().unwrap();
        let filepath = format!("{}/{}", download_path, filename);
        println!("Downloading file #{}", counter);
        counter+=1;
        match client.get(file_url).send() {
            Ok(r) => {
                match r.error_for_status() {
                    Ok(s) => {
                        match s.bytes() {
                            Ok(b) => {
                                let mut out = File::create(&filepath).unwrap();
                                io::copy(&mut b.as_ref(), &mut out).unwrap();
                            },
                            Err(e) => eprintln!("{}", e)
                        }
                    },
                    Err(e) => eprintln!("{}", e),
                }
            },
            Err(e) => eprintln!("{}", e),
        }
        println!("Download took {} seconds!", now.elapsed().as_secs());
        if let Some(usage) = memory_stats() {
            println!("Current physical/virtual memory: {}/{}", usage.physical_mem, usage.virtual_mem);
        } else {
            eprintln!("Couldn't get the current memory usage :(");
        }
    }
}

As is suggested here, I'm creating a client and reusing it. This all goes relatively well, with the reported RSS memory usage remaining in the tens of MB's out of the gate and for a while, and all downloads taking no more than a second or two to complete:
Download took 0 seconds!
Current physical/virtual memory: 14106624/155574272
Downloading file #2
Download took 0 seconds!
Current physical/virtual memory: 14176256/155574272
Downloading file #3
Download took 0 seconds!
Current physical/virtual memory: 14180352/155574272
Downloading file #4
Download took 0 seconds!
Current physical/virtual memory: 14213120/155574272

However, quite consistently, just after passing 100 file downloads, the process suddenly stalls, and it sees a considerable spike in memory consumption:
Downloading file #103
Download took 0 seconds!
Current physical/virtual memory: 25169920/159465472
Downloading file #104
Download took 0 seconds!
Current physical/virtual memory: 25182208/159465472
Downloading file #105
Download took 7 seconds!
Current physical/virtual memory: 91287552/226574336
Downloading file #106
Download took 0 seconds!
Current physical/virtual memory: 66891776/226574336
Downloading file #107
Download took 5 seconds!
Current physical/virtual memory: 98746368/226574336
Downloading file #108
Download took 0 seconds!
Current physical/virtual memory: 98967552/226574336

I would consider this to be a somewhat "light" example, because on my particular thing I'm writing, this consumption is just a runaway event, going into the GB's of RAM before oom-killer takes care of business, but I think this demonstrates the gist of what I'm experiencing. None of these files are what I'd consider to be particularly large, either.
What's also perhaps interesting is, on my real program, while I'm not doing anything (that I know of) to enable this, I do start seeing things like this written to the systemd journal right when memory consumption starts to spike:
May 20 16:50:45 host downloader[600]: Current physical/virtual memory: 15.34MB/213.72MB
May 20 16:50:45 host downloader[600]: starting new connection: http://local-mirror.lan/
May 20 16:50:45 host downloader[600]: resolving host="local-mirror.lan"
May 20 16:50:45 host downloader[600]: connecting to <IP_ADDRESS>:80
May 20 16:50:45 host downloader[600]: connected to <IP_ADDRESS>:80
May 20 16:50:45 host downloader[600]: pooling idle connection for ("http", local-mirror.lan)
May 20 16:50:45 host downloader[600]: reuse idle connection for ("http", local-mirror.lan)
May 20 16:50:45 host downloader[600]: pooling idle connection for ("http", local-mirror.lan)
May 20 16:50:45 host downloader[600]: reuse idle connection for ("http", local-mirror.lan)
May 20 16:50:45 host downloader[600]: pooling idle connection for ("http", local-mirror.lan)
...<there's actually a ton of these lines, before eventually starting to download again>...
May 20 16:50:50 host downloader[600]: Current physical/virtual memory: 104.19MB/878.01MB
...<downloads proceed again somewhat normally until it happens again>...
May 20 16:52:04 host downloader[600]: starting new connection: http://local-mirror.lan/
May 20 16:52:04 host downloader[600]: resolving host="local-mirror.lan"
May 20 16:52:04 host downloader[600]: starting new connection: http://local-mirror.lan/
May 20 16:52:04 host downloader[600]: resolving host="local-mirror.lan"
May 20 16:52:04 host downloader[600]: starting new connection: http://local-mirror.lan/
May 20 16:52:04 host downloader[600]: resolving host="local-mirror.lan"
May 20 16:52:04 host downloader[600]: connecting to <IP_ADDRESS>:80
May 20 16:52:04 host downloader[600]: connecting to <IP_ADDRESS>:80
May 20 16:52:04 host downloader[600]: connected to <IP_ADDRESS>:80
...<etc>...
May 20 16:52:05 host downloader[600]: Current physical/virtual memory: 114.73MB/1.05GB
May 20 16:52:05 host downloader[600]: reuse idle connection for ("http", local-mirror.lan)
May 20 16:52:05 host downloader[600]: pooling idle connection for ("http", local-mirror.lan)
May 20 16:52:05 host downloader[600]: Downloading next file...
May 20 16:52:05 host downloader[600]: Current physical/virtual memory: 128.09MB/1.05GB
May 20 16:52:05 host downloader[600]: reuse idle connection for ("http", local-mirror.lan)
May 20 16:52:05 host downloader[600]: pooling idle connection for ("http", local-mirror.lan)
May 20 16:52:05 host downloader[600]: Downloading next file...
May 20 16:52:05 host downloader[600]: Current physical/virtual memory: 134.77MB/1.05GB
May 20 16:52:05 host downloader[600]: reuse idle connection for ("http", local-mirror.lan)
May 20 16:52:05 host downloader[600]: pooling idle connection for ("http", local-mirror.lan)
May 20 16:52:05 host downloader[600]: Downloading next file...
May 20 16:52:05 host downloader[600]: Current physical/virtual memory: 146.29MB/1.05GB
...<steadily increasing until>...
May 20 16:52:55 host downloader[600]: Current physical/virtual memory: 3.33GB/4.87GB
May 20 16:52:58 host kernel: reqwest-interna invoked oom-killer: gfp_mask=0x1100cca(GFP_HIGHUSER_MOVABLE), order=0, oom_score_adj=0

I know based on bug #1771 with a similar situation, a client like this is supposed to maintain an internal connection pool. Based on the above, it looks like it's struggling to manage this pool after so many requests? And maybe just starts spawning endless connections with their own read buffers until there's no more RAM available?
Is this expected behavior? Or perhaps I'm doing something hilariously wrong here? I would be totally fine if this were just a coding mistake &/or poor connection handling that I could improve on.
Thanks for this crate and your time! Let me know if I can provide any more details!

Here's a few things that come to my mind:

Calling Response::bytes() will copy all the data into a single buffer (which must grow to fit it all). If you're always putting it in a file, you could just stream it instead: io::copy(&mut resp, &mut out). That should use less temporary memory.
The default memory allocator is often not very good. Many people find using a better one, like jemalloc or mimalloc, provides a better experience. They do a better job of managing fragmentation and releasing memory back to the OS.
It's certainly possible there's a bug in the connection pool code. Oh, if only we could write bugfree code! If the previous two points don't help, we could dig around some to see if the pool is holding something longer than it should.


Domain activation is Failed - [BRIEF DESCRIPTION]

App Id: d2tn8d1erecsjn
Region: us-east-1
Step: Domain activation
Status: Failed

Note: Do not include information that is sensitive in nature such as your domain name, company etc.

Issue/question
A clear and concise description of what the issue/question is.
Error message
If there is an error message, please include it here.
Note: Be sure to check the message for sensitive information.
Additional information
Please add any other relevant information. Please feel free to include screenshots.

Hello @welkomtravel
Can you please share the error message the you see on the domain management screen in the console?
Thanks!


SERVICE response check is not sufficiently flexible (regression)

In this commit by @UNEXENU:
https://github.com/ad-freiburg/qlever/commit/87e33299c49acdd1221f8ae12b9cecb30c58b5eb
The media type response of a SERVICE response was specified, and it now raises an exception on a query that does not match.
However, some services (Wikipedia, https://query.wikidata.org/sparql) can return a media type like this:
application/sparql-results+json;charset=utf-8
This is correct, except that the charset specifier is also included, which is allowed. The result is the SERVICE query fails:
Error while executing a SERVICE request to <https://query.wikidata.org:443/sparql>: QLever requires the endpoint of a SERVICE to send the result as 'application/sparql-results+json' but the endpoint sent 'application/sparql-results+json;charset=utf-8'.

So the new check for the media type of the response is not sufficiently flexible to allow for additional fields in the response type. This is a regression, since our SERVICE query used to work but now it raises an exception.

Thank you for reporting this issue.
It seems like i was unaware of the potential existence of parameters in content-types, which lead to this bug.
#1490 should fix this / allow parameters in the content-type again.


Add a Favicon to the site.

I would like to address this issue

I've already made a PR with this issue solved


Typos in README.md

There are few grammatical as well as typing errors in Readme.md

Closed, I think we have this covered.


Feat: Light and dark mode button size is too big in the mobile phone

Describe the feature
In the navbar Light and dark mode button size is too big in the small devices like mobile phone.
I want to do the work on this issue. assign me
Add ScreenShots

Record

[X] I agree to follow this project's Code of Conduct
[X] I have starred the repository
[X] I'm a gssoc-24-extd contributor
[X] I want to work on this issue

@Hemraj-7 ok . Just add the button in the hamburger menu

@Hemraj-7 ok . Just add the button in the hamburger menu

Ok


Added example of exporting to HTML

Addresses suggestion made in #1819.
A remaining issue is that there are various tornado messages printed in the notebook throughout, along with a Bokeh deprecation warning in cell 19 after pressing play and then pause:

If anyone knows a fix for those issues, feel free to push them to this PR.  Otherwise ready for review and merge.

Actually, is there some reason we need casual users to know about the concept of a "renderer"?  Shouldn't we provide a very simple top-level function for them to export an object using the current backend?  E.g. something like this in the top-level hv namespace:
def export(obj, filenamebase, backend=None):
    if backend is None:
        renderer = # look up current renderer
    else:
        renderer = hv.renderer(backend)
    renderer.save(obj,filenamebase)

So that hv.export(obj,'out3') would create out3.html if Bokeh is the current default plotting extension or out3.png if Matplotlib is the default (or if it is selected explicitly using  hv.export(obj,'out4',backend='matplotlib'))?  I'm not sure how to look up the current backend or the default renderer, but it seems like this would be much more obvious and convenient for users and would help dispel the notion that HoloViews is only really meant for use with Jupyter.


Pause when scrolled out of viewport

This is a question.
I'm trying to add multiple instances of wavesurfer.js on my page. I have my HTML structure like this:

and my script is like this (loaded in  wih defer attribute after the wavesurfer.js script):
window.addEventListener('load', function() {
  var mediaA = document.querySelectorAll('.audio-player')
  if (mediaA != null) {
    mediaA.forEach(medium => {
      var wavesurfer = WaveSurfer.create({
        /* wavesurfer config */
      })
      wavesurfer.load(medium.getAttribute('src'))
      window.addEventListener('scroll', function() {
        if (outOfViewport(medium) {
          wavesurfer.pause()
        }
      })
    })
  }
}

Basically, I'm trying to pause the wavesurfer instance once it's scrolled out of viewport and for that, this piece of code works perfectly (the outOfViewport is a helper function which returns a boolen based on my conditions).
As you can see from my code, I'm adding multiple scroll event listeners to the window and I would like to avoid that. I would like to group all my scroll events into one and make it a single function. However, I don't know how I can access the already created wavesurfer instance.
I'm looking for something like:
window.addEventListener('scroll', function() {
  /* my other scroll stuff */
  var mediaA = document.querySelectorAll('.audio-player')
  if (mediaA != null) {
    mediaA.forEach(medium => {
      if (outOfViewport(medium) {
        medium.paue()
      }
    }
  }
}

except, the medium.pause() should actually be called on the wavesurfer instance and not the HTML element (like it's doing in the code I posted).

However, I don't know how I can access the already created wavesurfer instance.

You create wavesurfer instances like this:
var wavesurfer = WaveSurfer.create({
        /* wavesurfer config */
      })

so store that instance in an array or object so you can access it later:
let surfers = [];

var ws = WaveSurfer.create({
        /* wavesurfer config */
      })
surfers.push(ws);

// later
surfers[0].pause();

However, I don't know how I can access the already created wavesurfer instance.

You create wavesurfer instances like this:
var wavesurfer = WaveSurfer.create({
        /* wavesurfer config */
      })

so store that instance in an array or object so you can access it later:
let surfers = [];

var ws = WaveSurfer.create({
        /* wavesurfer config */
      })
surfers.push(ws);

// later
surfers[0].pause();

Yes, this was indeed easy. I should have been able to think of it. Thanks a lot for the help!

Yes, this was indeed easy. I should have been able to think of it. Thanks a lot for the help!


[esprima] adds esprima.Syntax object with constants

See:
https://github.com/jquery/esprima/blob/master/esprima.js#L107
https://github.com/jquery/esprima/blob/master/esprima.js#L5616
The exported Syntaxwas not exposed in the type definition. Also bumped version to "2.4.1" since that is the latest released version (July 1).
Also corrected some code style issues, since the file was missing a few semicolons.

@fmilitao thanks mate!


Cannot send message to unsaved contact

hello, sorry if i miss the documentation,
but i have problem to send any message to contact who unlisted in my whatsapp contact
sorry for my bad in english
thank before

You must to read section Security on README - that's not allowed using this library.

Thanks before, now im understand


CRUD Operations in MongoDB

Story1: Get JavaScript code with CRUD operations on MongoDB for insert, Read, Delete and Update commands

Inserting new document to the mongo db using node js is working fine.

async function Insert1(client){
const database = client.db("DB1");
const Iteams = database.collection("Emids");
const doc = {
title: "Record of a Shriveled Datum",
content: "No bytes, no problem. Just insert a document, in MongoDB",
}
const data = Iteams.insertOne(doc);
//data.insertOne(db => console.log( -"having id: " ${db._id }," with name: " ${db.Name} ,${db.Contact} ,${db.Department},${db.Location}));
console.log(doc);
};

Crud operation with node js is working fine


Adding footer to Terms and services page

Hey @RIKI-05 ,
I would like to add the footer to terms and services page which will be the same as home page.
Also, there is no way of navigating back to the home page from the terms and services page as well as the contact page.
I would like to include a link to home page besides the other links in the footer.

is this issue assigned @AvadhootK

i would like to work on this

@AvadhootK sorry for the delay please do the necessary changes

@DhirajGilda if you want you can make a dummy policies page


cannot setup galleon

just tried to install galleon, get a fatal error on the setup:
TypeError: Cannot set property 'attachments' of undefined
at PromptUI.completed (/usr/local/lib/node_modules/galleon/bin/tasks/setup.js:77:90)
at PromptUI.onCompletion (/usr/local/lib/node_modules/galleon/node_modules/inquirer/lib/ui/prompt.js:69:10)
at AnonymousObserver.Rx.AnonymousObserver.AnonymousObserver.completed (/usr/local/lib/node_modules/galleon/node_modules/inquirer/node_modules/rx/dist/rx.js:1793:12)
at AnonymousObserver.Rx.internals.AbstractObserver.AbstractObserver.onCompleted (/usr/local/lib/node_modules/galleon/node_modules/inquirer/node_modules/rx/dist/rx.js:1730:14)
at AnonymousObserver.tryCatcher (/usr/local/lib/node_modules/galleon/node_modules/inquirer/node_modules/rx/dist/rx.js:567:29)
at AutoDetachObserverPrototype.completed (/usr/local/lib/node_modules/galleon/node_modules/inquirer/node_modules/rx/dist/rx.js:5288:56)
at AutoDetachObserver.Rx.internals.AbstractObserver.AbstractObserver.onCompleted (/usr/local/lib/node_modules/galleon/node_modules/inquirer/node_modules/rx/dist/rx.js:1730:14)
at InnerObserver.onCompleted (/usr/local/lib/node_modules/galleon/node_modules/inquirer/node_modules/rx/dist/rx.js:3425:65)
at InnerObserver.tryCatcher (/usr/local/lib/node_modules/galleon/node_modules/inquirer/node_modules/rx/dist/rx.js:567:29)
at AutoDetachObserverPrototype.completed (/usr/local/lib/node_modules/galleon/node_modules/inquirer/node_modules/rx/dist/rx.js:5288:56)
at AutoDetachObserver.Rx.internals.AbstractObserver.AbstractObserver.onCompleted (/usr/local/lib/node_modules/galleon/node_modules/inquirer/node_modules/rx/dist/rx.js:1730:14)
at AutoDetachObserver.tryCatcher (/usr/local/lib/node_modules/galleon/node_modules/inquirer/node_modules/rx/dist/rx.js:567:29)
at AutoDetachObserverPrototype.completed (/usr/local/lib/node_modules/galleon/node_modules/inquirer/node_modules/rx/dist/rx.js:5288:56)
at AutoDetachObserver.Rx.internals.AbstractObserver.AbstractObserver.onCompleted (/usr/local/lib/node_modules/galleon/node_modules/inquirer/node_modules/rx/dist/rx.js:1730:14)
at MergeObserver.onCompleted (/usr/local/lib/node_modules/galleon/node_modules/inquirer/node_modules/rx/dist/rx.js:3390:44)
at MergeObserver.tryCatcher (/usr/local/lib/node_modules/galleon/node_modules/inquirer/node_modules/rx/dist/rx.js:567:29)
steps:
1 - sudo npm install -g galleon
2 - galleon setup
3 - enter some local domain name (local)
4 - leave both options (Save attachments, Store raw emails) selected
5 - error shows
versions:
node v0.10.30
mongodb v3.0.4
if you need any other info, just ask   :)

I've failed to reproduce the issue on v0.10.30 but the setup script is still very immature so I doubt there are any issues with your setup. I'll commit some improvements to prevent issues like this in the next few days but in the mean time you can check if you have the .galleon directory in your Home directory (cd ~/.galleon) with a galleon.conf file. If yes then delete the entire folder and try galleon setup again. The issue should not persist as the config variables only initially default to the existing config (if any) to prevent overrides every time galleon setup is called. Let me know the result.

hey, thx for quick response
so, did not have a galleon.conf file, but removed the dir anyway.
then, jumped to node 0.12.7, got a new error.
then, unistalled, reinstalled with the 0.12.7, and it works. so, the build MUST be done with some minimum version. probably the one you have in your dev env  ;)
an update to the readme should save some pain to the slackers (like me) who update their VMs once in a century  :D

Glad that it was fixed, let me know what environment/OS your are running. I have run tests on Ubuntu 12.04 and Ubuntu 14.04 with multiple versions of Node and yet I can't reproduce your initial issue.

the original environment was a 32bit, 3.18 kernel arch linux with 0.10.30 installed, this is where i got the error. then reinstalled/rebuilt on 0.12.7 and worked good.
i  use the "n" package to test/jump node versions, and i'm not shy jumping from one to the other :)
also, it's a local testing VM, so i just hack it away without much care for stability. could have been some dependency that got updated too. so, don't loose any sleep over this one  :)

It would be nice to have a light version of galleon. Only for emails sending purposes.

@Globik Galleon uses smtp-connection module for sending emails under the hood, I would definitely recommend taking a look at https://www.npmjs.com/package/nodemailer, Galleon serves the purpose of queuing and dispatching emails over a long period of time but I would love to revisit the API over the holidays if I can find the time.


Avoid thread::spawn in CompatibleCiphers::new

Enable feature rt of dep tokio
Unify import from std::os::unix
Add new dep scopeguard v1.1.0
Make CompatibleCiphers::new async to avoid thread spawning and also return io::Result<Self> to avoid panic.
Fix Error: Add #[source] annotation
Refactor: Extract new async fn CompatibleCiphers::test_ciphers
Optimize accept_conns: keep them alive instead of reading from them.
Add test compatible_ciphers_single_thread

Signed-off-by: Jiahao XU<EMAIL_ADDRESS>

Did you manage to determine why it blocked when the accept future isn't spawned?
I'm not overly fond of the fixed-size array for conns — can you make it a Vec or use the previous strategy of draining the socket until it's closed by the other end? As-is it seems like there could be a race condition where accepted sockets replace already-accepted sockets in the ring buffer you've got there.
I'm aware it can't happen right now because we only do 6 test connections and the ring buffer has a capacity of 8, but by the time we add more checks, we'll have completely forgotten about CompatibleCiphers and it's potentially a pretty bad bug to have.

I'm not overly fond of the fixed-size array for conns — can you make it a Vec or use the previous strategy of draining the socket until it's closed by the other end?

Done, I've replaced the array with a Vec.
The problem of draining the socket is that it prevents test_cipher from running concurrently.
And both approaches are subject to denial-of-service attack.
Malicious attacker can just keep establishing new connection and keep writing data to the established connections to keep the process busy and the actual test_cipher might never have a chance of being executed.
I feel like we should filter the accepted connection by local addresses to prevent this attack.

Did you manage to determine why it blocked when the accept future isn't spawned?

I haven't figured that out yet.
I've added a new test compatible_ciphers_single_thread that runs under tokio's current_thread runtime and it worked without problem, so not sure why futures::future::join or futures::join! does not work.

Did you manage to determine why it blocked when the accept future isn't spawned?

I just figured out that there's another way to avoid spawning future.
It requires a little bit of refactoring, but should fix the problem in additional to preventing denial-of-service attack.

Did you manage to determine why it blocked when the accept future isn't spawned?

I know why it blocked forever...
That's because TcpListener::accept would always wait for more connections...

Did you manage to determine why it blocked when the accept future isn't spawned?

I've fixed that, but I didn't add the anti-DOS protection for now.

My ubuntu box and docker desktop on macos is having some problem running the kTLS.
It gives a lot of strange errors that is not related to the changes in this PR.
@fasterthanlime Can you run the test on this PR on your linux machine please?


HDF5 installation with luarocks error

Hello!
When typing sudo luarocks install hdf5 I get the following errors:

gcclua.c: In function ‘int gcclua_tree_get_type_anonymous(lua_State*)’:
gcclua.c:636:22: error: ‘TYPE_ANONYMOUS_P’ was not declared in this scope
lua_pushboolean(L, TYPE_ANONYMOUS_P(*t));
^~~~~~~~~~~~~~~~
gcclua.c:636:22: note: suggested alternative: ‘MAP_ANONYMOUS’
lua_pushboolean(L, TYPE_ANONYMOUS_P(*t));
^~~~~~~~~~~~~~~~
MAP_ANONYMOUS

How do I fix that?

I have the same problem. Any help would be appreciated! Thanks!

^ same

met the same problem. several ways were tried, no one worked.


Build failure: qutebrowser

Steps To Reproduce
Steps to reproduce the behavior:

build qutebrowser

Build log
error:
       … while calling the 'derivationStrict' builtin

         at /builtin/derivation.nix:9:12: (source not available)

       … while evaluating derivation 'home-manager-generation'
         whose name attribute is located at /nix/store/9avrpkh90hh5nciins6xz27xnf5rra49-source/pkgs/stdenv/generic/make-derivation.nix:300:7

       … while evaluating attribute 'buildCommand' of derivation 'home-manager-generation'

         at /nix/store/9avrpkh90hh5nciins6xz27xnf5rra49-source/pkgs/build-support/trivial-builders/default.nix:87:14:

           86|       enableParallelBuilding = true;
           87|       inherit buildCommand name;
             |              ^
           88|       passAsFile = [ "buildCommand" ]

       (stack trace truncated; use '--show-trace' to show the full trace)

       error: stem-1.8.2 not supported for interpreter python3.11

Additional context
this issue appears to match a recent comment added to the stem nix package
Notify maintainers
@rhendric (for lack of listed maintainers)
Metadata
Please run nix-shell -p nix-info --run "nix-info -m" and paste the result.
[user@system:~]$ nix-shell -p nix-info --run "nix-info -m"
 - system: `"x86_64-linux"`
 - host os: `Linux 6.1.59, NixOS, 23.11 (Tapir), 23.11.20231022.e7ce711`
 - multi-user?: `yes`
 - sandbox: `yes`
 - version: `nix-env (Nix) 2.17.1`
 - channels(kiara): `"home-manager"`
 - channels(root): `"home-manager, nixos-23.05"`
 - nixpkgs: `/nix/var/nix/profiles/per-user/root/channels/nixos`

qutebrowser's maintainers: @jagajaga @rnhmjoj @ebzzry @dotlambda @nrdxp
(Unsubscribing as I don't have anything to do with this software; re-@ me if you want my attention.)

from a brief examination it looks like the stem library is not available for python 3.10. No experience with this package but it looks like it is for a userscript. I see @dotlambda has already opened a PR to fix it, but according to the libraries own website, it is no longer maintained, so perhaps we should consider just removing it?


Update EtcdUtil.java byteSequceFromByteString

change the first line of the commit to rename byteSequceFromByteString to byteSequenceFromByteString so it describes what the commit is doing and this should be OK to merge. Thanks!

@heyitsanthony thanks, done


Firefox iOS flex getting removed from styles

Having trouble getting flex passed in iOS firefox browser. It seems like it gets removed from the styles completely.
Elements within a flex container with styles: { flex: 1 } won't grow as expected.

Can you provide an example input and output? Is this in the dynamic and/or static prefixer?

Would be good to know that. Thinking about dropping the dynamic prefixer is actually pretty realistic as I got no time to maintain both.

I have a similar issue while using FormidableLabs/radium. Seems to depend on user agent. User agent where the problem occurs is: Mozilla/5.0 (iPhone; CPU iPhone OS 10_3_3 like Mac OS X) AppleWebKit/603.3.8 (KHTML, like Gecko) FxiOS/10.6b8836 Mobile/14G60 Safari/603.3.8. Example of input css is display: 'flex' flexDirection: 'row' with output of     display: -webkit-box;     -webkit-box-orient: horizontal;     -webkit-box-direction: normal;

bowser._detect("Mozilla/5.0 (iPhone; CPU iPhone OS 10_3_3 like Mac OS X) AppleWebKit/603.3.8 (KHTML, like Gecko) FxiOS/10.6b8836 Mobile/14G60 Safari/603.3.8")
```
a: true
firefox: true
ios: true
iphone: true
mobile: true
name: "Firefox"
osname: "iOS"
osversion: "10.3.3"
version: "10.6"
webkit: true

I haven't looked into the code yet, but I would guess being `firefox` and `ios` with a lower version than what firefox normally reports (`10.6` vs 59 or whatever), things might be getting confused.


Generating WAST eosio.bios.wast error

While installing eos via the eosio_build.sh script I get the following error (mac OS):
[ 72%] Generating WAST eosio.bios.wast
<< _ZNSt3__16localeC1ERKS0_ >>
[[bad mustMatch:]]:
==========
.set _ZNSt3__16localeC1ERKS0_, _ZNSt3__16localeC2ERKS0_@FUNC
==========
/bin/sh: line 1: 44859 Abort trap: 6           /Volumes/main_disk/crypto/eos/eos/build/externals/binaryen/bin/eosio-s2wasm -o /Volumes/main_disk/crypto/eos/eos/build/contracts/eosio.bios/eosio.bios.wast -s 4096 eosio.bios.s
make[2]: *** [contracts/eosio.bios/eosio.bios.wast] Error 134
make[1]: *** [contracts/eosio.bios/CMakeFiles/eosio.bios.dir/all] Error 2
make[1]: *** Waiting for unfinished jobs....
[ 72%] Built target chainbase_test
[ 73%] Built target appbase_example
[ 78%] Built target fc
make: *** [all] Error 2

	>>>>>>>>>>>>>>>>>>>> MAKE building EOSIO has exited with the above error.

Fixed it finally:

OS restart
Remove eos git clone
clone again
Run again


Rename dist/style.css

I think it would be nice to rename this file to be something a bit more specific, like multiselect.css. This seems to be the common practice for angular libraries
In my case will prevent me from having to make changes to my build step. (it currently takes a list of css file paths and moves them into a folder named vendor.) So... for your library I will be including vendor/style.css which is a bit ambiguous.

Yes, sounds reasonable.

Fixed


Enable internal API proxy for not to rely on CORS on development

I understand the logic of not allowing too much flexibility on the CORS middleware, but that address should be configurable. For example, I've had to have multiple Volto sites running at the same time, while developing, to be able to compare between various problems or implementations.

Mmm I see. We can try to make it configurable, but I found it’s difficult
if you want to maintain backward compatibility. Any idea how to accomplish
that? I will give it a try today.
On Tue, 21 Apr 2020 at 06:37, Tiberiu Ichim<EMAIL_ADDRESS>wrote:

I understand the logic of not allowing too much flexibility on the CORS
middleware, but that address should be configurable. For example, I've had
to have multiple Volto sites running at the same time, while developing, to
be able to compare between various problems or implementations.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/plone/volto/pull/1390#issuecomment-616947016, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AADW4DY3NWY2FAAQKOTNA7DRNUPIDANCNFSM4ML34ZIA
.

--
Víctor Fernández de Alba
Github/Twitter/IRC: sneridagh

@tiberiuichim How about now? I didn't customize the /api... do you think it's required?

No, it looks perfect now

Maybe rename settings.proxyToApiPath to settings.devProxyToApiPath ?

@tiberiuichim good one, devProxyToApiPath it is: Done!


Iphone Network Spinner problem

Hi,
i like your Tool very much but i have a problem with the Network Spinner / Indicator. If i open a ViewController (Alamofire imported) and get some Data from Webserver all works fine only the Iphone Networt Indicator is not hiding. It spinns permanently like a endless loop.
Any idea why?
Thanks for support!

Hey mate,
This is the correct repo. Please follow that.
https://github.com/Alamofire/AlamofireNetworkActivityIndicator
Also please provide some code how your calling.

Thanks @hardikdevios! 🍻


Add healthcheck for nims

Description

Checklist

[ ] I am familiar with the Contributing Guidelines.
[ ] New or existing tests cover these changes.
[ ] The documentation is up to date with these changes.

Healthcheck added in #142


Swedish

resurrected old source with new dictionary generation from AOSP as discussed in https://github.com/AnySoftKeyboard/AnySoftKeyboard/issues/690#issuecomment-276605308

very nice. Thank you!

Changed according to the comments

Thanks. I might have some time tomorrow to prepare everything for a release.

Thanks!

Cool, thank you! No need to hurry just for me :-)

pushed to Play Store. Should be available in the next couple of hours.
Working on fdroid rn.

fdroid is on the way: https://gitlab.com/fdroid/fdroiddata/merge_requests/1994