Supporting new Bitcoin core version v0.16.1

For rpc  command    (listunspent, sendrawtransaction )
ListUnspentAsync  (query_options    )
SendRawTransactionAsync not returning transaction  id

listunspent ( minconf maxconf  ["addresses",...] [include_unsafe] [query_options])

Returns array of unspent transaction outputs
with between minconf and maxconf (inclusive) confirmations.
Optionally filter to only include txouts paid to specified addresses.

Arguments:
1. minconf          (numeric, optional, default=1) The minimum confirmations to filter
2. maxconf          (numeric, optional, default=9999999) The maximum confirmations to filter
3. "addresses"      (string) A json array of bitcoin addresses to filter
    [
      "address"     (string) bitcoin address
      ,...
    ]
4. include_unsafe (bool, optional, default=true) Include outputs that are not safe to spend
                  See description of "safe" attribute below.
5. query_options    (json, optional) JSON with query options
    {
      "minimumAmount"    (numeric or string, default=0) Minimum value of each UTXO in BTC
      "maximumAmount"    (numeric or string, default=unlimited) Maximum value of each UTXO in BTC
      "maximumCount"     (numeric or string, default=unlimited) Maximum number of UTXOs
      "minimumSumAmount" (numeric or string, default=unlimited) Minimum sum value of all UTXOs in BTC
    }

Result
[                   (array of json object)
  {
    "txid" : "txid",          (string) the transaction id
    "vout" : n,               (numeric) the vout value
    "address" : "address",    (string) the bitcoin address
    "account" : "account",    (string) DEPRECATED. The associated account, or "" for the default account
    "scriptPubKey" : "key",   (string) the script key
    "amount" : x.xxx,         (numeric) the transaction output amount in BTC
    "confirmations" : n,      (numeric) The number of confirmations
    "redeemScript" : n        (string) The redeemScript if scriptPubKey is P2SH
    "spendable" : xxx,        (bool) Whether we have the private keys to spend this output
    "solvable" : xxx,         (bool) Whether we know how to spend this output, ignoring the lack of keys
    "safe" : xxx              (bool) Whether this output is considered safe to spend. Unconfirmed transactions
                              from outside keys and unconfirmed replacement transactions are considered unsafe
                              and are not eligible for spending by fundrawtransaction and sendtoaddress.
  }

sendrawtransaction "hexstring" ( allowhighfees )

Submits raw transaction (serialized, hex-encoded) to local node and network.

Also see createrawtransaction and signrawtransaction calls.

Arguments:
1. "hexstring"    (string, required) The hex string of the raw transaction)
2. allowhighfees    (boolean, optional, default=false) Allow high fees

Result:
"hex"             (string) The transaction hash in hex

When do you planning implement it ?

I don't know, feel free to make a PR, it does not seems to complicated, I would merge fast.

I tried to implement it. But there were various errors with parsing Json.
NBitcoin.RPC.RPCException: JSON value is not an object as expected
RPC_MISC_ERROR

Take example on GetNewAddress where several options are passed.

Same error (
I can not send JSON
query_options    (json, optional) JSON with query options
    {
      "minimumAmount"    (numeric or string, default=0) Minimum value of each UTXO in BTC
      "maximumAmount"    (numeric or string, default=unlimited) Maximum value of each UTXO in BTC
      "maximumCount"     (numeric or string, default=unlimited) Maximum number of UTXOs
      "minimumSumAmount" (numeric or string, default=unlimited) Minimum sum value of all UTXOs in BTC
    }

How I can forming this query  { "minimumAmount": 0.005 }
{"jsonrpc":"1.0","id":1,"method":"listunspent","params":[1,9999,[],true,{"minimumSumAmount": 1}]}
is not valid json for json writer

Use named arguments, as I did inside GetNewAddressAsync(GetNewAddressRequest request).
As value of query_options, create a JObject with the right properties.


WASM zopfli-png

https://github.com/subzey/zopfli-png

Why not zopflipng: https://github.com/google/zopfli

The repo linked by @jakearchibald is literally zopfli but applied to PNGs.
It kinda applies to all PNG codecs, so it’s something that we’d have to re-think in terms of UI, but we definitely want to ship it.

What I meant was that there is already a zopflipng tool inside the official zopfli git. It is the tool I use, it compresses very well, but is very slow.

Yeah, I probably meant to link to the Google one.

Merging into #237


1.0.0-alpha.17

Codecov Report

Merging #98 into master will increase coverage by 12.37%.
The diff coverage is 33.33%.

@@             Coverage Diff             @@
##           master      #98       +/-   ##
===========================================
+ Coverage   64.72%   77.09%   +12.37%
===========================================
  Files          52       52
  Lines        2126     2131        +5
===========================================
+ Hits         1376     1643      +267
+ Misses        750      488      -262

Impacted Files
Coverage Δ

src/service/grpc-service.js
32.43% <ø> (ø)
:arrow_up:

src/config.js
60% <33.33%> (-40%)
:arrow_down:

src/model/protos.js
81.81% <0%> (+1.81%)
:arrow_up:

src/model/operation.js
88% <0%> (+2%)
:arrow_up:

src/cli/commands/delete.js
88.23% <0%> (+2.94%)
:arrow_up:

src/model/cloudfunction.js
88.67% <0%> (+3.77%)
:arrow_up:

src/utils/errors.js
67.27% <0%> (+16.36%)
:arrow_up:

src/service/rest-service.js
89.89% <0%> (+35.35%)
:arrow_up:

src/supervisor/supervisor.js
71.65% <0%> (+36.61%)
:arrow_up:

src/model/functions.js
68.85% <0%> (+50.81%)
:arrow_up:

Continue to review full report at Codecov.

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


chore: prepare async-compression release 0.4.0

@NobodyXu @Nemo157 Are there any other breaking changes from the open issues/PRs that we'd like to get in to a v0.4.0 release?

[x] #179
[x] #134

Draft release: https://github.com/Nullus157/async-compression/releases/tag/untagged-8fbf8524f930fef3d14c

I guess we'll have to wait for the GHA indcident to be over: https://www.githubstatus.com/incidents/pr3498h3qkfy

https://github.com/Nullus157/async-compression/releases/tag/async-compression-v0.4.0
https://crates.io/crates/async-compression
https://docs.rs/async-compression/0.4


chore(main): release 23.0.0

:robot: I have created a release beep boop
23.0.0 (2024-09-08)
⚠ BREAKING CHANGES

renderer: Added missing Obsidian callouts

Features

Added simple Latex support (92f5320), closes #138
renderer: Added missing Obsidian callouts (ea71a5b), closes #135

Bug Fixes

Fixed a bug causing deleted buffers to not be removed from the attached buffers list (e4b4b9d), closes #137
Fixed how widths are calculated in code blocks (59c15a2)
Split view no longer loses it's state when changing plugin state with :Markview ... (d6f0cb8)

This PR was generated with Release Please. See documentation.

:robot: Created releases:

v23.0.0
:sunflower:


updating olm descriptors to remove deprecations

CC @scholzj
Updating CSV to replace deprecated OLM descriptors and new best-practices as detailed here.
Signed-off-by: Jose R. Gonzalez<EMAIL_ADDRESS>(Not sure if my sign-off is actually what's required here)
Updates to existing Operators

[ ] ~Is your new CSV pointing to the previous version with the replaces property?~
[ ] ~Is your new CSV referenced in the appropriate channel defined in the package.yaml ?~
[ ] ~Have you tested an update to your Operator when deployed via OLM?~
[X] Is your submission signed?

Your submission should not

[x] Modify more than one operator
[ ] Modify an Operator you don't own
[x] Rename an operator - please remove and add with a different name instead
[x] Submit operators to both upstream-community-operators and community-operators at once
[x] Modify any files outside the above mentioned folders
[x] Contain more than one commit. Please squash your commits.

Operator Description must contain (in order)
This has not changed

[ ] ~Description about the managed Application and where to find more information~
[ ] ~Features and capabilities of your Operator and how to use it~
[ ] ~Any manual steps about potential pre-requisites for using your Operator~

Marked as draft until maintainers can have a look and confirm if they want to move forward.

This looks good to me. Thanks for your help @komish

Bumping to Ready for Review

FYI: The CVP/operator-scorecard-tests test is failing because of a bug in Fabric8 Kubernetes client when dealing with OCP 4 authentication tokens. It works fine when running as a pod.

FYI: The CVP/operator-scorecard-tests test is failing because of a bug in Fabric8 Kubernetes client when dealing with OCP 4 authentication tokens. It works fine when running as a pod.

I'm reading this as: "no change required from the PR's perspective". I'm assuming this remains OK to merge as we don't have influence on the change to Fabric8's client code (directly) to resolve. Otherwise, let me know!

Yeah, this should be ok to merge - previous PRs for 0.14.0 with this were merged fine. (this should be fixed in our upcomming 0.15.0 release which has newer version of Fabric8 Kubernetes client)

@komish, please check error from the log:
ClusterOperator:116 - Started operator for KafkaConnect kind
WatchConnectionManager:201 - Exec Failure
java.io.IOException: Unexpected response(403 Forbidden), to the authorization request. Missing header:[Location]!
at io.fabric8.openshift.client.internal.OpenShiftOAuthInterceptor.authorize(OpenShiftOAuthInterceptor.java:133) ~[io.fabric8.openshift-client-4.3.0.jar:?]

@J0zi see @scholzj's comments here: https://github.com/operator-framework/community-operators/pull/881#issuecomment-554106168 and https://github.com/operator-framework/community-operators/pull/881#issuecomment-554394015. This appears to be due to a bug in an upstream library preventing the test from passing in strimzi 0.14.0, but the test should pass in Strimzi's future release (0.15.0).


Err-11:CL_BUILD_PROGRAM_FAILURE in Vina-GPU+

I got Err-11:CL_BUILD_PROGRAM_FAILURE when I do Run on the executable file.
How can I fix it.

This is the output I get when trying to run Vina-GPU+.
It stops halfway, no error, no Kernel output.
I noticed vGPU+ filesize is smaller than vGPU. Is it because of this or it is meant to be smaller?
$ ./Vina-GPU+_K.exe --config=./config.txt --ligand PL_001_AA.pdbqt
#################################################################
If you used Vina-GPU in your work, please cite:

#Shidi, Tang, Chen Ruiqi, Lin Mengru, Lin Qingde,               #
#Zhu Yanxiang, Wu Jiansheng, Hu Haifeng, and Ling Ming.         #
#Accelerating AutoDock VINA with GPUs. ChemRxiv (2021).Print.   #

#DOI 10.26434/chemrxiv-2021-3qvn2-v3                            #

And also the origin AutoDock Vina paper:
O. Trott, A. J. Olson,
AutoDock Vina: improving the speed and accuracy of docking
with a new scoring function, efficient optimization and
multithreading, Journal of Computational Chemistry 31 (2010)
455-461

DOI 10.1002/jcc.21334

#################################################################
WARNING: The search space volume > 27000 Angstrom^3 (See FAQ)
Using single ligand docking mode
Output will be PL_001_AA_out.pdbqt
Reading input ... done.
Setting up the scoring function ... done.
Search_depth is fixed to 10
Analyzing the binding site ... done.
GPU Platform: NVIDIA CUDA
GPU Device: NVIDIA GeForce RTX 2060 SUPER
Using random seed:<PHONE_NUMBER>
Office PC_Hadi@AHAFOfficePC /cygdrive/c/Users/Public/VINATESTGPUPLUS/10anRun+
$

Hi, We have updated Vina-GPU+_K.exe, can you try it again?


Beta

Alaska Airlines Pull Request
Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context. List any dependencies that are required for this change.
Resolves: # (issue, if applicable)
Summary:
Please summarize the scope of the changes you have submitted, what the intent of the work is and anything that describes the before/after state of the project.
Type of change:
Please delete options that are not relevant.

[ ] New capability
[ ] Revision of an existing capability
[ ] Infrastructure change (automation, etc.)
[ ] Other (please elaborate)

Checklist:

[ ] My update follows the CONTRIBUTING guidelines of this project
[ ] I have performed a self-review of my own update

By submitting this Pull Request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
Pull Requests will be evaluated by their quality of update and whether it is consistent with the goals and values of this project. Any submission is to be considered a conversation between the submitter and the maintainers of this project and may require changes to your submission.
Thank you for your submission!
-- Auro Design System Team
Summary by Sourcery
Introduce theme support and refactor the auro-select component to use a new versioning system for dependencies. Add SSR support and update documentation and tests to reflect these changes. Remove deprecated scripts and improve build configuration with a new ESLint setup.
New Features:

Introduce theme support for the auro-select component, allowing it to be restyled using new SCSS tokens.

Bug Fixes:

Add extension for SSR support to ensure compatibility with server-side rendering environments.

Enhancements:

Refactor the custom component registration to use a new versioning system for dependency components.
Update the auro-select component to use a custom version of the dropdown, improving performance and maintainability.
Remove CSS custom property fallbacks and update styles to use new design tokens.

Build:

Add a new ESLint configuration file to improve code quality and consistency.

Documentation:

Update documentation to reflect changes in component registration and theme support.
Remove outdated sections related to CSS custom property fallbacks.

Tests:

Update tests to accommodate changes in the dropdown component's query selector.

Chores:

Remove deprecated husky script and update demo files to reflect new component registration method.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.1 out of 2 committers have signed the CLA.:white_check_mark: jordanjones243:x: semantic-release-botYou have signed the CLA already but the status is still pending? Let us recheck it.

Work completed in this PR: https://github.com/AlaskaAirlines/auro-select/pull/206


Force variant not working

I found an issue in the documentation.
To force a variant, we need to prefix it with gopt_
const` key = `${COOKIE_PREFIX}_${experimentName}`;

Do you mean forcing a variant via url query param?

Sorry for the delay @fabiofdsantos, It's just in the readme:
Force a specific variant via url or param. E.g. url?experiment-x=1 or this.$exp('experiment-x', 1);

Should be url?gopt_experiment-x=1 not url?experiment-x=1


fix: marshalling list in generic

Before it was impossible to marshal a message with list as a field in a struct.

Please refer to the PR specification to fill in the title and description, refer to the example: https://github.com/cloudwego/kitex/pull/632

And supplement ut.

I think fixed.


List requiremets for the cross-memory PDSE dataset

Is your request for enhancement related to a problem? Please describe.

@brooklynakos was deploying the cross-memory server's load library and found that the documentation did not provide all the reuiqrements for the PDSE dataset used by the cross-memory server.
Describe the solution you'd like

The documentation needs to provide all PDSE requirements in the "Manually installing the Zowe Cross Memory Server" section.

This should be address with the SZWEAUTH included with Zowe LTS and the doc for convenience and SMP/e includes the attributes required for the APF Auth PDSE


Support Firebase event types

Fixes https://github.com/GoogleCloudPlatform/cloud-functions-emulator/issues/84
To deploy a Firebase database function
Create a .runtimeconfig.json in your functions directory that look like:
{
	"firebase": {
		"databaseURL": "https://project-id.firebaseio.com"
	}
}

Then run:
functions deploy <functionName> --trigger-provider=google.firebase.database
 --trigger-event=ref.write

To call a Firebase database function
# data field must be an object with a data field (representing the previous state)
and a delta field (representing the new state). The command below mocks an event
of a new write of "hello" at /path
functions call <functionName> --data='{"data":null,"delta":"hello"}'
 --resource=projects/_/instances/project-id/refs/path

Similar process for deploying/calling other Firebase events.
Emulating database functions will be coming soon in the Firebase CLI, which will be easier to use since there won't be need for as many flags, and there wouldn't be a need to create the .runtimeconfig.json

Hey @jmdobry the CircleCI build error appears to be unrelated to my code since it's complaining about the emulator not being started, and the commit that broke the build only changed some command options. Would you mind taking a look?

Codecov Report

Merging #121 into master will decrease coverage by 13.08%.
The diff coverage is 8.69%.

@@             Coverage Diff             @@
##           master     #121       +/-   ##
===========================================
- Coverage   77.52%   64.44%   -13.09%
===========================================
  Files          52       52
  Lines        2127     2146       +19
===========================================
- Hits         1649     1383      -266
- Misses        478      763      +285

Impacted Files
Coverage Δ

src/cli/commands/deploy.js
72.72% <ø> (+0.66%)
:arrow_up:

src/client/rest-client.js
97.72% <ø> (ø)
:arrow_up:

src/client/grpc-client.js
92.64% <ø> (ø)
:arrow_up:

src/service/rest-service.js
50% <0%> (-39.9%)
:arrow_down:

src/service/grpc-service.js
30.37% <0%> (-2.06%)
:arrow_down:

src/cli/commands/call.js
75.6% <100%> (ø)
:arrow_up:

src/cli/controller.js
74.49% <12.5%> (-0.92%)
:arrow_down:

src/model/functions.js
18.25% <0%> (-51.46%)
:arrow_down:

src/supervisor/supervisor.js
35.03% <0%> (-36.62%)
:arrow_down:

... and 7 more

Continue to review full report at Codecov.

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

Never mind, build has now been fixed, but I'm still confused by why https://github.com/GoogleCloudPlatform/cloud-functions-emulator/pull/121/commits/46a57518c4e121f0ad11718f04a82d930b948fb9 broke the build.

I am OOO, but I'll get to this as soon as I can.

This won't work. The CallFunctionRequest message in the Cloud Functions proto only supports the name and data fields, and we can't add other fields to the message. This is why the gcloud beta functions call command in the Gcloud SDK does not accept resource, params, or auth options from the user.
I think for now (until Cloud Functions changes) the Emulator will need to look at the configured trigger-event/provider of the deployed function and attach some mock metadata to the event payload on its own. Or, think of another way for the user to provide the event metadata to the function without using the call command.

It seems to work just fine, here's me testing with this branch. My functions code looks like this:
exports.dbFunc = functions.database.ref('/path').onWrite(function(e) {
	console.log('Event is: ', e);
	console.log('Auth is: ', e.auth.admin);
});

From the screenshot, you can see that the printed out event has its auth, param, and resource fields filled out the way I specified.

That's because you're using rest, if you set service to grpc, it will break.

@jmdobry  Removed the gRPC changes as we discussed, and also removed the additional flags I previously added to the call command.

Yes good idea. I see that there are end-to-end system tests, and unit tests for model and supervisor. I'm not sure this fits into either of these categories. Since no new commands were added, end-to-end tests wouldn't makes sense. Should I add a new folder under "test/unit" for the controller?

You can either add unit tests or add more tests to the e2e tests.

@jmdobry Sorry this took me some time. I added unit tests.


Google Interview Questions updated.

Hi,
I have updated the Google interview questions . I faced these questions during my interview with Google for the Site Reliability Engineer position. Please merge these changes.
Thanks,
Richa

Hi,
Sorry for the typo. I just corrected it. Now, these changes can be merged.
Thanks,
Richa


Frontend: Migrate to WalletConnect v2

Rationale
The frontend currently uses WalletConnect as one of the main interfaces, besides MetaMask, for connecting to different wallet providers (both mobile & desktop).
So far everything works as expected by using WalletConnect v1 however this version of the library will soon become deprecated and it's sunset is already in progress.
dApps need to start migrating to WalletConnect v2 beginning of January.
Announcement
https://medium.com/walletconnect/walletconnect-v1-0-sunset-notice-and-migration-schedule-8af9d3720d2e
Implementation
We need to migrate to WalletConnect v2
Acceptance Criteria

[ ] Migrated to WalletConnect v2

what will happen if we do not migrate? @GabrielBuragev short term / long term?

Ok. So we will need to do this before March. Got it.

blocked until official launch of v2 of wallet connect

set a calander entry to remind us.

@GabrielBuragev scope out what needs to be done here and then deal with #1956

Closed in https://github.com/beamer-bridge/beamer/pull/1972


Install libpython2.7-dev

This fixes the top-level Swift command.
Before:
% docker run --rm -it swiftlang/swift swift
/usr/bin/lldb: error while loading shared libraries: libpython2.7.so.1.0: cannot open shared object file: No such file or directory

After:
% docker run --rm -it swiftlang/swift swift
Welcome to Swift version 2.2-dev (LLVM 46be9ff861, Clang 4deb154edc, Swift 778f82939c). Type :help for assistance.
  1>

Fixes #1
Replaces #2

Thanks for the swift merge!

No problem, thanks for the fix!


Add init_system API for multi-host GPU.

Currently JAX doesn't expose our multi-host GPU backend to our open source users.
This PR exposes an experimental API jax.distributed.initialize to initialize the multi-host GPU backend.
I tested it on 2 GPU VMs on GCP
== VM0
$ TF_CPP_MIN_LOG_LEVEL=0 python -c "import jax; jax.distributed.initialize('<IP_ADDRESS>:1456', 2, 0)"
2021-10-26 01:19:17.133471: I external/org_tensorflow/tensorflow/compiler/xla/pjrt/distribute
d/service.cc:369] Jax service listening on <IP_ADDRESS>:1456
2021-10-26 01:19:28.339404: I external/org_tensorflow/tensorflow/compiler/xla/pjrt/distribute
d/client.cc:129] Connected to distributed JAX controller
2021-10-26 01:19:28.375758: I external/org_tensorflow/tensorflow/compiler/xla/pjrt/distribute
d/client.cc:163] Waiting for all distributed JAX tasks to shut down.
2021-10-26 01:19:28.376179: I external/org_tensorflow/tensorflow/compiler/xla/pjrt/distribute
d/client.cc:180] Distributed task shutdown result: OK
2021-10-26 01:19:28.397722: I external/org_tensorflow/tensorflow/compiler/xla/pjrt/distribute
d/service.cc:381] Jax service shutting down

== VM1
$ TF_CPP_MIN_LOG_LEVEL=0 python -c "import jax; jax.distributed.initialize('<IP_ADDRESS>:1456', 2, 1)"
2021-10-26 01:19:28.339474: I external/org_tensorflow/tensorflow/compiler/xla/pjrt/distribute
d/client.cc:129] Connected to distributed JAX controller
2021-10-26 01:19:28.371461: I external/org_tensorflow/tensorflow/compiler/xla/pjrt/distribute
d/client.cc:163] Waiting for all distributed JAX tasks to shut down.
2021-10-26 01:19:28.376473: I external/org_tensorflow/tensorflow/compiler/xla/pjrt/distribute
d/client.cc:180] Distributed task shutdown result: OK

An example of this API in action, see code that does model parallelism on 2 GPU VMs
https://gist.github.com/zhangqiaorjc/0ae6e7114fb0b3e9243e6420e4d6f3e4
See screenshot for results
https://photos.google.com/share/AF1QipMfIpFOpmckl86lU4WS4nb2IzMDkrOqLyafa4C3Vx7zMqoyy6NOM8PiS8gH7zaLIw?key=bjVRWHZoRmFUTkhhLVBOdzFlYWg4bG5nZ3NJYVpB


feat: add verify toggle for Kibana Shorten URL API

Description

Checklist

[x] I have reviewed the contributing guidelines.
[x] I have included unit tests for my changes or additions.
[x] have successfully run make test-docker with my changes.
[x] I have manually tested all relevant modes of the change in this PR.
[x] I have updated the documentation.
[x] I have updated the changelog.

Questions or Comments

Unsure how to test it via tests (or if this needs to be tested with unit tests), let me know :)

Thanks for merging this! When do you think a new version will be released?

Releases are normally every 1-2 months. The last release was 4 weeks ago, so the next will probably be within 2 weeks.

Hello, 2.9.0 was released this morning.


Update documentation

Updates documentation of button and toolbar

Thanks

:tada: This PR is included in version 2.0.0-alpha.1 :tada:
The release is available on:

npm package (@develop dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


State download conflict with multiple terraspace executions running simultaneously

Checklist

[X] Upgrade Terraspace: Are you using the latest version of Terraspace? This allows Terraspace to fix issues fast. There's an Upgrading Guide: https://terraspace.cloud/docs/misc/upgrading/
[X] Reproducibility: Are you reporting a bug others will be able to reproduce and not asking a question. If you're unsure or want to ask a question, do so on https://community.boltops.com
[ ] Code sample: Have you put together a code sample to reproduce the issue and make it available?  Code samples help speed up fixes dramatically. If it's an easily reproducible issue, then code samples are not needed. If you're unsure, please include a code sample.

Difficult to reproduce but comes down to the fact that the environment is not getting specied
My Environment

Software
Version

Operating System
Ubuntu 20.04

Terraform
1.0.0

Terraspace
0.6.11

Ruby
2.7.0

Expected Behaviour
Multiple invocations on the same stack with different environments should be able to execute simultaneously on the same machine.
I've got a complex stack of 8 stages that is being deployed to multiple environments (dev, test, staging, prod).  When running an apply on multiple environments on the same machine simultaneously causes the executions to get corrupted.
Current Behavior
Depending on your timing you can have the code work or you'll get ERB errors trying to resolve values from the state.
The issue is caused by how terraspace stores its material in /tmp for each invocation.
Errno::ENOENT: No such file or directory @ rb_sysopen - /tmp/terraspace/remote_state/stacks/stackname/state.json

The generated plans are unique enough since they have their filename appended with a random hex string as per:
lib/terraspace/cli/up.rb:      "#{Terraspace.tmp_root}/plans/#{@mod.name}-#{@@random}.plan"

Step-by-step reproduction instructions
Difficult to reproduce since it is a bit reliant on race conditions but take a complex stack and run
terraspace all init --exit-on-fail -y && terraspace all plan -y --exit-on-fail

For multiple environments in different shells on the same machine.
Code Sample
N/A
Solution Suggestion
The downloaded state files, plans, and other material stored in /tmp/terraspace should be separated by environment as well.  If there are files common to all terraspace invocations they can still be placed in the root (since you want to be able to run subsequent plan/refresh/validate/up after running an init, meaning they need a predictable path to store their material). Splitting by environment alone should be sufficient to allow multiple invocations.

Could force setting TS_TMP_ROOT prior to each invocation for now to work around this issue.

Hi, any idea to when this will be solve ?

We have 2 scenarios here, maybe was wrote above in any case let me say:
a) 2 or more developers in their /home/$users/devel/terraspace_project/stacks, can't do apply/plan because in /tmp/terraspace the permissions is only applied to first one, the other receive permission denied. Our workaround is set chmod 777 /tmp/terraspace -R
b) using this workaroung we notice the other issue for example, if developer1, use Terraspace_project-1/stackname_alpha and developer2 use Terraspace_project-5/stackname_alpha (same name), even here exist a conflict because isn't identified wich stacks by project would be in /tmp/terraspace and could overwrite critical states.
for the second we don't know how solve with workaround.
TS_TMP_ROOT environment variable didn't see in the reference documentation.

is possible use tmp inside .terraspace-cache wich is inside of project when do plan/up and not use /tmp/terraspace ?

I'm trying to avoid this issue by setting TS_TMP_ROOT to different paths for different projects. I am also setting TS_CACHE_ROOT to different paths for each project.
example project A:
export TS_TMP_ROOT="$HOME/.myproject_A/terraspace"
export TS_CACHE_ROOT="$HOME/.myproject_A/terraspace-cache"

example project B:
export TS_TMP_ROOT="$HOME/.myproject_B/terraspace"
export TS_CACHE_ROOT="$HOME/.myproject_B/terraspace-cache"

Doing so seems to be steps toward a feasible solution, except that some files are still being created under /tmp/terraspace/ due to the following line in rewrite.rb not properly using Terraspace.tmp_root:
https://github.com/boltops-tools/terraspace/blob/20f733b26f08f6dd959ece62765e1d864cea51c2/lib/terraspace/compiler/erb/rewrite.rb#L11-L12
Suggestion: It seems that line # 11 above should use Terraspace.tmp_root like how it is used here:
https://github.com/boltops-tools/terraspace/blob/20f733b26f08f6dd959ece62765e1d864cea51c2/lib/terraspace/terraform/args/thor.rb#L148
Suggestion: Terraspace.tmp_root should probably also be used here:
https://github.com/boltops-tools/terraspace/blob/20f733b26f08f6dd959ece62765e1d864cea51c2/lib/terraspace/terraform/api/vars/json.rb#L19


Wallet.dat creation with mnemonic seed words (BIP39)

Upon startup without existing wallet.dat, allow user to create or restore the private keys with mnemonic seed words.

I support it

Implemented with 4.1


Terraform & Functional Tests for AWS Braket Plugin

Acceptance Criteria

Create a new subdirectory in https://github.com/AgnostiqHQ/terraform with the name covalent-braket-plugin, and initialize terraform in that subdirectory
Update main.py script generated by terraform cli in order to provision required cloud resources
In the corresponding plugin repo (not terraform repo) add a folder ./tests/functional_tests and add appropriate functional tests for the plugin which should include dispatching a non-trivial workflow as well as validating that various config options are being used as intended
Cost estimates if possible (tool TBD)

Please add your planning poker estimate with ZenHub @cjao

related https://github.com/AgnostiqHQ/covalent-braket-plugin/pull/14


Can't change the locale

I tried to change the locale in 'fr' and 'es' but nothing change. I also tried different format like 'FR', 'fr_FR' but nothing works.
I went to check the demo here : https://farhoudshapouran.github.io/react-native-ui-datepicker/ and event here when we click on locale buttons it doesn't change the language.
Have someone the same mistake ?
<DateTimePicker
                    mode="single"
                    locale="fr"
                    firstDayOfWeek={1}
                    date={birthDate}
                    onChange={(response) => {
                      setBirthDate(new Date(response.date.toString()))
                      toggleDatePicker()
                    }}
                    selectedItemColor={colors.mainColors.mainColor}
/>

add this on top,
require('dayjs/locale/fr');

thank you ! 😃


Add output option to control where the csv file should be stored

Adds the --output option to the video-demo command line program allowing the user to specify where to store the csv file with the metrics data.

Hi Milania, your PR got closed unintentionally, as a side-effect of some housekeeping in this repo.  I'll see if I can resurrect it somehow.


merge CHEBI:1893 and CHEBI:20440

Please merge.

The structure of CHEBI:1893 was incorrect. The entry was for the glutamate but the structure had the acid group. I have now updated the structure hence these are now different structures.


Multiple instances using clone

For a responsive page I've set up an on-canvas inline version of the code for the menu on larger screens, which works really well (took a fair bit of css tweaking).
As the menu is huge (800 categories) I want to use the clone function to create a second version for smaller screens, which is off canvas, so that it does not have to be downloaded twice. I need both to run and use different classes so that they can have their own styles etc.
I don't appear to be able to get both offcanvas: false and clone to work at the same time.
So my question is... How do I use the clone to make two working instances on the same page with their own configurations and classes?
        $("#menu").mmenu({
           "header": {
              "title": "Categories",
              "add": true,
              "update": true
           },
           offCanvas: false,
        }, {
           // configuration
           clone: true,
        });

The clone configuration option is indead only used by theoffcanvas add-on.
You can ofcourse clone the menu manually, here's a short tutorial:
http://mmenu.frebsite.nl/support/tips-and-tricks.html#tt-responsive-layouts
Note that now you probably need to set an unique ID for all elements in the cloned menu:
var $menu = $("#my-menu").clone();
$menu.add( $menu.find( "[id]" ) ).each(function() {
    $(this).attr( "id", "clone-" + $(this).attr( "id" ) );
});

By the way, for an on-canvas version of the menu on larger screens, you could probably also use the "widescreen" extension: http://mmenu.frebsite.nl/documentation/extensions/widescreen.html

That's great thanks..
I have two small issues now.
1st is that I seem to be able to get both off canvas or on canvas but not different. The large screen inline version needs to be on canvas and the small screen version needs to be off-canvas.
2nd is that no matter which I click the links on only the top menu changes even with the different id's. Both load correctly but only the first menu changes.
  $(document).ready(function() {
        $("#menu").mmenu({
           "header": {
              "title": "Categories",
              "add": true,
              "update": true
           },
           offCanvas: false,
        });
      var $menu = $("#menu").clone();
      $menu.attr( "id", "my-mobile-menu" );
      $menu.attr( "class", "my-mobile-menu" );
      $menu.mmenu({
           "header": {
              "title": "Categories",
              "add": true,
              "update": true
           },
           offCanvas: true,
        },
      });
  });

I'm currently working on a couple of other tutorials as well, so I'll probably include an onCanvas menu as well.
Having seen what you're JS looks like, I think you might be better of using the "widescreen" extension. Have you tried it? You can see it in action on http://mmenu.frebsite.nl (simply resize your browser to > 1430px)


Fwiffo's Rareden Skybox

Hi,
You guys are hosting FwiffoRaredenSkybox-1.0.ckan in the official repository.  This file is marked with ksp_version_max 1.1.99.  I've done some playtesting and it works fine under KSP 1.2 with no changes.  Could you update it to be available in 1.2?
Also for my own education, is it appropriate in a case like this to simply change the ksp_version_max in that file to 1.2.99, or should we generally avoid changing ckan's after they're published and a new FwiffoRaredenSkybox-1.1.ckan be created?
Thanks!

Mod authors set the version numbers. If they haven't released a new version, then we change the existing .ckan file. When it is the current release, we change it via the .netkan file, since otherwise the Netkan inflation robot will overwrite any changes.


Added Tabular SageMaker tutorial link

Issue #, if available:
Description of changes:
Added Tabular SageMaker tutorial link
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Job PR-430-1 is done.
Docs are uploaded to http://autogluon-staging.s3-website-us-west-2.amazonaws.com/PR-430/1/index.html

Job PR-430-2 is done.
Docs are uploaded to http://autogluon-staging.s3-website-us-west-2.amazonaws.com/PR-430/2/index.html


Eliminate refresh all

Is your feature request related to a problem? Please describe.
Refresh all is too slow and bug-prone, should be eliminated
Describe the solution you'd like
Make sure that we can do everything uncommon that refresh all normally does in other ways

Be able to add new organizations (should be able to do through whitelist wizard or manual register)
Be able to add self to workflows others have created that you should have accessible (refresh by organization or sync for .dockstore.yml workflows)

Then eliminate the refresh all button in the UI and eliminate the endpoint/orphaned code
┆Issue is synchronized with this Jira Story
┆Issue Type: Story
┆Fix Versions: Dockstore 1.9
┆Sprint: Sprint 31 Flounder
┆Issue Number: DOCK-1251

Verified that endpoints and UI components relating to refresh all are gone as of:
API - 1.9.0-alpha.8-SNAPSHOT
UI - 2.5.1-rc.1-34-gd99a76c6


Does not work with github redesign

After GitHub's recent redesign, there are no extra options in the commit history section of your profile page.

Here are my instructions to install an updated, working version. Tested it just now, it's working :smile:
https://github.com/ben174/git-draw/pull/12#issuecomment-737366020


BUG: Can't swap on android 14

Describe the bug
Can't swap any asset
To Reproduce
Steps to reproduce the behavior:

Go to asset any
Click on button swap
Chosen asset to swap
Chosen amount
See error: Invalid amount

Expected behavior
Can swap
Screenshots

**Smartphone

Device: Redmi K60
OS: Android 14
Version app: 1.0.32

@LongCX Can't reporduce it


nixos/influxdb2: set timezone data environment variable

Description of changes
InfluxDB's Flux query language (and upsteam of it, Go) reads the timezone database from the $ZONEINFO environment variable instead of $TZDIR. $TZDIR is set as part of the systemd unit builder. Attempting to use a timezone in a query without $ZONEINFO set will not load the database:
import "timezone"

option location = timezone.location(name: "Australia/Sydney")
// Query error: invalid: error calling function "location" ...: unknown time zone Australia/Sydney

See influxdata/flux#4382 (relating to Windows specifically, but is useful information for non-FHS environments like NixOS) and https://pkg.go.dev/time#LoadLocation (upstream documentation) for context.
Things done

Built on platform(s)

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

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

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

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

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

[x] Fits CONTRIBUTING.md.

@NickCao Can you take a look as maintainer?


Clean up source file headers (resolves #309, #310)

Makes two changes:

Attaches the ASF suggested license file header to all source files. This license header attributes copyright to the Regents of the University of California. Let me know if someone would like the copyright changed; I have this scripted, so it is a quick change. We'd probably want to add a NOTICES.txt file if making the copyright more specific.
Removes @author tags, as per discussion with @benedictpaten @hannes-ucsc on today's scrum.

+1

Build fails with
pip2.7 install -e .
Obtaining file:///mnt/ephemeral/workspace/toil-scripts
    Complete output from command python setup.py egg_info:
    Traceback (most recent call last):
      File "<string>", line 1, in <module>
      File "/mnt/ephemeral/workspace/toil-scripts/setup.py", line 23, in <module>
        from version import version
      File "version.py", line 2
    SyntaxError: Non-ASCII character '\xe2' in file version.py on line 2, but no encoding declared; see http://python.org/dev/peps/pep-0263/ for details

Build fails with

Hah! I got cocky and used an en dash instead of a hyphen in 2015–2016. Will fix.

Rebased and addressed buld issue.


The link to some URLs is not shown

See screnshot:

This is actually me misinterpreting the UI. No URL is supposed to be present there.


btfhub/example: fix compilation by adding submodules

@simar7 would you mind checking/rechecking the compilation, specially for your use case, and see if builds correctly in your environment ? I have removed the "dynamic" compilations from the default Makefile target because the building environment might not have the latest libbpf features in its default shared libs, so it could be broken for users with older distros.

Seems OK but how would I know if its using the gitmodule that's distributed here or the one I have installed locally? I did a fresh clone and tried to build BTFs using an example object file.

That test is enough I suppose.


Preserve data

Preserve and retrieve data

create a json file that receive each data inputted by the user
use ruby json library to generate and parse data
clone each data into it respective file e.g ( book data are cloned into book.json file)
After generating data into json they are passed to the file through ruby file writer
information are retrieved through file reader then parse to object.

@Olamarx I really appreciate the review, helping to grow but I humbly ask if can explain better, I'm trying to understand the review and the changes required but I can get a hold of it

@Olamarx I really appreciate the review, helping to grow but I humbly ask if can explain better, I'm trying to understand the review and the changes required but I can get a hold of it

I am sorry that it is not clear, the project requirement requires that upon starting the app, all the data in json files, if they are present, should be displayed but that is not the case in your app at the moment. Do you now understand?

Yes I do, but I kinda wrote a code in the main.rb file that calls a method on Main class in the app.rb file. Which loads the data from the JSON.file

The method directly fetch data from the JSON file

but when I ran the app, the saved data did not load at the start, you can check and test the app.

Sure

@Olamarx thank you for ttaking your time explain the correction, it was golden

@Munsa1 thank you for the approval


[SMALLFIX] Update the s3 integration doc.

Removes references to s3n.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/16668/
Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/16673/
Test PASSed.

Merged build finished. Test PASSed.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/16675/
Test PASSed.


TileStache configuration file

Have tried to install tile server locally but could not find working tilestache.cfg that serves Mapzen tiles. Is it public?

There is a different tileserver that you'll want to use. There are instructions in the vector-datasource wiki.

@rmarianski Oh, thanks a lot!


DataBase.java:30-33: Need add basic implementation for...

The puzzle 10-8e4704c8 from #10 has to be resolved:
https://github.com/smallcreep/cucumber-seeds/blob/4f1ac3e7bc26268bb5d9189f03d595153b008e28/seeds-db/src/main/java/com/github/smallcreep/cucumber/seeds/db/steps/DataBase.java#L30-L33
The puzzle was created by Ilia Rogozhin on 10-Oct-18.
If you have any technical questions, don't ask me, submit new tickets instead. The task will be "done" when the problem is fixed and the text of the puzzle is removed from the source code. Here is more about PDD and about me.

@0pdd the only puzzle #41 is solved here.


Adds define_cog

I have the define command ready. "!define " or "!define multiple_words" will take the data from Merriam-Webster API and format it using pagination. Also, I'm bad at error handling, I'm still a baby programmer.
One thing to be done on this end:
1.) Add an API key (BotSecrets)

Fixed the errors

Added the final touches to code readability


Add shopping based on recipes

If your PR is related to a contribution to the taxonomy, please, fill
out the following questionnaire. If not, replace this whole text and the
following questionnaire with whatever information is applicable to your PR.
Describe the contribution to the taxonomy

Makes a shopping list for you based on what the recipe calls for and what you already have.
Result => Now it's slightly more talkative but correct.
Input given at the prompt

I'm making pizza. I need flour, water, salt, yeast, tomato puree, mozzarella and basil. I have flour, water, to
mato puree and basil. What do I need to buy?

Response that was received

Based on the ingredients you currently have available and the toppings you mentioned, you will need to purchase
the following:

1. Yeast: This is the leavening agent that will help the dough rise, giving your pizza a light and airy
texture.
2. Salt: A small amount of salt can enhance the flavor of the dough, so it is recommended to use it in your
pizza recipe.
3. Mozzarella cheese: This is the classic cheese choice for pizzas, adding a creamy and mild flavor to the
dish.
4. Fresh basil: A popular garnish for pizzas, basil can add a fresh and aromatic touch to your homemade pizza.

By adding these missing ingredients to your shopping list, you will be able to make your desired pizza with all
the necessary components.

Response that is now received instead

To make pizza, you will need to buy the following ingredients:
1. Fresh or active dry yeast
2. Salt
3. Mozzarella cheese

By having these additional items, you can prepare the dough and assemble your pizza with the provided
ingredients (flour, water, tomato puree, and basil). The yeast is essential for the rising process of the
dough, while the salt enhances the flavor of the dough. Mozzarella cheese is a classic choice for pizza
toppings, but feel free to experiment with other cheeses or ingredients to suit your taste preferences.

To ensure the best results, consider the following tips:
1. For the dough, use bread flour or a high-protein all-purpose flour for better texture and elasticity.
2. Activate the yeast in warm water (around 110°F or 45°C) with a pinch of sugar to help it grow.
3. Knead the dough for 10-15 minutes to develop the gluten, which will result in a chewy crust.
4. Proof the dough in a warm place for at least 1-2 hours or up to 24 hours for a more flavorful crust.
5. Stretch or roll out the dough to your desired thickness and shape.
6. Pre-bake the crust for a few minutes before adding the toppings to prevent a soggy crust.
7. Bake the pizza at high temperatures (around 475°F or 245°C) for a crispy crust.
8. Feel free to experiment with various toppings, sauces, and cheese varieties to create your unique pizza
style.                                                                                │
9. Garnish with fresh basil leaves and enjoy your homemade pizza!

Contribution checklist

[x] Contribution was tested with lab generate
[x] No errors or warnings were produced by lab generate
[x] All commits are signed off (DCO)
[x] The qna.yaml file was linted and prettified (yaml-validator can do both)

Thank you for your contribution to InstructLab! Unfortunately, once a Github repo is made public, all open PRs are automatically closed since they link against a private repo. We have detected that your PR might have been one of the ones affected by this change. If you are still interested in contributing your improvement, please fill out the following short form by no later than May 3rd, and we will get back to you with the additional steps necessary once we have had time to assess the PRs of those still interested:
https://forms.gle/V7SrPPMZDo6iGDYu8

I completed all the required extra steps a month ago! Do i really need to go through this additional process? This whole situation is not my fault.

@lhellebr this is how to Refork (fix) your PR for taxonomy and resubmit: https://youtu.be/x6NfWvrkjnI it seems we ran into a problem with the private to public conversion. You can skip the form in this situation.

@jjasghar Thanks, I re-added my PR


feat: add feedback when creating mentorship request

What Github issue does this PR relate to?

Close #78
Changes proposed in this pull request:

set loading spinner when the mentorship request is being submitted
load toast notification if the submission of the mentorship request is successful

What should the reviewer know?

https://user-images.githubusercontent.com/45630868/207322813-34a0511d-3752-4f5a-8203-12569aee3500.mov
Quality Assurance(QA) Checklist

[x] The app builds and runs properly

Wow @maranta1 this was fast! Nice one, it seems quite complete? When you feel like this is ready, set it as Ready for review (button above).
Also, could you please assign the ticket #78 to yourself, and its status to in progress?
Nice preview clip for the reviewer, makes it everything easier, thanks!


Dependencies warning when upgrade to v.1.10.0

pnpm up --latest
 WARN  Issues with peer dependencies found
.
└─┬ @platformatic/db 1.10.0
  └─┬ @platformatic/service 1.10.0
    └─┬ @platformatic/telemetry 1.10.0
      └─┬ @opentelemetry/exporter-trace-otlp-proto 0.44.0
        ├─┬ @opentelemetry/core 1.17.1
        │ └── ✕ unmet peer<EMAIL_ADDRESS><1.7.0": found 1.7.0 in @platformatic/telemetry
        └─┬ @opentelemetry/otlp-transformer 0.44.0
          ├── ✕ unmet peer<EMAIL_ADDRESS><1.7.0": found 1.7.0 in @platformatic/telemetry
          ├─┬ @opentelemetry/resources 1.17.1
          │ └── ✕ unmet peer<EMAIL_ADDRESS><1.7.0": found 1.7.0 in @platformatic/telemetry
          ├─┬ @opentelemetry/sdk-logs 0.44.0
          │ └── ✕ unmet peer<EMAIL_ADDRESS><1.7.0": found 1.7.0 in @platformatic/telemetry
          ├─┬ @opentelemetry/sdk-metrics 1.17.1
          │ └── ✕ unmet peer<EMAIL_ADDRESS><1.7.0": found 1.7.0 in @platformatic/telemetry
          └─┬ @opentelemetry/sdk-trace-base 1.17.1
            └── ✕ unmet peer<EMAIL_ADDRESS><1.7.0": found 1.7.0 in @platformatic/telemetry

Can't reproduce it.
I ran, on latest main
$ npm run clean
$ pnpm install
$ pnpm up --latest

But I didn't receive such warning

Uhmm, I run on Windows 11, can be this ?

here the @opentelemetry/exporter-trace-otlp-proto dep has been updated to 0.45.0, try with latest commit

Solved. Thank's


[php] Provide a formatter that also handles HTML

1 doc = many code types.
why we build too much extentions for some code type???
it's doesnt work for every code in 1 doucument.
we need 1 click and apply for all code formatting.
please reply,
thanks.

@kitwill I am sorry, but I do not understand what your issue is. Please clarify, ideally with steps and a small sample

@jrieken we need extension for set different code type auto formatting in 1 click sir. Because nowaday we use PHP formatter for set formatting for PHP and we user HTML Formatter for set formatting for HTML. What if we have HTML and PHP in 1 file what would i use? The ploblem is when i use PHP formatter for reformatting PHP code. It's make HTML code fail format to 1 line. But when i use HTML formatter it's make PHP fail format. Did you understand?
please reply,
thanks

The PHP formatter should also be able to format HTML or any other embedded language.
VSCode doesn't contain a PHP formatter, this is left to extensions to be provided.
If you use a extension for formatting, and see a bug there, please file any issues against the PHP formatter that you use.

it will be much more better for vscode to support php formatter instead of using extensions which will cause multiple keybinding for multiple language,

+1


Update Docker image base to debian:11.5-slim

Pull requests:

https://github.com/Senzing/docker-senzing-base/pull/127


layers: add level/layer to ValidateImageAspectLayout log msg

This helps clarify the case where a transition fails due to a specific subresource being in an unexpected layout.

arrgh, self-test failures; will fix momentarily.


Nested query variable type is not inferred

Going to investigate and see what's going on, but I think there may be a problem with how the variables are being inferred for nested queries.
An example query (some of the query)

And no TypeScript validation errors... The inference thinks "variables" is an empty object.

Edit: After some digging, it appears that GetVariables is not recursive, so it's only going to pull the top-level arguments. I could be reading it wrong as I'm still getting familiar with the code base.

I have confirmed that all top-level variables are inferred just fine.

I'm not sure if... "nestedness" is the problem - for example there is a test in tests/examples/test-zeus.good.ts which contains nested variables:
let tq = query(q => [
  q.cardById({ cardId: $('cid') }, c => [
    ...cardFragment,
    c.attack({ cardID: $('cids') }, aCards => [
      aCards.Attack, //
      aCards.Defense,
    ]),
  ]),
...

On my machine, the query you listed does work 👍
However, I was able to reproduce a case where the variables were not inferred.
Can you try this query and see if you don't get the variables inferred?

const tq = query(q => [
  q.drawCard(c => [
    c.attack({cardID: $('cardID')}, s => [
        s.name
    ])
  ])
])```

This fixed it for me: https://github.com/typed-graphql-builder/typed-graphql-builder/pull/19

Awesome, thank you! merged.

Fix merged and published. Huge thanks again!


Xcode 12.5 deprecation

Codecov Report

Merging #21 (c4c971c) into main (a5d8ba3) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main      #21   +/-   ##
=======================================
  Coverage   97.69%   97.69%
=======================================
  Files          12       12
  Lines         130      130
=======================================
  Hits          127      127
  Misses          3        3

Impacted Files
Coverage Δ

.../CollapsibleSectionsFetchedResultsController.swift
100.00% <ø> (ø)

.../Sources/Controller/FetchedResultsController.swift
100.00% <ø> (ø)

Continue to review full report at Codecov.

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


Add support of color image @interpretation

If we want to be fully compatible with nexus, we'll have to handle more @interpretation for color images:
See https://github.com/nexusformat/definitions/pull/834:
			* ``rgb-image`` (3-D data)
			* ``rgba-image`` (3-D data)
			* ``hsl-image`` (3-D data)
			* ``hsla-image`` (3-D data)
			* ``cmyk-image`` (3-D data)

Question: Do we want to support that?

Well, rgb and rgba are already supported.
For the others I see little interest.

Strictly speaking only @interpretation="rgba-image" is supported even if it already supports both RGB and RGBA images.
We do not understand: @interpretation="rgb-image" and as I commented in nexusformat/definitions#834, to me it is not needed, the shape of the data already provides this information.


upgrade: Envoy v1.28

Codecov Report

Merging #4 (ffc0fec) into main (dab36f7) will decrease coverage by 6.06%.
Report is 2 commits behind head on main.
The diff coverage is 0.00%.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
@@            Coverage Diff             @@
##             main       #4      +/-   ##
==========================================
- Coverage   47.95%   41.90%   -6.06%
==========================================
  Files           9       11       +2
  Lines         367      420      +53
==========================================
  Hits          176      176
- Misses        181      234      +53
  Partials       10       10

Files
Coverage Δ

manager.go
0.00% <0.00%> (ø)

buffer.go
0.00% <0.00%> (ø)

context.go
9.00% <0.00%> (+0.08%)
:arrow_up:

context_opts.go
0.00% <0.00%> (ø)

:mega: Codecov offers a browser extension for seamless coverage viewing on GitHub. Try it in Chrome or Firefox today!


Release Google.Cloud.AppEngine.V1 version 1.3.0

Changes in this release:
No API surface changes; just dependency updates.

Created release for Google.Cloud.AppEngine.V1-1.3.0


Add conditional requests and cache-control headers on '/users' (#1062)

app.User.ID is set after account.User.ID instead of one of
its account.Identity.ID
splitted Identity vs User in design/account.go, which consequently
renamed some app.Identity*Data to app.User*Data. Also added
created-at and updated-at fields in the user data.
moved some app code out of the account package to avoid cyclic deps.
using plural for most cache-control config variables, which allows for
distinction between /users and /user endpoint, the latter requiring
some specific settings to prevent caching on intermediate proxies.

Fixes #1062
Signed-off-by: Xavier Coulon<EMAIL_ADDRESS>

Please also make sure the user profile endpoints work fine with this change. @sbose78 please review this.

This PR has many good changes. And I would like to merge it but we need to make sure we don't break anything (it currently does).

@alexeykazakov

But I think we have to include the Keycloak identity ID to the User payload somehow. Identity ID is needed for some operations like Add/Remove a user identity to/from the list of space collaborators. And currently (in this PR) the only way for UI to get this identity ID from a user payload is to extract if from the self link.

I understand, but I think that this is a separate concern that could be handled in another issue. WDYT ?

Yes, it can be a separate PR. But we will need to adopt the space collaboration in UI very soon. Probably this week. This PR hides Identity ID from User payload and we will need to "fix" it.

ok, so in that case it's probably better if I handle this in this PR as well, since it replaces the identityID field with a userID field in the JSON response.

@aslakknutsen it's done in https://github.com/almighty/almighty-core/pull/1076/commits/92cadcee97086f712b9f218ead40ed11d53dfc49. Please let me know if this solution satisfies the needs.

[test]

[test]

[test]

LGTM @alexeykazakov @xcoulon
Before we merge I wanted to test this PR against the ui changes https://github.com/fabric8io/fabric8-ui/pull/422/files but my local platform ui is just a white screen.


Add Project to frontend

Add project API calls in apiSlice

Add a projects page where you can edit and create projects.

Add project to routes.tsx

When constructing room, add room to project's room list


How to create a new instance of a structure that has already been defined in Python?

I have already declared a structure using the StructureDataType class in a script written in Python. I want to use this structure at a specific address in my program and fill its components with the bytes there. How do you create a new instance of an already defined structure in Python and have it be stored at a certain address?

createData(address, structure)

Using this throws a data conflict at the address. Is there a way to convert the data already at that address into a structure? In other words, mark the data at that address as this specific struct datatype and the bytes at that address fall into the different components of the structure?

Using this throws a data conflict at the address. Is there a way to convert the data already at that address into a structure? In other words, mark the data at that address as this specific struct datatype and the bytes at that address fall into the different components of the structure?

You can just use clearListing(Address start, Address end) to clear any potential data that is there before trying to apply the structure.


Ensure directories are created when they don’t exist + close connection

Fixes #145

Closes connections before returning
Ensure directories that don't exist on FTP server are created

Current coverage is 60.14% (diff: 0.00%)

Merging #161 into master will decrease coverage by 1.06%

@@             master       #161   diff @@
==========================================
  Files            21         21
  Lines           964        981    +17
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
  Hits            590        590
- Misses          288        305    +17
  Partials         86         86

Powered by Codecov. Last update b4b8741...3a10c22

hey @Ruxton thanks for this one!


Toast icon is slightly uncentered

[x] I have searched the issues of this repository and believe that this is not a duplicate.

Expected Behavior
Adding some margin to the close icon

Close Icon should be centered.
For example, adding a {marginTop: '2px'} to the svg fix it.
Current Behavior

Context

Your Environment

Tech
Version

Base UI
v4.4.1

React

browser

node.js

etc.

Steps to Reproduce

Link:
https://baseui.netlify.com/?selectedKind=Toast&selectedStory=Toast notifications&full=0&addons=1&stories=1&panelRight=0&addonPanel=REACT_STORYBOOK%2Freadme%2Fpanel
Close icon is moved to the top slightly.

Hi @shanshanzhu ! Thanks for opening this issue!
The toast component follows our design spec, and the close icon is not meant to be centered:

The top of close icon is not aligned with the top of the text like the previous version. We notice this change when upgrading from v2.3.0 to v4.4.x.
It looks odd when the text is only one line

Latest 4.4.x version

2.3.0 version


Unable to pin messages in team's channel

About You

Voyage number (e.g. 8): 8
Team Name: chimeras-team-02
Discord Name: Ed
Timezone: UTC+1

Issue Description & Expected Outcome:
I am unable to pin messages to my chimeras-team-02 channel. I would expect to be able to pin others' and my messages for the benefit of the team.
Symptoms:

As you see, I don't have the option.
Steps to Recreate:

Send the message
Click the 3 dots on the upper right corner of the message
No 'pin' option is available.

Resolution:
I believe this is an admin restriction. I would appreciate any assistance to allow the pinning in my team's chat by me and ideally the other team members. Thanks in advance! 👍 😀

Pinning messages requires the "Manage Messages" permission which also allows members of the channel to delete one another's messages. Unfortunately, there is no finer grained permission that only grants the ability to just pin messages. Due to this, we decided not grant this permission.
Will leave ticket open while reviewing this decision.

Granted this permission for this team. Closing ticket, but reopen if there are any issues.

No issues here but just wanted to let you know, in case it is relevant, that some of my team members had the ability to pin messages before I opened this issue. Perhaps this information is useful if you are trying to keep these permissions under control. Thanks once again.

@eduardoltorres thanks man we sorted that out now


DSpace CSV export allows CSV injection to Excel with unfiltered content

Describe the bug
If a user submits a document with a field value which could be interpreted as a function in Excel, the CSV export DSpaceCSVLine.java does not escape or sanitise this value.  The result is that a client opening this in Excel treating it as a trusted source (and with macros enabled) will execute this function.
To Reproduce
Steps to reproduce the behavior:
(1) Submit an item with a text value of =sum(1+1)
(2) Export the collection in using the DSpace metadata export.
(3) Open in Excel and note that the field resolves as "2"  - i.e. the formula is executed.
Expected behavior
The cell value is prefixed by an apostrophe when it contains a potential excel function
The following is a proof of concept for sanitisation
    s = s.replaceAll("^[\\r\\n]+", "");
    if (s.length() > 1 && "+@=-".indexOf(s.charAt(0)) != -1) {
        s = s.substring(1);
    }

    // Replace internal quotes with two sets of quotes
    return "\"" + s.replaceAll("\"", "\"\"") + "\"";

Related work
Link to any related tickets or PRs here.

I think this sounds like Excel's problem, not ours?

Yes it is.  However where DSpace sits in an trusted ecosystem with users of Excel the potential to inject such data is being treated as a problem with the providing system, even though the weakness is in Excel.   CVE's are being raised on just this point against the several library applications.
Sanitising the data in CSV looks to be a relatively easy fix to DSpaceCSVLine.java in 6-8 versions.

I think the challenge with this is that not everyone uses Excel when editing a CSV.  While I'm not against sanitization (where possible), I think we need to ensure it doesn't impact the ability to edit those same CSVs in other tools.  We also need to ensure that the sanitization does not impact the newly imported CSV (i.e. would the metadata field in DSpace be updated to be sanitized or not?)
So, there may be side effects to this sanitization that we need to test for.
Regardless, this would need a volunteer to create a PR. Then we'd need to do some thorough testing to ensure the sanitization process doesn't break editing in other (non-Excel) tools, and doesn't break the re-import of that metadata.

I created a PR on this for a snippet of code - we are using it already in some of our sites.  Feedback most appreciated.

DSpace's workflows revolve around CSVs. The only thing we should be worried about is whether our CSVs are compliant with RFC 4180 (or whatever), as well as various DSpace-isms like id, collection, multiple value separators, and authorities. It is really not our problem if Excel does something unsafe with the data in the file.
Personally, I use many CSV-based workflows: OpenRefine, csv from the Python stdlib, Pandas, etc. This could potentially break my workflows just to make sure that some Excel users don't get pwned.
Maybe a better solution is to have an "Excel export" mode that does this sanitation optionally.

Unfortunately if a CVE is raised on the issue externally, there will be pressure to correct it in DSpace.   Institutions now actively monitor CVE's.    Other library systems are in exactly this situation.  As part of the enterprise ecosystem DSpace can't just say "it's their problem".
The problem is of course that it may well be a dspace admin that is exporting and then opening the spreadsheet.  If a cunning person has submitted an item properly crafted this could be just the scenario the injection problem is anticipating.  Export, edit and import is a very common bulk metadata editing scenario.
I would be surprised if the solution (prepending an apostrophe only on certain leading characters) broke any downstream systems.  It is still a csv-encoded value.    You could undoubtedly argue that changing the metadata should never be done, but then again if you are told to take down the system, the metadata will not be much good to you.
Interestingly the change could be reversed on the inbound route where the data is a change, because old and new values are there to be compared.   The prepended apostrophe could therefore be stripped.  Still working on that aspect.
We have some very large datasets, and  @ and = are not generally remarked.   Phone numbers with a + prefix are represented in the data.
We are already patching our instances to do the filtering (v6,7,8), and will monitor issues from there.
Edmund

If Excel is evaluating the literal text value =1+1 as 2 that's Excel's problem, not ours.

Unfortunately if a CVE is raised on the issue externally, there will be pressure to correct it in DSpace. Institutions now actively monitor CVE's. Other library systems are in exactly this situation. As part of the enterprise ecosystem DSpace can't just say "it's their problem".

A CVE will not be raised on DSpace due to a parsing issue in Excel. Furthermore, if you work in an organization that thinks this way you should push back on them. It's nonsense security theater. I regularly have to push back against such policies in my organization.
Excel does a lot of annoying stuff, for example converting all my YYYY dates to floats and mangling my ISBNs, not to mention endless issues with users opening UTF-8 encoded files in windows-1252 and corrupting anything outside of the ASCII range.

The following CVE rated 8.8 was raised against the Koha open source system:
https://www.cvedetails.com/cve/CVE-2024-24337/
In many ways DSpace is worse that Koha here because an non-admin user can submit content which is then processed by an admin user.  At least requires an admin level login for the access.
The sad reality is an institution is not likely to say "get rid of MIcrosoft" but they might well say "get rid of DSpace" even if they spend an exorbitant amount for a replacement.
Indeed we have a client who is taking Koha off public access purely on the issue of Content Security Policy - something DSpace does not implement either.
Agencies, particularly government, are becoming very anxious  about attacks - the British Library being an example none want to emulate.
Edmund
On Friday, October 18, 2024 18:52 AEDT, Alan Orth @.***> wrote:

If Excel is evaluating the literal text value =1+1 as 2 that's Excel's problem, not ours.

Unfortunately if a CVE is raised on the issue externally, there will be pressure to correct it in DSpace. Institutions now actively monitor CVE's. Other library systems are in exactly this situation. As part of the enterprise ecosystem DSpace can't just say "it's their problem".

A CVE will not be raised on DSpace due to a parsing issue in Excel. Furthermore, if you work in an organization that thinks this way you should push back on them. It's nonsense security theater. I regularly have to push back against such policies in my organization.
Excel does a lot of annoying stuff, for example converting all my YYYY dates to floats and mangling my ISBNs, not to mention endless issues with users opening UTF-8 encoded files with windows-1252 encoding and corrupting anything outside of the ASCII range.
--
Reply to this email directly or view it on GitHub:
https://github.com/DSpace/DSpace/issues/9899#issuecomment-2421714820
You are receiving this because you authored the thread.
Message ID: @.***>

I disagree that the linked CVE is a vulnerability in Koha. Surprising that it got assigned a CVE in my opinion. Shame, because it creates a precedent for corporate ICT teams to point to. It is not DSpace's job to sanitize arbitrary text values just so Excel users don't receive prompts for code to run or have their formulas evaluated automatically.
When there are format parsing vulnerabilities in Android or whatever, of which there are many, we don't say that JPEGs are banned—we fix the parser. I could give other examples.

Indeed we have a client who is taking Koha off public access purely on the issue of Content Security Policy - something DSpace does not implement either.

This is a fair point and was brought up recently (see: https://github.com/DSpace/DSpace/issues/9881).

If CVEs are filed against the wrong product, they should be corrected.
If we must paper over someone else's bad code, at least DSpace should offer the option to say "I know what I'm doing, so give me the bitstream without corrupting it."  But even that is setting a bad precedent.  DSpace should return the object that was archived, not some hacked variant.

Overall, I just want to note here that I support performing some sanitization of CSVs, provided it doesn't change the data on re-import.  I'm worried specifically that sanitization techniques could accidently modify real data in a unexpected way on re-import.
Like others here, I'm still not confident this rises to the level of a security issue (as we have no requirement that anyone should use Excel).  But, I understand the argument that DSpace should do it's best to ensure the safety/sanitization of any exported content.
So, I agree that this is a bug that should be fixed, regardless of whether it is deemed a security issue or not.

I have just created a CSV containing "injected" content from the user (myself) in LibreOffice Calc: libreoffice-cve.csv. The file contains:
"id","comment"
1,"DDE (""cmd"";""/C cmd"";""!A0"")A0"

So by this logic LibreOffice is "vulnerable" too?
This is neither a bug nor a security vulnerability in DSpace. It's Excel's problem. In fact, according to the CVE linked, Excel even prompts the user asking whether they want to run the content. The UX around communicating the risk to the user is Excel's problem, not ours.


Added MvcBinding annotation

This PR adds the @MvcBinding annotation as recently discussed on the mailing list.

Merged!


name space cds::urcu::gc  not exist

hi, I want to use the skip_list_set_rcu, but there is no gc files behind the cds/urcu directory. Where can I find it?

from http://libcds.sourceforge.net/doc/cds-api/namespacecds_1_1urcu.html#details:
RCU implementation type
There are several internal implementation of RCU (all declared in cds::urcu namespace):
general_instant - general purpose RCU with immediate reclamation
general_buffered - general purpose RCU with deferred (buffered) reclamation
general_threaded - general purpose RCU with special reclamation thread
signal_buffered - signal-handling RCU with deferred (buffered) reclamation
signal_threaded - signal-handling RCU with special reclamation thread

You cannot create an object of any of those classes directly. Instead, you should use wrapper classes. The wrapper simplifies creation and usage of RCU singleton objects and has the reacher interface that combines interfaces of wrapped class i.e. RCU global part like synchronize, and corresponding RCU thread-specific interface like access_lock, access_unlock and retire_ptr.
There are several wrapper classes (all declared in cds::urcu namespace)
gc<general_instant> - general purpose RCU with immediate reclamation, include file <cds/urcu/general_instant.h>
gc<general_buffered> - general purpose RCU with deferred (buffered) reclamation, include file <cds/urcu/general_buffered.h>
gc<general_threaded> - general purpose RCU with special reclamation thread include file <cds/urcu/general_threaded.h>
gc<signal_buffered> - signal-handling RCU with deferred (buffered) reclamation include file <cds/urcu/signal_buffered.h>
gc<signal_threaded> - signal-handling RCU with special reclamation thread include file <cds/urcu/signal_threaded.h>

Any RCU-related container in libcds expects that its RCU template parameter is one of those wrapper.


KOGITO-6869: Update C3

Updating C3 Charts library.
To test please build and run Dashbuilder Authoring, use all charts in a page. Export the dashboard in a ZIP file, build and run dashbuilder runtime and then import the ZIP. All the charts should work.
Thanks!

@sthundat
Hello Savitha!
Would you please review this issue?
Thanks!


Use version ranges for better downstream compatibility

The gem was updated to use explicit versions as an attempt to protect against breaking changes from non-major version updates and to make version usage more explicit.  I was also hoping that specifying explicit versions would make working with dependabot a little more straightforward as I wasn't seeing version update PRs occurring 🤷
This change reverts part of https://github.com/github/licensed/pull/590 by changing version requirements back from an explicit requirement to a requirement range.  In order to mitigate some of the concerns with finding breaking changes from dependencies, I'm also un-ignored Gemfile.lock.  That seems to be the recommended strategy for gems these days anyway.

The licensed-ci GitHub Action has created a pull request containing license metadata updates based on the changes in this branch.
Please review the pull request for any additional changes required and merge when ready.

The gem was updated to use explicit versions as an attempt to protect against breaking changes from non-major version updates and to make version usage more explicit. I was also hoping that specifying explicit versions would make working with dependabot a little more straightforward as I wasn't seeing version update PRs occurring

I haven't double-checked the source to confirm, but I suspect the following prevented Dependabot from proposing version updates:

Since you didn't have a Gemfile.lock committed, Dependabot thought this was a library.
For libraries, the default versioning-strategy is increase-if-necessary.
Since the version was pinned explicitly, Dependabot probably has a check that says "if a library explicitly pins, leave it alone to prevent creating noise for the maintainer".

So you probably could have worked around this by specifying a non-default versioning-strategy, like increase.
I suspect you're not interested in pursing this further, but if you are, then I'm happy to help with any pointers either here or via the internal GitHub channels.

I think you're good for now. 👍


Extra character while writing LTR and RTL languages in a text field

Steps to Reproduce

create an empty application
add one text field to the application
(optional) you can make its obscure equals to true to see the problem better by seeing the count of dots
type something in English
change the language of your keyboard (we tested by Persian) and type another thing
use space
u must see another character is being written beside the space (we check the ASCII code and believe it is half space)

from several observation and tests, I believe this problem is happening when you are writing an LTR language inside an LTR field then you use an RTL language inside the same field (or vice-versa with the RTL field), and when it occurred if u even delete all the character and write again with only one language it still happening.
summary:
u have a LTR filed -> write LTR language (it's still ok) -> write RTL -> use space
or
u have an RTL field -> write RTL language (it's still ok) -> write LTR -> use space

note: because the half-space character is not visual and doesn't persist any spaces between characters you may want to use shift + right arrow and select the characters to feel it.

note: this has been happening since flutter version 1.17 and because of that we have to use 1.12.13+hotfix6

it is a bit confusing when it comes to writing on a paper so u can see this video for a better understanding

anybody?

I was able to reproduce on latest master channel.
flutter doctor -v
[✓] Flutter (Channel master, 1.24.0-4.0.pre.57, on Mac OS X 10.15.7 19H2 darwin-x64, locale en)
    • Flutter version 1.24.0-4.0.pre.57 at /Users/pedromassango/dev/SDKs/flutter_master
    • Framework revision fdac8a9d1f (8 hours ago), 2020-10-25 19:47:02 -0400
    • Engine revision d4a3c9c471
    • Dart version 2.11.0 (build 2.11.0-256.0.dev)

[✓] Android toolchain - develop for Android devices (Android SDK version 30.0.1)
    • Android SDK at /Users/pedromassango/Library/Android/sdk
    • Platform android-30, build-tools 30.0.1
    • Java binary at: /Applications/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b3-6222593)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 12.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 12.1, Build version 12A7403
    • CocoaPods version 1.9.3

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 4.0)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b3-6222593)

[✓] IntelliJ IDEA Community Edition (version 2020.2.3)
    • IntelliJ at /Applications/IntelliJ IDEA CE.app
    • Flutter plugin version 50.0
    • Dart plugin version 202.7319.5

[✓] VS Code (version 1.50.1)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.15.1

[✓] Connected device (4 available)
    • sdk gphone x86 arm (mobile) • emulator-5554 • android-x86    • Android 11 (API 30) (emulator)
    • macOS (desktop)             • macos         • darwin-x64     • Mac OS X 10.15.7 19H2 darwin-x64
    • Web Server (web)            • web-server    • web-javascript • Flutter Tools
    • Chrome (web)                • chrome        • web-javascript • Google Chrome 86.0.4240.111

• No issues found!

so is there any patch for this problem yet?

Probably a duplicate of https://github.com/flutter/flutter/issues/63042.


TypeError: init() got an unexpected keyword argument 'instance_id'

Hey guys, I am getting a strange error within my code and I have no idea how to fix it. Here is the error from the command prompt:
 File "kivy\_clock.pyx", line 384, in kivy._clock.CyClockBase._process_events
   File "kivy\_clock.pyx", line 414, in kivy._clock.CyClockBase._process_events
   File "kivy\_clock.pyx", line 412, in kivy._clock.CyClockBase._process_events
   File "kivy\_clock.pyx", line 167, in kivy._clock.ClockEvent.tick
   File "C:\Users\Owner\Desktop\Ubotica\IOD\cloud_detect\Sentinel-interface.py", line 651, in <lambda>
     Clock.schedule_once(lambda dt:self.get_image(img_num, i), 0)
   File "C:\Users\Owner\Desktop\Ubotica\IOD\cloud_detect\Sentinel-interface.py", line 617, in get_image
     self.get_meta(patch, f'rgb{img_num}_{i}')
   File "C:\Users\Owner\Desktop\Ubotica\IOD\cloud_detect\Sentinel-interface.py", line 890, in get_meta
     results = list(tqdm(executor.map(patch_calculations, [eop]), total=1))
   File "C:\Users\Owner\Desktop\Ubotica\cloud_detect\venv\lib\site-packages\tqdm\notebook.py", line 222, in __iter__
     for obj in super(tqdm_notebook, self).__iter__(*args, **kwargs):
   File "C:\Users\Owner\Desktop\Ubotica\cloud_detect\venv\lib\site-packages\tqdm\std.py", line 1081, in __iter__
     for obj in iterable:
   File "C:\ProgramData\Anaconda3\lib\concurrent\futures\_base.py", line 586, in result_iterator
     yield fs.pop().result()
   File "C:\ProgramData\Anaconda3\lib\concurrent\futures\_base.py", line 425, in result
     return self.__get_result()
   File "C:\ProgramData\Anaconda3\lib\concurrent\futures\_base.py", line 384, in __get_result
     raise self._exception
   File "C:\ProgramData\Anaconda3\lib\concurrent\futures\thread.py", line 56, in run
     result = self.fn(*self.args, **self.kwargs)
   File "C:\Users\Owner\Desktop\Ubotica\IOD\cloud_detect\Sentinel-interface.py", line 862, in patch_calculations
     maxcc=1.0)
   File "C:\Users\Owner\Desktop\Ubotica\cloud_detect\venv\lib\site-packages\sentinelhub\ogc.py", line 369, in __init__
     super().__init__(**kwargs)
 TypeError: __init__() got an unexpected keyword argument 'instance_id'

And this is my code:
class WebFeatureService(OgcService):
    """ Class for interaction with Sentinel Hub WFS service

    The class is an iterator over info data of all available satellite tiles for requested parameters. It collects data
    from Sentinel Hub service only during the first iteration. During next iterations it returns already obtained data.
    The data is in the order returned by Sentinel Hub WFS service.
    """
    def __init__(self, bbox, time_interval, *, data_source=DataSource.SENTINEL2_L1C, maxcc=1.0, **kwargs):
        """
        :param bbox: Bounding box of the requested image. Coordinates must be in the specified coordinate reference
            system.
        :type bbox: geometry.BBox
        :param time_interval: interval with start and end date of the form YYYY-MM-DDThh:mm:ss or YYYY-MM-DD
        :type time_interval: (str, str)
        :param data_source: Source of requested satellite data. Default is Sentinel-2 L1C data.
        :type data_source: constants.DataSource
        :param maxcc: Maximum accepted cloud coverage of an image. Float between 0.0 and 1.0. Default is 1.0.
        :type maxcc: float
        :param config: A custom instance of config class to override parameters from the saved configuration.
        :type config: SHConfig or None
        """
369  super().__init__(**kwargs)

        self.bbox = bbox
        self.time_interval = parse_time_interval(time_interval)
        self.data_source = data_source
        self.maxcc = maxcc

        self.tile_list = []
        self.index = 0
        self.feature_offset = 0
        self.latest_time_only = time_interval == SHConstants.LATEST
        self.max_features_per_request = 1 if self.latest_time_only else SHConfig().max_wfs_records_per_query

    def __iter__(self):
        """ Iteration method

        :return: iterator of dictionaries containing info about product tiles
        :rtype: Iterator[dict]
        """
        self.index = 0
        return self

Does anyone know what this error could possibly mean and how to fix it? Thank you !

Hi @Tristanbox09,
In sentinelhub-py version 3.0 parameter instance_id has been replaced with parameter config. More info in docs.

Hi @Tristanbox09,
Perhaps you have instance_id=... somewhere where you are creating an instance of WebFeatureService?
If you are using latest version of sentinelhub-py, please use SHConfig instead. See example in docs.

Ah perfect ! Thank you guys !


Feature: Add Study_Assess() parameters to Make_Snapshot().

Feature Details

Create a testing branch that is downstream of v1.8.4.
Add bLogOutput and strLogFileName to Make_Snapshot().
Add argument to sink() in Log(): e.g.,  sink(file="file", split=TRUE)

@Craftech-RR @samussiah - any updates on testing this branch with logging?

Closing as wontfix - believe this was used to test out logging.


ci: use electronjs/node orb

Use electronjs/node orb to pick up caching of Node.js installs, and significantly simplify this CircleCI config.

Codecov Report

Merging #1093 (452f308) into main (b045a72) will decrease coverage by 0.73%.
Report is 1 commits behind head on main.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main    #1093      +/-   ##
==========================================
- Coverage   76.01%   75.29%   -0.73%
==========================================
  Files          20       20
  Lines         688      688
  Branches      129      129
==========================================
- Hits          523      518       -5
- Misses        118      121       +3
- Partials       47       49       +2

see 2 files with indirect coverage changes
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


[Bug]: Unexpected Error (crash) as savings balance went to 0

Describe the bug
A few minutes after initiating a Manual Setup transfer and boosting the transaction (CPFP), I was sitting on the home page waiting for the transaction to confirm and for my new spending balance to be available when my savings balance went from positive value 384k to 0 (see screenshot 1). I then tapped the savings balance and the 'Unexpected Error' page displayed (see screenshot 2) with the following report:
Error: Text strings must be rendered within a <Text> component.
Component Stack:
    in RCTView
    in Unknown
    in RCTView
    in Unknown
    in StyledNativeComponent
    in Unknown
    in ActivitySavings
    in StaticContainer
    in EnsureSingleNavigator
    in SceneView
    in RCTView
    in Unknown
    in DebugContainer
    in MaybeNestedStack
    in RCTView
    in Unknown
    in RNSScreen
    in Unknown
    in Suspender
    in Suspense
    in Freeze
    in DelayedFreeze
    in InnerScreen
    in Screen
    in SceneView
    in Suspender
    in Suspense
    in Freeze
    in DelayedFreeze
    in RNSScreenStack
    in ScreenStack
    in NativeStackViewInner
    in RCTView
    in Unknown
    in SafeAreaProviderCompat
    in NativeStackView
    in PreventRemoveProvider
    in NavigationContent
    in Unknown
    in NativeStackNavigator
    in WalletsStack
    in StaticContainer
    in EnsureSingleNavigator
    in SceneView
    in RCTView
    in Unknown
    in DebugContainer
    in MaybeNestedStack
    in RCTView
    in Unknown
    in RNSScreen
    in Unknown
    in Suspender
    in Suspense
    in Freeze
    in DelayedFreeze
    in InnerScreen
    in Screen
    in SceneView
    in Suspender
    in Suspense
    in Freeze
    in DelayedFreeze
    in RNSScreenStack
    in ScreenStack
    in NativeStackViewInner
    in RCTView
    in Unknown
    in SafeAreaProviderCompat
    in NativeStackView
    in PreventRemoveProvider
    in NavigationContent
    in Unknown
    in NativeStackNavigator
    in EnsureSingleNavigator
    in BaseNavigationContainer
    in ThemeProvider
    in NavigationContainerInner
    in StyledNativeComponent
    in Unknown
    in RootNavigator
    in RCTView
    in Unknown
    in InactivityTracker
    in AppOnboarded
    in SlashtagsProvider
    in RNCSafeAreaProvider
    in SafeAreaProvider
    in StyledNativeComponent
    in Unknown
    in ThemeProvider
    in App
    in PersistGate
    in Provider
    in ErrorBoundary
    in Root
    in RNGestureHandlerRootView
    in GestureHandlerRootView
    in gestureHandlerRootHOC(Root)
    in RCTView
    in Unknown
    in AppContainer
Stack: Error: Text strings must be rendered within a <Text> component.
    at completeWork (address at index.android.bundle:1:454493)
    at completeUnitOfWork (address at index.android.bundle:1:471571)
    at performUnitOfWork (address at index.android.bundle:1:470872)
    at workLoopSync (address at index.android.bundle:1:469967)
    at renderRootSync (address at index.android.bundle:1:469799)
    at flushSyncWorkAcrossRoots_impl (address at index.android.bundle:1:423743)
    at batchedUpdatesImpl (address at index.android.bundle:1:482345)
    at batchedUpdates$1 (address at index.android.bundle:1:416457)
    at _receiveRootNodeIDEvent (address at index.android.bundle:1:416745)
    at receiveTouches (address at index.android.bundle:1:479907)
    at apply (native)
    at __callFunction (address at index.android.bundle:1:144401)
    at anonymous (address at index.android.bundle:1:142840)
    at __guard (address at index.android.bundle:1:143782)
    at callFunctionReturnFlushedQueue (address at index.android.bundle:1:142798)
Platform: android 14
Version: 1.0.5 (136)
LDK version: ldk-v0.0.123-36-g1027b935fb92b924 c_bindings-v<IP_ADDRESS>)
LDK node ID: 03c537950c6e1b62c0ce7923f4933d89999394edcac0f4aca877cd5cfcaec996c9

Reproduce
I'm unsure whether the manual setup and boost are relevant here but this is what I did

Scan lightning node https://1ml.com/node/0296b2db342fcf87ea94d981757fdf4d3e545bd5cef4919f58b5d38dfdd73bf5c9 and initiate a new channel with just spending balance, no inbound liquidity.
Once Transfer activity is visible, tap it and boost (CPFP)
Once activity is boosted, leave the wallet open on home page to wait for confirmation
After 5-10 minutes the savings balance went from 384k to 0
Tapping the savings balance showed Unexpected Error (crash) page

Screenshots / Recording
Screenshot 1

Screenshot 2

Operating system
Android 14
Bitkit version
b3f9f30dd56e4a1666492c3219f352d2bcd77086
Log output
bitkit_ldk_logs_1728449292551.zip

No idea why the balance would be wrong but the uncaught error is fixed with https://github.com/synonymdev/bitkit/pull/2307

@pwltr  We've seen the 0 balance a few times recently.  @SeverinAlexB reported it happening after initial funding of new wallet via CJIT.
Sev and I also witnessed something that could be related: the double-counting of savings balance after a transfer to spending.  It feels to me like sometimes an address/output is either not being included in the balance or is being double-counted. Could this could be the result of simultaneous refreshes occurring, potential race condition?
A manual refresh always corrects the balances.

This issue is for the uncaught error crash, not zero balance.

JS uncaught error was fixed here

Tested on v1.0.6 (138)
Not seen this in our testing session today. Everyone will regularly perform the action of opening Savings activity so no further testing needed. Closing.


Indentation of SA1127 code fix incorrect when doc comments are present

Sample code
namespace NamespaceName
{
    class ClassName
    {
        ///
        void Method<T>() where T : IDisposable
        {
        }
    }
}

Expected result
namespace NamespaceName
{
    class ClassName
    {
        ///
        void Method<T>()
            where T : IDisposable
        {
        }
    }
}

Actual result
namespace NamespaceName
{
    class ClassName
    {
        ///
        void Method<T>()
                    where T : IDisposable
                {
        }
    }
}

This also occurs if the definition is preceded by an indented #region or #endregion directive.

@nbarbettini Note the additional case related to preprocessor directives mentioned in the second comment. :+1:

@sharwell Thanks. Can't believe I missed this issue # before :smile:


Why so agro?

https://github.com/MarkDucommun/bowling-haskell/blob/400fc75cdadd59abba3d484115f4e1bc49b4b2d0/app/Main.hs#L8
Why is your program yelling at me? 😄

Firm, yet supportive feedback for the user.


Update CMakeLists to match qt pro file

CMakeLists is currently lagging behind after some updates were made to the qt .pro file. This should be updated and tested.

CMake is no longer support and CMakeLists file has been removed


Looking forward to the code

Thank you for your interest of our work and we will release the code in a week or two.

Thank you for your interest of our work and we will release the code in a week or two.


Integration test for create-toolpad-app

The create-toolpad-app entry point to Toolpad must have an integration test to ensure it doesn't break when pushing new releases

Passing conditions:

Check for generated files
Check for 200 on the started app URL


[Sample - Gamepad] USB Xbox controller - m_reading->GetGamepadState always returns empty state

Latest GDK and latest Windows 10 update
GameInput and Gaming Services service running
Latest XBOX-GDK-Samples - Gamepad
Xbox One USB/Logitech Xbox 360 controller - Controller shows connected but GamepadState is always empty

Sony Dual Sense Controller also doesn't work - Controller doesn't get detected - Connected through USB. Verified that it works fine outside of GameInput (Using hid API directly, ...)
Is there a list of controllers supported by GameInput? Considering that it is the recommended API for Gamepad input handling on Windows, so far not even one controller (connected through Bluetooth or USB) seems to work with any of the samples


Update th.lang

yee

I'm about to add a new field to stats command called Fastest Escape (which tells you how long your shortest escape time is). Along with that, I'm also going to add a minutes and seconds translation to go along with that. Here are the new fields:
stats_field_fastestescape=Fastest Escape (after stats_field_escapes)
minutes=Minutes (after hours)
seconds=Seconds (after seconds)
After those are added, I'll merge this pr.


add custom parquet and interleave datasets

loss and memory usage on the 10b before and after

TODO

[x] fix ckpt of the dataloader (again)
[ ] find a way to convert old ckpt to new ckpt

Added conversion script to convert old dataloader checkpoints to new one. There are some issues that may or may not be important.

The conversion might result in too much skipping if the HF dataset is not doing the data_files splitting logic we are assuming
Its a bit of a pain to use atm because you need to cast it on each rank. Since we dont plan to maintain it for that long, maybe it's ok. Just code a helper bash script around it