Is it possible to use getx with platform widgets

iPhone represents a considerable amount of phones out there, and NONE of the iOS users wan't to have their apps looking like an Android app.
There's a package that helps you to address the effort for both iOS and Android widgets,
Is it possible to use getX and platform widgets together?
https://github.com/stryder-dev/flutter_platform_widgets
also, there is an issue already at plataform widget's github:
https://github.com/stryder-dev/flutter_platform_widgets/issues/260
But I think this is more likely on the getx side

it seems this package is not ready to production yet, no one answers, 275 issues opened
quite dangerous to use a package like this.


[WIP] Implementing a basic ffi

This is a basic ffi I created, it's pretty basic and needs some tests still. Does this seem like it's going in the right direction? Also, I used it here, as sort of a testing ground.

@fitzgen fixed most of the nitpicks you brought up, still need the CI though. For that should I add a demangling test within c_api.rs or create a C file that calls that function?
And okay I'll start hacking on an example C project, that'll be pretty cool when it's done imo :smile_cat:

For that should I add a demangling test within c_api.rs or create a C file that calls that function?
And okay I'll start hacking on an example C project, that'll be pretty cool when it's done imo smile_cat

I think it should be a C file, and then this can incrementally grow to become the example C project :)

@fitzgen what do you think of this tiny C project?

Let me know how these comments look, and I believe all that should be left is probably hooking it up to CI right?

Let me know how these comments look, and I believe all that should be left is probably hooking it up to CI right?

Yep, looks good!

Added the c_api and the corresponding tests to CI, let me know how that looks. :)


0xe1ec44ff6f1198324bda891ba91245033a116c6e same farmer action

Related Safe Addresses
0x4e71e67f620e91c913182945a614ee19fc52bac0
0xb10aa150db8109c33cfd71494bab1a5929eb82e8
0xbe79702c319015891bf89f59c442f0d701de22fe
0xdd756b21adef1c0744f9e113325458aac0c6398b
0x73b98e7f2752332f9aba816012e51319e2363b48
0xb66b25aa7c094fbd69b96f3ddcee45aaa37e3ba7
0x8b0d8d09f3778c9fb450b353cc470a266654ca7a
0x10f32bc1b49a0d0251b415f4653dcf9e3060fee0
0x29cad380de413ef24f77a4f3aafeb6ecfbf82a0d
0x8acbc5cfad966982881ac430a9fa0c7c00801ac4
0x0be66b26946fcc430ece870c6587ea3b53621645
0x29714481e30dc4ce7eea47fb3b18149ee8c7055d
0x1d6e769e217d968f25b28d10a07c16a8464adf30
0x0d6a93b78eb9697e2d3ee87d298bfbc28bb0910b
0xb9efd0f77ab2ac427b405606132ea12b6d058f0e
Reasoning
15 safe address, check the transactions of the safe addresses one by one , find that the from of txs are all 0xe1ec44ff6f1198324bda891ba91245033a116c6e or 0x31c1bde6d298d15d7b924079b099488d0c99cc92, it's 2 human EOA address, and the the nums of txs of safe address are almost between 1 to 5(except one is 15 ,one is 13), and almost 200 days ago, the balance almost is 0.
Methodology
download txs of safe addresses ,and check the common action of the txs.
https://etherscan.io/address/0x4e71e67f620e91c913182945a614ee19fc52bac0
https://etherscan.io/address/0xb10aa150db8109c33cfd71494bab1a5929eb82e8
https://etherscan.io/address/0xbe79702c319015891bf89f59c442f0d701de22fe
https://etherscan.io/address/0xdd756b21adef1c0744f9e113325458aac0c6398b
https://etherscan.io/address/0x73b98e7f2752332f9aba816012e51319e2363b48
https://etherscan.io/address/0xb66b25aa7c094fbd69b96f3ddcee45aaa37e3ba7
https://etherscan.io/address/0x8b0d8d09f3778c9fb450b353cc470a266654ca7a
https://etherscan.io/address/0x10f32bc1b49a0d0251b415f4653dcf9e3060fee0
https://etherscan.io/address/0x29cad380de413ef24f77a4f3aafeb6ecfbf82a0d
https://etherscan.io/address/0x8acbc5cfad966982881ac430a9fa0c7c00801ac4
https://etherscan.io/address/0x0be66b26946fcc430ece870c6587ea3b53621645
https://etherscan.io/address/0x29714481e30dc4ce7eea47fb3b18149ee8c7055d
https://etherscan.io/address/0x1d6e769e217d968f25b28d10a07c16a8464adf30
https://etherscan.io/address/0x0d6a93b78eb9697e2d3ee87d298bfbc28bb0910b
https://etherscan.io/address/0xb9efd0f77ab2ac427b405606132ea12b6d058f0e
Safe Address
0xd8878417378CB18D315b4768503fC39487f90580

Reasoning
15 safe address, check the transactions of the safe addresses one by one , find that the from of txs are all 0xe1ec44ff6f1198324bda891ba91245033a116c6e or 0x31c1bde6d298d15d7b924079b099488d0c99cc92, it's 2 human EOA address, and the the nums of txs of safe address are almost between 1 to 5(except one is 15 ,one is 13), and almost 200 days ago, the balance almost is 0.

Owning several Safes is not airdrop farming per se. These Safes seem to have for some time been storing 2 ERC20s (HOTCROSS and RANKER). Store of value is a legitimate Safe usage.
Please elaborate on how this is airdrop farming and how you can be sure to no eliminate legitimate usage.

We haven't received any further information on this ticket and therefore deemed this submission invalid.
Unfortunately just the pure fact that Safes...

were created by the same address
have the same owner
conducted the same transactions
received assets from the same address

... is not enough reasoning for a valid submission, as there are legit user behaviour leading to such as well (relayer, scripts, power users, exchanges)
This submission didn't provide sufficient indicators to convince the team that there is some targeted airdrop farming behind the provided addresses. As per the rules, when in doubt we will default in favor of the airdrop recipients in order to not risk any false positives when sorting out airdrop hunters.


Add geo-replication support to config provider

This PR adds support for Geo-replication.
With the new Connect(IEnumerable<Uri> endpoints, TokenCredential credential) API, users can now specify multiple endpoints to use when the primary endpoint fails (i.e. When primary endpoint returns HTTP Status code 503 or 429).
The replication logic retries the primary configuration store after a failure using exponential backoff mechanism.
The samples are now updated to utilize the new Connect API as well.

internal class RefreshConstants

I don't see too much value is added by calling these "refresh" constants, "retry" constants, etc. Why don't we just call the type Constants and put all constants in it. We don't have that many constants anyway.

Refers to: src/Microsoft.Extensions.Configuration.AzureAppConfiguration/Constants/RefreshConstants.cs:8 in 9081eb1. [](commit_id = 9081eb1009e5c5091f84d461dc6653ec51f2ca0c, deletion_comment = False)


Documentation Details

Simply adding documentation for adding templates with mock data.
Signaling in the readme that this is JUST for testing purposes as of this moment.
Please don't merge until the other pr is merged in, and this is consequently rebased.

Anything you guys can catch here is greatly appreciated


Window transparency not showing window after build

We have a transparent window using setTransparent(true).  When we gulp run() this window works just fine.  When we gulp build() with platform 'win' and then run outputed exe, the window never shows.

I set transparency with package.json, and I had a same issue.

Does it work when running the app with nw?


Documented method to deploy to Java SE app service does not work

As documented in another doc page, I am using this line to deploy a self contained Spring Boot app as a jar to a Java SE app service from a DevOps pipeline;
- script: curl -X POST -u '$(username)':'$(password)' --data-binary @"target/app.jar" https://pim-01-prod.scm.azurewebsites.net/api/publish&type=jar
Notice I am using the recommended jar name app.jar.
These are the issues I noticed:

the official Kudu REST API doc does not list the /api/publish api that Azure recommends
using SSH into the container, I cannot find app.jar anywhere on the disk, so I cannot use a Startup File / Command to refer to the jar file.
my Linux Java SE app does not have a Startup File as documented, but a Startup Command

So this is my use case, please help me accomplish it by whatever means

I have a Linux App Server, the stack is Java SE - I could move to a Windows App Service if needed
I need an unattended deploy by a DevOps pipeline
I have a Spring Boot app that contains its own Tomcat, packaged as jar or war, does not matter.

What combination of deployment method and Startup Command I need to use? What am I missing?
[Enter feedback here]

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

ID: df68e904-7b05-38ba-4203-50eb8491946f
Version Independent ID: 921377f7-e428-143d-70e4-c9efc25e41d1
Content: Configure Java apps - Azure App Service
Content Source: articles/app-service/configure-language-java.md
Service: app-service
GitHub Login: @JasonFreeberg
Microsoft Alias: jafreebe

Hi @ciufudean , thanks for bringing this to our attention. I have assigned the issue to the content author to evaluate and update as appropriate

I found DevOps documentation that confirms my findings.
The auto detection logic says ZipDeploy will be used for a JAR file https://docs.microsoft.com/en-us/azure/devops/pipelines/tasks/deploy/azure-rm-web-app-deployment?view=azure-devops#auto-detect-logic.
Following on, ZipDeploy deploys the file contents to the wwwroot folder of the App Service.
So no JAR file anymore that I can start with java -jar ....

Hello, any progress on this one?

Hello, any progress on this one?

Hello, any progress on this one?

#needs-sme-input

This doc has been rearchitected. Please reopen another issue if help is needed on the updated doc.
#please-close


Pull .com, .net, or .org of only Available domains

Hey guys,
First, want to say this script is great, so thanks for your work on this. I have modified it for what works best for me, but since you guys wrote the base, I at least wanted to provide those changes to you. This might not be want you want to do with the project, so feel free to reject it or not merge it in. I just wanted to make sure to share my mods with you in case it is helpful.
The changes in this are:
The tool no longer pulls all domains from the table containing the expired domains search results. It only pulls .com, .net, or .org domains. It no longer skips a domain if archive.com doesn't have a record of it. Finally, it also only selects domains which are immediately "Available" vs. those which require a bid (and may take some time for the bid process to complete).
It will parse the table(s) and hold the applicable domains in a list. The domains are only checked for reputation after all domains have been sorted for applicable domains (vs. checking all of the results).
The "table" which is printed to the console might not be working correctly, not sure but might be from my modifications. However the html report which is saved after a run does properly display everything still.
Anyways, hope this helps in some way. Feel free to reject or use!
Thanks for your guys work!

Hey @ChrisTruncer, thanks for taking the time to make a PR! Just reading over your proposed changes and they definitely seem worthwhile. I will test it out in the next few days and hopefully get it merged.
Thanks again!

Hey Chris, thanks again for this idea. I had several new uncommitted changes locally and decided to just copy some of your ideas into the latest commit.
I still need to modify it to wait until the end of the domains retrieval before performing all the reputation checks, but that's for another day. :)


proxy support?

I saw an old feature request on the Google Code site where someone from the Guardian Project indicated plans to add Orbot/proxy support to K-9 mail. Has there been any movement on this? If not, how challenging would this be to implement? My coding experience is not extensive, but I'm ready to spend the time necessary to figure out how to do this if no one else is working on it and if it's not a horribly advanced project.

Please only use this issue tracker for bug reports or feature requests. If your title ends in a question mark it's neither.
I created issue #704 for this.


Issue with recurisve parsing and typescript

So I have the following setup for parsing a recursive rules tree
interface RuleNode {
    ruleId: string;
    description?: string[];
    children: RuleNode[];
    ruleOperation: RuleOperation;
    ruleField?: string;
    ruleValue?: string;
    ruleValueType?: RuleValueType;
}
export type RuleSyntaxTree = RuleNode;

export const RuleNode: z.ZodType<RuleNode> = z.lazy(() =>
    z.object({
        ruleId: z.string(),
        description: z.array(z.string()).optional(),
        children: z.array(RuleNode),
        ruleOperation: z.nativeEnum(RuleOperationInternal),
        ruleValue: z.string().optional(),
        ruleField: z.string().optional(),
        ruleValueType: z.string().optional(),
    }),
);

This gives me the following error on the validator.
Type 'ZodLazy<ZodObject<{ ruleId: ZodString; description: ZodOptional<ZodArray<ZodString, "many">>; children: ZodArray<ZodType<RuleNode, ZodTypeDef, RuleNode>, "many">; ruleOperation: ZodNativeEnum<...>; ruleValue: ZodOptional<...>; ruleField: ZodOptional<...>; ruleValueType: ZodOptional<...>; }, "strip", ZodTypeAny, { .....' is not assignable to type 'ZodType<RuleNode, ZodTypeDef, RuleNode>'.
  Types of property '_type' are incompatible.
    Type '{ ruleId?: string; description?: string[]; children?: RuleNode[]; ruleOperation?: RuleOperation; ruleValue?: string; ruleField?: string; ruleValueType?: string; }' is not assignable to type 'RuleNode'.
      Property 'ruleId' is optional in type '{ ruleId?: string; description?: string[]; children?: RuleNode[]; ruleOperation?: RuleOperation; ruleValue?: string; ruleField?: string; ruleValueType?: string; }' but required in type 'RuleNode'.t

is there something im doing wrong? Can this code be simplified.

The code compiles successfully on my machine (zod v3.18.0).
I wonder what's causing ruleId to be optional in your environment :thinking:

Hmm yah not sure. Might need to update my Typescript version.

could it potentially be a result of me not running in strict mode?

So I have create a brand new workspace, and I still get the same error. If I just make everything optional then it does actually work
enum RuleValueType {
  NUMBER,
}
interface RuleNode {
  ruleId?: string;
  description?: string[];
  children?: RuleNode[];
  ruleOperation?: 'AND';
  ruleField?: string;
  ruleValue?: string;
  ruleValueType?: RuleValueType;
}
export type RuleSyntaxTree = RuleNode;

// Cant figure out the typing here. However this works and is recommended by the Zod library
export const RuleNode: z.ZodType<RuleNode> = z.lazy(() =>
  z.object({
    ruleId: z.string().optional(),
    description: z.array(z.string()).optional(),
    children: z.array(RuleNode).optional(),
    ruleOperation: z.enum(['AND']).optional(),
    ruleValue: z.string().optional(),
    ruleField: z.string().optional(),
    ruleValueType: z.nativeEnum(RuleValueType),
  })
);

using     "zod": "^3.19.1"

Okay after further debugging I think I thought I had turned strict mode on... but on second look it was turned of.
Enabling strict mode seems to have solved my problem


Write a User Guide for implementing AccountActivity in a Client Application.

The AccountActivity API https://github.com/tweepy/tweepy/issues/1075 is implemented but the way to use it is quite obscure, especially if the developer doesn't have experience of webhooks. In the same way as the Authentication Tutorial helps with that the Account Activity Tutorial should give a clear example for implementing its use in a client application.

This is stale now that the Premium v1.1 Account Activity API has been deprecated.


Install game files from the base image

Probably fails because the DST game files don't have the proper permissions set or something after they get copied from the base image.

Setting the owner after copying the files seems to fix the issue although running the chown -R $STEAM_USER:$STEAM_USER $DST_HOME takes quite a while.

Maybe it's possible to use USER $STEAM_USER instead of running the chmod command but that may have other effects.

I definitely like the idea of splitting this images in two. But why do you use the multi-stage builds? What do we get from them?

In terms of programming language, multi-stage builds enable us to use to use composition instead of inheritance. We can copy/use different/multiple images into a final image instead of just use a single image. Also, depending on how it is used, the final image is potentionally smaller.

Also, I thought using scratch as base image helps to get the smallest possible image for dst-academy/docker-dontstarvetogether-base which only holds the game files. It's also faster to build. That means we can't use it as the base image for dst-academy/docker-dontstarvetogether, but that can be solved by just using a multi-stage build. It's just a prototype to find out if it works for us. So far I pretty much like it.

Okay, it seems using an environment variable for --chown doesn't work. :/


State of HCC support

The builds for the hcc variant got dropped in https://github.com/conda-forge/clangdev-feedstock/commit/b350f6903769d803ae09f42577ef1844450e8ff3 over 4 years ago, is anyone still using this? The only user I found is hcc itself, which hasn't seen a non-bot commit since https://github.com/conda-forge/hcc-feedstock/commit/e8836764cb658df5fc33ade19e99c78ee8e67d00.
More importantly, upstream HCC has also not seen any commits since 4 years. It notes:

Deprecation Notice
AMD is deprecating HCC to put more focus on HIP development and on other languages supporting heterogeneous compute.    We will no longer develop any new feature in HCC and we will stop maintaining HCC after its final release, which is planned for June 2019.  If your application was developed with the hc C++ API, we would encourage you to transition it to other languages supported by AMD, such as HIP or OpenCL.    HIP and hc language share the same compiler technology, so many hc kernel language features (including inline assembly) are also available through the HIP compilation path.

I know that there's interest in bringing up ROCm support in conda-forge, but I guess hcc won't be the way to do it. I suggest to clean out the remains from this repo.
CC @conda-forge/hcc

CC @conda-forge/clangdev, I'm planning to drop the hcc bits in #293

Closed by https://github.com/conda-forge/clangdev-feedstock/commit/9b933b12354c79c5984867d05dd2a85596b5c917


Fixed NIST links and redundancy.

Description of Work
Fixes #23
The NIST links are created only once now and are also correct.
Testing Instructions

In example_runScripts.py, set run_data = "nist"
Run example_runScripts.py in the mantidpython console.
Check if the links the the accuracy and runtime tables work.
Run example_runScripts.py in the mantidpython console with run_data = "neutron"
The tables should still have the working NIST links, this is a known issue.

tested and url to nist pages works


Building with module_webp_enabled="no" breaks svg support

Tested versions
Reproducible in: master, 4.2 branch, 4.2.1-sable
Notes

I built the 4.2.1 with many other disabled options so I am not so sure about it
When I exported the projects I used standard 4.2.1-stable editor because it just packs .pck file and place it near the pre-compiled export template...?
Tested with and without SCU
Tested with mold and partially with default gcc linker

System information
Godot v4.2.1.stable - Debian GNU/Linux testing (gcc12) - Wayland - GLES3 (Compatibility)
Issue description
Compiling export_template with module_webp_enabled="no" doesn't throw any compilation errors but when I try to export project that use svg it suddenly crash with following erros:
ERROR: Condition "img.is_null() || img->is_empty()" is true. Returning: Ref<Image>()
   at: load_image_from_file (./scene/resources/compressed_texture.cpp:344)
ERROR: Failed loading resource: res://.godot/imported/icon.svg-218a8f2b3041327d8a5756f3a245f83b.ctex. Make sure resources have been imported by opening the project in the editor at least once.
   at: _load (./core/io/resource_loader.cpp:277)
ERROR: Failed loading resource: res://icon.svg. Make sure resources have been imported by opening the project in the editor at least once.
   at: _load (./core/io/resource_loader.cpp:277)
ERROR: Can't load dependency: res://icon.svg.
   at: parse_variant (./core/io/resource_format_binary.cpp:459)
ERROR: Error when trying to parse Variant.
   at: parse_variant (./core/io/resource_format_binary.cpp:503)
ERROR: Error when trying to parse Variant.
   at: parse_variant (./core/io/resource_format_binary.cpp:490)
ERROR: Failed loading resource: res://.godot/exported/133200997/export-523cfd0014c18c3c941ab60c4947535f-icon.scn. Make sure resources have been imported by opening the project in the editor at least once.
   at: _load (./core/io/resource_loader.cpp:277)
ERROR: Failed loading scene: res://icon.tscn.
   at: start (main/main.cpp:3800)

Steps to reproduce
Compile export template with module_webp_enabled="no" and export project that use svg
Minimal reproduction project (MRP)
Project for export:

Create new project
Drag default godot svg icon to create new scene from this icon

custom.py script:
target="template_release"
debug_symbols="no"
lto="full"

linker="mold"

platform="linuxbsd"
arch="x86_64"

module_webp_enabled="no"

Like JPG and PNG, SVG files are rasterized to WEBP internally.
Disabling the webp module should make it fallback on saving as PNG, but already imported files may not be automatically reimported with the new format, and thus it fails loading the existing res://.godot/imported/icon.svg-218a8f2b3041327d8a5756f3a245f83b.ctex.
Could you check if it works after deleting the .godot folder, which should reimport icon.svg as PNG?

Could you check if it works after deleting the .godot folder, which should reimport icon.svg as PNG?

It looks like I also need to compile the editor itself with disabled webp. Am I right? Anyway I will try

Ah indeed, or use the rendering/textures/lossless_compression/force_png project setting.

Yeah it works now!
About the bug. At least if it is not too expensive for engine, I would like to know it is in the wrong/unsupported format rather then getting Make sure resources have been imported by opening the project in the editor at least once. which is a bit frustrating...
If I use only svg's so I don't have neither png or webp and want to minimize template (yeah I know it will give at max 100kb space but anyway...) I should disable png because webp is default or prefer png?

Reclassifying this as a core issue, as it would likely require changes to the resource loader to handle this better.
I'm not sure there's any easy solution, textures imported with a webp-enabled editor will be saved as webp (using force_png is enabled), and if an export template is then used which doesn't support webp, they can't be loaded. It can't fallback to png at runtime since the files will have been imported already, so there's no source file to load/re-import.
So it might well be that the best solution we have here is just better documentation on the consequences of disabling some features for export templates, and how to prepare one's project in the editor to use such a custom template.

Personally, I would make the WebP module a hard requirement. You'll probably save more space by enabling WebP and using WebP-compressed images than you'd save by disabling WebP and using PNG instead.

Personally, I would make the WebP module a hard requirement for compiling Godot. You'll probably save more space by enabling WebP and using WebP-compressed images than you'd save by disabling WebP and using PNG instead. Lossless WebP is a lot better than PNG at compressing images.

I think I might agree. I am not sure but it looks like currently the situation is diametrically opposite. PNG is a hard requirement for compilation. It is a "driver" not a "module" (idk, if it means anything about its hard requirementness). But anyway I didn't found an option to disable PNG in compile time while it is possible to disable WEBP


Issue-30 ; Fixed matchesResourceIdName resulting in NotFoundException on Android P when trying expand WebView PopUp list

Related issue: https://github.com/InflationX/Calligraphy/issues/30

When is this fix schedule for a release?


Roadmap for Next Year

Look at transmission code + CD4 / viral load for people not on treatment
Testing diagnosis (depends on date of infection)
Assume 3 drugs: tenofovir (ten), 3tc, dol
Only subset of resistance mutations are relevant to these drugs
Starting with ART & adherence, then resistance
ignore code with pop_wide_tld=1
Prep later if we have time!

Hi @mmcleod89 and @pineapple-cat
For when you return from holidays:
What would be the best way for us to comment on existing code ?  I tried to create a branch from development and was going to insert any questions or comments I had in that and then do a dummy pull request, but I'm sure there is a better way.  In any case, I failed to sync with pycharm so I clearly need some more help with doing that at some point.
Also, I think it would be helpful if you show us how to run a basic version of the model as it stands.  Even if all this does is update people's sexual behaviour it will be very useful for us to start to see outputs.
Thanks

Unfortunately, I don't believe it's currently possible to comment on existing code that hasn't been changed. This really should be a feature though, it would be very helpful for our use case (and it would help other translators too).
In any case, you seem to have had the right idea and I can't think of a better workaround than this:

Make a new branch (e.g. hivpy-discussion)
Make a new PR (e.g. HIVpy discussion)
Add various short remarks/questions to the Python code and push the changes
Then add further/longer GitHub comments to your initial Python comments

Find the Files changed tab in your PR
Click on the blue + icon next to any displayed line to leave a comment on that line (or click and drag the + icon down to select multiple lines to comment on)

We can use this PR as a forum for questions, answers, clarifications, suggestions, and anything else you can think of.

ok thanks Emily

Closing this roadmap issue as things have changed since our extension


Pull in Fix for Infinite Loops On Directories With 100+ Items

As was found by @slegdon the handling for how we were listing objects was incorrect causing it to enter an infinite loop when there were over 100 items within a search area (causing pagination to occur). This was due to an incorrect key being used as the offset for the operations.
This has been fixed with these changes and includes tests for the functionality to prevent regression.
Closes #138

Coverage remained the same at 82.765% when pulling a2e3888b84be6f9566e2d4e397544c9c1781a78b on slegdon-hotfix/hang_listing_large_directories into 4592e349116059450cbb45f4c45c67937efd91f4 on master.

Coverage remained the same at 82.765% when pulling a2e3888b84be6f9566e2d4e397544c9c1781a78b on slegdon-hotfix/hang_listing_large_directories into 4592e349116059450cbb45f4c45c67937efd91f4 on master.


New Version: UCBerkeley.BOINC version 7.16.20

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

[ ] Have you tested your manifest locally with winget install --manifest <path>?
[x] Does your manifest conform to the 1.0 schema?

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

Microsoft Reviewers: Open in CodeFlow

Wrong path

Sorry I messed this up


Add an optional adapter interface for configuring transaction behavior

pg and sqlite are autocommit right now; this could be configurable...
Would be nice  to have a checkbox (in the run query bar?) if the adapter supports this for enabling/disabling auto-commit.

Follow datagrip's lead:
Show a badge Tx: Auto; click that badge to switch to Tx: Manual.
Except that those states should be loadable from the adapter. So the adapter can declare a prop (class var) like TRANSACTION_MODES: list[str] = ["Auto", "Manual"],
The default mode should be configurable in a profile, although maybe that's a detail we leave t o the adapter author. In that case, maybe the right interface is something like
@property
def transaction_mode(self) -> str:
   """the name of the currently-active transaction mode"""
    ...

def toggle_transaction_mode(self) -> str:
   """Switches to the next transaction mode and returns the name of the new mode"""
   ...

this interface would also be compatible with older adapter versions, which is nice. We could just take the empty string to mean "not implemented"


Pcluster Manager not working on ap-south-1 region

Dear BarcoMasile and Marco Basile,
I hope this finds you well. I am writing to bring your attention to an issue that I have encountered with the PCLuster Manager in the ap-south-1 region.
I have noticed that the shell and nice DCV icons are not visible when using the PCLuster Manager in the ap-south-1 region, while it is working perfectly fine in the N Virginia region. I understand the importance of having a smooth and seamless experience when using the PCLuster Manager and I am writing to request your assistance in resolving this issue.
I would greatly appreciate your prompt attention to this matter as it is affecting my ability to use the PCLuster Manager effectively in the ap-south-1 region. If you could provide any information on the status of this issue or a timeline for its resolution, I would be most grateful.
Thank you in advance for your time and assistance. I look forward to hearing back from you soon.
Best regards,

Hello @dvpr-23, this issue has been fixed but not released. We'll release it soon!

Thank you @mtfranchetto . May i know the ETA on that.
Im really looking forward too it and very excited about that.

Hi @dvpr-23, unfortunately we do not provide ETA about future releases by policy, therefore I cannot provide you with an exact date for the time being. What I can tell you is that we are actively working on this and we plan to have it ready in a few weeks. I will provide an update on this ticket as soon as we will release the new version.

Thank you @mtfranchetto
but weeks seem to be bit long.
hoping and eagerly waiting for the release
Thank you though. Please update once the release happens
many of us in india are rooting for you

Hello @dvpr-23, you asked and we've released a new version of ParallelClusterManager, which has been renamed to ParallelCluster UI. Please head to the new repository to install the latest version https://github.com/aws/aws-parallelcluster-ui


Very slow fetching (includes STR)

Cloning Servo from a on-disk location to the same disk takes about one minute with the git command line. With git2, it can take hours. 've attached an example. It includes a checkout of Servo, that's why it's so big (400Mb).
Download, unzip, then cargo run --release. You should see something like progress: 3%. Time: 57 minutes.
https://drive.google.com/file/d/1kWlSwahyFyZirNzqd_d6O_gtmOGoR1X3/view?usp=sharing

I'm going to close this in favor of the upstream issue.


fix(versions): update Activiti/activiti-cloud-full-chart-security versions

UpdateBot pushed helm dependency: activiti-cloud-full-example to: 7.1.175

UpdateBot commands:
updatebot push-version --kind helm activiti-cloud-full-example 7.1.175


Feature: Support arbitrary downstream endpoints

The Ocelot config schema today only really works for http(s) endpoints. I'd like to be able to write my downstream endpoint as a fabric:/ uri, which would then be dynamically resolved by a custom IHttpRequester at the end of the pipeline.
Motivation for New Feature
Ocelot cannot dynamically resolve Service Fabric endpoints today (instead, it relies on SF Reverse Proxy, which adds an unnecessary hop). My proposal is a change in Ocelot that would enable me to build a custom IHttpRequester in my application (not a part of Ocelot) to dynamically resolve fabric:/ uri's and issue the appropriate request.
Proposal
The limitation today is how DownstreamScheme and DownstreamHostAndPorts are specified in ReRoutes:
"DownstreamScheme": "http",
"DownstreamHostAndPorts": [
  {
    "Host": "localhost",
    "Port": 80,
  }
]

which forces me to specify Host and Port, which only really make sense for http(s). My proposal is for Ocelot to support the following instead:
"DownstreamEndpoints": [
    "http://localhost:80",
    "fabric:/MyApp/MyService"
]

Benefits:

Convenient way to specify arbitrary downstream endpoints: http, https, or anything else, and even to mix them in a single ReRoute rule
Less verbose configs
Can be built backwards-compatible. Old config syntax DownstreamScheme + DownstreamHostAndPorts would still be accepted and parsed into the new model during init

Thoughts?

@davidni you don’t have to specify host and port for service fabric services. Please see docs http://ocelot.readthedocs.io/en/latest/features/servicefabric.html

Not quite. There are 3 common ways to reach Service Fabric services in a cluster:

Use SF Reverse Proxy explicitly (default is on port 19081): make an HTTP request to http://localhost:19081/app1/service1/path
Use SF DNS Service: make an HTTP request to http://service1.app1:8080/path which will resolve to the IP of a replica of the requested service. Downside is this can't do port resolution since DNS operates at the host name level
Explicit name resolution with SF Naming Service: C# code calls IServicePartitionResolver.ResolveAsync() to resolve the address of a service, then issues a call directly to it

Ocelot today uses (1). My proposal would enable me to do (3), which gives applications much finer grained control over endpoint resolution policies, retry loops, customized timeouts for each phase of the process, additional telemetry, etc.
The following articles might be helpful:

Connect and communicate with services in Service Fabric
DNS Service in Azure Service Fabric

@davidni thanks for more information. I’m happy to go with 3 as it has the most benefit for ServiceFabric users.
What is your solution at the moment? How would you envision doing this feature?

For now I am replacing Ocelot's IHttpRequester implementation with a custom class in my application. DI then injects my implementation into Ocelot's Middleware, and I issue the request myself using custom name resolution.
This has a few problems:

I am hijacking the entire pipeline to only issue Service Fabric service requests. I can't currently have a different route that would just issue a simple HTTP request with Ocelot's built-in HttpRequester. That's fine for my application, but probably not a good general assumption
Routing config overhead -- in order for the LoadBalancer middlewares to not complain, I am forced to specify a dummy DownstreamHostAndPorts for each of my ReRoutes. I literally have the following now:"DownstreamHostAndPorts": [
  {
    "_description": "These are ignored -- endpoint will be dynamically resolved",
    "Host": "localhost",
    "Port": 1
  }
]

My custom IHttpRequester adjusts the outgoing request with the proper hostname and port before issuing the request.

@davidni awesome thanks for this info! It really helps me out because I am not a ServiceFabric expert at all!! Could you share the custom IHttpRequester? I guess I would just do what I've done for all the other Ocelot things that need a different implementation and use a factory to get the ServiceFabricRequester depending on how the ReRoute is configured.

@davidni would you be able to share your custom IHttpRequester?

There's too much other code involved in my http outgoing request pipeline that I cannot easily strip out, sorry.
That said, my current course of action is to stay with Ocelot's existing model (1) Use SF Reverse Proxy explicitly. If I hit any blockers or insurmountable challenges I will revisit, but for now feel free to close this as it doesn't seem like it will be necessary and I am not actively working on it.
Thanks

@davidni OK cool, maybe we will come back to this one day :)


Code for Extended Simple Point Charge model used for water simulations?

Would it be possible to share the code used to perform the MD simulations for generation of the Water dataset used in this work? Even just an implementation of the force field would be great.

Hi, sorry for the delayed reply. Please find the LAMMPS code below.
water_lammps.zip


request failed after 5 retries with error:

About 15 days ago, without updating the version or making any changes, the error occurred unexpectedly. This error appears when executing any command to get list of themes, create new theme or theme watch.
request failed after 5 retries with error: <nil>

OS : Mac OS
Themekit version :  1.1.6 or any version

Please help, try multiple versions, but none works :/

BTW I also tried with different operating systems (Ubuntu 18.04) and I got the same error:
received a malformed response from shopify, this usually indicates a problem with your connection

I think the problem will prove to be with my store domain, because even if I put the wrong password it still shows the same error, it is important to mention that the error occurs in any method (get, download, deploy, watch).


Fix lower bound on python

Checklist

[x] Used a personal fork of the feedstock to propose changes
[x] Bumped the build number (if the version is unchanged)
[ ] Reset the build number to 0 (if the version changed)
[ ] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[x] Ensured the license file is being packaged.

fixes #3

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.

@conda-forge-admin please rerender


computed_rotation from OpenSFM in street view images?

"computed_rotation": [                 1.5361960317811,                 -0.57250186595465,                 0.54694338796389             ]
I saw in openstreetmap when we load the streetview images, theres is a field in response 'computed_rotation' which is coming from the OpenSFM. It seems it's reading the quaternion values present in camera Orientation and converting it into rotation vector, but it's not coming up correctly. Can someone explain how it's being calculated?

Hi @krishbhk ,
What do you mean not coming up correctly?
Can you provide an example with quaternion, the expected rotation and the computed rotation?
In case you do your own computations, make sure to apply the weight of the quaternion correctly. Sometimes it's at position 0, sometimes at position 3.

Hi @fabianschenk
So, got this data from a GoPro image
 'GRAV': '-0.4097, 0.9118, -0.0274',  'IORI': '1.0000, 0.0000, 0.0000, 0.0000',  'CORI': '0.3888, -0.0908, 0.8825, 0.2483',  'SHFX': '0.7278',  'SHFY': '1.9069',  'SHFZ': '2.0786',  'ANGX': '5.6211',  'ANGY': '7.2492',  'ANGZ': '19.3791',
I was trying to to calculate the computed_rotation value. I could be wrong, but it seems computed_rotation value is based on camera orientation (cori). I have trouble understanding how its converted to rotation_vector for the computed_rotation

Hi @krishbhk ,
I don't know how the value is computed but it can easily be in another reference system, so not at all matching the one from the image.
Don't have time to look into this any further.
Good luck,
Fabian

I was following mapillaryJS API, where the representation system was mentioned

have you made further discoveries ?
I am trying to do the reverse task: transformation this computed_rotation vector to quaternion (or Euler rotation vector). I asked the mapillary support team but they couldn't help.


Dano flow

Added flow types to kernel-manager (and jupyter-js-services-shim, but there wasn't anything to do)

@jdpigeon Welcome to nteract 🎉


feat: the Galois connection induced by an arbitrary relation

Define a Galois connection from an arbitrary relation and prove this induces inverse bijections between two sets of fixed points.

Thanks!
bors r+


Add in-game commands

Current in-game commands are doing ok, but could use some enhancement. Right now thinking /neo to keep them from getting confused with existing commands. I believe init.c will see them before any modded class since the modded class will be considered the base class to the custom mission.
Start with the following:

Unlock - unlock locks in-game

command would be '/neounlock <optional distance, default 2 meters ~6ft.>'

use neolocks lock manager list to find locked locks in radius and call unlock on them

may need to conditionally include that code in-case this mod is used stand-alone (git submodules?)
was originally considering the command to find all objects in a radius and detect locks, but apparently that function eats performance (probably iterates through all in-world objects checking distance).

God mode - avoid damage, etc.

command would be '/neogod <optional: on/off>'

toggles setting if not specified... leaving on/off in-case user wants to verify it is one or the other

features

no damage
no contaminated area effect
no AI targeting
no temperature effects
no hunger
no thirst
no sickness
no bleeds

Also add:
Teleport to dot

command would be '/neodot <optional distance, default 100m>'

Don't think server side code has access to player camera's, may need to use player object head bone like bohemia does for melee targeting.
https://github.com/BohemiaInteractive/DayZ-Script-Diff/blob/5579c3b9e879b55e947785d72184a1784081b565/scripts/4_world/entities/dayzplayerimplementmeleecombat.c#L334

Admin tier levels? What's max/min? Store in json config file? Normal config file may be difficult to organize.

Add function for follow-on mods to add commands. This will allow command addition from other mods to use command handling here, so new commands could go in without requiring changes to this mod.

Try and block chat messages to players if they start with '/'. Not sure if we can do this server side, but worth a try on the addChat method in PlayerBase


Bug on Cosmos 1.5 No such file or directory "/usr/local/aiflow/dags/dbt/dbt_venv/bin/python"

Two Astro customers reported that when updating to Cosmos 1.5, they started seeing the error:
No such file or directory "/usr/local/aiflow/dags/dbt/dbt_venv/bin/python"

As illustrated in the two screenshots below:

After lots of time trying to reproduce and understand this issue, I narrowed it down:

inside the customers dbt project folder, there was a symbolic link to a no longer existent file

Cosmos was raising an exception while trying to parse the file and create a version hash to represent the version of that dir.
I validated a possible solution with one of the customers earlier today, available in Cosmos 1.5.1a3 - and they confirmed it solved the problem. I'll make a quick PR with the solution and unittests today

We fixed the issue in #1097, and, in parallel to this, we would like to encourage users to have Python virtual environments outside the dbt project folder and outside of the Airflow DAGs folder.


Can't add comments on added lines

Hello. I am trying to add comments on added lines in a PR using the extension. I expect the white widget to show up in both sides, like this:

But it does not:

The blue circle shows the position of my cursor, which is not captured by my screenshots software.
It does work for the removed lines:

Possibly related to:

#1339
#1374

System info:

Extension version: v0.42.0
VSCode Version: 1.67.1
OS: Windows 10 (WSL 2)

Steps to Reproduce:

Show a PR in the extension
Hover over added (green lines)

The white widget doesn't show up.

@fepegar is your repository public and if so can you share a link to the pull request that you see this with?

I can repro when I use WSL.

Sure, here it is: https://github.com/microsoft/hi-ml/pull/337

Correction: It's not that I can repro with WSL, it's that I can repro when the file from the PR is not contained in folder I currently have open.

That reminds me of https://github.com/microsoft/vscode-pull-request-github/issues/1374#issuecomment-541179078

Thanks, @alexr00!


Dual Wield AC bons not parsed

Bonuses to AC from Dual Wielder feat is not being applied when character has a weapon marked as "dual wield" in the customization properties

Fixed


docs rs for deno_core is down

It's been down since I checked Friday evening, not sure how much longer before that. Not sure what change broke this or how it works, but I wanted to report this so people can get on top of it if nobody has noticed yet.
https://docs.rs/deno_core/0.221.0/deno_core/

It appears the build did just fine: https://docs.rs/crate/deno_core/0.221.0/builds/930245. Not sure what's the problem here. I'll be cutting another release tonight.

Looks like docs are working now: https://docs.rs/deno_core/latest/deno_core/.


Backups Not Encrypted

It would appear that the backups in the S3 bucket are stored in an unencrypted ZIP file. Is this intended?

Yes, that's how it works.
PG Back Web encrypts the data stored in the database.
Regarding storage, data is encrypted by default on the majority of S3 storage providers, so the responsibility is delegated to them.
https://docs.aws.amazon.com/AmazonS3/latest/userguide/UsingEncryption.html
https://www.backblaze.com/docs/cloud-storage-server-side-encryption
https://docs.wasabi.com/docs/how-secure-is-my-data-1
https://developers.cloudflare.com/r2/reference/data-security/
https://www.digitalocean.com/security/shared-responsibility-model-spaces

Maybe in the future when other features on the to-do list are done we can add a flag to add an encryption layer by PG Back Web


Spray w/ auto direction overwrites the direction of base texture

Description
Spraying with auto direction overwrites the direction on the base layer below. Spray should only change direction on the Spray pixels (where brush_alpha >. 1), I think. Due to blending, it might not work as we imagine. Might have to bump the threshold up to 0.3 or so.
https://discord.com/channels/691957978680786944/1065519581013229578/1275318563149840437

I think the condition for allowing spray to write angle/scale should require that the blend value that will be written to the control map, be 84 (1/3rd of 255)


Fixes #383

this may have drastic speed implications :/

Why? Because it *Sync function? I disagree.

This may not because

it happens once at start only (if I not mistake)
it is console app with single user, and block event loop is acceptable in this case

FuseBox use sync functions a lot see

@unlight the last file to write is the heaviest. It may take a long time to write it.

@nchanged
For me mainStr is:
#!/usr/bin/env node

require('gulp-cli')();

I suppose it is unexpected, but is another story :)
Default buffer is 64Kb, I do not think that's config will be so big.
Anyway it is better than:
fs.writeFile(configPath, mainStr, () => { });

Error swallowed
This happens before nukeCache (Window at this moment create file descriptor (file with empty content appears) and and prevent deleting directory)

Cache nuking happens only for the target folder. So it does not nuke everything.

And btw, https://github.com/fuse-box/fuse-box/issues/403

#407


Update README to point at RACCommunity

I can also change the bundle name if we think that's a good idea.

👍


IAppPartition may be not returned to the pool

Steps to reproduce the problem

make so engines borrow is failed here: https://github.com/voedger/voedger/blob/ba475f1ab0e8a360249ebb0223d5b18be5de0760/pkg/appparts/impl_app.go#L175

Result
*borrowedPartition taken from the pool here will not be returned to the pool
Expected behavior
take from pool -> error -> return to the pool before return

Why Release is necessary?


Remove or comment code for /peer-parameters in kernel

Its original purpose was to provide TURN servers from the catalysts.
The functionality never worked and should be removed.

I would like to raise PR for this

@RishiKumarRay please do it! https://github.com/decentraland/kernel

const peerParameters = await signedFetch(${commsServer}/peer-parameters, identity, { responseBodyType: 'json' })
Do I have to comment down this line only right ? @menduz

const peerParameters = await signedFetch(${commsServer}/peer-parameters, identity, { responseBodyType: 'json' }) Do I have to comment down this line only right ? @menduz

You need to remove the associated code to peerParameters and also that line, please make a PR and let's review the approach there.
    const peerParameters = await signedFetch(`${commsServer}/peer-parameters`, identity, { responseBodyType: 'json' })

    if (peerParameters.ok) {
      return { ...defaultPeerParameters, ...peerParameters.json }
    } else {
      throw new Error('Server response was not OK: ' + peerParameters.status)
    }

@leanmendoza please review https://github.com/decentraland/kernel/pull/188


av1.json: Firefox for Android can play AV1

Summary and context
The AV1 feature is compiled for Firefox for Android now.
In my testing the browser does attempt to play AV1 files now. That said, AV1 decoding is still off by default, behind a user preference flag.
Changes
This PR changes the support status of Firefox for Android 64 from "None" to "Off by default" and tags it with the note as follows:

"3":"Can be enabled in (all editions of) Firefox via the media.av1.enabled flag in about:config"

Source
Source (bugzilla.mozila.org):

Bug 1368843 - libaom doesn't build for android

Status: Resolved - Fixed in Firefox 64

Thanks!


API Endpoints Design

@simonv3 commented on Fri Dec 04 2015
Started a page for this: https://github.com/TheTorProject/ooni-api/wiki/API-Design
We can use this ticket to sketch out initial ideas.

@TylerJFisher commented on Sat Dec 05 2015
Updated: https://github.com/TheTorProject/ooni-api/wiki/API-Design

@hellais commented on Sun Dec 06 2015
Very good job on the API specification.
Just to be clear that I understand some things about it.
Your suggestion is to place all the test specific keys inside of the test_keys value (using the JSON feature of PostgreSQL) of the Metrics table and to generate some test specific views (that assess what Metrics present anomalies) based on some PgSQL procedures that get run in batch.
Is this accurate?

@TylerJFisher commented on Sun Dec 06 2015
Thanks, @hellais! That's correct - in order to implement this I'd likely opt to use a trigger within either PostgreSQL, or within middleware somewhere (e.g. ooni-pipeline using an ORM framework). The goal behind explicitly separating common and test-specific keys is to allow for test-specific tables, and views to be created.

@hellais Is this a ticket we can close?


Simple query causes crash (PG on RDS)

Sqlpad 2.7.1.
Postgres 10.4 on RDS.  Seems to crash less if a 'limit 1000' is added to the query.
Query:
select * from fake_complaint_rates;

Table structure:
CREATE TABLE "public"."fake_complaint_rates" (
  "complaint_rate" float8
);

Example data:
INSERT INTO "public"."fake_complaint_rates" VALUES (0.142220239778119);
INSERT INTO "public"."fake_complaint_rates" VALUES (0.032575241326039);
INSERT INTO "public"."fake_complaint_rates" VALUES (0.101116874108756);
INSERT INTO "public"."fake_complaint_rates" VALUES (0.0258135546366676);
INSERT INTO "public"."fake_complaint_rates" VALUES (0.125836686921209);

Debug output. Ran successfully with 'limit 1000' and 'limit 2000', crash on 'limit 3000'.
GET /api/schema-info/wlnOP0zanyyDfFaO 200 5.576 ms - 708
{"connectionName":"test","startTime":"2018-09-10T22:50:49.506Z","stopTime":"2018-09-10T22:50:50.224Z","queryRunTime":718,"rowCount":1000,"query":"select * from fake_complaint_rates\nlimit 1000"}
POST /api/query-result 200 781.426 ms - 37989
{"connectionName":"test","startTime":"2018-09-10T22:50:54.039Z","stopTime":"2018-09-10T22:50:54.738Z","queryRunTime":699,"rowCount":2000,"query":"select * from fake_complaint_rates\nlimit 2000"}
POST /api/query-result 200 774.988 ms - 75594
events.js:183
      throw er; // Unhandled 'error' event
      ^

Error: Connection terminated unexpectedly
    at Connection.con.once (C:\Users\me\AppData\Roaming\npm\node_modules\sqlpad\node_modules\pg\lib\client.js:179:21)
    at Object.onceWrapper (events.js:313:30)
    at emitNone (events.js:106:13)
    at Connection.emit (events.js:208:7)
    at Socket.<anonymous> (C:\Users\me\AppData\Roaming\npm\node_modules\sqlpad\node_modules\pg\lib\connection.js:76:10)
    at emitOne (events.js:116:13)
    at Socket.emit (events.js:211:7)
    at TCP._handle.close [as _onclose] (net.js:554:12)

Full data including table creation DDL is here (360kb, ~5k rows) -
http://delivio.net/fake_complaint_rates.sql

Hi @evan-burke  -- sorry for the late response to this. This might have something to do with the fact that SQLPad uses pg-cursor to enforce a query limit: https://github.com/brianc/node-pg-cursor
It could be something specific to RDS as well... Not sure how to go about debugging or testing this.
If you wanted you could try forking sqlpad and changing the postgres implementation here: https://github.com/rickbergfalk/sqlpad/blob/master/server/drivers/postgres/index.js
Actually... as a quick hack of a test, you could try the following query. 2 SELECTs being executed will try a query again without pg-cursor
select * from fake_complaint_rates;
select * from fake_complaint_rates;

Thanks, I'll give it a shot.

Closing due to inactivity. If this is still an issue we can reopen.

I just had this issue as well. Simple select fake_col from real_table; causes sqlpad to crash with pg connection.
Surely I can't be only one having such a basic issue for postgres? Make me think I'm missing something in config.

I should clarify PG is on RDS too. Maybe that's why others aren't experiencing this.

@kidtronnix Is anything noteworthy being logged at all?

I'm running into the same issue, data in PG running on a small EC2. Tried to run a "select * from table limit 400" causes server to fail; No logging showing system side at all.

Anyone willing to try things out now using v5?
Hoping this is fixed now. v5 no longer uses the postgres streaming/cursor implementation for running queries, which I suspect was the culprit.

Going to close this out assuming v5 fixes things. If not let me know and can reopen.


fix(sipi): Don't expect API v1 status code (DSP-1114)

resolves DSP-1114

Something very strange is happening in KnoraSipiIntegrationV2ITSpec.
If I do make stack-up and request http://localhost:1024/0803/incunabula_0000000002.jp2/full/499,630/0/default.jpg in a web browser, the image loads. Sipi's log in the Docker dashboard says:
Sipi: pre_flight called in sipi.init-knora.lua

Now I set a breakpoint at line 305 of KnoraSipiIntegrationV2ITSpec (on this branch), where it requests the same image.
https://github.com/dasch-swiss/dsp-api/pull/1763/files#diff-ccc8e1b4be7f09191193ca27b7b469d7b2e34f0ff97482fded3c02552ea27fc2R305
I run the test in IntelliJ's debugger. At the breakpoint, I can see that Sipi returned 404 Not Found. Why isn't the image there?
Now I set another breakpoint at line 438 and resume the test. At the breakpoint, the test has uploaded a file and is requesting the newly created file. I copy the IIIF URL (savedImage.iiifUrl), and I can load the image in a web browser. In the Docker dashboard, in the Sipi test container log, I can see that it didn't print pre_flight called in sipi.init-knora.lua.
So:

Images that are available when you do make stack-up are not available to the Sipi image running in the test.
The Sipi configuration used with make stack-up is not the same as the one used in the test.

I can't figure out where the Sipi configuration is coming from in the test. In org.knora.webapi.TestContainers, it says it's using sipi.knora-docker-config.lua, which loads sipi.init-knora.lua. So it should be the same, but it isn't.
Any ideas @subotic @SepidehAlassi ?

There is one small difference. In the tests, the Docker containers are started through the Testcontainers library, which does not allow mounting of volumes. When you run make stack-up then the sipi/images folder is mounted inside the Sipi container.

So where does the Sipi test container get its configuration files from? I suspect that the Lua pre_flight function isn’t being called at all inside the test container. How can I find out?

The config and scripts folders are backed into the Docker image. This is why we publish a DSP-API-specific Sipi Docker image.

you can see this here: https://github.com/dasch-swiss/dsp-api/blob/346873d17b994a04b40704f0ea595f55d81db459/docker/knora-sipi/BUILD.bazel#L11

The config and scripts folders are backed into the Docker image.

So I guess that means that if I need to change a Sipi script and test it, I need to publish a new Sipi Docker image. How do I do that?

hmm, something does not look right. The config folder should be added to the Sipi image, and also used when starting the container for the tests.

you recreate the docker containers locally with make docker-build. Then these images will be used when running the tests.

you recreate the docker containers locally with make docker-build. Then these images will be used when running the tests.

I did that, and it had no effect on the output that I described above.

ok, it should not be necessary to recreate the docker containers. I implemented it better: https://github.com/dasch-swiss/dsp-api/blob/346873d17b994a04b40704f0ea595f55d81db459/webapi/src/it/scala/org/knora/webapi/TestContainers.scala#L40

The sipi scripts are mounted into the image before the container is started. They are put on the class path.

https://github.com/dasch-swiss/dsp-api/blob/346873d17b994a04b40704f0ea595f55d81db459/webapi/BUILD.bazel#L140

or more correctly, they are part of the "it_library" which is a dependency for all it tests: https://github.com/dasch-swiss/dsp-api/blob/346873d17b994a04b40704f0ea595f55d81db459/webapi/BUILD.bazel#L199

The sipi scripts are mounted into the image before the container is started. They are put on the class path.

But in that case, Sipi should print the following when run inside the test container:
Sipi: pre_flight called in sipi.init-knora.lua

But it doesn't. That's what makes me think that the pre_flight script isn't being called inside the test container, and that this could be because it's not using the same configuration file.

@benjamingeer I found the error. I made a mistake when "mounting" the sipi.init-knora.lua file. I mounted sipi.knora-docker-config.lua twice. Once as itself and another time as sipi.init-knora.lua. I'm not sure why sipi didn't complain. Those files don't look anything alike.

@subotic No worries, I'm glad it turns out to be easy to fix. :)

@subotic But now I get this error in the Sipi log when I do bazel test //webapi/src/it/scala/org/knora/webapi:KnoraSipiIntegrationV2ITSpec:
BUILD_TIMESTAMP: 2020-11-06 10:22
BUILD_SCM_TAG: v3.0.0-rc.9
BUILD_SCM_REVISION: 453fd94601e0e732a65839e5fbde55bcc94bf72f
Sipi: BUILD_TIMESTAMP: 2020-11-06 10:22
Sipi: BUILD_SCM_TAG: v3.0.0-rc.9
Sipi: BUILD_SCM_REVISION: 453fd94601e0e732a65839e5fbde55bcc94bf72f
Sipi: Cache at "/sipi/cache" cachesize=104857600 nfiles=0 hysteresis=0.100000
Sipi: Closing cache...
Error at [/sipi/shttps/Server.h: 433]: initscript "/sipi/scripts/sipi.init-knora.lua" not found!

@subotic But now I get this error in the Sipi log when I do bazel test //webapi/src/it/scala/org/knora/webapi:KnoraSipiIntegrationV2ITSpec:
BUILD_TIMESTAMP: 2020-11-06 10:22
BUILD_SCM_TAG: v3.0.0-rc.9
BUILD_SCM_REVISION: 453fd94601e0e732a65839e5fbde55bcc94bf72f
Sipi: BUILD_TIMESTAMP: 2020-11-06 10:22
Sipi: BUILD_SCM_TAG: v3.0.0-rc.9
Sipi: BUILD_SCM_REVISION: 453fd94601e0e732a65839e5fbde55bcc94bf72f
Sipi: Cache at "/sipi/cache" cachesize=104857600 nfiles=0 hysteresis=0.100000
Sipi: Closing cache...
Error at [/sipi/shttps/Server.h: 433]: initscript "/sipi/scripts/sipi.init-knora.lua" not found!

If you use Bazel directly, then you first need to manually publish the new Docker image locally: make docker-build.

@subotic OK, now I get a different error: Sipi can't resolve the hostname api when it tries to ask for a file permission:
Sipi: Accepted connection from <IP_ADDRESS>
Sipi: pre_flight called in sipi.init-knora.lua
Sipi: webapi_hostname: api
Sipi: webapi_port: 3333
Sipi: pre_flight - knora_url: http://api:3333/admin/files/0001/1q0YiBup7zM-Bh4Xcy7eTDW.jp2
Sipi: pre_flight - knora_cookie_header: nil
Sipi: Server.http() failed: Error #1368: HTTP GET request to http://api:3333/admin/files/0001/1q0YiBup7zM-Bh4Xcy7eTDW.jp2 failed: Could not resolve host: api
Sipi: GET /0001/1q0YiBup7zM-Bh4Xcy7eTDW.jp2/full/499,630/0/default.jpg failed (Unauthorized): Unauthorized access

@subotic OK, now I get a different error: Sipi can't resolve the hostname api when it tries to ask for a file permission:
Sipi: Accepted connection from <IP_ADDRESS>
Sipi: pre_flight called in sipi.init-knora.lua
Sipi: webapi_hostname: api
Sipi: webapi_port: 3333
Sipi: pre_flight - knora_url: http://api:3333/admin/files/0001/1q0YiBup7zM-Bh4Xcy7eTDW.jp2
Sipi: pre_flight - knora_cookie_header: nil
Sipi: Server.http() failed: Error #1368:
HTTP GET request to http://api:3333/admin/files/0001/1q0YiBup7zM-Bh4Xcy7eTDW.jp2 failed:
Could not resolve host: api
Sipi: GET /0001/1q0YiBup7zM-Bh4Xcy7eTDW.jp2/full/499,630/0/default.jpg failed (Unauthorized): Unauthorized access

Thanks, will try to fix that.

Sipi config is now fixed, but another error has surfaced:
Sipi: webapi_hostname: <IP_ADDRESS>
Sipi: webapi_port: 3333
Sipi: pre_flight - knora_url: http://<IP_ADDRESS>:3333/admin/files/0803/incunabula_0000000002.jp2
Sipi: pre_flight - knora_cookie_header: nil
Sipi: Knora returned HTTP status code 404
Sipi: {"error":"org.knora.webapi.exceptions.NotFoundException: No file value was found for filename incunabula_0000000002.jp2"}
Sipi: GET /0803/incunabula_0000000002.jp2/full/499,630/0/default.jpg failed (Unauthorized): Unauthorized access

This seems to work. Locally

Sipi config is now fixed, but another error has surfaced:
Sipi: webapi_hostname: <IP_ADDRESS>
Sipi: webapi_port: 3333
Sipi: pre_flight - knora_url: http://<IP_ADDRESS>:3333/admin/files/0803/incunabula_0000000002.jp2
Sipi: pre_flight - knora_cookie_header: nil
Sipi: Knora returned HTTP status code 404
Sipi: {"error":"org.knora.webapi.exceptions.NotFoundException: No file value was found for filename incunabula_0000000002.jp2"}
Sipi: GET /0803/incunabula_0000000002.jp2/full/499,630/0/default.jpg failed (Unauthorized): Unauthorized access

This was maybe a temporary problem I had. The tests are now running through.

@subotic Can you pull the branch and try again? Now when I run the test, I get this error in Sipi's log:
Sipi: Server.http() failed: Error #1368: HTTP GET request to http://<IP_ADDRESS>:3333/admin/files/0001/KUZdTag1Vu6-BchuXtxRejX.jp2 failed: Failed to connect to <IP_ADDRESS> port 3333: Connection refused

@subotic Can you pull the branch and try again? Now when I run the test, I get this error in Sipi's log:
Sipi: Server.http() failed: Error #1368:
HTTP GET request to http://<IP_ADDRESS>:3333/admin/files/0001/KUZdTag1Vu6-BchuXtxRejX.jp2 failed:
Failed to connect to <IP_ADDRESS> port 3333: Connection refused

Locally, all tests are passing after the pull.

did you run make docker-build?

I've just done:
make docker-build
bazel test //webapi/src/it/scala/org/knora/webapi:KnoraSipiIntegrationV2ITSpec

I stopped the Ryuk container as soon as it appeared. The test failed with:
- should change a still image file value *** FAILED *** (1 second, 159 milliseconds)
  org.knora.webapi.exceptions.AssertionException: Got HTTP 401
 REQUEST: HttpRequest(HttpMethod(GET),http://localhost:33977/0001/5eHhWw5oHbM-GTicJJjHZjb.jp2/full/499,630/0/default.jpg,List(),HttpEntity.Strict(none/none,0 bytes total),HttpProtocol(HTTP/1.1)),
 RESPONSE: Unauthorized: Unauthorized access

The log in the Sipi test container says:
Sipi: Server.http() failed: Error #1368:
HTTP GET request to http://<IP_ADDRESS>:3333/admin/files/0001/GDvu6VsR7d0-CQjQNKQV4oh.jp2 failed:
Failed to connect to <IP_ADDRESS> port 3333: Connection refused
Sipi: GET /0001/GDvu6VsR7d0-CQjQNKQV4oh.jp2/full/499,630/0/default.jpg failed (Unauthorized): Unauthorized access

Should it be using <IP_ADDRESS> or <IP_ADDRESS>?

The very strange thing is that if I comment out the test reject an image file with the wrong file extension, the test change a still image file value passes.

Should it be using <IP_ADDRESS> or <IP_ADDRESS>?

It should actually be your machine's IP address.

It should actually be your machine's IP address.

Is there something I need to configure to fix that? Sipi says it's using <IP_ADDRESS>.

@subotic
% scala
Welcome to Scala 2.13.3 (OpenJDK 64-Bit Server VM, Java 14.0.1).
Type in expressions for evaluation. Or try :help.

scala> import java.net._
import java.net._

scala> val localhost: InetAddress = InetAddress.getLocalHost
val localhost: java.net.InetAddress = euler.local/<IP_ADDRESS>

scala> val localIpAddress: String = localhost.getHostAddress
val localIpAddress: String = <IP_ADDRESS>

scala
Welcome to Scala 2.13.4 (OpenJDK 64-Bit Server VM, Java <IP_ADDRESS>).
Type in expressions for evaluation. Or try :help.

scala> import java.net._
import java.net._

scala> val localhost: InetAddress = InetAddress.getLocalHost
val localhost: java.net.InetAddress = iml-stargazer.localdomain/<IP_ADDRESS>

scala> val localIpAddress: String = localhost.getHostAddress
val localIpAddress: String = <IP_ADDRESS>

This is what I get.
Also, I see that you have Java 14 installed.

I've pushed the fix for the testcontainer deprecation

Also, I see that you have Java 14 installed.

Maybe Brew installed it with Scala. The only JVM I've installed is AdoptOpenJDK 11:
% java -version
openjdk version "11.0.5" 2019-10-15
OpenJDK Runtime Environment AdoptOpenJDK (build 11.0.5+10)
OpenJDK 64-Bit Server VM AdoptOpenJDK (build 11.0.5+10, mixed mode)

public class Test {
    public static void main(String[] args) throws Exception {
        System.out.println(java.net.InetAddress.getLocalHost().getHostAddress());
    }
}

% java -version
openjdk version "11.0.5" 2019-10-15
OpenJDK Runtime Environment AdoptOpenJDK (build 11.0.5+10)
OpenJDK 64-Bit Server VM AdoptOpenJDK (build 11.0.5+10, mixed mode)
% javac Test.java
% java -cp . Test
<IP_ADDRESS>

Maybe this is the solution: https://stackoverflow.com/a/16444863

OK I found the problem! It actually was a static initialisation problem, but not the kind I expected. The problem is that there are several different TestContainers objects:

webapi/src/it/scala/org/knora/webapi/TestContainers.scala
webapi/src/test/scala/org/knora/webapi/TestContainers.scala
salsah1/src/test/scala/org/knora/salsah/TestContainers.scala

Which one is actually used seems to be random.
Could you clean this up so there is only one TestContainers object? To work on my machine, it needs to contain this:
import java.net.NetworkInterface
import scala.collection.JavaConverters._

val localIpAddress: String = NetworkInterface.getNetworkInterfaces
    .asScala.toSeq.filter(!_.isLoopback)
    .flatMap(_.getInetAddresses.asScala.toSeq.filter(_.getAddress.length == 4).map(_.toString))
    .headOption.getOrElse(throw TestConfigurationException(s"No suitable network interface found"))

OK I found the problem! It actually was a static initialisation problem, but not the kind I expected. The problem is that there are several different TestContainers objects:

webapi/src/it/scala/org/knora/webapi/TestContainers.scala
webapi/src/test/scala/org/knora/webapi/TestContainers.scala
salsah1/src/test/scala/org/knora/salsah/TestContainers.scala

Which one is actually used seems to be random.
Could you clean this up so there is only one TestContainers object? To work on my machine, it needs to contain this:
import java.net.NetworkInterface
import scala.collection.JavaConverters._

val localIpAddress: String = NetworkInterface.getNetworkInterfaces
    .asScala.toSeq.filter(!_.isLoopback)
    .flatMap(_.getInetAddresses.asScala.toSeq.filter(_.getAddress.length == 4).map(_.toString))
    .headOption.getOrElse(throw TestConfigurationException(s"No suitable network interface found"))

Strange. There should be only one of course. I will try to find out what is happening.

You added the ones in test and it in PR #1653, and you added the one in salsah1 in PR #1480.

Yes, it is three times in the codebase, but these hierarchies are completely separate. A test should only ever see one of those implementations. If it doesn't, then it is a configuration error in some Bazel build file.

I think that having three classes with the same class name, in the same package, in different parts of the codebase, is asking for trouble with the class loader, which is what has happened here (as I can see because, if I change all three, it works). And it creates a maintenance problem, because if the three classes are redundant and are supposed to be identical, it is too easy to change one and forget to change the others, which also just happened here. Each source file should exist only in one place.

And I would like to emphasise that I have now wasted several days debugging this problem, which would have been avoided if good basic software development practices had been followed, i.e. no redundant code.

I agree. We have the same problem with other test classes, too. Most base classes under it and test need to be updated simultaneously in both places. One way would be to merge the test and it hierarchies. Salsah is completely separate, so I'm not really sure, why the classloader sees all three implementations. How did you find it out?

And I would like to emphasise that I have now wasted several days debugging this problem, which would have been avoided if good basic software development practices had been followed, i.e. no redundant code.

The problems you are facing and I am trying to help you with is probably local to your system, as the test pass on my machine locally and on Github-CI. As for the redundant code, with SBT it wasn't possible to do it otherwise. If you want to clean up, please go ahead and merge the different test code hierarchies and eliminate the redundant code.

The problem is in the build file, where the test is defined. There, both the it and test libraries are added as dependencies. This was done in the PR #1724. Then of course, both implementations will be on the classpath.

One way would be to merge the test and it hierarchies.

That's what I was thinking, too.

The problems you are facing and I am trying to help you with is probably local to your system, as the test pass on my machine locally and on Github-CI.

This problem started because the admin API depended on API v1. That dependency shouldn't have been there.
Then there was the misconfiguration of the test containers, which apparently caused Sipi to run with the wrong configuration. As a result, tests passed when they should have failed.
As for the problem on my machine: after looking at https://stackoverflow.com/a/16444863, I think it's just luck that it worked on your system and on GitHub-CI. It seems that you can't assume that InetAddress.getLocalHost.getHostAddress won't return the loopback interface. If you don't want the loopback interface, it seems that you have to get all the available interfaces, and choose the one you want.

As for the redundant code, with SBT it wasn't possible to do it otherwise. If you want to clean up, please go ahead and merge the different test code hierarchies and eliminate the redundant code.

The different test code hierarchies could have been merged under SBT, too. So it was possible not to copy and paste redundant code with SBT.

The problem is in the build file, where the test is defined. There, both the it and test libraries are added as dependencies.

This is an example of how difficult it is to manage dependencies with Bazel. I think this is a good reason to get rid of Bazel and go back to SBT.


Added vault address to PRNG

@PierrickGT it was actually addressed before the C4 audit started:
L426: claimedPrizes[msg.sender][_winner][lastClosedDrawId][_tier][_prizeIndex] = true

This PR completes the first point: to include the vault address in the PRNG

(msg.sender == vault)


NativeImageLoader wrong RGB loading of Mat

Issue Description
Reporting here an issue discussed also at https://github.com/deeplearning4j/DataVec/issues/587.
The output of a Keras model (ComputationGraph) changes if we construct the INDArray from NativeImageLoader vs manually.
Version Information

Deeplearning4j 1.0.0-beta7
Windows 10
No CUDA

Here is a minimal reproducible example:
import org.datavec.image.loader.NativeImageLoader;
import org.deeplearning4j.nn.graph.ComputationGraph;
import org.deeplearning4j.nn.modelimport.keras.KerasModelImport;
import org.deeplearning4j.nn.modelimport.keras.exceptions.InvalidKerasConfigurationException;
import org.deeplearning4j.nn.modelimport.keras.exceptions.UnsupportedKerasConfigurationException;
import org.nd4j.linalg.api.ndarray.INDArray;
import org.nd4j.linalg.factory.Nd4j;
import org.opencv.core.Core;
import org.opencv.core.Mat;
import org.opencv.core.Size;
import org.opencv.imgcodecs.Imgcodecs;
import org.opencv.imgproc.Imgproc;

import java.io.IOException;

public class NILTest {
    public static void main(String[] args) throws InvalidKerasConfigurationException, IOException, UnsupportedKerasConfigurationException {
        System.load("opencv_world420d.dll");
        System.load("opencv_java420.dll");
        ComputationGraph model = KerasModelImport.
                importKerasModelAndWeights("D:\\model.hdf5",false);

        Mat img = Imgcodecs.imread("image.png", Imgcodecs.IMREAD_COLOR);
        Mat resized = new Mat();
        Imgproc.resize(img, resized, new Size(24, 16));
        Imgproc.cvtColor(resized, resized, Imgproc.COLOR_BGR2RGB);

        NativeImageLoader loader = new NativeImageLoader(16, 24, 3);

        INDArray inputNet = loader.asMatrix(resized);
        INDArray inputNet2 = Nd4j.zeros(1, 16, 24, 3);

        for (int y = 0; y < resized.height(); y++) {
            for (int x = 0; x < resized.width(); x++) {
                for (int ch = 0; ch < 3; ch++) {
                    inputNet2.putScalar(new int[]{0, y, x, ch}, resized.get(y, x)[ch]);
                }
            }
        }

        inputNet.divi(255.);

        inputNet2.divi(255.);

        inputNet = inputNet.permute(0, 2, 3, 1);

        INDArray[] outputs = model.output(false, inputNet);

        INDArray[] outputs2 = model.output(false, inputNet2);

        System.out.println(outputs[0]);
        System.out.println(outputs2[0]);
    }
}

When running the above code, outputs is totally different from outputs2 and the correct one seems to be the second.
The model is a very simple CNN with no particular layer inside and "image.png" can be any color image.
I think this issue would happen also with a standard model like MobileNet v2 but I wasn't able to import it in DL4j.

@marcoromelli we have workarounds for this in the record reader: https://github.com/eclipse/deeplearning4j/blob/master/datavec/datavec-data/datavec-data-image/src/main/java/org/datavec/image/recordreader/BaseImageRecordReader.java#L352
Generally, different frameworks will have different input image formats they expect.
The NativeImageLoader should be used with the expectation that you know which format is expected.
Also, can you attach which model you were trying to import? It'd be good to know about that.

I don't understand. The line you linked is to convert from NCHW to NHWC, which I already have in my code.
My problem is different: inputNet and inputNet2 look identical, but outputs and outputs2 are totally different.
The model I used can be downloaded from here.

Ah, sorry. I looked at the other github issue. That helped a bit. cc @saudet  here as well just in case. Let me spend some time in the debugger here.

@marcoromelli also you mentioned not being able to import mobilenet. Could you clarify what happened there maybe in a separate issue? We have multiple forms of input including keras and tensorflow.

@marcoromelli  just added
  Nd4j.getExecutioner().setProfilingConfig(ProfilerConfig.
                builder().checkForNAN(true)
                .checkForINF(true)
                .build());

to your script and I'm getting a JVM crash now. I'm guessing this is an addressing issue and it's bumping in to invalid memory or something. Let me keep looking in to this.

@marcoromelli  I looked in to this and noticed a dup() call will give consistent results. While it's not ideal, it's at least a workaround for now. It has something to do with the ndarray references and the way it's created.
I changed it to the below:
package org.example;

import org.datavec.image.loader.NativeImageLoader;
import org.deeplearning4j.nn.graph.ComputationGraph;
import org.deeplearning4j.nn.modelimport.keras.KerasModelImport;
import org.nd4j.graph.ProfilingMode;
import org.nd4j.linalg.api.ndarray.INDArray;
import org.nd4j.linalg.factory.Nd4j;

import java.io.File;

import org.nd4j.linalg.profiler.ProfilerConfig;
import org.opencv.core.Core;
import org.opencv.core.Mat;
import org.opencv.core.Size;
import org.opencv.imgcodecs.Imgcodecs;
import org.opencv.imgproc.Imgproc;

public class NILTest {
    public static void main(String[] args) throws Exception {
        Nd4j.getEnvironment().setDebug(true);
        Nd4j.getEnvironment().setProfiling(true);
        Nd4j.getExecutioner().setProfilingConfig(ProfilerConfig.
                builder().checkForNAN(true)
                .checkForINF(true)
                .build());
        System.load("C:\\Users\\agibs\\Documents\\GitHub\\dl4j-test-import\\opencv_world420d.dll");
        System.load("C:\\Users\\agibs\\Documents\\GitHub\\dl4j-test-import\\opencv_java420.dll");
        ComputationGraph model = KerasModelImport.
                importKerasModelAndWeights("model-cnn-test.hdf5",false);
        System.out.println(model.summary());

        Mat img = Imgcodecs.imread("download.jpg");
        Mat resized = new Mat();
        Imgproc.resize(img, resized, new Size(24, 16));
        Imgproc.cvtColor(resized, resized, Imgproc.COLOR_BGR2RGB);

        NativeImageLoader loader = new NativeImageLoader(16, 24, 3);

        INDArray inputNet = loader.asMatrix(resized).dup();
        INDArray inputNet2 = Nd4j.zeros(1, 16, 24, 3);

        for (int y = 0; y < resized.height(); y++) {
            for (int x = 0; x < resized.width(); x++) {
                for (int ch = 0; ch < 3; ch++) {
                    inputNet2.putScalar(new int[]{0, y, x, ch}, resized.get(y, x)[ch]);
                }
            }
        }

        inputNet.divi(255.);
        inputNet2.divi(255.);

        inputNet = inputNet.permute(0, 2, 3, 1);

        if(!inputNet.equals(inputNet2)) {
            throw new IllegalStateException("Inputs did not equal!");
        }
        for(int i = 0;  i < 4; i++) {
            INDArray outputs = model.outputSingle(false, inputNet.dup());
            INDArray outputs2 = model.outputSingle(false, inputNet2.dup());

            System.out.println("Output 1 " + outputs);
            System.out.println("Output 2 " + outputs2);
        }
    }
}

This gives consistent results.
If I get rid of the dup() calls when calling inputSingle I notice it is fine.
This is reference related. I'm guessing it has something to do with the way we fill in the NDArray:
https://github.com/eclipse/deeplearning4j/blob/master/datavec/datavec-data/datavec-data-image/src/main/java/org/datavec/image/loader/NativeImageLoader.java#L427
I'll keep poking at this, but that at least gets us to a starting point for debugging this. Thanks for reporting!


Color code buttons depending on actions in seller view

I made changes to button colors in seller view, so our page looks even more colorful, less monotonous. Please review

Green - View Interest
Yellow - Edit
Red - Delete
Yellow - Remove Interest


Start of advanced app development section

@julians1966 should I merge, or wait for @LuiseFreese 's review?

I don't need to review. Please merge so I can pull that again


Atom & Atom Beta both crashing with Mac OS 10.13

Description
Its freezes for 3-4 seconds and then become active again. if i do not stop at that point it crashes.
Tells me to report the problem here
Steps to Reproduce

Update form 10.12.6 --> 10.13
Atom && Atom Beta both pre-installed in 10.12.6

Expected behavior:
Not to crash
Actual behavior:
Crashes so frequently that i cannot write any code
Reproduces how often: 20 times in an hour
Versions
Atom - 1.21.0
Atom Beta - 1.21.0 beta -2

1+ in mac 10.12.6
Can't even open atom for more than 5 minutes without it crashing.
No action taken outside of atom .  inside the terminal to open a .json file

@michaelspeed - can you confirm if you went through the Prerequisites section of the issue template?  E.g. do you see the same behavior in safe mode (atom --safe)?
@PaulAndreRada - the latest stable release of Atom is 1.21.0, can you upgrade and if you're still seeing a crash, please open another issue with the issue template filled out and we'll take a look with you.

atom --safe also crashes(not so frequently). what i did was i removed atom and atom beta, and also its supporting folders and everything.Then installed it again. Currently in one hour its not crashing

@rsese Updated to 1.21.0, All working perfect so far. Thanks man

Good to hear that crashing has stopped for you @michaelspeed - we'll close this issue for now but if you see crashes again, let me know and we'll re-open.
Glad to hear things are looking good for you too @PaulAndreRada :+1:


[openimageio] Bump to 2.1.10

This pull requests bump OpenImageIO port version to 2.1.10

What does your PR fix? N/A

Which triplets are supported/not supported? Have you updated the CI baseline?
I have tested a custom triplet x64-windows-static-release, but I am waiting on CI to iterate and validate others
I haven't touched the CI baseline

Does your PR follow the maintainer guide?
Yes

Good morning, version 2.1.11 released. I was having issues using this latest version with the port. It seems the way the patches have to be applied have changed from 2.1.10 to 2.1.11.

Hi @huargovfx , I consider this PR on hold for now
The idea is to rebase my changes - and ideally upgrade to latest OpenImageIO - once #8543 is merged
If possible I would suggest to use this other PR is you don't need the latest OpenImageIO, otherwise please be a bit more patient :)

@theblackunknown My PR #8543 has been merged, you can continue your work now : )

Thanks ! Will do once I rebase my work !

/azp run

Need to test features.

Hi @theblackunknown
Sorry for the late reply.
All features except for field3d(only supports Windows) passed with the following triplets:

x86-windows
x64-windows
x64-windows-static

All features except for(libraw, pybind11) passed with x64-linux triplet.
Could you please help confirm and check the results about libraw and pybind11 on Linux?

Hey @NancyLi1013 , I have just tested those features on a Linux OS
I have not encountered any trouble for libraw, however I did for pybind11.
In my opinion, the main issue is how pybind11 port manage its dependencies :

On Windows, it relies on python3 from vcpkg port : this provide a consisten, controlled and reproducable behavior
On other OSes, it relies on whatever system python is present : this is very inconsistent, and error prone when trying to have reproducable builds on different installations

Now, even if I found the correct package to install to make it work on Linux, I had to further tweak the portfile to make sure openimageio will use the python shared library and not the static one because otherwise the port will fail (because the former is compiled with -fPIC which is what openimageio python bindings requires)
So my last commit is hacky but make it pass for me,
I would suggest to fix the following ports :

python3 by building shared library (currently tackled in #9979)
pybind11 by always relying on the upgraded python3 to get consistent and reproducable builds over any installations

What do you think ?

@theblackunknown
I think we must guarantee to use the built-in tools (git, python ...) of vcpkg to ensure that the actual effect in each triplet is consistent. And update these tools to make changes in all triplets at the same time.

@JackBoosY
Just to be sure to understand, in the current case what are you suggesting to fix the issue ?
Make pybind11 relies on vcpkg port ? vcpkg downloaded python ? or something else ?

@theblackunknown

If OIIO depends on pybind, use vcpkg port pybind11 (Add Build-Depends: pybind11 to CONTROL)
If OIIO use python3, use vcpkg installed tools python3(Use vcpkg_find_acquire_program)

Ok, we agree on this and this is what the current port is currently doing
However the pybind11 port needs an upgrade because the wrong python libraries gets picked up in its current state :

pybind11 relies on system python instead of vcpkg python
we need to have python with shared library

I am currently tackling this issue in #9979

/azp run

@NancyLi1013 @JackBoosY same as my other PR, this one is also on hold as it requires the ones that are now closed
I am closing it for now until I go back to it


cannot allocate memory

When I run the att2in2 model with cross entropy loss, I got 'cannot allocate memory' error. I don't know why. The detail is:
File "/usr/lib/python2.7/multiprocessing/process.py", line 130, in start
self._popen = Popen(self)
File "/usr/lib/python2.7/multiprocessing/forking.py", line 121, in init
self.pid = os.fork()
OSError: [Errno 12] Cannot allocate memory
because of my fault, I cannot get all the details. But I known it comes from the detaloader.py which loads the data with multiprocess. It seems that the error didn't happen regularly. Sometimes the whole training  can be over without errors, sometimes not. Have you ever met this error? @ruotianluo

It happens to me sometime too, but varies from machine to machine. https://github.com/pytorch/pytorch/issues/1355
I guess it's related to the setting of different machine.

Thanks a lot. Zan yi ge!


Singular/ plural mismatch in phrase

In the phrase:
“The resource owner can grant or deny your app (the client) access to the resources they own”
who is “they”?
(it seems it should be “he” or “she”, in singular, or “The resource owners”, in plural)

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

ID: b9abce49-5b87-65cf-8ef5-d3421619d535
Version Independent ID: 25c74f8d-9265-cb29-a80c-eaff4cfc6ec3
Content: OAuth 2.0 and OpenID Connect protocols on the Microsoft identity platform - Microsoft identity platform
Content Source: docs/identity-platform/v2-protocols.md
Service: identity-platform
GitHub Login: @henrymbuguakiarie
Microsoft Alias: henrymbugua

@carlos-quintero
Thanks for your feedback! We will investigate and update as appropriate.


KAFKA-16470; kafka-dump-log --offsets-decoder should support new records

When the consumer group protocol is used in a cluster, it is, at the moment, impossible to see all records stored in the __consumer_offsets topic with kafka-dump-log --offsets-decoder. It does not know how to handle all the new records.
This patch refactors the OffsetsMessageParser used internally by kafka-dump-log to use the RecordSerde used by the new group coordinator. It ensures that the tool is always in sync with the coordinator implementation. The patch also changes the format to using the toString'ed representations of the records instead of having custom logic to dump them. It ensures that all the information is always dumped. The downside of the latest is that inner byte arrays (e.g. assignment in the classic protocol) are no longer deserialized. Personally, I feel like that it is acceptable and it is actually better to stay as close as possible to the actual records in this tool. It also avoids issues like https://issues.apache.org/jira/browse/KAFKA-15603.
Committer Checklist (excluded from commit message)

[ ] Verify design and implementation
[ ] Verify test coverage and CI build status
[ ] Verify documentation (including upgrade notes)

Also, it seems the following classes are useless.

Yeah, I noticed those classes too. I am actually not sure what we kept them. It may be for backward compatibility. I need to dig a little more on it. It is anyway not related to this patch so we can treat it separately.

Not sure whether this will hurt the users who are using this tool to observe the assignments. Maybe we can parse subscription and assignment according to official kafka consumer, and convert the parse exception (https://issues.apache.org/jira/browse/KAFKA-15603) to log message instead of throwing exception. WDYT?

I don't think users rely on this tool to read the assignments. At least, I never did. The annoying part is that we cannot reuse the auto-generated structures if we want to preserve this. We need to introduce a new structure which is basically the autogenerated one + new fields for holding the deserialised arrays. I am not fan of this because they will for sure diverge in the future. I will think a little more about it.

@chia7712 I updated the PR with a different approach. I now print the records using the json converters. This allow me to post process the group metadata record to deserialize the subscription and the assignment as it was before. It looks as follow:
baseOffset: 1 lastOffset: 1 count: 1 baseSequence: 0 lastSequence: 0 producerId: -1 producerEpoch: -1 partitionLeaderEpoch: 0 isTransactional: false isControl: false deleteHorizonMs: OptionalLong.empty position: 99 CreateTime:<PHONE_NUMBER>539 size: 346 magic: 2 compresscodec: none crc:<PHONE_NUMBER> isvalid: true
| offset: 1 CreateTime:<PHONE_NUMBER>539 keySize: 7 valueSize: 269 sequence: 0 headerKeys: [] key: {"type":"2","data":{"group":"foo"}} payload: {"version":"3","data":{"protocolType":"consumer","generation":1,"protocol":"range","leader":"console-consumer-8fe09633-19c4-4894-ad9d-e26637e2e71f","currentStateTimestamp":1714661210525,"members":[{"memberId":"console-consumer-8fe09633-19c4-4894-ad9d-e26637e2e71f","groupInstanceId":null,"clientId":"console-consumer","clientHost":"/<IP_ADDRESS>","rebalanceTimeout":300000,"sessionTimeout":45000,"subscription":{"topics":["quickstart-events"],"userData":null,"ownedPartitions":[],"generationId":-1,"rackId":null},"assignment":{"assignedPartitions":[{"topic":"quickstart-events","partitions":[0]}],"userData":null}}]}}

Please take a look and let me know what you think.

There is a build having weird output, so I re-trigger QA

Yeah, I noticed those classes too. I am actually not sure what we kept them. It may be for backward compatibility. I need to dig a little more on it. It is anyway not related to this patch so we can treat it separately.

I open https://issues.apache.org/jira/browse/KAFKA-16666 to trace it. Please feel free to take over it if you have free time :)

@chia7712 Those two classes are meant to be used with the console consumer. This is why they are not referenced anywhere. I wonder if we should provide an equivalent one in java.

Those two classes are meant to be used with the console consumer. This is why they are not referenced anywhere.

thanks for the reminder !

I wonder if we should provide an equivalent one in java.

agree!
The conversion from ByteBuffer to ApiMessage can be rewrite by java. Also, ConsoleConsumer can internally replace the string kafka.coordinator.group.GroupMetadataManager\$OffsetsMessageFormatter by the name of new implementation to keep the compatibility of command line. WDYT?


Provision to switch between Front and Back camera

I tried the Scan barcode to input-field example, which opens Front Camera on Android Phone.  How can I change this functionality to make use of the Back Camera of Android Phone?

https://github.com/serratus/quaggaJS/issues?q=deviceId


WFLY-4518 Allow Undertow access log to write to the server log

Linux with security manager Build 1226 is now running using a merge of 30df64a07a8c47cb0ac99f081049b3cdaf7fc9d5

Linux Build 6261 is now running using a merge of 30df64a07a8c47cb0ac99f081049b3cdaf7fc9d5

Windows Build 1363 is now running using a merge of 30df64a07a8c47cb0ac99f081049b3cdaf7fc9d5

Linux with security manager Build 1226 outcome was SUCCESS using a merge of 30df64a07a8c47cb0ac99f081049b3cdaf7fc9d5
Summary: Tests passed: 823, ignored: 356 Build time: 0:04:57

Windows Build 1365 is now running using a merge of 30df64a07a8c47cb0ac99f081049b3cdaf7fc9d5

Linux Build 6261 outcome was SUCCESS using a merge of 30df64a07a8c47cb0ac99f081049b3cdaf7fc9d5
Summary: Tests passed: 2847, ignored: 412 Build time: 0:51:06

Windows Build 1365 outcome was SUCCESS using a merge of 30df64a07a8c47cb0ac99f081049b3cdaf7fc9d5
Summary: Tests passed: 2847, ignored: 412 Build time: 0:51:52

:+1:


Error during installation on ubuntu lts

I see this error after each sudo apt-get update.
GPG error: https://packages.microsoft.com/repos/vscode stable InRelease: The following signatures couldn't be verified because the public key is not available: NO_PUBKEY EB3E94ADBE1229CF

Fix this issue:
curl https://packages.microsoft.com/keys/microsoft.asc | gpg --dearmor > microsoft.gpg
sudo mv microsoft.gpg /etc/apt/trusted.gpg.d/microsoft.gpg
sudo sh -c 'echo "deb [arch=amd64] https://packages.microsoft.com/repos/vscode stable main" > /etc/apt/sources.list.d/vscode.list'
Thanks.

@gtzinos if i enter:
curl https://packages.microsoft.com/keys/microsoft.asc | gpg --dearmor > microsoft.gpg
I get the error: -bash: microsoft.gpg: Permission denied
(23) Failed writing body
do you know why this happens?

Can you try to use it with sudo ?

@ms-141630 it happens if you trying to issue command from directory where your user does not have write permissions. Sudo won't help, so it's better to browse on your f.e. home directory and issue curl/GPG from there.


Log File Empty

Team ran code where power board ID instead of motor board ID. This caused what seems to be logging crash with a totally empty log file created.
The code which caused this is listed below where FRONT_WHEELS is the power board ID. Expect to raise key issue. Instead no logs created. Lack of logs persists even when known good code is run, USB reformat fixes this.
from sr.robot3 import *
import time
import math

R = Robot()

BACK_WHEELS = "SR0WAF"
FRONT_WHEELS = "sr0XT25"

def drive_wheel(power: float, fb: str, lr: str):
    board = None
    side = None

    if fb in ["front", "f"]:
        board = FRONT_WHEELS
    elif fb in ["back", "b"]:
        board = BACK_WHEELS
    else:
        print("INVALID WHEEL SIDE '{}'".format(fb))
        return

    if lr in ["left", "l"]:
        side = 0
    elif lr in ["right", "r"]:
        side = 1
    else:
        print("INVALID WHEEL SIDE '{}'".format(lr))
        return

    try:
        R.motor_boards[board].motors[side].power = power
    except Exception as error:
        print("DRIVE WHEEL ERROR:", error)

def drive(power: float):
    drive_wheel(power, "front", "left")
    drive_wheel(power, "front", "right")
    drive_wheel(power, "back", "left")
    drive_wheel(power, "back", "right")

while True:
    drive(math.sin(time.time() / 2))
    time.sleep(0.1)

We have identified the cause of this bug as not having the sync mount option in the Udisks configuration.
Therefore this bug is not technically an issue with astoria. However, we should document this behaviour in the requirements.


@tacky need a logo

Hi, I can design a free logo/icon  for this project. If it's something you're interested in, please, let me know!

Hi, I can design a free logo/icon for this project. If it's something you're interested in, please, let me know!

Thanks! I need your help. It would be better if there were some sense of technology.

Hi @geoffgu , can you give me more informations about this project?

Hi @zuurw , this project is a front-end framework based on react. In the future, it will contain page router, state management framework, i18n, api call library, dev tool and so on.

@zuurw emmm... maybe a front-end development common solution, helping developer create a maintainable app in less time.

Hi, I would like to know what do you want for this logo, can you give me something to get inspired?
Thank you!
Best Regards!

Different things, gather together, become a new meaningful things...maybe different graphics, but it compose a new graphic as a whole, or insert the 'T' character for this logo


fix: multimodal search make shards possible

@bwanglzu ,
once the new Tirg Encoders versions are pushed, could you try that still works with 2 shards?

failed to index documents at the beginning:

failed to index documents at the beginning:

failed to index documents at the beginning:

You are lacking magic, maybe I need to change the requirements

failed to index documents at the beginning:

You are lacking magic, maybe I need to change the requirements


When the server restarts, the players' NBT is cleared.

I noticed another serious error, when restarting the server, things from the players' inventories disappear.

Is this a graceful shutdown? Or are you throwing a SIGINT or SIGKILL.

I just stop one of the servers, /stop.
I will make a video if I cannot repeat it, I will use the latest build on GitHub.

I apologize for the panic, I found out why this was happening, it was because of a conflict with the Golf IV mod, https://modrinth.com/mod/golfiv
I think it should be indicated so that others do not encounter this. :0

Sounds good


Dynamic styles dont use camel case default

I have the following style:
  box: {
    fontSize: ({ theme }: { theme: Theme }) => theme.font.size.form,
    height: 60,
    padding: 0,
    borderRight: 'solid 1px #fff',
  }

while borderRight is converted into border-right, fontSize is not converted in the resulting class:
.box-0-25 {
    fontSize: 14px;
}

.box-0-2 {
    height: 60px;
    padding: 0;
    border-right: solid 1px #fff;
}

yes https://github.com/cssinjs/jss/issues/441

Will keep it open because I have got this issue in this repo already multiple times.

Fixed, check out react-jss 6.0