GDScript line incorrectly marked as unsafe

Tested versions
v4.3.dev3.official [36e943b6b]
System information
Godot v4.3.dev3 - Windows 10.0.19045 - Vulkan (Forward+) - dedicated NVIDIA GeForce RTX 3060 Ti (NVIDIA; <IP_ADDRESS>23) - AMD Ryzen 5 5600X 6-Core Processor (12 Threads)
Issue description
The line 13 below is marked as unsafe while the line 12 is marked as safe:

Both lines are marked as safe if I replace the line 3 by:
const EXIT_DIR : Array[Vector2i] = [

It should work the same with both syntax.
Steps to reproduce
Create a new GDScript and paste this code:
extends Node

const EXIT_DIR := [
		Vector2i( 0, -1),
		Vector2i( 1,  0),
		Vector2i( 0,  1),
		Vector2i(-1,  0)]

var room := Vector2i(10, 10)

func change_room(i: int):
	room += EXIT_DIR[0]
	room += EXIT_DIR[i]

Minimal reproduction project (MRP)
See above.

Those two pieces of code aren't equivalent though, the types are different, the safety analysis might be missing some cases though but the change to the code means different types

I have noticed this issue often. Still somewhat of a nuisance, especially since these arrays cannot really be modified.

We should be able to do static checking on the elements of the array, but the clarification is that the issue isn't the type of the array but that the safety check doesn't seem to go deeper than the array type return, this might not be entirely simple either I don't know the specifics here

As per #46830, I thought the full array type was infered...
In any case, it could be a static analysis issue or an infered type issue.

The type is inferred, the type being an untyped array, otherwise you'd have bugs and unforseen behavior, like:
var arr := [1, 2, 3]
arr.push_back("Test")

That should still work and not throw an error
The issue is that the unsafe check doesn't look into the array, the type shouldn't be inferred to Array[int] here IMO

Note that even an array declared like const arr := [1, 2, 3] can't be safely inferred as Array[int] because the array itself is not constant and can be modified just as in @AThousandShips's example

Note that even an array declared like const arr := [1, 2, 3] can't be safely inferred as Array[int] because the array itself is not constant and can be modified just as in @AThousandShips's example

That's not the case for recent Godot versions. If you call e.g. append you will get an error like array is in read only mode. Same for dictionaries.


Allow to use '$' as the start of identifier

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 4 (100.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.02%) to 58.923%

Totals

Change from base Build<PHONE_NUMBER>:
0.02%

Covered Lines:
4335

Relevant Lines:
7357

💛  - Coveralls


Opening quote of a string literal not visible until you type another character

Description:
Refer to the following. Notice how the opening quote is not visible in the editor until you type the next character in the string. Not sure if this is an issue in the plugin or an issue in VS Code itself.

OS, DB, other environment details and versions:
VS Code 1.40.2
Windows 10

This is fixed in v1.1.0 with https://github.com/ballerina-platform/ballerina-lang/commit/cb6b8143679fa3773c0140edebc39a1f3252fdd3#diff-5f863dc8b57b2349508e03471421b37c
Hence closing the issue


postgresql: always create the bin directory in postgresqlAndPlugins

This is needed because some PostgreSQL plugins don't have a bin
directory. If only these plugins are listed in cfg.extraPlugins
buildEnv will turn $out/bin into a symbolic link to ${pg}/bin. Lateron
we try to rm $out/bin/{pg_config,postgres,pg_ctl} which will then fail
because $out/bin will be read-only.
Rebased from https://github.com/NixOS/nixpkgs/pull/44379, which didn't get merged
Closes https://github.com/NixOS/nixpkgs/issues/22653
backport please
Motivation for this change
Things done

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

[ ] NixOS
[ ] macOS
[ ] other Linux distributions

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

@marsam Is this PR still valid? Are you just waiting for review from someone like @thoughtpolice, @danbst, or @aszlig?

Is this PR still valid?

yes, is still valid. I'd really appreciate a review of the PostgreSQL maintainers

Thanks team!


No way to discover all resources of a certain type

testing on beta.hydroshare.org
go to discover
there should be a way to find all geographic rasters without having to type something in the search box.

This will already be the case with #987 because you'll see all resources and filters by default.

Fixed by new discovery, closing.


[Update Snap] Starknet

Checklist
All items in the list below needs to be satisfied.

[ ] Is the summary of the change documented in this ticket?
[ ] Has a MetaMask Snaps team member reviewed whether the changes need to be vetted?
[ ] If there are changes that need to be vetted, attach a description and the relevant fixes/remediations to this issue.
[ ] The corresponding pull request in this repo has been merged.

This update deprecates the Goerli testnet and replaces it with Sepolia: https://github.com/Consensys/starknet-snap/pull/187
This update has the following changes: https://github.com/Consensys/starknet-snap/pull/188

simplify recoverAccount to avoid duplicate called on isUpgradeRequired
chore hardcode cairo version 1 to constant CAIRO_VERSION
fix findAddressIndex should return address if the address match either cairo0 or cairo{N}
fix getCorrectContractAddress, use getVersion instead of getOwner to determine account has deployed or not
chore getCorrectContractAddress to return upgrade required attribute

The new version is 2.4.0
I have reviewed the changes and confirmed they do not introduce new vulnerabilities.


doesn't work on fullscreen for some apps

I'm not sure why, but it seems to not work when an app like zathura is in full screen (F11), but it does work for full-screen mode for say Google Chrome.
Any tips? Is this fixable in the extension?

This is also the case for Emacs: color inversion gets deactivated when switching to full screen.


Fix amd support and webpack

This changes the module dependency to the correct package name in npm.
It fixed webpack usage. Related issues: #141, #60

This issue will be obsolete when #160 is merged, because it uses https://github.com/bbatliner/vex-dialog and this rewrite doesn't contain the contain the code you patched.  You are encouraged tor review #160 and the new proposed vex-dialog replacement to see if would meet your needs.

See my comment on #60. Let me know if your issue is resolved! Thanks!

Assuming this issue is resolved with #160 :)


hardcodedSeaLevelFix causes a crash with Depth Meter from Additional Additions

MC 1.19.2
FAPI* 0.66.0
Carpet Fixes 1.12.2
Additional Additions 5.1.0
To reproduce:

Set hardcodedSeaLevelFix true in carpet fixes config
Create a world
Give yourself or search in creative inventory a depth meter (which is weird, considering you don't even need to actually have the item)
Crash

Logs: https://gist.github.com/SplendidAlakey/8e40d2ab6b0fd9319468accec755473c

Should be fixed in v1.13.0


RotatE: this implementation vs GraphVite

Hi, thanks for developing RotatE!
I was wondering, why are your results a bit different from the ones obtained by your other project https://graphvite.io/?

Hi,
Thank you for your interest in both projects. The essential differences between this implementation and GraphVite implementation are listed as following:

GraphVite uses ASGD rather than mini-batch SGD，the gradient information in GraphVite is relatively noisier.
Considering the numeric stability in ASGD, GraphVite reduces the number of negative samples from 256 to 64.
For a fair comparison with other large-scale KGE systems, the reported benchmark performance of GraphVite is undertrained. GraphVite can achieve better performance when using a single GPU and train for a longer time.

@AndRossi All the technical reasons have been pointed out in the author's comment above.
GraphVite is targeted at speed and scalability so there is slight difference in performance. Actually such gap could be easily covered by some hyperparameter hacking (e.g. double the dimension of embedding or training length), but we keep the original setting to make it a clean system work.
If you want to use KGE as an off-the-shelf tool, it is recommended to use GraphVite. If you want to develop new SOTA methods for KGE, it is recommended to use this repo.


fix: model name to gpt-3.5-turbo

Gpt4 is an incorrect model name also it is not widely available. So i have added most common used model 'gpt-3.5-turbo' as default one

This should be an option the user can toggle. Not just hard coding a model to use. For this project's purpose, GPT-4 is the right way to go. It doesn't make sense with this PR to just overwrite it again, since anyone can do that locally.

GPT-4 is the right way to, It is still not available widely also it will be paid so not all of them will use that.
It doesn't make sense with this PR to just overwrite it again since anyone can do that locally, If anyone can do it locally why even gpt4 is present as an option it should be a required argument instead of an optional one.
Not just hard coding a model to use, Please explain why gpt-4 is hardcoded there.

If you want to use another model please do so via the --model parameter
$ gpt-engineer --model gpt-3.5 -turbo

You can have a look at the options/parameters by running with the --help option:
$ gpt-engineer --help

Usage: gpt-engineer [OPTIONS] [PROJECT_PATH] [DELETE_EXISTING]

Arguments:
  [PROJECT_PATH]     path  [default: example]
  [DELETE_EXISTING]  delete existing files  [default: False]

Options:
  --model TEXT                    [default: gpt-4]
  --temperature FLOAT             [default: 0.1]
  -s, --steps [default|benchmark|simple|tdd|tdd+|clarify|respec|execute_only|use_feedback]
                                  decide which steps to run  [default:
                                  Config.DEFAULT]
  -v, --verbose
  --run-prefix TEXT               run prefix, if you want to run multiple
                                  variants of the same project and later
                                  compare them
  --install-completion [bash|zsh|fish|powershell|pwsh]
                                  Install completion for the specified shell.
  --show-completion [bash|zsh|fish|powershell|pwsh]
                                  Show completion for the specified shell, to
                                  copy it or customize the installation.
  --help                          Show this message and exit.


initdb better parameter naming

user and username were to close in my opinion so I renamed username to instance_username. I also set the username explicitly, which is no difference to the default if the username was not set before:
--username=username
Selects the user name of the database superuser. This defaults to the name of the effective user running initdb.


The workflowUI doesn't work with the new REST api

I tried to install the WorkflowUI on our system, but it didn't work because it only works with the old (deprecated) REST Api. Is that a fact? or am I doing something wrong?

Hi! What's your InterSystems Data platform version?

@schoenmakers Yes, you are correct, workflowUI works only with old REST api.
Currently please use old REST api with WorkflowUI.
If you're going to modify WorkflowUI or write your own, use new REST API.

In the future we are planning to update WorkflowUI to work with the new API but I can't give you a definite timescale, when it's going to happen.

Hello.
Here's hew UI: https://github.com/intersystems-ru/workflowUI-ngx which works with the new API: https://github.com/intersystems-ru/WorkflowAPI


Alternative strategy for testing with graphql17

Yarn resolutions can be scoped, though, so you should be able to tell it to still install v17 for supermassive. iirc something like:
"packages/supermassive/graphql": "^17.0.0"

@alloy sadly, this only works for "dependencies" and doesn't seem to work for "devDependencies" or "peerDependencies"


Something is wrong with InterfaceDefaults.GetHashCode_N4

I looked into your comparers and found that some of them improve performance quite significantly.
However the hashcode for Integer seems to kinda weird - its values are very clustered and not evenly distributed. You can notice this with your own dictionaries benchmark in the SystemRapid test. It is significantly slower than SystemSystem.

sglienke,
Many thanks for your attention to the project.
Could you develop the same fast, but better hash functions?

Could you develop the same fast, but better hash functions?

You could try FNV 1a:
https://stackoverflow.com/questions/4191523/implementation-of-fnv
http://codeverge.com/embarcadero.delphi.general/fnv-1a-hash/1064108


Provide the .urcap within this package

This shall make it easier to use the URCap right away without going
through the Maven build process.
Also add a step-by-step tutorial for the Polyscope installation.

Resolves #5

Would perhaps be nice to use the Releases feature for this.
Could avoid storing a non-versionable binary in a git repository.
Might also be possible to automate builds of binaries.

Yes, good point. I'll have a look at that.

I agree with  @gavanderhoorn. We have releases for our other URCaps in place already. I can take care about that at latest after my vacation. But it's not that hard to setup, so if you want to take a look into that @stefanscherzinger... You can check the CI pipeline for our other URCaps and use the same here. I usually download the artifact from the pipeline and upload it as a binary release.

@stefanscherzinger ok to close this in favor of #7?

@fmauch I'll drop the .urcap binary from this PR and change the title accordingly.
Let's keep the rest intact to merge the .urcap setup instructions.


cmd/kubectl-gadget: Do not fail if tracer controller is dead on node

Add logic to handle the case when some the trace controller is dead
in some nodes, i.e. some traces don't reach the expected state.
This is important because it should be possible to get some results
from the cluster even if the gadget pod is not running fine in all of
the nodes.
How to test
It's not so easy to kill the gadgettracermanager on a node because kubernetes restarts it, for this reason I have to use this hack:

Modify the gadget tracer manager not to start on a specific node:

$ git diff
diff --git gadget-container/gadgettracermanager/main.go gadget-container/gadgettracermanager/main.go
index 7325c875..777d4ee9 100644
--- gadget-container/gadgettracermanager/main.go
+++ gadget-container/gadgettracermanager/main.go
@@ -214,6 +214,10 @@ func main() {
                        log.Fatalf("Environment variable NODE_NAME not set")
                }

+               if node == "worker" {
+                       time.Sleep(24 * 60 * 60 * time.Second)
+               }
+
                lis, err := net.Listen("unix", socketfile)
                if err != nil {
                        log.Fatalf("failed to listen: %v", err)

Deploy without liveness probes to avoid kubernetes restarting it

$ ./kubectl-gadget deploy --liveness-probe=false

Run a gadget

./kubectl-gadget trace exec -A
NODE             NAMESPACE        POD                            CONTAINER        PID     PPID    PCOMM            RET  ARGS
Timeout waiting for condition on node "worker"
ubuntu-hirsute   kube-system      calico-node-4mx6g              calico-node      906422  5290    ipset            0    /usr/sbin/ipset list
ubuntu-hirsute   gadget           gadget-kq7rc                   gadget           906456  906446  sh               0    /bin/sh -c exec gadgettracermanager -call receive-stream -tracerid trace_gadget_execsnoop-lr69l
ubuntu-hirsute   gadget           gadget-kq7rc                   gadget           906456  906446  gadgettracerman  0    /usr/bin/gadgettracermanager -call receive-stream -tracerid trace_gadget_execsnoop-lr69l

Fix https://github.com/kinvolk/inspektor-gadget/issues/632.

Do you have more than one node on your cluster? I just tested on a minikube cluster with two nodes and worked fine.


[IOS] TextField with visual transformation single tap crash

Describe the bug
When I try to select text in a textfield with double tap, the application crashes.
Affected platforms

iOS

Versions

Kotlin version*: 1.9.22
Compose Multiplatform version*: 1.6.0-dev1397
OS version(s)* : IOS 16.7
OS architecture (x86 or arm64): arm64

To Reproduce
0. Use textfield with visual transformation which adds symbols to the current text

Double tap on TextField.

Log:
Uncaught Kotlin exception: kotlin.ArrayIndexOutOfBoundsException
at 0   Kalculator                          0x103b87c0f        kfun:kotlin.Throwable#(){} + 95
at 1   Kalculator                          0x103b80e57        kfun:kotlin.Exception#(){} + 87
at 2   Kalculator                          0x103b81077        kfun:kotlin.RuntimeException#(){} + 87
at 3   Kalculator                          0x103b819b3        kfun:kotlin.IndexOutOfBoundsException#(){} + 87
at 4   Kalculator                          0x103b81aa7        kfun:kotlin.ArrayIndexOutOfBoundsException#(){} + 87
at 5   Kalculator                          0x103bbc34f        ThrowArrayIndexOutOfBoundsException + 131
at 6   Kalculator                          0x103d330c3        Kotlin_String_subSequence + 91
at 7   Kalculator                          0x1047850a7        kfun:androidx.compose.foundation.text.isFirstHalfOfWordTapped#internal + 307
at 8   Kalculator                          0x104784e8f        kfun:androidx.compose.foundation.text#determineCursorDesiredOffset(kotlin.Int;androidx.compose.ui.text.input.TextFieldValue;androidx.compose.foundation.text.TextLayoutResultProxy;kotlin.String){}kotlin.Int + 687

I can't reproduce it with Compose 1.6.0-dev1397
@Lavmee can you please provide a minimal reproducible sample on GitHub?
Is it reproduced on iOS Simulator, or on real iOS Device?

This happens if you use VisualTransformation (in my case using DecimalFormat) on real iOS Device.

Perhaps this is not related to the selection, but to the cursor, because a single tap on a textfield also sometimes causes a crash

Should be fixed in the next beta.

Please check the following ticket on YouTrack for follow-ups to this issue. GitHub issues will be closed in the coming weeks.


Add Comb token

为确保 Token Profile 的顺利通过，务必逐一确保以下检查项并补充项目资料：
.json 及 .png 文件检查项

[ ] logo 尺寸：120*120 px
[ ] logo 背景：透明
[ ] 代币有至少一笔成功的转账记录
[ ] 此项对于 ERC-20 项目方：合约地址有大小写（做 checksum）
[ ] .json 和 .png 文件名一致（包括大小写）

项目资料：
官方公告：1、KuChain与Combo达成战略合作，Combo将加入KuChain生态
2、库币Pool-X平台于10月17日开启COMB BurningDrop
团队背景：Co-Founder&CEO 曾在CROC担任解决方案架构师的工作；CTO Khokhlov曾在ULMART中作为技术专家负责平台技术统筹与开发工作。
项目基本情况：KuCoin交易所公链KuChain的第一个DeFi生态，已上线质押挖矿。
媒体报道：已被金色财经、白话区块链等行业知名媒体报导
https://www.jinse.com/lives/193182.html
https://www.jinse.com/lives/199004.html
http://suo.im/5QIFIw
收录的交易所：暂无
转账所需 Gas Limit（对 ERC-20 项目方）：（默认为 60000）

Please check the following to ensure a successful verification of Token Profile
.json and .png document checklists

[ ] logo dimensions: 120*120 px
[ ] logo background: Transparent
[ ] At least one transaction on the token contract
[ ] For ERC-20 project teams: Upper and lower capitalized alphabets in token address (For checksum purposes)
[ ] Same document name for both .json and .png (Including capitalization)

Project background:
Official announcement: 1 KuChain reached strategic cooperation with Combo, Combo will join KuChain ecosystem. 2 The Treasury Pool-X platform opened the COMB BurningDrop in October 17th.
Project team background:Co founder & CEO worked as solution architect in Croc; CTO khokhlov was responsible for platform technology co-ordination and development as a technical expert in ulmart.
Project basic information: KuCoin exchange's public chain KuChain's first DeFi ecology has been on-line pledged to mine.
Media publications: It has been reported by famous media such as golden finance, vernacular block chain and other industries.
Tradeable exchanges:https://www.jinse.com/lives/193182.html
https://www.jinse.com/lives/199004.html
http://suo.im/5QIFIw
Recommended Gas Limit (For ERC-20 project teams) for transaction: (Default is 60,000)

请修改合约symbol

非常抱歉，由于该PR长时间未更新，我们先关闭这个PR
如果仍需提交，请参考教程重新提交：
https://github.com/consenlabs/token-profile#token-profile-guideline


Fix spanish menu typo

Fixed a small typo where the spanish word for "Extensions" had a space in between.

Hi @carmocca, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!

This seems like a small (but important) contribution, so no Contribution License Agreement is required at this point. Real humans will now evaluate your PR.

TTYL, MSBOT;

Coverage remained the same at 61.233% when pulling 6d32b38a40bb519c9f40fa4890b562b544b0975f on carmocca:master into 1459824cade4ac4cc80d51621ebfc0015cbcbcbe on Microsoft:master.

@lukaschal Please fix this typo in /Localize/esn/vscode/src/vs/code/electron-main/menus.nls.resx.lcl
  <Item ItemId=";miViewExtensions" ItemType="0;.resx" PsrId="211" Leaf="true">
    <Str Cat="Text">
      <Val><![CDATA[E&&xtensions]]></Val>
      <Tgt Cat="Text" Stat="Loc" Orig="New">
        <Val><![CDATA[E& &xtensiones]]></Val>
      </Tgt>
    </Str>
    <Disp Icon="Str" />
  </Item>

Fixed.
File: src\vs\code\electron-main\menus.nls.resx
ID: src\vs\code\electron-main\menus.nls.resx
SRC: E&&xtensions
TGT: E&&xtensiones


fix: punycode deprecation warning

Status
READY
Description
Orval indirectly depends on node-fetch, a shim for the Fetch API.
To handle URLs in a standards-compliant way, it depends on whatwg-url, a Node.js polyfill for the URL API.
The version of whatwg-url that Orval depends on uses punycode - a built-in (kinda) module that is deprecated in modern versions of Node.js.
Because it is deprecated, users of Orval will be given the following error message:
(node:95791) [DEP0040] DeprecationWarning: The 'punycode' module is deprecated. Please use a userland alternative instead.

This PR resolved the whatwg-url dependency to the latest version, which does not depend on punycode and so the message will no longer be shown. The URL API is stable, and swapping out the implementation does cause any breakages.
Alternative solutions
Orval bundles all dependencies into a single distribution file. This is good for locking dependencies, but causes the package to be bloated and prevents users of Orval from making this change themselves.
Making node_modules dependencies external in the build step might be worth considering.

Thanks running the tests now.

thank you. I'll check it later 👍

Hey, I'm on 7.2 and still see this warning.. how do I clean that up exactly?

@LukeChannings any thoughts?

I guess either the whatwg-fetch dependency has regressed or another dependency has been introduced that depends on punycode.
I'll have a closer look when I'm by a computer

Thanks @LukeChannings for checking


build(client): Update eslint config to latest version (5.5.1)

Updates the client release group to eslint config 5.5.1.

@RishhiB is actively working on turning these back on.


[local_auth] Update and migrate iOS example project

Description

Let https://github.com/flutter/flutter/pull/26630 flutter_assets migrator run
Run pod install so flutter_assets is removed from the asset copy build phase
Migrate deprecated "English" to "en" language
Allow Xcode to remove extraneous xcconfigs, see https://github.com/flutter/flutter/pull/38724
Let Xcode 11 update build settings
Remove DEVELOPMENT_TEAM references since having those provisioning profiles should not be required to run the examples (most examples don't have one)
Remove ARCHS, which was causing a compilation error
Remove extraneous framework outputs https://github.com/flutter/flutter/issues/20685

Related Issues
Get to a buildable baseline so I can start working on https://github.com/flutter/flutter/issues/41007.
Checklist

[x] I read the [Contributor Guide] and followed the process outlined there for submitting PRs.
[x] My PR includes unit or integration tests for all changed/updated/fixed behaviors (See [Contributor Guide]).
[x] All existing and new tests are passing.
[x] I updated/added relevant documentation (doc comments with ///).
[x] The analyzer (flutter analyze) does not report any problems on my PR.
[x] I read and followed the [Flutter Style Guide].
[x] The title of the PR starts with the name of the plugin surrounded by square brackets, e.g. [shared_preferences]
[x] I updated pubspec.yaml with an appropriate new version according to the [pub versioning philosophy].
[x] I updated CHANGELOG.md to add a description of the change.
[x] I signed the [CLA].
[x] I am willing to follow-up on review comments in a timely manner.

Breaking Change

[ ] Yes, this is a breaking change (please indicate a breaking change in CHANGELOG.md and increment major revision).
[x] No, this is not a breaking change.

Does a change to the example app require a build suffix version bump?

Does a change to the example app require a build suffix version bump?

Yes, I believe so.

All the changes are generated by the migrator, is that right?

Some of them are.

Let flutter/flutter#26630 flutter_assets migrator run

flutter migration.

Run pod install so flutter_assets is removed from the asset copy build phase

CocoaPods migration after flutter_assets is removed.

Migrate deprecated "English" to "en" language

Xcode migration.

Allow Xcode to remove extraneous xcconfigs, see flutter/flutter#38724

Xcode migration.

Let Xcode 11 update build settings

Xcode migration.

Remove DEVELOPMENT_TEAM references since having those provisioning profiles should not be required to run the examples (most examples don't have one)

I manually removed this.  The user of these examples shouldn't be expected to be logged into any particular account or have any particular Flutter development provisioning profiles installed.

Remove ARCHS, which was causing a compilation error

Xcode migration.

Remove extraneous framework outputs flutter/flutter#20685

CocoaPods migration.

We checked in the ios generated code as it was, is your work a way to remove these extra output automatically?

We checked in the ios generated code as it was, is your work a way to remove these extra output automatically?

Except for removing DEVELOPMENT_TEAM, these changes were all automatic migrations either from the Flutter tool, Xcode, or CocoaPods.  I wanted to avoid having changes in the working copy every time I ran the example app.


move interfaces into hardware_interface package

I am sorry, this change is pretty brutal, but I feel it makes more sense like this.

the interfaces for components were made as real interfaces with pure functions (= 0)
I shifted the logic for the urdf parser into the hardware interfaces as well. I could see that we put that into its own package or leave it inside there as-is.
the robot control components are somewhat default implementations of these interfaces and exported via pluginlib. No public headers are needed for this.
the linters are mostly run. There're some cpplint and test failures in the component parser.

Just went through your comments again.
As for claiming resources, I am not sure that each individual package should take care of this. I'd rather have this at a central place (possibly the hardware interface) which manages all resources "globally".

My observation regarding managing claims in each actuator is following:
CONS:

larger header files
addition member function calls

PROS:

no need for additional maps in container object
Actuators are self-contained
transparent interface - no functional influence from other classes (similar to previous one)
simpler management of Actuators in container object

Thanks for the fast answer.
I also added a few comments and the only question is, if we extent the final classes with some shared functionality...

no need for additional maps in container object
Actuators are self-contained

These two points are pretty important for me and I think it's the right approach. That would also go along with the discussion about using InterfaceValues in these interfaces.

I was thinking about the resource claiming a bit more. I haven't had time to prototype it, but essentially I'd like to implement a central resource manager within the hardware interface package. Each individual component (a single actuator, a single sensor or a complete system) has to implement a function in the style of get_claimed_resources() in which each component returns a list of resource names it tries to claim.
Upon loading, the hardware resource manager then verifies that every resource exists and can be claimed only once. That way, the only thing to do per component is to declare a list of resources.
Whenever the component is deactivated, the resources are being released and (re-)claimed when (re-)activated.

All that resource claiming would be specific to the actuators though. I don't really see a point in claiming a sensor resource. What do you think?

I agree completely, this is only an issue when writing variables.

Each individual component (a single actuator, a single sensor or a complete system) has to implement a function in the style of get_claimed_resources() in which each component returns a list of resource names it tries to claim.

I imagined that each component has/posses it’s resources and it is therefore responsible for it. My idea was that only through components hardware can be accessed. Nevertheless, I understand and agree with you on this. We can have one Ressource manager who takes care about that.

That would also go along with the discussion about using InterfaceValues in these interfaces.

To my knowledge, it is the same if we use some kind of complex objects (e.g. maps) or vectors/arrays of doubles since we are just passing pointers to them. Am I right on this? If so I would rather provide more data and than users can decide if and how they use them.

To summarize.
The main open questions are:

[ ] How resources (actuators) are claimed?
[ ] Which type of pointers should be used in read and write functions? (doubles or more complex data)

And TODO:

[ ] add constant with common interface types (for now: position, velocity, effort)

I imagined that each component has/posses it’s resources and it is therefore responsible for it. My idea was that only through components hardware can be accessed.

That is still true. The actual communication with the hardware is done through the components. The resource management is just purely virtual within the hardware interface. I don't think that a component should know much about its peers.

The resource management is just purely virtual within the hardware interface. I don't think that a component should know much about its peers.

I meant actually, that actuators are only managing resources. Nevertheless this is not the issue currently. I will merge this until Monday so we can continue to work on the one stream of code.
I hope this is ok with you :)


Swift Package Manager unable to find headers on subfolders

Apparently Swift Package Manager has a limitation when working with C headers, they need to be in a flat hierarchy otherwise SPM can't find them.
Transcript from Swift Evolution:

For C family library targets, we propose to add a publicHeadersPath property.
This property defines the path to the directory containing public headers of a C target. This path is relative to the target path and default value of this property is include. This mechanism should be further improved in the future, but there are several behaviors, such as modulemap generation, which currently depend of having only one public headers directory. We will address those issues separately in a future proposal.
All existing rules related to custom and automatic modulemap remain intact.

More on this context: https://github.com/apple/swift-evolution/blob/master/proposals/0162-package-manager-custom-target-layouts.md
This problem didn't arise before as the proposal for enabling Swift Package Manager was originally using the source code of master, on branch release/1.3.0 there are headers not contained on the root directory.
Symptoms:

Thanks @vauxhall , you are right. The travis script brought me in fault. It is putting the green tick also when SPM demo build doesn't start correctly.
Unfortunately It seems there are many frictions between SPM and ObjC libraries, at the moment.

For reference this issue is linked to: #474


[Javascript] Suggest to contributor adding ESLint extension to visual studio code

Description
This ESLint extension should help to guide the contributors to a standard development process. This will prompt Visual Studio Code to suggest the installation of the ESLint extension automatically.
You can review the extension documentation here.
Changes made

.vscode/extensions.json
.gitignore: Added exception for the new file, but ignoring the rest of the folder.
.gitignore: included exception for settings file (workspace).
.vscode/settings.json: contains configuration for eslint validation (typescript).

Testing
You can see now eslint lint warnings in visual studio code.
This IDE integration should improve productivity. The warnings detected will also appear in the Problems window.
IDE warning highlights

Extension Details

MS 1726


feat(JobCard): add selected effect

Reason for change
Add a props isSelected to set selected style for JobCard component.
https://realtimeboard.com/app/board/o9J_kzK68Z8=/?moveToWidget=3074457346150011938

Preview:
http://seek-asia-style-guide--e44281023d11f12998ffd7eb785409e59b05a2c4.surge.sh/jobCard

@mengtzu and @hscgavin please help to review again, sorry about that. Just got an update from designer there will be no frame / border for selected status, only background color changed.


Mask values written to mmap_write{8,16}

Fix for #460

I think tests are failing because write8 and write16 in memory.rs also need a mask.

Cheers!


Support TXT Records

mdns services may have an associated TXT record containing arbitrary key value pairs. This is very useful to expose meta information.

Ok, people keep asking for it and it is a cool feature, so here we go:
I wrapped (most of) the required functions. Converting JS objects in both directions works, too. Just need to

do a little argument checking (RFC conform keys &c.)
double check that JS objects always fit the bill
(if not) add an alternative, raw API

Take a look at the browser test for a preview ...

Habemus documentation.

Done.


lib/debug: remove functions that have been deprecated since 2018

traceShowVal
traceShowValMarked
attrNamesToStr
showVal
traceXMLVal
traceXMLValMarked
traceCall
traceCall2
traceCall3
traceValIfNot
addErrorContextToAttrs
traceCallXml
they were deprecated in https://github.com/NixOS/nixpkgs/pull/38368
Description of changes

Things done

Built on platform(s)

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

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

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[ ] Fits CONTRIBUTING.md.

To deprecate something, it first needs an explanatory warning in one release, an explanatory error in another, and then it can be removed in a further release down the line.
This means for functions without a warning you should add one and for ones with a warning you should make it an error instead.

i've added throws and FILL IN's for functions which i don't know how they were used and don't know the replacement


Client Crashing when mousing over fruit - The One Probe conflict?

Description:
No issue on the server side, but clients instantly crash when mousing over any fruit on trees.  We have experienced this with Dates on Dynamic Atum Palm trees, and now on Holly Trees from Oh the Biomes You'll Go.
Reproduction:
Mousing over any dynamic tree based fruit is all it takes.
Rendering:
We are using Rubidium and Oculus.
Using default textures from Atum/BYG.
Crashlog
https://pastebin.com/ycgUb6rk
Mod Notes:
ATM Spellbound
DynamicTrees-1.16.5-0.10.2
DynamicTreesBYG-1.16.5-1.0.2

this issue occurs when using Jade as well, as per https://github.com/DynamicTreesTeam/DynamicTreesPlus/issues/23

this fixes it:
diff --git a/src/main/java/com/ferreusveritas/dynamictrees/systems/fruit/Fruit.java b/src/main/java/com/ferreusveritas/dynamictrees/systems/fruit/Fruit.java
index 72ade9eb..abd2cc64 100755
--- a/src/main/java/com/ferreusveritas/dynamictrees/systems/fruit/Fruit.java
+++ b/src/main/java/com/ferreusveritas/dynamictrees/systems/fruit/Fruit.java
@@ -179,6 +179,9 @@ public class Fruit extends RegistryEntry<Fruit> implements Resettable<Fruit> {
      * @return a copy of this fruit's item stack
      */
     public final ItemStack getItemStack() {
+        if (itemStack == null) {
+            return ItemStack.EMPTY;
+        }
         return itemStack.copy();
     }


mock pa2 test case 3_1

Hi, does anyone know how to fix this?

Hi, it is likely that in your mul implementation you didnt account for 0 x 0 which may result in an infinite loop

There is infinite loop going on. Strange codecrunch doesn't give time limit exceeded though

did you check for 0s? maybe can add an if-else statement to account for any zeros!

I did account for 0s, and it seems to be working fine on jshell

you need to account for the situations that have either variable as 0, not just when both are zero

you need to account for the situations that have either one of the variables as 0, not just when both are zero

do you mean in the cases like zero.mul(one) or one.sub(zero)? because those seem to work too on jshell

Check your multiplication try both one.mul(zero) and zero.mul(one)

did you consider for zero.mul(zero)?
i ran into the same error because i forgot to consider for that case
so my loop with .equals() method became an infinite loop

I tried all of this but it all seems to work fine on jshell.. maybe I’m misunderstanding what you guys are suggesting? lemme know if I tried it all, I attached screenshots below
nested mul:

zero.mul(zero), and either one being zero:

Check your add() and sub() methods

did u try a.mul(b.mul(c))? if it doesnt obey the associative rules its likely a mistake within ur subtract method

Hey @sje5398p, I am facing the same problem unfortunately. Did you manage to figure it out maybe?

Notice that lines 69 and 71 tests the quadratic equation formula, where b is in fact negative. Hence, check your add() and sub() methods again

Hey @sje5398p, I am facing the same problem unfortunately. Did you manage to figure it out maybe? I think that the problem lies in the fact that time limit gets exceeded, but I wouldn't know how to deal with that.

hello, I haven't figured it out yet, will lyk when I do!

Notice that lines 69 and 71 tests the quadratic equation formula, where b is in fact negative. Hence, check your add() and sub() methods again

then how come b is not invalid?

@sje5398p Hey! I managed to fix my code! I hope this will solve your issue as well, but: In my mul() method I first had "result = this.add(result);" in each iteration of my while loop. Using your values of a, b, and c, I noticed that the operation a.mul(c.mul(b)) took considerably longer than a.mul(b).mul(c). So, then I changed my code to "result = result.add(this);", which solved the problem of one taking longer than the other! I hope this helps!

Hii, thank u for reaching out haha, my mul() method already incorporates "result = result.add(this);"

do you spot anything else that could be the problem?

@sje5398p
pls lmk if u spot anything wrong.... thank you sm

It is evident in your mul() you did not check if other is zero

How does your sub() method look like?

How does your sub() look like?
like this :) thank u for ur all help

Line 76 new Num(newOther) may be evaluating to an invalid Num

I tested for this, it seems to be valid?

Now try calling add() on minusTwo

Now try calling add() on minusTwo

realised it will give me all valid outputs even if it is negative.. maybe that is the problem?

It is highly probable this is the problem. Your add() should always check if the final result is valid


[Android-Wallet-App] Revocation Entry has Expired but Wallet App still Evaluates DCC-Code as REVOKED

Describe the bug
The revocation entry for this DDC

certificate of type REC was defined with 'expires': '2022-02-24T15:00:00Z'  I checked the code with the wallet app today  (24.02.22) at 17:34 but it is still shown as revoked (see attached photo).
Expected behaviour
From 15:00 o'clock 24.02.22 on, the wallet app should evaluate the code as valid again, due to the expired revocation.
Technical Details
Wallet App for Android, Version: 1.2.8-tst (41)
Galaxy XCover 4, Android 9

@RalicaY can you check again with this certificate. For me it is valid in the verifier app.

In wallet already expired certificates while claiming are shown as valid. But if a cerificate expires while it is stored in the wallet then this is shown as revoked after the expiration date too.

I will recheck, had claimed an certifcate that will expire on 30/03/22 9 a.m. So I can recheck at this time.

Retest with version 1.3.0 TST (39) in E2E-Testteam also positive. Revoked entries are correctly displayed as revoked. When the expiration-time occured for a QR-code, it will automaticallydisplayed as valid again. This is trigered by "Update revocation state" in Setting or by new start of the Wallet App.


Added category information to KGX export.

Adds category information to the KGX export.
Will close #10.

A version of this code has already been merged. Closing.


Remapping Physical Keys

I have a Unihertz Titan witch has a Physical keyboard on it. I want to use the Keys of it to control the games. Other emulators allow you to choose a Button (Like A, B or Start) and map any keys/button/input (external gamepad/connected Keyboard and so on) to it. Is this possible in gbcc too?

Not at the moment unfortunately. I just have gamepad remapping:

This was on my to-do list before I took a break from gbcc though. I'm trying to get back into development of it at the moment, so I'll take a look over the weekend and let you know. The main problem is one of UI I think.

omg! GREAT JOB!!!

:+1: happy to help!


Possible regression in parsing Google Cloud Storage log with latest version of goaccess

Given this log file taken from the original issue for GCS log format (https://github.com/allinurl/goaccess/issues/262#issuecomment-117279459)
"1435673425692000","<IP_ADDRESS>","1","","GET","/blog/data-ownership-in-nitropcr","301","0","0","211000","www.nitropcr.com","","Mozilla/5.0 (compatible; Baiduspider/2.0; +http://www.baidu.com/search/spider.html),gzip(gfe)","AEnB2Uo6_oFt6zlbUfcwodxyyKbvVNHi-PytOKYHJuxBOdWZgwcUe-ZGcBhJaCQwi3xS0wSaK7OPoVXmwYF2pwWH8L4s16TzoA","GET_Object","www.nitropcr.com","blog/data-ownership-in-nitropcr"
"1435673426067000","<IP_ADDRESS>","1","","GET","/blog/data-ownership-in-nitropcr/index.html","200","0","13654","179000","www.nitropcr.com","","Mozilla/5.0 (compatible; Baiduspider/2.0; +http://www.baidu.com/search/spider.html),gzip(gfe)","AEnB2UqRS8KJ9c9WfLWhpK-lG2wPir9rII5Dt0Dckw6g6oj-BVLXCMsqBpZQZ-11g4hRKp0mGQp4csi3NrQwGPXig3BPwLN7hQ","GET_Object","www.nitropcr.com","blog/data-ownership-in-nitropcr/index.html"
"1435673418492001","2602:30a:c037:b3b0:d63d:7eff:fef1:49b5","2","","GET","/storage/v1/b/www.nitropcr.com?fields=logging&alt=json&projection=full","200","0","97","50000","www.googleapis.com","","apitools gsutil/4.13 (linux2) Cloud SDK Command Line Tool 0.9.65,gzip(gfe)","b2ae6a88c12b69f55b5d3397cca80ba2","storage.buckets.get","www.nitropcr.com",""
"1435673978352000","<IP_ADDRESS>","1","","GET","/robots.txt","200","0","24","261000","www.nitropcr.com","","Mozilla/5.0 (compatible; Googlebot/2.1; +http://www.google.com/bot.html),gzip(gfe)","AEnB2Ur1ACazfDlNV-J3qw6_9bKb6YxmQQao7ZidoFgzl0A0RZTOe_Okoq0UibSYN2MwxvWG2rRPUc1ov0E1S8reNK0Eq49i_Q","GET_Object","www.nitropcr.com","robots.txt"

This command:
cat test.log | docker run --rm -i -e LANG=$LANG allinurl/goaccess:1.4.3 -a -o html --log-format CLOUDSTORAGE - > report.html

successfully generates the HTML report but this command:
cat test.log | docker run --rm -i -e LANG=$LANG allinurl/goaccess:latest -a -o html --log-format CLOUDSTORAGE - > report.html

fails with the following output:
==1== GoAccess - Copyright (C) 2009-2020 by Gerardo Orellana
==1== https://goaccess.io -<EMAIL_ADDRESS>==1== Released under the MIT License.
==1==
==1== FILE: -
==1== Parsed 4 lines producing the following errors:
==1==
==1== IPv4/6 is required.
==1== IPv4/6 is required.
==1== IPv4/6 is required.
==1== IPv4/6 is required.
==1==
==1== Format Errors - Verify your log/date/time format

Thanks for reporting this. Can I ask, are you using the stock GCS log? Thanks

Yes the test log file is a standard GCS "usage log" as described here.

This has been fixed upstream and will be pushed out in the upcoming release. It was an issue when the JSON parser was introduced. Please feel free to test this out by building from development and let me know how it goes.
Thanks again for reporting this!


Validate timeout greater than 0s for InformersSyncTimeout

In early stages of Beyla Alloy integration this flag was set to 0 and Beyla was crashing.
This PR adds a validation (tho i wasn't able to replicate the error this time, so just in case).
Fixes #739

Codecov Report
Attention: Patch coverage is 0% with 2 lines in your changes missing coverage. Please review.

Project coverage is 45.66%. Comparing base (f632c99) to head (62a08b5).

Files
Patch %
Lines

pkg/beyla/config.go
0.00%
1 Missing and 1 partial :warning:

:exclamation:  There is a different number of reports uploaded between BASE (f632c99) and HEAD (62a08b5). Click for more details.
HEAD has 3 uploads less than BASE

Flag
BASE (f632c99)
HEAD (62a08b5)

oats-test
1
0

k8s-integration-test
1
0

integration-test
1
0

Additional details and impacted files
@@             Coverage Diff             @@
##             main    #1103       +/-   ##
===========================================
- Coverage   73.50%   45.66%   -27.84%
===========================================
  Files         172      172
  Lines       13091    13093        +2
===========================================
- Hits         9622     5979     -3643
- Misses       2920     6713     +3793
+ Partials      549      401      -148

Flag
Coverage Δ

integration-test
?

k8s-integration-test
?

oats-test
?

unittests
45.66% <0.00%> (-0.03%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

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


Service as web Worker?

Hi, is it possible to create a service that runs as a web Worker, i need a messaging service that will get updates from the main server, do some work and give the results back to module, some updates are really big and it takes about 5-10 seconds to process.
Or i can have a regular t3 service starting a web Worker and then passing Worker messages to module?
a Box.Application.addWorker would be nice :)
Thanks

I went with the regular service stating it's web Worker.


Concurrency seems bad in Centos7.3

Issue Label:

[x] Bug

Description:
I try this command: currency CNY HKD 10  and get the output which not show as sample:
➜  ~ currency CNY HKD 10

=========================
| CNY to HKD
| Rate: 1.1474015352
1.0
| CNY: 10
| HKD: 1.1474015352
10.0
=========================

If its a bug make sure to include this section.
OS and OS version:

[x] Linux 64 Bit

OS Version:
➜  ~ hostnamectl
   Static hostname: china
... ...
  Operating System: CentOS Linux 7 (Core)
       CPE OS Name: cpe:/o:centos:centos:7
            Kernel: Linux 3.10.0-862.11.6.el7.x86_64
      Architecture: x86-64

Think this has been fixed in b270518de8c96ed1e67fafd7701ae08aa932ec1e, you will have to install from master to use the patch


Determine how to handle test for records.delete() Code Snippet

File path: /site/api/web5-js/dwn/records.mdx
Code Snippet
const response = await web5.dwn.records.delete({
  from: 'did:example:alice',
  message: {
    recordId: 'bfw35evr6e54c4cqa4c589h4cq3v7w4nc534c9w7h5'
  }
});

This one is tricky because the recordId is hard coded. Options could be testing with the expectation of failure.

Going to skip because of complexity around hard coded values and needing to refactor the code snippet itself to properly handle the error

Even though we have good coverage elsewhere for the delete method, we do need to figure how to get all snippets tested because these surface as docs, telling the dev the code to use. @dayhaysoos would like more time to talk over with DevRel how this code is surfaced in a testable way. @leordev had some ideas to resolve as well. Bumping to Phase 6.

@acekyd - pushed to next release because of the assumption that we couldn't get it in time for this one, but if you come to resolution on it in time for this release, by all means have at it and set to Phase 5 in the GH Project!


[Feat]: Adding Email Validation

Is your feature request related to a problem? Please describe.
Implement email verification for user registration to enhance security and reliability. This involves sending a verification otp upon registration, which users must enter to confirm their email addresses. The feature aims to prevent spam, improve account security, and enhance user experience.
Describe the solution you'd like.
No response
Describe alternatives you've considered.
No response
Additional context.
No response
Show us the magic with screenshots
No response
Checklist

[X] I have checked the existing issues
[X] I have read the Contributing Guidelines
[X] I want to work on this issue. (optional)

assigning to you @SamarthUrane, you can start working on it.

@Yash636261 Assign to me.
Steps i'll be doing:
1. User Email Collection: from the signup page.
2. OTP Generation and Emailing: Generate a 6-digit OTP and send it to the user’s email using Nodemailer.
3. OTP Verification Page: Create a page for users to enter the received OTP.
4. Backend Verification: Validate the OTP on the backend to complete the authentication process.
Benefits:

Enhanced Security: Adds an additional layer of security.

Improved Trust: Shows our commitment to user data protection.


Option in VM dict for packages

An option in the VM dictionary to add repos (e.g. EPEL) and multi-value list of dnf packages.

This would/should be in a follow up role separate from provision itself.  The register playbook will eventually be decoupled as it was really more of a demo of concept.  Next steps will be to create an "R2"(R squared = Randy and Randy)  collection.  This collection will contain all of the config management related roles/tasks/playbooks.


Additional classes ignored when first argument only has strings

var classes = bem({
    name: 'name'
}, 'additional');

Expected result:
'name additional'
Actual result:
'name'

@BurntCaramel I've released the fix in v1.0.7. Thank you for reporting the problem!

That’s alright! I had a fix in progress, just needed to install mocha and create a test – was going to finish it the next day. Thanks for the super speedy response!


Script cannot iterate through all organization for Enterprise

It is very painful to use this script for larger enterprise with 100s of Organization.
It is also better to provide switch to provide specific data by iterating through all organization and repo for specific informations.

It is very painful to use this script for larger enterprise with 100s of Organization.
It is also better to provide switch to provide specific data by iterating through all organization and repo for specific informations.

1100


Dependency not resolved by default

[ ✅] There is a similar issue, but not the same. I have searched existing issues and confirmed this is not a duplicate.

Problem
With Gradle, 'com.google.android:flexbox' artifact is not available in default jCenter repo, and also not available in MavenCentral. The error is "Failed to resolve artifact".
I checked the URLs that Gradle tries to resolve, and it's looking for the lib in jcenter.bintray.com, but the code is actually at google.bintray.com as found through your 'download' badge -- which is (I believe) the reason that causes the issue.
How to reproduce it?
Create a new empty project in Android Studio and try to add this dependency. It can be an empty project. Use Android Studio 3.0 or later. I don't know which plugins you have for project creation, but it's extremely important not to add any Google-specific dependencies, as everyone outside of Google would not have them included by default.
Having a Google repository pre-included is probably the reason why it's working for you and other people are complaining about not resolving the artifact (as in this issue and this one).
Version of the flexbox library
1.0.0 and 1.1.0
Preferred resolution
Looks like we need to add a Google jCenter repo somehow, can you tell us how?  There is a button to do so after you publish a lib to your local repo inside of jCenter.
Flexbox is not even visible by search in jCenter or MavenCental index pages, so please also check why that is the case.

Oh, and yes, I do have google(), jCenter() and mavenCentral() dependency repositories included for all modules, of course.

I have the exact same issue with version 0.2.5, It seems that artifacts con jcenter are down at the moment, on Friday it was working as expected

Interesting! But it's somehow hidden in search too... 🤔🤔
On Mon, Dec 10, 2018, 15:31 Mauricio Vignale<EMAIL_ADDRESS>wrote:

I have the exact same issue with version 0.2.5, It seems that artifacts
con jcenter are down at the moment, on Friday it was working as expected
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/google/flexbox-layout/issues/475#issuecomment-445835003,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABsJitKKNIDQJqBk6ElwKXymo9CBHOL2ks5u3nA8gaJpZM4ZLcjM
.

I have the same problem with the provided versions for those who do not use androidx

0.3.0 is not working for us either

Hello. I'm using version 0.3.2 as a dependency, and currently, my build fails.

Same issue here. It looks like the library vanished from the repository 🤔
In the meantime you can add this repository as a workaround:
maven {
    url "https://google.bintray.com/flexbox-layout"
}

Anyone experiencing this on other dependencies ?
I've been hitting this issue without various artifacts today ! (google-services, flexbox, firebase).. Wondering what's happening with jcenter...

@julienR2 Yes!  Though flexbox is the only one so far with a concrete solution for it.

Anyone experiencing this on other dependencies ?
I've been hitting this issue without various artifacts today ! (google-services, flexbox, firebase).. Wondering what's happening with google(), jCenter() and mavenCentral()

I'm having with all the dependencies you mentioned too, with no success with my builds until now.

Does anybody have something from Google about this problem?
I'm having the same problem with a lot of Google libraries.

@shrpereira Best I have is this one: https://issuetracker.google.com/issues/120759347

Yeah this is a huge blocker - a few new devs joined the team and none can perform their first builds.

We apologize for what happened here : its back available here https://jcenter.bintray.com/com/google/android/flexbox/, you no longer need the workaround.
https://b.corp.google.com/issues/120759347#comment97 : Please let us know if you run into any more issues.

Here's the public link to the comment from @jabubake https://issuetracker.google.com/issues/120759347#comment97 ;)

Apologies for the inconvenience.
The root issue has been resolved. Please reopen the issue if it still persists.

Execution failed for task ':app:checkDebugAarMetadata'.

Could not resolve all files for configuration ':app:debugRuntimeClasspath'.
Could not resolve com.google.android:flexbox:1.0.0.
Required by:
project :app > com.salesforce.service:servicesdk:224.0.1 > com.salesforce.service:chat-ui:4.1.2
When building with the latest gradle, and needing to remove jcenter()

Anyone else finding the same from saleforce?

Yes, this issue creeped back through the sales force chat-ui library. I experience the exact same error as @GordonBerrington . So @thagikura  how are we supposed to solve this problem in this case? Do you have any recommendations when the removed dependency is referenced in some library out of our control?


[5.1] Cache Fixes

This also fixes #9175. // cc @ifduyue

@GrahamCampbell Thanks. I confirm that this works.

Yeh, this bug was affecting me too. :)


Investigate consistency of account id/caller id in a direct call to function

When making a direct call to a contract method (not a test method), we should investigate:

Which accounts are present?
What are the values of <id>, <caller>, and <origin>.
Which account are we executing as?

In general, these should all be symbolic, but the account we're executing as should not be the hardcoded foundry one, but an arbitrary one, for generality of the generated KCFG.

@palinatolmach This one is very stale. Is it worth keeping, or should we close it?

Jumping in because I saw the notification. I think this issue can be closed since these items have already been addressed in the first iteration of CSE.

OK, sounds good to me. I thought it might be addressed with work done in the last 6 months, and happy to get it off the board. Thank you, @anvacaru!


Upgrade to 1.22

It might be good to extract the IMDSv2 feature to a separate PR, so it would be easier to backport (cherry-pick) to the older versions later on if we need?


I had installed Extention Yeoman corrently,but it can not work when i press 'yo' Enter

Please file issues against the extension you installed.


Release to maven central?

Hi,
Would it be possible to make a release to maven central? That would be really useful :)
Regards,
/Johan

Sorry I was too fast, I now see that it's already there. I was confused this I couldn't find it in the docs. But this great! Thank you :)

Thanks for asking @johanhaleby, feel free to follow up if you have any additional feedback on the library.


Revert "Add a helper type alias for __unsafe_unretained"

It turns out that this doesn't actually fully work. It appears to only suppress ARC inserting retain/release calls around places where the pointer is just passed around, but not when ivars are read.
This reverts commit f43997154e49b086be3ce69035ba50d48bc3a98e.

:sad_trombone: but :+1:


Use of the home directory?

I'm finally to the point where I'm trying to run things and I've just noticed that pyspextool_setup adds a file to the users home directory (.pyspextool_spex.dat). This makes me very uncomfortable for many reasons.
Could someone explain to me the purpose of the files in the users home directory?

It goes back to the IDL version.  The idea is to not have the user have to add the paths in each and every time they start the GUI up.  In principle if people are using the python version in batch or script mode this isn't necessary, but if they are using it at the command line, it might get annoying (although arrow up could be used).
I am not wedded to keeping this feature.  However, we have talked about writing out a log after each step and that is going to have to get written somewhere.  Could be in the proc/ directory which eliminates things going into the home directory.

Writing the directories was "disabled" via commenting in #112.  We can decide later if we want to bring this functionality back.


Support for Lifetimes in Structs Deriving Store and Patch

Describe the bug
I'm using the reactive_stores crate and I'm encountering an issue when trying to use lifetimes with structs that derive Store and Patch. Specifically, I'm getting errors related to undeclared lifetime parameters.
Leptos Dependencies
reactive_stores = "0.1.2"

To Reproduce
#[derive(reactive_stores::Store, reactive_stores::Patch)]  // throws a compile error
struct Todo<'a> {
    label: &'a str,
    completed: bool,
}

Expected behavior
no compile errors
Additional context
Diagnostics:
1. lifetime `'a` is missing in item created through this procedural macro [E0261]

I do not think this would be useful, as both Store and ArcStore require that their inner types be 'static, i,e., the store needs to own its value and cannot contain non-static references. So even if the derive macro worked you couldn't actually use it for anything.


Update manifest private data

Your module manifest should include private data with links to your license and this repo as the Project URI. When people search for your module, having the project link is helpful so they can review it before deciding to install it. If it helps, take a look at one of my module manifests.

Thanks for bringing that to my attention. I've updated the manifest to include links to those resources, as suggested.


Add more tests and jsdocs for saltcorn-admin-models & Add ability to limit file types to upload

Add more tests and jsdocs for saltcorn-admin-models
Add ability to limit file types to upload

@AleksandrSokolov it would be better to add the file filter to the upload fileview here:
https://github.com/saltcorn/saltcorn/blob/v0.8.0-beta.0/packages/saltcorn-data/base-plugin/fileviews.js#L101-L128
Add your field (https://github.com/saltcorn/saltcorn/pull/1472/files#diff-b67356cdcfedae3f65d51bff81d7350911b0be04245d24021b5b9feac7c983afR597-R602) here instead: https://github.com/saltcorn/saltcorn/blob/v0.8.0-beta.0/packages/saltcorn-data/base-plugin/fileviews.js#L113
and your implementation of the accepts attr to the input here: https://github.com/saltcorn/saltcorn/blob/v0.8.0-beta.0/packages/saltcorn-data/base-plugin/fileviews.js#L124
The admin should be able to upload any kind of file, but restricting the user is very useful (e.g. only images)

@glutamate please review. I reworked file filtering. Added accept filter to file field. So for every field column possible to define own filter. Also default filter applied if user does not define filter for particular field.

Thank you, looks good!


Component outputting HTML from different component

Description
Basically I have two components, one for SkipToContent, one for Header. The SkipToContent component is outputting the static HTML contents of the Header component, and I'm unsure why. I didn't want to make an issue for this but received no answer on stackoverflow.
I've described my issue on stack overflow in more detail, here: https://stackoverflow.com/questions/60646681/component-incorrectly-outputting-content-from-other-component
Steps to reproduce
I don't have this deployed anywhere but if the information in the stackoverflow question are not enough to debug, I can deploy the current project to Netlify tonight.
Expected result
SkipToContent should render its <a> element, not the contents of the Header component
Actual result
The Header component is rendered twice.
Environment
Paste the information here as shown by gridsome info

I'm not currently on my personal machine but I can grab this information when I get home.

@jtrobinson1993 how did you solved it?

I didn't, I'm going to switch to a different framework.


SMSBackup+ crashes when doing backup.

An FC happens immediately after the step "Calculating...".
SMSBackup+ version 1.4.3
Could you please help to investigate the root cause of the problem? Thanks! If there this any more debugging information needed, please feel free to let me know.
From logcat I grabbed the following information:
D/dalvikvm(12584): GC_FOR_MALLOC freed 9428 objects / 771376 bytes in 599ms
D/ACRA    (13232): Writing crash report file.
D/ACRA    (13232): Looking for error files in /data/data/com.zegoggles.smssync/files
D/dalvikvm(13232): GC_FOR_MALLOC freed 7558 objects / 492976 bytes in 89ms
D/NativeCrypto(13232): Freeing OpenSSL session
E/AndroidRuntime(13232): FATAL EXCEPTION: AsyncTask #1
E/AndroidRuntime(13232): java.lang.RuntimeException: An error occured while executing doInBackground()
E/AndroidRuntime(13232): 	at android.os.AsyncTask$3.done(AsyncTask.java:200)
E/AndroidRuntime(13232): 	at java.util.concurrent.FutureTask$Sync.innerSetException(FutureTask.java:273)
E/AndroidRuntime(13232): 	at java.util.concurrent.FutureTask.setException(FutureTask.java:124)
E/AndroidRuntime(13232): 	at java.util.concurrent.FutureTask$Sync.innerRun(FutureTask.java:307)
E/AndroidRuntime(13232): 	at java.util.concurrent.FutureTask.run(FutureTask.java:137)
E/AndroidRuntime(13232): 	at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1068)
E/AndroidRuntime(13232): 	at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:561)
E/AndroidRuntime(13232): 	at java.lang.Thread.run(Thread.java:1096)
E/AndroidRuntime(13232): Caused by: android.database.sqlite.SQLiteException: unknown error: Unable to convert BLOB to string
E/AndroidRuntime(13232): 	at android.database.CursorWindow.getString_native(Native Method)
E/AndroidRuntime(13232): 	at android.database.CursorWindow.getString(CursorWindow.java:329)
E/AndroidRuntime(13232): 	at android.database.AbstractWindowedCursor.getString(AbstractWindowedCursor.java:49)
E/AndroidRuntime(13232): 	at android.database.CursorWrapper.getString(CursorWrapper.java:135)
E/AndroidRuntime(13232): 	at com.zegoggles.smssync.CursorToMessage.cursorToMessages(CursorToMessage.java:172)
E/AndroidRuntime(13232): 	at com.zegoggles.smssync.SmsBackupService$BackupTask.backup(SmsBackupService.java:268)
E/AndroidRuntime(13232): 	at com.zegoggles.smssync.SmsBackupService$BackupTask.doInBackground(SmsBackupService.java:159)
E/AndroidRuntime(13232): 	at com.zegoggles.smssync.SmsBackupService$BackupTask.doInBackground(SmsBackupService.java:105)
E/AndroidRuntime(13232): 	at android.os.AsyncTask$2.call(AsyncTask.java:185)
E/AndroidRuntime(13232): 	at java.util.concurrent.FutureTask$Sync.innerRun(FutureTask.java:305)
E/AndroidRuntime(13232): 	... 4 more
W/ActivityManager( 1362):   Force finishing activity com.zegoggles.smssync/.SmsSync
I/WindowManager( 1362): updateFocusedWindowLocked newFocus=Window{45fb9548 com.zegoggles.smssync/com.zegoggles.smssync.SmsSync paused=true} mode=0 mCurrentFocus = Window{45fb9548 com.zegoggles.smssync/com.zegoggles.smssync.SmsSync paused=true}
I/ActivityThread(13232): enter process activity msg = 102

Can you send through the crashes from the /data/data/com.zegoggles.smssync/files folder on your device.

Hi,
Attached is the crash log. Could you please analyze that? Thanks.

On Mon, Dec 19, 2011 at 7:42 AM, David de Caires e Freitas
<EMAIL_ADDRESS>wrote:

Can you send through the crashes from the /data/data/com.zegoggles.smssync/files folder on your device.

Reply to this email directly or view it on GitHub:
https://github.com/jberkel/sms-backup-plus/issues/185#issuecomment-3197765

--
Li Fanxi
www.freemindworld.com

I have debugged this problem on 1.4.5.
In CursorToMessage.cursorToMessages, we have this code:
        final Map<String, String> msgMap = new HashMap<String, String>(columns.length);
        for (int i = 0; i < columns.length; i++) {
            msgMap.put(columns[i], cursor.getString(i));
        }

On my phone, when i == 20, it points to the column named "sms_pdu", when doing cursor.getString the exception happens, it seems that this column is BLOB type and cannot be converted to String.
I have quickly workarounded this problem by adding try...catch around msgMap.put(columns[i], cursor.getString(i)); to ignored the columns that causes exceptions. This works fine on my phone.
I'm not sure if this problem happens on how many devices, I have two ZTE phones (U880 and N760) and they are both suffering this problem. If you think this workaround has no side impact to SMSBackup+, I suggest to include this fix in the future releases.
Thank you.


refact(getter): refactor the KS Cloud client

This PRs proposes a significant revamping of the KSCloud API HTTP client.

Interfaces are unchanged

Deprecated: low-level API funcs marked for deprecation: HttpPost, HttpGetter, HttpDelete (an augmented version of the KS Cloud client will expose the post report API, which is currently the sole use-case of low-level API)

Doc: the package is now godoc-friendly

Style & code layout:

listed all exposed types via aliases, for clarity/less confusing imports
unexported private types
factorized query param logic
factorized type <-> JSON using generic func & io.Reader
"utils" are now limited to a few common utility functions
centralized hard-coded strings as (unexported) constants
concision: use higher-level http definitions such as constants, cookie methods, etc
included type-safety guards to verify that interfaces are actually implemented by the exported types

Tests: existing test assertions are unchanged

tests are beefed-up to assert proper authentication flow (token & cookie).
added unit tests for utility methods

Perf:

unmarshalling API responses is now flowing without extraneous memory allocation via string representation
request headers are now passed withot extraneous map allocation
JSON operations are now fully supported by jsoniter (no longer use encoding/json)

Changes in functionality:

the client is now fully extensible with KSCloudOption
use the option functor idiom to keep constructors short
methods that used to mute errors (i.e. return nil, nil) now bubble up errors
the captured cookie is now captured in full, not just its value (other cookie parameters returned are stored)
added a request/response dump option, for debugging

Overview

cc @dwertent @matthyx  a proposal for a cleaner, more extensible HTTP API client (now that the tests are merged)

@matthyx @dwertent
This change aims to centralize all dealings with the SaaS API in one place, thus paving the way for a simplification of other packages (to wit, reports uploading).
I believe it allows us to better cope with future changes of the API.

@fredbi we'll merge this after Sunday's hotfix release

@matthyx

Never ever such a big PR... took me 2 days to review and I would like to perform some manual tests with our platform before merging.

Ok. Copy that. Splitting other outstanding PRs in small parts now. My apologies for having gone overboard there.

@matthyx I've address your legit concern about jsoniter imports remaining here and there.
TL,DR: I just had to upgrade my vim plugin - the automatic goimport had some flaw with go1.19+...


chore: bdd-sbhg-1588647241 to 0.0.1

chore: Promote bdd-sbhg-1588647241 to version 0.0.1

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approvers:
If they are not already assigned, you can assign the PR to them by writing /assign  in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


Project name should validate against npm guidelines (eg. no caps, url-friendly)

I just created a project and gave it an invalid name:
donejs init
$ Project name: My Project
...

This resulted in invalid paths used in index.stache:
<can-import from="My Project/styles.less!">
<can-import from="My Project/app" export-as="viewModel" />

Npm has pretty specific guidelines on naming a project - donejs init should provide some sort of validation or at least instruction on this: https://docs.npmjs.com/files/package.json

Thanks!  Can you also file this same issue in https://github.com/donejs/generator-donejs?


DUMMY PR - Mm 44529 add panel trial info in subscription page dummy pr

Summary
DUMMY PR so QA can validate https://github.com/mattermost/mattermost-webapp/pull/10482 where spinwick started failing
Ticket Link
na
Related Pull Requests
n/a
Screenshots
n/a
Release Note
NONE

spinwick worked fine back again, closing this one, is no longer necessary.


distributed collectives

really just allreduce

CI test is failing I think but still getting green check

huh, odd.

Changes made in tinygrad/:
------------------------------------------------------------
files                             insertions       deletions
------------------------------------------------------------
tinygrad/runtime/ops_gpu.py                6               3
------------------------------------------------------------
lines added in the tinygrad folder: 3

so apparently wait_for just doesn't wait under certain conditions??????

ok should be good to go now.


pytorch 1.1 support tensorboard now, may maskrcnn-benchmark support it?

🚀 Feature

pytorch 1.1 support tensorboard now, may maskrcnn-benchmark support it?

check #163


Getting adler to run on a local machine

It would be handy for development to run adler from a local machine. This should be possible via pyvo and an authentication token: https://dp0-2.lsst.io/data-access-analysis-tools/api-intro.html
I have tried this but got errors, I will ask on the community forum to try get this working. Note the Meg's student Brian managed to get this working?

Related to #31

Update, I have managed to access RSP from some local machines using pyvo. I can't get it working on my mac though so I've asked on the forum: https://community.lsst.org/t/local-access-pyvo-dalserviceerror/8511
Looks like an M1 thing maybe.


Following is showing up in frontend

I am not shure why that is so? Maybe you can help me.
The newest version is running (1.1.0) and and it is typo3 6.2.30
And that is in install tool:

It seems like javascript error. Please refer the documentation for integrating the plugin. Run the DB Analyser tasks and clear the caches. Also check the network console for any JS errors.


Minification issues on class

If i write my class like this:
class MyClass
{

}

It will add a semicolon after MyClass, so it looks like this:
class MyClass;{}

But when writing it like this, it's all fine.
class MyClass {
}

Confirmed, I will come back with a fix.

Should be sorted now, will push another release shortly.


Swift Package Manager: FirebaseAnalyticsWrapper.modulemap Umbrella directory not found

Environment

Xcode version: Xcode 12 (12A7209)
Firebase SDK version: 6.33-spm-beta
Firebase Component: Analytics
Installation method: Swift Package Manager

Problem
After installing Firebase Analytics version 6.33-spm-beta, a new warning started to appear. This warning was not present in earlier releases.
FirebaseAnalyticsWrapper.modulemap
/Users/[redacted]/Library/Developer/Xcode/DerivedData/[redacted]/Build/Intermediates.noindex/GeneratedModuleMaps-iphonesimulator/FirebaseAnalyticsWrapper.modulemap:2:10: Umbrella directory '/Users/[redacted]/Library/Developer/Xcode/DerivedData/[redacted]/SourcePackages/checkouts/firebase-ios-sdk/FirebaseAnalyticsWrapper/Public' not found

Steps to reproduce:

Integrate the Firebase Analytics SDK into a new project following the documentation for Firebase Swift Package Manager support
Observe that there is a new warning related to the path in FirebaseAnalyticsWrapper.modulemap when building the project

@aschuch Thanks for the report. I've reproduced and will work on a fix. In the meantime I believe the warning is innocuous.

Thanks for your quick support, appreciated!

Should now be fixed with "Update to latest package versions"


Issue in "Connect/create Machine Learning Workspace step"

####Error Details:
get_workspace error using subscription_id=xxx, resource_group_name=mlops-ga-rg , workspace_name=mlops-ga-ws
Message: ***
"error_details": ***
"error": ***
"code": "AuthorizationFailed",
"message": "The client 'xxx' with object id 'xxx' does not have authorization to perform action 'Microsoft.MachineLearningServices/workspaces/read' over scope '/subscriptions/***/resourceGroups/mlops-ga-rg /providers/Microsoft.MachineLearningServices/workspaces/mlops-ga-ws' or the scope is invalid. If access was recently granted, please refresh your credentials."
***
,
"status_code": 403,
"url": "https://management.azure.com/subscriptions//resourceGroups/mlops-ga-rg%20/providers/Microsoft.MachineLearningServices/workspaces/mlops-ga-ws?api-version=2020-01-01"

InnerException None
ErrorResponse

####The object id that is getting shown here does not match the sp I created
####Double checked the permission of the sp and the value of the secret in github repo
Any ideas?

Hi @RajdeepBiswas,
Can you please check whether your Service Principle was configure as described here: https://github.com/machine-learning-apps/ml-template-azure#3-setting-up-the-required-secrets


Changing projects moves Sublime from Distraction Free to Fullscreen mode

As the title suggests, if you're in distraction free mode and change projects Sublime will suddenly be in the normal fullscreen view.

I don't think there is any fix for the moment. It depends if there will be any sublime official api for opening project files.


dolphin-emu-primehack: fix name in desktop file

Description of changes
Both the regular dolphin-emu and the dolphin-emu-primehack packages install desktop files for their respective application that carry the name "Dolphin Emulator" when shown in a desktop environment. This PR changes the name of dolphin-emu-primehack to PrimeHack in the desktop file.
It may be a little contentious to rename this, since PrimeHack is essentially a Dolphin fork with some changes, so here's my reasoning:

Shiiion's fork of Dolphin is listed as shiiion/dolphin on Github, but the readme clearly calls the project "PrimeHack".
The binaries carry two different names in nixpkgs (dolphin-emu and dolphin-emu-primehack), so why name the applications the same in desktop environments?
Searching for "PrimeHack" doesn't show anything, but searching for "Dolphin" shows both.
I have both packages installed by default, and this causes them to appear side-by-side with the same name, which is just not helpful:

In light of this, I think my proposed change makes a lot of sense.
Things done

Built on platform(s)

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

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

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

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

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

[x] Fits CONTRIBUTING.md.

Small note, all of these substitutions will need to be removed once (and if) https://github.com/shiiion/dolphin/pull/118 is merged.
Until then, approved.

Small note, all of these substitutions will need to be removed once (and if) shiiion/dolphin#118 is merged. Until then, approved.

Good catch, didn't see that PR.


don't attach view on package activation

fix for #108
I think the idea behind the option "Keep visibile" would be to keep the build output visible after a build, not to have the build output panel visible at all times. Package activation happens each time a new atom window is opened and it doesn't seem useful to me to have the build panel open right away (with no output).
I could be wrong though, and perhaps another option is needed. In any case, this PR changes the behaviour to keep the panel visible after a successful build, but not immediately after package activation.

As a bonus, this shaves off almost 50ms in package activation.

This is really cool. I do realize that we are changing the behavior with this change, but I agree with you that it makes more sense to show the panel on build, and then simply keep it. There's really no real need to have an empty build show up on activation!
Thanks a lot for this!

:+1: awesome!


[Bug]: Strange condition on UserInfo

Is there an existing issue for this?

[X] I have searched the existing issues

What happened?

This condition will always be false after refactoring in #5971
Steps to reproduce?
This condition should never be entered as it will always be false, this is just a note to remove this block at some point in time.
Current Behavior
No response
Expected Behavior
No response
Relevant log output
No response
Anything else?
No response
Affected Versions
9.13.3 (alpha)
What browsers are you seeing the problem on?
Chrome
Code of Conduct

[X] I agree to follow this project's Code of Conduct

Is it just a typo here (".UserInfo" should be just ".User")?

No, basically this is the user profile module, it used to be used by both users and admins but now admins have a new experience in the persona bar. So this code was all refactored to no longer user User and only UserInfo and that block of code is basically to remove, it is right now always false and this is just a reminder for code cleanup at some point.

No, basically this is the user profile module, it used to be used by both users and admins but now admins have a new experience in the persona bar. So this code was all refactored to no longer user User and only UserInfo and that block of code is basically to remove, it is right now always false and this is just a reminder for code cleanup at some point.

Do you just have to remove the condition?

@Mostafa-Moafi basically the whole if block can go away as one will never be both a superuser and not a superuser, so that code will never ever run anyway.


Add ability to write config to file and read back in

V0:

output all config elements to a file (yaml?). ideally hierarchical, but could be flat list to start with.
ability to read the config files back into an object. Tricky: how to handle complex object, like a Pytorch module

AB#4666

Now available in hi-ml with object_to_yaml https://github.com/microsoft/hi-ml/blob/7ddb45d04a46b73d811792b65230a21c787ba9bf/hi-ml-azure/src/health_azure/traverse.py#L133


Raw Linux Installation

I tried to follow the "How to install on Linux" instructions.
a1rwulf-i5:~ # wget https://bootstrap.pypa.io/ez_setup.py -O - | python Connecting to bootstrap.pypa.io (<IP_ADDRESS>:443) Downloading https://pypi.io/packages/source/s/setuptools/setuptools-30.2.0.zip Extracting in /tmp/tmp4jgJyJ Now working in /tmp/tmp4jgJyJ/setuptools-30.2.0 Installing Setuptools running install Traceback (most recent call last):   File "setup.py", line 181, in <module>     dist = setuptools.setup(**setup_params)   File "/usr/lib/python2.7/distutils/core.py", line 151, in setup   File "/usr/lib/python2.7/distutils/dist.py", line 953, in run_commands   File "/usr/lib/python2.7/distutils/dist.py", line 972, in run_command   File "/tmp/tmp4jgJyJ/setuptools-30.2.0/setuptools/command/install.py", line 67, in run     self.do_egg_install()   File "/tmp/tmp4jgJyJ/setuptools-30.2.0/setuptools/command/install.py", line 98, in do_egg_install     easy_install = self.distribution.get_command_class('easy_install')   File "/tmp/tmp4jgJyJ/setuptools-30.2.0/setuptools/dist.py", line 479, in get_command_class     self.cmdclass[command] = cmdclass = ep.load()   File "/tmp/tmp4jgJyJ/setuptools-30.2.0/pkg_resources/__init__.py", line 2287, in load     return self.resolve()   File "/tmp/tmp4jgJyJ/setuptools-30.2.0/pkg_resources/__init__.py", line 2293, in resolve     module = __import__(self.module_name, fromlist=['__name__'], level=0)   File "/tmp/tmp4jgJyJ/setuptools-30.2.0/setuptools/command/easy_install.py", line 48, in <module>     from setuptools.py31compat import get_path, get_config_vars   File "/tmp/tmp4jgJyJ/setuptools-30.2.0/setuptools/py31compat.py", line 2, in <module>     import unittest ImportError: No module named unittest Something went wrong during the installation. See the error message above.
Seems there is a dependency on unittest.
Did I miss it in the docs?
Imho this would be worth menioning in the Installation Instructions.

unittest module comes with Python. If your Python does not have unittest, there's something wrong.
$ python2
Python 2.7.12 (default, Nov  7 2016, 11:55:55)
[GCC 6.2.1 20160830] on linux2
Type "help", "copyright", "credits" or "license" for more information.
>>> import unittest
>>> unittest
<module 'unittest' from '/usr/lib/python2.7/unittest/__init__.pyc'>

Mhm, thought about that - should have mentioned that I was installing on some custom linux, that is all about being as lightweight as possible.
Sorry for the noise.


Bug 2111979: Set openshift.io/run-level to nil in openshift-controller-manager nam…

…espace

/retest-required

/test e2e-upgrade

/assign @gabemontero

/assign @adambkaplan

/assign @coreydaley
@jkhelil Corey is an approver in the owners file, so he can apply the approve label.  Neither Adam no I are approvers in the owners file.
FWIW I have no familiarity with the run-level label, so have no expertise to lend to this change.

/approve
/assign @bparees
for lgtm

/lgtm


would be nice to be able to collapse the list of usr sequences

in order to be able to get the tree and MSA viewer to be adjacent. alternatively, one could imagine going golden layout and giving the user the freedom to arrange the components as they see fit.

"Five Go Mad with GoldenLayout"?

It looks like it would be complicated to get Shiny and Golden Layout to work together. How about just toggling the highlighted features?
http://dev.lis.ncgr.org:50080/shiny/Funnotate/?job=P1X4F&family=L_4F5ZWF

yes, that's fine; but there will be no lashings of ginger beer distributed.

Not until we also hide the Vertical/Radial layout buttons and "You have focused on a subtree" elements?


A WebComponent build

I love what you've done here. 😊
If it is possible to build a framework-agnostic version (WebComponent), that would be lovely. I found an unvetted guide here on DEV.to by @sil-vio. I know https://github.com/ivosdc/svelte-datepicker by @ivosdc has both a Svelte and a WebComponent build so it might be worth looking at.
The advantage would be to be able to use this without any build process, and without including a Svelte build pipeline.
I don't have any Web Component or serious Svelte experience, but I hope this issue stays open for anyone else in the community to tackle. 😉

I'm happy for somebody to do this is they feel the need. I don't use WebComponents or really think they're particularly desirable, so I'd be equally happy to just output a vanilla Javascript version of the module too :)


ESP32-S3 Support

Version 2.0 of ZumoComSystem is expected to use an ESP32-S3 as its microprocessor. The main reason for this is due the USB-Host capabilities of the chip, removing the necessity for an external USB-Host IC as has been done until now.

[ ] USB Host: ACM component
[ ] Pinout
[x] IDF issue must be fixed first https://github.com/espressif/esp-idf/issues/12336

Pinout
v1.1 Configuration (old)

PIN_WIFI_AND_RESET_KEY = 4
PIN_DEVICE_RESET = 27
INFO_LED_R = 16
INFO_LED_G = 22
INFO_LED_B = 21
PIN_BATT_MEASUREMENT = 35

v2.0 Configuration (new)

PIN_WIFI_AND_RESET_KEY = 48
PIN_DEVICE_RESET = 5
INFO_LED_R = 38
INFO_LED_G = 1
INFO_LED_B = 2
PIN_BATT_MEASUREMENT = 10
LED_A = 6
LED_B = 16
LED_C = 18
Button_A = 7
Button_B = 15
Button_C = 17


live trade??

This Bot live trade?? or only show proposals??

Only shows proposals. Live trading is not added


Cannot edit free redemptions/tradebacks entries

Apparently those entries cannot be edited.

This should be fixed.


Escape commas for Emergency Banner Deploy task

Rake is not very good at handling arguments with commas in. It has
surprising behaviour, such as:
rake test[‘foo, bar’] will give you two arguments: foo and bar.
If you escape the comma, then they are provided as a single argument
with a comma: foo, bar.
This change escapes the commas in the provided arguments, in order
to reduce human error when manually escaping. Since the command is
executed inside speech marks, this change also escapes quotes.
The new task has been tested with arguments whose commas and
quotes were not manually escaped:

We had a trello card for this: https://trello.com/c/xAbDe1EK/232-the-emergency-banner-does-not-display-correctly-if-the-details-contain-a-comma


update various dependencies

This is a followup for https://github.com/lsd-rs/lsd/pull/938.

update dirs from v4 to v5 (no code changes)
update term_grid from v0.1 to v0.2 (small code changes)
update terminal_size from v0.1 to v0.3 (no code changes)
update lscolors from v0.15 to v0.16 (no code changes)
update serde_yaml from v0.8 to v0.9 (no code changes)

The term_grid update required small changes to the use of term_grid::Cell because with v0.2, it is possible to specify the alignment within the cell. Adding Alignment::Left should preserve the current behaviour.

Hi @decathorpe, thanks for the update, as you can see CI, once we upgrade the dirs dep to v5, the Windows build will fail.
can you help to fix it?

I briefly looked at the failure, and it seems to be a toolchain issue? I have no idea why updating Rust code should cause linker failures.
If it's dirs v5 that causes the issue, I can drop that one from the batch and we can look into the failure separately.

that would also be okay to leave dirs in a separate PR

I've dropped the dirs v4 → v5 update for now.
One thing that looks suspicious is that with v5, it now uses windows-sys instead of winapi ...

I tried something else, maybe it changes things, maybe not. 🤞🏼

Looks like good ol' cargo update fixed it. Judging by the previous contents of Cargo.lock, it hadn't been done in years, so it's not surprising that it pinned some crates to versions that were missing fixes.
(After cargo update I made small adjustments to keep the MSRV at 1.70 - clap v4.5 bumped its MSRV to 1.74, so I kept it at v4.4.)

thanks so much for the update!

Great, thanks!


Where are the templates?

The documentation (https://doc.micrium.com/display/kernel304/Conventions) says there are templates provided. I do not see any µC/CPU templates after downloading.

Please check the following link.


add cors documentation for the shell app

https://osc.github.io/ood-documentation-test/add-cors-1.6
OOD_SHELL_ORIGIN_CHECK docs for release 1.6.

I'll cherry pick from 1.7 once it is merged. There are more changes that are required. Closing this.


[globalopt] Use std::reverse instead of trying to reverse an array by…

… hand.
I also changed a dyn_cast to a cast since we assume here that the cast can never
fail.

@swift-ci smoke test and merge


Not working for Win 8.1(ver: 6.3.9600)

Hi,
I am getting the following logs and tested that it isn't hiding properly:
[TITANHIDE] Device \Device\TitanHide created successfully!
[TITANHIDE] Symbolic link \DosDevices\TitanHide->\Device\TitanHide created!
[TITANHIDE] SSDT RVA: 0x35CA80
[TITANHIDE] SSDT not found...
[TITANHIDE] SSDT RVA: 0x35CA80
[TITANHIDE] SSDT not found...
[TITANHIDE] SSDT RVA: 0x35CA80
[TITANHIDE] SSDT not found...
[TITANHIDE] SSDT RVA: 0x35CA80
[TITANHIDE] SSDT not found...
[TITANHIDE] HooksInit() returned 0
[TITANHIDE] HiderProcessData OK!
[TITANHIDE] HiderProcessData OK!
[TITANHIDE] HiderProcessData OK!
[TITANHIDE] HiderProcessData OK!
[TITANHIDE] HiderProcessData OK!
[TITANHIDE] HiderProcessData OK!

I have ran the TitanHideTest and used TitanHideGUI to hide that process. Then I attached a debugger on it and every debugger check returned 1.
Did Windows update 8.1 recently or something?
Currently I am creating a Windows 7 dual boot to see if it works there.

This is weird, because the RVA is actually correct. Or at least it matches the one on my Windows 8.1 VM, so I assume it's the same kernel. It's working for me though. Do you have a kernel debugger? Set a bp on SSDTFind and see where it's returning NULL (the reason for the "SSDT not found" message). Also use kb to see the call stack, because it looks like SSDTFind is working the first time it's called but not the second for some reason.

No response from @koemeet so I'll close it for the time being...


ILI9488 display on ESP32S3 initializes, but has no image

Using one of these ILI9488 based boards with this hardware_setup.py.  After importing gui.demos.simple, the screen is definitely reset as the backlight will dim a bit, but no graphics are displayed.  Touch does work, however.  The 18bit versions of the ILI9486 drivers for Arduino seem to work, so possibly it's related to that?  Any pointers on things to test would be appreciated.

It's hard to support hardware I don't possess.
Your SPI connections look correct, but it's always worth trying soft SPI. It's also worth trying the test script as it can indicate the cause of errors - however in your case this is a long shot as the display is blank. Another long shot is to issue
SSD.COLOR_INVERT = 0xFFFF  # Fix color inversion

in hardware_setup.py after instantiating SSD. This has proved necessary on some ILI9488 displays.
As a general comment, different display manufacturers connect the LCD to the display controller in different ways, requiring corresponding firmware changes. Implementing these can be difficult. However a completely blank display is uncommon and usually signifies a wiring error or other hardware problem.

Thanks for the feedback and totally understand on the supportability.  I modifed the interface pixel format from 0x55 to 0x66 to match the Arduino driver and got partial functionality:

Now that there is actual output, it should be easier to make progress.

After fixing the screen dimensions and trying the test script, something is still amiss. Suspecting it has to do with the color format being RGB565 vs RGB666 causing not enough bytes being sent to fill the framebuffer, but still digging.

You definitely need to set the chip up for rgb565. 18-bit mode would involve substantial modifications to the driver.
The color errors where a single pixel is displaying as multiple is unusual - perhaps the display is in 18bit mode? As a general point, if you can get the test script to produce pixel-perfect results, you can be highly confident that everything else will work.

Yep, that would be best, but it seems the display will only fully come up when set for 18-bit.  It is mostly working now with a much simpler driver, but the performance is awful since it writes directly.  The viper code in your driver is much faster, but I haven't fully grokked how the color and framebuf lookup stuff is working to adapt it.

Added a function to convert the color layout before writing to SPI.  Largely lifted from your style to keep it in viper.  Some issues with colors not rendering properly, but this bodge has it pretty close.

Well congratulations on getting it working, but it's very odd that RGB565 doesn't work. I've never heard of this before - I wonder if the chip is a clone?

If I understand the datasheet correctly, the driver chip can work with a number of different displays, with various connection and configuration options to support them.  I tried all of the different possibilities just to see what would work and with this module, only the one option does.  In researching this issue, there are others running into it, so it will be nice to have a solution.

I find the failure to work in 16 bit mode very surprising.
If you find a full solution please let me know: I could put a link to this issue in the drivers doc.


Update nixpkgs+ghc

Update nixpkgs to 23.11 and ghc to 9.6.4, inline with https://github.com/runtimeverification/haskell-backend/pull/3719

I believe we will have to also upgrade the K dependency in this PR, but IIRC to a custom version that adjusts the nix setup to avoid problems with our integration test :thinking:  ?


Maybe you can add Nihlatak's Hall of Pain layout?

Hi there,
did a Rework for "Halls of Pain" layouts in D2R Style.
Maybe you can add this to your Cheatsheet?

Halls of Pain Layout #2  is called  Layout 3/3 instead of 2/3. ;)


[Enhancement] Set priority of i/o task in pipeline by submitted times

What type of PR is this：

[ ] bug
[ ] feature
[x] enhancement
[ ] refactor
[ ] others

Problem Summary(Required) ：

Set priority of an i/o task in pipeline according to the number of submitted tasks by the scan operator.
For the inner scan i/o task queue in a workgroup, also replace FifoScanTaskQueue with PriorityScanTaskQueue.
Besides, make a scan i/o task yield, when it already costs 100ms.

run starrocks_admit_test


Ml5.js and face-api.js are not available at the same time.

I wanted to try face-api.js and installed it. But when I import both ml5.js and face-api.js at the same time, I have a problem. So I cloned https://github.com/ml5js/ml5-library/ And I tried to build, install and use it via yarn build myself but I get the same problem.

This error comes from ml5.js so I am not sure why this error occurs. My guess is that the tfjs-core versions of your ml5.min.js script and the version used by face-api.js are conflicting?

Closing because of inactivity.


Replace rustc-serialize with serde

rustc-serialize is deprecated in favor of serde, so I make this PR.

well, 3-alpha branch just seems fine. I seem to have done a meaningless job : P
Closed