Add FMS as a new external

To use GMAO_Shared 1.0.6 and higher, you need to add FMS as a new
external. This means every manage_externals .cfg file needs to
move up because of the change to src/Shared/CMakeLists.txt

@JustinPerket, note that hopefully with MAPL 2.0 we can un-require FMS because the FV3 dependency will be gone.

Tested to build, looks good to merge. 0-diff testing for jperket/PrepForMasterMerge will occur before merging to develop, but don't anticipate any issues
https://github.com/GEOS-ESM/GMAO_Shared/issues/40 can be marked as closed


gotools needs to be updated for compatibility with the debugger in the newest version of Ghidra

Describe the bug
The newest version of Ghidra, 10.0.1, operates with a debugger. Ghidra fails to update registers upon state change with the current version of gotools.
Go version: go1.16.6
Target os: Linux
Ghidra version: 10.0.1
To Reproduce
Steps to reproduce the behavior:

Unzip ghidra_9.1.2_PUBLIC_20200521_gotools.zip
Update version number to 10.0.1 in extension.properties
Re-zip
Move zip file to $GHIDRA_DIR/Extensions/Ghidra
Open Ghidra and install extension
Load a Go binary into debugger
Debug locally in GDB in VM
Set breakpoint anywhere
Click Resume
The error "Trouble updating registers on state change" appears in Debug Console

Minimal source to reproduce
//none

Expected behavior
When I run the Go binary, I expect the register values to populate in the Registers tab, accessible when I click "Display Decompiler". Instead, no register updates, and each remain set to 0.
Additional context
N/A

I was able to reproduce this issue. But I can also reproduce the issue without the plugin. So I assume this is not an issue with the plugin but with Ghidra.


Latest updates to production

Codecov Report

Merging #50 (48028c3) into main (0a85d7e) will decrease coverage by 10.49%.
The diff coverage is 71.81%.

@@             Coverage Diff             @@
##             main      #50       +/-   ##
===========================================
- Coverage   83.04%   72.54%   -10.50%
===========================================
  Files          36       42        +6
  Lines         985     1315      +330
  Branches      152      224       +72
===========================================
+ Hits          818      954      +136
- Misses        148      291      +143
- Partials       19       70       +51

Impacted Files
Coverage Δ

src/discordBot/commands/admin/sort_courses.js
100.00% <ø> (ø)

...rc/discordBot/commands/admin/update_instructors.js
100.00% <ø> (ø)

src/discordBot/commands/admin/update_invlinks.js
100.00% <ø> (ø)

src/bridge/service.js
12.26% <13.51%> (ø)

src/discordBot/commands/admin/update_database.js
21.42% <21.42%> (ø)

src/discordBot/services/service.js
69.35% <46.15%> (-11.94%)
:arrow_down:

src/discordBot/commands/student/instructors.js
87.50% <69.23%> (-12.50%)
:arrow_down:

src/discordBot/events/messageCreate.js
90.32% <71.42%> (-6.11%)
:arrow_down:

src/discordBot/commands/faculty/status.js
87.09% <80.00%> (-4.57%)
:arrow_down:

src/discordBot/commands/faculty/edit_course.js
53.65% <88.88%> (ø)

... and 18 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 0a85d7e...48028c3. Read the comment docs.


Stop running Luacheck on E2 extension files

They use a custom preprocessor so don't contain standard Lua syntax.
This should stop Travis builds failing when somebody touches these files.
See #1087 for a discussion on this.

better would be to only exclude those e2function
even better yet, run extpp to preprocess them into a secondary folder and luacheck that instead

Yeah, running extpp would be the ideal solution. Disabling it on these files for now is better than leaving it enabled and failing on everybody's PRs.


Compilation aborted for mds_reclen.f when building tutorial_barotropic_gyre

Building on Centos 7.7 using Intel 18.0.1 compilers
ifort   -c mds_reclen.f
mds_reclen.f(618): error #6404: This name does not have a type, and must have an explicit type.   [WORDLENGTH]
       MDS_RECLEN = nnn*WORDLENGTH
------------------------^
compilation aborted for mds_reclen.f (code 1)
make[1]: *** [mds_reclen.o] Error 1

@jmiezitis Please use the mitgcm-support mailing list for help regarding this type of problem, as suggested in the "new issue" remark:
Note: for general support questions, please use the mailing list<EMAIL_ADDRESS>  This repository's issues are intended for feature requests and bug reports.

Hi jm-c,
Sorry I thought i was reporting a bug as tutorial_barotropic_gyre is part of MITgcm.  Will move this to the list.
Cheers.
--JohnM


Abuse the JSON scalar

In regards to
https://github.com/graphql-java-generator/graphql-maven-plugin-project/issues/93
seems like many projects abuse the JSON scalar so that their services can return a graph of any shape. Calling GQL services applying the same approach I get an exception thrown in class AbstractCustomJacksonDeserializer.
switch (p.currentToken()) {
			case VALUE_FALSE:
			case VALUE_TRUE:
				value = new BooleanValue(p.getBooleanValue());
				break;
			case VALUE_NUMBER_FLOAT:
				value = new FloatValue(p.getDecimalValue());
				break;
			case VALUE_NUMBER_INT:
				value = new IntValue(p.getBigIntegerValue());
				break;
			case VALUE_STRING:
				value = new StringValue(p.getText());
				break;
			case VALUE_NULL:
				value = null;
				break;
			default:
				throw new JsonParseException(p, "Non managed JSON token: " + p.currentToken()); //$NON-NLS-1$
			}

the token in question is '{'. I am having a hard time trying to work around this issue. Say I wanted it to work I would apply the following to your code base

Extend AbstractCustomJacksonDeserializer
Override your deserialize function
Generate a new template based on your client_jackson_deserializers.vm.java and replace your base class with mine
I get stuck in the implementation of the deserialize function.

I don't mind using a JsonNode as the return type. Any suggestions?

Hello, it's the same issue as #205.
I'm working on it. But the issue is not in the plugin's code. It's in the graphql-java core tool.
I'll create a demo project, before filing an issue to the graphql-java team.
Etienne

Given this hack on the generated CustomJacksonDeserializers for just that mapping of JSON scalar  using the following implementation everything works as expected.
@Override public Object deserialize(JsonParser p, DeserializationContext ctxt) throws IOException, JsonProcessingException {
			return p.readValueAsTree();
}

Can you explain to me why you think it is not in your implementation of the AbstractCustomJacksonDeserializer deserialize function?

Hello,
Thank you for your comment.
I came to the same conclusion (that the issue is in the plugin).
This custom serialization / deserialization has become too complex. But I need to keep it for backward compatibility.
The main issue is to make it work for all possible cases.
Étienne


Use Vim 8 features for updating the quickfix list without replacing quickfix lists from other plugins

When the neomake plugin is updating a quickifx list asynchronously, some other
Vim plugin or user can create/modify the quickfix list resulting in the neomake
plugin updating the incorrect quickfix list. This can be avoided by using
the new features in Vim8.
Refer to the following thread for the steps to do this:
https://groups.google.com/forum/#!topic/vim_use/LTWNG0nRQCc
Patch 1023 for Vim 8 added the support for the quickfix list identifier.

Thank you!
I will consider doing it once the patch is in Neovim - but any PR / help is welcome before already of course.

Fixed in https://github.com/neomake/neomake/pull/2188, thanks!


please take this down

how exactly are you supposed to configure host networking from a docker container?

nevermind nevermind I just figured out what --net=host does XP


bug - onFocusChanged not called

hi, i've noticed that onFocusChange is not called sometimes on web when exiting from the field.
-When pressing Enter to unfocus the field
-Once you press outside the field to unfocus it, but i see that the autocompleting is working
I've noticed also that by pressing inside and outside (without making any change to the date) the focus change is called correctly.
Here's a console log
As you can see,
on focus changed date: FALSE is missing

Doing some more research,
if i select the text without focusing the field,
onFocusChange -> true IS called, when unfocusing it  onFocusChange -> false IS NOT called
if i press inside the field, then i select the text,
onFocusChange -> true IS called, when unfocusing it  onFocusChange -> true IS called

indeed this is an issue, a disappointing one, was so hyped for a second.

@matteoberla
Thanks for the research.

if i select the text without focusing the field,

What is the procedure for this?
In my environment, it seems that the callback is invoked even if the selection is made with the tab key, etc.
When unfocusing with Enter key or tapping outside, onFocusChanged is called in both cases

-Start with nothing focused
-Try to select the text until it's all highlighted (don't tap inside the field, just select the text directly)
-Make some changes to the text value
-Exit from the field by tapping outside
Hope you will be able to reproduce this, otherwise I'll make a short video
Thank you.

-Start with nothing focused -Try to select the text until it's all highlighted (don't tap inside the field, just select the text directly) -Make some changes to the text value -Exit from the field by tapping outside
Hope you will be able to reproduce this, otherwise I'll make a short video
Thank you.

I found out how to reproduce it. Thank you.

Released in v2.1.4

Thank you very much


Added Bench 2021 Logo

This logo is only used in the Bench README. The replacement has been
done so that the URL doesn't change.
Reference thread: https://github.com/frappe/bench/issues/1219

Pushed the logo on my fork. Check it out here => https://github.com/gavindsouza/bench#readme


Pagebuild global pages

Only display and allow to be added global pages which are not already in application.

fixed in [561608ec0c34b17d7fc6959bd0cff78fffa29511]


Add load test to GitHub Actions

Describe the problem
Create a new file in your repository under the .github/workflows/ directory, for example, load-test.yml and define load test GitHub Actions workflow.
Proposed solution
Create or add to an existing GH action the load tests using the cli
Alternatives you've considered
No response
Additional context
No response

Tasks:

[ ] Add documentation on running the load tests
[ ] Create github action that runs load tests in QA on merge


Error at startup

I am encountering issue when installing dbeaver and starting it for the first time. I have Java 6 installed (I'm on a mac) and when I try to start it I get the following:
An error has occurred.
See the log file
/Applications/DBeaver.app/Contents/Eclipse/configuration/1501870681528.log.
So it spits out a log file with errors like the one attached.
Anyone know the solution?
1501870645321.txt

Nevermind. Even though the install prompts me to install Java 6 for Mac, I really need the Java 8 developer kit: https://github.com/serge-rider/dbeaver/wiki/Installation


MissingFormatArgumentException

The String format that doesn't match with argument.
fixes: #6489

After offline discussion. The change stays the same.


[bug] Auto-scroll does not function on iOS Safari in v1.10.x

Describe the bug
Dragging a sortable item to the edge of the containing element does not result in auto-scrolling in iOS Safari in v1.10.x.
To Reproduce
Steps to reproduce the behavior:
Visit https://sortablejs.github.io/Sortable/ on iPhone or an iPad and drag a sortable element to the edge of the screen. It does not scroll like it does on OSx Safari or Google Chrome.
Expected behavior
It should scroll like on other platforms.
Information
The issue occurs from version 1.10.0 (I didn't test RCs, though), and doesn't happen in v1.9.0.

Same issue here


cannot consume compiled js files targeted to browser

This has been reported here  https://github.com/microsoft/TypeScript/issues/27287 but closed without being fixed.
Comment summarizing the issue:

I'm having the same problem, but succinctly, it's that when I compile
import {Foo} from "./module"
and target ES2015 or ESNext modules, it gets transpiled as:
import {Foo} from "./module"
Yes, exactly the same thing. Since I'm targeting javascript, it should be adding the .js on the end, but since it doesn't, any browser that supports modules gets a 404 back from the server, since the transpiled output is actually module.js.

Expected behavior:
Transpiled code should be
import {Foo} from "./module.js"
This prevents using TS packages in browsers (in modern development environment, without bundling and wepacking). Browsers expect import to point to the exact filename.

This isn't ideal but I'll note for the record that you can add the .js manually and TS will recognize it, even if the source is actually .ts.  The bigger issue is that automatically-added imports (added by the language service) don't include the extension, inevitably leading to this pain point.

TypeScript does not modify the module specifier.  This is very intentional, because any modification would work for some, but not work for anyone, so TypeScript is avoiding getting into that fight.

I generally agree with that sentiment, too - however unlike with paths, there is currently no way to deal with differences in file extension handling at the compiler/tsconfig level.
If there were an option to have it so automatically-added imports included the .js extension (which TS already happily recognizes), I would be happy.  Right now you can’t rely on auto-imports if you’re targeting ESM in the browser.

Probably we should have auto-import detect if the prevailing style of import in the file uses extensions or not, but rewriting the import path is not the correct solution.

Sorry to comment on a closed issue, but I have just encountered this issue myself.
The suggested solution to add the .js extension to imports in TS files does work but it's confusing and counter-intuitive.
Would it not be possible to add a moduleExtension option to the tsconfig.json file?
{
    "compilerOptions": {
        "module": "ES2020",
        "moduleExtension": "js" // or "mjs", or anything else a developer needs
    }
}

That seems like a sensible solution to this, and it would give developers the freedom they need on a per-project basis.

I'm also running into this issue; I think it's plainly almost unimaginable that a simple .ts file using modules and targeting modern browsers and ES modules cannot be compiled to a working javascript file in the browser.
I'm using the workaround suggested above (using an explicit .js module reference in the typescript file, like  "import {Foo} from './module.js' ", but this feels rather strange, although it works well in the browser.


contact us page

the image on the right is sticky to screen , we want same margin on the left and right of the page
@iaminsafb

Fixed.

Fixed.


The Badge is showing a retagles instead a circle in Xamarin.Forms.IOS

I'm using  BadgeView.Shared.BadgeView the componente is showing a rectangle instead a circle on Iphone.
Version: 2017.10.17
  Android View:

Iphone View

what version of Xamarin.Forms and did you initialize the renderer so it doesn't get linked out?

Fixed it by adding CircleViewRenderer.Initialize(); in my native projects


text wrap in report metadata table

Broken words are bad:

@lboeman can you take a quick look at this?

So the wrap for this table is set to break-word which was a fix for issues like this:

One thing we can do to ensure that at least, arbiter provided language such as interval labels and such don't wrap is to reduce the padding on these tables. The result of a reduction from .75rem to .5rem is below:

looks good to me, thanks!


Add Read At Indicator

This PR should add a new checkmark read at indicator if the message has
been read and is the last message in the conversation. The popup does
not look the best but can adjust with any feedback. There is a case in
the fixtures where a message can have two notifications, is that a use
case to worry about?

Add a new feature message_read_indicator
Add a new component Messages::ReadAt

[x] My code contains tests covering the code I modified
[x] I linted and tested the project with bin/check
[ ] I added significant changes and product updates to the changelog

Thanks for the PR! After playing with the feature a bit I'd like to make a few changes to the original issue.
If we are only going to show a single checkmark then I don't think it should be inside the message
. Instead, let's move it in between the message list and the input field.

Having it inside the message
 means we would have to also remove it when a new message is sent since they are done via Turbo Streams. Keeping it outside of the
 but inside the #messages Turbo Frame means it will get updated automatically when a new message is sent.
Here's some starting HTML for how it could look.
diff --git a/app/views/conversations/show.html.erb b/app/views/conversations/show.html.erb
index bb40fa7..340e513 100644
--- a/app/views/conversations/show.html.erb
+++ b/app/views/conversations/show.html.erb
@@ -22,6 +22,21 @@

     <%= render @conversation.messages %>
+
+
+

+

+        <%= inline_svg_tag "icons/solid/check.svg", class: "h-5 w-5 text-green-600" %>
+
Message read.

+

+

+
+
+

+

+
Message unread.

+

+

   <%= render Businesses::HiringFeeComponent.new(current_user, @conversation) %>

I also don't think we need the tooltip. If we migrate these read indicators to each message then I'd like to add them. But for now only having the checkmark feels good enough. Knowing when someone read the message, IMO, doesn't feel like it adds much value.

Finally, each message might have multiple notifications associated with it. The query should make sure to only check that the "other" user in the conversation has read it. You can also use Notification#read? instead of checking if the column isn't null.

@joemasilotti hopefully this is ready to go!

Looks great! Thanks for the contribution. Merging this in now.


Linear search for nearest neighbor doesn't always equal hwt nearest

A test has been added compare_to_linear to test this bug. This will be closed once the test executes successfully.

After fixing this, I am now blocked by rust-lang/rfcs#2603 because I am unable to write the source code in a way that doesn't have exponentially exploding type names.

I am changing the implementation to avoid the need for long iterator-combinator chains to alleviate this issue. It should perform just as well.


Does not work with https using self-signed certifcates

Hello, when running for a site with a self-signed certificate I get
ERROR : Post "https://<IP_ADDRESS>": x509: cannot validate certificate for <IP_ADDRESS> because it doesn't contain any IP SANs

Hello,
This seems to be related to how the certificate was generated, according to this

it would be better i think if tanker had a field/flag that allowed tanker to not verify the secure connection (which is the issue here) - I'm having the same issue.
Curl etc all have a flag to do this, so would be a nice touch if tanker also had one. Could be an option on a request....


Update LOCALIZATION_CONTRIBUTORS

Summary
Details and comments

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

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

Thank you for your interest in contributing to Qiskit translations. Please go ahead and follow steps 4 and 5 in the Contributing to Qiskit documentation.

@korbelan I don't see your request to join the Qiskit translation project on Crowdin. Which language are you interested in translating?
If the translation project is open in Crowdin go ahead and follow steps 4, 5 and 6 in the Contributing to Qiskit documentation.
If we don't receive your request in Crowdin or clarification about the language you want to translate by February 1, 2020, this pull request will be closed and you will need to open a new one.

Can you please move this PR here: https://github.com/Qiskit/qiskit-translations


Print errors, upgrade docker image

This PR prints errors instead of saving them to a file. The saving to file option is a bit annoying when using docker.
This can be tested with the following docker image:
quay.io/lunarway/aws-google-auth:0.0.36-print

@kaspernissen should we close this one?

Yes, we've moved on from this setup.


Remove resolution txn restrictions

Removes the need for a setup transaction for v2 contract resolutions

Why did we need a setup transaction? Is it purely to pay the miner fee? If so, perhaps some of the funds in a contract resolution could be used to pay the fee...? 🤔

Renter allowance, host collateral, and miner fee.

Looks like this would also have a side-effect of enabling the host/renter rollover to "escape"...

Another option is to timelock the outputs, i.e. the change output from a resolution wouldn't be spendable for 144 blocks. That might be confusing from a UX perspective though.

@lukechampine are you okay with this as is?

yeah, I think so, as long as we're a bit more cautious about immediately chaining those outputs into new transactions.

c.f. https://github.com/SiaFoundation/core/pull/43...
I'm thinking that we should probably not allow contract outputs to be ephemeral. That's enough of a deterrent to prevent the most egregious forms of output chaining.


[ATO-1236] Change slack notification for docs

Proposed changes:

Fix ATO-1236

Status (please check what you already did):

[ ] added some tests for the functionality
[ ] updated the documentation
[ ] updated the changelog (please check changelog for instructions)
[ ] reformat files using black (please check Readme for instructions)

@Urkem I'm not sure why the secret scanning workflow has suddenly started failing here given that the changes aren't related to this PR - you'll need to add docs/versioned_docs/version-3.x/deploy/deploy-rasa-plus.mdx to the trivy-secret.yaml file in the root of the repo to resolve.


Add Better Support for MyBatis Multi-Row Inserts that Return Generated Keys

Coverage remained the same at 100.0% when pulling 38db33d38672e9d5c88c55912e705bf6a95f131a on jeffgbutler:better-insert-multiple-support into b67ea69b73aa092307640c1db98233144f005c34 on mybatis:master.


Update Docker and installation instructions

Description of the changes
Modernize Docker base image and use conda-libmamba-solver for faster dependency resolution. Enhance Dockerfile CMD with improved format and ensure inclusion of activemq in docker-compose.yml dependencies.
Check all that apply:

[x] release notes are not necessary for this very minor change
[x] updated documentation
[ ] Source added/refactored
[ ] Added unit tests
[ ] Added integration tests

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

Project coverage is 89.51%. Comparing base (718d05b) to head (9913099).

Additional details and impacted files
@@           Coverage Diff           @@
##             next      #11   +/-   ##
=======================================
  Coverage   89.51%   89.51%
=======================================
  Files           3        3
  Lines         124      124
=======================================
  Hits          111      111
  Misses         13       13

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


Alb modules develop

Alb modules refactor now working with new logic.
Broke up core logic into several helper functions (some of which may be good candidates to standardize in alarm-tools).
Fixed broken deletion logic.
We correctly parse configs based on defaultCreate and tag values or lack thereof.
Limited changes to alarm-config as anomaly alarms are more limited to the values from the ComparisonOperators type. This has been notated and updated for ALB configs.
Some defaults have been initially agreed upon by the corresponding team owners of those alarms. These have also been updated in alarm-config.
Code has been tested and correctly sets default alarms (both warning and critical anomaly) with just autoalarm:enabled tag set to 'true' and the non-default alarms successfully create when overridden with tag values. Inversely, are also deleted when tag values override those alarms.
Other notable changes not related to alb-modules are:

Some testing changes were made to sqs-modules which can be ignored as a sqs-modules-develop branch was broken off to replicate the refactored logic found in alb-modules once this PR is approved.

Alarm tools have been changed and currently work for ec2-modules but do not work for the other modules. The current alarm creation functions are isufficent for the new workflows and logic between anomaly and static threshold alarms. For this reason, the only two functional modules for the time being are ec2-modules and alb-modules. Other modules are currently in progress for refactoring.

These other changes were necessary in the process of reworking the tag parsing and alarm creation logic or are a result of the the process to improve the preexisting logic with the new.

closing and pulling into alarm-configs-develop with other refactored modules.


manual Snapshot folder bug fix

again...

Does this PR relate to an existing Issue?

yes see #183
because you need a place to store, and i decided to use the vis.0 folder before years.


[1.6-rc.1] Wrong validation for Integer/Float fields for questions

width, maximum, minimum, or step raise a ValidationError when set and then removed.

Fixed.


[Bug] isAggregation ListTableAPI没声明TS

Version
1.7.8
Link to Minimal Reproduction
isAggregation ListTableAPI没声明TS
Steps to Reproduce
isAggregation ListTableAPI没声明TS
Current Behavior
isAggregation ListTableAPI没声明TS
Expected Behavior
isAggregation ListTableAPI没声明TS
Environment
- OS:
- Browser:
- Framework:

Any additional comments?
No response

可以做代码贡献 提pr呀  @kapeter

感谢你的反馈！我们已经收到你的问题，并将在后续进行处理。
如果有开发者对这个 issue 感兴趣，请在issue下方留言“认领该issue”，留言后开始你的创作提pr给我们！
欢迎大家参与开源共建！我们非常感激每一位贡献者，提交pr后我们将为参与共建的开发者准备一份礼品。
参与共建可参考：https://visactor.io/vtable/guide/Contribution_Guide
与官方开发者交流问题可以下载飞书app扫码加入飞书群：

再次感谢你的支持！
Thank you for your feedback! We have received your question and will deal with it later.
If any developer is interested in this issue, please leave a message "claim this issue" below the issue, and then start submitting your creation to us!
Welcome everyone to participate in open source co-construction! We are very grateful to every contributor. After submitting the PR, we will prepare a gift for the developers who participate in the co-construction.
For participation in co-construction, please refer to: https://visactor.io/vtable/guide/Contribution_Guide
To communicate with official developers, you can download the Feishu app and scan the QR code to join the Feishu group！
Thank you again for your support!


Can't run on Ubuntu 20.04 and 18.04

Tried the ashlar on both Ubuntu 18.04 and 20.04 both failed with the exception
Cycle 0:
reading 1.jpg
Traceback (most recent call last):
File "/home/simon/.local/bin/ashlar", line 8, in
sys.exit(main())
File "/home/simon/.local/lib/python3.8/site-packages/ashlar/scripts/ashlar.py", line 171, in main
return process_single(
File "/home/simon/.local/lib/python3.8/site-packages/ashlar/scripts/ashlar.py", line 202, in process_single
reader = build_reader(filepaths[0], plate_well=plate_well)
File "/home/simon/.local/lib/python3.8/site-packages/ashlar/scripts/ashlar.py", line 328, in build_reader
reader = reader_class(path, **kwargs)
File "/home/simon/.local/lib/python3.8/site-packages/ashlar/reg.py", line 391, in init
self.metadata = BioformatsMetadata(self.path)
File "/home/simon/.local/lib/python3.8/site-packages/ashlar/reg.py", line 207, in init
self._init_metadata()
File "/home/simon/.local/lib/python3.8/site-packages/ashlar/reg.py", line 228, in _init_metadata
options.setBoolean('zeissczi.autostitch', False)
File "jnius/jnius_export_class.pxi", line 879, in jnius.JavaMethod.call
File "jnius/jnius_conversion.pxi", line 72, in jnius.populate_args
File "jnius/jnius_utils.pxi", line 205, in jnius.check_assignable_from_str
TypeError: Invalid instance of 'java/lang/Integer' passed for a 'java/lang/Boolean'
Testing command:
ashlar 1.jpg 2.jpg --align-channel 0 --maximum-shift 50 --filter-sigma 1 -f ashlar_output_cycle{cycle}_channel{channel}.ome.tiff
Any clue to fix this?

I think this is caused by kivy/pyjnius#124 so downgrading pyjnius should fix it. Pyjnius v1.2.1 or v1.3.0 should work. We will investigate how to work around this issue in ashlar so newer pyjnius versions will work too, or adjust our minimum compatible version if they fix the bug.

@jmuhlich thanks so much! After downgrading pyjnius to 1.3.0 the issue solved.


Feature request : Tools support of Qwen2

Why ?
So i just found out that Qwen2 have tools support. However, ollama current do not have tools support for qwen2 models. And there is a section of Qwen2 github talking about ollama on tools use. And this seems prombles of ollama in older versions. I think this should be resolvable with version 0.3.x
Request

I would like to request tools support for Qwen2 models
If possible I would like to know how the process of adding tools support for a model (like ollama team can add tools to Qwen2 directly or participation of Qwen2 team is required)

Thanks for reading

may be you can pull sam4096/qwen2tools and test.
then check the modelfile，only the modelfile has been modified，and the model weight file has not changed, which is exactly the same as the original qwen2

So i just found out that Qwen2 have tools support. However, ollama current do not have tools support for qwen2 models. And there is a section of Qwen2 github talking about ollama on tools use. And this seems prombles of ollama in older versions. I think this should be resolvable with version 0.3.x

It work! thank you

hlstudio

It works! thank you

#6007 has most progress from the Qwen maintainer, so perhaps we can close this one and focus energy on getting the other sorted out?

@codefromthecrypt thanks! Will close for https://github.com/ollama/ollama/issues/6007


Don't set initial_fetch_duration if its unset

Currently by default we set this to 0s if its not set, which is changing
the default behavior undesirably.
The default recently changed to 15s in Envoy which is a very reasonable default

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

Test name
Commit
Details
Rerun command

gencheck_istio
1f39cf6817a96908f71259cdb1492cb75c602f65
link
/test gencheck_istio

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

/retest


Translate all product link

- Summary

- Test plan

- Description for the changelog

- A picture of a cute animal (not mandatory but encouraged)

👷 Deploy Preview for joyful-taffy-28aa41 processing.

Name
Link

🔨 Latest commit
91fd28dd3ff8d0c2ebd648833f9752bf4354198a

🔍 Latest deploy log
https://app.netlify.com/sites/joyful-taffy-28aa41/deploys/670e3b2acb585100085dd0a6


feat(es/compat): impl pure_getters for optional_chaining

Description:

BREAKING CHANGE:

Related issue (if exists):
Closes #7921

I don't believe it would be easy. Both are referenced many times.


Enable minerva for non-admin users

Look over #144 for details, but use a different approach.
Currently only admin users can use Minerva, and we are abusing GET requests by creating folders.
Change the approach to dealing with users to be the following

when Minerva is enabled as a plugin

create a minerva collection and a minerva group granting read access to that collection
create an analysis (es?) folder under the minerva collection
for any existing user

create a minerva folder under their user
create a dataset folder under the user minerva folder
create a session folder under the user minerva folder
create a source folder under the user minerva folder
add the user to the minerva group

when a new user is added and Minerva is already enabled, listen to the user creation event and follow the above procedure to enable the user for Minerva
figure out details related to running romanesco

maybe add the user to the list of enabled romanesco users
more ideally tie this into the minerva group
this may involve tying in the minerva collection analysis folder

http://www.datamation.com/imagesvr_ce/2498/keynesian-coding.jpg

Also #71.

@mgrauer this is a critical one I think.

#71


operationalinsights - add 202 definition for clusters_update

ARM (Control Plane) API Specification Update Pull Request
PR review workflow diagram
Please understand this diagram before proceeding. It explains how to get your PR approved & merged.

[1] ARM review queue (for merge queues, see [4])
The PRs are processed by time opened, ascending. Your PR may show up on 2nd or later page.
If you addressed Step 1 from the diagram and your PR is not showing up in the queue, ensure the label ARMChangesRequested
is removed from your PR. This should cause the label WaitForARMFeedback to be added.
[2] https://aka.ms/azsdk/support/specreview-channel
[3] List of SDK breaking changes approvers in pinned Teams announcement
[4] public repo merge queue, private repo merge queue (for ARM review queue, [1])
If you need further help with anything, see Getting help section below.
Purpose of this PR
What's the purpose of this PR? Check all that apply. This is mandatory!

[ ] New API version. (Such PR should have been generated with OpenAPI Hub).
[ ] Update existing version for a new feature. (This is applicable only when you are revising a private preview API version.)
[ ] Update existing version to fix swagger quality issues in S360.
[x] Other, please clarify:

adding missed response definition

Due diligence checklist
To merge this PR, you must go through the following checklist and confirm you understood
and followed the instructions by checking all the boxes:

[ ] I confirm this PR is modifying Azure Resource Manager (ARM) related specifications, and not data plane related specifications.
[ ] I have reviewed following Resource Provider guidelines, including
ARM resource provider contract and
REST guidelines (estimated time: 4 hours).
I understand this is required before I can proceed to Step 2, "ARM Review", for this PR.

Breaking changes review (Step 1)

If the automation determines you have breaking changes, i.e. Step 1 from the diagram applies to you,
you must follow the breaking changes process.
IMPORTANT This applies even if:

The tool fails while it shouldn't, e.g. due to runtime exception, or incorrect detection of breaking changes.
You believe there is no need for you to request breaking change approval, for any reason.
Such claims must be reviewed, and the process is the same.

ARM API changes review (Step 2)

If this PR is in purview of ARM review then automation will add the ARMReview label.
If you want to force ARM review, add the label yourself.
Proceed according to the diagram at the top of this comment.

Viewing API changes
For convenient view of the API changes made by this PR, refer to the URLs provided in the table
in the Generated ApiView comment added to this PR. You can use ApiView to show API versions diff.
Suppressing failures
If one or multiple validation error/warning suppression(s) is detected in your PR, please follow the
Swagger-Suppression-Process
to get approval.
Getting help

For general PR approval workflow, see the diagram at the top of this comment.
For guidance on fixing this PR CI check failures, see the hyperlinks provided in given failure
and https://aka.ms/ci-fix.
If the help provided by the previous points is not enough, post to https://aka.ms/azsdk/support/specreview-channel and link to this PR.

fixes #26678

Could someone take a look at this PR? it's fix the swagger so it might not should be "breaking change"?

@ziyeqf Can we update all versions of Clusters.json?

 Please get a breaking changes approval from Azure Breaking Changes Reviewers<EMAIL_ADDRESS>and ARM can review the changes afterwards. Once you have the approval, please remove the "ARMChangesRequested" label from the PR to make the PR visible to the ARM reviewers.

@ms-henglu I do not have the permission to remove ARMChangesRequested label, could you please help removing that? Thanks!

@ziyeqf, please fix errors in the following links, then remove ARMChangesRequested to continue the review.
https://github.com/Azure/azure-rest-api-specs/pull/26679/checks?check_run_id=22813622544
https://github.com/Azure/azure-rest-api-specs/pull/26679/checks?check_run_id=22813665009

@ms-zhenhua could you please help changing the labels as the checks passed now?


Possibility to hide all levels from "made in"?

Annoyance, since it won't even fit on screen.

Not possible, see #69


Update package registry snapshot

Relates to NixOS/nixpkgs#38966.
Side note: I think it'd be great if this repository was somewhere that isn't a "single point of failure" (for lack of a better term), but I don't know if the NixOS project itself has a better storage location for files like this. I asked on IRC but haven't gotten much feedback yet, may be sensible to post it on Discourse in addition.

Thanks! :100:


Request: Support JToken assertions

Hi Simon, is JToken supported here also?
The below doesn't fail for me...
[Fact]
public async Task Test7()
{
    var dt = DateTime.Now.ToLongTimeString();

    var json = "{\"Name\": \"one\", \"Value\": \"val\", \"A\": \"the\",\"B\":\"yes\",\"C\":\"p\",\"Dt\":\"" + dt + "\"}";
    var res = JToken.Parse(json);

    var result = await Verify(res)
        .ScrubInlineDateTimes("T")
        .Assert<JToken>(a => Assert.Equal("thee", a["A"].ToString()));
}

can u try Verify  26.1.6

I'm getting the below error:

This isn't a showstopper so it can wait, also I think JObject would also need changes to be supported.

there are many jtokens. your callback will be hit for all of them


Issues/588

Fixed the build

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@0ac7701). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master       #1   +/-   ##
=========================================
  Coverage          ?   81.81%
=========================================
  Files             ?      126
  Lines             ?     4570
  Branches          ?     1493
=========================================
  Hits              ?     3739
  Misses            ?      470
  Partials          ?      361

Impacted Files
Coverage Δ

src/generators/nodes/EachBlock.ts
82.5% <ø> (ø)

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 0ac7701...03d42ef. Read the comment docs.

Cool!
I moved the patch to branch https://github.com/talklittle/svelte/tree/test-btakita-588 instead of master


[Icon Request]: Support for Kotlin extensions (.kts, .ktm)

Housekeeping

[X] I'm sure this issue is not a duplicate

Icon Type

[X] File Icon
[ ] Folder Icon

Icon Type

[X] Extension
[ ] Filename
[ ] Language ID

Original Icon
Same as the kotlin icon.
Icon Information
Should be the same as the kotlin icon.
More Information:
The Kotlin language has multiple exensions (.kt, .kts, .ktm). It would be nice if they all had the kotlin icon. Currently, the .kts and .ktm file extensions show the default file icon.

If you install a Kotlin extension like this one the icons should appear. These icons are supported via language id.

Reopening this as the .ktm extension is not supported by that extension. @gillescoolen could you provide more context for the .ktm extension?

Alright, I have this Kotlin extension installed, which does not activate the icons for .kts or .ktm, but does activate an icon for .kt.
As far as the .ktm file extension goes, some developers use it for Kotlin modules, but I personally never use it. Just thought I might as well incude it.

Ok, that extension uses several ids for the extensions. We will add it to support it given that it's one of the kotlin most downloaded extensions. Btw, thanks for the clarifications!

No problem, thanks for the work!


Fix array item deletion issue

Description
Deleting array items which are objects will cause the fieldset field's sf-field directive $destroy event handler to try to remove itself from the array again. This was broken in various way previously because of incorrect form keys, but the issue became visible after changes from #870
Checklist

[x] I have read and understand the CONTRIBUTIONS.md file
[x] I have searched for and linked related issues
[x] I have created test cases to ensure quick resolution of the PR is easier
[x] I am NOT targeting main branch
[x] I did NOT include the dist folder in my PR

@json-schema-form/angular-schema-form-lead

Note failing test because travis still not running the build before the tests


Request to have Postgres as a Database option

Feature request:
The Dragonfly manager container currently supports using either MySQL or PostgreSQL:

https://github.com/dragonflyoss/Dragonfly2/issues/1423
https://github.com/dragonflyoss/Dragonfly2/pull/1459

I'd like the helm chart value.yaml to also support deploying with this option.
Use case:
My organization already uses PostgreSQL within our environment. Requiring MySQL would add another component instead of using our existing PostgreSQL environment.
UI Example:
N/A

Hello!  Is this still going to be added?  I see that manager supports PG but the chart has mysql hard coded so I am curious if it will be added eventually.


refactor: make some dark mode colors less contrast

Related to #199
After doing a little studying of dark mode sites, I've come to the conclusion that it's really worth using less contrast for base content color if dark mode is on:

Site
Contrast ratio (dark mode)

docs.microsoft.com
14.36

developers.cloudflare.com
11.45

Mk Docs Material
11.03

appwrite
11.96

stitches.dev/
15.34

developers.cloudflare.com
11.45

angular.io
12.64

Current dark mode contrast ratio - 16.27
New one - ~13.71~ 13.4
This is quite subjective, but the current contrast ratio is a bit of an inconvenience, so darkening the current content color would not be superfluous, given that in fact many sites use less contrast color in general.

Can we also brighten the background a bit so it approaches ~12.5?

I tried to do something with the background too, but it is quite unusually bright then, maybe decrease its brightness just a little bit so that the contrast ratio ends up being 13.4.

I'm looking at the CF docs which IMO is the closest to ours (the others are a bit too... colorful). Their background seems to be grayer than us. But looking at the deploy demo, this change already looks much, much better than before.

Yes, since I don't have a clear understanding of what dark mode should be at all, I just want avoid to make abrupt changes, so the new contrast ratio seems quite good.

Same, I don't even know what dark mode should look like 😄 But everyone seems to implement it differently anyways

Wondering if it's not worth it to make category/toc/navbar/title lighter, and only darken main text and walls of text?

I wouldn't do that yet, the current result is quite good.

IMHO code block is still not very visible (less than before)
And the pre-background too (not sure we have this case on Docusaurus but the Infima demo it's less visible than before)

Otherwise that seems nice

@slorber I'm not aware of a place where the pre background is actually used in Docusaurus, unless the user uses <pre> in JSX (unlikely). Everywhere else, we have Prism themes.

Should be better this time -- the background for inline blocks is darker, and it is also used for multi-line code blocks.

LGTM 👍


NuGet Package

Arash jan,
Would you please add simptip NuGet package. It's really useful for Microsoft software developers. You can find a less than a hour solution to build and deploy NuGet package in here http://docs.nuget.org/docs/creating-packages/using-a-gui-to-build-packages
Thank you ;-)

Hi
As soon as possible I'll do it and inform you.
Best wishes :)


feat: global secret

List

drone globalsecret ls

Add

drone globalsecret add --name foo --value bar

Info

drone globalsecret info --name foo

Update

drone globalsecret update --name foo --event pull_request

Remove

drone globalsecret rm --name foo

High quality PR.
Need a couple of days to crosscheck it with https://woodpecker.laszlo.cloud/environment-variables/#global-environment-variables
How the two features can co-exist.
What do you think about the feature overlap of the two?

env is for organization to setup non credentials data. e.g organization root domain, jira link or other data.
secret is for admin to set token like docker username and password.
in my opinion there are different purpose.

Thank you for the contributions. I'm now processing the backlog of PRs and my goal is to merge this in the coming days.

Thank you for the contributions. I'm now processing the backlog of PRs and my goal is to merge this in the coming days.

@CrabHo I see that this is building on top of the original Drone global secrets feature.
The problem here is that parts of that was not licensed with Apache2. I see that you pulled in the Apache code, but haven't got to the end in figuring out if all files are Apache or not.
It would help if you write down the approach you followed with the non Apache 2 code.

Hello folks,
What's up with this feature? :)
Moreover, are global secret shared with all repositories or only scoped with an organization?

@qwerty287 I think yes, my PR fully replaces this one, except for drone cli part.

I can implement CLI later on in other PR as I did not want to make PR too big :)


Optimize interpreter execution

optimize transitions from interpreter ==> compiled code and compiled code ==> interpreter
optimize interpreter execution

/cc @rmacnak-google @crelier who should own this issue?

"Optimize interpreter execution" is too vague.

There are two aspects to it

optimize the transitions from interpreted mode to compiled code and back (this make require unifying the execution stack)
figure out if the C++ implementation of the interpreter is not efficient enough and if so consider generating a more optimal version

Bytecode mode is removed from Dart VM.


Mandatory host header in http 1.1

The host header is mandatory in every HTTP/1.1 requests. Currently workerman doesn't validate it's presence.
https://www.rfc-editor.org/rfc/rfc2616#section-14.23

Thanks


Can't install

Whatever I try to install Foolslide, after clicking on on install's button I get a white page.
Refreshing I get Database Error 1146.
I am using MySQL 5.6 on hosting service "altervista"

Would need more information, but it's possible that this might be related to #63.

That may be because mysql_connect() has been removed from php7.
Try to open config.php and specify $db['default']['dbdriver'] = 'mysqli'; instead of mysql

So:
I am using PHP5.6 and I am unable to disable strict mode: the hosting provider (its free) doesn't give you all rights,


Adding From<f32>/From<f64> impls for f16/bf16

This adds from/into in the other way than is currently present, making the following code possible:
use half::f16;

fn add_me(a: f16, b: impl Into<f16>) -> f16 {
    a + b.into()
}

fn main() {
    println!("{}", add_me(f16::ZERO, 2.0f32));
    println!("{}", add_me(f16::ZERO, 2.0f64));
    println!("{}", add_me(f16::ZERO, f16::from_f64(2.0f64)));
}

While certainly convenient, these weren't implemented for the same reasons From<f64> isn't implemented for f32: the loss of precision means making the conversion explicit is preferable, to avoid unexpected loss of precision. Instead, use num-traits conversion traits for generic methods like this for f16.


Add statix checker

Substituted -command with -args

Hi, I installed the rust toolchain and statix executable. However, test cases do not seem to work. I manually executed elisp code used to output json data. The out file contains exactly the same lints described in test cases.

Yeah! It works! I added tests


Documentation needs to have examples

🐞 Bug Report
Describe the bug
It would appear that by randomly clicking types in the documentation, a great number, if not most of the types do not have examples nor do they declare their intent. Because of this I find this library to be incredibly difficult to make use of. actually I don't because I can never tell if its types can help solve my problem, or how to actually use them. It would be incredibly useful if someone(s) were to make a concerted effort to increase documentation coverage and ensure new types were covered to a higher standard.

Chiming in here with the same sentiment. A lack of examples and at least a description of what the types do makes this (probably great) library much less useful than it could be.
Here are some of the things that would help the docs site:

Add examples and a description of why you'd want to use it for every helper
Better search: Search seems broken, I just see Preparing search index. A cmd+k menu with quick search would be great.
Group helpers by category Any, Boolean, Class, etc with headings that are collapsible. There is a lot of extra "noise" with the prefixes in front of every sidebar link
Make the code samples legible, the highlighting theme is impossible to read (reported in issues  a few times)
Don't navigate to a new URL on every change as this breaks the scroll position (SPA would work)

For reference, ere are just a few of the other tickets describing docs/examples related topics:

https://github.com/millsp/ts-toolbelt/issues/223
https://github.com/millsp/ts-toolbelt/pull/304
https://github.com/millsp/ts-toolbelt/issues/112
https://github.com/millsp/ts-toolbelt/pull/329


feat: Add buttons to decrease and increase volume, clear the entire song queue, shuffle the song queue, and more. (REOPEN)

Log of changes for each file:

Add buttons for decreasing and increasing volume
Add buttons to clear and shuffle the song queue
Fix index.js to include handling for additional button interactions

The source has been changed to ytsearch because Lavalink can now play music from YouTube if certain plugins are installed. I installed them, and it works.

Alright, bro, now hurry up and merge the code. I'm afraid it'll conflict again, haha!

LGTM

What plugin did you install on lavalink to add it to mine or send me your application.yml

Okay

Im checking it

I will just use one of the free ones, its easier, thanks

The one with a lot of plugins, it works perfectly


Allow Tolk routes to be explicitly mounted in-app

allow routes to be explicitly mounted to avoid conflicts with existing routes in-app.
This is for rails 3.0 compatibility. With rails 3.1 we can make Tolk a
proper engine with an isolated namespace.

My bad, I pushed this to the wrong branch. I've got it fixed on the right branch now; I'll make a new pull request.


Pagination doesn't seem to work with iter_all

When I have code like the following:
from pdpyras import APISession

api_token = '<redacted>'
session = APISession(api_token)

for incident in session.iter_all('incidents', params={'service_ids[]': ['<redacted>'], 'since': '2020-08-01T00:00:00Z'}):
    print("{id}\t{created_at}\t{html_url}\t{description}\t{resolve_reason}".format(**incident))

I only get around 25 incidents. If I change the since date, I get a different 25-ish incidents, so it seems like the pagination isn't working. (Also, I know from the PD website that there are definitely much more than 25 incidents for this query).
Any ideas?
Thanks.

Have this same issue - attempting to bulk update incidents using this library with no luck getting pagination to work properly. My result set is limited the same way.

Hi @jacklund and @blardo,
In the incidents API, if one does not specify both ends of a date range, it will default to one month that begins at since or ends at until. In this example here it would only return incidents in the range 2020-08-01 thru 2020-09-01.
I have submitted an internal issue ticket to document this behavior in the appropriate place.
For all questions specifically pertaining to the API, Customer Support and the PagerDuty forums are the best places to get help.


#3795 raise sortChanged with proper arguments when grid state is restored

This request is related to the issue numbered #3795.
restoreColumns() raises sortChanged() with proper arguments as the event doc says.

Looks great thanks, merging.


fix: requires client field prop in server field components

Fixes a type error when using server components for field labels, descriptions, and errors. The clientField prop will always exist, so the types just need to be reflective of this. Here's an example:
import type { TextFieldServerLabelComponent } from 'payload'

import { FieldLabel } from '@payloadcms/ui'
import React from 'react'

export const MyServerFieldLabelComponent: TextFieldServerLabelComponent = ({ clientField }) => {
  return <FieldLabel field={clientField} /> // `TextFieldClientWithoutType | undefined` is not assignable to type `ClientFieldWithoutType`
}

🚀 This is included in version v3.0.0-beta.105


Talkdesk.Talkdesk version <IP_ADDRESS>0

[x] Have you signed the Contributor License Agreement?
[x] Have you checked that there aren't other open pull requests for the same manifest update/change?

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

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

Microsoft Reviewers: Open in CodeFlow

@microsoft-github-policy-service agree

@microsoft-github-policy-service agree company="Valgreenz"

@microsoft-github-policy-service agree
@microsoft-github-policy-service agree company="Valgreenz"


Make maximum query parallelism configurable.

Scope & Purpose
Some queries can take advantage of internal structure (currently only traversals) to run portions of the query in parallel. Currently there is a hard limit of on the maximum number of threads per query (4). This PR makes this configurable at the server level with a hidden option (but does not change the default of 4).

https://jenkins.arangodb.biz/view/PR/job/arangodb-matrix-pr/11108/

https://jenkins.arangodb.biz/view/PR/job/arangodb-matrix-pr/11109/

https://jenkins.arangodb.biz/view/PR/job/arangodb-matrix-pr/11111/


Update .vscode/tasks.json to run Debug Quarkus command with missing mvnw

Fixes #572
Signed-off-by: Jessica He<EMAIL_ADDRESS>

@rgrunber you were correct! I needed to use another way to find the wrapper command. It should work fine now


Cannot issue new command

EdgeDB Version: EdgeDB 1.0-alpha.3+dev.102.g176dac70
OS Version: Ubuntu

Steps to Reproduce:

Expose an HTTP Port
Send 2-3 requests at the same time

{
  message: 'cannot issue new command',
  type: 'InternalServerError',
   code: 16777216
 }

Does this happen on a particular query for you or regardless of a query?

Send 2-3 requests at the same time

What's your concurrency setting?

Concurrency is set to 4

But 4 doesn't mean 4 requests at the same time max. Does it?

Does this happen on a particular query for you or regardless of a query?

I did a reset Port and created a new Port and i was able to send queries just fine.
It seems some errors mess up with Ports.

When that happens, all queries return that same error. Even if i send one query.

But 4 doesn't mean 4 requests at the same time max. Does it?

It means that only 4 previously unseen queries can be compiled in parallel. But in any case, the server shouldn't error with what you see, so this is most definitely a bug. I'll investigate.

@1st1 I think i've found a reproduction scenario.

Create HTTP Port.
Make requests
Alter schema
cannot issue new command

I can reproduce this. After altering schema, the subsequent (compiled) query works once more, then fails with ISE "unexpected message type '3'" and subsequent requests fail with ISE "cannot issue new command".


[Fix #339] post uint8array instead of tensor to imshow

I guess we can write a test in nb_test.ts to determine if is there any canvas in the document or not (after calling imshow)

In nb_test, add along these lines:
testBrowser(async function notebook_fub4rba5() {
  resetPage();
  await new Promise((resolve) => {
    const el = h(nb.Notebook, {
      onReady: resolve,
      code: "imshow('foo/bar');"
    });
    render(el, document.body);
  });
  // Check that a canvas was created in the output div.
  ...
});

@piscisaureus I tried this and it failed:
testBrowser(async function notebook_imshow() {
  resetPage();
  await new Promise((resolve) => {
    const el = h(nb.Notebook, {
      nbId: "default",
      onReady: resolve,
      code: `import * as pr from "propel"
      const img = pr.zeros([10, 10, 3])
      pr.imshow(img)`
    });
    render(el, document.body);
  });
  const canvas = document.querySelector(".output-container > .output > canvas");
  assert(canvas !== null);
});

I guess it's because nb's ready event does not mean code has been executed and hence there is no canvas in the document.


Please consider fixing issues detected by repository checker

Notification from ioBroker Check and Service Bot
Dear adapter developer,
I'm the ioBroker Check and Service Bot. I'm an automated tool processing routine tasks for the ioBroker infrastructure. I have recently checked the repository for your adapter hausbus_de for common errors and appropiate suggestions to keep this adapter up to date.
Please see the result of the check below.
ioBroker.hausbus_de
   -

ERRORS:

[ ] :heavy_exclamation_mark: [E026] "{'engines': {'node'>='16'}}" is required at package.json, "{'engines':{'node'>='18'}}" is recommened
[ ] :heavy_exclamation_mark: [E033] @iobroker/adapter-core 2.6.12 specified. 3.1.4 is required as minimum, 3.1.6 is recommended. Please update dependencies at package.json
[ ] :heavy_exclamation_mark: [E162] js-controller 2.0.0 listed as dependency but 4.0.24 is required as minimum, 5.0.19 is recommended. Please update dependency at io-package.json.
[ ] :heavy_exclamation_mark: [E204] Version "1.4.0" listed at common.news at io-package.json does not exist at NPM. Please remove from news section.

WARNINGS:

[ ] :eyes: [W040] "keywords" within package.json should contain "ioBroker"
[ ] :eyes: [W105] Missing suggested translation into uk of "common.titleLang" in io-package.json.
[ ] :eyes: [W109] Missing suggested translation into uk of "common.desc" in io-package.json.
[ ] :eyes: [W115] "common.tier" is required in io-package.json. Please check https://github.com/ioBroker/ioBroker.docs/blob/master/docs/en/dev/objectsschema.md#adapter.
[ ] :eyes: [W181] "common.license" in io-package.json is deprecated. Please define object "common.licenseInformation"
[ ] :eyes: [W184] "common.main" is deprecated and ignored. Please remove from io-package.json. Use "main" at package.json instead.
[ ] :eyes: [W184] "common.materialize" is deprecated for admin >= 5 at io-package.json. Please use property "adminUI".
[ ] :eyes: [W519] Consider adding plugin "@alcalzone/release-script-plugin-manual-review".
[ ] :eyes: [W522] Please consider migrating to admin 5 UI (jsonConfig).

Please review issues reported and consider fixing them as soon as appropiate.
Errors reported by repository checker should be fixed as soon as possible. Some of them require a new release to be considered as fixed. Please note that errors reported by checker might be considered as blocking point for future updates at stable repository.
Warnings reported by repository checker should be reviewed. While some warnings can be considered as a suggestion and be ignored due to good reasons or a dedicated decision of the developer, most warnings should be fixed as soon as appropiate.
Feel free to contact me (@iobroker-bot) if you have any questions or feel that an issue is incorrectly flagged.
And THANKS A LOT for maintaining this adapter from me and all users.
Let's work together for the best user experience.
your
ioBroker Check and Service Bot
@mcm1957 for evidence

E162 and E204 should be fixed with https://github.com/hausbus/ioBroker.hausbus_de/pull/54

This issue has been replaced by new isse #57
This issue can be closed.
your
ioBroker Check and Service Bot


add possibility to modify ControllerBuilder in ControllerCommandConfig

It is hard to customize the StartController without a possibility to modify the builder. Otherwise it is necessary to copy paste a lot of code: https://github.com/openshift/cluster-policy-controller/pull/65/commits/a94cb90b7397b26d1fc72c2ffaa29d27d41a541e.
This change would be helpful for https://github.com/openshift/cluster-policy-controller/pull/65/

after a discussion, the default config should be enough for our use case


Second header indent

Is it possible to increase the indentation of the second header? The first and second header share the same indentation. Here's a screenshot of what I mean:

This is what I have in my config:
return {
    "hedyhli/markdown-toc.nvim",
    ft = "markdown",
    main = "mtoc",
    cmd = "Mtoc",
    keys = {
        { "<leader>ct", "<cmd>Mtoc<CR>", desc = "Insert TOC (mtoc)" },
    },
    opts = {
        toc_list = {
            indent_size = 2,
        },
    },
}

what's the use case for this? If you want something indented, you can increase the heading level.
# Heading 1
## Heading 2
### Heading 3
#### Heading 4

The heading level are increased accordingly but when I run Mtoc the indentation for header 1 and header 2 are the same:

It should look something like this:

Here's a short gif of what I mean:

Is this the expected result or would it be possible to indent header 2?

I se what you mean now. Will take a look, thanks!

This should be fixed now. Let me know if you're running into any other issues.


Why dont you try making a web app for this? (feature)

Basically its like the discord web app but its betterdiscord instead! :)

There was one at one point that was created by one of the contributors for BD, but AFAIK he has not been giving it full updates.
The main reasons that it existed was due to there being token loggers and other not-so-fun things on the ones that exist on the Chrome store, and that a lot of people do request this.
The main reasons that it isn't kept up is because it is a lot of work to try and make everything work with the browser version of Discord (mainly with stuff like people not fully understanding how to upload their addons, and some addons not working with the Web version of BetterDiscord), and it costs money to put it on the Chrome store, which is not something that the original developer (of the most official version of it) was not willing to do.
So if someone does end up making a new version of BD for the browser version of Discord, it'll most likely be at least slightly separated from the main BD.
So, I guess the reason is because it's a whole lot more work on top of the BetterDiscord that already exists, and because BetterDiscord was mainly designed for the desktop version of Discord.

If you want a client mod that runs in the web browser I'd recommend using GooseMod. A lot of BD's features are based on electron APIs and a lot of BD plugins are highly based around file system access and direct access to node/electron apis which BDBrowser tried to emulate but it was mostly based on compatibility hacks for a system and plugins that were never intended for the browser.

The problem with the store could be solved by making it into an userscript, using existing extensions like TamperMonkey, ViolentMonkey, and so on.

No thanks!


Spotify command issue

Hi AVS team,
SDK 1.8.1 , ARM based OpenWRT embeded linux.
I try the Spotify Music self-test. I found a strange issue while issue these commands:
Alexa, play amazon music.
Alexa, play spotify.
Alexa, stop music
Alexa, resume music.    ---> it will resume amazon music.
But, if I say: "Alexa, resume." it will resume the previous spotify music.
could you help to check your team if this behavior is expected ?
Thanks,

Hi @zeusshuang,
Can you please double check on the alexa portal / alexa app that she understood "resume music"?
Thanks,
Celina

Hi @celinval ,
This command is defined in AVS_SpotifyCertificationChecklist_4_03_2018 , item 9.
I checked Echo dot2, it's working as expected result. (i.e., it will resume Spotify correctly)
Regards,

Hi @celinval ,
I would like to add another issue for this same verification test.
In section 3, item 1, My playlist.
Is there any limitation for the "play list" naming ?
We found a strange result.
If I named it's "my_playlist", and ask, Alexa, play my myplaylist from Spotify, it's working and play the list.
But if I named "script verification", it will say cannot fine the song "script verification" from Spotify. (I could make sure the wordings are recognized in history log).
Could you help to check your team and see if there is any naming limitation ?
Thanks,

Hi @zeusshuang,
We were able to reproduce the spotify issue. I'll add to our backlog to fix the issue.
Regarding the name limitation, will it work if you say "Alexa, play script verification playlist from Spotify"?
Thanks

Hi @celinval ,
Thanks for the feedback.
And yes, it's working for "Alexa, play script verification playlist from Spotify".
If it's the case, will it need to review the AVS test plan for Spotify ?
Thanks,

Hi @celinval ,
I updated  the original issues report, please help to check it.

Hi @zeusshuang, is this issue still valid?  If not, could we close this?

I verified it on SDK 1.10 and it's working without problem.
We can close it now.


[Conference] PyJamas 2022

Conference Name
PyJamas
URL
https://pyjamas.live
Conference Dates
26-27 November 2022
Conference Type
online
Conference Location
No response
CFP URL
https://pyjamas.live/cfp/
CFP Dates
18 Sept 2022
Conference Description
24 hours streaming of talks about Python and connecting with Python communities around the world that you can access at home.

Thank you for adding this event! \n We'll happily consider it for the next episode. \n\n The Issue autocloses but we'll be reviewing it at https://github.com/users/kjaymiller/projects/4/views/3


Deal a bit better with Null quals

This moves us to only storing a single copy of the original QUAL value as a string. It is not converted explicitly. In vcfpaste, null values are handled explicitly.

Coverage remained the same at 44.944% when pulling 7020bbd3e191c9c66e516e0e483f5b4415fff6b7 on ernfrid:null_quals into<PHONE_NUMBER>1aa3fdc8e3eeea519003b36b7ead62 on hall-lab:master.


Add local testing targets (resolves #1741)

This PR adds two new targets to the makefile for testing locally:

test_offline and
integration_test_local.

The test_offline target gives a cleaner interface to run unit tests without running the docker/appliance tests. Previously, one had to set the environment variable TOIL_DOCKER_REGISTRY to the empty string and then run make test. Now, one can simply run make test_offline and get the results that they expect. Help messages were changed to clarify the new process. Name taken from
The integration_test_local target now runs in series and output is not captured. This can still use some improvement however because running all of the tests in series takes a looong time and produces a lot of output.
Other changes:

target that are only designed to be run on Jenkins have updated names (previously this was test_parallel and integration_test) prepended with jenkins_. The old target names still exist temporarily, but will be phased out soon. Probably no one was using the to develop locally anyway.
targets that meant to run locally: test, test_offline, and integration_test_local are now all run using pytest's -s option which allows the output to be printed to the stdout like one expects.

Things that still need to happen (though not necessarily right now):

add to the documentation information on how to use these targets, etc.
The "jenkins only" targets should depend upon the check_running_on_jenkins targets. This was held off because it's possible someone is using these targets locally and this would be disruptive.

@ejacox Sorry I had never pinged. This is ready for review.

Jenkins, test this please.

Jenkins test this please

I cannot explain why integration tests are not passing. The last substantive commit, 75fbb8a passed fine and the most recent commit, which was directly after, was only documentation.
I ran integrations test locally with complete success using the target that jenkins is configured to use: integration_test.
Have Jenkins configurations changed or something? @ejacox any ideas?

ah.. Possibly related to "slow registry operiations" currently reported for quay at status.quay.io ?

@ejacox why did that fix things? No target that was called by Jenkins used that option as far as I knew.
Also I thought that option would be useful for local testing.

-s disables capturing of test output, which is used to tell Jenkins if the tests passed I think

Can this get merged?

Woohoo, thanks @jessebrennan & @ejacox !


remove stotch.io

Pull Request Template
remove scotch.io referencing issue #201
Please ensure all items below are checked before creating a pull request:

[x] My changes were made in the resources folder, not in the auto-generated README.md file or db folder
[x] My submission is formatted according to the guidelines in the contributing guide
[x] My submission is ordered alphabetically based on the resource name
[x] My submission has a useful description
[x] I have searched the repository for any relevant issues or pull requests

let me know if you have any questions. tthanks! #201

Thank you!


resolve linting issues in the code

Describe the bug
resolve linting issues in the code
How to Reproduce?
Please list the steps to help development teams reproduce the behavior

Run any known linter to find and resolve the linting issue in the code

Expected behavior
resolve linting issues in the code

Linter issues are fixed by PR https://github.com/IBM/ibm-spectrum-scale-csi/pull/974


Russian translation: Basic Lessons status

[x] Collections
[x] Enum
[x] Pattern Matching
[x] Control Structures
[x] Pipe Operator
[x] Functions
[ ] Composition
[ ] Mix
[ ] Sigils
[ ] Documentation
[ ] Testing

See #220 :)


Merging 0.6.x-dev into 0.7.x-dev

@rquadling should these commits be merged into 0.7.x-dev ?

I'd like to do away with branched versions of development as we really have no need.
Also, merging into master and using /docs would allow github docs to work without the need of gh_pages (branch? second head?)
I think this will simplify ongoing development.

closing this PR i will merge directly to master instead.


Update list of validators and format to display

@geohacker , could you take a look on the improvement to display the validators.

the md file too: https://github.com/osmlab/osmlint/blob/doc/validators.md

Hey @Rub21 - thank you for the PR.

Let's call createValidators.md as CONTRIBUTING.md
I'm not sure if we need that shell script to print the help - this plus the validators.md file becomes redundant and adds one more place to update when we are contributing validators.
Can we just print the validators.md file?

@geohacker

Let's call createValidators.md as CONTRIBUTING.md

done!

I'm not sure if we need that shell script to print the help - this plus the validators.md file becomes redundant and adds one more place to update when we are contributing validators.
Can we just print the validators.md file?

Yes, could be redundant, lets print the  validators.md , Done!


Population Density widget control

Look into the possibility of changing from people per cell to People per Sq-Klmtr
Verify the advantages over the other before implementation

Updating. I recall that this for Map 2, right?


wittyer module

Description of feature
Push wittyer to nf-core/modules
example: https://github.com/nf-core/modules/tree/master/modules/nf-core/bcl2fastq

Relates to https://github.com/nf-core/modules/issues/4739

Was completed with https://github.com/nf-core/modules/pull/5171


Ffe-file-upload redesign

Big redesign of ffe-file-upload, new design can be seen here (requires access): https://www.figma.com/file/cn3VFs107XnyxeLo4Du4ny/Fil-Opplasting---Master?node-id=0%3A1
Some examples of the new design:

These are breaking changes.

It would be great if you could squash the commits since lerna will release a new version for each one. It's also good to include the text BREAKING CHANGE in the commit message as that will make lerna release this as a new major so noone accidentally gets API changes when updating minor/fix version.
I'm not sure if you need separate commits for the CSS and React packages or not.

You do need separate commits for CSS and React packages, as they are published to npm separately.

new PR: https://github.com/SpareBank1/designsystem/pull/694
closing this one


ECONNRESET. Message: tunneling socket could not be established, statusCode=503.

Troubleshooting
This issue is already fixed in the Deploy task. The same has to be replicated in this too. #8090
https://github.com/Microsoft/vsts-tasks/issues/8090#issuecomment-414916552
Environment
VSTS is being used for the deployment to a Windows web app on Azure using Service Principal
The Service Principal has contributor access on the resource group
Agent - Hosted or Private:
Private Windows Agent is being used over an Internet proxy
Issue Description
The task is being used for management of a Windows Web App. The Start/Stop operation goes through with in a min. But the task takes 8 to 9 mins to complete the execution. When debug logs are enabled we have observed that its failing on an attempt to update the deployment history and its re-attempting it 5 times after every 1 min. This is causing the delay of the execution and there is no way we can mask the task from attempting the deployment history update.
Error logs
2018-08-21T15:20:24.5754927Z ##[debug]build.sourceVersion=07c48833789f38ca37099893f39a1ec39bf7f4c9
2018-08-21T15:20:24.5755446Z ##[debug]build.repository.name=xxxxxxxxx
2018-08-21T15:20:24.5755939Z ##[debug]build.repository.provider=TfsGit
2018-08-21T15:20:24.5759274Z<EMAIL_ADDRESS>2018-08-21T15:21:24.0974449Z ##[debug]Encountered a retriable error:ECONNRESET. Message: tunneling socket could not be established, statusCode=503.
2018-08-21T15:21:26.0995672Z<EMAIL_ADDRESS>2018-08-21T15:22:26.0047963Z ##[debug]Encountered a retriable error:ECONNRESET. Message: tunneling socket could not be established, statusCode=503.
2018-08-21T15:22:32.0053097Z<EMAIL_ADDRESS>2018-08-21T15:23:32.3024458Z ##[debug]Encountered a retriable error:ECONNRESET. Message: tunneling socket could not be established, statusCode=503.
2018-08-21T15:23:46.0096952Z<EMAIL_ADDRESS>2018-08-21T15:24:46.0090902Z ##[debug]Encountered a retriable error:ECONNRESET. Message: tunneling socket could not be established, statusCode=503.
2018-08-21T15:25:16.0107429Z<EMAIL_ADDRESS>2018-08-21T15:26:16.0101812Z ##[debug]Processed: ##vso[task.logissue type=error;code=ECONNRESET;]

@pallavkaulwar , we fixed this issue in Azure App Service task as well, but only App Service Deploy task is deployed. Our daily deployment would deploy all the tasks and the issue should be resolved in ~1-2 weeks.

@vincentdass I don't see that fixed in "vsts-tasks/Tasks/AzureAppServiceManageV0/operations/AzureAppServiceUtils.ts"
Its still being used as "var response = await webClient.sendRequest(webRequest);"
At the same time I'm not sure if I'm looking at the right file.
Can we by any means push the fix faster, as we have multiple places this is effected on.

@pallavkaulwar , thanks for the notification. Will resolve the issue in Mange task.

@pallavkaulwar . Can you please send the debug logs of Manage task to<EMAIL_ADDRESS>We can get into call to investigate further on why the error occurred in first place.

@vincentdass Replied from my org ID. please provide a window to schedule a call

Closing this thread as there is already a open thread #8090


Consider splitting validators into multiple validators

now that we support multiple validators, consider splitting the current validators into more than one validator as appropriate, for example could pull the UniqueEmail validation out of UserValidator and into a seperate stand alone validator, which would let us remove the Option flag for it as well

Split the user validator into multiple ones.

@HaoK will this happen for Beta8?

This is a nice to have and not really sure this is the right thing to do.  Moving to 1.0.0 backlog is probably best


Python client

A Python client for Doorman:

A client talking to the Doorman service using gRPC.
A QPS based rate limiter.
Installable with Pip.

and possibly integration with Requests for throttling HTTP requests from the client side?

Is this being worked on?

@ryszard I would like to pick this up.


Add web.config to all samples to ensure Azure deployment works

I recently tried to deploy some of these samples to Azure using Teams Toolkit. However, the apps do not start properly since the Bicep file creates Windows App Services, yet there is no web.config overrride to point the IIS Server to the right JS start file. I copied a sensible web.config which is also used by other Teams samples to ensure users do not have trouble deploying these samples to Azure.

@Zenulous thanks for the catch and contribution! I'm just double-checking with the Toolkit Team before merging your PR; we really appreciate the community participation.


Initialising a dataframe with empty data and column names fails

The following fails:
pd.DataFrame([], columns="a b".split())

I'd expect this to be equivalent to
pd.DataFrame(None, columns="a b".split())

Is there any reason not to do it like this?

Why do you expect this to work, you've passed a list with a single empty element? It is clearly not the same shape when you pass a list of columns containing 2 elements?

Well, the following works:
pd.DataFrame([(1, 2), (2, 3)], columns="a b".split())

If I pass an empty list that is merely the empty variant of the same statement.

No, it's clearly different the DataFrame ctor is trying to deduce the final shape of the df, as you've passed an iterable object it'll assert that the shapes of the passed data and columns agree which they don't this is different to when you explicitly pass None

this is fixed by #9939. Its an edge case in the inference. You can't deduce the shape of the data, but you do have the column information so it just returns an empty frame. Either the data has to exactly match the number of columns in a list or has to be empty.


Live Gekko in ui can't load local data?

Note: for support questions, please join our Discord server

I'm submitting a ...
[ ] bug report
[ ] feature request
[ ] question about the decisions made in the repository

Action taken (what you did)

Expected result (what you hoped would happen)

Actual result (unexpected outcome)

Other information (e.g. detailed explanation, stacktraces, related issues, suggestions how to fix, links for us to have context, eg. stackoverflow, etc)

Stitching in the UI would be sooo cool! 👍

Will be supported in the next version of the UI :)

Wand

Did this actually get fixed? Since it seems to still not stitch in the UI.


Simultaneous audio playback

Its not really an issue, but more of a feature that would make this great. If we could listen to the song while making the midi, it makes it much easier to make small changes and its just great for quality of life. Thanks!

feature added in 105237e for 0.9.2! get the latest release and be sure to look at "info & usage" for details

Sorry for bothering you again, but when I put the song.wav file in the same folder as the .tmb file, it says "no ffmpeg loaded!" What does this mean and how do I fix it?

I have the same problem. I have tried everything I could think of to try and fix it, but nothing works


fix(doc): version correction

PR summary
Attempt to fix version mismatch was not correct. Refixing to v0.2.11

PR Checklist
Please make sure that your PR fulfills the following requirements:

[x] The commit message follows the Angular Commit Message Guidelines.
[ ] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been added / updated (for bug fixes / features)

PR Type

[ ] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] New tests
[ ] Build/CI related changes
[x] Documentation content changes
[ ] Other (please describe)

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


The visual features for STVQA

Hi, thanks for your sharing.
I find that the STVQA method needs the visual features with the shape [batch_size, fnum, feat_dim, w, h].
How can I get this kind of visual feature?
Looking forward to your reply and thank you.

Sorry that I am recently too busy to go back to this work and manage the feature (it's large). I will release it after next week.

Sorry that I am recently too busy to go back to this work and manage the feature (it's large). I will release it after next week.

Thanks a lot~

Hi, the spatial features are available for downloading.

Hi, the spatial features are available for downloading.

Thanks very much！

Hi, the spatial features are available for downloading.

Thanks very much！

Hi, can u share the spatial features ? The download link is not work now.

Sorry that I have removed the feature for the limited cloud storage as also the method is out-of-date. If you really want the feature, please use the code provided by HCRN to extract the feature yourself. Thanks for the interest~


Request - Nintendo switch account support

Hi ! Do GOG Galaxy can do the support of an nintendo account ?

I doubt it, there would have to be a website with this information available to the public and an API to access it which I'm not sure exists.

Every Nintendo Switch owner has an account that can be accessed via the website: https://accounts.nintendo.com/ You cannot check your games directly on it, but this information is probably attached to it because, first of all, there is such a list of games in the console Secondly, when you are logged in, when you enter individual games, you get information whether you already have it.
As for the API, take a look here: https://dev.to/mathewthe2/intro-to-nintendo-switch-rest-api-2cm7

Yes plssssss make it plssssss

Thanks for all your input. I'm closing this thread as it does not directly refer to the API.
For further discussion should be held within any community space for example: https://github.com/Mixaill/awesome-gog-galaxy/issues/19


fetch relative to layouts

#6337. Just a failing test for now, as I realised that there's no sense proceeding until #6541 is merged.
Couple of thoughts:

I think the only real way to make this work is to include a regex with each layout
In #6337 we concluded that a fetch in src/routes/foo/+layout.js should resolve relative to /foo/, not /foo. Since it would be very weird if src/routes/foo/+page.js then resolved relative to /foo, it probably makes sense for all relative fetches to resolve as though the trailing slash were there.

Please don't delete this checklist! Before submitting the PR, please make sure you do the following:

[ ] It's really useful if your PR references an issue where it is discussed ahead of time. In many cases, features are absent for a reason. For large changes, please create an RFC: https://github.com/sveltejs/rfcs
[ ] This message body should clearly illustrate what problems it solves.
[ ] Ideally, include a test that fails without this PR but passes with it.

Tests

[ ] Run the tests with pnpm test and lint the project with pnpm lint and pnpm check

Changesets

[ ] If your PR makes a change that should be noted in one or more packages' changelogs, generate a changeset by running pnpm changeset and following the prompts. All changesets should be patch until SvelteKit 1.0

we decided against this


Entity id can be hidden by id field

Today, we collate (box) data from multiple users into a single index to provide a simplified query ability.
However, there is no field in general on an Entity that we can use to identify the owner (or source) of that Entity.
Shell has a working solution that involves encoding the owner id into the Entity id itself, which works well. However, sometimes the Entity's schema creates an id field obscuring access to the Entity's (symbol) id.
There is a note in the code that (symbol) id shouldn't be exposed in any case, but it's very useful today for the boxing affordance.
Suggested fix incoming as PR.

Hiding the id is part of defense-in-depth for privacy protections, to make re-identification harder (we'll also have to be careful in the future with timestamps and other meta data). And really it should be the case that it never gets into the innerPEC anyhow!
However, owner or source or whatever this concept evolves to will be a thing. Maybe it would be better to introduce $owner() now and just return the part of the id where the owner is encoded?

I guess I'm inclined to do the tiny fix instead of a half-measure. I didn't make privacy worse, as the id was already available (all I did was alias it). But that's just my $0.02.
Fwiw, I will soon introduce this method into dom-particle:
/** @method boxQuery(box, userid)

Returns array of Entities found in BOXED data box that are owned by userid
*/

This is closer to what we ultimately want in the API. If we can implement this in the outerPEC, then we don't have to expose anything from the ID. Seems like we need new handle API to make the better solution work, I didn't want to do that without consultation with runtime team.

I looked at the userIDComponent stuff in runtime, and it looks like I can store the owner id in the uid: component on the Entity id. Then Particles can grab that information via entity.getUserID()].
This way I can remove the $id alias and generally none of this code will rely on whole id access.
@shans is it ok for me to use uid: this way?

This seem ok to me, yeah.

We used the userIDComponent to create getUserID API to avoid Particles handling id's directly.
We can consider variations and so on, but the immediate problem is resolved.


Added and updated tests

Updated and added more tests. Fixed some minor bugs

Codecov Report

Merging #31 (f124b7a) into main (f632209) will not change coverage.
The diff coverage is 0.00%.


New bug form

What does this PR do?
Created issue form using BETA YAML file support and removes MD issue form.
Does your PR introduce any breaking changes? If yes, please list them.
I'm not sure of the impact on this: https://github.com/Lightning-AI/lightning/pull/14017/files
Before submitting

[ ] Was this discussed/approved via a GitHub issue? (not for typos and docs)
[X] Did you read the contributor guideline, Pull Request section?
[X] Did you make sure your PR does only one thing, instead of bundling different changes together?
[ ] Did you make sure to update the documentation with your changes? (if necessary)
[ ] Did you write any new necessary tests? (not for typos and docs)
[ ] Did you verify new and existing tests pass locally with your changes?
[ ] Did you list all the breaking changes introduced by this pull request?
[ ] Did you update the CHANGELOG? (not for typos, docs, test updates, or minor internal changes/refactors)

PR review

[X] Is this pull request ready for review? (if not, please submit in draft mode)
[ ] Check that all items from Before submitting are resolved
[ ] Make sure the title is self-explanatory and the description concisely explains the PR
[ ] Add labels and milestones (and optionally projects) to the PR so it can be classified

Did you have fun?
Make sure you had fun coding 🙃
cc @borda @rohitgr7

We had good instructions how the user can collect the environment information, with a simple command and script we provide. This is now gone.
This is valuable information that is now completely missed. The result is:

More work for us: If the information is not provided, we have to ask and tell them how to collect it. This increases the time it takes for us to arrive at a reproducible problem, and delays the arrival of a fix.
More work for them: Instead of getting simple command to run, then copy paste the information into the issue, the user now has to collect this manually and type it into the issue page.

Can we consider adding this back? This is very important to me, and I believe also for the team.
cc @edenlightning @Borda @carmocca

@awaelchli @edenlightning @Borda @carmocca
https://github.com/Lightning-AI/lightning/pull/14736


add MiniMap

resolves #26

Love it!
However, I'm noticing some papercuts at both extremes of the zoom (really close, really far).
Really far:

Really close:

Would it instead be possible to "lock" the minimap to the closest fit of the full extent of the layer, that way we don't lose our location context as we zoom in and out far?

do you have a quick way to get the bounding box of each area? i think this will require hard-coding that in much as how the center of each area is currently included...
https://github.com/Saijin-Naib/UAVArena/blob/90bf96ca38fbf4000bc7f08bc8444826eb04eba6/main.js#L10

or do you have links to the processed orthophotos (not tiles) that I can then download and drop into QGIS to figure out the bounds for each of the 3 scenes? i wasn't able to load the folder locations into QGIS nor geojson.io to draw/calculate the bounds.


Right sidebar and visualiser

I have plans for a right sidebar in the ui, it should display statistics such as number of connected peers, current download/upload rate, etc.
Also a force-directed graph of the routing table would be awesome.

No longer relevant.


Error building sample app

I followed Steve Sanderson's very nice article "ASP.NET Core + Angular 2 template for Visual Studio". Everything seems OK, but when I try to run the project it complains and refuses to run (see stack trace below). So I'm filing an issue here as suggested in the article. Thanks!

An unhandled exception occurred while processing the request.
Exception: Call to Node module failed with error: Error: Zone.js has detected that ZoneAwarePromise (window|global).Promise has been overwritten.
Most likely cause is that a Promise polyfill has been loaded after Zone.js (Polyfilling Promise api is not necessary when zone.js is loaded. If you must load one, do so before loading zone.js.)
at Function.Zone.assertZonePatched (C:\Users\Alan\Source\Repos\Angular2ApplicationTest\Angular2ApplicationTest\node_modules\zone.js\dist\zone-node.js:38:23)
at new NgZoneImpl (C:\Users\Alan\Source\Repos\Angular2ApplicationTest\Angular2ApplicationTest\node_modules@angular\core\bundles\core.umd.js:6219:18)
at new NgZone (C:\Users\Alan\Source\Repos\Angular2ApplicationTest\Angular2ApplicationTest\node_modules@angular\core\bundles\core.umd.js:6368:30)
at PlatformRef_.bootstrapModuleFactoryWithZone (C:\Users\Alan\Source\Repos\Angular2ApplicationTest\Angular2ApplicationTest\node_modules@angular\core\bundles\core.umd.js:6913:26)
at PlatformRef.bootstrapModuleFactory (C:\Users\Alan\Source\Repos\Angular2ApplicationTest\Angular2ApplicationTest\node_modules@angular\core\bundles\core.umd.js:6904:25)
at C:\Users\Alan\Source\Repos\Angular2ApplicationTest\Angular2ApplicationTest\node_modules\angular2-platform-node\node-platform.js:439:38
at tryCatch (C:\Users\Alan\Source\Repos\Angular2ApplicationTest\Angular2ApplicationTest\node_modules\aspnet-prerendering\node_modules\es6-promise\dist\es6-promise.js:410:12)
at invokeCallback (C:\Users\Alan\Source\Repos\Angular2ApplicationTest\Angular2ApplicationTest\node_modules\aspnet-prerendering\node_modules\es6-promise\dist\es6-promise.js:425:13)
at publish (C:\Users\Alan\Source\Repos\Angular2ApplicationTest\Angular2ApplicationTest\node_modules\aspnet-prerendering\node_modules\es6-promise\dist\es6-promise.js:393:7)
at flush (C:\Users\Alan\Source\Repos\Angular2ApplicationTest\Angular2ApplicationTest\node_modules\aspnet-prerendering\node_modules\es6-promise\dist\es6-promise.js:121:5)

Same issue for me, was working previously

@alcohen, I have gotten around the issue by changing the dependency on Angular services to "1.0.0-beta-000019"
"Microsoft.AspNetCore.AngularServices": "1.0.0-beta-000019"
let me know if you come right ?

Same issue for me

@McGune have you tried my solution above ?

Yes, it worked form me.

Works. Thanks.

You can eather block AngularServices on version 1.0.0-beta-000019 as advised here or apply patches from 28. nov.
https://github.com/aspnet/JavaScriptServices/commits/dev

@hheexx Im not sure what that link is pointing out ?

commits on this repository

If you want to use AngularServices  1.0.0-beta-000020 or later you have to apply commits from 28.nov to template

Thanks for reporting this. Will publish a fix shortly.

thanks @SteveSandersonMS

The fix is live now. Please update to version 1.0.0-beta-0000021 (or just go back to specifying 1.0.0-* but make sure it fetches the latest).
If you still get any issues please let me know!


Remove Guice ExecModule

We're going to remove Guice from Embulk v0.10.33. This pull request is to remove Guice's ExecrModule, which has been used to manage types of Guice bindings -- logger, BufferAllocator, and TempFileSpaceAllocator. They will be managed directly in ExecSessionInternal, instead.

Thanks!


JSFiddle demonstration is broken

Just double checked with Chrome and Firefox - everything is ok.
Here is the screen from Chrome

I transferred the code to CodeSandbox - please try.

This one is working nice!


Support updating tags of provisioned AWS resources

cluster-image-registry-operator provisions S3 bucket to use as a registry backend for the cluster registry. During provisioning the operator reads the ResourceTags from the Infrastructure platform status and passes them as arguments during the creation call.
PR is for updating the tags of S3 bucket created by operator whenever ResourceTags in the Infrastructure gets modified.

/hold until openshift/api#1064 is merged

/cc @TrilokGeer @josefkarasek @arjunrn @lmzuccarelli

/assign @bharath-b-rh

/close


As support, I want an email to be sent to eduTEAMS support automatically when a user account is deleted from SBS

The email should contain the user's attributes and a request to delete the user from Perun.
Kunnen er meerdere addressen (of ccs) in de SBS mailconfig?

Configure the following email for sending the deleted-user email.

I've deleted a user from TEST, using impersonation. I've asked eduTEAMS if they have received the email asking them to delete this user from Perun.

Nee, dit zou moeten werken.  IK heb  zojuist gechecked dat er alleen een mail naar sram-beheer wordt gestuurd.
De var mail.eduteams_email staat correct ingesteld op test.

Fixed in https://github.com/SURFscz/SBS/commit/6d9ec6a8ecc43092be48db3e8ca3bb96426776e3

confirmed

confirmed


Summarize Filtered Data

When a file is filtered and then summarized the result given is the same as when the unfiltered file is summarized. The result should only be on the filtered records. To recreate, use the following steps:

Open the attached file Sample in Strata.
Right click the Amount field and click Summarize.
Save (untitled) as Unfiltered.
Filter Sample on Claim.
Right click the Amount field and click Summarize.
Save (untitled) as Filtered.

Notice that the values in Filtered are the same as Unfiltered. The values in Filtered that should be different are:

Maximum: 0.00
Sum_Amount: 0.00
Average: 0.00
Total Count: 1

Sample.zip
Version 5.0.0 of Kirix Strata

This has been fixed and will be included in the upcoming dot-release. Thanks for the report.


[Feature Request]: click on item task to view recipe in recipe viewer (emi/rei/jei)

Is your feature request related to a problem?
No response
Solution(s)
No response
Describe alternatives you've considered
No response
Mod Version
1.1.7
Mod Loader Version
fabric loader 0.14.25, fabric api 0.91.0+1.20.1
Mod Loader
No response
Additional context
No response

Implemented

Can we get this reopened for EMI explicitly?
FTB-Mods-Issues#312 is actually the reason we contributed to Heracles in the first place, and we'd be remissed if we didn't track whether Heracles can do better than a 1.5+ year turnaround time.