DECIMAL is specified without precision and scale, causing an syntax error.

Describe the bug
When a field/property has type decimal the statementgenerator maps the type to DECIMAL instead of DECIMAL(precision,scale) where precision and scale must be some integer number.
To Reproduce
see description
Expected behavior
Precision and scale are set on a DECIMAL type. Precision and scale need to be configurable.
Environment (please complete the following information):

ksqlDb version: 7.6.0
ksqlDb.RestApi.Client Nuget package version 4.0.0-rc.4
TFM net8.0

I would like to provide a PR

Cool, its already fixed in main using the DecimalAttribute.
Can you provide a new release candidate

would it make sense to use a documented default value in case the DecimalAttribute is missing?
      if (type == typeof(decimal))
      {
        var decimalMember = memberInfo.TryGetAttribute<DecimalAttribute>();

        if (decimalMember != null)
        {
          return $"({decimalMember.Precision},{decimalMember.Scale})";
        }
        else
        {
          return $"(14,14)";
        }
      }

Initially, I was hesitant to decide on a default precision and scale for the DECIMAL type. I'm considering whether it's better to throw a .NET exception when the DecimalAttribute is missing instead of setting a default. What do you think?

decimal in dotnet has 28 significat digits.
I defaulted to 14 for precision and 14 for scale.
I have two views.

On one hand I prefer being explicit and would set the attribute. This works great in a code first approach where I control the types that are serialized.
On the other hand I prefer configuration and would configure the contextoptions with the needed information to aid serialization/translation. This works great in a schema first approach where I do not control the types that are serialized but instead get them via code-gen.

Currently I would like to use the code as is from main that uses the DecimalAttribute and could implement the second approach as a feature later. This would cover StructAttribute, KeyAttribute and DecimalAttribute. The idea would be to set the properties that should be treated as if they have those attribites in the EntityCreationMetadata. The translator could check for the attribute on the member and the registered properties attribute tuple. This would allow for a schema first approach without the need to set teh attributes on types that are coming from code-gen.

I'm growing increasingly curious. Would you recommend developing a code-generation tool and incorporating a database schema-first approach alongside the current code-first approach? From what I gather, this seems more like an epic endeavor rather than just a simple story.

Ksqldb works with avro, protobuf and json-schema when leveraging schema-registry. There are already tools available that do the code-gen part from those schemas. I would like to leverage those tools and when using this library to tell it which properties/types should be treated as Key, Struct etc. without the need to touch the generated code.
This would allow to not commit generated code to a repo and keep the generation as part of the build process.

Thank you for the clarification, I think that now I understand your use case. I think that it is rather about introducing a more fluent API that would allow to configure entities (from-item types):
public class TweetConfiguration : IFromItemTypeConfiguration<Tweet>
{
    public void Configure(FromItemTypeBuilder<Tweet> builder)
    {
        builder
            .Property(c => b.Amount)
            .Precision(14, 14);
    }
}

The above code is of course inspired by EF Core.
The ideal scenario would involve effortlessly extending pre-existing (generated) classes/records with additional decorations.

Let me know in case it is safe to merge this PR "Enabled nullable reference type checks in the project" #65 to the main branch, please.

@mrt181 feel free to reach out if you require any assistance from my end. We could potentially divide the task into sub-tasks and collaborate on crafting the final solution for the fluent API together.

I created a branch with a model builder prototype. I'd appreciate your feedback when you have a moment.

@mrt181, could you please confirm if you intend to submit a PR as previously mentioned, or should I proceed with handling the task?

The requested functionality has been just released:
dotnet add package ksqlDb.RestApi.Client --version 5.0.0


As a dashboard viewer I would like the see all the different violation types in the past 30 days

Note that the API of what I believe is the way to group types currently returns an error and is blocked by Socrata:
https://opendata.miamidade.gov/resource/tzia-umkx.json?$select=issue_type, count(*) AS total&$group=issue_type&$limit=5

Did the ETL, so this is working now. Might recommend that we don't show all the different violation types as it clutters the chart, but take a look.


add MCP server badge

This PR adds a badge for the Perplexity MCP Server server listing in Glama MCP server directory.

Glama performs regular codebase and documentation scans to:

Confirm that the MCP server is working as expected
Confirm that there are no obvious security issues with dependencies of the server
Extract server characteristics such as tools, resources, prompts, and required parameters.

This badge helps your users to quickly asses that the MCP server is safe, server capabilities, and instructions for installing the server.

just noticed that the license is missing
would you mind adding one?
https://docs.github.com/en/communities/setting-up-your-project-for-healthy-contributions/adding-a-license-to-a-repository

Thanks, I will!

Thanks! are you on the MCP Discord?
Let me know your Discord username and I will give you a special flair that's only for MCP server authors.


Added echoing green

Made some edits to clarify what the fellowship is for (and that it's a grant rather than an investment). Thank you! ✨


Remove old GNUmakefiles

NOTE NOTE NOTE NOTE NOTE
This one is an unknown. Does Harvard still require these GNUmakefile to exist? If so, this pull request should be closed and the branch deleted.

Do the GNU makefiles in the repo still work (or could work with a bit of work), or are you removing files that are beyond help at this point?

If you desperately want them, they could be restored.  As you say they may need a bit of modification, but I can't imagine it would be very much.
Our strong preference is to delete them asap.   Please remind me GCHP's timeline for cmake?

We plan on moving to cmake at the same time that we incorporate the new git repos. We are meeting in a couple weeks to further discuss, but Seb and I both feel confident we will not issue a version with both GNU make and cmake, but will move straight to cmake only.


online version of docs would be nice

So people don't have to download and build them just to see if your project is worth downloading in the first place. Or at least some basic usage.

Absolutely!  I'll work on adding a readthedocs output and a few other cleanups for this project over the next couple of weeks.  Is there anything in particular you'd like to see in the documentation?

So I checked out the project again and I found that I do have online documentation available at https://crumbs.readthedocs.io/en/latest/.  I did spend a few pull requests trying to get this project updated but it's architecture is simply too old to be easily made modern.
My stance is that this shouldn't be used and I also am starting to think that argparse shouldn't be used either but I'll leave this available in case others are still finding it useful for their projects.
Otherwise, is there a simple way I could make the documentation more visible?  I'm guessing a link from the README would be helpful, but where did you look for documentation and find that none was available?

Thank you for responding. A link to that site in the readme where you have the pydoc line would be nice. A sample snippet of code showing usage would be good, too. When i found your project, i was looking for how others handled project configuration that could be set in different ways and could possibly be multi- layered, like inheritance.

Excellent.  I'll get the README updates I had planned worked out and add the link to that documentation as part of it.
If you want other examples of how I handle project configuration, you should check out my latest python project: https://github.com/alunduil/zfs-replicate.  It uses a modern toolset and follows the latest guidelines on setup.py vs requirements.txt.  Using direnv has made project work a lot easier to handle for me, but your mileage may vary.
If you have any specific questions about project configuration, let me know I'd be more than happy to have a discussion about various merits and share opinions.

Alright, I've got some updates to the README in #25.  I'd love to get your take on it before merging.

I've merged #25 so I'll mark this issue closed.  Thanks for the report and the check on sanity.  I'll get the new version published in the next week (tomorrow if I'm lucky).


Need error state for required form fields

Recommendation: If an input error is automatically detected and suggestions for correction are known, then the suggestions are provided to the user.
We have three required fields:

name
email
how we can help? text input

Design consideration

If the submit button is pressed and one or all of the above fields is missing, where should be place the error message? And what should it read? eg next to submit: "[Name, email, how can we help?] input is required to submit".

If the error state text is a color (say red) when shown on the blue background contact form on the homepage, consider the appropriate color on https://designsystem.webstandards.ca.gov/contact-us/ where there is a gray background. We need to ensure appropriate contrast.

Contact us form on https://designsystem.webstandards.ca.gov/

Contact us form on https://designsystem.webstandards.ca.gov/contact-us/

WCAG 2.1 Guideline
3.3.1 Error Identification (Level A)

3.3.3 Error Suggestion (Level AA)

@adam-little @kmbrlygl do you have input on a good pattern for error state after pressing submit, when all required fields are not filled out?
My first inclination is a message next to the Submit button because you just pressed that. Then list all the missing fields.

We'll look into this!
It relates to: https://github.com/cagov/design-system/issues/558

Any follow up on this?

Yes, sorry for the delay.
Please see Figma for updated form field states for the two contact forms on the Design System website.
https://www.figma.com/file/A1DSo765cWjvBWoLd6tvEE/DS-website-screens?node-id=0%3A1
We might need a separate ticket to document copy for error states. What do you think @m-sullivan7?
As a side note, we have additional thinking for form fields but will hold off on documenting that until we decide to make form components.

I took a look and made one edit in Figma ("cannot" for "can not"). Those look good to me!
Regarding a separate issue for documentation, would this be for standards around error state text or specifically for this use case?

Regarding a separate issue for documentation, would this be for standards around error state text or specifically for this use case?

Just for this use case (for now), so the updates you made in Figma should be good enough.

UPDATE:
We are adding Form as a pattern using various inputs options. These inputs will be broken out to the component level.
Needs:

[ ]
Labels
[ ] Required fields
[ ] Disabled fields
[ ] Inputs (check boxes, radio, text fields, etc)
[ ] Error messaging
[ ] Use guides and standards
[ ] Technical documentation
[ ] Integration considerations for data collection

Available components and design items:

Button
Input explorations

cc @rkojik @extinah @sharon-rabang @zakiya


Just cloned solution fails

The solution seems to work well when connecting to our service, yet the actual "demo" environment (https://iat-api.adp.com and https://iat-accounts.adp.com) throws an error when PostAsync is used:
ADPApiConnection.cs
Line 311: response = client.PostAsync(url, content).Result;
I know it should not matter because our service works, yet a vendor is checking its implementation of OAuth for us, and they are using this site to test.

Hi,
Can you please elaborate on the issue and please let us know what exact error are you seeing.
Thanks,
Sahithi Reddy

Good morning,
Testing the demo apps failed even after just downloading the solution (no changes).
After reviewing the code and calls (Wireshark), we discovered that ADP is using TLS 1.2 to negotiate the connection, and the code (because the computers are configured that way) is still using TLS 1.0.
Although the recommendation is to use the operating system setting, we temporarily "hard-coded" the TLS 1.2 protocol before making the actual PostAsync call:
System.Net.ServicePointManager.SecurityProtocol = System.Net.SecurityProtocolType.Tls12 | System.Net.SecurityProtocolType.Tls11
Regards.


Change to use classes to add sensors

Changing to class use and variety of other changes to support heating and hotwater sensors. Feedback pls!

Ok, I believe the conflicts are now sorted...as you say the refresh interval is the key item outstanding.


[FLINK-22317][table] Support DROP column/constraint/watermark for ALTER TABLE statement

What is the purpose of the change
This pull request aims to support ALTER TABLE table_identifier DROP statement.
Brief change log
This PR contains two commits

4b00202d implements the syntax
93b32c4a supports operation conversion

Verifying this change
This change added tests and can be verified as follows:

SqlToOperationConverterTest
TableEnvironmentTest
table.q

Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): no
The public API, i.e., is any changed class annotated with @Public(Evolving): no
The serializers: no
The runtime per-record code paths (performance sensitive): no
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Kubernetes/Yarn, ZooKeeper: no
The S3 file system connector: no

Documentation

Does this pull request introduce a new feature? yes
If yes, how is the feature documented? FLINK-22320 will document this new feature

Thanks for the update! I add a commit to improve some codes. Please take a look at the last commit when you are free.

Thanks for the update! I add a commit to improve some codes. Please take a look at the last commit when you are free.

Thanks for the improvement, LGTM 👍


Add files via upload

submitted by shivam manihar sahu

@shivammaniharsahu we loved your work but please make sure you read all instructions mentioned in README.md. You have not selected a folder according to README.md


Separate parsing and evaluation for number and decimal types ?

Hmm... should the evaluation of decimal and number types be separate ?? issue #10 was closed as the commit added the evaluation for decimal types . However, after this , evaluation of number types was removed entirely. Should this be kept ? :thought_balloon:

I suppose, I could look for CLISP and other languages for examples. :neutral_face: :

I suppose, I could look for CLISP and other languages for examples. :neutral_face: :

well... I have checked with CLISP:
root# clisp
  i i i i i i i       ooooo    o        ooooooo   ooooo   ooooo
  I I I I I I I      8     8   8           8     8     o  8    8
  I  \ `+' /  I      8         8           8     8        8    8
   \  `-+-'  /       8         8           8      ooooo   8oooo
    `-__|__-'        8         8           8           8  8
        |            8     o   8           8     o     8  8
  ------+------       ooooo    8oooooo  ooo8ooo   ooooo   8

Welcome to GNU CLISP 2.49 (2010-07-07) <http://clisp.cons.org/>

Copyright (c) Bruno Haible, Michael Stoll 1992, 1993
Copyright (c) Bruno Haible, Marcus Daniels 1994-1997
Copyright (c) Bruno Haible, Pierpaolo Bernardi, Sam Steingold 1998
Copyright (c) Bruno Haible, Sam Steingold 1999-2000
Copyright (c) Sam Steingold, Bruno Haible 2001-2010

Type :h and hit Enter for context help.

[1]> ( + 3 2)
5
[2]> ( + 3.5 27 2)
32.5
[3]>

... I presume that this could be added :+1:


Update translation and styles for the welcome page

I'm not native English speaker but shouldn't there be the in here:

Pledge free space on your computer
Pledge the free space on your computer

I think its good to go for now, if we want, we can fix the the later in an upcoming PR. No reason to hold this one back


Add a -splat common parameter

PR Summary
Similar to #10073, this PR enhances the splat functionality that is available in PowerShell. Instead of doing this by extending operators, this PR adds a common -splat parameter of type [IDictionary[]] that supports the following new features:

inline splat of one or more hashtables
splat of one or more hashtables by value returned from a:

variable
property
command

The functionality in this PR is only available if you have the PSCommonSplatParameter experimental feature enabled.
PR Context
This PR helps because it extends splatting to support quite a few features that were requested by the community and described in RFC0002, which can currently be found here. Specifically, this allows users to use inline splatting of literal hashtables, variables, properties, the result of methods, or the result of commands.
I have opened this PR to offer an alternative syntax to the syntax offered in #10073 while also supporting a broader set of scenarios necessary for easier splatting in scripts.
Here are some examples demonstrating how this can be used:
# Splat a literal hashtable
Get-Date -splat @{Year=1971;Month=10;Day=20}

# Splat a property value
$getDate = [pscustomobject]@{
    Parameters = @{Year=1971;Month=10;Day=20}
}
Get-Date -splat $getDate.Parameters

# Splat the results of a command:
function Get-ParameterSet {
    @{Year=1971;Month=10;Day=20}
}
Get-Date -splat (Get-ParameterSet)

PR Checklist

[X] PR has a meaningful title

Use the present tense and imperative mood when describing your changes

[X] Summarized changes
[X] Make sure all .h, .cpp, .cs, .ps1 and .psm1 files have the correct copyright header
[X] This PR is ready to merge and is not Work in Progress.

If the PR is work in progress, please add the prefix WIP: or [ WIP ] to the beginning of the title (the WIP bot will keep its status check at Pending while the prefix is present) and remove the prefix when the PR is ready.

Breaking changes

[X] None
OR
[ ] Experimental feature(s) needed

[ ] Experimental feature name(s):

Experimental features added

[X] PSCommonSplatParameter - Enabling this feature will make the -splat common parameter available to end users.

User-facing changes

[ ] Not Applicable
OR
[X] Documentation needed

[ ] Issue filed:

Testing - New and feature

[ ] N/A or can only be tested interactively
OR
[X] Make sure you've added a new test if existing tests do not effectively test the code changed

Tooling

[X] I have considered the user experience from a tooling perspective and don't believe tooling will be impacted.
OR
[ ] I have considered the user experience from a tooling perspective and enumerated concerns in the summary. This may include:

Impact on PowerShell Editor Services which is used in the PowerShell extension for VSCode (which runs in a different PS Host).
Impact on Completions (both in the console and in editors) - one of PowerShell's most powerful features.
Impact on PSScriptAnalyzer (which provides linting & formatting in the editor extensions).
Impact on EditorSyntax (which provides syntax highlighting with in VSCode, GitHub, and many other editors).

I see a couple of issues with this approach.
First, I think this breaks a common scenario:
sleep -s 10

Basically any command that has exactly 1 parameter beginning with S would no longer be callable by specifying -s.
Second, a common parameter would also not work with non-cmdlet binding functions because common parameters are not added to those functions. I think it would be a mistake to introduce an alternative syntax for splatting that doesn't work everywhere, especially when the incorrect syntax might appear to work silently as is typically the case where $args collects the "extra" parameters.

@PoshChan please restart static

@KirkMunro, successfully started retry of PowerShell-CI-static-analysis

First, I think this breaks a common scenario:
sleep -s 10

Basically any command that has exactly 1 parameter beginning with S would no longer be callable by specifying -s.

There were never any guarantees with non-alias shorthand for parameter names that such syntax will continue to work over time. Only parameter aliases can guarantee consistent syntax, and commands where it is very common to use that syntax would be better off assigning those aliases so that they do continue to work over time. For the specific scenario mentioned here, we can add a 's' alias for the -seconds parameter in Start-Sleep if that is a concern.

Second, a common parameter would also not work with non-cmdlet binding functions because common parameters are not added to those functions. I think it would be a mistake to introduce an alternative syntax for splatting that doesn't work everywhere, especially when the incorrect syntax might appear to work silently as is typically the case where $args collects the "extra" parameters.

You're right that this wouldn't work for basic functions. Rules should be added to PSSA to pick up on such things and warn users. If there isn't a rule already to identify when a basic function is passed common parameters, there should be, regardless of what happens with this PR. The argument about $args applies for all common parameters (-ErrorAction, -WarningAction, etc.). Users who create functions and who want common parameter support need to learn that CmdletBinding is required.

@PoshChan please restart static

@KirkMunro, successfully started retry of PowerShell-CI-static-analysis

@PoshChan please restart windows

@KirkMunro, successfully started retry of PowerShell-CI-Windows

The only concern that I have is the name Splat:
PowerShell's parameters are usually not shortened but more importantly the word splatting/splat is a word that is not very intuitive/explanatory to someone that does not know about PowerShell splatting or is not a native English speaker.
I do not have a better name but it would be great to have a name that is more self-explanatory.
In Python they call it 'parameter unpacking'

We could opt for something like -Parameters with a short alias -Params :thinking:

We could opt for something like -Parameters with a short alias -Params 🤔

While I like the sound of it, I think -Parameters is too generic and would be a high risk for collision. For example, Get-Help has a -Parameter parameter. It would be confusing to have both -Parameter and -Parameters in any cmdlet. OTOH, the -Splat name is not likely to result in collision or confusion.
Since the concept of splatting already needs to be learned, I like the explicit nature of the -Splat name, myself. Plus, visibility of that as a word gives users something to search for so that they can learn what it does.

I should note, I did consider including @ as an alias for the -Splat parameter, but since PowerShell treats -@ as a positional value (it isn't recognized as a parameter name today), that would be a breaking change. I'm not opposed to making such a change, but I didn't want it in the first implementation of this PR.

How about -ParameterSplat? Or -SplattedParameter? Or -ParameterObject?

@PoshChan please restart Linux

@KirkMunro, successfully started retry of PowerShell-CI-Linux

What about -PSArgumentList?

List is a type, that might be confusing. PS seems a bit redundant and not common for parameter names

We keep having discussions in various threads when adding common parameters how not to clobber existing parameters. It could have been avoided if a reserved prefix was used, like with cmdlets. It's slightly off topic for this specific PR but wanted to bring it up for the overall common parameter discussion.

Closing.

no -splat, no -@, no @@ yet, will there be anything like that in the future? It seems like all approaches to improve splatting and thus usability are declined. Is there anything else in the work?


Non-HR Sprites are scaled incorrectly

Non-HR Sprites are scaled incorrectly

Should be resolved for 2.0 by the required prototype changes, and I do not intend to fix this backwards for 1.0


wrench 1.1.0 fails to parse TIMESTAMP_SUB

Expected Behavior
wrench can apply DML including TIMESTAMP_SUB function.
Current Behavior
It seems that wrench 1.1.0 fails to parse DML including TIMESTAMP_SUB.
wrench 1.0.4 can apply such DML correctly.
Steps to Reproduce
The following reproduction steps assumes bash or zsh.
# $PROJECT, $INSTANCE, $DATABASE should be set properly

# setup
$ wrench apply --project $PROJECT --instance $INSTANCE --database $DATABASE --ddl <(echo "CREATE TABLE ParsingTimestampSub (CreatedAt TIMESTAMP NOT NULL) PRIMARY KEY (CreatedAt);")

# OK case
$ wrench apply --project $PROJECT --instance $INSTANCE --database $DATABASE --dml <(echo "DELETE FROM ParsingTimestampSub WHERE CreatedAt < CURRENT_TIMESTAMP();")
0 rows affected.

# error case
$ wrench apply --project $PROJECT --instance $INSTANCE --database $DATABASE --dml <(echo "DELETE FROM ParsingTimestampSub WHERE CreatedAt < TIMESTAMP_SUB(CURRENT_TIMESTAMP(), INTERVAL 31 DAY);")
Error command: apply, version: unknown
        /proc/self/fd/11:1.94: got "31", want ")" or ","

# cleanup
$ wrench apply --project $PROJECT --instance $INSTANCE --database $DATABASE --ddl <(echo "DROP TABLE ParsingTimestampSub;")

Context (Environment)

wrench version: 1.1.0

Ah, it might be due to we migrated the query parser for wrench from regex to spansql...
Let me take a deeper look 👀

I'll also dig it.

I was having the same thing with TIMESTAMP_ADD. Looks like spanner 1.40.0 might fix this.

@dtest Thank you for letting me know that! Let me try to fix this 🙏

If you still have a problem, please reopen.


Correct units

Just realized that acs monitor prints out Hz instead of MHz
Closes #262 #135

Also, sorry I didn't see this until now

Thanks so much for fixing this issue

yee np

am j gonna merge this


Cannot get the error information when requesting with SQL HTTP

What type of bug is this?
Incorrect result
What subsystems are affected?
Frontend
What happened?
Just execute the sample code:
curl -i 'http://localhost:4000/v1/sql?sql=show%20tables&db=public' \
        -H 'authorization: Basic Z3JlcHRpbWVfdXNlcjpncmVwdGltZV9wd2Q='

There is no response when the authorization is wrong.
HTTP/1.1 401 Unauthorized
Connection: keep-alive
Date: Tue, 18 Apr 2023 08:39:00 GMT
Keep-Alive: timeout=4
Proxy-Connection: keep-alive
Content-Length: 0

The server prints the ERROR logs:
2023-04-18T08:25:24.525778Z ERROR servers::http::authorize: authenticate failed: Failed to get user info, source: User not found, username: greptime_user
2023-04-18T08:39:00.200662Z ERROR servers::http::authorize: authenticate failed: Invalid base64 value, source: InvalidLastSymbol(13, 52)
2023-04-18T08:39:29.468231Z ERROR servers::http::authorize: authenticate failed: Invalid base64 value, source: InvalidLastSymbol(13, 52)

The error in the server logs should be returned to the HTTP response.
What operating system did you use?
Mac OS X
Relevant log output and stack trace
No response
How can we reproduce the bug?
please refer to the former description.

No longer an issue in main<PHONE_NUMBER>84c468ca398967a6a0a0c18b2e2f55 -
HTTP/1.1 401 Unauthorized
content-type: application/json
x-greptime-err-code: 7004
x-greptime-format: greptimedb_v1
x-greptime-execution-time: 0
content-length: 95
date: Wed, 20 Mar 2024 04:19:49 GMT

{"code":7004,"error":"Invalid base64 value: Invalid byte 61, offset 33.","execution_time_ms":0}


implement add-destination requests

implemented add-destination api calls (both for groups and testers types, still depends on swagger clients update PR)
added flags --mandatory and --silent
added unit tests

We took most changes from this PR and integrated them into our branch. I'm going to close this PR for now.


Chapter 8: Adding roles to Blazor - Cut&paste error

Hi,

The point 1) says In the Components project, open Shared/NavMenu.razor. In this edition there is no Components project

It's not clear where to add the code. In the source code here, I see the code in MainLayout.razorand NavMenu.razor. Is it maybe duplicated?

Sorry about that, I will add this to the errata, thanks for reporting this!

That should be BlazorWebApp project, Open the Components/Layout/NavMenu.razor

2."At the top of the component" it doesn't really matter as long as it is in the file.
The MainLayout has another AutorizeView, not the same

I will keep these open until I add them to the Errata


Runtime error: import: delegate library support not built-in

I'd like to try this little great project. First, to build it I had to install Pillow (v8.1.2) and PIL Tools (v1.0) in addition to the other dependencies mentioned in the Readme. Perhaps, the dependency list is incomplete?
Now, when I run the tool (even without arguments), I get this error:
import: delegate library support not built-in '' (X11) @ error/import.c/ImportImageCommand/1297.
/usr/local/Cellar/ii-pix/20210315/libexec/convert.py: line 22: syntax error near unexpected token `('
/usr/local/Cellar/ii-pix/20210315/libexec/convert.py: line 22: `def main():'

This may be related to my build issue mentioned above. Any idea what is wrong? Using Python v3.9.2, if that matters.

Sorry, that was an issue with my environment. Closing.


Handle tables with many columns

Boost.URL has this table that's too large and ends up outside the area of the main content:
https://www.boost.org/doc/libs/master/doc/antora/url/reference.html

There should be a solution where we can scroll right or some other option.
Of course, we can also rethink this individual Boost.URL table format. We migrated it from previous QuickBook documentation. But I expect this problem to reoccur with other libraries.

Tables are generally a pain to deal with in this format. My preferred method would be to allow the table to scroll inside the div, and not allow it to go outside the div, but there are other who are strongly opposed to that.
The alternatives would be to find ways to shrink the table to force it to conform to the size or allow the container div to expand, even if it goes outside of pre-set boundaries.

I committed a fix, it's currently part of https://github.com/boostorg/boostlook/pull/43/commits

One of the downsides to the way the data is handled is that we don't have much control over what we get to work with. So we have to do more of a "one size fits all" solution regarding tables.
Vinnie is not a fan of extra scrollbars within the document, so while adding scrolling is fine, try to hide or minimise the scrollbars themselves.


How can I clean selected choice

Hello Friend.
In this link there is an example of your project.
https://snack.expo.io/@wyrustaaruz/simple-radio-button-cleaner-problem
I want to clean my choice with cleaner button.
Changer button is working well with 0...999  or more index number but not working -1 like initial value.
how can I clean my choice?

@wyrustaaruz
I solved it...i think this Helps you...
Replace Function in given location of file.
node_modules/react-native-simple-radio-button/lib/SimpleRadioButton.js
  updateIsActiveIndex(index,type) {     if(type === "change"){       this.setState({ is_active_index: index });       this.props.onPress(this.props.radio_props[index], index)     }else if(type === "clean"){       this.setState({ is_active_index: index });     }else {       this.setState({ is_active_index: index });       this.props.onPress(this.props.radio_props[index], index)     }   }
if you want change selection
this.refs.radioForm.updateIsActiveIndex('3','change');
if you want clean selection
this.refs.radioForm.updateIsActiveIndex('-1','clean');

@wyrustaaruz
I solved it...i think this Helps you...
Replace Function in given location of file.
node_modules/react-native-simple-radio-button/lib/SimpleRadioButton.js
 updateIsActiveIndex(index,type) { if(type === "change"){ this.setState({ is_active_index: index }); this.props.onPress(this.props.radio_props[index], index) }else if(type === "clean"){ this.setState({ is_active_index: index }); }else { this.setState({ is_active_index: index }); this.props.onPress(this.props.radio_props[index], index) } }
after that restart your npm start
if you want change selection
this.refs.radioForm.updateIsActiveIndex('3','change');
if you want clean selection
this.refs.radioForm.updateIsActiveIndex('-1','clean');

Its throwing an Error: cannot read property 'updateIsActiveIndex' of undefined.
Can you please help

I solved it !!!!!!!!!!!!!!!!!!!!!!
all you need to do is add a ref attribute in the RadioForm Tag and then call the updateIsActiveIndex(-1) anywhere in your code.
This example should explain it better.
render() {
return (
<RadioForm
ref = {ref => this.radioFormClear = ref} // !! This is the main prop that needs to be there
radio_props={this.state.radio_props}
initial={-1} // set this so that your inital radioForm is not selcted
animation={true}
onPress={(value) => {this.onSelect(value)}}
/>
)
}
Now anywhere in your code ...
myDesiredFunction() {
this.radioFormClear.updateIsActiveIndex(-1); // just pass -1 and your radio button should clear
}
PS: I got this solution partly from @wyrustaaruz, although I couldn't understand what ref was. Also you don't need to change the updateIsActiveIndex( ) function as @wyrustaaruz mentioned. Passing -1 to the original code works. This is my first time contributing to an Issue so hope this helps someone!


MongoDB - Subdocument Examples

We could use some sample code on how to affect Subdocuments for objects in a mongo database.
Where in MySQL you use foreign keys.. the expected way of handling dependant records is to store them in an array on the model and manipulated that way.
I am new to Node/Mongoose and Mongo for that matter, but I really want to use some of the features that it offers so I'm having to use it. If you could provide some sample code for this that you know works it would be really helpful!
 I dont anyone would migrate from mongo to mysql and expect a nice subdocument solution.... For me its absolutely fine to have some features for mongo and others for mysql ...


Adding Xinanjiang direct runoff option, changed volume trackers to structure in CFE

[Short description explaining the high-level reason for the pull request]
Adding the Xinanjiang runoff scheme written by Rachel and approved by Fred, addressing issue #10 . Some other issues have been taken care of as well. See the changes.
Additions

Xinanjiang direct runoff method available as an option.

Removals

Changes

Put Fred's csv parser back in, addressing issue #18
Volume trackers are now in a stucture within CFE, this was done by Fred, addressing issue #19
Update until is now a simple loop over update, addressing issue #17 , but this might not be the desired fix. I removed some logic from that function.

Testing

Jess's unit tests
make and run, when the BMI_CFE reads the forcing data, then runs the model, once with update, and again with update until
make and run pass, this tests the bmi ability to get forcings from the bmi functions get_value, set_value
make and run pass pet, this tests running two models and passing information between them with BMI
All tests done with the XinanJiang runoff option.

Screenshots
Notes

Todos

Scott should update the variable_info structure.

Checklist

[X ] PR has an informative and human-readable title
[ ] Changes are limited to a single goal (no scope creep)
[ X] Code can be automatically merged (no conflicts)
[ ] Code follows project standards (link if applicable)
[ X] Passes all existing automated tests
[ ] Any change in functionality is tested
[ ] New functions are documented (with a description, list of inputs, and expected output)
[ ] Placeholder code is flagged / future todos are captured in comments
[ ] Visually tested in supported browsers and devices (see checklist below :point_down:)
[ ] Project documentation has been updated (including the "Unreleased" section of the CHANGELOG)
[ X] Reviewers requested with the Reviewers tool :arrow_right:

Testing checklist
Target Environment support

[ ] Windows
[ ] Linux
[ ] Browser

Accessibility

[ ] Keyboard friendly
[ ] Screen reader friendly

Other

[ ] Is useable without CSS
[ ] Is useable without JS
[ ] Flexible from small to large screens
[ ] No linting errors or warnings
[ ] JavaScript tests are passing

@hellkite500 and @robertbartel
Check out this PR. I modified the update_until to be like the example in the BMI docs (bmi_cfe.c lines 978-1010):
static int Update_until (Bmi *self, double t)
{
// https://bmi.readthedocs.io/en/latest/#update-until
// "the time argument can be a non-integral multiple of time steps"
{
cfe_state_struct* cfe_ptr = ((cfe_state_struct *) self->data);
double dt;
double now;

Get_time_step (self, &dt);
Get_current_time (self, &now);

{
  int n;
  double frac;
  const double n_steps = (t - now) / dt;
  for (n=0; n<(int)n_steps; n++) {
    Update (self);
    if (cfe_ptr->verbosity > 1)
        print_cfe_flux_at_timestep(cfe_ptr);

  }

  frac = n_steps - (int)n_steps;
  if (frac > 0)
      printf("WARNING: CFE trying to update a fraction of a timestep\n Figure out what to do here!\n");
}

}
return BMI_SUCCESS;
}

@madMatchstick can you look at this PR and the previous review comments? Thanks!


[GMOD] Command to update addons from github

Many server admins use addons coming from github such as ACF, Advanced Duplicator 2, Wiremod...).
Those are quite boring to keep updated.
LGSM could provide a way to help updating them, either by using github API (i don't know how to use it), or by pasting their zip link to a new file called for example github_addons.
Then a command (or a cron job) like ./gmodserver update-git-addons would update them.
Two problems to face : those addons usually have caps characters while they have to be lower case to work with Linux, and ultimately, it will have to deal with older addons.
So name greps and convert to lowercase commands will be required.
Do you feel like such a function has its place into LGSM ?

why not just a workshop ? this would mean (perhaps) the server gets content that works rather than bleading edge code that could crash the server ?

Workshop is already supported by LGSM for gmod.
Github addons are updated more frequently because uploading and updating a workshop addon is very painful. Some devs chose not to use the workshop, others release only major updates, just because it's long and boring.

I understand that however code that is new should be ignored until it is deemed to be good so adding content via the workshop should be better than git ... I suppose it could be said from your comment there is no need for steam .... perhaps the end user needs stable rather than beta ... If steam makes this pain full perhaps another method needs to be employed

Haha ! Nothing is ever stable with gmod ! I'm saying that because i started playing before it was even available on steam.  The game engine doesn't even support multi thread anymore (even if settings seems to allow it) because it breaks the lua engine synchronization and makes the game unstable !
I see your point, and it's very valid... Except not for gmod. Many admins are still using github addons, as some of them have more functionalities or correct some more bugs. Gmod server admins AND players usually prefer functionalities over stability. Everyone is used to a server crashing because of a coded by feed duplicator tool... Or because a guy welded the head of a ragdoll to a high enough skymap.
I'm thinking about mainly 2 addons that are Adv Dupe 2, and ACF that are only found onto github. But it could apply to ulx/ulib, utime, and uclip, which would be very cool, as they are serious developers that update their addons only when it's really required.

I am supprised I don't know you then ... back in they day the guys from wolverhampton used to come down to mine for the odd house party

That's just because i'm french. Otherwise i'd have come.
No seriously, you're just worried about admins having buggy addons in the end ?

Hi,
I have a more generic idea. Addons do not always use GitHub but many of them (including the ones hosted at GitHub) support Subversion (SVN).
And I guess that it must be easy to do because command-line SVN clients already exist.

@EstevanTH Clever. It's true that SVN has been used forever in gmod, and slowly disappeared. It shouldn't stop us from using it if possible. It would eliminate quite a few problems of managing old files.
That would probably require a new packet for the server, that hopefully will be available for any distro, and some SVN knowledge to code it that i don't have already. Any good documentation to provide ?

My thinking here is that an external tool might be a better idea, especially given how much extra functionality this will need to work robustly. Otherwise, we need to consider a few big issues with this:

All games will now not only need game data and scripts, but also a registry of mods and addons that we support. This will quickly become prohibitive and out of date.
Not everyone will have the same addons, or want to get the latest versions of all pieces. Granularity for the server admins needs to be a big concern here, so we need to fully understand how the update and use of these repos works across the games we will support.
Inventory and tracking of the repos (if we don't keep a centralized list) would need to be mapped out. So, will the script use a simple manifest file, or will it just $(find -type d -name .git) the directory and use all those repos?
This will require a lot of new dependencies (Git primarily) that will need to be taken care of in the setup phase. A lot of projects are still in SVN, Mercurial, or (shudder) CVS as well. Will we need to support them all?
How will we handle authentication, i.e. GitHub SSH requires a key. This is a big problem I ignored in my Insurgency server sync script that does something like this, it caused me quite a headache.

My suggestion here would be to clearly state what you want, so we can define a specification for the script. That way, we know what we want to do, and then can build the tool to the desired actions, not just go in blind. Something like this:
I want to be able to create a list of Git repos with the local paths they should be installed in. I want to be able to add repos to the file and have the tool clone them. I want to be able to update and pull all repos, or just individual ones. I want to be able to push changes back upstream as well, or branch my own versions.
Once we have a clear idea exactly what you're looking to do, we can look at creating a tool. I have a simple script for my Insurgency servers that clones out a bunch of repos, rsyncs some maps, and does some other cleanup afterwards, all driven by a config manifest. I'm sure that a lot of what you're looking to do could be done with this script, I'd be happy to take a crack at helping you add the features you need to it and sharing it.

@jaredballou
Well, thanks for your attention.
I guess you have some misconceptions about my idea.

We don't affect addons in any way
It's for gmod only
User chose addons that they wanna use, we have no control over that, no need to support specific addons.

What do you mean ? We can make it for gmod only. And as it didn't change for years that you put standalone addons into the addons folder , there's no obvious reason for it to change soon. Even if it had to change and that we had to remove that function, the gain of time that such a function would provide me overwhelms the time it would cost to develop it. I guess it would be problematic supporting addons for addons... I mean like addons for sourcemod, that would be extremely hard to maintain.
Ofc not everyone would have the same addons. But at least if we support addons from any github, that would require the user an initial configuration, that'd be a good start. We can make 2 commands : git-download-addons and git-update-addons. That way if you only wanna download addons that you put in the list once, you download, and if you're not worried about updates, you  use git-update-addons as a cronjob.
No need to keep it centralized, users chose what addons they want.
Not sure it needs dependencies, as LGSM is already able to download from github without any so... Are you sure ?
You don't need authentication to download, especially if using the .zip. I'm usually downloading them directly to my server to add or update them.

Is that clear enough ?

So I'm not clear on how the current manual process works. Any chance you can link me to a document that shows me how it's done? I bet if I saw what you're trying to do I'd be able to pick it up. And I am usually reluctant to put new stuff in if it's just for one game, but if it's common with other games we can certainly give it a shot.

I don't have a doc, but i can give you an example of how i'm doing it manually. It's actually the same process for a server than for the user.
I made a video for the user, just mute the sound if you don't speak french :p
https://www.youtube.com/watch?v=F6pDOAbm6-Q
Get your zip containing the addon.
Ensure that you get a folder containing at least a "lua" folder.
Copy that whole folder to your "addons" folder.
Done.
So if for example i wanna add that https://github.com/nrlulz/ACF addon to my server, i'll get the zip link (right click on "Download ZIP" then copy url) https://github.com/nrlulz/ACF/archive/master.zip
Then on the server :
cd serverfiles/garrysmod/addons
wget https://github.com/nrlulz/ACF/archive/master.zip
tar -zxvf ACF-master.zip
rm ACF-master.zip
mv ACF-master acf-master //gmod for linux dosn't support uppercase
Done !
Repeat that for all addons.
The idea would be that users could do that, by simply pasting github links into text document.
For example, at $serverdir, there would be a githubaddons.txt containing
Paste here addons from github you wish to install or update line by line
Then use the command ./gmodserver git-update-addons
Then user pastes links such as :
https://github.com/nrlulz/ACF
https://github.com/wiremod/advdupe2
https://github.com/wiremod/wire
[...]

Sorry i'm not uzing tar -zxvf on .zip files, but ofc unzip command !
BTW i'll start working on a standalone script first here : https://github.com/UltimateByte/zip-updater

My standalone function is pretty much working !
For now it's enough, any suggestion is very very welcome.
I'm gonna wait until we can make modules for lgsm (which seems likely to happen) to think about implementing it as an optional function :)

I still think workshop covers this ... if it's not in workshop the code is not at release standard ... you guys may want 'bleeding edge' but the users of this script may have no clue on how to do this  .... are you trying to over complicate ?

Workshop is not a "standard" in any way. Espacially for server based addons. Do you know about scriptfodder ? Sadly this script doesn't apply to it but still, it's light yearw away from the Workshop.
You most likely know ULX, the most famous admin plugin for gmod. Is this and its dependency Ulib on gmod workshop ? No. Such as utime, Uclip, ACF and extentions, advdupe2 and other wiremod extentions... I use all of that on several servers, and it's a damn pain to keep updated.
So i don't think i overcomplicate, it's event the opposite, i give a way to make boring tasks automatic with a bit of configuration instead of a lot of scripting. A guy i host a server for just told me "Finally ! It finally exists ! I won't be so bored doing this anymore", and that's exactly for that same thought thag i made this script ! If it's useful to us, it most likely is for others.

Let me take a crack at this tonight, I have some free time and might be able to draft up at least some pseudocode to make sure I have it.

Damn, wish I'd seen you already built this. Mine is fancy in a different way, mine takes an array of GitHub user,repo,branch,method and allows native Git cloning, Tarball downloads, or Zip downloads. For archives, it keeps tabs on what update it's on by querying the GitHub downloader for headers and divining the filename. The archive is then unpacked into the directory named the same thing as the archive, and the short name of the mod is symlinked. This obviously isn't production ready as it has multiple copies of the same mod, but it was more about me getting the logic working. For native Git, it uses the GitHub API and finds the latest commit, then makes sure the repo exists locally and is configured/updated properly, Let me know what you think, feel free to steal it and do whatever you want with it, I really don't have time to maintain it. I just wanted to poke at it a bit.
https://github.com/jaredballou/linuxgsm/blob/master/GarrysMod/addons.sh

That rocks mate !
I had no idea how to use GitHub API, so i did it with rough zipfiles.
Both can be complementary BTW, that's something to develop ! ^^

Will be able to do it through this :
https://github.com/GameServerManagers/LinuxGSM/issues/631
./gameserver install-mods
CU | Custom Mods List

First time launching :
"No custom mod list, generating config file"
"Config file created in /home/user/lgsm/custom_mods.cfg"

cat custom_mods.cfg
# Add mods using this syntax :
(Remains to define)

Obsoleted by https://github.com/GameServerManagers/LinuxGSM/issues/631


Message tracing

Codecov Report

Merging #879 into master will increase coverage by 0.02%.
The diff coverage is 76.19%.

@@            Coverage Diff             @@
##           master     #879      +/-   ##
==========================================
+ Coverage   61.75%   61.77%   +0.02%
==========================================
  Files          74       74
  Lines       12880    12899      +19
  Branches     2045     2045
==========================================
+ Hits         7954     7969      +15
- Misses       4268     4272       +4
  Partials      658      658

Impacted Files
Coverage Δ

autobahn/_version.py
100% <100%> (ø)
:arrow_up:

autobahn/wamp/message.py
63.87% <75%> (+0.14%)
:arrow_up:

autobahn/wamp/serializer.py
67.55% <75%> (+0.13%)
: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 9ad7878...cd25359. Read the comment docs.


Shared network drive - App does not run

Hi again,
I have deployed the app to a shared network drive where it will not run. I have checked and can confirm that the following end handler is contained within the app:
shinyServer(function(input, output, session){
...
session$onSessionEnded(function() {
stopApp()
})
...
})
In addition, I changed the error log file location in the config file.
Initially I had a similar issue to https://github.com/wleepang/DesktopDeployR/issues/19, with the error message saying that "run.wsf" could not be found, but now nothing is coming up at all.
Thanks

The latest update may have fixed this issue. It creates a *.bat file that uses absolute paths to launch the app.


Cmd+P search files should also search for opened files regardless of gitignore

Check for existing issues

[X] Completed

Describe the feature
After opening some files from node_modules, in a node.js repo, list of Cmd+P search files don't include the opened files in node_modules because it's git-ignored.
The opened files should be listed when searching for files, even if they're git-ignored.
If applicable, add mockups / screenshots to help present your vision of the feature
No response

In case for some reason the team doesn't want to add "node_modules" to the command pallette, I would still like to be able to access other .gitignored files. For example, my .env files :)

Related:

https://github.com/zed-industries/community/issues/1538


Webpack Dev Server picking up file changes but changes not persisted/page not refreshed

SO question: http://stackoverflow.com/questions/43569106/webpack-dev-server-doesnt-persist-changes-on-vagrant-vm

Do you want to request a feature or report a bug?
bug

What is the current behavior?
The webpack-dev-server job/process is detecting file changes but files are not written to disk.
If the current behavior is a bug, please provide the steps to reproduce.
use a website/spa in a Vagrant VM (for example https://github.com/geerlingguy/drupal-vm )

Here's my configuration:
https://gist.github.com/gkatsanos/321d97d12812302bdd173225231b18e0

What is the expected behavior?
As per the webpack documentation:

Make a change in one of your files and hit save. You should see that the console is recompiling. After that's done, the page should be refreshed.

That's not happening. (console recompiles but page isnt refreshed)
"webpack": "^2.4.1"
OSX 10.12.4 (16E195)
VirtualBox 5.1.14 r112924 (Qt5.6.2)
"webpack-dev-server": "^2.4.1"
node v6.3.1

My publicPath was wrong. For future reference: once you launch your dev-server, check from what path the bundle.js is server from. Use this relative path as the publicPath.
For example :
  output: {
    filename: "[name].js",
    path: path.resolve(__dirname, "./dist")
  },

generated bundle location: localhost:9000/themes/dist/js/bundle.js
publicPath should be /themes/dist/


Uploads to Minio do not work due to content-length

Describe the bug
I know this is documented in the wiki
Trying to use Minio breaks because the content-length header is not present. This seems like it could be addressed by changing  from using putObject to upload from @aws-sdk/lib-storage as commented here:
https://github.com/aws/aws-sdk-js-v3/issues/2348#issuecomment-841002347
and in the upgrade guide from v2 to v3: https://github.com/aws/aws-sdk-js-v3/blob/main/UPGRADING.md#s3-multipart-upload
To Reproduce
Steps to reproduce the behavior:
Point common-object-management-service to a Minio instance and upload a document.
Expected behavior
I should be able to use Minio for local development.
Screenshots

Desktop (please complete the following information):

OS: [e.g. iOS]
Browser [e.g. chrome, safari]
Version [e.g. 22]

Smartphone (please complete the following information):

Device: [e.g. iPhone6]
OS: [e.g. iOS8.1]
Browser [e.g. stock browser, safari]
Version [e.g. 22]

Additional context
I will see if I can find time this weekend to try my assertion that upload will work with minio.

Integrated with #96


StringInterpolation Rule

ensure string interpolation isn't used on certain attributes:
configurable

example:

 is not allowed

 is allowed

I think you shouldn't be using string interpolation within any kind of binding either; but will need to check.

Should be able to achieve this with a config parameter to AttributeValueRule


Security V1.3 User Database

User Database is now implemented, modelled after UserPref. List of accounts will be stored in root folder called users.json.
In main it is called AppUsers, which contains an arraylist (at the moment) of AccountCredential class, which contains attribute of username and hashedpassword, and method of hashing password (not implemented yet should be easy, just find a good hash library).
TODO:

[x] When new users register, add it to AppUsers.
[x] Exceptions calls in login, and register functions

Adding unit test 0162fe9 decreases coverage??

Pretty sure the coverage percentage is with reference to master.
Maybe it's because your unit test in 0162fe9 covered the same lines of code as 51818f3? So both were at -0.3%

Both of the commits are unit test for different Classes eh, wont be covering the same line of code

Ready to approve. Points raised will be addressed in V1.4 at #24

@bennchong your build is failing because of EOF

@bennchong your build is failing because of EOF

This is real sad.


Example/example re assembling fields

This new example shows how to do some matrix operations, including basic mathematical operation (power, add and multiply by
a constant, add field containers and invert ) and separating and assembling fields and fields containers

@luisaFelixSalles Thanks for the example. I think it would be good to have a follow-up on this one to explain users how to create their own matrix fields "by hand". Dpf Matrix Fields are created "by column" and not "by row", and we are not documenting it anywhere. This also affects the way the generalized_dot_product operator is working.
@PProfizi This caused a bug in the past, so I think it is something we would like to properly document. In addition, it works differently in numpy, so it is not very straigthforward to use without an example

@luisaFelixSalles Thanks for the example. I think it would be good to have a follow-up on this one to explain users how to create their own matrix fields "by hand". Dpf Matrix Fields are created "by column" and not "by row", and we are not documenting it anywhere. This also affects the way the generalized_dot_product operator is working.
@PProfizi This caused a bug in the past, so I think it is something we would like to properly document. In addition, it works differently in numpy, so it is not very straigthforward to use without an example

@luisaFelixSalles I think this remark could go both in a specific tutorial on Field creation, and in the User Guide documentation.


[KOGIGO-9149] Support arrays as knative response

There are also problems when sending json array as body in a post

(tests) - kogito-quarkus-examples job #911 was: UNSTABLE
Possible explanation: This should be test failures

Reproducer
export BUILD_MVN_OPTS_CURRENT=
build-chain build cross_pr -f 'https://raw.githubusercontent.com/kiegroup/kogito-pipelines/main/.ci/pull-request-config.yaml' -o 'bc' -p kiegroup/kogito-examples -u https://github.com/kiegroup/kogito-runtimes/pull/2985 --skipParallelCheckout
NOTE: To install the build-chain tool, please refer to https://github.com/kiegroup/github-action-build-chain#local-execution

Please look here: https://eng-jenkins-csb-business-automation.apps.ocp-c1.prod.psi.redhat.com/job/KIE/job/kogito/job/main/job/pullrequest/job/kogito-runtimes.tests.downstream.kogito-quarkus-examples/911/display/redirect
Test results:

PASSED: 166
FAILED: 1

Those are the test failures:

org.kie.kogito.quarkus.outbox.OutboxIT.(?)
java.lang.RuntimeException: org.testcontainers.containers.ContainerLaunchException: Timed out waiting for container port to open (localhost ports: [32800, 8080] should be listening)Caused by: org.testcontainers.containers.ContainerLaunchException: Timed out waiting for container port to open (localhost ports: [32800, 8080] should be listening)

jenkins run kogito-quarkus-examples test

jenkins run kogito-quarkus-examples tests


export redactions::Selector and Redaction

When writing a dynamic redaction, it'd be nice to be able to leverage the existing selector and redaction logic to redact parts of a structure.

Redaction is already exported. I was hoping that Selector can stay internal and there is no use for it. Do you have an example of how you would use it?

I don't remember, so I'll close this. Sorry for the noise.


Event user clicked ads

how to detect user click ads
Thanks

Hi.
In case of the banner, you can use the onClick listerner.
Example:
<AdMobBanner
   adSize="banner"
   adUnitID="ca-app-pub-3940256099942544/6300978111"
   onClick={() => {
        // Code to be execute when the user clicks on the Ad
  }}
/>

If you are using the Interstitial Ad, you can do something like this:
    AdMobInterstitial.setClickedListener(
       () => {
          // Code to be execute when the user clicks on the Ad
       }
    );

But, if you are using the Rewarded Video: There is no event listener to the click event. You can use setRewardedListener() function to know when the user watched the video until the end.

Thanks
Vào 21:21, T.2, 16 Th9, 2019 Matheus Alves<EMAIL_ADDRESS>đã
viết:

Closed #6
https://github.com/MatheusAlvesA/react-native-androide/issues/6.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/MatheusAlvesA/react-native-androide/issues/6?email_source=notifications&email_token=AIDIYZZSF4NHN7NFCY2OTI3QJ6I7BA5CNFSM4IUSEJIKYY3PNVWWK3TUL52HS4DFWZEXG43VMVCXMZLOORHG65DJMZUWGYLUNFXW5KTDN5WW2ZLOORPWSZGOTUTSUYY#event-2636589667,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AIDIYZ37JQYJ5YAEDGZGIXDQJ6I7BANCNFSM4IUSEJIA
.


feat(CX-2894): Change Highlight Icon Color

Description

This PR changes the color of the highlight icon

This PR resolves CX-2894
QA Testing

Acceptance Criteria

Setup Instructions

PR Checklist

[x] I tested my changes on iOS / Android.
[x] I added screenshots or videos to illustrate my changes.
[ ] I added Tests and Stories for my changes.
[ ] I added an app state migration.
[ ] I hid my changes behind a feature flag.
[ ] I have prefixed changes that need to be tested during a release QA with [NEEDS EXTERNAL QA] on the changelog.

To the reviewers 👀

[ ] I would like at least one of the reviewers to run this PR on the simulator or device.

Changelog updates
Changelog updates

Cross-platform user-facing changes

Change Highlight Icon Color - kizito

iOS user-facing changes

Android user-facing changes

Dev changes

Need help with something? Have a look at our docs, or get in touch with us.

LGTM! Great work Kizito! 🚀 🚀 I just have one concern about the color decision since (at least to me) it doesn't feel that noticeable due to the contrast of the color (Yellow100) and the background :) I will add a comment on the ticket about it, but this PR is good :)

Thanks @MrSltun . Dasha chose the color


Remove debug printing from the wasm distribution

The following piece of code makes using hacl-wasm way to verbose
// One MAY only call this function after all modules have been loaded and
// suitable calls to propagate have been performed.
function reserve(mem, size) {
  let m32 = new Uint32Array(mem.buffer);
  let p = m32[0];
  m32[0] += size;
  my_print("Reserved memory area 0x"+p32(p)+"..0x"+p32(m32[0]));
  return p;
}

See
Reserved memory area 0x0000bb55..0x0000bb75
Reserved memory area 0x0000bb15..0x0000bb55
Reserved memory area 0x0000bb55..0x0000bb76
Reserved memory area 0x0000bb15..0x0000bb55
Reserved memory area 0x0000bb55..0x0000bb76
Reserved memory area 0x0000bb15..0x0000bb35
Reserved memory area 0x0000bb15..0x0000bb55
Reserved memory area 0x0000bb55..0x0000bb75
Reserved memory area 0x0000bb15..0x0000bb55
Reserved memory area 0x0000bb55..0x0000bb75
Reserved memory area 0x0000bb15..0x0000bb55
Reserved memory area 0x0000bb55..0x0000bb76
Reserved memory area 0x0000bb15..0x0000bb55
Reserved memory area 0x0000bb55..0x0000bb76
Reserved memory area 0x0000bb15..0x0000bb35
Reserved memory area 0x0000bb15..0x0000bb55
Reserved memory area 0x0000bb55..0x0000bb75
Reserved memory area 0x0000bb15..0x0000bb55
Reserved memory area 0x0000bb55..0x0000bb75
Reserved memory area 0x0000bb15..0x0000bb55
Reserved memory area 0x0000bb55..0x0000bb76
Reserved memory area 0x0000bb15..0x0000bb55
Reserved memory area 0x0000bb55..0x0000bb76
Reserved memory area 0x0000bb15..0x0000bb35
Reserved memory area 0x0000bb15..0x0000bb55
Reserved memory area 0x0000bb55..0x0000bb75
Reserved memory area 0x0000bb15..0x0000bb55
Reserved memory area 0x0000bb55..0x0000bb75
Reserved memory area 0x0000bb15..0x0000bb55
Reserved memory area 0x0000bb55..0x0000bb76
Reserved memory area 0x0000bb15..0x0000bb55
Reserved memory area 0x0000bb55..0x0000bb76
Reserved memory area 0x0000bb15..0x0000bb35
Reserved memory area 0x0000bb15..0x0000bb55
Reserved memory area 0x0000bb55..0x0000bb75
Reserved memory area 0x0000bb15..0x0000bb55
Reserved memory area 0x0000bb55..0x0000bb75
Reserved memory area 0x0000bb15..0x0000bb55
Reserved memory area 0x0000bb55..0x0000bb76
Reserved memory area 0x0000bb15..0x0000bb55
Reserved memory area 0x0000bb55..0x0000bb76
Reserved memory area 0x0000bb15..0x0000bb35
Reserved memory area 0x0000bb15..0x0000bb55
Reserved memory area 0x0000bb55..0x0000bb75
Reserved memory area 0x0000bb15..0x0000bb55
Reserved memory area 0x0000bb55..0x0000bb75
Reserved memory area 0x0000bb15..0x0000bb55
Reserved memory area 0x0000bb55..0x0000bb76
Reserved memory area 0x0000bb15..0x0000bb55
Reserved memory area 0x0000bb55..0x0000bb76
Reserved memory area 0x0000bb15..0x0000bb35
Reserved memory area 0x0000bb15..0x0000bb55
Reserved memory area 0x0000bb55..0x0000bb75
Reserved memory area 0x0000bb15..0x0000bb55
Reserved memory area 0x0000bb55..0x0000bb75
Reserved memory area 0x0000bb15..0x0000bb55
Reserved memory area 0x0000bb55..0x0000bb76
Reserved memory area 0x0000bb15..0x0000bb55
Reserved memory area 0x0000bb55..0x0000bb76
...........

Here is a workaround I use
  /* We have to cheat to avoid the noise from hacl-wasm */
  var old_log = console.log;
  console.log = function () {};
  var _HACL_loader = require('hacl-wasm');
  console.log = old_log;

Thanks Hugo, you are correct and I ended up commenting out this exact line locally a few days ago. I've pushed a fix on the kremlin side, thanks for the bug report.


Don't create settings.json when it was removed

⚠ crazy 4am idea ⚠
👉 Open for discussion
Stop recreating settings.json if it's removed.
This improves UX, because users don't accidentally loose their settings anymore
and removes a source of crashes in the SUI. Our WinUI NavigationView crashes
if its MenuItems array is modified while a transition is ongoing.
The crash in particular occurs reliably if the settings are deleted.

PR Checklist

[x] I work here
[ ] Tests added/passed

Validation Steps Performed

Removing settings.json doesn't cause it to be created ✔

I don't think this is the right answer. What if it was removed purposely for the intention of getting a clean new settings.json? A PR was specifically added and merged to ensure that even while WT is running, a new settings.json could be generated.  This PR doesn't support the end user.  There must be other ways to address crashes than removing an option that users like myself rely on.

What if it was removed purposely for the intention of getting a clean new settings.json?

I imagined that you could just restart the app.

We can reopen/revaluate later (:


Fix comment in securityGroup.ts

Probably a copy-paste error

@stack72 I'm guessing that this comment is generated from a schema somewhere that needs to be fixed?

Fixed by #2215. Same root cause as #2133.


Finer-grained privacy control

Rust has a reasonable default meaning of pub and private items, but for larger projects it would be extremely useful to have finer-grained control -- especially "public to crate, private to world".

Triage: Fixed by #1422

Yup!


tarball job: more --show-trace

We ran into a really useless log here:
https://hydra.nixos.org/log/487b2k81x1nkhgi13b8z1xdw4ibj97iv-nixos-channel-23.05pre450719.c11c55dd08a.drv
Now we'd get a full trace instead, pointing to the bad package.

In this case the log would end by:
generating packages.json
error:
       … while querying the derivation named 'python3.10-zipfile36-0.1.3'

       … while evaluating call site

       at /build/source/pkgs/stdenv/generic/make-derivation.nix:451:14:

          450|
          451|   validity = checkMeta { inherit meta attrs; };
             |              ^
          452|

       … while calling 'assertValidity'

       at /build/source/pkgs/stdenv/generic/check-meta.nix:381:20:

          380|
          381|   assertValidity = { meta, attrs }: let
             |                    ^
          382|       validity = checkValidity attrs;

       … while evaluating call site

       at /build/source/pkgs/stdenv/generic/check-meta.nix:382:18:

          381|   assertValidity = { meta, attrs }: let
          382|       validity = checkValidity attrs;
             |                  ^
          383|     in validity // {

       … while calling 'checkValidity'

       at /build/source/pkgs/stdenv/generic/check-meta.nix:342:19:

          341|   # Along with a boolean flag for each reason
          342|   checkValidity = attrs:
             |                   ^
          343|     # Check meta attribute types first, to make sure it is always called even when there are other issues

       … while evaluating call site

       at /build/source/pkgs/stdenv/generic/check-meta.nix:376:13:

          375|     # Please also update the type in /pkgs/top-level/config.nix alongside this.
          376|     else if hasNoMaintainers attrs then
             |             ^
          377|       { valid = "warn"; reason = "maintainerless"; errormsg = "has no maintainers"; }

       … while calling 'hasNoMaintainers'

       at /build/source/pkgs/stdenv/generic/check-meta.nix:55:22:

           54|
           55|   hasNoMaintainers = attrs:
             |                      ^
           56|     attrs ? meta.maintainers && (lib.length attrs.meta.maintainers) == 0;

       error: value is a set while a list was expected

       at /build/source/pkgs/stdenv/generic/check-meta.nix:56:34:

           55|   hasNoMaintainers = attrs:
           56|     attrs ? meta.maintainers && (lib.length attrs.meta.maintainers) == 0;
             |                                  ^
           57|
build times:
user time for the shell             0m0.000s
system time for the shell           0m0.000s
user time for all child processes   0m0.000s
system time for all child processes 0m0.000s

Oh, OK, just by a couple minutes.


Being able to cross reference multiple identifiers

To handle multiple ways of identifying the same feature we could use identifier schemes. Are there are some well known schemes we could utilise?

My recommendation:

Define an identifier type for instance a FeatureIdentifier which is a composite key containing:

Component
Type
Notes

scheme
string
A unique identifier for the scheme that issues the IDs, typically in reverse domain format (e.g. nz.govt.linz)

id
string
The identifier within the scheme.

This would be complimented by a register of well-known schemes

Each feature object to have an array of FeatureIdentifiers, allowing its various IDs to be cross-referenced. Organisations don't have to know how to handle all identifiers, just to preserve them if interchanged.


Initial xlf files

We're changing the way we obtain localized builds of MSBuild. We will be storing localized resources in xlf files and keep them in sync with the resx files. This means that the community can contribute to translation.
I'm sending the xlf files ahead of the sync build code so the localization team can begin translating in advance

LGTM, assuming it's ok that everything's currently marked as "new"--I take it the loc team already has a plan to drag in the existing translations from their internal home?

@dotnet-bot test Windows_NT Build please

@dotnet-bot test Windows_NT Build please


whitespaceBeforeOpenBrace option makes weird formatting for array methods

System Details

System Details Output
### VSCode version: 1.56.1 e713fe9b05fc24facbec8f34fb1017133858842b x64

### VSCode extensions:
<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>
### PSES version: <IP_ADDRESS>

### PowerShell version:

Name                           Value
----                           -----
PSVersion                      7.1.3
PSEdition                      Core
GitCommitId                    7.1.3
OS                             Microsoft Windows 10.0.19042
Platform                       Win32NT
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0…}
PSRemotingProtocolVersion      2.3
SerializationVersion           <IP_ADDRESS>
WSManStackVersion              3.0

Issue Description
I am experiencing a problem with PowerShell script formatting where array methods are present. By default, rule powershell.codeFormatting.whitespaceBeforeOpenBrace is enabled and its producing weird formatting for such scenario. Kindly ask to consider either disabling that inside the methods parentheses or have additional rule to make the formatting look even.
Expected Behaviour
Whitespaces should not be changed between different type of braces or should be even.
$a = @(0..3)
$a.ForEach({ $_ * $_ })

or
$a = @(0..3)
$a.ForEach( { $_ * $_ } )

Actual Behaviour
$a = @(0..3)
$a.ForEach( { $_ * $_ })

Thanks @t3mi this looks related to https://github.com/PowerShell/PSScriptAnalyzer/issues/1113


Remove version: FreeCiv.FreeCiv version 2.6.3

[Automated] It returns code over 400 in all urls
Microsoft Reviewers: Open in CodeFlow

URL: https://sourceforge.net/projects/freeciv/files/Freeciv 2.6/2.6.3/Freeciv-2.6.3-win32-gtk3-setup.exe/download
Status Code: 200
(Automated message - build 896)

Close with reason: Package still available.;


[BUG] Static pages

Describe the bug
Static pages for Home-page, "Support CATAPA" (Previously called "members")
To Reproduce
Steps to reproduce the behavior:

Go to 'Portal Config' and General

Expected behavior
Find these two pages also editable.

"The registration page" also needs an editable section so we can adapt it when needed.

This is a Feature too


CSV functionality is not working Properly

the csv that my app downloads, is just an empty csv.. any idea what may happen..?

What django version are you using? And which version of the questionaire?

django==1.8.5 and the latest version of the questionaire from the github

okay with 1.9 it has another issue, so I replaced the methode export_cvs in views.py with that:
`@permission_required("questionnaire.export")
def export_csv(request, qid):  # questionnaire_id
"""
For a given questionnaire id, generaete a CSV containing all the
answers for all subjects.
"""
import csv
# Create the HttpResponse object with the appropriate CSV header.
response = HttpResponse(content_type='text/csv')

writer = csv.writer(response, dialect=csv.excel)

questionnaire = get_object_or_404(Questionnaire, pk=int(qid))
headings, answers = answer_export(questionnaire)

writer.writerow([u'subject', u'runid'] + headings)
for subject, runid, answer_row in answers:
    row = ["%s/%s" % (subject.id, subject.state), runid] + [
        a if a else '--' for a in answer_row]
    writer.writerow(row)

response['Content-Disposition'] = 'attachment; filename="export-%s.csv"' % qid

return response`

HTH

nice nice now it works! thank you very much!


Print source location for NoInfo errors

@sydow I was under the impression that with the improved error message thingy we would always be able to get some source location that would be sort of relevant. Isn't that the case?
I notice it is not currently printed. Can we print it? If there are multiple locations then just print them all?
Here's an example:
actor main(env):
    def test():
        foo: ?list[int] = None
        foo = [1, 2, 3]
        print(foo)
    test()
    env.exit(0)

resulting in this error NoInfo 86 without any source code location
kll@Boxy:~/terastream/acton/foo$ actonc src/foo.act
Building project in /home/kll/terastream/acton/foo
  Compiling foo.act for release

ERROR: Error when compiling foo module: Type error
NoInfo 86
kll@Boxy:~/terastream/acton/foo$

This is what we used to get:
kll@Boxy:~/terastream/acton/foo$ /usr/bin/actonc src/foo.act
Building project in /home/kll/terastream/acton/foo
  Compiling foo.act for release

ERROR: Error when compiling foo module: Type error
 5:13-18
  |
5 |        foo.append(4)
  |            ^^^^^^
Cannot infer W_18 : ?__builtin__.list[__builtin__.int].append < T_19

Well, what needs to be done for each NoInfo is to replace it with another form of ErrInfo, namely Origin SrcLoc String…
But, of course NoInfo should also take a SrcLoc as an argument; I can’t explain why I didn’t think of that :-(. I will do add that ASAP, but probably not today.
Unfortunately, I don’t think a location is immediately available for all NoInfo occurrences. We need to have some piece of source code to get the SrcLoc from and that is not always at hand. We will, I think, rewrite code to carry such source code pieces around, but that is a bigger job. So what I will do first is to add a SrcLoc to NoInfo but sometimes the location will be NoLoc…
After trying more examples I think that we should invoke some AI :-) when we get a type error, in particular when we get “Cannot solve" errors. Examples of what I think of is

if an actor has a parameter called env, guess that it is of type Env.
in an assignment such as
lst = [],
guess that lst has type [_] (and not a subtype).

Björn

On 13 Oct 2023, at 10:00, Kristian Larsson @.***> wrote:
@sydow I was under the impression that with the improved error message thingy we would always be able to get some source location that would be sort of relevant. Isn't that the case?
I notice it is not currently printed. Can we print it? If there are multiple locations then just print them all?
...
Already up to date, in 0.000 s
Compiling acton_streamer/nsoapi/schema.act for release
Already up to date, in 0.003 s
Compiling acton_streamer/tsdb/writer.act for release
Finished compilation in 0.162 s
Compiling acton_streamer/main/main.act for release
ERROR: Error when compiling acton_streamer.main.main module: Type error
NoInfo 86
@.***:~/terastream/acton-streamer$
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you were mentioned.Message ID: @.***>

fixed by #1557


Combine Pacman with GhostRepository

Just like the Ghosts, the Pacman should be selectable by name.
This should only be started after issue #41 is completed.

Merging this issue into issue #41 because it's a lot of work to maintain the PacmanRepository's separate functionality while modifying the Player identification by name.
The Pacman will be selectable by name, just like the Ghosts.


istioctl: advise user to monitor gtw deployment when applying waypoint

Please provide a description of this PR:
Fixes: #51294
Includes a kubectl command in the output that advises users to monitor their gateway deployment to ensure that it was actually deployed or not:
azureuser@dev-2024:~/istio$ k create ns test9000
namespace/test9000 created

azureuser@dev-2024:~/istio$ k apply -f samples/httpbin/httpbin.yaml -n test9000
serviceaccount/httpbin created
service/httpbin created
deployment.apps/httpbin created

azureuser@dev-2024:~/istio$ go run ./istioctl/cmd/istioctl x waypoint apply -n test9000 --name httpbin --enroll-nam
espace
waypoint test9000/httpbin applied
namespace test9000 labeled with "istio.io/use-waypoint: httpbin"
monitor status of waypoint deployment with `kubectl get gateways -n test9000`

azureuser@dev-2024:~/istio$ k create ns testing
namespace/testing created

azureuser@dev-2024:~/istio$ k apply -f samples/httpbin/httpbin.yaml -n testing
serviceaccount/httpbin created
service/httpbin created
deployment.apps/httpbin created

azureuser@dev-2024:~/istio$ go run ./istioctl/cmd/istioctl x waypoint apply -n testing --name httpbin
waypoint testing/httpbin applied
monitor status of waypoint deployment with `kubectl get gateways -n testing`

Isn't this the point of --wait?

I think ppl are in favor of training users to use this command to monitor their gateway to avoid confusion when a gateway isn't actually applied because of what's defined in the linked issue: https://github.com/istio/istio/issues/51294#issuecomment-2147833733
https://github.com/istio/istio/issues/51294#issuecomment-2147852730

monitor status of waypoint deployment with kubectl get gateways -n testing

Does not seem very helpful for a newer users. It will return istio gateways, not k8s ones, in some cases I think as a first issue. It also doesn't tell users what do look for. It also is hard to monitor because its a one-shot command

monitor status of waypoint deployment with kubectl get gateways -n testing

Does not seem very helpful for a newer users. It will return istio gateways, not k8s ones, in some cases I think as a first issue. It also doesn't tell users what do look for. It also is hard to monitor because its a one-shot command

That's a valid point

superceded by: #51559


Add uv python pin

Adds a uv python pin command to write to a .python-version file.
We support all of our Python version request formats. We also support a --resolved flag to pin to a specific interpreter instead of the provided version. We canonicalize the request with #4949, it's not just printed verbatim. We always attempt to find the interpreter so we can warn if it's not available. With --resolved, if we can't find the interpreter we fail. If no arguments are provided, we'll attempt to display the current pin.
In the future:

We should confirm that this satisfies the Requires-Python metadata if a pyproject.toml is present
We should support writing to a uv.python-version field if pyproject.toml or uv.toml are present
We should support finding and updating the "nearest" Python version file (looking in ancestors)
We should support finding version files in workspaces
We should support some sort of global pin

Is .python-version parsed by other tools, though?

I believe some tools will parse it, yes, but I'm not sure which. For most uses, we'll be writing a simple version. It may be fair to say if you go outside that it'll only be supported by us. We could also consider only supporting writing simple versions until we have pyproject.toml support, but we already support reading it so... idk.

I'll do some research, but this might just be a caveat in the documentation?

A brief Google search makes it look like just pyenv.

Mise also parses it and Rye does too IIRC.

So what are you thinking? What does mise do if it's not in the format it expects?

I'm not certain, I can test it out.

Actually, it looks like mise uses .tool-versions which is generic (not just for Python).

It looks like if Rye doesn't recognize it, it just gets ignored.

It just uses this:
/// Return the [`PythonVersionRequest`] from a `.python-version` file.
fn read_python_version(contents: &str) -> Option<PythonVersionRequest> {
    // Skip empty lines and comments.
    let ver = contents.lines().find(|line| {
        let trimmed = line.trim();
        !(trimmed.is_empty() || trimmed.starts_with('#'))
    })?;

    // Parse the version.
    let ver = ver.parse().ok()?;

    Some(ver)
}

Thanks for checking!
I think I'd prefer to support the full syntax of Python requests we support (e.g., it makes sense to allow pinning to<EMAIL_ADDRESS>if that's what your project needs) and just clearly document that other tools may not support those complex requests? We could prioritize pyproject.toml support and only allow the extended syntax there but then people who aren't working in "projects" don't have a way to pin local Python versions consistent with the rest of our interface.


Error on localhost login

Running localhost and attempting to login via email, the following error occurs.
Typed email and clicked "Continue With Email" a message appears "Secret must be a string with 64 hex characters." . When checking api logs I do not see any attempt to resend or to stripe. I also tried regenerating all new api keys, which did not resolve the issue.

Have you initialized the template with npx create-remix-saas@latest? If so, that TOTP string should be automatically set.
The TOTP_SECRET needs to be 64 hex characters as the error indicates, e.g.: "TOTP_SECRET=4570DE7251B98D442136595C4C129CB9932E2798F50F915CE8D09FB1239AD865"

https://github.com/dev-xo/remix-auth-totp/tree/main/docs#1-implementing-the-strategy-instance


Problem: test are broken

The test were broken on travis and locally. At least some of it is
because the control panel broke for containers with the reposiotry
in a strange place. With these modifications the tests run again
locally.
Let's see if travis is happy.

Boom! Merge please :smile:

It seems that the home handler has been removed. Is that intentional?

Yes, for some repos (like the one used in the test) the repo is in a strange place) the check if the repo was modified was broken leading to a 500. So removed that bit of code for the moment. We need a strategy for finding the repo dir inside the container.

We can/should add it back in, but this time not merge it until travis is green.

Okay, let's merge this.


Possibly add a version of the bindings with "internal" turned off

Hi there, thanks for keeping these bindings up to date, I use them every day in several projects via my Rust bindings that build on them.
One snag I have run into is that the version of the bindings in this repo has the internal API exposed, which is nice in terms of enabling the most APIs for users, but unfortunately gets me into trouble (I think, maybe I'm just doing something wrong) with the imgui Rust bindings that use a cimgui version without the internal API exposed (see https://github.com/imgui-rs/imgui-rs/issues/395#issuecomment-846580284).
I considered forking cimplot and running the generator with the internal flag off or vendoring everything altogether in implot-rs, but of course it would be even nicer to have both a version with "internal" and one without it already in the repo here.
What do you think? Thanks in advance for any help or advice.

I try to avoid the burden of having both versions.
What happens when you use this repo with no_internal cimgui?

Thanks for the quick reply!

I try to avoid the burden of having both versions.

Alright, then I'll come up with a solution on my end. I think I might take implot as a submodule and vendor in a no-internals version of cimplot created with your generator along with an attribution note and backlink to the repo here.

What happens when you use this repo with no_internal cimgui?

The build fails because "cimplot with internals" requires symbols from "cimgui with internals" - at least that was the conclusion I came to when trying to get things to work. I don't think it is possible to switch imgui-rs to use the imgui internals without forking the repo - there is an issue for adding support for this, but it does not look to be resolved soon.

The build fails because "cimplot with internals" requires symbols from "cimgui with internals"

Do you have a compiler error with symbols missing?
Could you post it?
As far as I know it should not happen.

I'll try to come up with one. The fact that you say things are expected to work is already very good input for me, it makes me suspect that I am doing something wrong on my end. I'll go experiment a bit (might take a while, I'm not sure if I'll get to it today) and get back to you once I have some results.

Another question for me to experiment with: The imgui-rs version I point to seems to vendor in an older version of cimgui from back in april, so I am not sure which version of cimplot I should pair this with to try and get things working. If I naively try out the current master, I get errors in the bindings generation process with bindgen about ImRect and ImPoolIdx being missing:
[...]
.../cimplot/cimplot.h:64:94: error: unknown type name 'ImPoolIdx'
.../cimplot/cimplot.h:66:76: error: unknown type name 'ImPoolIdx'
.../cimplot/cimplot.h:68:76: error: unknown type name 'ImPoolIdx'
.../cimplot/cimplot.h:70:82: error: unknown type name 'ImPoolIdx'
.../cimplot/cimplot.h:439:5: error: unknown type name 'ImRect'
.../cimplot/cimplot.h:468:5: error: unknown type name 'ImRect'
.../cimplot/cimplot.h:486:5: error: unknown type name 'ImRect'
.../cimplot/cimplot.h:488:5: error: unknown type name 'ImRect'
.../cimplot/cimplot.h:501:5: error: unknown type name 'ImRect'
.../cimplot/cimplot.h:502:5: error: unknown type name 'ImRect'
.../cimplot/cimplot.h:503:5: error: unknown type name 'ImRect'
.../cimplot/cimplot.h:504:5: error: unknown type name 'ImRect'
.../cimplot/cimplot.h:515:5: error: unknown type name 'ImRect'
.../cimplot/cimplot.h:516:5: error: unknown type name 'ImRect'
.../cimplot/cimplot.h:583:5: error: unknown type name 'ImRect'
[...]

... and, looking around imgui, I find that type in the imgui_internal.h C++ header here.
I'm not sure what follow-up things to ask, because I don't want to waste your time with problems specific to the way I create and build my Rust bindings - I just describe things here in case there is something super obvious you can help me with without having to deal with all the details.

You could try to compile implot with that back in april
If it complains then you should change to an older implot until you find a compatible one.

Thanks for the hints, I'll do more experiments.

Perhaps in your compilation you are forgetting to include imgui_internal.h

So far I have been using cimplot so I can create an FFI to a C API - if that C API requires the inclusion of a C++ header, I have to deal with C++ code at the FFI boundary, and then I might as well attempt to create bindings to the C++ code directly. Maybe I am just confused though and am missing the point.

But this shouldnt be as cimplot.cpp already includes imgui_internal.h

For cimplot.cpp things should be fine (that is a C++ build anyway), but cimplot.h itself refers to things like ImRect as well (for example here). If paired with an internal-enabled version of cimgui, a C version of ImRect is available here, which works, but for the combination of an non-internal-enabled cimgui and an internal-enabled cimplot, I don't yet see a way to create bindings using only C headers because types like ImRect and ImPoolIdx are not available as C types.
In any case, the original issue is answered and I'll see if I can work around the problem in implot-rs itself a bit, so we can close the issue I think.
Thanks for all your help!

If paired with an internal-enabled version of cimgui, a C version of ImRect is available here,

But this is not docking_inter branch but master branch.
In cimgui repo, master branch it is the no internal branch (the same used by imgui-rs)

Update: without internal in cimgui and internal in cimplot compilation succeds but the problem appears in the part of cimplot.h revealed with CIMGUI_DEFINE_ENUMS_AND_STRUCTS, used in the binding generation, where ImPoolIdx is not defined.
Solutions are:
1 - generate cimplot without internal
2 - generate cimplot with internal but manually add ImPoolIdx definition (and posibly others) doing cut and paste from cimgui.h (with internal generation)

Thanks for looking into this further! I did not yet get around to do more experiments, but I'll see if I can make one of your two suggestions work.


Feature/254

#254 develop finish, compile success, but not test

Merged directly without any review.


Text generation isn't predicting correctly

hello,
I am building a text generation model in Chinese and trying to pull out the probabilities between words for another task. This is the model I follow : https://www.tensorflow.org/text/tutorials/text_generation.
import tensorflow as tf
from tensorflow.keras import models, layers, preprocessing, Model, callbacks, activations

inputShape = (X_converted_final.shape[1],)
input = layers.Input(shape=inputShape)
x=layers.Embedding(vocab_size2, embedding_dim, input_length=50, trainable=True, embeddings_initializer="uniform")(input)
x = layers.GRU(rnn_units, return_sequences=True, bias_initializer="zeros")(x)
output= layers.Dense(vocab_size2, activation=activations.softmax)(x)

gru_fpi_model = models.Model(input, output)
gru_fpi_model.compile(loss=tf.losses.SparseCategoricalCrossentropy(from_logits=False),
              optimizer='adam', metrics=["accuracy"])
callback = callbacks.EarlyStopping(monitor='accuracy', patience=1)
gru_fpi_model.summary()

So this model should be able to give me '天' as the highest possible next word if ’昨’ is provided in the sequence, since '昨天' means 'yesterday' in Chinese and we have a lot of examples in the training data. This model always takes a sequence so I tried to give the model a sequence and regard the highest probabilities of next words as the best candidate, given a previous word.  Here is what I did.
prediction=np.argmax(loaded_model(sentence)[:,2,:], axis=-1)
With this line, if I give the model a sentence '我昨天晚上去上学', the model should give me the word that has the highest probabilities after '昨' (in the second position, including the beginning marker of sentence), which should be '天' or other words, but it instead gives me '昨'. I couldn't find any sequence '昨昨' in the training data so there must be something wrong. For some words, it predicts well, e.g. '我', it gives me '們'and it means 'we' in English. So I think this model still works for some words, but it just doesn't work for some.
Could anyone give me some hints on this? Thank you so much

Hi @limkhaiin1012 !
Please post this issue on keras-team/keras repo.
To know more see;
https://discuss.tensorflow.org/t/keras-project-moved-to-new-repository-in-https-github-com-keras-team-keras/1999 . Thanks!


<IP_ADDRESS> breaks import in Google Colab examples

Running example number 4 (Multi molecule tracking) stops at the second cell when importing deeptrack :
---------------------------------------------------------------------------
ImportError                               Traceback (most recent call last)
<ipython-input-2-c0187a06b858> in <module>()
      2 import matplotlib.pyplot as plt
      3
----> 4 import deeptrack as dt
      5
      6 from deeptrack.extras import datasets

4 frames
/usr/local/lib/python3.7/dist-packages/deeptrack/models/autotracking/generators.py in <module>()
      1 from ...generators import ContinuousGenerator
      2 from ...augmentations import Affine
----> 3 from ...features import Value
      4 from .equivariances import (
      5     ScaleEquivariance,

ImportError: cannot import name 'Value' from 'deeptrack.features' (/usr/local/lib/python3.7/dist-packages/deeptrack/features.py)

!pip install deeptrack installed deeptrack <IP_ADDRESS>.
Explicitely stating !pip install deeptrack==0.11.4 results in the same error.
!pip install deeptrack==0.11.3 solves the issue.
I haven't tried replicating it on a local install yet.
EDIT :
This also seems to be the case on local install.
I created a fresh conda environment, ran pip install deeptrack, then ran import deeptrack as dt, which results in :
Exception has occurred: ImportError
cannot import name 'Value' from 'deeptrack.features' (C:\Users\Magnetic Tweezers\anaconda3\envs\test_dt\lib\site-packages\deeptrack\features.py)
  File "C:\Users\Magnetic Tweezers\Documents\user\isol\test.py", line 1, in <module>
    import deeptrack as dt

Installing deeptrack 0.11.3 also solves the issue here.

Same issue on Google colab

This seems to be Git not removing empty folders on checkouts, which conflicts with Python modules. Should be fixed in 0.11.5. Please reopen if the issue persists!


Porting PhantomJS to ppc64le.

PhantomJS is an important software used to headless browser testing. At present PhantomJS is not available on ppc64le platform. The main hurdle is that the two main components viz. Google Breakpad and QtWebkit need to be ported to ppc64le first. Towards this, two issues have been created for Breakpad and QtWebkit:
https://bugs.chromium.org/p/google-breakpad/issues/detail?id=673
https://bugreports.qt.io/browse/QTBUG-49219
Once these packages are ported to ppc64le, they need to be integrated in PhantomJS and then PhantomJS code needs to be compiled for ppc64le. At present, no substantial effort is identified in order to port PhantomJS code (excluding above mentioned components) to ppc64le platform.

Any updates ? We need this too.

We don't use Breakpad anymore. Porting QtWebKit won't be trivial.
Please track the progress, if any, in those upstream issues. PhantomJS won't be ported until those issues are resolved.

any update? lack of phantomjs support for Power architecture is breaking Apache Zeppelin build on Power as well..

i am able to build phantomjs 2.1.1 and the development branch as such without any fixes in ppc64le (RHEL 7.1)
Ran some of the examples which comes with the source.
Ran the test-suite as well..
$ python run-tests.py
ff...................f.........f.............s................P............s....f...
module/webpage/render: XPASS
XPASS: JPEG
XPASS: JPEG (format option)
XPASS: JPEG (quality option)
XPASS: JPEG (format and quality options)
17.470s elapsed
203 passed
4 passed unexpectedly
12 failed as expected
2 skipped
I am interested in contributing the binary to the official download site.
https://bitbucket.org/ariya/phantomjs/downloads

I also managed to build the binary with few dynamic linkings
$ ldd phantomjs
linux-vdso64.so.1 =>  (0x00003fff884c0000)
libz.so.1 => /lib/powerpc64le-linux-gnu/libz.so.1 (0x00003fff88480000)
libfontconfig.so.1 => /usr/lib/powerpc64le-linux-gnu/libfontconfig.so.1 (0x00003fff88410000)
libfreetype.so.6 => /usr/lib/powerpc64le-linux-gnu/libfreetype.so.6 (0x00003fff88330000)
libdl.so.2 => /lib/powerpc64le-linux-gnu/libdl.so.2 (0x00003fff88300000)
libpthread.so.0 => /lib/powerpc64le-linux-gnu/libpthread.so.0 (0x00003fff882c0000)
libstdc++.so.6 => /usr/lib/powerpc64le-linux-gnu/libstdc++.so.6 (0x00003fff88150000)
libm.so.6 => /lib/powerpc64le-linux-gnu/libm.so.6 (0x00003fff88060000)
libgcc_s.so.1 => /lib/powerpc64le-linux-gnu/libgcc_s.so.1 (0x00003fff88030000)
libc.so.6 => /lib/powerpc64le-linux-gnu/libc.so.6 (0x00003fff87e60000)
/lib64/ld64.so.2 (0x000000005da70000)
libexpat.so.1 => /lib/powerpc64le-linux-gnu/libexpat.so.1 (0x00003fff87e00000)
libpng12.so.0 => /lib/powerpc64le-linux-gnu/libpng12.so.0 (0x00003fff87dc0000)

Uploaded the binary tar in the below link
https://github.com/ibmsoe/phantomjs/releases/download/2.1.1/phantomjs-2.1.1-linux-ppc64.tar.bz2

I am interested in contributing the binary to the official download site.
https://bitbucket.org/ariya/phantomjs/downloads


Installation/Setup

Hello, sorry but i don't understand well how to include textra typist in my project, could someone be so kind as to explain it in details? Thank you in advance.

I guess I'd recommend gdx-liftoff (link) for this; it's an alternative to gdx-setup. You can select TextraTypist as a dependency under Third-Party. It would probably help to read this libGDX wiki article about Gradle dependencies if you want to add additional libraries to an existing project. Essentially, the README.md for this project lists the dependencies you need, which is probably just implementation "com.github.tommyettinger:textratypist:0.6.3". That goes in the dependencies section of your core/build.gradle (for gdx-liftoff projects) or the core section of your root build.gradle (for gdx-setup projects). That's the later dependencies block if there is more than one (it doesn't go in the buildscript section).
Any extra stuff would probably involve adding assets. To use the preconfigured fonts in KnownFonts, you need some of the .fnt, .png, and license files from https://github.com/tommyettinger/textratypist/tree/main/knownFonts (just any files you want to use, you don't need all of them). It may be easiest to just download the whole repo as a .zip and get the files that way if you need many. You can also use your own fonts, such as those from a Skin produced by Skin Composer; Skins are pretty important to scene2d.ui usage but they're a little complex, so check the Skin Composer wiki for lots of documentation and instructions for how scene2d.ui works. Generally, if a Skin works in regular scene2d.ui widgets, it probably will work similarly in the widgets here that use TypingLabel or TextraLabel.
If you have more questions, the libGDX Discord can probably walk you through any setup questions you have,