fix Make PHHashMap exception safety

Proposed changes
It's better to erase the key when bad_alloc was thrown.
Problem summary
Describe your changes.
Checklist(Required)

[ ] Does it affect the original behavior
[ ] Has unit tests been added
[ ] Has document been added or modified
[ ] Does it need to update dependencies
[ ] Is this PR support rollback (If NO, please explain WHY)

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall


Wikipedia summary GUI

Description
Script added to generate summary about any topic from wikipedia using a GUI
Fixes #671
Have you read the Contributing Guidelines on Pull Requests?

[x] Yes

Type of change

[x] New feature (non-breaking change which adds functionality)

Checklist:

[x] My code follows the style guidelines(Clean Code) of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have created a helpful and easy to understand README.md
[x] My documentation follows Template for README.md
[x] My changes generate no new warnings
[x] I have added tests/screenshots(if any) that prove my fix is effective or that my feature works.

@kaustubhgupta  looking into it...will update you once I figure it out

@Ayush7614 did you go the workaround?

or shall I help in debugging?

@kaustubhgupta I have made the changes , can you please test it again and let me know if it is fine now

Done, I have updated the requirements file

@antrikshmisri


lib.licenses: refactor internal mkLicense to avoid future typo bugs

Description of changes
This is a follow-up to https://github.com/NixOS/nixpkgs/pull/325380 where @emilazy discovered that we have some typos in field names.
This refactoring wraps the implicit lib.licenses schema up in a factory function to ensure consistency.
While a more proper type checker like the one we use for meta checks would be better I didn't want to depend on that in more places. In fact, we might want to make meta type checks more strict on the license field.

Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
24.11 Release Notes (or backporting 23.11 and 24.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

Marking as draft until https://github.com/NixOS/nixpkgs/pull/325380 is merged.

I should have thought of this :) The // stuff in that CUDA package made me assume that there was no real way to get reliable checking, but I guess even that could be covered by the meta type‐checking stuff?

The // stuff in that CUDA package made me assume that there was no real way to get reliable checking, but I guess even that could be covered by the meta type‐checking stuff?

Indeed that would catch ad-hoc attrset additions too.

I should have thought of this :) The // stuff in that CUDA package made me assume that there was no real way to get reliable checking,

We definitely want to make this less ad hoc

If no one else is inclined I'll merge this shortly.

As the author of the original code: Thanks, much better :)


Requirements.txt fails to install due to openssl

OpenSSL throws several errors in crypto.py when trying to run pip install -r requirements.txt
Module lib is missing dozens of attributes mentioned within python3/dist-packages/OpenSSL/

Could you help to provide more details of the OS and the errors?


How do I redraw radio button with fill custom color in disable mode?

as title, please give me some help just  like below image thx

If you only want set title text color for disabled state. You can use setTitleColor:forState: and specify Disabled state. Let me know if it works.

Resolving this issue now, please feel free to re-open it if still have problem.


[v3.9.3] don't attach ui on reset if headlessMode is true

Summary
Will publish 3.9.3 from this branch without merging this, as master is on 4.0.0 and this issue does not apply to 4.0.0
How did you test your changes?
Manually

published 3.9.3 on npm based on this branch, closing this PR


Add support for cancellation of simple functions

[x] Simple requests
[x] Multipart requests

Also fixes https://github.com/BlurredSoftware/BSWFoundation/issues/10


allow custom opts.resolve

All other options are forwarded along to module-deps and browser-pack directly.

please.

Hello @wenbing. You're right, the documentation about that is incorrect. I'll leave this open for now as a documentation issue. The specific issue about opts.resolve is being tracked in #1006.

:+1:


Bug: not human readable

I got
> const cronstrue = require('cronstrue');
> cronstrue.toString('0 15 10 * * L')
'At 10:15 AM, on the last undefined of the month'

But Quartz specs says:

L (“last”) - has different meaning in each of the two fields in which it is allowed. For example, the value “L” in the day-of-month field means “the last day of the month” - day 31 for January, day 28 for February on non-leap years. If used in the day-of-week field by itself, it simply means “7” or “SAT”. But if used in the day-of-week field after another value, it means “the last xxx day of the month” - for example “6L” means “the last friday of the month”. When using the ‘L’ option, it is important not to specify lists, or ranges of values, as you’ll get confusing results.

@sanjorgek Thanks for the report!  Want to take a crack at a PR?

Fixed and released: https://github.com/bradyholt/cRonstrue/releases/tag/v0.10.9


Auto-indentation in jupyter notebooks failing

VSCode Version: 1.46.1
OS Version: Windows 10

Steps to Reproduce:

Open any function definition e.g. def func(x,y):
Pressing enter afterwards doesn't enter the scope of the function by indenting properly

Does this issue occur when all extensions are disabled?: Yes
On pressing enter the cursor moves to the following position

Instead of

This issue was not present before the previous vscode update

/extPython


Failures on meta: Introduce TargetKind.directive...[web] Add static js interop .call() tests

There are new test failures on meta: Introduce TargetKind.directive...[web] Add static js interop .call() tests.
The test changed from flaky -> RuntimeError (expected Pass)
pkg/frontend_server/test/macros_test RuntimeError (expected Pass)

On the configuration:
cfe-unittest-asserts-release-linux

cc @johnniwinther @davidmorgan

Thanks Nick! @jakemac53 FYI
Turns out that this was already broken due to the move from using package:_fe_analyzer_shared to package:_macros, but that was missed because it was flaky.
Fix to the dep is easy, and I also increase the timeout which should address the flakiness: https://dart-review.googlesource.com/c/sdk/+/360821


Validator dev

Please make sure these boxes are checked before submitting your PR, thank you!

[x] Make sure you follow DiDi's contributing guide.
[x] Make sure you are merging your commits to dev branch.
[x] Add some descriptions and refer relative issues for you PR.

Codecov Report

Merging #98 into dev will increase coverage by 0.52%.
The diff coverage is 95.18%.

@@            Coverage Diff             @@
##              dev      #98      +/-   ##
==========================================
+ Coverage   88.49%   89.02%   +0.52%
==========================================
  Files          62       68       +6
  Lines        1156     1239      +83
  Branches      159      166       +7
==========================================
+ Hits         1023     1103      +80
  Misses         73       73
- Partials       60       63       +3

Impacted Files
Coverage Δ

src/common/helpers/validator/messages.js
100% <100%> (ø)

src/common/helpers/validator/types.js
100% <100%> (ø)

src/modules/validator/index.js
100% <100%> (ø)

src/common/helpers/validator/rules.js
100% <100%> (ø)

src/common/helpers/util.js
77.77% <77.77%> (ø)

src/components/validator/validator.vue
92% <92%> (ø)

... and 3 more

Continue to review full report at Codecov.

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


nixos/testing: set the nixos version suffix to a fixed value

Description of changes
This avoids the outPath of the test nodes being different between classical nix and flake-enabled nix builds, so that we don't have to re-run the tests unnecessarily.

Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
24.05 Release Notes (or backporting 23.05 and 23.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

Hey @r-vdp, was your problem solved?
We didn't really have a discussion.
Maybe there's something simple we can do to make it work better, or is my suggestion less important, so that we should just proceed?


Import new table from existing Data source(PowerQuery)

What are you trying to achieve?
I am trying to import tables from existing Azure SQL Datawarehouse Datasource , when i try to right click and select import tables
the import tables wizard does not allow to import from existing datasource from the model .
Option is greyed out, is there something i am missing ?

Please see this article. The Import Table Wizard does not support Power Query data sources.

Thank you @otykier


Update/Hide invoice statuses on project detail page for Applicant/Vendor.(see design)

As per design sugestions:
Invoice status for applicants/vendors should only be:
Pending approval
Approved (when it is approved by staff and finance)
Declined
Request for change or more information
Paid
Payment failed (new status to be added for finance when a payment goes wrong because of some errors)

These changes has to be done on project detail page.

All these statuses need to be designed over the current statuses as wrapper for applicant like APPLICANT_INVOICE_STATUS.
Payment failed is a new status, we don't have it in our current system. Do we need it throughout the system like for staff and finance as well? @fourthletter @frjo

@fourthletter so we'll just continue showing Finance pending till it's resolved?
although, we don't need to show any specific reason for Payment failed just that applicant know. But it's okay if we want to cover it under Finance pending

We could check with individuals working in Finance, but Finance pending makes sense for now.


fix the scikit-learn package name

Summary
package name for sklearn is changed to scikit-learn
Implementation Notes :hammer_and_pick:

Briefly outline the overall technical solution. If necessary, identify talking points where the reviewer's attention should be drawn to.

External Dependencies :four_leaf_clover:

Breaking API Changes :warning:

Simply specify none (N/A) if not applicable.

@niravnshah thank you. Does the image build now?

@niravnshah thank you. Does the image build now?

Yes, it does.

@niravnshah thank you for fixing this


Variable context and value inheritance

I couldn't find a way to make Twig act this way — and I'm not sure if this is a Twig design decision, or if perhaps something like that can be resolved in the php implantation of Twig — but I would have liked a variable to "understand" its scope in the context of the json file, and inherit values from parent scopes if not locally defined.
By "understand its context" I mean that if I have the following teaser.twig partial:

{{ headline }}

  {% if byline %}
    <address>{{ byline }}</address>
  {% endif %}

I'd like to be able to include it like this, which could get very complex:
{% for article in articles.items %}
  {{ include('molecules-teaser') }}
{% endfor %}

Instead of having to do:
  {{ include('molecules-teaser', {headline: article.headline, byline: article.byline}) }}

Meaning, always look for the variable in the local scope first, then, if not defined, traverse up the blocks towards global scope in resolving the variable's value. This is how it works in the Mustache version of Pattern Lab, and it makes a lot of sense to me for templating.
Means you could set defaults in the model instead of in the view, enabling us to have a json like:
{
  "headline": "Lorem ipsum",
  "articles": {
    "byline": "Author Name",
    "items": [
      {},
      {
        "headline": "Some text"
      },
      {
        "byline": "John Doe"
      }
    ]
  }
}

and this Twig

{% for article in articles.items %}
  {{ include('molecules-teaser') }}
{% endfor %}

{{ include('molecules-teaser') }}

output this html:

Lorem ipsum

  <address>Autor Name</address>

Some text

  <address>Autor Name</address>

Lorem ipsum

  <address>John Doe</address>

Lorem ipsum

Phew, this came out long, sorry. Hope it makes any sense.
Unless I am conceptually very wrong in my understanding of templating best practices (very plausible), this pretty much annuls the benefits of Twig over Mustache in the context of Pattern Lab for me.

@TxHawks -
This is completely untested but I'd think you'd need it to look like this:
{
  "article": {
     "headline": "Lorem Ipsum",
     "byline": "Author Name"
  },
  "articles": {
    "byline": "Author Name",
    "items": [
      {},
      {
        "headline": "Some text"
      },
      {
        "byline": "John Doe"
      }
    ]
  }
}

Because you'd have to reference the vars as article.headline in the molecule. And you've about reached my limit on my knowledge of Twig ;) Let me know if that works.

@dmolsen -
No, I've already thought of that, but unfortunately even that doesn't work...
The output is

  <address></address>

Some text

  <address></address>

  <address>John Doe</address>

My knowledge of php < 0, but I would think it is an issue of how the json is parsed.
I imagine that the json is parsed as a native php object, and when calling article.headline from the for loop, it actually looks at articles.items[i].headline. article.items[0] does not have a headline key, so it returns undefined or null or whatever it is php would return (sorry for my ignorance and using javascript syntax).
One way I could think of resolving this is in parsing always including the parent scope in its childern (this sounds like a horrible idea, I know).
Something like:
{
  "headline": "Lorem ipsum",
  "articles": {
    "byline": "Author Name",
    "items": [
      {},
      {
        "headline": "Some text"
      },
      {
        "byline": "John Doe"
      }
    ]
  }
}

being parsed into the following object:
{
  "headline": "Lorem ipsum",
  "articles": {
    "headline": "Lorem ipsum",        // From global scope
    "byline": "Author Name",
    "items": [                        // This is an array, so don't include parent keys directly
      {
        "headline": "Lorem ipsum",    // from 'aritcles' scope
        "byline": "Author Name",      // from 'articles' scope
      },
      {
        "headline": "Lorem ipsum",    // from 'aritcles' scope
        "byline": "Author Name",      // from 'articles' scope
        "headline": "Some text"       // from current scope, overrides inheritance
      },
      {
        "headline": "Lorem ipsum",    // from 'aritcles' scope
        "byline": "Author Name",      // from 'articles' scope
        "byline": "John Doe"          // from current scope, overrides inheritance
      }
    ]
  }
}

This is probably not such a great way to solve it from an architectural point of view, and because it will create huge object, but maybe a better implementation could be conceived by someone with better programming skills than mine (confession: I'm really much more a designer than a developer).

@TxHawks -
K. This uses Twig's filters and seems to works. This is about the best I can come up with. Have no idea how performance would stack up over time/multiple patterns/depth/etc. Let's start with the data:
{
	"article": {
		"headline": "default headline",
		"byline": "default byline"
	},
	"articles": {
		"items": [
			{},
			{
				"headline": "Some text"
			},
			{
				"byline": "John Doe"
			}
		]
	}
}

And the atoms-article template:

{{ article.headline }}

	<address>{{ article.byline }}</address>

And the atoms-articles template (obv just quick testing this):
{% for articleStore in articles.items %}
	{% set articleStore = article|merge(articleStore) %}
	{% include 'atoms-article' with { 'article': articleStore } %}
{% endfor %}

It's a little verbose. It does allow for the merging of the iterated content with the article attribute from the global context without having to set attributes individually. Note that the include has access to the global context regardless. It just comes down to how do we override the article attribute. If we don't use the with the article attribute in the global context picks up bad data.

@dmolsen  -
Thanks for that - I did have the merge filter in mind, though it can become very verbose. I was hoping to come up with something as trivial as the traversing Mustache does. For some of the less technical designers on my team that would have been a great help, but I guess it would be trying to force Twig into something that it is not.
Just wanted to say again how much I appreciate you work on this project.
Since not many people are currently using the Twig version, I was wondering if there are any specific features you'd like me to test.

@TxHawks -
Yeah, looks like Twig prefers scoping and "redefining" variable names from the template. That kind of "logic" is anathema to Mustache. On what you can test... really just think about project workflow. You've already given me a number of ideas on what I need to tweak.
FWIW, I've updated a few things in core and the Twig PatternEngine. One of those things was watch so you may be interested in upgrading. To do so run:
composer update pattern-lab/core
composer update pattern-lab/patternengine-twig

@dmolsen -
I generally use grunt for watching changes, but I'll update and test the watch function on a few systems.
BTW, is there a place were new features of PL2 are documented? I remember something about you adding pattern-specific annotations, is that right?
If so, I'll open another issue asking how to use them, for better searchability

@TxHawks -
No documentation. We have a plan for tackling it but I don't feel like PL2 is solid enough for me to spend time on it. Please feel free to open a new issue.


[Authorization Logic] Look Into StructOpt for command line args

StructOp
This is also possibly a good opportunity to make the way inputs and outputs are specified a bit more conventional.
Suggested by: @conradgrobler @tiziano88

I think this was done in #461 and #462 (as re-implementation of #245), so I'm marking this closed

Thanks @bMacSwigg for both fixing this and properly taking managing the issue :)


Success feature

Alaska Airlines Pull Request
Summary:
This pull request addresses two new features.

New success use case
New noIcon use case

Type of change:
Please delete options that are not relevant.

[x] New capability
[ ] Revision of an existing capability
[ ] Infrastructure change (automation, etc.)
[ ] Other (please elaborate)

Checklist:

[x] My update follows the CONTRIBUTING guidelines of this project
[x] I have performed a self-review of my own update

By submitting this Pull Request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
Pull Requests will be evaluated by their quality of update and whether it is consistent with the goals and values of this project. Any submission is to be considered a conversation between the submitter and the maintainers of this project and may require changes to your submission.
Thank you for your submission!
-- Auro Design System Team

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Document and enforce source name requirements

Sources are written to the file system using the provided input from yaml.
We should have a set of rules to define what is a valid source name in order to prevent weird behaviors/breakages due to characters that won't play nice with the filesystem.
An example is path separators (/).
Quotes (single or double) are also likely to cause some issues without escaping.
I think yaml gives us at least some safety, e.g. you can't put a control character in there, but even this should be validated beyond just the yaml parser.

Just to add a few other thoughts here, any shell special characters such as *, ?,  $  etc could potentially cause issues without proper quoting or escaping


download attribute on a tag does download file, instead performs a navigation

I have an anchor tag like so
            <a
                href={asset("/file.pdf")}
                download="file"
            >

file.pdf resides in the static directory of the project. Now, when you click the anchor tag, the file does not download, instead, the URL in the browser is changed to  localhost:8000/file.pdf. And then also, the file doesn't actually load till I hit refresh.
I also tried setting f-client-nav={false} on the a tag, to no avail. If any other information is required, please let me know.

Needed a target="_blank". Closing the issue.


Error with Einthusan

Keep getting the following error when launching Einthusan
Please check and update the Base URL in Addon Settings and restart the addon.

Please try the below steps and comment if you are still facing the issue:
- Kindly uninstall the older version of the Einthusan addon
- Update to EinTamil Kodi Repository Version 1.1.0
- Install Einthusan addon Version 3.1.0


Clarify pinning certificates on web service and REST calls

[TW-1216]

Hi Ela, @ilagordhan
This is for both integration and cloud, do you want to have a quick look over it and let me know what you think?
Thanks.

Hi Mark,
I need some time to get my parallels to behave so if you need me to also verify against the product, I can do this early next week.
Otherwise, looks good!
Ela
From: Mark van Ments<EMAIL_ADDRESS>Reply to: mendix/docs<EMAIL_ADDRESS>Date: Friday, 5 March 2021 at 09:10
To: mendix/docs<EMAIL_ADDRESS>Cc: Ila Gordhan<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [mendix/docs] Clarify pinning certificates on web service and REST calls (#3106)
Hi Ela, @ilagordhanhttps://eur03.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Filagordhan&data=04|01|ila.gordhan%40mendix.com|2226edffab364876a8e808d8dfae175d|b4e3c78d8e3b46d8bc565540da23ba4d|0|0|637505286104425372|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=FIeoieWsBIBJeVcl%2BqZh0PJze0pC1NotkJsuZNz6fIY%3D&reserved=0
This is for both integration and cloud, do you want to have a quick look over it and let me know what you think?
Thanks.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://eur03.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fmendix%2Fdocs%2Fpull%2F3106%23issuecomment-791241839&data=04|01|ila.gordhan%40mendix.com|2226edffab364876a8e808d8dfae175d|b4e3c78d8e3b46d8bc565540da23ba4d|0|0|637505286104425372|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=KFl8fqsrKacihOWEPr80FdJ27yg4gZVKwhb303EJ4EI%3D&reserved=0, or unsubscribehttps://eur03.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAOJFHGSQPZW34XVHKB3QBO3TCCGV5ANCNFSM4YTHLAUA&data=04|01|ila.gordhan%40mendix.com|2226edffab364876a8e808d8dfae175d|b4e3c78d8e3b46d8bc565540da23ba4d|0|0|637505286104435366|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=gZnqZF%2BpwPKgt%2BV3m5AcXus7Bs1MNVvBUG4vREe2YQM%3D&reserved=0.

Thanks again for the review Ela @ilagordhan ,
You were right - I had written the stuff about WEB SERVICE CALL NAME from the wrong direction in each of the documents.
I've corrected that now and will merge the doc.
Let me know if you see other improvements.
Mark.


Update NiceSpinner.java

Fix java.lang.NullPointerException: Attempt to invoke virtual method 'java.lang.Object org.angmarch.views.NiceSpinnerBaseAdapter.getItemInDataset(int)' on a null object reference
()

When I called the getSelectedItem () method when the data was not initialized, NullPointerException occurred.I hope you can fix this problem.(Translate by Google)

You make a really good point, but i think you can give users friendly tips, not NPE.(Translate by Google)


[Feature Request] AWS Batch multi node deployment

At my company we deal with millions of hourly jsonline encoded records that we store in s3. Since our workloads are batch oriented, we use AWS Batch for cost efficiency.
However, the AWS Batch resources are capped at 64vcpus for a single machine. AWS batch supports [multi node deployment][(https://docs.aws.amazon.com/batch/latest/userguide/multi-node-parallel-jobs.html), which would be ideal for us since we can process thousands of files in parallel.
However, since this doesnt have yarn, I dont think we could use dask-yarn, is there any way/plan to support this use case?

There are many ways to deploy Dask on the cloud.  Given that you are on AWS you might find dask-cloudprovider useful.  https://cloudprovider.dask.org/en/latest/

ok @mrocklin i have to say i had no idea dask-cloudprovider existed, that is awesome! THanks so much for sharing it, will close this then since it seems that project seems to suit my need :)

I'm glad to hear it.  In the future you may want to look through
https://docs.dask.org/en/latest/setup.html
On Sun, Mar 22, 2020 at 10:13 AM Manuel Garrido<EMAIL_ADDRESS>wrote:

Closed #6033 https://github.com/dask/dask/issues/6033.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/dask/dask/issues/6033#event-3152875371, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AACKZTFQBGHF4I2PGLW5PADRIZBM5ANCNFSM4LRJZZDQ
.


[Scenario] The Sea Devils

https://github.com/jlederer2/Valkyrie/tree/master/TheSeaDevils
What kind of mission do you want to add (mark with an X, e.g. [X] Descent)?

[X] Mansions of Madness
[ ] Descent

What expansions does the scenario require?

[ ] Conversion Kit (CK)
[ ] First Edition (MoM1E) - Also Recurring Nightmares
[ ] Fatal Alchemy (FA) - Also Suppressed Memories
[ ] Call of the Wild (CotW) - Also Suppressed Memories
[ ] Beyond the Threshold (BtT)
[ ] Streets of Arkham (SoA)
[ ] Sanctum of Twilight (SoT)
[X] Horrific Journeys (HJ)
[ ] Path of the Serpent (PotS)

What expansions can the scenario optionally use?

[ ] Conversion Kit (CK)
[ ] First Edition (MoM1E) - Also Recurring Nightmares
[ ] Fatal Alchemy (FA) - Also Suppressed Memories
[ ] Call of the Wild (CotW) - Also Suppressed Memories
[ ] Beyond the Threshold (BtT)
[ ] Streets of Arkham (SoA)
[ ] Sanctum of Twilight (SoT)
[ ] Horrific Journeys (HJ)
[ ] Path of the Serpent (PotS)

Please enter the GitHub Url to your scenario here
https://github.com/jlederer2/Valkyrie/tree/master/TheSeaDevils

Nice to see you back to business, @jlederer2

Added to Manifest and to statistics sheets


Add support for switch statements in qasm3_to_qir converter

Feature Description

openqasm 3 also supports switch statements which allow users to condition statement execution based on integral variables.

Implementation (Optional)

The pyqir._native module does have a definition of the Switch construct but it is not entirely clear how that can be incorporated into the converter. A good starting point would be the pyqir._native definition and the openqasm switch reference

It would be good to have a handler for the switch, like the _visit_branching_statement. Since switch is also a type of branching instruction, we can either expand this method or create a new handler.

I will take a look at this and work on it

Hi @anushkrishnav, I got an email about your comment on this issue.
You can surely go for it if you're still interested!


Add hashes to cnn_dailymail

The URL hashes are helpful for comparing results from other sources.

Looks you to me :)
Could you also update the json file that goes with the dataset script by doing
nlp-cli test ./datasets/cnn_dailymail --save_infos --all_configs

It will update the features metadata and the size of the dataset with your changes.

@lhoestq I ran that command.
Thanks for the helpful repository!


Support for wings ngs embroidery format

Hi, Are you planning to add support to wings ngs embroidery format? if it is not in the plan, I would be up to contribute on this with some guidance, thanks!

I have looked at NGS in the past I don't think I was able to make much progress other than reading some of the metadata. IIRC it uses MS-CFB: Compound File Binary File Format I am not certain but I believe the files inside that are potentially encrypted. It is worth another look, but we haven't had much demand for it. Are you a software dev?

I have looked at NGS in the past I don't think I was able to make much progress other than reading some of the metadata. IIRC it uses MS-CFB: Compound File Binary File Format I am not certain but I believe the files inside that are potentially encrypted. It is worth another look, but we haven't had much demand for it. Are you a software dev?

Yes, I am a software dev. But would be my first time working with files formats and reverse engineering


createMeeting, createAttendee

I did run the amazon-chime-react-native-demo using serverless code with SAM CLI it and worked well.
Implementing same meeting feature in my react native project.
using Amazon Chime mobile SDK iOS and Android both.
I have followed the all the steps here.
Now I am stuck in creating AWS service APIs such as createMeeting, createAttendee.
To achieve this I have written this code in my  src/utils/Api.js
npx react-native run-ios
ERROR    TypeError: undefined is not a constructor (evaluating 'new _awsSdk.default.Chime({
region: 'us-east-1'
})')
ERROR    Invariant Violation: Module AppRegistry is not a registered callable module (calling runApplication)
Is src/utils/Api.js  is the correct place to write server side code createMeeting, createAttendee and so on?
If correct why am I getting above error?
If incorrect where should I write create meeting and other API related code for my app?
Because in my app users will create the meeting and choose the attendee.

@Devo-React react-native is mobile cross platform source code, so if you are trying to write server side code, you would need to start new project in your preferred language and implemented (server)[https://docs.aws.amazon.com/chime/latest/dg/mtgs-sdk-mtgs.html].

you would need to start new project in your preferred language and implemented chime doc in the preferred language.

Do I need to follow below steps:

write server side code in separate project in any preferred language
Deploy that code to get SERVER_URL
Use that url in the SERVER_URL and SERVER_REGION in src/utils/Api.js

Are these above steps correct?

That is correct step with demo application. If you want to have custom logic, for example, having login and get some token, the flow will be

Login
Get token
Make request to server url to create meeting and attendee
Use Chime Mobile SDK to join meeting after getting CreateMeeting Response and pass those information to DefaultMeetingSession

These implementation changes based on your logic. For example, the reason why we require SERVER_REGION is to pass to the server region parameter. You can also take this within your app and pass to JS server. These logics can change based on your need and your server.
The server also createMeeting. Any reason why you would like to move that logic to client?

. Any reason why you would like to move that logic to client?

Basically I am creating a voip/voice calling simple app. User logs in (using aws Auth) and clicks on call button which generates voice call to a random user.
So have planned to create meeting and Attendee when user clicks on call button.
Because user has no scope to enter here any meeting id and attendee info.

@Devo-React I am not sure if that is currently possible. It looks there is no Chime SDK in the Android to createMeeting unless you call HTTP request directly with payload https://docs.aws.amazon.com/chime/latest/APIReference/API_CreateMeeting.html. In this case, you might need to sig4, yourself. I'll update the comment if there is any update on it.


proposal: grouping all of the sealos's subcommands

As more and more subcommands are implemented in the sealos command, for better understanding of each commands and improving user experience, we might need to group all of those subcommands like what kubectl does.
Using module k8s.io/kubectl/pkg/util/templates might do that trick

ping @cuisongliu

sealctl
addHostsCommandFmt    = "hosts add --ip %s  --domain %s"
	deleteHostsCommandFmt = "hosts delete  --domain %s"
	hostnameCommandFmt    = "hostname"
	tokenCommandFmt       = "token"
	cGroupCommandFmt      = "cri cgroup-driver --short"
	socketCommandFmt      = "cri socket"


Negative inexact raised to a power

I believe the following illustrates some undesirable behavior.
   var sn = SchemeNumber;
   var a = sn('#i-1');
   var p = sn('1');
   var result = sn.fn['expt'](a, p);
   console.log(a + "^" + p + " = " + result);
   console.log(a + "^" + p + " = " + result.toString());

Raising an inexact negative number to a (non-complex) power results in a complex number.
And, printing this number gives NaN unless the toString method is explicitly called.

I agree the behavior is undesirable: a real raised to an exact integer should result in a real.  Here is a patch to master.  I do not have time to test for regressions, so I hesitate to push it.
diff --git a/schemeNumber.js b/schemeNumber.js
index b44fc24..b089623 100644
--- a/schemeNumber.js
+++ b/schemeNumber.js
@@ -3642,6 +3642,7 @@ function implementPluginLibrary(plugins) {
     // Functions to be provided by number implementations.
     var nativeToExactInteger, divideReducedNotByOne;
     var exactRectangular, inexactRectangular;
+    var nativeToInexact;

     // Imports from ECMAScript.
     var g                = plugins.get("es5globals");
@@ -3678,6 +3679,7 @@ function implementPluginLibrary(plugins) {
         divideReducedNotByOne    = plugins.get("divideReducedNotByOne");
         exactRectangular         = plugins.get("exactRectangular");
         inexactRectangular       = plugins.get("inexactRectangular");
+        nativeToInexact          = plugins.get("nativeToInexact");

         ZERO                     = plugins.get("ZERO");
         ONE                      = plugins.get("ONE");
@@ -3970,6 +3972,8 @@ function implementPluginLibrary(plugins) {
     function complex_or_exact_expt(n) {
         if (isExact(this))
             return expt_N_EI_fn(this, n);
+        if (isReal(this))
+            return nativeToInexact(Math_pow(this, n));
         return Complex_expt_fn(this, n);
     }
     function tan_via_divide_sin_cos() {

As for having to call toString() explicitly after the + operator, that is unavoidable.  I would write String(result) instead of result.toString() to handle nulls.

Applied the patch and did'nt see any regressions in my (indirect and insufficient) set of tests.
Thanks for your work!


Remove dependency on puppetdb-terminus if storeconfigs is not used

Overview
This pull request updates the documentation for configuring the path to Puppet, including tips on the puppetdb-termini gem, and eliminating the dependency on that gem in the Puppet run if storeconfigs is not used. It also adds (hopefully) reasonable defaults for locating the Puppet binary.
Inspired by https://github.com/github/octocatalog-diff/issues/6

Hey @dalen could you take a look at this and confirm this is doing what I expect and not introducing any side effects?
Basically in lib/octocatalog-diff/catalog-util/builddir.rb I changed the temporary copy of routes.yaml in the compilation directory to have facts - terminus = puppetdb changed to facts - terminus = yaml (or facts - terminus = facter). Keep in mind that :octocat:alog-diff pulls facts from PuppetDB and sticks them in a YAML file before puppet is ever invoked, so puppet itself is not ever hitting puppetdb unless it's doing storeconfigs as well.


embed link for image

When I call the API to get an embed link for an file, then the API returns an Iframe. (In the request data: {"type": "embed"}). But when I call this request for an image, then the API returns also an Iframe and not like the OneDrive Web an vaild <img> tag.
How can I get the real <img> tag for embeding an image with the API?

Any guidance on this? I would also appreciate a solution. Thanks!

Anything new on this topic? Having the same problem.
Thanks in advance.

To get a URL compatible with <img> tags you'll need to make a couple of requests.
Create an embed link
POST https://api.onedrive.com/v1.0/drive/root:/test.jpg:/oneDrive.createLink HTTP/1.1
Authorization: bearer *snip*
Content-Type: application/json
Host: api.onedrive.com
Content-Length: 23

{
  "type": "embed"
}

{
  "id": "PermissionId",
  "link": {
    "type": "embed",
    "webUrl": "https://onedrive.live.com/embed?resid=ABCD%211234&authkey=!ABCdEfgHiJklMNO",
    "webHtml": "<iframe src=\"https://onedrive.live.com/embed?resid=ABCD%211234&authkey=!ABCdEfgHiJklMNO"\" width=\"98\" height=\"120\" frameborder=\"0\" scrolling=\"no\"></iframe>"
  },
  "roles": ["read"],
  "shareId": "s!Aasdjfgasldghlfgafg",
  "expirationDateTime": "0001-01-01T00:00:00Z",
  "hasPassword": false
}

Request thumbnails for the shared item
GET https://api.onedrive.com/v1.0/shares/s!Aasdjfgasldghlfgafg/root/thumbnails HTTP/1.1
Content-Type: application/json
Host: api.onedrive.com
Content-Length: 0

{
  "value": [{
    "id": "0",
    "large": {
      "height": 800,
      "url": "https://bak0la.bn1301.livefilestore.com/y4mHlotsohstuffjhe0",
      "width": 533
    },
    "medium": {
      "height": 176,
      "url": "https://bak0la.bn1301.livefilestore.com/y4m0lotsohstuffc7s",
      "width": 117
    },
    "small": {
      "height": 96,
      "url": "https://bak0la.bn1301.livefilestore.com/y4mDlotsohstuffQr8",
      "width": 64
    }
  }]
}

You can then use any of the url values with an <img> tag to render the image directly.

How stable are these thumbnails? If I generate one today how far in the future will it be around?

Using this approach the thumbnails should work until the file is either deleted, or the embed permission is revoked.

If I were to create a slideshow website from OneDrive folder that contains both images and videos, should I follow the steps above? What would be the best way to get permanent links to multiple media files from OneDrive?
So for example if I followed the method above, would I first list all the children in the folder, then send POST request for each individual media to get embed link and then finally send a GET request to get the thumbnails?
(Unsure if this should be a separate issue)
Thanks.

@raRaRa it really depends on how your slideshow will function. Here's a brain dump that may or may not help you :)
Approach 1: On Demand
With the on demand approach you wouldn't worry about pulling all of the URLs before you start. You'd basically make calls to /children with a suitable page size, and consume all of the relevant URLs in sequence until the page is exhausted. Once the page is exhausted you'd make another query using the @odata.nextLink to get the next set with fresh URLs. That way you'd theoretically never have URLs that would expire before they're useful, since thumbnail URLs should last for 24 hours without the shares dance mentioned above, so unless your page of children is nothing but long videos you're probably fine.
This approach will not work if your website wants to try and rely on any form of caching based off of the URLs though.
Approach 2: Embed Link for Folder
You should be able to generate an embed link for the parent folder, then using a query similar to the above you could get the URLs for the children:
GET https://api.onedrive.com/v1.0/shares/s!Aasdjfgasldghlfgafg/root/children

Hello, "GET https://api.onedrive.com/v1.0/shares/s!Aasdjfgasldghlfgafg/root/thumbnails HTTP/1.1" this only return the large, median, small, three sizes of the image. Could you please tell how to get the url of original size of image? Thank you very much.


React Router v5.1.1 Link Breaks with React v15.6.2

Version
v5.1.1
Steps to reproduce

Using react v15.6.2 and react-router v5.1.1
Production build (minified) with webpack v2 and Uglify (not sure if pertinent to the issue but part of the setup I am using)
Create a component that renders with:

<div>
  <Link to={`/watch=${someVar}`}>Direct Link{</Link>
</div>

Expected Behavior
No errors.
Actual Behavior
Console will show:

https://reactjs.org/docs/error-decoder.html/?invariant=119
Note, the problem doesn't seem to happen when we are doing a development build where nothing is minified. I am not sure exactly why that is the case. Our production build is targeting ES5.
This problem does not happen at all when we are on v5.0.1 which we have been using for months. As a work around we have pinned our RR back to this version but could you please look into this as we would like to stay in sync with your latest releases.

Could this be a follow-up error of the #6934 fix?
The 'minified react error #119' says:
addComponentAsRefTo(...): Only a ReactOwner can have refs. You might be adding a ref to a component that was not created inside a component's `render` method, or you have multiple copies of React loaded (details: https://fb.me/react-refs-must-have-owner).

https://reactjs.org/docs/error-decoder.html/?invariant=119
That smells pretty much like another ref error.

This should be fixed in 5.1.2, released soon


[Feature]Task submit with ECM health check

Is your feature request related to a problem? Please describe.
Add ECM health check when select available node to execute task
Screenshots

https://github.com/apache/incubator-linkis/pull/1174


Can't animate a Shape2D using "Add Track" button

Godot version:
Godot 3.1.1 Mono Stable
OS/device including version:
Windows 8.1
Issue description:
Using the "Add Track" button, a CollisionShape2D's shape property can be animated, but there is no way to animate shape:extents:x or shape:radius without manually typing the property name into the property path text box. Because of this issue, I always thought collision shapes couldn't be animated, and I think others may have been confused into thinking this too.
To animate a Shape2D, the CollisionShape2D has to be selected in the scene tree. Keys will appear next to the shape's properties, allowing them to be animated. There is no way to do this using the "Add Track" button in the animation player. Instead, only the shape itself can be animated, which is usually not desirable because a Resource can't be animated over time.
I don't know what the UX for keying a Shape2D through the 'Add Track' menu should be, or what would be simplest to implement. This problem applies to keying all resources' properties, not just collision shapes.
One idea is to display the properties of the current resource in the "Select Property" menu:

It's confusing that there are two ways to add animation tracks, and neither of them can key everything the other can.
The key button in the inspector can't do these things (as far as I know):

Add 3D transform tracks
Add call method tracks
Add audio playback tracks
Add animation playback tracks

The "Add Track" button can't add keys for properties of resources (like the size of CollisionShapes).
To prevent confusion, I think these features should meet one of these conditions:

One of them should be able to do everything, and the other can be for convenience,
or both of them should have feature parity,
or one of them should be merged into the other so there is one, consistent way to key animations

Steps to reproduce:
With an AnimationPlayer and a CollisionShape2D in a scene, try to animate the CollisionShape2D's Shape resource.
a) Select the CollisionShape2D in the scene. Open its shape in the inspector, and add a key for its extents (if its a RectangleShape2D). This allows you to animate the shape's properties over time.
b) Using the "Add Track" button, this isn't possible. Only the Shape2D resource itself can be animated. The only way to animate its properties is to manually add a dummy track and change its path to CollisionShape2D:shape:extents in the path text box.

Related: https://github.com/godotengine/godot/issues/27999

It should be fine to remove "Add Property Track" button altogether. I don't even remember when I used it last time. Probably never, because the key buttons are just too convenient and are the "standard" way of doing animations.

@KoBeWi How would you add method track if you remove the "Add track"?

I'm talking about a single button. The dropdown would still be there, just without the Property option. Others are useful.

This here is the perfect example of the importance of having more than just one UI path to provide the best UX.
Simply removing broken UI paths instead of fixing them is the perfect example what made the 3.1 AnimationPlayer UX worse than it was before.


[OPPRO-227] Change the copy vector pool to base vector veloxPool to avoid memory copy with string type

https://github.com/oap-project/velox/blob/main/velox/vector/FlatVector.cpp#L346
This will check whether the pool is same with src BaseVector pool
Our gluten velox execute pool use wrapped allocator pool
https://github.com/oap-project/gluten/blob/main/cpp/velox/compute/VeloxPlanConverter.cc#L344
But then copy, use the default pool, so the pool is different, cause the string type memory copy

hi @zhztheplayer Can you take a look?


Documentation typo

https://k2-fsa.github.io/sherpa/onnx/tts/pretrained_models/vits.html

Since there are 109 speakers available, we can choose a speaker from 0 to 198. The default speaker ID is 0.

s/198/108/

Thanks! Will fix it.

Closing as it has been fixed.


Are there no item tooltips in the inventory or is this a bug?

when I hover over an item with the cursor (left stick/d-pad) no tooltip is shown. so I can't see the name of the item, and more importantly any other text - this makes it impossible to know what enchantments are on a book/tool among other things.
Are tooltips not in this mod yet, or am I experiencing a bug? I cannot find any reference to them being an issue anywhere.
I'm using the latest version available for download from Curseforge as of today.

Unfortunately I can replicate the issue. Is this still a problem on latest version of Controllable? (e.g. 1.19+)

I believe so, though I haven't played in a while. I assume you meant can't replicate?


[1.0.1] Fix #5901 - Query: Wrong data in included navigation when usi…

Tweak logic in IncludeExpressionVisitor to not apply EXISTS optimization if the parent query uses Skip without a predicate.

@smitpatel

@maumar

:shipit:


K1

When trainning a semantic query function on the public dataset ImageNet, the K1=1? The highest accuracy?

Hi, you need to choose the classifier obtaining the highest val accuracy as your semantic query function. K1 represents the number of public images that each sensitive image votes for and is not used during training the semantic query function.


CodeFixImports doesn't suggest to import from a file when this file is actually exported

TypeScript Version:  2.5.2
Code
index.ts
export * from './MyClass';
var t = new MyClass();

MyClass.ts
export class MyClass { }

Expected behavior:
CodeFixImport should suggest to import 'MyClass' from the 'MyClass.ts' file.
Actual behavior:
No suggestion at all

Looks like this works now (probably thanks to @andy-ms).  I suspect it's included in TS 2.6.2, but it'll be in 2.7 otherwise.


Repository Size is too large for use with Bower

Description
When I download materialize using bower, it takes alot of time to download on a broadband connection [usually 1mbps]. I know its a network speed thing but downloading 20mb file everytime is insane. Please update repository so that bower usable.
Repro Steps
Just run bower install materialize in ur cmd

@Dogfalo @acburst can't we delete the bin directory as these files should be created with each compilation?
Imho it makes more sense to move the zip task to a release task.
@tomscholz @fega your opinions on this?

Actually I think a better way would using a Bower Ignore List
Have a reference to - http://stackoverflow.com/questions/18245394/ignore-in-bowers-bower-json
Because in the bower installation, we would only require the dist folder so it makes everything else "not important". I did not wanted to use the word "useless" as it is important for some people.

I don't agree. It also makes sense to include / import Sass files and other source files, overwrite variables and use these with a bundler like webpack. It is normal that you always have also the source files. Take a look at the bower packages for Bootstrap, jQuery, Roboto and so on.

@DanielRuf the problem with the repo is all of those web page stuff. including images...
the website should in another repo!
@SamWillson I agree with @DanielRuf the source files should in the download

@fega right, a separate repository for the docs makes sense, but just if this is possible.
But why not deliver the docs with the package so developers have the docs of the installed version?
Anyway, dependencies are just installed once and just a few files imported.

I'm not really sure why the bower size is 20mb, since the materialize folder after bower install materialize completes is 4.27mb

I'm not really sure why the bower size is 20mb, since the materialize folder after bower install materialize completes is 4.27mb

$ bower install Dogfalo/materialize --force --verbose
bower resolve       https://github.com/Dogfalo/materialize.git#*
bower download      https://github.com/Dogfalo/materialize/archive/v0.98.2.tar.gz

It is mainly the images folder.
$ du -sh * | sort -h
...
97K     tests
112K    extras
128K    test
140K    css
271K    sass
410K    js
644K    jade
772K    fonts
1,1M    res
1,5M    dist
2,2M    bin
3,3M    templates
13M     images

I don't see images when I bower install
On Sun, Apr 30, 2017 at 12:57 PM, Daniel Ruf<EMAIL_ADDRESS>wrote:

It is mainly the images folder.
$ du -sh * | sort -h
...
97K     tests
112K    extras
128K    test
140K    css
271K    sass
410K    js
644K    jade
772K    fonts
1,1M    res
1,5M    dist
2,2M    bin
3,3M    templates
13M     images
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/Dogfalo/materialize/issues/4623#issuecomment-298253220,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ACpax-Jx9bwDsGSq_6d8jfyfiPrwzPjDks5r1OergaJpZM4NMIB8
.

--
Doggy sends his greetings from Mars.

The du command was on the manually downloaded tarball. Not the final bower_components directory.
Bower deletes the files and directories fro the ignore key.
Running PNGGauntlet just saves ~2 MB (png files).

Download https://github.com/Dogfalo/materialize/archive/v0.98.2.tar.gz and extract it ;-)
A diff with the final files and directories in the bower_coponents/materialize directory makes clear that the ignored files are not there after the cleanup process of Bower.

Anyway, imho 20MB is nothing compared to other archives and Bower / npm directories with hundreds of MB dependencies. You always use just a fraction of this. I know, this takes a while and the first install of big projects with an empty cache can take some minutes, this is not unusual.

@DanielRuf But we always could do better 🥇

Should not be a good idea create o new organization on Github and split the site itself and the framework?
Things would be more clear with that, and no unnecessary files on each repo ...

When we put the whole documentation in a new folder we could just put that folder to the bower ignore file...

When we put the whole documentation in a new folder we could just put that folder to the bower ignore file...

Well but this would not decrease the size of the tarball which is downloaded and which is so big ;-) Only solution would be to use another branch for the site or https://github.com/Dogfalo/materialize/issues/4623#issuecomment-300791268 to decrease the size of the tarball (master branch, the one which you download at the top right with the clone / download button).
At least the archive is too big https://github.com/Dogfalo/materialize/issues/4623#issuecomment-298254441

How does bootstrap do it? How do they keep their bower package so small then? They have the docs in same repo...

It is the exact same size like the v3-dev / master branch. Bower downloads, extracts it and then it is cleaned up (the ignored stuff deleted).
They do not host such big images in their repository like we do ;-)
$ bower install bootstrap --force --verbose
...
bower progress      bootstrap#* received 1.9MB of 4.0MB downloaded, 48%
bower progress      bootstrap#^3.3.7 received 1.9MB of 4.0MB downloaded, 48%
...
bower progress      bootstrap#^3.3.7 received 3.9MB of 4.0MB downloaded, 97%
bower extract       bootstrap#^3.3.7 archive.tar.gz

I think since Bower is now deprecated, we can close this issue. What do you think?

Also it would not make any difference for the most alternatives as they still download the full archive first, extract it, clean it up and then you have the final (smaller) size. The first download (the archive) is still big and bigger in some other projects and it is locally cached after this.

Yes I do not think this is a huge issue. We will work on reducing the size with future updates anyways


feat: hertz binding refactor

What type of PR is this?

feat
Check the PR title.

[ ] This PR title match the format: <type>(optional scope): <description>
[ ] The description of this PR title is user-oriented and clear enough for others to understand.

(Optional) Translate the PR title into Chinese.
hertz 绑定重构文档
(Optional) More detailed description for this PR(en: English/zh: Chinese).

en: hertz binding refactor doc
zh(optional):  hertz 绑定重构文档
(Optional) Which issue(s) this PR fixes:

PATL @Skyenought  @chaoranz758

记得 + 英文 @FGYFFFF


Update release assets to include MacOS ARM64 links

As mentioned in #4509, the release notes & assets.json file did not properly include the MacOS ARM64 download links.
This small PR fixes that issue.

@microsoft-github-policy-service agree

/azp run

Merged in this PR.


Error message: No credentials form found

Command:
weconnect-cli --username<EMAIL_ADDRESS>--password VerySecret
Gives error message:
2022-01-28T13:18:24+0200:CRITICAL:There was a problem when communicating with WeConnect. If this problem persists please open a bug report: No credentials form found
Tested version:
weconnect-cli --version
weconnect-cli 0.23.1 (using WeConnect-python 0.35.0)

VW changed the login form. Will release an update as soon as possible!

Ok. Keep up the good work!

Should be fixed with 0.23.2 will work on the details the next days, there was no time for testing. But I think it should be robust enough for now

Done a quick test and looks that this works!


[BUG] Wrong crit. chance for Imbued Wand

Imbued Wand has a base crit. chance of 7.00% instead of the 8.00% listed in PathOfBuilding.
Edit:
Nevermind, updated the program and got the correct values.

I've just checked, and Imbued Wand definitely has 7%.
Are you sure you're running the latest version? The wand bases were updated in 1.3.18, which was released 2 weeks ago.


Branch fix delete contact

Codecov Report

Merging #334 (cb55965) into master (5658619) will decrease coverage by 0.05%.
The diff coverage is 47.05%.

@@             Coverage Diff              @@
##             master     #334      +/-   ##
============================================
- Coverage     56.60%   56.55%   -0.06%
- Complexity      720      723       +3
============================================
  Files           139      139
  Lines          3196     3213      +17
  Branches        420      426       +6
============================================
+ Hits           1809     1817       +8
- Misses         1286     1293       +7
- Partials        101      103       +2

Impacted Files
Coverage Δ

src/main/java/seedu/awe/commons/core/Messages.java
87.50% <ø> (ø)

src/main/java/seedu/awe/model/Model.java
100.00% <ø> (ø)

src/main/java/seedu/awe/model/ModelManager.java
79.09% <25.00%> (-6.63%)
:arrow_down:

...seedu/awe/logic/commands/DeleteContactCommand.java
100.00% <100.00%> (ø)

...a/seedu/awe/logic/commands/EditContactCommand.java
97.05% <100.00%> (+0.08%)
: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 5658619...cb55965. Read the comment docs.


Unify validation and execution

Right now we do validation in batch but we do execution order by order. Would be better if we did validation order by order as well. We can unify validation and execution into a single function, where a section at the top handles validation.

I think this is the right approach.
@itamarreif and I talked about this a bit. The biggest question for me is how much overhead there is for each message server request. If overhead is negligible each msgserver request should just specify a single action and SDK/frontend should handle batching. If overhead is high we should stick with batching but probably handle verification and execution in a more symmetrical way

I don't think there is any real advantage to batching from a computational perspective.


[Bug]: Deleted binding path does not show up as an error by the linter

Is there an existing issue for this?

[X] I have searched the existing issues

Description
When a widget's binding path has been deleted, for instance, a table widgets column or a button group widget's button, we don't display an error for incorrect path
Steps To Reproduce

Add a button group widget and bind the first button to a text widget
Delete this button and observe there are no errors displayed for this missing path

Public Sample App
No response
Version
Cloud

@ramsaptami looks like a duplicate of https://github.com/appsmithorg/appsmith/issues/15905


GitHub + SauceLabs authentication access tokens should be Readium, not individual developers

Related issue: https://github.com/readium/readium-js-viewer/issues/565

GitHub (fork updater, invoked during npm run prepare):
https://github.com/readium/readium-cfi-js/blob/master/readium-build-tools/gitHubForksUpdater.js#L18

SauceLabs:
https://github.com/readium/readium-cfi-js/blob/master/.travis.yml#L9

TODO check what this secure field is for:
https://github.com/readium/readium-js-viewer/blob/master/.travis.yml#L102
(IIRC this is for GitHub integration in TravisCI)


Unable to open PVA.pbit

Hi
Downloaded the files. On opening the pbit I am getting the message that it can't be opened.
"Either the file is encrypted or corrupted"
Running Version: 2.95.983.0 64-bit (July 2021)

Ignore. Redownloaded Git & file opened with no issues.


many strings plotted w/ one command: ANNOTATE x,y,string

Reported by Steven C Hankin on 29 Apr 2004 15:45 UTC
It is often desirable to label many points at once -- for example numerical
labels showing temperature extrema, or names of islands.  Ferret already
contains the ability to handle arrays of strings, and functions to convert
arrays of float values into string arrays.  To add the needed PLOT
functionality a relatively simple solution might be
ANNOTATE xpos, ypos, string
(Optionally ANOTATE could be an alias for PLOT/VS/ANNOTATE)
Other qualifiers might include
/SIZE=textSize
/FONT=PLOT+font
/OFFSET=xinch,yinch
/COLOR=
/APPEND="degrees C." (e.g.)
/PREPEND="string"  (is this needed?)
/NOCLIP (or whatever to indicate ok to go past axis box)
It should also possess the /I/J/K/L/X/Y/Z/T qualifiers
Migrated-From: http://dunkel.pmel.noaa.gov/trac/ferret/ticket/868

Comment by<EMAIL_ADDRESS>on 5 Nov 2004 19:05 UTC
changing "Severity" tag of bugs under Ferret Tasks to "enhancement"

Comment by<EMAIL_ADDRESS>on 5 Nov 2004 19:20 UTC
Changing Component of bugs in "Ferret Tasks" from All to Main


⚠ Spryr has degraded performance

In 62807fb, Spryr (https://spryr.com) experienced degraded performance:

HTTP code: 200
Response time: 3447 ms

Resolved: Spryr performance has improved in fe543c5 after 15 minutes.


No module named simvg.models.fusions

Hello,
First of all, thank you for the excellent work on SimVG! I am interested in this project and am currently trying to reproduce the results.
However, I encountered an issue during implementation. When running the code, I got the following error:
Traceback (most recent call last):
  File "tools/train.py", line 13, in <module>
    from simvg.models import build_model, ExponentialMovingAverage
  File "/home/gpu18/workspace/SimVG/simvg/models/__init__.py", line 4, in <module>
    from .fusions import *
ModuleNotFoundError: No module named 'simvg.models.fusions

I have checked the project structure but couldn't locate the fusions module. Could you kindly provide guidance on whether this is a missing file or if I might have missed some configuration steps?

Hello,
First of all, thank you for the excellent work on SimVG! I am interested in this project and am currently trying to reproduce the results.
However, I encountered an issue during implementation. When running the code, I got the following error:
Traceback (most recent call last):
  File "tools/train.py", line 13, in <module>
    from simvg.models import build_model, ExponentialMovingAverage
  File "/home/gpu18/workspace/SimVG/simvg/models/__init__.py", line 4, in <module>
    from .fusions import *
ModuleNotFoundError: No module named 'simvg.models.fusions

I have checked the project structure but couldn't locate the fusions module. Could you kindly provide guidance on whether this is a missing file or if I might have missed some configuration steps?

Thank for your attention.
We have fixed this issue in the newest comment b7319ca5f24cd3f46cbe566010288f83b8c9268b.

Thanks!


TrafficShaper not Working on 18.1_1

Hi there,
I have 2 TrafficShaper Rule's in use to Limit the Download & Upload Speed for our guest users (in Guest WiFi). Since on or two Updates back i need to "re-apply" (only hit apply without canging any settings) the rules/pipes to get the correct limitation. Otherwise (lets say one day without "re-apply") i only get very low speeds.
The Limitations are 15 Mbit/s for Download and 2 Mbit/s for Upload.
When one Client is there and i "Re-apply" the rules the SpeedTest show the correct speed; Otherwise i get 1 Mbit/s Download and 0,1 Mbit/s Upload.
Any Ideas about that behaviour ?

To troubleshoot I would create the same rules and Pipes for other Ethernet Nic. If also occurs check status page while testing and check the logs.
Perhaps deleting all, reboot and recreate also helps?

It might be older than 18.1, I remember I noticed a similar drop in effective throughput after the winter holidays, OPNsense being left alone since 12.23.17 until 01.08.18 - 17.7.11 or 12, I can't really remember the sub-subversion. The effective speed was crawling, and a quick disable/ enable restored everything to normal.
But I don't use TS any more, so I didn't experienced anything like this since then.

Docs updated https://github.com/opnsense/docs/commit/64663c9fb26a7fb27b502c8f3f8ee9c5f6c0ee01


Connection failed

help me please.
i can't login from udeler version 1.6.3

I have released a new version v1.7.0 which should allow you to login easily using the Udemy website itself.
The new version will support all kind of accounts, be it business, google or facebook. Download, test and provide feedback.


fix ci error

// Describe your PR here; eg. Fixes #issueNo

Make sure that:

[ ] You have read the contribution guidelines.
[ ] You submit test cases (unit or integration tests) that back your changes.
[ ] Your local test passed mvn clean install -Dmaven.javadoc.skip=true.

Codecov Report

Merging #3 (1020399) into master (18e9c50) will decrease coverage by 0.37%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master       #3      +/-   ##
============================================
- Coverage     59.86%   59.49%   -0.38%
+ Complexity     4473     4429      -44
============================================
  Files           691      691
  Lines         20045    20042       -3
  Branches       1803     1802       -1
============================================
- Hits          12000    11923      -77
- Misses         7033     7107      +74
  Partials       1012     1012

Impacted Files
Coverage Δ

...henyu/admin/service/impl/UpstreamCheckService.java
42.85% <0.00%> (-20.64%)
:arrow_down:

...yu/sync/data/http/refresh/AbstractDataRefresh.java
70.00% <0.00%> (-13.34%)
:arrow_down:

...a/org/apache/shenyu/common/utils/VersionUtils.java
67.85% <0.00%> (-10.72%)
:arrow_down:

...apache/shenyu/admin/utils/ShenyuDictH2Trigger.java
63.63% <0.00%> (-9.10%)
:arrow_down:

...ache/shenyu/admin/utils/PluginHandleH2Trigger.java
63.63% <0.00%> (-9.10%)
:arrow_down:

...henyu/plugin/grpc/resolver/ShenyuNameResolver.java
56.38% <0.00%> (-7.45%)
:arrow_down:

.../apache/shenyu/admin/model/dto/BatchCommonDTO.java
50.00% <0.00%> (-7.15%)
:arrow_down:

...apache/shenyu/admin/model/dto/AuthPathWarpDTO.java
57.14% <0.00%> (-7.15%)
:arrow_down:

...che/shenyu/sync/data/http/HttpSyncDataService.java
75.55% <0.00%> (-5.93%)
:arrow_down:

...org/apache/shenyu/admin/model/dto/AuthPathDTO.java
58.82% <0.00%> (-5.89%)
:arrow_down:

... and 25 more

Continue to review full report at Codecov.

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


Add traceback info to common error handler

Still up for discussion: right now, when an internal server error occurs, the logged error is not really useful for the user.
For example, a KeyError will only print the missing key instead of the full stacktrace and error.
Adding it to the common error handler as that one is meant to catch all remaining errors, HTTPExceptions are handled by another error handler.
Another option would be to do like Flask and Starlette by adding a debug option, but that seems a lot of work while this current solution doesn't really have big downsides (?)

Coverage: 94.042%. Remained the same when pulling ae41a30699db7686db17db399c850b04e5b9386c on Ruwann:feature/exception-logging into 0c0c517cf6d698bd7683e0a516d6aad9564a2f5e on spec-first:main.

logger.error(exc, exc_info=exc)
This was much needed, thanks!

@Ruwann @RobbeSneyders I think it would it make sense to apply this same change to the http_exception and problem_handler functions in this file because neither connexion.middleware.exceptions.ProblemException nor starlette.exceptions.HTTPException have a decent __str__ implementation.
two other ideas - in case dumping the exc_info is overkill for each HTTPException or ProblemException:

logger.error(repr(exc))  would be useful.
OR just implement the __str__ method for those Exception subclasses


DOCS: long term fix to kafka not starting up in multi region clusters  Fixes #252

DOCS: add note explaining suspected long term fix to kafka not starting up in multi region clusters
Fixes #252

...vscode has stripped some trailing whitespaces


Eliminate multiple seed phrases

Upon initial implementation, https://github.com/dashhive/DashWallet.js lacked the ability to add multiple addresses to one wallet (seed phrase) but should have that ability now.
We should eliminate the one-to-one wallet-address pairing and instead allow generating new public addresses within a single wallet (seed phrase), while maintaining the ability to signup for CrowdNode with each public address.

https://youtu.be/dNI6SAB9GMY
https://youtu.be/CIZoBbUragA


DCOS-47925: remove manual trigger for rerun script

Testing

Check if this branch has a job with head that was triggered automatically without a manual check and ran successfully without rerunning the tests
Check if this branch has a job with head that was triggered automatically without a manual check and ran successfully with rerunning a subset of tests
Check if a .only in an integration test can be commited without a warning, renders the merge test red, but triggers the branch test to run reruns

Trade-offs

Decided that ~10min of work for not running is okay, compared to maintaining an extra check before the tests run. As always, feel free to challenge me on this 😇

Dependencies

Screenshots

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


composite unique constraint

Is it possible to define composite unique constraints like this ?
const ItemTag = sequelize.define('item_tag', {
  id : {
    type: DataTypes.INTEGER,
    primaryKey: true,
    autoIncrement: true
  },
  tag_id: {
    type: DataTypes.INTEGER,
    unique: 'item_tag_taggable'
  },
  taggable: {
    type: DataTypes.STRING,
    unique: 'item_tag_taggable'
  },
  taggable_id: {
    type: DataTypes.INTEGER,
    unique: 'item_tag_taggable',
    references: null
  }
});

With the Column annotation:
@Column({
  unique: 'item_tag_taggable',
  ...,
})

tnx

Is it possible to define two composite unique constraints where there is an intersection between columns?
I mean:
 await queryInterface.addConstraint('a_b', ['a_id', 'b_id'], {       type: 'unique',       name: 'a_b_unique_constraint',     });     await queryInterface.addConstraint('a_c', ['a_id', 'c_id'], {       type: 'unique',       name: 'a_c_unique_constraint',     });
As you see there is a column a_id which is a part of two composite unique indexes.
How can I define this property in model description?
Smth like ?
@Column({     unique: [       'a_b_unique_constraint',       'a_c_unique_constraint'     ]     type: DataType.INTEGER,   })

Hey @NickTaylor98, I'm not sure. What does the sequelize docs say?

@NickTaylor98 were you able to solve 2 unique constraints issue?

@NickTaylor98 were you able to solve 2 unique constraints issue?

@NickTaylor98 @RobinBuschmann @akoidan
You can define composite multiple composite indices in the table options, which should in effect solve the issue.
`
// This string binds unique constraint to the combinations
const UNIQUE_CONSTRAINT_EMAIL: string = "unique_email_per_tenant"
const UNIQUE_CONSTRAINT_PHONE: string = "unique_phonenumber_per_tenant"
@Table({
indexes: [
{
name: UNIQUE_CONSTRAINT_PHONE,
unique: true,
fields: ['phone', 'userId']
},
{
name: UNIQUE_CONSTRAINT_EMAIL,
unique: true,
fields: ['email', 'userId']
},
]
})
`


Adding manifest files for SSMS 21.0 Preview 1.0

Checklist for Pull Requests

[x] Have you signed the Contributor License Agreement? Yes
[x] Is there a linked Issue? No

Manifests

[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.6 schema?

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

We released SSMS 21.0 Preview 1.0 today, so these are the winget manifest files.
Couple of things (vs older SSMS manifests):

They are based on the template used by Visual Studio
There's only one language, since the Visual Studio Installer takes care of all the languages directly
Links, descriptions, etc... partially carried over from the old SSMS manifest files

Microsoft Reviewers: Open in CodeFlow

Hi @,
Do you approve of these changes?
(Automated response - build 897.)


913 add null constraints and update org create service

🔗 Issue
#913
✍ Description
This Pr adds null constraints to the Organization model name, email, and slug attributes.
Initially, I tried to add the null constraints by simply adding the columns.
Rails didn't approve and gave the following error
"Dangerous operation detected #strong_migrations ===
Setting NOT NULL on an existing column blocks reads and writes while every row is checked"
It then guided me to add a check constraint and validate it in a separate migration.
Adding NOT NULL constraints on an existing column can be dangerous in a production environment and
can lock the table and block reads and writes, which is why strong_migrations recommended a safer approach.
This will also allow for easier rollbacks if needed.
Also adds:

email attribute on Organizations Create Service
validation for presence and format for the following organization attributes:
name, slug, and email
All corresponding test updates

@Gabe-Torres Ah yes the strong migrations gem, rightfully so, will not allow you to apply these constraints. However, we don't have any production data t the moment and we don't have to adhere to these process the gem sets out. Have you tried writing the migration to add the null constraints on the three columns by wrapping it in a safety_assured block?
We had strong migrations installed early on in this project and ran a lot of migrations it moaned about...and we had to wrap each migration in the safety_assured block. See this PR that removed the gem and this block in the migrations https://github.com/rubyforgood/pet-rescue/pull/400/files
We can take the same approach here to write a simpler migration that just adds the null constraints. You will probably need to drop your db and go from there to run the migrations and seed it for it to work properly. That is fine as we do not care about the data.
Also, should we not also update the seed files to add the required attributes to the code creating the Organizations?
Let me know if you have any questions.

@Gabe-Torres how's this one going?

@Gabe-Torres bump

I'll get these changes submitted asap! @kasugaijin

@Gabe-Torres Ah yes the strong migrations gem, rightfully so, will not allow you to apply these constraints. However, we don't have any production data t the moment and we don't have to adhere to these process the gem sets out. Have you tried writing the migration to add the null constraints on the three columns by wrapping it in a safety_assured block?
We had strong migrations installed early on in this project and ran a lot of migrations it moaned about...and we had to wrap each migration in the safety_assured block. See this PR that removed the gem and this block in the migrations https://github.com/rubyforgood/pet-rescue/pull/400/files
We can take the same approach here to write a simpler migration that just adds the null constraints. You will probably need to drop your db and go from there to run the migrations and seed it for it to work properly. That is fine as we do not care about the data.
Also, should we not also update the seed files to add the required attributes to the code creating the Organizations?
Let me know if you have any questions.
Updated!
note -
looks like both the alta and baja seed files organization instances have the required attributes! But maybe I'm missing something?

@organization = Organization.create!(
  name: "Baja",
  slug: "baja",
  email<EMAIL_ADDRESS>  phone_number: "250 816 8212",
  custom_page: CustomPage.new(hero: "hero text", about: "about us text")
)

@organization = Organization.create!(
  name: "Alta Pet Rescue",
  slug: "alta",
  email<EMAIL_ADDRESS>  phone_number: "250 816 8212",
  custom_page: CustomPage.new(hero: "Where every paw finds a home", about: "Alta was founded by an incredible group of ladies in April of 2020. Our initial goal was to have both a rescue and a spay/neuter clinic, however, we quickly realized that it would be more efficient to separate into two organizations.")
)

Gotcha! Made those changes as well 👍🏽


Rework intro page

Mostly UI changes and a few improvements to UX.
To test:
Look at Intro page on UI and ensure everything works as expected.
Look at About page on UI and ensure everything works as expected.
Look at Create page on UI and ensure everything works as expected.
In particular, open the 3 scene previews and ensure that multiple keys will now trigger the fullscreen mode exit (incl. e.g. Alt, Ctrl, Space, Backspace and as previously Esc).
Specifically, take a look at the Topnav and the Footer on the UI and ensure everything looks in place and works as expected.
Make sure that the Login functionality from the Topnav works as expected. Also ensure that you stayed logged in across the different pages (and that they display accordingly).
In a particular Space, ensure the Inspector now opens in the same position in the space (i.e. the view doesn't change to a default position).

When I went to the explore and clicked view a scene. It made me log in again

Tested it and looked good!

thanks, for the additional login request, we'll tackle that as part of this issue: https://github.com/Bebb-Protocol-and-Apps/PWS/issues/83


Add fix to display the current version of the crate

Add a constant that get the current version of the crate
Override --version option to display the current version of the crate and the current version of cargo

Thanks again, @k0pernicus!
@homunkulus r+

:pushpin: Commit 14abd6b has been approved by japaric

:hourglass: Testing commit 14abd6b with merge 14abd6b...

:sunny: Test successful - status-appveyor, status-travis
Approved by: japaric
Pushing 14abd6bb31c98d79db155b3e16b51f64a6e075bf to master...


set distributed=True, return AttributeError: 'NoneType' object

For example:
tp.tensor_parallel(module=model, device_ids=["cuda:0", "cuda:1"], distributed=True)
AttributeError: 'NoneType' object has no attribute 'make_distributed_shard'

@rocketsearch Good morning!
I've released version 1.2.2 which is supposed to fix this issue.
Distributed functionality hasn't been properly tested in a long time, so it was plain broken. I'm planning to write test for it sometime later.

Thanks


fsck reports false-positives when comparing recipients

gopass fsck can report false positives if the .gpg-id files don't contain full size hashes but either short ones or recipient names.
The comparison must first normalize to long hashes before comparing them.

Yep, just been bitten by this one:
[] ❌ Missing recipients on irc.freenode.net<EMAIL_ADDRESS>Run fsck with the --decrypt flag to re-encrypt it automatically, or edit this secret yourself.

[] ❌ Extra recipients on irc.freenode.net: [FC41BCF0479672265B8D2B64753E1D018EC24F4C]
Run fsck with the --decrypt flag to re-encrypt it automatically, or edit this secret yourself.

Not only the fsck message is a little confusing, but re-encrypting the secret does nothing!

Please note that my idea was to look up the long hash to whatever is in the .gpg-id file - not rewrite that.


Upgrade configmap-reloader

How to categorize this PR?

/area monitoring
/kind enhancement
What this PR does / why we need it:
Upgrade configmap-reloader.
Special notes for your reviewer:
/cc @istvanballok
Release note:

The following image is updated:
- ghcr.io/prometheus-operator/prometheus-config-reloader: v0.56.2 -> v0.58.0

/retest


Publish persistent version on GitHub CI

I saw this commit lying around for the 1.1.2 version and I thought we might want this all the time lol

Good point. In that case we have one stable release plus one with the version number.

In that case we don't have any stable tag anymore or? That means we have to update the versions on the webpage to point to the new version?

That's why I initally kept the stable tag lol
Should I just put it back?

Haha ys I think we have to do it ^^. Not that elegant to have a stable, latest and a version release but there is no other way or?

Take a  look at this: https://github.com/exactly-one-kas/Gittyup/releases/tag/v13.37.69

That means we have to update the versions on the webpage to point to the new version?

I'd say get rid of the "stable" tag altogether.
GitHub supports a keyword (not a tag) called "latest" that will always be a fresh link to the latest tagged release marked as stable in their API. Note that the URL for this is not quite the same as for a tag, two segments of the path are reversed. This is far superior to having a confusing mess of sliding tags like "stable".

Currently latest is pointing to the latest commit on master, but this is not indended to be a stable version

Currently latest is pointing to the latest commit on master, but this is not indended to be a stable version

That's why I specified that there is a latest keyword not a tag.
This is the link to your tag (which I think should also be dropped, see below):
https://github.com/Murmele/Gittyup/releases/tag/latest
This is the keyword link:
https://github.com/Murmele/Gittyup/releases/latest
This will update automatically to whatever the most recent release (not just tag, but GitHub Release not marked as pre-release).
As such this keyword can be used to completely replace what you are currently doing redundantly with the "stable" tag.

The "stable" system can definitely go with no loss of function. However secondarily the "latest" tag doesn't really serve a purpose that isn't accomplished other ways already. If you want to expose CI artifacts for HEAD you can just like to the actions on the branch.

Currently latest is pointing to the latest commit on master, but this is not indended to be a stable version

That's why I specified that there is a latest keyword not a tag.
This is the link to your "stable" tag (and also "latest" tag which I think should also be dropped, see below):
https://github.com/Murmele/Gittyup/releases/tag/stable https://github.com/Murmele/Gittyup/releases/tag/latest
This is the keyword link:
https://github.com/Murmele/Gittyup/releases/latest
This will update automatically to whatever the most recent release (not just tag, but GitHub Release not marked as pre-release).
As such this keyword can be used to completely replace what you are currently doing redundantly with the "stable" tag.
The "stable" system can definitely go with no loss of function. However secondarily the "latest" tag doesn't really serve a purpose that isn't accomplished other ways already. If you want to expose CI artifacts for HEAD you can just like to the actions on the branch.

But is it then also possible to get a link to the latest release, prerelease and stable?

@alerque @exactly-one-kas what do you think about my solution?
In that case the latest tag will always set and if we have a release we additionally creating a stable release?

Good idea

@Murmele I pushed a change to clarify what the releases names mean. If you're okay with that, you can merge

@Murmele I pushed a change to clarify what the releases names mean. If you're okay with that, you can merge

great thanks!


lammps datafile for UFF - redefinition of the unitcell

Dear,
Willing to generate the UFF datafile for Lammps for the structure in attachment, I get the message 'bonded atoms (nr) and (nr) are 7.730 Angstroms apart. This probably has something to do with the redefinition of the unitcell to a supercell. Please contact the developers!'
The final files, which I hoped to get, have not been generated and the program hangs. I attached the input structures (pdf and cif, which I had to use in the program) in attachment.
problem_uff.zip
Remark that the structure consists out of layers, and that inside each layer a compound is used, which has symmetry. For completeness, I attached as well this symmetric part of the compound in one layer (*_ori_cif.zip).
PR122_ori_cif.zip
Can you please have a look at it?
Thank you!
Stefan

@peteboyd @danieleongari Is somebody willing to look into this?

(From my side, I am still interested...)

I'll have a look

Stefan, I have looked at your cif files and have found the problem. The one titled 'PR122_uff.cif' contains symmetry elements that the lammps-interface code cannot currently process. This is something we should address, and there is another issue that raises this concern (issue #5).
The one titled 'PR122_try.cif' is P1 symmetry, however there appears to be too many molecules described in the unit cell. If I look at the number of atoms in the small bounding box, I count 42. However in your file there are 504! It is likely that whatever software you were using provided replicated molecules from a supercell.  One of the cif processing steps the code will do is try to fit all the atoms within the unit cell, prior to computing bonding. I'm a little surprised I didn't implement an atom overlap check at this stage, but I've done so now and will exit with a message telling the user to check their system before proceeding.
I've provided a modified unit cell for you, that only contains 42 atoms and works with the lammps-interface code.
PR122_unit.txt
(just change .txt to .cif)

Dear Pete,
Thank you!
I understand now that the Lammps-interface requires for the triclinic case one unit cell, which helps me a lot.
If I may, the point where I am still a bit puzzled now is how the program determines on its own the dimension of the final slab. In the code, I find the reconversion of the a,b,c-alpha,beta,gamma unit cell to lx,xy,... Are then the final lx, ly and lz of the supercell calculated with respect to a half-cell cutoff? If I would like to have a much larger slab, how should that be said to the lammps_interface.py?
Thank you again for the clarifications and help!
Stefan

Indeed, thanks!
Op ma 26 aug. 2019 om 19:33 schreef Peter Boyd<EMAIL_ADDRESS>
Stefan, try the --replication command line option
if you type lammps-interface --help you will see a number of command line
arguments and a description of what they do.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/peteboyd/lammps_interface/issues/17?email_source=notifications&email_token=AC76WPVV45DPEVNIXG567FTQGQHYHA5CNFSM4IDBEXEKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD5FCMAI#issuecomment-524953089,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AC76WPQSMZXDQJCXO3PZWN3QGQHYHANCNFSM4IDBEXEA
.


Char Symbol dont't change

The problem
I was setting up the user's configuration and I noticed that the SPACESHIP_CHAR_SYMBOL don't change when I set a new value for It
The default char ➜ still remains:

Relevant Zsh configuration
export ZSH="/home/carlos.junior/.oh-my-zsh"

SPACESHIP_PROMPT_FIRST_PREFIX_SHOW="true"
ZSH_THEME="spaceship"

plugins=(git)

source $ZSH/oh-my-zsh.sh

SPACESHIP_PROMPT_ORDER=(
  user          # Username section
  host          # Hostname section
  dir           # Current directory section
  git           # Git section (git_branch + git_status)
  exec_time     # Execution time
  line_sep      # Line break
  jobs          # Background jobs indicator
  exit_code     # Exit code section
  char          # Prompt character
)

SPACESHIP_CHAR_SYMBOL="$"
SPACESHIP_CHAR_PREFIX="└─"
SPACESHIP_CHAR_SUFFIX=" "

SPACESHIP_USER_SHOW="always"
SPACESHIP_USER_PREFIX="┌──("
SPACESHIP_USER_SUFFIX=""
SPACESHIP_USER_COLOR="green"

SPACESHIP_HOST_SHOW="always"
SPACESHIP_HOST_PREFIX="%{$fg[green]%}@"
SPACESHIP_HOST_SUFFIX="%{$fg[gray]%}:"
SPACESHIP_HOST_COLOR="green"

SPACESHIP_DIR_TRUNC_REPO=false
SPACESHIP_DIR_PREFIX=""
SPACESHIP_DIR_SUFFIX=") "
SPACESHIP_DIR_COLOR="blue"

### Added by Zinit's installer
if [[ ! -f $HOME/.local/share/zinit/zinit.git/zinit.zsh ]]; then
    print -P "%F{33} %F{220}Installing %F{33}ZDHARMA-CONTINUUM%F{220} Initiative Plugin Manager (%F{33}zdharma-continuum/zinit%F{220})…%f"
    command mkdir -p "$HOME/.local/share/zinit" && command chmod g-rwX "$HOME/.local/share/zinit"
    command git clone https://github.com/zdharma-continuum/zinit "$HOME/.local/share/zinit/zinit.git" && \
        print -P "%F{33} %F{34}Installation successful.%f%b" || \
        print -P "%F{160} The clone has failed.%f%b"
fi

source "$HOME/.local/share/zinit/zinit.git/zinit.zsh"
autoload -Uz _zinit
(( ${+_comps} )) && _comps[zinit]=_zinit
### End of Zinit's installer chunk
zinit light zdharma/fast-syntax-highlighting

What is version of Spaceship are you using?
v4.13.1
What is version of Zsh are you using?
5.8.1
What operation system are you running?
Ubuntu 22.04.2 LTS jammy
Do you use any Zsh framework or plugin manager?
zinit
Which terminal emulator do you use?
Terminator
Additional information
No response

any solution? im have same problem

same here, I  made a workaround replacing the icon in  ~/.oh-my-zsh/custom/themes/spaceship-prompt/sections/char.zsh

any solution? im have same problem

My workaround is: set SPACESHIP_CHAR_SYMBOL before loading. I'm using antigen, this is my working config:
SPACESHIP_CHAR_SYMBOL='%6{🌿🦥🍃%} '
antigen apply

any solution? im have same problem
have the same problem fix with this "change CHAR to PROMPT

SPACESHIP_PROMPT_SYMBOL=

My workaround is: set SPACESHIP_CHAR_SYMBOL before loading. I'm using antigen, this is my working config:
SPACESHIP_CHAR_SYMBOL='%6{🌿🦥🍃%} '
antigen apply

Can confirm this solution does help.
in my case:
SPACESHIP_PROMPT_SYMBOL=">>"
SPACESHIP_CHAR_SYMBOL=">>"
...
prompt spaceship

I am not using antigen, and I found no workaround so far.

It's not a bug i think, you need create a custom spaceship.zsh file and put this config into them.
Follow this docs:

https://spaceship-prompt.sh/config/intro/

Use the first method with "touch", the other didn't work for me.
After, open this file it has been created.
$ open ~/.spaceshiprc.zsh
And put your CHAR configuration in it.
For me, it worked. 😄


Style/MapToHash incorrectly flags non-hash transformations

%w[path other_path].map { |path| Pathname.new(path) }.to_h { |path| [path.basename, path.extname] }

leads to
Offenses:

/tmp/1.rb:1:21: C: Style/MapToHash: Pass a block to to_h instead of calling map.to_h.
%w[path other_path].map { |path| Pathname.new(path) }.to_h { |path| [path.basename, path.extname] }

despite the fact that the #map is not creating a Hash.
RuboCop version
`1.24.0 (using Parser <IP_ADDRESS>, rubocop-ast 1.15.0, running on ruby 3.0.3 x86_64-linux)``

Not sure if this is a bug. Same can be rewritten as following to avoid offense:
%w[path other_path].to_h do |path|
  pathname = Pathname.new(path)
  [pathname.basename, pathname.extname]
end

We have tests to specifically check for this case map <block> to_h <block> - and register offense but do not autocorrect - which seems correct.
https://github.com/rubocop/rubocop/blob/9f39dad99d6d4cdc2c1f96e205000ca081a820ce/spec/rubocop/cop/style/map_to_hash_spec.rb#L79-L88

Yeah it’s not a bug IMO as @tejasbubane mentioned it can be rewritten without the map.

Yep, but unfortunately it clashes with Avoid do…end when chaining. if there are further method calls.
CONST = %w[path other_path].to_h do |path|
  pathname = Pathname.new(path)
  [pathname.basename, pathname.extname]
end.freeze

wouldn't fly.


please bump peerDependency of react to allow v15.x.x

@pedronauck could you do it, please? 😏

yes!! could you doit please?

@pedronauck you're not going to maintain this repo anymore, right?

giving up. just one more zombi module.


Support multiple response class depending on an input parameter

I have a get operation that return two different classes (and it could be more) depending on a parameter input.
I could resolve it separating to multiple operations. One for each result class.
But it forces me to add a path for the GET.
Example: /persons/{id}/info=?
It returns PersonDto or EmployeeDto for BASIC or EMPLOYEE parameter.
The result are two classes totally different.
And I don't want to publish something like: /persons/{id}/full

Parent: #574.

@perabello Is this still an issue with the latest OAI v3 draft?

@RobDolinMS - There's no solution for this in 3.0. Given that this didn't get much traction, I'll close it now. We can revisit if we get more requests for it.

Stinks that this is closed. This would be so incredibly useful.
being able to treat:
GET /foo?view=full returning a list of fully hydrated foos vs
GET /foo?view=compact returning say, a list of ids for instance would be incredibly useful to have in the spec
as distinct endpoints would be incredibly useful.


Order DESC in scope not possible with Sequelize.col

What I want to do is get the average value of a column grouped by an id. This is like a rating. Therefore I would need to get this code-snippet to work:
@Scopes({
   ranked: {
       attributes: [[Sequelize.fn('AVG', Sequelize.col('review.points')), 'avg']],
       order: [[Sequelize.col('avg'), 'DESC']],
       group: 'myId'
   }
})

This right now doesn't work but gives me the error:
[ts]
Argument of type '{ ranked: { attributes: [fn, string][]; order: (string | col)[][]; group: string; }; }' is not assignable to parameter of type 'IDefineScopeOptions'.
  Property 'ranked' is incompatible with index signature.
    Type '{ attributes: [fn, string][]; order: (string | col)[][]; group: string; }' is not assignable to type 'Function | IScopeFindOptions | undefined'.
      Type '{ attributes: [fn, string][]; order: (string | col)[][]; group: string; }' is not assignable to type 'IScopeFindOptions'.
        Types of property 'order' are incompatible.
          Type '(string | col)[][]' is not assignable to type 'string | col | literal | (string | number | ModelClassGetter | { model: ModelClassGetter; as?: st...'.
            Type '(string | col)[][]' is not assignable to type '(string | col | literal | (string | number | ModelClassGetter | { model: ModelClassGetter; as?: s...'.

I think this is due to wrong typings, because when I build it, everything works as expected. So maybe in IScopeFindOptions the following has to be changed:
order?: string | col | literal | Array<string | number | ModelClassGetter | { model: ModelClassGetter, as?: string }> |
  Array<string | col | literal | Array<string | number | ModelClassGetter | { model: ModelClassGetter, as?: string }>>;

to ->
order?: string | col | literal | Array<string | col | number | ModelClassGetter | { model: ModelClassGetter, as?: string }> |
  Array<string | col | literal | Array<string | col | number | ModelClassGetter | { model: ModelClassGetter, as?: string }>>;

Any thoughts on this? I can add an according mergerequest. I'm kinda new to sequelize in general so maybe I'm also missing something and it's possible to solve my problem easier.

@patsimm You're right, this need to be fixed. Thanks for reporting

Hello, is this fix integrated in the current release ? I have a recent release and meet the same problem. I'm using 0.6.5.
Here is my code, which is similar :
BasePlanning.ts
 @DefaultScope({
    include: [
        { model: () => BaseEtape, as: 'etapes', order: [[Server.sequelize.col('positionInParentList'), 'ASC']] }
    ]
})
@Table({ tableName: 'base_planning' })
export class BasePlanning extends BaseModel<BasePlanning> {

    @BelongsToMany(() => BaseEtape, () => BasePlanningEtapes)
    public etapes: Array<BaseEtape>;

}

And the typescript error :
myPath/BasePlanning.ts(11,49): error TS2345: Argument of type '{ include: { model: () => typeof BaseEtape; as: string; order: (string | col)[][]; }[]; }' is not assignable to parameter of type 'Function | IScopeFindOptions'.
  Type '{ include: { model: () => typeof BaseEtape; as: string; order: (string | col)[][]; }[]; }' is not assignable to type 'IScopeFindOptions'.
    Types of property 'include' are incompatible.
      Type '{ model: () => typeof BaseEtape; as: string; order: (string | col)[][]; }[]' is not assignable to type '(ModelClassGetter | IScopeIncludeOptions)[]'.
        Type '{ model: () => typeof BaseEtape; as: string; order: (string | col)[][]; }' is not assignable to type 'ModelClassGetter | IScopeIncludeOptions'.
          Object literal may only specify known properties, and 'order' does not exist in type 'ModelClassGetter | IScopeIncludeOptions'.


Sharing is Caring Power App Sample

Pull Request Body:
Category

[x] New Sample

What's in this Pull Request?
Power Apps Calendar Component

Thank you @mschuess !


Need a gnomansland_contributor.yml file

Need to make a GnomansEnv.yml file so users can run
conda env create -n GnomansLand --file gnomansland_contributor.yml

As the project evolves, more will be added to the file.