Add integration test for CSV bulk upload to verify FHIR bundle

Background
In order to increase our test coverage of our FHIR conversions, we should add additional testing that will test from our backend API to our queue. Previously we have had an issue where a change to our result object caused a regression in our FHIR bundle, but was not caught by conversion tests alone.
Action requested

Add integration testing for submitting a csv file and verify the FHIR bundle sent to ReportStream's /api/waters matches what is expected

Acceptance Criteria

An integration test is added for submitting a csv file and

Additional context

Look at 690c92b for a rough outline of how we could add an integration test and verify that ReportStream is called correctly

see comments on #6075 for thoughts on how to handle date/times


Add Validator/StakingFacilities

Please add our toml for StakingFacilities to the Testnet2 Genesis Validator

merged manually into public-testnet-3


Add more curves

See https://en.wikipedia.org/wiki/List_of_curves

Deprecated issue - #14 already implemented. The user will now be able to include their own custom curve path function.


ng-table 显示页码时, 同时添加数据 修改当前页码到最后一页, 页码条当前页渲染错误

Version
0.6.14
Environment
chrome latest 0.6.14
Reproduction link
https://stackblitz.com/edit/ng-zorro-antd-setup-umkwhw?file=app%2Fapp.component.ts
Steps to reproduce
https://stackblitz.com/edit/ng-zorro-antd-setup-umkwhw?file=app%2Fapp.component.ts
同时往 dataset 中加数据
一直点最后一个按钮, pageIndex 改变, 但是 当前 页码 渲染错误
What is expected?
页码 随 pageIndex 改变
What is actually happening?
页码 随 pageIndex 不改变
Other?

fixed in version 0.7.0 closed


[Suggestion] MultiVerse Chat Extension

I've been scouring the internet for a plugin that can control chat per world how I would like it to, but am failing to find anything othat than local/global chat.
I already use Multiverse Core and Inventories, so if I can keep as much of it within Multiverse as possible, it would makes my life, and i imagine others admins lives so much easier.
I wonder if this would be good to have as another branch of Multiverse > Multiverse Chat
Essentially, I'm looking for per-world chat, with the ability to talk globally given they have the permission, and a toggleable admin mode that shows all chat across worlds.
There are a number of permissions and configs that I think could make this suggested plugin very flexible and useful.
Chat Format - Configurable
[(mv world alias)] (Existing format) (message)
i.e. [Survival] {Admin} JCOnline >> Hello Survival!
[Global] (Existing Format) (message)
i.e. [Global] {Admin} JCOnline >> Hello World!
Commands
(Message) - (Default) Send message to current world only.
!(Message) - Sends message globally (Prefix can be configured).
/mvchat global <on/off> - Allows user to toggle speaking in global by default.
/mvchat admin <on/off> - Allows user to toggle viewing all worlds chats.
/mvchat (world) (message) - Allows user to send message to a specific world.
Permissions
- mv.chat.world.(worldname)/(*) > Permits users to chat in specific worlds, and if they lack permission have a configurable message stating so.
- mv.chat.global > Allows user to talk globally across the server in all worlds.
- mv.chat.global.toggle > Allows users to toggle speaking globally by default.
- mv.chat.admin > Allows users to toggle viewing chat across all worlds for moderation.
- mv.chat.admin.world > Allows users to send a message to a specific world if they are not in it for moderation purposes.
This is something i've been looking for for a while now and am struggling to find plugins that are either up to date, or cover the diversity in the configs for per-world usage.

I believe there is already various plugins that does the above mentioned features. Some include Chatcontrol, Chat Manager and Venturechat.
https://www.spigotmc.org/resources/chatcontrol™-the-ultimate-chat-plugin-500-000-downloads-1-2-5-1-15-2.271/
https://www.spigotmc.org/resources/chat-manager-1-7-1-15-with-30-features-and-40-commands.52245/
https://www.spigotmc.org/resources/venturechat.771/

As mention, there are already many great alternatives, so I don't see MV implementing this.

Would be nice to have this in the same family/eco-system of plugins.


[Proposal]Metadata(URL) problems and possible solutions

Hello, community
We have previously roughly discussed the problems with Dubbo's metadata in GitHub issues and mailing list,  here when saying metadata I mainly refer to URL in Dubbo. Now I think it's time to bring the discussion onto the table and consider solving it.  Below is the proposal I've drafted:
Problems
URL works as the carrier for Dubbo internal data transfer at runtime, and is also works as a protocol integrating with the external system: registration center, service query, service management, etc., resulting in serious coupling between the configuration and the registration center, and the data load pushed by the registration center is too large.
Targets

Keep URL unchanged at runtime, and even enrich the runtime URL; Keep the immutability of the URL during a single invoke from start to end.
Simplify Registry url as much as possible.
Refactor the way of doing service configuration, rely on Registry currently, switch to third-party Config Center while keeping compatible with old Overrides protocol.
Some data in URL are for service query purpose, we should register this part to another place instead of Registry. And it should be realized as extensible and users can decide to enable it or not.

Solutions
Here is a typical URL in Registry:
dubbo://<IP_ADDRESS>:20880/com.xxx.compose.ic.service.vas.ValueAddServiceCompose?anyhost=true&application=goods-compose&default.actives=400&default.delay=-1&default.dispatcher=all&default.group=online&default.loadbalance=leastactive&default.service.filter=-monitor&default.threads=400&default.timeout=2000&default.version=1.0&delay=1&dubbo=2.6.2&environment=product&interface=com.xxx.compose.ic.service.vas.ValueAddServiceCompose&logger=slf4j&methods=queryValueAddServiceEditVOs,auditPassByValueAddServiceIdList,getValueAddServiceVOByServiceIds,getValueAddServiceInfoByGoodsIdList,getRuleMatchVOByGoodsIdList,getValueAddServiceEditVOById,getValueAddServiceEditVOByIds,queryValueAddServiceVOs,disableValueAddService,auditRejectByValueAddServiceIdList,saveValueAddServiceEditVO,enableValueAddService&organization=someorg&owner=somebody&pid=54812&revision=1.18.0628.3&side=provider&timestamp=1530671441040

We can see this URL showed here is really long and redundant, here are my suggestions to refactor it step by step:

Only keep necessary part in Registry
For example:

dubbo://<IP_ADDRESS>:20880/com.xxx.compose.ic.service.vas.ValueAddServiceCompose?timeout=1000&group=online&version=1.0&weight=100&timestamp=1530671441040

Find a new place for those parameters only for query purpose
For example:

delay=1&dubbo=2.6.2&interface=com.xxx.compose.ic.service.vas.ValueAddServiceCompose&logger=slf4j&methods=queryValueAddServiceEditVOs,auditPassByValueAddServiceIdList,getValueAddServiceVOByServiceIds,getValueAddServiceInfoByGoodsIdList,getRuleMatchVOByGoodsIdList,getValueAddServiceEditVOById,getValueAddServiceEditVOByIds,queryValueAddServiceVOs,disableValueAddService,auditRejectByValueAddServiceIdList,saveValueAddServiceEditVO,enableValueAddService&organization=someorg&owner=somebody&pid=54812&revision=1.18.0628.3&side=provider

Deprecate the Override protocol in Registry
Typical Override protocol to change timeout value at runtime:

   override://<IP_ADDRESS>/com.xxx.compose.ic.service.vas.ValueAddServiceCompose?category=configurators&dynamic=false&application=foo&timeout=1000

We can instead integrate with thirdparty Config Center to do that, like Apollo, Archaius, Diamond and so on, these systems are more likely to be used in a miscroservice architecture.
Furthermore, we can also consider separate the routing rule config from Registry and switch to Config Center,  using Config Center to dynamically add new group policies that can group service instances into different groups according to environments.
Kindly ask for your suggestions, don't hesitate to let us know if you have any different ideas.

Remove some unused info and use compression maybe better.
Integrate other Config Center make operation and maintenance complex. It means that we should deploy a Config Center to use dubbo, not very suitable for me

Config Center is not necessary in Dubbo, sometime I want to directly connect to the provider.

As for many company use config center. I think it can be an option for user

Integrate other Config Center make operation and maintenance complex. It means that we should deploy a Config Center to use dubbo, not very suitable for me

We need to keep compatibility with current Overrides policies. I listed 'Apollo', 'Archaius', 'Diamond' above, but Zookeeper can still be an option for configuration, Archaius itself also doesn't need too much effort to deploy. So it's not mandatory to deploy a Config Center.
Besides, Config Center seems to be a common system for a Microservice architecture, users usually don't need to deploy it for Dubbo purpose, it is usually already there.
Consider a cluster with a certain scale, it may have lots of rules that need to be stored and notified at runtime, a separate configuration center may be more scalable.
What do you think now, @carryxyh ?

hi, I think chose URL as the carrier for Dubbo is ingenious, and the goal you descripted here is keep Registry clean. but I have some questions:

Config Center dependency on Registry? or provider directly expose to Config Center?
Config Center is duplicate with Admin? and will community should talk about service governments around with Config Center? at  least we can list some topic and best patterner.

@carryxyh make a abstract composite for integration with different config centers, and then every contributor will be PR a specified one.

@kimmking
yeah
make it optional and configure.

The Registry URL for dubbo may contanins many information. I simply divided into several categories：
1.Service registration

where is the service? so comes ip and port
What services are provided? for dubbo use interface or path
how contact with the service? we may use protocol, codec, exchanger, serialization...
support multiple versions and isolated， can use group, version
...

2.Service config
We can use configuration in provider side to give a suggestion behavior of the remote call.

How long will timeout?
Do we need retries? or jsut failfast?
how to select one instance fron a list? loadbalance, weight...
how about service downgrade? can use mock...
Do we need cache invoke result?
generic invoke?
methods level configuration
...

3.Provider Config

Thread pool
Delay pubilsh
Register or not
Application info
Parallel control(executes)
Which transporter
...

4.Feature depends on registry

Callback parameter
Token Authorization
Tag a service deprecated
...

5.Runtime
methods, timstamp, dubbo, pid, revision, side...
I think we may retain the Service registration and Feature depends on registry in registry URL.
Some common serivce config is also needed， may for compatibility.
And finally, there will also be many keys

Parameters need to transfer to Registry
The URL for provider parameters comes from:

ApplicationConfig
ModuleConfig
ProviderConfig
ProtocolConfig
ServiceConfig
MethodConfig&ArgumentConfig
others: side/dubbo/timestamp/pid/generic/methods/revision/default/token/notify/dynamic

ApplicationConfig
application(name) need to transfer.
ModuleConfig
Module info may not need to transfer to registry.
ProviderConfig
ProviderConfig ‘ keys are subsets of keys in ServiceConfig and ProtocolConfig, And It's default. prefix should be removed.
 Use configurators instead.
All transfer to Consumer side
others
others: side/dubbo/timestamp/pid/generic/methods/revision/default/notify/dynamic
the dubbo? &  timestamp  should be transfered.
TranferKeys

application
codec
exchanger
host
protocol
payload
port
serialization
cluster
connections
deprecated
group
loadbalance
mock
path
timeout
token
version
warmup
weight
dubbo?
timestamp
pid
Any method level configuration of the listed keys above

Parameters to search center

All parameters should be stored to search center
Further enrich method signature information: method name, param name, JSON string that fully reflects the parameter structure.


Remove duplicate code for aggregating vote power

The aggregation of total vote power was implemented slightly differently
in two areas of the validator_verifier.rs file.
This commit refactors those implementations into a single function and
adds tests for the case in which a validator increases their voting
power to a value large enough to cause an integer overflow.
Motivation
Simplify code that aggregates total voting power in validator_verifier.rs
Have you read the [Contributing Guidelines on pull requests]
Yes
Test Plan
cd types; cargo test -- --nocapture test_very_unequal_vote_quorum_validators

/land


svg size not detected; assumed to be 1x1

I've got a valid svg file:
<?xml version="1.0" encoding="utf-8"?>
<!-- Generator: Adobe Illustrator 18.1.1, SVG Export Plug-In . SVG Version: 6.00 Build 0)  -->
<svg
  xmlns:xlink="http://www.w3.org/1999/xlink"
  id="Layer_1"
  enable-background="new 0 0 24 24"
  version="1.1"
  viewBox="0 0 24 24"
  x="0px"
  xml:space="preserve"
  xmlns="http://www.w3.org/2000/svg"
  y="0px"
>
  <path
    d="M18,20H6c-1.1,0-2-0.9-2-2V6c0-1.1,0.9-2,2-2h12c1.1,0,2,0.9,2,2v12C20,19.1,19.1,20,18,20z"
    fill="#686868"
  />
</svg>

The generated pngs are:
build/generated/res/debug/svg/drawable-mdpi/test.png     1x1
build/generated/res/debug/svg/drawable-hdpi/test.png     2x2
build/generated/res/debug/svg/drawable-xhdpi/test.png    2x2
build/generated/res/debug/svg/drawable-xxhdpi/test.png   3x3
build/generated/res/debug/svg/drawable-xxxhdpi/test.png  4x4
Victor apparently thinks this is a 1x1 svg, though Adobe Illustrator and Inkscape recognize it as 24x24.
Manually adding width and height attributes to the file fixes it:
<?xml version="1.0" encoding="utf-8"?>
<!-- Generator: Adobe Illustrator 18.1.1, SVG Export Plug-In . SVG Version: 6.00 Build 0)  -->
<svg
  xmlns:xlink="http://www.w3.org/1999/xlink"
  id="Layer_1"
  enable-background="new 0 0 24 24"
  height="24"
  version="1.1"
  viewBox="0 0 24 24"
  width="24"
  x="0px"
  xml:space="preserve"
  xmlns="http://www.w3.org/2000/svg"
  y="0px"
>
  <path
    d="M18,20H6c-1.1,0-2-0.9-2-2V6c0-1.1,0.9-2,2-2h12c1.1,0,2,0.9,2,2v12C20,19.1,19.1,20,18,20z"
    fill="#686868"
  />
</svg>

I thought this might be related to not specifying svgDpi in build.gradle, but adding:
victor {
    // Any assets defined in relative terms needs a base DPI specified
    svgDpi = 72
}

...doesn't change the result.

Hi,
See issue 20 and also the sample project for how to specify the width in a way that Victor currently recognizes.
https://github.com/trello/victor/issues/20
https://github.com/trello/victor/blob/master/sample/src/main/svg/ic_laptop.svg

Thanks ciskeboekelo.  Adding width/height attributes does work, but is slightly tedious and seems unnecessary given that every other tool (Adobe Illustrator, Inkscape, ImageMagick, etc.) seems to properly determine the size.  I was hoping this issue could prompt a fix to make these changes unnecessary.

@stephentalley Ah yes, I see now you already knew the workaround, sorry.
@dlew I'm wondering if this bit in SVGResource.groovy isn't the cause of this:
    width = UnitProcessor.svgHorizontalLengthToUserSpace(svgElement.width.baseVal.valueAsString, '', context)
    height = UnitProcessor.svgVerticalLengthToUserSpace(svgElement.height.baseVal.valueAsString, '', context)

Now I'm new at SVG, at Batik, at Groovy, at building Gradle plugins, so forgive me if I say something stupid, but it seems that that bit of code reading the width/heigth directly from the svg tag (which may not be specified), whereas this:
    width = context.viewportWidth;
    height = context.viewportHeight;

would seem to get the viewport dimensions regardless of whether the width is specified in the svg element. I would have tested it, but I currently don't have the time to figure out how to builld and test the victor gradle plugin myself.

Due to the oddities of Gradle, I find the easiest way to develop is to move the plugin to the buildSrc directory. (I know there are complicated setups you can use where it publishes the plugin automatically before building, but bleh.)
I'll look into this soon. I can't remember exactly why but I think there's a reason I was using UnitProcessor - probably it has to do with evaluating the value with DPI applied.

I've spent the last hour reading up on SVG sizing. More specifically, what happens if you don't define width or height because that's the crux of this issue.
The key information is provided by the specs here (emphasis mine):

SVG needs to specify how to calculate some intrinsic sizing properties to enable inclusion within other languages. The intrinsic width and height of the viewport of SVG content must be determined from the 'width' and 'height' attributes. If either of these are not specified, the lacuna value of '100%' must be used. Note: the 'width' and 'height' attributes are not the same as the CSS width and height properties. Specifically, percentage values do not provide an intrinsic width or height, and do not indicate a percentage of the containing block. Rather, they indicate the portion of the viewport that is actually covered by image data.

Since neither width nor height is provided, it assumes 100% for both. Percentage values do not determine intrinsic width/height.

The reason it works in other contexts:

In HTML, a percentage width/height determines its size based on other, extrinsic values provided by the webpage.

In SVG editors, you don't need an intrinsic width/height since it's vector graphics. You can edit in entirely abstract units; it just doesn't have any intrinsic size.

In the context of Victor, there are no extrinsic values that can be relied upon to determine the width/height. The SVG must have an intrinsic width/height.

Ultimately, I've decided this is not an issue for Victor to solve.
The only possible solution would be to allow the definition of a default width/height if neither is specified (as a way of providing extrinsic values). However, this seems a poor solution vs. being more explicit with your own SVGs.
It's been one of those fun bugs that has expanded my knowledge. Thanks!


Update the npm package name

The pulumi-buildkite is not correct FYI

thank you for spotting this!
the change lgtm.
however, could you please change your commit message to follow conventional commits, and force push again? this is why the build check fails now.
in your case i‘d suggest something like „docs: fix npm package name“

fyi: this change will be included going forward with #50


Opened windows don't have keyboard focus on macOS

Describe the bug
On MacOS, newly opened windows are in weird status when it comes to their focus, keyboard focus. Newly opened windows look like they are focused, their IsActive property is true, but no keyboard events are received until you click anywhere on the window. The window is receiving pointer events properly tho.
To Reproduce
Repro: https://github.com/BAndysc/AvaloniaTestFocus

Open application on macOS
Press "Open window" button (and do not click anything else now)
New window looks like it is active, IsActive is true, textbox looks focused, but no keyboard event is received by the window until you click anywhere in that new window.

Expected behavior
It works properly on Windows - the newly opened window should have proper focus - the textbox should receive keyboard events.
Screenshots

The red dot shows Pointer Events work correctly.
(you can see mouse click effect, before I click on the window, I was typing keys on my keyboard, but they weren't received)
Desktop (please complete the following information):

OS: MacOS BigSur 11.4 on M1 via Rosetta
Version 0.10.6

For a further test, I put a breakpoint here: https://github.com/AvaloniaUI/Avalonia/blob/1bfa7a6aa6dd96b02f7dabbd72abb7d4c4f22881/src/Avalonia.Native/WindowImplBase.cs#L211 and I can confirm the key events are not received at all, so I guess it is problem in macos interop code

Hi @BAndysc - yeah this has been reported by a customer too. We're in the middle of a bug-bashing session for OSX so hopefully will be fixed in the next day or two.


[TECH-DEBT] Improve Log levels

ISSUE:

Most of the logs are in INFO level
Some are in WARN and ERROR level
Very less lines are in DEBUG level

There is no functional issues, but for reporting or debugging can be enhanced.

FIX:
AC1:
Level - INFO:

[ ] Write logs relevant to the request / response only

AC2:
Level - WARN:

[ ] Write logs relevant to the any information to be propagated to developers/testers which could be a blocker

AC3:
Level - ERROR:

[ ] Write logs relevant to the any exception scenarios, errors, program halt situations

AC4:
Level - DEBUG:

[ ] Write logs relevant to the anything technical developer or tester, that will help in debug or analysing a problem. Verbose mode

can you assign this issue to me? i want to work on it.

I can help with this.

In 2 days or earlier. I thought it was not urgent.

In 2 days or earlier. I thought it was not urgent.

Sounds good. Even a week's time is also fine imho. 👍
Just for visibility purpose, it was needed as it helps other participants 😄

pull request created #580

Fixed the build issue and updated pull request

Is this issue closed now?

@idevkshitij ,
Still there is a build failure in the latest build(click).
The failure may not be due to the logging-fix, but it needs a fix.

PR https://github.com/authorjapps/zerocode/pull/580 has been merged.
This will be available in the next release automatically.


uuu tool issue - doesnt detect USB device

Hi,
I have a eval kit from NXP MX8QXPMEK. I can boot it with the pre-configured SD-Card without problems. When I doesnt insert the SD-Card board switches to SDP. In my device manager the board is detected. The old MFGTools shows "HID device detected".
Now  I am trying to download the pre-compiled u-boot. When I start the uuu (Version 1.2.91) it shows me the message Wait for Known USB Device Appear... and wait. It look like the USB device (board) is not detected.
How can I solve this?
I am using Windows10 as host system.
regards Haturi

You are using QXP A0 chip, which is not support by UUU.
A0 chip is not production chip.

Okay but there must be a way how to get it to work. I mean we have purchased those CPUs from NXP so we can´t develop if it´s doesnt work. Furthermore we have bought a development kit (i.MX8QXP-MEK) and it also doesnt work with uuu.
So what should I modify to get it to work?
regards,
Haturi

The whole boot flow is changed in QXP B0 totally.  A0 Chip is not production one. If you purchased it, you should contact sale to replace it with B0 chip.

Okay thanks for the explanation.


(fix): use typing_extensions for NamedTuple

I was off on the version in my comment or we just missed I meant "greater than": https://github.com/python/cpython/pull/92027
In any case, we can just use typing_extensions: https://github.com/python/typing_extensions/pull/44
Previously, this was causing issues with 3.10 as this was the case not covered (this feature broke in 3.9, but is fixed in 3.11, and we special cased <3.10).

[ ] Closes #
[ ] Tests added
[ ] Release note added (or unnecessary)

Hmmm, I thought typing_extensions was either already a dependent or included in python.  Better not to add a dependency just for this.


TypeError: filename.endsWith is not a function

🐞 Bug report
Command (mark with an x)

[ ] new
[X] build
[ ] serve
[ ] test
[ ] e2e
[ ] generate
[ ] add
[ ] update
[ ] lint
[ ] xi18n
[ ] run
[ ] config
[ ] help
[ ] version
[ ] doc

Is this a regression?
Not sure.
Description
I'm building an angular elements project with ngx-build-plus with the following command and I'm getting the following error:
ng build --prod --output-hashing none --single-bundle --bundle-styles=false --keep-styles=false

ERROR in chunk styles [entry]
({
              chunk: chunkData
            }) => this.options.moduleFilename(chunkData)
filename.endsWith is not a function
TypeError: filename.endsWith is not a function
    at /Users/hisham/src/ess-app/frontend/node_modules/@angular-devkit/build-angular/src/angular-cli-files/models/webpack-configs/common.js:106:72
    at SyncWaterfallHook.eval [as call] (eval at create (/Users/hisham/src/ess-app/frontend/node_modules/tapable/lib/HookCodeFactory.js:19:10), <anonymous>:5:16)
    at MainTemplate.getAssetPathWithInfo (/Users/hisham/src/ess-app/frontend/node_modules/webpack/lib/MainTemplate.js:527:40)
    at Compilation.getPathWithInfo (/Users/hisham/src/ess-app/frontend/node_modules/webpack/lib/Compilation.js:2227:28)
    at Compilation.createChunkAssets (/Users/hisham/src/ess-app/frontend/node_modules/webpack/lib/Compilation.js:2138:31)
    at /Users/hisham/src/ess-app/frontend/node_modules/webpack/lib/Compilation.js:1397:10
    at AsyncSeriesHook.eval [as callAsync] (eval at create (/Users/hisham/src/ess-app/frontend/node_modules/tapable/lib/HookCodeFactory.js:33:10), <anonymous>:7:1)
    at AsyncSeriesHook.lazyCompileHook (/Users/hisham/src/ess-app/frontend/node_modules/tapable/lib/Hook.js:154:20)
    at Compilation.seal (/Users/hisham/src/ess-app/frontend/node_modules/webpack/lib/Compilation.js:1342:27)
    at /Users/hisham/src/ess-app/frontend/node_modules/webpack/lib/Compiler.js:675:18
    at /Users/hisham/src/ess-app/frontend/node_modules/webpack/lib/Compilation.js:1261:4
    at AsyncSeriesHook.eval [as callAsync] (eval at create (/Users/hisham/src/ess-app/frontend/node_modules/tapable/lib/HookCodeFactory.js:33:10), <anonymous>:40:1)
    at AsyncSeriesHook.lazyCompileHook (/Users/hisham/src/ess-app/frontend/node_modules/tapable/lib/Hook.js:154:20)
    at Compilation.finish (/Users/hisham/src/ess-app/frontend/node_modules/webpack/lib/Compilation.js:1253:28)
    at /Users/hisham/src/ess-app/frontend/node_modules/webpack/lib/Compiler.js:672:17
    at _done (eval at create (/Users/hisham/src/ess-app/frontend/node_modules/tapable/lib/HookCodeFactory.js:33:10), <anonymous>:7:1)

🌍 Your Environment

Angular CLI: 9.0.3
Node: 13.2.0
OS: darwin x64

Angular: 9.0.2
... animations, common, compiler, compiler-cli, core, elements
... forms, language-service, platform-browser
... platform-browser-dynamic, router
Ivy Workspace: Yes

Package                           Version
-----------------------------------------------------------
@angular-devkit/architect         0.900.3
@angular-devkit/build-angular     0.900.3
@angular-devkit/build-optimizer   0.900.3
@angular-devkit/build-webpack     0.900.3
@angular-devkit/core              9.0.3
@angular-devkit/schematics        9.0.3
@angular/cdk                      9.1.0
@angular/cli                      9.0.3
@angular/material                 9.1.0
@ngtools/webpack                  9.0.3
@schematics/angular               9.0.3
@schematics/update                0.900.3
rxjs                              6.5.4
typescript                        3.7.5
webpack                           4.41.2

Anything else relevant?
In https://github.com/angular/angular-cli/blob/c9a5f3ced6689feac9cc0ccbd15b0fed6f454d6e/packages/angular_devkit/build_angular/src/angular-cli-files/models/webpack-configs/common.ts#L154, it appears that filename can sometimes be a function, putting a line of code such as:
if (typeof filename === 'function') { filename = filename(chunkdata); }
resolves the issue.
See https://github.com/pirelenito/git-revision-webpack-plugin/pull/42/files and https://github.com/webpack-contrib/mini-css-extract-plugin/issues/417 for similar discussions and fixes.

Thanks @alan-agius4 for filing. If this is a regression, do you know what angular cli version would this be working in that I can downgrade to?

For now I force downgraded mini-css-extract-plugin to 0.6.0 and the issue got resolved.


add compounds/drugs on pre-clinical stage

@deniseOme commented on Mon Oct 23 2017
This is one compound ARUK Oxford DDI where interested in:
https://www.ebi.ac.uk/chembl/compound/inspect/CHEMBL1957266
We've got it here:
http://www.targetvalidation.org/summary?drug=CHEMBL1957266
More info on this compound (including mechanism of action):
http://www.chemicalprobes.org/jq1
https://www.ncbi.nlm.nih.gov/pubmed/20871596
They would like to see this compound listed under drugs for this target:
http://www.targetvalidation.org/target/ENSG00000239713
There is some target information from chemical probes under target details:

@iandunham commented on Tue Oct 24 2017
I can't see APOBEC3G amongst the list of targets that ChEMBL curates at https://www.ebi.ac.uk/chembl/target/results/keyword for this drug, and the chemical probes site just gives the BRD set of proteins. Where does the information that APOBEC3G is a target for this come from? i.e. how would we get the information, other than someone telling us?

@deniseOme commented on Tue Oct 24 2017
This is what I found:
https://www.ebi.ac.uk/chembl/target/inspect/CHEMBL1741217
Would this suffice?

@iandunham commented on Tue Oct 24 2017
Is APOBEC3G in the list of compounds found in that page (there are a lot)? The request seemed to be about adding the compound under the APOBEC3G drugs but is it actually a target or just has some activity towards that target?

@deniseOme commented on Tue Oct 24 2017
As far I understood, it had some activity towards the target and this is known in the pre-clinical stage.
They want to get compounds targeting a protein, not only drugs, not only drugs already marketed or approved for clinical trials.

Issue triage notes:

Platform now includes chemical probes information from the Structural Genomics Consortium, the Chemical Probes Portal, Open Science Probes, and ProbeMiner
Example given - CHEMBL1957266 - appears in the Platform as JQ1, which can be found on the BRD2 target profile page in the Chemical Probes section
Introduction of drug entity will also increase coverage of drugs/compounds in pre-clinical stage

Issue closed as per triage notes above


amplify push doesn't deploy for container based rest apis

How did you install the Amplify CLI?
npm
If applicable, what version of Node.js are you using?
v20.8.0
Amplify CLI Version
12.7.0
What operating system are you using?
Mac
Did you make any manual changes to the cloud resources managed by Amplify? Please describe the changes made.
No manual changes made
Describe the bug
I created a rest API using a Docker container a few months ago. When I try to update it now, using amplify push, I get Deployment state saved successfully in the end, but no change is made to the ecs. I checked the link for the code pipeline, and above the page it is written:
As of 2 months agoChange 2 months ago to UTC-6:00 time formatChange 2 months ago to UTC-6:00 time format, CodePipeline has disabled polling in this pipeline because it hasn't run in the 30 days prior. Use the recommended event-based mechanism to start new pipeline executions.
Also, when I checked the pipeline, it was from 3 months ago, and it does not get updated.
Expected behavior
I expect the changes to be deployed.
Reproduction steps

Make an amplify rest api using a docker container
Don't update it for three months
Try updating it and redeploying it

Project Identifier
64645b8840605258eb3b7de823f2c831
Log output

# Put your logs below this line

Additional information
No response
Before submitting, please confirm:

[X] I have done my best to include a minimal, self-contained set of instructions for consistently reproducing the issue.
[X] I have removed any sensitive information from my code snippets and submission.

Hey @zandvakiliramin, thanks for raising this issue and providing the repro steps.
We will reproduce this internally and report back soon.

Hi @zandvakiliramin, We were unable to reproduce the issue by following the steps mentioned in the description. We recommend trying the latest CLI version. If you still encounter the same problem after upgrading, please feel free to leave a comment. This will allow us to reopen the issue and provide you with the necessary support.


MOGP Fitting tests

Occasionally the fitting tests for multiple outputs fail in an incorrect way because the fitting is done in parallel. Fix this to ensure that a single process is used and that the case expected to fail comes first to ensure reproducible failure as expected.

Closing, fixed in #127.


Fix type queries for persisted partitions

Please don't merge yet, this seems to affect queries on non-persisted partitions.


Waiting.........

Can some one help me with this bug, that its stuck on waiting and does not show anything.

When that happened to me, It meant that the ptc account that I created needed to accept the TOS. So either manually log into the game with the account you plan to use for the scanner or find a program that will automatically accept the TOS for you. It will not scan until the account accepts TOS.

Thanks its fixt!

i've accepted it + caught my first pokemon with that acc but it's the same after that.

Same here, TOS has been accepted on the account but it never gets past waiting.


Union with nptyping not handled properly

First, thanks for your great work!
The problem I'm facing is illustrated in the following code:
from typing import Optional

import numpy as np
from beartype import beartype
from nptyping import Float64, NDArray, Shape

@beartype
def fun1(arg1: NDArray[Shape["*"], Float64]) -> None:
    pass

@beartype
def fun2(arg1: NDArray[Shape["*"], Float64] | None) -> None:
    pass

@beartype
def fun3(arg1: Optional[NDArray[Shape["*"], Float64]]) -> None:
    pass

def test_fun() -> None:
    fun1(np.array([i / 2 for i in range(1, 101)], dtype=np.float64))  # <- works ok
    fun2(np.array([i / 2 for i in range(1, 101)], dtype=np.float64))  # <- fails
    fun3(np.array([i / 2 for i in range(1, 101)], dtype=np.float64))  # <- works ok

All three functions fun1, fun2, and fun3 accept the same argument, except that in fun2 and fun3 it is optional (can be None). The same input argument for fun1 and fun3 is considered OK, while for fun2 it fails with:
parameter arg="array([ 0.5,  1. ,  1.5,  2. ,  2.5,  3. ,  3.5,  4. ,  4.5,  5. ,  5.5,
E           6. ,  6....])" violates type hint nptyping.ndarray.NDArray | None, as <protocol "numpy.ndarray"> "array([ 0.5,  1. ,  1.5,  2. ,  2.5,  3. ,  3.5,  4. ,  4.5,  5. ,  5.5,
E           6. ,  6....])" not <class "builtins.NoneType"> or <class "nptyping.ndarray.NDArray">

Am I doing something wrong? Any workaround?
Thanks!
UPDATE: Meanwhile I've discovered that it works with Optional and added fun3, so this is a workaround. However, it would be nice to fix this bug because | None notation is more commonly used and suggested than Optional (Ruff UP007).

First, thanks for your great work!

Thank you so much for the kind words, @peske. Serbia looks like such a fascinating and beautiful country, too. I'd love to visit someday. Who wouldn't? So much green space and florid history all over the place there! :open_mouth:

The problem I'm facing is illustrated in the following code:

Thanks so much for that detailed example. You are about to change your mind about both me and @beartype, because... I can't replicate your problem. I tried really hard on a late Friday night – but Python 3.10 through 3.12 all quietly accept and run your code without complaints, warnings, or exceptions.
Let's Do Some Canada-splaining

Canada-splaining: It's like man-splaining, only worse.

I have a few ideas about what might be happening here. Are you perhaps attempting to run this example:

Under Python < 3.10? Like, under Python 3.8 or Python 3.9? If so, that would be your problem. Sadly, Python only supports the PEP 604-compliant | pipe operator that your fun2() function annotates its type hints with under Python ≥ 3.10. I weep. :weary:
With an older version of @beartype? Please ensure you've installed the most recent stable release of @beartype as of this writing: @beartype 0.16.4. It is good.
With an older version of nptyping? Likewise, please ensure you've installed the most recent stable release of nptyping as of this writing: nptyping 2.5.0. It is possibly good.

Lastly, your irrational hatred of me is about to intensify. I am about to suggest another unpleasant thing that will ruin your beautiful but bitterly cold weekend in Serbia: consider dropping nptyping. Seriously. nptyping is currently unmaintained and likely to die shortly.
Instead, you almost certainly want to refactor your code to leverage @patrick-kidger's jaxtyping. Yes, this will be work.  Yes, you will eventually thank me after shaking your fists in anger at the dismal grey sun. That's good.
Unlike nptyping, jaxtyping provides well-maintained type hints covering NumPy, JAX, PyTorch, and TensorFlow. It's pretty intense. It's also @beartype's official FAQ recommendation for type-checking NumPy arrays.
Let's Quietly Go Home
Sadly, I can't replicate this on my end. Likewise, nptyping is unmaintained and possibly broken on your end. I hope you don't mind too much if I quietly close this issue and pretend everything is okay without doing anything.
Please reopen this if I am a raving lunatic. Thank you. Have an awesome winter in Serbia, bro! :snowboarder:

Hey @leycec!
Thanks for your kind response!
There are several things to say, so let's start from the original problem: it is caused by my stupidity. I've used both beartype_this_package at the package level, and @beartype at the function level. It looks that this caused the weird behavior and my problems.
About Serbia - thanks for your kind words! Nothing special, but there are a few things to see and experience... Since you've turned out to be so friendly and helpful - I can offer you an apartment in Serbia for several weeks for free, so you're welcome to see first hand... And if you decide to come in September or October, the invitation includes visiting Corfu island, Greece (I also have apartment there). Serbia+Greece is a perfect combo if you enjoy eating :) Not to mention history and beaches of Greece...
To be completely honest, my opinion about Canada is not so good though... News that are coming from there are quite ugly. Censorship, government blocking people's banking accounts, cancel culture... Sounds more like North Korea or Belarus... Sorry for brutal honesty. But my criticism is not for the people, of course - some are very friendly and helpful ;)
When it comes to nptyping / jaxtyping - thanks for the advise! I wasn't aware about jaxtyping. I'm a newbie in Python... I will definitely give it a chance. Talking about replacing things, I would be happiest to replace the language (Python) itself. Making a dynamically typed language is probably one of the stupidest idea ever. And then trying to fix it with enormous effort with type hints (that are not enforcing???), mypy, things like beartype, etc. I'm software engineer for almost 25 years, probably wrote millions of line of code, but was lucky to work with decent languages (first C# and C++, later Golang and Rust). This is my first Python project (it's a science project so...), and it's very frustrating. Irony is that scientific community is doomed with some of the ugliest languages in the world (Python and MATLAB).
My Canadian friend, enjoy your winter there, and think about options for the next summer/fall... ;) Seriously.
Semi-bitter Serbian

After additional research, I've concluded that my problem was not caused by using both beartype_this_package and @beartype as I've said in my previous message, but with some very strange bug in beartype. I've opened another issue: https://github.com/beartype/beartype/issues/304


WFLY-4179 add permissions.xml to work around test issue

The test involves multiple contexts and by default contexts are not given
permission to read files from a different context.

Linux Build 6071 is now running using a merge of ff1cd3f6daa6d082367b93041f63c558302147be

Linux with security manager Build 1049 is now running using a merge of ff1cd3f6daa6d082367b93041f63c558302147be

Windows Build 1185 is now running using a merge of ff1cd3f6daa6d082367b93041f63c558302147be

Linux with security manager Build 1049 outcome was SUCCESS using a merge of ff1cd3f6daa6d082367b93041f63c558302147be
Summary: Tests passed: 843, ignored: 319 Build time: 0:05:24

Windows Build 1185 outcome was SUCCESS using a merge of ff1cd3f6daa6d082367b93041f63c558302147be
Summary: Tests passed: 2927, ignored: 311, muted: 2 Build time: 0:48:31

Linux Build 6071 outcome was FAILURE using a merge of ff1cd3f6daa6d082367b93041f63c558302147be
Summary: Tests failed: 1 (1 new), passed: 2928, ignored: 311 Build time: 0:49:41
Build problems:
Failed tests detected
Failed tests
org.jboss.as.test.integration.ejb.stateful.passivation.PassivationTestCase.testPassivationMaxSize: <no details avaliable>

Linux with security manager Build 1050 is now running using a merge of b082c662036c344447632b74740ef98408c325c6

Linux Build 6072 is now running using a merge of b082c662036c344447632b74740ef98408c325c6

Windows Build 1186 is now running using a merge of b082c662036c344447632b74740ef98408c325c6

Linux with security manager Build 1050 outcome was SUCCESS using a merge of b082c662036c344447632b74740ef98408c325c6
Summary: Tests passed: 843, ignored: 319 Build time: 0:05:24

Windows Build 1186 outcome was SUCCESS using a merge of b082c662036c344447632b74740ef98408c325c6
Summary: Tests passed: 2927, ignored: 311, muted: 2 Build time: 0:46:46

Linux Build 6072 outcome was FAILURE using a merge of b082c662036c344447632b74740ef98408c325c6
Summary: Tests failed: 29 (29 new), passed: 2824, ignored: 311, muted: 1 Build time: 1:05:30
Build problems:
Failed tests detected
Failed tests
org.jboss.as.test.manualmode.ws.WSAttributesChangesTestCase.testWsdlHostChanges: <no details avaliable>

org.jboss.as.test.manualmode.ws.WSAttributesChangesTestCase.testWsdlPathRewriteRuleChanges: <no details avaliable>

org.jboss.as.test.manualmode.ws.WSAttributesChangesTestCase.testWsdlPortChanges: <no details avaliable>

org.jboss.as.test.manualmode.ws.WSAttributesChangesTestCase.testWsdlUriSchemeChanges: <no details avaliable>

org.wildfly.test.integration.security.picketlink.federation.SAMLMetadataTestCase.testFederationWithGlobalLogout: <no details avaliable>

org.wildfly.test.integration.security.picketlink.federation.SAMLMetadataTestCase.testFederationWithLocalLogout: <no details avaliable>

org.wildfly.test.integration.security.picketlink.federation.SAMLPostBindingTestCase.testFederationWithGlobalLogout: <no details avaliable>

org.wildfly.test.integration.security.picketlink.federation.SAMLPostBindingTestCase.testFederationWithLocalLogout: <no details avaliable>

org.wildfly.test.integration.security.picketlink.federation.RestoreOriginalRequestTestCase.testPostOriginalRequest: <no details avaliable>

org.wildfly.test.integration.security.picketlink.federation.RestoreOriginalRequestTestCase.testPostOriginalRequestWithParams: <no details avaliable>

org.wildfly.test.integration.security.picketlink.federation.RestoreOriginalRequestTestCase.testRedirectOriginalRequest: <no details avaliable>

##### there are 19 more failed tests, see build details


Incorrect profile selection for GetCompositeSchedule.req handler

OCPP Version
OCPP2.0.1
Describe the bug
Bug
Currently the GetCompositeSchedule.req handler does not include all required profiles of purpose TxDefaultProfile and TxProfile in its calculation.
Reason
This is an excerpt of the GetCompositeSchedule.req handler:
std::vector<ChargingProfile> valid_profiles = this->smart_charging_handler->get_valid_profiles(request.evseId);

auto schedule = this->smart_charging_handler->calculate_composite_schedule(
        valid_profiles, start_time, end_time, request.evseId, request.chargingRateUnit);

The issue is inside get_charging_profiles_for_evse which is called in get_valid_profiles:
std::vector<ChargingProfile> SmartChargingHandler::get_valid_profiles_for_evse(int32_t evse_id) {
    std::vector<ChargingProfile> valid_profiles;

    auto evse_profiles = this->database_handler->get_charging_profiles_for_evse(evse_id);
    for (auto profile : evse_profiles) {
        if (this->validate_profile(profile, evse_id) == ProfileValidationResultEnum::Valid) {
            valid_profiles.push_back(profile);
        }
    }

    return valid_profiles;
}

This function calls validate_profile with a profile that has already been validated when it was added. This leads to the fact that evse specific profiles are not returned as part of get_valid_profiles because reasons like DuplicateProfileValidityPeriod or TxProfileConflictingStackLevel occur since the profiles are basically compared to themselves.
To Reproduce
Run OCTT test case TC_K_41_CS
Anything else?
While I'm not completely aware of all implications, I think this can simply be solved by removing the validate_profile call inside get_valid_profiles_for_evse to avoid the repeated validation with the consequences described above.
We can then also think of naming this function get_profiles rather then get_valid_profiles assuming that all profiles that have been added previously are valid.
This would require us to drop K08_GetValidProfiles_IfInvalidProfileExists_ThenThatProfileIsNotReturned.
I think we should make add_profile private and only provide validate_and_add_profile as part of the public API.

While I'm not completely aware of all implications, I think this can simply be solved by removing the validate_profile call inside get_valid_profiles_for_evse to avoid the repeated validation with the consequences described above.

I don't think this the correct solution. Profiles can be invalidated by internal changes to the system, such as changes to EVSEs, ongoing transactions, or device model configuration values. So we should check that all profiles we compile into a composite schedule are valid. 1.6 has a get_valid_profiles () that checks for similar state that can change and is used in the same context.
With regards to the bug:

This leads to the fact that evse specific profiles are not returned as part of get_valid_profiles because reasons like DuplicateProfileValidityPeriod or TxProfileConflictingStackLevel occur since the profiles are basically compared to themselves.

Generally when comparing profiles, we ensure that we let pairs of profiles with the same profile.id pass through. There's one exception to this: the comparison for TxProfileConflictingStackLevel within validate_tx_profile(). It seems like we forgot to accept profiles with the same ID there. In all other cases, we shouldn't be running into any issues comparing profiles against themselves. To verify this we wrote tests to check all of the validation functions that would compare given profiles against existing profiles, and only validate_tx_profile() failed.
So the proper solution would be to add a check in validate_tx_profile() to allow profiles with the same ID.

If you would like to check for yourself, these are the tests:
TEST_F(SmartChargingHandlerTestFixtureV201, K01_ValidateChargingStationMaxProfile_AllowsExistingMatchingProfile) {
    auto profile =
        SmartChargingTestUtils::get_charging_profile_from_file("max/ChargingStationMaxProfile_grid_hourly.json");
    auto res = handler.validate_and_add_profile(profile, STATION_WIDE_ID);
    ASSERT_THAT(res.status, testing::Eq(ChargingProfileStatusEnum::Accepted));

    auto sut = handler.validate_charging_station_max_profile(profile, STATION_WIDE_ID);

    EXPECT_THAT(sut, testing::Eq(ProfileValidationResultEnum::Valid));
}

TEST_F(SmartChargingHandlerTestFixtureV201, K01_ValidateTxDefaultProfile_AllowsExistingMatchingProfile) {
    auto profile = SmartChargingTestUtils::get_charging_profile_from_file("singles/Absolute_301.json");
    auto res = handler.validate_and_add_profile(profile, DEFAULT_EVSE_ID);
    ASSERT_THAT(res.status, testing::Eq(ChargingProfileStatusEnum::Accepted));

    auto sut = handler.validate_tx_default_profile(profile, DEFAULT_EVSE_ID);

    EXPECT_THAT(sut, testing::Eq(ProfileValidationResultEnum::Valid));
}

TEST_F(SmartChargingHandlerTestFixtureV201, K01_ValidateTxProfile_AllowsExistingMatchingProfile) {
    auto profile = SmartChargingTestUtils::get_charging_profile_from_file("baseline/TxProfile_1.json");
    this->evse_manager->open_transaction(DEFAULT_EVSE_ID, profile.transactionId.value());

    auto res = handler.validate_and_add_profile(profile, DEFAULT_EVSE_ID);
    ASSERT_THAT(res.status, testing::Eq(ChargingProfileStatusEnum::Accepted));

    auto sut = handler.validate_tx_profile(profile, DEFAULT_EVSE_ID);

    EXPECT_THAT(sut, testing::Eq(ProfileValidationResultEnum::Valid));
}

The first two pass, and the last one fails with TxProfileConflictingStackLevel.

The DuplicateProfileValidityPeriod occured because the check for the overapping validity period of profiles used the combination:
stackLevel - chargingProfileKind- evseId
although
K01.FR.06 requires
stackLevel - chargingProfilePurpose- evseId


Unable to change the width of the widget rendered in the stepBuilder

Hello,
thanks to the version 3.0.0 of your package, we can now return a custom widget in the stepBuilder to render pretty much everything we want.
This is a great feature, however i noticed that the rendered widget takes the full width of the device and I can't change or constraints the width.
I tried returning, in the stepBuilder, a SizedBox with a width of half the one given by the OnboardingStepRenderInfo but it doesn't change the end result.
stepBuilder: (
    BuildContext context,
    OnboardingStepRenderInfo renderInfo,
) {
    return SizedBox(
        width: renderInfo.size.width / 2,
        child: Material(
            child: ...,
        ),
    );
},

I also tried to add some padding at the top of the widget in the stepBuilder method, but only worked by removing the labelBoxDecoration, and with that loosing the arrow, which isn't really what i want.
stepBuilder: (
    BuildContext context,
    OnboardingStepRenderInfo renderInfo,
) {
    return Padding(
        padding: const EdgeInsets.symmetric(horizontal: 20),
        child: Material(
            child: ...,
        ),
    );
},

So my question is, is there a way to adjust the width of the widget returned in the stepBuilder? Or does it not exist yet?

I haven't tested that use-case. Will see what I can do and will be back with to you, maybe next week.

fixed in 3.1.0-pre.2
please test


Is there a documentation on creating custom protocol in gatling ?

NOTE: This link did not work to post the question to the mailing list: https://gatling.io/community-mailing-list/
I am looking to perf test AWS lambda, kinesis etc. Wanted to see if there is any plugin recommended or if I have to create my own custom protocol and what are the steps I need to follow to do so.

Hi!
Where did you get this link?
You are right, the link is wrong, it should be https://gatling.io/community/
Please let us know where is the wrong link as we can fix it.
(about your question, you should definitely ask it on the mailing list)
Cheers!

Where did you get this link?

In the original CONTRIBUTING.md. It was an issue on our side, I've fixed it.

No, there's currently no tutorials for implementing custom protocol supports. Our internals are not stable so we don't want to turn them into a public API at this time.


"Proceed anyway" option doesn't work correctly if website is blocked by rule with $all modifier

Steps to reproduce

Add this rule:
||adguard.com^$all
Go here - https://adguard.com/
Click on "Proceed anyway" button (like on a screenshot below)

Screenshot

Expected behavior

Website should loads correctly.
Actual behavior

Website loads, but requests are still blocked.
Screenshot

If I use $document modifier instead of $all like - ||adguard.com^$document then website loads fine (after clicking on "Proceed anyway" button), so I think that with $all modifier it should works the same.
Your environment

AdGuard for Windows Version 7.4 nightly 3 (build 3093, CL 1.5.170)

Actually, this is expected behavior at least in the current implementation.
We should change the way it's implemented, though.
Instead of unblocking one request, we should disable the whole rule ||adguard.com^$all at least temporary.

After some thinking, I came to the conclusion that we cannot implement this solution, it's too risky as it may expose some ways to avoid blocking to third-parties.
The only more or less viable solution would be to add an "Unblock for 30 seconds" button to the access denied page that will call AdGuard using custom scheme scheme:: https://github.com/AdguardTeam/AdguardForWindows/issues/2938

Hi! How you added the popup message?


how to excute @complete event after all cards are scratched within a div?

I have a total of 9 cards iterated with v-for when all 9 cards are fully scratched I want to alert something to the user, how do I do it?
currently when any one of the card is scratched @complete event works

The best way I can think of is wrapping each one inside a wrapper to handle the logic

Instead i added a ref in scracth card component by  <scratch-card ref="CardRef" >
and did this
  methods: {
    onComplete() {
      var CardArray = this.$refs.CardRef;
      for (var i=0; i<CardArray.length; i++) {
        console.log(CardArray[i].isDrawing);
      }
      const allTrue = CardArray.every((c) => c.isDrawing);
      if (allTrue) {
        // alert("all card is scratched");
        this.cards[0].scratched = true;
        this.HideScoreBoard = true;
      }
    }
  }


FN: No FLD_NEW_FUNDING alert for new account funding

BOT ID - 0x186f424224eac9f0dc178e32d1af7be39506333783eec9463edd247dc8df8058
Funding Tx -
https://bscscan.com/tx/0xe4cdfc5e491b07b13a61239ba52326bea7095770e50663413031be09ff5ba43c

the problem about not receiving txns from the networks is fixed.
However, since this tx was triggered by "WithdrawETH" function, it wasn't caught by the bot.
this event will also be added

yes pls. thanks Tayfun.
On Tue, Oct 31, 2023 at 2:51 AM Tayfun Sur @.***> wrote:

the problem about not receiving txns from the networks is fixed.
However, since this tx was triggered by "WithdrawETH" function, it wasn't
caught by the bot.
this event will also be added
—
Reply to this email directly, view it on GitHub
https://github.com/forta-network/starter-kits/issues/300#issuecomment-1786871160,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AYF4T4YJWNOJJ6CEOYAUCVDYCDCZHAVCNFSM6AAAAAA5EN5G56VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTOOBWHA3TCMJWGA
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

Christian, WithdrawETH only includes value. in order to fetch the receiver we need traces. for BSC I think we can't do it? should I close this?

has the fix been deployed yet? could you pls close the bug once deployed?

I'll close it as soon as it's deployed.

Is it deployed yet? @0xtaf ?

Deployed in https://github.com/forta-network/starter-kits/pull/419


Cannot login to hub - response is undefined

node app
/Users/d/Dropbox/Dev/HomeAutomation/HarmonyTest/node_modules/harmonyhubjs-client/lib/login/hub.js:63
if(response.identity && response.identity !== undefined) {
^
TypeError: Cannot read property 'identity' of undefined
at Client. (/Users/d/Dropbox/Dev/HomeAutomation/HarmonyTest/node_modules/harmonyhubjs-client/lib/login/hub.js:63:15)
at emitOne (events.js:78:13)
at Client.emit (events.js:170:7)
at Client.onStanza (/Users/d/Dropbox/Dev/HomeAutomation/HarmonyTest/node_modules/harmonyhubjs-client/node_modules/node-xmpp-client/index.js:231:14)
at emitOne (events.js:78:13)
at Connection.emit (events.js:170:7)
at Connection.onStanza (/Users/d/Dropbox/Dev/HomeAutomation/HarmonyTest/node_modules/harmonyhubjs-client/node_modules/node-xmpp-core/lib/Connection.js:370:14)
at StreamParser. (/Users/d/Dropbox/Dev/HomeAutomation/HarmonyTest/node_modules/harmonyhubjs-client/node_modules/node-xmpp-core/lib/Connection.js:226:14)
at emitOne (events.js:78:13)
at StreamParser.emit (events.js:170:7)

please refer to #15 and ensure you are using the latest version.


game-rs: init at 0.1.3

Description of changes

Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[x] Tested, as applicable:

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

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

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

[x] Fits CONTRIBUTING.md.

The vendor hash is showing up as different for me:
 specified: sha256-0W9FeGdlVQujbUJS9wRlMSw9q/m1W9t5NxRmRZ/fDEw=
    got:    sha256-dBUx7xh9b1gWe0Uc7GYti0r/qib+6jXxeSLu3Uk633c=

is that the cargo256sum? oh it needs updating after every change in dependencies, i'll use the lockFile argument then

Result of nixpkgs-review pr 218590 run on x86_64-linux 1

1 package built:

game-rs

Could you change the commit message to maintainers: add amanse?

done! apologies for the delay

oh no duplicate commits, I'll push the fix in a while

Vendor hash is different for me again:
         specified: sha256-8gpWV96E9BbvObwuzX6IEBEjrcDArbt04MM+FqAUvGQ=
            got:    sha256-aq58sFK4/Zd8S4dOWjag+g5PmTeaVAK3FS3fW/YlCLs=

i just did a nix-build -A game-rs on a new VM and it compiled successfully, is there something I'm doing wrong here?

I'm pretty sure it has to do with fixed-output derivations. Since you built the previous version, and the hash was the same, Nix did not build the vendor derivation and thus did not discover a mismatch. Nix will always build with an empty hash. So, you should specify an empty hash whenever you update before testing. This guarantees that you will always see the correct hash.

that's what i did... that's why i created a new VM too so that nix store would be empty, it gave the mismatch error on empty string and then i used the hash it provided.

Did you ever figure this out?

Result of nixpkgs-review pr 218590 run on x86_64-linux 1

1 package built:

game-rs

does wozeparrot need to approve too? or it's just waiting time?

This pull request has been mentioned on NixOS Discourse.


Pushing to ACR (Azure Container Registry)

Following the auth documentation did not work for me when pushing to ACR from an on-premise devops (with no docker installation).
I was trying the "last-resort" environment variables (as documented) with an Azure Service Principal, but that did not work either - not even with the DevOps Service Connection configured.
Only the ACR Admin user/password combination worked (at least for me).
Might be worth mentioning this in the documentation - maybe even look into why the configured DevOps Service Connection does not work (would be the best solution IMO), or enhance the project to support such use-cases out-of-the-box.
Thanks

@dviry have you continued to be blocked in this way? I've got a few samples where I'm authenticating using OIDC and Managed Identities successfully so I'm uncertain where the blockage could be.


Cannot save a file when the name consists only from numbers

This may be related to https://github.com/Microsoft/vscode/issues/451, possibly a regression.

VSCode Version: 1.11.2
OS Version: Windows 10 x64 Pro

Steps to Reproduce:

Open VSCode
Open a new tab and write something
Open the Save dialog
Switch the file type to All files (*)
Enter a filename consisting only from numbers (like<PHONE_NUMBER>06)
Click Save
You get an error saying "The file name is not valid"

Weird, either an Electron bug or actual Windows upstream issue.

You have to pick the "No Extension" entry for "Save as type":


Point users towards the Setting Up guide

Before going through the pmo or chat guide the user needs to have set up
all prerequisites. This adds a note at the top of those guides which
points people at that guide. Closes #571

Coverage remained the same at 96.396% when pulling b51f58d4098d6fed68f868c78c69e8e0e9bf652b on node-version into 06c9c4bdd18eaa19339148f519dcd3cb49c47dcd on major.


Return value != 0 after an error

I am using acme-tiny from a bash script, but i can't properly react to errors since acme-tiny always returns 0 upon exit.
Is it possible to change acme-tiny on master to make it return something != 0 if an error occurs?

I just made a simple error in bash, i redirected the acme tiny output and always got 0 as the return value (return of the redirection).


User not receiving notification when his comments are replied to.

Whenever a new comment is made by the user , Example Foo is notified by email.
However when there is a reply to the comment, the commentor(user)  is not being notified, even thoughreply-notifications= true in the server and the client being configured.
I see the checkbox though.
Server side:
[general]
reply-notifications= true

[smtp]
host = smtphost
port = 25
security = none
from = "Example Foo"<EMAIL_ADDRESS>to =<EMAIL_ADDRESS>timeout = 10

Client side:
        data-isso-reply-notifications="true"

Am I missing something?

I just set this up and it works. Could your SMTP server be limiting which addresses you can send to? That happened with my AWS SES implementation.

@vegeta123254 could you kindly verify whether the issue is indeed caused by what @thomasjsn suggested?
Without further logs/setup info it will be hard to verify and fix your issue.


Modal form change  proposal

Right now it's not possible to add a form inside a Modal as it throws this error:
Warning: validateDOMNesting(...): <form> cannot appear as a descendant of <form>.
Clicking on any button inside the modal closes it as well, which is not always intended.
This PR proposes to change this behavior, if a user want to close the modal he can always use:
<form method="dialog">
           <Button>Close</Button>
</form>

or
 <Button onClick={toggleModal}>Close me</Button>
inside the modal.
This is a proposal change for this opened issue: https://github.com/daisyui/react-daisyui/issues/415

Related to this commit
https://github.com/saadeghi/daisyui/commit/314761e35fd7b186e6556d21417ec8aa8e43dcec
LGTM 👍


InvalidCharacterError when using multiple classes

Using multiple classes, for example 'button success', within the swal options results in an InvalidCharacterError and prevents the alert from opening.
This can also be seen within the demos on the alert labelled '... and by passing a parameter, you can execute something else for "Cancel"' where the example uses successButtonClass: 'btn btn-success'.
The full error message (which occurs at line 439 of the prettified sweetalert2.min.js):
sweetalert2.min.js:1 Uncaught InvalidCharacterError: Failed to execute 'contains' on 'DOMTokenList': The token provided ('btn btn-success') contains HTML space characters, which are not valid in tokens.

Thank you @oli-laban for reporting this issue! Fixed in v4.0.10


Command injection attack does not work

It actually looks like the email address is being properly validated by this app, which prevents the command injection attack from working:
https://github.com/k-tamura/easybuggy4django/blob/01f060efbd186c2794119219f60e33ddceea467b/easybuggy/views.py#L552-L554
It seems like you need to be using a less stringent check for the email address in order for this attack to work.
It's kind of funny to ask you to make your app more vulnerable 😆

@drdavella
Oops, I messed up. validate_email() is not needed as you said.
Thank you!


Warnings with Angular 8 and above application

The following warnings shows up with recent versions of angular
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>< 8.0.0 but none is installed. You must install peer dependencies yourself.
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>< 8.0.0 but none is installed. You must install peer dependencies yourself.

the new version is out and it resolves this.


Factorial Trailing Zeroes

Given an integer n, return the number of trailing zeroes in n!.
Example :-
Input: n = 3
Output: 0
Explanation: 3! = 6, no trailing zero.
Input: n = 5
Output: 1
Explanation: 5! = 120, one trailing zero.
Input: n = 10
Output: 1
Explanation: 10! = 3628800, two trailing zero.
Input: n = 0
Output: 0
Explanation: 0! = 1, no trailing zero.

/assign

Doing it till tomorrow.


⚠ Bofrost.at has degraded performance

In 4b53984, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 1967 ms

Resolved: Bofrost.at performance has improved in 2e6fa81.


Added product Category into Product Model

Codecov Report

Merging #43 into master will increase coverage by <.01%.
The diff coverage is 100%.

@@             Coverage Diff              @@
##             master      #43      +/-   ##
============================================
+ Coverage     98.91%   98.91%   +<.01%
- Complexity      394      395       +1
============================================
  Files            51       51
  Lines          1015     1017       +2
============================================
+ Hits           1004     1006       +2
  Misses           11       11

Impacted Files
Coverage Δ
Complexity Δ

src/Product.php
100% <100%> (ø)
8 <1> (+1)
:arrow_up:

Continue to review full report at Codecov.

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


Support for pictures or image files instead of icon

This is a bit similar to #46 but I was curious how to add images to the notifications. Similar to the news article notifications that you would see on the Windows Action Center, I'd love to be able to attach a picture with sizing dimensions in addition to my message.
The syntax could be something like this:
toaster.show_toast("Today's Weather!",
                    "It's Cloudy Today",
                    image_path="http://icon-park.com/imagefiles/simple_weather_icons_cloudy.png",
                    image_width="600px",
                    image_height="600px",
                   icon_path="custom.ico",
                   duration=10)

I have dug around the code a bit and I don't think that is possible.
Since this library uses pywin32 and pywin32 does not support the notification API in Windows 10 (which normally uses C#) the most you can get is to be able to convert an image to .ico
I have implemented IMAGE TO ICO haphazardly but I do not think it is a desired feature


The shadow is being cut on the bottom

It seems that the demo gif also has this problem.

Fixed in 1.0.7.


Update pin

Rebuild for the new libuuid pin and update the recipe.

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


Add info about cross-origin resource loading for loadImage, loadFont, etc.

This was suggested in #822, but as best I can tell never made it into the reference (nor a ticket).
Seems to cause lots of confusion with students using images, so would be great to have some clear docs on it...

whoops, thanks. will add.


[type:Integration Test] Add rewrite-plugin Integration Test

This pr is an integration test written for the rewrite plugin.
After enabling the rewrite plugin, you can rewrite the url.
The test case is to rewrite /waf/pass to /waf/deny.
Accessing /waf/pass will receive a status code 403 of /waf/deny.

Codecov Report

Merging #3385 (c7f273b) into master (b485752) will decrease coverage by 0.03%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master    #3385      +/-   ##
============================================
- Coverage     65.41%   65.37%   -0.04%
- Complexity     5266     5277      +11
============================================
  Files           761      761
  Lines         21251    21250       -1
  Branches       1931     1931
============================================
- Hits          13902    13893       -9
+ Misses         6063     6058       -5
- Partials       1286     1299      +13

Impacted Files
Coverage Δ

...plugin/logging/rocketmq/LoggingRocketMQPlugin.java
22.72% <0.00%> (-72.73%)
:arrow_down:

...ogging/rocketmq/body/LoggingServerHttpRequest.java
0.00% <0.00%> (-20.00%)
:arrow_down:

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

...ache/shenyu/plugin/grpc/cache/GrpcClientCache.java
87.50% <0.00%> (-12.50%)
:arrow_down:

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

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

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

...ng/rocketmq/rocketmq/RocketMQLogCollectClient.java
70.00% <0.00%> (-6.00%)
:arrow_down:

...rg/apache/shenyu/plugin/rewrite/RewritePlugin.java
66.66% <0.00%> (-5.56%)
:arrow_down:

.../logging/rocketmq/utils/LogCollectConfigUtils.java
71.11% <0.00%> (-4.45%)
:arrow_down:

... and 10 more

Continue to review full report at Codecov.

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


Adding exports section to package.json

This allows 'normal' import/require of the package in node.js apps. Should not effect the usage in browser based apps.

:tada: This PR is included in version 2.3.0 :tada:
The release is available on npm package (@latest dist-tag)
Your semantic-release bot :package::rocket:


FlutterViewController should not show the Launch storyboard in add2app scenarios

Even with https://github.com/flutter/engine/pull/6834, the following code will result in a flash of the launch storyboard.
  FlutterViewController* flutterViewController =
      [[FullScreenViewController alloc] init];
  flutterViewController.splashScreenView = nil;
  [self.navigationController pushViewController:flutterViewController
                                       animated:NO];

Pre-warming the engine makes it marginally better but not much, e.g.:
  // marginally better, but still flashes
  FlutterViewController* flutterViewController =
      [[FlutterViewController alloc] initWithEngine:[self engine]
                                            nibName:nil
                                             bundle:nil];
  flutterViewController.splashScreenView = nil;
  [self.navigationController pushViewController:flutterViewController
                                       animated:YES];

still flashes the launch screen.
/cc @jamesderlin

Can't we derive the intent from the way it's initialized rather than having the user call 2 things? We should also make sure the Android API is consistent.

I'm trying to re-test this right now - I'm pretty sure I had tested it with a regular flutter app and everything worked fine.  Verifying.


date_spine is mentioned in the documentation but is not provided.

Describe the bug
Reference date_spine macro in a model
Steps To Reproduce
There was an error at execution because date_spine is not included in the macros available though it is mentioned as a custom provided macro in the documentation. What is offered is a datediff.
Expected behavior
Either the custom macro is provided or it is not mentioned as a custom provided macro in the documentation.
Screenshots and log output
If applicable, add screenshots or log output to help explain your problem.
The output of dbt --version:
1.3.3

The operating system you're using:
Windows
The output of python --version:
3.10
Additional context
Add any other context about the problem here.

Daniel,
When this plugin was released, data_spine macro was provided by dbt-utils. The only thing that dbt-teradata-utils had to do to be compatible with dbt-utils was to override generate_series. The macro worked even though it was not implemented in the plugin.
Now the situation got a little bit complex. In version 1.3 dbt moved the cross-db macros to dbt-core. To match that, we also moved our cross-db macros from dbt-teradata-utils to dbt-teradata. We didn't move generate_series.
Have you tried installing the extension and using data_spine with the extension installed?

Hi Adam,
If by the extension you mean dbt_utils, yes both dbt_utils and dbt_teradata are installed and the dispatch is set to look for first at teradata_utils for the dbt_utils macro namespace.
The macro date_spine does exist in the dbt_utils package, though, as mentioned it doesn't exist in teradata_utils.
I venture, in light of your comment that we are facing the following:

I see that date_spine in dbt utils does mentions both generate_series and datediff, both of which do exist still as part of teradata_utils.
generate_series in dbt_utils still is called under the namespace dbt_utils, so I guess that one is not the issue.
datediff, on the other hand, is directly under the dbt namespace, I think that is the culprit since being under that namespace it doesn't point to the teradata version.
In line of the above I think either we will have to move our datediff to core, or still provide the custom macro.

Hi @Daniel-Itzul ,
This issue has been resolved with the latest dbt-teradata-utils 1.1.1 and dbt-teradata 1.6.7
Although, with 1.7.x dbt has moved the date_spine() to core , which we are working on right now.
But for now, date_spine() is working and generate_series() has also been overridden in dbt-teradata-utils latest release.


Started handling messages from background isolates.

companion engine pr: https://github.com/flutter/engine/pull/35174
design doc: https://docs.google.com/document/d/1yAFw-6kBefuurXWTur9jdEUAckWiWJVukP1Iay8ehyU/edit#
issue: https://github.com/flutter/flutter/issues/13937
example
main.dart
import 'package:flutter/material.dart';
import 'package:flutter/services.dart';
import 'dart:isolate';

void isolateMain(BackgroundIsolateBinding binding) async {
  BackgroundIsolateBinding.initializeBackgroundIsolate(binding);
  const BasicMessageChannel channel =
      BasicMessageChannel('dev.flutter.aaclarke.echo', StandardMessageCodec());
  final String result = await channel.send('hello');
  print(result);
}

void main() {
  final BackgroundIsolateBinding binding =
      BackgroundIsolateBinding.initializeRootIsolate();
  Isolate.spawn(isolateMain, binding);
  runApp(const MyApp());
}

class MyApp extends StatelessWidget {
  const MyApp({super.key});

  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      title: 'Flutter Demo',
      theme: ThemeData(
        primarySwatch: Colors.blue,
      ),
      home: const MyHomePage(title: 'Isolate Platform Channels'),
    );
  }
}

class MyHomePage extends StatefulWidget {
  const MyHomePage({super.key, required this.title});
  final String title;

  @override
  State<MyHomePage> createState() => _MyHomePageState();
}

class _MyHomePageState extends State<MyHomePage> {
  @override
  Widget build(BuildContext context) {
    return Scaffold(
      appBar: AppBar(
        title: Text(widget.title),
      ),
      body: Container(),
    );
  }
}

AppDelegate.swift
import UIKit
import Flutter
import os.log

@UIApplicationMain
@objc class AppDelegate: FlutterAppDelegate {
  override func application(
    _ application: UIApplication,
    didFinishLaunchingWithOptions launchOptions: [UIApplication.LaunchOptionsKey: Any]?
  ) -> Bool {
    let registrar = self.registrar(forPlugin: "echo");
    let channel = FlutterBasicMessageChannel(name: "dev.flutter.aaclarke.echo", binaryMessenger: registrar!.messenger());
    channel.setMessageHandler { (input, reply) in
      if let input_string = input as? String? {
        os_log("%@", input_string ?? "null");
      } else {
        os_log("unrecognized message");
      }
      reply(input)
    }
    GeneratedPluginRegistrant.register(with: self)
    return super.application(application, didFinishLaunchingWithOptions: launchOptions)
  }
}

Pre-launch Checklist

[ ] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[ ] I read the Tree Hygiene wiki page, which explains my responsibilities.
[ ] I read and followed the Flutter Style Guide, including Features we expect every widget to implement.
[ ] I signed the CLA.
[ ] I listed at least one issue that this PR fixes in the description above.
[ ] I updated/added relevant documentation (doc comments with ///).
[ ] I added new tests to check the change I am making, or this PR is test-exempt.
[ ] All existing and new tests are passing.

If you need help, consider asking for advice on the #hackers-new channel on Discord.

The overall idea seems fine. I'm not sure I understand the specifics of the implementation on the Dart side. Maybe we can go over the PR together on VC?

@gaaclarke here's a proposed reworking of the API as discussed in our chat:
https://github.com/flutter/flutter/compare/master...Hixie:flutter:isolate-platform-channels
It's semantically equivalent to the original code, just presented differently.

@gaaclarke here's a proposed reworking of the API as discussed in our chat: master...Hixie:flutter:isolate-platform-channels
It's semantically equivalent to the original code, just presented differently.

Thanks, I've incorporated most of it now to see how it worked out.  I still need to clean it up and incorporate your docs.  I made 2 changes:

Not requiring the binary messenger in the channel constructor as we discussed in https://github.com/flutter/flutter/commit/04194fd1ada2a94125b16fc2a9d21e2eed715ebc#commitcomment-81568761
I put the logic for acquiring the the RootIsolateToken on that class and started exporting it from services/platform_channels.dart.  I wanted to make sure that importing services was sufficient to allowing one to use platform channels on background isolates.

Thanks, I've incorporated most of it now to see how it worked out. I still need to clean it up and incorporate your docs.

Oops, I missed that you hadn't finished yet, feel free to ignore my review comments until you have. :-D

@dnfield I got everything cleaned up and all the tests passing and ready to review.  I introduced bool get useBackgroundIsolateBinaryMessenger to handle the web and tests case.

It looks like this pull request may not have tests. Please make sure to add tests before merging. If you need an exemption to this rule, contact Hixie on the #hackers channel in Chat (don't just cc him here, he won't see it! He's on Discord!).

False positive, there are integration tests.

Over the course of responding to feedback I broke the integration test which apparently doesn't run as a presubmit.  Reverting to open the tree.


Fix GelfHandlerFactory to be compatible with ServiceManager v3

The GelfHandlerFactory is currently not compatible with version 3 of the ServiceManager because the options array in the constructor is mandatory. In v3 the options are passed via the __invoke method.
This fix made the constructor options optional (to be compatible with v2) and also allows the options to be passed via the __invoke method (v3). Afaik no BC breaks.

Coverage increased (+1.09%) to 89.552% when pulling f0de3393828243fdd000e09f915e9c858a19e2f4 on arjanvdbos:fix-deprecated-factory-constructor into 237c9ac4eed24c47c5fc55d990c81ef8e4a3919a on neeckeloo:master.

Coverage increased (+1.09%) to 89.552% when pulling f0de3393828243fdd000e09f915e9c858a19e2f4 on arjanvdbos:fix-deprecated-factory-constructor into 237c9ac4eed24c47c5fc55d990c81ef8e4a3919a on neeckeloo:master.

Thanks.


use '|' to preserve line breaks in welcome message

What was the end-user problem that led to this PR?
I used the wrong block in the welcome message YAML. > strips the new line breaks, we want to use | instead.

@bundlerbot r+


Xalia v0.4.4 crashing inside GudlTokenizer

Please help. Actually I'm getting Xalia crash in the steam, but it also reproduced in standalone version:

Unhandled exception in Xalia:
System.FormatException: The input string '0.2' was not in a correct format.
at System.Number.ThrowFormatException[TChar](ReadOnlySpan1 value)    at System.Double.Parse(String s)    at Xalia.Gudl.GudlTokenizer.<>c.<.cctor>b__6_7(<>f__AnonymousType72 <>h__TransparentIdentifier0, TextSpan dec) in D:\a\xalia\xalia\xalia\Gudl\GudlTokenizer.cs:line 60

Message looks weird to me but I cannot get what is going on. More info in the logs:
system_info.log
crash.log

I forgot CultureInfo.InvariantCulture. Sorry about that. Should be an easy fix at least.

Just so I can make sure I test it, what locale are you using?

I wasn't able to reproduce this locally (I'm probably just not overriding my locale correctly) so I just went through and added InvariantCulture anywhere I could find that might be a problem.
Could you try with the current CI build? (https://github.com/madewokherd/xalia/actions/runs/11262113557/artifacts/2036979861)

Hi, thanks for the fix!
Not sure if it's working fine but at least it stop crashing now

Thanks, closing.


Slack / Discord Channel during Beta testing

Is your Enhancement request related to a problem? Please describe
Would be nice to have more open conversations than bug/enhancement tickets.
Having a Slack/Discord channel would allow us to interact with the Dev's, and answer/suggest ideas directly. Once chewed a bit, we could then create a Issue in GitHub?

I'm going to suggest the Forums as a place for this type of discussion: https://forums.universaldashboard.io/c/universal-automation/14
I'd rather not do real-time discussions on Slack\Discord as it's far too distracting. I'm happy to have a call if you'd like something more real-time. We should involve @leeberg if we schedule that.


License: 'Free' is ambiguous

Even if flatpak provides better sandboxing capabilities than previous package management systems in Linux, I want to know if the software I'm installing is opensource or not.
When the "License" field contains "Free", this is clearly ambiguous, because what does this exactly mean? Free as in beer (free-ware) or free as in freedom(FLOSS)? Please make this explicit. A way to do this would be to mention the specific name of the license. Or if this is too technical, just mention if the licence is approved by the OSI or FSF.
One example where this is not explicit: https://flathub.org/apps/details/org.gimp.GIMP

This is what it looks like in GNOME:

Great! no bug there. I guess we need a tooltip in the website.

I guess we need a tooltip in the website.

It's shown on click, for what it's worth.

It's shown on click, for what it's worth.

I guess you mean in Gnome. Yeah, on click or via tooltip, either way would be great to show in the website.

This is what it looks like on Clear Linux using Firefox. No tool tip when hovering.

It not implemented (yet)

Fixed in master. You can test it in the dev server
This is how it works now:

Proprietary Apps: just show Proprietary  (example: Spotify)
Open source apps with short license value: show the license(s)  (example: GIMP)
Open source apps with long license value: show dialog (example: Darktable)

This is great. I think the specificity of the license in the 'license' field helps visitors to flathub identify exactly what type of license the software has before they download it.


Add parse package with new AST structure

As described in the recent PR #381 that introduced more tests for model parsing/reading and code generation, I've been working on refactoring those phases of the compiler so that they work with a generic AST structure instead of directly depending on the antlr4-vensim library.  There are a few different motivations for this work:

Removing the direct dependency on antlr4-vensim from the existing model reading and code gen modules makes it much easier to support other input formats, primarily XMILE (as used by Stella Architect).

Making code gen mostly input-format agnostic will make it possible (and easier) to support other output formats besides C.  For example, I am interested in adding pure JS as an output format (can rely on much of the existing C code gen for this), which will make it easier to get up and running with SDE without the Emscripten C to WebAssembly compilation step.  We have also considered generating WebGPU code, which will be easier to explore once we remove the Vensim dependency in code gen.

Working with the AST can be much easier than implementing reading and code gen using the ANTLR visitor pattern.  Currently to add support for a new Vensim function, it is sometimes necessary to spread the implementation over a few different visit methods, which means that class instance-level state is needed to share information between those methods, which can be hard to understand and keep track of.  With the AST structure, all of that logic and state can generally be maintained in one place, making the code more readable and easier to maintain.

Using an AST structure also makes it easier to implement compile-time optimizations, such as more aggressive constant folding and dead code elimination.  (I implemented a simple version of this optimization in #102, but it could only easily analyze simple expressions.  With the AST, I was able to implement a more aggressive algorithm that is capable of reducing a broader set of expressions.)

There are other types of tools (not related to the compiler) that can benefit from the AST structure.  For example, the hyperlinked documentation tool I wrote for the En-ROADS Technical Reference guide (see this Model Equations section) uses an earlier version of the parse package that I'm proposing here.

I have a branch available that adds a new @sdeverywhere/parse package (with the new AST types) and updates the existing compiler code to use new code paths (disabled by default) that use the parse package.
To avoid introducing lots of changes all at once, I'm proposing to make the changes in a few steps:

This first issue/PR will add the parse package to the repo (no changes to the compile package yet).
Once the initial work is merged, I will file a second issue/PR that includes the changes to the compile package, but disabled by default (can be enabled with a hidden environment variable to allow for testing).  The goal for this new code is that it is 100% compatible with the existing code, i.e., it generates the same code as the old implementation.
Once we are satisfied with the new code, I will file a third issue/PR to make the new code the default and remove the old implementation.

(None of these steps should result in any behavior changes for users of SDEverywhere.)

I'm looking forward to trying this out on EPS in step 2. I will be curious to see how the code generation changes. The need to parse parts of the model repeatedly and to build up the code gen in several parts of the parse tree are the hardest parts of adding new features for me.
It would be great if we could target pure JS and drop Emscripten from the tool chain. We would only be able to do that if there was no significant performance hit. That's becoming a limiting factor for us as the model grows. I remember asm.js that came before Emscripten did all kinds of tricks to get the JS running fast, so I'm thinking a pure JS target won't be simple to implement. But first things first!

I'm looking forward to trying this out on EPS in step 2. I will be curious to see how the code generation changes.

If the tests are any indication, the new code gen should produce 100% identical code to that of the old (excepting maybe for slight differences in the constant conditional optimization).  But we can evaluate that in the next round.

It would be great if we could target pure JS and drop Emscripten from the tool chain. We would only be able to do that if there was no significant performance hit.

Yeah, I see the pure JS target being a good default output format with acceptable performance for smaller models, but likely too slow to use in production for larger models like En-ROADS and EPS.  In my initial evaluation, it looks like the JS code gen can reuse most of the C code gen, and the more time consuming part will be porting over the C runtime parts to JS.  But like you say, first things first!


Ability to serve custom BSP to flasher locally

Hi,
We're patching the pinmux on the BSP and have been manually changing the l4t URL for the AGX Orin in lib /resin-jetson-flash.js; It would be super nice for the flasher to be able to take a local BSP archive instead of making changes to the flasher repo.

Hi, you can use the same approach used for the uefi firmware: https://github.com/balena-os/jetson-flash/blob/master/lib/resin-jetson-flash.js#L214
Just note that custom board configuration files are flashed into the QSPI and if you are using the devkit image, the UEFI capsule will overwrite them during a OS update, with the ones used for the Devkit


Text Classification

This module is regarding the code assessment task.
It contains the text classification model in Keras and integrated with MLFlow pipeline.

Any update here?

Any update here?
Let me know if anything is required from my end.


Red dot added when the css is set to compressed.

The plug-in works fine when it's set to nested, but as you can see on the left hand panel, there is a small red dot before all of the compiled css which is stopping the all selector from working.

Seems to be a zero-width no break space, found from here: https://apps.timwhitlock.info/unicode/inspect?s=
Brackets shows it as a red dot. Any idea why this is being compiled into my css?
This is the beginning of my scss file:

I've now found that this disappears if I export the css into its own folder, so I'll use this as a workaround for now.


Can't delete topic using pykafka/cli/kafka_tools.py script

Hello, folks,
I'm trying to remove a topic from Kafka using the pykafka/cli/kafka_tools.py script, but getting an error message pykafka.exceptions.SocketDisconnectedError: <broker b'<IP_ADDRESS>':9092>. Please, suggest, if I do something wrong.
You may find shell command and it's traceback below.
Thanks!
(py36) macmini987:notifications_api vserhei$ python .tox/py36/lib/python3.6/site-packages/pykafka/cli/kafka_tools.py --broker=<IP_ADDRESS>:9092 --broker_version=0.10.0 delete_topic test_topic_1
Traceback (most recent call last):
  File ".tox/py36/lib/python3.6/site-packages/pykafka/cli/kafka_tools.py", line 466, in <module>
    main()
  File ".tox/py36/lib/python3.6/site-packages/pykafka/cli/kafka_tools.py", line 460, in main
    args.func(client, args)
  File ".tox/py36/lib/python3.6/site-packages/pykafka/cli/kafka_tools.py", line 280, in delete_topic
    client.cluster.controller_broker.delete_topics([args.topic], args.timeout)
  File "/Users/vserhei/src/notifications_api/.tox/py36/lib/python3.6/site-packages/pykafka/broker.py", line 45, in wrapped
    return fn(self, *args, **kwargs)
  File "/Users/vserhei/src/notifications_api/.tox/py36/lib/python3.6/site-packages/pykafka/broker.py", line 584, in delete_topics
    return future.get(DeleteTopicsResponse)
  File "/Users/vserhei/src/notifications_api/.tox/py36/lib/python3.6/site-packages/pykafka/handlers.py", line 74, in get
    raise self.error
  File "/Users/vserhei/src/notifications_api/.tox/py36/lib/python3.6/site-packages/pykafka/handlers.py", line 212, in worker
    res = shared.connection.response()
  File "/Users/vserhei/src/notifications_api/.tox/py36/lib/python3.6/site-packages/pykafka/connection.py", line 217, in response
    raise SocketDisconnectedError("<broker {}:{}>".format(self.host, self.port))
pykafka.exceptions.SocketDisconnectedError: <broker b'<IP_ADDRESS>':9092>
(py36) macmini987:notifications_api vserhei$
(py36) macmini987:notifications_api vserhei$
(py36) macmini987:notifications_api vserhei$ pip freeze | grep pykafka
pykafka==2.7.0
(py36) macmini987:notifications_api vserhei$

Hi @vikt0rs, and thanks for your report. I see that you've set --broker-version to 0.10.0, but what version of Kafka are the brokers you're talking to actually running? A mismatch there is a likely cause of this error.

Hello, @emmett9001.
Yes, it is 0.10.0 (at least, I think so based on the server ps command). If you suggest a trusty way to check it, I'll do it and inform you about the result.
Thanks!

Oops, looks like topic creation/deletion APIs were added in 0.10.1, not 0.10.0. I'll fix that check in the Broker code.

Fixed here https://github.com/Parsely/pykafka/commit/7cf21a68b2a8848c632001a88e7ede0d959eea8b


Feature request: hot reload on 'lib http'

It would be really useful to have hot reloading on the lib http local development server, so we don't have to kill and relaunch it.

We'll be refactoring a lot of how the local lib instances work so we can definitely touch on this :).
Will get back to you when we push something.

For the time being, you can use nodemon
$ npm install -g nodemon
$ nodemon -i node_modules -x "lib http"


#21720 add the drop statement part for routines definition

It will work only with the "Show header" command
You can check it on Redshift and Greenplum as well.

verified


Update open-amp licence md5sum

The LICENSE.md file was changed after v2024.05.
This PR is required for CI to build from master

@bentheredonethat @mhatle This is trivial and required. It should be easy to approve.

LGTM


I get the wrong in Training stage 1

python Segment3D/third_party/segment-anything/scripts/process.py
--checkpoint "/hdd0/qinmy/checkpoints/external/Grounded-Segment-Anything/sam_vit_h_4b8939.pth"
--model-type vit_h
--input dataset/scannet/scannet_2d
--output output/segment3d/test
this is my training instruction, but i get some wrong response as following:
发生异常: ValueError
too many values to unpack (expected 4)
File "/ssd0/qinmy/code/test/11/Segment3D/third_party/segment-anything/scripts/process.py", line 200, in get_sam
masks, masks_s, masks_m, masks_l = mask_generator.generate(image) #segment-anything/segment_anything/automatic_mask_generator.py
File "/ssd0/qinmy/code/test/11/Segment3D/third_party/segment-anything/scripts/process.py", line 337, in main
sam_groups, sam_s_groups, sam_m_groups, sam_l_groups = get_sam(color_image, mask_generator)
File "/ssd0/qinmy/code/test/11/Segment3D/third_party/segment-anything/scripts/process.py", line 353, in
main(args)
ValueError: too many values to unpack (expected 4)
my datasets as follows:

the wrong code as follw picture ：
for image_file in rgb_images:
image_id = image_file.split('.')[0]
        color_path = os.path.join(rgb_folder, image_file) #'/hdd0/qinmy/dataset/scannet/scannet_2d/scene0000_00/color/3700.jpg'
        color_image = cv2.imread(color_path)  #(240, 320, 3)
        color_image = cv2.cvtColor(color_image, cv2.COLOR_BGR2RGB)
        color_image = cv2.resize(color_image, (640, 480))  #(480, 640, 3)

        sam_groups, sam_s_groups, sam_m_groups, sam_l_groups = get_sam(color_image, mask_generator) （this is the wrong position）
        sam_masks = extract_masks(sam_groups)
        sam_l_masks = extract_masks(sam_l_groups)

        contained_indices = find_contained_masks(sam_masks, sam_l_masks, iou=0.9)
        sam_whole_masks = [sam_masks[i] for i in range(len(sam_masks)) if i not in contained_indices]
        sam_whole_masks.extend(sam_l_masks)
        sam_whole_groups = return_group_ids(color_image, sam_whole_masks)

        sam_image_path = os.path.join(sam_folder, f'{image_id}.png')
        img = Image.fromarray(num_to_natural(sam_whole_groups).astype(np.int16), mode='I;16')
        img.save(sam_image_path)

def get_sam(image, mask_generator):
masks, masks_s, masks_m, masks_l = mask_generator.generate(image)  (this is the wrong position）
group_ids = return_group_ids(image, masks)
group_ids_s = return_group_ids(image, masks_s)
group_ids_m = return_group_ids(image, masks_m)
group_ids_l = return_group_ids(image, masks_l)
return group_ids, group_ids_s, group_ids_m, group_ids_l
i find the "mask_generator.generate(image)" get many results when i debug this code. i want to know why i get this issue? i didn't
change any code or datasets. How can i deal with it ?

I have already resolved it

I have already resolved it

I get same issue, how do you resolved it?

I have already resolved it

I get same issue, how do you resolved it?
The author said that one of the files needs to be moved to segment anything. You can check the installation environment guide to solve this problem


Add Raphnet.net's 4NES4SNES

Plug in four original Nintendo Famicom or Super Nintendo controllers and
convert them to USB.  (Arduino source code is available at raphnet.net.)

Added from fork. Thank you for the PR!


Fix the 1 Day price chart

Description
The historical price end time is currently getting rounded to the start of day UTC, regardless of interval. Fix to not round the end time when the interval is hourly, this allows it to show up to date pricing on the 1 day chart.
Both of the screenshots below were taken at the same time:
Before:

After:

Type of change

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Other

How should this be tested?

[ ] Navigate to the swap page, hover over the latest hourly price, ensure it is in line with the current time

Checklist:

[x] I have performed a self-review of my own code
[x] I have commented my code where relevant, particularly in hard-to-understand areas
[x] My changes generate no new console warnings
[x] The base of this PR is master if hotfix, develop if not

Thanks for this @danielmkm !


Omni-channel payments tab missing from version 10.0.29

This page and other pages makes reference to enabling the omni-channel payments on the Retail and Commerce > Headquarters setup > Parameters > Commerce shared parameters page. however this option is missing in version 10.0.29.
Is this a bug in 10.0.29 or has the option been moved or deprecated?
Can this page be updated to reflect how to set this option in 10.0.29 onwards?

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

ID: afc32bd1-d8ea-aff8-aa80-8c668dd26582
Version Independent ID: 0f24c912-b1cf-3fd8-6a4e-5d57635d3513
Content: Linked refunds of previously approved and confirmed transactions - Commerce | Dynamics 365
Content Source: articles/commerce/dev-itpro/linked-refunds.md
Service: dynamics-365-commerce
Product: ****
Technology: ****
GitHub Login: @josaw1
Microsoft Alias: anupamar

Thank you for your feedback. @anupamar-ms can you please review?

@josaw1 I have PR 16101 created to address this request.