Allow from subqueries with read models

With read models supporting subqueries, it would be cool if they supported them for the from of a read model. This would work similar to the joins:
class MyReadModel
  include MR::ReadModel
  from MyRecord do
    read_model do
      select 'a_column'
      where MyRecord.active
    end
    as 'subquery_records'
  end

where this would generate something like:
SELECT *
FROM (
  SELECT a_column
  FROM my_records
  WHERE my_records.active = 't'
) AS subquery_records

Also add where and select subqueries. That would cover almost all the main use cases I know of.


[WIN M4] WIN (ZIP/MSI) Publishing

Tasks(ZIP)
Tasks(MSI)
Estimate
Status(ZIP)
Status(MSI)
Notes

Update the link of the WIN artifact on S3/CloudFront to our website download pages
Same
1
Not Started
Not Started

Provide ability to manually download the WIN artifact with the link
Same
2
Not Started
Not Started

The build system should try to install the WIN artifact after build/assemble and verify the commands running correctly
Same
3
Not Started
Not Started

The build system should attempt to start the service of the installed packages and verify the outcome through task manager of the service (see more details in the test requirements)
Same
Not Started
Not Started

Once verified, the WIN artifact should be uploaded to the dedicated production S3 bucket linked to artfacts.opensearch.org
Same
2
Not Started
Not Started

Once deployed, user can then download the ZIP/MSI to install the latest WIN artifact from our repository on S3, with the help of CloudFront cache in the front
Same
1
Not Started
Not Started

The publishing is actually added in this PR:

https://github.com/opensearch-project/opensearch-build/pull/2869
https://github.com/opensearch-project/opensearch-build-libraries/pull/31


Upgrade Job: Missing delete endpoint notification for remote clusters

What happened:
Latest release to Latest version tests are failing
https://github.com/submariner-io/submariner/pull/1010/checks?check_run_id=1562617791
The e2e test fails on the [redundancy] job. As part of e2e redundancy tests, we perform a gateway-failover on "cluster2", wait for a new Engine Pod to be created on the newly labelled node and ensure that all connections are established. The last step fails - i.e., connection to cluster1 stays in "connecting..." stage itself.
Looking at the logs on cluster1 enginepod, we see the following error. Also, there is no delete event for cluster2 old endpoint in the logs of cluster1 enginePod. This needs to be investigated and fixed.
cluster1 EnginePod logs: https://pastebin.com/raw/UU0j1cHt
2020-12-16T09:26:24.5140018Z [cluster1] E1216 09:20:56.988121       1 tunnel.go:59] error installing cable for Endpoint types.SubmarinerEndpoint{Spec:v1.EndpointSpec{ClusterID:"cluster2", CableName:"submariner-cable-cluster2-172-17-0-10", HealthCheckIP:"<IP_ADDRESS>", Hostname:"cluster2-worker2", Subnets:[]string{"<IP_ADDRESS>/16", "<IP_ADDRESS>/16"}, PrivateIP:"<IP_ADDRESS>", PublicIP:"", NATEnabled:false, Backend:"libreswan", BackendConfig:map[string]string(nil)}}, found a pre-existing cable "submariner-cable-cluster2-172-17-0-5" that belongs to this cluster cluster2

What you expected to happen:
The redundancy test should pass and "cluster1" enginePod should be getting the delete notification for old Endpoint.
How to reproduce it (as minimally and precisely as possible):
Anything else we need to know?:
Environment:

Submariner version (use subctl version):
Kubernetes version (use kubectl version):
Cloud provider or hardware configuration:
OS (e.g: cat /etc/os-release):
Kernel (e.g. uname -a):
Install tools:
Network plugin and version (if this is a network-related bug):
Others:

This issue is specific to upgrade to 0.8.0 which changed the datastore syncer to use admiral. It uses a label, submariner-io/clusterID,  on the resources to keep track of the originating cluster to avoid sync loops. In cluster1,  we see the initial update for the EP from cluster2 however notice that the label points to cluster3:

2020-12-16T09:26:24.4764862Z [cluster1] I1216 09:18:54.403435       1 tunnel.go:51] Tunnel controller processing added
 or updated submariner Endpoint object: &v1.Endpoint{TypeMeta:v1.TypeMeta{Kind:"", APIVersion:""},
ObjectMeta:v1.ObjectMeta{Name:"cluster2-submariner-cable-cluster2-172-17-0-5", ...
Labels:map[string]string{"submariner-io/clusterID":"cluster3"}, Annotations:map[string]string(nil), OwnerReferences:[]v1.OwnerReference(nil), Finalizers:
[]string(nil), ClusterName:"", ManagedFields:[]v1.ManagedFieldsEntry(nil)}, Spec:v1.EndpointSpec{ClusterID:"cluster2",
CableName:"submariner-cable-cluster2-172-17-0-5", HealthCheckIP:"", Hostname:"cluster2-worker", ...}

From the prior release run, it was synced to cluster3 but w/o the submariner-io/clusterID label. On restart after the upgrade, cluster3 gets notified of the EP from the local datastore and sees that there's no submariner-io/clusterID label and assumes it's local so it syncs to the broker. But, in doing so, it sets the submariner-io/clusterID label to cluster3. cluster2 eventually sees this and syncs it locally b/c it thinks it's remote. So now the local EP on cluster2 has the submariner-io/clusterID label  set to cluster3.  After restart during failover, it deletes it locally but it doesn't get propagated to the broker since it's not seen as local.
Not sure right now how to safely address this - just reporting my initial findings.


Add linting to web gitlab checks

We have prettier as a gitlab check on the branch for frontend but not linting, means it's possible to merge a branch with linting errors.
Here's my attempt to add it, feel free to yell at me if there's a reason I was unaware of that it's not there.

Closing as I have another with the same code open, was just tryin to understand if the branch names effecting which gitlab tests ran.


Why include AuthInfoField and not GraphqlError() ?

AuthInfoField makes things complex especially if you use Relay.
A lot of errors of the type : Object doesn't match crop up.
Can the error simply be a graphQLError ?
(from the library graphql.error.base import GraphQLError )

I try to include relay connection like this.
class PostObject(SQLAlchemyObjectType):
    class Meta:
        model = Post
        interfaces = (graphene.relay.Node, )

class ProtectedPost(graphene.Union):
    class Meta:
        types = (PostObject, AuthInfoField)

class PostConnection(graphene.Connection):
    class Meta:
        node = ProtectedPost

class Query(graphene.ObjectType):
    node = graphene.relay.Node.Field()
    my_posts = graphene.ConnectionField(PostConnection)

However, I get an error when the "token" expires.  error is -

Resolved value from the connection field have to be iterable or instance of PostConnection. Received "<flask_graphql_auth.fields.AuthInfoField object at 0x10c34e160>"

This is because AuthInfoField returns an Object Literal whereas IterableConnection expects an Iterable list.
Throwing an Exception or GraphQL error should solve so many complications.


#I Improve snapshot test

This PR improves the way snapshot tests are executed. By creating an
extension to UIViewController, we can now create snapshot tests with
less code, time, and effort. And, it is much more readable. Since our
usages of snapshot tests are very limited (for now), this approach serves
those purposes very well.
This commit also updates the following:

Naming of files - some voiceover test files did not have a proper name.
Naming image references - voiceover image references didn't have a
correct naming, leaving out the dimensions of the view. This PR fixes that
AccessibilityImage tests didn't have frame that was aligned with layout
and extra3LargeFont tests
Fixes the issue, that some landscape images came out portrait before.

MOB-2696

Please re-review as the complete update has been added

I think assertSnapshot in more appropriate method name than testSnapshot.

fixed d23621c


Add matomo tracking

Rationale
We want to use matomo to get more user behavior insights to improve the website and app.
Implementation

[ ] Add the following to our websites:

Make sure this code is on every page of your website. We recommend pasting it immediately before the closing  tag

[ ] Add a disclaimer to our privacy policy that we are using matomo

Web analysis tool Matomo
We analyze the use of our website with the open-source web analysis tool Matomo (formerly Piwik; data protection declaration can be found at https://matomo.org/privacy-policy/). For this purpose, the automatically collected IP addresses are anonymized before the evaluation. The analysis is therefore based on anonymized data records, and there is no personal evaluation. The web analysis is used exclusively to optimize the website in terms of user-friendliness and to provide useful information about our services. This data is not merged with other personal data sources or passed on to third parties.

In addition to the data mentioned, Matomo also uses so-called “cookies” (see below).

[ ] Allow opt out from the tracking via privacy policy by adding matomo opt in / out option bellow added privacy text

@GabrielBuragev can you give me more info about if and for when this issue was prioritized? Just want to get an estimation about when this can be done. I assume this was taken to Jacob already as he was aware of it when we had our marcomms sprint planning.

@DBecane i am already working on it in parallel to other critical issues. Should be live by end of tommorow.

@andfletcher Is that the right place to put the matomo opt-out element/

@GabrielBuragev ne there needs to be a new number (make it 13. and make contact us to 14.) and add the text that is above:
So should look like this:

Web analysis tool Matomo
We analyze the use of our website ...........

then directly bellow "the opt out element" you have in the screenshot

There is already an entry for this at 10.
I adjusted the text based on your requirements.
I will put the opt-out right below that as you described.


Unable to delete when using withoutTenantRestriction

I happened upon this issue during my development.  Currently the hibernate filter prevents deletes from executing when the current tenant id is null.  It is my understanding that having no current tenant id should allow unrestricted access to view, update or delete assets.
Please let me know if the delete functionality was not intended to work with the "withoutTenantRestriction" feature.

That sounds about right, but it's a bit hard to see what has changed in the patch..

Oh, I didn't notice how awful the merge was. I expect it has something to do with line endings. I'll see if I can clean it up to make the patch easier to read.

Ok, those files should be easier to read now.  Somehow I also had some bad line endings in documentation that I had written the other day, and apparently the license had CRLF in it as well.  Sorry for the clutter in the diff view, but it should be easier to see the changes I made for the delete functionality.


Progress bar for brew cleanup

A detailed description of the proposed feature
I would like to see a progress bar showing the status of brew cleanup, similar to the progress bar that appears during downloading.
The motivation for the feature
brew cleanup can take a while if a lot of formulae are installed. I'd like to be able to have some sense of how long it will take to finish.
How the feature would be relevant to at least 90% of Homebrew users
brew cleanup automatically runs every 30 days, so this is relevant to every user who hasn't turned that off.
What alternatives to the feature have been considered
Not sure what would be an alternative, other than not having this feature

We don't implement the progress bar, it comes from curl. We'd consider a PR to do this but it'll likely be pretty tricky.


problems about the docs

hi,
when study the picorv32, some confused without documents, where can i find the descriptions or request for help on these?

about the interrupt vector table description, from the De-assambled  code of the example, seems the ram 0-F both for the reset handler, how should i arrange the vectors, or just like the one in original riscv?

is there any Unit for Debug implementation(i mean the support for breakpoint, step...)? it should important for the fpga debug, while can not find in the readme.

any PCPI detail descriptions? can not find the modules in the readme for pcpi functions, while the PCPI interface always toggle even the parameter PCPI closed.

thanks.

about the interrupt vector table description, from the De-assambled code of the example, seems the ram 0-F both for the reset handler, how should i arrange the vectors, or just like the one in original riscv?

I cannot parse this. What do you mean?

is there any Unit for Debug implementation(i mean the support for breakpoint, step...)? it should important for the fpga debug, while can not find in the readme.

You don't need any extra hardware for this (I assume this is what you mean with "unit").
The core execute the illegal instruction trap handler for EBREAK.
You have to write your own gdb stub, I don't provide one.

any PCPI detail descriptions?

There is a chapter in the readme file about the PCPI.

can not find the modules in the readme for pcpi functions, while the PCPI interface always toggle even the parameter PCPI closed.

I have no idea what that means.

about the interrupt vector table description, from the De-assambled code of the example, seems the ram 0-F both for the reset handler, how should i arrange the vectors, or just like the one in original riscv?

I cannot parse this. What do you mean?
<<what i mean is where should I place the reset entry address and each ISR entry address in the memory?  for example in ARM system the 0x0 is for SP, 0x4 is for reset handler and so on.>>
is there any Unit for Debug implementation(i mean the support for breakpoint, step...)? it should important for the fpga debug, while can not find in the readme.

You don't need any extra hardware for this (I assume this is what you mean with "unit").
The core execute the illegal instruction trap handler for EBREAK.
You have to write your own gdb stub, I don't provide one.
<<can not understand  this, RISCV debug specification should need dedicated HW to support, like PC address compare, step exec.., like coresight in ARM.>>
any PCPI detail descriptions?

There is a chapter in the readme file about the PCPI.
can not find the modules in the readme for pcpi functions, while the PCPI interface always toggle even the parameter PCPI closed.

I have no idea what that means.
<<i try the basic testcase simulation, in the result waveform dumped, the interface for the PCPI input all floating while PCPI output keep toggle, and can not see the connections for the PCPI co-processors, any case for test this implementation? >>
thanks.

what i mean is where should I place the reset entry address and each ISR entry address in the memory? for example in ARM system the 0x0 is for SP, 0x4 is for reset handler and so on.

To whatever you have set the PROGADDR_RESET and PROGADDR_IRQ of course. Theyt are both documented in the README file.

can not understand this, RISCV debug specification should need dedicated HW to support

PicoRV32 does not support the interfaces described in the debug specification.

i try the basic testcase simulation, in the result waveform dumped, the interface for the PCPI input all floating while PCPI output keep toggle, and can not see the connections for the PCPI co-processors, any case for test this implementation?

(1) What do you care if some of the output pins are toggling if you have disabled the interface? Presumably you are not using them anyways (because you have disabled it), and as long as pcpi_valid is low anything connected to the port should ignore all the other lines anyways.
(2) Re tests: e.g. MUL/DIV are implemented using PCPI.


Run registered application on an image

Something like ecli slide run application_id to analyze an image.

Duplicates #8.


Revise make-cross-deps helper script

Align with conventions of other helper scripts.

Implement --config option to configure without building.

Implement --ninja option to build using Ninja.

Can we update docs to reflect what the command looks like when using ninja? Is it just ./make-cross-deps.sh --ninja?

The command would be:
./make-cross-deps.sh --host=HOSTDEPS --prefix=PREFIX --ninja

Ninja support is in an intermediate state right now. I'm in the process of adding it to the scripts, but I haven't updated their help text to list --ninja as a supported option. Building the Stratum Dependencies has a Ninja example, but it's for the cmake command-line, not the helper script(s).
I think the correct approach is:

Create individual documents for the make-host-deps.sh and make-cross-deps.sh helper scripts. Document the individual parameters and include a number of practical examples.
Create a separate guide on building the Stratum dependencies with CMake. Document the command-line variables and include a number of practical examples.
Revise Building the Stratum Dependencies and Building the Stratum Dependencies for ACC. Remove the parameter documentation and include a couple of the most common use cases, referring the customer to the other guides for parameter documentation and additional use cases.

There are limitations on the use of Ninja. It doesn't work with OVS (too much Autotools-specific logic), and it doesn't play well with --sudo (no password prompt, and just typing it in doesn't work for Ninja the way it does for Unix Makefiles). I'd like to understand the problems better (and possibly solve them) before we announce the availability of the feature. I'd like to time a couple of builds, to determine whether Ninja really is faster than Unix Makefiles, and it would probably behoove me to learn more about working with Ninja.
None of the above should be taken as a criticism of your suggestion, which is a sound one. I'd just like to stay under the radar a little longer, in hopes of achieving one or more of the attendant tasks mentioned above.


feature: people you might know feature

  On LinkedIn their success came from the "people you may know" section. It identifies folks connected with people you know.

This is also hi the tiktok recommendations work with a certain extent with its "for you" page.

OpenSauced should provide a "projects for you" or similar projects. Same with contributors to follow.
Not sure how this would surface but recommendations is the backbone of hot.opensauced.pizza and a potential place we can start surfacing this data.
Originally posted by @bdougie in https://github.com/open-sauced/insights/discussions/171

Using the fullstack network and saw they use the "insta stories" model.
https://thefullstack.network/explore

This work can be attributed to the highlights work in this PR https://github.com/open-sauced/insights/pull/1347


Got unexpected error when use ticker.remove on destroyed ticker

Expected Behavior
Just don't throw the error, or expose an reliable function to let devloper known is the ticker has been destroyed.
Current Behavior
throw an error like:
Uncaught TypeError: Cannot read property 'next' of null
    at Ticker.remove (Ticker.js:310)

Possible Solution
Just don't throw the error, or expose an reliable function to let devloper known is the ticker has been destroyed.
Steps to Reproduce
First called ticker.destroy(), then try to removed anything by called ticker.remove(fn).
My reason is that:
Think for this case, we have some scenes used different ticker. If the player exit one of the scenes, the we should destroy corresponding ticker.
Then, in one of the scenes, there are some children segments, each time the player exit a segment, all the ticking methods should been removed.
Now, if the player exit a scene, the problem appeared, we don't know if the ticker has already been destroyed. Maintain it manually is unwised, it expend additional mind.
An simply method to resolved this problem is just prevent throw the error, consistent with the browser built-in method window.removeEventListener .
Environment

pixi.js version: 4.8.6
Browser & Version: _e.g. Chrome 72.0.3626.109
OS & Version: Win10

This is by design. You cannot remove ticker handlers after the ticker has been destroyed. In generally, in PixiJS for all objects, you should not use them after you call destroy.
I don't completely understand your use-case. Could you provide some more code about why the ticker is unknown or you cannot maintain a reference to it?

In fact, I used React with PIXI, and in the parent node, there are a ticker mounted once the node will mount, and unmounted once the node was unmounted.
Then in the children nodes, there are lot's of ticker methods, that should unmounted once the child node was unmounted too.
However, in React, the unmount behavour of the parent node was executed before the children nodes unmounted, so this would caused this problem.
On the other hand, I also can't think of some use-case that the remove method should throw an error. Or at least, expose a function to let the developer known is the ticker was destroyed?

As a work around, why not do something like this?
class CustomTicker extends PIXI.Ticker {
  remove(listener, context) {
    if (this._destroyed) {
      return this;
    }
    return super.remove(listener, context);
  }
  destroy() {
    this._destroyed = true;
    super.destroy();
  }
}

I don't think this is something we should fix, since it's a little too specific to how React works. And we have some default assumptions built into how destroy works across all the PixiJS APIs.

Nice idea, thank you.


Create a controller to sync Aegis state to k8s secrets that it interpolates

if k8s secret is deleted (non-existent) or empty but secret in ageis safe exists, then the pod crashes (for the using-init-container use case)
if the secret is a k8s-enabled secret,
then Aegis Safe can be updated check the secret’s data and if there is a difference (i.e. data is stale or empty)
it can first update the secret and then return a response to the workload.
but that would mean a round-trip to api server for every k8s-using secret.
instead; we can write a controller, and it can be optionally deployed.
if someone deletes the secret, the controller will recreate it
if someone changes the value of the secret the controller will update its value.
that looks like a better approach.

Closing this in lieu of kind: VSecMSecret


Update yearn meta-German-Protocols 1

can please review and let me know if anything needs to be changed?


zksync-ethers V5 and V6: Add documentation related to SmartAccount

What :computer:

Add documentation related to SmartAccount that would be part of next zksync-ethers V5 and V6 releases.

Relevant to: https://github.com/zksync-sdk/zksync-ethers/pull/49
In-progress @dutterbutter .


error while using guidance with azure openai

The bug
A clear and concise description of what the bug is.
ValueError: Fail to create device flow. Err: {
"error": "invalid_request",
"error_codes": [
900144
],
....
}
To Reproduce
Give a full working code snippet that can be pasted into a notebook cell or python file. Make sure to include the LLM load step so we know which model you are using.
# put your code snippet here

System info (please complete the following information):

OS (e.g. Ubuntu, Windows 11, Mac OS, etc.):
Guidance Version (guidance.__version__):

This causes because AzureOpenAI uses msal. If you are unfamiliar with Azure and simplily wanna use Azure OpenAI Servise, try my temporary solution #87

@jhkim-tf

I think this is fixed now. If not, please reopen and give us a snippet to reproduce.


check alerts length

check alerts length in  FanoutStage.Exec , which is done in MultiStage.Exec.
if len(alerts) == 0

No need to create goroutines for each stages.

@simonpasquier PTAL


Shuffle list

In my project I've got shuffle feature. I can make pull request if anyone like it.

Closing this due to the PR being closed as out of scope: https://github.com/swanandp/acts_as_list/pull/111


Maven centralize

My apologies: my ineptitude at using/reading the GH pull request UI on my fork led to creating this PR in error.


Opening modal from outside of modalComponent

Hi,
I am attempting to use reactstrap modal where I programmatically open the modal, but I do not see anything in the docs explaining how to open the modal from a parent component. Any help is appreciated.
Thanks.

The docs outline this pretty clearly. Modal visibility is controlled with the isOpen prop.
http://reactstrap.github.io/components/modals/

Hey @virgofx , it does seem clear yes. But Here is an example of it not opening:
https://jsfiddle.net/n5u2wwjg/123932/
I'd really appreciate any ideas on why its not working.

In your example, your toggle button was too small ... clicking it worked fine. You're initial state was also set to false ignoring the isOpen
Working example:
https://jsfiddle.net/qex0s4wd/

@virgofx Thanks man, I appreciate your help. I have one more question, is it possible to re-open the modal from the parent component (without the toggle button) after closing it?

I don't really understand .. Your parent component controls the state of the modal. So you can open it however/whenever --- the example uses a button.... this could change to a timer and open/close every 10 seconds .. or react to anything else in your application and then appropriate control the state of the Modal (<Modal .../>) via the isOpen property which is stored as state in your parent object.

Thank you, I'll ask there.


master

Using kotlin Coroutines, LiveData, ViewModel and Databinding.

What i did was incorporating LiveData, Kotlin Coroutines, ViewModel and DataBinding as well as reorganizing the code. This is just my take, feel free to change it.


removed style3 table class from widgets and pdfs

#1890

@dclarizio Please review

@epwinchell Specs will need to be changed, maybe ping @h-kataria

@dclarizio Spec has been updated


experimental:module and /std:c++20

I believe that, if you want to use the standard library modules (e.g., std.core) with MSVC, you need to be using /std:c++latest, and /std:c++20 is not sufficient.
Indeed, if you do just use /std:c++20, then you get a warning (which becomes an error with /WX):

https://developercommunity.visualstudio.com/t/possible-incompatible-environment-while-importing/1567681

This then makes:

You can use the modules feature to create single-partition modules and to import the Standard Library modules provided by Microsoft. To enable support for modules, compile with /experimental:module and /std:c++20 or later (such as /std:c++latest).

be misleading, because to use the standard library modules, you cannot just use /std:c++20.

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

ID: ad1fef9a-9c55-a8ab-faf4-8b9ceb9ff688
Version Independent ID: 0eadb3ed-bb38-58bc-b335-687b10f0fd31
Content: Overview of modules in C++
Content Source: docs/cpp/modules-cpp.md
Product: visual-cpp
Technology: cpp-language
GitHub Login: @corob-msft
Microsoft Alias: corob

@andrewvaughanj
Thanks for the report. You are correct. I've added some clarification to the relevant docs. C++ Standard modules are supported under /std:c++20, but the extension that implements standard library modules requires /std:c++latest and /experimental:module as those are proposed for the next standard.


[DOC] Feedback on Time Filter page

What do you want to do?

[x] Request a change to existing documentation
[ ] Add new documentation
[ ] Report a technical problem with the documentation
[ ] Other

Tell us about your request. Provide a summary of the request and all versions that are affected.
We received feedback 2/28/23 on this page: https://opensearch.org/docs/latest/dashboards/discover/time-filter/
"Calendar icon does not exist on new "Discover" option."
What other resources are available? Provide links to related issues, POCs, steps for testing, etc.

@hdhalter Do you know the context of this issue? The calendar icon shows in Discover on both the playground and localhost:5601. Do you know what version of OpenSearch Dashboards the user is using? The following image shows the calendar icon in Discover i

n OpenSearch Dashboards v 2.5.

Hi Melissa, thanks for looking into this. I don't have any additional information. It could be that the user couldn't find it?
The one thing we can do proactively is update the instructions in the documentation. For example instead of starting with "Select the calendar icon", we provide more context in the steps themselves (From Dashboards, select Stack Management > Advanced Settings...), in case the user is coming straight here and wants to quickly perform this task.

Thanks, @vagimeli !


Speed up some time-consuming tests

This makes the total of tests finish in 5 minutes on my local desktop. The changes are:

use smaller PDB structures in the tests
allow multiprocesssing where possible
make sure that some time consuming setup functions are not called repeatedly

Great! Let's wait for the checks, then we can merge. Hopefully the tests will be faster also in GitHub workflow.

Great! Let's wait for the checks, then we can merge. Hopefully the tests will be faster also in GitHub workflow.

The checks just completed successfully. I'll merge.


History Action sheet broken iOS 14.5

On iOS 14.5, the action sheet to clear history is broken. Active bug to fix in another v0.3 build.
Found by @owoswordsman

Fixed in 9231f7b73898ccb3102a0a0448c4c8eb3a5483b4


feat: upgrade strict-event-emitter to 0.5.0

I faced the same problem as https://github.com/mswjs/http-middleware/issues/27.
This problem caused by strict-event-emitter version.
In another repositories uses higher than v0.4.3 so version but in this repository one is v0.2.0.
New API named Emitter is used from v0.3.2 and StrictEventEmitter was removed from v0.4.0.

Released: v0.13.0 🎉
This has been released in v0.13.0!

📄 Release notes
📦 npm package

Make sure to always update to the latest version (npm i @mswjs/data@latest) to get the newest features and bug fixes.

Predictable release automation by @ossjs/release.


Jjlee050 edit reposense tags

I am closing this PR. All reposense tweak PR go to #297

Pull Request Test Coverage Report for Build 1255

0 of 0   changed or added relevant lines in 0 files are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage remained the same at 72.238%

Files with Coverage Reduction
New Missed Lines
%

src/main/java/seedu/clinicio/model/ClinicIo.java
1
84.11%

Totals

Change from base Build 1252:
0.0%

Covered Lines:
3406

Relevant Lines:
4715

💛  - Coveralls


Nix GUI Support

The nix-gui project has the potential to make NixOS an awesome user experience even for those without programming language experience. It might be worthwhile to include in DevOS as it may allow us to target a broader audience.
It is at least worth discussing.

Related https://github.com/nix-gui/nix-gui/issues/150


fix: update pw to 1.43.1 and fix expect types

Updates PW to the latest release 1.43.1 as part of the regular cadence #614
PR removes some of the build techniques that we used for importing expect library matcher from test package, Instead since the expect package has been includes in the playwright library itself, we no longer need that workaround since test-runner is separated already to @playwright/test package.
Fix the expect types that was raised in https://github.com/elastic/synthetics/issues/917
To workaround the module resolution issues, we resolve to the absolute path and use Node.js resolution algorithm to find the package that was installed along with the Synthetics library. This is safer than manually recursing.

@devcorpio It might have worked before because of the NPM synthetics linking, if you create a fresh project and make sure there is no links to your local @elastic/synthetics repo. Then expect types wont work since the resolution of import would break as the issue pointed out.
With the new change, This is not the case anymore as we are using the Node.js Import resolution which would find it in the node_modules directory. Probably a mistake from my part when adding the expect support.


Update to Tracy v0.9

Fixes #52

Uh, unfortunately I cannot reproduce the CI failures locally on my W10 VM… :/

This is not reproducible on a fresh win2k22 install either… Very curious this only occurs on nightly, so would be very nice if this was actually reproducible, as right now this looks like potentially something genuinely serious, but I’ve no means whatsoever to actually do anything about it.
TBH I’m inlined to ignore the failure for now and just merge.

I ran out of steam to investigate the sigsegv past what I wrote down in https://github.com/wolfpld/tracy/issues/483. Either way this seems like not-a-bindings issue (except maybe for the fact that we’re using the manual lifetime mode).


Implement archival_config timeline endpoint in the storage controller

Implement the timeline specific archival_config endpoint also in the storage controller.
It's mostly a copy-paste of the detach handler: the task is the same: do the same operation on all shards.
Part of #8088.

Can you update existing tests to invoke the API via the storage controller instead of directly on the pageserver, so that we have some coverage of this?

There haven't been any tests so I wrote #8824 to address that.


Is your feature request related to a problem?
Depends on how you define problem.
Describe the solution you'd like
Be able to choose prepared spells during character creation, like a final tab before the cosmetics that's just the spellbook.
Describe alternatives you've considered
Unless there's a way to create actual custom modules to load a new character into, prepare spells and rest there, save the character, and load that saved character into the actual game... we're stuck with what the game automatically chooses for us until we get a chance to rest if we're prepared casters.
Additional context
There are any number of reasons from RP to minmax to want to be able to do this.

Can't you just use the ToyBox rest function first thing into the game?


Automated Package Upgrade

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Resolver module separation

This change is

Current dependencies on/for this PR:

main

PR #2675

PR #2676   👈

This comment was auto-generated by Graphite.


To edit a relay stuck the connection

When it try to edit a relay for the first time, the connection still frozen until the relay is edited in the server (database).

Test: what if another client try to modify a relay state at the same time?

It seems like is a server side problem..
For every connection, there are two threads, one for the sender and the other for the receiver.
The thing is that the receiver edite/deletes/updates in the same thread. It must be in another thread so the receiver can continues receiving things


Exposes subscription mode for consumer

The default subscription mode is Durable but some use cases would benefit from a NonDurable mode for consumers and are not necessarily compatible with a reader. The consumer property was not exposed through the pulsar4s wrapper hence this proposed change.

Hello @judu @RemiBardon ,
sorry for the direct ping but the repository doesn't look very active and I'm not sure if anyone has seen my PRs :D
This one is because we need it for the apache/james project pulsar integration. the other one https://github.com/CleverCloud/pulsar4s/pull/570 is for upgrading the pulsar client version to match the pulsar server version hosted on clever cloud :D
thanks !

hello again, any chance to have this merged or do you need me to do additional work ?

hello again @judu @RemiBardon,
any chance to have this merged or do you want me to do additional work ?
thanks
(also a release would be a nice xmas gift :D )

@jeantil I don't work at Clever Cloud anymore, I can't help you.

Hello and happy new year !
@judu any chance you could review this very small change and make a release :pray: ?

Thanks for this feature @jeantil !
I'm planning on releasing pulsar4s with it before the end of the week.

thank you ! that's great news :D

Hello, @judu
I come back to the project on a regular basis and I refrained from doing a direct ping so far but I would really like a release :D
thanks !


Only returning first character on macOS.

I have a strange issue here that seems to only be affecting macOS, not Windows.
I connect to an ODBC via a MySQL driver just fine but when I run a query like:
const query = "SELECT * FROM tasks LIMIT 10";

const tasks = odbc.querySync( query );

console.log( "tasks", tasks );

The resulting fields from the tasks table only have the first character returned.
For example, here's what the column and value looks like in the MySQL table:

And here is what is returned to the console for that same record:

When I use iODBC Demo Ansi to connect and run the same query, I get the correct, full value:

So that makes me think that the ODBC connection is fine and it might be something to do with how the node-odbc is handling the data.
Even stranger, when I run this on Windows, it returns the correct, full value through node-odbc just fine.

In the event that others have run into this or anybody knows what I could investigate, I've added it here.
Thanks for any help!

@joshuapinter What is the datatype of that field?

VARCHAR(50)

I tried changing it to TEXT and VARBINARY and a couple others to see if it helped. No change. It seems only the numeric values, like INT come through with more than the first character.
But, everything works fine on iODBC Demo application.


Mature Accel

MVP:

one dependency per cell
can grab a (set path) of all kids of dependency (resolves #26)
can send output as a poke to a pith

Stretch Goals:

faces (a1, a2, b1, b2) with values from other cells pre-assigned (resolves #25)
cells with state (assign dynamic variables to individual cells, can modify them from outside)
can use output poke as a cron job
can send output poke to face (a1 etc)

All of these features need to be usable in the UI to be considered complete

This is resolved per dev summit context, need to remake this issue with specific and up-to-date deliverables


embassy-rs now compiles on stable

embassy-rs just releases a new version on crates.io and it now compiles on stable.

Thanks for the update!


partial updates

Can you add the update request to the gRPC version of the index as well? pinecone/core/grpc/index_grpc.py. The interface should be similar to the upsert API, including async_req option

@yaakovs  I dont see the updated spec files, did you forget to add them?

@yaakovs I dont see the updated spec files, did you forget to add them?

https://github.com/pinecone-io/pinecone-python-client/pull/48/files

@rajat08 the docs are synced with the client repo or with the core repo?

@mutayroei they are in both places at the moment, but make gen does change them simultaneously. Also didn't see Yaakov's other PR in the client repo

@mutayroei they are in both places at the moment, but make gen does change them simultaneously. Also didn't see Yaakov's other PR in the client repo
I meant to ask - which one of them is the source for Redocly portal?


The write of websocket.Conn gets blocked if called from different thread when blocking_mode is true

I am not sure if I am using it wrong, but using default httpz config, blocking_mode is true, after receiving websocket.Conn when I try to write to it from a different thread, it blocks that thread and never properly writes it meaning the Client never receives the message, did a litttle investigation call to std.posix.writev hangs, so I think I am using it the wrong way? Window's WriteFile API doesn't support different thread "write"(s)? or it's the Zig's standard library at fault? Any suggestions would be helpful.

If I'm being honest, these window issues have been my bane. See https://github.com/karlseguin/websocket.zig/issues/46 for another example. For one, I have no means of reproducing them.
It doesn't sound like you're doing anything wrong.  And I'm unsure if my libraries are wrong, or if Zig's windows abstractions are wrong.
I believe my use of writev is correct, or at least, it does the same thing as Zig'd stdlib: https://github.com/ziglang/zig/blob/2e2927735d26fc6047343f0c620f20e9048ebaa5/lib/std/net.zig#L1884
I find the windows documentation frustrating - I don't see it mention anything about thread-safety, although non-official sources seem to indicate that it is thread-safe. Anyways, *websocket.Conn.write itself uses a mutex to ensure a single thread writes at any given time.
The best thing to do is to strip out httpz and websocket.zig and see if the problem can be reproduced using Zig's standard library - as we did with the linked issue above.


Using CSI # S to scroll screen buffer is incorrect and can lead to data loss

Environment data
PS version: 5.1.17712.1000
PSReadline version: 2.0.0-beta2
os: 10.0.17712.1000 (WinBuild.160101.0800)
PS file version: 10.0.17712.1000 (WinBuild.160101.0800)

Steps to reproduce or exception report
Make sure that the prompt is at the bottom of the screen, then press enter any number of times. One should do.
PSReadline will emit ^[[1S (from ConsoleLib.cs, perhaps via ReadLine.cs).
Scrolling the viewport in this manner can:

destroy the user's scrollback buffer
force an expensive full-buffer repaint over a VT link (like ssh or pty)

cc @zadjii-msft

In your scenario, I think the call is here.
v1.2 was calling ScrollConsoleScreenBuffer, and the non-VT code for this method also calls ScrollConsoleScreenBuffer. The documentation states that this escape is equivalent.
I don't think any of the scrollback buffer is being destroyed (barring bugs) other than the part that needs to be scrolled off the screen because the buffer is full.
As for performance - is it worse than calling ScrollConsoleScreenBuffer? In this scenario, it's scrolling a single line, basically what would happen if you wrote a single newline.

Wait, why were you ever calling that API? In this case, aren't you effectively just newlining down from the previous output? Why would you need to call ScrollConsoleScreenBuffer to handle that? In the console since time immemorial, if you emit a newline at the bottom of the buffer, conhost will automatically discard the top lines and create new ones at the bottom. It certainly does that a lot more efficiently than ScrollConsoleScreenBuffer does. ScrollConsoleScreenBuffer needs to copy the contents of each row into another row, while the "newline at the bottom" case just reallocates a single row.
Plus, this might be getting hit more frequently (esp on *nix) where there's no "screen buffer", there's only the viewport. So the cursor is going to be hitting the bottom of the "buffer" after only ~30 lines (the height of your window) instead of after 9000 lines.
As far as "destroy the user's scrollback buffer", what we mean can be easily repro'd with the following VT sequence. You can test this out from bash with the following commandline:
printf "\033[2J\033[HThis line of text will disappear\n1\n2\n3\n4\n5\n6\n7\n8\015\033[1S"

This string:

Clears the buffer (forcing existing contents into scrollback, for terminals that support one)
moves the cursor to 0,0
prints a line of text. This line of text is going to disappear from the terminal.
Prints 8 more lines of text, each with a number
Uses Scroll Up to move the viewport contents up by one row.

You can see the output here:

If you scroll up into the terminal's history, you'll see that the line "This line of text will disappear" does not appear in the scrollback.

While this example is trivial, in the real world case, this means that if you hit enter a couple of times at the prompt line after outputting a bunch of text, the history gets deleted instead of moving into the scrollback.

@lzybkr  It's here #607

@MVKozlov - are you sure? I think that issue is unrelated.

I doesn't look in ScrollConsoleScreenBuffer  code exactly, but I remember that cursor at the end of a line - it s the problem with issue I mentioned

Ah, I see, thanks for the clarification, and sorry I haven't gotten around to fixing your issue.

Ah, this also causes some weird viewport things during WSL Interop. Since it's "scrolling the viewport" instead of emitting line feeds, this happens:


auto CI deploy scheme

Should auto-deploy on merge to main.
(Prod and staging deploy actions are being worked on already and should be manually triggerable by end of today - 26 Feb.)
Need to confirm merge protections are working correctly.

Manually triggered workflows now exist for this.


URL  in a code  is  not accepted my GitHub .

Looking forward for reporting a security issue:
Please report security issues by sending an email to<EMAIL_ADDRESS>instead of raising a GitHub issue.
Describe the bug
A clear and concise description of what the bug is.
A bug is in contributors.md file  in which i give my URL as all the contributors had added but it creates trouble in pushing the code .
To Reproduce
Steps to reproduce the behavior:

Go to '...'
Click on '....'
Scroll down to '....'
See error

Expected behavior
A clear and concise description of what you expected to happen.
I expected  to accept the  my changing in a code by GitHub.
Screenshots
If applicable, add screenshots to help explain your problem.

Desktop (please complete the following information):

OS: [windows 10 pro]
Browser [chrome, ]
Version [e.g. <IP_ADDRESS>]

Smartphone (please complete the following information):

Device: [ iPhone6]
OS: [ iOS ]
Browser [ safari]
Version [12.4.1]

Additional context
Add any other context about the problem here.
kindly fix my issue ASAP

This is not an issue with the "file". You need not add links to your GitHub on the contributing guide.
Thanks for the issue.
You can seek additional help on the forum if needed: https://www.freecodecamp.org/forum/t/pull-request-is-creating-error/318466/5


node-pre-gyp WARN Hit error EACCES: permission denied, mkdir '/Users/ms/myapp/node_modules/fsevents/lib'

MS-Mac:myapp ms$ node -v
v11.9.0
MS-Mac:myapp ms$ npm -v
6.7.0
node-pre-gyp WARN Using needle for node-pre-gyp https download
node-pre-gyp WARN Pre-built binaries not installable for<EMAIL_ADDRESS>and<EMAIL_ADDRESS>(node-v64 ABI, unknown) (falling back to source compile with node-gyp)
node-pre-gyp WARN Hit error EACCES: permission denied, mkdir '/Users/ms/myapp/node_modules/fsevents/lib'
gyp ERR! configure error
gyp ERR! stack Error: EACCES: permission denied, mkdir '/Users/ms/myapp/node_modules/fsevents/build'
gyp ERR! System Darwin 17.7.0
gyp ERR! command "/usr/local/bin/node" "/usr/local/lib/node_modules/npm/node_modules/node-gyp/bin/node-gyp.js" "configure" "--fallback-to-build" "--module=/Users/ms/myapp/node_modules/fsevents/lib/binding/Release/node-v64-darwin-x64/fse.node" "--module_name=fse" "--module_path=/Users/ms/myapp/node_modules/fsevents/lib/binding/Release/node-v64-darwin-x64" "--napi_version=3" "--node_abi_napi=napi"
gyp ERR! cwd /Users/ms/myapp/node_modules/fsevents
gyp ERR! node -v v10.15.1
gyp ERR! node-gyp -v v3.8.0
gyp ERR! not ok
node-pre-gyp ERR! build error
node-pre-gyp ERR! stack Error: Failed to execute '/usr/local/bin/node /usr/local/lib/node_modules/npm/node_modules/node-gyp/bin/node-gyp.js configure --fallback-to-build --module=/Users/ms/myapp/node_modules/fsevents/lib/binding/Release/node-v64-darwin-x64/fse.node --module_name=fse --module_path=/Users/ms/myapp/node_modules/fsevents/lib/binding/Release/node-v64-darwin-x64 --napi_version=3 --node_abi_napi=napi' (1)
node-pre-gyp ERR! stack     at ChildProcess. (/Users/ms/myapp/node_modules/fsevents/node_modules/node-pre-gyp/lib/util/compile.js:83:29)
node-pre-gyp ERR! stack     at ChildProcess.emit (events.js:189:13)
node-pre-gyp ERR! stack     at maybeClose (internal/child_process.js:970:16)
node-pre-gyp ERR! stack     at Process.ChildProcess._handle.onexit (internal/child_process.js:259:5)
node-pre-gyp ERR! System Darwin 17.7.0
node-pre-gyp ERR! command "/usr/local/bin/node" "/Users/ms/myapp/node_modules/fsevents/node_modules/node-pre-gyp/bin/node-pre-gyp" "install" "--fallback-to-build"
node-pre-gyp ERR! cwd /Users/ms/myapp/node_modules/fsevents
node-pre-gyp ERR! node -v v10.15.1
node-pre-gyp ERR! node-pre-gyp -v v0.10.3
node-pre-gyp ERR! not ok
Failed to execute '/usr/local/bin/node /usr/local/lib/node_modules/npm/node_modules/node-gyp/bin/node-gyp.js configure --fallback-to-build --module=/Users/ms/myapp/node_modules/fsevents/lib/binding/Release/node-v64-darwin-x64/fse.node --module_name=fse --module_path=/Users/ms/myapp/node_modules/fsevents/lib/binding/Release/node-v64-darwin-x64 --napi_version=3 --node_abi_napi=napi' (1)
npm WARN<EMAIL_ADDRESS>No description
npm WARN<EMAIL_ADDRESS>No repository field.
npm WARN optional SKIPPING OPTIONAL DEPENDENCY<EMAIL_ADDRESS>(node_modules/fsevents):
npm WARN optional SKIPPING OPTIONAL DEPENDENCY<EMAIL_ADDRESS>install: node install
npm WARN optional SKIPPING OPTIONAL DEPENDENCY: Exit status 1
audited 3693 packages in 4.353s
found 0 vulnerabilities

Closing due to staleness, although I'm assuming all the reaction emoji here means others are experiencing similar issues.
Here's my brief advice on EACCES: permission denied errors like this: try removing the node_modules directory that's causing this problem and install again. Perhaps something has been installed with sudo (or root perms by some other means) and you're running into ownership conflicts. In this case I'd rm -rf /Users/ms/myapp/node_modules and npm install again. If the rm -rf fails with a permission problem then you know your culprit and may have to sudo rm -rf it.


Add removal instructions for (boring people)

Looks Nice!! tysm :)

@LuanderFarias hi. I was the ghost in this pull (not lying, clone the repo and check this pull.) thanks for merging this. it was my first pull request ever!

@LuanderFarias hi. I was the ghost in this pull (not lying, clone the repo and check this pull.) thanks for merging this. it was my first pull request ever!

NO PROBLEM! Great you were able to get your first contribution through here :)

@LuanderFarias hi. I was the ghost in this pull (not lying, clone the repo and check this pull.) thanks for merging this. it was my first pull request ever!

NO PROBLEM! Great you were able to get your first contribution through here :)

yep!
btw im going to be a mac user as of december 26 but I'll see what features I can bring from mac arc!

@LuanderFarias hi. I was the ghost in this pull (not lying, clone the repo and check this pull.) thanks for merging this. it was my first pull request ever!

NO PROBLEM! Great you were able to get your first contribution through here :)

yep!
btw im going to be a mac user as of december 26 but I'll see what features I can bring from mac arc!

NICE, sorry for the arcfox being so abandoned lol. Hope everything goes fine with arc :3


chore(deps): upgrade dependencies

Upgrades project dependencies. See details in workflow run.

Automatically created by projen via the "upgrade-main" workflow

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

Comparison is base (abc06f3) 95.69% compared to head (7bcae6b) 95.69%.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@           Coverage Diff           @@
##             main     #874   +/-   ##
=======================================
  Coverage   95.69%   95.69%
=======================================
  Files           1        1
  Lines         395      395
  Branches       53       53
=======================================
  Hits          378      378
  Misses         17       17

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


Feature/terraform openstack

Add basic OpenStack functionality

On the inventory plugin for terraform there are a couple of items.
Currently the args parser is defined globally, and there are other places in the code where there are local variables that are referencing a variable named "parser". By moving the definition of the args parser into main, that is completed avoided and there would be no confusion by about which variable is intended.
On the definition of the args parser,
modes = parser.add_mutually_exclusive_group()
If you include the following:
modes = parser.add_mutually_exclusive_group(required=True)
Then the following block of code is not required at all (argparser) will take care of it for you:
    if not args.list and not args.host:
        print('error: one of --list or --host is required', file=sys.stderr)
        print('{}')
        return 1

Was this tested when using this inventory plugin as part of executing an ansible playbook? The reason I ask, when I included the line:
from __future__ import print_function

In another plugin I created, it resulted in ansible generating errors during the playbook execution because ansible has been adamant about be python2 compatible such that much of their code you will see print called as built-in vs. function call.

@kenjones-cisco good point about the groups, I didn't know about the required param. I won't be able to get to it today but it should be pretty simple if you'd like to submit a PR.
The print function works as part of the Ansible run. We did test it, of course. If you'd like to give it a shot and let me know of any errors, we can try to improve compatibility on various systems.

@BrianHicks PR submitted.
As to the print comment, it could be the difference between library module and plugin module. Too bad ansible doesn't support py2 and py3 such that even thinking about this would be simpler.

@ChrisAubuchon @kenjones-cisco  do we feel this is this ready to merge?

The feedback I provided has already been merged into the branch feature/terraform.


DirectoryTree download support

What this PR does / why we need it:
The standard configuration now provides a downloader for resources of type directoryTreeand the legacy type filesystem.
It acts on the mimetypes for an artifact set (application/vnd.oci.image.manifest.v1+tar+gzip) and a tar/tgz archive (application/x-tar, application/x-tar+gzip, application/x-tgz). The default configuration extracts the content to a
filesystem folder. If the blob format is an artifact set, for example provided by the access method ociArtifact,
the default configuration accepts the image config mimetype (application/vnd.oci.image.config.v1+json).
In this case the final filesystem content provided by the image is download by evaluating the layered image file system.
The default behaviour can just be  used with

import "github.com/open-component-model/ocm/pkg/contexts/ocm/download"
download.For(octx).Download(printer,rsourceAccess,targetdir,vfs)

If the resource access describes an appropriate artifact, the new handler is automatically selected.
As usual, the target is always a virtual filesystem.
Like all download handlers. the dirtree download handler can also explicitly be used with

import "github.com/open-component-model/ocm/pkg/contexts/ocm/download/handlers/dirtree"
dirtree.New().Download(...)

In this mode, the behaviour can be influenced by specifying any list of accepted OCI artifact config mime types.
With New(mimetypes ...string).SetArchiveMode(true) it is possible to enable the archive target mode. The content is downloaded to an archive instead of extracted filesystem content.
The handler checks the mimetypes, only, but the default registration is done exclusively for the directory content resource types.
A context can be extended for other resource types with
download.For(octx),Register(type, [mimetype], dirtree.New...(...))

The handler also provides additional methods, which can be used to execute more specific tasks, for example
methods for an optimized content access providing an internal virtual filesystem or an archive byte stream trying to avoid unnecessary conversions depending on the actual input format,
The localization package has been adapted, accordingly. The localize.Instantiate method now prefers to use the
download handlers to get to the filesystem content to be configured, instead of expecting an archive resource.
Therefore, any (potentially own) resource type with any format can be used, as long as there is an appropriate downloader configured for the used OCM context.  An optional additional parameter can be used to restrict the accepted resource types
to an explicitly given set of types.
Which issue(s) this PR fixes:
Fixes #
Special notes for your reviewer:
Release Notes:

With the following component:
NAME      VERSION IDENTITY TYPE          RELATION
config    1.0.0            PlainText     local
image     6.3.5            ociImage      external
manifests 1.0.0            directoryTree local

I find that I have to use dirtree package in order for the  resource to be extracted correctly into the filesystem:
`	memfs := memoryfs.New()
_, _, err = dirtree.New().Download(common.NewPrinter(io.Discard), res, "", memfs)
if err != nil {
	log.Fatal(err)
}

vfs.Walk(memfs, "", func(path string, info fs.FileInfo, err error) error {
	fmt.Println(info.Name(), info.IsDir())
	return nil
})

`
From reading the PR I believe it should be possible to simple execute download.ForContext(...

@phoban01  If you use dirtree.New(...).Download(...) you explicitly use the dirtree downloader and nothing else.
It only checks the mimetypes, but not the resource types. So, you can enforce to use it on resources, regardless of their type to download dirtree-like  resources (with a matching mime type).
If you use download.For(...).Download(...) it tries to find a registered downloader with registration criterias matching the actual resource. This can be used without bothering with the kind of actually used resource (to just download it, whatever it is in a standard manner). If a matching downloader is found, it is used, otherwise just the blob is downloaded as provided by the access method. Here, for sure the dirtree downloader is used for the standard scenarios (it is registered for). This is especially the directoryTree  resource type with the tar-like mimetypes and the oci artifact archive mime type. But it is not used for other scenarios.
So, if you want to use an own resource type (directly expressing the dedicated new meaning of a general filesystem content), for example gitOpsTemplate, which is more expressive than just directoryTree. you can

either register the dirtree handler in advance for your OCM context and for this resource type at the registry (then it would automatically be chosen for all downloads using this context)
or you know what you are doing, and explicitly call the dirtree downloader on such a resource.

If, for example gitOpsTemplate should be a standard resource type, we should add such a registration as part of the standard.
Another possible scenario, where you might want to use the explicit dirtree usage is to overwrite the standard behaviour for a special use case. For example, an OCI image is typically downloaded as OCI artifact with the distribution spec format. But. if you want to access the effective filesystem, you could explicitly use the dirtree downloader for an OCI image, which handles this for you. It would make less sense to use it on a helm chart OCI artifact, because here the layers have a different meaning than building a directory tree.

Thanks for the explanation @mandelsoft. My assumption was that handlers for built-in OCM types would be registered by default but it seems that I need to explicitly register handlers which is fine.

I don't personally ever use this type gitOpsTemplate   so I don't think it needs to be added. There are other types and formats which we should start considering (for example web assembly) but I think that's a separate discussion :smile.

@phoban01 , may be you could approve this PR, It just added the explanation to the docu of the dirtree package,


download VRTNU not working

Jan$ youtube-dl --verbose -fmp4 -u<EMAIL_ADDRESS>-p xxx https://www.vrt.be/vrtnu/a-z/faqda/1/faqda-s1a10/
[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: ['--verbose', '-fmp4', '-u', 'PRIVATE', '-p', 'PRIVATE', 'https://www.vrt.be/vrtnu/a-z/faqda/1/faqda-s1a10/']
[debug] Encodings: locale UTF-8, fs utf-8, out UTF-8, pref UTF-8
[debug] youtube-dl version 2020.11.19
[debug] Python version 3.6.4 (CPython) - Darwin-15.6.0-x86_64-i386-64bit
[debug] exe versions: ffmpeg 3.4.2, ffprobe 3.4.2
[debug] Proxy map: {}
[VrtNU] Logging in
[VrtNU] Requesting a token
[VrtNU] faqda-s1a10: Downloading webpage
[VrtNU] faqda-s1a10: Downloading JSON metadata
ERROR: Unable to download JSON metadata: HTTP Error 410: Gone (caused by <HTTPError 410: 'Gone'>); please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; see  https://yt-dl.org/update  on how to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
File "/Library/Frameworks/Python.framework/Versions/3.6/lib/python3.6/site-packages/youtube_dl/extractor/common.py", line 632, in _request_webpage
return self._downloader.urlopen(url_or_request)
File "/Library/Frameworks/Python.framework/Versions/3.6/lib/python3.6/site-packages/youtube_dl/YoutubeDL.py", line 2238, in urlopen
return self._opener.open(req, timeout=self._socket_timeout)
File "/Library/Frameworks/Python.framework/Versions/3.6/lib/python3.6/urllib/request.py", line 532, in open
response = meth(req, response)
File "/Library/Frameworks/Python.framework/Versions/3.6/lib/python3.6/urllib/request.py", line 642, in http_response
'http', request, response, code, msg, hdrs)
File "/Library/Frameworks/Python.framework/Versions/3.6/lib/python3.6/urllib/request.py", line 570, in error
return self._call_chain(*args)
File "/Library/Frameworks/Python.framework/Versions/3.6/lib/python3.6/urllib/request.py", line 504, in _call_chain
result = func(*args)
File "/Library/Frameworks/Python.framework/Versions/3.6/lib/python3.6/urllib/request.py", line 650, in http_error_default
raise HTTPError(req.full_url, code, msg, hdrs, fp)

Apparently, some mssecurevideo.json file got removed.
A similar issue in another tool appears to have been fixed here:
https://gitea.caret.be/jens/vrtnu-app/commit/24a495625ef9863486e81c4e2653aec30bec9fee?lang=fi-FI


added support for coin marginate futures

Hi,
the current "future" support only works for usdt marginated futures
added support for coin marginated futures
and renamed to usdt_futures and coin_futures accordingly

this looks like breaking change due to change in method names. please preserve the current standing api names to avoid users from changing their codebase from upgrading.

@bsliran please make changes to prevent breaking changes. else I have to close this PR

@bsliran please make changes to prevent breaking changes. else I have to close this PR

Thanks @bsliran this has been superseded by newer commits


WIP - chore(ci): add OpenShift 4 CI using Microshift

Related to #2473
Draft for adding E2E testing on OpenShift 4 leveraging Microshift.
TODO:

There are still many tests that fail. Make sure to pass all the tests before committing the workflow.
It uses tadayosi/microshift-action. Once it's ready, I'd like to move it to container-tools org.

Release Note
NONE

So maybe I should move tadayosi/microshift-action to container-tools/microshift-action first and get it accepted to run it in the pull req.
https://github.com/apache/camel-k/actions/runs/1244368425

<EMAIL_ADDRESS>is not allowed to be used in apache/camel-k. Actions in this workflow must be: within a repository owned by apache, created by GitHub, verified in the GitHub Marketplace or match the following: /@[a-f0-9][a-f0-9][a-f0-9][a-f0-9][a-f0-9][a-f0-9][a-f0-9]+, AdoptOpenJDK/install-jdk@, JamesIves/github-pages-deploy-action@5dc1d5a192aeb5ab5b7d5a77b7d36aea4a7f5c92, TobKed/label-when-approved-action@, actions-rs/, al-cheb/configure-pagefile-action@, apache/, burrunan/gradle-cache-action@, bytedeco/javacpp-presets/.github/actions/, chromaui/action@, codecov/codecov-action@, conda-incubator/setup-miniconda@, container-tools/kind-action@, dawidd6/action-download-artifact@, docker://pandoc/core:2.9, eps1lon/actions-label-merge-conflict@, golangci/, gr2m/twitter-together@, gradle/wrapper-validation-action@, julia-actions/julia-runtest@, julia-actions/setup-julia@, manusa/actions-setup-minikube@, mikepenz/action-junit-report@, msys2/setup-msys...

@nicolaferraro If it's ok, could you give me the right to move it to container-tools?

Wow, the integration test suite almost passes! There are only the routes tests that fail. Impressive!
Out of curiosity, I don't see the oc login command used to run the test suite as a non-admin user. Is the OAuth server that comes out-of-the-box with OCP 4 also present in Microshift?

Yes, Microshift runs with the fixed non-admin user user and there is no OAuth server running in the cluster out of the box. You can create OpenShift resources but many of them have no effects. So if there are some e2e tests that rely on OAuth server they can be false positive.

Great, as long as the test suite is executed with a non-admin user, that is alright. There are no dependency on the OAuth server. One of the main reason we keep the Minishift workflow is to test RBAC, and the ability to install the operator as a standard user (which we may reconsider in the future).


Read 4 byte integers and floats and handle more unknown bytecode

Just an attempt to handle more bytecode and remove some of the "UNK_xx" found when viewing in lscrtoscript.
For example, when uploading hh_photo.cst
on countCS(me, tImg)
  tL = [3, 2, 73, 28, 83, 21, 43, 90, 92, 91, 37, 4, 3, 84, 12, 102, 103, 108, 97, 43, 44, 89, 109, 65, 61, -4, 76]
  tA = 0
  tW = tImg.width
  tH = tImg.height
  i = 1
  repeat while i <= 100
    tA = ERROR mod ERROR mod tA + tImg.getPixel(i mod tW, i * i mod tH).paletteIndex * tL.getAt(i mod tL.count + 1).undefined
    i = 1 + i
  end repeat
  return(tA)
  exit
end

Now turns into
on countCS(me,tImg)
  tL = [3, 2, 73, 28, 83, 21, 43, 90, 92, 91, 37, 4, 3, 84, 12, 102, 103, 108, 97, 43, 44, 89, 109, 65, 61, -4, 76]
  tA = 0
  tW = tImg.width
  tH = tImg.height
  i = 1
  repeat while i <= 100
    tA = tA + tImg.getPixel(i mod tW, i * i mod tH).paletteIndex * tL.getAt(i mod tL.count + 1) mod 85000
    i = 1 + i
  end repeat
  return(tA)
  exit
end

Awesome progress!

i'll review this when i'm not so tired...
however, this repo is technically inactive
https://github.com/Earthquake-Project
is the current place to find everything

I pushed it before because I couldn't find lscrtoscript project on the Earthquake-Project 😛

I pushed it before because I couldn't find lscrtoscript project on the Earthquake-Project 😛

I believe this is the new repo:
https://github.com/Earthquake-Project/ProjectorRays

you should indeed PR projectorrays
tbh, it is unfortunate I am the one to have to try answering this question, as the bytecode still largely confuses me. it doesn't help, either there's still a bunch of those unknowns...
my personal opinion is to be careful with this...
at least, by our earlier observations, the highest length for a bytecode has thus far been four bytes, this would mean there are five byte codes. Of course, anything is possible with this format.
All I ever really did with bytecode is resolve some ambiguity/inaccuracy from previous efforts made by other people, and assign names to all the opcodes. well, all the known ones. also note the 'special' opcodes (all getters/setters), which are context-sensitive in what a previous opcode had done.
you're only showing the output lingo script, and not the disassembly, so it's hard to follow your example, anyways... also, is this a cst obtained by unprotecting a cct/cxt? or do you have the original source code?
anyways, you'd be much better off to re-submit this to the typescript port of projectorrays

you're only showing the output lingo script, and not the disassembly, so it's hard to follow your example, anyways... also, is this a cst obtained by unprotecting a cct/cxt? or do you have the original source code?

It was an unprotected .cct who used 4 byte integers, demonstrating how fixing the bytecode reading is beneficial to recovering workable Lingo script.

anyways, you'd be much better off to re-submit this to the typescript port of projectorrays

I know, but you had no repo for lscrotscript on the new project page.

seeing the actual example file would be far more beneficial to improving the decompiler. we're otherwise flying blind and don't really know which way to turn next if this is wrong.
https://github.com/Earthquake-Project/ProjectorRays
it's right there
lscrtoscript in its original form, is abandoned


Row-order after df1.join(df2, how="right")

Disclaimer:
This is not an issue or bug, just an observation.
If correct, I propose to add a bit of text in the documentation to clarify the matter.
What:
I was using .join() and noticed that the order of elements in self may change when using how="right" or how="left". This is even visible in the provided example of .join() method where how="right" is used. For example, in case we use:
f1 = pr.from_dict({'Chromosome': ['chr1', 'chr1', 'chr1'], 'Start': [3, 8, 5],
                   'End': [6, 9, 7], 'Name': ['interval1', 'interval3', 'interval2']})
f2 = pr.from_dict({'Chromosome': ['chr1', 'chr1'], 'Start': [1, 6],
                   'End': [2, 7], 'Name': ['a', 'b']})

print(f1)
print(f1.join(f2, how="left"))

we will see
+--------------+-----------+-----------+------------+
| Chromosome   |     Start |       End | Name       |
| (category)   |   (int32) |   (int32) | (object)   |
|--------------+-----------+-----------+------------|
| chr1         |         3 |         6 | interval1  |
| chr1         |         8 |         9 | interval3  |
| chr1         |         5 |         7 | interval2  |
+--------------+-----------+-----------+------------+
Unstranded PyRanges object has 3 rows and 4 columns from 1 chromosomes.
For printing, the PyRanges was sorted on Chromosome.
+--------------+-----------+-----------+------------+-----------+-------+
| Chromosome   |     Start |       End | Name       |   Start_b | +2    |
| (category)   |   (int64) |   (int64) | (object)   |   (int64) | ...   |
|--------------+-----------+-----------+------------+-----------+-------|
| chr1         |         5 |         7 | interval2  |         6 | ...   |
| chr1         |         3 |         6 | interval1  |        -1 | ...   |
| chr1         |         8 |         9 | interval3  |        -1 | ...   |
+--------------+-----------+-----------+------------+-----------+-------+
Unstranded PyRanges object has 3 rows and 7 columns from 1 chromosomes.
For printing, the PyRanges was sorted on Chromosome.
2 hidden columns: End_b, Name_b

Notice that the order of rows in f1 is no longer the same.
I understand that this is most likely a "design choice" considering that interval1 and interval3 are not overlapping with any interval in f2. Regardless, I think it makes sense to add a few sentences in the documentation and emphasize that the order is not preserved after join().
If this is not a "design choice", I wonder if preserving the order is a "useful" feature?
In case of the former, I would be happy to provide this as a pull request if my reasoning is agreed by the developer(s).

It would not be hard to preserve the ordering. Perhaps it could be done with a keyword flag?
I have some days in May where I have the time to fix this btw. I can do it, but in a few weeks.

Thank for your remark @endrebak.
Happy to see that you agree with the utility of this functionality.
I took the liberty of implementing a suggested pr, you may see it in #304.
Hope you like it.
I also added a bit of documentation to reflect this change.
Thanks for PyRange, its fun/fast to work with it!

Merged into master branch. Thanks for contributing and reporting :D


Update id properties

Updates IDProperty and ReferenceProperty to support both stix 2.0 and 2.1 rules regarding identifiers.   Behavior is chosen via a new "spec_version" constructor parameter.  Also changes all uses of IDProperty and ReferenceProperty to specify a particular spec_version.

Codecov Report

:exclamation: No coverage uploaded for pull request base (stix2.1@5b6a0dc). Click here to learn what that means.
The diff coverage is 100%.

@@            Coverage Diff             @@
##             stix2.1     #269   +/-   ##
==========================================
  Coverage           ?   98.58%
==========================================
  Files              ?      117
  Lines              ?    12477
  Branches           ?        0
==========================================
  Hits               ?    12301
  Misses             ?      176
  Partials           ?        0

Impacted Files
Coverage Δ

stix2/test/v21/test_indicator.py
100% <ø> (ø)

stix2/test/v21/test_sighting.py
100% <ø> (ø)

stix2/v20/sro.py
100% <ø> (ø)

stix2/test/v20/test_sighting.py
100% <ø> (ø)

stix2/v21/common.py
100% <ø> (ø)

stix2/v20/common.py
100% <ø> (ø)

stix2/test/v20/test_report.py
100% <ø> (ø)

stix2/v20/bundle.py
100% <ø> (ø)

stix2/v21/sdo.py
92.07% <ø> (ø)

stix2/v21/sro.py
94.59% <ø> (ø)

... and 10 more

Continue to review full report at Codecov.

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


fix getEURegions filter

The countries for Frankfurt and London are de and uk. We just left out de.
Resolves M3-426.

Closed in favor of https://github.com/linode/manager/pull/3235

https://www.youtube.com/watch?v=-Y2R01k4SXI


Add blog post about the optimizations in OTP 26

@frej, thanks for the comments. I have addressed the min two fixup commits.

@bjorng: Looks good to me!


Add proper dependency for this gem

I don't see any require statements in this gem
And there is no dependency declared in gemspec
You properly need rake, some-gem-with-railties

There is no need for any require statements. The library will only be required in a Rails application where Rails::Railtie is already available. If it isn't available, an error would be appropriate as it won't work. There's no need to require Rack as Rails handles that already and we're simply providing an additional set of tasks in the Railtie.
The only dependency needed in the gemspec would be Rails along with what versions on Rails are required. I haven't tested in anything less than Rails 4 but it probably works fine in Rails 3 too. Feel free to submit a pull request :smile:


Separate view state and proof state

As suggested by Manuel in #830, it might be a good idea if undo/redo didn't change the current view of the workspace (i.e. which slice you're viewing).
To do this, I propose we split the "view state" consisting of the slice path and view dimension from the proof state, so that it's not stored in the history. Moreover, we could also include the attachment/slice highlight. Basically everything except diagram and attach in Workspace.
https://github.com/homotopy-io/homotopy-rs/blob/a4cfd16d8e70e6382ce5e5c782ee12f975c98af9/homotopy-model/src/proof.rs#L42-L49
Perhaps we could combine this with the existing View struct.
https://github.com/homotopy-io/homotopy-rs/blob/a4cfd16d8e70e6382ce5e5c782ee12f975c98af9/homotopy-model/src/proof.rs#L30-L32
We would also need to have a separate class of "view actions" which update the view. These are precisely the actions that are marked as not relevant wrt history (so we could get rid of this function as all proof actions will be relevant):
https://github.com/homotopy-io/homotopy-rs/blob/a4cfd16d8e70e6382ce5e5c782ee12f975c98af9/homotopy-model/src/proof.rs#L193-L202
There are a few subtleties:

We need to be careful about updating the view, as a change to the proof state (e.g. by undo/redo) can render the current view invalid so we need to update it accordingly.
Some actions such as Attach depend on the current view, so we would need to refactor these actions so they don't depend on how you were viewing the workspace at the time of the action but rather encode that information as part of the action (e.g. see Contract and Expand). This is related to #836 and doing this would fix that.

This seems to have a lot of benefits, but it's a big refactoring job so we should definitely discuss this.

I absolutely love this idea.

I really like this as well, but I expect it will cause some problems labelled conflicts. Should we wait for labelled to be merged before implementing this?

Yes, let's wait for labelled to be merged, it's pretty much ready to merge now.

I have tried to implement this and I don't think it's a good idea anymore, therefore I am closing this as "not planned".
The issues with the action dump being non-determinist have already been fixed.


MRML - Tf2 synchronization

[ ]  Review existing code (Broadcaster and Listener)
[ ]  Come up with UI to select which MRML transformations to synchronize with Tf2.  Maybe support chain of transformations?  We should look at how IGTL transformations are supported in Slicer.


Animated scrollTop not working on Chrome nor Firefox

Hello, short version:
installed smoothState, site is up http://stavebnikancelar.eu/ but no animated scrolling seems to be taking place when changing between pages.
Does anybody see the problem?

jquery-1.11.0.min.js should be above jquery.smoothState.js
<script src="js/jquery.smoothState.js"></script>
<script src="js/functions.js"></script>
<script src="js/jquery-1.11.0.min.js"></script>

to below;
<script src="js/jquery-1.11.0.min.js"></script> <!-- JQuery should be on top of all jqs plugins -->
<script src="js/jquery.smoothState.js"></script>
<script src="js/functions.js"></script>

Okaaaaay... thanks a lot, seems that scrolling is fixed, but now lightbox does not work. At least I know where to seek the problem.

@J4Y4G3 Did you ever get the lightbox to work?


Feature/endpoint to check if dataset exists

New endpoint .../publication-imex/ac to check if a publication exists and get its AC from its IMEX id. This will be used to check publications curated in Intact when searching in imex consortium.
Currently we make a call to export the publication just to check if it exists and we ignore the response, so we don't need to actually do any export, we just need to get the publication and return a 200.

Tested locally. To be tested in dev before deploying in prod.

Tested in dev: http://ves-ebi-46.ebi.ac.uk:8080/intact/dev/editor/service/export/mi/publication-imex/ac?imex=IM-14434


Docker for Mac nginx example doesn't run

I posted this on StackOverflow but doesn't appear to have a known solution/workaround:
http://stackoverflow.com/questions/39082936/docker-for-mac-nginx-example-doesnt-run

Cutting and pasting your command line works fine on my mac.
Are there any firewalls running on your local machine?

Works fine for me too. Does docker logs webserver show anything?
There is support for Docker for Mac at https://github.com/docker/for-mac

@mikegcoleman - no firewall
@justincormack - no, there is no output when I run your command


mmdetection/mmdet/core/evaluation/mean_ap.py",   num_classes = len(det_results[0])  # positive class num IndexError: list index out of range

Describe the bug
Reproduction
the command I use was: python ./tools/train.py ./configs/faster_rcnn/faster_rcnn_r50_fpn_1x_coco.py
I meet this bug when the first epoch training over,in eval,the terminal shows
  File "/home/wangyh21/assignment/mmtest/mmdetection/mmdet/core/evaluation/mean_ap.py", line 354, in eval_map     num_classes = len(det_results[0])  # positive class num IndexError: list index out of range
I want to train a model to detect the vichle,so, that was only one class 'vehicle' in my dataset ,and my dataset is voc format.
I changed  four file under:
`
mmdetection/mmdet/datasets/voc.py
mmdetection/mmdet/core/evaluation/class_names.py
mmdetection/configs/faster_rcnn/faster_rcnn_r50_fpn_1x_coco.py
mmdetection/configs/base/models/faster_rcnn_r50_fpn.py
`
Environment
sys.platform: linux
Python: 3.7.0 (default, Jun 28 2018, 13:15:42) [GCC 7.2.0]
CUDA available: True
GPU 0: Tesla V100-SXM2-16GB
CUDA_HOME: /usr/local/cuda
NVCC: Build cuda_11.0_bu.TC445_37.28540450_0
GCC: gcc (Ubuntu 7.5.0-3ubuntu1~18.04) 7.5.0
PyTorch: 1.10.0+cu102
PyTorch compiling details: PyTorch built with:

GCC 7.3
C++ Version: 201402
Intel(R) Math Kernel Library Version 2019.0.0 Product Build 20180829 for Intel(R) 64 architecture applications
Intel(R) MKL-DNN v2.2.3 (Git Hash 7336ca9f055cf1bfa13efb658fe15dc9b41f0740)
OpenMP 201511 (a.k.a. OpenMP 4.5)
LAPACK is enabled (usually provided by MKL)
NNPACK is enabled
CPU capability usage: AVX512
CUDA Runtime 10.2
NVCC architecture flags: -gencode;arch=compute_37,code=sm_37;-gencode;arch=compute_50,code=sm_50;-gencode;arch=compute_60,code=sm_60;-gencode;arch=compute_70,code=sm_70
CuDNN 7.6.5
Magma 2.5.2
Build settings: BLAS_INFO=mkl, BUILD_TYPE=Release, CUDA_VERSION=10.2, CUDNN_VERSION=7.6.5, CXX_COMPILER=/opt/rh/devtoolset-7/root/usr/bin/c++, CXX_FLAGS= -Wno-deprecated -fvisibility-inlines-hidden -DUSE_PTHREADPOOL -fopenmp -DNDEBUG -DUSE_KINETO -DUSE_FBGEMM -DUSE_QNNPACK -DUSE_PYTORCH_QNNPACK -DUSE_XNNPACK -DSYMBOLICATE_MOBILE_DEBUG_HANDLE -DEDGE_PROFILER_USE_KINETO -O2 -fPIC -Wno-narrowing -Wall -Wextra -Werror=return-type -Wno-missing-field-initializers -Wno-type-limits -Wno-array-bounds -Wno-unknown-pragmas -Wno-sign-compare -Wno-unused-parameter -Wno-unused-variable -Wno-unused-function -Wno-unused-result -Wno-unused-local-typedefs -Wno-strict-overflow -Wno-strict-aliasing -Wno-error=deprecated-declarations -Wno-stringop-overflow -Wno-psabi -Wno-error=pedantic -Wno-error=redundant-decls -Wno-error=old-style-cast -fdiagnostics-color=always -faligned-new -Wno-unused-but-set-variable -Wno-maybe-uninitialized -fno-math-errno -fno-trapping-math -Werror=format -Wno-stringop-overflow, LAPACK_INFO=mkl, PERF_WITH_AVX=1, PERF_WITH_AVX2=1, PERF_WITH_AVX512=1, TORCH_VERSION=1.10.0, USE_CUDA=ON, USE_CUDNN=ON, USE_EXCEPTION_PTR=1, USE_GFLAGS=OFF, USE_GLOG=OFF, USE_MKL=ON, USE_MKLDNN=ON, USE_MPI=OFF, USE_NCCL=ON, USE_NNPACK=ON, USE_OPENMP=ON,

TorchVision: 0.11.1+cu102
OpenCV: 4.5.4-dev
MMCV: 1.4.0
MMCV Compiler: GCC 7.3
MMCV CUDA Compiler: 10.2
MMDetection: 2.19.0+f08548b
Error traceback
2021-12-10 15:06:03,589 - mmdet - INFO - Epoch [1][3050/3053]   lr: 2.000e-02, eta: 1:26:38, time: 0.154, data_time: 0.007, memory: 2304, loss_rpn_cls: 0.0066, loss_rpn_bbox: 0.0287, loss_cls: 0.1110, acc: 95.3926, loss_bbox: 0.1749, loss: 0.3212
2021-12-10 15:06:04,083 - mmdet - INFO - Saving checkpoint at 1 epochs
completed: 0, elapsed: 0s
---------------iou_thr: 0.5---------------
Traceback (most recent call last):
File "./tools/train.py", line 185, in
main()
File "./tools/train.py", line 181, in main
meta=meta)
File "/home/wangyh21/assignment/mmtest/mmdetection/mmdet/apis/train.py", line 203, in train_detector
runner.run(data_loaders, cfg.workflow)
File "/home/wangyh21/anaconda3/lib/python3.7/site-packages/mmcv/runner/epoch_based_runner.py", line 127, in run
epoch_runner(data_loaders[i], **kwargs)
File "/home/wangyh21/anaconda3/lib/python3.7/site-packages/mmcv/runner/epoch_based_runner.py", line 54, in train
self.call_hook('after_train_epoch')
File "/home/wangyh21/anaconda3/lib/python3.7/site-packages/mmcv/runner/base_runner.py", line 307, in call_hook
getattr(hook, fn_name)(self)
File "/home/wangyh21/anaconda3/lib/python3.7/site-packages/mmcv/runner/hooks/evaluation.py", line 267, in after_train_epoch
self._do_evaluate(runner)
File "/home/wangyh21/assignment/mmtest/mmdetection/mmdet/core/evaluation/eval_hooks.py", line 58, in _do_evaluate
key_score = self.evaluate(runner, results)
File "/home/wangyh21/anaconda3/lib/python3.7/site-packages/mmcv/runner/hooks/evaluation.py", line 362, in evaluate
results, logger=runner.logger, **self.eval_kwargs)
File "/home/wangyh21/assignment/mmtest/mmdetection/mmdet/datasets/voc.py", line 89, in evaluate
use_legacy_coordinate=True)
File "/home/wangyh21/assignment/mmtest/mmdetection/mmdet/core/evaluation/mean_ap.py", line 354, in eval_map
num_classes = len(det_results[0])  # positive class num
IndexError: list index out of range

maybe you should check your num_classes in your config file

really thanks，but which file  you mean? I have modify four file under
`mmdetection/mmdet/datasets/voc.py
mmdetection/mmdet/core/evaluation/class_names.py
mmdetection/configs/faster_rcnn/faster_rcnn_r50_fpn_1x_coco.py
mmdetection/configs/base/models/faster_rcnn_r50_fpn.py`

@BIGWangYuDong
I'm really appreciate your  reply，but which file you mean(config file)? I have modify four file under
mmdetection/mmdet/datasets/voc.py
mmdetection/mmdet/core/evaluation/class_names.py
mmdetection/configs/faster_rcnn/faster_rcnn_r50_fpn_1x_coco.py
mmdetection/configs/base/models/faster_rcnn_r50_fpn.py

It is suggested to give us your code details, seems that your config file num_classes in bbox head is not equal to your labels:
https://github.com/open-mmlab/mmdetection/blob/83eea4bec684708171f0b6433636df0873e50dac/configs/_base_/models/faster_rcnn_r50_fpn.py#L47

Have you solved the problem yet？Thank you~


Crash

I changed the graphics settings, when I clicked ready I crashed minecraft
crash-2022-01-18_14.29.31-client.txt

You are outdate to start with.


feat(core): add evm feature

What this PR does / why we need it:
Which issue(s) does this PR fixes?:

Fixes #
Additional comments?:
Developer Checklist:

[ ] Read your code changes at least once
[ ] Tested on iOS/Android device (e.g, No crashes, library supported etc.)
[ ] No console errors on web
[ ] Tested on Light mode and Dark mode*
[ ] Your UI implementation visually matched the rendered design*
[ ] Unit tests*
[ ] Added e2e tests*
[ ] Added translations*

Codecov Report

Merging #4052 (4f922f1) into main (fb8a96d) will decrease coverage by 27.39%.
The diff coverage is 32.84%.

@@             Coverage Diff             @@
##             main    #4052       +/-   ##
===========================================
- Coverage   57.33%   29.94%   -27.39%
===========================================
  Files         421      438       +17
  Lines       11543    12305      +762
  Branches     3760     4063      +303
===========================================
- Hits         6618     3685     -2933
- Misses       4848     8573     +3725
+ Partials       77       47       -30

Files
Coverage Δ

mobile-app/app/api/persistence/domain_storage.ts
100.00% <100.00%> (ø)

mobile-app/app/components/BottomSheetHeader.tsx
100.00% <ø> (ø)

...-app/app/components/BottomSheetTokenListHeader.tsx
0.00% <ø> (-100.00%)
:arrow_down:

mobile-app/app/components/HeaderNetworkStatus.tsx
90.00% <100.00%> (-10.00%)
:arrow_down:

mobile-app/app/components/InputHelperText.tsx
50.00% <ø> (ø)

mobile-app/app/components/WalletTextInputV2.tsx
46.15% <ø> (-12.31%)
:arrow_down:

...le-app/app/components/icons/assets/ConvertIcon.tsx
100.00% <100.00%> (ø)

mobile-app/app/components/icons/assets/index.ts
100.00% <ø> (ø)

mobile-app/app/hooks/useAddressBook.ts
70.00% <100.00%> (-30.00%)
:arrow_down:

mobile-app/app/hooks/useAddressLabel.ts
85.71% <100.00%> (ø)

... and 74 more

... and 140 files with indirect coverage changes
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


spykingcircus example fails for me

Tried https://github.com/magland/spikeforest2/blob/master/examples/example_spykingcircus.sh example (while in a branch for #3).  FWIW I did not have docker image locally
(git)lena:~/proj/spikodrome/spikeforest2[bf-mandatory-recording]
$> sf-sort -a spykingcircus -r sha1://961f4a641af64dded4821610189f808f0192de4d/SYNTH_MEAREC_TETRODE/synth_mearec_tetrode_noise10_K10_C4/002_synth.json -o sorting_sc.json
===== Hither: running spykingcircus in container: docker://magland/sf-spykingcircus:0.8.6
unknown flag: --gpus
See 'docker run --help'.
Traceback (most recent call last):
  File "/home/yoh/proj/spikodrome/spikeforest2/venvs/dev3/bin/sf-sort", line 7, in <module>
    exec(compile(f.read(), __file__, 'exec'))
  File "/home/yoh/proj/spikodrome/spikeforest2/bin/sf-sort", line 44, in <module>
    main()
  File "/home/yoh/proj/spikodrome/spikeforest2/bin/sf-sort", line 26, in main
    sorting_path = sort(algorithm=algorithm, recording_path=recording_path)
  File "/home/yoh/proj/spikodrome/spikeforest2/bin/sf-sort", line 36, in sort
    result = sorter.run(recording=recording_path, sorting_out=hither.File())
  File "/home/yoh/proj/spikodrome/spikeforest2/hither/_core.py", line 180, in run
    local_modules=local_modules
  File "/home/yoh/proj/spikodrome/spikeforest2/hither/_run_function_in_container.py", line 188, in run_function_in_container
    raise Exception('Non-zero exit code ({}) running {} in container {}'.format(retcode, name, container))
Exception: Non-zero exit code (125) running spykingcircus in container docker://magland/sf-spykingcircus:0.8.6

BTW, spotted above, not sure if you intend to keep compatibility with some older pythons, but if >= 3.6 you might love to start using implicit .formatt()ing via f-strings -- makes code more readable and concise.

What version of docker do you have? The docs say >=19.
The issue is the --gpu flag. This is a bit mysterious to me. I had to install https://github.com/NVIDIA/nvidia-docker on my host system -- but it's a bit mysterious to me. This will need to be included in the installation docs.

oh, that is probably from docker itself (might be worth adding a check for the version.  I could introduce our helper from datalad so you could ask if external_versions['cmd:docker'] < '19':.  I have
$> docker --version
Docker version 18.09.9, build 039a7df

upgrade to 19.03.4+dfsg2-2

be sure not to miss my comment of 1 minute ago.

yeap... indeed -- would need that beast (nvidia-docker). Just FYI with regular 19.x docker I get
===== Hither: running spykingcircus in container: docker://magland/sf-spykingcircus:0.8.6
docker: Error response from daemon: could not select device driver "" with capabilities: [[gpu]].
Traceback (most recent call last):
...

that is why singularity in the longer run generally would be better IMHO for users on linux, although would also need more recent (iirc some 3.0.) version with --nv flag.
Which sorter is non-GPU based so I could try without reinstalling docker ATM?

(a) It already works with singularity. Just set HITHER_USE_SINGULARITY=TRUE environment variable. You need to have singularity >=3.3 I believe (I have 3.5). Yes, it uses the --nv flag.
(b) the only gpu sorters now are ks2 and irc. sc and ms4 do not use gpu. so spykingcircus would have worked. We just need to expose the gpu option in hither. Maybe I'll do that now.

I just pushed a version that only uses --gpus (--nv) flags for the gpu sorters. Therefore you my try either spykingcircus or mountainsort4 without needing to install nvidia-docker


[SPARK-46141][SQL] Change default for spark.sql.legacy.ctePrecedencePolicy to CORRECTED

What changes were proposed in this pull request?
Change default for spark.sql.legacy.ctePrecedencePolicy to CORRECTED
Why are the changes needed?
This change has been long planned. We have been running with EXCEPTION for years with an announcement that CORRECTED will eventually become the only option. To get there, changing the default to CORRECTED is the next step.
Does this PR introduce any user-facing change?
Yes, we now allow nested CTEs to share the same name by default.
How was this patch tested?
Run the existing QA: cte-nested.sql
Was this patch authored or co-authored using generative AI tooling?
No

thanks, merging to master!


Style  as a button

Django generate forms with submit buttons represented as <input type="submit" value="submit">. It's my understanding that this is the semantic way to represent the submit button for a form, so I expected mvp.css to style these like a <button>.

@MatthiasPortzel I think it makes sense and is an easy enough implementation, so I've added it here: https://github.com/andybrewer/mvp/releases/tag/v1.12
@ChristopherBilg It looks like regarding functionality, they're interchangeable: https://stackoverflow.com/questions/7117639/input-type-submit-vs-button-tag-are-they-interchangeable

Thanks for the update @andybrewer ! Sounds good!

Finally got back around to this. Looks great @andybrewer, thanks.


Allow running lighthouse tests on Edge and Opera

Allow running lighthouse tests on Edge and Opera as those are Chromium based browsers and supports lighthouse
Users run Playwrigth tests on Edge and Opera on LambdaTest and also in their local environment by adding a custom browser path(https://playwright.dev/docs/api/class-browsertype#browser-type-launch-server-option-executable-path).
Thus, we need to allow Edge and Opera to generate lighthouse reports.

@abhinaba-ghosh , can you review and release the updated version?

The changes should be available for v3.2.1


Apache Phoenix support in Flyway 5.x

I see that Flyway 5.0 dropped support for Phoenix. I guess this was due to lack of interest in a fairly weird, niche DB. Looking at past issues it seems that Phoenix provides the necessary bits and pieces, although the Phoenix support in Flyway 4.x doesn't handle schemas.

Yes - please add support back. I did not noticed it was removed when looking at the change logs, I must have missed it. We are running Flyway 4.x, and Phoenix.

Any idea when this will be coming back? We are running 4.x, and were holding off on upgrading so we don't lose Phoenix support. If this won't get added in 6 we may need to fork and build ourself, maybe grabbing https://github.com/flyway/flyway/pull/2113

Can you recommend a good Docker image to test this with tags for the various versions?

Sorry, just noticed this comment. This looks like a suitable for testing Phoenix.
https://hub.docker.com/r/boostport/hbase-phoenix-all-in-one/dockerfile

On investigation, we have hit a number of snags with reimplementing Phoenix support.

The phoenix jdbc driver currently requires protobuf 2.5. We rely on protobuf 3.6 due to other dependencies, and do not wish to downgrade them to support a 7 year old library.
Downgrading the protobuf version we rely on does allow us to bring in the phoenix-core dependency, and query the database. However as the library pulls in a huge number of dependencies we do not want this to be a default addition to flyway, instead being an optional driver. The phoenix-[version]-client driver should be the driver we need to include, but replacing the dependency added driver with the 'fat' one from the package does not work, failing on some internal reflection trying to create org.apache.phoenix.query.DefaultGuidePostsCacheFactory.
Each version of the jdbc driver currently only supports a single phoenix/hbase version (5.0 phoenix with 2.0 hbase for example). As we need to compile against a single api that supports all versions users may intend to use, this provides us with a problem. There has been no statement on the side of phoenix about the volatility of the api, and as such we cannot commit ourselves to 10 years of support.

Obviously pull requests adding this support are welcome, but until the problems above are resolved we will not be committing to support Apache Phoenix.


Stylesheets not associated for LEMDO-dev

It looks like our stylesheets for lemdo-dev aren't associated again. Can you please take a look into that?

That was fixed yesterday; see also issue #150. Closing.


Add node conditions info to Taints and Tolerations

Overview
Page to update: https://kubernetes.io/docs/concepts/scheduling-eviction/taint-and-toleration/#taint-nodes-by-condition
Blocked by #27739
Description
Based on this discussion thread in PR #27739, we should add the following information about node conditions to the Taint Nodes by Condition section of this page.
Proposed content
The API server also respects node conditions
when making scheduling decisions. For example, If the DiskPressure node condition is active, the system doesn't allow any new Pods to schedule on the node. If the MemoryPressure node condition is active on a node, the system doesn't allow new BestEffort pods on the node.
Questions
What is "the system"? The API server?
/cc @alculquicondor
/assign
/sig docs
/language en

"The system" is a mix of components:

the kube-controller-manager sets the taints via the node controller.
apiserver sets the toleration for DiskPressure for non-BestEffort pods.
scheduler checks if pods have tolerations for the taint

But that's too much implementation detail. "The system" or "the control plane" should be enough. WDYT @sftim ?

Ah, if it's the control plane then we can say that haha I thought it was something else :P

/triage accepted


ChatCompletionRequestMessage doesn't allow null content

Describe the bug
The docs indicate that null is an expected value for message "content":

However, the type definition for OpenAPI.ChatCompletionRequestMessage only allows string or undefined.
When passing message that utilize "function_call" to createChatCompletion(), I have to specify the message content as null. If I simply omit the "content" field or set it to undefined, I get a 400 "Bad Request" error.

To avoid this 400 error I specify the content as null, producing a type error:

Similarly, the type for OpenAI.ChatCompletionResponseMessage doesn't reflect the fact that a response's "content" can contain a null value.
To Reproduce
Specify a message such as:
{
  role: "assistant",
  content: null,
  function_call: {
    name: "...",
    arguments: "..."
  }
}

TypeScript Playground reproduction:
https://www.typescriptlang.org/play?#code/JYWwDg9gTgLgBAKjgQwM5wPJgKYDsCCAknAGZQQhwDkEOuywVcA9M3DAJ46oBccAzADohABgBQYgMYRcqeCGypUyAOaK+WPEUEBhABbIYOimAA22GMBkAlbAEcAropgBZRcrUBtALpwAvHA+EgpKqoqCYA6oegAUAN5icHDk5nwARGiowHLIuDBpADSJcNJ5eDB8uA6mpkVJJA64kpYyAPqSyDV8CUlJ9ArpIBytDU0tuIXFSchQKg4KebxwaYKracUAvmIbAJRAA
Code snippets
No response
OS
macOS
Node version
Node v18.16.0
Library version
openai v3.3.0

I believe this bug is fixed in our upcoming v4 beta; can you give that a try instead?


Allow loading files with content for better integration with language servers

A while back I opened #188, but I closed it because I thought that language server integration would be better if it was decoupled from elm-analyse itself, so I pared this down to only the features that really helped in integrating it, they are:

Allowing you to update file contents in elm-analyse without needing them written to disk.
Allowing file loading ports to accept file content along with the file path if available.
Reparsing on the fly as the contents are updated

I've been using it for a while through Vim, and it is fast enough to leave on insert mode and have it lint as I type.
Misc Fixes:

tsc was complaining about the return type in ts/util/cache.js

I added some support for workspaces with 0678a427b91ecf31c9fafd48215e9d897361c2b1, so that you can open a folder like /project in VSCode that has a /project/frontend and /project/backend component, the LocalCache class now stores the .elm-analyse cache directory relative to where the elm.json file is, previously it used the current working directory, which would cause it to create /project/elm-stuff/.elm-analyse.