[10.0] Ship migrations with package

This adds Cashier's migrations to the package itself so they're shipped with Cashier and can be published just like with Passport and Telescope. This also directly integrates the migrations into the test suite so they're tested. Orchestra testbench helps us to implement this.
I needed to update migrations for a future PR so I thought I'd implement this first.
Closes https://github.com/laravel/cashier/issues/639

Am I missing something here...?
If we've already run the migrations as per the documentation for Laravel <5.8 the migrations will fail, if we use the disable migrations function; future migrations for the package wont run...
Shouldn't there be command to populate the migrations table with details of the migration to let the system know the migrations don't need running without preventing future migrations running?

Also the migration assumes the users table is the model that is holding the payee entity, which isn't always the case... (such as in ours)

@SlyDave any breaking changes to the table structure will be documented in the upgrade guide so you can always reference that when upgrading.

Also the migration assumes the users table is the model that is holding the payee entity, which isn't always the case... (such as in ours)

You can publish the migrations and alter the table name.

Thanks @driesvints
For anyone else:
publish the migrations: php artisan vendor:publish --tag="cashier-migrations"
and remove the up and down content, leaving them as empty stubs.
This will allow  the migrations to run and allow future package migrations to work.
(Note however, if you're changing the CASHIER_MODEL you'll always need to check future migrations anyway...)

Glad you figured it out 👍
This is also documented here btw: https://laravel.com/docs/5.8/billing#installation

There is one unfortunate side effect of this that isn't as easily overcome.
The original migration that was used before the migration was bundled has the same class name...
If I change the published migration, the package will use it's own version again.
If I change my migration, it'll be seen as a new migration.
In my case, because I'm changing the CASHIER_MODEL I'm going to disable migrations from Cashier, and I can simply not use the published migrations. But for people not changing the model, this will present a difficult to overcome problem.
They will have to update their migration class name and filename to be different, then manually changed the migrations table setting the 'migration' to the new name.


Request for resources to understand deploy-webapi.yml and deploy-webapp.yml

Is it possible to provide references to understand the meaning of all the fields in the two files below referenced by the article?
deploy-webapi.yml
apiVersion: apps/v1
kind: Deployment
metadata:
  name: webapi
  labels:
    app: weather-forecast
spec:
  replicas: 1
  selector:
    matchLabels:
      service: webapi
  template:
    metadata:
      labels:
        app: weather-forecast
        service: webapi
    spec:
      containers:
        - name: webapi
          image: exploredocker.azurecr.io/webapi:v1
          imagePullPolicy: IfNotPresent
          ports:
            - containerPort: 80
              protocol: TCP
          env:
            - name: ASPNETCORE_URLS
              value: http://+:80
---
apiVersion: v1
kind: Service
metadata:
  name: webapi
  labels:
    app: weather-forecast
    service: webapi
spec:
  ports:
    - port: 80
      targetPort: 80
      protocol: TCP
  selector:
    service: webapi

deploy-webapp.yml
apiVersion: apps/v1
kind: Deployment
metadata:
  name: webapp
  labels:
    app: weather-forecast
spec:
  replicas: 1
  selector:
    matchLabels:
      service: webapp
  template:
    metadata:
      labels:
        app: weather-forecast
        service: webapp
    spec:
      containers:
        - name: webapp
          image: exploredocker.azurecr.io/webapp:v1
          imagePullPolicy: IfNotPresent
          ports:
            - containerPort: 80
              protocol: TCP
          env:
            - name: ASPNETCORE_URLS
              value: http://+:80
            - name: WebApiBaseAddress
              value: http://webapi
---
apiVersion: v1
kind: Service
metadata:
  name: webapp
  labels:
    app: weather-forecast
    service: webapp
spec:
  type: LoadBalancer
  ports:
    - port: 80
      targetPort: 80
      protocol: TCP
  selector:
    service: webapp

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

ID: 95aa8619-2154-00fb-5994-09b7a9673e03
Version Independent ID: 182cacc8-12d1-70ac-6005-71d4f6f420d5
Content: Build ASP.NET Core applications deployed as Linux containers into AKS/Kubernetes clusters
Content Source: docs/architecture/containerized-lifecycle/design-develop-containerized-apps/build-aspnet-core-applications-linux-containers-aks-kubernetes.md
Product: dotnet-architecture
Technology: containerized-lifecycle
GitHub Login: @nishanil
Microsoft Alias: nanil

Hi @PingPongSet, thank you for reaching out.
You can refer Deployments section in kubernetes.io to learn more about each field of that provided yaml file.

Hi @PingPongSet , considering the original question has been answered. So I am closing this issue as of now.
Please feel free to reopen if you have any further questions. Thank you.


main_branch input for build_main_documentation.yml

if this comment is true, then this PR is needed

No as said before, the name for the default branch is already customizable with the doc config. This would also only build the documentation for the main version and not the released versions.

oh indeed. forgot about that solution 😊


Correct initial dynamic table limit

The correct value is given in rfc9113 paragraph 4.3.1:
When a connection is established, the dynamic table size for the
HPACK decoder and encoder at both endpoints starts at 4,096 bytes,
the initial value of the SETTINGS_HEADER_TABLE_SIZE setting.

This fixes "Header table index 82 out of range" errors ocurring when using Cro::HTTP::Client with HTTP2 and multiple requests per connection.
fixes croservices/cro-http#122
fixes croservices/cro-http#156

Thanks; released!


[BD-46] feat: Providing default fallback images for Card.ImageCap

Description

add a default fallback PNG image to the Card.ImageCap component in Paragon, and use it as a default value for the fallbackSrc prop.

Related links

Brand-edx
Brand-openedx

Deploy Preview
Card component
Merge Checklist

[x] If your update includes visual changes, have they been reviewed by a designer? Send them a link to the Netlify deploy preview, if applicable.
[x] Does your change adhere to the documented style conventions?
[x] Do any prop types have missing descriptions in the Props API tables in the documentation site (check deploy preview)?
[x] Were your changes tested using all available themes (see theme switcher in the header of the deploy preview, under the "Settings" icon)?
[x] Were your changes tested in the example app?
[x] Is there adequate test coverage for your changes?
[x] Consider whether this change needs to reviewed/QA'ed for accessibility (a11y). If so, please add wittjeff and adamstankiewicz as reviewers on this PR.

Post-merge Checklist

[ ] Verify your changes were released to NPM at the expected version.
[ ] If you'd like, share your contribution in #show-and-tell.
[ ] 🎉 🙌 Celebrate! Thanks for your contribution.

Thanks for the pull request, @PKulkoRaccoonGang!
When this pull request is ready, tag your edX technical lead.

The patterned/textured fallback image can live in @edx/brand-edx.org. The fallback image for the paragon repo can be a solid gray background.
Also add solid  gray background image to @edx/brand-openedx.
Open question:

Can we get the fallback image in SVG? Yes.

@PKulkoRaccoonGang 🎉 Your pull request was merged!  Please take a moment  to answer a two question survey so we can improve your experience in the future.

:tada: This PR is included in version 20.26.2 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This PR is included in version 21.0.0-alpha.12 :tada:
The release is available on:

npm package (@alpha dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Add kubernetes shapes

Hey,
The kubernetes community has a set of icons for different kubernetes resources. It would be nice to have them natively in draw.io
https://github.com/kubernetes/community/tree/master/icons

I made a stencil that load the icons in the sidebar, u can use it as a plugin.
Plugin URL for Draw.IO:
https://yacdn.org/serve/https://raw.githubusercontent.com/CoolZeroNL/draw-io-plugins/master/%23 Draw-io-plugin-K8-Icons/draw-io-plugin-k8-icons-v1.js?maxAge=10

We did our own implementation and it will be in the next release.

The kubernetes community icons set has a much richer set of icons, the option of having them labeled or unlabelled, and a slide deck to guide their usage.

How can I import the kubernetes community icons?

Where is the source to the other implementation?
What is the license for that set?

As per the initial request.
https://github.com/kubernetes/community/tree/master/icons
Apache-2.0 or CC-BY-4.0

And that is in the set originally referenced?


Add Stacks Connect content to Docs

This issue is to add all the Stacks connect functions listed in Docs.
E.g., https://connect.stacks.js.org/modules/_stacks_connect.html
I got to know from Jannik that these Connect functions are auto-generated. So, we need to work with him to get a link to all the auto-generated functions and link them in Docs.

https://connect.stacks.js.org/

Related to https://github.com/hirosystems/docs/issues/129

Done!


IPv6 addresses don't fit in the table (Inspect tab)

Describe the bug
In the "inspect" tab, IPv6 addresses sometimes occupy 2 or 3 table columns, so the content to the right no longer aligns with the table header:

And to see all content, it requires vertical scrolling, but this doesn't scroll the table header:

To Reproduce
Describe steps to reproduce the bug, e.g.:

Inspect traffic with IPv6 packets

Expected behavior
The table header should always be aligned to the content
Desktop (please complete the following information):

OS: Linux
Version 1.2.1

First of all, thank you very much for opening all this issues since you are finding really interesting points.
In this case it's a duplicate of #63 though.

Happy to announce that #63 is now completely fixed by PR #442.
The relative changes are now in the main branch, and will be officially released in v1.3.0.


Cleaning up the project structure.

As a first refactoring step, we should delete some unused folders. In primis, Zeus and Magnus directories should be removed as those systems are no longer here (for those wondering, note that those files will still be present in the git history, so they are not actually lost).
I also suggest removing the deprecated directory, probably there to act as an archive, but git handles that part for us.
This branch should not be merged yet, but feel free to add suggestions!

@pelahi @ddeeptimahanti I have implemented most of the changes we discussed during the last meeting.
Tomorrow I will create a subdirectory for secondary scripts, and will go ahead and test shpc. Deva or I will create a 'setonix_datamover' system.


Pages failing

https://github.com/weaversa/cryptol-course/runs/6719289492?check_suite_focus=true#step:4:213
Ummm...@WeeknightMVP, seems some triple quoting double curly braces etc. is mucking up the html render. Any ideas?

specifically here:
self.points_to(player, cry_f("""{{ name = repeat 0x41 : [{MAX_NAME_LENGTH}][8],
                                   level = 1 : [32],
                                   hp = 10 : [32],
                                   atk = 5 : [32],
                                   def = 4 : [32],
                                   spd = 3 : [32] }}"""))

Maybe try wrapping fenced code blocks with conflicting ({{ ... }} or {% ... %}) syntax in {% raw %} ... {% endraw %} or {% highlight %} ... {% endhighlight %} tags?

Funny thing, the code block one or two above the broken one looks almost identical and works fine!

@WeeknightMVP your {% raw %} idea worked. The second issue is was the "click here" hidden text block. I removed it and now everything seems good.


Typo in README

On Getting Started you should migrate the database and it says
Then run rails db:migrate
It should say
Then run rake db:migrate

@santiago-su In rails 5 you can run rails db:migrate. rake commands will be removed in future.

Correct. It works in Rails 5 and we recommend you to use the newest version of the framework.


Remove Tilt from list

Tilt has been acquired by Airbnb and will continue payouts from their platform until June 12, 2017. I think it is safe to say that they won't be hiring anyone in the near future.

Thanks for noticing and helping keep this list up-to-date!


Specifying bio_sample_id in SearchReadGroupSets requests is awkward and inconsistent

The current definitions of SearchReadGroupSetsRequest has the following:
  //  Specifying the id of a BioSample record will return only readgroups
  //  with the given bioSampleId.
  string bio_sample_id = 3;

This is currently interpreted in the reference server to mean that (a) we should only return read group sets that contain read groups that refer to this bio_sample_id and (b) for these read groups sets, we filter out all read groups that do not refer to this bio_sample_id. This is semantically very awkward, and inconsistent with other parts of the API.
It would be far simpler if we had a SearchReadGroupsRequest which provided a set of filters on the ReadGroups (including bio_sample_id) and returned a list of ReadGroup objects.

Agreed! Having a read groups endpoint helps to make the API consistent. Let me know what you think of #652.

Looks good to me @david4096, a rethink of the reads API is long overdue IMO.


[FEATURE] Communicate implemented extension interfaces to OpenSearch

Is your feature request related to a problem?
Extensions may implement one or more interfaces beyond Extension.  For example, script engine and context extension points will be implemented on the ScriptExtension.  The ExtensionsManager list of discovered extensions needs to be able to filter that list to determine a subset of extensions which implement specific functionality.
What solution would you like?
Create an Enum as an inner class of the ExtensionsManager with all of the extension interfaces (see #315 for a list of plugins that need corresponding extensions).
Communicate a list of enum values to OpenSearch during initialization and save it in the appropriate object (probably a List<EnumName> on the DiscoveryExtensionNode object).
Do you have any additional context?
Review how various plugin interfaces are filtered in Node.java.  For example, ActionPlugin are sent to the ActionModule to register rest handlers.  ScriptPlugin are sent to the ScriptModule etc.

Hi @dbwiddis, I would like to implement this.  I have some work for next couple of days so I will be touching the code on this weekend if it's okay.
Thanks and Regards
Varun

Hi @dbwiddis,
Can I assign myself this issue?
Thanks and Regards
Varun

Hey @varuntumbe sorry for the delay replying here.  Go ahead and work on this if you like.
Note that at this point none of the other extension interfaces have been written (see #318, #320, #321, #322, #323, #324, #325, #326, #327, #328, #329, #330, #331, #332, and #333).  You probably want to do at least one of those to incorporate into the code logic, and then whoever handles those issues can build on the work you did.
This will require a PR both here and on the main OpenSearch project.

Okay, I will assign myself one of those first. Is there any communication channel like slack or discord incase if I want to clear up some doubts?
Thanks and regards
Varun

Just ask here on Github if you have any questions!  Feel free to tag my username to get my attention faster.

Sure! I have cloned the repo and trying to get it running in local.
Thanks and regards
Varun

I am quite new to Java, as per my understanding it is trying to get some artifacts as it is declared in the repositories section of build.grade
repositories {
mavenLocal()
maven { url "https://aws.oss.sonatype.org/content/repositories/snapshots" }
maven { url "https://d1nvenhzbhpy0q.cloudfront.net/snapshots/lucene/"}
mavenCentral()
}

Hey @varuntumbe I can build fine but see access denied trying to go to that repository.  As a troubleshooting step, try to comment out that lucene snapshot URL in bulid.gradle.  I think the appropriate snapshots are published at the aws snapshot url.
@saratvemulapalli any idea here?

Hi @dbwiddis ,
It did not work. It tries to look for lucene snapshot in .m2/repositories and in other two locations, but it could find those. Let me troubleshoot some more and will get back to you.

Execution failed for task ':compileJava'.
Could not resolve all files for configuration ':compileClasspath'.
Could not find org.apache.lucene:lucene-core:9.5.0-snapshot-0878271.
Searched in the following locations:
- file:/C:/Users/varun/.m2/repository/org/apache/lucene/lucene-core/9.5.0-snapshot-0878271/lucene-core-9.5.0-snapshot-0878271.pom
- https://aws.oss.sonatype.org/content/repositories/snapshots/org/apache/lucene/lucene-core/9.5.0-snapshot-0878271/lucene-core-9.5.0-snapshot-0878271.pom
- https://repo.maven.apache.org/maven2/org/apache/lucene/lucene-core/9.5.0-snapshot-0878271/lucene-core-9.5.0-snapshot-0878271.pom

Hey @varuntumbe I can build fine but see access denied trying to go to that repository. As a troubleshooting step, try to comment out that lucene snapshot URL in bulid.gradle. I think the appropriate snapshots are published at the aws snapshot url.
@saratvemulapalli any idea here?

Usually these artifacts (lucene snapshots) are open to the world. Not sure why @varuntumbe is seeing problems.
@mch2 are these snapshots not open to public?

These snapshots are not published to aws.oss or repo.maven given they are not oss artifacts.  They exist here right now - https://d1nvenhzbhpy0q.cloudfront.net/snapshots/lucene/.  Example:
https://d1nvenhzbhpy0q.cloudfront.net/snapshots/lucene/org/apache/lucene/lucene-core/9.5.0-snapshot-0878271/lucene-core-9.5.0-snapshot-0878271.pom exists & is public.

@varuntumbe @dbwiddis  I believe you will want to list mavenCentral before the snapshot repo like this:
repositories {
    mavenLocal()
    mavenCentral()
    maven { url "https://aws.oss.sonatype.org/content/repositories/snapshots" }
    maven { url "https://d1nvenhzbhpy0q.cloudfront.net/snapshots/lucene/" }
}

So that we aren't searching aws.oss & the lucene snapshot repo for released dependencies.

I was just coming to that conclusion myself!

Hey @varuntumbe thanks for your patience.  Looks like we need to improve our build.  For now, can you try reordering the repositories as indicated in this comment. That should at least resolve the attempt to pull the jackson dependency.   We'll fix that on the repo in any case.

Hey @varuntumbe thanks for your patience.
Looks like our build is pulling a very old 3.0.0-SNAPSHOT version of OpenSearch.  We are continuing to dig into this.

@varuntumbe looks like changing the repo order per this comment will fix things.  I'll update our main branch here, you can update your local copy and hopefully build.

Yes. I was able to get it up and running. Thanks @dbwiddis @saratvemulapalli  @mc2 I will assign myself #331 and later will work on this

Hi @dbwiddis,
So far what I have understood is we need to define one enum datatype consisting of all the existing extension Interface inside ExtensionManager ( which is 1st part, which is an easy part :) ) and then during instantiation of Node class it initializes ExtensionManager based on some setting Flag and it has List as one of the data member  which has information on all discovered extensions. OpenSearch node does the tcp handshake with the each extension which is running on separate node and exchange some messages ( custom settings etc ). In this communication, we need to inform opensearch Node about the interfaces being implemented by an extension ( which is the 2nd part, little tricky ). Did I understand the issue correctly ? or am I missing something ?
Thanks and Regards
Varun

Yep.  I think you have it right!

Hi,
I facing issues with compiling opensearch node locally. I was trying to follow steps from 'To run OpenSearch from a compiled binary:' and I gave this command ./gradlew assemble . This is the error I am getting.

Task :server:compileJava
Build cache key for task ':server:compileJava' is 8511ee27ab378ac8c2824573dd386fe7
Task ':server:compileJava' is not up-to-date because:
Task has failed previously.
The input changes require a full rebuild for incremental task ':server:compileJava'.
Full recompilation is required because no incremental change information is available. This is usually caused by clean builds or changing compiler arguments.
Compiling with toolchain 'C:\Program Files\Java\jdk-17'.
Starting process 'Gradle Worker Daemon 2'. Working directory: C:\Users\varun.gradle\workers Command: C:\Program Files\Java\jdk-17\bin\java.exe --add-opens=java.base/java.util=ALL-UNNAMED --add-opens=java.base/java.lang=ALL-UNNAMED @C:\Users\varun.gradle.tmp\gradle-worker-classpath1213878160051673014txt -Xmx3g -Dfile.encoding=windows-1252 -Duser.country=IN -Duser.language=en -Duser.variant worker.org.gradle.process.internal.worker.GradleWorkerMain 'Gradle Worker Daemon 2'
Successfully started process 'Gradle Worker Daemon 2'
Started Gradle worker daemon (0.376 secs) with fork options DaemonForkOptions{executable=C:\Program Files\Java\jdk-17\bin\java.exe, minHeapSize=null, maxHeapSize=3g, jvmArgs=[], keepAliveMode=SESSION}.
Compiling with JDK Java compiler API.
F:\Open_Source_Projects\OpenSearch\server\src\main\java\org\opensearch\index\engine\EngineConfigFactory.java:77: error: cannot find symbol
codecService = enginePlugin.getCustomCodecService(idxSettings);
^
symbol:   method getCustomCodecService(IndexSettings)
location: variable enginePlugin of type EnginePlugin
F:\Open_Source_Projects\OpenSearch\server\src\main\java\org\opensearch\index\engine\EngineConfigFactory.java:79: error: cannot find symbol
} else if (enginePlugin.getCustomCodecService(idxSettings).isPresent()) {
^
symbol:   method getCustomCodecService(IndexSettings)
location: variable enginePlugin of type EnginePlugin
Note: Some input files use or override a deprecated API.
Note: Recompile with -Xlint:deprecation for details.
Note: Some input files use unchecked or unsafe operations.
Note: Recompile with -Xlint:unchecked for details.
2 errors

Task :server:compileJava FAILED

FAILURE: Build failed with an exception.
I have synced my fork to the latest and pulled latest to local as well. I dont see this getCustomCodecService method in EnginePlugin class.
What can I do regarding this ? I need to start opensearch locally so that I can debug the application to understand the transport communications with opensearch SDK
Thanks and Regards
Varun

A few ideas:

make sure you have enabled the feature flag.  In the gradle/run.gradle file add the following under runTask:

systemProperty 'opensearch.experimental.feature.extensions.enabled', 'true'

try with ./gradlew run rather than assembling.

Hi,
I have done the 1st step and it gives the same error for run as well

I think I had deleted here as well by mistake, I did the fresh clone now and its coming up, but one last question, where do we have to place extension.yml file ? I have placed it under extension folder under project root dir. its not finding the yml file while loading
[2023-02-03T00:01:27,043][INFO ][o.o.e.ExtensionsManager  ] [runTask-0] ExtensionsManager initialized
[2023-02-03T00:01:27,043][INFO ][o.o.e.ExtensionsManager  ] [runTask-0] Loading extensions : F:\Open_Source_Projects\OpenSearch\build\testclusters\runTask-0\distro\3.0.0-ARCHIVE\extensions
[2023-02-03T00:01:27,044][WARN ][o.o.e.ExtensionsManager  ] [runTask-0] Extensions.yml file is not present.  No extensions will be loaded.

When you start up OpenSearch it will have a log entry telling you the extensions config dir.  Mine looks like:

[2023-02-02T09:29:13,071][INFO ][o.o.e.ExtensionsManager  ] [runTask-0] Extensions Config Directory :/Users/widdisd/git/OpenSearch/distribution/archives/darwin-tar/build/install/opensearch-3.0.0-SNAPSHOT/extensions

I actually have copies of the extensions.yml and run.gradle files in my git root and a bash file in my home directory that I execute to start up OpenSearch and re-copy the files over (they sometimes get erased during a recompile):
#!bash

mkdir -pv /Users/widdisd/git/OpenSearch/distribution/archives/darwin-tar/build/install/opensearch-3.0.0-SNAPSHOT/extensions
cp -v ~/git/extensions.yml /Users/widdisd/git/OpenSearch/distribution/archives/darwin-tar/build/install/opensearch-3.0.0-SNAPSHOT/extensions
cp -v ~/git/run.gradle ~/git/Opensearch/gradle
./gradlew run

This is helpful, yes I had copied to that location it was getting erased. I will make a batch file or equivalent to this.
Thank you.

Hi @dbwiddis ,
I have added Request and RequestHandler classes in Open Source Extension package, those classes not getting imported locally for open search SDK ( I am still trying to understand how it works ), i will get this by this week
Thanks and regards
Varun

Note the extensions have their own version of ExtensionRestRequest and ExtensionRequestHandler. Not sure which request and handler you're dealing with but if you can be a bit more specific I can offer more specific advice.

Yes, I have created a separate classes for Request and Request handler under extension package for registering interfaces. But after defining everything on the open search side, i need to register the request in SDK side in extensionRunner.java, here I am not able to reference ExtensionsManager.<My_New_Defined_Req> or anything that i have defined in the open search side.
I thought ./gradlew publishMavenToLocal on open search ( which adds all the artifacts and jars to .m2/repositories/.. ) and running gradlew build --refresh-dependencies on SDK would fix it. But it did not work. Did I miss anything here ?
Thanks and regards
Varun

Yes, making the change on your OpenSearch branch and using publishToMavenLocal on that branch should work (if it built successfully) to make those classes available on the SDK side.  Can you double check visibility of the classes you're adding?

Yes I published to MavenLocal and in sdk I reloaded the workspace and its taking changes. I tested it as well now, its working. I will do a commit today or by tomorrow
Thanks and Regards
Varun


Remove community.kubernetes dependency for ansible-operator

Description of the change:
Remove community.kubernetes dependency for ansible-operator
Motivation for the change:
Cannot test the ansible-operator with the kubernetes.core collection.

/cc @fabianvf @asmacdo

Hi, @alinabuzachis,
Thank you for your contribution 🥇
It seems open for a while without interactions, so I hope that you do not mind if we close this one. However, if you still thinking that we would need to move forward here please feel free to re-open or reach out. Also, in this case could you please check https://github.com/operator-framework/operator-sdk/pull/4982#pullrequestreview-741049487 ?


chore(docs): add note on new initializer

updates the getting started guide to discuss initializer instead of constructor

Docs Preview
Hey there! 👋 You can check your preview at https://65f35c2e306343284d709e00--aztec-docs-dev.netlify.app


what is .include() for and what should it do?

OrientDB Version: 2.2.17
Java Version: 1.8.0_74
OS: macOS Sierra (10.12.3 - 16D32)
Expected behavior
I thought SELECT @this.include(nome) FROM Persona should behave similar to SELECT nome FROM Persona and give something like:
 {
     "result": [
         {
             "@type": "d",
             "@rid": "#-2:0",
             "@version": 0,
             "nome": "Marco"
         },
         {
             "@type": "d",
             "@rid": "#-2:1",
             "@version": 0,
             "nome": "Luigi"
         }
     ]
 }

Actual behavior
The value of "nome" field becomes the key in new records, making impossible to .include() @rid and other LINK fields, as ":" is not allowed in field names.
 {
     "result": [
         {
             "@type": "d",
             "@rid": "#-2:0",
             "@version": 0,
             "this": {
                 "@type": "d",
                 "@version": 0,
                 "Marco": null
             }
         },
         {
             "@type": "d",
             "@rid": "#-2:1",
             "@version": 0,
             "this": {
                 "@type": "d",
                 "@version": 0,
                 "Luigi": null
             }
         }
     ]
 }

Steps to reproduce
CREATE CLASS Persona
CREATE PROPERTY Persona.nome STRING
INSERT INTO Persona (nome) VALUES ('Marco'), ('Luigi')
SELECT @this.include(nome) FROM Persona

hi @UltraMarco
from the example in the docs, you should use strings instead of identifiers.
http://orientdb.com/docs/2.2/SQL-Methods.html
SELECT @this.include('nome') FROM Persona

Hope it helps


Wallet Access

I am trying to access my wallet i created with the old Neo-Gui . Its a DB3 file , i have this saved on a usb but no longer have access to the Neo Database . I have downloaded the new Neo_gui but it will not accept my db3 file only wants json file .
Any information to access this wallet would be greatly appreciated .

In the Open Wallet dialog box, the file format can be selected in the lower right corner.

Hey erik thanks for messaging back , im still unable to see anything on that tab is there any other ways to access the wallet . See picture attached

Where did you download your software?

The new software came from neo.org

The new software came from neo.org

The new gui is for neo N3 and N3's  address is incompatible with neo legacy.  Recommand you to download neo-gui v2.12.2 or using web wallet like O3 or neonline directly.

Thank you i will give that a try


Queries are slow?

I am using AWS and mLab. I upgraded mLab to the first option but it did not make a difference, so that means that the open source parse server is slow. When I was on parse.com it was a lot faster so I am not sure what changed? If any one could help that would be great!

The hardware you use on AWS, and the indexes you have on your database also make a difference. If you can provide a profiling trace or database query (or series or database queries) that you think could be faster, that would be welcome, but "everything is slow" isn't really something we can debug.

@drew-gross I have ran it on my computer on localhost and it was the same speed. Is this a issue many people face or is this just me.


Added 7 comments on core_rnn_cell_test.

I added 7 comments on core_rnn_cell_test.
Thanks!

Can one of the admins verify this patch?

hi @chris-chris we usually try to reflect the content of tests in test names. I will close this PR for consistency. Thank you for your contribution!


[ZEPPELIN-2647] Bypass auth logic when a user logins as admin role

What is this PR for?
For administrator, make new admin role that assigned user can see all notebooks.
What type of PR is it?
Improvement
What is the Jira issue?
https://issues.apache.org/jira/browse/ZEPPELIN-2647
How should this be tested?

Set role name to use as admin through ZEPPELIN_OWNER_ROLE =  or zeppelin.owner.role = .
Default role name is admin
Login as user who is not assigned as admin and create notebook.
Logout the user and login another user who is assigned as admin, open the created notebook.

Questions:

Does the licenses files need update? N
Is there breaking changes for older versions? Y/N
Does this needs documentation? Y

continued from #2525 #2416

Thanks for the contribution @yu74n
Looks Good to Me. Let me test more cases and then left feedback soon.

LGTM.

I wanted to merge now. But cound't find the build history.

https://builds.apache.org/job/zeppelin-pull-request/2960/

@yu74n if you don't mind close and reopen this issue to trigger jenkins?

@1ambda Sure

The 3/4 jobs failed. Please rebase this PR based on master. And Could you resolve them?

I rebased and ran test again, but this test is still failed.
https://travis-ci.org/yu74n/zeppelin/jobs/286508544
I think probably my PR doesn't affect the test.
I ran test several times, but failed tests are changed.

Hi, Thanks for rebasing and testing. But this recent PR passed the CI.
If you don't mind please re-run few times until successful and check the failure reasons?
If we can verify they are a flaky test, we can merge, but without evidence, we can't do that.

test

Rerun all tests.
'testPerUserScopedAction' is passed this time, but other 4 tests is failed.
These 4 tests was passed before.

All tests are passed, but NotebookTest has an error.
java.lang.RuntimeException: org.apache.thrift.transport.TTransportException: java.net.SocketException: Connection reset

and rerun the failed travis job manually, the job is passed.
https://travis-ci.org/yu74n/zeppelin/jobs/295676369

What do you think about that? @1ambda @felixcheung

@felixcheung if unset zeppelin.notebook.default.owner.username, set 'admin' as default value.
https://github.com/apache/zeppelin/pull/2585/files#diff-179ea3a816210bb9904baabec1a552e4R716

I've changed default value is blank.
if set blank as zeppelin.notebook.default.owner.username, disable admin role.

ok, could you explain how

if set blank as zeppelin.notebook.default.owner.username, disable admin role.
?
I think also it'll be good to match template to the default value

if set blank as default owner username, apply blank('') to the admin role. but, I think, a user cannot set blank as login user name. so nearly equal as disable admin role.

hmm, since this is around security perhaps we need to be more careful?
how about we add a check for the owner username is valid (at least not "") and not add to the admin role if it is not valid?

Actually, I thought so too when I explained how if the owner name is blank. I've fixed that.
I've changed config template to be disable admin role.
Can you review that again?
@felixcheung

could you check if it's related? https://travis-ci.org/yu74n/zeppelin/builds/299517426

CI failed at testRunOnSelectionChange()
java.lang.AssertionError: Even if 'RunOnSelectionChange' is set as false, still can run the paragraph by pressing ENTER
Expected: "My selection is 1"
but: was "My selection is 2"

This test is related to 'Run on selection change' checkbox in paragraph menu.
I think the reason why that test is failed would be timing error,  cause of the test code ran the paragraph before the checkbox was unchecked. Then I think the test is not related to my PR.
I rerun the CI job manually, all tests are passed.
https://travis-ci.org/yu74n/zeppelin/builds/299517426

ok thanks

merging if no more comment


ci: set matrix strategy fail-fast to false

Description
This pull request changes the following:

set matrix strategy fail-fast to false

Related Issues

Closes #696

:tada: This PR is included in version 0.31.1 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This PR is included in version 0.31.1 :tada:
The release is available on:

npm package (@release-0.31.x dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Add illumination and seismic Intensity entities to the list of sensors.

Some sensors like Fibaro Motion or Aeotec Multi-sensors have the entities for illumination and seismic intensity.
-->
Checklist

[x ] I checked for similar existing requests (both open and closed) before posting.
[x ] My request is generic, other users may benefit from it too.

Proposal
I think it would be beneficial to have these type entities available as sensors in Alarmo.
Additional info
Maybe set up the same as a door or motion sensor.

Any binary_sensor can be added.
But I don’t see specific types defined for seismic / illumination sensors: https://www.home-assistant.io/integrations/binary_sensor/
This list is used for the support within alarmo .

I should have known that about binary, it is plain as day. I apologize for jumping the gun. I'll think before I leap next time.
Thank you all for the good work and your patience with me.


[Lint] Fixes some trivial SwiftLint warnings

Description
This is the first follow-up smaller PR to https://github.com/corona-warn-app/cwa-app-ios/pull/104. Together with the other PRs I will be posting, all SwiftLint warnings should be fixed to allow us to enable the --strict mode of SwiftLint on the CI to prevent any new warnings getting merged in the future.
Related PRs
https://github.com/corona-warn-app/cwa-app-ios/pull/518, https://github.com/corona-warn-app/cwa-app-ios/pull/522, https://github.com/corona-warn-app/cwa-app-ios/pull/527, https://github.com/corona-warn-app/cwa-app-ios/pull/531, https://github.com/corona-warn-app/cwa-app-ios/pull/534.
Note that all these PRs are completely independent and can be merged in any order.

Thank you very much for this PR. I will put this on hold for the next couple of days because we are in a critical phase in the project at the moment. I hope you understand. We will come back to you once this phase is over.

@ChristianKienle Honestly? I'm waiting since 11 days now (see https://github.com/corona-warn-app/cwa-app-ios/pull/104) to get these SwiftLint warnings fixed and I'm not even changing any logic here. I decided to start with this because it's a no-brainer to get them merged, but if even these PRs are being put on hold, I'm really worried about the overall community aspect of this project.
I wonder why you even put this into the README then before the first public release:

At the same time our commitment to open source means that we are enabling -in fact encouraging- all interested parties to contribute and become part of its developer community.

It's clearly not true as of yet which I have seen in these PRs and also from how https://github.com/corona-warn-app/cwa-app-ios/issues/106 was handled.

@Jeehut
Sorry for the late response. We decided to close this PR due to two Reasons:

The PR can`t be merged since there have already been changes in our SwiftLint config
We will clean up our Project/SwitLint Rules in the next 2 Week-Sprint (starting 30th Sept.) with the Goal to have 0 Warnings coming from SwiftLint.

Sorry again for the late response and the now resulting close of this PR.


[CRITICAL] Simplecast integration is broken

Problem
This repo contains a custom gatsby plugin (/plugins/gatsby-source-simplecast) to gather our podcast episodes and display them as a list (https://frontside.com/podcast/) and pages for each podcast episode (ex. https://frontside.com/podcast/open-telemetry-with-austin-parker).
We had used simplecast v1 by recommendation of Mandy, our podcast producer. However, simplecast sent us an email saying that they would deprecate completely v1 and move our show to v2, which has a different UI and API. Therefore, our integration gastby plugin broke down.
No deployments of the frontside website can happen until this bux is fixed, as it will lead to a build failure
Solution
The solution that comes to mind first is refactoring the plugin to use simplecast v2 API. However, I don't know how v2 API looks like so I can't tell how much effort it'll take, or if it's even a feasible.

After digging into this, there is some good news, but mostly bad news.
The good news is that I kinda got it working,
The bad news is that new simplecast API doesn't have author information on the  /episodes REST collection like the old API did. That means that instead of doing one query for our entire catalog, we have to do N+1 queries. In our case this is about 150 queries, which is quite a lot, and slows down the time to render the site considerably.
The other bad news is that this number of queries pushes us over the default rate limit threshold and so we can't actually build the site without introducing artificial delays into the fetch. I've spoken with simplecast about increasing our limit.
And finally, there is a third piece of bad news. The slug format has changed between their legacy api and their new api, and not for the better. Descriptive and SEO friendly slugs have been replaced by a hexidecimal number.
For example, the slug for our latest podcast in the legacy api was open-telemetry-with-austin-parker. Now, it's 509620d7 which means our previously search engine optimal /podcast/open-telemetry-with-austin-parker is now the far less google juice-y /podcast/509620d7. But worse, it means that we need to set up redirects from the old url to the new.

Note: we could just start generating our own slugs within gatsby, but we'd still need to manually verify each redirect since there'd be no guarantee that their algorithm and ours would arrive at the same result for every podcast.

I think our gatsby architecture needs a re-think. It's not good that we have source data which is now effectively lost to posterity via the sunsetting of the legacy API. We should never dynamically pull resources from the network and use them directly as a gatsby source. Instead, we should use a script to update a file on disk that is under source control and then source pulls from that static file. This has several benefits

You can always render the site, even when you're not connected to the network, or when 3rd party services like simplecast go down.
Development is faster, because 99% of the time you aren't even working on network related aspects of the site, so why should you have to query simplecast?
If we find ourselves in a situation like this one where we can no longer access the 3rd party service, we nevertheless have a record of what the old value was.

In summary, the new gatsby rule should be "Thou shalt check into source control everything that is needed to render the site"
So here's we have to do now:

[ ] fix the rate limiting problem
[ ] implement the slug algorithm on our gatsby code since it is no longer implemented on simplecast
[ ] manually check each url and set up a redirect for it if it's different

What we can do in the future: use a webhook to trigger a workflow_dispatch event that downloads and submits a PR for the podcast content.

Progress Update:
It sounds as if our rate limit was increased for the time being so we can hopefully brute force query for the time being.
I began work on writing the query out to a file, and building the gatsby nodes from a file. I believe that is working, but it appears that the API key on netlify is not working for me? I receive a 503 Service Temporarily Unavailable whenever I try to query it with the key. Even a single query fails with this error. I am also unable to login with the credentials in 1Password so I think I will need some of your assistance with this @cowboyd.
It looks like we have the slugifying working in the code Charles committed. I used the sitemap on the production version of our website, and some nushell piping to grab a list of all of the podcast URLs. Whatever we end up with, I can set up redirects for the ones that don't conform. It looks like some of the old URLs had the episode number in it and the new URLs do not. I presume we want to remove the episode number from all of the old URLs?
Simplecast doesn't have any webhooks so the next best option to trigger a website feed update / publish is likely through something like zapier, n8n or a cron job somewhere. It would read the XML from simplecast for changes, and hit a workflow dispatch in github actions to PR the change to the json file. Speaking of the rss feed, it seems that simplecast is having us update the URL for our feed. As we request adding it to other services, I think it would be prudent to have a URL on our website that redirects to their RSS feed. This would give us the option to more easily make host adjustments if necessary.


Add option to make angle yaw not around gravity

Add an option to make angle mode yaw act on the quads yaw rather than gravities yaw. This will add an option to allow for more of an Emu/BF feel to angle mode. Won't effect how well QS handles crashes or any other angle mode improvements QS has made. From personal experience Emu users tend to prefer this yaw feeling compared to the yaw around gravity feeling. Would be a nice option to help with adoption. And no this isn't an effort to make this like BF or to copy BF. Its simply an effort to allow me to more happily convert my whoops over to QS, haha.

if this is to be a feature to increase adoption by removing migration churn, it should really be a runtime configurable thing.

if this is to be a feature to increase adoption by removing migration churn, it should really be a runtime configurable thing.

Any standard operating procedure for making something runtime configurable?

if this is to be a feature to increase adoption by removing migration churn, it should really be a runtime configurable thing.

Any standard operating procedure for making something runtime configurable?

add a new member to the profile struct, both in the struct itself and in the appropriate *_MEMBERS define.
although not perfectly fitting, the pid section is probably most appropriate.
https://github.com/BossHobby/QUICKSILVER/blob/master/src/main/config/profile.h#L124
this will make it available in the variable profile.pid.<whatever for you to switch on in the codebase and automatically generate the cbor encoding/decoding code to expose it via usb.
this will already allow you to change the value in the profile*.json file as downloaded via the config, adding a new toggle input to the configurators vue code would be the next step

this will already allow you to change the value in the profile*.json file as downloaded via the config, adding a new toggle input to the configurators vue code would be the next step

I'm a complete noob when it comes to dealing with configurators, but i can see what I can do, haha, thanks for pointing me in the right direction.

this will already allow you to change the value in the profile*.json file as downloaded via the config, adding a new toggle input to the configurators vue code would be the next step

I'm a complete noob when it comes to dealing with configurators, but i can see what I can do, haha, thanks for pointing me in the right direction.

i can take care of that part in an upcoming relase, would be good tho if the firmware side was already done tho.

Silverware originally maintained yaw around the quads Z axis.  The change
was made to yaw around the gravity vector because when a yaw is performed
around the z axis - it just dumps a ton of orientation error into roll and
pitch.  The gravity vector system was created to alleviate the system of
being flooded with error every time a yaw manuver is performed.  This
change was a massive improvement to angle mode performance and handling.
It is also worth noting that angle pid system uses angle error to move a
setpoint weight between two sets of angle pids (high and low error).
Flooding roll and pitch axis with error is going to also cause a response
by the angle pid system as this setpoint weight will shift on roll and
pitch with every yaw input until the dumped error is eliminated.
The end result of yaw around gravity vector vs Z axis is going to be the
same eventually anyway.  The quad will find the same orientation in both
cases eventually.  Yaw around gravity will use math to get to the right
spot faster and with less error in the system - yaw around Z axis will rely
on multiple loop cycles gradually moving the quad to eliminate the yaw
dumped error to get to the same place.
This approach seems inadvisable to me.
On Wed, Dec 15, 2021, 12:22 PM Benedikt Kleiner @.***>
wrote:

this will already allow you to change the value in the profile*.json file
as downloaded via the config, adding a new toggle input to the
configurators vue code would be the next step
I'm a complete noob when it comes to dealing with configurators, but i can
see what I can do, haha, thanks for pointing me in the right direction.
i can take care of that part in an upcoming relase, would be good tho if
the firmware side was already done tho.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/BossHobby/QUICKSILVER/pull/40#issuecomment-995002554,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AIRZCHPSUUW2C5LQYFLCSCLURDFF3ANCNFSM5KCO4WKQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

This approach seems inadvisable to me. It is going to create unpredictable impact to flight feel, crash handling, and the quad's ability to stabilize itself based on user set angle pids.

I disagree, I've tested both approaches before. It does not create an unpredictable flight feel, in fact for many users they feel that yaw feels more predictable. This is simply an option given to users and should not create any problems for users choosing it over the traditional method. Some will find it preferable, others will not. I'm also looking to change how the code calculates its angle error, removing some issues that euler angles bring to play, and a yaw around gravity type approach isn't the most compatible with this. So I definitely feel this is worth letting users decide what they want.

Every yaw input is going to change the active roll and pitch angle pids.
This is going to impact stick feel.  It will create unpredictable results
for different users because different users run different values for these
and because different amounts of error will be dumped into roll and pitch
based on different amounts of yaw command or different preceding roll and
pitch orientations when yaw is commanded.  You're going to need to toss out
the high error/low error angle pid system (and its resulting improved crash
recovery) along with this.
On Wed, Dec 15, 2021, 2:02 PM Kevin Plaizier @.***>
wrote:

This approach seems inadvisable to me. It is going to create unpredictable
impact to flight feel, crash handling, and the quad's ability to stabilize
itself based on user set angle pids.
I disagree, I've tested both approaches before. It does not create an
unpredictable flight feel, in fact for many users they feel that yaw feels
more predictable. This is simply an option given to users and should not
create any problems for users choosing it over the traditional method. Some
will find it preferable, others will not. I'm also looking to change how
the code calculates its angle error, removing some issues that euler angles
bring to play, and a yaw around gravity type approach isn't the most
compatible with this. So I definitely feel this is worth letting users
decide what they want.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/BossHobby/QUICKSILVER/pull/40#issuecomment-995094349,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AIRZCHJ6VYEKWFUTPY4BZ5LURDQ3ZANCNFSM5KCO4WKQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

Every yaw input is going to change the active roll and pitch angle pids.
This is going to impact stick feel.  It will create unpredictable results
for different users because different users run different values for these
and because different amounts of error will be dumped into roll and pitch
based on different amounts of yaw command or different preceding roll and
pitch orientations when yaw is commanded.  You're going to need to toss out
the high error/low error angle pid system (and its resulting improved crash
recovery) along with this.

That's totally wrong. I have used this exact system before with the high error/low error pid system and ended up with incredible crash recovery. I rewrote the way that angle mode works in emu, and took a bunch of ideas from silverware. This will work and some users will like it. Some will not. This doesnt make angle mode unflyable all of a sudden. Itll still handle crashes and itll still be predictable on the sticks. I do even think that this would help with the yaw problem mr shutterbug mentioned a while back, given that he is used to this style more. There is nothing wrong about this version or the way quicksilver currently does things. This is just some more options for users that may want things to feel a little different.

Let's make one thing very clear:
As this is an optional thing, all changes in this or following PRs can not degrade the performance of the gravity vector based approach, that one is definitely here to stay, and will stay the primary one going forward.
Furthermore i am not keen on building two distinct and competing systems in one firmware. However obviously there different schools of thought on how the rotation around yaw should work so i am fine with having two variants for that, but everything down the line must be made in a way that it works for both.

Furthermore i am not keen on building two distinct and competing systems in one firmware.

I don't either. This is really just a simple option, if some users like it, then I think it has a place. If not then it should go the way of the dodo.

Closing as outdated. If we were to got forward, we can pull changes manually.


Bug: Using /ride to mount an entity results in desync

Using /ride in context below does not visually teleport the rider as riding the specified entity.
/ride @e[type=pig,limit=1,sort=nearest] start_riding @p teleport_rider if_group_fits
Using evict_riders afterwards does display the entity disembarking from the mount though

Hmm, must be due to the way players riding entities is different to entities riding entities. I'm not sure if players support riding but I'll look into it

We have ride in vanilla now
swagalicious signing out


CLOUD-797 fixed status change

@keyki

Jenkins build finished, all tests passed.
Refer to this link for build results: http://ci.sequenceiq.com/job/cloudbreak-pull-request/1308/


Release

Hi :)
Is there a possibility to make a new release? There wasn't any since 1.8.1 (18 May 2016).
After that release there was a fix for EmptyDataSetView offset issues, that ensures correct origin of the view:
https://github.com/dzenbot/DZNEmptyDataSet/pull/293
There are some issues on gh regarding this problem. Is there any reason why it cannot be included in official release?

yep


Error when trying to issue Let's Encrypt certificate.

I am using Cloudflare tunnel on my server, and in its yaml conf I have pointed the internal IP of the NPM container. Currently I am using origin certificate from Cloudflare to access my apps with https, which I have added with the "Custom" option. However, I am unable to issue a let's encrypt certificate and I get this error:
There is a server found at this domain but it returned an unexpected data. Is it the NPM server? Please make sure your domain points to the IP where your NPM instance is running.
My domain doesn't point to an IP but to the tunnel id.
What can I do to fix this?

Try disabling proxy (yellow icon on cloudflare dns) or using API key from Cloudflare.

I'm trying to set-up the same. uing Cloudflare as DNS provider. Is there a manual or how-to on how to set this up? I Checked the installation manuals, but couldn't find anything. Disabling the Proxy on Cloudflare didn't help me, as I get the same error.

Hello,
This is an Automated message.

This email will stop working soon!
Please, use the following email<EMAIL_ADDRESS>Thank you and have a nice day!

Здравейте,
Това е автоматично съобщение.
Скоро, този имейл няма да работи!
Моля, използвайте следния имейл<EMAIL_ADDRESS>Благодаря и приятен ден!

--
  CBA Bulgaria & BG MARKET
  Volodya Evtimov
  m: +359 896 841 665
  e<EMAIL_ADDRESS>  w: cba.bg & bgm.bg
  a: blvd. Vasil Levski 61, 2700 Blagoevgrad, Bulgaria


Query: use of hardcoded values

Hi, this is more of a general query, than anything else.
I noticed that in the AdamOptimizer class in apply_grad there was a hardcoded value (1e-8) added to the result of an operation (which I assume is to prevent values exploding in the division). Would it make more sense to use a machine epsilon value here, such as np.finfo(float).eps or np.finfo(np.float32).eps?
I don't expect it would make much difference to any result, and removes the use of arbitrary values. If the value is chosen for a specific reason, then feel free to ignore this. Thanks!
https://github.com/XanaduAI/pennylane/blob/4aeb842f09ec3795ef9208ace13fc7c46e60d43d/pennylane/optimize/adam.py#L94

Hi @mlxd, yes that numerical value would be there to ensure that the denominator doesn't explode for small s values.
@mariaschuld can comment why that specific value was chosen, since she wrote that section of the code

Closed with PR #132


Flush socket on write automatically

This change is

r? @canndrew
(maidsafe_highfive has picked a reviewer for you, use r? to override)

Reviewed 4 of 4 files at r2.
Review status: all files reviewed at latest revision, 1 unresolved discussion.

Comments from Reviewable


Minor spelling fixie

Thanks!

Thank you for the great repo! saved me lots of trouble :D

Good to know it's still relevant - haven't updated in awhile.


protractor.getInstance() is undefined

var ptor;
beforeEach(function () {
browser.get('http://localhost:8383/v2');
ptor = protractor.getInstance();
});
.getInstance() is undefined

Please read the contributing guidelines; these types of questions are ill-suited for GitHub. Thanks!
FYI, http://stackoverflow.com/questions/27690814/object-object-has-no-method-getinstance-when-trying-to-run-protractor-tests


use my own API to upload file rather than begin_write and end_write

For my case, I already have an API, an Python function api_upload(path), to implement write. For example, a file with local path filepath="/home/foo/1.png", can be "writen" directly by  a function call api_upload(path=filepath). That is all. How can I simplify begin_write and end_write?

Have a look at FileLikeQueue

See also #253


Fonts on Sway

i'm trying to use unifont
hint_font: unifont and hint_font: Unifont aren't working
no fonts are working, i can't change the default one

font is currently not configurable on wayland. I'll release a patch for
this when I have some time, but be aware wayland is generally not well
supported and has several warts.
On Fri, Jun 17, 2022 at 7:55 PM DNDEBUG @.***> wrote:

i'm trying to use unifont
hint_font: unifont and hint_font: Unifont aren't working
—
Reply to this email directly, view it on GitHub
https://github.com/rvaiya/warpd/issues/137, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAOP7GNTLREOM4UP3MJ3VHTVPUF5TANCNFSM5ZDX4NEA
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

This should work in the latest commit.


Explorer sidebar opens on new file if activity bar is hidden, after latest update

Type: Bug
I've had VSCode set to a rather minimal layout with only text mostly, for over a year.
After the latest update, whenever I open it, the 'Explorer' opens on the sidebar.
I've reset my settings and the issue persisted, so I began adding to the settings.json one by one, until I've hit the offending setting.
It seems to be "workbench.activityBar.location": "hidden",
And here's my current settings.json
{
    "workbench.editorAssociations": {
        "{git,gitlens}:/**/*.{md,csv,svg}": "default"
    },
    "editor.minimap.enabled": false,
    "breadcrumbs.enabled": false,
    "workbench.startupEditor": "newUntitledFile",
    "window.restoreWindows": "none",
    "workbench.editor.untitled.labelFormat": "name",
    "workbench.editor.restoreViewState": false,
    "files.hotExit": "off",
    "window.closeWhenEmpty": true,
    "workbench.activityBar.location": "hidden",
}

Expectation: VSCode will open without 'explorer' sidebar
Reality: VSCode will open with an 'explorer' sidebar
Reproduction: Close vscode entirely. Open vscode, witness that it opens with an explorer sidetab.
I've also tried all associated settings I could find like "explorer.autoReveal": false, but they had no effect on mitigating this issue.
VS Code version: Code 1.94.0 (d78a74bcdfad14d5d3b1b782f87255d802b57511, 2024-10-02T13:08:12.626Z)
OS version: Windows_NT x64 10.0.19045
Modes:

System Info

Item
Value

CPUs
Intel(R) Core(TM) i9-10900K CPU @ 3.70GHz (20 x 3696)

GPU Status
2d_canvas: enabledcanvas_oop_rasterization: enabled_ondirect_rendering_display_compositor: disabled_off_okgpu_compositing: enabledmultiple_raster_threads: enabled_onopengl: enabled_onrasterization: enabledraw_draw: disabled_off_okskia_graphite: disabled_offvideo_decode: enabledvideo_encode: enabledvulkan: disabled_offwebgl: enabledwebgl2: enabledwebgpu: enabledwebnn: disabled_off

Load (avg)
undefined

Memory (System)
127.88GB (112.07GB free)

Process Argv
--crash-reporter-id 6cf1cdf9-0be8-4dac-b5a5-b9e955d4dbd9

Screen Reader
no

VM
0%

Extensions (23)

Extension
Author (truncated)
Version

vscode-office
cwe
3.4.3

dart-code
Dar
3.98.0

flutter
Dar
3.98.0

gc-excelviewer
Gra
4.2.62

blender-development
Jac
0.0.24

csdevkit
ms-
1.10.18

csharp
ms-
2.45.25

vscode-dotnet-runtime
ms-
2.1.7

debugpy
ms-
2024.10.0

isort
ms-
2023.10.1

python
ms-
2024.16.0

vscode-pylance
ms-
2024.9.2

jupyter
ms-
2024.9.1

jupyter-keymap
ms-
1.1.2

jupyter-renderers
ms-
1.0.19

vscode-jupyter-cell-tags
ms-
0.1.9

vscode-jupyter-slideshow
ms-
0.1.6

remote-wsl
ms-
0.88.4

cpptools
ms-
1.21.6

vscode-yaml
red
1.15.0

cmake
twx
0.0.17

intellicode-api-usage-examples
Vis
0.2.8

vscodeintellicode
Vis
1.3.1

A/B Experiments
vsliv368cf:30146710
vspor879:30202332
vspor708:30202333
vspor363:30204092
vswsl492cf:30256860
vscod805cf:30301675
binariesv615:30325510
vsaa593:30376534
py29gd2263:31024239
c4g48928:30535728
azure-dev_surveyone:30548225
a9j8j154:30646983
962ge761:30959799
pythongtdpath:30769146
welcomedialog:30910333
pythonnoceb:30805159
asynctok:30898717
pythonmypyd1:30879173
h48ei257:31000450
pythontbext0:30879054
accentitlementst:30995554
dsvsc016:30899300
dsvsc017:30899301
dsvsc018:30899302
cppperfnew:31000557
dsvsc020:30976470
pythonait:31006305
dsvsc021:30996838
0ee40948:31013168
a69g1124:31058053
dvdeprecation:31068756
dwnewjupytercf:31046870
impr_priority:31102340
nativerepl1:31139838
refactort:31108082
pythonrstrctxt:31112756
flighttreat:31134774
wkspc-onlycs-t:31132770
wkspc-ranged-t:31151552
cf971741:31144450
autoexpandse:31146404
iacca2:31150323
notype1:31151523
cc771715:31146322

seems related to my issue: https://github.com/microsoft/vscode/issues/230489

This is very annoying issue without any workaround that takes 25% of the edit area for no reason.
I'm using vscode for editing everything like random txt, ini, json files and in 95% of cases I have to close this sidebar as it only contains Open Folder and Clone Repository buttons.
For me it's also annoying for workspaces as I'm using this sidebar very rare because I'm using breadcrumb navigation in most cases.
Also for me when I open vscode then it restores ~5 vscode windows because I have opened more workspaces at once like dotfiles, current project, another random txt file, or ahk workspace and it everywhere opens this sidebar, very annoying for geek users.
The best solution would be to save it to the state per workspace and also add a setting to control this at startup for new vscode windows, or at least restore the previous behavior. I hope this will not be the new standard like a regression in search #206511

@silverqx as a temporary workaround, maybe have a look at https://marketplace.visualstudio.com/items?itemName=sirmspencer.vscode-autohide . The sidebar will still be open for a moment, but then gets hidden automatically.

Thx @samcarter, I know about this extension but I don't like it either, the auto-hide when you click on the edit area is also weird, I might give it a shot, I will hope that this will be fixed soon.

Thanks for reporting, I will be reverting the change which is causing this to happen.

Thx for asap fix, I'm happy about it. 👍

Thank you so much for reverting! Can't wait for the fix :). Been bugging me for weeks!
I just got frustrated enough to file an issue, but it looks like multiple others did for me and even fixed the issue ;).

This is still happening for me on 1.95.0.
I think it's less about the sidebar than it is about Explorer. I say this because sometimes when I have the sidebar open for Search, it will jump to Explorer for no apparent reason. If the sidebar is closed, it will open as described in this thread.

I can confirm this is fixed in Windows 11 64-bit 23H2 in 1.95. I had the sidebar closed, and when I opened up VS Code after updating, the sidebar was still closed.
I tried this a few times, and the results are the same. Fantastic! Thanks for this fix!

It's okay in v1.95. The sidebar open state is even persisted, which is great; I like it. 🕺


Enable osx

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

Needs the osx_image set. Please see PR ( https://github.com/conda-forge/conda-smithy/pull/325 ).

I cancelled the builds so they don't take time for other feedstocks.

Please re-render with conda-smithy. This is required to enable the disabled CIs (e.g. Travis CI). In this particular case, it is very important to re-render with the latest release 1.3.2 to get a critical bug fix for Travis CI. Please ping when you have done this. Would like to double check that it went through smoothly.

@jakirkham done


Fixes on the front end

Make the navbar sticky in landing
Reduce space between sections in landing page
Get back the navbar in the dashboard
fix upload image in the profile section, add small element like pen in the corner to make it understoodable that user can change it.
add cursor pointer on hover on soil maintainance
On section predict revenue: add small explanation that shows when user hovers the (+2000$) (+3000kg)... different message for each index and the same repeats over all crops
Ensure that the font used in elements in the dashboard is the same in the side bar and the same in the landing (use the one in the sidebar)
Remove select button from selected land section
Make the logo agrisistance in the dashboard clickable if possible and it redirects to home of the dashboard

modify chatbot border and send button try make it green

land coordinates are entred separetly
Disable button when no land exists
Move all parameters of your land inside componenet (reactFC) by state
Fix budget rendering dynamiclly also the button is disabled when no land is selected
Remove selected button from already selected


chore: use eslint config internally for project

:tada: This PR is included in version 1.0.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Chant Proofread page missing Edit Syllabification link

Compare http://<IP_ADDRESS>/proofread-chant/669030?pk=669065&folio=007v to http://<IP_ADDRESS>/edit-chants/669030?pk=669065&folio=007v. I haven't checked whether OldCantus leaves this link off of the proofread page, but this seems like an important oversight.

I don't seem to have the relevant permissions on staging to see the links provided...
In OldCantus you can edit the syllabification via edit-volpiano but not /node/blah/edit; c.f.

Highly annoying in my opinion, especially since there are different permissions and paths involved to get to each of those places...

I have permissions to access https://cantus.uwaterloo.ca/proofread/699578, and I don't see an Edit Syllabification link anywhere.
If this is a feature we should close this issue. If it's a bug, we should fix it. @annamorphism, is this worth checking in with Debra about the intended functionality?

It seems to me that editing syllabification should be a past of the proofreading process, and that we do want it here. The OldCantus pages don't work exactly the same way, but the one that is closer to the proofread functionalities (edit-volpiano) is the one where you can edit syllabification. So we should fix this.


[#1319] Separate spend and spendorder behavior

This change adds a new possible error to POST /ob/spendorder. Our error handling is a little all over the place so it simply returns a string "ERROR_ORDER_NOT_FOUND" if the param is empty or if the order cannot be retrieved. Hopefully we can round back on to the error handling at some point.

Coverage decreased (-0.1%) to 33.565% when pulling 4b5b2791011cd2b1e41145c8bd65d325386979a4 on 1319-api-orderspend into cda68e29970ba486a295e06938a30da033d53da8 on master.


Getting Platform System Information Using Python

What type of PR is this? (check all applicable)

[x] 🚀 Added Name
[x] 🌟 stared the repo
[x] 🐛 Grammatical Error
[ ] 📝 Documentation Update

Description
Getting Platform System Information Using Python
Add Link of GitHub Profile
https://github.com/surjyaneeh

@keshavsingh4522
Sir Please Add Hacktoberfest-Accepted Tag


fix(CurrentRun): check if file exist when adding output

It's better to put that check on the openhexa-app in the addOutput resolver to cover the case of a user that calls the mutation.


vyper: 0.3.5 -> 0.3.6

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/vyperlang/vyper/releases.
meta.description for vyper is: "Pythonic Smart Contract Language for the EVM"
meta.homepage for vyper is: "https://github.com/vyperlang/vyper"
meta.changelog for vyper is: ""
Updates performed

Version update

To inspect upstream changes
Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 0.3.6 with grep in /nix/store/fqwnmymp5lnqg9i364v87jra4kcn52h5-python3.10-vyper-0.3.6
found 0.3.6 in filename of file in /nix/store/fqwnmymp5lnqg9i364v87jra4kcn52h5-python3.10-vyper-0.3.6

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

3 total rebuild path(s)

3 package rebuild(s)

First fifty rebuilds by attrpath
python310Packages.vyper
python39Packages.vyper
vyper

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/fqwnmymp5lnqg9i364v87jra4kcn52h5-python3.10-vyper-0.3.6 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

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

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/fqwnmymp5lnqg9i364v87jra4kcn52h5-python3.10-vyper-0.3.6
ls -la /nix/store/fqwnmymp5lnqg9i364v87jra4kcn52h5-python3.10-vyper-0.3.6/bin

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

2 packages built:

vyper (python310Packages.vyper)
python39Packages.vyper

Maintainer pings
cc @siraben for testing.

Result of nixpkgs-review pr 186036 run on aarch64-darwin 1

2 packages built:

vyper (python310Packages.vyper)
python39Packages.vyper


[flowsheet-data-grid] Hide columns headers

Summary

This PR hides the column headers in the flowsheet-data-grid to meet functional requirements.
Testing

This change was tested using:

[x] WDIO
[ ] Jest
[ ] Visual testing (please attach a screenshot or recording)
[ ] Other (please describe below)
[ ] No tests are needed

Reviews
In addition to engineering reviews, this PR needs:

[ ] UX review
[ ] Accessibility review
[ ] Functional review

Additional Details

This PR resolves:
UXPLATFORM-9771

Thank you for contributing to Terra.
@cerner/terra

@sdadn  Based on Charles's comment, and your response, looks like you are addressing this. But from a documentation/examples perspective I think we should have a specific example without column headers and then keep the others to have headers.

@eawww , @chrismichalewicz  I added the no header example, so it should now be ready for review.

@eawww , @chrismichalewicz I added the no header example, so it should now be ready for review.
@sdadn  Did you update the right example?
The "Columns Header Hidden" example is showing headers, it looks like the "No Results Cell" might have been updated by mistake as it shows with no headers.

Also can "Columns Header Hidden" be updated to "Column Headers Hidden"?

@chrismichalewicz thanks for pointing that out, I duplicated the files and updated the wrong one 🤦♂. It should be fixed now.

@sdadn  Not sure if it's related to the isSelectable observation @eawww  made but in testing with JAWS in Edge the column headers are also being communicated when tabbing into the table.

@sdadn Not sure if it's related to the isSelectable observation @eawww made but in testing with JAWS in Edge the column headers are also being communicated when tabbing into the table.

@chrismichalewicz I believe that is intended because the implementation to hide column headers only makes them invisible but doesn't remove them from the DOM completely. @cm9361 / @eawww can correct me if I'm wrong.

@sdadn Not sure if it's related to the isSelectable observation @eawww made but in testing with JAWS in Edge the column headers are also being communicated when tabbing into the table.

@chrismichalewicz I believe that is intended because the implementation to hide column headers only makes them invisible but doesn't remove them from the DOM completely. @cm9361 / @eawww can correct me if I'm wrong.

We discussed this and this is working as expected so there are no updates needed for this specific comment. +1


Add LocalRedirect to Page/PageModel

Pages need to have the same set of Redirect-related helpers as controllers

Dupe of #6007


Add global timeout to E2E test

Extend the timeout for each test

The one report on my fork, it's possible to run more than two seconds...


Swap ERC-20 support

Looks like the swap example is not compatible with swapping to/from ERC-20s.
Currently, we're swapping the whole incoming amount to target ZRC-20.
https://github.com/zeta-chain/example-contracts/blob/51373fb8d2c23fcee8e907a329296747a10d1c65/omnichain/swap/contracts/Swap.sol#L50-L58
Then we check if the gas token is not equal to the target ZRC-20.
https://github.com/zeta-chain/example-contracts/blob/51373fb8d2c23fcee8e907a329296747a10d1c65/omnichain/swap/contracts/Swap.sol#L60-L64
When we want to swap tMATIC for USDC (Goerli), the target ZRC-20 (USDC) will not match the gas token (gETH), so the swap will fail. This is issue 1.
If I understand correctly, for the swap to succeed the contract has to have uint256 gasFee amount of address gasZRC20 to cover the gas costs when withdrawing the target ZRC-20. Right now we're only swapping to the target ZRC-20 and the contract does not have gas tokens. This is issue 2.
@andresaiello please, let me know if you agree that both issues are valid.
To solve these issues I'm proposing we do the following:

Determine the required gas fee amount using IZRC20(targetTokenAddress).withdrawGasFee();
Swap to get the required gas fee amount using SwapHelperLib.swapTokensForExactTokens
Swap the rest to get the target amount using SwapHelperLib._doSwap.

For this to work I've added a swapTokensForExactTokens helper:
https://github.com/zeta-chain/toolkit/blob/bc629af99832689697b2c45a6560b1b5df382d66/contracts/SwapHelperLib.sol#L127-L164
The problem I'm facing right now is that when I try to interact with a contract:
https://zetachain-athens-3.blockscout.com/address/0x7F665c57796E19a6A938f2B6dDbc5Ac5576c5De7
npx hardhat interact --contract 0x7F665c57796E19a6A938f2B6dDbc5Ac5576c5De7 --amount 1 --network mumbai_testnet --target-token 0x0cbe0dF132a6c6B4a2974Fa1b7Fb953CF0Cc798a --recipient 0x2cD3D070aE1BD365909dD859d29F387AA96911e1
🔑 Using account: 0x2cD3D070aE1BD365909dD859d29F387AA96911e1

🚀 Successfully broadcasted a token transfer transaction on mumbai_testnet network.
📝 Transaction hash: 0xfcc13472c25b7ae1b1aa0dd29514266f8b3183b60daa97aa902737e529d1eae4

The contract isn't swapping into the gas token gETH (the balance is 0), but swaps fine to the destination token USDC.
I need help figuring out why swapTokensForExactTokens doesn't work as intended.

Alternatively, we can use a different approach were we swap all incoming tokens into the gas token using _doSwap, deduct the gas fee amount, and swap the rest again using _doSwap into the destination token.
@andresaiello please, let me know if you think this solution is ok or if we should try to fix swapTokensForExactTokens.

@fadeev I agree with the approach. This contract was designed before ERC20 support. I like the first approach

@fadeev I agree with the approach. This contract was designed before ERC20 support. I like the first approach

Then I need help debugging swapTokensForExactTokens 😬

I've tried the second approach (swapping all to gas tokens, deducting the gas fee, then swapping to destination token), and I'm getting the same result (tMATIC -> Goerli USDC, I'm getting 0 gETH and a proper amount of USDC).
        (address gasZRC20, uint256 gasFee) = IZRC20(targetTokenAddress)
            .withdrawGasFee();

        uint256 gasOutput = SwapHelperLib._doSwap(
            systemContract.wZetaContractAddress(),
            systemContract.uniswapv2FactoryAddress(),
            systemContract.uniswapv2Router02Address(),
            zrc20,
            amount,
            gasZRC20,
            0
        );

        SwapHelperLib._doSwap(
            systemContract.wZetaContractAddress(),
            systemContract.uniswapv2FactoryAddress(),
            systemContract.uniswapv2Router02Address(),
            gasZRC20,
            gasOutput - gasFee,
            targetTokenAddress,
            0
        );

https://zetachain-athens-3.blockscout.com/address/0x6bf6ae7d9763eb1e42d8c7e9D31A5C23Be192a70

mmmm targetTokenAddress is USDC? I guess to calculate the fee should be the native token of destination chain

mmmm targetTokenAddress is USDC? I guess to calculate the fee should be the native token of destination chain

@andresaiello you mean IZRC20(targetTokenAddress).withdrawGasFee()? When target token is USDC it returns ZRC-20 gETH address, so I don't think there is a problem there.

withdrawGasFee() always returns a gas token:
https://github.com/zeta-chain/protocol-contracts/blob/fbf8687395ce15429ec0bb6ef8342e3c9f23cb6e/contracts/zevm/ZRC20.sol#L237

are we sure gasFee is returning >0 ?

are we sure gasFee is returning >0 ?

It is returning a positive integer, yes.

I'm just using public variables to output some debug information as events don't show in the explorer.

When I manually increase gasFee to a larger amount, I can see that swapTokensForExactTokens actually swaps tokens just fine. The problem is with IZRC20(targetTokenAddress).withdraw(recipientAddress, outputAmount), it's throwing an error:
0x8e03367d2cd6ac000e3186c86fe5939daa98ed544ffbf6238ee31000ab056470: 80001 → 7001: PendingRevert (contract call failed: method 'depositAndCall', contract '0xEdf1c3275d13489aCdC6cD6eD246E72458B8795B': execution reverted: ret 0x10bad147: evm transaction execution failed)

Amounts 0 will be the same you sent as input
Then your line 70 is returning 0

@andresaiello I don't think that's the case.
Here's an example of me swapping 10 tMATIC for Goerli USDC. The state of the contract is a result of a single transaction.
https://zetachain-athens-3.blockscout.com/address/0x7c732C6A614ecaD95B2953C624d39825f58C6F5C

I’m using public variables to debug the contract. inputForGas (amount 0 of swapTokensForExactTokens) is not the same as input (which is amount). inputForGas actually represents the amount of gETH tokens that was swapped to cover gas fees. Please, ignore Blockscout's bug where it shows  0.00 gETH, you can check the actual value through the ZRC-20 gETH contract.

All in all, inputForGas is an amount in gETH that ZRC-20 USDC contract required for gas, so that should be enough gas. amount - inputForGas

In a separate contract I've also tried setting gasFee manually and the contract had up to 0.1 gETH for fees, but withdraw still kept failing, so it's probably not due to lack of tokens for gas.

Could it be that it's getting confused because USDC has uses different decimal amount that gas tokens?

@SocketSecurity ignore<EMAIL_ADDRESS>

@andresaiello please, review.


Sustainability Trends in Modern Technology

To ensure environmental sustainability, efficient practices are being used in modern-day workplaces such as:

Carbon Footprint Tracking: Companies measure and report their carbon footprints regularly to ensure that no actions are taken by the company that largely affect the environment.

Electric and Autonomous Vehicles: The automotive industry is transitioning to electric vehicles (EVs) and exploring autonomous driving technology, which can reduce emissions and improve fuel efficiency.

Regulatory Compliance: Governments are implementing stricter regulations on emissions and sustainability, prompting tech companies to adopt more environmentally friendly practices.

Search for new issue :Repeated


Support for Saved Searches (functions) in Log Analytics workspaces

Issue
I want to add Saved Searches (functions) in the Azure TerraForm provider, but due to the lack of fields in the REST API specs the Go SDK does not contain the property I want to use:

functionAlias

Query
curl --location --request PUT 'https://management.azure.com/subscriptions/<<SUBSCRIPTION>>/resourcegroups/<<RESOURCEGROUP>>/providers/Microsoft.OperationalInsights/workspaces/<<WORKSPACE>>/savedSearches/test-6?api-version=2015-03-20' \
--header 'Authorization: Bearer <<TOKEN>>' \
--header 'Content-Type: application/json' \
--data-raw '{
  "properties": {
    "category": "Saved Search Test Category",
    "displayName": "Create or Update Saved Search Test",
    "query": "Heartbeat | summarize Count() by Computer",
    "version": 2,
    "functionAlias": "test-function-alias",
    "tags": [
      {
        "name": "Group",
        "value": "Computer"
      }
    ]
  }
}'

Definition file
https://github.com/Azure/azure-rest-api-specs/blob/master/specification/operationalinsights/resource-manager/Microsoft.OperationalInsights/stable/2015-03-20/OperationalInsights.json

We are working to release a new SDK version.

@shemers Are you talking about the 2020-03-01 preview API? Or is there something else?
If there's something like a beta version: I'd love to give feedback

@yossi-y can you please follow up here.

There is an implementation for Terraform done based on the preview for Saved Searches.
Thanks a lot @shemers !


feat: 96/abort session and clear steps

Resolves #96.
This is a reposting of https://github.com/elastic/synthetics-recorder/pull/97; after we brought this repo public my fork lost association with open PRs and new commits would not show up/trigger the CI process.
After numerous attempts to re-establish the connection I opted to simply re-open a new PR. Description and previous comments available at the link above.

:green_heart: Build Succeeded

the below badges are clickable and redirect to their specific view in the CI or DOCS

Expand to view the summary

Build stats

Start Time: 2021-12-03T15:22:34.515+0000

Duration: 43 min 57 sec

Commit: 60c42e7c449c5e3aaccfef41f20468dc12c6a536

Test stats :test_tube:

Test
Results

Failed
0

Passed
31

Skipped
0

Total
31

:robot: GitHub comments
To re-run your PR in the CI, just comment with:

/test : Re-trigger the build.

@vigneshshanmugam sorry, I should've run this branch before re-opening the PR. The functionality is not working the way it was when I last developed on this branch, I will ping you when it's ready for review.

@vigneshshanmugam I think this is now working as the design expects.


Edit user guide

Add addPatient command.
Edit formatting for command summary
Edit formatting for admonition
Add login to command summary.
Edit select to selectPatient command.
Edit find to findPatient command.

Pull Request Test Coverage Report for Build 189

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

Totals

Change from base Build 173:
0.0%

Covered Lines:
1716

Relevant Lines:
2031

💛  - Coveralls

I will add rest of the new command in another PR.


Patient diary to update locations

Descriptions:
As a patient, I would like to update my diary (building, the places have been using postcode, GPS ... diary is optional) daily. So that doctors can view the patient's symptoms, and health officers can track patients' possible contact or self-quarantine.
Acceptance criteria:
Given: The user is logged in as a patient.
When: The user clicks on add diary.
Then: The user is able to modify their diary.
Sub-task:

[x] #157
[x] #158

merged to develop!


Non-intrusive usage for temu.com

What website(s) would this filter apply to?
https://www.temu.com/
What content should be blocked?
Intrusive pop-ups and psychological triggers like:

90% OFF!!
ONLY 2 ARTICLES LEFT!
175 CUSTOMERS ALSO HAVE THIS ARTICLE IN THEIR SHOPPING CART!
Quick, buy now!
Do you really want to remove this article from your shopping cart? You will miss out the awesome discount!

What options should be offered?
No response
Did you write a custom uBlock filter already?
my-ublock-static-filters_2023-08-17_14.37.07.txt
Additional notes
The static filters are not perfect on all websites.
It is also very aligned to my needs. Things, like the strikedthrough original total price and discount percentages, are blocked.
Intrusive things, like stopping me from my decision and begging to reconsider, are blocked as well.

@JohnyP36, thanks for the quick reply!
I see!
I can try to figure something out to make the filter rules more generic, but I am not very good at defining filter rules.
Most of the stuff is coming from kwcdn.com, but blocking this breaks the entire page:

@keks24 When I have time (probably somewhere within 3 months) I can make an account and verify and beautify your rules, but you have to be patient.

Nice, just within 3 days?! (Measuring in adult time) :smile:
Sure thing, no pressure!
I am also going to invest some time and will update here from time to time.

Hello @keks24, thanks a lot for your contribution to the project!
My intuition is that this template would be a pretty high-maintenance one:

their markup use autogenerated class names, for which we have no guarantee of stability. They could all change next month, invalidating all of the rules
there will probably be additions or changes of these prompts all year long, as they A/B test their conversion rate

This is definitely not a template that the contributors team can maintain on a reactive mode (not using the target website and waiting for people to open issues when it breaks). I think that the best way forward would be for you to commit to maintain it proactively and contribute updates as needed.
Because the scope of things to hide will vary repeatedly as nags are added and removed, I don't recommend making a customizable template that lists the things it hides. I'd go with a "remove all of that stuff" non-customizable template like we do for the high-maintenance youtube-shorts.yaml. To make the maintenance easier, I'd rather you add comments that explain what the rule hide, like we do for the shorts template.
How would you feel about giving it a try? I'm happy to help and review it of course! Adding a template is achieved by adding a yaml file to the repo, you can just copy youtube-shorts.yaml as temu-cleanup and modify it. data/filters/README.md documents the file format. Because this template will not have any option, tests are not needed.

@xvello

I think that the best way forward would be for you to commit to maintain it proactively and contribute updates as needed.

How would you feel about giving it a try?

I feel honoured to get this kind of opportunity!
But I am not sure, if I will be capable of maintaining proactively on a regular basis, since my current living situation still has some incoming unpredictable twists and turns.

To make the maintenance easier, I'd rather you add comments that explain what the rule hide, like we do for the shorts template.

Sure!
I will have a look later on and will see, if I can do it. :grin:

I thought about it and I want to give it a try!
Though, I still have not looked into the yaml files in detail and still need to figure out how the process flow looks like.
I can maintain it proactively from time to time, but it will be very irregularly later on, if this would be OK.

Hello @keks24, sorry for the delay! This month has been busy and I'll be out of town for the rest of the month. Feel free to open a PR as I described above (by copying youtube-shorts.yaml and changing the rules). I won't be able to test the rules, but can review and merge it.

@xvello, sure! Once I found the time to do it finally.
I am currently prioritising personal goals. I will come back later to this.
Meanwhile, temu.com may have changed a lot of things already, so less editing for me, if I would have started now. :smile:

I just want to inform you, that I do not have the capacities for this anymore, sorry.

Hello @keks24, thanks for the head's-up. Re-closing as "not done" to better reflect the status, but happy to re-open if someone wants to start work on this!


Documentation related issue template

Add documentation related issue template

@Pranjal360Agarwal  i can solve this issue please assign me this issue

I have submitted a pull request for this issue , PR #179
Please check and merge
Thanks


Create Coffees UserControl

[x] Create CoffeeForm UserControl
[x] Create CoffeeForm Validation
[x] Implement Logic for CoffeesManager
[x] Create Layout

resolved by 205434e87f93043c08d3adbf93912ed247403c93


Add TFM_PROVIDES_BL1 option used to use stage 1 bootloader (BL1) from TF-M as part of final merged hex

Is your enhancement proposal related to a problem? Please describe.
Both Zephyr and TF-M contains board support code for the same board.
A Zephyr board config sets the corresponding TF-M board to be used when building TF-M.
Some of the TF-M boards produce a stage 1 (BL1) bootloader. The custom board that I use builds BL1.
The TF-M produced BL1 (elf|bin|hex) file is not accessible from Zephyr when making a merged hex image.
The image contains only BL2 and the secure/unsecure images merged into one HEX file (tfm_merged.hex).
I used the following command for testing:
west build -p -b b_u585i_iot02a_ns samples/tfm_integration/tfm_ipc
Describe the solution you'd like
Add a config option: "TFM_PROVIDES_BL1" which instructs CMake to expect to find a BL1 (elf|bin|hex) after TF-M has been built.
The option doesn't change TF-M configuration. The board used for TF-M is expected to output bl1.(elf|bin|hex) based on its config. It it doesn't the zephyr build will fail.
The added option should be disabled per default.
Describe alternatives you've considered
The alternative is making flashing scripts that are not part of Zephyr.
Additional context
No.

I added the proposed solution here:
https://github.com/zephyrproject-rtos/zephyr/pull/49453


Bumping version in build.gradle to 4.2.2

See similar https://github.com/cedar-policy/cedar-java/pull/237
Bumping version in build.gradle to 4.2.2 before publishing 4.2.2 to Maven to make tagging support available

Closing as this is targeting main and not release/4.2.x as intended

You can change the target without closing the PR :)


help center: Mention stream section labels.

Updates the relevant help center articles to reflect that the sections for Pinned and Inactive streams in the left sidebar now have labels.
Fixes #23423.
Screenshots and screen captures:

https://zulip.com/help/manage-inactive-streams

https://zulip.com/help/pin-a-stream

Self-review checklist

[x] Self-reviewed the changes for clarity and maintainability

Communicate decisions, questions, and potential concerns.

[x] Explains differences from previous plans (e.g., issue description).
[x] Highlights technical choices and bugs encountered.
[x] Calls out remaining decisions and concerns.
[x] Automated tests

Individual commits are ready for review (see commit discipline).

[x] Each commit is a coherent idea.
[x] Commit message(s) explain reasoning and motivation for changes.

Completed manual review and testing of the following:

[x] Visual appearance of the changes.

@alya This PR is ready for your review.

I've made the necessary changes in response to your feedback.

Looks good to me, thanks!

Merged, thanks @drrosa!


feat(client): completion modal progress bar

Re-land #37775

There was a bit of miss-understanding and the linked PR was merged and then reverted. The actual intent was to cleanup the commit messages.
Anyways, I am re-landing the original PR as is with as cherry-pick of the commit.
We are unable to make any force pushes to master to avoid any builds and Travis CI checks on all the currently open PRs.
This makes the commit history look ugly, but it is better than having broken builds. Apologies about the confusion.
For the uninitiated, everything is normal and you can ignore the activity on this PR.


[WIP] - Refactor into Client

New Client for managing HTTP reqs
Unit & Integration Tests
Abstracted request layer

Codecov Report

Merging #4 (4a59077) into master (577ffbb) will increase coverage by 33.87%.
The diff coverage is 73.44%.

@@             Coverage Diff             @@
##           master       #4       +/-   ##
===========================================
+ Coverage   40.00%   73.87%   +33.87%
===========================================
  Files           2        7        +5
  Lines          70      245      +175
===========================================
+ Hits           28      181      +153
- Misses         37       51       +14
- Partials        5       13        +8

Impacted Files
Coverage Δ

pay.go
0.00% <0.00%> (ø)

request.go
60.00% <60.00%> (ø)

get_payment.go
86.84% <86.84%> (ø)

profile.go
90.00% <88.46%> (+68.37%)
:arrow_up:

api.go
94.87% <94.87%> (ø)

client.go
100.00% <100.00%> (ø)

utils.go
100.00% <100.00%> (ø)

... and 3 more

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 577ffbb...4a59077. Read the comment docs.


Added better node position logging

What problem does this PR solve?
Maybe #910 -> How do I get the position of an AST node.
The previous solution logged the starting offset of the original text,
this pull request added more detailed position information to ast.Node
What is changed and how it works?
Moved the struct Pos from parser/lexer.go into parser/ast/ast.go
Added 2 methods
func (ast.node) SetOriginTextPos(ast.Pos)
func (ast.node) OriginTextPos() ast.Pos

to the interface ast.Node
Check List
Tests

Unit test

Code changes

Has exported function/method change

Side effects

Possible performance regression
Breaking backward compatibility

Related changes
N/A

 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.Yifan Luo 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.

Hi @TeemoKill, would you like to explain why we need this feature? Did you encounter any problems?

 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.Yifan Luo 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.


[5.6] fix for route url generator when request()->basePath exists

This is a fix for a bug that was introduced with #24051 and tagged in version 5.6.19 yesterday.
Basically, if you had another script (non Laravel) in a sub-folder in the public root that loaded Laravel (by requiring the bootstrap.php file), every route url would include that sub-folder path.
This is a common thing to do in legacy code you are slowly converting to the framework, or when trying to utilize laravel resources in non laravel code (like with wordpress for instance).
In the past, the url generator stripped the whole url root but that changed in #24051 and introduced this side effect.
This fix simply removes the base path string if it exists on the request.
/cc @staudenmeir

What if the base is in the query string for some reason? Maybe holding a redirect value or something? Will this remove it?

I'm not sure the base could ever be a query string... that should only be set by to where the script was initialized from by PHP.
Am I missing something there?


feat: Add Website Carbon badge

Closes #73
Checklist

[x] I didn't over-scope my PR
[x] My PR title matches the commit convention
[x] I did not include breaking changes
[ ] I included unit tests that cover my changes
[ ] I added/updated the documentation about my changes
[x] I made my own code-review before requesting one

Description of the changes
Add carbon badge in the footer

:tada: This PR is included in version 1.1.0-beta.4 :tada:
The release is available on:

GitHub release
v1.1.0-beta.4

Your semantic-release bot :package::rocket: