[BUG] when selecting text then replacing new text

I'm not familiar with acode version 1.8.6,
and two versions before it.
I have experienced 2 updates before, and in version 1.8.4 I have difficulty writing code.
I don't know if this is a bug or just an additional feature but it's not finished.
but I think this is a bug, because for me this is an oddity.
I use HP Oppo A53, Android 12 [Acode Premium], Please fix this BUG immediately 👍
Video Detail BUG

you can go into the setting and change keyboard mode to no suggestions or learn to properly end the keyboard suggestion with a space, symbol, enter or by pressing the word for now
This seem to be an issue with keyboard suggestion, there also other weirdness with it like the following.
tapping on 'randomish' part of the editor and pressing a button
https://github.com/deadlyjack/Acode/assets/4752645/d04a19cb-a449-414e-b7cb-e8e036c429db
alternating between enter and backspace
https://github.com/deadlyjack/Acode/assets/4752645/b0904ae4-680e-4821-b8f0-96ffb7493a17

you can go into the setting and change keyboard mode to no suggestions or learn to properly end the keyboard suggestion with a space, symbol, enter or by pressing the word for now
This seem to be an issue with keyboard suggestion, there also other weirdness with it like the following.
tapping on 'randomish' part of the editor and pressing a button
Screenrecorder-2023-08-31-06-42-17-478.mp4
alternating between enter and backspace
Screenrecorder-2023-08-31-06-52-10-404.mp4

wow, thanks for helping with this problem, I hope this Acode Editor will continue to develop well and can help someone who doesn't have a computer

I don't think this should be closed as this is a fairly buggy behavior that still exists
@deadlyjack I believe the following behavior would be desirable

completion/suggestion should ends on touch/input
when backspace delete a newline, it shouldn't try to do completion/suggestion

Does that mean this is a fairly serious bug? I'll help reopen this issue so that other developers can hear or listen to it


Add a reverse ll program

Pull Request Template
What have you Changed(must)
Write here what you have changed in codebase
Issue no.(must) -
Pr will be closed and marked as spam. If issue number not found or issue was assigned to someone else.
Marking as spam can block your account from HacktoberFest.
Self Check(Tick After Making pull Request)

[x] This issue was assigned to me if not it will marked as spam.
[x] My file is in proper folder
[ ] I am following clean code and Documentation.My code is well linted and I am using flake8 standard for python
[x] I have added title and what program will do first in the file

README - How to Contribute

please merge my file I have added in C language

please merge this pr


✅ Implement tzinfo.fromutc method for TzInfo in pydantic-core

Change Summary
This PR adds tests for https://github.com/pydantic/pydantic-core/pull/864
Related issue number
Fix #6735
Checklist

[x] The pull request title is a good summary of the changes - it will be used in the changelog
[x] Unit tests for the changes exist
[x] Tests pass on CI
[x] Documentation reflects the changes where applicable
[x] My PR is ready to review, please add a comment including the phrase "please review" to assign reviewers

please review


The golang "true" is 0 ,but Rust "true" is 1

The int variable value of the bool variable in Golang language is different from that of rust。
In golang
// bool is the set of boolean values, true and false.
type bool bool
// true and false are the two untyped boolean values.
const (
true  = 0 == 0 // Untyped bool.
false = 0 != 0 // Untyped bool.
)

I'm not sure I understand this.
KillRequest is protobuf generated structure and boolean all field.
Where do you see it inverted?

In 'task.rs' kill function  ,    there is    "  info!("Kill request for {:?}", req);"     this way can print result .     but when you send the KillRequest in containerd ,
when ctr set all "true"   but shim get false ,  I find it in test function "TestContainerDrainExecIOAfterExit " ， Maybe I was wrong, But I feel confused , ctr send true ，but shim get false . so I think golang deal with bool  "0" represent true .

Can you provide more defails, such as the log msg of info!("Kill request for {:?}", req);? It has a long way between ctr and task.kill(), I think it is not a bug of rust shim.

this is containerd's problem I will bring issus  to containerd


Multiple CSProj files - some locked some not. how to score?

Looking at a repo such as this where there are three csproj files, one of them is leveraging the RestoreLockedMode feature, what do we expect?

A relative score (i.e. 1/3 nuget dependencies are pinned)
A zero score (i.e. unpinned dependencies found in the repo)
A positive score (i.e. dependencies are pinned)
Any other thought?

Update in daily: Current implementation is to have a relative score (1/3 nuget dependencies are pinned)

"pinned dependencies" generally refer to specifying exact versions of dependencies in your project's configuration files (e.g., package.json, requirements.txt, csproj, etc.)
the 1/3 projects having a lock file I think might be an ADDITIONAL check ontop of that requirement.
thus my knowledge here, this project would be 3/3 given each .csproj has a pinned dependency.
we may need to improve detection if we only check lock files. lock files can be used separately to pinned versions and would account for the same "check" if my knowledge is correct here.

If it were me, I would use an all or nothing approach. Supply chain security is as good as the weakest link.


There can be only one variable named 'where'

Describe the bug
Duplicate variable name for $where when querying an array field that references a type.
Error: GraphQL error: graphql: There can be only one variable named 'where'. (line 1, column 12):
query call($where: UserWhereInput, $where: CallWhereUniqueInput!) {
           ^
 (line 1, column 36):
query call($where: UserWhereInput, $where: CallWhereUniqueInput!) {
                                   ^

To Reproduce
example datamodel.prisma:
type User {
	id: ID! @unique
	credit: Int! @default(value: 0)
	createdAt: DateTime!
	updatedAt: DateTime!
}

type Call {
	id: ID! @unique
	source: User! @relation(name: "UserCalls")
	guests: [User!]! @relation(name: "UserCallsReceived")
	createdAt: DateTime!
	updatedAt: DateTime!
	endedAt: DateTime
}

example prisma.yml:
endpoint: "http://localhost:4466"
datamodel: datamodel.prisma
generate:
  - generator: go-client
    output: ./generated/prisma-client/

Steps to reproduce the behavior:

Create a new GoLang project
Run prisma deploy
Run prisma generate
Try to run the following with prisma generated packages imported:

u := getCurrentUser(ctx)
users, err := prismaClient.Call(prisma.CallWhereUniqueInput{ID: &callId}).Guests(&prisma.GuestsParamsExec{Where: &prisma.UserWhereInput{ID: &u.ID}}).Exec(ctx)
if err != nil {
	log.Fatal(err)
	return
}

Expected behavior
Run the query without error and return a slice of Users.
Screenshots
If applicable, add screenshots to help explain your problem.
Versions (please complete the following information):

Connector: Postgres
Prisma Server: 1.25
prisma CLI: prisma/1.26.6 (darwin-x64) node-v8.12.0
OS: macOS Mojave 10.14.3
other dependencies:

Additional context
It seems to me that this is a problem with query generation.

@raeesbhatti
I have created a PR to fix this:
https://github.com/prisma/prisma-client-lib-go/pull/7
While it is not merged, feel free to use my fork:
https://github.com/voluzi/prisma-client-lib-go


Can't build on OGRE 2.x

The latest commit on ogre-next branch prevents MeshMagick from build with OGRE 2.x.

Some using are missing (e.g. v1::HardwareIndexBuffer)
using isn't probably enough by itself in every case because of ambiguous symbols (e.g. here)
Some v1 namespace prefixes have been lost (e.g. here)

should be fixed now


When no virtualenv is activated, vf all doesn't restore state

If you're not in a virtualenv, and then you run vf all (or allvirtualenv) you're left in a state where whatever the last virtualenv you have is left activated.

Hi Alex, Thanks for letting me know! I'll push up a fix now.


Firfox Horizontal Nav Bug

In Firefox the Horizontal nav is taller than the list inside which is causing extra space underneath the list, you can inspect it here: https://pf4.patternfly.org/demos/Page/examples-full/?component=Page component horizontal nav example#
This is also causing a bug in the Patternfly website: https://github.com/patternfly/patternfly-org/issues/865

Also noticed this when the nav breaks to a mobile layout in firefox, the nav doesn't overflow properly.

Also this misalignment

:tada: This issue has been resolved in version 2.6.15 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Add -fPIC compiler flag

Hi,
Is there any reason the -fPIC flag isn't added in the library's CMakeLists.txt?
set(CMAKE_C_FLAGS "${CMAKE_C_FLAGS} -fPIC")
Cheers,
Jan

When building shared libraries, CMake adds -fPIC by default. For static libraries, most of the time you probably don't need position independent code anyway.

ARM Internal Ref: IOTSSL-711

mbed TLS is used in many different ways on many different platforms, both as as a static library and a dynamic library, so I don't think we want to include much in CMAKE_C_FLAGS as that's too inflexible.
If you look at the Makefile we ship with mbed TLS you'll see we include -fPIC when building the shared library, and if you use CMake, as @jcowgill has already said, CMake includes the option for the shared library target.


Pass zipfile as arg to onEntry

Fixes #32.

@maxogden - Can you think of any good reason not to merge this?

added docs and released as 1.6.0, thanks

thanks!


before_info callback for priv checks and configuration for tools/nodes

Some testing should be done, unit tests would be fairly easy to implement and very good for stuff added/changed here.

Added unit testing with busted, after I've also added few more tests then that should be enough for things around tool/node settings (privileges and definition.settings table).

Basic tests for configuration merging done and config merging was also fixed / updated while writing tests.
Some basic tests for tool registration and node registration done.
Little bit cleanup done.
Changes are getting pretty wide, unit testing ensures few things but this still needs some in-game testing through all basic tool functionality and to double check that unit tests is done correctly:

sharetool

[ ] cannot be used without privs
[ ] protected books
[ ] protected travelnets

tubetool

[ ] formspec works correctly
[ ] protection works correctly
[ ] tp tube copy/paste
[ ] sorting tube copy/paste

luatool

[ ] protection works
[ ] copy/paste luatube

Everything seems to work now, a lot of cleanup, some unit testing, tool API is a lot cleaner and everything is prepared for even more cleanup.


Add markdown style guide to CONTRIBUTING

This PR adds a Markdown style guide to the CONTRIBUTING.md. I've lifted the bullets from @ralfhandl 's PRs on style fixup and added the .markdownlint.yaml file from @lornajane 's PRs for markdown updates.
There may be other things to add but I think this is a pretty good start and since we are already acting on these I thought we should make sure we have consensus and then capture that in the docs.

We should add the rules for "property" vs "field" vs "keyword" in #4020

In the TDC meeting we discussion that we should be using  instead of the  format for link anchors - it's been deprecated.

Can I merge an "editorial" PR with just one approval?

Unfortunately we haven't yet resolved

#3581

Btw., I forgot that you want to add .markdownlint.yaml in this PR in the project root.
In PRs #4040, #4041, and #4025 I add it in the versions folder, I can clean that up once this is merged.
Would you mind to update this PR and
npm i -D markdownlint-cli

so that we have the tool to interpret .markdownlint.yaml as a dev dependency?

Resolved conflict in package.json, refreshed package-lock.json

@mikekistler I was intending to actually look over this by Thursday, but I have no objection to you merging it as we can always tweak it later.

Refreshed package-lock.json due to action errors in my fork


typo in query 2

Hi, the query
SELECT count()
FROM PARTITION BY PartitionId
GROUP BY PartitionId, clusterid, tumblingwindow (minutes, 5)
is missing an input name, I think it should be
SELECT count()
FROM input PARTITION BY PartitionId
GROUP BY PartitionId, clusterid, tumblingwindow (minutes, 5)

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

ID: da35754e-933e-c93f-96e4-415c89cd3dca
Version Independent ID: 375300d6-9bed-ad51-307c-5c2e5de92ee8
Content: Understand and adjust Streaming Units in Azure Stream Analytics
Content Source: articles/stream-analytics/stream-analytics-streaming-unit-consumption.md
Service: stream-analytics
GitHub Login: @JSeb225
Microsoft Alias: jeanb

@SqlWaldorf Thanks for the feedback. We are actively investigating and will get back to you soon.

@SqlWaldorf That's a good catch, fixing this typo right now.

Thanks for your contribution! The typo has been fixed. #please-close


findOrFail() works like find() when array provided

For some reason, findOrFail() method doesn't throw exceptions when array of IDs is provided. For example, if I have a Animal model:
id | name
-----------
1  | Dog
2  | Cat
3  | Turtle

Those are the results of various find() / findOrFail() combinations:
// single existing
Animal::find(1);       //=> instance of Animal
Animal::findOrFail(1); //=> instance of Animal

// single missing
Animal::find(4);       //=> NULL
Animal::findOrFail(4); //=> ModelNotFoundException

// multiple existing
Animal::find([1, 3]);       //=> Illuminate\Database\Eloquent\Collection with 2 elements
Animal::findOrFail([1, 3]); //=> Illuminate\Database\Eloquent\Collection with 2 elements

// multiple mixed
Animal::find([1, 4]);       //=> Illuminate\Database\Eloquent\Collection with 1 element
Animal::findOrFail([1, 4]); //=> Illuminate\Database\Eloquent\Collection with 1 element

// multiple unexisting
Animal::find([4, 5]);       //=> Illuminate\Database\Eloquent\Collection with 0 elements
Animal::findOrFail([4, 5]); //=> Illuminate\Database\Eloquent\Collection with 0 elements

I believe it would be more intuitive, if findOrFail() threw ModelNotFoundException if any of requested ID in the array doesn't exist.

@kokokurak now that #7048 has been merged, has this been fixed?

Not working @4.2

Not working @4.2

4.2 is not supported. Upgrade to 5.x, and you'll find this is fixed. :)

@GrahamCampbell thanks, good to know that :+1:

By the way, I have tested it after the merge and it works now. Sorry for not telling :)


exposing BaseName on main naming namespace

Coverage decreased (-0.2%) to 99.833% when pulling 0bcd283c7e12c02c10e6954dce3a43e74e4145fe on hotfix/0.2.1 into ed0efbd2a3718f977c01cc15b33aeb1aa4fb299c on master.


Help command

Saw some dude griping about the name command not being intuitive. Didn't know what direction you wanted to end up with that, but at least a /help command will benefit the ux :)

what if the placeholder was /nick name

That may not always be the only command users can issue in the chat.

i still feel like a /nick name placeholder is fine. /help can be added if we ever get more commands

Fair enough.


false positives

Hi there,
I've been looking into adding your list to my auto updated dnsmasq filter, though I found quite a lot of possible false calls.
You might want to consider removing the following items:
vmall.com
taobao.com
toplinkxchange.com
shellcat.ru
sigidwiki.com
jappy.tv
kingsoft.com
skyrock.mobi
skyrock.net
pwnews.net
rusnext.ru
nianticlabs.com
nikopik.com
nordpresse.be
nrj.fr
minimilitiahackcheats.com
magicbricks.com
launchforth.io
ksosoft.com
hideipaddr.com
fairtop.in
fedsig.com
fireeye.com
dailymaza.co
delcomeyersdale.com
columbiarecords.com
birddl.com
azadmob.in

@notracking Thanks for reporting!


Validate Thingy:91 with all samples

This pull request collects the changes needed to make the Thingy:91 work with all Golioth samples.

I've restructured this PR to isolate the changes related to adding Thingy:91 support.
This PR is rebased on #169 which should be merged first.

@szczys Can you rebase this PR, so that merge conflicts are resolved?
One of the things I am missing now is build tests for this platform. We do have build tests for all other platforms, so that we don't accidentaly break them.
Additionally, we there is a Device Matrix to be updated in main README file.
Last, but most important, are you commited to testing this platform regularly and maintaining support for it? IMO we already have too many platforms with no automated runtime tests, so I don't want to take that on me :)

Can you rebase this PR, so that merge conflicts are resolved?

Yes, done!

One of the things I am missing now is build tests for this platform. We do have build tests for all other platforms, so that we don't accidentaly break them.

I haven't looked into these but with some guidance I'm happy to work on this.

Additionally, we there is a Device Matrix to be updated in main README file.

I can work on that. Is the SDK version the hash for this commit once it's merged? Where do I get golioth version info?

Last, but most important, are you commited to testing this platform regularly and maintaining support for it? IMO we already have too many platforms with no automated runtime tests, so I don't want to take that on me :)

Sure? I agree that autoated runtime tests are important and we need to solve that. For now, I can run the tests on this device when needed.

I made an update to the device matrix and added yaml entries in the samples for thingy91_nrf9160_ns.

However I think that we should assume that users understand west when working with SDK repo directly (as opposed to following docs repo).

It would be better assume the user understands enough to remove the -p. This is a possible tripping point when trying out Golioth. I was confused when I first encountered the need for a pristine build, especially since there is no west clean command (I'm used to typing make clean).
But, we can debate this another time. I'll remove these and push a commit so we can get this PR merged. Thanks!

@szczys Thanks a lot!


[SPARK-49703] Publish Java 21 Docker image for preview1

What changes were proposed in this pull request?
This PR aims to publish Java 21 Docker image for preview1 and will be extended for preview2.
Why are the changes needed?
Apache Spark supports Java 21 via SPARK-43831.
Does this PR introduce any user-facing change?
No, this is a new image.
How was this patch tested?
Pass the CIs.

Thank you, @HyukjinKwon .

For the record, while preparing 4.0.0-preview2 release, I'm making this repo up-to-date by re-implementing the Apache Spark patches like the following. The following three are published to https://hub.docker.com/r/apache/spark/ .

#66
#67
#68

Thank you, @viirya .

Thank you, @Yikun .

Thank you, @HyukjinKwon , @viirya , @Yikun .
Merged to master.

For the record, Java 21 images are published successfully.

https://github.com/apache/spark-docker/actions/runs/10934045296
https://hub.docker.com/layers/apache/spark/4.0.0-preview1-java21/images/sha256-5e0bb8becda30b59fcbd712dfac99aad22d50683c0df986a77f3db7c9d78e5e6?context=explore


Enforce branch policies on this repository

To improve the security of the ORAS project we need to enforce the branch policies for this repository. I propose that we enforce the policies as follows:

Use the following rules for main and release/* branches:

Require PR before merging

[ ] Require 3 approvals
[ ] Dismiss stale PR approvals when new commits are pushed
[ ] Require review from Code Owners
[ ] Require status checks to pass before merging
[ ] Require conversation resolution before merging
[ ] Require signed commits
[ ] Do not allow bypass the above settings

Please add your comments and proposals for additional changes to this issue.

Few comments:

release/* does not apply to libraries.
"Require 3 approvals" is not applicable to this repository since we only have 3 code owners.
Additionally, we require branches to be up to date before merging, which is useful but not captured in this issue.

It is worth noting that "require branches to be up to date before merging" somehow conflicts with "dismiss stale PR approvals when new commits are pushed".

I've updated the branch policies here to be similar to other oras libraries.


Authentication failure AAD

Given I have 2 Azure Active directories:

mycompany.onmicrosoft.com - domain verified as "mycompany.org.au"
myazurehosting.onmicrosoft.com

I have a user account "me" in the directory "mycompany.onmicrosoft.com", my user name is<EMAIL_ADDRESS><EMAIL_ADDRESS>is also an external "member" user in the directory "myazurehosting.onmicrosoft.com"
I have a number of Azure SQL instances in subscriptions associated with the "myazurehosting.onmicrosoft.com" tenant.
When I try to create a new connection to an Azure SQL Server I:

Enter my-servername.database.windows.net
Authentication type - Azure Active Directory - Universal with MFA support
Account - Select "Add Account" and follow the process to sign
Azure AD Tenant - "myazurehosting.onmicrosoft.com"
Database - "mydatabase"

When I press connect the error "Login failed for user 'NT AUTHORITY\ANONYMOUS LOGON' is shown
I have tried '' and 'master' as the database and get the same error.
The connection can be made without issue with SQL Server Management Studio

Thanks for the report! This seems to be a dupe of #4085 so closing this one - you can follow the linked issue for updates.


[FEATURE] Support Clang compilation

My testing indicates that compiling with Clang gives a noticeable (on the order of 10%) speed improvement - likely because it optimizes the Eigen matrix code a bit better than the VS compiler.
It probably makes sense not to switch the VS project to Clang, since it will make it harder for people to compile out-of-the-box (most people won't have Clang installed). I think it can be specified in the release build process, though.
The only issues I encountered with building the plugin with Clang were a couple of minor iPlug2 things that seem to have already been fixed - but in a later version of iPlug2 than the fork the plugin is currently using.

I am not sure why NAM is using a non-fork fork of iPlug2 :)

I think this can be closed since the repo now tracks the iPlug2 repo, where https://github.com/iPlug2/iPlug2/commit/dbdf118fc8273eba0241ffe90202e415de4968a4 is merged

I think this can be closed since the repo now tracks the iPlug2 repo, where iPlug2/iPlug2@dbdf118 is merged

That should hopefully enable clang support, but we still don't actually use it. If it has a significant performance impact (as it seems to), we should consider using it for the build.

i think this can be closed now?

@sdatkinson Can you re-open this?
I think it is worth looking into having an official clang build. Recent experiments with clang on the LV2 plugin are showing around a 30% performance improvement for standard WaveNet models:
https://github.com/mikeoliphant/neural-amp-modeler-lv2/discussions/51

PR here that gets a start on this: https://github.com/sdatkinson/NeuralAmpModelerPlugin/pull/383


docs and docx regex clashes

$ mkdir -p test-regex-clash && cd test-regex-clash
$ drive new --mime-key docs a1 a2 a3
$ drive new --mime-key docx b1 b2 b3
$ for i in $(seq 1 10)
> do
>   drive list --match-mime docs
> done

That undeterminastically produces alternating results between a1 a2 a3 and b1 b2 b3

Fixed by PR https://github.com/odeke-em/drive/pull/501


Release notes for 5.0.1

According to npm there has been a new release 5.0.1 and I wonder why and what has changed, as there is no pull request in the repository matching this release.
Also, why has the version 4 been skipped?
https://www.npmjs.com/package/nuxt-directus?activeTab=versions

Ah I see, there is a changelog file within the package, so I'll close it :)

@mklueh There was a Version 4.0, regarding to a breaking change the version was bumped to a new major version.

@Intevel, The documentation website needs to be updated?

@casualmatt Im working on a completely new documentation.

Do you need any help, I'm not that grate of a dev but i can help in writing.

Sure, send me a message on Discord!


cabal install hangs

ArchiCurry@AdaWorkPlace-TGQ35IA:~/.emacs.d/structured-haskell-mode$ cabal install
Resolving dependencies...
Configuring structured-haskell-mode-1.1.0...
Building structured-haskell-mode-1.1.0...
Failed to install structured-haskell-mode-1.1.0
Build log ( /home/ArchiCurry/.cabal/logs/ghc-8.2.2/structured-haskell-mode-1.1.0-HknZ85FckEP1czqhIT4oBS.log ):
cabal: Entering directory '.'
Configuring structured-haskell-mode-1.1.0...
Preprocessing executable 'structured-haskell-mode' for structured-haskell-mode-1.1.0..
Building executable 'structured-haskell-mode' for structured-haskell-mode-1.1.0..
[1 of 1] Compiling Main             ( src/Main.hs, dist/build/structured-haskell-mode/structured-haskell-mode-tmp/Main.o )
src/Main.hs:165:18: error:
• The constructor ‘Deriving’ should have 3 arguments, but has been given 2
• In the pattern: Deriving _ ds@(_ : )
In the pattern: Just (Deriving _ ds@( : ))
In a case alternative:
Just (Deriving _ ds@( : _))
-> [spanHSE
(show "InstHeads")
"InstHeads"
(SrcSpan
(srcSpanFilename start)
(srcSpanStartLine start)
(srcSpanStartColumn start)
(srcSpanEndLine end)
(srcSpanEndColumn end)) |
Just (IRule _ _ _ (IHCon (SrcSpanInfo start _) _)) <- [listToMaybe
ds],
Just (IRule _ _ _ (IHCon (SrcSpanInfo end ) )) <- [listToMaybe
(reverse ds)]]
|
165 |            Just (Deriving _ ds@(:)) ->
|                  ^^^^^^^^^^^^^^^^^^^
cabal: Leaving directory '.'
cabal: Error: some packages failed to install:
structured-haskell-mode-1.1.0-HknZ85FckEP1czqhIT4oBS failed during the
building phase. The exception was:
ExitFailure 1

what can i do  ?

Looks like it needs an update to cope with the latest version of haskell-source-exts.

structure-haskell-mode  & intero is pretty much the same isnt  ?

intero was done from scratch from what @chrisdone learnt from doing structured-haskell-mode IIUC.

Closing in favour of #156


Best way to get IP address and subnet mask?

For IP scan, it needs the IP address and the subnet mask to decide what IP range to scan on. The runnable jar is abstracting those details by running command 'netstat -rn', I am not 100% sure it works for all Linux systems.
The alternative approach is using Java network interface API, but there is no good way to tell which network interface to choose. In Evercam Discover Android App, it's on WiFi network by default and I added an option in Settings to let the user select network interface (for example, wlan0, eth0 and so on).
Any suggestions around this? My questions basically are:

Does command 'netstat -rn' work on all our gateway device and is it always Debian on the gateway device?
Do we want to select network interface manually, or scan both of WiFi and Ethernet?

@scandox @marcoherbst

@liutingdu I had this issue also with the Gateway discovery. My solution was to enumerate all interfaces (i.e. all Network cards, wireless cards etc) and run discovery on all of them. After all - a device might be connected to numerous networks.
So specifically answer to your questions are:

Yes it probably does, but we should not use it. Much better to use Java Network APIs...
Scan all network interfaces (even if multiple ethernet and WiFi.

@liutingdu additional: you need to implement an "ignore" list, as I have to be able to exclude interfaces by name: usually the local loopback and also the VPN Network Interface. So we would have a commandline switch that takes a list like "eth1 lo vpn_ether" - and specifically doesn't scan those named interfaces...

Created a new issue for switching to use Java NetworkInterface and scanning all interfaces #17
But it needs router IP for the port mapping discovery, I think it will need to keep the 'netstat' request for router IP because I didn't find a better solution than that

There appears to be no better solution to this. Other command that one can
use is
ip route

bleedin' Java.
On Fri, Jul 31, 2015 at 12:11 PM, Liuting Du<EMAIL_ADDRESS>wrote:

Created a new issue for switching to use Java NetworkInterface and
scanning all interfaces #17
https://github.com/evercam/evercam-discovery-java/issues/17
But it needs router IP for the port mapping discovery, I think it will
need to keep the 'netstat' request for router IP because I didn't find a
better solution than that
—
Reply to this email directly or view it on GitHub
https://github.com/evercam/evercam-discovery-java/issues/15#issuecomment-126656783
.


Update TypeDoc when it releases support for TS 1.7

Update TypeDoc dependency when it releases support for TS 1.7. Angular2 has 1.7 features that can't be parsed by the current version 1.6.
Blocker: https://github.com/sebastian-lenz/typedoc/issues/162

Apparently it works with angular2.beta.0. ¯_(ツ)_/¯


hdf5 formatter does not implement proper interface for write_metadata

The write_metadata function has an argument read_first:
https://github.com/QCoDeS/Qcodes/blob/master/qcodes/data/format.py#L107
The HDF5 formatter does not have this argument:
https://github.com/QCoDeS/Qcodes/blob/master/qcodes/data/hdf5_format.py#L238
This generates an error here:
https://github.com/QCoDeS/Qcodes/blob/master/qcodes/data/data_set.py#L764
A one line fix would be to add the read_first argument to the hdf5 formatter, but ignore it.
@giulioungaretti @AdriaanRol

@peendebak , I thought I took care to preserve the interface when making the hdf5 formatter. Maybe this got added later? (or maybe I just messed up). I approve of your immediate solution.


Purchase price rounds to 0

When i "purchase" a product on Grocy it will change the price to 0 if i put in a value less than 1. I.E. 0.50 changes to 0.00 when addedd. Can anyone help me?

This here is the repo of grocy-desktop which is only the desktop application wrapper.
grocy itself is maintained in grocy/grocy, so first transferring this to the right place...


Can't download extensions.

Visual Studio Code Version: 1.8.1
Code Sync Settings Version: 2.3.8
Operating System: Windows 7
Occurs On: Download
Proxy Enabled: Yes
It should show network problem instead of showing "downloading extension in the background".

can you please provide the developer console error logged picture so i can see where is the problem


Add graphql sso guide

Adds a guide about how to set up SSO with GraphQL 🎉

@keithpitt it would be great to get your feedback on the layout of the doc, and the accuracy of the explanations/examples 😄If it's all looking reasonable, I'll make all the dot points into full sentences 👌🏻

Given the complexity of supporting the variety of different providers, what would you think of splitting into specific guides for setting up SSO with SAML, and SSO with (Github|Google)?

Alright @harrietgrace — I've just done a retake of the guide.

Can you review it and let me know what you think / find? 🙏🏼

Thanks so much for the review @harrietgrace! I'll do another round of edits on the document first thing next week.


Allow passing in actions icon for menu

Is your feature request related to a problem? Please describe.
When using a icon font other than material, it is impossible to fix the more actions button.
Describe the solution you'd like
Add property to allow this icon to be set.
Describe alternatives you've considered
Right now I can't think of a way to fix this without forking and forcing the new version in the build.
Edit: I'm going to use a custom header.
Additional context
The icon is hard-coded in the render function:
https://github.com/quasarframework/quasar-ui-qwindow/blob/d4923d85d077cf0dba3e651a05eba0cd9c03d0a3/ui/src/components/QWindow.js#L1571-L1587

Good catch. I'll look into a fix for this on the weekend.

I will also implement this feature in the new version of qwindow 2


Does not work with Swift 3.0.2 (swiftlang-800.0.63 clang-8<IP_ADDRESS>)

Produces hundreds of errors when installed via Cocoapods. Can be reduced down to 5 in "misterfusion" by converting all to latest syntax dialog, all of which are "Cannot invoke 'filter' with an argument list of type, etc"

Seems like an awesome library though 😄

I've released 0.8.1!
Please try it😄


Mention 'using namespace' in readme

I thought this could be helpful!
https://github.com/landelare/ue5coro/pull/35

From what I've seen, most people use UE5Coro without any using namespace directives. It's entirely optional, and up to you.


Deploy App Engine from Docker Image

First step on our App Engine to Cloud Run migration #973

Made a Docker image for our App Engine website, and moved to App Engine flex to start using it in deployment.
Experimenting with using BuildKit for faster Docker builds.
I've kept the App Engine deployment workflow the same for now, just added the image building steps.

To preserve the current behaviour, images end up duplicated on both gcr.io/oss-vdb and gcr.io/oss-vdb-test

I also haven't changed how we run the App Engine locally.

Unfortunately, the App Engine flex/custom deployment (without the docker builds) seems to take a few minutes longer than current, which will make our deployments take a bit longer.
The website performance doesn't seem to be changed: current vs docker
There are still some things using App Engine that I haven't looked into for Cloud Run migration:

The cron-service (which doesn't actually get automatically deployed?)
The static files in the app.yaml

can you add a build.sh script in the blog page that just runs hugo -d ../dist/static/blog since it's no longer in the deploy script.

It already gets run in the Makefile, so I don't know if it's necessary

The website performance doesn't seem to be changed: current vs docker

Do you happen to know if any of the metrics we're using for SLO monitoring will require adjusting as part of this change?

I've been running into some issues with App Engine Flex:

Deployment, promotion, and rollbacks all take a long time
There's downtime/server errors between promotions

Instead of trying to deal with these App Engine issues, it'd be better to just wholesale move to Cloud Run, so I'm going to close this PR for now. I'll probably use this branch to work on the Cloud Run migration, since the Dockerfile should be useful for that.

I think this is the right call, App Engine Flex is a bit slower and cumbersome on a number of dimensions so if your going to the effort of containerising might as well go to Run.

Instead of trying to deal with these App Engine issues, it'd be better to just wholesale move to Cloud Run, so I'm going to close this PR for now. I'll probably use this branch to work on the Cloud Run migration, since the Dockerfile should be useful for that.


Remove max_depth parameter to Call*Request. [phab:D2195]

Created by jrtom at 2018-02-21 03:05:57:
This makes Call*Request a single-ply request.
This will give users control over how deep they want to recurse, and over how they want to filter the response for further queries.
Also, update proto-generated Go files and add proto-generated link.pb.go.

jrtom wrote at 2018-02-26 18:44:30:
ping to anyone who feels like reviewing this (I think that it may have been overlooked :) )

fromberger wrote at 2018-02-26 18:51:06:
Apologies for the delay, this one slipped off my radar.

fromberger accepted this revision at 2018-02-26 18:51:06

jrtom closed via commit 830c67c2c6a4994022a274f065001942a4f21746 at 2018-02-26 19:23:39


Download fails with os error 3 if a part of download path has trailing char sanitized to space

(Windows) When downloading a book bundle where one of the books was named "The Best of Make:" , the download failed with os error 3.
Fixed when the replacement char at util.rs:25 was changed from space to underscore, so probable cause was trailing space in directory name which caused the subdirectory be created as "The Best of Make" but trying to use "/The Best of Make /" with the trailing space in the download path failed as the directory with that name did not exist.

Hi!
Thank you for the analysis.
Indeed, it seems Windows ignores any whitespace as suffix.

Could you please test 0.15.1 to see whether it downloads the bundle properly?

It does work correctly now, thanks!

Thank you for confirming! Will close the issue.


How to predict rating using your code and what is the data format and file extension?

Hi Deven, I also am a bit lost in the sense that I cold successfully execute the code in python! The iterations result are printing but I am not sure how to get the predictions! Could you figure this out?


remove unused use statement for Phinx\Db\Action\DropColumn;

Phinx\Db\Action\DropColumn doesn't exist, remove the use statement in Phinx\Db\Table

thanks!


fix: mongodb multiple error rollback transaction

Description
fixes https://github.com/payloadcms/payload/issues/4133
When one transaction is used where multiple operations error, the result is that rollbackTransaction is called more than once. If this is happening async the second call to rollbackTransaction would hang on await sessions[id].abortTransaction().
This change makes it non-blocking and the function will not error when called multiple times.
PR also hs a type change to in Payload so the database interface allows the return type for rollbackTransaction to not be async.

[x] I have read and understand the CONTRIBUTING.md document in this repository.

Type of change

[x] Bug fix (non-breaking change which fixes an issue)

Checklist:

[x] I have added tests that prove my fix is effective or that my feature works
[x] Existing test suite passes locally with my changes
[ ] I have made corresponding changes to the documentation

When one transaction is used where multiple operations error, the result is that rollbackTransaction is called more than once.

This sounds very strange to me. Only one operation should be responsible for rolling back or committing the transaction, right? How is rollbackTransaction called more than once with the same transaction?
After cloning the reproduction repository from #4133 I saw that it was using Payload 2.1.1, which does not include #4164. In that reproduction I am seeing seeing an issue in commitTransaction, called from the docAccess operation (both when I reproduce it locally as well as in the stacktrace in the issue). That is exactly what I fixed in #4164 with this: https://github.com/payloadcms/payload/pull/4164/files#diff-9d83c8ca3813b690c6a9b61f1e4fd57959d364ee5d9530eb1013c9f10d5f85b3L37-R57, this makes sure that all operations nested within docAccess do not start their own transaction (creating a big race over req.transactionID) but instead just run in one big one.
When updating the reproduction repo to Payload 2.2.1, the reported issue goes away.
After all this: What exactly is this now fixing, if not #4133? :D

When one transaction is used where multiple operations error, the result is that rollbackTransaction is called more than once.

This sounds very strange to me. Only one operation should be responsible for rolling back or committing the transaction, right? How is rollbackTransaction called more than once with the same transaction?

See this test case: https://github.com/payloadcms/payload/pull/4245/files#diff-2a01d79fb3010bfe668ca296a4b7e534f28386e38f8fa1fa379dd309ed0f67c1R173
I'm firing off two payload.create calls that will both fail. This will cause rollbackTransaction to be called more than once. This should help if projects have complex hooks or access controls that are failing and the transaction rollback bug makes it hard to see the actual errors.

After cloning the reproduction repository from #4133 I saw that it was using Payload 2.1.1, which does not include #4164. In that reproduction I am seeing seeing an issue in commitTransaction, called from the docAccess operation (both when I reproduce it locally as well as in the stacktrace in the issue). That is exactly what I fixed in #4164 with this: #4164 (files), this makes sure that all operations nested within docAccess do not start their own transaction (creating a big race over req.transactionID) but instead just run in one big one.
When updating the reproduction repo to Payload 2.2.1, the reported issue goes away.
After all this: What exactly is this now fixing, if not #4133? :D

This is all great. I am glad that your changes addressed #4133.
The other thing this PR does is prevent endSession() errors from causing another throw. I found that DocumentDB specifically does not support it.
https://docs.aws.amazon.com/documentdb/latest/developerguide/mongo-apis.html#mongo-apis-dababase-sessions

I'm firing off two payload.create calls that will both fail. This will cause rollbackTransaction to be called more than once. This should help if projects have complex hooks or access controls that are failing and the transaction rollback bug makes it hard to see the actual errors.

Honestly, this kind of code should be actively discouraged. It is impossible to tell what will happen if two operations run in parallel like this. Even if both succeed, one will eventually be faster. If this faster operation is the one that "owns the transaction", then it will try to commit before the other, potentially causing parts of the other operation to happen outside (or in another) transaction.
It gets even worse when one or the other receive errors:

If the one that errors is not the one that "owns the transaction", it will also not roll back. As such the transaction will be committed (by the other operation), even when one of the operations threw an error!
If the one that errors is the one that owns the transaction, it will roll the transaction back, but the other operation may not be done yet, thus leading to parts of that operation running outside the transaction.

If there are multiple operations, we must either start a transaction beforehand or have them use separate request objects (making them run in separate transactions). The outcome of the code in your test case is imho entirely unpredictable, because it contains a race condition. Yes, we can just ignore and carry on if the transaction is already gone, but that only hides the issue of that race condition.
It would be great to have a useful error message if we can possibly detect this race condition, explaining what went wrong and telling users how to fix it.

I'm firing off two payload.create calls that will both fail. This will cause rollbackTransaction to be called more than once.

I'm sorry to keep going about this, but I would rather we get this correct.
If I understand everything correctly, the test case you linked will never roll back a transaction and if this test passes (with transactions enabled) then we have a bug (or I have a fundamental misunderstanding).
Assuming transactions are enabled:

await initTransaction(req)  happens, now req.transactionID is set.
payload.create is called to create first. This sees the existing transactionID thus will never commit or roll back (its initTransaction will return false).
The same goes for both payload.create calls that go into promises.
Any errors from those two create calls are entirely ignored ("// catch errors and carry on"). At this point nothing should have committed or rolled back the transaction.
expect(req.transactionID).toBeFalsy() should always fail, because the transaction is still alive
await commitTransaction(req) - now we commit

If there are multiple operations, we must either start a transaction beforehand or have them use separate request objects (making them run in separate transactions)

Yup, this is exactly what the test is doing: await initTransaction(req) https://github.com/payloadcms/payload/pull/4245/files#diff-2a01d79fb3010bfe668ca296a4b7e534f28386e38f8fa1fa379dd309ed0f67c1R155
This is how synchronous operations should be handled and now there is a test case for it. Without the changes to rollbackTransaction this test was hanging indefinitely.
How else would we handle this?

It would be great to have a useful error message if we can possibly detect this race condition, explaining what went wrong and telling users how to fix it.

I did have logger.warns in earlier releases. Unfortunately they were not helpful and only caused more confusion. If you have some fresh ideas I'd like to hear them.
@diesieben07 are you against merging this PR, do you see it causing other issues?

Okay... I found both my misunderstanding and the bug. All operations unconditionally call killTransaction without checking shouldCommit. This is not correct. The operation must only touch the transaction if initTransaction returned true.
Consider the scenario from your test case, where multiple operations run concurrently within the same transaction. Neither of them is "in charge" of the transaction, it was started before (like in your test case). Now one of the operations encounters an error and calls killTransaction. However it has no idea if the other operations (that are running in parallel with it!) are done yet or if they are still using the transaction. You might have the situation that killTransaction is completed, but afterwards one of the other (slower) operations still writes things to the database. Those writes will now use a new transaction (because req.transactionID was removed by killTransaction). Now you have the situation where an operation is rolled back partially!
killTransaction must be handled just like commitTransaction. Only who started the transaction must commit or roll it back.


re: Just a thought (resize->thumbnail)

Example One:
http://imgur.com/a/by9uu
Original: 2048x1366
->thumbnail(440, 330, 'center')
= completely fucked showing only a tiny portion of the image near center (no resize)
I actually thought thumbnail would have resize built into the function scaling down the original image to a near representation of the thumbnail size specifications. Then completing a crop of outer edges to match thumb specs exactly. The end result being essentially the original image with tiny portion of the edges cropped creating a uniform thumbnail. The above is useless...
Example Two:
Trying to resize before thumbnail has no affect
$image
->fromFile($filenameOut)
->resize(width, height) << using this to reduce dimensions of overly large files before creating thumb but does nothing
->thumbnail(850, 637, 'center')
->overlay($_SERVER['DOCUMENT_ROOT'].'/images/watermark.png', 'bottom right')
->toFile($newFile, 'image/jpeg', 94);
End result the only way I see this working now is by running it twice. (which should not be required).
Run 1: Resize to near Thumbnail Size
Run 2: Create thumbnail
My suggestion is to either have "thumbnail" resize the org. image automatically then create the thumbnail.
If not that then at least allow resize to be called before thumbnail.
Thoughts?
Options?

I just ran the following code using your test image:
$image
  ->fromFile('CU0eTPj.jpg')
  ->thumbnail(440, 330, 'center')
  ->toScreen();

Result:

I don't have your original watermark.png file, but here's what it looks like with an overlay using another PNG:
$image
  ->fromFile('CU0eTPj.jpg')
  ->thumbnail(440, 330, 'center')
  ->overlay('flag.png', 'bottom right')
  ->toScreen();

Result:

You wrote:

completely fucked showing only a tiny portion of the image near center (no resize)
I actually thought thumbnail would have resize built into the function scaling down the original image to a near representation of the thumbnail size specifications. Then completing a crop of outer edges to match thumb specs exactly. The end result being essentially the original image with tiny portion of the edges cropped creating a uniform thumbnail. The above is useless...

 Things seems to be working as intended. You can see that the library does perform a resize and a crop in the source.
Maybe your original code has an issue, but the library appears to be working as intended and there haven't been any other reports of this working incorrectly.

If you have a minimal test case demonstrating this not working, feel free to post it and I'll reopen.

Let me send originals!
I believe the code is yours.
Origional - 2048x1366
http://imgur.com/WFVBFrM
After Thumbnail - 440x330
http://imgur.com/6q59cQF
CODE
$image
->fromFile($filenameOut)
->thumbnail(440, 330, 'center')
->toFile($newFile, 'image/jpeg', 94);
Not overly complex code. It simply takes original and runs it through
thumbnail and saves as jpeg.
All that appears to be happening on my end is thumbnail taking 440x330 from
the center of the image, then cropping off the rest.
I will try run the above again as I have made further changes to try and
figure out a solution.
Sincerely,
Grant
On Jun 3, 2017 3:00 PM, "Cory LaViska"<EMAIL_ADDRESS>wrote:

Closed #186 https://github.com/claviska/SimpleImage/issues/186.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/claviska/SimpleImage/issues/186#event-1108940190, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AEGBNkrwSJYmAzVOjAnu3k9V4Lu8ViDyks5sAbtFgaJpZM4NvKhK
.


Detect premoves with zero time and analyse if it was a good premove or not

A premove is a locked move, meaning that some opponent moves that would not be objectively good could have led to disaster

Not worth it. It would get a lot of positive in bullet. And bullet is not chess :D


Any ideas on how to debug flakey dynamixel connections?

Having trouble consistently being able to connect to motors and scan them via dynamixel, wondering if there's ways to debug it more

When attaching them one by one, it works most of the times, and nothing shows up on scanning via Dynamixel wizard rest of the time
When trying to connect one motor to another, nothing shows up on scanning via Dynamixel wizard 100% of the time
When have the whole setup wired, the smaller motors are blinking red (seems like that's incorrect voltage error) – and nothing shows up on Dynamixel wizard scanning

The Dynamixel wizard shows you the input voltage. You could check that this is correct. Otherwise, I have no idea.

Gotcha, will debug more, thanks!


Improve tolerance at fully extended configurations

Once a system has been fully extended like the following it can get "stuck" in that pose with any attempt to solve resulting in divergence because moving one of the fully extended joints even a small amount will result in it diverging before it converges.
Some things to look in to:

Find joint angles that cause these fully extended configurations (the angles that cause two connected links to be parallel) and avoid them.
Adjust the angles of the extended joints if they happen to be parallel and the solve diverged. Or adjust all angles to afford more "slack" in the system.
Analyze the resulting joint deltas to see if we can adjust certain joints based on that.

Taking a step towards a known good pose like the rest pose before solve seems to alleviate this issue a bit but can cause jumping when full extended as the solve diverges and resets the previous position and another step is taken the next frame. Perhaps the step can be undone if the solve diverges?


Upgrade from python optparse to argparse

optparse has been deprecated since Python 2.7: https://docs.python.org/2/library/optparse.html
Would be a fun project for someone just getting familiar with python, git, or grep.

Closing as dup.  See #1125 and #1729.  @rishig You may want to take a quick look at the PR.


Zap build cannot be verified

I tried to verify the build from Google Play against the source code but got a diff in binary files.
Could you please investigate and improve the build instructions so the releases can be verified.
I think this is highly important also for internal use, as the release manager under distress might put the users' funds at risk and only deterministic builds allow the team to react.

@Giszmo
Thanks for doing this. This is indeed very important.
We generate two APKs so we can keep the download size at a minimum.
I don't know though why the diffs appear.
I always built it using Android Studio, but that shouldn't make a difference, I think.
We have to investigate this and will inform you if we have any news on this.

@michaelWuensch thanks for your interest and understanding. Yes, I was surprised that not more apps were building multiple APKs but yours was the first and only one I remember. We failed to compile some apps, so ... who knows.
Sorry for boring you with my problems but would you have an idea how to get all APKs as they get released? My best idea is a monitor app that hopefully one user per category (architecture, country, screen size, ...) runs on their phone to hash every relevant apk and check my server if we know that one and if not, upload it.

@Giszmo
I used the following firefox extension which allows you to download all the apks from google play store:
https://addons.mozilla.org/en-US/firefox/addon/direct-apk-downloader-2018/
I am not sure though if this really helps to automate this in a feasible way.
I now also started to investigate the diffs.
I found out that the 2 json files for example only differ in linebreaks

The line breaks in json and the png diffs I would have allowed for now, although later I would need some criteria that can be automated, so finding a careful balance to what is allowed in a diff and what might open the door for malicious insertions would be a tricky one. My main concern are the .so files.
For the png files you would have to disable pngcrunch. You can run it once and commit the crunched files to avoid the compile time non-determinism.
Thanks for the firefox tip. If the extension can download apks, console tools can, too.

Three months later ... is there any progress regarding verifiability?

@Giszmo
Thanks for doing this. This is indeed very important.
We generate two APKs so we can keep the download size at a minimum.
I don't know though why the diffs appear.
I always built it using Android Studio, but that shouldn't make a difference, I think.
We have to investigate this and will inform you if we have any news on this.

Is there a reason the 0.3.1 release doesn't include the APK's?

Hey @githorray,
no reason other than I have forgot to, have attached them now.

Attaching a build to GitHub releases is unrelated to this issue. How is the issue coming along?

@Giszmo
We have not forgotten about it. There is still so much left to do on Zap Android that we just didn't take the time to investigate further where this diffs come from. Eventually we will tackle this issue as well.
In the meantime everyone who does not trust us has the possibility to build it from source.

I know it is important. But usability is as well.
There is no use in having a verifiable wallet if it is complicated to use and people flock to a custodial wallets instead.
It is a question of priority.
Just so you can better understand me:
You are the only one so far who asked for verifiability (which actually surprises me), while we get lots of other requests.
This means it might not be the top priority for everyone.
If more people ask for it, I will prioritize this above other things.
And if not I will still try to make it verifiable, but it might take some time.
By the way, I don't feel like i owe anything to anyone. I am doing this 100% for free in my spare time. Nobody is forced to use Zap Android.

Well, ... I'm working on getting more people up to speed on the topic. Non-techies that learn about open source, assume that that is enough and that somehow magically thousands of security researchers review code of any tiny project.
Re: What you owe: This project is open source under the MIT license, so you the software developer don't owe anything to anybody and I totally agree with that. You do put the binary out there through Google Play though, where you make at least implicit promises of this app being good to use. From that angle, I see a big duty to provide certain levels of security and not knowing how the binary came to be, is scary.

I'm a zap user and would like to see this as a priority. Without reproducibility it's not much different from closed source software.

@Giszmo
Hi, I want to get this done now, but have problems to find out, what causes the differences.
If I do a fresh clone of the Source Repo and build a unsigned version with
$ ./gradlew assembleRelease
And right after that create again a unsigned version with:
$ docker run --rm -it --volume=$PWD:/mnt --workdir /mnt mreichelt/android:latest bash
root@84c836f4577a:/mnt# ./gradlew assembleRelease
And compare the two results with:
diff --brief --recursive BuiltOnPC/ BuiltWithDocker/
I get this as result:
Dateien BuiltOnPC/lib/armeabi-v7a/libiconv.so und BuiltWithDocker/lib/armeabi-v7a/libiconv.so sind verschieden.
Dateien BuiltOnPC/lib/armeabi-v7a/libzbarjni.so und BuiltWithDocker/lib/armeabi-v7a/libzbarjni.so sind verschieden.
All other differences seem to have vanished, but the .so files still differ. (Both are unsigned, therefore the signing differences are gone as well.)
The only difference in the build process seems to be building with the docker image vs building with Android Studio installed from the official website.
Do you have any idea what could cause this or how to proceed from here?

Hallo Michael!
Nice to hear you are working on this! So those two native libs are built in a non-deterministic way and there are several things that might cause this. I would check in this order:

Do those libraries have dependencies that are not pinned to one specific version?
Are the build tools pinned to a specific version?
Try with disorderfs. Many projects process files in folders in the order the underlying file system provides them and that causes non-deterministic results. Disorderfs is a thin FS layer that allows to control the sorting.

@Giszmo
Thanks for your help.
I found out, that both .so files are part of the QR-Code scanner library I used.
Luckily I have made a PR on a new QR-Code scanner library a long time ago but did not merge it, as it didn't really make a difference. With this library the app seems to be 0.6 mb smaller though and no longer has those .so files.
So I think the easiest way is to make some last improvements on that PR and merge it.
I will try that first.

@Giszmo
I just released a new version (v0.3.8-beta) which should be reproducible. It will probably take some time until it is available on the playstore though. On github the release is already available.
Please note that it is now only one apk and no longer two. This will later change again though, when local LND is added.
Please let me know if it worked.
This should be the instructions you will need to reproduce it:
$ cd zap-android/
$ git tag | grep 0.3.8
v0.3.8-beta
$ git checkout v0.3.8-beta
$ docker run --rm -it --volume=$PWD:/mnt --workdir /mnt mreichelt/android:latest bash
root@84c836f4577a:/mnt# ./gradlew assembleRelease
root@84c836f4577a:/mnt# mv app/build/outputs/apk/release/zap-android-0.3.8-beta* .
root@84c836f4577a:/mnt# exit
$ ls *.apk
'zap-android-0.3.8-beta(26)-release-unsigned.apk'

Please note that it is now only one apk and no longer two. This will later change again though, when local LND is added.

Can you please elaborate?
Compilation is quick and works. I don't have the Google Play version yet though.

@Giszmo
It is now available on google play as well. Hope it works and we have reproducible Zap for Christmas :)

Can you please elaborate?

I changed the QR-Code scanner library which removed the .so files. Appearently after that change the splitted apks were all identical, as there was no architecture specfic code anymore.
Google play even recognized all that apks as ABI "all". So I actually had to make it build a single one again with a higher store id, to ensure all devices get updated.
Later, when local lnd is added, we will most likely switch back to a splitted deploy, as it reduces the app size a lot when we don't include all the different architectures of lnd.

Welcome in the top ranks!
That split part will make things complicated on my end as I don't have many channels to obtain binaries.  I will approve the rebuild for one of those, only and it will get a bit a pain to get all architectures to broaden the checks.


Fixed #26215 -- Fixed RangeField.value_to_string() serialization

Sounds good to backport to 1.8/1.9. Might be worth checking the other serialization formats too. We have the infrastructure in tests/serializers/test_data.py for creating other tests as needed. I'd be fine if you test this manually for now and create a new ticket to improve the postgres_tests if it's warranted.

I had to restart Jenkins so this build got lost - I'll retrigger. Feel free to use the PR title as a release note.

buildbot, test this please.


[feat] Quest progress bar

On the quest's frontend tab, the button to claim a quest is either red if not claimed or green if claimed. It'd be cool to have the button act like a progress bar aswell. So given 2 values progress and needed, say needed=4 for example:
progress = 0 -> full red button how it is now
progress = 1 -> 1/4 full dark orange-ish
progress = 2 -> 1/2 full orange-ish
progress = 3 -> 3/4 full orange-yellow-ish
progress = 4 -> full yellow and pulsing button indicating claimable
Then the button will be green after claiming like it is now. Make the colors be a steady transition based on the % of progress/needed. You can use hardcoded values for progress and needed for now.
References

Quest item

Would love to work on this :D

Thank you for assigning me @b-j-roberts,
I have started working on it. I will be quick to complete the issue.

Hey @b-j-roberts,
I wanted to know if there is any TG group for the project. I wanted to communicate and discussing on GH will just fill it up ig.


Videos don't autoplay

Provide a test case please.

same here!

Same again! We are using mediaelement autoplay and after adding the plugin it seems to stop this. if you have any ideas please let us know. Great plugin btw!

Same here. :(

any solution yet?


Blockchain Explorer

How would you describe your project?
Blockchain Explorer - Browser for the blockchain for everyone
Functional Requirements of blockchain explorer:

The system should be able to track all transactions in real time
List all transactions by creation date and show details of the transactions on clicking on each one
The system shall display total no of blocks added to the chain
Wealth Management should be there to see how wealth is distributed amongst the members in the network
Account page should be there showing all transactions done by that particular account
People should be able to see active banks and system should help them decide which bank one should they prefer
Nodes can be highlighted on a world-map
Powerful search
Analysis like volume of transactions, etc
Mainnet/ Testnet

What problem does it solve?
Blockchain explorer serves the need to view and read detailed data about transactions as well as ensure transparency of transactions and blockchain. It will enable reading of blockchain transaction histories, wallet balances, and to confirm the status of a transaction if or not it is confirmed. It can be used to extract virtually any data related to transactions, wallets, and blockchains including real-time market price, rich lists and wealth distribution, etc.
Who is the target market?
Anyone who uses the listed blockchain(s) network, or who wants to learn explore transactions/ ledger blockchain
What do you think is the opportunity of this project?
Applying data driven approach can benefits all individuals. For validators/ miners, explorer will provide helpful resources.
How does it integrate with thenewboston digital currency work?
thenewboston digital currency will be exclusively supported on the platform
Are there any competitors?
Blockchain.com, Etherscan.io and many many more
Will you need specific information or advice?
Yes, access to right APIs would be critical for this project along with
What are your goals and objectives?
Our aim for the blockchain explorer is a user-friendly tool that gives you easy access to data from multiple search points, and provides a more intuitive understanding of the blockchain network itself.
How will you evaluate project success?

User base growth
Powerful Data API
Real-Time Analysis

What risks should you plan to manage?
Exclusive support to thenewboston comes with it's risk of dependence on the success of thenewboston
What are your milestones?

Proof of concept https://itsnikhil.github.io/tnb-analysis completed!
Hiring Phase
Discovery Phase (Planning and evaluating the tasks for the project)
Designing Phase (UI/UX Designers prepare the components for the project)
Architecture phase (Decide/Construct a certain architecture for the Project)
Development of the components & logic which are going to be integrated into the website

Tell us about your team members:
@itsnikhil | Nikhil Taneja | Project Lead
List the roles of any additional team members you would be looking for?

1-2 Backend Developer
1-2 FE Lead (Preferably one that can design UI/UX as well)
More as per requirements

Congratulations @itsnikhil , we liked your project proposal. Now you need to provide your project repository link (You can comment down here). You can provide an empty repository link also if you have not started working on your project yet.

Thank you so much :)
I have created an organization https://github.com/open-blockchain-explorer as there could be need for having more than one repository for the project

Hey, @itsnikhil  your project has been added to the repository.
Blockchain Explorer: https://github.com/thenewboston-developers/Project-Proposals/tree/master/projects/23-Blockchain Explorer

Hey @itsnikhil, I think this would be a really cool project to implement and I would like to be one the Front End developers working on this. I mainly code in javascript and use Web Technologies like HTML, CSS, and React to make Responsive Web Apps.

Hi @tomijaga
I would love to connect with you regarding this. Have you joined the slack group?.. if you have not then please join it and message me personally over there on slack. You can find me by my username itsnikhil


Error: JSDOC_ERROR: ERROR: Unexpected token import

I am getting the following error when trying to do a dynamic import within my component (a Webpack feature?):

JSDOC_ERROR: ERROR: Unable to parse /var/folders/cj/xqqjbxpj6ylbz5rs_89w8q2r0000gn/T/quxuq4lmsz8a5hd8zhg16b.js: Unexpected token import
Cannot parse src/elements/ImageMedia.vue: Error: JSDOC_ERROR: ERROR: Unable to parse /var/folders/cj/xqqjbxpj6ylbz5rs_89w8q2r0000gn/T/quxuq4lmsz8a5hd8zhg16b.js: Unexpected token import
It usually means that vue-docgen-api does not understand your source code or when using third-party libraries, try to file an issue here:
https://github.com/vue-styleguidist/vue-docgen-api/issues

Component script (reduced for brevity)
<script>
const lazySizes = () => import("lazysizes") // This is the problem line.
</script>

I am actually building my components via Vue Design System (https://github.com/viljamis/vue-design-system) which I beieve used Vue Styleguidist under the hood.
So, I'm not sure whether to report the problem there too.
Thanks.

Hello @michaelpumo, I believe this request was fixed by the last version of vue-docgen-api.
If you encounter it again, do not hesitate to open a new request. If you can, please include a reproduction repo.
Bart


Add support for allow-hotplug

Related to issue #16.

Probably it is better to add a new list parameter allow to add support for multiple allow- stanzas. According to http://manpages.ubuntu.com/manpages/hardy/man5/interfaces.5.html allow-XYZ is possible.

@berendt Thank you for your pull request. I don't know if any other stanza than auto and hotplug is used.
Is it allowed to have multiple stanzas per interface? like auto and hotplug?

Added more generic #22


New Web Service - Download Lyrics For Music File

Now that there is Lyrics support in Tagger (#239), I'd be cool to have a web service to download the lyrics for a music file
Needs planning: I have no idea what APIs are available to use for this

Probably worth taking a look at GiveMeLyrics

Rhythmbox also has a lyrics plugin, but it's a bit outdated and contrived to work with (no specific APIs, it actually sends HTTP requests, parses the HTML pages and extract the lyrics).

https://github.com/NickvisionApps/Tagger/releases/tag/2023.9.0-beta1


Fix typo

github -> GitHub

@changeworld : Thanks for your contribution! The author, @SnehaGunda, has been notified to review your proposed change.

Merging typo fix


Adds time-of-flight sensor support (VL53l0X)

Supports two messages:

Enable
Read

Enable is necessary to support multiple ToF sensors, since they must be brought online one at a time and set to new I2C addresses.

LGTM!


gce-production-3: resurrect, managed instance groups

The next step in this saga is to resurrect the managed instance group. It is still set to 0 instances for now.
We need to announce the new NAT IPs before we can start shifting traffic to this project.
This patch puts the infrastructure in place. Once we're ready to do that, we can play with the numbers in instance-counts.auto.tfvars, and use worker-deploy to drain some of the instances in the other two projects.
refs travis-ci/reliability#217

Applying to gce-production-3.
I got one error:
Error: Error applying plan:

1 error(s) occurred:

* module.gce_worker_group.google_storage_bucket.gcloud_cleanup_archive: 1 error(s) occurred:

* google_storage_bucket.gcloud_cleanup_archive: googleapi: Error 409: You already own this bucket. Please select another name., conflict

I went ahead and manually deleted the gcloud-cleanup-production-3 bucket, in order to allow terraform to re-create it.
The second apply ran successfully. The worker instance groups have been created.

Applied to gce-production-1, in order to grant image permissions.


Disable field validation until field loses focus

Hi, the validation runs on every keystroke so the error message is always seen on every form field. The field content will be invalid until the users finishes typing.  So is it possible to delay the field's validation until the field loses focus?

I'm not quite sure your requirement. Do you meam that you wanna trigger the validation only when onBlur rather than onChange?
Because when the user is typing, the partial message could be wrong, but it's not the final state of that field because the user hasn't finish typing?
You can simply not binding the onChange for that component.
:)

Hi, yes, I only want to trigger the validation when onBlur is called.  If I comment out the 'onchange={onChange} in the  component then typing in the field does nothing:

             <input                 type={type}                 id={name}                 name={name}                 className={status}                 placeholder={placeholder}                 value={value}                 /*onChange={onChange}*/                 disabled={disabled}                 autoFocus={autoFocus}                 ref={ref}             />
{errorText}

How can I allow the field to accept input without this onChange handler?

In conjuction with binding the value to 'defaultValue' instead of 'value', this works :)
So the  component's  (from your TextInput.js file) looks like this:

             <input                 type={type}                 id={name}                 name={name}                 className={status}                 placeholder={placeholder}                 defaultValue={value}                 /*onChange={onChange}*/                 disabled={disabled}                 autoFocus={autoFocus}                 ref={ref}             />
{errorText}

This works. Many thanks.

Previously, I used:
value={value}
And when I wanted to update a field's value in code, I would do:
this.setState({email:{value:''}})
But with the change above:
defaultValue={value}

I can no longer update the field's value using:
this.setState({email:{value:''}})

How do I update this field's state programmatically?

The form's submit button is disabled until the isFormOk = true.  By removing the onChange handler, each form field is only validated when onBlur is called. This means I need to:

click once outside the form field for it to be validated and enable the submit button and
click a second time on the button to submit the form.

So i've realised that the onChange handler needs to remain. BUT, only show any relevant validation message when the related form field's onBlur is called.
Apologies for this use-case, but is this possible?

How do I update this field's state programmatically when I use defaultValue={value} when creating the component?

There are hacky ways if you google this. But allow me to answer this question along with the one below.

So i've realised that the onChange handler needs to remain. BUT, only show any relevant validation message when the related form field's onBlur is called. Apologies for this use-case, but is this possible?

I think the original way is better, think of this use case, if you don't handle the onChange, or only show the error message when onBlur happens even onChange triggers before it. There is no way for the user to know whether his input is right or wrong unless he changes the focus.
Which means when the user at the last field, after he finishes entering the content.
He wants to click the submit button right after entering content.
But if he gets the wrong input, the submit button will be disabled and he can't click, but he doesn't know what is wrong there, because the onBlur hasn't been trigger yet.
Now this becomes confusing for the user. Why I can't click the submit button while there are no errors?
So, I recommend the original way which is been used by most of the site there.
Let me know if you have a use-case which isn't been covered. :)

Ok, I agree that the onChange must stay as you have described. Many thanks.
Regarding the previous question - How do I update this field's state programmatically when I use defaultValue={value} when creating the component? ...I didn't see your answer to this...

Hey, sorry for the delay. I kept the onChange handler in the end.  But any additional functionality that you add can only increase your framework's usefulness :) I'll happily test it for you.
How do I update a field's state programmatically? eg reset a single field?


Embedded/Linked documents use forwardslash instead of backslash

Here are my current settings:

I drag a sample image into my vault and choose these import settings.

The image was correctly placed under _resources/Untitled but the note says that the file can not be found.
![[_resources\Untitled\Mister_Puck_064-2024_07_07T15_18_21.png]]
Upon switching these to backslashes, the image embeds properly.

Thanks a lot for the feedback. Today is a busy day. But the bug will
certainly be fixed.
Are you running under Windows? I should do some more tests. In MacOS and
Linux (where I did most of the tests), the backslash is not really used
for files. I would expect Windows to use it. However, perhaps Obsidian
expects forward slash instead. It kind of makes sense to keep
compatibility across the Unix devices too.

Here are my current settings:

I drag a sample image into my vault and choose these import settings.

The image was correctly placed under _resources/Untitled but the note
says that the file can not be found.
![[_resources\Untitled\Mister_Puck_064-2024_07_07T15_18_21.png]]
Upon switching these to backslashes, the image embeds properly.

--
Reply to this email directly or view it on GitHub:
https://github.com/alberti42/obsidian-import-attachments-plus/issues/1
You are receiving this because you are subscribed to this thread.
Message ID:
@.***>

Ok, I released a new version where now the links are imported correctly. However, I think this breaks other features since now the links are stored with forward slash, but then Obsidian needs internally the path with backslash to operate on the attachment.
I think it is straightforward how to fix it, but I don't have time until Thursday/Friday.

Honestly, I realized a few features are broken in Windows. I should have tested it better. I did only for MacOS and Linux. I will fix the bugs in the following days.
Meanwhile you can already use it to import attachment by moving them to the vault. This works as intended.

Hello, yes I should have specified, I'm using Windows 10!
I will let you know if I encounter any more windows-specific issues.

... well, there are issues with additional features that you may want to
switch on, such as automatic renaming of the attachment folder if you
rename the note, and other small things that may not work as intended
now on the Windows version.
All these niuances will be fixed for sure.

Hello, yes I should have specified, I'm using Windows 10!
I will let you know if I encounter any more windows-specific issues.
--
Reply to this email directly or view it on GitHub:
https://github.com/alberti42/obsidian-import-attachments-plus/issues/1#issuecomment-2214871925
You are receiving this because you commented.
Message ID:
@.***>

Ok. I believe all the problems are now fixed with the new version I just released. I would be grateful if you could send a feedback.

I believe the issue is now solved. Feel free to open the ticket again if you find more problems on that.


Make seed lease/duration configurable

How to categorize this PR?

/area scalability
/kind enhancement
What this PR does / why we need it:
Make seed lease/duration period configurable.
Which issue(s) this PR fixes:
Fixes #4504
Special notes for your reviewer:
Release note:

Two fields `LeaseDurationSeconds` and `LeaseResyncSeconds` added under `SeedControllerConfiguration` to make Seed lease and duration configurable. Both field have default value of 2 seconds.

Thank you @ary1992 for your contribution. Before I can start building your PR, a member of the organization must set the required label(s) {'reviewed/ok-to-test'}. Once started, you can check the build status in the PR checks section below.

/invite @shafeeqes @acumino

/reviewed/ok-to-test

Can you add a proper release note to make operators aware that this is now configurable?
Can you also extend https://github.com/gardener/gardener/blob/master/example/20-componentconfig-gardenlet.yaml and https://github.com/gardener/gardener/tree/master/charts/gardener/gardenlet accordingly, please?

@ary1992 The verify step is failing, can you check please?

09a0e66 looks good, thank you! Please proceed with the other improvements as discussed yesterday :)

lue in the Lease object to be different compared to the duration how of the lease is renewed? Or wouldn't this rather be always equal?

Earlier two separate fields were defined as constant integer  LeaseDurationSeconds and LeaseResyncSeconds. Hence two separate int variables are defined with the same name corresponding to constant integer  under SeedControllerConfiguration  as mentioned in issue #4504 . Moreover LeaseResyncGracePeriodSeconds was defined in terms of  LeaseResyncSeconds  i.e  LeaseResyncGracePeriodSeconds = LeaseResyncSeconds * 10. If we can define LeaseResyncGracePeriodSeconds in terms of LeaseDurationSeconds, then one variable is enough. Need more insight on such use cases where only LeaseDurationSeconds is enough rather that two separate variables.

Need more insight on such use cases where only LeaseDurationSeconds is enough rather that two separate variables.

I would rather say the opposite: We need to know about use cases where only one configuration value is NOT enough.
@timebertt Since you originally opened the issue - do you have information about such use cases?

/merge squash

/merge squash


Generation of the module name: added the ability to pass the url inde…

On my project, when generating api using your library, we faced the problem that all urls of requests for api start with / api / and we generate one large module with all methods inside the api. In the code, I saw that the selectable index was hardcoded. Perhaps it will be convenient not only for me if it will be possible to select the index through the config.

Yes, got your problem, thanks for this PR with fix
I will merge it today soon

Thank you. If I missed something and need to add, write, I'll do it.


Plan for 1.0.0

[x] Rename to ssn-validator.
[x] Add strict mode option to isValid to consider dashes and spaces when validating or not.
[x] Add node 5 to the test matrix (https://github.com/seegno/is-valid-ssn/blob/master/.travis.yml#L4) or consider using docker-compose run --rm sut.
[x] Add missing "Instances" comment (https://github.com/seegno/is-valid-ssn/blob/master/src/index.js#L25).
[x] Convert to ES2015 template string (https://github.com/seegno/is-valid-ssn/blob/master/src/index.js#L51).
[x] Fix README typos (e.g. "This modules allows you to check if a number is a valid")
[x] Invert order of "See" with the introduction (https://github.com/seegno/is-valid-ssn/blob/master/src/index.js#L3).
[x] Update package.json (e.g. based on this one), with special focus on scripts.
[x] Update README to include lodash-like arguments (based on this).
[x] Update eslint-config-seegno to 2.0.0 and subsequently the rc file, including renaming to yml.
[x] Export named functions instead (https://github.com/seegno/is-valid-ssn/blob/master/src/index.js#L58).
[x] Remove lodash dependency (https://github.com/seegno/is-valid-ssn/blob/master/src/index.js#L14).

Closes by #2.


Update python_api.md to fix typo

Noticed a typo in Carla.Transform section of the docs under 'getters'. get_right_vector description previously said "rotatio" instead of "rotation". Added the missing letter.

Description

Fixes #
Where has this been tested?

Platform(s): ...
Python version(s): ...
Unreal Engine version(s): ...

Possible Drawbacks

This change is

Hi, it seems that this PR was using master instead of dev branch, so it is quite old. We are only merging PR to dev branch. So if you want, make a new PR with that fix for the dev branch, if it is still applicable. Thanks.


Github templates confusing

name: Bug report
about: Create a report to help us improve

Describe the bug
The Github templates are confusing since the "TODO"s are not wrapped in comments, ie. <!-- -->
Also, the --- .... --- section is confusing
Expected behavior
All this stuff should be added or removed! :^)

Also, we'll likely want to add a "contributing" section.


CVE-2022-27664 (High) detected in golang.org/x/net-v0.0.0-20220708220712-1185a9018129

CVE-2022-27664 - High Severity Vulnerability
Vulnerable Library - golang.org/x/net-v0.0.0-20220708220712-1185a9018129
Library home page<EMAIL_ADDRESS>Dependency Hierarchy:
github.com/hyperledger/fabric-contract-api-go-v1.1.1 (Root Library)
github.com/go-OpenAPI/spec-v0.19.4
github.com/go-openAPI/jsonreference-v0.19.2
github.com/puerkitobio/purell-v1.1.1
x golang.org/x/net-v0.0.0-20220708220712-1185a9018129 (Vulnerable Library)
Found in base branch: master
Vulnerability Details
In net/http in Go before 1.18.6 and 1.19.x before 1.19.1, attackers can cause a denial of service because an HTTP/2 connection can hang during closing if shutdown were preempted by a fatal error.
Publish Date: 2022-09-06
URL: CVE-2022-27664
CVSS 3 Score Details (7.5)
Base Score Metrics:
Exploitability Metrics:
Attack Vector: Network
Attack Complexity: Low
Privileges Required: None
User Interaction: None
Scope: Unchanged
Impact Metrics:
Confidentiality Impact: None
Integrity Impact: None
Availability Impact: High
For more information on CVSS3 Scores, click here.

This should be fixed with v1.2.0

This is still reported with following go.mod file
require (
github.com/hyperledger/fabric-chaincode-go v0.0.0-20220920210243-7bc6fa0dd58b
github.com/hyperledger/fabric-contract-api-go v1.2.0
github.com/stretchr/testify v1.8.0
)

How are you creating that report? I'm still not seeing the same vulnerability in the nightly scans, and I checked the tutorial contract with the following go.mod file and that was ok too...
module github.com/hyperledgendary/cagcve

go 1.19

require github.com/hyperledger/fabric-contract-api-go v1.2.0

require (
	github.com/go-openapi/jsonpointer v0.19.5 // indirect
	github.com/go-openapi/jsonreference v0.20.0 // indirect
	github.com/go-openapi/spec v0.20.6 // indirect
	github.com/go-openapi/swag v0.21.1 // indirect
	github.com/gobuffalo/envy v1.10.1 // indirect
	github.com/gobuffalo/packd v1.0.1 // indirect
	github.com/gobuffalo/packr v1.30.1 // indirect
	github.com/golang/protobuf v1.5.2 // indirect
	github.com/hyperledger/fabric-chaincode-go v0.0.0-20220720122508-9207360bbddd // indirect
	github.com/hyperledger/fabric-protos-go v0.0.0-20220613214546-bf864f01d75e // indirect
	github.com/joho/godotenv v1.4.0 // indirect
	github.com/josharian/intern v1.0.0 // indirect
	github.com/mailru/easyjson v0.7.7 // indirect
	github.com/rogpeppe/go-internal v1.8.1 // indirect
	github.com/xeipuuv/gojsonpointer v0.0.0-20190905194746-02993c407bfb // indirect
	github.com/xeipuuv/gojsonreference v0.0.0-20180127040603-bd5ef7bd5415 // indirect
	github.com/xeipuuv/gojsonschema v1.2.0 // indirect
	golang.org/x/net v0.0.0-20220708220712-1185a9018129 // indirect
	golang.org/x/sys v0.0.0-20220715151400-c0bba94af5f8 // indirect
	golang.org/x/text v0.3.7 // indirect
	google.golang.org/genproto v0.0.0-20220719170305-83ca9fad585f // indirect
	google.golang.org/grpc v1.48.0 // indirect
	google.golang.org/protobuf v1.28.0 // indirect
	gopkg.in/yaml.v2 v2.4.0 // indirect
)

Which version of Go are you using? I think 1.18.6/1.19.1 or later should resolve the vulnerability. I've updated the vulnerability scan to use the latest patch level to avoid any false positives in the future #77

I've opened hyperledger/fabric#3654 to update the Go version used to build chaincode.


[$1000] Android/iOS - Chat - jpg image opens always with loading spinner in preview

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Action Performed:

Open the App
Login with any account
Go to any chat
Send a jpg image
Open the image you just sent in the preview

Expected Result:
jpg image opens the first time with the loading spinner, but the next times without it
Actual Result:
jpg image opens always with loading spinner
Workaround:
Unkown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[x] Android / native
[ ] Android / Chrome
[x] iOS / native
[ ] iOS / Safari
[ ] MacOS / Chrome / Safari
[ ] MacOS / Desktop

Version Number: <IP_ADDRESS>
Reproducible in staging?: Yes
Reproducible in production?: Yes
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
https://github.com/Expensify/App/assets/93399543/e721ec50-d21e-4e3f-96eb-5707770af77d
Expensify/Expensify Issue URL:
Issue reported by: Applause - Internal Team
Slack conversation:
View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~010b826cfb93c10d6b
Upwork Job ID:<PHONE_NUMBER>439746560
Last Price Increase: 2023-06-04

I was able to reproduce this consistently on both Android and iOS native apps, it's working fine on other platforms. Also, PDF works fine on native apps.
This may be a duplicate of https://github.com/Expensify/App/issues/19369, I've asked over there if it's the same root cause and if we should fix them together.

Sounds like this bug is unique from the other one, adding the external label. @jliexpensify I've also added you as another BZ team member to watch over this since I'm out all next week.

@tienifr can you clarify this for me?

We can remove that InteractionManager.runAfterInteractions logic and set isLoading to false directly after the image finishes loading. There

Are you saying remove all of the logic in InteractionManager.runAfterInteractions? or just removing the setting of the isLoading state from outside of the runAfterInteractions?

@tienifr Thanks for the proposal. Can you please explain why this happen on jpg images only? (or it does not?)

@tienifr Thanks! Your solution looks good to me, if the image is loaded we should set isLoading to false without further delay.  Based on the code comment we did this to prevent stutter, this was first added by https://github.com/Expensify/App/pull/6442. Can you please use the image in that PR (and others) to navigate through and confirm the animation is smooth?
I have also asked here for better reproduction steps here https://github.com/Expensify/App/pull/6442#discussion_r1219621596.

@tienifr There seems to be some lag. Can you please attach another video that keeps the InteractionManager.runAfterInteractions delay.

@tienifr I think we still have some lag in the second video as well. Do you see that?

@tienifr Since the lag is not a regression here we can move forward and handle the lag bug separately.
:ribbon: :eyes: :ribbon: C+ reviewed
cc @bondydaa

Invited @tienifr and @s77rt

Cool, good to know - cheers @tienifr . We'll add the 50% bonus then.

Sent offers to @tienifr and @s77rt in Upwork.

All paid, thanks everyone!


Customizable Over-Scroll

It would be really nice if I could tweak the amount of over-scroll that is allowed in Windows.
With overscroll, I mean how in MacOS when I scroll and I hit the end of a page the inertia causes the page to scroll slightly beyond its real boundaries to simulate the physical effects the weight would have.
Understand that some people might not prefer that so would never end up in real Windows, but to me, it makes the overall design feel more polished and think it would work really nicely with the Fluent design language.

Hi,
Thank you for the suggestion.
I'm adding the "Suggested PowerToy" label, as it may be possible to achieve it through a PowerToy.
In any case, the place to send generaç Windows suggestions would be through the Feedback Hub: https://support.microsoft.com/en-us/help/4021566/windows-10-send-feedback-to-microsoft-with-feedback-hub

Good idea but I do not believe this is something PowerToys will implement anytime in the near future based on xref / upvotes.  I think the best way for this feedback is Feedback hub.


"Cannot get url of stream..."

HASS 2022.4 seems to break this integration - all of my cameras say "cannot get url of stream [camera name]..." over a black box instead of the video. a few minutes after a restart, it changes to "restart connection. have re installed master version but no luck. don't seem to have any errors in my logs relating to this.

I checked the latest version. It's working.
I think you using entity: camera.xxx in your config. Via which integration is the camera connected?

i figured it out - something changed in 2022.4 such that if you have your cameras configured via yaml, it no longer ignores a bad still image url (which apparently mine were bad). previously i guess it just ignored it but as of 2022.4 it won't load them. so if anyone else has this issue -- it can be fixed by just removing the still image url from your yaml or, alternatively, just adding them via the generic camera integration in the UI (leaving the field for still image url blank). thanks for everything alex!


HTML extraction: try to identify common elements on page and "substract" these from articles to get cleaner text extraction

Moved to pol-harvester repo, which has a research folder


Homepage nerd days

Description
Change homepage Nerd Days event to APAC date and time

@jaesius It looks like we are continuing with NerdDays promotion on the home page, thanks for making these changes. approved and merged!


Test Finite Markov

Remove let blocks, add a few tests for the toy examples, and test invariance on the exercises.

FYI, we got this message when running this section of code. Should we file the issue? cc: @jlperla


grub module: efiInstallAsRemovable description wrap  in

Motivation for this change
Removed first  <para> and wrapped others in <note> to avoid build failure introduced by: #24978
Things done

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

[X] NixOS
[ ] macOS
[ ] Linux

[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[X] Fits CONTRIBUTING.md.

@vcunat Reverting #24978 still leaves the original issue. Should I make a new pull request for wrapping it in <note>?

I don't think any original issue exist.  I guess the comments are automatically wrapped by <para> </para> pairs.  In any case, we do validate the resulting XML, as that's what got broken.

In any case it's confusing and the situation might be worth investigating and perhaps documenting (though people probably wouldn't get to read that bit).

Looking at it some more, I think the issue is with the website. Before the xml is parsed it gets wrapped in a <div>  instead of a <para> so during parsing the <div> gets ignored and it tries to close a non existent <para>.

Indeed, http://nixos.org/nixos/manual/options.html doesn't validate.


ValidateSet for optional parameters.

I was attempting to validate an optional parameter, but I realized that also more or less makes it mandatory, because now it is forced to have a value. So I attempted to use empty string as a valid possibility, but that also has issues.
So I have two issues here:

If a parameter is not mandatory, why is validate set running when the parameter is not supplied, does it look the same as passing in a null value and therefore cannot be determined if it is absent?
If empty string is a valid value in ValidateSet, then I expect tab completion to work. This would also be fine on mandatory parameters if AllowEmpyString or AllowNull are also provided.

Steps to reproduce
param([ValidateSet('', "Test")]
          [string] $Param)

Expected behavior
Empty string should either show up in tab completion OR tab completion should ignore it, display all non-empty values, and allow the parameter to be left out (i.e. empty).

Actual behavior
Tab completion does NOT work at all when empty string is provided as a valid value. Note: that the set validation DOES work fine, i.e. I can still only provide empty string or "Test" as values and all other values error, but the empty string breaks tab completion. This seems like very confusing behavior, if it works I would expect tab completion, if it is not supported I would expect it error or get ignored all together.

Environment data

> $PSVersionTable
Name                           Value
----                           -----
PSVersion                      6.1.0-preview.2
PSEdition                      Core
GitCommitId                    v6.1.0-preview.2
OS                             Microsoft Windows 10.0.16299
Platform                       Win32NT
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0...}
PSRemotingProtocolVersion      2.3
SerializationVersion           <IP_ADDRESS>
WSManStackVersion              3.0

If a parameter is not mandatory, why is validate set running when the parameter is not supplied,

If you simply omit the parameter, no validation is performed. Do you mean something else by "not supplied"?

If empty string is a valid value in ValidateSet, then I expect tab completion to work

Yes, that cycling through valid values stops working if you tab-complete without having typed at least one characters is unexpected.

Hm... Well I just tested it again and now I am not seeing what I saw before. I can close this as it does appear to work as I wanted it to.

@Tim Biggin
I am facing similar issue,Can you help us with the solution that you've tried to resolve this issue?
Thanks

@vivekgite what's the issue you're seeing?
In general terms, if you don't mark a parameter as mandatory, and you also apply a ValidateSet, the validation is only performed if and when the parameter is supplied.

@vivekgite I didn't really have a solution, the problem just went away when I went to test it again.


Lists and blockquotes should be inside the text block

Lists, numbered lists and blockquotes should be intended and not outside the text block (see discussion in skohub/skohub-blog).
Functional review: @acka47
Code review: @fsteeg

These changes can also be reviewed in the deployment to https://metafacture.github.io/metafacture-blog/

Hmm, at https://metafacture.github.io/metafacture-blog/new-beginnings/ bullet points and quotes are not indented,

These changes can also be reviewed in the deployment to https://metafacture.github.io/metafacture-blog/

Sorry, that isn't correct. These are two different branches so these changes can't be reviewed on  https://metafacture.github.io/metafacture-blog/.
Please git checkout -b 4-listsAndBlockquotes origin/4-listsAndBlockquotes and run gatsby develop to review.

+1


Add a testing utility library

Right now the testing utilities TestTask and Temporary.swift are part of different unrelated libraries. We should split them into its own testing library so we import that library only in tests.

Somewhat related https://github.com/fdiaz/SwiftInspector/issues/16, since we may be able to remove Temporary.swift?


Licenses mismatch

The license file is of apache 2.0, but package.json states that license is ISC. These should be the same

I was trying to find the license when I was creating by init, but I couldn't. Does typing the name of the license on the package.json valid?

it is valid if you type the correct thing :)


Example for AI backed client

Not tied to this implementation specifically, but this was just a proof of concept on how you could start incorporating LLMs into a brainstorm server.

@mr-joshcrane many thanks for contributing!


Remove another google-http-client dependency from geoip2

This pulls in bad dependencies that can make GCP extensions have classloader problems.

already fixed in master