GitHub Oidc and Latest Flagscript Cicd

Add optional GitHub oidc provider to accounts
Use latest flag cicd

closes #16
closes #17

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


MoveChild doesn't set parent of childs correctly

Let's say I have an HtmlDocument that has some child nodes in DocumentNode like

. I want to encapsulate the existing childs into a new  Node:
private void InsertHtmlRootElement(HtmlDocument doc) {
    HtmlNode HtmlTag = doc.DocumentNode.ChildNodes.FindFirst("html");
    if (HtmlTag is null) {
        // Create new  Node
        HtmlTag = new HtmlNode(HtmlNodeType.Element, doc, 0) { Name = "html" };
        // Move all childs from root into the new HTML Node
        HtmlTag.MoveChildren(doc.DocumentNode.ChildNodes);
        // Append new HTML Node into Document
        doc.DocumentNode.AppendChild(HtmlTag);
        // HtmlTag.ChildNodes.ForEach(c => c.SetParent(HtmlTag));
    }
}

This works but all children of HtmlTag have their ParentNode set to null. I guessed that I first need to add the HtmlTag to the document and then insert the childs to fix it:
private void InsertHtmlRootElement(HtmlDocument doc) {
    HtmlNode HtmlTag = doc.DocumentNode.ChildNodes.FindFirst("html");
    if (HtmlTag is null) {
        // Append new  Node
        HtmlTag = new HtmlNode(HtmlNodeType.Element, doc, 0) { Name = "html" };
        doc.DocumentNode.AppendChild(HtmlTag);
        // Move all childs from root into the new HTML Node
        doc.DocumentNode.ChildNodes.Except([HtmlTag]).ToArray().ForEach( c => {
            HtmlTag.MoveChild(c);
            // c.SetParent(HtmlTag);
        });
    }
}

As it turns out, the children still have no parent. To work around that, I have to call c.SetParent(HtmlTag); explicitly.

Hello @dartrax ,
The method MoveChilden seems to have been created to move a node from one document to another document, not within the same document.
The MoveChildren method calls the RemoveChildren method: https://github.com/zzzprojects/html-agility-pack/blob/master/src/HtmlAgilityPack.Shared/HtmlNode.cs#L1770. This causes an issue because the child is fully removed from the document after being correctly added with a parent.
I don't know if that's a bug or if this works as intended by the original author. What I would recommend you is probably this code:
public void InsertHtmlRootElement(HtmlDocument doc)
{
	HtmlNode HtmlTag = doc.DocumentNode.ChildNodes.FindFirst("html");
	if (HtmlTag is null)
	{
		// Create new  Node
		HtmlTag = new HtmlNode(HtmlNodeType.Element, doc, 0) { Name = "html" };

		var childNodes = doc.DocumentNode.ChildNodes.ToList();
		childNodes.ForEach(x => doc.DocumentNode.RemoveChild(x));

		// Append new HTML Node into Document
		doc.DocumentNode.AppendChild(HtmlTag);

		// Move all childs from root into the new HTML Node
		childNodes.ForEach(x => HtmlTag.AppendChild(x));
	}
}

All childs get removed from the document and after added to the HTML node.
Best Regards,
Jon

Hi Jon,
thanks for your reply. Your recommended code works, thank you!
I would suggest to add the =MoveChildren= method to the docs, because currently I couldn't find any documentation for it. Then the intended behavior could be documented.
Ideally, of course this use case should be added to make it work in both cases ;-)

Thank you for your suggestion.
Both methods have been added with a note about the "different document": https://html-agility-pack.net/traversing
Unfortunately, .NET Fiddle is currently down so I was not able to add the example
Best Regards,
Jon


Fix typo in CMakeLists.txt

At the time generated dynamic library is named libnetcdf.so.19} which looks like a typo.

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

I would like to contibute on behalf of Yandex LLC and thus need to sign a corporate version of the CLA.
Do you have such option?


Fetch metafields for an order

This is not tested. Please test it before merging!
I can't test it because I get an error during yarn install.

The error if you are interested
➤ YN0000: ┌ Resolution step
➤ YN0061: │ natives@npm:1.1.6 is deprecated: This module relies on Node.js's internals and will break at some point. Do not use it, and update to<EMAIL_ADDRESS>➤ YN0061: │ tslint@npm:6.1.3 is deprecated: TSLint has been deprecated in favor of ESLint. Please see https://github.com/palantir/tslint/issues/4534 for more information.
➤ YN0002: │ alsatian@npm:3.2.1 doesn't provide typescript (pfb597), requested by ts-node
➤ YN0000: │ Some peer dependencies are incorrectly met; run yarn explain peer-requirements <hash> for details, where <hash> is the six-letter p-prefixed code
➤ YN0000: └ Completed in 2s 425ms
➤ YN0000: ┌ Fetch step
➤ YN0013: │ yargs@npm:14.2.3 can't be found in the cache and will be fetched from the remote registry
➤ YN0013: │ yargs@npm:16.2.0 can't be found in the cache and will be fetched from the remote registry
➤ YN0013: │ yn@npm:3.1.1 can't be found in the cache and will be fetched from the remote registry
➤ YN0013: │ yoga-layout-prebuilt@npm:1.10.0 can't be found in the cache and will be fetched from the remote registry
➤ YN0013: │ typescript@npm:4.3.2 can't be found in the cache and will be fetched from the remote registry
➤ YN0066: │ typescript@patch:typescript@npm%3A4.3.2#builtin<compat/typescript>::version=4.3.2&hash=a45b0e: Cannot apply hunk #2 (set enableInlineHunks for details)
➤ YN0000: └ Completed in 3s 451ms
➤ YN0000: Failed with errors in 5s 882ms

I took the information about the endpoint from here https://shopify.dev/docs/admin-api/rest/reference/metafield

Thank you!
This looks fine, and I tested it and it works.
The only issue I see with it is the name of the method "metafields" on the Orders service, since there are also endpoints to count, get (single field), create, update and delete metafields for an order (etc.).
So I will rename this to listMetafields and would later add the remaining methods (countMetafields, getMetafield, createMetafield, updateMetafield, deleteMetafield).
(About the error during yarn install: This happens commonly with yarn2 when the Typescript version is too new for the used yarn version, because yarn always needs to apply patches to make Typescript work with it. For TypeScript 4.3.2 it fails with yarn 2.4.1 but works with yarn 2.4.2.)

FYI: I already added a new commit including the other methods, also for Product, Customer, Page, Blog, ... etc.

Any plan for making a release in the near future?
I would like to use the metafield part in this plugins, many thanks!

@vip30 We are still fixing the tests and will then publish a new release. If you want to contribute tests for the metafields, please do :)


Adding a DDOS page on the wiki

DDOS
Introduction
A Memcached attacks operates similarly to all DDoS amplification attacks such as NTP amplification and DNS amplification.
The attack works by sending spoofed requests to a vulnerable server, which then responds with a larger amount of data than the initial request, magnifying the volume of traffic.
This method of amplification attack is possible because memcached servers have the option to operate using the UDP protocol.
UDP is a network protocol that allows for the sending of data without first getting what’s known as a handshake, which is a network process where both sides agree to the communication.
UDP is utilized because the targeted host is never consulted on whether or not they’re willing to receive the data, allowing for a massive amount of data to be sent to the target without their prior consent.
How does an attack with memcached work?

An attacker implants a large payload of data on an exposed memcached server.
Next the attacker spoofs an HTTP GET request with the IP address of the targeted victim.
The vulnerable memcached server that receives the request, which is trying to be helpful by responding, sends a large response to the target.
The targeted server or its surrounding infrastructure is unable to process the large amount of data sent from the memcached server, resulting in overload and denial-of-service to legitimate requests.

Mitigation
Disable UDP
For memcached servers, make sure to disable UDP support if you do not need it.
You can disable UDP by adding the following line to your /etc/memcached.conf:
# Disable UDP protocol
-U 0

Save /etc/memcached.conf and restart the service (On Ubuntu using service memcached restart).
You can check it worked by using netstat:
$ sudo netstat -plunt

Active Internet connections (only servers)
Proto Recv-Q Send-Q Local Address           Foreign Address         State       PID/Program name
tcp        0      0 <IP_ADDRESS>:11211         <IP_ADDRESS>:*               LISTEN      16079/memcached

As you can see, memcached is listening using only UDP.
Firewall memcached servers
If your memcached server is only used by the local machine, you can limit the listening address to <IP_ADDRESS>.
If other machines need to connect from a private network, force the listening on a private IP (for example <IP_ADDRESS>.1, to be adapted to your network class).
You can change those settings in the file /etc/memcached.conf:
# Only listen on localhost
--listen <IP_ADDRESS>

By default, Ubuntu and Debian bind Memcached to the local interface <IP_ADDRESS>.
You can also use firewalls to block external traffic from reaching your memcached server.
External links

What is a memcached DDoS attack?

Did some editing and put it up: https://github.com/memcached/memcached/wiki/DDOS
thanks for the push.


[SPARK-20092][R][PROJECT INFRA] Add the detection for Scala codes dedicated for R in AppVeyor tests

What changes were proposed in this pull request?
We are currently detecting the changes in R/ directory only and then trigger AppVeyor tests.
It seems we need to tests when there are Scala codes dedicated for R in core/src/main/scala/org/apache/spark/r/ and sql/core/src/main/scala/org/apache/spark/sql/api/r/ too.
This will enables the tests, for example, for SPARK-20088.
How was this patch tested?
Tests with manually created PRs.

Changes in sql/core/src/main/scala/org/apache/spark/sql/api/r/SQLUtils.scala https://github.com/spark-test/spark/pull/13
Changes in core/src/main/scala/org/apache/spark/api/r/SerDe.scala https://github.com/spark-test/spark/pull/12
Changes in README.md https://github.com/spark-test/spark/pull/14

These PR above were made against detect-tests which has a diff - https://github.com/spark-test/spark/compare/master...spark-test:detect-tests and therefore the PR against branch detect-tests enabled the tests.

cc @shivaram and @felixcheung.

Test build #75208 has started for PR 17427 at commit cfb3a88.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/75208/
Test FAILed.

retest this please

Test build #75209 has started for PR 17427 at commit cfb3a88.

Test build #75209 has finished for PR 17427 at commit cfb3a88.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/75209/
Test PASSed.

Thanks! this has always bug me.
Please add mllib/src/main/scala/org/apache/spark/ml/r too

Test build #75225 has started for PR 17427 at commit 1ce14a2.

Test build #75225 has finished for PR 17427 at commit 1ce14a2.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/75225/
Test PASSed.

I guess AppVeyor doesn't run with this change and it's expected

merged to master, thanks!

Thank you @felixcheung


add a link to cccmo devel docs

This adds a link directly to the integration docs in the cccmo
repository to help give the reader more information about the steps
involved in the process.

@candita ptal

lgtm


Issue#721 - not-sorted and duplicate entries of COO format

PR for #721

2nd commit:

More comments are added.
Explicit zeros are not intentionally removed.


Add (GPN:Beam) to the user-agent string of Python GCS SDKs.

For a valid user-agent string, the version number should be separated with the product name by "/". In addition, the requested string of "(GPN:Beam)" is considered a comment, and could not be the start of the user-agent string.
addresses #26644

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Mention the appropriate issue in your description (for example: addresses #123), if applicable. This will automatically add a link to the pull request in the issue. If you would like the issue to automatically close on merging the pull request, comment fixes #<ISSUE NUMBER> instead.
[ ] Update CHANGES.md with noteworthy changes.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.
To check the build health, please visit https://github.com/apache/beam/blob/master/.test-infra/BUILD_STATUS.md
GitHub Actions Tests Status (on master branch)

See CI.md for more information about GitHub Actions CI.

Run Python PreCommit


Add CONTRIBUTING.md

Add CONTRIBUTING.md to describe Contributor License Agreement

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


[Bug][gitextractor] Azure devops repos fail to clone with 400 errors

Search before asking

[X] I had searched in the issues and found no similar issues.

What happened
Soemwhere around the v1.0 release the underlying implementation for cloning git repos was changed, and since that point cloning azure deveops git repositories has not been working (on prem, though I would assume cloud is the same)
The error reported is a 400 response.
What do you expect to happen
I expected git repos to continue to clone successfully, they were working fine before I updated the code base (to pick up an unrelated fix)
How to reproduce
Configure the azure devops connector, provide a valid PAT  and setup one or more repositories. run collect data.
I believe this would happen to azure cloud, if not then this could be specific to azure devops server which I know is technically unsupported. however this was working prior to picking up v1 changes and I can make it work with code changes now
Anything else
So from what I can tell the azure devops git server requires an AUTHORIZATION Basic header to be set with the base64 encoded version of an empty user and the pat token, eg-   :  encoded as base64. It seems to need this in addition to the values being passed on the commandline in the url where we are setting https://git:@url
In testing a fix locally I found that invoking the git cli directly as happens from this line: https://github.com/apache/incubator-devlake/blob/8a6bbe2b84d709e30d329cc7317ac85aa2398f10/backend/plugins/gitextractor/parser/clone_gitcli.go#L170C32-L171C2
did not work, something about hte way go was passing the parameters to the command were corrupting the header. However if I isntead call bash and pass the git command as an argument then repository cloning works no problem
Obviously my local fix is very specific to me, in taht I know I'm talking to azure devops so I can extract the PAT from the passed in url and encode it into a header etc. I'm not sure what hte general purpose fix for something like this would be.
Version
master
Are you willing to submit PR?

[ ] Yes I am willing to submit a PR!

Code of Conduct

[X] I agree to follow this project's Code of Conduct

Hi, can you share the error message of the 400 response? Thanks.

@klesh  sorry I didn't capture the exact wording, and my machine rebooted since. obviously I'm now running a version with my own local fix. Ostensibly it was just a 400 bad request response from AZD reported by the git cli. I guess I'm assuming that other people using AZD would be seeing the same issue with the latest code?
I can confirm that for me switching the base command to being bash and the args to be:
args := []string{"-c", "git -c http.extraHeader="Authorization: Basic "+sEnc+"" clone "+ taskData.Options.Url+" "+localDir+" --bare --progress"}
fixed my issue. where I obviously pulled the PAT out of the Options.Url anad b64 encoded it into the sEnc value.

@wouldd It is working fine on my local machine, I wonder if it is caused by the configuration on the Azure Portal. Is it possible for you to provide a Restricted token and Repo for me to investigate? You may find me on the Slack

So we are not using azure devops services, we're currently still using on-prem. so I guess if this is working for you then the issue relates to something odd about the on-prem setup. I'm happy to consider this a quirk of me being unsupported pointing to our server rather than services.
It's weird that it used to work prior to the code changes in the gitextractor logic. but I suppose for now I can continue to solve for our case with my local fix.


Prevent form submission if data types are invalid

Webapp should aim to prevent a user from sending invalid data (e.g., input == string when should have been an int)

resolved with #61


[DRAFT] Convert ORM to FHIR for TI Service

User Story

User Story
As a Hospital, I want ReportStream to be able to convert incoming ORM messages to FHIR format, so that I'm able to submit electronic orders the form that my EHR supports.
Description/Use Case
We would like to support the following FHIR resources:

Patient
ServiceRequest
Observations
Organization
Practitioner
MessageHeader
Specimen
PractitionerRole (otherwise, I can't find a way to associate both the ordering physician and hospital to the ServiceRequest)

Risks/Impacts/Considerations

Dev Notes

Example ORM Message:
MN ORM
Dev note: Proposed FHIR sample message https://github.com/CDCgov/trusted-intermediary/blob/fhir-sample-data/examples/fhir/MN NBS FHIR Order Message.json -- but we can certainly make adjustments to that as needed
Acceptance Criteria

[ ] Sample FHIR messages are flowing to the TI Service
[ ] Confirm receipt of the FHIR messages by ETOR team

Regarding the list of FHIR resources, I think we'll also need:

MessageHeader
Specimen
PractitionerRole (otherwise, I can't find a way to associate both the ordering physician and hospital to the ServiceRequest)

Dev note: Proposed FHIR sample message https://github.com/CDCgov/trusted-intermediary/blob/fhir-sample-data/examples/fhir/MN NBS FHIR Order Message.json -- but we can certainly make adjustments to that as needed

This is a bit blocked on #9675 at the moment.

Fair point @MauriceReeves-usds -- let me discuss with the team to make sure everybody agrees with the split.

@JohnNKing @avnieldravid I made a ticket to track publishing and presenting the draft order IG you're creating to the HL7 WG. It is here: https://app.zenhub.com/workspaces/platform-6182b02547c1130010f459db/issues/gh/cdcgov/prime-reportstream/10382

Thanks @MauriceReeves-usds
@avnieldravid Now that I've merged my PR, the sample links referenced above have changed to:

Example ORM: https://github.com/CDCgov/trusted-intermediary/blob/main/examples/hl7/MN/MN NBS ORM_O01 Lab Order Message.txt
Example FHIR Order: https://github.com/CDCgov/trusted-intermediary/blob/main/examples/fhir/MN NBS FHIR Order Message.json

I believe we are no longer blocked on authenticating against TI.  So this ticket's acceptance criteria will involve that step as well.  Does that align with your thinking @JessicaWNava ?

also @JessicaWNava please note the new links that JOhn provided.  I'm also updating the main ticket.


OrderBy a field in expanded element does not work as expected

This:
var settings = new ODataClientSettings();
settings.BaseUri = new Uri("http://services.odata.org/V4/Northwind/Northwind.svc/");

var client = new ODataClient(settings);

var result = await client.For<Product>()
	.Expand(p=>p.Category)
	.Select(p => new { p.ProductName, p.Category.CategoryName})
	.OrderBy(p=>p.Category.CategoryName)
	.FindEntriesAsync();

var list = result.ToList();

Produces following query:
http://services.odata.org/V4/Northwind/Northwind.svc/Products?$expand=Category($select=CategoryName;$orderby=CategoryName)&$select=ProductName
The resulting list is not ordered by CategoryName. Expected would be the outcome of this query:
http://services.odata.org/V4/Northwind/Northwind.svc/Products?$expand=Category($select=CategoryName)&$select=ProductName&$orderby=Category/CategoryName

Thank you for the issue report. I believed it's fixed now, you can test the version 4.18 that I just uploaded to NuGet.

This is not fixed for the initial scenario. However it works when the navigation property(ies) involved in the order by are not contained by the expand clause.
Working:
var result = await client.For<Product>()     .Expand(p=>p.Supplier)     .OrderBy(p=>p.Category.CategoryName)     .FindEntriesAsync(); var list = result.ToList();
Not working:
var result = await client.For<Product>()     .Expand(p=>p.Category)     .OrderBy(p=>p.Category.CategoryName)     .FindEntriesAsync(); var list = result.ToList();

Thanks for the answer.
Since this is not yet fixed, do I have to add the same issue again?

I am reopening the issue and will investigate it.

Fixed in 4.20.


Workaround caching bug

Closes #9, but not a very satisfying implementation.
I found that std:index delays the entire result, so without specifically calling $success, errors were never emitted.

Oh yeah, though indexing stderr as before would do the same (if it weren't already broken earlier).

:facepalm: It just occurred to me that it could also be
{
  child = std:exec ...
  child:success # (or just $child)
  child:stderr ...
}

I purposefully avoided suggesting that before thinking that it would propagate the id evaluation up and evaluate the {...} block instead (which is as undesirable as id calculation evaluating the exec in the first place), however I forgot that the rules around the index operator for id evaluation is limited to only the expression being immediately evaluate-able (i.e. all captures available), it doesn't propagate upward. Again, the fact it's so hard to reason about that even for the person who wrote it is a bad sign :) Not to mention that rule around the index operator is a total hack.


[SliderIOS] Apply value after minimum/maximumValue in order to ensure it is properly set

value is clamped between min/max and so order of prop application matters - value always ended up being set first in my tests, and consequently a value outside of the default range 0-1 would not work. So this applies the value when the min/max are set.
Gist of broken example

^ the second slider here should have it's cursor in the middle
/cc @tadeuzagallo

While this solution works, I do wonder if it's worth adding a hook on view manager classes that allows us to specify in what order props should be applied, either in JS or Objective-C. eg: instead of having RCTSetViewProps [iterate over each prop in an unreliable order as provided by NSDictionary] the manager class could optionally expose a function that specifies order which RCTSetViewProps can then use to sort.
// RCTSliderManager.m

- (NSArray *)applyPropsFirst
{
  return @[ @"minimumValue", @"maximumValue" ];
}

or...
// RCTSliderManager.m

- (NSArray *)propsOrder
{
  return @[ @"minimumValue", @"maximumValue", @"value" ];
}

This would have been helpful for me on react-native-overlay because I need to ensure visible is set before setting the windowLevel here and on react-native-video because most things are dependent on the src being set.
@tadeuzagallo - thoughts? cc @vjeux

ping @tadeuzagallo - simple change here that we discussed in IRC, would be good to get it into next sync if possible

I've been trying to land this today, but ran into an unexpected merge conflict. Should be sorted shortly.


I'm using node-red-contrid-whatsapp-link, what i do sir to colaborete with whatsapp-web js... helpme please sir.. thanks...

i have do https://stackoverflow.com/questions/78692742/whatsapp-web-js-package-failing-when-scanning-the-qr-code/78692943#78692943,
I have modif file Constants.js
webVersion: '2.3000.1014590669',
webVersionCache: {
type: 'remote',
remotePath: 'https://raw.githubusercontent.com/wppconnect-team/wa-version/main/html/2.3000.1014590669-alpha.html',
},
but still whatapp not running well.. helpme please sir..thanks...

 I only send text messages, I haven't checked if everything works. Good luck.


Update .gitignore

Updated from Python gitignore https://raw.githubusercontent.com/github/gitignore/master/Python.gitignore

Hi, thanks, appreciated!
Please put these in your global gitignore, rather than copy pasting them into every project. E.g., here's mine: https://github.com/Julian/dotfiles/blob/master/.config/git/ignore


no yolov3.tf file

Hi,
I'm trying to run the detect.py file.
There're no files in the checkpoint folder.

My bad,
The convert did its job and I had to refresh to the file.


Add graphviz as a runtime dependency

graphviz is required as a development dependency. So without having graphviz already installed, loading fpgrowth will fail:
/usr/local/lib/ruby/gems/2.7.0/gems/bootsnap-1.4.6/lib/bootsnap/load_path_cache/core_ext/kernel_require.rb:23:in `require': cannot load such file -- graphviz (LoadError)

Any chance on getting this merged?


Enable Xpack security in elasticsearch container

Is there a simple way to enable xpack security inside the elasticsearch container?
Perhaps there can be a choice of enabling elasticsearch-with-ssl and elasticsearch-without-ssl.

In the current state, there isn't a way to do this simply. However, this brings up a great conversation about how we can either customize images (adding a build layer?) or pass config files into the container during the run process.
I created a backlog issue for this in #91.
Thanks for taking Takeout for a spin!


fix problem with OperatorGroup invalid spec (when using gitops)

What is this PR About?
Describe the contents of the PR
The chart produces an invalid spec for OperatorGroups in some scenarios. This becomes a problem when using the chart with ArgoCD.
Example error from ArgoCD
one or more objects failed to apply, reason: OperatorGroup.operators.coreos.com "whatever" is invalid: spec.targetNamespaces: Invalid value: "null": spec.targetNamespaces in body must be of type array: "null".

How do we test this?
Provide commands/steps to test this PR
Apply this application to an ArgoCD instance...
apiVersion: argoproj.io/v1alpha1
kind: Application
metadata:
  name: test-operators-installer
  namespace: openshift-gitops
spec:
  destination:
    namespace: openshift-gitops
    server: 'https://kubernetes.default.svc'
  project: default
  source:
    chart: operators-installer
    repoURL: https://redhat-cop.github.io/helm-charts
    helm:
      valuesObject:
        approveManualInstallPlanViaHook: true
        installPlanApproverAndVerifyJobsImage: quay.io/openshift/origin-cli:4.14
        operatorGroups:
        - name: whatever
          targetOwnNamespace: false
          createNamespace: true
    targetRevision: 2.4.3
  syncPolicy:
    automated:
      selfHeal: true
    syncOptions:
      - ApplyOutOfSyncOnly=true
      - PrunePropagationPolicy=foreground
      - PruneLast=true
      - ServerSideApply=true
      - FailOnSharedResource=true
      - RespectIgnoreDifferences=true
      - CreateNamespace=false

cc: @redhat-cop/day-in-the-life

hey @trevorbox , thanks for the PR! would you mind to bump up the Chart version to make the linter happy please? 💚 thanks!

hey @trevorbox , thanks for the PR! would you mind to bump up the Chart version to make the linter happy please? 💚 thanks!

done

thanks @trevorbox 🙌


[5.5] Fix SQLiteGrammar whereTime method with correct formatting.

strftime('time', created_at) is not working for sqlite and we need to apply correct formatting %H:%M:%S.
It also should be merged to 5.6 and master branches.
You can read more about sqlite date functions here: https://www.sqlite.org/lang_datefunc.html

What is the current behavior, and how can we be sure that this doesn't cause problems for existing installations?

@sisve currently, ->whereTime() method does nothing and no filtering is performed.
SQLite strftime function accepts formatting string. Passing 'time' to it returns 'time' string instead of expected %H:%M:%S formatted time string.

Wrong branch. 5.6 is the current release.

@taylorotwell It's a bugfix. Shouldn't it be merged to current LTS version?
From Laravel contribution doc:
All bug fixes should be sent to the latest stable branch or to the current LTS branch (5.5). Bug fixes should never be sent to the master branch unless they fix features that exist only in the upcoming release.

@taylorotwell it should be merged with #23321

@sisve @taylorotwell ping

I do not have the power of merging. Try sending a new PR with identical content.


Drop support for Python <=3.8 and TensorFlow <=2.4

This drops support for version 3.8 and older which also means that we now require TensorFlow 2.5 and newer as it is the first release that shipped with wheels for Python 3.9.
See https://github.com/larq/compute-engine/pull/786#issuecomment-1668196582

I started a test release here let's see what CI thinks about this.

I started a test release here let's see what CI thinks about this.

Looks like the build was successful apart from some issues with the Android benchmarker build: #788

I started a test release here let's see what CI thinks about this.

Looks like the build was successful apart from some issues with the Android benchmarker build: #788


Problems setting a different default theme in site.yml

I'm not sure if this is a bug or just my own incompetence, but I'm looking for a little help setting a theme in site.yml.
I'm trying to use this theme: repo link
From my understanding of the documentation, all I need to do is change the theme parameter in site.yml to the URL of the .zip of my desired theme, so my site.yml reads:
theme: https://github.com/digisavvy/some-like-it-neat/archive/master.zip

I have also tried it as:
theme: 'https://github.com/digisavvy/some-like-it-neat/archive/master.zip'

I get the following error:
==> spwp.dev: FATAL: Stacktrace dumped to /var/chef/cache/chef-stacktrace.out
==> spwp.dev: ERROR: bash[WordPress https://github.com/digisavvy/some-like-it-neat/archive/master.zip activate] (wpcli::install line 185) had an error: Mixlib::ShellOut::ShellCommandFailed: Expected process to exit with [0], but received '1'
==> spwp.dev: ---- Begin output of "bash"  "/tmp/chef-script20150718-26805-m5zhk6" ----
==> spwp.dev: STDOUT:
==> spwp.dev: STDERR: Error: The 'master' theme could not be found.
==> spwp.dev: ---- End output of "bash"  "/tmp/chef-script20150718-26805-m5zhk6" ----
==> spwp.dev: Ran "bash"  "/tmp/chef-script20150718-26805-m5zhk6" returned 1
==> spwp.dev: FATAL: Chef::Exceptions::ChildConvergeError: Chef run process exited unsuccessfully (exit code 1)
Chef never successfully completed! Any errors should be visible in the
output above. Please fix your recipes so that they properly complete.

I'm not sure if maybe linking directly to the github .zip download is an issue or not. Should I try hosting it on a server of my own instead? Or maybe I'm just missing something simple somewhere.

Thanks!
I released 2.12.0.
https://github.com/vccw-team/vccw/releases/tag/2.12.0

:+1:
Awesome. Thanks for the quick response and fix!


[CodeCompletion] Don’t report a call pattern as convertible if its result type doesn’t match

To compute the expected type of a call pattern (which is the return type of the function if that call pattern is being used), we called getTypeForCompletion for the entire call, in the same way that we do for the code completion token. However, this pattern does not generally work. For the code completion token it worked because the code completion expression doesn’t have an inherent type and it inherits the type solely from its context. Calls, however, have an inherent return type and that type gets assigned as the typeForCompletion.
To fix this, check if we recorded a fix that allowed a type mismatch between ParentCall and its surrounding context. If this is the case use the surrounding type as the ExpectedCallType. Finally, to make this work, we need to ensure that we don’t suppress the recording of such type mismatch fixes in code completion mode.

@swift-ci Please smoke test

@swift-ci Please smoke test

@swift-ci Please smoke test


is there anyway for normal PIP mode for without video player for both android and iOS?

Is there any way to show our custom widget in PIP in ios?

No. iOS does not support custom views in PIP. Only the system video player is supported (as of iOS 16).

can someone tell that how to integrate this normal video player in pip mode in ios?.for me its not working.


CI fails on alpine linux

Hi!
Here is the log - http://sparrowhub.io:2222/report/159
HTH
Alexey

Thanks, it looks like the failing test is system-dependent, but has also uncovered some general issues when fontconfig can't find a matching font.
Will investigate.

Sure. If you want to test on alpine linux (SparkyCI),  please see my PR - https://github.com/pdf-raku/FontConfig-raku/pull/2 and please setup repo on SparkyCI , thanks

Latest commit handles the situation where a FontConfig pattern doesn't match any fonts, both in the library and basic.t test.
Also added explicit empty-config.t to test the case of an empty configuration file that shouldn't match any tests.

@melezhik not sure if I'm in the right place or doing the right thing.
I logged into http://sparrowhub.io:2222/ and authenticated github.
Ended up here after trying to add a repo.

Did you click "Add repo" button?

oh, it's listed in the active repos under your account, so I think you did ... let me check within SparkyCI database if your repo is added ...

I just got to enter a repo name, but nothing else

oh, yeah, looks like the issue is that FontConfig-raku belongs to pdf-raku org not to @dwarring  user ... , SparkyCI expects that a repo belong to user, so if one just enter a repository manually ( ignoring suggestion list ), Sparky ends up in a wrong github repo.
Let me adjust it for you , but I need to think how to deal with org based repositories in the future ...

I changed repo name to correct one in internal SparkyCI database, you will have to see build report soon.
Please be aware that now repo is named as gh-dwarring-FontConfig-raku and this is what you should see when login and list your active repositories ...

here we are! build succeeded now with your latest commit - http://sparrowhub.io:2222/report/187 🍾

thanks!

Fixes released with FontConfig 0.0.4


[Call for Volunteers] WeCom Puppets: wechaty-puppet-wxwork is going to ready for testing!

Hello to developers who are using Wechaty,
We have excellent news for you:
We have launched a new puppet wechaty-puppet-wxwork. It is ready for testing, and Alpha Testing Volunteers are needed.
wechaty-puppet-wxwork is a WeCom(original WeChat Work) solution. Currently it supports following features:

Send / Receive Text, Image, Video, File, UrlLink, MiniProgram messages
Get Corporation and Individual contacts
Get All kinds of rooms with room members
Modify alias, phone, corporation and description on a contact
Add / Remove members to / from room
Modify room topic
Room events(join, leave, topic)

If you want to benefit from the new puppet, you are welcomed to apply for joining the Alpha Testing List: https://github.com/juzibot/wxwork-tester
You can leave a comment under this issue, introduce yourself about who you are and what you want to do (please DO follow the following TEMPLATE and fill it in details, or you might not be able to be selected), so that we can be able to add you to the alpha testing list if you are qualified.

 (YES/NO)

Gambling
Adult & Sex

Your Email:

Your Wechat[must, please make sure this WeChat is searchable]:

I Had Joined the Wechaty Developers' Home Wechat Room by Scan QRCode from README.md: (Please join the group if you are not, and then say YES at here)

I Had Read the Wechaty API Doc: http://wechaty.js.org/docs/introduction :(YES/NO)

I Had Used wechaty-puppet-padplus or wechaty-puppet-donut Before Taking Part in This Test: (YES/NO)

I understand that the Wechaty Puppet Wxwork is in very Early Alpha Stage, and I have the necessary engineering technics to deal with the bugs instead of just asking for support: (Please say YES, then followed by your Name at here)

Optional:

[ ] Strong willingness & motivation in wechaty-puppet-wxwork
[ ] Have the ability & engineering power to fix the basic bug instead of just asking for support
[ ] Good to communicate with the developers in the community
[ ] The basic requirement for bot online time: we might take back the test token back if the bot is offline for more than 1 day.
[ ] Willing to write a blog to share the experience of using wechaty-puppet-wxwork

Wechaty Puppet Wxwork Alpha Tester Apply Template

Introduce Yourself(who you are): Simon Liang, CEO of https://xanthous.cn
Explain the Problem You Want to Solve:(what you do): Automate WxWork groups management using bots, and integrate into https://github.com/xanthous-tech/chat-operator
Are Your Bot Related with the Following Tasks? (YES/NO)

Gambling - NO
Adult & Sex - NO

Your Email<EMAIL_ADDRESS>Your Wechat[must, please make sure this WeChat is searchable]: 27326835
I Had Joined the Wechaty Developers' Home Wechat Room by Scan QRCode from README.md: (Please join the group if you are not, and then say YES at here) YES
I Had Read the Wechaty API Doc: http://wechaty.js.org/docs/introduction :(YES/NO) YES
I Had Used wechaty-puppet-padplus or wechaty-puppet-donut Before Taking Part in This Test: (YES/NO) YES
I understand that the Wechaty Puppet Wxwork is in very Early Alpha Stage, and I have the necessary engineering technics to deal with the bugs instead of just asking for support: (Please say YES, then followed by your Name at here)

YES, Simon Liang
Optional:

[x]  Strong willingness & motivation in wechaty-puppet-wxwork
[x]  Have the ability & engineering power to fix the basic bug instead of just asking for support
[x]  Good to communicate with the developers in the community
[x]  The basic requirement for bot online time: we might take back the test token back if the bot is offline for more than 1 day.
[x]  Willing to write a blog to share the experience of using wechaty-puppet-wxwork

@lhr0909 your application are approved!

Wechaty Puppet Wxwork Alpha Tester Apply Template
Introduce Yourself(who you are): wu yiqun, developer of http://www.xinxiaodian.com
Explain the Problem You Want to Solve:(what you do): Automate WxWork groups management using bots
Are Your Bot Related with the Following Tasks? (YES/NO)
Gambling - NO
Adult & Sex - NO
Your Email<EMAIL_ADDRESS>Your Wechat[must, please make sure this WeChat is searchable]: wuyiqun_9
I Had Joined the Wechaty Developers' Home Wechat Room by Scan QRCode from README.md: (Please join the group if you are not, and then say YES at here) YES
I Had Read the Wechaty API Doc: http://wechaty.js.org/docs/introduction :(YES/NO) YES
I Had Used wechaty-puppet-padplus or wechaty-puppet-donut Before Taking Part in This Test: (YES/NO) YES
I understand that the Wechaty Puppet Wxwork is in very Early Alpha Stage, and I have the necessary engineering technics to deal with the bugs instead of just asking for support: (Please say YES, then followed by your Name at here)
YES, wu yiqun
Optional:

[x]  Strong willingness & motivation in wechaty-puppet-wxwork
[x] Have the ability & engineering power to fix the basic bug instead of just asking for support
[x] Good to communicate with the developers in the community
[x]The basic requirement for bot online time: we might take back the test token back if the bot is offline for more than 1 day.
[x] Willing to write a blog to share the experience of using wechaty-puppet-wxwork

I believe the testing period is closed because we are using https://github.com/wechaty/puppet-service-providers for our Wechaty Puppet Service Providers to accept the token applications.
Please go to https://github.com/wechaty/puppet-service-providers/issues/new/choose to apply a token that you need, and file an issue with the right token type to send the application.
Link to https://github.com/wechaty/puppet-service-providers/issues/1


RF5: Recovery Flow using Seed Words

As a user, I want to restore my Hexa wallet using my seed word backup, so that I don't have to use cloud or level backup mechanism to restore.
This is part of RF5, please refer to RF5: https://trello.com/c/Od0akFNu
This issue covers the creation of screens and adding navigation based on UX flow including Screens, navigation, actions, and reducers.
Changes to backend and business logic are covered by https://github.com/bithyve/hexa/issues/3086
Trigger: User opts for recovering using Seed Words
Precondition: User has backed up using SW
Flow:

The user chooses recover using Hexa Seed Words.
They are asked to enter the SW and scan QRs from the Recovery Kit
Question is asked.
Wallet is restored with wallet image from the server for wallet meta data

(If they use this outside it will not have meta data)

Screens names with xd links to follow

https://xd.adobe.com/view/3df97271-f272-4dae-975d-95d4785f1b30-68aa/
Restore using Recovery Kit
Restore using Seed Words – Q$A
Restore using Recovery Kit – Question Confirmed
Restore using Seed Words –None Entered
Restore using Seed Words – Enter 2 Word Suggestions
Restore using Seed Words – All Words Entered
Restore using Recovery Kit – Seed Words Confirmed
Restore using Seed Words – Scan PDF
Restore using Recovery Kit – Recovery Kit Confirmed
Success Screen – Wallet restored using sw
Restore button screen seems to be pending.


Pull folder from NetSuite on Mac

Steps to reproduce:

On a Mac
Open the SuiteScript folder as the root in VS Code
Right-click in the SUITESCRIPT sidebar and select Pull folder from NetSuite

What happens:

Nothing changes in VS Code even after hitting the refresh button.
Open the SuiteScript folder in finder and you will see the all the files at the same level as SuiteScript, but with the folder names embedded with back slashes.

What should happen:

It shouldn't use back slashes for the path separators.
The files should be placed in the correct folders on the HD.

I still have this issue.

If you go to my fork of the repo here: https://github.com/joshhidley/netsuite-upload/tree/platform-path-fix/build, I implemented the fix.
If you don't want to build it, and just want the vsix file to install it, I put it here: https://github.com/joshhidley/netsuite-upload/tree/platform-path-fix/build. I'm not sure of the best way to down load it from there.
I make no claims about this. Use at your own risk.

Thanks so much it works great!
This is how I implemented @joshhidley vsix file.

Download the file vsix file from Github.
In VS Code open extensions
At the top right click the menu icon
select the option Install from VSIX...
find your vsix file in folders and select it
YOUR GOOD TO GO!!

This should be good to go in the 1.0.0 release. See releases tab of this project.
Once I get some beta testing done, I'll release the update to the marketplace.


Decouple from router (only build routes), simplify route processing, add lifecycle events, add redirects

Issues I was running into
I was having issues with the  component and which routes were supposed to be rendering but weren't because Switch will render the first route that matches and none else.
Ex: { path: '/app/:someVar' } will render if it appears in a list above  { path: '/app/path' } and '/app/path' will never render.
Also - If you've got child routes, and your route doesn't match then none of your child routes will either
So if you have { path: '/app', exact: true, childRoutes: [{ path: 'subRoute' }] }
Then you can never navigate to the component at /app/subRoute

To remedy this, I you can have structural route parts that don't have any component like here: https://github.com/BigRoomStudios/nearpeer-frontend/blob/955e042957a4f2ef96c27cdc38bf1ccf6b6c4459/src/routes/app/index.js#L158-L160

Some improvements provided in this PR

Decouple the use of any routers inside this lib, only use it to render routes, given a config
Remove the server-side rendering bit until we have a chance to test it out
Introduce redirect setting that will redirect if no routes were matched at that level

Tips

It helps to organize your paths by having a structural route part like linked above. See in this example for the path app/dms/ here: https://github.com/BigRoomStudios/nearpeer-frontend/blob/955e042957a4f2ef96c27cdc38bf1ccf6b6c4459/src/routes/app/index.js#L109-L120
Referencing the above link, to match a route at the structural path property you create a slash route with { path: '/', exact: true } -- making sure to have exact set to true

Hey @wswoodruff could you summarize the problems you encountered and changes made here?  Maybe best to in person?  Also, it seems like there are two major changes in here: 1. lifecycle methods and 2. a big refactor to fix some bugs.  We might want to update the title/description to reflect that.

This config will now redirect to 404 if no other routes matched.
Without the redirect setting, the unmatched url will stay in the address bar and any routes that matched will be rendered (The app will probably look broken)
[
    App(store),
    {
        path: '404',
        component: FourOFour
    },
    {
        path: '/',
        component: PlainLayout,
        childRoutes: [
            Home,
            ...Signup,
            ...Login,
            ...Legal
        ],
        redirect: {
            to: '404'
        }
    }
]

So are these two identical?
    {
        path: '/',
        component: PlainLayout,
        childRoutes: [
            Home,
            ...Signup,
            ...Login,
            ...Legal
        ],
        redirect: {
            to: '404'
        }
    }

and
    {
        path: '/',
        component: PlainLayout,
        childRoutes: [
            Home,
            ...Signup,
            ...Login,
            ...Legal,
            { redirect: {  to: '404' } }
        ]
    }

Currently that redirect object you have will require a path prop. Then those 2 are identical
{ path: '', redirect: { to: '404' } }

The latter is the more general case, and the one most important to support.  From the code I think it will try to render a <Route/> component.

Ok I'm on it

This example from the docs seems relevant,
<Switch>
  <Redirect from='/old-path' to='/new-path'/>
  <Route path='/new-path' component={Place}/>
</Switch>

Worth keeping in mind that 404 handling is now just a special case of the more general feature of redirecting!

Yeah that's really cool that 404 handling is a special case now 👍

With commit #e8ec420 you can have a childRoute item that looks how you specified it: { redirect: { to: '404' } }

Awesome!
Can you describe what the redirect means here?  Is it really just a way to define a catchall?  If so, is there a reason it shouldn't also be able to be a route?  Should we have two ways of defining a catchall, given that we can already place a catchall at the end of childRoutes?  Don't want to overthink this, but also want to keep things simple.  Having one way to do a given thing is a useful constraint.  CC @bmleight for some thoughts.
    {
        path: '/',
        component: PlainLayout,
        childRoutes: [
            Home,
            ...Signup,
            ...Login,
            ...Legal
        ],
        redirect: {
            to: '404'
        }
    }

The configuration you have there ☝will add a redirect component to the bottom of the switch component for childRoutes, if childRoutes are listed.
If there are no childRoutes, then this just renders a redirect component, not wrapped in a Switch component.
Since the component prop is also optional on routes, you can have configs like this:
{
    path: 'oldRoute',
    redirect: {
        to: 'newRoute'
    }
}

Question: do we care or even want that? loLL

I would expect that to be,
{
    redirect: {
        from: 'oldRoute',
        to: 'newRoute'
    }
}

Where from is the fully resolved path (when nested deeply).

I don't understand what it would mean to have a from on a route's redirect—my impression is that that was specifically for catch-alls.  And I'm not feeling great about redirects not being supported in childRoutes if there is a redirect on the route itself.  For me all signs are still pointing to removing the route's redirect.  It seems to be causing some issues but not offer any more flexibility.

Yeah it is confusing. We should make the redirect prop exclusive then and not allow any other props if that one exists

👌
// This isn't a redirect, or it is a redirect and only has the "redirect" property

!route.redirect || Object.keys(route).length === 1

With these latest changes, the onWillMount that calls history.replace is equal in functionality to the redirect below.
{
    path: 'connections',
    // onWillMount: (route, match, location, history) => history.replace('/app/connections/dms'),
    component: ConnectionsTabbedSearch,
    childRoutes: [

        // This redirect may be used in place of the onWillMount func above
        { redirect: { from: '/', exact: true, to: 'dms' } },

        {
            path: 'dms',
            childRoutes: [
                {
                    path: '/',
                    exact: true,
                    component: DirectMessagesPage,
....

@wswoodruff also keep in mind that to can be an object, in which case we probably want to adjust its pathname.


Support discrete action in HER replay buffer

Currently, the code doesn't work because it does not convert the action into a one-hot vector.

Addressed with #32


Cleaned Up Client-Side Login-Related Temporary Code

Resolved #22

Hi Taiheng,
I've reviewed your recent pull request involving the deprecation of FileUserDataAccessObject, removal of test clients, and various code updates. Here's my feedback based on our standard review criteria:

Coding Style:

Imports: The use of wildcard imports (e.g., import client.*;) is generally discouraged as it can lead to namespace conflicts and make the code less readable. Prefer explicit class imports.
Consistency: There's consistency in the code's formatting, which is good. However, ensure consistent use of indentation and spacing, especially in LoginUseCaseFactory.
Naming Conventions: Naming conventions are followed well. Class and variable names are descriptive and adhere to Java standards.

Correctness:

The code compiles and runs as expected. However, there is a use of RuntimeException in main which might not be the best practice for exception handling. Consider more specific exception types or custom exceptions for better error resolution.
The refactoring from FileUserDataAccessObject to ServerDataAccessObject seems appropriately handled.

Test Coverage:

With the removal of TestClientUI.java and TempClient.java, ensure there are sufficient tests covering the functionalities previously tested by these.
Consider adding unit tests for new methods in ServerDataAccessObject to ensure their correctness, especially for network interactions.

Architectural Alignment:

The changes adhere to Clean Architecture, with clear separation between entities, use cases, and interface adapters.
However, the presence of TODO comments (e.g., in LoginUseCaseFactory) suggests unfinished work which should be addressed or tracked via issues.

Documentation:

Javadoc comments are missing for new methods and classes. It's important to keep the documentation updated to match the code changes for maintainability and clarity.
The TODO comments should be accompanied by more detail or a reference to an issue in the project's tracking system.

Clarity of Accompanying Information:

The pull request description adequately explains the changes made, but it would benefit from a more detailed explanation of why certain files were removed and how the deprecation of FileUserDataAccessObject impacts the system.
Ensure commit messages clearly describe the changes for an accurate and useful version history.
Action Items:
Update and expand test coverage for new and modified code.
Add comprehensive Javadoc comments for new classes and methods.
The changes made in this pull request show good progress towards enhancing the system's structure and maintainability. Addressing these points will further improve code quality and project robustness.
Best,
Hannah


Stop deleting runs on hard deletion

Description

Following the recent work around the run execution. This PR stops deleting all dust app runs when we hard delete a workspace. Those runs will be deleted automatically after 30 days anyway (see https://github.com/dust-tt/tasks/issues/726).
Risk

Deploy Plan

I don’t see the runs diff is that expected?

I don’t see the runs diff is that expected?

What do you mean, the function still exists and will be used by the cron. This PR just removes the call to the stored function that was attempting to delete all the runs.


Question about using optical flow while inference

Hi, sorry for disturbing again. I wanted to say that when I am using optical flow while implementing TMO I am getting the same results as paper, but when I don't use optical flow while inference, I am getting lower results.(I am implementing your pretrained model and I am setting flow path to JPEGImages instead of JPEGFlows)
I am asking this question because paper says that TMO can run with and without optical flow.
I just want to know that when you run TMO with and without optical flow, are you getting the same results? or the results of using optical flow is higher than the results of not using it?

Hi, TMO can handle both RGB images and optical flow maps as motion input, but the accuracies are different.
The optimal source information varies by the target dataset (check Table 3 in the paper).


Add CosmosDB-compatible Document and Index Stores

Description
Added Document and Index stores compatible with CosmosDB.
Motivation from existing MongoDB stores being incompatible with CosmosDB due to incompatible naming schemes. Simple change of connection names' / characters changed to _.
Fixes #3423
Example Usage for CosmosDB URI
from llama_index.storage.docstore import CosmosDBDocStore
from llama_index.storage.index_store import CosmosDBIndexStore

doc_store = CosmosDBDocStore.from_uri("mongodb://server:port")
index_store = CosmosDBIndexStore.from_uri("mongodb://server:port")

Type of Change
Please delete options that are not relevant.

[ ] Bug fix (non-breaking change which fixes an issue)
[X] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[X] This change requires a documentation update

How Has This Been Tested?
Ran it as a drop-in replacement for MongoDB Document and Index stores in an existing deployment with a CosmosDB backend. Not a major change at all.
Suggested Checklist:

[X] I have performed a self-review of my own code
[X] I have commented my code, particularly in hard-to-understand areas
[ ] ~I have made corresponding changes to the documentation~ Added CosmosDB Doc and Index Stores to the llama_index.storage.docstore and llama_index.storage.index_store packages for use, drop-in replacement where docs are same as existing MongoDB stores
[X] My changes generate no new warnings
[X] I have added tests that prove my fix is effective or that my feature works
[X] New and existing unit tests pass locally with my changes

@ntflix Since this is the exact same as the mongo index, maybe a better solution is allowing the user to customize the suffixes of the namespaces?


Observing Istio control plane

Istio control plane
These changes intend to add a new MAL rule and UI template to analyze and render Istio metrics exported by the Istio control plane(Istiod), main changes include：

[x] MAL rule file
[x] UI template file
[x] Add OpenTelemetry receiver to replace OpenCensus receiver
[x] Create metric tables or indices on the initial phase instead of runtime.
[x] Remove istio mixer receiver

Check list:

[x] Update the documentation to include this new feature.
[x] If it's UI related, attach the screenshots below.
[x] If this pull request closes/resolves/fixes an existing issue, replace the issue number. Closes #5810 .
[x] Update the CHANGES log.

UI screenshot:

Look like tests fail, please recheck or rerun.

I have feeling whether this PR creating too many metrics to cause the unstable.

I have feeling whether this PR creating too many metrics to cause the unstable.

Yep, I also fall in with that. How about disabling Istio relevant tables generation in e2e since there is not a single test case for it. That means we don't have a test flow to guarantee whether all of the metrics work together, does it bring and concerns to you?


ipam: Use static service loopback address

Reduce the number of allocations required in preparation for external IPAM and
ENI support. The service loopback address can be arbitrary as long as the
address is unused, default to <IP_ADDRESS> and allow the user to overwrite it.

This change is

Coverage decreased (-0.003%) to 41.559% when pulling 00bbc6f4f378fedc30e420adc999b3cff08640de on pr/tgraf/loopback-ip into 60f2731d00986928a6d815dc8c129838121bdf90 on master.

test-me-please

test-me-please


Allow arbitrary catch clause type annotations

Fixes #20024
Playground
This is a follow-up to #52240 to exercise my comment in https://github.com/microsoft/TypeScript/pull/52240#issuecomment-1382691157, pulled into a PR for design meeting demoing.
The actual change is https://github.com/microsoft/TypeScript/commit/db0c9b08823c9b8365d45ce746e385c0ac99a94c (i.e. deletions only!), the rest is #52240 as it's not merged in main.
This probably conflicts with #13219 somehow, but I expect that the code in #52240 and this PR that grabs the type for the catch clause would just be updated. That or if we do something like #51390 this should still compose (as it's just another "default" implicit annotation).

@typescript-bot pack this

Weird:

We have no test that exercises this; I guess the catch clause isn't taking part in code flow, somehow.

Semantic tokens! shakes fist

I posted this because it was easy after #52240, but per design meeting it seems that this feature isn't palatable (which is totally fair!); closing.


Use decoded command to determine length

The old method of determining an end of packet was a threshold duration at the end of the bit. However, controllers may respond almost instantly, which would cause end-of-packets to not be recognized. This is basically a complete rewrite of the analyzer to use the command to determine how long the packet should be. This also implements taking advantage of FrameV2 to analyze the data.

This should fix #2


QoL: File Select Shows Items, Songs, Dungeon Rewards, and other information

We talked about this last weekend but with Pre-Completed dungeons being a thing, I think this file select feature would be nice to have now compared to a few months ago. It's a bit more essential, to know which dungeon rewards you start with besides OOT's (so including the Remains on the file select) as well as what songs (if any) you start with and what not. Ideally, similar in design to below:

I'm unsure how you would realign or restructure this to fit MM's Remains and any of its items (Deku, Goron, Zora) and also taking into account shared items could make this trickier, too. But I think at the least if we were to do just a version 1 of this for whatever reason I think it should primarily be dungeon rewards and songs. But if you can figure out something that looks good and encompasses many settings combinations in a version 1, great!

Are you still interested in doing something like this @Nax ? At the very least we could have a basic one showing your remains?

Still on the menu. We're looking to first port the file select actor so we can more easily edit it, rather than hacking into it with assembly. This will take a bit though.

It's there, lacking some stuff tho


Mod resets custom keybinds on use.

I always used custom keybindings, but once i start using the mod blueprint mode it jumps back to default keybindings.
Very annoying, sadly.

Would you please describe which key is reseted, and how it happens?

Ehm i thought that would be clear: ALL OF THEM.
Every custom keybind of Satisfactory (not of the mod) is set back to default once i edit a blueprint manager.
Before:

After:

Thanks for reporting. I will fix it in 0.1.1 today, both EA and Experimant build are supported.

I think its fair to inform you that its finally working for me like a charm with new Version.
Great Mod, keep up the good work.


New aka.ms link

Aka.ms link name
joincloudycounty
Category (Optional. Select None to auto-select)
None

Thank you for submitting aka.ms/joincloudycounty. Your link already exists at akaSearch.net. 🙏✅


Adding complement string representation for conns

#380 (base PR)
issue #441

in this PR, adding a complement for string representation of a connection-set.
for example: writing All but: UDP 5353 instead of "SCTP 1-65535,TCP 1-65535,UDP 1-5352,5354-65535"

currently: the decision if to use the connectionSet string as is or its complement string is choosing the shorter string between the two.

@shireenf-ibm  Le'ts close this PR for now, and decide later if/when this enhancement should be added.

to be decided later when to add this enhancement


MSC2771: Bookmarks

Rendered
Signed-Off-By: MTRNord<EMAIL_ADDRESS>

I am working on a tui based reference impl "utility" (Not a full MX Client but a todo application focused on just this feature) which implements both approaches but defaults to the main versions when sending.
The unstable prefixes are dev.nordgedanken.bookmarks.room for the main proposal and dev.nordgedanken.bookmarks.global for the alternative.
(I might also build a small flutter app for this allowing synced todos across devices)

Closing due to not having the time to maintain


Fix bug where client with TrustNT enrolment can't accept invitation

@umhmrc The problem is that Services.supportedTrustServices is used, among other things, to populate the list of selectable services in the invitation journey. So then we would have to de-duplicate trust and trustNT before displaying that page and we would end up with a different hack elsewhere.
I agree that we should think up a less clumsy model to take into account that services, client id types and enrolment keys are not necessarily in a 1:1:1 relationship, but that's for when we have more time...


Using create_training_graph with act_quant not found error

System information

Have I written custom code (as opposed to using a stock example script provided in TensorFlow):
Yes
OS Platform and Distribution (e.g., Linux Ubuntu 16.04):
Linux Ubuntu 16.04
TensorFlow installed from (source or binary):
Binary
TensorFlow version (use command below):
1.9.0
Python version:
3.6
CUDA/cuDNN version:
9.0
GPU model and memory:

After the addition of the quantization training code like:
......
full_precision_graph = tf.get_default_graph()
with tf.variable_scope(params['model_scope']):
build_the_network()
tensorflow.contrib.quantize.create_training_graph(input_graph=full_precision_graph, quant_delay=0)
......

I intend to train a int8 model from a pre-trained f32 ckpt model. However, the error raised as
......
netname/act_quant/max not found in checkpoint
......

I am wondering if something wrong with my usage of the quantization creating. I only defined the train quantization net and the screen did print out something like the following information before the error occurred
......
INFO:tensorflow:Skipping netname/conv3_eltwise4/add, because its followed by an activation.
....

Okay I know the problem. No such error if I train from the scratch...So what's the correct way to train from a pre-trained f32 model that contains no elements like /act_quant/max?

I kinda solved it...In case of fine-tuning from full-precision model without quantization parameters like .../act_quant/min, one just restore the weight and let the graph fill the default quantization parameters. Passing the tf.saver APIs using the existed weights and biases will do this. And I think the current tf APIs is enough to achives this and no need of extra support for this. Anyway, thanks for the attention from @ymodak.

Thanks for the update, closing since it seems to be worked around now.

@asjmasjm what is the workaround? I am using the export_inference_graph.py to export a quantized model, but I am getting the error FeatureExtractor/MobilenetV3/Conv/conv_quant/max not found in checkpoint. I have trained the model myself (starting from f32 but trained a quantized model).

Problem still unsolved.

I'm also experiencing this issue as well, any insights or solutions would be great


haddock-api 2.21.0 and haddock-library 1.6.1 release

Updates .cabal files,`CHANGELOG's, README, and Travis CI.
Also accepted an incorrectly failing test case.

Will the new version support ghc-8.6.1? It would be great to get that release soon because currently haddock-library blocks all kinds of dependent packages, like pandoc, from building with the latest compiler.

Will the new version support ghc-8.6.1? It would be great to get that release soon because currently haddock-library blocks all kinds of dependent packages, like pandoc, from building with the latest compiler.

Yes, it will support ghc-8.6. The release should be coming soon.

Ping?

Ben, I am travelling for some time. Herbert and Alec are in charge here.
But I guess nothing is stopping here, right @hvr, @harpocrates?
Ben Gamari<EMAIL_ADDRESS>schrieb am Do., 11. Okt. 2018, 14:23:

@alexbiehl https://github.com/alexbiehl, do you think you will get to
this soon?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/haskell/haddock/pull/945#issuecomment-429123476, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AByiiTi4WkOT-7KRhLR6J0gFREI_V7tpks5uj7bPgaJpZM4W1WG8
.

This is ready.
I don't have the permissions to release to Hackage. I've been pinging Herbert regularly, but I think he's been busy with other things. The plan is to merge this right before publishing on Hackage.


Cosmos emulator download script with retry

We are experiencing some reliability issues with the download of the Cosmos emulator from the edge CDN. It will occasionally fail resulting in the build failing. To help add more resiliency we should improve the script to add retry logic.
Rather than add more complex loops to the YAML file we should instead extract out the existing download logic and put it into a script and then create another YAML file which wraps the script. Because we download the emulator across all platforms we should place this script, and the YAML file in the azure-sdk-tools repository common directory. But initially we can develop and debug it in the Java repo.

@kushagraThapar @moderakh Is this still an issue? Have there been any recent failures? If so can you point to them, otherwise can we close this.

@chidozieononiwu This happens every now and then, but definitely less frequent than it used to happen back in the day (last year when we filed this issue). Did we add any retries or is it just better networking now ?


fix(settings): obfuscate api keys

Hide saved API keys in the UI.

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


chore: add semaphore artifacts

Description

Since the Semaphore v4 circuit changed, the zk artifacts were updated.

More tree depth will now be supported, from 2 to 32.

Note: This PR should be merged after the Semaphore repo is updated to support the circuit changes as well.
Related Issue(s)
Re https://github.com/semaphore-protocol/semaphore/issues/778

@sripwoud re changeset and versioning. Considering https://github.com/privacy-scaling-explorations/snark-artifacts/issues/14#issuecomment-2105154616, I guess we should bump version 4.0.0-beta for this package right?

@vplasencia Thanks for the PR.
With @cedoor we are still in the process of defining the best way to version those artifacts.
What do you guys think of
https://github.com/privacy-scaling-explorations/snark-artifacts/issues/14#issuecomment-2106981820
If we agree on it, then this PR should set the version in @zk-kit/semaphore-artifacts
https://github.com/privacy-scaling-explorations/snark-artifacts/blob/2ca667565384b9b6bcee705715e00e5d0023d75c/packages/semaphore/package.json#L5
to what we have in @semaphore-protocol/circuits
https://github.com/semaphore-protocol/semaphore/blob/47bfebd4ed4b38ee0c19fff67849380ee79ea3f1/packages/circuits/package.json#L3
and mention @semaphore-protocol/circuits in the @zk-kit/semaphore-artifacts description
https://github.com/privacy-scaling-explorations/snark-artifacts/blob/2ca667565384b9b6bcee705715e00e5d0023d75c/packages/semaphore/package.json#L3

@vplasencia I assume you scripted the creation of these artifacts? Asking because of #33  :smile:

Hey @sripwoud
Regarding this comment: https://github.com/privacy-scaling-explorations/snark-artifacts/pull/32#pullrequestreview-2052078554
I agree. Please, let me know when we all agree so that I can change those values.
Regarding this comment: https://github.com/privacy-scaling-explorations/snark-artifacts/pull/32#issuecomment-2107011708
Yes, we have scripts for everything related to the artifacts. Here is the repo: https://github.com/vplasencia/semaphorev4-generate-zk-artifacts
There is one script to generate the files with the specific format for this repo: https://github.com/vplasencia/semaphorev4-generate-zk-artifacts/blob/main/scripts/getWasmZkeyFiles.ts
I was thinking about integrating those scripts into circomkit (which is what Semaphore uses for circuits) or maybe create a plugin but a new tool adapted to this repo should work too.
Let me know if I can help with the issue https://github.com/privacy-scaling-explorations/snark-artifacts/issues/33

@vplasencia we agreed on keeping the same version of the circuit package if it exists. (in this case @semaphore-protocol/circuits), and update that link only when the circuit is updated.
A new circuit field needs also to be added to the package.json file, like here: https://github.com/privacy-scaling-explorations/snark-artifacts/blob/4023ed5a9c87c58de6b84475accdd30e8b4ff73c/packages/semaphore-identity/package.json#L5

Hey @cedoor! Then the version of these artifacts should be 4.0.0-beta.9 because it's the current version of the Semaphore circuit, right?

I just added the new circuit field. When the new Semaphore version is released, I can update the version here.

@vplasencia Yes, 4.0.0-beta.9 is fine for now!

@cedoor perfect, done 👍

Great! Ready to be merged :)


Tons of indexing

Could we mark bower_components and node_modules as excluded? Unsure if it indexes them as library root or not, but I get a ton of indexing for no reason all the time.

done 😉
see https://github.com/Turbo87/intellij-emberjs/issues/24

@Turbo87 if this is done, do you have any idea why indexing would be happening so much and for so long then?

@rwwagner90 you need to explicitly disable indexing of those folders in the plugin settings. Indexing them has some advantages, but at this time I tend to agree that they are mostly not worth the extra CPU cycles so I'm thinking of switching the default in the near future.

@Turbo87 how would I go about doing that? I'm unsure how to change the settings.

Never mind, I found it

hmm, indeed that settings dialog page isn't really documented anywhere. do you feel like opening a PR to fix that?


Don't hard-code the name of the webhook service

This allows users to rename the service and deployment, so long as they
also update the env var value specified in webhook.yaml
More progress toward #1713
cc @silverlyra
Submitter Checklist
These are the criteria that every PR should meet, please check them off as you
review them:

[x] Includes tests (if functionality changed/added)
[ ] Includes docs (if user facing)
[x] Commit messages follow commit message best practices

See the contribution guide for more details.
Double check this list of stuff that's easy to miss:

If you are adding a new binary/image to the cmd dir, please update
the release Task to build and release this image.

Reviewer Notes
If API changes are included, additive changes must be approved by at least two OWNERS and backwards incompatible changes must be approved by more than 50% of the OWNERS, and they must first be added in a backwards compatible way.
Release Notes
The webhook service and deployment can have a name specified by the user.

/hold
@ImJasonH do we want webhook-asdfasdf in there though ? :sweat_smile:

Whooooops! Good catch. Fixed.

/test pull-tekton-pipeline-integration-tests


⚠ Samsung has degraded performance

In 932baef, Samsung (https://samsung.com) experienced degraded performance:

HTTP code: 200
Response time: 2418 ms

Resolved: Samsung performance has improved in 17ba82c.


Apply arbitrary Matching Logic simplification rules

At present, the backend only recognizes simplification rules of the forms:
// σ is a symbol
rule σ(φ₁, ...) => ...
rule #Ceil(σ(φ₁, ...)) => ...

The backend should also apply simplification rules over arbitrary Matching Logic patterns. The pattern matcher should be able to do this, but it isn't enabled in the simplifier.

The architectural blocker to implementing this feature is that the term simplifier is currently monolithic. It would be very difficult to break up (we have tried that before). Instead, we should build the new simplifier around the monolithic simplifier. We would delegate calls to the monolithic simplifier, until we have built up enough of the new simplifier that the legacy simplifier isn't needed any longer.


Update wast to 6.0.0

Due to API changes in wast, this removes all AssertReturn* directives and expands the functionality the single AssertReturn directive with the nan:canonical and nan:arithmetic patterns. See https://github.com/bytecodealliance/wat/pull/42 and https://github.com/WebAssembly/spec/pull/1104 for context.

Let's merge https://github.com/bytecodealliance/wasmtime/pull/798 first and I can rebase on that.


bug in simulating EAG with QOutMax when using numba

When simulating EAG with QOutMax, this parameter is not passed to the numba calc_waterbalance method. When passing it, the result seems off. Probably a sign problem in the code.
Fix using Snelrewaard example.

Comparison between outflux and QOutMax used wrong sign, values are negative.


Update gamesData.json

PR Description 📜
Please include summary related to the issue you have fixed and describe your PR in brief over here by specifying the issue number on which you were working below
Fixes # <your_issue_number>

Mark the task you have completed ✅

[x ] I follow CONTRIBUTING GUIDELINE & CODE OF CONDUCT of this project.
[ x] I have performed a self-review of my own code or work.
[ x] I have commented my code, particularly in hard-to-understand areas.
[ x] My changes generates no new warnings.
[ x] I have followed proper naming convention showed in CONTRIBUTING GUIDELINE
[ ] I have added screenshot for website preview in assets/images
[x ] I have added entries for my game in main README.md
[x ] I have added README.md in my folder
[x ] I have added working video of the game in README.md (optional)
[x ] I have specified the respective issue number for which I have requested the new game.

Add your screenshots(Optional) 📸

Thank you soo much for contributing to our repository 💗

Thank you @ayushlohmod ,for creating the PR and contributing to our GameZone 💗
Review team will review the PR and will reach out to you soon! 😇
Make sure that you have marked all the tasks that you are done with ✅.
Thank you for your patience! 😀

Hey @ayushlohmod,
Please make sure to link the relevant issue using the appropriate syntax, such as "#issueNumber" 👀.
Follow the proper guideline and make a new PR again 😀.
Happy Hacking 💗

Thank you  @ayushlohmod , for your valuable time and contribution in our GameZone 💗.
It’s our GameZone, so Let’s build this GameZone altogether !!🤝
Hoping to see you soon with another PR again 😇
Wishing you all the best for your journey into Open Source🚀


Lint Jira Integration

Bring the Jira Integration up to our Slack linting standards.
Currently the Jira project uses Standard, but Slack (and all of dotcom) use ESLint with the airbnb styleguide and some other rules turned on. This is probably too much work for a single PR, but it's giving me exposure into the project, which is generally how I learn. We can throw this away/part it out if we decide it's too much.

Will revisit this idea later.


DESENG-452 : Applying pending migrations

Issue: https://apps.itsm.gov.bc.ca/jira/browse/DESENG-452
Description of changes:

Change some foreign key field to nullbale false in model files

Change `rejected_reason_other` to nullable true in `submission` model

Generated new migration file based on the pending model changes which confirmed to be valid

Updated Unittest of Email verification to send `type` to the api

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of the met-public license (Apache 2.0).

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

Comparison is base (23faaae) 71.80% compared to head (2d80b6e) 71.82%.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #2378      +/-   ##
==========================================
+ Coverage   71.80%   71.82%   +0.01%
==========================================
  Files         509      509
  Lines       16916    16916
  Branches     1288     1288
==========================================
+ Hits        12147    12150       +3
+ Misses       4529     4526       -3
  Partials      240      240

Flag
Coverage Δ

metapi
86.45% <100.00%> (+0.03%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

Files
Coverage Δ

met-api/src/met_api/models/submission.py
80.21% <100.00%> (ø)

met-api/src/met_api/models/timeline_event.py
75.00% <100.00%> (ø)

met-api/src/met_api/models/widget_timeline.py
100.00% <100.00%> (ø)

... and 1 file with indirect coverage changes


Make loading reporting in dvm easier

You need to do the DCO. You can do git commit --amend -s and force push to fix it.


Few fixes to Hangman.hs

Good stuff, this version of hangman is much nicer than the previous (easier to understand) :)

Thanks @pyko !


Hello, I want to ask if has a kernel/qemu patch for ctr currently?Thank you.

Hi @solarises,
I am currently cleaning up patches. I will try to get those out in upstream mailing-list by end of this week.

Hi @solarises,
I am currently cleaning up patches. I will try to get those out in upstream mailing-list by end of this week.

thank you.

@solarises Please refer to #26.

thank you .


Use scorch index type for bleve

Gitea version (or commit ref):1.12.0-dev
Git version:2.11
Operating system:Debian9
Database (use [x]):

[ ] PostgreSQL
[ ] MySQL
[ ] MSSQL
[x] SQLite

Can you reproduce the bug at https://try.gitea.io:

[ ] Yes (provide example URL)
[ ] No
[x] Not relevant

Log gist:
Don't need

Description
The bleve indexer file was too large.
I have found that the bleve has two index type, refer this https://github.com/blevesearch/bleve/issues/783
The current index type gitea used is no longer actively maintained.
We should replace boltdb with bbolt. I had a try. It works fine, and the index file size decrease about 80%.
May i pull a request or wait someone handle it.
Screenshots

We've already switched to bbolt, and you'll see that it was me that contributed most of the PR to bleve to switch for this exact purpose.

But i found the content in index_meta.json was not {"storage":"scorch","index_type":"scorch"} but {"storage":"boltdb","index_type":"upside_down"}.

We've already switched to bbolt, and you'll see that it was me that contributed most of the PR to bleve to switch for this exact purpose.

My statement may be inaccurate, I mean we should use the new index form-Scorch.

My statement may be inaccurate, I mean we should use the new index form-Scorch.

In that case I will re-open. Thank you for suggestion :)

https://github.com/go-gitea/gitea/blob/ed5140d2353a1d3721db73c5fcba19b67e22931d/modules/indexer/code/bleve.go#L188
My attempt is use another bleve api to init a index, it can specify the storage type and index type
indexer, err := bleve.NewUsing(path, mapping, "scorch", "scorch", nil)
Then you need remove the previous index file and re-index, and you will find the size of index file decrease drastically.
It was a big change to indexer, we should consider cautiously about the user update to the new index type.

My statement may be inaccurate, I mean we should use the new index form-Scorch.

In that case I will re-open. Thank you for suggestion :)

The bleve library lack concrete document, and i was confused in some concept. Sorry for this misleading title.

@L0veSunshine I would like you create a new indexer type bleve-scorch so that it will not break the previous index. Pull request is welcome!!!

In order to change the index type, the code only need change a little. But the index file will change completely.
You meaning give user a choice to select the index type? I think no one will select the previous index type, because of it's enormous size. I think it is necessary to change the index type thoroughly. The key issue is how to smoothly transition

Add a new indexer type bleve-scorch and make it as default if user cannot set the type. And user could delete old indexer files and set to the new type.
We can use the new implementation share most codes with the previous codes.

Wouldn't it be possible to convert index file? Like opening old index, creating new with other name and then streaming all records from one to other. When successfully done old one could be deleted.

Add a new indexer type bleve-scorch and make it as default if user cannot set the type. And user could delete old indexer files and set to the new type.
We can use the new implementation share most codes with the previous codes.

The only difference of creating two types index is this line:
The original type
indexer, err := bleve.New(path, mapping)
content in index_meta.json file is{"storage":"boltdb","index_type":"upside_down"}
The scorch type
indexer, err := bleve.NewUsing(path, mapping, "scorch", "scorch", nil)
content in index_meta.json file is{"storage":"scorch","index_type":"scorch"}
I think lafriks's solution was the best. But i am a beginner in golang. I am afraid i can't implement this because of it needs to consider a vary of compatibility issues😅.
I found the new index type it by accident,i had try it a period of time and it has no difference with previous type meanwhile it's size decrease obviously. I open this issue aim to share this discovery with yours.
Thanks for listening❤

You may try it on a experimental instance, to see the actual effect, and decide whether the index type worths to change.

If different kinds of indexes will be used, we'd need to update the way we handle the rupture file (i.e. include the type of indexer).

I would prefer we would support only one index type

So we should have a new bleve index gitea version, so that when upgrade Gitea, the index will be regenerated.

It seems we have upgraded to scorch when upgraded to bleve search v2, so I just close this one.


feat: dependencies

Description

chore: upgrade dependencies

Testing required outside of automated testing?

[x] Not Applicable

Screenshots (if appropriate):

[x] Not Applicable

Rollback / Rollforward Procedure

[ ] Roll Forward
[x] Roll Back

Reviewer Checklist

[ ] Description of Change
[ ] Description of outside testing if applicable.
[ ] Description of Roll Forward / Backward Procedure
[ ] Documentation updated for Change

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


fixed error parsing

closes #147

Ah, such a simple oversight on my part sorry. Thanks for correcting it

Thank you @uDaiko


Possible bug for the Resize class in augmentations.py

Hi all,
When image is resized, bounding boxes should be scaled accordingly, but the Resize class in augmentations.py does not scale the bounding box. Is this a bug?

Because ToPercentCoords() is applied before resize.

oh, I see. thanks @lzx1413


Why can't insertion enum type to createStandardAction

First of all checkout our gitter channel: https://gitter.im/react-redux-typescript-guide/typesafe-actions
Then please use Stack Overflow or other resources for help resolving your specific use-cases.
The only exceptions to use issue tracker for questions would be for some unusual "special cases".
I am using this library with react. (create-reactapp)
I don't know why can't insertion enum type to createStandartAction
enum UserActionTypes {
LOGIN = 'LOGIN'
};
interface LoginPayload {
aa: string;
bb: string;
};
// ERROR in create react app
const loginAction = createStandardAction(UserActionTypes.LOGIN)();

Hey @NoMoreViolence, please provide detailed error and minimal reproduction repository only then I'll be able to help you

import { createStandardAction } from 'typesafe-actions';

enum UserActionTypes {
  LOGIN = 'LOGIN'
}

export interface LoginPayload {
  email: string;
  password: string;
}

export const userActions = {
  login: createStandardAction(UserActionTypes.LOGIN)<LoginPayload>(),
}

TypeError: Cannot read property 'LOGIN' of undefined

I don't think it's related to typesafe-actions. Have you used a debugger to check UserActionTypes value?

You are right, It's about typescript compile, I will close this issue.


How to use with Rails?

I've got a Rails app in which I've built a simple API using grape. I'd like to use my existing RABL templates for this API, so I've stumbled upon this gem. I can't get it to function with my Rails app, though.
I've got my /app/api/api.rb setup like this:
require 'grape/rabl'

use Rack::Config do |env|
  env['api.tilt.root'] = Rails.root.join('app', 'views')
end

class API < Grape::API
...

but that gives me this error:
app/api/api.rb:3:in `<top (required)>': undefined method `use' for main:Object (NoMethodError)

Any ideas what I'm doing wrong? Or perhaps this wasn't meant to work in a Rails environment? On a related note, you've got this in your README:
object @user => :user # in the RABL template

How does this work? I feel like updating all my RABL templates with lines like this might make other parts of my app that use the templates break; hence my thinking this won't work in a Rails app. ...but feel free to set me straight. hehe
Let me know if you need any other information.

Try move
use Rack::Config do |env|
  env['api.tilt.root'] = Rails.root.join('app', 'views')
end

to RAILS_ROOT/config.ru
About object syntax see on https://github.com/nesquena/rabl and search for Object Assignment

I actually ended up taking a different approach with the project in question, so I haven't had a chance to test this. I'll close this now, and try to test this in another project ASAP. Thanks for the info.


Found what looks like an issue with spring.cloud.config.awsS3 with multiple profiles

I have a application.yml and 4 more files application-local.ym, application-dev.yml,  application-qa.yml,  application-prod.yml. The way it works is that
spring:
profiles:
active:
@activatedProperties@
Will depending on what @activatedProperties@ is, will  pull the correct application so if @activatedProperties@ is local, it will pull application-local.ym.
The problem is that to run
cloud:
config:
server:
awss3:
bucket: hp-fetcher-config
region: us-west-2
order: 1
I need to replace
spring:
profiles:
active:
@activatedProperties@
with
spring:
profiles:
active:
awss3
I cant find any docs or examples that show how to run awss3 in multiple profiles

You have not stated what the actual problem is.
Please learn how to format code on GitHub.

I did state the problem I cant find a way
spring:
  profiles:
    active:
      @activatedProperties@

I need to be able to run awss3: with multiple profiles.  I have the following application.yml  that calls a eureka service discovery , the eureka service discovery is calls relied on what @activatedProperties@ resolves to.
server.port: 8080
spring:
  profiles:
    active: @activatedProperties@
  application:
    name: @artifactId@
  # do not serialize nulls
  jackson:
    default-property-inclusion: NON_NULL
    mapper:
      accept-case-insensitive-enums: true

# service discovery
eureka:
  client:
    validateCertificates: false
    registerWithEureka: true
    fetchRegistry: true
    healthcheck:
      enabled: true
    serviceUrl:
      defaultZone: "http://${eureka.server.host}:8761/eureka/"
  instance:
    statusPageUrl: "http://${eureka.hostname}/actuator"
    healthCheckUrl: "http://${eureka.hostname}/actuator/health"
    homePageUrl: "http://${eureka.hostname}/"
    nonSecurePortEnabled: true
    preferIpAddress: false

if @activatedProperties@ resolves to local, then it calls another yml file application-local.yml
server.port: 8082
eureka:
  client:
    serviceUrl:
      defaultZone: "http://localhost:8761/eureka/"

This works, the problem arises when make this a config service using awss3. Because to use awss3 you have to make it the active profile spring:profiles:active:awss3.  Which means I can no longer pass in @activatedProperties@  to set the correct discovery service address.
spring:
  profiles:
    active:
      awss3
  application:
    name: HP-FETCHER-CONFIG-SERVICE

  cloud:
    config:
      server:
        awss3:
          bucket: hp-fetcher-config
          region: us-west-2
          order: 1

# service discovery
eureka:
  client:
    validateCertificates: false
    registerWithEureka: true
    fetchRegistry: true
    serviceUrl:
      defaultZone: "http://ip-172-31-39-54.us-west-2.compute.internal:8761/eureka/"
  instance:
    statusPageUrl: "http://${eureka.hostname}/actuator"
    healthCheckUrl: "http://${eureka.hostname}/actuator/health"
    homePageUrl: "http://${eureka.hostname}/"
    nonSecurePortEnabled: true
    preferIpAddress: false

When I run your previous sample you supplied (https://github.com/spring-cloud/spring-cloud-config/issues/2017#issuecomment-997465541) with ./mvnw spring-boot:run -P local
I see both profiles active
2021-12-21 09:26:05.333  INFO 74372 --- [           main] c.o.s.h.f.HpFetcherConfigApplication     : The following profiles are active: awss3,local

Yes there are two profiles, but it does not run local, All that does is say is I have two profiles. I need a way for it to say run  as both profiles

What I am trying to do is run the config server in the cloud on a ECS, I have a client that runs in the cloud as a ECS and Eureka running as a EC2 on the cloud. The example was so easy.  Add  the config data to a S3 bucket. Then the client just uses the ServiceId to connect to the Configserver. And that is what I have done and it should work but it wont.

The problem trying to run Eureka with Spring cloud config, I have 4 applcations.
I have eureka
I have zuul
I have a spring config server which is a client of Eurka and regsiters with it
And I have a application which  I want to use to connect to the config server by its ServiceId it then downlods from the store config data which includes data such as the path to Eurka so it can register itself.

There is something that is even more annoying on the client side. Because I am having problems with setting Eureka, I decided leave all the config data in the yml, and just have one item in the repo a simple hello string. The idea being that maybe Eureka should not be added to the config , but other data can. That would at least give me something.  Now the following code without using spring cloud config works.
server.port: 8080
spring:
  profiles:
    active: @activatedProperties@
  application:
    name: @artifactId@

# service discovery
eureka:
  client:
    validateCertificates: false
    registerWithEureka: true
    fetchRegistry: true
    healthcheck:
      enabled: true
    serviceUrl:
      defaultZone: "http://${eureka.server.host}:8761/eureka/"
  instance:
    statusPageUrl: "http://${eureka.hostname}/actuator"
    healthCheckUrl: "http://${eureka.hostname}/actuator/health"
    homePageUrl: "http://${eureka.hostname}/"
    nonSecurePortEnabled: true
    preferIpAddress: false

But when I add  config server
server.port: 8080

spring:
  profiles:
    active: @activatedProperties@
  application:
    name: @artifactId@
  config:
    import: "configserver:"
  cloud:
    config:
      fail-fast: true
      enabled: true
      #uri: http://<IP_ADDRESS>:8080
      discovery:
        enabled: true
        serviceId: HP-FETCHER-CONFIG-SERVICE

eureka:
  client:
    validateCertificates: false
    registerWithEureka: true
    fetchRegistry: true
    healthcheck:
      enabled: true
    serviceUrl:
      defaultZone: "http://${eureka.server.host}:8761/eureka/"
  instance:
    statusPageUrl: "http://${eureka.hostname}/actuator"
    healthCheckUrl: "http://${eureka.hostname}/actuator/health"
    homePageUrl: "http://${eureka.hostname}/"
    nonSecurePortEnabled: true
    preferIpAddress: false

It throws an exception
23:27:50.411 [main] ERROR org.springframework.boot.SpringApplication - Application run failed
java.lang.IllegalArgumentException: Not enough variable values available to expand 'eureka.server.host'
        at org.springframework.web.util.UriComponents$VarArgsTemplateVariables.getValue(UriComponents.java:370)

I thought the idea was that it would look local first, before it looked to config server. So I cant even get it working to test if it would just give me a simple string.

This is what I have at the moment this is my client application.yml
server.port: 8080

spring:
  profiles:
    active: @activatedProperties@
  application:
    name: @artifactId@
  config:
    import: "configserver:"
  cloud:
    config:
      fail-fast: true
      enabled: true
      #uri: http://<IP_ADDRESS>:8080
      discovery:
        enabled: true
        serviceId: HP-FETCHER-CONFIG-SERVICE

But does the service Id work the same way as a uri. With a url you have a http://localhost:8060 and then @artifactId@ is the name of my application call client and @activatedProperties@ is the profile call it dev
so it will be realized as http://localhost:8060 /client/dev and in the and in my S3 repo I have config data client-dev.yml
But if I am using ServiceId to call the data, will it still be looking for client-dev.yml. I think this may be the real problem.

Back to my original question it looks as if spring.cloud.config.awsS3 can only work as its own profile, and is not designed to be portable between environments


add ChangePitch for Look Up & Look Down

changePitch shouldn't change heading.

that's what I thought. fixing


Updated Türkçe(Turkish) translation

Türkçe(Turkish) language support has been updated.

Thank you for submitting translations to Ghost! 🙏🏻 We really appreciate every string you’ve added. There might be a few new strings that need translation in Turkish to ensure your site. If you spot any untranslated or missing strings, let us know, and we’ll fix them ASAP! 🎉


Base Distribution has no _localSetState

Issue Description
The base class Distribution calls _localSetState as part of its __setstate__ method, but there's no implementation for that method in the base class (and it is missing in some inheritors such as the mutlivariate normal).  There should be a passthrough at minimum.
This bug cannot cause incorrect results, only code crashes.

For Change Control Board: Issue Review
This review should occur before any development is performed as a response to this issue.

[x] 1. Is it tagged with a type: defect or improvement?
[x] 2. Is it tagged with a priority: critical, normal or minor?
[x] 3. If it will impact requirements or requirements tests, is it tagged with requirements?
[x] 4. If it is a defect, can it cause wrong results for users? If so an email needs to be sent to the users.
[x] 5. Is a rationale provided? (Such as explaining why the improvement is needed or why current code is wrong.)

For Change Control Board: Issue Closure
This review should occur when the issue is imminently going to be closed.

[ ] 1. If the issue is a defect, is the defect fixed?
[ ] 2. If the issue is a defect, is the defect tested for in the regression test system?  (If not explain why not.)
[ ] 3. If the issue can impact users, has an email to the users group been written (the email should specify if the defect impacts stable or master)?
[ ] 4. If the issue is a defect, does it impact the latest stable branch? If yes, is there any issue tagged with stable (create if needed)?
[ ] 5. If the issue is being closed without a merge request, has an explanation of why it is being closed been provided?

The following are also missing and need this method (perhaps it should be abstract in the base class):

Categorical
Custom1D
LogUniform
NDimensionalDistributions

NDInverseWeight
NDCartesianSpline
MultivariateNormal

@mandd it is critical


annotations have wrong addresses associated

See offset 0x201068 in the attached bndb. If you double click the string reference, you'll be taken to 0, and likewise if you click the string the xrefs will be incorrect, showing a number of locations where a null argument is passed to functions.

rand2.bndb.zip

This is fixed before stable 2.2


Prototype tailwind support

Few issues:

This loads a pretty large file from CDN (300kb+) and isn't recommended by Tailwind for production use cases
Can't load it via ES6 import due to CORS issue, this seems like something they could potentially fix
Not sure whether every component should accept classes
Potential class name conflicts?

@wwwillchen

This loads a pretty large file from CDN (300kb+) and isn't recommended by Tailwind for production use cases
This is fine to me as the page is intended to use in internal services
Not sure whether every component should accept classes
I only need for Box now, but every component seems cool
Potential class name conflicts?
We also have this problem in FE project if not using tailwind-merge lib

I've answered some questions