Code Scan Issue - Insecure Randomness in tinysort.js

Hi ,
We are using the tinysort plugin version 2.3.6 in our website. Our security team ran the code scan and found the “HIGH” security risk in the tinysort.js at line no. 280.
sortReturnNumber = criterium.sortFunction(a,b);         } else if (criterium.order=='rand') { // random sort           sortReturnNumber = Math.random()<0.5?1:-1;
Attaching the detailed information of the issue.
Please let us know if you have a solution for this available in some patch or some newer version.
Thanks,
M
TinySort-Insecure Randomness.docx

Hi Monika,
Has your security team read what this code does?
It does a random sort, using Math.random().
Explain to me how exactly this is a “HIGH” security risk and I might take this claim serious but for now I'm closing this issue as invalid.
You could always fork and patch the code yourself if you feel uncomfortable using it in its current state.
ps: I've read the docx and it doesn't explain a thing. In the future you might want to refrain from attaching files like that. It looks very dubious especially since your account is fresh. I had to go through virus-scan-proxy-open loops to read a file that you could have just copy pasted as text.


[oneDNN] Add oneDNN version of SparseMatrixMatMul (v2)

This is version 2 of #62883, which should fix the build on ARM64.
Adds _MklNativeSparseMatrixMatMul and its accompanying kernel, which uses oneDNN to multiply a CSR sparse matrix by a dense tensor. The op is enabled with an environment variable (TF_ENABLE_ONEDNN_SPMM), so is entirely opt-in. It also includes tests and a benchmark, which we've used below to measure its performance against the existing kernel.
The performance looks promising particularly for larger shapes, and is optimized to use the AVX2 and AVX512 ISAs. These results were collected using the new benchmark in tensorflow/core/kernels/mkl/mkl_sparse_matrix_matmul_op_benchmark.cc on an Intel Xeon Platinum 8480 with hyperthreading enabled. To minimize NUMA effects, we bound it to the first socket.

 Currently, a reference implementation will be used when running on ARM.

Great -- hopefully this time, we've fixed the nightly tests!

Thanks for the heads up and for fixing the failure! I should say that if this flag goes from opt-in to opt-out before there’s an optimized implementation of SPMM in oneDNN for AArch64, then it will need a platform specific guard to stick with Eigen.

@jondea No problem! Yes, definitely: that's exactly one of the reasons that it's opt-in.

So all of the internal overnight checks are successful, now?


4.5.0 django_redis unsupported

as titled.
should be related to https://github.com/niwinz/django-redis/issues/220

Fixed in #28


Chef Zero provisioner

The current provisioner: Chef Solo does not support policyfiles. It will be awesome to include chef zero as one of the options so that use of policyfiles will be supported. Also, chef_zero allowed you to do in-memory runs.

I was new to chef, I just found chef zero is part of Class: Kitchen::Provisioner::ChefBase. I was able to use add chef-zero as my provisioner in the test/.kitchen.yml file.
Ref link: https://www.rubydoc.info/github/test-kitchen/test-kitchen/windows-guest-support/Kitchen/Provisioner/ChefBase


[FLINK-16885][hive] Remove wilcard excludes that don't work on maven 3.1.X

What is the purpose of the change
The sql-connector-hive modules added in FLINK-16455 use wildcards imports to exclude all transitive dependencies from hive.
This is a maven 3.2.1+ feature. This may imply that Flink cannot be properly built anymore with maven 3.1 .
Brief change log
Just remove the exclusions.
The shade-plugin will remove all transitive dependencies since promoteTransitiveDependencies is false.
All declared dependencies are included in the artifactSet.
Verifying this change
This change is a trivial rework without any test coverage.
Does this pull request potentially affect one of the following parts:

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

Documentation

Does this pull request introduce a new feature? (yes / no)

Merged #11620 to master.
Thanks @lirui-apache for the review.


Codespell action for BotKube

This commit adds GitHub action which checks for common misspellings across the repository.
ISSUE TYPE

Bug fix Pull Request

SUMMARY
This commit adds codespell action in CI workflow which enables verification of common misspellings across the repository.
Also fixed existing codespell issues.

Hello @sanketsudake ,
Thanks again for the PR!
Thank you for contributing to BotKube. Could you please fill out this form, so we can send the well deserved awesome swags :slightly_smiling_face:
Team BotKube


Update Kafka ingestion tutorial

Update the Kafka ingestion tutorial to use a nested data source.
The updated tutorial has been tested and reviewed by @ektravel and reviewed by @techdocsmith
This PR has:

[x] been self-reviewed

One outstanding item is to decide where to host the kttm-kafka-supervisor.json file and then update the reference to it. It's not currently in the Druid distribution along with the other sample data files (quickstart/tutorial).

@writer-jill , can we make sure to include an explicit mention on querying/nested-columns.md that streaming ingestion is supported and link out to the tutorial for an example.
FYI @gianm

Would it be possible to address #13261 as well, by updating querying/nested-columns.md? i.e. adding an example to this page: https://druid.apache.org/docs/latest/querying/nested-columns.html


zun zoned error

The Flutter bindings were initialized in a different zone than is now being used. This will likely cause confusion and bugs as any zone-specific configuration will inconsistently use the configuration of the original binding initialization zone or this zone based on hard-to-predict factors such as which zone was active when a particular callback was set.
It is important to use the same zone when calling ensureInitialized on the binding as when calling runApp later.
To make this warning fatal, set BindingBase.debugZoneErrorsAreFatal to true before the bindings are initialized (i.e. as the first statement in void main() { }).
When the exception was thrown, this was the stack
#0      BindingBase.debugCheckZone.
binding.dart:497
#1      BindingBase.debugCheckZone
binding.dart:502
#2      runApp
binding.dart:1080
#3      main.
main.dart:40

════════════════════════════════════════════════════════════════════════════════
Future main() => bugsnag.start(
apiKey: 'mykey',
runApp: () async {
WidgetsFlutterBinding.ensureInitialized();
await Firebase.initializeApp(
options: DefaultFirebaseOptions.currentPlatform,
);
await configureDependencies();
// FlutterError.onError = FirebaseCrashlytics.instance.recordFlutterError;
runApp(const MyApp());
},
);
zone mismatched error for this code

The plugin is outdated. They need to update their plugin to meet the new error collection requirement.
Instead of creating a new Zone, PlatformDispatcher must be used for error collection.
Also, don't update Gradle to 8, the plugin is not compatible yet.

Hi @J-Nokwal,
Thanks for reaching out.
I just wanted to let you know that we are aware of this issue and have an item on our roadmap aimed at replacing these deprecating Zone calls. We don't currently have an ETA for the release of this fix but we will be sure to keep you informed of any updates.

@matth-bugsnag Thanks for the response. I am also running into this issue, while evaluating whether to use bugsnag as a vendor for my team. Any update on a fix ETA?

I happened to be looking at bugsnag as one of the integrations for my team as well. I'm also running into the same issue.

@matth-bugsnag any update on this?

Hi @brokenicedeveloper, we are working on this at present and are hopeful of having an update soon.

@johnkiely1 waiting)))

I encountered a similar issue and was able to find a solution. The problem was caused by multiple zones, and the application needed to run on only one zone. The following code snippet was helpful to me:
Zone.root.runGuarded(() {
runApp(MyApp());
});

Hi all, just a quick update, we have the changes in what we hope is their final stages of testing now. Unless we get any major problems, we hope to release this next week.

Hey all, just wanted to let you know we have now released v3.0.0 which should address the issue.
We have also created an upgrading guide here as there are some additional changes from bugsnag-flutter v2
Do let us know if you see any issues when you upgrade.


How to get notification url from notifcation

I want to get notification url from gimbal place notification while tapping on it?

Sorry we missed this earlier. Below is some example Objective C code on how to retrieve the Communicate.url when the notification is clicked. In the AppDelegate you need to listen for the notification callback.
- (void)userNotificationCenter:(UNUserNotificationCenter *)center
didReceiveNotificationResponse:(UNNotificationResponse *)response
         withCompletionHandler:(void(^)())completionHandler {

    GMBLCommunication *gimbalCommunication = [GMBLCommunicationManager communicationForLocalNotification:notification];

    if (gimbalCommunication)
    {
        // do something with the Gimbal Communication URL
        gimbalCommunication.URL;
    }

    completionHandler();
}

Thank you @garydamm 👍


ValueError: Missing valid docker_host.Either DOCKER_HOST or local sockets are not available.

I copy example code from README
I install all dependencies
I run it
I get an error message: ValueError: Missing valid docker_host.Either DOCKER_HOST or local sockets are not available.

Full error message:
Traceback (most recent call last):
  File "/home/synalice/Coding/Tests/main.py", line 55, in <module>
    asyncio.run(main())
  File "/usr/lib/python3.10/asyncio/runners.py", line 44, in run
    return loop.run_until_complete(main)
  File "/usr/lib/python3.10/asyncio/base_events.py", line 646, in run_until_complete
    return future.result()
  File "/home/synalice/Coding/Tests/main.py", line 29, in main
    executor = runbox.DockerExecutor()
  File "/home/synalice/Coding/Tests/venv/lib/python3.10/site-packages/runbox/docker/docker_api.py", line 28, in __init__
    self.docker_client = docker_client or Docker(url)
  File "/home/synalice/Coding/Tests/venv/lib/python3.10/site-packages/aiodocker/docker.py", line 99, in __init__
    raise ValueError(
ValueError: Missing valid docker_host.Either DOCKER_HOST or local sockets are not available.

Process finished with exit code 1

I do have docker installed:
$ docker --version
Docker version 20.10.14, build a224086349

All I had to do is to enable docker daemon:
$ sudo systemctl enable docker
$ sudo systemctl start docker


how to disable translation for a specific query

hi,
I would like to disable , for a query specific, the translation layer .
indeed I executes a query that return me 1500 objects.
the problem is that for every object, an additional request is made for traduction.i would like to disable it just for this request for to get only a single query.
this my query :
SELECT a0_.id AS id_0, a0_.ordre AS ordre_1, a0_.dateCrea AS dateCrea_2, a0_.dateModif AS dateModif_3, a0_.datePubli AS datePubli_4, a0_.highlight AS highlight_5, a0_.actif AS actif_6, a0_.slug AS slug_7, a0_.actifLocale AS actifLocale_8, a0_.slugcustom AS slugcustom_9, a0_.title AS title_10, a0_.text_intro AS text_intro_11, a0_.meta_description AS meta_description_12, a0_.content AS content_13, a0_.oldurl AS oldurl_14, a0_.oldid AS oldid_15, a0_.oldurlen AS oldurlen_16, a0_.oldiden AS oldiden_17, a0_.oldurles AS oldurles_18, a0_.oldides AS oldides_19, a0_.auteur_id AS auteur_id_20, a0_.vignette_id AS vignette_id_21, a0_.vignette2_id AS vignette2_id_22, a0_.vignetteoriginal_id AS vignetteoriginal_id_23 FROM article a0_ ORDER BY a0_.ordre ASC
and for every object this query is made :
SELECT t0.id AS id_1, t0.locale AS locale_2, t0.field AS field_3, t0.content AS content_4, t0.object_id AS object_id_5 FROM article_translations t0 WHERE t0.object_id = ? Parameters: [2178]
do you have a solution for to disable all this queries.
thanks

see query hint it autojoins translations.

Hello. I am also interested in this option. I want to disable translation at all for a specific query. Is it possible?


Add PyCon Finland 2011 videos

Fixes #36

@logston , @Daniel-at-github , any feedback?

Looks great. Thanks!


V4.5 - The recent addition of 'min-width: 0' on .'col' class causing issues

Referring to the following markup, the intended behaviour for this row setup would be when viewing at less than the SM breakpoint the two columns will stack on top of each other. As of v4.5, there seems to be an issue causing the 'col' (which should auto size) to not stack and have a width of zero appearing next to the other column.
I think the issue is caused from the addition of min-width:0 on .col class with v4.5.

col-sm-6

Auto col

Looks to be a duplicate of: #30852


What is support email address?

I wrote email to<EMAIL_ADDRESS>(what is described on jodit pro website)
about license question 2021.12.21.
but still today, There is no reply.
Is this not correct email address?
If then, What is correct email address?

Yes, this is a correct email, but it is better to write questions about using the PRO version in the issue tracker https://xdsoft.net/jodit/pro/cab/issues

Thanks @xdan.
I wrote question about license on your comment's link.
If I wrote inappropriate question, then tell me.
I just wanna know communication method, before get pro or oem license.

Wrote answers https://xdsoft.net/jodit/pro/cab/issues/42073037-15e8-406f-86e9-71acae5b0474


Can not develop authentication locally

I have a big project that I have migrated to dingo/api and I am now trying to install authentication via tymondesigns/jwt-auth (which I installed & configured - and then turned on in dingo/api's config file)
Because I am working completely locally, I am serving my Laravel4 application using: php artisan serve & my angular app using: grunt serve
Because going from one port (angular on 9000 & laravel on 8000) to another qualifies as a CORS request I normally add a before filter to my Laravel app which returns open X-Origin headers when I am working locally.
Once I try to turn on protection the dingo/api's auth-filter executes prior to my Laravel filter -- so I get X-Origin errors and cannot continue.
How can I continue to develop locally with Laravel 4 & dingo/api ?

I figured it out I think -- I needed to send those headers in the standard Laravel auth filter.


Add semver tags?

Hi man,
Hope you are doing well :)
Is that possible to tag the releases with semver versioning?
I'm planning to use your library but I'm afraid about none compatibilities issues in the future...
Thanks in advance!
Cheers!

sorry wasn't seing that it's being tagged in npm :)


Localize “email address” placeholder on home page

https://github.com/kubernetes/website/issues/19415
/language it

@Fale would you be willing to take a look at this one?

/lgtm

@sftim it seems like I can not lgtm it :-D. I think it would be better to put it in the localization branch otherwise it's a messy business, but that's just my 2cc. The translation by itself is right

Thanks.

This is fixed in #20139

@npu21 are you happy to close this?


Upgrading process looks a bit weird on AppVeyor

npm-windows-upgrade per se works great, but upgrade output on AppVeyor is a bit weird.
Any chance you might consider to look into this rather cosmetic-only request?
https://ci.appveyor.com/project/Mapbox/node-mapnik/build/1.0.839/job/r29p489lwrvbyg68#L25

/cc @FeodorFitsner

Oh, absolutely. I might be too pragmatic, but would it work for you if just disabled the spinner whenever it's run with --no-prompt?

would it work for you if just disabled the spinner whenever it's run with --no-prompt?

Sure, would do.
Maybe I'm missing something, but I already use npm-windows-upgrade like this:
CALL npm install -g npm-windows-upgrade
CALL npm-windows-upgrade --version:3.3.2 --no-dns-check --no-prompt

Sorry, I wasn't clear - I would disable the spinner for those who use --no-prompt. Patch coming right up :)

...and fixed (in 0.4.5)! Should work fine now, I'm closing this - let me know if you have any other troubles.

@felixrieseberg WOW, that was fast, thanks.
Looks a lot cleaner now:


Fix rtd build 2

📚 Documentation preview 📚: https://plone6--874.org.readthedocs.build/

This is strange. The link that RTD generates and writes back to the description is incorrect. It should be https://plonetraining--874.org.readthedocs.build/. I'll merge this PR and try another, as perhaps the GH workflow needs to have the correct slug on main first.


Set node labels on kube-worker role in v1.19.7

Set correct node-label on Kubernetes worker nodes provisioned with kube-worker role.

Hi Miroslav! Have you tried this? We had to change the label because in the latest versions this label can't be used from the kubelet for security reasons, see:
https://github.com/weaveworks/eksctl/issues/2197


Async and await not working as expected in for loop

Here is a code snippet which prints Log: {i} for every one second, which is not working.
Also tried changing to ES6/Babel but this gives error "SyntaxError: Unexpected token ';'
console.clear();

const customDelay = () => {
  return new Promise(resolve => {
    setTimeout(() => resolve(), 1000);
  });
}

const init = async () => {
  for(let i=0; i<=5; i++) {
    await customDelay();
    console.log('Log: ' + i);
  }
}

init();

I had a same problem with the code below running as ES6/Babel.
async function init() {
  console.log( "ready!" );
}

document.addEventListener("DOMContentLoaded", () => {
  init();
});

The async init function above was compiled as below. The first statements inside switch statement without label seems to cause the error.
function init() {
    return regeneratorRuntime.async(function init$(context$1$0) {
        {
            ; window.runnerWindow.protect.protect({ line: 5, reset: true });
            while (1)
                switch (context$1$0.prev = context$1$0.next) {
                    ;
                    if (window.runnerWindow.protect.protect({ line: 5 })) break;
                case 0:
                    window.runnerWindow.proxyConsole.log("ready!");

                case 1:
                case "end":
                    return context$1$0.stop();
                }
        }
    }, null, this);
}

Duplicate of #2495
According to #2495, the workaround is // noprotect comment and babel-polyfill like below.
<script src="https://cdnjs.cloudflare.com/ajax/libs/babel-polyfill/7.12.1/polyfill.min.js"></script>


Allow users to schedule rentals one day in advance

As an Admin, I would like to be able to schedule rentals day of.
As a User, I would like to be able to schedule rentals one day in advance.

Settled on Admins will be able to schedule rentals day of, external users will be required to schedule one day in advance

at least one day in advance ( (Rental.last.start_time-Rental.last.created_at) > 1.day) or exactly one day in advance
advance = (Rental.last.start_time-Rental.last.created_at) advance < 1.day && advance < 2.days

Perhaps I misspoke. What I meant was...
'As an administrator, I would like to be able to book reservations, even for Date.today'
'As a user (renter), I am required to make reservations at least one day in advance'.

@tomecho , does that help?

@phantompatrol thank you i see now

@owlee label enhancement plz

@owlee I closed #183 and assigned you to this one

Rental reservation works for today


delete_all_using_limit_in_batches generates slow SQL query

This strategy generates following SQL query:
DELETE FROM "tasks" WHERE "tasks"."id" IN (SELECT "tasks"."id" FROM "tasks" WHERE (end_at < '2023-05-13 08:28:12.447024') ORDER BY "tasks"."end_at" ASC LIMIT 500)

For big tables sorting by custom field could cause:
ActiveRecord::QueryCanceled: PG::QueryCanceled: ERROR: canceling statement due to statement timeout
So, I think it's better to rewrite the something like:
      def call(collection)
        while true
          keys = collection.limit(batch_size).pluck(collection.primary_key)
          break if keys.blank?
          collection.where(collection.primary_key => keys).delete_all
        end
      end

@smily-ivan Are you sure that this is caused by ORDER and not WHERE statement? ordering by whatever you have in filters is the most optimal solution here, so it looks like the issue is not caused by ordering statement but rather a lack of index

Yes, I checked.
Here is explain with order

And same query without order:

We switched to delete_all strategy, it sorts by id (using index).


i get some alert

i get some alert 1 from Eslint and 2 from ?
any tips to fix?

 type '100' type 'string | 0'.
The expected type comes from property 'height' which is declared here on type 'Properties<string | 0>'

Oh well, now it's cannot receive number type, maybe for now, you should use string with px for a while, i will fix it later


Fixes #416 Scope moved to a seperate elb

Fixed #416 Move Scope to a seperate ELB

Can either @errordeveloper or @2opremio do a review please?

@vlal could you please elaborate on how this fixes #416?

Judging by the issue the idea was to remove WeaveScope off the original load balancer, but the discussion here weaveworks/integrations#115 mentioned running one app behind the load balancer, and the original reason for the load balancer being the URL listed in the outputs section of CloudFormation stack status page. So I figured putting each app behind their own load balancer would solve the split brain and allow the url to be displayed in the status page.

I think you may be missing one detail here, when we talk about 'app' in
this case, we mean the UI part of Scope... Sorry if this is a bit
confusing. Anyhow, I am going to try sticky sessions for Scope app, and I
am not sure if that would require a separate ELB, so I might need to merge
this first :)
On Tue, 22 Nov 2016, 15:00 Vishal Lal<EMAIL_ADDRESS>wrote:

Judging by the issue the idea was to remove WeaveScope off the original
load balancer, but the discussion here weaveworks/integrations#115
https://github.com/weaveworks/integrations/issues/115 mentioned running
one app behind the load balancer, and the original reason for the load
balancer being the URL listed in the outputs section of CloudFormation
stack status page. So I figured putting each app behind their own load
balancer would solve the split brain and allow the url to be displayed in
the status page.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/microservices-demo/microservices-demo/pull/424#issuecomment-262263081,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAPWS5rU3hO-9UKuthe8oEIgn8CYbJxuks5rAwN1gaJpZM4K4-yk
.

Oops, cool no worries I'll leave this open just in case


Unable to see the images in the Blob view

Description of the bug
Images fail to load in the NuxtHub admin and show an error with status code 500. The error indicates that the Fetch API is unable to retrieve the image URLs.
Here is some information that may be relevant:

The image names contain :.
This happens in a team. Is it possible that there are some security rules in the cloudflare account that prevent the images from loading?
I am using my own custom endpoint to serve the image and it seems to work. So the images are uploading correctly.
I don't see any errors or placeholders, maybe show a fallback?

Error from server

{
    "url": "/api/teams/nimiq/projects/crypto-map/production/blob/location:bea8377e-42b0-4289-9b0e-d94bcb1565e2",
    "statusCode": 500,
    "status message": "",
    "message": "[GET] \"location:bea8377e-42b0-4289-9b0e-d94bcb1565e2\": <no response> Fetch API cannot load: location:bea8377e-42b0-4289-9b0e-d94bcb1565e2",
    "stack": ""
}

Thank you in advance! 🙌

If you right-click and open in a new tab? What is displayed? Any content type answered?

If you right-click and open in a new tab? What is displayed? Any content type answered?

Does it work in the devtools locally?

Using local storage does work.
Using the remote storage does not work.

Should be fixed now with https://github.com/nuxt-hub/core/releases/tag/v0.7.34
Make sure to update the dependency and deploy, then refresh the admin.

Yes, it worked. Thank you!


Use IOU/Expense reportID for IOU actions

Add the IOU reportAction to the IOU/Expense report instead of the parent chatReport.
We are gonna start creating IOU reportActions directly in the IOU/Expense report and as part of the switch, we are temporarily skipping test so that Auth integration tests pass. The plan to make the switch is:

Merge https://github.com/Expensify/Web-Expensify/pull/37375 and CP to prod
Web-E Integrations tests in this https://github.com/Expensify/Auth/pull/7863 should pass
Merge the https://github.com/Expensify/Auth/pull/7863
Deploy the Auth PR
Merge https://github.com/Expensify/Web-Expensify/pull/37337 that re-enables tests using the correct reportID
CP the https://github.com/Expensify/Web-Expensify/pull/37337 PR to prod
Merge https://github.com/Expensify/App/pull/18604, CPing it to staging to minimize the amount of time IOUs are broken
Run the bulkCQ from https://github.com/Expensify/Expensify/issues/282420 to change the reportID of all previous IOU actions

Fixed Issues
$ https://github.com/Expensify/Expensify/issues/281893
Tests
In dev, pull these Auth and Web-E branches
Pre-requisites: 2 accounts you have access to
Request money

In accountA, go offline
Request money from a user you haven't chatted with before, + > Request money
Verify:

A new chat report is created between you and accountB. The chat should be greyed out (pending state)
A new iou report is created between you and accountB and is displayed in the LHN (this report has a different header showing the request total). The report should be greyed out (pending state)
In the iou report, you see the IOU request preview. The preview should be greyed out (pending state).

Go online and verify that all pending states are resolved, i.e. the reports and preview are no longer greyed out
From the chat report, request money again to the same user.
Verify that a new preview is added to the IOU report.

Delete request
8. Tap on the request preview and delete the request
9. Verify that a deleted the $X request message is added to the IOU report
Pay money request
10. In accountB, navigate to the IOU report with accountA
11. Tap on the pay button and settle elsewhere
12. Verify that a settled up $X message is added to the IOU report
Workspace case

In account A, create a new Workspace + > New workspace
Navigate to the Workspace chat
Repeat the steps for Request money, Delete request and Pay money request above.

[x] Verify that no errors appear in the JS console

Offline tests
Covered above
QA Steps
Same as above.

[ ] Verify that no errors appear in the JS console

PR Author Checklist

[x] I linked the correct issue in the ### Fixed Issues section above
[x] I wrote clear testing steps that cover the changes made in this PR

[x] I added steps for local testing in the Tests section
[x] I added steps for the expected offline behavior in the Offline steps section
[x] I added steps for Staging and/or Production testing in the QA steps section
[x] I added steps to cover failure scenarios (i.e. verify an input displays the correct error message if the entered data is not correct)
[x] I turned off my network connection and tested it while offline to ensure it matches the expected behavior (i.e. verify the default avatar icon is displayed if app is offline)
[x] I tested this PR with a High Traffic account against the staging or production API to ensure there are no regressions (e.g. long loading states that impact usability).

[x] I included screenshots or videos for tests on all platforms
[x] I ran the tests on all platforms & verified they passed on:

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

[x] I verified there are no console errors (if there's a console error not related to the PR, report it or open an issue for it to be fixed)
[x] I followed proper code patterns (see Reviewing the code)

[x] I verified that any callback methods that were added or modified are named for what the method does and never what callback they handle (i.e. toggleReport and not onIconClick)
[x] I verified that comments were added to code that is not self explanatory
[x] I verified that any new or modified comments were clear, correct English, and explained "why" the code was doing something instead of only explaining "what" the code was doing.
[x] I verified any copy / text shown in the product is localized by adding it to src/languages/* files and using the translation method

[x] If any non-english text was added/modified, I verified the translation was requested/reviewed in #expensify-open-source and it was approved by an internal Expensify engineer. Link to Slack message:

[x] I verified all numbers, amounts, dates and phone numbers shown in the product are using the localization methods
[x] I verified any copy / text that was added to the app is correct English and approved by marketing by adding the Waiting for Copy label for a copy review on the original GH to get the correct copy.
[x] I verified proper file naming conventions were followed for any new files or renamed files. All non-platform specific files are named after what they export and are not named "index.js". All platform-specific files are named for the platform the code supports as outlined in the README.
[x] I verified the JSDocs style guidelines (in STYLE.md) were followed

[x] If a new code pattern is added I verified it was agreed to be used by multiple Expensify engineers
[x] I followed the guidelines as stated in the Review Guidelines
[x] I tested other components that can be impacted by my changes (i.e. if the PR modifies a shared library or component like Avatar, I verified the components using Avatar are working as expected)
[x] I verified all code is DRY (the PR doesn't include any logic written more than once, with the exception of tests)
[x] I verified any variables that can be defined as constants (ie. in CONST.js or at the top of the file that uses the constant) are defined as such
[x] I verified that if a function's arguments changed that all usages have also been updated correctly
[x] If a new component is created I verified that:

[x] A similar component doesn't exist in the codebase
[x] All props are defined accurately and each prop has a /** comment above it */
[x] The file is named correctly
[x] The component has a clear name that is non-ambiguous and the purpose of the component can be inferred from the name alone
[x] The only data being stored in the state is data necessary for rendering and nothing else
[x] For Class Components, any internal methods passed to components event handlers are bound to this properly so there are no scoping issues (i.e. for onClick={this.submit} the method this.submit should be bound to this in the constructor)
[x] Any internal methods bound to this are necessary to be bound (i.e. avoid this.submit = this.submit.bind(this); if this.submit is never passed to a component event handler like onClick)
[x] All JSX used for rendering exists in the render method
[x] The component has the minimum amount of code necessary for its purpose, and it is broken down into smaller components in order to separate concerns and functions

[x] If any new file was added I verified that:

[x] The file has a description of what it does and/or why is needed at the top of the file if the code is not self explanatory

[x] If a new CSS style is added I verified that:

[x] A similar style doesn't already exist
[x] The style can't be created with an existing StyleUtils function (i.e. StyleUtils.getBackgroundAndBorderStyle(themeColors.componentBG))

[x] If the PR modifies code that runs when editing or sending messages, I tested and verified there is no unexpected behavior for all supported markdown - URLs, single line code, code blocks, quotes, headings, bold, strikethrough, and italic.
[x] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[x] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.
[x] If a new page is added, I verified it's using the ScrollView component to make it scrollable when more elements are added to the page.
[x] If the main branch was merged into this PR after a review, I tested again and verified the outcome was still expected according to the Test steps.
[x] I have checked off every checkbox in the PR author checklist, including those that don't apply to this PR.

Screenshots/Videos

Web
https://github.com/Expensify/App/assets/22219519/c691d0e4-67be-4c7f-bb4d-1626f7738dfd

Mobile Web - Chrome

Mobile Web - Safari

Desktop
https://github.com/Expensify/App/assets/22219519/9af2a988-66b9-4a81-89c8-725cb9d21966

iOS

Android

What is the progress on this?

Updated! At this point we can refactor splitBill to also use buildOnyxDataForMoneyRequest, since the logic is very similar. I'll do that in a follow up though.

Updated!

Thanks @s77rt it will hopefully be early on Monday

thanks for the reviews @s77rt @mountiny!

@luacmartins lint failing

@mountiny resolved

@s77rt this is ready to be tested in staging. FYI we have an issue with the call to OpenReport it'll be in an infinite loop right now because it doesn't return data for reports of type = iou or expense. This will probably make testing challenging.

I think there might be some new places which need updating the chat report/ iou report, I will try to see if I can find them

Updated

@s77rt that should be fixed soon as we are in the midst of deploying those changes.

Clicking IOU preview opens a new report instead of RHN IOU details
https://github.com/Expensify/App/assets/16493223/abe407c3-ef28-483a-b84d-2376c4db2dbf

Clicking IOU preview opens a new report instead of RHN IOU details

This is not a bug but feature. Its a transaction thread, you can chat with your friend/ manager about that particular request there. We are adding the update header there

How am I supposed to delete a money request?

Ah I got it

Requesting money from workspace fails
https://github.com/Expensify/App/assets/16493223/e48f3a4c-f20e-4c23-bda9-8c1459395c6c
https://github.com/Expensify/App/assets/16493223/b3809c86-3067-4a6d-9933-7a71e1ff2837

@s77rt yes, we are aware and working onti

@luacmartins Is it a blocker or can we merge and fix that in a follow up PR?

@s77rt the issue is in the API, so we can continue here

Other two minor issues:

The avatar in LHN always shows the current user's avatar
There is an option to request money from IOU report but it's broken (requesting money from chat report works well)

@s77rt both of those are known and being worked on

https://expensify.slack.com/archives/C01GTK53T8Q/p1684254793691709
https://github.com/Expensify/App/pull/19033

@luacmartins Okay one more :grin:

You can't delete the last existing money request
The IOU preview for deleted request is still shown

https://github.com/Expensify/App/assets/16493223/b8f5dfae-81bb-40d8-87fb-ef3aa44b4998

I'm not able to reproduce the delete error. What error are you getting back from the API?
As for the IOUPReview, it should still be shown.

Error 401
{
    "jsonCode": 401,
    "message": "401 Unauthorized report NVP.",
    "onyxData": [
        {
            "onyxMethod": "merge",
            "key": "reportActions_6792924156649370",
            "value": {
                "errors": {
                    "1684260166419151": "Auth DeleteMoneyRequest returned an error"
                }
            }
        },
        {
            "onyxMethod": "merge",
            "key": "reportActions_6792924156649370",
            "value": {
                "errors": {
                    "1684260166419151": "Auth DeleteMoneyRequest returned an error"
                }
            }
        }
    ],
    "requestID": "7c8588974f9c9a17-SJC"
}

@s77rt that seems unrelated to the changes in this PR. We already have another internal issue to track that error.

Reviewer Checklist

[X] I have verified the author checklist is complete (all boxes are checked off).
[X] I verified the correct issue is linked in the ### Fixed Issues section above
[X] I verified testing steps are clear and they cover the changes made in this PR

[X] I verified the steps for local testing are in the Tests section
[X] I verified the steps for Staging and/or Production testing are in the QA steps section
[X] I verified the steps cover any possible failure scenarios (i.e. verify an input displays the correct error message if the entered data is not correct)
[X] I turned off my network connection and tested it while offline to ensure it matches the expected behavior (i.e. verify the default avatar icon is displayed if app is offline)

[X] I checked that screenshots or videos are included for tests on all platforms
[X] I included screenshots or videos for tests on all platforms
[X] I verified tests pass on all platforms & I tested again on:

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

[X] If there are any errors in the console that are unrelated to this PR, I either fixed them (preferred) or linked to where I reported them in Slack
[X] I verified proper code patterns were followed (see Reviewing the code)

[X] I verified that any callback methods that were added or modified are named for what the method does and never what callback they handle (i.e. toggleReport and not onIconClick).
[X] I verified that the left part of a conditional rendering a React component is a boolean and NOT a string, e.g. myBool && <MyComponent />.
[X] I verified that comments were added to code that is not self explanatory
[X] I verified that any new or modified comments were clear, correct English, and explained "why" the code was doing something instead of only explaining "what" the code was doing.
[X] I verified any copy / text shown in the product is localized by adding it to src/languages/* files and using the translation method
[X] I verified all numbers, amounts, dates and phone numbers shown in the product are using the localization methods
[X] I verified any copy / text that was added to the app is grammatically correct in English. It adheres to proper capitalization guidelines (note: only the first word of header/labels should be capitalized), and is approved by marketing by adding the Waiting for Copy label for a copy review on the original GH to get the correct copy.
[X] I verified proper file naming conventions were followed for any new files or renamed files. All non-platform specific files are named after what they export and are not named "index.js". All platform-specific files are named for the platform the code supports as outlined in the README.
[X] I verified the JSDocs style guidelines (in STYLE.md) were followed

[X] If a new code pattern is added I verified it was agreed to be used by multiple Expensify engineers
[X] I verified that this PR follows the guidelines as stated in the Review Guidelines
[X] I verified other components that can be impacted by these changes have been tested, and I retested again (i.e. if the PR modifies a shared library or component like Avatar, I verified the components using Avatar have been tested & I retested again)
[X] I verified all code is DRY (the PR doesn't include any logic written more than once, with the exception of tests)
[X] I verified any variables that can be defined as constants (ie. in CONST.js or at the top of the file that uses the constant) are defined as such
[X] If a new component is created I verified that:

[X] A similar component doesn't exist in the codebase
[X] All props are defined accurately and each prop has a /** comment above it */
[X] The file is named correctly
[X] The component has a clear name that is non-ambiguous and the purpose of the component can be inferred from the name alone
[X] The only data being stored in the state is data necessary for rendering and nothing else
[X] For Class Components, any internal methods passed to components event handlers are bound to this properly so there are no scoping issues (i.e. for onClick={this.submit} the method this.submit should be bound to this in the constructor)
[X] Any internal methods bound to this are necessary to be bound (i.e. avoid this.submit = this.submit.bind(this); if this.submit is never passed to a component event handler like onClick)
[X] All JSX used for rendering exists in the render method
[X] The component has the minimum amount of code necessary for its purpose, and it is broken down into smaller components in order to separate concerns and functions

[X] If any new file was added I verified that:

[X] The file has a description of what it does and/or why is needed at the top of the file if the code is not self explanatory

[X] If a new CSS style is added I verified that:

[X] A similar style doesn't already exist
[X] The style can't be created with an existing StyleUtils function (i.e. StyleUtils.getBackgroundAndBorderStyle(themeColors.componentBG)

[X] If the PR modifies code that runs when editing or sending messages, I tested and verified there is no unexpected behavior for all supported markdown - URLs, single line code, code blocks, quotes, headings, bold, strikethrough, and italic.
[X] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[X] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.
[X] If a new page is added, I verified it's using the ScrollView component to make it scrollable when more elements are added to the page.
[X] If the main branch was merged into this PR after a review, I tested again and verified the outcome was still expected according to the Test steps.
[X] I have checked off every checkbox in the PR reviewer checklist, including those that don't apply to this PR.

Screenshots/Videos

Web

https://github.com/Expensify/App/assets/16493223/c1a05f8b-b6c5-4970-a735-4e6b18e5d779

Mobile Web - Chrome

Mobile Web - Safari

Desktop

https://github.com/Expensify/App/assets/16493223/7c51fd8c-f797-484a-a58b-0513f07818a3

iOS

https://github.com/Expensify/App/assets/16493223/77bb1f87-2fd7-4693-bc5d-7839e492351e

Android

@luacmartins Checklist is ready. Just waiting for a confirmation for the reported bug if it's something we should fix in this PR.

Thanks for the thorough review @s77rt

🚀 Cherry-picked to staging by https://github.com/luacmartins in version: 1.3.14-13 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

@Expensify/applauseleads please QA this PR and check it off on the deploy checklist if it passes.

🚀 Deployed to production by https://github.com/yuwenmemon in version: 1.3.14-14 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

🚀 Cherry-picked to staging by https://github.com/AndrewGable in version: 1.3.28-2 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

@Expensify/applauseleads please QA this PR and check it off on the deploy checklist if it passes.

🚀 Deployed to production by https://github.com/AndrewGable in version: 1.3.28-5 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅


Bug

After update to v. 0.10.10 C# "Language mode" disappeared from the list.
Reinstalling VS Code didn't helped.

C# is now an extension, instructions to isntall are here https://code.visualstudio.com/Docs/languages/csharp#_installing-c35-support

Thank you.


Change firstDayOfTheWeek on calendar day/month/week picker

Hey currently there is no way to change the firstDayOfTheWeek when you click on the header of the calendar even when I try to override the locale it seems I am unable to change it to start from Monday.
is it possible in future build to apply locale to the calendar based on user locale as param ?

Hi @dimityrivanov, Could you please specify what things you need to change? If you need to change firstDayOfTheWeek on calendar then you can go ahead with startDay field in week and month view.

I am closing this issue for now, feel free to reopen if this is not the case.


Help running tests

I’m having some trouble running py.test for OG-USA.  Which directly should I run it from and which command should I use?
I keep getting this error:
(ogusatemp) HDoupe-MacBook-Pro:ogusa henrydoupe$ py.test
============================================================================================================================ test session starts =============================================================================================================================
platform darwin -- Python 2.7.13, pytest-3.1.2, py-1.4.34, pluggy-0.4.0
rootdir: /Users/henrydoupe/Documents/OG-USA, inifile: setup.cfg
collected 6 items / 1 errors

=================================================================================================================================== ERRORS ===================================================================================================================================
_____________________________________________________________________________________________________________ ERROR collecting Python/ogusa/tests/test_basic.py ______________________________________________________________________________________________________________
ImportError while importing test module '/Users/henrydoupe/Documents/OG-USA/Python/ogusa/tests/test_basic.py'.
Hint: make sure your test modules/packages have valid Python names.
Traceback:
tests/test_basic.py:8: in <module>
    from ogusa.utils import comp_array
E   ImportError: No module named ogusa.utils
============================================================================================================================== warnings summary ==============================================================================================================================
None
  [pytest] section in setup.cfg files is deprecated, use [tool:pytest] instead.

-- Docs: http://doc.pytest.org/en/latest/warnings.html
!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!! Interrupted: 1 errors during collection !!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!
==================================================================================================================== 1 warnings, 1 error in 1.73 seconds =====================================================================================================================

where ogusatemp is a temporary anaconda environment I made from PR #306.

@hdoupe I'm not sure this py.test is working at all of OG-USA.  I'm using the environment in OG-USA:master and I'm getting the same error.
I think it would be safe to just start from scratch with the local testing.  You might  look to what is done in Travis CI for OG-USA to see what testing has been working.

@jdebacker said

I think it would be safe to just start from scratch with the local testing. You might look to what is done in Travis CI for OG-USA to see what testing has been working.

Yeah, I agree.  There's a problem with how py.test sets up the paths.  I made a fake test file to see what was in sys.path.
import os
import sys

def test_not():
    with open('/Users/henrydoupe/Documents/OG-USA/Python/ogusa/tests/test_status.txt', 'w') as w:
        w.write('\n'.join(os.sys.path))

Then, I ran py.test ogusa/tests/test_fake.py from the Python directory.  The contents of sys.path are
/Users/henrydoupe/Documents/OG-USA/Python/ogusa/tests
/anaconda/envs/ogusatemp/bin
/anaconda/envs/ogusatemp/lib/python27.zip
/anaconda/envs/ogusatemp/lib/python2.7
/anaconda/envs/ogusatemp/lib/python2.7/plat-darwin
/anaconda/envs/ogusatemp/lib/python2.7/plat-mac
/anaconda/envs/ogusatemp/lib/python2.7/plat-mac/lib-scriptpackages
/anaconda/envs/ogusatemp/lib/python2.7/lib-tk
/anaconda/envs/ogusatemp/lib/python2.7/lib-old
/anaconda/envs/ogusatemp/lib/python2.7/lib-dynload
/anaconda/envs/ogusatemp/lib/python2.7/site-packages
/anaconda/envs/ogusatemp/lib/python2.7/site-packages/setuptools-27.2.0-py2.7.egg

Then performing the same fake test in taxcalc, I got the following result
/Users/henrydoupe/Documents/Tax-Calculator/taxcalc/tests
/Users/henrydoupe/Documents/Tax-Calculator
/anaconda/envs/taxcalc-dev/bin
/anaconda/envs/taxcalc-dev/lib/python36.zip
/anaconda/envs/taxcalc-dev/lib/python3.6
/anaconda/envs/taxcalc-dev/lib/python3.6/lib-dynload
/anaconda/envs/taxcalc-dev/lib/python3.6/site-packages
/anaconda/envs/taxcalc-dev/lib/python3.6/site-packages/pytest_cache-1.0-py3.6.egg
/anaconda/envs/taxcalc-dev/lib/python3.6/site-packages/pytest_pep8-1.0.6-py3.6.egg
/anaconda/envs/taxcalc-dev/lib/python3.6/site-packages/setuptools-27.2.0-py3.6.egg

So this path is missing in the ogusa test path/Users/henrydoupe/Documents/OG-USA/Python/ which corresponds to this path in taxcalc: /Users/henrydoupe/Documents/Tax-Calculator.  I'm trying to figure out how to solve this now.
In the meantime, you can run python -m pytest ogusa/tests and it works.

@hdoupe @jdebacker This issue appears related: https://github.com/open-source-economics/OG-USA/issues/290 I encountered the same issue you are talking about (ogusa.utils import problem) after doing python setup.py develop but did not encounter that after doing python setup.py install .  I can look into this more before the meeting.

@PeterDSteinberg, let @hdoupe pursue this one. You've got a full plate.

@PeterDSteinberg I ran python setup.py install and it works now.  But I'm still not sure why I have to do that for ogusa but not for taxcalc.  I'll look into it some more.
Thanks for you help!

No problem @hdoupe - I have been meaning to fix that issue for a while ( #290 ).  I'll close this issue as a duplicate essentially of #290 .  I'll take a look at the setup.py before tomorrow afternoon

@hdoupe - I didn't read clearly the whole thread inclusive of @MattHJensen 's comment that you'll be handling this setup.py bug ( #290 ) - let me know if you have any trouble.


Topology definition rework

This PR will be in Draft whilist it undergoes development. The changes will appear hear, I will try to keep commits per sub-feature to make the review easier

renamed flag to provide docker network name (from --bridge <name> to --network <name>) 8c48f50
moved flag variables that are defined in root.cmd from config.go b07a8d1
Added comment clarifying the role of cLab.Init() in b7d42d2
reworked GetTopology (removed shortname for files, renamed File struct to TopoFile) in 3ad1543
renamed cLab.Prefix to cLab.name (5b0b5e0)
renamed docker_info section to mgmt section. Also removed ipv4/6 gateway calculation, since it will be used by docker automatically as a GW in the management network (1fec139)
renamed duts config element to topology with a nested structure oftype Topology struct {
   	    Defaults NodeConfig
    Kinds    map[string]NodeConfig
    Nodes    map[string]NodeConfig
}

renamed lab examples topologies in ac98ce9
closes #97

https://github.com/srl-wim/container-lab/blob/3d5ee2bd1144c60f0ed5632c160e884c066d6a6f/clab/config.go#L376
This message needs to be changed as well, to reflect the new topology format.
It's a good idea to get rid of the panic while we are here, make it return an error, that can be handled in clab.ParseToplogy then in whatever cmd that called it.

@karimra addressed this in dff9317


skywalking-agent 8.2 NullPointException when mysql update

Please answer these questions before submitting your issue.

Why do you submit this issue?
[Question ] Question or discussion
[ Bug ] Bug
[ ] Requirement
[ ] Feature or performance improvement

Question

What do you want to know?

Bug

Which version of SkyWalking, OS and JRE?
skywalking-agent 8.2 NullPointException

Which company or project?

What happened?
If possible, provide a way to reproduce the error. e.g. demo application, component version.

Requirement or improvement

Please describe your requirements or improvement suggestions.

Check your codes with our tested demo, including versions, https://github.com/apache/skywalking/tree/master/test/plugin/scenarios/mysql-scenario.


ledger data problem?

public void testLedger {
        final String paymentVal = TransactionType.PAYMENT.value();
        final String accountDeleteVal = TransactionType.ACCOUNT_DELETE.value();
        final HttpUrl rippledUrl = HttpUrl.get("https://s2.ripple.com:51234/");
        XrplClient xrplClient = new XrplClient(rippledUrl);
        ImmutableLedgerRequestParams builder = LedgerRequestParams.builder().transactions(true).ledgerSpecifier(LedgerSpecifier.of(Integer.valueOf("70232798"))).build();
        LedgerResult ledgerResult = xrplClient.ledger(builder);
        LedgerHeader ledgerHeader = ledgerResult.ledger();
        List<TransactionResult<? extends Transaction>> transactions = ledgerHeader.transactions();
        // 如果事务列表为空直接解析结束
        if (transactions.isEmpty()) {
           return ;
        }
        // 过滤不必要的事务  只保留转账 账号删除
        transactions.stream().filter(f -> paymentVal.equals(f.transaction().transactionType().value()) || accountDeleteVal.equals(f.transaction().transactionType().value())).forEach(item -> System.out.println(item));
}

Encountered the following problem, unable to obtain transaction data under this ledger:
Caused by: com.fasterxml.jackson.databind.JsonMappingException: No matching AccountSetFlag enum value for int value TicketCreate (through reference chain: org.xrpl.xrpl4j.model.client.ledger.ImmutableLedgerResult$Json["ledger"]->org.xrpl.xrpl4j.model.ledger.ImmutableLedgerHeader$Json["transactions"]->java.util.ArrayList[43])
how to process this data?

When will 2.3.0 be released

It was just released. It will take a few hours to reflect in maven central. See here to confirm.

Closing this ticket as resolved. If v2.3.0 doesn't fix your issue, please re-open.

If I get the approximate data of each transaction through the above method, but I can't get the success or failure of each transaction! When the transfer method is account deletion, I cannot get the transfer amount because the metadata data is empty!
  TransactionMetadata transactionMetadata = item.metadata().get()
  transactionMetadata.deliveredAmount();     // Get the transfer balance when the account is deleted.
  transactionMetadata.transactionResult();    // tesSUCCESS
  // If the above fields can be returned in the metadata, then my block scanning will become more efficient! There is no need to additionally request the result of the transaction!


Cover for fantasy books.

the cover for fantasy books are not displaying

https://i.imgur.com/s1UdNcR.jpg

Thank you for your report, I will try to investigate this problem further

Fixed, the updated app can be downloaded in here

Thanks man:)


Pasting a domain name should  work when linking two related domains

This is such a minor nuisance that I didn't think much of it when the domain linking feature was badly wounded; but now that it basically works, it would be really nice if you could just paste the name of the domain you want to link to in the linking dialog box.
The way it currently works, you can paste, remove the last character, then select the correct domain from the dropdown. Take care to select the correct one; there will sometimes be candidates which share a substring match which are being offered up, too.
Something in there gets wedged sometimes, as well; a few times a day, I end up closing the tab where I opened the dialog, and starting over by reopening the domain page. Other times, it eventually comes back to life if you leave it open and go do something else for a while. I can't say how to repro this, though it's not impossible that fat-fingering the copy/paste somehow might be a contributing factor. I think I have still experienced this after the latest changes, but let me get back and check that still, probably within a few days.
I imagine this is a side effect of the logic to dynamically suggest completion candidates when you manually type the name of the target, but in our use case, chances are you already have it in your system clipboard, and don't really need this feature.
(I suppose this might be an upstream bug in the library we use; is this component using Turbolinks?)
Ideally, I hope metasmoke should ultimately be consistent in how keyboard completion works in different subcomponents. The autocompletion for tags when you assign them to a post or a domain has slightly different quirks, and so you have to learn different muscle memory behaviors for different contexts. I imagine this inconsistency will also be bewildering to some newcomers to the system.

The above PR/commit resolves the titular problem. The fix resolves the issue of not searching upon paste for the New Domain Link dialog and anywhere else we use that search extension.
However, the text of the above issue is substantially more expansive than just the issue of pasting not being an effective method of triggering a search (which is an extension MS adds to the bootstrap-select dialogs). It would help to have those problems in a separate issue or issues.
I've created Issue #943, Need to prevent re-performing some JavaScript page setup actions when they've already been performed, which covers a wide variety of problems which occur with navigation. The problems which are a result of that issue will mostly be when using the browser's forward and back functionality to navigate, but might also occur when revisiting a page through other navigation, depending on how the browser handles caching information from the previous visit. Unfortunately, until that issue is fully addressed, it's going to be difficult to separate out any problems which are not caused by it, but which require navigating around the site prior to the problem manifesting. I haven't seen anything associated with that issue manifest without previously performing a Turbolinks navigation from one page to another (not something a user generally chooses to do or not do, other than through an explicit page reload). So, if the more general issues which you've mentioned happen without previously navigating within that tab, then they are unlikely to be issue #943 (unless your browser is serving you a cached DOM, rather than a cached raw HTML fetch).


Readme suggestion

Please add to the Readme.md instructions how to add library to dependencies in package.json.

good idea :+1:

Thank you for answering on all issues. I hope I did not spam you too much.
Thank you for making this great library !
Cheers :)

No problem! I added some notes into the new release 1.0.14 from #18
Let me know if this is sufficient: https://github.com/0xStarcat/CircularNatalHoroscopeJS#installation
Feel free to add any suggestions! I'll close this issue in a day or so

Looks good 👍

Thanks! :)


[Doc Improvement][Build 2024 what's new and landing page]

Acrolinx Scorecards
We currently enforce a minimum score of 80.
Click the scorecard links for each article to review the Acrolinx feedback on grammar, spelling, punctuation, writing style, and terminology:

Article
Score
Issues
Scorecard
Processed

msteams-platform/mstdd-landing.yml
84
11
link
:white_check_mark:

msteams-platform/whats-new.md
90
80
link
:white_check_mark:

More info about Acrolinx
Use the Acrolinx extension, or sidebar, in Visual Studio Code to check spelling, grammar, style, tone, clarity, and key terminology when you're creating or updating content. For more information, see Use the Visual Studio Code extension to run Acrolinx locally.


Add functionality for uploading profile picture

WIP
Closes #86

LGTM, I will take over from here.


FR adding brand labels

Hi since I have added the zha pairing guide for the Bosch TRV (#1082) I have take a look at my previous purchased TRVs and saw that two of my Moes are not listed anymore (HY368 and BY-100)
Instead they are listed as the Hysen HY368 and Beca BRT-100 but you wont find them if you search by Moes.
Could we discuss revisiting the manufacture labeland links + introducing brands/relabels?
My Moes packagings have their manufacture facility printed:

Greetings.

No need to discuss, if you find them distinctly different you can make a PR for the other brand


Ionic2 RC0 ios build fails >ngc failed:  NGC encountered an error

My Ionic project works fine with $ionic serve but fails when trying to build:
I get errors, that some my variables are private and then it ends with

ngc failed:  NGC encountered an error

This is my npm-debug.log. I can't figure out why it's not building. Any idea?
0 info it worked if it ends with ok
1 verbose cli [ '/usr/local/Cellar/node5/5.12.0/bin/node',
1 verbose cli   '/usr/local/bin/npm',
1 verbose cli   'run',
1 verbose cli   'build' ]
2 info using<EMAIL_ADDRESS>3 info using<EMAIL_ADDRESS>4 verbose run-script [ 'prebuild', 'build', 'postbuild' ]
5 info lifecycle ionic-hello-world@~prebuild: ionic-hello-world@
6 silly lifecycle ionic-hello-world@~prebuild: no script for prebuild, continuing
7 info lifecycle ionic-hello-world@~build: ionic-hello-world@
8 verbose lifecycle ionic-hello-world@~build: unsafe-perm in lifecycle true
9 verbose lifecycle ionic-hello-world@~build: PATH: /usr/local/lib/node_modules/npm/bin/node-gyp-bin:/Users/floriannorbertbepunkt/Dropbox/Dropbox Arbeit/Mister Bishop Studios/Ionic Tests/roomary/node_modules/.bin:/usr/local/Cellar/node5/5.12.0/bin:/usr/local/bin:/usr/bin:/bin:/usr/sbin:/sbin:/opt/X11/bin
10 verbose lifecycle ionic-hello-world@~build: CWD: /Users/floriannorbertbepunkt/Dropbox/Dropbox Arbeit/Mister Bishop Studios/Ionic Tests/roomary
11 silly lifecycle ionic-hello-world@~build: Args: [ '-c', 'ionic-app-scripts build' ]
12 silly lifecycle ionic-hello-world@~build: Returned: code: 1  signal: null
13 info lifecycle ionic-hello-world@~build: Failed to exec build script
14 verbose stack Error: ionic-hello-world@ build: `ionic-app-scripts build`
14 verbose stack Exit status 1
14 verbose stack     at EventEmitter.<anonymous> (/usr/local/lib/node_modules/npm/lib/utils/lifecycle.js:255:16)
14 verbose stack     at emitTwo (events.js:100:13)
14 verbose stack     at EventEmitter.emit (events.js:185:7)
14 verbose stack     at ChildProcess.<anonymous> (/usr/local/lib/node_modules/npm/lib/utils/spawn.js:40:14)
14 verbose stack     at emitTwo (events.js:100:13)
14 verbose stack     at ChildProcess.emit (events.js:185:7)
14 verbose stack     at maybeClose (internal/child_process.js:850:16)
14 verbose stack     at Process.ChildProcess._handle.onexit (internal/child_process.js:215:5)
15 verbose pkgid ionic-hello-world@
16 verbose cwd /Users/floriannorbertbepunkt/Dropbox/Dropbox Arbeit/Mister Bishop Studios/Ionic Tests/roomary
17 error Darwin 15.5.0
18 error argv "/usr/local/Cellar/node5/5.12.0/bin/node" "/usr/local/bin/npm" "run" "build"
19 error node v5.12.0
20 error npm  v3.10.8
21 error code ELIFECYCLE
22 error ionic-hello-world@ build: `ionic-app-scripts build`
22 error Exit status 1
23 error Failed at the ionic-hello-world@ build script 'ionic-app-scripts build'.
23 error Make sure you have the latest version of node.js and npm installed.
23 error If you do, this is most likely a problem with the ionic-hello-world package,
23 error not with npm itself.
23 error Tell the author that this fails on your system:
23 error     ionic-app-scripts build
23 error You can get information on how to open an issue for this project with:
23 error     npm bugs ionic-hello-world
23 error Or if that isn't available, you can get their info via:
23 error     npm owner ls ionic-hello-world
23 error There is likely additional logging output above.
24 verbose exit [ 1, true ]

Precisely my problem: #https://github.com/driftyco/ionic-app-scripts/issues/114

@florianbepunkt BTW paste the terminal output here as well OK?

Issue solved. Problem was wrong typings definition. Please take a look here: https://github.com/backand/backand-ionic2-starter/issues/4
Will close this.

@florianbepunkt Good luck.


why do some original functions not exist in this project?

I could't found them in go-sdl project, but we really need them!
why not add these functions?

These functions are for SDL1 and are removed from SDL2. Bindings are for ver2 and there you create SDL_Texture with YV12 format for something like you want to do.

Thanks~


cmd/cue: export ignores trailing component of --path value

What version of CUE are you using (cue version)?

$ cue version
cue version v0.9.2

go version go1.22.4
      -buildmode exe
       -compiler gc
       -trimpath true
     CGO_ENABLED 0
          GOARCH amd64
            GOOS linux
         GOAMD64 v1
cue.lang.version v0.9.2

Does this issue reproduce with the latest stable release?
0.9.2 is latest.
What did you do?
! exec cue export f.yml --path baz
stderr baz
! exec cue export f.yml --path foo:bar:baz
stderr baz
-- f.yml --
data: true

What did you expect to see?
I expected either:

that the same error would strike twice, indicating that the bare baz (without a trailing :) couldn't be resolved, or
that the successfully-exported output would contain the non-CUE data at a path location that included a literal baz component.

What did you see instead?
$ testscript -continue cue.export.ignoresTrailingPathElement.txtar
> ! exec cue export f.yml --path baz
[stderr]
error evaluating label baz: reference "baz" not found
[exit status 1]
> stderr baz
> ! exec cue export f.yml --path foo:bar:baz
[stdout]
{
    "foo": {
        "bar": {
            "data": true
        }
    }
}
FAIL: /tmp/testscript295385525/cue.export.ignoresTrailingPathElement.txtar/script.txtar:3: unexpected command success
> stderr baz
FAIL: /tmp/testscript295385525/cue.export.ignoresTrailingPathElement.txtar/script.txtar:4: no match for `baz` found in stderr
> stdout baz
FAIL: /tmp/testscript295385525/cue.export.ignoresTrailingPathElement.txtar/script.txtar:5: no match for `baz` found in stdout
error running cue.export.ignoresTrailingPathElement.txtar in /tmp/testscript295385525/cue.export.ignoresTrailingPathElement.txtar

I expected either:

that the same error would strike twice, indicating that the bare baz (without a trailing :) couldn't be resolved, or
that the successfully-exported output would contain the non-CUE data at a path location that included a literal baz component.

I think the only valid answer here is option 1 FWIW.

I think the only valid answer here is option 1 FWIW.

Agreed. The test script reflects that option.


Remove IDamageableComponent.HealthChangedEvent

About the PR
Makes DoAfters use DamageChangedMessage instead of the event, as it was the last thing left using it.
Could cache the doafters that actually need to care about damage but I'm not sure how important that is @metalgearsloth

90% of the time there's only 1 DoAfter active at most (outside of stripping) so I doubt it matters.


Missing documentation how to use labels or env for advanced log tag options

File: config/containers/logging/log_tags.md does not explain how to use labels or env. Although drivers like syslog reference this page for advanced log tag options config/containers/logging/syslog.md

I could not find a way to use env variables. But for labels it looks like the following works:
{{.ContainerLabels.labelname}}

or
{{index .ContainerLabels \"labelname\"}}

I'm not sure if this is they way the label should be access. The naming looks somehow internal.
If a label is not set we get a "<no value>" string. To suppress it we can use " -" before to closing curly braces
{{index .ContainerLabels \"labelname\" -}}


chore: Cap urllib3<2

Feature or Bugfix

Bugfix

Detail

Cap urllib3 to keep compatibility with AL2 (where OpenSSL <1.1.1)
Re-built lambda layers for 3.4.1 with urllib3<2

Relates

https://github.com/aws/aws-sdk-pandas/issues/2512
https://github.com/aws/aws-sdk-pandas/pull/2447

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

AWS CodeBuild CI Report

CodeBuild project: GitHubCodeBuild8756EF16-4rfo0GHQ0u9a
Commit ID: 163b8701e1eaee2e28e262c6595a483f04f901dc
Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

Upd: Checking if we can override the version that is packaged with the lambda zip instead, or avoid packaging it altogether and rely on lambda to provide it.

AWS CodeBuild CI Report

CodeBuild project: GitHubDistributedCodeBuild6-jWcl5DLmvupS
Commit ID: 163b8701e1eaee2e28e262c6595a483f04f901dc
Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: GitHubCodeBuild8756EF16-4rfo0GHQ0u9a
Commit ID: 4ad84d53194211de7242df8c974ae7ef9513fc52
Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: GitHubDistributedCodeBuild6-jWcl5DLmvupS
Commit ID: 4ad84d53194211de7242df8c974ae7ef9513fc52
Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository


rtl: true problem

When rtl is true i cannot click on any date, I use it for Arabic locale.
It doesn't react on click event on any date.
    $('.datepicker-popup').datepicker({
      todayHighlight: false,
      language: 'ar'
    });

Any solution for this?

I found the solution, when I use settings orientation: 'bottom' it works fine with rtl.


Feature Request: ng cli option for angular-cli.json name and/or location

I have found a need to modify properties inside the angular-cli.json, particularly the styles and scripts property, depending on what libs I want compiled for various clients.
The ability to provide the name or location of an angular-cli.json config file would solve this easily.
Something like:
ng  -config angular-cli.alt.json
Im assuming the optional property would need to default to angular-cli.json and be available for all of the cli commands. I know there is already error handling for improper config files so I dont see much of a risk with this property.
I would be willing to provide a PR if a) this isn't already in the works, b) there isn't an alt approach that doesn't require modification to this project and c) someone quickly points me to the file where this would occur in.

Another use case is for changing the apps.index, apps.main file names and or locations as well.

This is part of a bigger feature that we want to do, and the reason why apps is an array. The plan is to be able to have several apps inside the same project, with per-app settings. It's a post 1.0 thing though.

@filipesilva  I know you have said this is a post 1.0 feature but would you be able to provide a rough estimate? Would it be available sometime this year or 2018?
Would there be anything I could potentially help with?


About commas

Hello @IonicaBizau , why do you use:

commas at the beginning of the line, not at the end.

Any reason  or is just particulary thing?
Thank you

Hey @juliomatcom!
To be really honest, it's just because I like how it looks like, and I feel more productive too.
Here are some interesting discussions:

http://ajaxian.com/archives/is-there-something-to-the-crazy-comma-first-style
https://gist.github.com/isaacs/357981
http://stackoverflow.com/a/10483845/1420197

For me it's more like a particular thing, but it probably has its pros & cons. 😄


Reuse React Native Flow types

React Native will offer Flow types soon. Use them in Este custom components like TextInput etc.

And this https://github.com/necolas/react-native-web/blob/master/packages/website/guides/getting-started.md#using-flow

Related https://github.com/necolas/react-native-web/issues/1007

After RN update.

Soon TextInput, Text, etc. will have React Native Flow types.

Flow is a dead babe, is dead. TypeScript types for React Native work OOTB.


Turn OSV TXT report into an HTML report

OSV reports are currently plain TXT files.
They should be parsed and turned into proper HTML reports (see Trivy reports).

Closed by d1bdad945f3c09acb614e6ceb52208a88c5cb7f6


Load Model NHITS|Core

What happened + What you expected to happen
Having create an NHITS model, able to fit and to predict, and save the model.
When I load the model and predict, receive this error:
....
File /local_disk0/.ephemeral_nfs/envs/pythonEnv-8adfd926-64bf-41db-939e-3f917799867b/lib/python3.10/site-packages/yaml/representer.py:330, in Representer.represent_object(self, data)
328     listitems = list(listitems)
329 if dictitems is not None:
--> 330     dictitems = dict(dictitems)
331 if function.name == 'newobj':
332     function = args[0]
ValueError: dictionary update sequence element #0 has length 1; 2 is required
....
Any idea?
Versions / Dependencies
torchvision==0.16
neuralforecast==1.6.4
torchmetrics==1.2.0
pandas==1.4.4
protobuf==3.19.4
PyYAML==5.4.1
Reproduction script
model2=NeuralForecast.load("/nhits_study_20231103144339")
model2.predict(futr_df = ts_eval)
Issue Severity
High: It blocks me from completing my task.

I was able to isolate the issue, maybe will be helpful to some one
     trainer_kwargs = {         'accelerator': 'gpu',         'enable_progress_bar' : verbose,         "callbacks": [EarlyStopping("train_loss", min_delta=0.0001, patience=early_stop_patience_steps, verbose=verbose)],
and then build the model
with parameter **trainer_kwargs

Hey @pmelo, thanks for using neuralforecast. Can you provide the configuration of the NHITS and the command you used to save it?


Add EnumerateControllerData method.

I have some use case that I want to filter out just some of the device type, instead of get all the controller device EnumerateControllerData is more compact than the GetAllControllerData in many way.

Hello, thanks for cvontributing.
I would prefer a method that just returns an IEnumerable<Device> which you could use Where() on after. Would that be good enough for your use case?

After taking a second look at  GetAllControllerData signature I agree with you, I will submit only non-predicate version of EnumerateControllerData.

It was easier to add it on my own than fixing the conflict. Thanks for contributing!


libffi-dev package needed for installation on Debian

PIP installation was failing because of a needed package "libffi-dev". After installing it, the build continues.

Also the package "python3-dev".

also: build-essential libssl-dev cargo pkg-config

rust is too old in Debian bullseye to compile the cryptography pip dependency. so "curl --proto '=https' --tlsv1.2 -sSf https://sh.rustup.rs | sh" is needed swell.

I did not experience any of these problems on Raspberry PI OS. However, it could be the case that I had them already installed. But should be everything in the dependencies that is required when installing python3 and python3-venv.
I'm honestly a bit confused about this issue.

It's the Homebridge Raspberry Pi Image (https://homebridge.io/raspberry-pi-image) based on Raspbian Lite (maybe that makes a difference)

Okay, actually I am using exactly that for my productive system ... but I have probably installed all these packages at some point before which is why I did not stumble across that ... seems like I have to test it on a fresh install and check / document what packages need to be installed.
Thank you for your valuable feedback.

@failbit I have just flashed an SD Card with the latest Homebridge Image for Raspberry Pi. I am running a Raspberry PI 3B.
I updated the system with sudo apt update and sudo apt upgrade -y. I have then upgraded the Node Version to 20.9.0 via the hb-service update-node command. Then updated the Homebridge UI and Homebridge itself ... besides the upgrade to bookworm, everything should be uptodate.
I have then installed the plugin, configured it via Homebridge UI and restarted the homebridge.
The plugin complained that python3-venv is not available ... as it should be ... I installed it via apt and restarted again ...
After that the plugin created the venv, updated pip, installed the requirements, discovered my Apple TV, initilized the pairing request and after enter the PIN it showed up in the Home app.
So basically as it should be. I could not reproduce the problems you were mentioning.
Any thoughts on that?

Okay good to know. The only difference then is, that my Homebridge Base-Installation ist really old. I did 2-3 Distribution-Updates, so I guess that's where my missing packages came from. If it works with fresh installation of newer Homebridge-Image, all is good. Can be closed here.


ERROR with CTE names

I'm using hiverunner 3.0.0.
The following query fails if there's an uppercase letter in cteName in the select * clause, regardless of whether or not there's an uppercase in cteName in WITH clause.
To illustrate,
CASE 1:
hiveShell.executeQuery(
      """
        |WITH cteName as
        |  (SELECT someColumn
        |   FROM someTable
        |   WHERE someCondition
        |  )
        |SELECT *
        |FROM cteName
      """.stripMargin)

It fails.
CASE 2:
hiveShell.executeQuery(
      """
        |WITH ctename as
        |  (SELECT someColumn
        |   FROM someTable
        |   WHERE someCondition
        |  )
        |SELECT *
        |FROM ctename
      """.stripMargin)

Succeeds.
CASE 3:
hiveShell.executeQuery(
      """
        |WITH cteName as
        |  (SELECT someColumn
        |   FROM someTable
        |   WHERE someCondition
        |  )
        |SELECT *
        |FROM ctename
      """.stripMargin)

Succeeds.
Using hive and beeline:
The query  succeeds in all cases.
Here's the stack trace:

java.lang.AssertionError
at org.apache.hadoop.hive.ql.parse.QB.rewriteViewToSubq(QB.java:237)
at org.apache.hadoop.hive.ql.parse.QB.rewriteCTEToSubq(QB.java:242)
at org.apache.hadoop.hive.ql.parse.SemanticAnalyzer.addCTEAsSubQuery(SemanticAnalyzer.java:1008)
at org.apache.hadoop.hive.ql.parse.SemanticAnalyzer.getMetaData(SemanticAnalyzer.java:1492)
at org.apache.hadoop.hive.ql.parse.SemanticAnalyzer.getMetaData(SemanticAnalyzer.java:1449)
at org.apache.hadoop.hive.ql.parse.SemanticAnalyzer.genResolvedParseTree(SemanticAnalyzer.java:10205)
at org.apache.hadoop.hive.ql.parse.SemanticAnalyzer.analyzeInternal(SemanticAnalyzer.java:10256)
at org.apache.hadoop.hive.ql.parse.SemanticAnalyzer.analyzeInternal(SemanticAnalyzer.java:10141)
at org.apache.hadoop.hive.ql.parse.BaseSemanticAnalyzer.analyze(BaseSemanticAnalyzer.java:222)
at org.apache.hadoop.hive.ql.Driver.compile(Driver.java:432)
at org.apache.hadoop.hive.ql.Driver.compile(Driver.java:305)
at org.apache.hadoop.hive.ql.Driver.compileInternal(Driver.java:1119)
at org.apache.hadoop.hive.ql.Driver.compileAndRespond(Driver.java:1112)
at org.apache.hive.service.cli.operation.SQLOperation.prepare(SQLOperation.java:99)
at org.apache.hive.service.cli.operation.SQLOperation.runInternal(SQLOperation.java:170)
at org.apache.hive.service.cli.operation.Operation.run(Operation.java:268)
at org.apache.hive.service.cli.session.HiveSessionImpl.executeStatementInternal(HiveSessionImpl.java:410)
at org.apache.hive.service.cli.session.HiveSessionImpl.executeStatement(HiveSessionImpl.java:391)
at org.apache.hive.service.cli.CLIService.executeStatement(CLIService.java:245)
at com.klarna.hiverunner.HiveServerContainer.executeStatement(HiveServerContainer.java:112)
at com.klarna.hiverunner.builder.HiveShellBase.executeStatementWithCommandShellEmulation(HiveShellBase.java:103)
at com.klarna.hiverunner.builder.HiveShellBase.executeStatement(HiveShellBase.java:99)
at com.klarna.hiverunner.builder.HiveShellBase.executeQuery(HiveShellBase.java:89)
at com.klarna.hiverunner.builder.HiveShellBase.executeQuery(HiveShellBase.java:82)

Sorry for the late reply. I've been on vacation. Thanks for the report. I've marked it as a bug for now until we get the time to fix it.
regards
/Pelle

We can reproduce the issue in HiveRunner.
Looks like a bug in Hive 1.2 and I found a very similar case  https://issues.apache.org/jira/browse/HIVE-10698. Solved in Hive 1.3 and 2.0.
When you run the query on the Hive CLI/Beeline, what version of hive are you using?

I tested your CTE queries in Hive CLI 0.14 and 1.2. They all work regardless of case (as you pointed out). Maybe the Hive CLI does some toLowerCase() before passing the query to the HQL Parser.
The SemanticAnalyzer (and QueryBlock) in Hive 1.2 tries to do lower case on most of the identifiers, but sadly missed to do so for all identifiers in the CTE query...
I'll try to see if the Hive CLI does some magic or sets some properties that we can mimic in HiveRunner, otherwise I fear you have to use lower case all over your CTE's until we upgrade to Hive 1.3/2.0

I'm using Hive 1.1.0

Yeah I agree, I believe Hive CLI does something like toLowerCase() to every query because all the table and column names are stored in lower case in Hive.
For now I do toLowerCase() for all my CTEs before running the query, and everything seems to be working fine.

We ran into this issue today (Hive 1.2.1/ latest hiverunner build). Is there a plan to squash this?

Could you please try HiveRunner 3.2.1 (which was released today) and let us know if this is still an issue?

Closing this since no feedback received on above comment. I hope that https://github.com/klarna/HiveRunner/issues/73 fixed this.


Using Multi Search after merging to Laravel Scout

No idea if this is the right place to ask, but;
This driver has been merged upstream to the Laravel Scout repository, however it seems the Laravel team has no intention of implementing multi search for any of the drivers, including Typesense. How would you recommend one to do federated/multi search? Use the driver from this repository or?
Thanks in advance

Also interested to know if there's any clean workaround.

Also have this requirement so have been digging into the libraries a bit to see if there's a lower level undocumented workaround without falliing back to this library.
If we look at the Laravel Scout core typesense engine we can see that it proxies methods dynamically onto the underlying client:
https://github.com/laravel/scout/blob/8af517e61e88c423b9e9b466713e5f3a56e588ac/src/Engines/TypesenseEngine.php#L534
This means we can get hold of the client multisearch by doing this:
$multiSearch = app(Laravel\Scout\EngineManager::class)->engine('typesense')->getMultiSearch();
We can then call perform on this and pass in our query similarly to https://github.com/typesense/laravel-scout-typesense-driver?tab=readme-ov-file#multi-search - syntax slightly different e.g. $multiSearch->perform(['searches' => $searchRequests, $options]);
See https://github.com/typesense/typesense-php/blob/4e41737b4ee416e6b2041f81f5506bdac7c823b2/src/MultiSearch.php#L39 for the signature and how it's implemented in the client.


[query] Constraint-based algebrizing

The current algebrizer — mainly ConjoiningClauses — is deliberately procedural and predictable. Paired with the simple query translator, we quite directly translate Datalog into SQL.
CC performs two roles: it tracks the metadata it needs to not be completely clumsy (the types deduced for variables and known bindings), and it accumulates some information that should really be part of a planner: table allocations and mappings between their columns and variables.
Again, that works fine for simple and predictable translation.
Extending this approach, however, to attain more power gets… hacky.

Query simplification/optimization (e.g., removal of redundant patterns and constraints).
Algebrizing queries down to static results (e.g., [:find ?x :where [?x :db/ident :foo/bar]]).
Value space intersection and optimization (e.g., in [:find ?x :where [?x :foo/age ?z][?y :foo/age ?o] [(!= ?x ?y)] [(> ?z 20)] [(< ?o 30)] [(> ?o ?z)]] we can derive a lower bound for ?o).
Deducing attributes from their ranges (e.g., [:find ?x :where [?x ?y :foo/bar]] can deduce the value of ?y in the current database from the schema alone, and [:find ?x :where [?x ?y true] [?z ?y ?z]] can never bind a single ?y).

At some point it might be worthwhile to more fully separate algebrize+translate into algebrize+plan+translate, with the algebrizing stage being primarily a constraint system used to derive something like CC.
The risk of doing so is that the planner input would be less ordered, and thus the translated output would likely be less predictable and less related to the input. Small changes to the input query could cause large changes to what we can deduce.

@fluffyemily this is what we were chatting about yesterday.

I'm not against doing this, especially in the far future after we learn some lessons about what really happens in the wild, but it's worth pointing out what the fundamental design trade-off encoded into our conversion from Datalog to SQL is.  In my mind, we have always been betting that SQLite's planning (re-order clauses, remove redundant constraints, choose indices efficiently, etc) would be "better" than our own: possibly more efficient, definitely quicker to market.

Yes, this is far future (unless predicates become troublesome for me!). But there are three caveats.

SQLite's planner is good but not sophisticated; it also aims for simplicity and predictability. We should not have high expectations about it 'rescuing' queries — users will need to do tuning. Most of our bet is that even its naïve plans are very quick indeed.
If we ever start caching things in memory or on disk (we already do the former: the schema!) then we unavoidably get into query planning, because we're choosing where and how to join our own data into the SQL query.
We store lots and lots of information that isn't available to SQLite when planning queries. Type ranges and uniqueness constraints, for example, are part of our data and not our disk schema, and so don't factor into query planning. Simple things like reordering queries to put unique patterns first will often improve results.

I realized that I didn't have good conceptual clarity around what exactly CC does. I spent a lot of time thinking this morning, and spent some time with some of my old Prolog books, and I think I have a phrasing that — at least right now — satisfies me.
Datalog, like Prolog, backward-chains for query evaluation. Given a Datalog program/query — a goal — a Datalog engine attempts to find rule and variable substitutions from the knowledge base that prove the goal.
In the very simplest case, this might be the literal existence of a fact in the store — with a store like [123 :foo/name "Alice"], a query like [?x :foo/name "Alice"] is trivially proved by the substitution ?x = 123.
CC — indeed, the algebrizer as a whole — is not a Datalog solver. It's a compiler. We compile a subset of Datalog into SQL that performs query evaluation and substitution of places.
However, compiling and simplifying a Datalog query/program involves something that passes for forward-chaining inference. This is confusing; at times it looks as if we are solving parts of the query, and indeed sometimes we do. However, both the space of inference and the mechanism of evaluation differ from those used by the Datalog engine itself, which in our case is SQLite plus our database schema.
If the SQL type system were a superset of our Datalog type system, CC would do only two things:

Forward-chain rules from the schema and the query to achieve simplification and consistency checking. That is: for a given Datalog query, find a simpler Datalog query that produces the same substitutions. In the special case of the constraints being unsatisfiable, we can statically eliminate the entire query.
Most of our optimizations so far are about pruning patterns that cannot match: absence of an attribute from the schema, for example, causes the containing clause to disappear. But we can go further.
A query like
[[?x :foo/age ?agex]
 [?y :foo/age ?agey]
 [(!= ?x ?y)]
 [(> ?agex 10)]
 [(> ?agey 12)]
 [(> ?agex ?agey)]]

can — with sufficient inference — simplify to
[[?x :foo/age ?agex]
 [?y :foo/age ?agey]
 [(!= ?x ?y)]
 [(> ?agex 12)]
 [(> ?agey 12)]
 [(> ?agex ?agey)]]

and with a lot of eleven-year-olds in the store, this might make a significant difference to execution time.

Transform predicate-oriented query input into column-oriented input. In the same way that we — in a manner of speaking — turn a predicate (foo/name 123 "Alice"), the data tuple [123 :foo/name "Alice"], into a set of columns in the store, we implement [?x :foo/name "Alice"] in SQL by applying constraints to columns: the e of a row can be substituted for ?x if the a is :foo/name and v is "Alice".
Our table shape is not necessarily fixed (#33) and not necessarily matching on a single table (fulltext values). In a conventional SQL database, the presence of a value in a column for a row is equivalent to an assertion, so we have a level of indirection.

Finally, because we'd like to run a single SQL query for a single Datalog query, and because SQL is sufficiently powerful, CC needs to recursively build and combine other CCs to express flow-control predicates like or, not, and get-else.

The result of the CC doing its work is a Datalog program simplified in the context of the current Datalog schema, and compiled to SQL for our particular SQL schema.
Unfortunately, the SQLite type system is a subset of our Datalog type system. As such, a naïve translation of Datalog into SQL will produce spurious results as values unify (in the Prolog sense) incorrectly — entid(123) will unify with double(123.0).
CC therefore has an additional pair of responsibilities:

Encode type constraints from the query such that the resulting compiled SQL is equivalent to the Datalog. Do so as efficiently as possible, eliding type constraints wherever they are implied by other parts of the query, because unnecessary type checks are unnecessary expense.

Preserve information about types and type codes to allow the projector to translate SQL result rows into Datalog substitutions.

Where things get a little fuzzy is the scope of simplification and consistency checking. The implication of this issue is that a sufficiently smart CC can model, via constraints, the work that will be done by the SQL engine, and the space of possible results, using those to derive a simpler Datalog query. The simpler query can be naïvely translated into SQL that does less work than the original query but yields the same results. We know that there's space to do this because some of the knowledge available to the algebrizer — cardinality constraints, for example — is not available to the SQL 'solver' itself.
Three good examples, expanded from the first comment in this thread:

ground can be naïvely implemented via a SQL subquery projecting a single row with two columns (the value and its type code)… but we can do much better if we crunch the numbers up-front and avoid having to walk tables.

Schema queries. A query like [:find ?ident :where [?x :db/ident ?ident]] can obviously be computed without touching disk. We could special-case these schema properties, but it's also worth considering how they can be phrased as something like ground, using conventional query substitution and simplification to transform this into:
[:find ?ident :where [(ground [":db/cardinality" ":db/valueType" …]) [?ident])]]

which we can optimize more generally.

Attribute inference: if there's only a single boolean attribute in the schema, then given a pattern like [?x ?a true] we can immediately deduce the binding of ?a, potentially pruning the search space substantially when substituted elsewhere in the query. The SQL solver would need to reach that pattern and run it to find out what ?a might be, which is less efficient.


Max line length broken on 1.22.2-android

On the latest version 1.22.2-android of the plugin, the max line length gives false positives at around 120 characters even though my .editorconfig and detekt configuration yml specify 140 characters. Reverting back to 1.22.1-android resolves this issue.
If required I can create an example project for this

That's really strange. Yes, a reproducer project would be ideal thanks.

Hi @arturbosch I tried reproducing it in a new project with the same configs, but it worked fine.
After some investigating we found that removing the detekt-configuration.yml from Android Studio settings and applying it again, fixed the issue.
We haven't had this before so it's something that broke in the latest version. After re-applying, it works fine and updates to the configuration do to.
I think this issue can be closed


[Feature Request]: Interactive Team Details Pages

Is there an existing issue for this?

[X] I have searched the existing issues

Feature Description
This feature enables users to click on icons within the "Teams" section (e.g., "Service Manager," "Rescue Specialist") to view detailed information about each team member or group. When an icon is clicked, the website will display a dedicated page or popup with profiles, roles, and other relevant information.
Use Case
Users visiting the site can click on team icons to access specific information about each team, such as their responsibilities, expertise, and team members. This provides a more engaging and informative experience, allowing users to better understand the services and personnel available.
Benefits

Improved User Experience: Offers an interactive and engaging way for users to explore the teams and their roles.
Transparency: Provides clear information about the teams, building trust and credibility with users.
3 .Efficient Navigation: Allows users to access relevant information quickly and easily by clicking on icons, enhancing website functionality.
4 .Enhanced Engagement: Keeps users engaged by offering more in-depth information, encouraging them to explore the site further.

I hope may you understand the feature, if not then please tell me i will explain it more clearly.
Please assign me this issue under the labels of Gssoc-extd, hacktoberfest, hacktoberfest-accepted and relevant level.
Priority
High
Record

[X] I agree to follow this project's Code of Conduct
[X] I'm a GSSOC contributor
[X] I want to work on this issue
[X] I'm willing to provide further clarification or assistance if needed.

Thank you for assigning me this issue. I will resolve this issue very soon .

Thank you for assigning me this issue. I will resolve this issue very soon .

actually homepage broken, you have to work on this issue with that.

Hey !, if you don't mind, Can you please assign level 2 to this issue . it will be very helpful for me.
Its my humble request . else there is no issue I will work over it with level 1 as well.

Yes I understand , I will develop the complete teams page for this site., that when the user click on the teams page on the navbar , then he will get to see the team members and their details

Thank you so much for assigning level 2 on this issue, I will resolve this very soon.

Hey @mansiruhil13 ! I  had merge the PR , but I am getting conflicts in it. can you please help me to resolve them , Actually I am new to GitHub so I had not such knowledge that how to resolve them. It will be highly appreciable if you help me to resolve these conflicts.


Collision offset

I can not do that the rigidbody would fall to different heights.
Maybe add option like as offset?

What I want:

Feature and improvement proposals for the Godot Engine are now being discussed and reviewed in a dedicated Godot Improvement Proposals (GIP) (godotengine/godot-proposals) issue tracker. The GIP tracker has a detailed issue template designed so that proposals include all the relevant information to start a productive discussion and help the community assess the validity of the proposal for the engine.
The main (godotengine/godot) tracker is now solely dedicated to bug reports and Pull Requests, enabling contributors to have a better focus on bug fixing work. Therefore, we are now closing all older feature proposals on the main issue tracker.
If you are interested in this feature proposal, please open a new proposal on the GIP tracker following the given issue template (after checking that it doesn't exist already). Be sure to reference this closed issue if it includes any relevant discussion (which you are also encouraged to summarize in the new proposal). Thanks in advance!