Preserve TypeScript type coercions in Closure code

Input:
function foo(str: string) {}
foo(<string>JSON.parse('"foo"'));

Outputs:
// @param {string} str
function foo(str) { }
foo(JSON.parse('"foo"'));

We drop the <string> so Closure doesn't know about the type cast, producing a warning.

Due to https://github.com/Microsoft/TypeScript/issues/6982 , I can't find any way to make TS preserve an inline Closure type coercion.
Start from this:
http://www.typescriptlang.org/Playground#src=let x %3D %2F** foo *%2F(3)%3B
and see if you can find some combination of parens etc. to make it work... hmm.

@martine maybe we could contribute an upstream patch?
Evan Martin<EMAIL_ADDRESS>schrieb am Do., 18. Feb. 2016 um
00:27 Uhr:

Due to Microsoft/TypeScript#6982
https://github.com/Microsoft/TypeScript/issues/6982 , I can't find any
way to make TS preserve an inline Closure type coercion.
Start from this:
http://www.typescriptlang.org/Playground#src=let x %3D %2F** foo *%2F(3)%3B
and see if you can find some combination of parens etc. to make it work...
hmm.
—
Reply to this email directly or view it on GitHub
https://github.com/angular/sickle/issues/65#issuecomment-185458344.


3d Charts

I searched all issues and didn't find any references to 3D capabilities. My question is, has implementing 3d charts been considered?

Yes, would be nice to see 3D charts.
Google Charts offers an option like is3D: true and it becomes a 3D chart. HightCharts on the other hand provides a lot of options (which are complex to understand and configure).
A simple is3D: true would be the best option in my opinion.

I'm not sure what considerations have been made prior to v2.0-dev, but to the best of my knowledge, 3D charts are not a priority right now for future releases.  There is still much to be done with the breadth of existing chart types, new chart types, and the interactivity of those charts.  If 3D enters the pipeline, it won't be until much further down the road if at all.

Closing since we aren't going to implement this at this time. We'd welcome any extensions that made it possible.


add logging assertions to dropwizard-example IntegrationTest

Problem:

dropwizard-example logging should be robust
Solution:

add more logging assertions to IntegrationTest.java
Result:

All tests pass.

ready for review @joschi


twitter isn't twitter any more

Refs #9677
Tbh, we could just drop linking to this.
@PyvesB - are you still checking https://twitter.com/shields_io ? I think you're the only person who has access to it.

I'm going to just merge this for now, but we could remove in future.
It would still be nice to hear back from @PyvesB on this.

I've not actively monitored it in some time, but still have access to it.

Thanks. I've followed up in https://github.com/badges/shields/pull/9685


formula: make extract_macho_slice_from private

[x] Have you followed the guidelines in our Contributing document?
[x] Have you checked to ensure there aren't other open Pull Requests for the same change?
[x] Have you added an explanation of what your changes do and why you'd like us to include them?
[ ] Have you written new tests for your changes? Here's an example.

[x] Have you successfully run brew typecheck with your changes locally?
[ ] Have you successfully run brew tests with your changes locally?

Based on feedback from #11756.

Review period will end on 2021-08-19 at 03:53:14 UTC.

Review period ended.


refactor(select-mode): Refactor select mode action implementation

This PR resolves AMBER-491
Description
This significantly refactors how we manage select mode interactions in the app, due to a bug I noticed yesterday where the "select mode" button was still visible on the Shows screen, which has no items that one can select. Digging deeper, realized that we were injecting three versions of the buttons into the screen, but outside the normal react lifecycle via a "portal".
I must have been :zombie: when i wrote that, because that is... quite strange 🙄
So this updates things by moving select-mode active tab items into the proper location - global state - and then we simply render one single <SelectModeActions> component at the top of the react tree screen (ie, on ArtistTabs) and then update our global state whenever a given tab becomes active via
useSetSelectModeActiveTab({
  name: "ShowArtworks",
  items: presentedArtworks,
})

From there, we can render the appropriate select mode buttons and everything gets way simpler, while also unlocking some nice behaviors like animated enable / disable / hiding select all, etc.
https://github.com/artsy/energy/assets/236943/b76e955a-a0b4-498d-946a-03b80f8c7df5

Warnings

:warning:

It looks like you've modified a store model. If necessary, please be sure to add a migration to migrations.ts. See the migration-docs for more info.

  Generated by :no_entry_sign: dangerJS against a375018f2bd6cd864c92d9e4d6e88ca59344ac5d


Can't add transformation for collection mapping

Add Transformation link doesn't work for Repeating Mapping

Transformations enabled on arrays/ collections:

Generated XML:


AppTaskQueue end-to-end test suite postgres setup fix

The AppTaskQueue end-to-end test suite fails to configure postgres as the package is now installed in the image but the postgresql service is not running.
Other minor changes align this setup script with the third party postgres configuration script.
https://github.com/AppScale/appscale-thirdparties/blob/65905287ba21198c14a9d2c0f265b415d234a6c5/postgres/configure-and-start-postgres.sh#L76-L99

Daily build no. 6956

This is not needed now that #3177 is merged


feat(restock): Add a delete and view button to delete and see restock

create a 2 new button which:

one call the already existing delete method on RestockApi
the other create a table to see information about the restock

fix the getAllRestock because it was getting the deleted ones

lgtm 👍


Error while running the application on windows after install

Describe the bug
I have successfully installed it on docker as guided
To Reproduce
After installation i have created a user, and when i am trying to add a service, it is giving me this error -

Desktop (please complete the following information):
I am on windows 11

Could you run docker-compose logs and upload the file here?

I downloaded another zip and things worked. Now i have successfully logged in. I want to ask couple of questions regarding best practices, do i open another tread for that ?

It's better to create a discussion for this:
https://github.com/exadel-inc/CompreFace/discussions


Impossible to select the child elements of the table when filled with function

Version
2.6.11
Reproduction link
https://jsfiddle.net/lucasmsoares96/2x0mpLt7/1/
Steps to reproduce
open the browser console
What is expected?
the browser console is expected to show the childre elements of the selected table.
What is actually happening?
Child elements are not selected.

child elements are selected when the items attribute is defined within data(), but not when items are returned from a function

Hi, thanks for your interest but Github issues are for bug reports and feature requests only. You can ask questions on the forum, the Discord server or StackOverflow.

This is related to bootstrap vue. Also, you are accessing table[1] but table has only one element


Better support for self signed certificates

From<EMAIL_ADDRESS>on June 29, 2012 14:05:03
What steps will reproduce the problem? 1. test an ssl connection with a self signed certificate (no hostname specified) 2. 3. What is the expected output? What do you see instead? I would like to be able to do this but am prevented to because I cannot specify the default host name verifier What version of the product are you using? On what operating system? 1.6.2 (windows) Please provide any additional information below. I would like something like this to work. Many thanks!
    /*
     * The following code basically nullifies all SSL checks - this is not recommended to be copied without thought
     * of the consequences!! Sadly rest assured ignores all this so we may well have to ditch rest assured
     */

    TrustManager[] certs = new TrustManager[]
    { new X509TrustManager()
    {
        @Override
        public X509Certificate[] getAcceptedIssuers()
        {
            return null;
        }

        @Override
        public void checkServerTrusted(X509Certificate[] chain, String authType) throws CertificateException
        {
        }

        @Override
        public void checkClientTrusted(X509Certificate[] chain, String authType) throws CertificateException
        {
        }
    } };
    SSLContext ctx = null;
    try
    {
        ctx = SSLContext.getInstance("TLS");
        ctx.init(null, certs, new SecureRandom());
    }
    catch (java.security.GeneralSecurityException ex)
    {
    }

    if (ctx != null)
    {
        HttpsURLConnection.setDefaultSSLSocketFactory(ctx.getSocketFactory());
    }

    HttpsURLConnection.setDefaultHostnameVerifier(new HostnameVerifier()
    {
        public boolean verify(String hostname, SSLSession session)
        {
            return true;
        }
    });

Original issue: http://code.google.com/p/rest-assured/issues/detail?id=182

From<EMAIL_ADDRESS>on June 29, 2012 05:05:33
Actually this should be an enhancement - sorry

From<EMAIL_ADDRESS>on July 13, 2012 23:54:26
And you just execute this before you make a request with HTTP Client and it should work?
Status: Accepted
Labels: -Type-Defect Type-Enhancement

From<EMAIL_ADDRESS>on October 22, 2012 06:17:18
I just posted to the forum a HttpClient+Jetty sample that might help Johan implement this.  included in that post is a suggestion for how I think the API should work.  The forum post hasn't show up yet so I can't link it here.  I have however attached the sample.
Attachment: self-signed-certs.zip

From<EMAIL_ADDRESS>on December 06, 2013 06:20:10
Also see https://github.com/jayway/rest-assured/pull/22

From<EMAIL_ADDRESS>on December 07, 2013 09:42:25
Great! I've now merged the pull request.

From<EMAIL_ADDRESS>on December 07, 2013 11:03:35
I've actually modified the API (it's not backward compatible). You can now specify the host name verification check using "CertificateAuthSettings" that you may pass in to the "certificate" method. Please try this out and tell me if it works and if you like the API. Depend on version 2.0.2-SNAPSHOT after having added the following Maven repo:
<repositories>
<repository>
<id>sonatype
<url> https://oss.sonatype.org/content/repositories/snapshots/ <snapshots />
</repository>
</repositories>
Status: Fixed


Fill-in TODO for runtime table feature flag

Explain your changes:

While working on https://github.com/o1-labs/o1js/issues/1859, I found a TODO when computing the runtime table feature flag inside the verifier index in the WASM Kimchi bindings (which was set to false by default). Instead, it is now true if the selector for runtime tables exists.

Explain how you tested your changes:

Unfortunately, this is not sufficient to make https://github.com/o1-labs/o1js/pull/1858 be able to use runtime tables from o1js. In particular, tests show that it still fails when a table is configured (so the selector is set), but the feature flag interpreted from the circuit is still false.
Changing featureFlagsFromGates() to undefined or true is also not sufficient.

Checklist:

[ ] Dependency versions are unchanged

Notify Velocity team if dependencies must change in CI

[ ] Modified the current draft of release notes with details on what is completed or incomplete within this project
[ ] Document code purpose, how to use it

Mention expected invariants, implicit constraints

[ ] Tests were added for the new behavior

Document test purpose, significance of failures
Test names should reflect their purpose

[ ] All tests pass (CI will check this if you didn't)
[ ] Serialized types are in stable-versioned modules
[ ] Does this close issues? List them

Looks to me a relevant change.


docs: add Wildlife Studios to USERS.md

Signed-off-by: Guilherme Oki<EMAIL_ADDRESS>Please ensure your pull request adheres to the following guidelines:

[X] For first time contributors, read Submitting a pull request
[ ] All code is covered by unit and/or runtime tests where feasible.
[X] All commits contain a well written commit description including a title,
description and a Fixes: #XXX line if the commit addresses a particular
GitHub issue.
[X] All commits are signed off. See the section Developer’s Certificate of Origin
[X] Provide a title or release-note blurb suitable for the release notes.
[ ] Thanks for contributing!

Added Wildlife Studios to Cilium user list

This change is

Thanks!


Android: Can't access videos saved as .zip expansion file

I'm trying to access videos from an Android expansion file, which was uploaded in .zip format with no compression. Looking at the react-native-video source code, it seems like .zip expansion files are supported, but it's not working and I can't figure out why. Here's the basic code:
<Video source={{ uri: 'v1', mainVer: 4, patchVer: 0 }} />
(With 4 being the version number from the uploaded expansion file)
Any ideas why this isn't working? Is it the zip format?

Is your zip file in the right location on your Android device? If you step throigh the code in debug mode using Android Studio, bookmark here https://github.com/react-native-community/react-native-video/blob/master/android/src/main/java/com/brentvatne/react/ReactVideoView.java#L256 and check that expansionFile is not null.

I have the same problem. My zip file is in Android/obb/<My.Package>/. When I put a breakpoint on the line you pointed, expansionFile is null.


Scala 2.12 milestones support

Hi @lihaoyi
Would it be possible to publish cross-built versions so projects depending on utest can give Scala 2.12 milestones a try, please?
Regards

This should go out as part of https://github.com/lihaoyi/utest/issues/85


what do you suggest when you want to add an item on the page?

i just thought of how to inject a newly added item on the pagination, how?

In my opinion the best solution is to add new entity to the backend and then refresh whole ReactJS list (remove all cache and fetch page again). It helps to have consistend data on backend and frontend site.
If you use react-window library and need fetch from paginated api you can use simpler react-window-paginated


Add shade property next to the color property

[X] I have searched the issues of this repository and believe that this is not a duplicate.

Summary 💡

I believe that there should be a way to set a different shade than the main in any component that accepts color as a property.
Examples 🌈
I can think of two ways:

<AppBar color="secondary" shade="900">
<AppBar color="secondary" palette-type="dark">

In Flutter (which is a cross-platform framework) they do:
AppBar(backgroundColor: Colors.blueGrey.shade500)

I know that this syntax is less familiar in JS, but it might give you some other ideas.
Motivation 🔦
Currently, I need to make a new palette and set the default main to the desired color. I'm not sure if there's a better way of doing this. Adding shade/palette-type will make this library much more customizable without any additional modifications.

@oliviertassinari can you reference the issue you believe it's a duplicated from?

Sure, we link #13875 in the roadmap. I think that we should support color import path. e.g.
<AppBar color="secondary.900">

in your case.


Sub generators

In this PR:

Now uses composition for generating services and models
can generate services standalone

depending on the db used a model get generated automatically

can generate models standalone
can generate before and after hooks
database connection setup happens in services/index.js instead of the generated service

except for nedb because you create 1 file per resource.

various bug fixes

Still left to do:
I'd like to automatically import the generated hooks, models, and services. Right now that part has to be done manually.

Refs https://github.com/feathersjs/feathers/issues/159


Hackathon Submission: Corinna AI - MultiChatbot

Hackathon Submission: Corinna AI - MultiChatbot
Name of Team Lead
Akshay Waghmare
GitHub handles of Team Members

@akshayw1

Are all of your team members registered on our Hackathon Signup page?

[x] Yes
[ ] No

Rules and Code of Conduct
By submitting this PR, you agree to follow our Rules and Code of Conduct.

[x] Yes

Anything Else?
Looking forward to sharing this project and seeing the community's ideas!

Feel free to replace "Corinna AI" with your project title and add other team members as needed. Let me know if you'd like more customization.

The project does not use Appwrite, making it ineligible for the hackathon.


MNT: rerender

Hi! This is the friendly automated conda-forge-webservice.
I've started rerendering the recipe as instructed in #1.
If I find any needed changes to the recipe, I'll push them to this PR shortly. Thank you for waiting!
Here's a checklist to do before merging.

[ ] Bump the build number if needed

@conda-forge-admin please rerender

Hi! This is the friendly automated conda-forge-linting service.
I failed to even lint the recipe, probably because of a conda-smithy bug :cry:. This likely indicates a problem in your meta.yaml, though. To get a traceback to help figure out what's going on, install conda-smithy and run conda smithy recipe-lint --conda-forge . from the recipe directory.
This message was generated by GitHub actions workflow run https://github.com/conda-forge/conda-forge-webservices/actions/runs/11371138787.

Hi! This is the friendly automated conda-forge-webservice.
I tried to rerender for you but ran into some issues. Please check the output logs of the GitHub actions workflow below for more details. You can also ping conda-forge/core for further assistance or you can try rerendering locally.
The following suggestions might help debug any issues:

Is the recipe/{meta.yaml,recipe.yaml} file valid?
If there is a recipe/conda-build-config.yaml file in the feedstock make sure that it is compatible with the current global pinnnings.
Is the fork used for this PR on an organization or user GitHub account? Automated rerendering via the webservices admin bot only works for user GitHub accounts.

This message was generated by GitHub actions workflow run https://github.com/conda-forge/conda-forge-webservices/actions/runs/11371128239.

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

@conda-forge-admin please rerender


feat: show list of children using a ui recipe specified in the child

To show a list of objects a new generic UI plugin is needed.

Collapsible Object Table

Each object is shown as a row.
There is a config element to define what items to show as rows for each child on the banner
The ui-plugin inside each element is defined based on the "View" concept as defined in the Grid plugin.
The table has general functions such as, add, delete, and move (reorder)
(optional) Config allows for adding buttons, icon and call-back function, at the end of each row.
Default behaviour is to get the default view of each component.

Object Table
Same as above, without the collapsible part. Instead the rows include editable data fields.

There is a config element to define what items to to show on the columns
config has an option to be editable to read only.
The table has general functions such as, add, delete, and move (reorder)
(optional) Config allows for adding buttons, icon and call-back function, at the end of each row.

https://mui.com/material-ui/react-table/#collapsible-table


spelling: use_sarif is incompatible with only_check_changed_files

The option use_sarif is incompatible with only_check_changed_files.
This option is still quite new, see https://github.com/check-spelling/check-spelling/wiki/Feature:-Sarif-output

LGTM! why is the warning still there?

Not sure if it relates to how pull_request_target event works in this context or to something else :)


A question about the contribution of semantic data

Hi, thank you very much for your great work and for the code.
I have read your paper and I appreciated the level of information that you provided. However I still have a question about the encoding function that you have used for your descriptor when you didn't use semantic data (while calculating the contribution of individual components).
Can you please share the block representation used and the similarity score used in this case ?
Thank you very much !
Best regards

Hi, thank you very much for your great work and for the code.
I have read your paper and I appreciated the level of information that you provided. However I still have a question about the encoding function that you have used for your descriptor when you didn't use semantic data (while calculating the contribution of individual components).
Can you please share the block representation used and the similarity score used in this case ?
Thank you very much ! Best regards

Hi, you can refer to the ‘makeScancontext’ and ‘distanceBtnScanContext’ functions in the original Scan Context (paper, code).

@MohammedCHGHAF hi，I have a question to ask you.How to get semantic data,like 000720.label and 001500.label?


Milestone M3 Summary Document

Milestone M3 Summary Document
This document provides a summary of the completion of Milestone M3, including the acceptance criteria and evidence of milestone completion as required.
More details can be found here:
Project Catalyst Milestone
Details on Acceptance Criteria Fulfillment

Kupo database snapshots are automatically taken at least once per epoch for each public network (mainnet, preprod, and preview):
The latest two snapshots for each network are retained, at minimum:
Anyone can access key metadata about the database snapshots programmatically in JSON format (containing download URL and any other relevant information):

Snapshots are automatically created by the system for each public network, ensuring they are taken at least once per epoch and are available here

Metadata for each snapshot, including the download URL and other relevant details, is accessible programmatically in JSON format. This allows users to programmatically retrieve information about the snapshots.

Files downloaded from each URL are valid Kupo database snapshots:

The restore script successfully restores the downloaded snapshot (allowing Kupo to continue syncing from that point):

A restore script is provided that successfully restores the downloaded snapshot, enabling Kupo to continue syncing from the restored state without issues. The usage of the restore script is shown in the demo video

The portion of the source code related to creating the snapshots, as well as the restore script, are publicly accessible with an open-source licence:

Any documentation required for utilizing the caching service is made publicly available:

The source code for creating the snapshots and the restore script are available publicly under an open-source license. This promotes transparency and allows the community to contribute to and utilize the code.
Comprehensive documentation for utilizing the caching service is made available to the public, ensuring that users can easily understand and use the service in this repository

Evidence of Milestone Completion
The following evidence supports the completion of Milestone M3:

[x] Source code and documentation are available in this GitHub Repository.
[x] Test suite verifying acceptance criteria (excluding those not applicable to automated testing) is implemented (see below).
[x] Video demo explaining how to utilize the caching service: YouTube Demo
[x] Summary document that links to specific video timestamps, tests, and acceptance criteria, where applicable: Summary Document
[x] Social media post announcing the availability of the caching service: Twitter Announcement

Video Timestamps
The following sections can be found in the video demo:

Intro: 0:11
Cardanow Overview: 0:35
Kupo Restore Overview: 1:35
Restore Script Example: 3:57

Testing
Tests run automatically in CI when the Nix flake checks are evaluated. The infrastructure code is tested "by construction" as Nix would refuse to evaluate invalid configurations. Application logic tests can be found here

This was approved by Catalyst Team


Edge field makes required field nullable in database

Before adding edge field, the required field was defined as NOT NULL
However once I added a required edge field for this field (https://entgo.io/docs/schema-edges#edge-field) the field was changed to be of type NULL which is incorrect

[x] The issue is present in the latest release.
[x] I have searched the issues of this repository and believe that this is not a duplicate.

Current Behavior 😯
Define the following schema:
// Fields of the Post.
func (Post) Fields() []ent.Field {
    return []ent.Field{
        field.Int("author_id"),
    }
}

// Edges of the Post.
func (Post) Edges() []ent.Edge {
    return []ent.Edge{
        edge.To("author", User.Type).
            // Bind the "author_id" field to this edge.
            Field("author_id").
            Required(),
    }
}

Database migration code is:
CREATE TABLE IF NOT EXISTS "post"(
  "author_id" BIGINT NULL,
  PRIMARY KEY("id")
);

Expected Behavior 🤔
The database migration code should be:
CREATE TABLE IF NOT EXISTS "post"(
  "author_id" BIGINT NOT NULL,
  PRIMARY KEY("id")
);

Your Environment 🌎
I also ran the migration with WithForeignKeys(false)

Tech
Version

Go
1.16

Ent
0.8

Database
Postgres

Driver
https://github.com/jackc/pgx

Hey @naormatania 👋
Sorry for the delay, but the issue was resolved with #1703 - Thanks to github.com/atlas/ariga. That means if you're using MySQL and need to migrate existing columns, you should use the WithAtlas option.
cc @rotemtam

Hey @a8m !
Would it work if I'm using Postgres ?

Yes.

@a8m Are you planning to release a new version v0.10.1?


stream model must use 1-3second to recognize

use too many times to recognize. is there any way to reduce it?

@trrwilson

Hi @wangzhenxing, can you please provide more details about your scenario? Here are some questions that would be helpful:
How are you using the SDK?
What is behavior you are expecting?
What is the issue you are observing?

Closing this thread as it hasn't had any activity in a bit over a month. Please feel free to reopen if more help is needed.


Update Azure Coverity pipeline

/azp run openvino_contrib-coverity-lin-arm64

/azp run openvino_contrib-coverity-lin-arm64

/azp run openvino_contrib-coverity-lin-arm64

/azp run openvino_contrib-coverity-lin-arm64

/azp run openvino_contrib-coverity-lin-arm64

@azhogov it seems I've fixed the pipeline, could you please approve and merge fixes?


Enable support for different major versions of Camel

Right now the Camel plugin is tied to the single version of Camel model derived from a Camel 3.x version. But Camel 4 will be released soon so we'll need to support Camel 4 as well.
Having two different branches of hawtio-next for different versions of Camel is a bit awkward solution.  It should be better to enable support for multiple versions of Camel in a single @hawtio/react version.
One possible approach would be to have two editions of Camel plugin, i.e. Camel (current one) and Camel 4, and activating either of them depending on the version of Camel returned from a backend MBean.

cc @phantomjinx @lhein

@lhein FYI, here it's only talking about the frontend hawtio-next project. For the main hawtio project, we'll still need two editions (?) of Hawtio standalone to support both Camel 3 and 4.


Modernize design a bit

Just a few spacing modifications:

Let me know if this is acceptable ;)

Amazing! Do you have a screenshot of how it looks right now? Before/after kind of

Added!

Looking good! 👍

looks great!


Killercoda new issue link redirect correction

Environment

Killercoda

Affected Component

[ ] Approval
[ ] Bridge
[ ] CLI
[ ] Datastore
[X] Docs
[ ] Distributor / Event Broker
[ ] Helm
[ ] Istio
[ ] Jmeter
[ ] Kubernetes Integration
[ ] Openshift Integration
[ ] REST API
[ ] Webhook

Describe the bug
After completing the tutorial of keptn on killercoda the last page the link to "Create a issue on GitHub" redirects you to the GitHub homepage.
To Reproduce
Visit this link and follow the steps mentioned in the tutorial until the last section appears.
Link: https://killercoda.com/keptn/scenario/keptn-end-to-end-delivery
Expected behavior
It should redirect to the keptn repo of creating a new issue.
Maybe here: https://github.com/keptn/keptn/issues/new/choose
Screenshots

Additional context
Before working on this issue should get along with issue #9501
cc- @DavidPHirsch @bradmccoydev @agardnerIT

I would like to take this!!

Thanks @SuperAayush. The file you need to modify is here.
You can action this now and have it point to https://github.com/afzal442/keptn-pg-killerway/issues/new then when we transfer, we'll update the link. How does that sound?

Thanks @SuperAayush. The file you need to modify is here.
You can action this now and have it point to https://github.com/afzal442/keptn-pg-killerway/issues/new then when we transfer, we'll update the link. How does that sound?

Understood!!
Made a PR - https://github.com/afzal442/keptn-pg-killerway/pull/75

@agardnerIT should I close this issue as completed?

Yup, all done and completed. Thanks!!


Enables searching for @Provide(type=SET) injection points

This patch fixes fetching the injection points referenced by @Provides(type=SET), and also enables searching for the provider sources of type=SET for when the injected value is of type Set. For example:
@Inject SomeConstructor(Set<Long> longSet) { ... }

Will now be able to resolve to a definition like:
@Provide(type=SET) Long provideSetElement() { return 1L; }

And vice versa.
Details:

Adds a listener class to ShowUsagesAction that is called when the search result completes. The listener receives a boolean indicating whether results were found or not. This enables chaining additional actions that must be taken after one action completes.
Adds a CollectionElementParameterInjectDecider that adds an additional constraint to IsAProviderDecider, which is that the method element's @Provide annotation must include a type=SET provider. This class can be used to search only for SET providers.
Adds a boolean argument to PsiConsultantImpl#getReturnClassFromMethod that when set true, elements annotated with @Provides(type=SET) returns a class type of Set<ELEMENT_TYPE>.
Updates PsiConsultantImpl#getTypeParameters so that elements annotated with @Provides(type=SET) returns a list with just the method's return type.
Updates ConstructorInjectToProvidesHandler such that for @Provide with type annotation param set, first check for methods annotated with @Provide(type=SET) that match the search criteria. If search finishes with no results, then resort to original search.

Thanks!


Check length of current directory on Windows in install.hs

My suggestion is:

if we are on Windows
if length <$> System.Directory.getCurrentDirectory is above a threshold (currently some 160 chars isn't buildable)
create some long path and attempt to setCurrentDirectory there.
If it succeeds then we have long paths enabled.
if it fails print a warning and/or abort building

(a blind advice to enable long paths systemwide is not a good idea. Also note that for most of the users Haskell is a toy so the toy shouldn't break their systems used for more serious tasks)

Sounds like a nice idea in general. Isn't it possible to detect long paths via some system variable or something? Creating and cding there sounds very ad-hoc.

The setting is only read during boot, and only by certain builds of Windows 10 and Windows Server 2016/2019.
Also, it's actually a bug in GHC older than 8.6.1 which the Windows setting manages to work around, see https://downloads.haskell.org/~ghc/latest/docs/html/users_guide/win32-dlls.html#windows-file-paths
So by checking for a breakage using a broken GHC API we make it very robust against both these issues.
Oh, and my idea about cd actually seems wrong as SetCurrentDirectoryW keeps broken after long-paths setting is applied, so we need to check using another API which is fixed by long-paths (but I'm not sure).


Re-upload dataset to hub to avoid using script upload

This should solve issue #1311.
I re-uploaded the dataset to parquet format here: https://huggingface.co/datasets/mteb/mlsum as suggested by @lhoestq, to avoid using the script generation for loading.

Looks good, as long as you've tried out at least one model on the new dataset.

Yes I tested it locally with average_kominos model. Let's merge?

Great fix, @imenelydiaker!


Add config.php on default controllers to clear

Resubmission of #44 with conflicts solved.

Thanks!

Shouldn't this also be mentioned in README.md?


Add APIs to work with commit statuses

Closes #64.
This implements functions for all of the API endpoints that work with commit statuses. One caveat here is that the repo and owner for the commit is "deduced" from the URL of the commit, as this is faster and cheaper than requesting the owner when parsing the commit, or having the call site provide the repo.

There seems to be an inconsistency between the latest tagged image of ASH and what's on main. I assume this is due to the 1.21 changes, but the "combined" commit status is different between the two versions. I'm letting this go for now and will make the change when 1.21 becomes the default image.


Update to  RepoToolset 1.0.0-beta2-63103-01

Progress toward https://github.com/dotnet/arcade/issues/724.
Close to working; needs a couple of tweaks to the release that does ingestions to keep it working for 15.9 and 16.0 builds, but this can go in before that--it'll just require minor manual intervention when we want to insert.

NuGet made a breaking change were licenseUrl has been deprecated. We are working to have this fixed in arcade directly. For now, I would just add a NOWARN for that NuGet warning to unblock the PR.

That's a good idea, but it is unfortunately not the major problem with this PR at the moment; the update somehow horks task loading, leading to errors like
   24:6>/home/vsts/work/1/s/artifacts/.dotnet/2.1.500/sdk/2.1.500/Roslyn/Microsoft.Managed.Core.targets(90,5): error MSB4062: The "Microsoft.CodeAnalysis.BuildTasks.MapSourceRoots" task could not be loaded from the assembly /home/vsts/work/1/s/artifacts/.dotnet/2.1.500/sdk/2.1.500/Roslyn/Microsoft.Build.Tasks.CodeAnalysis.dll. Assembly with same name is already loaded Confirm that the <UsingTask> declaration is correct, that the assembly and all its dependencies are available, and that the task contains a public class that implements Microsoft.Build.Framework.ITask. [/home/vsts/work/1/s/src/Package/MSBuild.VSSetup/MSBuild.VSSetup.csproj]

when built from .NET Core MSBuild.

I had similar issues with this in the CLI. There is a property that you can set to tell arcade to use the roslyn assemblies from sdk or to use assemblies from the package. Here is the property: <UsingToolMicrosoftNetCompilers>false</UsingToolMicrosoftNetCompilers>.

Closing this one in favor of https://github.com/Microsoft/msbuild/pull/3993.


Feature request: Allow only module-based imports

I would request that black also modifies imports so that it does not allow any importing of symbols, but just Python modules. So if somebody writes code like:
from foo.bar import MyClass

MyClass()

It would be changed to:
from foo import bar

bar.MyClass()

Or:
from foo import MyClass

MyClass()

To:
import foo

foo.MyClass()

This is outside the scope of Black. Even if we did start doing AST-modifying changes, this is unlikely to be one of them. The change you're proposing would require us to analyze symbol tables to make sure the new variable name doesn't create conflicts. Also, I don't think there is general agreement that your proposed code style is better.

It is used in Google Python style: https://github.com/google/styleguide/blob/gh-pages/pyguide.md#22-imports

But that style guide allows from sound.effects import echo where echo is a module. How is Black supposed to know whether echo here is a module imported from the sound.effects package, or a function imported from the sound.effects module?

I would probably answer static analysis and you would answer that this is out of scope of Black? :-)


MAME scraper "unknown image format" when fetching .jpeg extension

I set up my hosts file to redirect mamedb.com to mamedb.blu-ferret.co.uk to be able to use the MAME scraper function.
(If anyone wants to know how to do this, add these lines to /etc/hosts on Linux and Mac systems or C:\System32\drivers\etc\hosts on Windows systems.)
<IP_ADDRESS>  mamedb.com
<IP_ADDRESS>  www.mamedb.com
Anyway, when it fetches an image with the .jpeg extension I get "unknown image format". This is not a problem with images with the .jpg extension.
Example of problem: http://mamedb.blu-ferret.co.uk/game/arabian/ with the marquee image (http://mamedb.blu-ferret.co.uk/marquees.small/arabian.jpeg).

The underlying issue is that the marquee image isn't on the server and is returning a 404. This wasn't an issue on the original site because it didn't display the link if there wasn't an image to back it up. I need to add better recovery when there is an error fetching the image.


Router executed multiple requests to same subgraph to resolve entities

Component(s)
router
Component version
0.130.0
wgc version
Just the router
controlplane version
Just the router
router version
0.130.0
What happened?
Description
While experimenting with the simple router example provided here, I noticed a missed opportunity for batching when resolving entities from a subgraph. Which is a blocker for us to adopt cosmo :/
Steps to Reproduce
After starting the router I executed the following request:
{
   employees {
    id
    products
  }
  employee(id: 1) {
    id
    products
  }
}

This query touches two subgraphs and created the following Request Trace

Expected Result
I was surprised by the two calls to the "product" subgraph and would have expected them to be combined into a single call. These objects are even from the same source; however, I believe this approach should also work for different types by simply using the following syntax.
query ($representations: [_Any!]!) {
  _entities(representations: $representations) {
    ... on Employee {
      __typename
      products
    }
   ... on OtherObjectFromProductsSubgraph {
      __typename
      someField
    }
  }
}

Actual Result
Group all your calls together and send a single request to each subgraph.

Hi @alexus37
Not sure, how it is blocking you, as other router implementations are doing the same
It is not a bug, currently, it is designed like that
It is not a trivial thing to implement - you have nested calls for elements of a list of items (which will be batched) and then you have a nested call on another root query field which is an object
The tricky part here is to be able to multiplex representations from different places and then demultiplex them back, you should take into consideration that for each separate item in a list, each representation could already be null, and you still need to filter them
We have some thoughts on how to implement it, but it is not a priority right now
And we are always open for PRs :)
Thanks,
Wundergraph Team

Hi @devsergiy,
thanks for the quick reply.

Not sure, how it is blocking you, as other router implementations are doing the same

I’d like to provide a bit more context to clarify why this is a blocker. We are currently exploring various options to enhance the capacity of our GraphQL API. Our current plan is to rewrite certain parts in Go and utilize federation to manage the components that are still on the old system as well as the transition period. This approach allows us to gradually improve capacity rather than implementing a large change all at once. However, if the call (to provided all the data that is not yet moved to the new go service) to the legacy API isn’t batched into a single request, we will end up using more capacity than before. This would make this approach unfeasible.

We have some thoughts on how to implement it, but it is not a priority right now
And we are always open for PRs :)

I would be willing to help get this done but I have no context of the code base. Would you mind sharing some context as well as links to important files or any thoughts you guys had?

@alexus37 could you explain how this blocks you? Or better yet, which alternatives are using the approach you described?

@alexus37 could let us know which alternatives you think are using the approach you described?

@alexus37 could let us know which alternatives you think are using the approach you described?

I haven't tested how the Apollo Federation router behaves in these situations yet. If it generates the same number of requests, we may need to come up with a new plan for the migration. :(

@alexus37 We didn't know the details of your current/feature schema

@devsergiy Let me share an example schema show casing the issue. Imagine you have the following schema.
# subgraph 1 - faster one
extend type Passagner @key(fields: "id") @shareable  {
  id: ID! @external
}

extend type Flight @key(fields: "id") @shareable  {
  id: ID! @external
  passangers: [Passanger!]!
}

extend type Airport @key(fields: "id") @shareable {
  id: ID!
  flights: [Flight!]!
  allPassagner: [Passagner!]!
}

type Query {
  airportV2(id: ID!): Airport
}

# subgraph 2 - slower one
type Passagner @key(fields: "id") @shareable {
  id: ID!
  name: String!
}

type Flight @key(fields: "id") @shareable {
  id: ID!
  name: String!
  passangers: [Passanger!]!
}

type Airport @key(fields: "id") @shareable {
  id: ID!
  name: String!
  flights: [Flight!]!
  allPassagner: [Passagner!]!
}

type Query {
  airport(id: ID!): Airport
}

and the following query:
query {
  airportV2(id: "1") {
    id
    name # not supported in subgraph 1
    flights {
      id
      name  # not supported in subgraph 1
      passangers {
        id
        name  # not supported in subgraph 1
      }
    }
    allPassagner {
      id
      name  # not supported in subgraph 1
    }
  }
}

This will created the following request execution graph
graph TD
    A[Start] --> B[Subgraph 1]
    B -- Query 1 (Entity) --> C[Subgraph 2]
    B -- Query 2 (Batched Entity) --> D[Subgraph 2]
    B -- Query 3 (Batched Entity) --> E[Subgraph 2]
    B -- Query 4 (Batched Entity) --> F[Subgraph 2]

While all these queries (1-4) are executed in parallel they require roughly ~4 (this is not fully correct since if they would be batched more work would be done) times more cpu as compared to being batched together.
Executed queries

query 1 ($representations: [_Any!]!) {
  _entities(representations: $representations) {
    ... on Airport {
      __typename
      name
    }
  }
}

query 2 ($representations: [_Any!]!) {
  _entities(representations: $representations) {
    ... on Flight {
      __typename
      name
    }
  }
}

query 3 ($representations: [_Any!]!) {
  _entities(representations: $representations) {
    ... on Passagner {
      __typename
      name
    }
  }
}

query 4 ($representations: [_Any!]!) {
  _entities(representations: $representations) {
    ... on Passagner {
      __typename
      name
    }
  }
}

We would like to have a single query made to subgraph 2 to fetch all these information.
graph TD
    A[Start] --> B[Subgraph 1]
    B -- Query 5 (Batched Entity) --> C[Subgraph 2]

the executed query could look like this:
query 5 ($representations: [_Any!]!) {
  _entities(representations: $representations) {
    ... on Airport {
      __typename
      name
    }
    ... on Flight {
      __typename
      name
    }
    ... on Passagner {
      __typename
      name
    }
  }
}

We believe that making one heavier request instead of four might be a bit slower but will save a significant amount of CPU cycles.
As for the feature, it might be possible to have this as a configuration option on the router, allowing it to run as an optional optimization after the query plan has been computed.

Hey @alexus37, this is a very interesting topic we've been thinking about for quite some time.
Are you available for a conversation to see how we can help you?
You can use this link to book: https://wundergraph.com/meet/jensneuse


Update C# bitwise Enums to be treated as bitfields

The C# example HowToUse_Dll.cs is treating the Status Enum as a flagged Enum. Although the example technically works, the Enums should be decorated with the Flags attribute to get the full support of flagged Enums in C#. Looks like the InfoFileOptions is a bitwise Enum as well, so decorating with the Flags attribute, too.

Thanks.


Unknown or unsupported HipChat API version: [v2 ]

Hi @hbakkum ,
@hbakkum I am trying to setup this plugin. But I am getting error v2 is unknows or not supported
when I specified following settings;
configure framework:
framework.plugin.Notification.HipChatNotification.apiVersion=v2
framework.plugin.Notification.HipChatNotification.apiAuthToken=value
configure project:
project.plugin.Notification.HipChatNotification.apiVersion=v2
project.plugin.Notification.HipChatNotification.apiAuthToken=value
Do you know why am I getting this?
Also on rundeck in hipchat section, you only specify room id and that's it right?
@hbakkum This is what I am getting:
class com.hbakkum.rundeck.plugins.hipchat.HipChatNotificationPluginException: Unknown or unsupported HipChat API version: [v2 ]
I am using new version of plugin 1.7.0.
Please let me know. Thanks
Regards,
Waqas.

Hey Waqas,
I'm thinking that you have a trailing space after the v2 in your rundeck properties file. I should probably be trimming this value. Can you check for a trailing space and remove it if its there.
Cheers,
Hayden

@hbakkum Thank you, I will check and test it.
Really appreciate your help. Thanks


Adding UnitTest tags dotnet version support

Motivation and Context
Unit tests are currently targeting net6.0 and the UTs will only run on machines with net6.0 SDK installed. That is, if a machine has only net7.0 or net8.0, installed, the UT projects will build, but the tests will not run.
Description
With this change, the UT projects self-identify as test projects with the tag:
<IsTestProject>true</IsTestProject>
...and we add the RollForward tag which ensures the UTs build for the highest installed version of the SDK:
<RollForward>LatestMajor</RollForward>
This ensures that the tests are runnable on any installation net6.0 or greater, including the new docker-based CI builds, each of which targets only a single SDK version.
Contribution Checklist

[X] The code builds clean without any errors or warnings
[X] The PR follows SK Contribution Guidelines (https://github.com/microsoft/semantic-kernel/blob/main/CONTRIBUTING.md)
[X] The code follows the .NET coding conventions (https://learn.microsoft.com/dotnet/csharp/fundamentals/coding-style/coding-conventions) verified with dotnet format
[X] All unit tests pass, and I have added new tests where possible
[X] I didn't break anyone :smile:

I'm guessing the UT failure is similar to the change that was required when we tried transitive pinning.
https://github.com/microsoft/semantic-kernel/pull/938/files#diff-69e3fabad2e4646e9bd7d79e0cc573d286fb811afce4bf9084c84a04e357e27d

I'm guessing the UT failure is similar to the change that was required when we tried transitive pinning.
https://github.com/microsoft/semantic-kernel/pull/938/files#diff-69e3fabad2e4646e9bd7d79e0cc573d286fb811afce4bf9084c84a04e357e27d

Yup, it's a dotnet 7 failure. I can repro on net7 build. I'll change to "draft" until I can fix it.


Provide option to disable auto screenshot taken for failed web test

1.20
Client
Due some issue in the test case selenium session got terminated. SeLion is trying to take the screenshot for the session already terminated in SeleniumGridListener.afterInvokation(), so user are getting different error than actual error. Need to have option to disable this auto screenshot taken feature.
May we can add SeLion config to decide this feature is required or not. By default make it enable this feature.

Hi, is this issue still unsolved? if not I would like to try

hi @michiboo I don't believe it has been resolved. here is the current code on the develop branch - https://github.com/paypal/SeLion/blob/develop/client/src/main/java/com/paypal/selion/internal/platform/grid/SeleniumGridListener.java#L319-L322


[TM] Add spec for DeviceInfo

Summary
Part of #24875. Causes a flow issue in Dimensions.js, trying to access DeviceInfo.Dimensions.
Changelog

[General] [Added] - Add TM spec for DeviceInfo
Test Plan

Flow tests pass.

@RSNara pinging you on this one, which should also be good to go!

Hey @ericlewis - it looks like there's merge conflict in this PR. It also turned out that we have a lot of internal callsites to fix for this module, so we ended up writing an internal PR for it. So I'm going to close this one to speed up the land process.
Thanks for working on this one!


Crash on 10.12.4

I am able to build the DMG add the application etc and running ike from the terminal seems ok (from far as I can tell). But when I launch the application I get the error below.  Unfortunately I am complete noob with AppKit and Python and I was unable to see a way around it in the scripts.
**Error Message
MacShrew (AppKit)
Subsystem: com.apple.AppKit Category : Layout
Detected missing constraints for .  It cannot be placed because there are not enough constraints to fully define the size and origin. Add the missing constraints, or set translatesAutoresizingMaskIntoConstraints=YES and constraints will be generated for you. If this view is laid out manually on macOS 10.12 and later, you may choose to not call [super layout] from your override. Set a breakpoint on DETECTED_MISSING_CONSTRAINTS to debug. This error will only be logged once

I will try in next days to update the package in order to work with newest version of Sierra + some enhancements. Sorry for the delay.

If you open ~/.macshrew/MacShrew.conf and set the verboselogging setting to off then start the app it will work (It did for me)
[UI]
profile =
verboselogging = off
[IKE]
ikedpath = /usr/local/sbin/iked
ikecpath = /usr/local/bin/ikec

Thanks it is now working. But i am unable to add or select profiles but that might be a RTF.  This can be closed i think.


breaking! make ellipsis configurable

I would like to ellipsis_char to be configurable. When the value is nil, then not show anything. Or define whatever you like.

LGTM, thanks!


How to train my dataset

Thank you for the excellent work and publicly available code.
I want to ask a quaetion.How to train my dataset, I did not find the data input port in the program, may I ask how to do it?

@bichongyuan Please follow the input pipeline.


[Feature]add support for be graceful exit

What type of PR is this：

[ ] bug
[x] feature
[ ] enhancement
[ ] refactor
[ ] others

Which issues of this PR fixes ：

Fixes #
Problem Summary(Required) ：

change graceful exit behaviour to:

BE tell FE it will exit soon
FE won't dispatch new fragments to BE exiting
BE loop wait all fragments done (maximum loop count is configurable), then stop servers and exit

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

[FE PR Coverage Check]
:disappointed: fail : 15 / 28 (53.57%)
file detail

path
covered_line
new_line
coverage
not_covered_line_detail

:large_blue_circle:
com/starrocks/system/ComputeNode.java
7
18
38.89%
[217, 218, 295, 296, 302, 341, 349, 350, 421, 434, 435]

:large_blue_circle:
com/starrocks/system/HeartbeatMgr.java
6
8
75.00%
[304, 311]

:large_blue_circle:
com/starrocks/system/BackendHbResponse.java
2
2
100.00%
[]

run starrocks_fe_unittest

[FE PR Coverage Check]
:disappointed: fail : 15 / 28 (53.57%)
file detail

path
covered_line
new_line
coverage
not_covered_line_detail

:large_blue_circle:
com/starrocks/system/ComputeNode.java
7
18
38.89%
[217, 218, 295, 296, 302, 341, 349, 350, 421, 434, 435]

:large_blue_circle:
com/starrocks/system/HeartbeatMgr.java
6
8
75.00%
[304, 311]

:large_blue_circle:
com/starrocks/system/BackendHbResponse.java
2
2
100.00%
[]

[FE PR Coverage Check]
:disappointed: fail : 15 / 28 (53.57%)
file detail

path
covered_line
new_line
coverage
not_covered_line_detail

:large_blue_circle:
com/starrocks/system/ComputeNode.java
7
18
38.89%
[217, 218, 295, 296, 302, 341, 349, 350, 421, 434, 435]

:large_blue_circle:
com/starrocks/system/HeartbeatMgr.java
6
8
75.00%
[304, 311]

:large_blue_circle:
com/starrocks/system/BackendHbResponse.java
2
2
100.00%
[]

run starrocks_admit_test


Fixed a couple issues with Bool and Time

Checked time string with regex to make sure that it contains a parseable string. Bool and Times can be passed as strings or Bool | Time.

released v0.5.9


Fix build on s390x

Checklist before submitting

[x] Did you read the contributor guide?
[ ] Did you update any affected documentation?
[ ] Did you write any tests to validate this change?

Description
Fixes # (issue).
Review process to land

All tests and other checks must succeed.
At least one maintainer must review and approve.
If any  maintainer requests changes, they must be addressed.

Closing this as it is not needed. The fix is added directly in bazel recipe to handle this error.


Epinio UI test - Testing Epinio upgrade on Rancher

We need to test Epinio upgrade through Rancher (Cypress).
EDIT: standby on this issue for now, as a new standalone UI should come in the next weeks. We will see if this issue is still pertinent after this release.

Maybe we need a test for epinio upgrades using cli first (helm upgrade)


Add fix

[x] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)
[x] My pull request targets the master branch of freeCodeCamp.
[x] None of my changes are plagiarized from another source without proper attribution.
[x] My article does not contain shortened URLs or affiliate links.

If your pull request closes a GitHub issue, replace the XXXXX below with the issue number.
Closes #XXXXX

Closed and Reopened this PR to attempt to resolve a specific Travis build failure.

Congrats @TIPOKA3HUK on your first pull request (PR)! 🎉
Thank you for your contribution to the page! 👍
We're happy to accept these changes, and look forward to future contributions. 📝


[BUG/QUESTION] weather_model._valid_date - should it be consistenly typed?

I am working on  #529. Just noticing that certain weather models have "Present" (or a string) as their end date for the _valid_range property, but certain models do not.
For example:
https://github.com/dbekaert/RAiDER/blob/dev/tools/RAiDER/models/hres.py#L50 - has "Present"
https://github.com/dbekaert/RAiDER/blob/dev/tools/RAiDER/models/era5.py#L27 - has the precise end-date
All models have _lag_time property (in the latter case this is used to compute the end date).
Just posting this in case it's worth making sure the second date in the tuple might be made to be similar to the ERA5 case. It can be closed if nothing will be changed.
Note: not even sure how the latency of SLCs available on ASF compare to the weather models and whether this is an issue.

@cmarshak just noting that as of #627 was merged all the end dates have been updated to be valid time zone-aware datetime objects, with datetime defaults given as well, so all typing is consistent now.


Watchmode issue with MacOS mojave

🐞 Bug report
Description
Kill the process with "ctrl+c" after press "r+intro"
🔬 Minimal Reproduction
💻Your Environment
Angular Version:

Scully Version:

🔥 Exception or Error

Closing as we found no way to reproduce this issue.


Pangram, String, Python

Is your feature request related to a problem? Please describe.
A string containing every letter of the alphabet is called a Panagram
Example: "  A quick brown fox jumps over the lazy dog " --> This string is a pangram
Describe the solution you'd like
I would like to add this program under string
Do you want to work on it
Pyhton

I would like to work on this issue under GSSOC'21.

I would like to implement pangram in c++

issue at #925


Bug in quickstart guide on Windows

Migrated from GitLab: https://gitlab.com/meltano/meltano/-/issues/722
Originally created by @keithemanuel7 on 2019-06-18 05:15:28

Hi, I saw a meltano post and wanted to try it out. I'm trying to get through the quickstart and ran into issues. I didn't see a duplicate on this, but I did see an issue about creating a Windows VM, so thought it might be a Windows specific issue. It might also be incomplete or outdated documentation.
What is the current bug behavior?
Following the tutorial here, I ran into an issue when trying to run
meltano elt tap-carbon-intensity target-sqlite

However, I'm not 100% sure the documentation is complete. Step 1-4 kinda just say 'do this' with no real details. Only the step 4 image really seems to show info. The output makes it seem like tag-carbon-intensity is missing configuration, but in the GUI it says it doesn't need configuration. Looks like there is also a sqlite issue. I have postgres installed on my machine, but didn't try it due to the tap error.
What is the expected correct behavior?
The tutorial ELT runs successfully.
Steps to reproduce
I tried both using Meltano on Windows and using the Ubuntu subsystem on Windows. Same error each time. Unless I'm missing something, you should be able to reproduce by following the installation and getting started guide on Windows.
Relevant logs and/or screenshots
This is from the run on Windows Subsystem for Linux (Ubuntu):
keith@KeithsThinkpad:~/first-project$ meltano elt tap-carbon-intensity target-sqlite
Running extract & load...
[MainThread][WARNING][2019-06-17 23:50:56,022] Could not find tap.properties.json in /home/keith/first-project/.meltano/extractors/tap-carbon-intensity/tap.properties.json, skipping.
[MainThread][WARNING][2019-06-17 23:50:56,023] Could not find state.json in /home/keith/first-project/.meltano/extractors/tap-carbon-intensity/state.json, skipping.
[MainThread][WARNING][2019-06-17 23:50:56,082] No state was found, complete import.
[MainThread][ERROR][2019-06-17 23:50:56,119] Failed to start plugin <Plugin name=target-sqlite type=loaders>.
[MainThread][ERROR][2019-06-17 23:50:56,120] Cannot start tap or target: [Errno 2] No such file or directory: '/home/keith/first-project/.meltano/loaders/target-sqlite/venv/bin/target-sqlite': '/home/keith/first-project/.meltano/loaders/target-sqlite/venv/bin/target-sqlite'
ELT could not complete, an error happened during the process.
Cannot start tap or target: [Errno 2] No such file or directory: '/home/keith/first-project/.meltano/loaders/target-sqlite/venv/bin/target-sqlite': '/home/keith/first-project/.meltano/loaders/target-sqlite/venv/bin/target-sqlite'
Aborted!

Possible fixes
No idea.
Further regression test
Ensure we automatically catch similar issues in the future

[ ] Write additional adequate test cases and submit test results
[ ] Test results should be reviewed by a person from the team

View 8 previous comments from the original issue on GitLab


return which work day it is.

I would be really happy if you'll add a function that returns which buisness day is entered.
it is crucial for Statistics.
For example 2019-01-10 will return the number 9.

@Evilu
nice idea, but can you please explain how 2019-01-10 should return 9? Why not 7? As 1st of January isn't a business day, you have 2,3,4,7,8,9,10

As 1st of January isn't a business day

Well, it all depends on locale settings…
Regarding the feature, if on non-business day like 2019-01-10 (which is a Sunday, if Sundays are non business days in the locale setting, of course) should it count 1 up or one 1 down?

1 down is basically saying the amount of business days before the date, 1 up is considering the day itself as an extra day… I might be wrong but the latter might be more consistent

Also, should it be like a total count of business days or per month?
f.e.:
January 10, 2019, is 9 days
What about June 10 2019, should it be counted from 1 of June or 1 of January?


Earn > Pools ux feedback

[ ] Can't sort by protected icon (sort only by name at the moment). Please fix

[ ] Please design tooltip with the dimensions from the design system
https://www.figma.com/file/37rEn9Up8BK377t7KFQZ91/?node-id=103%3A3

[ ] Please make sure chevrons are the same icon

closing duplicate


Vaulting through spaces you can't fit through

Essentially - you can reliably vault through spaces your character can't fit through as long as your character fits at the landing spot.
Example 1... Separated stairs with complex collision: from underneath, you can vault through them and appear on top.
Example 2... Barn with wood slats for roof. You cannot fall down through the slats because there's not enough space, but you can vault from below to the roof through the gaps.

Could you please provide me with mesh assets that I can test this on?

I have attached the bar with the open roof. When set to Use Complex Collision as Simple (I do this for most buildings for accuracy of geometry, maybe that's bad), you can vault through these areas from underneath, but not fall through them.
The issue is worse if you increase the capsule size of your character, as it doesn't seem to increase the vault size. This is probably a setting in the settings that I haven't messed with, but at the default capsule size this area can be vaulted out of, but you can't fall in to it.

https://drive.google.com/file/d/1VO6uFO_GSFKHy8jZWh6lyH1gbX7Dc34l/view?usp=sharing
(can't upload FBX here I guess so link above)

Can't reproduce this issue. Do you have the latest changes of this plugin from the repository? Maybe you can provide me with a video of this issue?
I've tried to mantle from inside the barn in the location shown in your screenshot, but no luck, while in the other, wider two locations, I can because there's actually enough room for the capsule.
Make sure that if you want to scale the character, you change the scale of the capsule component, not its size, in which case the mantling traces will be scaled accordingly.
Also in this case it's better not to use "Use Complex Collision as Simple", because as you can see, even character movement can't work with it correctly.

I have slightly increased the height that the player can actually vault which is likely why you can't reproduce in a default project, sorry about that, totally spaced it. Mine is about 30 units higher than the default settings in the plugin.
The other two spots on the barn you can indeed fall through at the default capsule size. Changing just the capsule diameter messes with the vaulting setup?

Should be fixed in 9302e047af0b28260b6e2b3f212faa5c393822e8.

Changing just the capsule diameter messes with the vaulting setup?

Yes, most all settings are configured with the absolute size of the capsule in mind.

Should be fixed in 9302e04.

Changing just the capsule diameter messes with the vaulting setup?

Yes, most all settings are configured with the absolute size of the capsule in mind.

Thank you for the information. I was unaware that the mantling mechanic didn’t use the capsule diameter itself. I have currently reverted back to using the default 30.


Use readline for command-line editing in chats

Just importing the standard Python readline module allows editing with GNU readline keys. This is not explicitly documented in click, but is implied by their Changelog.
On Windows, the readline module is not standard. This could be fixed by using pyreadline3 in that case, but that should be a separate change. This change ignores the failure to load readline on Windows.
The import ignores symbol unused error F401 from ruff, as that is by design. The readline module just needs importing to work.
No specific unit tests, as click doesn't process mocks in a way that uses readline.

@frabcus I've quickly reviewed the info about readline, seems that the one I'm missing for navigating and editing the text!
+1


Better Interactive Shell

Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]
Describe the solution you'd like
A clear and concise description of what you want to happen.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.

the default environemnt should be scene root(get_tree().root.get_last_child())？
to make life easier.

remove register env


Update docs with verified status

Closes #3895

@bigkevmcd @foot I've had to take new screenshots to show the verified status (I used demo-01). Some bits of the description are not as detailed as before so I'm wondering if we want to go back to that (I might need some help getting the sources/apps that we previously had in the screenshots on a local cluster).


UI Overhaul?

I'd like some input from users of TaskBoard on this one. I'm considering reworking the UI to remove Bootstrap and change the way I've done a few things.
The impetus behind this is to allow for a more flexible Dashboard/Overview page to include some of the many features requested so far (email, charts, etc.).
What do you think about reworking the UI, and do you have any recommendations if this happens?

You can still use Bootstrap and accomplish all this if you wanted to. If you were gonna switch the UI framework to something else. I like these two: Semantic-UI, and Material CSS. I think reworking the UI is a good idea though. Good luck!

Be prepared to have some people hating it and other loving it ;)
Seriously, I like the idea of an Dashboard for each user. However, even more I would like to see
LDAP and Email support ;)

Yeah, that's the danger of UI changes. I've been thinking about this, and I may have presented it poorly.
I think what I really want to do is front-end framework changes. I also want to do UI changes, but I'll probably make those ideas into a theme for #126 (I do want a Material-style theme).
And don't worry, LDAP and email are on the list of things to add!

Found something you might like since you use Angular. https://material.angularjs.org/#/

I was looking at that this morning!
I think I'm going to avoid any framework changes, and just use #126 as a way to provide optional looks for now. Thanks to everyone for your input!


Cannot create a resource using the resource type that contains a resource label selector

Describe the bug

Steps to reproduce

Create a resource definition with resource labels selector.

Create a resource using the resource definition added above.

Result

The newly created resource definition is not displayed.

Expected behavior

Display the type of the newly created resource definition when creating a resource.

Environment

Walrus version: v0.4.1-rc2

Additional context
Add any other context about the problem here.
It may be introduced by changes related to issue https://github.com/seal-io/walrus/issues/1580 , but resource labels selector cannot be listed out in advance.

Environment

Walrus version: main 0ce7516077eac143abadf67373426215f867d593
UI version: dev-90d701d

Test Result: pass


Fix more mypy full name checks

Continuation of #1077
I didn't realise we change more paths. This PR fixes all of them 😊

Thanks for adding the RELEASE.md file!

Here's a preview of the changelog:

This releases improves the MyPy plugin to be more forgiving of
settings like follow_imports = skip which would break the type checking.
This is a continuation of the previous release and fixes for type checking issues.


kW to BTU/HR

BTU/min is not standard for the construction industry.  It's BTU/Hr.  I want to add BTU/Hr conversion.

This is your friendly Microsoft Issue Bot. I've seen this issue come in and have gone to tell a human about it.

Thanks for submitting your idea. This sounds great! I think I understand your idea, but there might be a couple smaller details to iron out. For improvements like these, we like to follow the feature pitch template from our New Feedback Process. Can you take a look and update the details of your idea?

@Tak-Au When you get the chance, can you update your idea with more details following the feature pitch template from our New Feedback Process?

Problem Statement
Gas appliances have a BTU/hr rating. Electric appliances have a kW rating. Smart meters will give you your current gas usage in BTU/hr but your current electric usage in kW. In order to compare the two power units, they will look to a converter, but Win10calc only has BTU/min. Adding this feature will help people understand their energy usage and empower them to reduce their carbon footprint.
Evidence or User Insights
Google search results for the following searches:
"BTU/m" 62000
"BTU/min" 70000
"BTU/h" 4760000
"BTU/hr" 3930000
Proposal
Add British Thermal Units per hour (BTU/h) to list of power conversions, equal to 60 BTU/min.
Goals
User can convert power units to and from BTU/h
Requires translation to other languages, perhaps as simple as copying the translations of "British Thermal Units" and "per hour" from "kilowatts per hour".
Non-Goals
We are not getting rid of BTU/min, as it seems 2% of websites use this.
We are not imposing the superior metric system on the US and Imperial measurement world.
Low-Fidelity Concept
Requested Assignment
I'm just suggesting this idea.  I don't want to implement it.

Thanks, @I-Campbell!

I note that EIA uses the formula 1 BTU = 1 055.055 852 62J
https://www.eia.gov/totalenergy/data/monthly/pdf/sec13_17.pdf
Whereas the codebase uses 1055.056
https://github.com/microsoft/calculator/blob/9f01c8168b921a82ada80be24203de7729b1b6fa/src/CalcViewModel/DataLoaders/UnitConverterDataLoader.cpp#L831
Do we care that we're 0.15ppm out in this case? Do any other jurisdictions use 1055.056? It kind of makes gas look slightly more attractive than it is, which is bad if we want to reduce global carbon emissions.

It appears 1055.056 comes from ISO 31-4 on Quantities and units—Part 4: Heat. While the EIA measurement has higher accuracy, I think we probably want to stick to ISO standards whenever possible.

#515 aims to cover gaps in conversion units more broadly. Let's continue the conversation there, including what to do in the case here with power conversions.


Feature request: Arranging images into galleries

While the list of images grows it's getting inconvenient to browse. So let's arrange the images into galleries! That way users will be able to:

filter image list by gallery
combined with page images engine images can be added by given gallery

What do you think about my idea?
If you accepted it I volunteer for its implementation.

OK, I'll contact him.

@mirosr, I am close to completing a task (end of this week) that includes adding tags, captions, name, alt & title, and a searchable description to images, and it includes a tag switcher in the "Insert Image" dialogue. We will be putting in our pull request soon. Thanks!

Wow! That's great news, Joe! As soon the image galleries get in the repository I'll begin playing with them. :)


[mesheryctl] Fix Registry usage error message link

Notes for Reviewers
This PR fixes #10502
Signed commits

[x] Yes, I signed my commits.

Yay, @tico88612! 🎈


IMBA 7.00 Bug Compilation Issue

Feel free to post anything here if you don't want to make a separate issue or something.
Silencer - this one's mine

[ ] Some sound stuff that's played at the middle of the map;
[x] Ulti blows speakers, as I feared - use playsoundforclient; (FIXED)

Lion:

Mana Drain can be targeted at creeps without mana
* Finger of Death Frog Panic ignores cast duration. This means there is no chance to avoid it for the targeted enemy.
* Mana drain particles are yuuuggee. They are also kinda weird. Not a fan.

Ursa's and Lycan's innate passives are not in the list of innates skills.

Slardar: Seems to gain and lose an ability any time he gets an item.
Also his Raincloud ability is only mentioned in what it does, not how to get it -- aghs is blank.

Abbadon
Ability layout of 4 skills while he has 5.
Mist Coil does not show damage.
Curse of Avernus does not slow the target nor increase allies' move and attack speed (at least not in the UI, though the target didn't look like it was slowed at all)

Axe
Enemies affected by Berserker's Call do not show bonus attack speed.
Battle Hunger's debuff ends immediately (instead of fading out) which looks bad visually.
Battle Hunter doesn't show base damage or % of maximum HP as damage.
Battle Hunger doesn't describe the amount of units needed to kill to dispel (just says that lasts until the target kills another unit), nor shows the amount of units needed to kill to dispel.
Counter Helix shows only the base damage, but not how much of Axe strength's is calculated in it.
Culling Blade shows "speed" (probably movement speed) as a constant, not percentage.
Axe doesn't heal from Culling Blade as described in the Trello or the infodump. Instead, it seems to add % of Axe's maximum health to the damage, which is not intended.
Either way, Culling Blade's % of Maximum Health is not shown in the description.
Culling Blade doesn't show duration, neither before or after purcashing Aghnaim Scepter.
Axe doesn't have talents of his own: he'll show the talent of the last hero whose talents were hovered on.

Centaur
Has an ability layout of 4 skills even though he has 5.
Return's stacking damage block does not work.
Return works on towers.
Centaur doesn't have talents of his own: he'll show the talents of the last hero whose talents were hovered on.

Lycan
Doesn't get his innate.
(Probably) since Lycan doesn't have his innate, the cooldown can't be calculated properly. Either way, we can probably get rid of nettables calculations by using GetModifierStackCount() instead of setting it.
When using Shapeshift, an error pops because there is no IsServer() block in function modifier_imba_shapeshift_transform:OnCreated() function (line 980).

**Skadi **
Purcashing a skadi pops errors.

Magnus
Mid-Skewer activation should probably have a No Target behavior as it's practically possible to click anywhere to achieve the knockback effect.
Skewer does not "heaves" polarized enemies, which means that it ends prematurely and feels weird, especially if there's at least one enemy that was not polarized: that enemy would be heaved, while the other one stays at the location. They should always be heaved, and if Polarized, should be all heaved towards the exact same spot.
Using Manta while having Aghnaim Scepter "purges" the passive Empower on Magnus, causing the scepter duration to tick. When the duration ends, it stays there forever, but without the green circle. Casting Empower on self resets the process. The effect never ends, just stays there forever like this:

This doesn't happen if you have no scepter, by the way. Having only Manta doesn't cause that bug.
It also pops errors in the console, line 392.

Night Stalker
Still has Void icon for his innate (we really gotta find something cool to replace that).

Slardar
Doesn't explain what is Rain Cloud or how you get it. Should probably add a "Upgradeable by Aghnaim Scepter", with the aghnaim scepter flavor showing up when holding a scepter.
Towers can be bashed with Bash of the Deep.

Anti Mage
Descriptions are not updated.

Curseblade, Hellblade
DISPELLABLE_DEBUFF_LIST is not loaded anywhere (that I could find), which makes the actives fail completely.
Hellblade's file is missing, game's trying to route it into curseblade's file (?).

Desolator
Active particle is probable not being precached (I don't see a precache block), so it is invisible. Still deals damage and applies Desolator as intended

Bloodseeker
HP bonus indicator from Blood Rite has a weird placement above Bloodseeker - a bit too high, and it sometimes move to the right or to the left of the caster.
Thirst: Seems like the on-damage bonus only counts when the target is below 75% health. Is it intended?
Has no scepter upgrade: we wanted to do a scepter that pushes targets away from Blood Rite with Rupture on them, but we didn't in the end.
I've got some error after killing a hero just as he got hit by Blood Rite, but I can't repeat it.
stack traceback:
    ...465778740.vpk:scripts\vscripts\hero\hero_bloodseeker.lua:301: in function <...465778740.vpk:scripts\vscripts\hero\hero_bloodseeker.lua:285>
[VScript] Script Runtime Error: ...465778740.vpk:scripts\vscripts\hero\hero_bloodseeker.lua:301: attempt to compare nil with number
stack traceback:
    ...465778740.vpk:scripts\vscripts\hero\hero_bloodseeker.lua:301: in function <...465778740.vpk:scripts\vscripts\hero\hero_bloodseeker.lua:285>
[VScript] Script Runtime Error: ...465778740.vpk:scripts\vscripts\hero\hero_bloodseeker.lua:301: attempt to compare nil with number
stack traceback:
    ...465778740.vpk:scripts\vscripts\hero\hero_bloodseeker.lua:301: in function <...465778740.vpk:scripts\vscripts\hero\hero_bloodseeker.lua:285>```

it seems to be very rare and needs extremely precise timing, so I think it's OK.

Bounty Hunter
HeadHunter is not innately assigned.
Ability layout is for 4 skills even though he has 5.

Drow Ranger
Doesn't get her innate assigned at game start.
Frost Arrows animation is replayed over and over when spamming the unit target spell. This is fixed by adding Immediate behavior to the skill.

Faceless Void
Descriptions are not updated, and talents are not named.
Time Lock has 100% proc at level 1.
Faceless Void has no animation for Chronosphere.
While in Chronosphere, a weird animated copy of himself stays at the cast location. This only happens when Faceless Void has Chronocharges when casting.
Chronosphere doesn't seem to interrupt Forced Movement of enemies, allowing Force Staff/Hurrican Pike to move units out of the Chronosphere. (wiki says it shouldn't happen)
Chronosphere doesn't seem to provide Flying vision in its area of effect.

Juggernaut
When using Blade Fury while Omnislashing, and Omnislash ends, Juggernaut doesn't keep the Blade Fury animation: the particle stays but he restarts his animations normally.
When Juggernaut gets his Aghnaim Scepter, Omnislash doesn't show how many slashes do you actually get (need to look at base value and add +3 to it, hard math for most players)

Mirana
Sacred Arrow's first damage instance doesn't crit (that code should be on the damagefilter and it isn't)
Leap doesn't get extra leap distance at night, both server and client sides.
Moonlight Shadow doesn't have a cast animation.

Riki
All skills have 4/3/2/1 (ult has 3/2/1) cooldown.
All skills have 4 levels (ult has 3).
Riki's Blink Strike has only 2 damage values: 200 and 555.
Riki's Blink Strike will not do anything if the target is out of range: might be annoying sometimes if you just want to make Riki run towards it and jump to it, but instead, Riki stays stationary and does nothing.
Instead of showing a custom error, it might be better to just make Riki move towards it and try to jump to it normally, if no jump targets were found.
Riki's Blink Strike doesn't seem to slow turn speed of enemies: if it does work properly, then it definitely needs to be increase, as it takes almost no time to rotate yourself to face Riki.
Cloak and Dagger immediately apply when leveling up the skill instead of having a fade out time.
Riki has no custom talents in the talent tree.

Ursa
Doesn't get his innate skill.
Fury Swipes work fully on towers.

Bane
Has an ability layout of 4 while having 5 skills.
Seems like skill descriptions aren't updated at all, while talents are actually updated. "new" modifiers have no description at all.
Enfeeble's attribute reduction in descriptions should be with positives (it's a positive reduction..) instead of negatives.

Dazzle
Poison Touch: Description shows %movement_speed%% instead of a value.
Shadow Wave: Bonus Armor doesn't seem to stay for 3 seconds like it's stated.. more like 1.5 seconds.
Shadow Wave: Doesn't state what is counted as "injured" for armor increase purposes.
Weave: I think the stack count is unneccesary, but that's my own opinon, not really a bug.

Disruptor
Disruptor's innate is not assigned at game start.
Ability layout of 4 skills while he has 5.
Static Storm: CAPS IN THE SKILL DESCRIPTION (the word "BY").

Jakiro
Fire Breath and Ice Breath: Description states -10% move slow - I prefer the values to be positives and not negatives.
Fire Breath and Ice Breath: Jakiro literally can keep going and going and going forever until reaching the target location, without ever stopping.
Fire Breath: There is no visual effect for a target being hit by Fire Breath. Likewise for Ice Breath.
Fire Breath and Ice Breath: it seems that Jakiro will break trees much ahead of him, instead of strictly around his flying location
Ice Path: Targets do not have a particle of being frozen or for being slowed by the path's aftereffects.
Liquid Fire: Uses normal attack particle (not Liquid Fire's).
Macrypyre's alt description "jakiro buffs increases..." - should be changed to "jakiro buff durations increases..."

Lion
Hex's secret Firetoad model doesn't go up in flames - throws "xxxx" insteaded.
Hex's secret Firetoad sound is not loaded in the soundevent.
Lion's Mana Drain Aura particle is invisible (probably not precached).
Lion's Mana Drain particles are huge. This is probably because the "laser" goes to the enemy's feet, while the rest of the drain particle goes to the enemy's hitloc.
Lion's Mana Drain particles do not follow allies (but follow enemies), and it goes purely to its feet instead to its hitloc.
Finger of Death: Frog Panic turns the main target to a frog immediately, removing muh skill cup dodge powa.

Necrophos
Heartstopper Aura: I still insist that the aura particle is too bright and is kinda invisible, and probably should use "stronger" green colors.
Sadist: Doesn't show the duration in the buff (so you can't really know when it's going to end)
Reaper's Scythe: Sick enemies have no particle on them to show the sickness: probably should have a status effect at least? maybe light green, to show nausea?

IAS cap on ALL heroes seems to be uncapped (at least on Abaddon for sure)

@Saniok88 intended.
Angelic Alliance
Can be cast on self (shouldn't be possible)

Tutela Plate
Doesn't state how much damage block it has. (not Greater Parry's damage reduction)
Same for Crimson.

Clinkz
Missing some Searing Arrows special particles.

Morbid Mask, MoM, Satanic, Vladimir
Special attack particle missing, resulting in absolutely no attack particle for ranged heroes.
Vladmir only: Doesn't heal the attacker, but heals others affected by the aura.

Omniknight
When Omniknight heals a target with the double purification talent, it seems to give the target 2 stacks of amped heal instead of just one.

Shadow Fiend
PR is up, but I couldn't make it show the Scepter's effect in it, for reasons unknown.

Sniper
Passive range increase doesn't update (refresh) when leveling the ability.
Scepter Assassination doesn't target creeps.

Whew. Almost all bugs are fixed, except for Faceless Void (no talents and old description), Riki, and Bane.

Closing bug compilation. Remaining bugs are few and are listed on reddit.


resample_poly returns float64 output for float32 input

The resample_poly() function returns float64 output even when both the input array and the window array are float32.
The underlying call to upfirdn, on the other hand, does return float32 output for float32 inputs.
If we replace the following line in resample_poly()
h = np.concatenate((np.zeros(n_pre_pad), h, np.zeros(n_post_pad)))

with
h = np.concatenate((np.zeros(n_pre_pad, dtype=h.dtype), h, np.zeros(n_post_pad, dtype=h.dtype)))

then resample_poly() will return float32 output for float32 inputs.
Reproducing code example:
import numpy as np
from scipy import signal

x = np.arange(10, dtype=np.float32)
h = np.array([1,1,1], dtype=np.float32)
y1 = signal.resample_poly(x, 1, 2, window=h)
y2 = signal.upfirdn(h, x, 1, 2)
print(y1.dtype, y2.dtype)     # float64 float32

Scipy/Numpy/Python version information:
1.0.1 1.13.3 sys.version_info(major=3, minor=6, micro=5, releaselevel='final', serial=0)

This behaviour is also present in the master branch:
https://github.com/scipy/scipy/blob/master/scipy/signal/signaltools.py#L2376

That sounds like a good fix, can you make a PR?


Test for the 3211 printer fails with ooREXX 4.2.0 (Windows 11, VS 2019 Community Edition) but not with ooREXX 5.0.0

The two tests for the 3211 printer fail with both Aethra and SDL-Hyperion when using the REXX included with "hercules-helper-windows" (on Windows 11 with the Visual Studio 2019 Community Edition); all the other tests pass.  On the other hand, all the tests pass when using "hercules-helper" on WSL Ubuntu 22.04.4.  Given that all the tests pass on WSL, but not Windows 11, I guessed that the problem might of been with ooREXX.  The version of ooREXX included with "windows-helper-windows" is 4.2.0, but according to the ooREXX web site (https://www.oorexx.org/) the current GA version is 5.0.0.  Consequently, I uninstalled ooREXX 4.2.0 and replaced it with 5.0.0.  With this change, all the tests pass with both Aethra and SDL-Hyperion.
Incidentally, version ooREXX 5.1.0 Beta appeared less than a day ago (as of 28 Aug 2024).

Daniel,
I will check that out for myself and update the ooREXX installer that I included.
Thanks, Bill

Thank you, Bill--for both "hercules-helper" and "hercules-helper-windows"!
The two 3211 test failures may have been due to something hinky about my Windows 11 (Version 6.3.26100) and/or Visual Studio 2019 Community.  I can't believe that this test failure wouldn't have been noticed by someone else.  I have observed the problem for about six months.
Let me know if you want the various log files; I saved them for both the most recent failure (28 Aug 2024) and the subsequent success after installing ooREXX 5.0.0 (also 28 Aug 2024).

OK, I switched it to ooRexx 5.0.0 and it works for me, too.
(Only tested on Windows 11, though.  Not Windows 10 yet)
Changes have been committed.

You're welcome, but did the 3211 tests fail for you before the change from ooREXX 4.2.2 to 5.0.0?

I foolishly didn't check that on Windows 11.  Made the change.  It worked.
Made the change in Windows 10.  It worked.
By the time you asked if it failed for me, I didn't feel like unzipping VM backups to retest it with 4.2.
But, I did just fire up a Windows 7, installed ooRexx 4.2, and it works.
Snippet from the test output...
Variable $cmpxchg8           set to "1"
Variable $cmpxchg16          set to "1"
Variable $hatomics           set to "msvcIntrinsics"
Variable $rexx_package       set to "OORexx"
Variable $rexx_VERSION       set to "REXX-ooRexx_4.2.0(MT)_64-bit 6.04 22 Feb 2014"
Variable $rexx_SOURCE        set to "WindowsNT"
Variable $rexx_Mode          set to "Command"
Variable $rexx_MsgLevel      set to "0"
Variable $rexx_MsgPrefix     set to "0"
Variable $rexx_ErrPrefix     set to "0"
Variable $rexx_Resolver      set to "1"
Variable $rexx_SysPath       set to "1"
Variable $rexx_RexxPath      set to ""
Variable $rexx_Extensions    set to ".REXX;.rexx;.REX;.rex;.CMD;.cmd;.RX;.rx"
Test "1403 printer Load UCS Buffer":  2 OK compares.  All pass.
Test "3211 printer (OORexx)":  2 OK compares.  All pass.
Test "agf 20161211 12.44":  10 OK compares.  All pass.
Test "S/370 AWS tape BSF into Load Point":  5 OK compares.  All pass.
Test "BC-mode ILC Handling":  4 OK compares.  All pass.
Test "bfp-001-divtoint":  1 OK compares.  All pass.


WIP: Serialize image loads to kind

This seemed to fix issues publishing multiple images to kind, which were cleared up when image loads were serialized.

I'm not seeing the issue anymore after deleting and recreating my KinD cluster. This is weird. Keeping this PR open in WIP/draft mode to collect further data.

Another workaround may be to use set --jobs 1 which should overcome this?


Release v0.12

Release the compatibility version for AngleSharp 0.12.

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.0 out of 2 committers have signed the CLA.:x: FlorianRappl:x: georgiosdYou have signed the CLA already but the status is still pending? Let us recheck it.


Some words/phrases not in Translatewiki

From Jorn Ari:
" have translated the INat to Danish language and I have found several words in INat which is not to find in Translatewiki.
https://www.inaturalist.org/projects/nogensnegle-i-danmark-nudibranchs-of-denmark
English = Danish translation

Preferred membership model = Foretrukket medlemskabsmodel
Preferred submission model =  Foretrukket indberetningsmodel
Coordinates shareable by project curators = koordinater må deles af kuratorer
4.Back to observation field = Tilbage til observationsfelt
About Editing Observation Fields = Om redigering af observationsfelter
Påkrævet felts = påkrævet felter
title = titel
created = oprettet
Email = E-mail
10.Name = Navn
Time zone = Tidszone
Locale = Lokal
Search place = Søgested
Blocked Users = Blokerede brugere
About Blocking = Om blokering
Muted Users = Slukkede brugere
About Muting = Om slukning
Place = Sted
Show Scientific Name First = Vis videnskabeligt navn først
Preferred observation fields by = Foretrukne observationsfelter af
Only You = Kun dig"

And a few more:
"When in a Guide, printing a Journal, some of the words is not translated
https://www.inaturalist.org/guides/6816.journal.pdf?print=t
English = Danish translation
Weather = Vejr
Notes = Noter
Drawings = Skitser"


Facing os path error in windows 10

Traceback (most recent call last):
File "F:\Courses\NeoUdeler\main.py", line 58, in
main()
File "F:\Courses\NeoUdeler\main.py", line 54, in main
course.download_all_contents('.')
File "F:\Courses\NeoUdeler\neoudeler.py", line 364, in download_all_contents
os.mkdir(save_dir)
OSError: [WinError 123] The filename, directory name, or volume label syntax is incorrect: '.\20230812_Tailwind CSS 3 | Learn By Building Multiple Projects A-Z - Kazi Ariyan'

@sabitahmad
Thank you for the issue report.
By default, NeoUdeler sets the course title as the name of the directory where the videos are saved. However, in the Windows operating system, special characters such as <, >, :, ", /, , |, ?, * cannot be used, leading to the error. To circumvent this issue, I've made changes to replace any disallowed characters in the directory name with _.
I'm a MacOS user, so this issue was an oversight on my part.

All right. I will test it now.

Its fixed now downloading perfectly.


[WFLY-19621] Upgrade the keycloak-services and keycloak test dependencies to the latest version

Issue: https://issues.redhat.com/browse/WFLY-19621

<--- THIS SECTION IS AUTOMATICALLY GENERATED BY WILDFLY GITHUB BOT. ANY MANUAL CHANGES WILL BE LOST. --->

Wildfly issue links:

WFLY-19627

<--- END OF WILDFLY GITHUB BOT REPORT --->
More information about the wildfly-bot[bot]

Thanks @PrarthonaPaul


Styles text inside buttons

I would like to allow styling of button text using ckeditor.
When I import mjml code with a button component like so:
<mj-button href='...'>
	Hello
</mj-button>

It creates this html inside the canvas:
 	Hello
I suspect that the model considers the styled text as a text component by itself. Not sure how it gets to decide that...
These attributes are set for every component inside the button, and it causes it to act funny.
When I go to the mjml core view and comment out this.renderContent();  all the content is gone. Button content is gone, but NOT the text above. It's like mjml doesn't consider the Hello text to be the content of the component. Which makes me think it creates the text inside the button as a new and separate component.
Any ideas how to make the text inside the button part of the button element?

I don't think we can easily provide a way to mix html and mjml code, however I added a way to style buttons as italic (the font-style property was missing)


Issue with Special Characters in File Names When Downloading and Converting Videos

Problem Description:
When using the download and conversion feature to MP3 format, it has been observed that if the video title contains special characters (such as question marks, quotes, non-ASCII characters, etc.), the process fails when attempting to save the MP3 file. This is due to these characters not being valid in file names on certain file systems, such as Windows.
Steps to Reproduce:
Attempt to download a video whose title contains special characters, for example, "¿Por qué no me comprendes?".
Notice that the process fails when trying to save the MP3 file, with an error message related to "invalid argument".
Proposed Solution:
Implement a file name cleaning function that removes or replaces special characters before saving the MP3 file. Here is an example of how this function could be implemented:
import re

def clean_filename(title):
    """
    Cleans the video title to ensure it's a valid file name.
    Removes special characters and replaces spaces with underscores.
    """
    cleaned_title = re.sub(r'[\\/*?:"<>|\']', '', title)
    cleaned_title = re.sub(r'\s+', '_', cleaned_title)  # Replace spaces with underscores
    return cleaned_title

hey did you happen to solve this?

hey did you happen to solve this?

Hey @hrantvirabyan , I didn't see your comment, sorry for that. But yes, this function solved the problem.


The bot should specify a user agent

The Mastodon.py library gives "mastodonpy" as its default user agent. It is possible to specify a different user agent, which would allow instance operators to block this bot script (or distinguish it from others). PR coming.

Closed by #36


Switching Loaders / Vectors

If i need to switch a vector, i can switch dynamically based on the setVector and Path settingss and changing these to different values.
The question is, loaders, for example if i wanted to use a defined loader i used with a vector db, how to retrieve or remember it or reload it back when switching from one to other. I have a listLoaders api i wrote, to return me the list, types and id's but if you reload server, they all go.  So it's about persisting without have to call the reloaders again such as re-adding web pages once they are in the vector db/llm cache.
If ti can be managed, this way you can potentially have multiple rags, and even build a graph type llm or a simpler router llm at the front of it later to decide which rag to to switch to.
should there be a loadVector/LoadEmbeddings function?
Any advice is appreciated. I think your doing an awesome job, I will try raise that FAISS PR this weekend!

Switching vectors is tricky because they need to be initialized. I have been thinking myself also around adding support for multiple RAG sets. One can then set which RAG set to use for which query and can also multiple RAG sets each for a few documents.
However, this complicates the API quiet a bit making it difficult for simpler use cases. Needs more thought this one.

Hi there, been thinking too about this, and wanted to share my thoughts and whether this can help and if these scenarios are possible
What I obsere today it should be possible to

Create loader intances and track them in memory and switch object of objects or array of objects with a unique rag key
The  rag can be switched based on the vector and llm cache on the fly

It's about persistence (i.e. if needed to restart server) and or load the last one that was in use
Is possiblefor the ragbuilder/app instance object state to be somehow stored in some kind of binary o other format, saved to disk,  Basically a loader representation no disk

saveLoaderStateToDisk(ragid)

saves current state of existing loader updating it

removeLoaderStateFromDisk(ragid)

removes loader from disk

switchLoaders(rag id)

switches loader from one to other in that current object (if no current loader, simply loads in the specifed loader if exist)

removeAllLoaderStatesFromDisk();

removes all stored loaders

getLoaderIsntance(rag id) into instance of variable type ragApplication etc (useful for in memory switching) so you could load multiple rags and switch on the go, This could then be fronted by a LLM router or LLM knowledge Graph to determine which sub rag to use, switch to and query.


The 'clusters' tab doesn't look highlighted when selected

Describe the bug
The home, helpful links and FAQ tabs are highlighted when selected. But the clusters tab is not highlighted when selected.
To Reproduce
Steps to reproduce the behavior:

Go to 'https://www.covid19india.org/'
Observe that the home tab is highlighted
Click on 'CLUSTERS'
Notice that the clusters tab isn't highlighted though it's the currently open tab

Expected behavior
When the clusters tab is clicked, it's expected to be highlighted like how the other active tabs are
Screenshots

Desktop (please complete the following information):

OS: Windows 10
Browser: Chrome
Version 80.0.3987.149 (Official Build) (64-bit)

Additional context
Home tab:

Fixed in
https://github.com/covid19india/covid19india-react/pull/92#issue-393477338
Commit: 8de3d90f899ed97cf8e355f9f03f22804d1e45de

Are you working on this issue?

I've already made a PR on this.

Nice

Seems to be fixed now. Closing


CSV upload skeleton

User can upload a CSV file. Contents of CSV file are printed to terminal.

Added bulk blueprint, which will contain all steps of bulk resource upload (CSV file upload, multi-stage review process, final resource saving).
Added route and form to handle CSV file upload.

r @armant

r @dzhang55

LGTM


[feature] Content in iFrame not rendered

Describe the bug
The content I wish to monitor is within an iFrame and CD does not seem capable of capturing it's content.
Version
v0.39.11
To Reproduce
Steps to reproduce the behavior:

Add a site which contains an iFrame
Content within iFrame is not detected

Note
I can share a link to the site in question privately, but do not wish to do so publicly.

not a bug, but not implemented, I'm open to PR's if you want to contribute something here

oh, i just came over the same limitation when using changedetection with playwright chromium - no iframe is loaded in 'browser steps'


Larger charset

Release 1.0.1 seems to be restricted to pure ASCII.
Would it be possible to extend it to PDF encoding as described on https://gist.github.com/zmughal/5130211  ?
Unless it diverts work away from full unicode support of course.
I tried to use one-file-pdf for Swedish in https://github.com/jonasgit/pdfhw but it seems to fail. Unless I did something wrong?

I'm sorry, other commitments have been keeping me from working on this project. Thank you very much for your feedback. I think it'll be all right to have some Latin support beyond ASCII, as much as provided by the PDF standard. It seems easier to implement than full Unicode support. I'll look into that and how to make it work for you. Just give me a few days.

Thanks. No need to hurry, it is fine whenever.