Upload: prevent concurrent uploads with same lbry name

If you make an upload with the same lbry name and same channel, 2 uploads are happening in the background with only 1 item shown in the GUI (you'll see the progress bar jumping to reflect 2 uploads).  I guess the first one to finish gets the lbry url, while the second fail will fail at the SDK call.  I encountered this outside of abuse testing -- forgot to change the lbry name, and since there was no conflict error, went ahead to publish.
I think the issue has been there since the beginning since I did not change the upload key format (name#channel) when adding tus.

This is one of the reason for stat - no such file error


LICENSE empty

Hi, I've got GPLv2 project wtfbbqhax/tremulous which I want to scoop in this semver implementation for our auto-updater.
Cheers,
wtfbbqhax

Added MIT license.


Footer no longer has sitemap

The current footer is missing some content. I think the content type name changes didn't get applied to the footer.
Current footer:

Old footer:

Fixed a28967724b9de9846d7c68290e04eb3e0c3a10de


Swipe-right on touchscreen laptops triggers Chromes previous-page function.

Simply swiping to the right moves the current page aside revealing the previous one, even if preventDefault is called on TouchStart. Is there any way to disable it?

Iiiiiiiinteresting. Wonder if we need evt.preventDefault() to account for that.

Yes, this navigation feature/gesture of Chrome interferes with these samples and many other web apps with similar functionality and it's VERY annoying as it can completely break the user experience because it's not the expected outcome when swiping on a sidebar, etc.
There's a Chrome flag that users can disable but really there needs to be a way for developers to disable it in code when there's a risk of a clash - if not then it should be disabled by default instead of enabled as it clearly hasn't been thought through.


Dataset format

Hello, first of all thank you for such an organised package. Before I use your package for my own custom dataset, I wanted to test your methods on UCSDped dataset. The format of the dataset is quite different that what your package needs.
Do you have a script to change UCSDped to the desired format?


Base16-tomorrow-night theme isn't working properly

The colors in the "tomorrow-night" theme are miss matched. For instance, when I open st and then vim, the colors look different when I launch vim within the terminal at the same time, like using the command "st -e vim" in dmenu or another terminal. Please fix it soon

I figured it out, I just wasn't using the 256-colors variant for my terminal. More specifically I didn't modify x.c to add 256 colors support. Thank to @yshch for the help.


hey quick question/ possible error

hey in misc functions
	if [ -n __BMT_START_WATCH ]
		then
			local stopWatchVar=0
			stopWatchVar=$(bmtMiscFunc now)
			bmtMiscFunc elapsed "$__BMT_START_WATCH" "$stopWatchVar"
			return 0
		else
			return 2
		fi
	;;

under stopwatch #6 shouldnt it be
if [[ -n "$__BMTSTART_WATCH" ]];
     then

i could be wrong I'm fairly new to programming.

HI
Yes that's correct. that's a typo, I should have spotted that as shellcheck flags it as an  issue(SC2154) and I always run shellcheck on scripts before i use/publish them.
The stopwatch function still works  , that "if statement" is only there to catch a user error: if they run "stopWatch" before they run "startWatch". and return error code 2.
thanks

hey no worries i just wasn't sure if there was some convention i didn't know about, i only noticed it because I'm building a bash snippet manager / command line IDE  and that and i happened to be doing it in vs code and it point it out to me. I've only been programming 6 months and exclusively in bash so far and its got some jacked up exceptions (like I've seen code that's set up to intentionally fail which still doesn't make sense to me).


trigger by modifying terramate file

Ran Plan for dir: stacks workspace: default
Plan Error
running "/usr/local/bin/terraform plan -input=false -refresh -out \"/home/atlantis/.atlantis/repos/oponomarov-tu/terramate-atlantis/3/default/stacks/default.tfplan\"" in "/home/atlantis/.atlantis/repos/oponomarov-tu/terramate-atlantis/3/default/stacks": exit status 1
╷
│ Error: No configuration files
│
│ Plan requires configuration to be present. Planning without a configuration
│ would mark everything for destruction, which is normally not what is
│ desired. If you would like to destroy everything, run plan with the
│ -destroy option. Otherwise, create a Terraform configuration file (.tf
│ file) and try again.
╵

Ran Plan for 2 projects:

dir: stacks workspace: default
dir: stacks/stack-2 workspace: default

1. dir: stacks workspace: default
Plan Error
running "/usr/local/bin/terraform plan -input=false -refresh -out \"/home/atlantis/.atlantis/repos/oponomarov-tu/terramate-atlantis/3/default/stacks/default.tfplan\"" in "/home/atlantis/.atlantis/repos/oponomarov-tu/terramate-atlantis/3/default/stacks": exit status 1
╷
│ Error: No configuration files
│
│ Plan requires configuration to be present. Planning without a configuration
│ would mark everything for destruction, which is normally not what is
│ desired. If you would like to destroy everything, run plan with the
│ -destroy option. Otherwise, create a Terraform configuration file (.tf
│ file) and try again.
╵

2. dir: stacks/stack-2 workspace: default
Terraform used the selected providers to generate the following execution
plan. Resource actions are indicated with the following symbols:
+ create

Terraform will perform the following actions:

  # null_resource.example will be created
+ resource "null_resource" "example" {
      + id = (known after apply)
    }

Plan: 1 to add, 0 to change, 0 to destroy.

:arrow_forward: To apply this plan, comment:

atlantis apply -d stacks/stack-2

:put_litter_in_its_place: To delete this plan click here
:repeat: To plan this project again, comment:

atlantis plan -d stacks/stack-2

:fast_forward: To apply all unapplied plans from this pull request, comment:

atlantis apply

:put_litter_in_its_place: To delete all plans and locks for the PR, comment:

atlantis unlock


Broken symlinks in kbfs

I had a git repo inside my kbfs (I wanted the auto-sync of kbfs and the history inspection of git). I realize this is probably not ideal but I'm very good at git and thought I could handle it. Unfortunately now that I have moved this repo elsewhere, all of my other files are broken symlinks to a some git-annex repo that I don't know how to access.
What can I do to fix this or troubleshoot further?
my log id: 5a7c1a151119c7169b861e1c

I don't really understand what's wrong here.  You made KBFS files that are symlinks to data that you have now deleted from KBFS?
If you need to recover data that has been deleted within the last 2 weeks, it's possible -- see the Time Travel section of this doc.
Or are you saying that KBFS itself turned perfectly good files into broken symlinks somehow?

(And we do not recommend having a git repo stored directly in KBFS, because KBFS doesn't provide the locking mechanisms that it relies on for multiple clients to use it at the same time, and it can't really handle the file system interrupts that git throws at it when not in quiet mode.  We have native git repos that are much better for this

Or are you saying that KBFS itself turned perfectly good files into broken symlinks somehow?

This is what happened. For example:
$ file ~/keybase/private/bsima/docs/resume.pdf
/home/ben/keybase/private/bsima/docs/resume.pdf: broken symbolic link to .git/annex/objects/fW/Vp/SHA256E-s35843--d15125e50ac5cbc7e46c4e8a7d80b80014e2838baac70875f6f9a5ce190517f0.pdf/SHA256E-s35843--d15125e50ac5cbc7e46c4e8a7d80b80014e2838baac70875f6f9a5ce190517f0.pdf

Where is this .git/annex directory supposed to be?

@bsima that's not something created by Keybase; something on your side created that link.  .git/annex is not a Keybase thing.  Again, if you need to recover data, see the link above.


How to start the docusaurus with /docs

When i start docusaurus using npm start
The Page is opening with baseURL /, How do I change it to /docs
For Example:
When I start docusaurus using npm start
The page should directly land in http://localhost:3000/docs/
How do I do it ??

It would be better to explain why you want to do that, and particularly why is this important in dev?
You can create an index page, use ReactRouter <Redirect to="/docs"/> component (exposed as @docusaurus/router)
You can also create a static HTML file in /static/index.html on which you perform a HTML redirect using JS
You can also setup the redirect only in your production env by using a server-side redirect on your host provider config (best solution)


RuntimeError: !!Loop Protection!! We have tried to solve 3 time(s) in a row.

getting this error (with Python 3.7):
> python.exe javcloudscraper.py eyan-064
> <_io.TextIOWrapper name='C:\\Temp\\jav\\eyan-064-html.txt' mode='w' encoding='cp1252'>
> eyan-064
> RuntimeError: !!Loop Protection!! We have tried to solve 3 time(s) in a row.

my code:
> import cloudscraper,sys
>
> tempdir = "C:\\Temp\\jav\\"
> outputfile = open(tempdir + sys.argv[1] + '-html.txt', 'w')
>
> scraper = cloudscraper.create_scraper()  # returns a CloudScraper instance
> print(outputfile)
> print(sys.argv[1])
> print(scraper.get("http://www.javlibrary.com/en/vl_searchbyid.php?keyword=" + sys.argv[1]).content,file = outputfile)
> outputfile.close()

What version of the library are you using?

> import cloudscraper,sys
>
> tempdir = "C:\\Temp\\jav\\"
> outputfile = open(tempdir + sys.argv[1] + '-html.txt', 'w')
>
> scraper = cloudscraper.create_scraper()  # returns a CloudScraper instance
> print(outputfile)
> print(sys.argv[1])
> print(scraper.get("http://www.javlibrary.com/en/vl_searchbyid.php?keyword=" + sys.argv[1]).content,file = outputfile)
> outputfile.close()

🤢 , pep8!!
import cloudscraper
import os
import sys

with open(os.path.join(tempdir, '{}-html.txt'.format(sys.argv[1])), 'w') as outputFile:
    print(
        scraper.get(
           'http://www.javlibrary.com/en/vl_searchbyid.php',
            params={'keyword': sys.argv[1]}
        ).content,
        file = outputfile
    )

my bad, i should've checked for an update before i posted this. didnt realize you had a new update! works fine with the new update! great job!

All good man, glad you have it sorted :)


Readme should show where setup commands are run

Users are often confused when steps happen on the device and which steps should be run from their personal computer. This edit should make that distinction more clear.
Change-type: patch
Signed-off-by: Stef Kors<EMAIL_ADDRESS>

@StefKors thank you for the update!


Only overwrite HasOne/MorphOne field names on display

refs: #3971

... found out that the problem is that we parse the dot.notation field names into dot[notation] names, and when using fluent syntax after changing the names the subsquent ->type() etc calls would not find the field in the array of fields because it would check userDefined.test against userDefined[test]

This PR aims to change the field names only when displaying them on page, keeping the "crud fields" with the dot notation name.
I think this could be NON-BC, because this behaviour applies only to HasOne and MorphOne relations, and nothing changed in terms of field apresentation, validation or definition. But I could be wrong here, please let me know.
Anyway atleast in 4.2 this should be merged, I am just waiting for one of you guys green ligth to merge this into the relationships branch.

A new inspection was created.

A new inspection was created.

The inspection completed: 2 updated code elements

The inspection completed: 2 updated code elements

I'm rewriting Pedro's PR above and the issue using the new Pull Request template, to put things into perspective:

WHY
BEFORE - What was wrong? What was happening before this PR?
@ale1981 reported this bug in https://github.com/Laravel-Backpack/CRUD/issues/3971
When you try to add a HasOne or MorphsOne relationship using the fluent syntax, it won't work:
            $this->crud->field('userDefined.test')
                ->type('boolean')
                ->label('Test')
                ->entity('userDefined')
                ->attribute('test')
                ->model('App\Models\userDefined');

But the same thing using the array syntax WILL work:
            $this->crud->addField([
                'name' => 'userDefined.test',
                'type' => 'boolean',
                'label' => 'Test',
                'entity' => 'userDefined',
                'attribute' => 'test',
                'model' => 'App\Models\userDefined',
            ]);

AFTER - What is happening after this PR?
You can use the fluent syntax above, no problem.
HOW
How did you achieve that, in technical terms?

... found out that the problem is that we parse the dot.notation field names into dot[notation] names, and when using fluent syntax after changing the names the subsquent ->type() etc calls would not find the field in the array of fields because it would check userDefined.test against userDefined[test]

This PR aims to change the field names only when displaying them on page, keeping the "crud fields" with the dot notation name.

Is it a breaking change or non-breaking change?

I think this could be NON-BC, because this behaviour applies only to HasOne and MorphOne relations, and nothing changed in terms of field apresentation, validation or definition. But I could be wrong here, please let me know.

Anyway atleast in 4.2 this should be merged, I am just waiting for one of you guys green ligth to merge this into the relationships branch.

How can we test the before & after?
Use both of the syntaxes above to add a HasOne / MorphsOne field. They should return the exact same thing.

I love this @pxpm . Thank you - so from what I understand it's better to overwrite the field names as a GETTER instead of a SETTER. That makes TOTAL sense, don't know why we didn't think of this before.
I have some refactoring ideas/requests, but I totally agree with this ❤

A new inspection was created.

A new inspection was created.

The inspection completed: 3 new issues, 3 updated code elements

A new inspection was created.

A new inspection was created.

A new inspection was created.

A new inspection was created.

@tabacitu I pointed this into 4.2 branch because we are messing with public functions. As a rule of thumb, changing public functions should be considered a BC.
Let me know if you disagree with me here and we should merge this in 4.1 anyway, if so I will create a branch with this changes for 4.1.
Pedro

The inspection completed: 3 new issues, 2 updated code elements

The inspection completed: 3 new issues, 3 updated code elements

I pointed this into 4.2 branch because we are messing with public functions. As a rule of thumb, changing public functions should be considered a BC.

Totally agree. 4.2 it is. I've merged 4.2 into this so the diff is cleaner. But there are still changes in 13 files (!!!) for something that only affects one/two fields. That's a huge red flag 🚩🚩🚩 - it means we're doing something wrong. I have an idea on how to easily fix this - see my inline comments please @pxpm

Basically I see two options here:
Option 1 - overwrite stuff only on the getter (instead of the setter)
Detailed in the inline comments, more specifically:

https://github.com/Laravel-Backpack/CRUD/pull/3981#discussion_r763718622
https://github.com/Laravel-Backpack/CRUD/pull/3981#discussion_r761805530

The good part about this is that we could probably also do that for relationships.
Option 2 - stop overwriting stuff
What if we store both user.category.name and user[category][name]  on the field? A field could look smth like this:
[
    'name' => 'user.category.name',
    'input_name' => 'user[category][name]',
    'type' => 'text',
    'label' => 'User Category Name',
]

Basically:
// when you do
CRUD::field('user.category.name');
// or
CRUD::addField([
    'name' => 'user.category.name',
]);

// what CRUD will do is that it will create a CrudField with
// - name: user.category.name
// - input_name: user[category][name]

PROs:

no more overwriting stuff;
it wouldn't be very different from the key in Columns, which is something that's calculated automatically when you give it the name, if you don't specifically give it a key;

CONs:

we have to change all fields to use input_name instead of name;

All in all, from the two options above, I think OPTION 1 would be better, since 4.2 is supposed to be a SMALL breaking change, and we've already gone a bit overboard with the BC.
OPTION 2 would have been great as a rewrite... I wish we had the foresight to consider the first thing to give a field / column / filter / button the key, not the name. And then intelligently deduct the name in most cases, or allow the user to define a certain name. Maybe we can do that when we completely refactor to objects instead of arrays, who knows. It will certainly make more sense to to do the overwriting on $field->getName() rather than on $crud->addField().

So I say... let's go with OPTION 1. What do you think @pxpm ?

closing in favor of #3996


Audit and clean up amen-player

Many unanswered questions in here, much room for cleanup.

Done with #30, hooray!


Feature/dawgpath 146

Jira: https://jira.cac.washington.edu/browse/DAWGPATH-146

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

1 of 1 (100.0%)  changed or added relevant line in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.05%) to 63.496%

Totals

Change from base Build<PHONE_NUMBER>:
0.05%

Covered Lines:
494

Relevant Lines:
778

💛  - Coveralls


Windows trayballoon support

Windows trayballon example

Thanks for the PR!
Please read CONTRIBUTING.md. You would have seen that the best course of action for a feature PR is to open an issue first for a discussion, as this may save everyone time.
We could have explained two things:

this functionality is better implemented as a reporter
we will likely not be accepting more reporters into core

I'm going to close this PR, but it's a good idea. I encourage you to implement this as a reporter in a fork, but keep in mind the proposed plugin API would likely break compatibility when it is delivered.
As an alternative something like grunt with grunt-notify may be a good bet.

You are right about the reasons thanks for the reply :+1:


Gepard Shield 3.0 Bypass Code 7

hi
https://media.discordapp.net/attachments/469615764920467458/706875588731994172/unknown.png?width=586&height=253
Before .vmx changes i got Gepard Shield 3.0 Bypass Code 2, after changes and loaded kernel driver i get Gepard Shield 3.0 Bypass Code 7.
maybe somebody know how to bypass?

did u solve it? if yes need ur assist with it


Search std for module name when using with module.

related #7737

@msullivan Not entirely sure this is what you had in mind, but I've added schema and describe tests.

@msullivan the tracer code already did what we wanted, it was just the schema that needed updating.


chore: output declaration files for neuroglancer when installed with NPM

esbuild does not support generating declaration files https://github.com/evanw/esbuild/issues/95

I improved the quality of the declarations by using the tsconfig file. The bulk of the change comes from "moduleResolution": "bundler",
It does lose a little bit of type information for some reason but it adds a lot more than it removes
-    spatiallyIndexedSources: Set<Borrowed<AnnotationGeometryChunkSource>>;
+    spatiallyIndexedSources: Set<AnnotationGeometryChunkSource>;

-    references: Map<AnnotationId, Borrowed<AnnotationReference>>;
-    localUpdates: Map<AnnotationId, LocalUpdateUndoState>;
+    references: Map<string, AnnotationReference>;
+    localUpdates: Map<string, LocalUpdateUndoState>;

Thanks --- we could probably just emit the declarations all the time --- I don't think there is a time when we don't want the declarations.

Having the type aliases resolved in some cases is fine, I think, and I agree that some annotations are better than none.
When I tried to generate declarations previously I ran into some errors due to some exported things having types depending on non-exported types, I think.  But I guess it works now for some reason.

Removed the flag making it optional. Added that in case the performance benefit of leaving it off was useful.
I agree imperfect types are far better than none. I was fortunate that this solution https://github.com/Microsoft/TypeScript/wiki/Using-the-Compiler-API#getting-the-dts-from-a-javascript-file was very easy to drop in

I just realized const host = ts.createCompilerHost(options); is probably unnecessary but it does change the declaration files. Removes ".ts" from the imports
-    get changed(): import("../util/signal.ts").Signal<() => void>;
+    get changed(): import("../util/signal").Signal<() => void>;

Checking to make sure that isn't a problem.

check(new Uint64(1423829346,<PHONE_NUMBER>), 28); in toString parseString round tripis a real error. I'm trying to debug it
check(new Uint64(0, 79741775), 29); also fails, and it seems to fail with any low value.
79741775 seems to be the smallest high that causes this test to fail with base 29.


Added note for Mill users

Added a brief note to help direct Mill users who would like similar functionality.
Feel free to use or not or let me know if you'd like changes, just trying to add a sign post of the next traveller. 😄

Before this is merged, I just wanted to drop a note that I'm thinking about archiving mill-tpolecat and signposting scalac-options instead as mentioned on the mill-tpolecat PR: https://github.com/DavidGregory084/mill-tpolecat/pull/54#issuecomment-2410493258


Support multiple headers at the same time

Swagger supports multiple headers but requires that each one uses a unique name. The existing approach overwrote the previous header definition. This also caused issues when changing the value of #input_apiKey as it stores the api key with the same name as all defined headers, leading to complete header loss on api key addition. This fix ensures each header is given a unique name.
For more information see: https://github.com/swagger-api/swagger-ui#header-parameters

This makes total sense. Needs a CHANGELOG entry please.

CHANGELOG entry added, good to merge?

Squash-merged via https://github.com/TinkerDev/grape-swagger-rails/commit/d6ed84427ea787c736f8b9709c46ff62e7f54951. Thanks.


Update dependencies

Hi Heiko. Just an update to the dependencies. This will get rid of the "possible binary incompatibility" warnings in one of my current projects.
Also included is the dependency update checking plugin.

Heiko, please also release it after the merge...
Jsoniter-scala evolved a lot since v0.27.1, there was binary incompatible changes and change of artifact names: https://github.com/plokhotnyuk/jsoniter-scala/compare/v0.27.1...v0.29.20

Something wrong happened...
v1.21.1 is tagged but artifacts aren't available on the Maven Central:
http://repo1.maven.org/maven2/de/heikoseeberger/akka-http-argonaut_2.12/
http://repo1.maven.org/maven2/de/heikoseeberger/akka-http-avro4s_2.12/
http://repo1.maven.org/maven2/de/heikoseeberger/akka-http-circe_2.12/
http://repo1.maven.org/maven2/de/heikoseeberger/akka-http-jackson_2.12/
http://repo1.maven.org/maven2/de/heikoseeberger/akka-http-json4s_2.12/
http://repo1.maven.org/maven2/de/heikoseeberger/akka-http-jsoniter-scala_2.12/
http://repo1.maven.org/maven2/de/heikoseeberger/akka-http-play-json_2.12/
http://repo1.maven.org/maven2/de/heikoseeberger/akka-http-upickle_2.12/
The same is with Scala 2.11.x artifacts :(


[Enhancement]: autodeploy doesn't make full use of the hardware resource.

Check Before Asking

[X] Please check the issue list and confirm this bug is encountered for the first time.
[X] Please try full text in English and attach precise description.

Environment
In the ECS: 32C 128G Memory, 500G essd

    devname: eth0
    # External port for OceanBase Database. The default value is 2881. DO NOT change this value after the cluster is started.
    # mysql_port: 2881
    # Internal port for OceanBase Database. The default value is 2882. DO NOT change this value after the cluster is started.
    # rpc_port: 2882
    # Defines the zone for an observer. The default value is zone1.
    # zone: zone1
    # The maximum running memory for an observer. When ignored, autodeploy calculates this value based on the current server available resource.
    memory_limit: 115G
    # The percentage of the maximum available memory to the total memory. This value takes effect only when memory_limit is 0. The default value is 80.
    # memory_limit_percentage: 80
    # The reserved system memory. system_memory is reserved for general tenants. The default value is 30G. Autodeploy calculates this value based on the current server available resource.
    # system_memory: 22G
    # The size of a data file. When ignored, autodeploy calculates this value based on the current server available resource.
    datafile_size: 200G
    # The size of disk space used by the clog files. When ignored, autodeploy calculates this value based on the current server available resource.
    log_disk_size: 66G
    # System log level. The default value is INFO.
    # syslog_level: INFO
    # Print system logs whose levels are higher than WARNING to a separate log file. The default value is true. The default value for autodeploy mode is false.
    enable_syslog_wf: true
    # Enable auto system log recycling or not. The default value is false. The default value for autodeploy mode is on.
    enable_syslog_recycle: true
    # The maximum number of reserved log files before enabling auto recycling. When set to 0, no logs are deleted. The default value for autodeploy mode is 4.
    # max_syslog_file_count: 4
    # Cluster name for OceanBase Database. The default value is obcluster. When you deploy OceanBase Database and obproxy, this value must be the same as the cluster_name for obproxy.
    # appname: obcluster
    # Password for root. The default value is empty.
    # root_password:
    # Password for proxyro. proxyro_password must be the same as observer_sys_password. The default value is empty.
    # proxyro_password:

but after install the cluster, the server's resource is as following:
mysql> SELECT * FROM oceanbase.GV$OB_SERVERS;
+---------------+----------+-------+----------+--------------+------------------+--------------+------------------+--------------+--------------+-------------------+-------------------+-----------------+--------------------+------------------+-------------------------+--------------+-------------------------+-----------------------+
| SVR_IP        | SVR_PORT | ZONE  | SQL_PORT | CPU_CAPACITY | CPU_CAPACITY_MAX | CPU_ASSIGNED | CPU_ASSIGNED_MAX | MEM_CAPACITY | MEM_ASSIGNED | LOG_DISK_CAPACITY | LOG_DISK_ASSIGNED | LOG_DISK_IN_USE | DATA_DISK_CAPACITY | DATA_DISK_IN_USE | DATA_DISK_HEALTH_STATUS | MEMORY_LIMIT | DATA_DISK_ABNORMAL_TIME | SSL_CERT_EXPIRED_TIME |
+---------------+----------+-------+----------+--------------+------------------+--------------+------------------+--------------+--------------+-------------------+-------------------+-----------------+--------------------+------------------+-------------------------+--------------+-------------------------+-----------------------+
| <IP_ADDRESS> |     2882 | zone1 |     2881 |           30 |               30 |           30 |               30 |<PHONE_NUMBER>6 |<PHONE_NUMBER>6 |<PHONE_NUMBER>4 |<PHONE_NUMBER>4 |       536870912 |<PHONE_NUMBER>00 |        469762048 | NORMAL                  |<PHONE_NUMBER>60 | NULL                    | NULL                  |
+---------------+----------+-------+----------+--------------+------------------+--------------+------------------+--------------+--------------+-------------------+-------------------+-----------------+--------------------+------------------+-------------------------+--------------+-------------------------+-----------------------+
1 row in set (0.04 sec)

The memory capacity is lower than expected.

### Fast Reproduce Steps

as before

### Actual Behavior

as before

### Expected Behavior

_No response_

### Other Information

_No response_

You can add ‘production_mode: True’ in the configuration, or select "Maximized Deployment" through OBD web to fully utilize the hardware resources.


OCPBUGS-31536:OCPBUGS-33554: Fix mirroring operators on fedora systems - no sig verification

Description
2 issues are fixed here:

catalog images are mirrored as manifest lists
catalog images are mirrored without signature verification by default
to enable signature verification, one can use the new flag --enable-operator-secure-policy
existing flag oci-insecure-signature-policy marked deprecated

Fixes # OCPBUGS-31536
Type of change
Please delete options that are not relevant.

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

How Has This Been Tested?
$ ./bin/oc-mirror -c isc_31536.yaml file:///home/skhoury/31536 --enable-operator-secure-policy

Expected Outcome
Above test is no proof of fix: this needs to be tested on a fedora system

with no /etc/containers/policy.json
AND without using the flag
to show that the mirroring is successful on systems that don't have the policy setup for checking image signatures from redhat.

/label qe-approved

/label acknowledge-critical-fixes-only
/lgtm


Reduce & Merge Activity contents

Currently there are very much fragments and activity in here, containing a lot of code duplicates. E.g. there is no need to have Acitivy_hideapps.java in here. Should all be integrated in one.
Of course, e.g. settings need to have it's own activity/fragment. And maybe some kind of common BaseActivity/Fragment. I tried to understand whats going on, but thats really hard to currenty, because it's hard to guess what gets really called/executed.
This should be one of our top priority issues, before integrating any new feature. Else this gets more bad with each new stuff, and at a certain point the project will become unmaintainable if not done.

Alright, the activities we can probably modify first are MinbarEdit, About, and HideApps. All three will need custom layouts first and then we will need to modify the settings activity to open the new fragments. LauncherActions will also need modifications to support the MinbarEdit fragment. Normally MinbarEdit will be attached to the settings, but maybe the launcher action should use a base activity instead of the settings.

I also don't know how much important the launcheranim, materialpreffragment, smoothviewpager libraries are, and if it's important to keep them as seperate lib. I cannot see much code and stuff in there, I think they can be merged into the app. This will reduce build and maintaining overhead, e.g. we don't need to update library versions double and twice

@BennyKok wanted to keep the materialpreffragment separate but the other two can be merged. Also, I think it might clean up the issues page if we tag duplicates, reference the main issue, and then close them to better organize the page, what do you think?

I would like to extract materialpreffragment later to a new repository as a standalone library. And as for the launcheranim, I have used the vector drawable flag in the gradle file so we need to be careful when merging this one.

Honestly, I dislike the materialpreffrag. Its quite a hacky solution, with everything done in code. which is quite unmaintainable. Android already supplys very good stuff here, no need to use a workaround library here.
Can easily done in XML:
https://github.com/Diaspora-for-Android/dandelion/blob/master/app/src/main/res/xml/preferences__master.xml
and very small piece of code: https://github.com/Diaspora-for-Android/dandelion/blob/master/app/src/main/java/com/github/dfa/diaspora_android/activity/SettingsActivity.java.
And all of this also works in older versions + is material too + easy manageable strings, resources :).

Because I use Gson for the entire settings and even desktop and dock data, so I created a custom workaround for me to easier inflate the settings to the interface. Why I did the whole thing because of I don't like switching between xml code and java at the same time need to pull the settings out from the SharedPreferences for each variable, using the Gson I can easily define list and array and custom object to store to the device, which is very easy in a lazy way. It is true "easy manageable", "unmaintainable", but that is the way I got started, I am fine with a total rewrite and switch to the traditional way.
Each Materialpreffrag has it own scrollview.
Great job man, a deep analyzing report. But before we ditch the Materialpreffrag out, we need to take care of the existing LauncherSettings.

@BennyKok Good a idea with JSON, I think e.g. launcher positions&arrays can't be managed any better

I think first I will clean up the Home activity before I go any further (and fix some bugs along the way)

@gsantner are u still using xmpp, I found you id in you website, did you got my msg in xmpp

@BennyKok Nope, I didn't receive a msg, but a contact request now, from "itechbenny", is that you?

yes


tox.ini: Syntax check files in cli/

Cc: @tamarrow

Can one of the admins verify this patch?

Can one of the admins verify this patch?


run the target binary in Ubuntu

Hi, how do I run the complied linux in Ubuntu? I tried "./complied_binary". This does not work

Try sudo ./complied_binary!
I don't know why but for me the files that are generated to linux only work when using sudo

@hkbluesky
try it
chmod -x complied_binary
sudo ./complied_binary


Attachment file type is not allowed

Gitea version (or commit ref): 1.8.1
Git version: 2.7.4
Operating system: Linux
Database (use [x]):

[ ] PostgreSQL
[x] MySQL
[ ] MSSQL
[ ] SQLite

Can you reproduce the bug at https://try.gitea.io:

[ ] Yes (provide example URL)
[ ] No
[ ] Not relevant

Log gist:

Description
I've the following config for the attachments
[attachment]
ENABLED       = true
PATH          = /var/lib/gitea/data/attachments/
MAX_SIZE      = 4
MAX_FILES     = 5
ALLOWED_TYPES = image/jpeg|image/png|application/zip|application/gzip

However, when uploading a zip file it says "File type is not allowed".
Screenshots

Hi pditommaso,
please try to add:

application/octet-stream,
application/x-zip-compressed
multipart/x-zip

Tried this, still no joy:
ALLOWED_TYPES = image/jpeg|image/png|application/zip|application/gzip|application/octet-stream|application/x-zip-compressed|multipart/x-zip

File input allows specifying types by extension too, try adding the .zip extension directly:
https://developer.mozilla.org/en-US/docs/Web/HTML/Element/input/file#accept

If above does not work, we can probably add support by it by passing the value of attachment.ALLOWED_TYPES directly to the <input>, replacing | with , in process.


Add support for Arch Linux

Add support for Arch Linux package manager and update tests.

Merged with minor fix for flake8 style tests.
Thanks!

Awesome! And thank you for maintaining testinfra!


docs: added a page for portusctl

Fixes #868
Signed-off-by: Miquel Sabaté Solà<EMAIL_ADDRESS>

lgtm

LGTM

Note to self: I should also note that the --local-registry option doesn't do anything onto the DB (from #711). I'll update this PR later :smile:


Graph API

I am not able to update the Nuget package for Microsoft Graph. I get:
Severity	Code	Description	Project	File	Line	Suppression State
Error		Could not install package 'System.Runtime.InteropServices.RuntimeInformation 4.0.0'. You are trying to install this package into a project that targets '.NETPortable,Version=v4.5,Profile=Profile7', but the package does not contain any assembly references or content files that are compatible with that framework. For more information, contact the package author.			0
I am not sure what the resolution is. Any ideas? Thanks!

Just to clarify: you're trying to install v 1.3.0 of the Microsoft Graph SDK? I was just able to reproduce this. I'll look into it and let you know what I discover.

I am trying to Update to v1.3.0 in Nuget. Thank you sir.

I've looked into this a bit, and here is what I know. First, it looks like you need to install this package to a Xamarin solution in order to update to v 1.3.0 of the Microsoft Graph SDK:
NETStandard.Library v 1.6.1
I should let you know, though, that I've tested this sample with v 1.3.0 of the Graph SDK and discovered a bug that breaks the call that creates the sharing link. I would, therefore, advise against upgrading the Graph SDK right now (for this sample, at least).
I should also let you know that I'm working on a set of updates that will make the work-around I describe above unnecessary. I'll be updating the MSAL library, and that change will install the NETStandardLibrary that appears to be a dependency for the Graph upgrade.
Does this information help you? At the moment, my advice is to wait both for an updated Graph SDK and for my coming updates to this sample.

I've checked in changes that update the MSAL library used in the sample. This library has a dependency on NETStandard.Library v1.6.1, so that library is now in the sample and you shouldn't have to add it. You should be able to update the version of the Microsoft Graph library, but as I said, I'd advised against it for now because of the sharing link bug.

That helps. Thank you for your attention to this. I will await the next version.


Bug plotting root functions

Hi,
Plotting f(x) = x^(1/2) or f(x) = x^(1/3) does not produce the correct graph. For f(x) = x^(1/2) there are no solutions for minus x but they are plotted. Also for f(x) = x^(1/3) all solutions for x below zero should be negative, not positive.

Function-plot - f(x) = x^(1/2)

Correct - f(x) = x^(1/2)

Function-plot - f(x) = x^(1/3)

Correct - f(x) = x^(1/3)

Thanks for the report! I'll solve it on the interval arithmetic package https://github.com/maurizzzio/interval-arithmetic/issues/3

Identifying if the exponent plots something in the negative x-axis is quite a challenging problem, I've seen that supercalc (google's plot) actually analyzes the expression to be evaluated and sees if the denominator is even to plot stuff in the negative x-axis (e.g. y = x^(1 / 3)) however since this is an static analysis it doesn't handle cases like y = x^(1 / 3 + 0)
Doing such a complex static analysis of the expression's ast is quite complex, so I've created a function in interval arithmetic called nthRoot (also available in math.js), what you can do to plot roots is use that function instead
e.g. y = x^(2/3) needs to be plotted with nthRoot
functionPlot({
  target: '#root-finding',
  data: [{
    fn: 'nthRoot(x, 3)^2'
  }]
})


Revert "Turn back on macOS shard Cirrus caching"

This reverts part of commit e002698c3511166a2bf85a39a2c084eb5f03f98d.
Description
Due to https://github.com/cirruslabs/cirrus-ci-docs/issues/566 cache population is causing tasks to run forever when there's a mysterious hang.
Related Issues
Reverts turning on macOS Cirrus caching in https://github.com/flutter/flutter/pull/50496.
Keep the --offline flag.

Landing without waiting for tests, this is causing havoc in other PRs.


MFW-458: Add Rules List example for Magic Firewall

This reads to me as more of a procedure/how to use account-level rules lists, so I'd suggest adding this under the How to section. Content looks good overall though!

This reads to me as more of a procedure/how to use account-level rules lists, so I'd suggest adding this under the How to section. Content looks good overall though!

@dcpena addressed comments


Key error when loading the embedding

Hello, I encountered an error when loading gtr_wikipedia_index.pkl. It only seems to contain the Git LFS details
version https://git-lfs.github.com/spec/v1
oid sha256:211b64c73c2c1fc26371b17d30a31e63017d1ca8155b70a3bc464d59a430decc
size<PHONE_NUMBER>3

I have tried getting the original archive with oid and size. However, I think this is only possible when these details are within a repository.
As constructing the dense embeddings may be too expensive considering the resources that I have, can you kindly take a look at this issue?
Thank you for the help

Hi @HCY123902, thank you for finding this bug. I'm currently looking into the problem and will update you as soon as I fix it :)

Thank you for the help. I think it works now

Just another comment related to sparse retrieval with BM25, the h.raw attribute on line 37 does not exist in the h.

When I run retrieval.py --retriever bm25, there is a corresponding error message

An example dict of a hitted passage is {'id': '7281f52d-a82e-11eb-8e33-e778e9b30943', 'url': 'https://www.elitedaily.com/news/politics/third-party-candidates-huge-impact/1564095', 'title': 'This Is How Third-Party Candidates Hugely Impact Elections Even When They Lose', 'sha': 'sha1:VSJGRHYGFDS3IH7IMIHEDQWXR5QB3HNY'}, which does not contain the raw key.
Did you take any extra steps to process the sparse index before running the retrieval script? I assume that getting the raw passages will take a huge space given that the sparse index already takes around 800 GB after compression

Hi @HCY123902 it should be hit.raw instead of h.raw. The hit object from LuceneSearcher has the raw attribute. I have updated the code to reflect the change. Thanks for finding the bug :)


Gen 2 - Amplify Outputs type issues after adding groups

Environment information
System:
  OS: macOS 15.1.1
  CPU: (8) arm64 Apple M1
  Memory: 161.67 MB / 8.00 GB
  Shell: /bin/zsh
Binaries:
  Node: 20.15.1 - ~/.nvm/versions/node/v20.15.1/bin/node
  Yarn: 1.22.22 - ~/.nvm/versions/node/v20.15.1/bin/yarn
  npm: 10.8.3 - ~/.nvm/versions/node/v20.15.1/bin/npm
  pnpm: undefined - undefined
NPM Packages:
  @aws-amplify/auth-construct: 1.5.0
  @aws-amplify/backend: 1.8.0
  @aws-amplify/backend-auth: 1.4.1
  @aws-amplify/backend-cli: 1.4.2
  @aws-amplify/backend-data: 1.2.1
  @aws-amplify/backend-deployer: 1.1.10
  @aws-amplify/backend-function: 1.8.0
  @aws-amplify/backend-output-schemas: 1.4.0
  @aws-amplify/backend-output-storage: 1.1.3
  @aws-amplify/backend-secret: 1.1.5
  @aws-amplify/backend-storage: 1.2.3
  @aws-amplify/cli-core: 1.2.0
  @aws-amplify/client-config: 1.5.2
  @aws-amplify/deployed-backend-client: 1.4.2
  @aws-amplify/form-generator: 1.0.3
  @aws-amplify/model-generator: 1.0.9
  @aws-amplify/platform-core: 1.2.2
  @aws-amplify/plugin-types: 1.5.0
  @aws-amplify/sandbox: 1.2.6
  @aws-amplify/schema-generator: 1.2.5
  aws-amplify: 6.10.0
  aws-cdk: 2.171.1
  aws-cdk-lib: 2.171.1
  typescript: 5.6.3
No AWS environment variables
No CDK environment variables

Describe the bug
Hello,
This is my first ever AWS project, so I'm not exactly certain this is a real issue, even though I can't find a solution to this problem.
I'm building a small website with both an admin and a user dashboard, so I decided to add groups. So I followed this documentation and added 2 groups, 'ADMIN' and 'USER' to my amplify/auth/resource.ts file like so:
export const auth = defineAuth({
    loginWith: {
        email: { ... },
    },
    groups: ['ADMIN', 'USER'],
    triggers: {
        postConfirmation,
    },
    access: allow => [
        allow.resource(postConfirmation).to(['addUserToGroup']),
    ],
    userAttributes: {
        'custom:role': {
            dataType: 'String',
            mutable: true,
        },
    },
})

I also followed this documentation to automate the group adding on registration process. It works well, I just set 'USER' as the group a user will join.
But when trying to build the project, I got the following errors:
Argument of type '{ auth: { user_pool_id: string; aws_region: string; user_pool_client_id: string; identity_pool_id: string; mfa_methods: never[]; standard_required_attributes: string[]; username_attributes: string[]; ... 4 more ...; unauthenticated_identities_enabled: boolean; }; data: { ...; }; storage: { ...; }; version: string; }' is not assignable to parameter of type 'ResourcesConfig | LegacyConfig | AmplifyOutputs'.
  Type '{ auth: { user_pool_id: string; aws_region: string; user_pool_client_id: string; identity_pool_id: string; mfa_methods: never[]; standard_required_attributes: string[]; username_attributes: string[]; ... 4 more ...; unauthenticated_identities_enabled: boolean; }; data: { ...; }; storage: { ...; }; version: string; }' is not assignable to type 'AmplifyOutputs'.
    The types of 'auth.groups' are incompatible between these types.
      Type '({ ADMIN: { precedence: number; }; USER?: undefined; } | { USER: { precedence: number; }; ADMIN?: undefined; })[]' is not assignable to type 'Record<string, UserGroupPrecedence>[]'.
        Type '{ ADMIN: { precedence: number; }; USER?: undefined; } | { USER: { precedence: number; }; ADMIN?: undefined; }' is not assignable to type 'Record<string, UserGroupPrecedence>'.
          Type '{ ADMIN: { precedence: number; }; USER?: undefined; }' is not assignable to type 'Record<string, UserGroupPrecedence>'.
            Property '"USER"' is incompatible with index signature.
              Type 'undefined' is not assignable to type 'UserGroupPrecedence'.

15 Amplify.configure(outputs)

The relevant part of the auto generated amplify_outputs looks like so:
{
  "auth": {
    // ...
    "groups": [
      {
        "ADMIN": {
          "precedence": 0
        }
      },
      {
        "USER": {
          "precedence": 1
        }
      }
    ]
  }
}

Is there anything I didn't do correctly or at all? The auto generated output seems good to me.
Thank you in advance
Reproduction steps
"Simply" add the following to the amplify/auth/resource.ts file:
groups: ['ADMIN' 'USER'],

And wait for the sandbox to regenerate.

Hey @Drillan767, thank you for reaching out. Transferring the issue over to our Amplify Js repository for further investigation.


By clicking on highlighted WS user, the checkbox is not selected or opening the details of the member

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

Version Number: 1.4.49-0
Reproducible in staging?: y
Reproducible in production?: no, new feature
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Expensify/Expensify Issue URL:
Issue reported by: applause internal team
Slack conversation:
Action Performed:

Access staging.new.expensify.com
Sign into an account
Create a workspace
Go to WS settings and add members
Tap on the users highlighted name

Expected Result:
User expects the checkbox to highlight when the user selects the name, or a redirect
Actual Result:
The checkbox does not highlight when the user clicks on the members name, clicking on the name "highlights" as if something should happen, but nothing does.
NOTE : In production checkbox getting selected
Workaround:
unknown
Platforms:

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

[x] Android: Native
[x] Android: mWeb Chrome
[x] iOS: Native
[x] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[x] MacOS: Desktop

Screenshots/Videos
Add any screenshot/video evidence
https://github.com/Expensify/App/assets/43996225/b076c42e-41eb-402d-8b8f-fa9ce8a2b8c7

View all open jobs on GitHub

@marcaaron FYI I haven't added the External label as I wasn't 100% sure about this issue. Please take a look and add the label if you agree it's a bug and can be handled by external contributors.

I think it is related to #37715 and will fix in a follow up issue #37927.
to fix this we need to navigate to user profile on item click and keep select by checkbox only by add Navigation.navigate(ROUTES.PROFILE.getRoute(item.accountID)); before return here

cc @luacmartins @burczu

@ahmedGaber93 I don't see this issue in that list so assuming this is a blocker unless @luacmartins says otherwise.

@marcaaron
I think it is the No. 1

Navigate to global profile if not an admin of paid policy. We shouldn't just early return here, instead let's navigate the admin of a free policy to the global profile, like we do above.

Yes, that's the first issue. The policy in the test video is a free policy (we can tell by the pages that are visible), so we just have an early return there. I can work on a quick fix.

This is done and CPed

Tested in staging, it's a pass!

This feature is still a WIP so I don't think we need to worry about the checklist above now. We're addressing all these issues in follow ups.


$MODS_OVERRIDES doesn't seem to be set correctly

I'm using $MODS_OVERRIDES to set custom mods configurations, but it seems that the script, which creates the modoverrides.lua file, ignores it or can't read it or whatever.
It seems like this line fails:
https://github.com/dst-academy/server/blob/develop/build/static/boot/mods.sh#L18
When actually printing the ENV var inside the container, it's available and holds the correct value.
The actual value is:
return {
	['workshop-378160973'] = { enabled = true, -- Global Positions
		configuration_options = {
			SHOWPLAYERSOPTIONS = 2,
			FIREOPTIONS = 2,
			OVERRIDEMODE = false,
			SHAREMINIMAPPROGRESS = false,
			SHOWFIREICONS = true,
			SHOWPLAYERICONS = true,
		},
	},
	['workshop-360823660'] = { enabled = true, -- Simple Protection
		configuration_options = {
			backpack_protection = true,
			bushes_dig_protection = true,
			bushes_protection = true,
			container_protection = true,
			fire_spread_protection = false,
			ghost_protection = false,
			min_days = 3,
			starting_day = 0,
			stewer_protection = true,
		},
	},
	['workshop-519266302'] = { enabled = true, -- Season Starting Items
		configuration_options={
			is_season_items = 1,
			food_give = 1,
			num_cutgrass = 2,
			num_twigs = 2,
			num_flint = 2,
			num_goldnugget = 0,
			num_log = 0,
			num_rocks = 0,
		},
	},
	['workshop-375850593'] = { enabled = true }, -- Extra Equip Slots
	['workshop-599498678'] = { enabled = true }, -- Tools are Fuels
	['workshop-503919639'] = { enabled = true }, -- Equipment Switcher
}

When looking into the generated file, this is what I see:
return {
  ["workshop-378160973"] = { enabled = true },
  ["workshop-375850593"] = { enabled = true },
  ["workshop-599498678"] = { enabled = true },
  ["workshop-503919639"] = { enabled = true },
  ["workshop-360823660"] = { enabled = true },
  ["workshop-519266302"] = { enabled = true },
}

This looks like it's executing this lines:
https://github.com/dst-academy/server/blob/develop/build/static/boot/mods.sh#L20-L28
Not sure why this happens.

Nevermind, seems to be a weird issue running docker/docker-machine on Windows using a VirtualBox guest.


feat(dashboard): nullify hardcoded id

When id is hardcoded in a GrafanaDashboard, the operator is likely to fail to create the dashboard in a Grafana instance.
The PR makes sure id is nullified before dashboard model is passed to Grafana client.
(Strictly speaking, id should never be defined by a user in the first place, so it's more around improving UX.)
More details
Basically, when id is set, Grafana tries to update a dashboard by the id, not uid. And, expectedly, that can cause various issues, e.g. if the id is not known to Grafana, Grafana responds with 404.
Example of a failing CR
apiVersion: grafana.integreatly.org/v1beta1
kind: GrafanaDashboard
metadata:
  name: grafanadashboard-sample
spec:
  resyncPeriod: 30s
  instanceSelector:
    matchLabels:
      dashboards: "grafana"
  json: >
    {
      "id": 146,
      "title": "Simple Dashboard",
      "tags": [],
      "style": "dark",
      "timezone": "browser",
      "editable": true,
      "hideControls": false,
      "graphTooltip": 1,
      "panels": [],
      "time": {
        "from": "now-6h",
        "to": "now"
      },
      "timepicker": {
        "time_options": [],
        "refresh_intervals": []
      },
      "templating": {
        "list": []
      },
      "annotations": {
        "list": []
      },
      "refresh": "5s",
      "schemaVersion": 17,
      "version": 0,
      "links": []
    }

Fixes: #1108

Will there be a patch release soon for this fix? Currently looking to migrate form v4 -> v5 and this is a little bit of regression.  I know v5 is a major release. Its just a shame Grafana does not  by default exclude this id when exporting dashboards.

@owenhaynes it's now part of v5.0.2 that got released today :)


Create a CLI archetype for quickstart

We need to support it in the CLI

Fixed as part of https://github.com/oracle/helidon/pull/2035


How to retrieve transfer details?

So when you call the following withdrawal endpoint 'withdrawals/crypto' you will get an id in the response. My guesses are that, that is the transfer id.
My question is, once you have the transfer id how can you get it's details? (E.g. Estimated Payout and Transaction Hash , etc.)
Once a withdrawal occurs I could use the transaction hash to follow it's confirmations on the blockchain, if it has 3 or more confirmations I can assume the withdrawal is complete. However it would even be better if I could call an API to poll if a withdrawal has been completed.

Right now this isn't possible, but I agree the feature would make a nice addition. Stay tuned :)

:+1: I'd find this feature useful as well.


Add mobile robot kinematics 101 and improve steering library docs

This is the first step of #931, but I focused only on steering on the front axle for now.
Edit: I also added the diff-drive kinematics to have a complete documentation.
After peer-reviewing of the equations I'd start to rename the variables of the library to match the doc and fix possible bugs. E.g., if I'm not wrong:
Wrong IK in case of traction on the back wheels?
https://github.com/ros-controls/ros2_controllers/blob/dcd15ee6bac003aecd668943f3d4c93c1e0ee171/steering_controllers_library/src/steering_odometry.cpp#L226
Wrong odometry?
https://github.com/ros-controls/ros2_controllers/blob/dcd15ee6bac003aecd668943f3d4c93c1e0ee171/steering_controllers_library/src/steering_odometry.cpp#L108-L109
https://github.com/ros-controls/ros2_controllers/blob/dcd15ee6bac003aecd668943f3d4c93c1e0ee171/steering_controllers_library/src/steering_odometry.cpp#L134-L136
https://github.com/ros-controls/ros2_controllers/blob/dcd15ee6bac003aecd668943f3d4c93c1e0ee171/steering_controllers_library/src/steering_odometry.cpp#L156-L157
https://github.com/ros-controls/ros2_controllers/blob/dcd15ee6bac003aecd668943f3d4c93c1e0ee171/steering_controllers_library/src/steering_odometry.cpp#L169-L171
rendered docs

:eyes: @aleksandrsizmailovs @roncapat @franzrammerstorfer @petkovich

Ad odometry of ackermann steering: There is a not negligible error imho. (average is the equation implemented in the library, plotted for l = 1, w = 1; the equation from above gives the correct angle phi)

Impressive job! I think this is very valuable. At least for now everything seems ok to me, I would maybe suggest to be more explicit in the derivation of some formulas, maybe with collapsable sections.
For example, for the unicycle:
 this implies at least one or two intermediate steps and to make the description more appealing to readers I may tend to add detail (so the attention of the reader is shifted from understanding the formulas to trusting them) even if sometimes it seems trivial :)
The point is... there are not so many places on the internet where you can find a clean explanation. This can become one of these 👍🏻
About the "overdetermined" odometry solution of ackermann with traction, I would start to think about a set of recommendations or methods to find the best solution.

Impressive job! I think this is very valuable. At least for now everything seems ok to me, I would maybe suggest to be more explicit in the derivation of some formulas, maybe with collapsable sections.
For example, for the unicycle:  this implies at least one or two intermediate steps and to make the description more appealing to readers I may tend to add detail (so the attention of the reader is shifted from understanding the formulas to trusting them) even if sometimes it seems trivial :)
The point is... there are not so many places on the internet where you can find a clean explanation. This can become one of these 👍🏻

Thanks! I was not sure where to start and where to end. I think about it.

About the "overdetermined" odometry solution of ackermann with traction, I would start to think about a set of recommendations or methods to find the best solution.

We should at least assume, that ever actuated joint has a feedback sensor (e.g., no combination of traction joints on the rear axle but speed sensors on the front axle) -> this would get very complicated to parameterize for every combination.
But as far as I understood: the claim of this lib at the current state was bicycle, tricycle, ackermann; all with front or back steering; and with the traction axle being the other than the steering axle? (the steering+traction ackermann description is not part of this then, but part of this proposal of a "general" library https://github.com/ros-controls/ros2_controllers/issues/692#issuecomment-1749748573)
This is the first real difference to the tricycle_controller for #850, which expects the steering wheel be the only traction wheel, and the two behind are just following.

Am I right that for a trailing steering axis (front_steering=false) one has just to use a negative value for the wheel-base l, and that's it?

@petkovich this may be interesting to you

Very nice @christophfroehlich this looks like a very good documentation page! Nice work!
Looks alright to me.
IMHO I would finish and merge this PR.
The documentation page will then be available to the public and more sets of eyes can look at it and possibly spot errors or recommend improvements?

Very nice @christophfroehlich this looks like a very good documentation page! Nice work! Looks alright to me. IMHO I would finish and merge this PR. The documentation page will then be available to the public and more sets of eyes can look at it and possibly spot errors or recommend improvements?

Thanks! Would you mind adding a review in the UI? (Below "Files Changed" tab -> Review Changes). Yes, this will be published on control.ros.org.


Using UTF16 characters messes up indexOf function

When we have a string with UTF16 characters, indexOf returns -1 when the searched substring is at the very end of the string.
Code to reproduce:
package main

import (
	"github.com/dop251/goja"
)

func main() {
	vm := goja.New()
	// here we have a space after "END", works properly
	v, err := vm.RunString("'case 字 END '.indexOf('END');")
	if err != nil {
		panic(err)
	}

	if num := v.String(); num != "7" {
		panic("First check, index should be 7 but it is: " + num)
	}
	// here we don't have a space after "END", doesn't find the string
	v, err = vm.RunString("'case 字 END'.indexOf('END');")
	if err != nil {
		panic(err)
	}

	if num := v.String(); num != "7" {
		panic("Second check, index should be 7 but it is: " + num)
	}
}

Output: panic: Second check, index should be 7 but it is: -1
Actual behavior:
Returning -1
Expected behavior:
Return right location (7)
Might be related to https://github.com/dop251/goja/issues/59 ?

Hi @hirshbergshachar,
Are you sure you're using the latest version, it's not vendor locked or something? 'case 字 END'.indexOf('END') returns 7 for me.

@dop251 you are right checked with the latest version and it doesn't occur there. Will update, thank you for your time.


snappify LuCI "Save & Apply" button

When applying changes in the interface, the first check if changes were applied happens after 5 seconds. Can we lower that since changes are likely applied within split seconds? In case it's a more complex change, a recheck can happen again after x seconds, like it's now.
I'd propose a recheck pattern of like second 1,2,5,10,15...

Duplicate of your #4881 ?
Same answer again as I gave there a month ago:
Trying to remember the original discussions from 2018.
I think that it was originally shorter and that caused problems with some apps, so it was made a bit longer (and configurable)
Discussion e.g. here in the original topic about the "new" rollback: #1769 (comment)

Thanks!


Added a script to help create a default ceph storageclass

@stacyped ^^

The yum upgrade will only work on certain OS.

The yum upgrade will only work on certain OS.

Noticed these when running:
====> Updating OS. This might take a spell
<==== Yum update did not succeed

./rook-ceph-setup.sh: line 76: determine_system_arch: command not found
====> Checking if git is installed
<==== git is installed
./rook-ceph-setup.sh: line 91: login_oc_cmdline: command not found

Noticed these when running:
====> Updating OS. This might take a spell
<==== Yum update did not succeed

./rook-ceph-setup.sh: line 76: determine_system_arch: command not found
====> Checking if git is installed
<==== git is installed
./rook-ceph-setup.sh: line 91: login_oc_cmdline: command not found

Please let me know once changes have been made and it is oK to merge

Please let me know once changes have been made and it is oK to merge

thanks @stacyped and @taylormgeorge91 , let me review and update later

thanks @stacyped and @taylormgeorge91 , let me review and update later

we will not need this one, but will use some yaml templates to deploy a default storageclass for PoC.

we will not need this one, but will use some yaml templates to deploy a default storageclass for PoC.


0xC0000005: Access violation reading location 0xFFFFFFFF.

cocos2d-x version:
devices test on:
developing environments

NDK version:
Xcode version:
VS version:
browser type and version:

Steps to Reproduce:

add SSE check at cocos/CMakeLists.txt

check_cxx_compiler_flag("/arch:SSE2" HAVE_ARCH_SSE2)
if(HAVE_ARCH_SSE2)
    target_compile_definitions(cocos2d PUBLIC __SSE__=1)
    message(STATUS "Building with SSE2")
endif()

run cpp-empty-test

Root Cause: caused by address alignment issue, everything must be 16byte boundary when pass to sse instructions functions _mm_xxx


spec: move specs to tendermint/spec repo

We want to consolidate all spec work at the tendermint/spec repo so that it becomes a central access point for specs

All active work on specs is now happening at tendermint/spec. The plan was to delete the specs here, or more precisely, keep empty files with links to the new location (as we referenced the files in our recent publications). However, for the model-based testing, @andrey-kuprianov needs the TLA+ specs within the tendermint-rs repository. We should figure out how to proceed here.

In fact, for MBT I have copied and modified the spec files I need into the LightClient crate. It's safe to remove the old spec files now

Closing for https://github.com/informalsystems/tendermint-rs/issues/601


bump up Go version for Docker builds

This fixes build error:
github.com/intel/intel-device-plugins-for-kubernetes/internal/deviceplugin
../../internal/deviceplugin/server.go:186: cannot use srv (type *server) as type v1beta1.DevicePluginServer in argument to v1beta1.RegisterDevicePluginServer:
	*server does not implement v1beta1.DevicePluginServer (wrong type for Allocate method)
		have Allocate("context".Context, *v1beta1.AllocateRequest) (*v1beta1.AllocateResponse, error)
		want Allocate("github.com/intel/intel-device-plugins-for-kubernetes/vendor/golang.org/x/net/context".Context, *v1beta1.AllocateRequest) (*v1beta1.AllocateResponse, error)

Codecov Report

Merging #65 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master      #65   +/-   ##
=======================================
  Coverage   77.99%   77.99%
=======================================
  Files           8        8
  Lines         936      936
=======================================
  Hits          730      730
  Misses        189      189
  Partials       17       17

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 f8c3e9a...fac7971. Read the comment docs.


Compacting upper section while writing email

Describe the bug
Half of screen is taken by keyboard and the upper section cannot be rolled up which leave you only with one line of space visible.
To Reproduce
Steps to reproduce the behavior:

Go to 'compose email'
Switch to 'section for writing email content'
What's happening? 'there is only one line visible in composing section'
See display layout on screenshot attached

Expected behavior
Be able to compact upper area of email while writing email content.
Screenshots
Added screenshot to help explain the problem.

Thanks

@mezysinc The bug has been fixed and will be available in the next release. Now via scroll, you can use more area for writing the letter. Thanks for your reporting!

@mezysinc The bug has been fixed and will be available in the next release. Now via scroll, you can use more area for writing the letter. Thanks for your reporting!


Tooltip clipped if shown inside scrollable container

As per this screenshot:

To reproduce place the container inside a div with overflow-x: auto and a fixed width.

The normal solution to this problem is to make the tool-tip a child of some other element (e.g. the body) and absolutely position the tool-tip in the correct position.
This means JavaScript is needed - for backwards compatibility we can have the JavaScript modify the existing HTML rather than generate the tool-tip from scratch, meaning if JavaScript isn't present they will still work as they currently do.


Added test to verify that pointer is valid for first frame after hand appears - issue 5053

Overview
This PR addresses issue #5053.
This test checks whether the pointer is valid for the first frame.  It also checks if the pointer direction is facing the same general direction as camera forward.
Video of Pointer Test
Changes

Added PointerTests.cs to Assets/MixedRealityToolkit.Tests/PlayModeTests/

Verification

In Unity: Window -> General -> TestRunner
Select PlayMode at the top of the window
Select PointerTests and then Run Selected

/azp run

@julenka do you have a suggestion for the test name/file name/location? I see that you've added some pointer tests, and there are a bunch of pointer related tests now in the root which seems not optimal.

@julenka do you have a suggestion for the test name/file name/location? I see that you've added some pointer tests, and there are a bunch of pointer related tests now in the root which seems not optimal.

For now PointerTests is fine. We have some inconsistency in how tests are organized. For example, most folders contain things like prefabs, components, except for core/providers/hands/BaseHandVisualizerTests. We should probably fix that in a separate change.
I think PointerTests is fine -- it generally tests things about pointers, but not pointer events.

/azp run

/azp run


Running in PR mode

Thank you very much for quick implementation of branch name patterns. I'm running into one more issue. I would like to add Jira comments for each build  triggered in Pull Request mode but the response is:
WARN[13:44:44] The step's<EMAIL_ADDRESS>Run-If expression evaluated to false - skipping
INFO[13:44:44] The Run-If expression was: .IsCI | and (not .IsPR)

Would it be possible for you to add this functionality?
best
Kris

If you enable it for pull requests you might end up with "duplicate" messages. Are you sure you want to have this ?

What do you mean by "duplicate" messages? We have separate workflow for PRs and we need to update information in Jira with current build public instal page. I think it's better to do this once when PR is opened instead of posting every time there is a push to feature branch.

Hey @lycha
I don't have the moment right now but feel free to make a PR.
We should somehow give the choice to run for PR or not to avoid to introduce a BC break.

Hey @dagio. I've opened PR with possible solution.

Closing for now. @maciejmiarecki @lycha feel free to reopen if you are still looking for that feature


Please add a license

This looks like a great library, but I can't legally use it unless you add a license. A really popular and easy to understand is the MIT license, but you can choose any one you want.
Not adding a license isn't being more lenient, it is actually completely locking your library from being used publicly.

Hi! Thanks for the heads up, I just included a simple demo app and the LICENSE file.


[question] where to draw the line with state

I love this concept but I have some questions around where you envision the line being drawn with actions and state.
For instance, I know that in a UITabBarController example you override the default behavior and handle the event manually. How does something like this translate to a UISwitch? Is it overkill for this example and better to just receive the change and then post and action that the state of a switch changed?
Additionally there are lots of things in the UI realm, for instance scroll events on a tableview. In order to FULLY capture the state change these would be tracked but unless it is relevant to a given application this also seems like overkill.
Thoughts?
Thanks and great work!

We had a very similar question on another repo in this org: https://github.com/ReSwift/CounterExample-Navigation-TimeTravel/issues/1
Check it out and let me know if it answers your question :)

Ha, that is almost exactly the same question. That answers most of my questions but i'd love to get your opinion something like a slider or a switch.
For example you can register a selector to listen for when the slider is done changing, or a switch's value has changed. This could emit an action to update the store which in turn would emit a "store updated" event.
For the case of the switch would the flow be like this: switch changed by user -> update store action -> store emits change -> view controller re-sets the switch with the same value it already has?

For the case of the switch would the flow be like this: switch changed by user -> update store action -> store emits change -> view controller re-sets the switch with the same value it already has?

You will definitely run into a few cases where you need to force UIKit to work nicely with ReSwift.
IMO what you suggest would be fine for the switch! Ideally you would stop the UIKit component from updating directly (e.g. like I do with UITabBarController, by setting up a delegate that doesn't allow the controller to update), but in many cases that won't be possible and there your proposed solution would be the way to go.

Thanks for the advice! I look forward to seeing how this evolves.


Plugin updated with skpm won't install

Plugin in question:
https://github.com/larsenwork/sketch-easing-gradient
Steps
1. Successfully run `skpm publish patch` in the branch
2. Sketch picks up the update
3. Click the `Update` button but nothing happens

Screenshot

It's a bug in Sketch, I filled an issue in our repo so I can track it there (BohemianCoding/Sketch#18523)

Actually no it's not a bug in Sketch. You specify the version in your manifest.json so skpm doesn't use the one specified in the package.json. That means that your published version 0.4.2 is actually still 0.4.0 so Sketch cannot find the updated plugin.


Allow to instrument bluebird multiple times (with different namespaces)

Issue description
From instana >= 1.65.0, you cannot longer use cls-bluebird to instrument (the currently require-able) version of bluebird with any cls namespace if the instana sensor was required before the instrumentation process.
Possible cause
The following features, when combined are causing the issue:

From instana >=1.65.0, the package started registering a require hook which is currently used to ensure all bluebird versions get patched properly by using cls-bluebird.

The cls-bluebird package is used to instrument the currently require-able bluebird version (by not specifying the bluebird instance to patch) with any cls namespace.

When developers combine both features, (I think) the following happens:

Instana registers a require hook, intercepting all requires to bluebird.
Developers try to instrument bluebird with a cls namespace without specifying a bluebird instance
cls-bluebird tries to require bluebird to instrument it
Since the require hook is already registered, it will intercept the bluebird require call
Instana require hook will try to instrument the intercepted bluebird call by calling cls-bluebird again (this time with the intercepted bluebird instance)
We have now a recusive call to cls-bluebird. As described by the Node.js modules Cycle documentation, Node.js deals with this scenario by providing a temporal incomplete export value (an empty object in this case) to the module causing the cycle call.
The module causing the cycle call is the instana require hook, which tries to immediately call the cls-bluebird's require result, which under normal conditions is a function
The function call fails, because the required value is not a function (yet), it's an object

Additional information
As described in the test cases provided below, you will get an error stating that bluebird cannot be required. That is because cls-bluebird ignores the underlying error. Print that error, to get the underlying error:
TypeError: require(...) is not a function
    at patchBluebird (/Users/jsamines/dev/personal/cls-bluebird-with-recursion/instana-master/node_modules/@instana/core/src/tracing/instrumentation/control_flow/bluebird.js:19:26)
    at Function.patchedModuleLoad [as _load] (/Users/jsamines/dev/personal/cls-bluebird-with-recursion/instana-master/node_modules/@instana/core/src/util/requireHook.js:58:36)
    at Module.require (internal/modules/cjs/loader.js:637:17)
    at require (internal/modules/cjs/helpers.js:22:18)
    at Object.<anonymous> (/Users/jsamines/dev/personal/cls-bluebird-with-recursion/instana-master/node_modules/cls-bluebird/lib/index.js:15:13)
    at Module._compile (internal/modules/cjs/loader.js:701:30)
    at Object.Module._extensions..js (internal/modules/cjs/loader.js:712:10)
    at Module.load (internal/modules/cjs/loader.js:600:32)
    at tryModuleLoad (internal/modules/cjs/loader.js:539:12)
    at Function.Module._load (internal/modules/cjs/loader.js:531:3)
    at Function.patchedModuleLoad [as _load] (/Users/jsamines/dev/personal/cls-bluebird-with-recursion/instana-master/node_modules/@instana/core/src/util/requireHook.js:25:32)
    at Module.require (internal/modules/cjs/loader.js:637:17)
    at require (internal/modules/cjs/helpers.js:22:18)
    at Object.<anonymous> (/Users/jsamines/dev/personal/cls-bluebird-with-recursion/instana-master/test.js:12:28)
    at Module._compile (internal/modules/cjs/loader.js:701:30)
    at Object.Module._extensions..js (internal/modules/cjs/loader.js:712:10)

Suggested solution
Require the cls-bluebird module, before the instana require hook is registered. By doing that, we avoid the recursive call
Alternate solutions
Perhaps validating cls-bluebird require value is actually a function before invoking it.
Please check this reproduction example to verify the working solution.
Solves #151

Thanks a ton for the perfect analysis and this model PR. ❤ 🎉 👍
Merged via 27663d96f643015e1f95f10d5ddd9637031d3fb6

This has landed in 1.68.4.


feat: add Kotlin model generator

This PR deals with adding support for Kotlin.
Todos:

[x] Copy, Rename & Replace stuff in and from tepmplate files
[x] Add generator to generators index file
[x] Adapt logic & implement basic functionality
[x] ~Use inline classes~ don't make sense in this context, also unnecessary performance optimization
[x] Add Constraint Validation
[x] Ensure that Classes can be compiled
[x] ~Jackson Integration~ Not needed, Kotlin interop good enough
[x] Add relevant reserved keywords to constants
[x] Add mocked renderer
[x] Testing

[x] Test constrainer
[x] Test restrained keywords
[x] Test typemapping
[x] Test presets
[x] Test renderer
[x] Add examples

[x] Add FileGenerators test
[x] Add Blackbox Tests
[ ] Doc

[x] Add to main README
[x] Add to list of generators
[ ] Add language specific Doc

[ ] Cleanup

For all steps, see contributing.md
Resolves https://github.com/asyncapi/modelina/issues/1073

I'm pushing a bunch of malformed commit messages in here, I'll squash anything but the first chore commit once I'm done

One thing I'm not familiar with or quite so sure of is the whole __snapshots__ part during testing.
I suppose it's a regression testing thing? But who generates the snapshots the first time and what do I really test against?
I'll figure it out tomorrow.

I'm pushing a bunch of malformed commit messages in here, I'll squash anything but the first chore commit once I'm done

There are no specific format for commit messages in your local branch, everything is squashed when merged into next 🙂

One thing I'm not familiar with or quite so sure of is the whole __snapshots__ part during testing.
I suppose it's a regression testing thing? But who generates the snapshots the first time and what do I really test against? I'll figure it out tomorrow.

They are generated by the testing framework Jest, and can be updated by running the script npm run test:update, which will then update any non-matching snapshots with new expected results.
https://github.com/asyncapi/modelina/blob/9e3a823dd7ccd5f206f1028277189291a51f56b3/test/generators/java/JavaGenerator.spec.ts#L50
It is used instead of relying on using something like the following:
    expect(models[0].result).toEqual(expectedResult);

Which just makes it easier to work with then having to manually change the expected values 🙂

just fyi: I'll be back working on this tomorrow!

My roadmap to completing this PR:
Constraint Validation Preset -> Test Annotation Renderer -> Test Presets -> Create Doc
Apart from that it seems to be working at this point.
Blackbox tests are passing, code can be compiled, unit test and examples passing.
Code probably needs a bit of care before I can complete this, since some parts are still copies, but all in all, this is nearly done.
P.S.:
Sorry for taking so long. I would have liked to be done with this last week.

Awesome @LouisXhaferi! Want me to do a first review?

Awesome @LouisXhaferi! Want me to do a first review?

Yes @jonaslagoni , I think it's time for a first review. 💪
I'll start doing a language specific doc and then go over all the code as well, looking if I missed anything important.

One thing I find interesting is sonarcloud reporting 0% of duplication, whereas IntelliJ is complaining all over the place.
There's a ton of duplicated functionality, maybe it would be beneficial to share some of it?

Btw, I would love it if you would like to become a champion for the Kotlin generator! It gives you full write access and freedom to maintain and own it how you see fit (This also makes you an AsyncAPI TSC member). Let me know if that would be something you would be interested in! 😄

Got some very minor comments, but that's it, perfect as perfect goes, nicely done! 🔥
Some general remarks:

Remember to add the examples to the list: https://github.com/asyncapi/modelina/blob/next/examples/README.md
Might even though Kotlin works in the same way as Java and is indirectly described in the black-box documentation, but do you mind making an explicit statement about it here: https://github.com/asyncapi/modelina/tree/next/test/blackbox#running-the-tests. It might not be everyone who knows Kotlin and Java 🙂

I'm not sure I quite understand the second one though! Is this about me having to add that you also have to have a JDK installed? (Because that's totally the case, oops)

I'm not sure I quite understand the second one though! Is this about me having to add that you also have to have a JDK installed? (Because that's totally the case, oops)

Yes! It's just to provide some help to the contributor if they want to run the black-box tests for Kotlin, that they need to have a JDK installed (in the same way it's described for Java), just want to have it explicitly defined for Kotlin, so its not indirectly described 🙂

Btw, I would love it if you have interest in becoming a champion for the Kotlin generator! It gives you full write access and freedom to maintain and own it how you see fit (This also makes you an AsyncAPI TSC member). Let me know if that would be something you would be interested in! 😄

Thanks for considering me as the Kotlin champion. In general I'm interested in becoming the Kotlin generator champion for modelina!
I'll have to check something at work about this, but I'll let you know on Monday or Tuesday 💪🏻

I also just fucked the review setup my trying to resolve it...

Sorry...

Gonna try replicating it on my computer later today 🙂

Gonna try replicating it on my computer later today 🙂

That would be nice. But just so you know, I haven't specifically looked at the Windows blackbox issue so far. I'll try replicating it later as well, but am limited by macOS and a weak mobile internet connection 😂

Yea, macOS will definitely not trigger the error (tried) 😅

Another thing I just saw: I did not install Kotlin in the Docker image..

I also added a failing but necessary blackbox tests for the constraint validation blackbox test. Don't understand why it is failing (or rather why it is doing what it is) yet, but I'll look at it later.

Yep, installing the kotlin-compiler-1.8.0.zip works 👌
kotlinc -version
info: kotlinc-jvm 1.8.0 (JRE 19.0.1+10-21)

kotlinc -cp C:\\Users\\Lagoni\\Downloads\\modelina-kotlin-next-add-kotlin-generator\\test\\blackbox\\dependencies\\ko
tlin\\* C:\\Users\\Lagoni\\Downloads\\modelina-kotlin-next-add-kotlin-generator\\test\\blackbox\\output\\JsonSchemaDraf
t-6\\draft-6-core\\kotlin\\class\\default\\*.kt -d C:\\Users\\Lagoni\\Downloads\\modelina-kotlin-next-add-kotlin-genera
tor\\test\\blackbox\\output\\JsonSchemaDraft-6\\draft-6-core\\kotlin\\class\\default

Guess you will have either adapt the GitHub workflow for windows to install the correct dependency or adapt the commands to run with the native compiler 🤔
But, not 100% sure as I have only worked with it for an hour, so take it with a gram of salt 😆

Yep, installing the kotlin-compiler-1.8.0.zip works 👌
kotlinc -version
info: kotlinc-jvm 1.8.0 (JRE 19.0.1+10-21)

kotlinc -cp C:\\Users\\Lagoni\\Downloads\\modelina-kotlin-next-add-kotlin-generator\\test\\blackbox\\dependencies\\ko
tlin\\* C:\\Users\\Lagoni\\Downloads\\modelina-kotlin-next-add-kotlin-generator\\test\\blackbox\\output\\JsonSchemaDraf
t-6\\draft-6-core\\kotlin\\class\\default\\*.kt -d C:\\Users\\Lagoni\\Downloads\\modelina-kotlin-next-add-kotlin-genera
tor\\test\\blackbox\\output\\JsonSchemaDraft-6\\draft-6-core\\kotlin\\class\\default

Guess you will have either adapt the GitHub workflow for windows to install the correct dependency or adapt the commands to run with the native compiler 🤔
But, not 100% sure as I have only worked with it for an hour, so take it with a gram of salt 😆

First of all: Thank you for investigating the problem.
I’ll probably have a to get Windows to use the correct dependency, since JVM is definitely the primary compilation target.

In case you wanted to become a champion for Kotlin, add yourself to the codeowners file by adding the following:
# Language Champions for Kotlin and it's presets
*/generators/kotlin @LouisXhaferi

I have not found out why the windows runner would use the native compiler so far.
I checked out the Installer script, which looks fine and seems to install the correct version....
Since I have no idea where it goes wrong, I haven't opener an Issue yet.
I hope I'll find out later, otherwise I'll temporarily open one in this repo.

Aaah I need to do one more thing: install kotlin in the docker image

I hope I'll find out later, otherwise I'll temporarily open one in this repo.

Temporary issue is all good, you can clarify it later once you figure it out 👍

I think.... I'm all done

/rtm

@all-contributors please add @LouisXhaferi for code, test, doc, example, maintenance.

LGTM 🔥 Well done 💪
I will invite you on as a collaborator, so please check your invites/email. You can then add yourself to the TSC in the community repository, you can use this PR as guideline: asyncapi/community#570
If you encounter any problem just reach out 😄
@magicmatatjahu @Samridhi-98 @ron-debajyoti @leigh-johnson pinging you to let you know the Modelina family grows 😊

Thank you for accompanying and supporting me through my first issue and pr here 💪🏻


Don't sort profile to avoid memory allocation

Pull request details
Describe the changes made in this pull request
Sorting creates intermediary data, which makes it slow. Assuming that profiles are created in order will avoid this and make it faster.
List of related issues or pull requests
Part of #294
Collaboration confirmation
As a contributor I confirm

[x] I read and followed the instructions in README.dev.md
[x] The documentation is up to date with the changes introduced in this Pull Request (or NA)
[x] Tests are passing
[x] Lint is passing

Please test on the NLBE dataset locally

/run-benchmark

Looks like the benchmark is too small. I'm increasing it. Also, the Input_NLBE is not in the correct format anymore.

Updated the docstring


Disable vertical drag to dismiss

The previous method of disabling vertical drag was using a workaround that we will not be able to give proper support to long term. To remedy this we've recently introduced the shouldVerticalDragDismissMessage method to the OneSignalIamApi. Now we can call this much simpler method anywhere before the browser load event and the behavior will be handled.

@gardenFiend138, I removed the carousel template in PR #20.
cc/ @Koji23


Deployment: Add bicep deployment to AKS

Alongside the existing option to deploy to ACA using bicep, add the ability to deploy to AKS

add bicep and scripts
update docs

Is the title of this issue correct?
Should it be `Deployment: Add AKS deployment of ACA"

Ignore. I misread/misunderstood.

I'll take this one @stuartleeks

@stuartleeks happy to help with this one :)


Feature Request: add setting to allow running in a Chrome tab like previous versions

I really liked running ARC in a Chrome tab. It's one of the reasons I kept using it instead of switching to Postman. I assume it was made to use a pop-out window because most people prefer that. Would it be possible to add a setting to ARC that tells it to use tabs instead of windows?

Hi,
Thank you for your issue report.
As for popup window issues, please, refer to the issue #510 for detailed discussion end explanation.
I'm closing this as a duplicate.
Thanks.


.yml file

which language do we use for .yml file

.yml is for files written in the YAML language which is (sort of) a markup language, though there appears to be disputes on whether or not it's really a markup language. You can read more about it here (https://en.wikipedia.org/wiki/YAML)


Solves Swiper's adaptation problem on Android

Is it a bugfix ?

Yes
solve #868 problem

Is it a new feature ?

NO
Include documentation, demo GIF if applicable

Describe what you've done:
ViewPagerAndroid Component have been officially abandoned by react-native, requiring the @react-native-community/viewpager component

https://github.com/leecade/react-native-swiper/commit/a808eed80918fffb5e6ea516851ca903b744a449


Add fake chalk in browser builds in order to support IE10

Summary
Adds a new module jest-fake-chalk which mocks out the entire chalk API in order for expect to work in older browsers.
The implementation is a noop (just returns the string it got as input).
Closes #4074
(First commit is from #4366, so you can merge that and I'll rebase, or just merge this).
/cc @skovhus @mjackson

Test plan
./jest and manually running browser test in IE10
We should really look into getting browserstack or something set up!

Codecov Report

Merging #4367 into master will increase coverage by 0.06%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master    #4367      +/-   ##
==========================================
+ Coverage   55.85%   55.91%   +0.06%
==========================================
  Files         189      190       +1
  Lines        6383     6392       +9
  Branches        6        6
==========================================
+ Hits         3565     3574       +9
  Misses       2815     2815
  Partials        3        3

Impacted Files
Coverage Δ

packages/jest-fake-chalk/src/index.js
100% <100%> (ø)

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 db4465e...e5b6fac. Read the comment docs.

As an alternative to this, could we use the previous version of Chalk?

As an alternative to this, could we use the previous version of Chalk?

I don't think browser support should block updates to modules that doesn't even work in the browser anywasy

Nice, thanks for fixing this stuff. I would prefer not to create a separate package for it, and putting packages/expect/src/fake-chalk.js makes sense to me (assuming that we transform that file afterwards?)

I can move it to expect, sure.
Can we set up browserstack or something similar?

Could we somehow mock out the environment in Jest? Could that be sufficient for now?

For chalk's case in specific it's just Object.setPrototypeOf. I don't think messing up the env really proves support. I only think real browser test provides that value. We can make sure to manually test before release until it's set up?
After moving fake_chalk it's still green, BTW:

I can confirm that using this expect package makes https://github.com/ReactTraining/history/pull/468 green.
Example of assertion error in the terminal:

Awesome @SimenB!

I only think real browser test provides that value.

Agree.


Add python bindings

I had forgotten how messy python is. If you want to do things properly, you apply all sorts of band-aids ...
all, slots, explicit inheritance from object, if's for python2 vs. python3, ...
Thx a lot for catching all of those!


docs: fixing docs bugs made by quetzalli

I just realized I accidentally merged a couple of mistakes to docs, so fixing them 🥲
The Kafka section is in the wrong order in the nav (should be after new WebSocket section) and there is a period in one of the WebSocket page titles.

@thulieblack or @akshatnema, can you please review the new correct order via the preview link and help approve this PR? Our new Kafka section is in the wrong place in our docs navigation, creating a confusing user experience. 😿

@alequetzalli is this the right position for Kafka directory? I see this in the preview.

Based on the PR description, it is Akshat; it's supposed to be below the WebSocket.

/rtm

damn I forgot, please @akshatnema do approve and merge please

thank you so much for helping out, @thulieblack! 🍻 appreciate you


Add CAS (Central Authentication Server) support

As discussed (briefly) in the Google Group, I need support for CAS.  I've implemented the protocol before, but have struggled with integrating it into the Play framework in an acceptable way.  Silhouette looks like an ideal way to do that!
Help and guidance appreciated!

@bwbecker Any news on this? I would like to release the 2.0 in the next few days.

Christian —
Thanks for the poke.  Unfortunately, I’ve had a number of projects come out of the woodwork and haven’t gotten to this yet.  It’ll happen, but not in time for 2.0.
Sorry,
Byron

On Jan 18, 2015, at 6:51 AM, Christian Kaps<EMAIL_ADDRESS>wrote:
@bwbecker https://github.com/bwbecker Any news on this? I would like to release the 2.0 in the next few days.
—
Reply to this email directly or view it on GitHub https://github.com/mohiva/play-silhouette/issues/198#issuecomment-70405191.

Byron Weber Becker
School of Computer Science
University of Waterloo, Waterloo, Ontario  N2L 3G1
Email<EMAIL_ADDRESS>Phone:<PHONE_NUMBER> x34661
Fax:<PHONE_NUMBER> (include my email on cover page)
Advising FAQ:  www.cs.uwaterloo.ca/current/faq

It's all good. Let me know if you can find the time.

Fixed with #450


add e2e testCase to run workflow and timing

Tips

Thanks very much for contributing to Apache DolphinScheduler.
Please review https://dolphinscheduler.apache.org/en-us/community/index.html before opening a pull request.

What is the purpose of the pull request
(For example: This pull request adds checkstyle plugin.)
Brief change log
(for example:)

Add maven-checkstyle-plugin to root pom.xml

Verify this pull request
(Please pick either of the following options)
This pull request is code cleanup without any test coverage.
(or)
This pull request is already covered by existing tests, such as (please describe tests).
(or)
This change added tests and can be verified as follows:
(example:)

Added dolphinscheduler-dao tests for end-to-end.
Added CronUtilsTest to verify the change.
Manually verified the change by testing locally.

Codecov Report

Merging #3146 into dev will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##                dev    #3146   +/-   ##
=========================================
  Coverage     33.47%   33.47%
  Complexity     2389     2389
=========================================
  Files           442      442
  Lines         20596    20596
  Branches       2522     2522
=========================================
  Hits           6894     6894
  Misses        13042    13042
  Partials        660      660

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 0692dca...f9bcf10. Read the comment docs.


sc-11862 : fix permission issue

Scope of changes
At the first user login, we are facing some issues with permission which is not set.
Type of change

[x] bug fix
[ ] new feature
[ ] documentation
[x] other (refactor)

Acceptance criteria
Author checklist

[ ] I have manually tested the change and/or added automation in the form of unit tests or integration tests
[ ]  I have updated the dependencies list
[ ]  I have recompiled and included new protocol buffers to reflect changes I made
[ ]  I have added new test fixtures as needed to support added tests
[ ]   Check this box if a reviewer can merge this pull request after approval (leave it unchecked if you want to do it yourself)
[ ]  I have moved the associated Shortcut story to "Ready for Review"

Reviewer(s) checklist

[ ] Any new user-facing content that has been added for this PR has been QA'ed to ensure correct grammar, spelling, and understandability.

Codecov Report
Base: 43.99% // Head: 37.96% // Decreases project coverage by -6.02% :warning:

Coverage data is based on head (c2dad18) compared to base (9c917cd).
Patch has no changes to coverable lines.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #953      +/-   ##
==========================================
- Coverage   43.99%   37.96%   -6.03%
==========================================
  Files         688      168     -520
  Lines       25577     2568   -23009
  Branches     1565      667     -898
==========================================
- Hits        11252      975   -10277
+ Misses      12878     1409   -11469
+ Partials     1447      184    -1263

Flag
Coverage Δ

unittests
37.96% <ø> (-6.03%)
:arrow_down:

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

Impacted Files
Coverage Δ

.../pages/app/details/BasicDetails/dashboard/Tasks.js
59.09% <0.00%> (ø)

...user-ui/src/components/UserProfile/UserDetails.tsx

...b/gds-user-ui/src/modules/auth/login/user.slice.ts

...gds-user-ui/src/components/NetworkStatus/index.tsx

...nts/CertificateManagement/CertificateDataTable.tsx

...com/trisacrypto/directory/pkg/gds/client/editor.go

.../gds-user-ui/src/components/Loader/MinusLoader.tsx

...i/src/modules/auth/register/register.validation.ts

...ser-ui/src/modules/dashboard/registration/utils.ts

web/gds-user-ui/src/hooks/useMeasure.ts

... and 511 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Container image scanning can take a lot of time, increase timeout

In downstream builds there were few images that the CSO could scan,
so finding the one with the vulnerability shouldn't take too long. In
upstream there are lots of images that don't have vulnerabilities, so
finding the one with vulnerabilities can take much longer.  My local
testing was finding vulnerabilities within about 3 minutes, so it's
certainly possible the canary test is consistently taking longer than
the 2 minutes allotted.  Logs haven't been useful to identify a root
cause because tests clean up.  Theory on failure is from local tests
Refs:

https://github.com/stolostron/backlog/issues/21734

Signed-off-by: Gus Parvin<EMAIL_ADDRESS>

Here are image manifest vulnerability timestamps from my local environment:
default                               sha256.a2d98652e07897c388689795e63d6b9d0512162dffc65f40f34942b66abdcc9f   2m57s
hive                                  sha256.bc91986eaaf994b4bf34b77fed4f96a798ee09a4170ddef797d165c825e010dd   3m58s
hypershift                            sha256.3e595a200ffae53f5773d1988e009547fd4cc97fdfbd00d8af2133e324bdecd8   2m46s
open-cluster-management-agent-addon   sha256.736377cf9374b22a58f38b22abff8038103052a089b789ec9b453f2e263fbba0   2m13s
open-cluster-management-hub           sha256.71d6109420a951620138a9219c31f350b295abe3d53edfc3959e1b7032342f8f   4m5s
open-cluster-management               sha256.9031f3a89d0f2bc8124e80cfdde5ea4f090d2d9841481e673d0524d0982446d0   3m2s
open-cluster-management               sha256.c92a1480b2628d50d8fe158df9451598406e2e5cc7177cf5efcb81caba120afd   4m39s
openshift-cluster-version             sha256.347fcefa4cff84074fa56ff73a483b9fee7ba98b9a71752763502f11182a11af   4m26s
openshift-marketplace                 sha256.9e5dbc2677344b38a627386760b2d1e63e83316a38a6f8890d64edfeced102fc   4m52s

This is a subset of what was found, but notice the oldest age is 4m52s and the default pod is 2m57s.  Of course many
of these vulnerabilities are specific to my local environment -- any vulnerability would trigger the NonCompliance we want,
but getting the timing set to something more reasonable is my aim right now based on the timing we see here.

/cherry-pick release-2.5


remove 819⌶ from lowercase function to use a syntax error trapped ⎕C for backwards compatibility

Hi Aarush... I'm not going to merge this pull request as I already have a commit in the wings that deals with removing 819⌶.  As for the test suite, the tests need to be run individually until I fix the ]DTEST integration.

Arg! I'm an idiot... I didn't realize that your changes were to HttpServer and not HttpCommand.  (Note to self - drink coffee before reviewing pull requests).