Reward curve uses floats

Describe the bug
The reward curve uses floats, this will cause inconsistencies across machines. Didn't see it tracked anywhere so I thought i'd make an issue for it.
https://github.com/ava-labs/gecko/blob/master/vms/platformvm/reward.go#L13
Additional context
Just wanted to make a note on one possible solution to the problem. Take a look at how substrate accomplishes this: https://github.com/paritytech/substrate/blob/master/frame/staking/src/inflation.rs#L32.
Hope it helps!
By submitting this issue I agree to the Terms and Conditions of the Developer Accelerator Program.

Thanks for putting this into an issue. This is fixed by #381 where the real minting function is defined.


add message

Already present, cant merge

I add the msg!
On Thu, Oct 27, 2022, 11:33 Arunim Malviya @.***> wrote:

Closed #177 https://github.com/Arunim313/Hackto.py/pull/177.
—
Reply to this email directly, view it on GitHub
https://github.com/Arunim313/Hackto.py/pull/177#event-7678960067, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AYTG5JIQ374SEB2HB6KQXTLWFILKPANCNFSM6AAAAAARPVWWZM
.
You are receiving this because you authored the thread.Message ID:
@.***>


COBOL textmate - Support for comment in SQL statements

Description of the enhancement requested
Hi,
We would like the texmate COBOL grammar manage comments included in SQL statements.
COBOL code in IBM Z Open Editor:

SQL code in IBM Db2 for z/OS Developer:

Thanks.

See:

#402

This rule could work without the problem described in issue #402:
    {
      "begin": "(?i:exec\\s+sql)",
      "name": "string.quoted.cobol.sql",
      "patterns": [
        {
          "match": "(\\:([0-9a-zA-Z\\-_])*)",
          "name": "variable.cobol"
        },
        {
          "match": "(--.*)",
          "name": "comment.line.modern"
        },
        {
          "include": "source.sql"
        }
      ],
      "end": "(?i:end\\-exec)",
      "endCaptures": "string.quoted.cobol.sql"
    },

PS: rule for SQL comment is not mandatory if problem describe in #402 was solved, « source.sql » apply rule for SQL comment.
But COBOL floating comment « *> » should be managed in SQL comment.

that's not how "endCaptures" work
it should be "endCaptures": { "0": { "name": "string.quoted.cobol.sql" } }


[Build] Generate ObjC compatibility header for library Swift targets

This will generate the ObjC header for library Swift targets so they can be
used in other ObjC targets. This is only done for macOS.
rdar://problem/51780725

@swift-ci smoke test

@swift-ci smoke test

@swift-ci smoke test

@aciidb0mb3r: Is this released in Xcode 11.2? Because I'm still seeing the issue. I have a sample if it helps?

No, this change is not present in Xcode 11.2

Is this going to be released with Xcode 11.3 or we have to wait for Swift 5.2?

How about Xcode 11.3?

Maybe with 11.4?

@aciidb0mb3r was there a particular reason for supporting only macOS? Would be great to have ObjC support for iOS too.

This is only done for macOS.

@aciidb0mb3r Any reason not to do this for iOS, as well?

I wish this could be supported in iOS as well，currently I facing cases like delivering Swift packages to ObjC based App target.  Without able to directly import the autogenerated swift header in ObjC, it will be very painful to add a Swift adaptor layer in the app.

When can we have this for iOS? Thanks

This is working in iOS project on Xcode 11.4.1 for me.
I've tested it on an old Objective-C project and successfully can use Swift Package manager with it. Here is what I did:

Swift Packages - add package dependency, as normal to include my package
In an Objective-C .m file, include the module like so:
@import YourSwiftPackageName;
Ensure in your swift package that you have used @objc to expose Swift to your Objective-C
Ensure you have used public to expose your Swift in your package to your main project, as it's in another module.
Ensure your Swift Package is listed in your targets 'Frameworks, Libraries and Embedded Content'. I note this because I have 2 targets - and it only added them to one of my targets, not both, so I kept getting Module not found errors.

I'm able to add a SwiftPackage into MainTarget Objective-C.m file
But no chance when trying to add it to my Objective-C.h file.
Any idea to solve this ?

Why is this only done for macOS?


Retry also on initial connect

Initializing the bunyan logger with this stream without a running logstash instance results in an unhandled error event
events.js:137
      throw er; // Unhandled 'error' event
      ^

Error: connect ECONNREFUSED <IP_ADDRESS>:9999
    at Object._errnoException (util.js:1003:13)
    at _exceptionWithHostPort (util.js:1024:20)
    at TCPConnectWrap.afterConnect [as oncomplete] (net.js:1195:14)

Maybe the retry function can also be applied on the error event? I wasn't able to handle the error event on my own.

This already works, i just missed the
.on('error', console.log)

line...sorry!


refactor: move default actions to TILE_DEFAULTS

also introduce the option to disable actions by setting action: false or secondaryAction: false
also show "clickable" tick not for fixed tile types, but for tiles with action defined

Thanks.


Migrate to travis-ci.com to run PRs

(template did not apply)
Currently, no PR will be able to run the CI since Travis migrated to travis-ci.com and stopped support for Travis-ci.org completely on June 15. This is how you can migrate to travis-ci.com: https://docs.travis-ci.com/user/migrate/open-source-repository-migration#migrating-a-repository

Closing. @andymccurdy moved over to GitHub actions.


Cannot read 6ch FLAC file

sf.read(r'D:\temp\6ch_silence.flac')
Out[6]: (array([], shape=(0, 6), dtype=float64), 16000)

sf.read(r'D:\temp\2ch_silence.flac')
Out[7]:
(array([[0., 0.],
        [0., 0.],
        [0., 0.],
        ...,
        [0., 0.],
        [0., 0.],
        [0., 0.]]),
 16000)

multi_ch_flac.zip
When I try to read 6ch flac file, soundfile returns empty array.
2ch is fine.

That's probably an error in libsndfile. I'd advise you to reopen the issue there.

So, I investigated this a bit:
It seems that there is a limit on how many samples can be read from flac files at the moment. With your 2ch-example, the entire file is smaller than the limit, but with the 6ch-example is bigger. Reading in chunks of up to 2:45 minutes (or thereabouts) seems to work. More than that does not work.
This is most probably related to https://github.com/libsndfile/libsndfile/pull/858, and will therefore hopefully be fixed in the next release of libsndfile.
If you don't think this is related, please open an issue at libsndfile. Silent files can be weird edge cases sometimes.

I confirmed it is https://github.com/libsndfile/libsndfile/pull/858.

Duplicate of #236

(By the way, if you urgently need a fix, put an updated libsndfile in your load path, and soundfile will use it instead of its own)


Multiple links in API reference TOC Tree appear on same line

The HTML docs generated for ansys-openapi-common show this in the API reference TOC tree (0.9.6 and 0.9.5):

Compared to the current output in the published docs (package version 0.7.1)

This then produces a strange layout when clicking one of the links:

We do not have multi-version docs set up yet for openapi-common, so see the artifacts generated by the CI in this PR https://github.com/pyansys/openapi-common/pull/338 for the actual generated HTML. It looks like the change occurred somewhere between 0.7.1 and 0.9.5, I just only noticed when checking the 0.9.5 to 0.9.6 PR.

@Andy-Grigg it is because of the sidebar_nav_bar.html present in the templates, which overrides the existing sidebar settings by pydata and ansys-sphinx-theme. i removed the html files. and hopefully it will solve the issue.

Thanks @Revathyvenugopal162 , your commit has indeed solved this issue. See https://github.com/pyansys/openapi-common/pull/338/commits/eb857e49857bcc738657a687c612b30b0da8af34 for the commit.
Closing this issue.


Content update - Dependency properties overview (user story 1878471)

Hi @adegeo, this PR is ready for your review (article and code projects).
Notes:
Some of the links point to .NET Framework 4.8 and others to .NET 5.0 - all are hardcoded per your previous instructions.
However, if we don't hardcode the .NET version when linking to framework articles, the server redirects to the latest version. For example:
/dotnet/desktop/wpf/advanced/xaml-in-wpf?view=netframeworkdesktop-4.8&preserve-view=true  (will only point to the .NET 4.8 page)
/dotnet/desktop/wpf/advanced/xaml-in-wpf  (will auto-redirect to the .NET 5.0 page)
Internal preview link

If the page is available in .NET 5, you should link to the .NET 5 version.

I'll check all links in the 3 print articles and submit a separate PR for any issues found. I'm glad we're hashing this out early!
Conditional link update should be automatable, like image/snippet syntax update in docs-toolset.

Another thing I was thinking of, if you run into XAML that happens to be different between VB and C# (generally just the root nodes are different, such as Window or Class because VB does it differently) I call out that the XAML may look different in Visual Basic and all XAML is from C#. There isn't any of that here, but I just want you to be aware of that 😄

@adegeo thanks for the clarification on links, which are now fixed in this article.
I'll keep an eye out for any XAML changes between C# & VB for future articles.

@adegeo, I've relocated the article & snippets to wpf/properties folder.
Should we rename wpf/documents to wpf/printing?

One of the other projects I worked on, had articles marked with this kind of comment below the metadata section:

Would you be interested in that score getting added for future reference?

@adegeo, I've relocated the article & snippets to wpf/properties folder.
Should we rename wpf/documents to wpf/printing?

Nah, the printing stuff is highly related to the overall documents story which will contain other things. Originally I was thinking this article would go in the systems folder since it's related to the WPF systems (properties, events, resources) but I think there will end up being a lot in that folder and I want to avoid the situation where 100 articles live in the same generic folder like the advanced folder in the .NET Framework content.

@adegeo, a different project I worked on had articles marked with this kind of comment below the metadata section:

Would you be interested in that score getting added for future reference?

Sure that sounds good!

@v-trisshores Here is the PR for you to accept to get the TOC/Redirects into your branch for this PR... woah that is a lot of PRs... https://github.com/v-trisshores/docs-desktop/pull/1

Awesome thanks again!

Glad to help!


Only lower case slugs

Slugs should only contain of the following characters:

a-z
0-9
_
.

Inputs of users should be lowercased and than have to be validated.

Should we also disallow starting with a number?

Good thinking about the not starting with a number. https://github.com/vutuv/vutuv/issues/78


Android-Java: 3.0 overview of Android-Resources file (res)

Description :
Give an overview of the Android-Resource file (res):-
References :

https://developer.android.com/guide/topics/resources/providing-resources
https://medium.com/@doyouseeitmyway/creating-and-using-resource-files-in-android-studio-8b4208c08862

Changes should be made in Android_Development/Java/ directory under Android_Development_With_Java branch.
You have to push your PR to the Android_Development_With_Java branch only.
You can take help to the references but don't simply try to copy it. Just explain in your own word.
This issue is for GWOC'21 contributors only.
You need to push in a single commit. If there exists more than one commit then you need to squash your commits into one commit.

sir i would like to work on this issue plz assign it to me.

Hello Supervisor/ Mentor, Can I start working on this issue I just created? 😅
canyou assign this work to me?

@Manmeet2607 you can start working on this issue.

@LegendSumeet Actually we assign issues on a first come first serve basis. No problem I will soon create some issues you can definitely start working with them. Otherwise, you also can create an issue by yourself too.

@LegendSumeet Actually we assign issues on a first come first serve basis. No problem I will soon create some issues you can definitely start working with them. Otherwise, you also can create an issue by yourself too.

ok sir

@Manmeet2607 you can start working on this issue.

OK sir

does it need to be a simple doc file or HTML ?

@Manmeet2607 you can make a markdown file. I think it will be better


tinyllama quantize can't inference correctly

Description of the bug:

ai-edge-torch version: 2.0
command

CC=/usr/bin/clang-18 bazel run -c opt //ai_edge_torch/generative/examples/c++:text_generator_main -- --tflite_model=/mnt/data/nigel_wang/ai-edge-torch/ai_edge_torch/generative/examples/tiny_llama/ttiny_llama_seq512_kv1024.tflite --sentencepiece_model=/mnt/data/nigel_wang/ai-edge-torch/ai_edge_torch/generative/examples/tiny_llama/TinyLlama-1.1B-Chat-v1.0/tokenizer.model --prompt="<|user|> \n Write and email:\n <|assistant|>" --start_token="<s>" --stop_token="</s>" --num_threads=1

output

INFO: Running command line: bazel-bin/ai_edge_torch/generative/examples/c++/text_generator_main '--tflite_model=/mnt/data/nigel_wang/ai-edge-torch/ai_edge_torch/generative/examples/tiny_llama/ttiny_llama_seq512_kv1024.tflite' '--sentencepiece_model=/mnt/data/nigel_wang/ai-edge-torch/ai_edge_torch/generative/examples/tiny_llama/TinyLlama-1.1B-Chat-v1.0/tokenizer.model' '--prompt=<|user|> \n Write and email:\n <|assistant|>' '--start_token=<s>' '--stop_token=</s>' '--num_threads=1'
normalizer.cc(52) LOG(INFO) precompiled_charsmap is empty. use identity normalization.
INFO: Created TensorFlow Lite XNNPACK delegate for CPU.
Prompt:
<|user|> \n Write and email:\n <|assistant|>
Output text:

reference

I found some points, if i set quantize equal false, it can inference correctly.
quantize bool = True : can decode successfully.
quantize bool = false : fail decode. e.g., above log, all is ??
def convert_tiny_llama_to_tflite(
checkpoint_path: str,
prefill_seq_len: int = 512,
kv_cache_max_len: int = 1024,
quantize: bool = True,
):

Actual vs expected behavior:
No response
Any other information you'd like to share?
No response

Hi @nigelzzz, I'm getting this issue when I run your command:
ERROR: Didn't find op for builtin opcode 'STABLEHLO_COMPOSITE' version '1'. An older version of this builtin might be supported. Are you using an old TFLite binary with a newer model?

ERROR: Registration failed.

Error at ai_edge_torch/generative/examples/c++/text_generator_main.cc:93

Can you either share/provide your .tflite file or provide reproduce steps for your model conversion? i.e. the branch/version you are on when you do the conversion, whether you are using cuda when converting as well. Generally the more information I know, the faster we can help you. Is there any special reason your model name is ttiny_llama_seq512_kv1024? or is that just a typo? Thanks.

Hi @pkgoogle,
ttiny_ llama_seq512_kv1024 is typo, i just classify quantize or no quantize.
by the way i think the root cause is WORKSPACE tensorflow version isn't correctly

i using v0.2.0 branch
build command

 /user/: CC=/usr/bin/clang-18 bazel run -c opt //ai_edge_torch/generative/examples/c++:text_generator_main -- --tflite_model=/mnt/data/nigel_wang/ai-edge-torch/ai_edge_torch/generative/examples/tiny_llama/ttiny_llama_seq512_kv1024.tflite --sentencepiece_model=/mnt/data/nigel_wang/ai-edge-torch/ai_edge_torch/generative/examples/tiny_llama/TinyLlama-1.1B-Chat-v1.0/tokenizer.model --prompt="<|user|> \n Write and email:\n <|assistant|>" --start_token="<s>" --stop_token="</s>" --num_threads=1

output

Extracting Bazel installation...
Starting local Bazel server and connecting to it...
DEBUG: /mnt/data/nigel_wang/tensorflow_cache/153a550227f3ff2fa4e4811633058a05/external/org_tensorflow/third_party/repo.bzl:132:14:
Warning: skipping import of repository 'com_google_absl' because it already exists.
DEBUG: /mnt/data/nigel_wang/tensorflow_cache/153a550227f3ff2fa4e4811633058a05/external/org_tensorflow/third_party/repo.bzl:132:14:
Warning: skipping import of repository 'XNNPACK' because it already exists.
INFO: Analyzed target //ai_edge_torch/generative/examples/c++:text_generator_main (147 packages loaded, 3826 targets configured).
INFO: From Compiling src/google/protobuf/generated_message_tctable_lite.cc [for tool]:
external/protobuf~/src/google/protobuf/generated_message_tctable_lite.cc:347:14: warning: unused function 'Offset' [-Wunused-function]
  347 | inline void* Offset(void* base, uint32_t offset) {
      |              ^~~~~~
1 warning generated.
INFO: From Compiling src/google/protobuf/compiler/cpp/helpers.cc [for tool]:
external/protobuf~/src/google/protobuf/compiler/cpp/helpers.cc:197:25: warning: unused function 'VerifyInt32TypeToVerifyCustom' [-Wunused-function]
  197 | inline VerifySimpleType VerifyInt32TypeToVerifyCustom(VerifyInt32Type t) {
      |                         ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~
1 warning generated.
INFO: From Executing genrule @@org_tensorflow//tensorflow/lite/acceleration/configuration:configuration_schema:
When you use --proto, that you should check for conformity yourself, using the existing --conform
INFO: Found 1 target...
Target //ai_edge_torch/generative/examples/c++:text_generator_main up-to-date:
  bazel-bin/ai_edge_torch/generative/examples/c++/text_generator_main
INFO: Elapsed time: 276.290s, Critical Path: 109.56s
INFO: 1493 processes: 601 internal, 892 linux-sandbox.
INFO: Build completed successfully, 1493 total actions
INFO: Running command line: bazel-bin/ai_edge_torch/generative/examples/c++/text_generator_main '--tflite_model=/mnt/data/nigel_wang/ai-edge-torch/ai_edge_torch/generative/examples/tiny_llama/ttiny_llama_seq512_kv1024.tflite' '--sentencepiece_model=/mnt/data/nigel_wang/ai-edge-torch/ai_edge_torch/generative/examples/tiny_llama/TinyLlama-1.1B-Chat-v1.0/tokenizer.model' '--prompt=<|user|> \n Write and email:\n <|assistant|>' '--start_token=<s>' '--stop_token=</s>' '--num_threads=1'
ERROR: Didn't find op for builtin opcode 'STABLEHLO_COMPOSITE' version '1'. An older version of this builtin might be supported. Are you using an old TFLite binary with a newer model?

ERROR: Registration failed.

Error at ai_edge_torch/generative/examples/c++/text_generator_main.cc:93

above this error, i see in newer version has add stablehlo_composite,
https://github.com/tensorflow/tensorflow/commit/f4f2393888af78879dc9b299786023fe87fbbcfc
in WORKSPACE version, doesn't add

_TENSORFLOW_GIT_COMMIT = "26d4ea90364daa14bbb2bc5c2aa68f5b70c4641f"
https://github.com/tensorflow/tensorflow/blob/26d4ea90364daa14bbb2bc5c2aa68f5b70c4641f/tensorflow/lite/core/kernels/register.cc#L385

and i have other question in this ticket.
https://github.com/google-ai-edge/ai-edge-torch/issues/109

or i can help open a pr to fix it

Hi @nigelzzz, thanks for the info -- I'm having trouble building against that particular commit/version of TF -- did you modify your BUILD file? https://github.com/google-ai-edge/ai-edge-torch/blob/release/0.2.0/ai_edge_torch/generative/examples/c%2B%2B/BUILD
You can open a PR if you feel it is actually fixing the root cause. And this might just be a new issue as HEAD/nightly should work as well :).

Hi @pkgoogle ,
I just do some patch in my local tensorflow library.
I guess we just update tensorflow version, change version in WORKSPACE file. e.g., (https://github.com/google-ai-edge/ai-edge-torch/blob/main/WORKSPACE)

Hi @nigelzzz, for better reproducibility can you produce a diff between your local files and the github repo (maybe pull the latest changes).
something like this:
# navigate to tf root
git diff origin/master > diff.txt

Then share/upload that diff.txt file... that will help me a lot, thanks.


Fix for ProvidedTypesContext.fs when compiling with dotnet.exe

With old VS everything works fine, but when compiling with dotnet.exe, for whatever reason, FSharp couldn't resolve type: String.StartsWith(value: char) vs String.StartsWith(value: string)
C:\git\FSharp.TypeProviders.StarterPack\src\ProvidedTypesContext.fs(95,20): error FS0041: A unique overload for method 'StartsWith' could not be determined based on type information prior to this program point. A type annotation may be needed. Candidates: String.StartsWith(value: char) : bool, String.StartsWith(value: string) : bool [C:\git\myProj.fsproj]
C:\git\FSharp.TypeProviders.StarterPack\src\ProvidedTypesContext.fs(95,70): error FS0041: A unique overload for method 'StartsWith' could not be determined based on type information prior to this program point. A type annotation may be needed. Candidates: String.StartsWith(value: char) : bool, String.StartsWith(value: string) : bool [C:\git\myProj.fsproj]

This PR will fix the problem just by giving a type hint to the compiler.

Seems to happen only with target netcoreapp2.0
Another question is if that should ever be a target for a type provider.


Reconcile ipset when osnetconfig changes

When e.g. a network definition changes, the IPsets have to reconcile to update its information. This allows changing network definition, but needs to be done carefully and tests in a pre-prod env as it is an invasive change.
The process looks like this:

update the OpenStackNetConfig to do the network change
validate the IPSet for the role and the OpenStackNetConfig status reflects the new IPs for the nodes

<IP_ADDRESS>/24

Make sure you have a parameter file to allow update the network config for the role:

  network-environment.yaml: |
    parameter_defaults:
      ControllerNetworkConfigUpdate: True
      ComputeNetworkConfigUpdate: True
      ComputeLeaf1NetworkConfigUpdate: True

use the OpenStackConfigGenerator resource to create a new configversion
when config generator process finished, depending on the number of changed, the new created openstackconfigversion will show the diff (gets trunkated if it is too big)

  diff: |
    diff --git a/source-templates/environments/deployed-network-environment.yaml b/source-templates/environments/deployed-network-environment.yaml
    index 168a9242caa93630d17ade3d254604b967a3a3f0..d08508054b2fc8e01d8f6499b08fba25fbe84860 100644
    --- a/source-templates/environments/deployed-network-environment.yaml
    +++ b/source-templates/environments/deployed-network-environment.yaml
    @@ -99,2 +98,2 @@           storage_subnet:
    -            cidr: '<IP_ADDRESS>/24'
    -            gateway_ip: '<IP_ADDRESS>'
    +            cidr: '<IP_ADDRESS>/24'
    +            gateway_ip: '<IP_ADDRESS>'
    @@ -208 +207 @@       storage:
    -      - <IP_ADDRESS>/24
    +      - <IP_ADDRESS>/24
...

use the OpenStackDeploy resource to apply the generated configversion to the overcloud

Notes:

from checking the overcloud node, the /etc/os-net-config/config.yaml and /etc/sysconfig/network-scripts/ifcfg-enp5s0 are correct.
When checking the running interface configuration we still see the old IP address <IP_ADDRESS>. Reboot the node/if-down|if-up would fix it.
This is not an OSPdO issue.

6: enp5s0: <BROADCAST,MULTICAST,UP,LOWER_UP> mtu 9000 qdisc mq state UP group default qlen 1000
    link/ether 02:c8:20:00:00:04 brd ff:ff:ff:ff:ff:ff
    inet <IP_ADDRESS>/32 brd <IP_ADDRESS> scope global enp5s0
       valid_lft forever preferred_lft forever
    inet <IP_ADDRESS>/24 scope global enp5s0
       valid_lft forever preferred_lft forever
    inet <IP_ADDRESS>/32 scope global enp5s0
       valid_lft forever preferred_lft forever
    inet6 fe80::c8:20ff:fe00:4/64 scope link
       valid_lft forever preferred_lft forever

General on changing networks. when e.g. removing a network from the openstackclient will still have a reservation on that network

Also restart openstackclient pod if network attachment list changes currently if a new network gets added to the openstackclient network list, the ipset and pod annotation gets updated, but the interface is not configured. Therefore this change makes sure that the openstackclient pod gets restarted when the network list changes
resolves: https://bugzilla.redhat.com/show_bug.cgi?id=2325902

/cherry-pick v1.3.x


Dashboard-next- Could not create product with Product attributes

What I'm trying to achieve
Create a new product based on a Product-Type, which has Product Attribute.
…
Steps to reproduce the problem

Create a new product-type(Say.,  X) which has product attribute.
Try Create a new product for the product-type (X)
3.While using the drop-down and trying to select the product-type (X), application opens up a blank page and fails to load

What I expected to happen
While trying to select the product-type (X), application should load the product-type (X) along with its attributes.
Screenshots

Debug Logs:
TypeError: Cannot read property 'name' of undefined
ProductOrganization.tsx:48
at getAttributeName (\saleor\static\dashboard-next\products\components\ProductOrganization\ProductOrganization.tsx:48:87)
System information
Operating system: Windows 10
Browser:Chrome

Same as #3420 - I couldn't reproduce this problem.

With the latest master, issue is not there


Rework deployment so dist folder is not necessary in git

Right now I'm storing the 'dist' folder in git because that's what is used to serve the processed server/client code.  Modify the deployment process to create the dist folder, then remove it from git.

http://stackoverflow.com/questions/11175923/heroku-github-deployment-strategies


pip install requirements is failing

On Ubuntu 23.10 I cannot run your pip install.
Error log is very long and have attached it.
Do I need to install mkl in a separate step?
time-llm-pip-error-log.txt

I was using python 3.11
I switched to 3.9 but then got errors installing torch 2.0.1.  I manually installed torch 2.0.2 at the terminal and then ran the pip requirements again.  It now installed torch 2.0.1 and then uninstalled torch 2.0.2
Everything is good now.

Closing this.

was this error also with deepspeed or just torch?

As far as I can tell it was the fact that at the time this project required Python 3.9.  The issue with torch resolved itself after switching to 3.9  I did not have an issue with deepspeed.


Not possible to create service principal database user

The instructions for setting up the sql user no longer works as expected. The prerequisite is:

Once a service principal is created in Azure AD, create the user in SQL Database. You'll need to connect to your SQL Database with a valid login with permissions to create users in the database.

I am signed in using the super admin account for the SQL Server and receive the following error.
CREATE USER [AppSP] FROM EXTERNAL PROVIDER
GO

---

Started executing query at Line 1
Msg 33159, Level 16, State 1, Line 1
Principal 'AppSP' could not be created. Only connections established with Active Directory accounts can create other Active Directory users.
Total execution time: 00:00:00.027

It seems this approach is no longer supported.
I would also suggest to clarify the usage and relations of AppSP, myapp, the app registrations, enterprise applications and the managed identity. It is a bit confusing. We have system managed identity which results in an enterprise application and service principal entry. It does not generate an app registration however.

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

ID: 2b0443f7-69c6-1e5d-cb14-7876595b8b54
Version Independent ID: a0fd8e29-b356-6e36-3a8a-8ec5fb68bb26
Content: Create Azure AD users using service principals - Azure SQL Database
Content Source: articles/azure-sql/database/authentication-aad-service-principal-tutorial.md
Service: sql-database
Sub-service: security
GitHub Login: @GithubMirek
Microsoft Alias: mireks

I'm assigning this to the author for the feedback review

Regarding error 33159.
For Azure SQL DB is it required that only AAD users can create other AAD users. It means that any SQL user with SQL authentication including a server admin cannot create an AAD user. The initial AAD user who can create AAD users in SQl DB is the AAD admin. After this any AAD user with proper permissions can create other AAD users.

Thanks, @span! I've added some clarification to the Create the service principal user in Azure SQL Database section that hopefully clears up your issue.
#assign:VanMSFT

Thnak you @VanMSFT
#please-close


Emit runner ending events to plugins on SIGTERM, SIGHUP, and SIGINT

Seems plugins need to be informed about gemini is about to quit. Example use case - if gemini is being killed and tunnel plugin working there will be ghost ssh process.

/cc @SevInf


can't generate normal image?

i can't generate normal image use instantid

but if i lower the weight of the instantid, i get a normal image:

no warning no error:

https://github.com/cubiq/ComfyUI_InstantID/issues/36


re: solicitation for maintainers

Responding per this tweet, I'd be happy to contribute to this project. I'm a long-time user of scientific/geospatial Python stack, and have plenty of experience contributing to open source Python projects. I wouldn't call myself a Dask power user, but I'm familiar enough with the library to have bumped up against the limitations of the Dask + Geopandas workflow, which is why I'm eager to contribute to this effort. I'm not sure that Cython-oriented tasks are where I would be most effective from the start, as I don't have much development experience with Cython, but hopefully this can be a good excuse for me to gain some.

Thanks for stepping up!  Spatialpandas is at the stage of a fully usable proof of concept, sufficient to illustrate how to do fast Numba-powered native-Python spatial data processing, but not yet fully fleshed out with a robust user community. We (the HoloViz team) developed it to address limitations with the stack for rendering spatial data in Datashader, but it deserves to have a much wider userbase and range of applications, far beyond just viz.
Spatialpandas gets its speed from Numba rather than Cython, which in some ways lets you focus on Python syntax and concepts more than Cython does. In the best case, with Numba all you have to do is put @jit in front of your slow functions, though in practice getting good performance out of it typically takes understanding what Numba is doing at a deeper level, same as for Cython.
Probably the best way to get started is to look at the open issues here on this repo and see if any of what people have raised as issues are within your expertise.  If they are, jump right in!  If nothing jumps out at you or you want a bit more handholding, let me know and you can join our weekly HoloViz issue-triaging meetings where we try to stay on top of what users are reporting.

Sounds good. Looks to me like most of the open issues should be covered by #37 apart from the .ipynb issue which sounds like a problem with either memory or the kubernetes config. I'll watch the repo for any new issues, but do feel free to tag me if something pops up that I miss. Don't think I can commit to a weekly meeting, but I'd be happy to sit in for one or two to help identify where work is needed.

hey all, I've went ahead and contributed a recipe to the conda-forge for spatial pandas, anyone in this issue want to help maintain the feedstock? see #40

@mxndrwgrdnr @jbednar interested in becoming maintainers for the conda forge feedstock as well? https://github.com/conda-forge/spatialpandas-feedstock

I'm trying to avoid spreading me and my team too thin; we're focusing on viz and hoping this library will have a life outside of our viz tools. So I'm trying to stay back except for making sure it has a logo so that I can easily provide proper attribution for it in our docs and blogposts. That approach has worked out perfectly for xarray-spatial, another viz-related but not viz-specific project that started as part of Datashader and has now gone on to an exciting new life beyond HoloViz. So I can be listed as a maintainer for the feedstock, but I'm really hoping someone more directly involved will step up for it!


New rule: forbid unsafe enum assignment

enum ZeroOrOne {
  Zero = 0,
  One = 1,
}

// Compiles
const zeroOrOne: ZeroOrOne = 2;

const foo = (a: ZeroOrOne): void => {
  return;
};

// Compiles
foo(2);

You can ban enum declaration with the no-restricted-syntax rule, like so:
    // Ban enums.
    // TODO make this a real rule instead of using `no-restricted-syntax`: https://github.com/danielnixon/eslint-config-typed-fp/issues/284
    "no-restricted-syntax": [
      "error",
      {
        selector: "TSEnumDeclaration",
        message: "Don't declare enums",
      },
    ],

Indeed, we do exactly that over in https://github.com/danielnixon/eslint-config-typed-fp/blob/master/src/index.ts#L47
But that won't help you if the enum is declared by a library and you're, for example, passing an argument to a function that expects such an enum. So we want a rule to detect the unsafe assignment whether we have banned enums or not.

Here is a POC: https://github.com/danielnixon/eslint-plugin-total-functions/pull/new/ban-enum-assignment

It might be nice to:

not flag string enums, only numeric
not flag assignment of number literals to numeric enum values that we can verify at compile time are safe (i.e. actually belong to the enum)

Counterargument to the second bulletpoint in above comment would be:
// Let's start with an implicit numeric enum:
enum AOrB {
  A, // Implicitly 0
  B, // Implicitly 1
}

// And then let's use it
const b: AOrB = 1; // At the time I wrote this, I believed 1 === AOrB.B

// But if someone later inserts a new value in the middle...
enum AOrB {
  A, // Implicitly 0
  C, // Implicitly 1
  B, // Implicitly 2 now!
}

// This existing code still compiles, but is now misleading/wrong, because 1 actually corresponds to AOrB.C
const b: AOrB = 1;

// If we had instead written this, there would be no issue:
const b: AOrB = AOrB.B;

// If we had instead written this, there would be no issue:
const b: AOrB = AOrB.B;

Unless of course we had a bunch of enum values stored in a database using their numeric values, in which case by adding AOrB.C we have accidentally changed their meaning in a disastrous way. Another argument against enums at all.


Alter ETag based on contained resources

JIRA Ticket: https://jira.lyrasis.org/browse/FCREPO-3541
What does this Pull Request do?
We don't change the ETag when a container has a new or deleted contained resource.  This attempts to fix that by generating a hash of all the children of the requested resource.
I think this might be slowing down ingest as I was creating 5,000 children of a single parent and it seemed to average about 0.14 per resource (which is slower than normal on my machine).
Also there is probably a limit to how many contained resources this might work with. Probably need the big test machine to see if and when this crashes.
How should this be tested?
Create a container, note it's ETag and Last Modified time.

Interested parties
Tag (@ mention) interested parties or, if unsure, @fcrepo/committers

So I'm playing with this, but I could probably use some help. Here is my current idea.
On commit, check which resources are having their containment updated (like in the current PR). But add those to a list, then asynchronously (so a separate thread) have it pull the getContains result, chop them to 100 characters max, concatenate them together and generate a SHA-256 hash, store that back into the DB. Inside a long running transaction we'd have to calculate the hash on the fly, but 🤷


Smooth Hover Transition and Box Radius for Bestselling Books and Audiobooks Box

Description
I would like to address an issue regarding the bestselling books and audiobooks box on our website. Currently, the box lacks a smooth hover transition and box radius, which affects the overall user experience and visual aesthetics.
I want to solve this issue, please assign me :)
Screenshots

Checklist

[X] I have checked the existing issues
[X] I have read the [Contributing Guidelines]
[X] I am willing to work on this issue (optional)
[X] I am a GSSoC'23 contributor

@Anshkaran7 Ok go for it

completed


Add the ability to use .env files

Add the ability to use .env files to the api-client
TODO

[ ] Install https://www.npmjs.com/package/dotenv
[ ] Update the .gitignore file in the API-Client and ignore the .env file
[ ] Create a .env.sample file in the api-client root with the following

[ ]    API_VERSION=v1
[ ]  API_BASE_PATH=http://localhost:3000/

[ ] Update API-Client install guide with a section on how to copy the .env.sample to a new .env file and make the required changes.

@pfwd Can help with installing environmental variables


[stable/hackmd]: fixes pvc mount path

Is this a new chart

NOTE: We're experiencing a high volume of PRs to this repo and reviews will be delayed. Please host your own chart repository and submit your repository to the Helm Hub instead of this repo to make them discoverable to the community. Here is how to submit new chart repositories to the Helm Hub.

What this PR does / why we need it:
Which issue this PR fixes
(optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged)

fixes pvc mount path

Special notes for your reviewer:
Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

[x] DCO signed
[x] Chart Version bumped
[x] Variables are documented in the README.md
[x] Title of the PR starts with chart name (e.g. [stable/chart])

/assign @prydonius

/ok-to-test


Fix URL in README for Python language

I don't understand what i need to do(

@veblush

Help me please. What I need to do to merge this request?

Thanks for the fix! You'll have to follow the instructions above to sign a CLA before we can accept your contribution.

Thank you! I have sign a CLA.

Currently the link appears to correspond to "Quick start" python doc. Does this need to change to Language/Python page? If so, conflict needs to be resolved to get this merged.


A question about the gradients of sampling

Hello, thank you for your great works. I have a question about how you update the sampling parameters (arch parameters). You update the the sampling parameters with the validation set. But your do not augment the validation set with the sampled augmentation. That means that, the gradients of the loss respect to the validation data is None. Then how do you update the sampling parameters by the validation set?

The gradient is estimated by the bi-level optimization algorithm. You can find it on page 9 of DADA.
Actually, the gradient of augment parameters is calculated by the second-order gradient of $$L_{train}(w - d L_{val}(w), alpha)$$, which is not the same as gradient w.r.t. augment parameters on validation data.


Error when querying SPARQL endpoint using the sparqlify example

I have converted the pizza ontology (https://protege.stanford.edu/ontologies/pizza/pizza.owl) to n-triples using OWL api and then gave it as input to the sparqlify example. It sets up the sparql endpoint ok and generic queries execute without problem, such as:
SELECT * WHERE {
  ?s ?p ?o .
}

however when I try to search for a literal with a language tag:
PREFIX rdfs: <http://www.w3.org/2000/01/rdf-schema#>
SELECT * WHERE {
  ?s rdfs:label "American"@en .
}

I get an error:
Exception in thread "Thread-45" java.lang.RuntimeException: java.lang.RuntimeException: java.lang.RuntimeException: No SQL conversion found for NodeValue: "American"^^<http://www.w3.org/1999/02/22-rdf-syntax-ns#langString> ( http://www.w3.org/1999/02/22-rdf-syntax-ns#langString)
	at org.aksw.jena_sparql_api.web.utils.RunnableAsyncResponseSafe.run(RunnableAsyncResponseSafe.java:29)
	at java.lang.Thread.run(Thread.java:748)
Caused by: java.lang.RuntimeException: java.lang.RuntimeException: No SQL conversion found for NodeValue: "American"^^<http://www.w3.org/1999/02/22-rdf-syntax-ns#langString> ( http://www.w3.org/1999/02/22-rdf-syntax-ns#langString)
	at org.aksw.jena_sparql_api.web.servlets.SparqlEndpointBase$3.run(SparqlEndpointBase.java:354)
	at org.aksw.jena_sparql_api.web.utils.RunnableAsyncResponseSafe.run(RunnableAsyncResponseSafe.java:26)
	... 1 more
Caused by: java.lang.RuntimeException: No SQL conversion found for NodeValue: "American"^^<http://www.w3.org/1999/02/22-rdf-syntax-ns#langString> ( http://www.w3.org/1999/02/22-rdf-syntax-ns#langString)
	at org.aksw.sparqlify.core.cast.TypeSystemImpl.convertSql(TypeSystemImpl.java:231)
	at org.aksw.sparqlify.core.cast.TypedExprTransformerImpl.translate(TypedExprTransformerImpl.java:766)
	at org.aksw.sparqlify.core.cast.TypedExprTransformerImpl.rewrite(TypedExprTransformerImpl.java:268)
	at org.aksw.sparqlify.core.cast.TypedExprTransformerImpl.rewrite(TypedExprTransformerImpl.java:320)
	at org.aksw.sparqlify.core.cast.TypedExprTransformerImpl.rewrite(TypedExprTransformerImpl.java:277)
	at org.aksw.sparqlify.core.cast.TypedExprTransformerImpl.rewrite(TypedExprTransformerImpl.java:320)
	at org.aksw.sparqlify.core.cast.TypedExprTransformerImpl.rewrite(TypedExprTransformerImpl.java:277)
	at org.aksw.sparqlify.core.cast.TypedExprTransformerImpl.rewrite(TypedExprTransformerImpl.java:320)
	at org.aksw.sparqlify.core.cast.TypedExprTransformerImpl.rewrite(TypedExprTransformerImpl.java:277)
	at org.aksw.sparqlify.core.cast.TypedExprTransformerImpl.rewrite(TypedExprTransformerImpl.java:320)
	at org.aksw.sparqlify.core.cast.TypedExprTransformerImpl.rewrite(TypedExprTransformerImpl.java:229)
	at org.aksw.sparqlify.util.SqlTranslatorImpl2.translate(SqlTranslatorImpl2.java:73)
	at org.aksw.sparqlify.core.algorithms.MappingOpsImpl.createExprSqlRewrites(MappingOpsImpl.java:481)
	at org.aksw.sparqlify.core.algorithms.MappingOpsImpl.createSqlConditionItems(MappingOpsImpl.java:500)
	at org.aksw.sparqlify.core.algorithms.MappingOpsImpl.createSqlCondition(MappingOpsImpl.java:522)
	at org.aksw.sparqlify.core.algorithms.MappingOpsImpl.filter(MappingOpsImpl.java:1276)
	at org.aksw.sparqlify.core.algorithms.OpMappingRewriterImpl.rewrite(OpMappingRewriterImpl.java:161)
	at org.aksw.sparqlify.core.algorithms.OpMappingRewriterImpl.rewrite(OpMappingRewriterImpl.java:324)
	at org.aksw.sparqlify.core.algorithms.OpMappingRewriterImpl.rewriteList(OpMappingRewriterImpl.java:91)
	at org.aksw.sparqlify.core.algorithms.OpMappingRewriterImpl.rewrite(OpMappingRewriterImpl.java:101)
	at org.aksw.sparqlify.core.algorithms.OpMappingRewriterImpl.rewrite(OpMappingRewriterImpl.java:336)
	at org.aksw.sparqlify.core.algorithms.OpMappingRewriterImpl.rewrite(OpMappingRewriterImpl.java:207)
	at org.aksw.sparqlify.core.algorithms.OpMappingRewriterImpl.rewrite(OpMappingRewriterImpl.java:320)
	at org.aksw.sparqlify.core.interfaces.SparqlSqlOpRewriterImpl.rewrite(SparqlSqlOpRewriterImpl.java:98)
	at org.aksw.sparqlify.core.algorithms.SparqlSqlStringRewriterImpl.rewrite(SparqlSqlStringRewriterImpl.java:44)
	at net.sansa_stack.query.spark.sparqlify.QueryExecutionSparqlifySpark.executeCoreSelect(QueryExecutionSparqlifySpark.java:39)
	at org.aksw.jena_sparql_api.core.QueryExecutionBaseSelect.execSelect(QueryExecutionBaseSelect.java:378)
	at org.aksw.jena_sparql_api.web.servlets.ProcessQuery.processQuery(ProcessQuery.java:121)
	at org.aksw.jena_sparql_api.web.servlets.ProcessQuery.processQuery(ProcessQuery.java:79)
	at org.aksw.jena_sparql_api.web.servlets.SparqlEndpointBase$3.run(SparqlEndpointBase.java:351)
	... 2 more

this seems to be valid syntax (https://www.w3.org/TR/rdf-sparql-query/#matchingRDFLiterals) so might be a bug

Ok an update: rewriting it as this works:
SELECT * {
?s ?p ?o .
    FILTER (str(?o) = "American")
    FILTER (lang(?o) = 'en')
}

I think it is related to the "@" symbol being in the string sent to the server and the protocol complains about it? Maybe this is related?
17/11/20 18:10:19 WARN WebComponent: A servlet request to the URI http://localhost:7531/sparql?query=SELECT+%2A+%7B%0D%0A%3Fs+%3Fp+%22American%22%40en+.%0D%0A%7D contains form parameters in the request body but the request body has been consumed by the servlet or a servlet filter accessing the request parameters. Only resource methods using @FormParam will work as expected. Resource methods consuming the request body by other means will not work as expected.

This might be either a missing feature; or a regression back at the time when Jena added support of RDF1.1. Will investigate.

I guess it's because of rdf:langString literal. Not sure how Sparqlify handles literals with this datatype.

Added potential fix to the underlying jena-sparql-api https://github.com/SmartDataAnalytics/jena-sparql-api/commit/c2f4b905cc5c7b28cf0232da0a1fe2172a6faf8b

Closing it as it should be fixed.


Calculate touchpad y-axis inverted coordinates

The Y axis of the touchpad is inverted. This issue has been fixed.

It wasn't required on the board where I have tried it. Maybe ST has changed the hardware in the meantime.

I knew. understand.


sync fix

If we find a transaction in a block that we have not seen before then we save it but make sure its marked as not in the mempool because it must now be onchain.  This achieves that be enabling a new parameter for calling transactionSubmittedEventHandler, so that mempool can be set (true or false, false in this case).
We also have to set the blockNumberL2 of the block that this transaction is in (by default it will be assigned -1)

Closed because code incorporated in later PR


About Apple privacy manifests

I recently learned about the new privacy management feature, Privacy Manifest, that Apple announced at this year's WWDC. I understand that relevant support will be subject to review from April 2024.
I'm particularly interested in when Maplibre will be implementing support for this new feature. Could anyone provide some insight or direct me to resources where I could find more information? Your help would be greatly appreciated. Thank you.

This is the first time this is brought up, thanks for that.
MapLibre Native is vendor-agnostic. You specify where it needs to pull data (tiles, fonts, etc.) from. So you need to include a privacy manifest that specifies the specific domains that you are using for this.
MapLibre Native itself does not do any tracking
https://developer.apple.com/documentation/bundleresources/privacy_manifest_files/describing_data_use_in_privacy_manifests?language=objc
This anti-fingerprinting measure seems relevant for us (we use NSUserDefaults).
https://developer.apple.com/documentation/bundleresources/privacy_manifest_files/describing_use_of_required_reason_api?language=objc
WWDC Video: https://developer.apple.com/videos/play/wwdc2023/10060/

We should add a PLIST file called PrivacyInfo.xcprivacy to the XCFramework that we distribute.

Adding dependency signatures would be a beneficial enhancement .
https://developer.apple.com/videos/play/wwdc2023/10061/?time=731
By the way, could you please inform me about the planned version for the addition of the privacy file?

No ETA at this point. MapLibre is not listed in the first batch of third-party SDKs that require a privacy manifest. https://developer.apple.com/support/third-party-SDK-requirements/

Should stop using .systemUptime in MLNMapView.mm or declare NSPrivacyAccessedAPICategorySystemBootTime`.
Should declare NSPrivacyAccessedAPICategoryUserDefaults.
Perhaps also NSPrivacyAccessedAPICategoryFileTimestamp?

Hey, we use your SDK in an App on the Apple Store and we just got the warning on our latest release.
TMS-91053: Missing API declaration - Your app’s code in the “APPNAME” file references one or more APIs that require reasons, including the following API categories: NSPrivacyAccessedAPICategoryUserDefaults.  While no action is required at this time, starting May 1, 2024, when you upload a new app or app update, you must include a NSPrivacyAccessedAPITypes array in your app’s privacy manifest to provide approved reasons for these APIs used by your app’s code. For more details about this policy, including a list of required reason APIs and approved reasons for usage, visit: https://developer.apple.com/documentation/bundleresources/privacy_manifest_files/describing_use_of_required_reason_api.
Same Warning for NSPrivacyAccessedAPICategoryDiskSpace,  NSPrivacyAccessedAPICategorySystemBootTime.


Test PR checker

Also merge checkReviewers and checkReviews because their purposes don't really differ that much..

Codecov Report

Merging #37 into master will increase coverage by 15.79%.
The diff coverage is 100%.

@@             Coverage Diff             @@
##           master      #37       +/-   ##
===========================================
+ Coverage   79.88%   95.67%   +15.79%
===========================================
  Files          10       10
  Lines         353      347        -6
===========================================
+ Hits          282      332       +50
+ Misses         71       15       -56

Impacted Files
Coverage Δ

lib/pr_checker.js
96.96% <100%> (+49.35%)
:arrow_up:

lib/reviews.js
98.14% <0%> (+3.7%)
:arrow_up:

lib/collaborators.js
82% <0%> (+4%)
: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 b0d428d...840083b. Read the comment docs.


is mcause inhv writeable?

Is mcause inhv writeable so if firmware writes to the xcause CSR, the value will stay?   If firmware does set inhv to 1 and it stays, when is inhv updated to reflect the actual hardware vectoring status?  after interrupts and exceptions?  only if a hardware vector successfully completes and sets inhv to zero?
We are doing some randomized verification and writing values to xcause csrs and this situation showed up.
Thanks

Based on TG discussion on 20201006:
This bit is writable, so the value will stay if firmware writes to it. Normally, it is set and clear by HW state machine for vectored mode interrupts. It is not updated for exceptions. xret does not clear it (to maintain backward compatibility). When this bit is set, it enters exception but other parts of xcause does not get updated/cleared.
Need to determine which of the following options works the best.  If HW fetch for vector table is not successful, the machine will execute the SW exception routine with mcause reflecting one of the following:

mcause.interrupt = 1, exception code = vector number, inhv = 1
mcause.interrupt = 0, exception code = some existing fault,       inhv = 1
mcause.interrupt = 0, exception code = a new type (group) of interrupt fetch faults,   inhv = N/A

As long as SW does not clear this interrupt, the same interrupt will trigger again after the fault is handled by SW.
Is this type fault even recoverable at all?
Can we ignore the vector lookup failure and fall back to pure SW ISR? Can we fall back to pure SW mode if HW vector mode fails? Do we need 2 versions of ISR in this case? Any corner cases when we have nesting interrupts?
Without vector number, it may be difficult to figure out which part of the vector table went wrong (could be captured in mtval).
Will check with Krste about his original intention.
Need to clarify the details in the spec.

The inhv bits are only written by hardware during the table vector read operation.  The inhv bits can be written by software, including when hardware vectoring is not in effect. The inhv bit has no effect except when returning from interrupt.

One use case for restart is when there is a bare-S mode running on top of a security M-mode monitor.  The M-monitor can take M-mode interrupts in the middle of an S-mode interrupt sequence - the inhv bit allows the hardware to preempt the S-mode vector table read to take the M-mode interrupt.  The M-mode interrupt handler does not have to be aware of the S-mode interrupt handler state and can simply mret as before, in which case the hardware will replay the table read.
An implementation could choose to make the interrupt/table-read sequence non-interruptible (at the cost of worst-case interrupt latency), so inhv is never reported.  But some mechanism is still needed to indicate that the table read itself caused a trap, even if fatal.   We use inhv to report this case, as the bit is used to support the other scenarios.  Using a separate cause field for this case would prevent restart in other cases.
The cases when the table read exceptions are not fatal traps are when this code is running in a lower-privilege mode under some higher-privilege mode that takes action on the table read trap then resumes (.e.g,  user-mode interrupts with S-mode paging, or S-mode guest OS on hypervisor, or U/S-mode code under M-mode that has access logging on turned on).
When the table read causes a horizontal trap, this usually signifies an error in the software configuration.  These horizontal table read traps are not generally recoverable, so some separate debug mechanism will usually be required to determine the cause of the error.  The table address will be available in mtval  for debugger to inspect.

I think there are three possible mechanisms for handling a trap of any kind during a hardware vector table lookup.  Let's say the privilege level of the vectored interrupt that has its lookup in progress is b and the privilege of the taken trap is a.  The following three ways may be ways in which the trap could be handled:

The trap can be delayed until the vector table lookup completes and the lookup value placed in aepc (this only works if the trap is not caused by the vector table lookup).  The trap at privilege a will be handled and will return to privilege b to execute the handler at aepc.
The vector table lookup can be abandoned and the state of privilege level b left with the interrupt taken but the vector table lookup remaining to be done.  Then, bcause.inhv is set and the privilege a trap taken.  It's not clear what aepc should be set to.  On return, because the bcause.inhv bit is set, the return process is entirely different.  The table location is recomputed from bcause and looked up again.  When the lookup is successful, bcause.inhv is cleared and the handler entered.
The entire interrupt can be postponed.  For this, the b interrupt's ip bit must be left set and the entire status register and current privilege kept at (or set back to) what they were before the privilege b interrupt was taken.  Then aepc set to the current bepc, and status updated for the trap to privilege a as if it were from the original instruction.  The return will be to the original instruction being executed before the b privilege vectored interrupt and the b privilege vectored interrupt taken all over again.  Interrupts work smoothly for this mechanism.

An implementation may only want to use a single mechanism.  That leaves out mechanism 1 as it can only be used for interrupts (and latency may be a problem even there).  Mechanism 2 can be used for interrupts and vertical traps and can be straightforwardly recovered from for those, but cannot be used for horizontal traps.  Mechanism 3 may be able to recover from horizontal traps, but causes the software more difficulty in figuring out what happened even for vertical traps.  It has to notice that atval points to a vector table address and know that the trap has nothing at all to do with the code that was apparently executing at the time of the trap but was because of a vectored interrupt.  This may be possible, but needs to be understood architecturally.
Mechanism 2 works straightforwardly except that we define horizontal traps as unrecoverable where we define them to do something for debug purposes.  Mechanism 3 could be used for all cases, but would need to be considered architecturally so that software would be able to deal correctly with it.  The fact that mechanism 3 needs special trap consideration even for vertical traps may make mechanism 2 and fatal horizontal traps more attractive.
If we choose mechanism 2 and don't allow mechanism 3 (by not saying what the software must handle), we need to say what aepc is set to and we need to define what a horizontal trap does.

Closed by PR #131


Best way to copy a RoaringBitmap

Hello everyone,
I am using the project RoaringBitmap in an application at work and trying to integrate RoaringBitmaps in it.
In that process, I am facing situations where our API needs to return a copy of the result.
For internal reasons, I can't just return the result of operations such as RoaringBitmap#and(RoaringBitmap, RoaringBitmap) and need to copy the result.
I see that the RoaringBitmap has no copy constructor - nor RoaringArray. There is a #clone() method, and I use it in some situation.
So far, my solution has been to create a new RoaringBitmap and using #or(RoaringBitmap) to copy all values. But I suppose that it is always heavier than just making a basic copy of the RoaringArray.
Thanks for the tips

@Kineolyan what's wrong with RoaringBitmap.clone() method?

@Kineolyan  As @maciej  may be hinting at... It is uncommon for Java classes to have "copy constructors". That's a C++ construct. In Java, we use "clone" instead.  A clone is equivalent for all practical purposes to a copy construction. I guess that doing a or with an empty bitmap would also work nearly just as well, though I expect it is going to be a tad slower.

I am going to close this as I believe it is an invalid issue. Reopen if you disagree.

@lemire I hear you. Let me process your answer and see if I can rely only on clone.
So far, no need to reopen. Thanks for the answers.


[BUG/ISSUE] Scoop not working properly in locales other than English

Before submitting the issue

[X] I have checked that the bug is not a duplicate or, if it is, it needs to be reopened
[X] I have checked that I am running the latest version of WingetUI
[X] I have tried resetting and reinstalling WingetUI and the issue persisted or happened again
[X] I have checked the FAQ section for solutions
[X] I have read the contributing guidelines and I agree with the Code of Conduct

Describe the bug
When running WinGetUI in a language other than English, the parsing of Scoop commands fails as it expects the program to be always running in English. For instance, attempting to list Scoop packages with the system language set to Spanish outputs the following:

(in English, the pieces of words jumbled in the middle of the list stand for "In line 1" and "Try again")
When trying to add a bucket to Scoop, it outputs a wrong error message in Spanish:
"scoop" no se reconoce como un comando interno o externo,
programa o archivo por lotes ejecutable.

(This stands for "Scoop is not recognized as an internal or external command, program or batch file")
The error above is incorrect, as running scoop from the terminal does show the program as running and functional - and furthermore, the program runs in English as per default.
To Reproduce
Steps to reproduce the behaviour:

On a computer with a language set to other than English, install WinGetUI.
When requested to install Scoop, do so.
Go to the "Software Updates" tab.
See the error shown above.

Expected behaviour
The program should be able to correctly list and manage Scoop buckets, regardless of the locale.
Screenshots

The Log

C:\Users\csoli\AppData\Local\Programs\WingetUI\sudo\gsudo.exe
C:\Users\csoli\AppData\Local\Programs\WingetUI\sudo
🟢 checkQueue Thread started!
🟢 Using cached lang file
🟡 Using bundled lang file (forced=True)
It took 0.003000497817993164 to load all language files
🔵 Starting main application...
Scanning for instances...
Found lock file(s), reactivating...
🔵 Starting update check
b'v1.4.10173\r\n'
None
updating winget
🔵 Language file up-to-date
b''
None
list index out of range
🟢 Downloaded latest metadata to local file
🔵 Loaded metadata from local file
b'1.2.1\r\n'
None
🔵 Version URL: https://versions.marticliment.com/versions/wingetui.ver
🟢 Updates not found
Reactivation signal ignored: RaiseWindow_[1675438766.195291]
{'wingetFound': True, 'scoopFound': False, 'chocoFound': True, 'sudoFound': True, 'wingetVersion': 'v1.4.10173\r', 'scoopVersion': 'Unknown', 'chocoVersion': '1.2.1\r', 'sudoVersion': 'gsudo v2.0.4 (Branch.tags-v2.0.4.Sha.506efa024af0cef6e4b0cfec42e0c8c5d0b1472c)\r'}
Reached main ui load milestone
🔵 Is Dark
🟢 Starting winget search, winget on C:\Users\csoli\AppData\Local\Programs\WingetUI\winget-cli\winget.exe...
🟢 Starting scoop search...
🟢 Discover tab loaded
🟢 Starting winget search, winget on C:\Users\csoli\AppData\Local\Programs\WingetUI\winget-cli\winget.exe...
🟢 Starting scoop search...
🟢 Starting choco search, choco on C:\Users\csoli\AppData\Local\Programs\WingetUI\choco-cli\choco.exe...
🟢 Upgrades tab loaded
🟢 Starting winget search, winget on C:\Users\csoli\AppData\Local\Programs\WingetUI\winget-cli\winget.exe...
🟢 Starting scoop search...
🟢 Starting choco search, choco on C:\Users\csoli\AppData\Local\Programs\WingetUI\choco-cli\choco.exe...
🟢 Discover tab loaded
IndexError: list index out of range
🟢 Scoop search finished
🔴 Traceback (most recent call last):

🔴   File ".\scoopHelpers.py", line 87, in searchForUpdates

🔴 IndexError: list index out of range

🔴 Note this traceback was caught by reporter and has been added to the log (list index out of range)
🟢 Scoop search finished
🟢 Starting scoop search...
🟢 Settings tab loaded!
🟢 About tab loaded!
🟢 Scoop bucket search finished
🟢 Main application loaded...
🟢 Chocolatey uninstallable packages search finished
🟡 Package inténtelo found in the updates section but not in the installed one, might be a temporal issue, retrying in 3 seconds...
🟡 Package En found in the updates section but not in the installed one, might be a temporal issue, retrying in 3 seconds...
🟡 Package + found in the updates section but not in the installed one, might be a temporal issue, retrying in 3 seconds...
🟡 Package + found in the updates section but not in the installed one, might be a temporal issue, retrying in 3 seconds...
🟡 Package + found in the updates section but not in the installed one, might be a temporal issue, retrying in 3 seconds...
🟢 Searching for string ""
🟢 Searching for string ""
Name                                                         Id                          Version      Available
🟢 Winget search finished
🟢 Searching for string ""
🟡 package Kanjozoku Game レーサー                            Stea failed parsing, going for method 2...
Kanjozoku Game レーサー                            Steam App 2073470                                  Unknown 102
🟢 Winget uninstallable packages search finished
🟢 Searching for string ""
IndexError: list index out of range
🟢 Scoop search finished
🟢 Chocolatey search finished
🟢 Searching for string ""
🟢 Searching for string ""
🟢 Total packages: 160
🟢 Searching for string ""
🟢 Searching for string ""
🟡 Can't get custom interval time! (got value was '')
🟢 Total packages: 6
None
🟢 Winget search finished
🟢 Total packages: 11335
🟢 Found reactivation lock file...
🟠 Icon inténtelo not found in json
🟠 Icon inténtelo not found in json
🟢 Starting get info for title nuevo./Inténtelo
🔵 Scoop does not support specific version installs
🟢 Starting get info for title nuevo./inténtelo
🔵 Scoop does not support specific version installs
🟢 Starting winget search, winget on C:\Users\csoli\AppData\Local\Programs\WingetUI\winget-cli\winget.exe...
🟢 Starting scoop search...
🟢 Starting choco search, choco on C:\Users\csoli\AppData\Local\Programs\WingetUI\choco-cli\choco.exe...
🔴 Traceback (most recent call last):

🔴   File ".\scoopHelpers.py", line 87, in searchForUpdates

🔴 IndexError: list index out of range

🔴 Note this traceback was caught by reporter and has been added to the log (list index out of range)
🟢 Scoop search finished
🟢 Searching for string ""
Name                                                         Id                          Version      Available
🟢 Winget search finished
🟢 Searching for string ""
🟢 Chocolatey search finished
🟢 Searching for string ""
🟢 Searching for string ""
🟡 Can't get custom interval time! (got value was '')
🟢 Total packages: 6
🔵 Announcement URL: https://www.marticliment.com/resources/wingetui.announcement
🔵 Image URL: https://www.marticliment.com/resources/choco.png
main
🟢 Waiting for install permission... title=main Scoop bucket, id=, installId=1675439009.5456626
🔵 Current program set to<PHONE_NUMBER>.5456626
🟢 Have permission to install, starting installation threads...
🟢 winget installer assistant thread started for process <Popen: returncode: None args: 'scoop bucket add main'>
"scoop" no se reconoce como un comando interno o externo,
programa o archivo por lotes ejecutable.
1
🟢 Starting scoop search...
['inténtelo', 'de', 'nuevo.']
IndexError: list index out of range
['+', 'scoop', 'bucket', 'list']
IndexError: list index out of range
['+', '~~~~~']
IndexError: list index out of range
['+', 'FullyQualifiedErrorId', ':', 'CommandNotFoundException']
IndexError: list index out of range
🟢 Scoop bucket search finished
🟢 Starting scoop search...
['inténtelo', 'de', 'nuevo.']
IndexError: list index out of range
['+', 'scoop', 'bucket', 'list']
IndexError: list index out of range
['+', '~~~~~']
IndexError: list index out of range
['+', 'FullyQualifiedErrorId', ':', 'CommandNotFoundException']
IndexError: list index out of range
🟢 Scoop bucket search finished
🟢 Starting scoop search...
['inténtelo', 'de', 'nuevo.']
IndexError: list index out of range
['+', 'scoop', 'bucket', 'list']
IndexError: list index out of range
['+', '~~~~~']
IndexError: list index out of range
['+', 'FullyQualifiedErrorId', ':', 'CommandNotFoundException']
IndexError: list index out of range
🟢 Scoop bucket search finished
🟠 Icon + not found in json
🟠 Icon + not found in json
🟢 Starting get info for title +
🔵 Scoop does not support specific version installs
🟢 Starting get info for title +
🔵 Scoop does not support specific version installs
🟢 Starting winget search, winget on C:\Users\csoli\AppData\Local\Programs\WingetUI\winget-cli\winget.exe...
🟢 Starting scoop search...
🟢 Starting choco search, choco on C:\Users\csoli\AppData\Local\Programs\WingetUI\choco-cli\choco.exe...
🔴 Traceback (most recent call last):

🔴   File ".\scoopHelpers.py", line 87, in searchForUpdates

🔴 IndexError: list index out of range

🔴 Note this traceback was caught by reporter and has been added to the log (list index out of range)
🟢 Scoop search finished
🟢 Searching for string ""
Name                                                         Id                          Version      Available
🟢 Winget search finished
🟢 Searching for string ""
🟢 Chocolatey search finished
🟢 Searching for string ""
🟢 Searching for string ""
🟡 Can't get custom interval time! (got value was '')
🟢 Total packages: 6
🔵 Sending cancel signal...
🔵 Announcement URL: https://www.marticliment.com/resources/wingetui.announcement
🔵 Image URL: https://www.marticliment.com/resources/choco.png
🟢 Starting scoop search...
['inténtelo', 'de', 'nuevo.']
IndexError: list index out of range
['+', 'scoop', 'bucket', 'list']
IndexError: list index out of range
['+', '~~~~~']
IndexError: list index out of range
['+', 'FullyQualifiedErrorId', ':', 'CommandNotFoundException']
IndexError: list index out of range
🟢 Scoop bucket search finished
main
🟢 Waiting for install permission... title=main Scoop bucket, id=, installId=1675439355.981606
🔵 Current program set to<PHONE_NUMBER>.981606
🟢 Have permission to install, starting installation threads...
🟢 winget installer assistant thread started for process <Popen: returncode: None args: 'scoop bucket add main'>
"scoop" no se reconoce como un comando interno o externo,
programa o archivo por lotes ejecutable.
1
🟢 Starting scoop search...
['inténtelo', 'de', 'nuevo.']
IndexError: list index out of range
['+', 'scoop', 'bucket', 'list']
IndexError: list index out of range
['+', '~~~~~']
IndexError: list index out of range
['+', 'FullyQualifiedErrorId', ':', 'CommandNotFoundException']
IndexError: list index out of range
🟢 Scoop bucket search finished
🔵 Copying log to the clipboard...
🟢 Log copied to the clipboard successfully!
🔵 Copying log to the clipboard...
🟢 Log copied to the clipboard successfully!
🔵 Copying log to the clipboard...
🟢 Log copied to the clipboard successfully!
🔵 Copying log to the clipboard...

Additional context
Fresh installation of WinGetUI and of Scoop in my computer.

This might be a regression, as bug #122 (same issue but with Chinese locale) claims to have had the issue fixed. Also related to bug #427 .

hi, can you please send me the location where scoop is installed?

could you please install scoop from the settings window?

Hello, I checked one of the other bugs and tried rebooting the computer after installing Scoop from the settings. This solved the issue for me.

All right, then


Unable to use with python project

We have an issue with a python project (unfortunately, I can't share the requirements.txt file).
pip install is running successfully, but license_finder crashes with:
root@c616ea3b9557:/code# license_finder
LicenseFinder::Pip: is active
Traceback (most recent call last):
  File "/usr/local/rvm/gems/ruby-2.5.1/gems/license_finder-5.3.0/bin/license_finder_pip.py", line 29, in <module>
    in pkg_resources.working_set.resolve(requirements)]
  File "/usr/local/lib/python2.7/dist-packages/pip/_vendor/pkg_resources/__init__.py", line 778, in resolve
    raise DistributionNotFound(req, requirers)
pip._vendor.pkg_resources.DistributionNotFound: The 'flower>=0.7.3; extra == "celery"' distribution was not found and is required by airflow
Traceback (most recent call last):
        22: from /usr/local/rvm/gems/ruby-2.5.1/bin/license_finder:23:in `<main>'
        21: from /usr/local/rvm/gems/ruby-2.5.1/bin/license_finder:23:in `load'
        20: from /usr/local/rvm/gems/ruby-2.5.1/gems/license_finder-5.3.0/bin/license_finder:5:in `<top (required)>'
        19: from /usr/local/rvm/gems/ruby-2.5.1/gems/thor-0.20.0/lib/thor/base.rb:466:in `start'
        18: from /usr/local/rvm/gems/ruby-2.5.1/gems/thor-0.20.0/lib/thor.rb:387:in `dispatch'
        17: from /usr/local/rvm/gems/ruby-2.5.1/gems/thor-0.20.0/lib/thor/invocation.rb:126:in `invoke_command'
        16: from /usr/local/rvm/gems/ruby-2.5.1/gems/thor-0.20.0/lib/thor/command.rb:27:in `run'
        15: from /usr/local/rvm/gems/ruby-2.5.1/gems/license_finder-5.3.0/lib/license_finder/cli/main.rb:95:in `action_items'
        14: from /usr/local/rvm/gems/ruby-2.5.1/gems/license_finder-5.3.0/lib/license_finder/license_aggregator.rb:13:in `any_packages?'
        13: from /usr/local/rvm/gems/ruby-2.5.1/gems/license_finder-5.3.0/lib/license_finder/license_aggregator.rb:13:in `map'
        12: from /usr/local/rvm/gems/ruby-2.5.1/gems/license_finder-5.3.0/lib/license_finder/license_aggregator.rb:15:in `block in any_packages?'
        11: from /usr/local/rvm/rubies/ruby-2.5.1/lib/ruby/2.5.0/forwardable.rb:223:in `any_packages?'
        10: from /usr/local/rvm/gems/ruby-2.5.1/gems/license_finder-5.3.0/lib/license_finder/core.rb:76:in `decision_applier'
         9: from /usr/local/rvm/gems/ruby-2.5.1/gems/license_finder-5.3.0/lib/license_finder/core.rb:81:in `current_packages'
         8: from /usr/local/rvm/gems/ruby-2.5.1/gems/license_finder-5.3.0/lib/license_finder/scanner.rb:15:in `active_packages'
         7: from /usr/local/rvm/gems/ruby-2.5.1/gems/license_finder-5.3.0/lib/license_finder/scanner.rb:15:in `flat_map'
         6: from /usr/local/rvm/gems/ruby-2.5.1/gems/license_finder-5.3.0/lib/license_finder/scanner.rb:15:in `each'
         5: from /usr/local/rvm/gems/ruby-2.5.1/gems/license_finder-5.3.0/lib/license_finder/package_manager.rb:90:in `current_packages_with_relations'
         4: from /usr/local/rvm/gems/ruby-2.5.1/gems/license_finder-5.3.0/lib/license_finder/package_managers/pip.rb:12:in `current_packages'
         3: from /usr/local/rvm/gems/ruby-2.5.1/gems/license_finder-5.3.0/lib/license_finder/package_managers/pip.rb:52:in `pip_output'
         2: from /usr/local/rvm/rubies/ruby-2.5.1/lib/ruby/2.5.0/json/common.rb:440:in `JSON'
         1: from /usr/local/rvm/rubies/ruby-2.5.1/lib/ruby/2.5.0/json/common.rb:156:in `parse'
/usr/local/rvm/rubies/ruby-2.5.1/lib/ruby/2.5.0/json/common.rb:156:in `parse': 765: unexpected token at '' (JSON::ParserError)

I'm using the latest  image on https://hub.docker.com/r/licensefinder/license_finder/tags/
Any idea how to get rid of this?
Thanks

Have you tried pip installing your requirements.txt before running license_finder? I had the same sort of error, and I apparently fixed it by installing my requirements. I got the idea from GitLab's use of LicenseFinder for Python projects. This is their LicenseFinder Dockerfile entrypoint script:
https://gitlab.com/gitlab-org/security-products/license-management/blob/master/run.sh#L106

Alternatively: "If you do not want to manually run an individual package manager's prepare command (ex: bundle install, npm install, etc) to ensure your project is fully prepared to be scanned, use the --prepare or -p option which will run each active package manager's prepare command for you." Haven't tried that myself, but that's direct from the docs.

I was inspired from https://gitlab.com/gitlab-org/security-products/license-management/blob/master/run.sh#L106 as well.
The issue is the licensefinder/license_finder image is based on ubuntu where python links to /usr/bin/python2.  It's also missing the python3-pip package.
What I did to solve the issue was I made another image based on licensefinder/license_finder that did some setup as part of the entrypoint and installed python3-pip.
if test -f requirements.txt; then
  if test ${PYTHON_VERSION:-3} = "3"; then
    ln -sf $(which python3) $(which python)
    ln -sf $(which pip3) $(which pip)
  fi
fi

I think this is a suitable workaround until python2 is EOL.

I think this is a suitable workaround until python2 is EOL.

Even after Py2 is EOL (in a few months), python should continue to refer to python2 ("for the time being"), so that workaround could stay around "forever". In general, using python is discouraged, unless it’s code that’s known to work under both Py2 and Py3. See https://www.python.org/dev/peps/pep-0394/

You can now also add the Python version via a command line parameter:

$  license_finder help
....
[--python-version=PYTHON_VERSION]                # Python version to invoke pip with. Valid versions: 2 or 3. Default: 2
...

As the python version used for license finder defaults to Python 2 it will fail on Python 3 projects.
This was introduced with https://github.com/pivotal/LicenseFinder/pull/622


Rename structural constituent of carboxysome -> carboxysome shell

See previous discussion #19379
id: GO:0043886
name: structural constituent of carboxysome
namespace: molecular_function
def: "The action of a molecule that contributes to the structural integrity of a carboxysome, an organelle found in all cyanobacteria and some chemoautotrophs, consisting of a proteinaceous coat and enzymes for the fixation of CO(2)." [GOC:jl, PMID:28934381]
is_a: GO:0005198 ! structural molecule activity
intersection_of: GO:0005198 ! structural molecule activity
intersection_of: occurs_in GO:0031470 ! carboxysome
property_value: term_tracker_item https://github.com/geneontology/go-ontology/issues/19743 xsd:anyURI

I find this odd on a number of levels.
The name is strange. I think it just means carboxysome shell
also why the lopsided structure? of all the 3 BMCs why does this one only have the structural constituent pattern instantiated?

It looks like we have duplicative/redundant annotations between the SC term and the CC term. I think it makes sense to have two terms to distinguish the shell from things in the shell (e.g. rubisco). But I think we need clear guidelines as to which term to annotate when.
Regarding the name:
https://scholar.google.com/scholar?hl=en&as_sdt=0%2C5&q="structural+constituent+of+carboxysome"&btnG=

contrast:
https://scholar.google.com/scholar?hl=en&as_sdt=0%2C5&q="carboxysome+shell"&btnG=

I realize carboxysome shell is not a MF name, but do we need MFs for these? Do we need to duplicate parts of CC in MF just so people can check the box with an MF annotation?

also: isn't the 'structural constituent' a complex?

Hi @cmungall
I also questioned this term here #19743
but Andrea confirmed that this was the MF of some proteins.
Close?

do you think 'carboxysome shell' is a good synonym?

do you think 'carboxysome shell' is a good synonym?

Maybe this is a tangent -
Quoting from PMID: 24006469,

Carboxysomes encapsulate the CO2-fixing enzyme within the selectively permeable protein shell and simultaneously encapsulate a carbonic anhydrase enzyme for CO2 supply from a cytoplasmic bicarbonate pool.

and

Carboxysomes are specialized protein microcompartments composed of a polyhedral protein shell within which cyanobacteria concentrate CO2 around their primary carboxylating enzyme, d-ribulose 1,5-bisphosphate carboxylase/oxygenase (RubisCO).

So on the one hand a carboxysome appears to have the structure of a large complex, although perhaps not one with a fully defined composition and structure, but on the other distinctive functions are mediated by shell components and inside components, so maybe two cell component terms are needed for shell and innards, and a grouping term "carboxysome" for cases where a mire precise localization is not possible? (Analogously to the usual treatment of a eukaryotic intracellular membrane-bounded vesicle?
Also, the definition of GO:0043886 structural constituent of carboxysome, "The action of a molecule that contributes to the structural integrity of a carboxysome, an organelle found in all cyanobacteria and some chemoautotrophs, consisting of a proteinaceous coat and enzymes for the fixation of CO(2)," appears not to include the transport and catalytic functions of gene products, but onluy to their role in keeping the structure intact.

@cmungall wouldn't 'carboxysome shell' be a CC?

Decision on ontology call: change label to 'structural constituent of carboxysome shell'


start preclimate and charging not possible (VW eUP)

Start/stop charging and climate, as well as read sensors like light and door status not available

Same for Peugeot e208 climate, and it is possible via PSA API so it should also be possible for tronity


Deadcode elimination optimizatio pass breaking (invalid jump position: 0)

Hi, first of all thanks for such a great project! We wanted to report a recent issue we found.
When using return statements that don't return any value, we got the next error from the compiler:
panic: invalid jump position: 0

This triggers usually when there are multiple exit points in the function (returns), and also there are conditional jumps (IFs or FORs loops with return statements). This is a small example that triggers the bug:
fmt := import("fmt")

fmt.println("Hello")

f := func(a, b) {
    if a > 0 {
        return
    }

    // This standalone return statement causes the deadcode optimization pass to break,
    // if you remove it, everything works as expected.
    return

    return 123
}

r := f(0, 2)

fmt.println("Bye")
fmt.println(r)
fmt.println(is_undefined(r))

I think the issue must be or in the optimization pass or even before on how the bytecode for the standalone return statement is generated. At the moment I didn't have time to investigate further, but simply disabling for the time being the deadcode elimination solves the problem for us as a stopgap solution.
Let us know if you need more details or how we can help. Will post here if I discover more details.
We are running with version: v2.10.0 (4846cf524306)

Thanks for reporting the bug. I have a fix for this in PR now


Disk image compress pipelines refactor

Draft as a followup for https://github.com/osbuild/images/pull/504 - as a starting point for a discussion how to avoid duplicating the compression and image stages, they are extremely similar right now but also subtly different (like edge images set no ImageFormat right now). The individual commits also need some splitting and more tests

Mark this ready?

Needs a rebase.

Closing for now as this needs some more time/thinking with the new all-manifest


Dataset download issue

I saw a comment saying that You will uploading a dataset using another cloud. When can you upload it?

could you please open the source code for making dataset ?

OK, I can send the code to your E-mail.

so, can you seed me a copy? Thanks a lot. @Hydrogenion

so,can you send me a copy too ? @Hydrogenion  my address<EMAIL_ADDRESS>

OK, I can send the code to your E-mail.
can you send me the code for making dataset too, thank you very much!  my email<EMAIL_ADDRESS>

could you please open the source code for making dataset ?
Have you received the code given to you by the original author? If so, can you send me a copy? my email<EMAIL_ADDRESS>

Can you send me a copy for making dataset too? Thanks a lot! my email<EMAIL_ADDRESS>@Hydrogenion @duzhenjiang113 @songwei100110 @ProgrammerSakura

@Hydrogenion  can you plz also mail me too<EMAIL_ADDRESS>Thanks

@Hydrogenion can you plz also mail me too<EMAIL_ADDRESS>Thanks

Already sent to you!

@Hydrogenion Could you please also send me a cope of datasets generation. Because I cannot download the dataset. Thanks a lot!!!

@Hydrogenion Could you please send me dataset? I can't join baidu. my email<EMAIL_ADDRESS>Thanks!

could you send me the code for making dataset too, thank you very much! my email<EMAIL_ADDRESS>

could you email me too at<EMAIL_ADDRESS>

could you send me the code for making dataset too, thank you very much! my email<EMAIL_ADDRESS>@Hydrogenion


Security

So I finally got the button to work after giving in and actually reading the fine-print: "remove paypal security"
Quoted from https://github.com/jeffharrell/MiniCart:

I installed the Mini Cart, but my website still redirects to PayPal ...

The other cause is that the Mini Cart doesn’t yet work with what PayPal’s call their “hosted” or “encrypted” buttons which is why this is most likely happening. To fix your buttons, you’ll need to log into paypal.com and do the following steps:
Create a button on PayPal’s website and uncheck the Save button at PayPal checkbox under Step 2: Track inventory, profit & loss.
Once you’ve created the button click Remove code protection before copying your button’s code.

I'd think that this means that any cracker/hacker could make a copy of my paypal button (from my site's view source), place it on a site of their own after changing the price from e.g. $200 to $0.05, and place a legitimate order?!
Am I wrong? What do you suggest? Are you currently working on integrating security so that Paypal can verify the legitimacy of a button before allowing checkout?
Other than that, I really appreciate you making this code publicly available since I'm not a programmer but am currently redeveloping my fiancées site ... provided I can make secure use of your code, be happy to make a donation  ;-)
Cheers,
=) Detlev

Hi Detlev,
Yes, your assumption is correct and a button's source code can be changed if someone had malicious intentions. It's recommended to always check the payment amount received from a buyer before shipping the products, whether your using the Mini Cart or not.
This isn't as much a Mini Cart caveat as it is a PayPal non-hosted button one. The Mini Cart takes advantage of the unencrypted data and renders the cart with it. The Mini Cart is an attempt at removing friction from both buyers and sellers and was designed to be a one-size fits all for installation.
The PayPal APIs do not currently support:

A way to consume from data from them without using some type of properly configured scripting technology on the host web server (removing the one-size fits all solution capability)
Hosted button data being sent as a complete cart to their servers (aka cart upload).

I've provided feedback on this to them, but they're both currently blockers in supporting hosted / encrypted buttons.
That being said, I may end up hacking my way through it and creating a variety of scripts (PHP, etc.) which can be configured by a merchant to look up hosted button data. That's a larger scale change though, so in the meantime I'd recommend you just keep an eye out for invalid order totals.

FYI, I'm also going to re-title this Issue and mark it as a feature to look at for the next major release.

Might be important to mention this somewhere on the homepage. Otherwise someone could end up using minicart without realizing the security concerns.


setData with hidden columns shows incorrect values

Hi everyone!
When I set data into a table that has hidden columns, the new data is, at first, shown on incorrect columns. Data follows the visible columns order and not the name property. After a change in the visibility status, data is correctly organized.
An example. Columns Start, Finish and Duration.

If I hide Finish column and load data into table again, values from finish are show on Duration column.

And by changing finish column visibility, data are back on the correct columns.
In addition, the filter options remain the same, after hiding columns, as shown below.
Before hiding "Finish" column

After hiding "Finish" column

Is this an issue or is there any workaround that I should be doing?
Thanks in advance.

Hi everyone!
For the data problem we did a "workaround" showing all columns before setting data and then hiding again the ones that were previously hidden.
For the filter problem we made a 2 line change on hideColumn and showColumn functions and we're submitting a pull request.
https://github.com/jspreadsheet/ce/pull/1337

I just ran into this same issue. I guess no one is going to merge #1337?!

I just ran into this same issue. I guess no one is going to merge #1337?!

I will release a patch including this one.

Any fix on this?


added support for nested password fields option in hash password hook

Summary
Hash password hook does not currently support nested fields e.g. field: 'nested.password'
No open issues currently.
Added lodash get and set to check and set the nested property, also added a test case.

Thank you! Released in v1.2.1


Text Classification using Transformers

The idea is to make a new beginner-level NLP tutorial combining this transformers example and this Text CNN notebook.

Hi! Do you guys have any other tutorial that you think would be nice to have here? I would like to help here

@LiviaCavalcanti Hi, thank you for your interest !
Would you want to help on this tutorial ? Currently I think the slot is free and your contribution on this would be really helpful and appreciated 😊

@LiviaCavalcanti Hi, thank you for your interest !
Your contribution would be great, very helpful and appreciated. Can you join our discord channel to discuss further ? See https://github.com/pytorch/ignite#communication

@LiviaCavalcanti we were also thinking about

https://github.com/pytorch/ignite/issues/1735

@Priyansi can you open issues in this repository for cross-validation tutorial and hyperparameter tuning, please ?

@LiviaCavalcanti we were also thinking about

Port Ray example to Ignite pytorch/ignite#1735

@Priyansi can you open issues in this repository for cross-validation tutorial and hyperparameter tuning, please ?

@vfdev-5 sounds good. I am interested in working on it. I am gonna wait for the issues right?

@Priyansi can you open issues in this repository for cross-validation tutorial and hyperparameter tuning, please ?

Cross-validation would work better as a how-to guide IMO #28 and here is the hyperparameter tuning tutorial #29 (no ideas on how to proceed with this right now).

Thanks @Priyansi. I will think about how we could do it and bring the discussion to the issues. Does it work?


Refactor the TTD usage and add test

CM-1182

Define uid1Eids for the trade desk identifier and refactor its usage.
the uid1 respect the priority when ttd is present in different modules.

PR opened: https://github.com/prebid/Prebid.js/pull/11174


Update swagger documentation

Fixes #2635. This PR fixes the following:

Add docs for missing endpoints
Add documentation for request parameters
Make parameter naming consistent
Fix response documentation

This PR has no functional change.

LGTM

Codecov Report

Merging #2899 into master will increase coverage by 0.05%.
The diff coverage is 78.94%.

@@            Coverage Diff            @@
##           master   #2899      +/-   ##
=========================================
+ Coverage   26.85%   26.9%   +0.05%
=========================================
  Files          89      89
  Lines       17607   17616       +9
=========================================
+ Hits         4728    4740      +12
+ Misses      12193   12190       -3
  Partials      686     686

Impacted Files
Coverage Δ

routers/api/v1/misc/version.go
0% <0%> (ø)
:arrow_up:

routers/api/v1/misc/markdown.go
82.25% <100%> (+4.25%)
: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 47f40cc...d4e9781. Read the comment docs.

Two little comments that should be fixed. For the rest LGTM. Thanks for doing it, it takes time and few good catch like https://github.com/go-gitea/gitea/pull/2899/files#diff-2620fdbeeb83ce43692534e6c2c39452R315.

@sapk Fixed. The "Terms of Service" link doesn't work, so that is still removed, everything else is restored.


Cannot execute GraphQL queries on GitHub Enterprise Server with hub api

Thanks for this great tool! ✨
I'm able to execute REST API requests using hub api against GitHub Enterprise Server:
GITHUB_HOST=git.acme.org hub api user

Unfortunately, GraphQL requests don't seem to work:
GITHUB_HOST=git.acme.org hub api graphql -F<EMAIL_ADDRESS>
I receive the error:
{
  "message": "Not Found",
  "documentation_url": "https://developer.github.com/enterprise/2.18/v3"
}

I'm using hub version 2.12.7.

Unfortunately, GraphQL requests don't seem to work:

Thanks for reporting. This is a bug. Hub is trying to POST to git.acme.org/api/v3/graphql, whereas it should be posting to git.acme.org/api/graphql.
I will fix this soon. In the meantime, you can use this as workaround:
GITHUB_HOST=git.acme.org hub api /api/graphql ...

This style of invocation will work on GHE; unfortunately it's not portable back to dotcom.


Price return from uniswap QuoterV2 on arbitrum incorrect

Describe the bug
Hi,
im using the latest hummingbot version and getting a completly wrong price back from the gateway.

i had to disable useRouter, because when using the UniswapSmartRouterV2, im getting the error "price query failed 404"
my config:

the tokenList contains the correct token addresses and decimals

gateway_files/conf/uniswap.yml
# how much the execution price is allowed to move unfavorably from the trade
# execution price. It uses a rational number for precision.
allowedSlippage: '2/100'

# the maximum gas used to estimate gasCost for a uniswap trade.
gasLimitEstimate: 150688

# how long a trade is valid in seconds. After time passes uniswap will not
# perform the trade, but the gas will still be sent.
ttl: 600
useRouter: false
feeTier: 'MEDIUM'
# For each swap, the maximum number of hops to consider.
# Note: More hops will increase latency of the algorithm.
maximumHops: 4

contractAddresses:

  arbitrum_one:
    uniswapV3SmartOrderRouterAddress: '0x68b3465833fb72A70ecDF485E0e4C7bD8665Fc45'
    uniswapV3NftManagerAddress: '0xC36442b4a4522E871399CD717aBDD847Ab11FE88'
    uniswapV3QuoterV2ContractAddress: '0x61fFE014bA17989E743c5F6cB21bF9697530B21e'

hummingbot_files/conf/strategies/conf_xemm_1.yml
#######################
###   None config   ###
#######################

strategy: cross_exchange_market_making

# The name of the maker exchange connector.
maker_market: whitebit

# The name of the taker exchange connector.
taker_market: uniswap_ethereum_arbitrum_one

# The name of the maker trading pair.
maker_market_trading_pair: HDN-USDT

# The name of the taker trading pair.
taker_market_trading_pair: HDN-USDT

# The minimum estimated profitability required to open a position.
min_profitability: 1.0

# The strategy order amount.
order_amount: 1000.0

# Adjust order price to be one tick above the top bid or below the top ask.
adjust_order_enabled: true

# Refresh orders by cancellation or by letting them expire.
order_refresh_mode: {}

# Volume requirement for determining a possible top bid or ask price from the order book.
top_depth_tolerance: 0.0

# Minimum time limit between two subsequent order adjustments.
anti_hysteresis_duration: 60.0

# Taker order size as a percentage of volume.
order_size_taker_volume_factor: 25.0

# Taker order size as a percentage of the available balance.
order_size_taker_balance_factor: 99.5

# Order size as a maker and taker account balance ratio.
order_size_portfolio_ratio_limit: 16.67

# Convert between different trading pairs using fixed conversion rates or using the rate oracle.
conversion_rate_mode: rate_oracle_conversion_rate
# Allowed slippage to fill ensure taker orders are filled.
slippage_buffer: 5.0

# Usd the debug price shim to mock gateway price.
debug_price_shim: false

# Gateway transaction cancellation timeout.
gateway_transaction_cancel_interval: 120

Steps to reproduce
see above
Release version
latest
Type of installation
Docker
Attach required files
No response

Could you please provide the exact steps to reproduce and How you was able to disable UniswapSmartRouterV2.
The possible workaround could be changing the nodeURL to infura.
Could be that XEMM needs a fix since we had fix for router for uniswapLP in the past.

Hi @philipp1992 I was having similar issues with Arbitrum this month. From my experience, the quoterV2 works properly.
The reason your quoted price doesn't match your expectation could be caused by different pool from which you are referring to. You can log the pool address in your gateway/src/connectors/uniswap/uniswap.ts: getPool() function to see which exact pool quoterV2 returns.
The getPool method takes only three parameters: tokenA.address, tokenB.address and feeTier. So one thing you can try is tweaking the config feeTier by running gateway config uniswap.feeTier [LOWEST/LOW/MEDIEUM/HIGH]

Hi @nikspz However for alpha-router, I'm still having the same 404 issue. I tried different nodes including ankr, allnodes and infura. By adding logging into gateway/src/connectors/uniswap/uniswap.ts I can see the program is interrupted at gateway/src/connectors/uniswap/uniswap.ts:299 const route = await this._alphaRouter.route(...
If I set useRouter to False by gateway config uniswap.useRouter False, I can get the price quote successfully from quoterV2.

Did you tried to use amm_arb strategy? @Quantum-C

I was running script amm_get_price_example

Did you tried to use amm_arb strategy? @Quantum-C

Will give it a try later today


Cell output height for multiple pyplot  images in Notebook Editor is too small

Bug: Notebook Editor, Interactive Window, Editor cells
When using pyplot  multiple times in same cell ( to display multiple images one below other), VS Code Notebook cell outputs are almost always with reduced height showing just part of image in first row, and need to be scrolled down to show other images.
Expected behavior
Expected is for output height to scale with output, just like in JupyterLab.
Alternative workaround:

if cell is unable to determine correct height, it should be at least possible to set output height on per cell basis.

Impact:
It makes large notebooks even harder to use in VS Notebook Editor.  It is hard to find correct image by scrolling large notebook, when half or 2/3rd of that image (or multiple row images ) is hidden .
And large notebooks are already hard to use in VS Code due to several other issues :

inability to hide input cells ( like cells with large code )
inability to use links to jump to document sections ( for example in summary )
this partial showing of image outputs makes it even harder to find correct spot in large notebook

Needless to say, all of above works in Jupyter without issues.

Hi @gmnenad VS Code currently supports the following settings:
Python › Data Science: Enable Scrolling For Cell Outputs --> If you set this to false you can see your entire cell outputs
Python › Data Science: Max Output Size --> (This does not matter if you have the previous setting set to false) This allows you to resize the cell outputs to your designated height
Do these help?

@claudiaregio
Thanks, that solves this issue .


[action][unlock_keychain] Fix randomly failing unit tests for Ubuntu

Checklist

[x] I've run bundle exec rspec from the root directory to see all new and existing tests pass
[x] I've followed the fastlane code style and run bundle exec rubocop -a to ensure the code style is valid
[x] I've read the Contribution Guidelines
[x] I've updated the documentation if necessary.

Motivation and Context

unlock_keychain spec was failing randomly on CI only for Ubuntu
Failing CI job: https://app.circleci.com/pipelines/github/fastlane/fastlane/2973/workflows/d0d44212-5f02-4d39-8bcc-d1875b9d50de/jobs/69125
or
https://app.circleci.com/pipelines/github/fastlane/fastlane/2972/workflows/768e3b5c-0aa6-4910-9e98-36770f013f74/jobs/69120

Description

Now using mock for the FastlaneCore::Helper keychain_path

Testing Steps

CI should pass now for all the jobs!
Run spec in local using: bundle exec rspec fastlane/spec/actions_specs/unlock_keychain_spec.rb

Does just moving the Tempfile.new into a let fix this? 🤷♂ I’d like to not mock keychain_path if possible since it’s good to test that out in this action 😊

Seems to work in this PR 💪 and it was a very random failure...I will keep an 👀  on this spec in future CI jobs
For now, Removed the keychain_path mocking...please re-review!


TagUI cannot be closed after closing the browser - expected, pending details

Hi kensoh,
When tagui runs' r.lose ()' after closing the brower, the process cannot be closed. I found this line of code  ' while _process.poll() is None: pass' at funtion close() in tagui.py got stuck in an infinite loop until I restart.  Could you please help to fix it? The test code  is as fellow:
r.init(visual_automation=True)
r.debug()
r.url('https://github.com/tebelorg/RPA-Python')
r.wait(2)
keyboard('[alt][space]')
keyboard('[alt][F4]')
r.close()

Hi @RichiardChi, the expected design is using close() to close the process and the browser. If user accidentally close browser before close() function runs, you can ignore it. The next time init() runs, it will kill dead processes and start clean.