Add features to interactive finder.py.

Here's a sketch of list from the docs.

[ ] Draw the slit (width + length).
[ ] Identify which stars are in the slit, print their info.
[ ] Try to show TESS light curves of stars in the slit.
[ ] Calculate the total flux (= photon noise) of comparison stars along the slit.
[ ] Explore interaction on different platforms. Seems best in jupyter notebook, only halfway works in VSCode.
[ ] Improve behavior for clicking just one star (=have it print a single-star catalog entry?)

I have a vague memory that @catrionamurray might have done some of these last year, particularly the drawing a slit onto the finder?

Added plot slit function here:
https://github.com/zkbt/kosmos-craftroom/pull/7


Build docs on how to use the CI pipelines (if necessary)

issue already created: #159

Recreated with issue #159


Warn about duplicate dependencies

Add duplicate-package-checker-webpack-plugin as proposed here: #1844 .
Display compilation warnings on build.

This feature can be quickly tested by creating an app, installing an old version of fbjs (yarn add<EMAIL_ADDRESS>and importing fbjs from App.js. This will result in having duplicate fbjs packages since React already requires the most recent version of fbjs.

"duplicate-package-checker" is not a very friendly way to log it. The end user doesn’t know what a "duplicate-package-checker" is. Instead, it should say:
The following packages are duplicated in the bundle:

and also add some instructions on how to avoid it.

(I understand this may not be possible, but we need to work with its author to ensure it provides a good experience to our users. We also need to make sure we have some sensible instructions to follow to solve the issue.)

Been a while 😄 I've updated the plugin to display a better warning message and beefed up the Readme in the repo with some possible solutions for resolving duplicate package issues (https://github.com/darrenscerri/duplicate-package-checker-webpack-plugin)
As for the warning message, Webpack prepends all warnings with a "Warning in:" which is usually followed by a package name or a path to the file that the warning refers to, so I've settled down on starting the warning with the duplicated package name followed by relevant information.
CRA:

Non-CRA Webpack:

Fancy!


Improve the File.endian_swap documentation

This adds more information about endianness, which is useful for beginners to file access.

Thanks!

Cherry-picked for 3.2.4.


Added link to linear search in all README files

It seems like there is no link to linear search that I just created in README files(Chinese, German, Korean and Japanese), so I added them :)

Pull Request Test Coverage Report for Build 383

0 of 0   changed or added relevant lines in 0 files are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage increased (+0.03%) to 91.693%

Files with Coverage Reduction
New Missed Lines
%

algorithms/sort/top_sort.py
1
92.31%

Totals

Change from base Build 382:
0.03%

Covered Lines:
2870

Relevant Lines:
3130

💛  - Coveralls


docs: add the diff standard

In order to categorize a certain change to AsyncAPI document as "breaking" or "non-breaking", we need to have a "standard" which documents what changes can be considered "breaking" or "non-breaking".
When the community agrees on this "standard", it will be mapped to a JSON file which will contain all the information in here, which the diff library can utilize to categorize changes.
Please note that the standard has been written from the Point of View of a Consumer(client/server), thus some changes were considered as "non-breaking" because the consumer won't be affected by that change.

When the community agrees on this "standard", it will be mapped to a JSON file which will contain all the information in here, which the diff library can utilize to categorize changes.

I love the idea of a machine-readable diff-map which can be applied to a target document. Thus (if there was a JSON Schema for the diff-map object) we could create one for other JSON/YAML based standards (cough) not just AsyncAPI.
My only query would be about treating a change in $ref automatically as a breaking change. Possibly the diff should run over a de-referenced input file? Thus converting a direct object into a $ref or vice-versa would itself not be treated as a change at all.

When the community agrees on this "standard", it will be mapped to a JSON file which will contain all the information in here, which the diff library can utilize to categorize changes.

I love the idea of a machine-readable diff-map which can be applied to a target document. Thus (if there was a JSON Schema for the diff-map object) we could create one for other JSON/YAML based standards (cough) not just AsyncAPI.
My only query would be about treating a change in $ref automatically as a breaking change. Possibly the diff should run over a de-referenced input file? Thus converting a direct object into a $ref or vice-versa would itself not be treated as a change at all.

My 2c on whether $refs are breaking changes... ideally I'd like to see both, something indicating that the $ref is broken, and something indicating that all those things that reference are naturally broken. This prompt is based on how validation that occurs on the resolved spec can be noisy as hell. As well as the fact that the breaking change may being address/fixed in one place, not many -- both on code (if the $ref'd model is reusable) and in the definition. </2c>

I would love if the document defines the thought behind when a change is considered non-breaking vs breaking.

@jonaslagoni Working on it :)

For example, any change to id might not be considered a breaking change in general (again depending on the thought behind the diff), but it depends on the use case. If a user uses the id to generate specific code that suddenly breaks because for that user id IS a breaking change.

Interesting. You are right, it might depend on the use case whether a change in breaking or not.

As @MikeRalphson touched upon, I suggest that, while we define a standard diff, it should be adaptable and modified by the user, which means instead of hardcoding/forcing what is breaking and what is not, maybe it can be done using a configuration? At least consider whether it would be worth investing in from the start, maybe it won't be that difficult compared to hardcoding it.

I really like this idea. I'm thinking of having some defaults configs that come with the package, but it can be easily overridable. So, In case if someone doesn't like the default configs, they can provide their own config, which our diff will use instead of using the default one. Any thoughts?

I completely agree with @MikeRalphson that changes to $ref cannot on its own be considered a breaking change. It needs to be realized first.

Yup, I think you are right. The parser will inline those $refs, so i think it should not be considered a breaking change.

did you already think on how this standard would be coded?
did you consider that what you have here could be later declared in a yml file that user can override as a config?

Kind of. Will need to discuss on that as well, but right now, as a POC, I have a standard.json file which has this key:value pair. Like "asyncapi": { "type":"breaking" }, but Like I said, nothing is set right now. We will need to discuss on that.
So, if a user provides a json or yml(which we can convert to json), we will merge that change into the JSON using some JSON merging technique.

@aayushmau5 yup, a definitely separate topic, good for another GitHub Issue 😄 and something that can be added later, which will be easy if you already take config from a json

Hey @vinitshahdeo There's an ongoing discussion about the output in this issue: https://github.com/asyncapi/diff/issues/14
Definitely need your input here :)

@aayushmau5 what plans do you have with this PR?

I was thinking about this, and I'm not too sure what to do with this document. If we only care about the categories of changes, one can see them inside our standard object.
I guess we can use this document to justify why a change has been considered breaking, non-breaking etc. What do you say?

for sure useful part is how we classify changes and why but rest is redundant with the standard I think. Maybe just move some of it to readme and that is it, and in the readme link to standard after initial explanation

Closing this one. Main points that should be addressed will be addressed in the README.


Slight difference between implementation and paper.

Namaste!
I think this is my final issue :D
In the implementation of the classifier, you start with a single-head-attention, then you do the spatial transformation followed by multi-head-attention. In the paper, on the other hand, you start with the transformation.
I have the feeling that I have missed something. What is your opinion?

Another difference. Correct me if I am wrong: The implementation of attn_feature does not 100% match. Especially "edge_feature = input_feature_tiled - neighbors" seems to be missing on the picture.

Yes. The paper mentions that a single-head attention happens as part of the spatial transform step.


It takes too long time to generate lmdb from images. How can I improve this issue?

I've tried convert from images to lmdb but it took about 3 hours. And now I wanna convert about 1.6 million image, so it wastes too much time. Can you help me to improve this? Thank you so much!

There may be something wrong with your HDD, I guess. Check it or try a SSD.

Thank you, I'm trying that.


Update Composer dependencies (2024-06-29-00-19)

Visual regression test passed!
View the visual regression test report


Type of FormioComponents.components

Isn't this supposed to be an array of form wrapper objects or am I getting something wrong?
Currently:
// ...
export class FormioComponents {
    public static components:FormioComponentWrapper = {};
// ...

My Idea:
// ...
export class FormioComponents {
    public static components:FormioComponentWrapper[] = [];
// ...

https://github.com/formio/ng2-formio/blob/master/src/components/components.ts#L19
Or maybe an object that contains FormioComponentWrapper objects? Anyway, the type seems a bit confusing.

Closed due to no activity. Please reopen if you still have the issue.


CONTACT FORM

[ ] Contact form header - mobile
[ ]  Contact form description - mobile
[ ]  Contact form - mobile
[ ]  Contact form footer - mobile
[ ]  Contact form header - tablet
[ ]  Contact form description - tablet
[ ]  Contact form - tablet
[ ]  Contact form footer - tablet
[ ]  Contact form header - desktop
[ ]  Contact form description - desktop
[ ]  Contact form - desktop
[ ]  Contact form footer - desktop
[ ]  Published online


OpenApiPaths and Operations

Hi all
I have a question regarding OpenApiPaths and their operations:
Let's assume I have the following two endpoints:
/api/mp/v1.0/order/{ordernumber}
/api/mp/v1.0/order/{id}

Is this valid from an OpenApi spec point of view?
According to online.swagger.io and OpenApiDiagnostic this is no issue.
Whey are they not combined? Or why is this not a validation issue?
With the following example from an API point of view I don't know what has been called:
/api/mp/v1.0/order/10
code to generate OpenApi spec:
        /// <summary>
        /// Gets the resource with the specified id.
        /// </summary>
        /// <param name="id">The resource id.</param>
        /// <response code="200">The resource with the specified id.</response>
        /// <response code="404">A resource with the specified id could not be found.</response>
        [HttpGet("{id}")]
        [Authorize(PolicyNames.OrderManagement.CanRead)]
        [ProducesResponseType(typeof(OrderReadDTO), StatusCodes.Status200OK)]
        [ProducesResponseType(typeof(BaseErrorResult), StatusCodes.Status404NotFound)]
        public async Task<IActionResult> GetById(int id)
        {
            var result = await _OrderService.GetSingleAsync(id);
            return new ObjectResult(result);
        }
        /// <summary>
        /// Gets the resource with the specified order number.
        /// </summary>
        /// <param name="orderno">The resource order number.</param>
        /// <response code="200">The resource with the specified order no.</response>
        /// <response code="404">A resource with the specified order no could not be found.</response>
        [HttpGet("{orderno}")]
        [Authorize(PolicyNames.OrderManagement.CanRead)]
        [ProducesResponseType(typeof(OrderReadDTO), StatusCodes.Status200OK)]
        [ProducesResponseType(typeof(BaseErrorResult), StatusCodes.Status404NotFound)]
        public async Task<IActionResult> GetByOrderNo(int orderno)
        {
            var result = await _OrderService.GetSingleAsync(orderno);
            return new ObjectResult(result);
        }

The current version of the OpenAPI specification says this scenario is invalid.  http://spec.openapis.org/oas/v3.0.3#path-templating-matching
The following paths are considered identical and invalid:

  /pets/{petId}
  /pets/{name}

This has been the case since the specification was created.  This is one of the many challenges of trying to infer an OpenAPI specification from annotated code.  The general consensus in the space is that explicitly creating the OpenAPI document as a design artifact up front is the best way to design APIs using OpenAPI.
There is a proposal on the table to introduce an priority property that would allow tooling to choose one operation over the other if there is ambiguity.  I don't believe this would help the specific case you have shown.

thanks for your answer @darrelmiller
so shouldn't there be an error when loading a file like this in OpenApiDiagnostic?
/pets/{petId}
/pets/{name}

I have no issue with this being invalid but the tooling is not reflecting this properly imho.


Product types should be searchable

What I'm trying to achieve
In #3363 it was pointed out that product type field should use autocomplete rather than fetch all product types automatically. To achieve that, productTypes query needs an additional argument query, so the API consumer would be able to search it by name fragment.

Closing in favor of https://github.com/mirumee/saleor/issues/4199.


change message to lowercase before profanity check

This fixes #71 by using a lowercase version of the message to check for profanity.

Glad I could help!


PR to fix Issue 124 in reInvent branch

Issue #, if available:
https://github.com/awslabs/autonomous-driving-data-framework/issues/124
Description of changes:
removed reference to package apache-airflow[amazon] that is causing conflicts in reInvent branch
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

wrong PR


Redesign Message Tracker Details Receiver Table

Problem statement
We want to implement some changes to how the data is conveyed on the Message Tracker to improve the visual hierarchy and overall readability based on internal usability findings.
What you need to know

The new design includes two primary sections -- one about the Sender and one list the Receivers -- with more emphasis on actionable data. See the Figma for reference.

Style Guide specs in Figma

Acceptance criteria

[ ] Receivers section

[ ] Each Receiver's data is organized into a table row with the following columns:

[ ] Name (receivingOrg)
[ ] Service (receivingOrgSvc)
[ ] Date (createdAt)
[ ] Report ID (reportId)
[ ] File Location

[ ] Parent (parsed from fileUrl)
[ ] Child (parsed from fileUrl)
[ ] File Name (parsed from fileUrl)

[ ] Transport Results (transportResult or N/A if null)

[ ] Each row is sortable by Name, Service, Date, Parent, Child, File Name, and Transport Results
[ ] Each table cell truncates longer content
[ ] Each table cell can be clicked to expand to reveal full content

For context, Penny, Jillian and I discussed next steps of this ticket and the design decisions we made going forwards: https://nava.slack.com/files/T03M7E63A/F04MMMKTPSS

For posterity, I'll paste the above notes here in case someone can't access the above:
Notes:
Average number of 9 receivers
Discussion of alternate cell expansion behavior for cells with long text fields
Current Figma design: Complex for engineers to build; difficult UX challenge
Possible Solutions: Creating a horizontal scroll for the entire table; Create horizontal & vertical scroll within the cell itself; Add copy & paste button if needed
Use Case: O&O engineers will want to expand the field to read the file name to look it up in Azure; O&O engineers will want to read the full Transport Results field (don’t need to copy paste it anywhere)
Warnings & Errors section
Tracking ID session long; unclear what it’s pulling from
Difficult to distinguish between rows
The error descriptions are very long
Decisions & Action Items
Etan: The File Name and Transport Results column will open in a modal to show the full, non-truncated text field
No additional UI needed
Use default USWDS modal
Etan: Preserve column behavior as sorting one-at-a-time for this MVP
Jillian: Connect with Audrey about including the Tracking ID column and the Error Description field length
Penny: Restore default striped style to Warnings & Errors table

Increased to 5 points per Etan's revised estimate.


Add support for automaticDataCollectionEnabled

Version info

Angular: N/A
Firebase: 4.13.0
AngularFire: 5.0.0-rc.6.0
Other (e.g. Ionic/Cordova, Node, browser, operating system): N/A
How to reproduce these conditions
Firebase Javascript SDK version 4.13.0 added a new field in the FirebaseApp class called automaticDataCollectionEnabled. Upgrading Firebase to this version causes TypeScript error.
Debug output

Expected behavior

Build should pass.
Actual behavior

Build is failing.
More information
Refer to firebase/firebase-js-sdk#625

Temporary fix: https://github.com/angular/angularfire2/issues/1571#issuecomment-383079493

fixed in rc7


make gsutil rsync recursive

Small change to allow gsutil to do rsync recursively, syncing all the folders.
Context: We had some issues using it with gcloud, since make sync_data_up would not upload all the data folders. Adding the -r flag fixes this issue.

Thanks @KBodolai. Rebasing on to the latest v2 should work now to run the tests.

ok, I've done that, hopefully it'll be it!

ok, so these passed (yay!)
There's another parameter that can be fed to gsutil for threaded processing (-m), which provides huge performance boosts, although they mention in some cases it may slow the syncing down.
I believe it will typically only result in speedups for the setting this is used in, but not quite certain, what do you think?

@KBodolai I agree that 99% of the use cases here will be cloud <-> local, not local <-> local so -m likely makes sense. Happy to wait on this til that id added

right, I'll add it and set up a test project to double check it does well, I'll ping you when it's ready :)

@pjbull , actually, I stayed a bit late after work and tested if with a bunch of heavy lidar files, all working fine!


Apply (optional) startup checks when mechs are shutdown

In TT mechs must pass a skill check to restart when they are shutdown. Apply these semantics.

Added in 0.3.9 was feature: startupChecks.


feat: upgrade to 3.4 version

Proposed Changes
Upgrading from Invenio v3.3 to v3.4 following this guide
Please review

[ ] @libremente

Search bar does not work properly


Live transcribe

Description of the new feature / enhancement
adding live transcription feature on onenote application to power toys.
Scenario when this would be used?
Anytime can be useable for example while watching local videos on your drive.
Supporting information
No response

This must be available in latest Windows 11.

Built into windows 11 22h2. https://support.microsoft.com/en-us/windows/use-live-captions-to-better-understand-audio-b52da59c-14b8-4031-aeeb-f6a47e6055df


hostPlatform.arch: treat all aarch64 as arm64

If using the nix-darwin linux-builder (which is aarch64-linux), isDarwin is false, but the processor is still aarch64, and we want the build arch to be arm64 in this case too.

Thanks!


Get and Set JSON in localstorage

Description
Update the storage utility to provide convenience functions to get and set JSON.

Completed as part of #1 .


New version: RandomEngy.VidCoder version 9.18

[x] Have you signed the Contributor License Agreement?
[x] Have you checked that there aren't other open pull requests for the same manifest update/change?
[x] This PR only modifies one (1) manifest
[x] Have you validated your manifest locally with winget validate --manifest <path>?

[x] Does your manifest conform to the 1.5 schema?

Note: <path> is the name of the directory containing the manifest you're submitting.

Microsoft Reviewers: Open in CodeFlow

/AzurePipelines run

/AzurePipelines run

/AzurePipelines run

@wingetbot waivers Add Validation-No-Executables


Add references and tutorials for forge clone

Now that forge clone command has been merged into foundry main repository in https://github.com/foundry-rs/foundry/pull/7576, here we also add command references, tutorials, and examples for forge clone in the foundry book.
Credits to @ZhangZhuoSJTU and @Troublor.

@all-contributors
please add @Troublor for code.
please add @ZhangZhuoSJTU for code.

@all-contributors please add @ZhangZhuoSJTU for code

@all-contributors please add @ZhangZhuoSJTU for code


HHH-15057 JdbcSQLException: NULL not allowed for column "BOOKS_ORDER" when inserting into many-to-many list

https://hibernate.atlassian.net/browse/HHH-15057
Explanation (copied from the ticket):

See https://hibernate.zulipchat.com/#narrow/stream/132094-hibernate-orm-dev/topic/List.20are.20no.20longer.20bags.20by.20default.3F
The problem seems to be that we add an order column for both sides of the association, which is not supported (even when using @OrderColumn explicitly, even in ORM 5.x).
So the non-owning side of a many-to-many association will have to remain a bag, independently from whatever is configured as the default list semantics.
From what I’ve seen this will have to be implemented in org.hibernate.cfg.annotations.CollectionBinder#determineCollectionClassification(java.lang.Class<?>, org.hibernate.annotations.common.reflection.XProperty, org.hibernate.boot.spi.MetadataBuildingContext)

I decided to keep applying the default list semantics to the owning side of the many-to-many association.

@dreab8 Hopefully this solves the TCK failure with a bi-directional many-to-many :)

@yrodiere not super happy that we cannot support the order on both side for a bidirectional manytomany but not sure if/how we can implement it, so I would say the change is ok. I think we have to document this!
@sebersole what do you think?

@dreab8 Yes unfortunately that's the conclusion we reached there: https://hibernate.zulipchat.com/#narrow/stream/132094-hibernate-orm-dev/topic/List.20are.20no.20longer.20bags.20by.20default.3F/near/269744314


Mapping properties of one type on another

I'm looking for something like this:

var form = new CarFormModel();
form = Map(details, form);

public static CarFormModel Map(ContactDetailsCar source, CarFormModel target)
{
    // Do mapping here

    return carFormModel; // Return target
}

Please take a look at https://github.com/cezarypiatek/MappingGenerator/issues/93


Implement support for \text

Problem
The \text command is not parsed (and thus not renderable):
rex::parser::engine::parse("\text{a b}");
// Err(UnrecognizedCommand("text"))

Workaround
To get text-like format, one can use \mathrm{...}. Problematically, \mathrm{...} just changes the font style ; it keeps ignoring spaces as in the rest of maths mode. \text{...} does not ignore spaces
Leads
Prior to sb3k's rewriting, ReX seems to implement or have envisioned implementing \text{..}, cf here.

There is experimental support for this since  36b60310537e295a404d73309283116c7c2e2b01


Corrected the tube and added new geometries for a torus arc (partial torus) and a wedge (a pie shape).

I'm new to GitHub, but this appears to be the right way to submit changes. This addresses one of the defects raised and adds two new geometries.

Thanks for your additions and fixes!! all merged to main now, I'll publish a release soon


Require Origin header

Based on the previous PR #68, this delightful piece of code sends a 403 if people try to access crossorigin from:

the same origin (ie. crossorigin.me)
a browser, without using XHR
anything that doesn't set the Origin header

Pros:

Harder to abuse the proxy for casual browsing, eg http://crossorigin.me/http://scratch.mit.edu/ or http://crossorigin.me/http://imgur.com

Cons:

Not as easy to debug your code?
Trivially workaroundable -- curl -H 'Origin: foo' ...

Based on  #68

Here's the gist of my concern about this: any AJAX/XHR request from JS should Just Work™. I'm not sure if standard ajax/xhr request things add these headers?

XHR requests add CORS headers. I believe Origin is part of the CORS spec. This should allow any JS requests, while disallowing casual browsing.

(btw, can you rebase this on master? Looks like there are some conflicts in the test files)

done :)


More REST compliant APIs

Rework on REST APIs to be self explaining (proposition will follow)

Yep, routes should not contains any verb IMHO.

any suggestions are welcome !

I know our current APIs are not so nice, but without any suggestions, it can't be improved.
So your opinions are needed here, @fdelbrayelle @hdurix

@hdurix Maybe we can discuss this soon?

today? :)

Things I have in mind:

add /clients before Vue.js, React, angular... As we have /servers/ for servers API:

/api/servers/spring-boot is compliant
/api/vue is not compliant

try to have something like <collection>/<id>/<collection>/<id>/.../<element> (URL "leaf" can be either a collection, an id or an element):

/api/servers/spring-boot/databases/mysql is compliant
/api/svelte/styled is not compliant

maybe try try to thing about using request bodies to specify some options like:

/api/servers/spring-boot/banner with a required request body attribute of the wanted type instead of /api/servers/spring-boot/banner/jhipster-v2 & others
merge /api/servers/sonar/java-backend and /api/servers/sonar/java-backend-and-frontend in /api/servers/sonar (btw, sonar is not a server, this could also be changed)

some good idea here
To ease this ticket, it's better to improve each context in separated PR, it will be easier to review and discuss

OK I'm creating a ticket by topic.

I think the work on modules covers that, what do you think about closing that and keep migrating modules?


User is logged out after a few hour in hybrid app

Hi, I think I found issue in SPiDHybridApp - we use this library in BT app: https://itunes.apple.com/pl/app/bt.no/id435546727?mt=8
Issue: User is logged out after a few hour.
How reproduce this error:

Install app
Login
Read something
4 ... long break f.ex. whole night
run app
read closed article

When I try do that I see salesposter.
I think, issue exists because webview lost cookie: SP_ID important for Varnish. App/library should recreate it. This cookie exists only for a few hours.

Hi! This sdk doesn't have a real maintainer and I'm probably the closest to being one at the moment. So I'll ask som basic questions to get those squared away:
Which version of the sdk are you using?
Does this occur on any specific versions of iOS or devices?
Is the bug reproducible in the SPiDHybridApp example in this repo?

Hi, we use newest version.
I can't reproduce the error in example app because app uses static HTML.
I think key point is that SPiD Varnish module uses cookie SP_ID to works properly. I don't see it in repository so SDK does not set it :/

The SDK does not set the varnish cookie. The cookie is set by SPiD through the webview itself via the Javascript SDK running on it. The cookie timeout is specified in the initialisation of the javascript sdk.
The way I see it this is solvable in many ways:

Extend the varnish cookie timeout for webviews in the javascript initialisation
Re-initiate the authentication (and following cookie refresh) after a certain period in the app itself. The user's session in the webview lasts for 2 hours of inactivity. So if the user opens the app after 2 hours you know the user wont be logged in. The varnish cookie can have a longer timeout, but  since the app knows the user isn't logged in anymore a refresh of the session is probably what you need.
Implement the APi for retrieving and setting the varnish cookie in the webview (http://techdocs.spid.no/endpoints/GET/user/{userId}/varnishId/)

That makes sense - thanks for your help and explanations.


chore(deps): update common-dev-assets digest to c72cc0b

This PR contains the following updates:

Package
Update
Change

common-dev-assets
digest
909dabb -> c72cc0b

Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
👻 Immortal: This PR will be recreated if closed unmerged. Get config help if that's undesired.

[ ] If you want to rebase/retry this PR, check this box

This PR has been generated by Renovate Bot.

Edited/Blocked Notification
Renovate will not automatically rebase this PR, because it does not recognize the last commit author and assumes somebody else may have edited the PR.
You can manually request rebase by checking the rebase/retry box above.
⚠ Warning: custom changes will be lost.


swiftlint 0.8.0

What to do if the build fails?
At the moment contributors do not have access to the CI workflow triggered by StepLib PRs. In case of a failed build, we ask for your patience. Maintainers of Bitrise Steplib will sort it out for you or inform you if any further action is needed.
New Pull Request Checklist
Please mark the points which you did / accept.

[x] I will not move an already shared step version's tag to another commit
[x] I read the Step Development Guideline
[x] I have a test for my Step, which can be run with bitrise run test (in the step's repository)
[x] I did run bitrise run audit-this-step (in the step's repository - note, if you don't have this workflow in your bitrise.yml, you can copy it from the step template.)
[x] I read and accept the Abandoned Step policy

 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.

Hello @kimdv,
Can you please sign the CLA here: https://cla-assistant.io/bitrise-io/bitrise-steplib?pullRequest=2608.
Thank you.

Hi @lpusok 👋
I should already have signed it?
🧐

Hi @kimdv,
It seems that the signature is all right, just did not show up in the CLA widget.


tab-bar support

Current status:

After the change (feel free to adjust the colours)
twilight:

tomorrow night:

solarized:

Fixes: https://github.com/tinted-theming/base16-emacs/issues/132

I've somehow never seen tab-bar before, but this looks like a massive improvement. Thanks for the PR!


Simpler way to convert String to ByteArray

https://groups.google.com/d/msg/ceylon-users/O_T78CVFxTM/fGqEZzwTQqgJ
A lot of java api's require Byte Arrays, it seems really complex to convert some Ceylon Types into byte arrays.

@FroMage wrote the code for character encodings and decodings, maybe he has an idea on the best way to handle this?

I think we should add Array<Byte> bytes() to ByteBuffer.
I also think the signature of newByteBufferWithData() is wrong. It should accept a {Byte*} stream, not a variadic Byte*.

Not going to make it into 1.2.

I think we can close this issue… this works and is relatively simple IMO:
String string = "“Hello, World!”, said the \{ELEPHANT}.";
ByteBuffer byteBuffer = utf8.encodeBuffer(string);
Array<Byte> byteArray = byteBuffer.array;
ByteBuffer byteBuffer_ = ByteBuffer.ofArray(byteArray);
String string_ = utf8.decode(byteBuffer_);

Try it!
(Note that, due to #649, string != string_, but that’s an unrelated issue.)

Hm, actually, not so simple… the seemingly obvious byteBuffer.array is wrong, that’s just the underlying storage without regarding position and limit. The correct version is byteBuffer.array.measure(byteBuffer.position, byteBuffer.available). Is that non-obvious enough to warrant keeping this issue open?


api-docs: Update /set-typing-status endpoint documentation.

Updates made:

Adds Changes notes for feature level 58 where support was added for stream messages for the /set-typing-status endpoint parameters. Relevant CZO conversation for this part of the  changes.

Updates formatting for references to the type parameter in the descriptions of other endpoint parameters.

Improves readability of and updates links in the endpoint's main description.

Screenshots and screen captures:

Self-review checklist

[x] Self-reviewed the changes for clarity and maintainability
(variable names, code reuse, readability, etc.).

Communicate decisions, questions, and potential concerns.

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

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

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

Completed manual review and testing of the following:

[x] Visual appearance of the changes.
[ ] Responsiveness and internationalization.
[ ] Strings and tooltips.
[ ] End-to-end functionality of buttons, interactions and flows.
[ ] Corner cases, error conditions, and easily imagined bugs.

Merged, thanks @laurynmm!


can't use enum for String[] type

Since prisma doesn't support Enum Arrays as types, I'd like to map a String[] field to a typescript Enum array. Something like this:
declare global {
  enum UserPermission {
    CanCreateCourse = 'CanCreateCourse',
  }
}

model User {
  id             String            @id @default(cuid())
  /// [UserPermission]
  permissions    String[]          @default([])
}

While this works fine today during runtime, when I run npx prisma generate, I get these errors:
Running generate... - Prisma Json Types Generator
✘ Type UserPayload.permissions is not supported.
✘ Type UserGroupByOutputType.permissions is not supported.
✘ Type UserWhereInput.permissions is not supported.
✘ Type UserScalarWhereWithAggregatesInput.permissions is not supported.
✘ Type UserCreateInput.permissions is not supported.
✘ Type UserUncheckedCreateInput.permissions is not supported.
✘ Type UserUpdateInput.permissions is not supported.
✘ Type UserUncheckedUpdateInput.permissions is not supported.
✘ Type UserCreateManyInput.permissions is not supported.
✘ Type UserUpdateManyMutationInput.permissions is not supported.
✘ Type UserUncheckedUpdateManyInput.permissions is not supported.
✘ Type UserCreateWithoutPasswordInput.permissions is not supported.
✘ Type UserUncheckedCreateWithoutPasswordInput.permissions is not supported.
✘ Type UserUpdateWithoutPasswordInput.permissions is not supported.
✘ Type UserUncheckedUpdateWithoutPasswordInput.permissions is not supported.
✘ Type UserCreateWithoutActivityInput.permissions is not supported.
✘ Type UserUncheckedCreateWithoutActivityInput.permissions is not supported.
✘ Type UserUpdateWithoutActivityInput.permissions is not supported.
✘ Type UserUncheckedUpdateWithoutActivityInput.permissions is not supported.
✘ Type UserCreateWithoutCoursesInput.permissions is not supported.
✘ Type UserUncheckedCreateWithoutCoursesInput.permissions is not supported.
✘ Type UserUpdateWithoutCoursesInput.permissions is not supported.
✘ Type UserUncheckedUpdateWithoutCoursesInput.permissions is not supported.
✘ Type UserCreateWithoutInstitutionInput.permissions is not supported.
✘ Type UserUncheckedCreateWithoutInstitutionInput.permissions is not supported.
✘ Type UserScalarWhereInput.permissions is not supported.
✘ Type UserCreateWithoutRubricsInput.permissions is not supported.
✘ Type UserUncheckedCreateWithoutRubricsInput.permissions is not supported.
✘ Type UserUpdateWithoutRubricsInput.permissions is not supported.
✘ Type UserUncheckedUpdateWithoutRubricsInput.permissions is not supported.
✘ Type UserCreateWithoutStudentsInput.permissions is not supported.
✘ Type UserUncheckedCreateWithoutStudentsInput.permissions is not supported.
✘ Type UserUpdateWithoutStudentsInput.permissions is not supported.
✘ Type UserUncheckedUpdateWithoutStudentsInput.permissions is not supported.
✘ Type UserCreateWithoutTokensInput.permissions is not supported.
✘ Type UserUncheckedCreateWithoutTokensInput.permissions is not supported.
✘ Type UserUpdateWithoutTokensInput.permissions is not supported.
✘ Type UserUncheckedUpdateWithoutTokensInput.permissions is not supported.
✘ Type UserCreateWithoutCreationsInput.permissions is not supported.
✘ Type UserUncheckedCreateWithoutCreationsInput.permissions is not supported.
✘ Type UserUpdateWithoutCreationsInput.permissions is not supported.
✘ Type UserUncheckedUpdateWithoutCreationsInput.permissions is not supported.
✘ Type UserCreateManyInstitutionInput.permissions is not supported.
✘ Type UserUpdateWithoutInstitutionInput.permissions is not supported.
✔ Generated Prisma Client (4.11.0 | library) to ./node_modules/@prisma/client in 302ms
✔ Generated Prisma Json Types Generator (2.2.3) to ./prisma in 1.26s

I'm assuming it's somewhat related to #34, but wanted to create another issue since the end goal is different.

To clarify: I could use a Json type in the database for this, but then I'd lose all the array functionalities that prisma provide

The internal implementation should be the same as #34. I'll close this issue to keep things clean.

To clarify: I could use a Json type in the database for this, but then I'd lose all the array functionalities that prisma provide

Until it gets implemented, I think Json is the only solution.


Macchiato Theme

Hello kind humans, would you able to add catppuccin machiatto to the list?

Thank you for your input. I will add themes for Latte, Frappe, and Macchaito. It's just a matter of changing the color values.

@wrinkledeth I've added themes for Latte, Frappe, and Macchiato. Please let me know if they are acceptable, and then I will close this issue. :-)

@hyperreal64  This is perfect, thanks so much!
Have a wonderful day :)


IW acronym

I guess IW means Information Worker , but don't think all users know this acronym.

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

ID: 28550025-fde2-9a83-a826-f7e3ccb313db
Version Independent ID: 2a304432-b10c-220f-3453-e19e7a155c67
Content: Manage the IW trial offer for Microsoft Teams
Content Source: Teams/iw-trial-teams.md
Service: msteams
Product: unspecified
GitHub Login: @chuckedmonson
Microsoft Alias: chucked

Thank you for the feedback. It has been clarified.


Validation not correct when swapping Mainnet to ETH

The validation for the swapgate code is currently balance - fee === able to swap.
But it should be balance - fee - 0.1 === able to swap
Example
If the user has 1000 mainnet GNY and the fee is set to 500 GNY he could fill out in the textbox 500. This wouldn't work because 500 + 500 + 0.1 (transaction fee) will be over the 1000 GNY balance.

Fixed in c6aff081b2cbae30660fd40845cf47d4b1d0ab1a


issue when confirming license agreements

Describe the bug
When upgrading, the script asks for accepting all license agreements. However, I didnt know how to accept them other than an educated guess, as the "[Y/N]" only popped up AFTER i entered a Y.
To Reproduce
Steps to reproduce the behavior:
upgrade something with license agreements to accept
Expected behavior
i expected the "[Y/N]" to be before entering input, otherwise it's hardly a useful hint.
Screenshots
sorry, i had to restart immediately, so dont have screenies.

this was fixed


You may need an additional loader to handle the result of these loaders.

Module parse failed: Unexpected token (96:16)
File was processed with these loaders:
 * ./node_modules/cache-loader/dist/cjs.js
 * ./node_modules/vue-loader-v16/dist/index.js
You may need an additional loader to handle the result of these loaders.
|     console.log('props', props)
|     const fileExt = ref<string>(props.fileExt)
>     const qrCode: QRCodeStyling = new VQRCodeStyling({
|       width: props.width,
|       height: props.height,

 @ ./node_modules/vue3-qr-code-styling/src/vue3-qr-code-styling.vue?vue&type=script&lang=ts 1:0-165 1:0-165 1:166-320 1:166-320
 @ ./node_modules/vue3-qr-code-styling/src/vue3-qr-code-styling.vue
 @ ./node_modules/cache-loader/dist/cjs.js??ref--0-0!./node_modules/vue-loader-v16/dist??ref--0-1!./src/components/common/qrcode/QRCode.vue?vue&type=script&lang=ts
 @ ./src/components/common/qrcode/QRCode.vue?vue&type=script&lang=ts
 @ ./src/components/common/qrcode/QRCode.vue
 @ ./node_modules/cache-loader/dist/cjs.js??ref--12-0!./node_modules/babel-loader/lib!./node_modules/cache-loader/dist/cjs.js??ref--0-0!./node_modules/vue-loader-v16/dist??ref--0-1!./src/views/Home.vue?vue&type=script&lang=js
 @ ./src/views/Home.vue?vue&type=script&lang=js
 @ ./src/views/Home.vue
 @ ./src/router/index.js
 @ ./src/main.js
 @ multi (webpack)-dev-server/client?http://<IP_ADDRESS>:8080&sockPath=/sockjs-node (webpack)/hot/dev-server.js ./src/main.js

Hi, weird I get the same issue.
Using Laravel 8, Vue 3
Did the npm install as per doc.
Created a component called VueQr3 and added example code snippet.
As soon as I import this new component into any Vue, it gives the same error above.
Do you need to adjust the webpack config to handle?
Any suggestions?
Might be nice to be able to simply (something like):
import Vue3QrCode from 'vue3-qr-code-styling'

then

<vue3-qr-code ...options... >
</vue3-qr-code>

Cheers.

download the component directly you can't use npm or yarn for now very busy with other project i will add the package to npm soon once am less busy
 components: {
    VueQr3: defineAsyncComponent(() =>
      Promise.resolve(import('components/vue3-qr-code-styling'))
    )
  },

or
import Vue3QrCode from 'components/vue3-qr-code-styling'

try this npm install vue3-qr-code-styling

working

<VueQr3
              :width="800"
              :height="800"
            ....
            ></VueQr3>

if any other issue you can reopen

Hey, thanks for your help.
Ok, I see you just pushed a new version. It just updated "vue3-qr-code-styling": "^1.4.5".
So, still getting the same issue when I create a component with the code snippet.
Do I still need to copy the src folder into a local components folder?
Not sure why it doesn't like this line const qrCode: QRCodeStyling = new VQRCodeStyling({
Cheers.

if you downloaded the package create folder named vue3-qr-code-styling in your  component folder then copy all content in dowloaded src to components/vue3-qr-code-styling done it work
components: {
    VueQr3: defineAsyncComponent(() =>
      Promise.resolve(import('components/vue3-qr-code-styling'))
    )
  }

``

if error share your code and send a screenshot of vue3-qr-code-styling in your component

I did it as you explain here, but still, not working... i'm using Vue3 with script setup with typescript, could that be the reason? any incompatibility?


Javascript methods like flatmap and flat are not supported

How do we get the support for enhanced javascript functions like flatmap , flat added to html unit ?

This simplest way to get this is to make a pull request for rhino (https://github.com/mozilla/rhino) that adds support for this.
Second option is to motivate me to add the implementation :-); but keep in mind i'm doing all the work on HtmlUnit/Rhino in my spare time and i have a real job and a real family.

Ok I will try and add the support in my spare time. Any guides to get started on that will be very helpful.

Sounds great, Rhino is a monster but adding this functions might be not that tricky.
You might start by lucking at these commits:

https://github.com/mozilla/rhino/commit/2d89acb883576c0acc62ab4300da6c0029dc565d
https://github.com/mozilla/rhino/commit/833133ca7df1fb6ba85b4d8e47b62e63216f183d
to get a first idea how to add new functions to a class (prototype).

And regarding the implementation of flatmap and flat you might find many hints inside the existing functions like map().
And finally you have to add testcases for your new implementations. There might be something like

https://github.com/mozilla/rhino/commit/af30e136a93fe02ece892e92fffaf09c84b509ad a good starting point.

The rest is the usual game- provide your PR for Rhino and someone will jump in and add some comments.

Cool. I will do this work from this personal account. Thanks. If i need help where do I ask?

Am 31. Dezember 2019 19:16:46 MEZ schrieb Som Shankar Bhattacharyya<EMAIL_ADDRESS>
Cool. I will do this work from this personal account. Thanks. If i need
help where do I ask?
--
You are receiving this because you commented.
Reply to this email directly or view it on GitHub:
https://github.com/HtmlUnit/htmlunit/issues/113#issuecomment-569969139

Great, please open an issue for Rhino - if you need some help we can use the issue for discussing.

Looks like there are already testcases for these functions (currently disabled) in rhino.

Awesome ! Thanks for checking up on that. I will get cracking the moment I return from vacation.


Cannot click on a plugin to add to a aplaceholder

I have configured content placeholder
    CMS_PLACEHOLDER_CONF = {
        'content': {
            'plugins': [
                'TextPlugin', 'PicturePlugin', 'IframePlugin',
                'LinkPlugin', 'PubDatePlugin'
            ],
            'text_only_plugins': ['LinkPlugin'],
            'name': gettext('Content'),
            'language_fallback': True,
        },
    }

And there is an issue with placeholder editing frontend. Look at screenshot. There are 5 plugins in the list, but the latest is hidden under scroll hint block (but there is not scroll in that control). So user cannot click on a latest plugin in a list. If I will add more plugins the scrolls appears and I can choose a plugin, but not in this case.

Hi, thanks for reporting this. Can you tell us which browser and OS?

Google Chrome 40.0.2214.115, Mac OS X

Duplicate of #3456
This has been plaguing us for a very long time. I think this can be solved only by rethinking the structure mode UI (which is on our todo list)

I think its critical bug.
I am new to django-cms
I spend 2 hours, trying to guess how to add text to placeholder.
Text plugin always hidden by "friendly" scroll button for me.

Closed by #4014

@mkoistinen this still appears to be an issue, at least with Chrome. The threshold should be 247 instead of 245 to account for a 1px top and bottom border.


visual studio code 1.2 on mac is getting quite slow

VSCode Version:1.2
OS Version: mac, 10.11

After upgrading to 1.2, it becomes extremely slow. For example, it could take 1~2 seconds to open a 10 lines of files on 2015 MBP.

after uninstall all extensions and then reinstall them again, it works fine now.


Support seeking in bounded/adaptive storage

src/storage/bounded.rs:

//! Because the buffer will never resize, it's important to ensure the buffer is large enough to
//! hold all of the data you will need at once. This needs to account for any seeking that may occur
//! as well as the size of the initial prefetch phase.

Suggest seeking further then the bounded storage's capacity is not possible. My use-case is huge podcast episodes which I want to stream without filling up to much disk space or heap memory.

I think I answered this here https://github.com/aschey/stream-download-rs/discussions/26. This is likely best handled by a different storage implementation.

Closing due to answer above.

As you indicated you build the bounded/adaptive storage for things like internet radio I have build a (still work in progress) streaming solution specifically for (hours long) podcasts.
It provides seeking in bounded storage and bandwidth throttling. I would not recommend it for internet radio, stream-download-rs is way better tested and more stable.
If anyone runs into this issue however it might be worth a look: https://github.com/dvdsk/stream-owl

Awesome, thanks for the heads up! Is that repo private? Seems to 404 for me.

Awesome, thanks for the heads up! Is that repo private? Seems to 404 for me.

good catche! Fixed that :)


[WFRP4e System] adding pay button in tables in journals for items

A suggestion for journal entries with items that player characters can buy. The tables in journals with weapons, trappings etc, could have the value cell be pay buttons. This would make this journals basically shops. the player clicks the pay button and drags the item, done.From: telles007 #0570foundry: 10.291wfrp4e: 6.3.1wfrp4e-altdorf: 2.0.2wfrp4e-archives1: 2.4.1wfrp4e-core: 3.0.4wfrp4e-dotr: 2.3.4wfrp4e-eis: 3.0.1wfrp4e-starter-set: 3.0.2wfrp4e-up-in-arms: 3.0.1Active Modules: CautiousGamemastersPack, advanced-drawing-tools, ambientdoors, autocomplete-whisper, betterroofs, chat-portrait, df-curvy-walls, dice-so-nice, dragupload, find-the-culprit, forge-vtt-shared-compendiums-carreiras-criadas-por-mim, forge-vtt-shared-compendiums-ingredients-feitos-por-mim, forge-vtt-shared-compendiums-landing-page, foundryvtt-simple-calendar, image-hover, lib-df-buttons, lib-wrapper, mels-masterworks, monks-active-tiles, monks-enhanced-journal, monks-little-details, monks-wall-enhancement, notokenanim, popout, remote-highlight-ui, smart-doors, smarttarget, token-action-hud, token-attacher, tokenmagic, wfrp4e-dotr-maps, wfrp4e-gm-toolkit, wfrp4e-ubersreik-maps, forge-vtt

The pay button inside journals is not intended to be used by players, it is for GMs to post a pay prompt to chat.
In this scenario, players should post the item to chat and pay from there.


Error while using update_direct_lake_partition_entity() Function!

In import mode I had spaces in the table names and as lakehouse we have removed them to map it we can use this function update_direct_lake_partition_entity()! so I tried but got the below error!

I have also tried changing the partition names manually in tabular editor but still while I am doing the refresh it says the below error:

Any alternative way to map the lakehouse tables to semantic model!?

It looks like there is a data type issue with one of the columns in one of your tables. Check the data types between the columns in the delta table in the lakehouse and the columns in the table in the semantic model.

Are you still experiencing this issue with the latest version?


Laravel 8 Compatibility

Codecov Report

Merging #2104 into master will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master    #2104   +/-   ##
=========================================
  Coverage     86.90%   86.90%
  Complexity      664      664
=========================================
  Files            33       33
  Lines          1573     1573
=========================================
  Hits           1367     1367
  Misses          206      206

Continue to review full report at Codecov.

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

I need Laravel 8 support too.
I examined Laravel 8 upgrade docs. It seems There's no issue for v8.

Why was this PR closed?

@stefnats for example you can see on this change https://laravel.com/docs/8.x/upgrade#queue-retry-after-method

Why was this PR closed?

I have answered this multiple times. Check my comments over here. This not just a dependency upgrade code changes are required.
Let me remind you, I will push my own PR soon, why you can't just wait?

@Smolevich makes sense, i have not seen this in the upgrade guide
@divine Sorry, i just wanted to learn from mistakes. That's nothing about a hurry. Plenty of other packages are not Laravel 8 ready yet.
Makes sense to not merge in this state.


Summary in TF2.3 wraps unexpected name_scope

When tf.summary is in a condition scope, the tag in tensorboard will wrap unexpected 'cond/'.
For example, when executed the following code, the tag in tensorboard is cond/foo, not foo.
import tensorflow as tf

writer = tf.summary.create_file_writer('log')
step = tf.Variable(0, dtype=tf.int64)

@tf.function
def test():
    if step % 5 == 0:
        with writer.as_default():
            tf.summary.scalar('foo', 0.5, step=step)
            writer.flush()

    step.assign_add(1)

for _ in range(100):
    test()

This bug only shows in TF2.3. Tags are correct in tf below 2.3.

@BlueFisher
I ran the code on tf 2.3 and on tf 2.2 and see that there is no error, please refer tot he gist here.

@Saduf2019
The code has no error, but the tag in tensorboard is cond/foo, not foo. Please refer to this gist

@BlueFisher I think this is intended behavior. The name argument for tf.summary.scalar clearly explains name is prefixed by any active name scopes. Please check here for more details on tf.summary.scalar. Thanks!

name: A name for this summary. The summary tag used for TensorBoard will
be this name prefixed by any active name scopes.

@jvishnuvardhan Thanks a lot


‘…you need to sign in’—misleading action buttons

Some buttons redirect you to pages that redirect you back upon telling you that you should be logged in to access them. This is bad in two ways:

A newcomer has no way of knowing that/if a button is hidden behind a login-wall;
The newcomer is not taken to a page where they can login and follow through to the page that they desire to open.

So I suggest that:

Actions requiring the user to be logged in be greyed out, with an appropriate disclaimer provided. (I think this shouldn’t be that much of a hassle if we can read incoming cookies.)
Upon clicking on such an action, the browser navigate to a sign-on page whence upon successful login/signup, the user is allowed to continue with the desired action. (Most websites implement this with query strings: /some-webpage → /login?continue=/some-action → /some-action.)

Thank you for the feedback!
I will start working on this soon (unless someone else would like to take this).
(2) should be working already (using the Referer header rather than the querystring). Do you know a specific place where it's not working?

It did not work on my computer when I checked. It just took me back to where I started.


Stream processing is creating tasks in the context of the last user that updated a stream filter but it should be the user who created the filter

There is no referential integrity between User and StreamProcessingFilter so all filters will need to be altered to have been created with the 'admin' account and care will need to be taken when deleting users. In v6 this should be catered for when performing integrity checking.

Fixed but leaving for further consideration in v6.0


Cleanup unused ChartRepo

This was both unused and wrong; it was pointing to release-1.1. We
should never use this anyways, as this would mean we are running stuff
that has already been merged rather than what is being tested in the PR.
Please provide a description for what this PR is for.
And to help us figure out who should review this PR, please
put an X in all the areas that this PR affects.
[ ] Configuration Infrastructure
[ ] Docs
[ ] Installation
[ ] Networking
[ ] Performance and Scalability
[ ] Policies and Telemetry
[ ] Security
[ ] Test and Release
[ ] User Experience
[ ] Developer Infrastructure

@howardjohn: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

integ-security-k8s-tests_istio
9da0e4518d93de62aa3735c2e10df64cdc8cecf6
link
/test integ-security-k8s-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/retest

@howardjohn: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

integ-security-k8s-tests_istio
9da0e4518d93de62aa3735c2e10df64cdc8cecf6
link
/test integ-security-k8s-tests_istio

gencheck_istio
9da0e4518d93de62aa3735c2e10df64cdc8cecf6
link
/test gencheck_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/retest


fix: HOME env var not exist on Windows

Close: #98

Gonna cut a new release, will also respond in the corresponding issue.


double quote in .merlin is not right

this line in .merlin:
FLG -ppx "_build/.ppx/4.04.0/ppx_type_conv+ppx_fields_conv/ppx.exe --as-ppx"
cause error when open .ml file.
"main.ml" 48L, 1028Csh: 1: Syntax error: Unterminated quoted string

(merlin) backtrace:
(merlin) project flags: unknown flag: --as-ppx"
Error detected while processing function merlin#Register..merlin#LoadProject:
line    2:
Traceback (most recent call last):
  File "<string>", line 1, in <module>
  File "/home/hwu/.opam/4.04.0/share/merlin/vim/autoload/merlin.py", line 740, in setup_merlin
    failures = command("project","get")
  File "/home/hwu/.opam/4.04.0/share/merlin/vim/autoload/merlin.py", line 201, in command
    return merlin_process().command(context(cmd))
  File "/home/hwu/.opam/4.04.0/share/merlin/vim/autoload/merlin.py", line 168, in command
    raise Error(value)
merlin.Error: {u'message': u'Error while running external preprocessor\nCommand line: "_build/.ppx/4.04.0/ppx_type_conv+ppx_fields_conv/ppx.exe \
'/tmp/camlppx8a78b1\' \'/tmp/camlppx16180d\' 1>/dev/null 2>/dev/null', u'valid': True, u'type': u'type', u'sub': []}

change double quote to single quote, the problem disappear.
FLG -ppx '_build/.ppx/4.04.0/ppx_type_conv+ppx_fields_conv/ppx.exe --as-ppx'

@def-lkb Should .merlin accept double quotes here?

This should work with dev version, not the released one yet.
Try with:
opam pin add merlin --dev-repo

@let-def, moving forward, is the recommendation to use single or double quotes?

Any, I am trying to follow shell conventions, so single quote for exact string, double quote might allow some expansion in the future (right now it is also just quoting).
(The change in git version is that quoting is handled, previously merlin wasn't really trying to interpret the string.)

Does it make to just stick to single quotes for now then? At least until jbuilder needs to generate .merlin files with interpolation in the quotes.

I don't think the released merlin behave properly even with single quotes, and the git version works with both. So either should be fine :).

I don't think the released merlin behave properly even with single quotes

Confirmed.
(I'm guilty)

I think this is still an issue - not sure if it's a jbuilder or merlin issue though. I see the same issue as the linked mirage/charrua-core#62 issue discusses but in a local project.  If I manually remove the --cookie '"library=foo"' portion of the FLG line in .merlin then merlin works fine during editing.


Combined with cefnet, after the focus leaves the program, it returns to the program page, and the input method switch fails in the input box.

Sorry, my English is terrible；
Combined with cefnet, after the focus leaves the program, it returns to the program page, and the input method switch fails in the input box.
结合cefnet ，第1步焦点进入页面输入框，可以切换输入法，能输入中文，第2步焦点离开页面或者该程序后，第3步再返回页面里，在输入框中输入法切换失败，不能输入中文了。
Look at the code first；

Let me try to translate.
Step 1: input in webview, IME works.
Step 2: focus on other controls or outside the window.
Step 3: focus back on this page, IME cannot switch between English and Chinese.
Can you share this project with us via Github repo?
你能把项目上传到Github repo分享出来吗？

I'm not sure if this is an issue with Avalonia 0.10.x, as IME was not working well on those versions. I tried to upgrade to V11 but build failed as CefNet is not compatible with Avalonia preivew yet.

@rabbitism
Sorry, Thank you for waiting. I finally uploaded it.
https://github.com/lovexionger/xiongavatest.git


make toast shorter

Why
Too long, want short.
Test Plan
Goes away faster

2.5 much long, 1.75 much short, 2 gud.
(Everyone can give an opinion i think once we get on device, but this feels a lot better. Less blocking of buttons like back)


Interested in status of this project

Hi there,
I'm interested in knowing the status of this project. Wanting to connect to a Firebase DB and came across your project. Trying to decide if I roll my own or adopt your framework which looks good and provides the basic functionality I'm after.
Thanks in advance,
PC

Hallo,
Great that you found some interest, It will have a first version completed within 10 days from now. In the meantime you could also just take the index.R and copy the code you need for your firebase connection.


Chrome scrolls to reveal full map when user clicks a zoom button

[x] I'm reporting a bug, not asking for help
[x] I've looked at the documentation to make sure the behaviour is documented and expected
[x] I'm sure this is a Leaflet Draw code issue, not an issue with my own code nor with the framework I'm using (Cordova, Ionic, Angular, React…)
[x] I've searched through the issues to make sure it's not yet reported

How to reproduce

Leaflet version I'm using: 1.3.1
Leaflet Draw version I'm using: 1.0.0
Browser (with version) I'm using: Chrome 63.0.323.132
OS/Platform (with version) I'm using: MacOS 10.13.2
Draw a map in a container with a height higher than the browsers window.
Click any draw or edit toolbar buttom.

What behaviour I'm expecting and which behaviour I'm seeing
This error is reported by https://github.com/Leaflet/Leaflet/issues/4125 and diagnosed as a Chrome issue when using map.getContainer().focus() The work-around is to monkey patch _refocusOnMap. This works for leaflet but does not work for leaflet.draw because leaflet.draw does not use leaflet's _refocusOnMap but instead directly used map.getContainer().focus() which is not monkey-patchable.
Minimal example reproducing the issue

[x] this example is as simple as possible
[x] this example does not rely on any third party code

See issue https://github.com/Leaflet/Leaflet/issues/4125

Having the same issue here. Any news on this? Thanks

using map.getContainer().focus(); it maybe scroll to the center of the window


Custom Image Marker with actual size

Case 1 :
I try to load the custom image in marker with original height X width. Image was coming dynamically
from server so i don't know the size of it.
  <Image                       style={{flex:1, height: undefined, width: undefined}}                       resizeMode={'contain'}                       source={{ uri: 'http://www.xyz.com/images/' + this.state.IconPath }} />
it shows nothing on map but if i set the height to any random number then it start appearing over map.
Case 2 :
If i set image property of Marker then it display a tiny image on map.
 <Marker
                    coordinate={this.state.region}
                    title={this.state.title}
                    description={this.state.title}
                     **image**={{ uri: 'http://www.xyz.com/images/' + this.state.IconPath }}
                    rotation={this.state.rotation}>
                  </Marker>

I have to load thousand of marker on map with different size of images, so how do i solve the issue?

Thank

It's ok that it does not work like you expect, because it's not web. You must set height and width. And you must have 1x, 2x and 3x images to fit all screen densities.
And actually it's a duplicate: #1870 #1422


raft-canonical: 0.9.6 -> 0.9.15, dqlite: 1.0.0 -> 1.2.2, lxd: 3.18 -> 3.21

Motivation for this change
Update.
Things done

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

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[x] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

Notify maintainers
cc @

@GrahamcOfBorg build raft-canonical

@GrahamcOfBorg build lxd dqlite raft-canonical

@GrahamcOfBorg build dqlite
@GrahamcOfBorg build raft-canonical

@GrahamcOfBorg lxd

What's the status? I see ofborg reported a failure.

@GrahamcOfBorg build raft-canonical dqlite lxd
@FRidh Probably finished, let me run a last test

@GrahamcOfBorg build raft-canonical dqlite lxd

@GrahamcOfBorg build lxd

@wucke13 thank you for working on this, please ping me if a PR gets stuck


Python has stopped working (Spyder crash)

Description of your problem
Spyder crashes when I try to open a file. The description is copied and pasted as follows:
Problem signature:
Problem Event Name:	APPCRASH
Application Name:	pythonw.exe
Application Version:	<IP_ADDRESS>3
Application Timestamp:	585d65af
Fault Module Name:	ntdll.dll
Fault Module Version:	6.1.7601.23572
Fault Module Timestamp:	57fd0651
Exception Code:	c0000005
Exception Offset:<PHONE_NUMBER>04f23c
OS Version:	6.1.76<IP_ADDRESS>.256.48
Locale ID:	7177
Additional Information 1:	a745
Additional Information 2:	a745a5fe3d4bc55416c4959c3f9fee16
Additional Information 3:	3f00
Additional Information 4:	3f008431c988f5c1d2cbbb05cf3bf3b4
What steps will reproduce the problem?

When I open Spyder after installing. Then attempting to open an existing program

What is the expected output? What do you see instead?
A program to run.
Please provide any additional information below
The got the Spyder IDE through Anaconda.
Versions and main components

Spyder Version: 3.1.2
Python Version: 3.6
Operating system: Windows 7 professional

Dependencies
Please go to the menu entry Help > Optional Dependencies (or
Help > Dependencies), press the button Copy to clipboard
and paste the contents below:
jedi >=0.8.1     :  0.9.0 (OK)
matplotlib >=1.0 :  2.0.0 (OK)
nbconvert >=4.0  :  4.2.0 (OK)
numpy >=1.7      :  1.11.3 (OK)
pandas >=0.13.1  :  0.19.2 (OK)
pep8 >=0.6       :  1.7.0 (OK)
pyflakes >=0.6.0 :  1.5.0 (OK)
pygments >=2.0   :  2.1.3 (OK)
pylint >=0.25    :  1.6.4 (OK)
qtconsole >=4.2.0:  4.2.1 (OK)
rope >=0.9.4     :  0.9.4-1 (OK)
sphinx >=0.6.6   :  1.5.1 (OK)
sympy >=0.7.3    :  1.0 (OK)

This is a duplicate of issue #3903, so I'm closing it.
Unfortunately, we don't know how to fix it, sorry.

Select compatibility in spyder‘s properties and check Run spyder as administrator can solve the problem.


Annotation files for two datasets: NPC and thymic

Hello,
I am interested in the analysis of the clustering analysis. 15 Organ dataset was used in this notebook (https://github.com/StatBiomed/CamoTSS/blob/main/notebook/Fig2E.ipynb) and I noticed that its cluster annotation file was provided in your GitHub
(https://github.com/StatBiomed/CamoTSS/blob/main/notebook/cell_type_annotation/15organ_cell_annotation.txt).
If I am also interested in clustering analysis for NPC and thymic datasets, how should I get similar annotation files?
Thanks,
Allen

Hi,
All of the annotation file were located here. The thymus datasets was the development dataset.
https://github.com/StatBiomed/CamoTSS/tree/main/notebook/cell_type_annotation
Best
Ruiyan


Golangci lint runs twice in github action

There is no need to run twice there.

Fixed by a908775a6cbb3057569ca895fcc278b586914192


Package Information in About window needs to be updated

Open up About window.
Details need to be filled in.

Updated string for help about and VSIX gallery entry.  Commit on 10 Feb 2015


Eliminate redundancy of literate programming material here and into Intro lesson

from @kcranston (switching issue from org1):
From first Duke workshop: In the literate programming section, much of the content in the slides feels redundant after the morning, or perhaps just needs trimming / reorganization (remember the things - Rmd, knitr, HTML generation - we saw this morning? here is some more detail…).
Likely that this issue will get solved when we split this repo into two (issue #24) and remove the literate programming from the Intro lesson (Reproducible-Science-Curriculum/rr-intro#10 ).

This has now been split, resolution is just pending removal of literate programming content from intro.


datect secret audit doesnt work

After I generated baseline file i got around 100 entries in file. I know that some of them are secrets.
When I run audit it produce error, cant figure out what is the problem.
I used latest version 1.3.0
".circleci/config.yml": [
      {
        "type": "Hex High Entropy String",
        "filename": ".circleci/config.yml",
        "hashed_secret": "963d6519f46177770aad8c3ffa8713824daff92c",
        "is_verified": false,
        "line_number": 141
      },

Secret:      1 of 62
Filename:    .circleci/config.yml
Secret Type: Hex High Entropy String
----------
ERROR: Secret not found on line 5!
Try recreating your baseline to fix this issue.

Try recreating your baseline to fix this issue.

Did this fix the issue?

Try recreating your baseline to fix this issue.

Did this fix the issue?
no)

@garry-t Can you post a snippet of the .circleci/config.yml file that is failing?

@garry-t Can you post a snippet of the .circleci/config.yml file that is failing?

sorry I cant, there are a lot of confidential info.
But string 141 it is slack webhook url https://hooks.slack.com/services/[A-Z][0-9]/[A-Z][0-9]/[0-9]

@garry-t could you then provide us with a mock .circleci/config.yml (i.e. without any of your info) that we could still use to reproduce the issue?

@garry-t could you then provide us with a mock .circleci/config.yml (i.e. without any of your info) that we could still use to reproduce the issue?

let me try

Mocking file on 1000 strings hard job. Not sure how to properly mock it and preserve structure

@garry-t I am mainly curious of the structure around the failing secret. I do not need a full mocked file.

@jpdakran
docker-ruby-config: &docker-ruby-config
    image: cimg/ruby:2.5.8
    environment:
      AWS_IVS_REGION: eu-west-1
      BUNDLE_JOBS: 3
      BUNDLE_RETRY: 3
      BUNDLE_PATH: vendor/bundle
      PGHOST: <IP_ADDRESS>
      PGUSER: circleci-demo-ruby
      RAILS_ENV: test
      DEVISE_SECRET_KEY: REAL_SECRET_HERE
      DEVISE_JWT_SECRET_KEY: REAL_SECRET_HERE
      DEVISE_JWT_PRIVATE_KEY:  REAL_SECRET_HERE
      SECRET_KEY_BASE: REAL_SECRET_HERE
      LOCKBOX_MASTER_KEY: REAL_SECRET_HERE
      HOST: test.host
      PROTOCOL: http
      CDN_HOST: 'localhost:3000'
      CDN_PROTOCOL: 'http'
      E_API_SECRET:  STRING
      ACTIVE_CAMPAIGN_API_KEY:  API_TOKEN
      ACTIVE_CAMPAIGN_ENDPOINT: HTTPS
      MAIL_HOST: smtp
      LW_DIRECTKIT: https
      LW_BACKOFFICE: https:
      LW_WEBKIT: https:
      LW_LOGIN: login
      LW_PASS: pass
      LW_PDV: string
      API_PROTOCOL: http
      API_HOST: api.test.host
      SLACK_PAYMENT_TOKEN: 'https://hooks.slack.com/services/REAL_ID/REAL_ID/NUMBERS’
      M_PAY_CLIENT_ID: string
      M_PAY_API_KEY: API_KEY
      M_PAY_PREPRODUCTION: 'true'
      M_PAY_SYSTEM_USER: string
      M_PAY_USD_CREDIT_WALLET: 'CREDIT_USD'
      M_PAY_EUR_CREDIT_WALLET: 'CREDIT_EUR'
      M_PAY_EUR_SYSTEM_WALLET: SOME_ID
      M_PAY_EUR_FEE_WALLET: SOME_ID
      M_PAY_USD_SYSTEM_WALLET: SOME_ID
      M_PAY_USD_FEE_WALLET: SOME_ID
      EE_CONNECT_ACCOUNT_ID: ACC_ID
      EE_CONNECT_SECRET: REAL_SECRET
      EE_CONECT_PUBLISHABLE: string
      PDF_PATH: system_path

141 string it is this one SLACK_PAYMENT_TOKEN: 'https://hooks.slack.com/services/REAL_ID/REAL_ID/NUMBERS’

@garry-t Ok I was unable to reproduce it. Looking at the original query - is it failing on the secret at line 5 or line 141 of .circleci/config.yml? How many secrets are in that file?

@garry-t Ok I was unable to reproduce it. Looking at the original query - is it failing on the secret at line 5 or line 141 of .circleci/config.yml? How many secrets are in that file?

I added file to exclude.
Originally I gave only part with env variables where is problematic string. all other staff above and below section omitted.
I think issue can be closed sine workaround found.


Field' object has no attribute 'is_'

.is_ method does not seems to be supported now

Please give more detailed information and an example that gives you this error.

from pypika import Table, Query

customers = Table('customers')

sql = (
    Query
    .from_(customers)
    .select('*')
    .where(customers.first_time.is_(True))
)

print(sql.get_sql()

received this error after running
'Field' object has no attribute 'is_'

@cheunhong What's your real purpose? if you want to compare value, you can use ==, or if you want to use IS NULL syntax you can use isnull()

@lntuition not much really, was just thinking to flag to you guys on this.  as i saw another closed issue which showcase the usage of this.


Bead class has redundant methods

Why does the bead class have vec& getVel() and vec& Vel() and vec& getPos() and vec& Pos(). Why on earth are there two methods the getVel checks a bool to see if the value is set or not but the Vel method doesn't... I don't understand what the reasoning is for this @ruehle?

getPos ist const, so you can use it even if bead is passed as const ref. Pos is not which allows you to change it. There is actually no reason to call them differently, but you should have 2 methods, one with const and one without.
Getvel is checked as there is not always a velocity. This depends on the trajectory. However we assume that there always must be a position.
Hope this answers your questions.

That the vel method doesn't check could be an error. Also as vel is read/write, it is not so obvious what to do. I don't really remember. Sorry

Ok, thanks for the clarification. The lack of the asserts in the Vel(), Pos(), W() U() and F() methods seems to be an error then.

Sorry for my exasperation,

It seems though that passing out a reference might completely defeat the purpose of having a boolean indicating that the instance has a defined position, velocity etc...

@junghans do you have any input? I would not normally care but seeing that I want to derive some classes off of this one I would like to be sure the bead class is air tight.


Add vsphere CPI and CSI charts

@rayandas @PennyScissors
This is how I believe the chart should be added.
Note: I didn't test the deployment of the chart itself. I only added the chart contents and ensured that it was able to be pulled from Apps and Marketplace when added as an indpendent repo.

As @PennyScissors said, this repo should be similar like harvester charts repo. They use a release workflow with chart-releaser-action which make the tarballs and release them and generate index.yaml. There are three branches. main contains the charts. When we want to release we have to edit the charts which are in main and push/PR to release branch. The CI will build and release the tarballs and generate the index.yaml and store it on gh-pages branch.
We can also do the same. All the things are in my repo https://github.com/rayandas/vsphere-charts-1 but I had a question which I asked on team-3 channel.


pyodbc v4-virtuoso build error

On Windows 7 with Microsoft Visual Studio 2015 and Python 3.5.1, I have these build errors out of the box:
src\virtuoso.cpp(17): error C3861: 'strncasecmp': identifier not found
src\virtuoso.cpp(54): error C3861: 'wcsncasecmp': identifier not found
error: command 'C:\\Program Files (x86)\\Microsoft Visual Studio 14.0\\VC\\BIN\\amd64\\cl.exe' failed with exit status 2

I observe that in src\pyodbc.h, there is a macro defined for strcasecmp when _MSC_VER is not defined.  Going the other way to avoid impacts on upstream, I added the following to src\virtuoso.h:
#ifdef _MSC_VER
#  define strcasecmp _stricmp
#  define strncasecmp _strnicmp
#endif

This leaves wcsncasecmp - I cannot figure out easily from Dr. Google how this should be solved, especially as wchar_t is only 16-bits on windows.   I will do my development in vagrant or on a server, and I suspect this will work, but I will want eventually to be able to do this from Windows as well - most of the developers here work from Windows, and I'm an exception in being able to use vagrant (and also dabble in C/C++ even though I don't write in it any longer).

On Windows, maybe you can use '_strnicmp' and '_wcsnicmp', which may work.

@xjtusedy , I thinkalready succeeded in building it by changing the tests on MSC_VER to understand more modern versions of Visual C++, this was just a friendly bug report.  Users should not have to modify source code to build.


Add Tinkerbell taints and labels test

Description of changes:
Add e2e test for worker node groups with taints and labels for tinkerbell provider on Ubuntu
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

/approved
/lgtm

/approve

/unhold

/cherry-pick release-0.10


ITMS-90809: Deprecated API Usage - New apps that use UIWebView are no longer accepted. Instead, use WKWebView for improved security and reliability.

ITMS-90809: Deprecated API Usage - New apps that use UIWebView are no longer accepted. Instead, use WKWebView for improved security and reliability. Learn more (https://developer.apple.com/documentation/uikit/uiwebview).
此插件使用了UIWebView 吗?

同问

没有


Handling of byte slice/array and time.Time

If I create a new type out of []byte (type ByteArr []byte), the postgres dialect (specifically, isByteArrayOrSlice) will recognize it as a byte slice, and will use the postgres bytea type. The others will not.
Here's an example comparing the two methods:
http://play.golang.org/p/PyBc8eGOF6
The Postgres dialect also supports byte arrays (the go type), the others do not.
time.Time has a similar problem, you can't use a custom type that derives from time.Time.
The other types all use Kind, so custom types derived from them work.
You could use reflect.Type.ConvertibleTo to support custom types derived from time.Time:
http://play.golang.org/p/kzXczO4xef
The same would work for []byte, but isByteArrayOrSlice is better, because it supports byte arrays too.
An example of where this would be useful is json.RawMessage. I can't use it as the field's type, I have to use []byte, but that means I can't just take the model and convert it to json, because the byte slice will become a base64 encoded string. (I know I can specify the sql type manually in the tag, but if I use the mysql type longblob, it won't work on postgres, and vice versa).

Looks like this isn't as easy as I thought. I implemented it here: https://github.com/nkovacs/gorm/tree/byte-time-handling. Migration works, and reading works as well, but inserting doesn't, because the sql package or the driver isn't smart enough to convert the go time.Time or json.RawMessage to the appropriate sql type, and I get an unsupported type error from Exec.

Write a scanner, valuer for this.

Gorm still won't be able to figure out the type during migration.

If it is a scanner, gorm will use the scanner's first field's type to generate the SQL type.
or you could just use the sql tag to set the type.

We need a FAQ out of all these closed issues...
and need a couple days to create new website.. (time.time.time)
seems to be same question reappearing....

Thanks, I'll try it out.

@pedromorgan most closed issues, not really problems, and the answer could be find in the documents or past issues...
I think we should be more strict to ask a runnable script for issues, only people who want to contribute his time by writing the script worth our time, it is pretty time costing for me to setup testing env for every issues.


Update ids in json_metadata on import

Currently import fixes ids for position_json but not json_metadata. As the results default_filters for example are broken.

Are default_filters those applied by the filter widgets in the dashboard? And when you refer to the ids fixing on position_json, is this something that was done on our side only or included upstream? Do you have a link to the related PR/issue?

And when you refer to the ids fixing on position_json, is this something that was done on our side only or included upstream?

it was always in upstream: https://github.com/src-d/sourced-ui/blob/master/superset/superset/models/core.py#L550

Are default_filters those applied by the filter widgets in the dashboard?

those filters applied to the filter widget and to all charts in the dashboard. (unless they are in filter_immune_slices or filter_immune_slice_fields which we also don't update but should) Ref: http://superset.apache.org/faq.html#how-to-add-dynamic-filters-to-a-dashboard

thanks a lot!


[Bug] line chart markPoint not display

In my vue file, create line chart. but display in the echarts demo project. May I miss some markPoint options?
Version
5.0.1
Link to Minimal Reproduction
https://echarts.apache.org/examples/zh/editor.html?code=PYBwLglsB2AEC8sDeAoWsAeBBDEDOAXMmurGAJ4gCmRA5AMYCGYVA5sAE7m0A0J6AE2aMiAbVoBZGL1i0AKgFcqM2gHUqAlXIAWClQDEOEFQGVmphdFoBdEgF8-6cjnxFUpMpRqyAbowA2SrT2jrB4VEZUhLCi_MQegsJiAEwArAAMPLAAjACcmbDJyQAsWWkFAMwVqVnZxQDsZRXp1qEeFNR0_hDQym2keAC2wMBg2kRgHEr96IOMHADWAArAPWBucR545IMARsD-RABEID1HWQD0F4DgFoAXNoDePoDR6igAkNt7ByYQAF7eGbCXG6ABujAOSagHgdV7vfb-ADyADM4eF1jECrQALTZdIAUhsM1I_kYuyoh3iCQ89AOnDoAGIEXDgmT0A5NqQhGARDEkBTOAIxNlavlWrA_IFvHl0g5kNyOLyYhUyhkhSKlERypKuSMZWIajkGkqAirdfU7LYycy4nYSLY7ABuIA
Steps to Reproduce

in vue file, create lint chart. but display in the echarts demo project.
chart option:

option = {
  xAxis: {
    type: 'category',
    data: ['Mon', 'Tue', 'Wed', 'Thu', 'Fri', 'Sat', 'Sun']
  },
  yAxis: {
    type: 'value'
  },
  series: [
    {
      data: [250, 190, 224, 250, 335, 147, 230],
      type: 'line',
      smooth: true,
      markPoint: {
        symbol: "pin", //标注类型
	symbolSize: 50 , //标记大小
	symbolOffset: [0, '-10%'],
        label: {
          color: '#fff'
        },
        data: [{coord: [1, 190], value: 190}, {coord: [3, 250], value: 250}, {coord: [5, 147], value: 147}]
      },

    }
  ]
};

Current Behavior
markPoint not display in the chart
Expected Behavior
markPoint should display in the chart
Environment
- OS: macOS Monterey
- Browser: Chrome 111.0.5563.146
- Framework: Vue@3

Any additional comments?

https://echarts.apache.org/en/faq.html#others
Pleas refer this faq

https://echarts.apache.org/en/faq.html#others Pleas refer this faq

this faq list can't help. The issue of my case is the line chart can display, but the markPoint doesn't display. any insight? thanks.

Do you mean the markPoint works without vue but doesn't work in vue environment? I think you should first make sure in the vue enironment, ECharts package is imported as import * as echarts from 'echarts' rather than partially imported like import xxx from 'echarts/lib/xxx'.

If you are using on-demand import, please don't forget to import and register the components that you need. This might be an issue that needs to reproduce in a bundler environment. If not, please show us your code, and don't let us guess what it is.
import * as echarts from 'echarts/core';

import {
  MarkPointComponent
} from 'echarts/components';

echarts.use(MarkPointComponent);

import * as echarts from 'echarts/core';

Yes, This helps to solve my issue. I didn't import MarkPointComponent. Thanks

MarkPointComponent

Thanks, after import MarkPointComponent, it works.


Enabling formatter on/off tags

Allows overriding Eclipse formatter in order to comply with the
guidelines, when Eclipse formatter doesn't work.

This would fix #27, which I initially reported on Google Code before the project was transitioned to GitHub.

+1 for this PR, it would be great to have it merged

@eglaysher, can this PR get some love? Should be an easy one to review :)


feat: add new container flags

Related issue
Closes https://github.com/kyverno/website/issues/987.

Proposed Changes
Add two new container flags to customize certificate secret names.

Checklist

[x] I have read the contributing guidelines.
[x] I have inspected the website preview for accuracy.
[x] I have signed off my issue.

Flags are alphabetically ordered.

I thought it was within new flags.
How should we change the flag indexes? By manual?

The index numbers are manual or auto-added when you add a new line between two indices. You'll need to put them in proper order as they aren't automatically ordered upon rendering.

The index numbers are manual or auto-added when you add a new line between two indices. You'll need to put them in proper order as they aren't automatically ordered upon rendering.

Ok, I've updated it manually. Will probably have another conflict after https://github.com/kyverno/website/pull/992.


Test Python 3.9

Codecov Report

Merging #9 into master will increase coverage by 0.08%.
The diff coverage is 95.08%.

@@            Coverage Diff             @@
##           master       #9      +/-   ##
==========================================
+ Coverage   89.80%   89.89%   +0.08%
==========================================
  Files          29       29
  Lines         942      950       +8
==========================================
+ Hits          846      854       +8
  Misses         96       96

Impacted Files
Coverage Δ

stochopy/optimize/_common.py
85.84% <93.87%> (+0.55%)
:arrow_up:

stochopy/optimize/cmaes/_cmaes.py
90.22% <100.00%> (+0.07%)
:arrow_up:

stochopy/optimize/cpso/_cpso.py
89.90% <100.00%> (+0.09%)
:arrow_up:

stochopy/optimize/de/_de.py
90.52% <100.00%> (+0.10%)
:arrow_up:

stochopy/optimize/vdcma/_vdcma.py
87.65% <100.00%> (+0.07%)
:arrow_up:

Continue to review full report at Codecov.

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


Generate signed vsix for bicep in visual studio

Generate signed vsix for bicep in visual studio

Closing the issue as we are now able to generate signed vsix.


⚠ Bofrost.be has degraded performance

In 12a37f0, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 2558 ms

Resolved: Bofrost.be performance has improved in a9adffd.


Export of variant monomers to Sequence/FASTA doesn't work for presets

Steps to Reproduce

Toggle to Macro - Flex
Load using paste from clipboard way following peptide via sequence: Y

Try to export canvas to Sequence (or FASTA - result is the same)

Actual behavior
System throws exception: Convert error! Error during sequence type recognition(RNA, DNA or Peptide)

Expected behavior
No exception. Export works as intended - sequence string: Y
Versions

Ketcher Version 2.25.0-rc.1 Build at 2024-09-03; 15:31:03
Indigo Toolkit Version 1.24.0-rc.1.0-gb8f099851-wasm32-wasm-clang-19.0.0
Chrome Version 128.0.6613.114 (Official Build) (64-bit)
Win10

Reference issues: https://github.com/epam/Indigo/issues/2015

Verified.

Versions

Ketcher Version 2.25.0-rc.2 Build at 2024-09-14; 10:47:47
Indigo Toolkit Version 1.24.0-rc.2.0-g5f2751d70-wasm32-wasm-clang-19.0.0
Chrome Version 128.0.6613.138 (Official Build) (64-bit)
Win10


Credit

Your note splash code is mostly based on my pull request at Psych repo, can you at least give me credit?

you made a pr? I just glanced at 0.7.3's notesplash code with my own little twist on it.

you did infact make a pr, but I've never seen it before.


[5.2] Make MessageBag constructor consistent with add method

Currently, the behaviour between initialising a new MessageBag and using the add method is inconsistent.
The add method tests for uniqueness (of the key, and the message in the keyed bag), whereas the constructor will accept duplicate messages.
$messageBag = new MessageBag(['messages' => ['first', 'second', 'third', 'third']]);

// $messageBag->getMessages()
// ['messages' => ['first', 'second', 'third', 'third']]

$messageBag = new MessageBag;
$messageBag->add('messages', 'first');
$messageBag->add('messages', 'second');
$messageBag->add('messages', 'third');
$messageBag->add('messages', 'third');

// $messageBag->getMessages()
// ['messages' => ['first', 'second', 'third']

I'm not certain if this is on purpose, so I have opened a PR with a failing test to demonstrate the proposed behaviour, which will amend should the desire exist to match the behaviour between the constructor and public API of the MessageBag class.
It appears this has been around since January, 2013, so this would likely be targeted to next release due to backwards compatibility breakage.

Will need to go to 5.3, yes.

Please discuss proposals in the correct place as per the contribution guidelines.


improve error handling for LP code Bicycle and GB constructors

This short PR aims to improve the error handling for Bicycle and Generalized Bicycle Constructors.
The a_shift and b_shift vectors can have values that even exceed/equal the order of the Group Algebra.
For example, l=127, the following do not throw error:
julia> c = generalized_bicycle_codes([0, 15, 130, 28, 66], [0, 58, 59, 100, 129], 127);

julia>  # or

julia> c = generalized_bicycle_codes([0, 15, 20, 28, 66], [0, 58, 59, 10000, 127], 127);

suppose l=12,
julia> GA = group_algebra(GF(2), abelian_group(12))
Group algebra
  of Z/12
  over prime field of characteristic 2

julia> GA[1].coeffs
12-element Vector{FqFieldElem}:
 1
 0
 0
 0
 0
 0
 0
 0
 0
 0
 0
 0

julia> dim(GA)
12

# Considering above example,  since we consider 0 as well, the range in this case for shift vectors
# is element is [0 - 11].

[x] The code is properly formatted and commented.
[x] Substantial new functionality is documented within the docs.
[x] All new functionality is tested.
[x] All of the automated tests on github pass.

It seems that because of reintroducing type piracy, we can't use @test_throws.
julia> @test_throws generalized_bicycle_codes([0, 15, 20, 28, 200], [0, 58, 59, 100, 20], 127)
ERROR: LoadError: MethodError: no method matching var"@test_throws"(::LineNumberNode, ::Module, ::Expr)

Closest candidates are:
  var"@test_throws"(::LineNumberNode, ::Module, ::Any, ::Any)
   @ Test ~/.julia/juliaup/julia-1.10.4+0.x64.linux.gnu/share/julia/stdlib/v1.10/Test/src/Test.jl:761

in expression starting at REPL[11]:1

So, manual checking after adding error handling:
julia> generalized_bicycle_codes([0, 15, 20, 28, 200], [0, 58, 59, 100, 20], 127)
ERROR: DimensionMismatch: Each element in [0, 15, 20, 28, 200] must ≥ 0 and < the order of Group Algebra 127
Stacktrace:
 [1] generalized_bicycle_codes(a_shifts::Vector{Int64}, b_shifts::Vector{Int64}, l::Int64)
   @ QuantumCliffordHeckeExt ~/Desktop/bounds/QuantumClifford.jl/ext/QuantumCliffordHeckeExt/lifted_product.jl:172
 [2] top-level scope
   @ REPL[7]:1

Based on the papers, I have not seen a shift vector element ≥ the order of GA itself. Hopefully, this is a sensible approach. The PR is ready for review, Thank you!

I think shifts exceeding the order are not a problem and should not be banned.
The underlying groups are cyclic. So, all integers for shifts make sense---shifts that differ by multiples of the group order are naturally equivalent.


Cannot load such file -- jekyll-sitemap (LoadError)

I have installed jekylland all dependencies like described in http://mmistakes.github.io/minimal-mistakes/theme-setup/. Trying to build or serve the content of this page with jekyllI'm getting following  error
kso@lenovo:~/work/projects/priv/sobkowiak.github.io.git$ jekyll serve
WARN: Unresolved specs during Gem::Specification.reset:
      pygments.rb (~> 0.6.0)
      jekyll-watch (~> 1.1)
WARN: Clearing out unresolved specs.
Please report a bug if this causes problems.
Configuration file: /home/kso/work/projects/priv/sobkowiak.github.io.git/_config.yml
/var/lib/gems/2.1.0/gems/jekyll-2.5.3/lib/jekyll/plugin_manager.rb:29:in `require': cannot load such file -- jekyll-sitemap (LoadError)
	from /var/lib/gems/2.1.0/gems/jekyll-2.5.3/lib/jekyll/plugin_manager.rb:29:in `block in require_gems'
	from /var/lib/gems/2.1.0/gems/jekyll-2.5.3/lib/jekyll/plugin_manager.rb:26:in `each'
	from /var/lib/gems/2.1.0/gems/jekyll-2.5.3/lib/jekyll/plugin_manager.rb:26:in `require_gems'
	from /var/lib/gems/2.1.0/gems/jekyll-2.5.3/lib/jekyll/plugin_manager.rb:19:in `conscientious_require'
	from /var/lib/gems/2.1.0/gems/jekyll-2.5.3/lib/jekyll/site.rb:79:in `setup'
	from /var/lib/gems/2.1.0/gems/jekyll-2.5.3/lib/jekyll/site.rb:41:in `initialize'
	from /var/lib/gems/2.1.0/gems/jekyll-2.5.3/lib/jekyll/commands/build.rb:29:in `new'
	from /var/lib/gems/2.1.0/gems/jekyll-2.5.3/lib/jekyll/commands/build.rb:29:in `process'
	from /var/lib/gems/2.1.0/gems/jekyll-2.5.3/lib/jekyll/commands/serve.rb:26:in `block (2 levels) in init_with_program'
	from /var/lib/gems/2.1.0/gems/mercenary-0.3.5/lib/mercenary/command.rb:220:in `call'
	from /var/lib/gems/2.1.0/gems/mercenary-0.3.5/lib/mercenary/command.rb:220:in `block in execute'
	from /var/lib/gems/2.1.0/gems/mercenary-0.3.5/lib/mercenary/command.rb:220:in `each'
	from /var/lib/gems/2.1.0/gems/mercenary-0.3.5/lib/mercenary/command.rb:220:in `execute'
	from /var/lib/gems/2.1.0/gems/mercenary-0.3.5/lib/mercenary/program.rb:42:in `go'
	from /var/lib/gems/2.1.0/gems/mercenary-0.3.5/lib/mercenary.rb:19:in `program'
	from /var/lib/gems/2.1.0/gems/jekyll-2.5.3/bin/jekyll:20:in `<top (required)>'
	from /usr/local/bin/jekyll:23:in `load'
	from /usr/local/bin/jekyll:23:in `<main>'

This is a list of my gems
kso@lenovo:~/work/projects/priv/sobkowiak.github.io.git$ bundle show
Gems included by the bundle:
  * blankslate (<IP_ADDRESS>)
  * bundler (1.10.6)
  * celluloid (0.16.0)
  * classifier-reborn (2.0.3)
  * coffee-script (2.3.0)
  * coffee-script-source (1.9.1)
  * colorator (0.1)
  * execjs (2.3.0)
  * fast-stemmer (1.0.2)
  * ffi (1.9.6)
  * hitimes (1.2.2)
  * jekyll (2.5.3)
  * jekyll-coffeescript (1.0.1)
  * jekyll-gist (1.1.0)
  * jekyll-paginate (1.1.0)
  * jekyll-sass-converter (1.3.0)
  * jekyll-sitemap (0.8.0)
  * jekyll-watch (1.2.1)
  * kramdown (1.6.0)
  * liquid (2.6.2)
  * listen (2.8.5)
  * mercenary (0.3.5)
  * octopress (3.0.0.rc.34)
  * octopress-deploy (1.1.0)
  * octopress-escape-code (2.0.6)
  * octopress-hooks (2.6.0)
  * parslet (1.5.0)
  * posix-spawn (0.3.10)
  * pygments.rb (0.6.2)
  * rb-fsevent (0.9.4)
  * rb-inotify (0.9.5)
  * redcarpet (3.2.2)
  * safe_yaml (1.0.4)
  * sass (3.4.13)
  * timers (4.0.1)
  * titlecase (0.1.1)
  * toml (0.1.2)
  * yajl-ruby (1.2.1)

It works using bundle exec. Probably problem described here

bundle update solved my problem.


bump python-ecobee-api version to 0.0.8

This version broke being able to set hold mode

Noticing the same behavior, this version seems to have broke setting holds