[WIP] First implementation of density reweighting

Description
I have finished a first pass at roughly implementation for allowing reweighting of densities. This required quite large changes to parts of the storage system - in particular to add support for serializing data for properties such as enthalpy of mixing.
For now most of the reweighting implementation lives in the density.py class. While this might be ok (but not the best way) for a small number of built in properties (i.e reweighting implementation in the class), this needs to be assessed and redesigned to be more modular and flexible in the future.
This PR also mostly fixes up the enthalpy_of_mixing implementation, adding support for storing all of the pure component trajectory data.
Status

[ ] Ready to go

This pull request introduces 3 alerts when merging 73febe2cf8dc2cfd6d909a8e7710ed9d07759b35 into d1b0c0b4c74d01e63e0660889cccd749e1a8c306 - view on LGTM.com
new alerts:

2 for Unused import
1 for Incomplete ordering

Comment posted by LGTM.com

In particular, I bet there's a way to greatly simplify some of the Protocol code by using openmmtools.mcmc.
I think there's also an opportunity to simplify the handling of Ensembles by extending the API of ThermodynamicState to know what thermodynamic ensemble it defines. In essence, it already knows this, since it knows how to compute the reduced potential from relevant thermodynamic parameters and dynamical variables---which is all that really matters---but it could expose some useful additional API points to return human-interpretable ensembles (e.g. NVT, NPT, µVT) if that would be helpful.

Also, it would be great to eventually migrate the Protocol code into openmmtools if it looks like a flexible way to build simulation pipelines and do distributed computing with them!

This pull request introduces 3 alerts when merging e1dfbb73cb4b29a6a3fc3012974d5e92477aa7e5 into d1b0c0b4c74d01e63e0660889cccd749e1a8c306 - view on LGTM.com
new alerts:

2 for Unused import
1 for Incomplete ordering

Comment posted by LGTM.com

This pull request introduces 2 alerts when merging 8559498eb7647d877eef8da308bddda695f7112a into 9424366d119cad0c35127dedebfb1bf70fe77bb8 - view on LGTM.com
new alerts:

1 for Unused local variable
1 for Unused import

Comment posted by LGTM.com

This pull request introduces 1 alert when merging 6cb57c6d918b92f446153926b497ff9ad4385bc3 into 9424366d119cad0c35127dedebfb1bf70fe77bb8 - view on LGTM.com
new alerts:

1 for Unused import

Comment posted by LGTM.com

I've gone ahead and implemented the majority of the suggestions. Once the travis and lgtm checks have completed, I'm going to merge and close this PR. If any one has any other feedback feel free to drop it here or in an issue and I'll open a new PR to address to save this PR getting even more bloated.

This pull request introduces 4 alerts when merging 02c8dc927c8984099c8f6261414729dbe8117da5 into 9424366d119cad0c35127dedebfb1bf70fe77bb8 - view on LGTM.com
new alerts:

4 for Unused import

Comment posted by LGTM.com


Add baseUrl and paths in tsconfig.json

What does this PR do?
This will help VSCode and other IDE to resolve path in TypeScript
Any background context you want to provide?
https://create-react-app.dev/docs/importing-a-component/#absolute-imports
What are the relevant tickets?
Fixes #
Checklist

[ ] Added relevant tests
[x] Didn't break anything

@hackerwins  @dc7303
I added it as it would be convenient during development.
https://medhatdawoud.net/blog/paths-and-baseUrl-in-tsconfig.json

@hackerwins  @dc7303
I added it as it would be convenient during development.
https://medhatdawoud.net/blog/paths-and-baseUrl-in-tsconfig.json


Claim cartification card shows completed step without signing in.

Affected page
Certification pages
Expected behavior
Users should be singed in before their requirements are checked.
Screenshots

@ahmadabdolsaheb  This will be fixed in the pr I made #43335.


Cannot connect to Metro (Flipper)

Description
Hello.
Hermes Debugger doesn't work in Flipper.
Error:
Cannot connect to Metro. Please check that metro is running and Flipper can connect to it.

I'm not sure but as I know localhost:8081/json should work but in my case it returns error:
Cannot GET /json/
Metro is started and works well.
How it can be fixed?
Thank you.
Version
0.68.2
Output of npx react-native info
System:
OS: macOS 12.4
CPU: (12) x64 Intel(R) Core(TM) i9-8950HK CPU @ 2.90GHz
Memory: 29.08 MB / 32.00 GB
Shell: 3.2.57 - /bin/bash
Binaries:
Node: 16.13.2 - ~/.nvm/versions/node/v16.13.2/bin/node
Yarn: 1.22.11 - ~/.nvm/versions/node/v16.13.2/bin/yarn
npm: 8.1.2 - ~/.nvm/versions/node/v16.13.2/bin/npm
Watchman: 2<IP_ADDRESS> - /usr/local/bin/watchman
Managers:
CocoaPods: 1.11.3 - /usr/local/bin/pod
SDKs:
iOS SDK:
Platforms: DriverKit 21.4, iOS 15.5, macOS 12.3, tvOS 15.4, watchOS 8.5
Android SDK:
Android NDK: 21.3.6528147
IDEs:
Android Studio: 2021.2 AI-212.5712.43.2112.8609683
Xcode: 13.4.1/13F100 - /usr/bin/xcodebuild
Languages:
Java: 11.0.15 - /usr/bin/javac
npmPackages:
@react-native-community/cli: 7.0.3 => 7.0.3
react: 18.1.0 => 18.1.0
react-native: 0.68.2 => 0.68.2
react-native-macos: Not Found
npmGlobalPackages:
react-native: Not Found
Steps to reproduce
Run the mobile app. Open the debug menu. Tap "open debugger". In Flipper choose "Hermes Debugger (RN)"
Snack, code example, screenshot, or link to a repository
no code

How it can be fixed?

Not enough information to debug this. Are you on a Android or iOS emulator?
On Android have you tried to adb reverse tcp:8081 tcp:8081 to see if this solves?

localhost:8081/

Are you opening this from your computer or from the device? Does it work in any of the two?

It happens on my computer (macOS).
http://localhost:8081/json returns error Cannot GET /json/
http://localhost:8081 works fine.
So my question is how to fix http://localhost:8081/json

Can I ask you to open an issue on https://github.com/facebook/metro/issues
explaining your environment and mentioning that http://localhost:8081/json returns error Cannot GET /json/
You can link this issue to speed up the triaging there.
Closing as this is not a React Native issue per se.

How it can be fixed?

Not enough information to debug this. Are you on a Android or iOS emulator? On Android have you tried to adb reverse tcp:8081 tcp:8081 to see if this solves?

localhost:8081/

Are you opening this from your computer or from the device? Does it work in any of the two?

it worked, thanks alot.

How to solved it for iOS simulator?

i am facing same issue with a live device


Prevent recursion into the body elements of the pact file

Hi,
I had an issue where, while the pact file was being verified, it was uppercase-ing an element in the request body, the element was named method.
The recursiveTransformJson method, runs over the whole pact including request and response bodies, this means that if the element names being transformed here appear in the request or response they will be incorrectly transformed.
Please consider this fix for this issue.
Thanks,
Simon

Thanks for the PR!
I might move the uppercasing of the method out of the recursive call as it does not need to be there.

I've actually removed the recursion all together as it did not make much sense.

Fair enough, I wasn't going to be so bold!
Thanks for looking into it :)


Create instance of ArrayParser directly

Q
A

Is bugfix?
❌

New feature?
❌

Breaks BC?
❌

Fixed issues
#297

Need to add line to changelog only


Expired certificate for https://influxdata.com

The certificate to https://influxdata.com expired 25 November 2019.

There is a couple of links to this page from this repository:

Just wanted to let you guys know about it :)

@groenlid thanks for reporting.  @tkyocum is working on it.

This has been fixed. Thanks for reporting it.


Test all exercises

After updating all exercises, go through all exercises and follow the steps, verifying that all steps are correct and produce the correctly working workshop apps.

[ ] Android
[ ] AppStudio
[x] iOS
[ ] Java
[x] macOS
[ ] .NET
[ ] Qt

I completed testing for iOS at #126 / 57c1bcf6a0e0cfd1661bcf5f71de685cdfafe327.

I completed testing for macOS at #127 / bbe2797cbdf342193e15e74f234e4c80755ebe78.

I completed testing for Android at #128 / 51a0de37cbd2785a59758abcdabd2dbe960a9cac .

I completed testing for Qt at #129 / b7e1ada89b8cdbb25a6776cb53bdf047cebdcb62 .

I completed testing for Java at #130 / 2e36c554e292edd96b94bc44a128032e952f643b .


Does transaction malleability (TXID change) break this tool?

(Because Non-segwit transactions can be malleated)

no.
the worst case situation would be that the refund transaction has to be recreated, because the contract transaction was malleated.


Documentation Error? san.cnf sample file missing

Describe the bug
A clear and concise description of what the bug is.
May need to follow up on this error that I'm trying to debug, but under the 'TLS Certificates and HTTPS' and 'Creating a PEM File', step #2, you said that there would be a sample san.cnf file to use as reference in the documentation (as attached below)

Hey @MaanasPeri23 well spotted 👏 I've just fixed the documentation.
You can find this file in the support folder at:
https://github.com/Voyz/ibeam/blob/master/support/san.cnf
Thanks for catching this 😊


Add CMake commands to install library for find_package.

Codecov Report
Patch and project coverage have no change.

Comparison is base (d75e4cd) 97.35% compared to head (6baf4bd) 97.35%.

:exclamation: Current head 6baf4bd differs from pull request most recent head 552f560. Consider uploading reports for the commit 552f560 to get more accurate results

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
Additional details and impacted files
@@           Coverage Diff           @@
##           master      #11   +/-   ##
=======================================
  Coverage   97.35%   97.35%
=======================================
  Files          29       29
  Lines        2535     2535
  Branches       48       48
=======================================
  Hits         2468     2468
  Misses         39       39
  Partials       28       28

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Enable GHC 8.8.1 windows ci

Since https://github.com/jacobstanley/unix-compat/pull/42 has been closed.
Closes #121

I am getting an error, i think it could be cause i have ghc-8.6.5 as default ghc in $PATH but i am running the tests with ghc-8.8.1:
> cabal test -w ghc-8.8.1
....
 Initial module load (0.79s)
    bios-tests.exe: unable to load package `ghc-prim-0.5.3'
    simple-stack:              bios-tests.exe:  | D:\bin\stack\x86_64-windows\ghc-8.6.5\lib\ghc-prim-0.5.3\HSghc-prim-0.5.3.o: unknown symbol `stg_atomicModifyMutVarzh'

Access violation in generated code when reading 0x30

D:\bin\stack\x86_64-windows\ghc-8.6.5\lib\ghc-prim-0.5.3\HSghc-prim-0.5.3.o
Looks weird.
Well, CI succeeded T_T

I did not execute it locally, so, I am not sure. Previously, CI failed for GHC 8.8.1.


There is an activity log if clicking the 'X' to close the dialog prompted when uploading files to one SAS attached blob container/file share with permissions (List, Write)

Storage Explorer Version: 1.27.0-dev
Build Number: 20221020.1
Branch: main
Platform/OS: Windows 10/Linux Ubuntu 22.04/MacOS Monterey 12.6 (Apple M1 Pro)
Architecture: ia32/x64
How Found: Ad-hoc testing
Regression From: Not a regression
Steps to Reproduce

Expand one storage account -> Blob Containers.

Select one blob container -> SAS attach the blob container with List & Write permissions.

Go to the attached blob container -> Upload one blob.

There is a prompt dialog -> Close the dialog by clicking the 'X'.

Check there is no activity log.

Expected Experience
There is no activity log.
Actual Experience
There is an activity log.

Additional Connext
This issue also reproduces for attached file shares/ADLS Gen2 blob containers.

Verified this issue on the main build 20221026.5. Fixed.


add single quotes for credentials

also strip inital credentials for single quotes, for backward compatibility

Thank you for a pull request! Please check this document for how the Jenkins project handles pull requests

A possible alternative solution would be to shellquote().

@iakovgan This PR was fixed up and merged as #395.  Thank you for contributing!


feat: add structured logging for hmi-server

This PR adds the ability to enable structured logging to hmi-server. We need to be able to parse logs in Loki, to do this we can enable json logging in staging and production.  With the logs provided I was able to create the following dashboard:

We can provide a breakdown of

Request duration by user (top graph)
Request duration by endpoint (bottom graph)

This allows us to see who's using (and potentially taxing) the system, and what endpoints are slower than others

We also add some tables that allows filtering/sorting by request as well as a simple table of exceptions that occur in the backend.

Sorry for jumping in :P.  Wanted to make a comment on whether an update to LoggingService.java is also needed.

Sorry for jumping in :P. Wanted to make a comment on whether an update to LoggingService.java is also needed.

As we discussed offline, this isn't needed and is just handled already.


Update to react v15.0

See https://facebook.github.io/react/index.html
After upgrading to<EMAIL_ADDRESS>npm starts complaining:
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none was installed.

Patch example: https://github.com/STRML/react-grid-layout/commit/ef335bcc11652df0ff7cd649d9bc265acf339faf
(not submitting a PR since I don't know whether it is just about changing a version or there should be some modifications in the code too)

I think if since there's no warnings it shouldn't need any changes -  || ^15.0.0 to peer deps and should be good to go


[feature] --codimd style generator without lowercasing the words

When trying to use doctoc with a codimd https://demo.codimd.org/ markdown document which renders an HTML document (either side by side or as "view only") it failed to work because the auto-generated anchors keep the uppercase caracters, so # Introduction gets you an #Introduction anchor, not #introduction as is generated by github and others.
I've filed an issue on codimd https://github.com/hackmdio/codimd/issues/1590 but I guess this could also be changed on doctoc's side.
Thanks for your feedback.

Based off that thread, it sounds like maaaaaybe that's a thing on codimd side? I don't know what that system does.


[Variogram] [Block] [inblock_semivariance] [L41] Check if commented part is faster when dataset size increases

Is your feature request related to a problem? Please describe.
n/a
Describe the solution you'd like
Currently, inblock_semivariance calculations are depended only on the numpy. But there COULD BE a chance that calculations may run faster for the very large datasets if performed only for unique distances. Test this assumption with commented function:
...
    # unique_distances, uniq_count = np.unique(distances_between_points, return_counts=True)  # Array is flattened here
    # semivariances = variogram_model.predict(unique_distances)
    # multiplied_semivariances = semivariances * uniq_count
...

Describe alternatives you've considered
n/a
Additional context
n/a

Ok, it is almost never faster, code to check it:

import numpy as np

def some_model(x):
    return (100 + 9.8*x)

def inblock_semivariance(distances_between_points) -> float:
    semivariances = some_model(distances_between_points)

    average_block_semivariance = np.sum(semivariances) / len(semivariances)
    return average_block_semivariance

def inblock_semivariance_unique(distances_between_points) -> float:

    # TODO: part below to test with very large datasets
    unique_distances, uniq_count = np.unique(distances_between_points, return_counts=True)  # Array is flattened here
    semivariances = some_model(unique_distances)
    multiplied_semivariances = semivariances * uniq_count

    average_block_semivariance = np.sum(multiplied_semivariances) / len(semivariances)
    return average_block_semivariance

if __name__ == '__main__':
    import pandas as pd
    from datetime import datetime

    outputs = []

    for idx in range(0, 20):
        for arr_size in np.logspace(1, 7, 20):
            for x_limit in [10, 100, 1000, 5000, 20000, 100000]:
                arr_size = int(arr_size)
                x_limit = int(x_limit)

                print(idx, arr_size, x_limit)

                arr = np.random.randint(0, x_limit, arr_size)

                t_start_0 = datetime.now()
                _ = inblock_semivariance(arr)
                secs_0 = (datetime.now() - t_start_0).total_seconds()

                t_start_1 = datetime.now()
                _ = inblock_semivariance_unique(arr)
                secs_1 = (datetime.now() - t_start_1).total_seconds()

                d = {
                    'idx': idx,
                    'arr_size': arr_size,
                    'max_element': x_limit,
                    'plain time': secs_0,
                    'uniq time': secs_1,
                    'experiment': 'integer random numbers'
                }

                outputs.append(d)

    df = pd.DataFrame(outputs)
    df.to_csv('inblock_times_integer.csv')


[Doc] Archive Documentations by version

Search before asking

[X] I had searched in the issues and found no similar issues.

Documentation Related
Docs in website shoule be archived by version, which can be updated in this path: https://github.com/apache/incubator-eventmesh-site/tree/master/docs
Are you willing to submit PR?

[ ] Yes I am willing to submit a PR!

Completed in https://github.com/apache/eventmesh-site/pull/126. Further improvements are needed in https://github.com/apache/eventmesh-site/issues/135.


Finding a Public Agent IP - Improve Oneliner

What are your changes?
Add and .attributes.public_ip == "true" to the select(...) so that we get just the public agents, not the public agents and masters.
I'm new to dc/os so I'm not actually sure this is guaranteed to work forever, as I'm not that familiar with the promises of the dcos node api, but it seems to be working for me. Comments are appreciated. Maybe the correct fix is actually to make sure slave_public is null for masters? That, however, would be a code change for which I'm certainly not qualified.
What type of changes does your doc introduce?

[X] Bug fix
[ ] New feature

What is the urgency level of this change?

[ ] Blocker
[ ] High
[X] Medium

I have completed these items:

[X] I have tested all commands and code snippets.
[ ] I have built locally and tested for broken links and formatting.
[ ] I have made this change for all affected versions (e.g. 1.7, 1.8, and 1.9).
[ ] My doc follows the style guide: https://github.com/dcos/dcos-docs#contributing.

Ping @emanic, @sascala, or @joel-hamill for reviewing pull request changes.
If you included a comment with your commit, it appears here:
Add and .attributes.public_ip == "true" to the select(...) so that we get just the public agents, not the public agents and masters.
I'm new to dc/os so I'm not actually sure this is guaranteed to work forever, as I'm not that familiar with the promises of the dcos node api, but it seems to be working for me. Comments are appreciated. Maybe the correct fix is actually to make sure slave_public is null for masters? That, however, would be a code change for which I'm certainly not qualified.

Can one of the admins verify this patch?

The bash script is correct so a NOP here but the output should indeed be corrected (last line removed).

Thanks to Michael Hausenblas for help debugging some redherrings in my cluster while looking at this oneliner. After talking with him we realized we don't need a change to the code in this documentation. (Hence the second commit and changes to this PR)


Firebase Storage : File at URL is not reachable

[REQUIRED] Step 1: Describe your environment

Xcode version: 13.3
Firebase SDK version: 8.11.0
Installation method: CocoaPods
Firebase Component: Storage
Target platform(s): iOS

[REQUIRED] Step 2: Describe the problem
Here is a kind of error i am getting from crashlytics as a non-fatal error :
Non-fatal Exception: FlutterError
[firebase_storage/unknown] File at URL: file:///var/mobile/Containers/Data/Application/C7947755-75F0-4B08-A1FB-DA09B2E75018/Documents/tempdir/posts/c9ba4b10-a695-11ec-4f446ee21d88-tempcompress-main.mp4 is not reachable. Ensure file URL is not a directory, symbolic link, or invalid url.. Error thrown Uploading main video file failed...

I know that this is a flutter error, however i have checked their source code and found out that they simply use the native SDK error handling to throw their errors, so i guess this error comes from the native SDK.
I would like to understand where i can find this error in the SDK implementation please especially to understand what's going on here.
It's an error that is thrown on a simple UploadTask that fails to putFile.  I know it despite i can't reproduce it in debug mode, because of the Error thrown Uploading main video file failed... that is my custom error message and not part of the Firebase storage error message.
I have been trying to look for File at URL and could not find it here
I have also not been able to find it in the firebase_storage source code.
Could you please highlight me on this ? So i can understand why an UploadTask would look for a local file as an URL or something ?
If you know that the error does not come from the native SDK i will make the issue in the flutter repo.
Thanks a lot in advance !

The source for the error is at https://github.com/firebase/firebase-ios-sdk/blob/master/FirebaseStorage/Sources/FIRStorageUploadTask.m#L207

Closing for staleness. Further investigation could be done by setting a breakpoint in Xcode.


Is there a way to capture the inference output through the code into a dataframe?

System Info
I am trying to capture this data to be concise.
Is there a way to store this data into a dataframe? or record this data while a particular prompt is running?

Information

[ ] Docker
[X] The CLI directly

Tasks

[X] An officially supported command
[ ] My own modifications

Reproduction
I need to capture the following data which recording when an inference is passed.

Expected behavior
Any code or any way to do it>

Those logs are made to be consumed with telemetry: https://github.com/huggingface/text-generation-inference#distributed-tracing


I am experiencing a 403 error when sending tracking data to my Matomo server using the matomo_tracker package in my Flutter app

Describe the bug
I am experiencing a 403 error when sending tracking data to my Matomo server using the matomo_tracker package in my Flutter app. The same URL works fine when used in native apps (Swift).
To Reproduce
Steps to reproduce the behavior:

Create a new Flutter app and add the matomo_tracker package as a dependency
Configure the MatomoTracker with the following settings:MatomoTracker.instance.initialize(
  siteId: 6,
  url: 'https://matomo.exemple.com/piwik.php',
);

The server responds with a 403 error.

Expected behavior
The tracking data should be sent successfully to the Matomo server without a 403 error.
Smartphone (please complete the following information):

Device: iPhone 14
OS: iOS16.2

Additional context
The same Matomo URL works fine in native apps (Swift).
I have already checked the Matomo server settings, site ID, and token authorization, but the issue persists.
Please let me know if there's any known issue or solution for this problem.

Can you provide the cURL of the failing request ?

Thank you Maniak for your answer
the same request works with postman and not with flutter i used 2.0.0 i tried both piwik.php and matomo.php

Ok but can you provide the cURL of the failing request (you can find in the network tab of the dart devtools). This will help to identify what might be going wrong with the request sent from the package.

You may use
MatomoTracker.instance.initialize(
siteId: 6,
url: 'https://myhost.matomo.cloud/matomo.php',
);


update some fields option

Codecov Report

Merging #50 (dbb5cf2) into master (771d08c) will not change coverage.
The diff coverage is n/a.

:exclamation: Current head dbb5cf2 differs from pull request most recent head 6e4b021. Consider uploading reports for the commit 6e4b021 to get more accurate results

@@           Coverage Diff           @@
##           master      #50   +/-   ##
=======================================
  Coverage   98.73%   98.73%
=======================================
  Files          73       73
  Lines       12136    12136
=======================================
  Hits        11983    11983
  Misses        122      122
  Partials       31       31

Impacted Files
Coverage Δ

pkg/camt_v05/camt.go
100.00% <ø> (ø)

pkg/camt_v08/camt.go
98.07% <ø> (ø)

pkg/camt_v09/camt.go
100.00% <ø> (ø)

Continue to review full report at Codecov.

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

updated tests...


Prep package.json for publish

FYI @slimbuck, lint check had flagged up some 'errors' around the order of getter and setters


Add UTC date converter

I sometimes have to edit a cell in the database. But only accepted format is UTC ISO format.
I would love to have a tool which:

can use date picker to pick any date
user can also enter a date by hand
default value for input would be now
convert date to UTC format
easy copy UTC date

The tool Unix timestamp converter can convert timestamp to Unix time. I think we just need to improve it to easily pick datetime, instead of manually entering timestamp. Will that solve your use case?

@Morasiu now you can use date picker to easily pick the timestamp like this.
Test it: https://truedevtools.com/unix-timestamp-parser

Almost. I need to have ability to copy ISO 8601 formatted string.
Example:
2024-05-04T04:19:50Z
Also not sure if Date picker works on mobile

well.. bad UX for sure

@Morasiu @kokoropie Now the UTC time follows ISO 8601 format. I 've decided to remove the date picker on mobile.
Additional big update: dark mode supported!

@dathoangnd mode switch don't show on width < 640px

@kokoropie I have added the dark mode button on mobile now.


[AdaptiveCards- Samples - Input with Validation]: Screen Reader conveys the incorrect role for the 'Submit' button.

https://microsoft.visualstudio.com/OS/_workitems/edit/35920091

This issue is not repro, hence closing this issue.


Poll: should the location directory be created recursively?

Follow-up for https://github.com/Level/classic-level/pull/6.
In the following example, should the location directory be created recursively? Such that, if the foo directory does not exist, it will be created (on open) rather than yielding an error?
const db = new ClassicLevel('foo/bar')

Creating it recursively is the current behavior of classic-level (and new compared to leveldown) which may break expectations given typical filesystem behavior, or it could be a convenient feature, if the database is considered to abstract away the filesystem.
React with thumbs up to create the directory recursively, react with thumbs down to yield an error. This question is about what the default behavior should be, so I'm purposefully not including a poll option to make either behavior opt-in.

@juliangruber Could you please check that the above text is not biased towards my personal preference?


Updated REAMDE.md by removing a dead link.

Summary of your changes
Removed a dead link in the README.md file.
Description
Fixes: #1863

Section: Interview Preparation section, link: Core Java Interview questions - Interview question on each topic
The link is dead and the site is up for sale, so I've removed the link.

Checklist

[x] My change follows the Contributing Guidelines
[ ] I have added only one new link to the list.
[ ] I have checked that the link that I added does NOT exist in the project already.
[ ] I have sorted the link alphabetically under the related section.

I have found an alternative in the Javatpoint website for core-java-interview-questions.
Is it ok if I commit those changes in this PR only?
Or should I open another one after #1863 is resolved and this pr is closed?

@sdmg15 what do you say?

@sdmg15 is the pr alright?
Or are there any changes to be made?

@sdmg?


Refactor functions in stream_ui_updates

This PR refactors functions in static/js/stream_ui_updates.js file.
Followup of #12025

Nice cleanups, merged, thanks @YJDave!


update the arg to new Makefile logic

Signed-off-by: Michael Fornaro<EMAIL_ADDRESS>Description
Testing logic to fix csi-external-provisioner to allow for new Makefile logic in upstream
Type Of Change
To help us figure out who should review this PR, please put an X in all the areas that this PR affects.

[ ] Docs
[ ] Installation
[ ] Performance and Scalability
[ ] Security
[x] Test and/or Release
[ ] User Experience

Issue Ref (Optional)
Which issue(s) this PR fixes (optional, using fixes #(, fixes #<issue_number>, ...) format, will close the issue(s) when the PR gets merged): Fixes #144
Notes
Add special notes for your reviewer here.

The Problem is now also for csi-attacher and longhorn-csi-plugin

@Trackhe it will be all the updates to this particular version as the upstream has changed their main implementation, I saw locally that it didn't respect the GOOS & GOARCH values we provided previously so we'd need to update that logic in the Dockerfile to work with the new Makefile logic. Feel free to try as I won't be able to address this any time soon unfortunately.

This is working for me, but I need this for a couple other images, too.
Please let me know if you require some help.

Fixed in https://github.com/raspbernetes/multi-arch-images/pull/154


Function to add blank lines to a predetermined number in DOCX

It would be handy to have a function that shortens the syntax for something like this:
{{p some_answer }}
{%p line for line in range(10-((some_answer/80) | int) %} (where 80 is the actual number of characters that fit on a line, at least on average--you can check what actually fits on your page based on the font, etc, and 10 represents the existing number of blank lines (replace with real numbers))
____________________
{%p endfor %}

Even better if we could just use a tab stop and use an underline to the end of the margin?

See this snippet suggested by ChatGPT:
from docx import Document
from docx.shared import Pt

# Create a new Document
doc = Document()

# Add a paragraph
p = doc.add_paragraph()

# Set a single tab stop at the right margin (assumed to be at 6 inches for US Letter size)
tab_stop = p.paragraph_format.tab_stops.add_tab_stop(Pt(6*72), leader='underline')

# Add underlined text with a right tab to extend the underline
run = p.add_run('Your Text Here\t')
run.underline = True

# Save the document
doc.save('your_doc.docx')


C_NUMBER_RE has incorrect behavior with unary -

That makes sense.
I modeled my regex after the C_NUMBER_RE which starts with -?:
https://github.com/highlightjs/highlight.js/blob/3a2934d7e7af99631527bf55465fa229522735b9/src/lib/modes.js#L8
Originally posted by @Hirse in https://github.com/highlightjs/highlight.js/pull/2808#discussion_r528422486

We should look at how many grammars use this rule.  It's possible worth discussing if we should change our long policy on this (though I'm still against atm).  I definitely am not sure we should get into "technically unary - is an operator in this language and a part of the number in this other languages" (recent Swift point).  I think at a highlighter level we should take a consistent position and enforce it across all the grammars.
Until I saw this I thought that position was "unary - is not included in numbers" because it's impossible to tell a unary - from a binary.   So we end up mis-highlighting things like: (highlighting the - as a number when it's an operator)
let x = 5-3;

I think the correct fix is to change the C_NUMBER rule.
This is of course less problematic in well linted code with spaces:
let x = 5 - -3;

But I don't think we can count on that.

C_NUMBER is used in 14 grammars and our other number regexes (in our mode library) do not have this issue.  It's possible some grammars do, but thats really out of scope here without a full review... If we agree C_NUMBER is indeed in the wrong then we fix it, update any tests, and save the remaining grammars for whenever it comes up in the future.

Also make sure to exclude -8 in UTF-8.

Huh?  Not sure I'm following.

Ah okay so you want to exclude - from the number token globally.

Ah okay so you want to exclude - from the number token globally.

Yes, that has been the general rule because it's  mostly impossible to simply know if the - is an operator or a unary on a number (without being a full parser).
We can't look backwards yet... so all we see is "-3".. so is that:
[operator]- [number]3
OR
[number]-3

Just flagged it as an operator always (or not flagging it at all) has the appears of us making fewer 'mistakes'.

I made the change (just fixing the mode) but it leads to +12 broken markup tests across different languages that it looks like (because of their grammar) that they could actually get away with a unary, such as say YAML:
value: -32

Where from context it's clear the - is associated with the number.  Also technically this change would be breaking.  Closing this for now and we'll continue to deal with numeric issues one at a time as they come up in the future.
Our overall trend has been getting away from using things like C_NUMBER_RE and instead using much more specific grammar rules per grammar.


A J: Watch svedrillingtools.in

A J requests the watch of the watch_keyword svedrillingtools\.in. See the Metasmoke search here and the Stack Exchange search here.

svedrillingtools\.in has been seen in 1 true positive, 0 false positives, and 0 NAAs.

!!/approve


Simple frontend

Hi,
I have built a simple frontend module using Scala.js, I'm not a CSS or frontend master so it isn't great. What do you think about it? So far I have run it using https://github.com/gr2m/CORS-Proxy. Now I'm working on a codecard form.

Hi, thanks for contribution! I haven't played with it yet, however I'm really excited about it :). One thing I don't like is monstrous Page.scala - I'd rather have it as SPA (in Angular or whatever - there're bindings for Scala.js) in pure HTML not this weird DSL :). What do you think?


[HOW-TO] Reset camera from picamera2.start() after programme crash

Is there any way to reset a camera after programme crash?
Scenario: The running python programme using picamera2 crashes. The camera is left in a busy state as per the following error message:
Pipeline handler in use by another process
Camera init sequence did not complete.
RuntimeError: Failed to acquire camera: Device or resource busy
Aside from restarting the Raspberry Pi, is there any other less obtrusive way to bring the camera back online?
Thanks

Hi, could you maybe provide a short example that we can try where the Python program is crashing and leaving the camera in a bad state. I assume you've tried closing the Picamera2 instance, but in the abstract it's a bit hard to guess what may have become unhappy. Sorry not to have any ideas off the top of my head, but if you can provide a bit more detail about how things are going wrong, that would be useful and give us something to look at. Thanks!

Thanks for quick reply.
Perhaps I have not explained clearly enough. It is not picamera2 that is crashing the programme. It is my shenanigans. Mainly whilst in development mode.
If the programme crashes before the camera is sent a picamera2.stop() then that leaves the camera in "device busy" mode, which then doesn't allow access to it. How can the camera be recovered from this state, without a reboot of the RPi?
Thanks

Thanks for the update. I guess I'd still like to get a better idea of how things are broken. For example

What happens if you call picam2.stop()?
What happens if you call picam2.close()?
What happens if you kill the Python process completely? Can you now start something like rpicam-hello?

I'd have expected picam2.close() might free up the camera, but maybe that's hanging or not working now? I'd also have expected killing the Python process to sort things out too. Unless libcamera underneath or the kernel has become unhappy. Probably worth seeing what rpicam-hello reports after killing your Python process, and if that doesn't work, maybe a look at dmesg.

The only thing that appears to work is a process KILL command.
Thanks for the help.

I guess I'm still curious, but if things are behaving better for you, then that's great!!


Move bootstrapping script from machine controller to OSM

Signed-off-by: Waleed Malik<EMAIL_ADDRESS>
What this PR does / why we need it:
Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #95 #97 #96
Special notes for your reviewer:
Does this PR introduce a user-facing change?:

Support for dynamic bootstrapping configuration in OSM

/retest
Flake, the pod was never scheduled for the job.


simplified example

As my own self-directed learning I redid the basic example as a single file instead of spread among lots of files/folders, and added a lot of documentation.
I wrote it in typescript, the .TSX can be found here:   https://gist.github.com/jasonswearingen/44068ebe04b1b077c8c9
Maybe consider using this as a replacement for the current example?  If no, no hard feelings! :+1:
PS: sorry it's in typescript (not ES6)   it's almost the same, just need to remove the "typings" (things like  :number in var x:number = 5;

I think the current basic example is extremely simple and an excellent starter guide for using redux simple router. Except for the slight conflict between it using version 2.0 at the moment, which means you have to use the https://github.com/rackt/redux-simple-router/tree/1.0.2/examples/basic tag version, which I think I will create a pull request to give new users a hint on.
Your example looks nice, I do not use TypeScript myself, I use Babel and es6 which I think a lot of react developers do, I would consider if you wanted to make your gist a simplified example repo, as minimal as you possibly can but still possible to build it and run easily with instructions. Then if you like create a pull request on the README to add your example and the maintainers will have a look at it and possibly add it to the list of examples.
Nice work :+1:

If you don't mind, submitting it as a PR would be best. However, given its written in a language none of us use, that would make it harder to accept. We could definitely link to a repo in the examples section!


Force a rendering update before updating the status bar.

The comment found in in DisplayInteractor was describing a
required re-calculation of the plane positions, however
rendering was not being updated and thus not able to provide
up-to-date position calculations.
This missing update was showing consequences when a user
was scrolling through slices of an image using the mouse wheel
and observing the displayed position in the status bar.
These positions would 'lag behind' when changing the scroll direction.

Thank you for pointing this out. May I ask if you experienced a problem in a real-world scenario?
I'm asking because the use of the DisplayInteractor was changed to DisplayActionEventBroadcast in most cases (a new class was introduced some time ago - the std display already uses the DisplayActionEventBroadcast).
The DisplayInteractor was only used in some specific cases, which have just been cleaned in T29234.
However, with the DisplayActionEventBroadcast the same bug exists - the displayed position 'lags behind' and is also the root cause for the already existing issue T29219.
I tested your change (included the missing code line in the DisplayActionEventBroadcast-class) and see that it solves the problem.
In general: We are working on T25105, which will remove the UpdateStatusbar-function and move the status bar info calculation to another class / file.
In case you are working on / with related classes: Do you see any problems in moving the function? (Main reason was that the function is duplicated at several locations / in several classes and the interactor / action event broadcast class should not be responsible for updating the info).

Hi Amir. Yes, I observed a problem in a real-world scenario. The same that you now observe as lagging. I came across DisplayInteractor because I was working on an application based on the 2018.04 release.
I see my proposed change as a workaround. The base problem is: when the DisplayInteractor class asks SliceNavigationController and related classes to show some other slice, then this change is not taken into account by DisplayToWorld calculations until mappers were asked to update. Ideally we'd see immediate effect of a changed slice position. However, I know how rendering is structured and understand that this would mean a big planned change.

Regarding your question "Do I see problems with moving...":
I don't know your plans for refactoring and cannot really express a qualified opinion. One thing I understand from your description is that you want to centralize calculation of the 'current position' into a single place. This is a very good change! As seen with the present PR just make sure that rendering is updated before the position is calculated (perhaps one time too often) or calculate it in a different way (one could get relevant plane definitions and calculate the intersection point).

I added your suggested changes in a merge-commit, see https://phabricator.mitk.org/D715.
I needed to adapt your solution because - as written above - the DisplayInteractor was replaced with the DisplayActionEventBroadcast.
(Re)moving the UpdateStatusBar function is on our list but will be tackled later.

Merged into develop branch


Small fixes and new features

Fix identity operator and add some docs (we need a lot more...)
Beginning of a way to override BCs, should use a similar system like the sites system but we can deal with that later
Add more of the needed definitions of Digit sites, which allows for product states
include delta_x and delta_xyz to create a product state bit string configuration

[ ] From discussion: swap left,right boundary to tuple and include default of size 1 to exand

Can you add a compat entry for Graphs.jl to this PR too (v1.8)? Just so we can register the package post this PR

Looks good to me! Do you want me to merge now and do the implement boundary conditions via a Tuple point above in a separate PR or do it in this PR? Up to you.


SBOM and dependency track

you maybe already did but I can't find

add a new readme in some subdirectory of sbompoc that explains the instance of Dependency track was created for Oct28 workshop, and how to recreate if needed another copy with other sbom input. It was done manually so describe the steps and add issues in the todo section of this project for suggested improvements to automate various of the manual steps.


Update azure-storage-blob-batch to 12.21.3

About this PR
📦 Updates com.azure:azure-storage-blob-batch from 12.21.1 to 12.21.3
Usage
✅ Please merge!
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

⚙ Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "com.azure", artifactId = "azure-storage-blob-batch" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "30 days" },
  dependency = { groupId = "com.azure", artifactId = "azure-storage-blob-batch" }
}]

labels: library-update, early-semver-patch, semver-spec-patch, commit-count:1

Superseded by #1081.


Zoom Image Hover Not Working When Swiftly Hovering

When you hover outside the image and then back swiftly, the zoom no longer works unless you repeat but slower. Not major but something to consider.

Hi @hfall, thanks so much for your feedback. Could you please give me the steps to reproduce the issue? I tried on my site and could not.
Zoom Image Demo.webm
Could you share with me the example you found that issue or CodeSandbox link will be great? Thank you :)

I'm testing on my chrome/safari:
test.webm

Sorry @hfall, I accidentally closed your issue. It should be fixed by now. Please check again on your end to see whether it's still available. Thanks again for your input 😉

Excellent! It works fine now. 🥇


[W6.1d][W14-B4]Leon Chow WenHao

@LeonChowWenHao
Comments given. Check and acknowledge and close the PR thereafter. Good effort!


Is logrotation supported by the kubelet on windows nodes?

What is your question?
A clear and concise description of what your question is. If you have bug/blocker, please open a 'Bug Report' issue instead.

There is a known issue in k8s https://github.com/containerd/containerd/issues/7075

I'm going to resolve this issue as there's another issue tracking the same thing on a more related repo.

heres the issue hes refering to: https://github.com/containerd/containerd/issues/7075


feat(stream): add approx percentile cache for global approx percentile

I hereby agree to the terms of the RisingWave Labs, Inc. Contributor License Agreement.
What's changed and what's your intention?
Just load the entire state into memory on bootstrap. When processing chunks, we will immediately persist the updates, rather than waiting for barrier to write to state tables, to avoid contributing to barrier latency. On barrier, we will just scan the cache and output the percentile agg.
Other improvements

We delete buckets once their counts reach 0 to save space.
We keep old output in memory. If there's no change in the output of the approx percentile, we can just clone and dispatch the old output rather than do a table scan and rediscover the approx percentile.

Checklist

[ ] I have written necessary rustdoc comments
[ ] I have added necessary unit tests and integration tests
[ ] I have added test labels as necessary. See details.
[ ] I have added fuzzing tests or opened an issue to track them. (Optional, recommended for new SQL features #7934).
[ ] My PR contains breaking changes. (If it deprecates some features, please create a tracking issue to remove them in the future).
[ ] All checks passed in ./risedev check (or alias, ./risedev c)
[ ] My PR changes performance-critical code. (Please run macro/micro-benchmarks and show the results.)

[ ] My PR contains critical fixes that are necessary to be merged into the latest release. (Please check out the details)

Documentation

[ ] My PR needs documentation updates. (Please use the Release note section below to summarize the impact on users)

Release note
If this PR includes changes that directly affect users or other significant modifications relevant to the community, kindly draft a release note to provide a concise summary of these changes. Please prioritize highlighting the impact these changes will have on users.

Seems like performance got worse.......

Let me test it with fs storage rather than in-memory.

Performance improved.

Ready for review.


slot init error

Rasa Core version:
rasa-core 0.10.0
Python version:
python 2.7.12
Operating system (windows, osx, ...):
ubuntu 16.04
Issue:
Traceback (most recent call last):
  File "/usr/lib/python2.7/runpy.py", line 174, in _run_module_as_main
    "__main__", fname, loader, pkg_name)
  File "/usr/lib/python2.7/runpy.py", line 72, in _run_code
    exec code in run_globals
  File "/home/lxp/.local/lib/python2.7/site-packages/rasa_core/train.py", line 155, in <module>
    additional_arguments)
  File "/home/lxp/.local/lib/python2.7/site-packages/rasa_core/train.py", line 106, in train_dialogue_model
    max_history=max_history))])
  File "/home/lxp/.local/lib/python2.7/site-packages/rasa_core/agent.py", line 51, in __init__
    self.domain = self._create_domain(domain)
  File "/home/lxp/.local/lib/python2.7/site-packages/rasa_core/agent.py", line 390, in _create_domain
    return TemplateDomain.load(domain)
  File "/home/lxp/.local/lib/python2.7/site-packages/rasa_core/domain.py", line 402, in load
    return cls.load_from_yaml(read_file(filename), action_factory=action_factory)
  File "/home/lxp/.local/lib/python2.7/site-packages/rasa_core/domain.py", line 411, in load_from_yaml
    slots = cls.collect_slots(data.get("slots", {}))
  File "/home/lxp/.local/lib/python2.7/site-packages/rasa_core/domain.py", line 455, in collect_slots
    if "type" in slot_dict[slot_name]:
  File "/home/lxp/.local/lib/python2.7/site-packages/rasa_core/slots.py", line 177, in __init__
    self.values = [str(v).lower() for v in values] if values else []
UnicodeEncodeError: 'ascii' codec can't encode characters in position 0-1: ordinal not in range(128)

it seem that yaml data is in unicode, when init Slot class by unicode values, it raise UnicodeEncodeError,
does it mean encode to str first when init a class? but How can we control this process by replace 'ascii' codec operation as intent to  use non-ascii char?
Content of domain file (if used & relevant):

can you post your domain file please?

slots:
  fund_sacle:
    type: categorical
    values:
    - "小于2亿元"
    - "小于10亿元"
    - "小于50亿元"
    - "小于100亿元"
    - "大于100亿元"

it only occur when slot type is categorical.


Updating detection of viridiplantae in taxonomy and Golgi in compartment detection

Implementing this independently so we have a minimal change to code.


Icons are not rendering

This is a screenshot of an avatar:

This is what I'm using to render it:
      <Avatar
        icon={'face'}
        backgroundColor={this.getBackgroundColor()}
      />

I've made sure I copied the correct font file and restarted the package, but it still doen't load. Any idea?
Thanks!

Hmm strange, try a few others... Potentially the icon package might be out
of date and not have some newer ones.
On 18 Dec 2015 17:16, "Marc Riera"<EMAIL_ADDRESS>wrote:

This is a screenshot of an avatar:
https://camo.githubusercontent.com/43611032cb80f83237dc7ae33daa11e1dcad80df/68747470733a2f2f692e696d6775722e636f6d2f346530787364622e706e67
This is what I'm using to render it:
  <Avatar
    icon={'face'}
    backgroundColor={this.getBackgroundColor()}
  />

I've made sure I copied the correct font file and restarted the package,
but it still doen't load. Any idea?
Thanks!
—
Reply to this email directly or view it on GitHub
https://github.com/react-native-material-design/react-native-material-design/issues/3
.

If I enter a random string, the warning shows a list of all available
icons. "face" is there :/
On Fri, Dec 18, 2015 at 5:42 PM Elliot Hesp<EMAIL_ADDRESS>wrote:

Hmm strange, try a few others... Potentially the icon package might be out
of date and not have some newer ones.
On 18 Dec 2015 17:16, "Marc Riera"<EMAIL_ADDRESS>wrote:

This is a screenshot of an avatar:
<
https://camo.githubusercontent.com/43611032cb80f83237dc7ae33daa11e1dcad80df/68747470733a2f2f692e696d6775722e636f6d2f346530787364622e706e67
This is what I'm using to render it:
<Avatar
icon={'face'}
backgroundColor={this.getBackgroundColor()}
/>
I've made sure I copied the correct font file and restarted the package,
but it still doen't load. Any idea?
Thanks!
—
Reply to this email directly or view it on GitHub
<
https://github.com/react-native-material-design/react-native-material-design/issues/3
.

—
Reply to this email directly or view it on GitHub
https://github.com/react-native-material-design/react-native-material-design/issues/3#issuecomment-165830620
.

--
Marc Riera

Hmm okay, which version of react native are you running? If you check in the build.gradle file in the android dir it'll tell you the exact version.

Oh, it looks like I missed fixing this file when upgrading to RN 0.16,
sorry. Will check if this is the source of the bug.
On Fri, Dec 18, 2015 at 5:49 PM Elliot Hesp<EMAIL_ADDRESS>wrote:

Hmm okay, which version of react native are you running? If you check in
the build.gradle file in the android dir it'll tell you the exact version.
—
Reply to this email directly or view it on GitHub
https://github.com/react-native-material-design/react-native-material-design/issues/3#issuecomment-165834463
.

--
Marc Riera

Yeah 0.16 introduced auto font loading, also make sure the directory is called 'fonts' (note all lower case) too.

Hi, I rechecked it and I'm really using the correct RN version in the build.gradle file:
compile "com.facebook.react:react-native:0.16.+"

Strange then. If you head over to
https://github.com/oblador/react-native-vector-icons try running through
the installation guide on there, it should be getting loaded in for you.
On 19 Dec 2015 12:45, "Marc Riera"<EMAIL_ADDRESS>wrote:

Hi, I rechecked it and I'm really using the correct RN version in the
build.gradle file:
compile "com.facebook.react:react-native:0.16.+"
—
Reply to this email directly or view it on GitHub
https://github.com/react-native-material-design/react-native-material-design/issues/3#issuecomment-165978401
.

I'm having the same problem. Same version of RN and latest version of react-native-vector-icons. I followed up the installation guide a couple of times, no success. @mrcasals did you solve the issue?

Okay I'll look at this tomorrow - I've got it working myself so hopefully something simple.

Icons work here on both 0.16 & 0.17... However I did upgrade the vector icons package (https://github.com/oblador/react-native-vector-icons/releases/tag/v1.0.3). Maybe that was causing the problem. The latest release (0.2.1) has this in it, so if you could give that a go, ta.

Considering its fixed post 0.2.1, will be closing this issue.

Have anyone problem with this?
My versions:
react-native-material-design: 0.3.2
react-native-vector-icons: 1.2.0
I copied fonts and tried the instal of react-native-vector-icons - no success


LaserIO Redstone interaction with Modular Routers in Direwolf20 1.18 v1.8.0 Modpack

LaserIO version
1.3.2
Minecraft Version
1.18
Forge Version
No response
Modpack & Version
Direwolf 20 1.18 - 1.8.0
Do you have optifine installed?
No
Describe the issue
When using LaserIO redstone to fire a Modular router, an undesirable behavior occurs. The redstone passed through LaserIO is is affecting modular routers adjacent to the intended router.
Steps to reproduce

Three modular routers are configured as placers and placed adjacent to each other in a horizontal row.
All Three routers are set to action on High redstone signal in pulse mode.
Laser IO linked to and configured to send redstone signal to only the center Router.
Redstone signal is sent to LaserIO via button.
All three routers receive the signal. (why?)

FYI - This behavior does not occur if using Redstone Transmitters.
Expected behaviour
One would expect LaserIO to only affect the blocks for which it is directly attached.,
Screenshots
No response
Log files
No response
Additional information
No response

Is the redstone output card set to 'Strong signal'? If so that is intended behavior -- I can't reproduce this bug when the output signal is set to 'weak' which is the default.

Yes it is/was set to high. I was unaware of this difference between these modes. Is this documented anywhere? Is it a LaserIO behavior or a minecraft behavior?
Thanks! I'll try it on our world.


incorrect conversion to grayscale

https://github.com/muslll/neosr/blob/f2a2be9f45f1dd53b509d0adec16316ccf2ad22c/neosr/data/single_dataset.py#L51
in this piece of code you have an error, you are converting float32 to int8 directly, i.e. the output is int 0-1
img_lq = Image.fromarray(img_lq.astype(np.uint8)) if we add multiplication by 255 here
img_lq = np.array(img_lq, dtype=np.float32) and here division by 255
then the output will be a correct output in the range from 0 to 1
otherwise, everything below 1 will be 0 and 1 will be equal to 1
but this is a straightforward solution, I just want to report a bug

Fixed on #1ddad19, thanks for reporting.


Understanding of num_classes in code

Dear Authors, thanks for sharing high performance models.
I am reading through the DINO model code and get some questions below. Could you please help me?

Is the num_classes = actual_num_classes + 1(background class) in detrex?
https://github.com/IDEA-Research/detrex/blob/697f5e9dafab6ea1769ec2ea1e0b65351273aa32/projects/dino/modeling/dino.py#L101

The reason I am asking is because background class should also need a label, as I can see in DINO repo.
https://github.com/IDEA-Research/DINO/blob/66d7173cc4167934381a898b07c08507bdd96b63/models/dino/dino.py#L81
self.label_enc = nn.Embedding(dn_labelbook_size + 1, hidden_dim)

src_logits.shape[batch_size, n_dn_queries=900, num_classes] and target_classes.shape is [batch_size, n_dn_queries=900]. Does num_classes include background class, which means the last logit value is for background class? Is label_id for classes starting from "1" (I can see category_id is starting from 1 in coco dataset)?
https://github.com/IDEA-Research/detrex/blob/697f5e9dafab6ea1769ec2ea1e0b65351273aa32/detrex/modeling/criterion/criterion.py#L112

https://github.com/IDEA-Research/detrex/blob/697f5e9dafab6ea1769ec2ea1e0b65351273aa32/detrex/modeling/criterion/criterion.py#L122
If num_classes already includes background class, then +1 in this line is not needed(but cross-entropy loss is not in use, so it does not matter.)?
https://github.com/IDEA-Research/detrex/blob/697f5e9dafab6ea1769ec2ea1e0b65351273aa32/detrex/modeling/criterion/criterion.py#L103
I was trying to apply DINO model in my custom dataset. So far it can train, but the performance is not so good. I think I might misunderstand num_classes.
=======UPDATE======
https://github.com/IDEA-Research/detrex/blob/697f5e9dafab6ea1769ec2ea1e0b65351273aa32/detrex/modeling/criterion/criterion.py#L141
I went through the code second time. It looks like for Focal Loss, num_classes only needs to = actual_num_classes(without +1 for background class). For example, there is a dataset with 2 classes: 0, 1. The logits for each prediction only needs 2 numbers, e.g., [0.0145, 0.0111]. If it is mapped to background class '2', onehot encoding will be [0, 0, 1]. We can cut off the last digit of onehot so that [0.0145, 0.0111] is comparing with [0, 0].
So with Focal Loss, we only need to set num_classes=actual_num_classes(without +1 for background class) for all, including the following two locations.
https://github.com/IDEA-Research/detrex/blob/697f5e9dafab6ea1769ec2ea1e0b65351273aa32/projects/dino/modeling/dino.py#L92
https://github.com/IDEA-Research/detrex/blob/697f5e9dafab6ea1769ec2ea1e0b65351273aa32/projects/dino/modeling/dino.py#L101
Background class concept is only limited inside SetCriterion class when trying to produce one-hot encoding and the last digit will be cut off, becoming all zeros for background class.
https://github.com/IDEA-Research/detrex/blob/697f5e9dafab6ea1769ec2ea1e0b65351273aa32/detrex/modeling/criterion/criterion.py#L116
Is my understanding correct?
Thanks,
Cheng

@HaoZhang534 Thanks for your time and help.
I meant the loss was decreasing as expected. Take below loss output for example.
loss=7.69, lr=0.0001, loss_class=0.00258, loss_bbox=0.0155, loss_giou=0.222...]

And I loaded the trained model and tested on images, it gave fairly good predictions on many images. Therefore, I think the model was learning with the custom dataset.
I am not sure where I mis-configured the evaluator part, as it depends on annotation file in coco format self._coco_api =COCO(json_file).

Hi @weicheng113, I suggest you visualize the predictions on the validation set to see wether the problem is on the model or the evaluator.

Hi @HaoZhang534 , thanks a lot. I got it working by using MeanAveragePrecision from torchmetrics, which has simpler interface. I will continue with training. When I get more metrics information, I will ask for your advice with finetuning.

@HaoZhang534 may I ask for your suggestion.
I have a dataset about 1,100 examples, 20% for validation and 80% for training. The dataset only has two categories, 0 and 1.
Below is the histogram of number of instances in each image file. Most of them has two objects in an image and maximum 4 objects in an image.

Category 1 has more less instances than category 0 as shown below, which I think it is ok for focal loss.

=========
I tried with the following configuration.
num_queries: 40
num_dn_queries: 10(if there is 2 instances in an image, it will have 5 DN groups - 5* [2 * 2])
num_select: 20 ( select top 20 confident prediction for validation)
num_epoch: 50 ( with StepLR(optimizer=optimizer, step_size=20) )
train_batch_size: 2
gradient_accumulation_steps: 1 (only on single gpu machine. doing gradient decent at every step).

I got below, which is not as good as our yolo version:
{'map': tensor(0.7773),
 'map_50': tensor(0.9670),
 'map_75': tensor(0.9138),
 'map_large': tensor(0.8001),
 'map_medium': tensor(0.4979),
 'map_per_class': tensor(-1.),
 'map_small': tensor(-1.),
 'mar_1': tensor(0.5967),
 'mar_10': tensor(0.8588),
 'mar_100': tensor(0.8595),
 'mar_100_per_class': tensor(-1.),
 'mar_large': tensor(0.8821),
 'mar_medium': tensor(0.5853),
 'mar_small': tensor(-1.)}

=======
After looking the above instance stats, I am thinking having experiments with the following two configuratiion:
num_queries: 60
num_dn_queries: 16(if there is 2 instances in an image, it will have 8 DN groups - 8* [2 * 2])
num_select: 6 ( instead of using 4, leaving a bit room)
num_epoch: 50 ( with StepLR(optimizer=optimizer, step_size=20) )
train_batch_size: 2
gradient_accumulation_steps: 1 (only on single gpu machine. doing gradient decent at every step).

num_queries: 60
num_dn_queries: 16(if there is 2 instances in an image, it will have 8 DN groups - 8* [2 * 2])
num_select: 6 ( instead of using 4, leaving a bit room)
num_epoch: 50 ( with StepLR(optimizer=optimizer, step_size=20) )
train_batch_size: 2
gradient_accumulation_steps: 3 (only on single gpu machine. doing gradient decent at every 3 steps).

Any advice is highly appreciated.
Thanks, Cheng

Is the mAP 0.777% or 77.7%? Given that you have at most 4 instances per image. You can use small num_queries such as 40. num_select can be set larger such as 40. You can also load our coco pre-trained model to fine-tune the model on your dataset.

@HaoZhang534 Thanks a lot. It is mAP 77.7%.
Could you explain why to use larger num_select? I thought num_select should be close to max number of instances?
Good suggestion. I will try with pre-trained model out.
Thanks, Cheng

@weicheng113 Larger num_select usually leads to higher AP. About category 1, you may try some tricks to balance the proportions of the two categories. For example, you can add some copies of images from category 1 in the training data.

@HaoZhang534 Thanks. For final inference, is there a rule to pick the confidence_threshold? I saw 0.5 as default in detrex and 0.3 in DINO repo.

@weicheng113 Which confidence_threshold do you mean? We do not use confidence_threshold to evaluate. We use confidence_threshold=0.3 for visualization.

Got you, thanks a lot @HaoZhang534 .

@HaoZhang534 I realized I can't directly load a pretrained dino model, as there is an incompatibility in num_classes, num_queries and num_dn_queries. I can load other weights like transformer weights, but something like class_embed, label_enc need to be retrained.
I will have a try to see if loading a pretrained dino model is helpful. Thanks.

@weicheng113 You are welcome. Your concern is reasonable. It's really a problem when objects are crowded. Maybe some improvements can be made to fix this such as only use negative examples when objects are not crowded.

Excuse me, can you tell me how to load the custom dataset?
Would be very grateful if it could be done.

Hi @HaoZhang534 , thanks a lot. I got it working by using MeanAveragePrecision from torchmetrics, which has simpler interface. I will continue with training. When I get more metrics information, I will ask for your advice with finetuning.

halou i got the same question，can you give more detail about how to fix this bug？


Nested arrays in custom headers

First of all hello there, and thank you for this wonderful project.
I want to access nested data in a nested array. I have like 30 custom headers in a separate file, where all of the headers work perfectly except 2. These two headers gets their value one more level down, than the other headers.
An example could be this:
--------- THIS WORKS -------------
{
    name: 'value.fields.system.name',
    label: System,
    options: {
         filter: true,
         sort: true,
     }
}

------- THIS DOES NOT WORK --------
{
    name: 'value.fields.system.applications[0].name',
    label: 'App Types',
    options: {
         filter: true,
         sort: true,
     }
}

I dont know if there is any way I can loop over these or similar?
All I need is this array to be joined, so for example I have applications[0].name joined with the rest applications[rest].name.
Once again thank you for this great repo! It really solves a lot of problems and frustrations.
The table doesn't have any way to combine objects into arrays for you as it builds the table. This is something you would need to do with your data before you pass it in, using something like reduce to get the structure that you want. Also, be aware that array data is not well supported for cell individual cell contents. You may need to also use customBodyRender to get the kind of display you're looking for.

Hi @gabrielliwerant,
Thank you for your response. That was also what I thought could be the case, but great to get it confirmed. I will look into the customBodyRender, as there are loads of individual and complex data being inserted into the table. However if it isn't well supported, I would like to say that it is only 'really' nested data that isn't showing correct for me, otherwise the Datatable handles the data quite good actually.
I will hereby close this issue. :-)


Newer version of conversational-form lost support for older android (4.x) and ios (9.x) devices

Will not render, showing an empty page.

Re: Safari the issue is #131 - I assume it's the same issue on android devices.


EL-786 hide irrelevant rows from vehicle results

Jira ticket
What changed and why
Before
If a vehicle was not in regular use (only possible for certificated checks), the assessed value of the vehicle was equal to the value. Outstanding finance was not deducted, and the “disregards and deductions” figure we got back from CFE was actually the outstanding finance figure multiplied by -1.
After
We now show neither of those rows in the vehicle table when in_regular_use is false in the vehicle payload.
For vehicles not in regular use, the vehicle table shown on the results page only has the “Value” and “Assessed value” rows.
Checklist
Before you ask people to review this PR:

Tests and rubocop should be passing
Branch is generally up to date with main Github - definitely no conflicts
No unnecessary whitespace changes. These make diffs harder to read and conflicts more likely.
PR description says what changed and why, with a link to the JIRA story.
Diff has been checked for unexpected changes being included.
Commit messages say why the change was made.

(I'm not going to review this one as I paired writing the code so have biased eyes. I'll leave it to Hettie or Will.)


Containers

Hey, I am wondering if there's a solution to my problem..
My problem is that I want the child container, to always have a z-index higher than the other children.
Basically, think of a game with a leaderboard. I'm having that drawn into a container, and added to the main container, but the problem I stated is what I'm currently facing.
Is this not the use of containers? Or, how could I achieve with what I'm trying to do?
BTW: I've found solutions as to re-organizing the array, but that is not an efficient way of doing so, and then again, im already using a sorting system to sort out the other children.
I've tried placing the child at an index, stage.childrens.length + 1, and some high number as 9999, but looks as if I cannot do that. Currently, im left with no more options to work with.
If anyone can help in which what I shall do that would be great. Thanks.
PIXI

would be more awesome if a z vector for a container,sprite, and or object(child) existed.

This one: https://github.com/pixijs/pixi-display
Or new version (unstable but more stuff): https://github.com/pixijs/pixi-display/tree/layers

As @ivanpopelyshev notes, pixi-display is a great plugin that will let you set z array on objects regardless of their hierarchy in the scene.
Another way, without using plugins, would be to have parent containers that represent the main objects in your scene (scoreboard, game area) ordered such that the scoreboard is in front.  Then when you're messing around with adding/removing elements to your game, you only do so inside the game area container and the scoreboard always stays in front.

@ivanpopelyshev @staff0rd Thanks, I'll try this out. Hopefully it works.


Remove remaining refs to SITE_URL

Thanks for creating an issue! Please fill out this form so we can be sure to have all the information we need, and to minimize back and forth.

What are we trying to do?

(another multitenancy fix; description tk)

What feature or behavior is this required for?

How could we solve this issue? (Not knowing is okay!)

Anything else?

(I don't think we actually need to do this for reasons)


kie-kogito-apps-2069: Improve Jobs Service liveness check to limit the amount of time to get the leader status

Many thanks for submitting your Pull Request :heart:!
Please make sure that your PR meets the following requirements:

[X] You have read the contributors guide
[X] Pull Request title is properly formatted: KOGITO-XYZ Subject
[X] Pull Request title contains the target branch if not targeting main: [0.9.x] KOGITO-XYZ Subject
[X] Pull Request contains link to the JIRA issue
[X] Pull Request contains link to any dependent or related Pull Request
[X] Pull Request contains description of the issue
[X] Pull Request does not include fixes for issues other than the main ticket

How to replicate CI configuration locally?

Build Chain tool does "simple" maven build(s), the builds are just Maven commands, but because the repositories relates and depends on each other and any change in API or class method could affect several of those repositories there is a need to use build-chain tool to handle cross repository builds and be sure that we always use latest version of the code for each repository.
build-chain tool is a build tool which can be used on command line locally or in Github Actions workflow(s), in case you need to change multiple repositories and send multiple dependent pull requests related with a change you can easily reproduce the same build by executing it on Github hosted environment or locally in your development environment. See local execution details to get more information about it.

How to retest this PR or trigger a specific build:

for pull request checks
Please add comment: Jenkins retest this

for a specific pull request check
Please add comment: Jenkins (re)run [kogito-apps|kogito-examples] tests

for quarkus branch checks
Run checks against Quarkus current used branch
Please add comment: Jenkins run quarkus-branch

for a quarkus branch specific check
Run checks against Quarkus current used branch
Please add comment: Jenkins (re)run [kogito-apps|kogito-examples] quarkus-branch

for quarkus main checks
Run checks against Quarkus main branch
Please add comment: Jenkins run quarkus-main

for a specific quarkus main check
Run checks against Quarkus main branch
Please add comment: Jenkins (re)run [kogito-apps|kogito-examples] quarkus-main

for quarkus lts checks
Run checks against Quarkus lts branch
Please add comment: Jenkins run quarkus-lts

for a specific quarkus lts check
Run checks against Quarkus lts branch
Please add comment: Jenkins (re)run [kogito-apps|kogito-examples] quarkus-lts

for native checks
Run native checks
Please add comment: Jenkins run native

for a specific native check
Run native checks
Please add comment: Jenkins (re)run [kogito-apps|kogito-examples] native

for native lts checks
Run native checks against quarkus lts branch
Please add comment: Jenkins run native-lts

for a specific native lts check
Run native checks against quarkus lts branch
Please add comment: Jenkins (re)run [kogito-apps|kogito-examples] native-lts

How to backport a pull request to a different branch?

In order to automatically create a backporting pull request please add one or more labels having the following format backport-<branch-name>, where <branch-name> is the name of the branch where the pull request must be backported to (e.g., backport-7.67.x to backport the original PR to the 7.67.x branch).

NOTE: backporting is an action aiming to move a change (usually a commit) from a branch (usually the main one) to another one, which is generally referring to a still maintained release branch. Keeping it simple: it is about to move a specific change or a set of them from one branch to another.

Once the original pull request is successfully merged, the automated action will create one backporting pull request per each label (with the previous format) that has been added.
If something goes wrong, the author will be notified and at this point a manual backporting is needed.

NOTE: this automated backporting is triggered whenever a pull request on main branch is labeled or closed, but both conditions must be satisfied to get the new PR created.

Quarkus-3 PR check is failing ... what to do ?

The Quarkus 3 check is applying patches from the `.ci/environments/quarkus-3/patches`.
The first patch, called 0001_before_sh.patch, is generated from Openrewrite .ci/environments/quarkus-3/quarkus3.yml recipe. The patch is created to speed up the check. But it may be that some changes in the PR broke this patch.
No panic, there is an easy way to regenerate it. You just need to comment on the PR:
jenkins rewrite quarkus-3

and it should, after some minutes (~20/30min) apply a commit on the PR with the patch regenerated.
Other patches were generated manually. If any of it fails, you will need to manually update it... and push your changes.

@ricardozanini @gmunozfe
Due to weird CI erros I have close the original PR https://github.com/apache/incubator-kie-kogito-apps/pull/2087 and opened this new one. Basically same stuff + some code review suggestions.
Would you mind guys re-approve?

PR job #1 was: UNSTABLE
Possible explanation: This should be test failures

Reproducer
build-chain build full_downstream  -f 'https://raw.githubusercontent.com/${AUTHOR:apache}/incubator-kie-kogito-pipelines/${BRANCH:main}/.ci/buildchain-config-pr-cdb.yaml' -o 'bc' -p apache/incubator-kie-kogito-apps -u https://github.com/apache/incubator-kie-kogito-apps/pull/2096 --skipParallelCheckout
NOTE: To install the build-chain tool, please refer to https://github.com/kiegroup/github-action-build-chain#local-execution

Please look here: https://ci-builds.apache.org/job/KIE/job/kogito/job/main/job/pullrequest_jobs/job/kogito-apps-pr/job/PR-2096/1/display/redirect
Test results:

PASSED: 1256
FAILED: 13

Those are the test failures:

org.kie.kogito.it.jobs.JobServiceLeaderIT.(?)
Can't get Docker image: RemoteDockerImage(imageName=org.kie.kogito/jobs-service-postgresql:999-SNAPSHOT, imagePullPolicy=DefaultPullPolicy(), imageNameSubstitutor=Chained substitutor of 'DefaultImageNameSubstitutor (composite of 'ConfigurationFileImageNameSubstitutor' and 'PrefixingImageNameSubstitutor')' and then 'Kogito Image Name Substitutor')

PR check / Build projects / org.kie.kogito.it.jobs.CallbackStateTimeoutsIT.callbackStateTimeoutsExceeded
java.util.concurrent.CompletionException: java.lang.RuntimeException: Unable to start Quarkus test resource class org.kie.kogito.test.resources.JobServiceCompositeQuarkusTestResource

PR check / Build projects / org.kie.kogito.it.jobs.CallbackStateTimeoutsIT.callbackStateTimeoutsExceeded
java.util.concurrent.CompletionException: java.lang.RuntimeException: Unable to start Quarkus test resource class org.kie.kogito.test.resources.JobServiceCompositeQuarkusTestResource

PR check / Build projects / org.kie.kogito.it.jobs.MultipleTimerInstancesIT.boundaryTimerEvent
java.util.concurrent.CompletionException: java.lang.RuntimeException: Unable to start Quarkus test resource class org.kie.kogito.test.resources.JobServiceCompositeQuarkusTestResource

PR check / Build projects / org.kie.kogito.it.jobs.MultipleTimerInstancesIT.boundaryTimerEvent
java.util.concurrent.CompletionException: java.lang.RuntimeException: Unable to start Quarkus test resource class org.kie.kogito.test.resources.JobServiceCompositeQuarkusTestResource

PR check / Build projects / org.kie.kogito.it.jobs.ProcessAsyncIT.testAsync
java.util.concurrent.CompletionException: java.lang.RuntimeException: Unable to start Quarkus test resource class org.kie.kogito.test.resources.JobServiceCompositeQuarkusTestResource

PR check / Build projects / org.kie.kogito.it.jobs.ProcessAsyncIT.testAsync
java.util.concurrent.CompletionException: java.lang.RuntimeException: Unable to start Quarkus test resource class org.kie.kogito.test.resources.JobServiceCompositeQuarkusTestResource

PR check / Build projects / org.kie.kogito.it.jobs.ProcessTimerIT.testDeleteTimerCycle
java.util.concurrent.CompletionException: java.lang.RuntimeException: Unable to start Quarkus test resource class org.kie.kogito.test.resources.JobServiceCompositeQuarkusTestResource

PR check / Build projects / org.kie.kogito.it.jobs.ProcessTimerIT.testDeleteTimerCycle
java.util.concurrent.CompletionException: java.lang.RuntimeException: Unable to start Quarkus test resource class org.kie.kogito.test.resources.JobServiceCompositeQuarkusTestResource

PR check / Build projects / org.kie.kogito.it.jobs.SwitchStateTimeoutsIT.switchStateEventConditionTimeoutsTransitionTimeoutsExceeded
java.util.concurrent.CompletionException: java.lang.RuntimeException: Unable to start Quarkus test resource class org.kie.kogito.test.resources.JobServiceCompositeQuarkusTestResource

PR check / Build projects / org.kie.kogito.it.jobs.SwitchStateTimeoutsIT.switchStateEventConditionTimeoutsTransitionTimeoutsExceeded
java.util.concurrent.CompletionException: java.lang.RuntimeException: Unable to start Quarkus test resource class org.kie.kogito.test.resources.JobServiceCompositeQuarkusTestResource

PR check / Build projects / org.kie.kogito.it.jobs.WorkflowTimeoutsIT.workflowTimeoutExceeded
java.util.concurrent.CompletionException: java.lang.RuntimeException: Unable to start Quarkus test resource class org.kie.kogito.test.resources.JobServiceCompositeQuarkusTestResource

PR check / Build projects / org.kie.kogito.it.jobs.WorkflowTimeoutsIT.workflowTimeoutExceeded
java.util.concurrent.CompletionException: java.lang.RuntimeException: Unable to start Quarkus test resource class org.kie.kogito.test.resources.JobServiceCompositeQuarkusTestResource


kodi: copy web assets instead of symlinking

Kodi refuses to follow symlinks that lead outside of a small whitelist
of allowed directories. See
CFileUtils::CheckFileAccessAllowed
for the relevant code.
This feels like a pretty brittle workaround, but I can't think of a
better solution, so I'm going to say that this fixes
https://github.com/NixOS/nixpkgs/issues/145116.
Motivation for this change
I ran into https://github.com/NixOS/nixpkgs/issues/145116 and wanted to fix it.
Things done

Built on platform(s)

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

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

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

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

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

[x] Fits CONTRIBUTING.md.

Looking through the source code makes me wonder if upstream would even be willing to reconsider that logic at all... 😟 Still, it might be worth opening an issue upstream and presenting our scenario. Any takers? ❤

I'm happy to try to take this upstream, but I'm not clear on what exactly we'd want from kodi, so I'd want to do a little discussion among us before I'd feel comfortable filing an issue upstream. Do you have any concrete requests in mind, @aanderse? Some ideas:

We could ask kodi for an option to "dangerously" turn off this realpath. This might be analogous to apache's FollowSymLinks option, although I'm not fully grok-ing the docs on it. It is interesting to note that maybe this isn't such a dangerous behavior to have. From the apache docs:

Omitting this option should not be considered a security restriction, since symlink testing is subject to race conditions that make it circumventable.

Ask kodi for an option to make this whitelist an arbitrarily long list of folders.

Something else entirely? Is it possible that nix already has to deal with this with other embedded webservers in other places? Perhaps there's some "prior art" we can use for inspiration? I'm very new to nix, so I don't know how to find any such inspiration.

Thanks for the discussion @jfly. It is probably luck we haven't ran into this issue until now. I'm not sure what the answer is either... but a whitelist distros can append to might be a the best idea - what do you think?
For now we should merge this and get people back up and running. Are you fine with me merging this PR now?

@aanderse, I'm fine with merging this up! I'll try to start a discussion upstream as well.

Great! Thanks again.

I've started a thread over here: https://forum.kodi.tv/showthread.php?tid=366338.
@aanderse I'd greatly appreciate any backup on that thread. I feel like I spent most of the time explaining arcane NixOS details and not really motivating why Kodi should change for us.


[DC-1189] Read from settings for name and description

Jira Ticket: https://broadworkbench.atlassian.net/browse/dc-1189

Summary of changes:

What

Why

Testing strategy

[ ]

@pshapiro4broad I already did an API PR to add the fields, but they aren't required so as to not break things. Next step is going to be updating all our settings, making the API fields required, and then coming back to make the parameters non-optional in the UI


Tooltips are turned into arrays

When using Vega Editor to display tooltips, they are rendered correctly. However, the exact same code in Deneb produces tooltips that look like they are turned into some kind of array as below.

.pbix attached for investigation.
Tree Test2.zip
Using the following works fine:
"tooltip": {
            "signal": "datum"
          }

But using below produces the weird result. Using field instead of signal does the same and interestingly trying to access a number instead of text (i.e. datum.size) produces an empty tooltip.
     "tooltip": {
            "signal": "datum.name"
          }

I'm probably missing something obvious so appreciate any pointers.

Looks like it could be an issue with how we process the signal into Power BI's VisualTooltipDataItem elements if a single datum field is provided. most of our focus in getting the integration working was for Vega-Lite, so that may be something specific to deal with here. I'll try to find some time to explore further and see if there's anything we need to take into account.
As a workaround, the following approach from the map Vega tooltip example should help: specify a valid object of key/value pairs inside the signal expression, e.g.:
"tooltip": {
    "signal": "{'Name': datum.name, 'ID': datum.id}"
}

In your example specification, this now looks as follows when a mark is hovered-over:

Just some notes, while I have the debugger running - turning off the Power BI tooltip handler and falling back to Vega's correctly shows datum.name as a string, e.g.:

In this case, the signal's tooltip property is a plain string, rather than an iterable:

As we normally assume an object of key/value pairs (because this is how Power BI tooltips need to be structured), this will indeed get split into an array for each letter.

This will be fixed in the next 1.2 alpha build, which will trigger on the next pull request.
Because Power BI tooltips need key/value pairs as part of their definitions, if a tooltip signal contains a scalar value rather than an object, we will reconstruct it with a blank key and use the scalar value as the corresponding property value in order to game the Power BI tooltip handler into similar behaviour as the inbuilt vega tooltip, e.g.:


Styling Upload button

Hi
After following the getting started example, i was able to get the following button :

May i know if there is any way for me to style this button to at least an ionic button?

you could put a regular button where you like and hide the file input. Then do
$("#yourRegularButton").click(function(){
 $("#yourFileInput").click();
});


Adding vtnerd GPG signing key to repo

This public key was just used to sign 0.2.0 last night.

Maybe extras? https://api.csswg.org/bikeshed/?force=1&url=https://raw.githubusercontent.com/vector-of-bool/pitchfork/develop/data/spec.bs#tld.extras

I'm leaning towards misc due to the link provided by @selsta . The extras directory is supposed to be submodules, and I don't see that happening with gpg_keys (for a little while).
In other words, I'll update this PR to make it misc/gpg_keys/vtnerd.asc and perhaps in the future we'll have a top-level repo in monero-project/gpg_keys at which point this would be extras/gpg_keys/vtnerd.asc.

FYI - I'm trying to go with the pitchfork layout because I like what its trying to do, etc.


Support TempData attribute properties on all activated types

The [TempData] backed properties are currently only supported on pages and controllers. We should extend it to work on all activated types, including:

View Components
Partials
Tag Helpers

Can we support complex types to be stored in the TempData too?

It's a bit of a Pandora's Box due to the implications of round-tripping objects beyond primitive values. Perhaps we could support POCOs by serializing to JSON, so it would be limited to public primitive members, and perhaps that could be composed via a separate attribute and filter.

We tried making ViewData work on partial models and it turned out to be difficult since we don't manage the type's lifetime. We can consider supporting TempData properties on tag helpers if we get enough customer feedback.


snabbnfv traffic: warn when config file does not exist

snabbnfv traffic should print a warning when the configuration file does not exist. This will help users detect when they use the wrong filename by accident.
(This should be a warning and not an error because the behavior is to poll until the configuration file is created. The OpenStack setup allows the traffic process to start before its configuration is defined.)

I can look into this if you like. Just let me know so we don't end up duplicating effort again. ;)

Fixed in #679.


Project Addition

I want to add my project Machine Learning link in the project section.

Assign this issue to me under OSEN CollabCode Carnival Hacktoberfest 2023


SHARED_VERSION is used but not defined in master's Makefile.template

It looks like SHARED_VERSION was renamed to SHARED_VERSION_CORE in master branch's Makefile.template, but there are still a few Makefile targets that try to use the old SHARED_VERSION
This seems to be one of the main problems in https://github.com/grpc/grpc/issues/8124 - the ruby opt/linux package build tries to use one of those bad Makefile targets.
I'm not sure how the rest of the targets that still use SHARED_VERSION should be changed though.

Ruby is probably what has to change there.

This is specifically a Makefile problem. There are variables in the Makefile that are used but not defined.

Just for clarity adding this to 1.1 release pre-reqs.
At least for ruby package builds though, this is fixed by #8957.


replace the Makefile symlink to use sourcing

obsolete, we dont have a makefile anymore

obsolete, we dont have a makefile anymore


Push packages to GHCR (rebased. the credit goes to @sollie)

I was going to handle #849 by awesome @sollie in a usual merge process but failed to do so due to some permission issues in GH and also perhaps some kind of (unusual?) conflicts among our git histories.
This pull request is made of the rebased and reworked version of the commit included in @sollie's, so that it should be good to be merged as usual. Thank you for your contribution @sollie!
Ref #849
Ref #566

This is excellent. I try to avoid using hub.docker.com as much as possible after the rate limits were introduced.


added my data

Please resolve the conflicts and update the PR. @Dhanush-2605

hi, can you check it once?


complete refactoring of Grid class

The Grid class needs some minor refactoring to complete the conversion from the use of std::vector to std::array for the representation of fixed-size, arrays of length 3. In particular, a number of public getters on the grid class still return std::vector instead of std::array. This results in unnecessary, and sometimes expensive, conversions between the types.

Well, in the proposed refactoring work, one should also move all the function implementations to the .cpp file, instead of leaving them in the .h file. That's because:

the current headers are often more than 1000 lines long, kinda ridiculous.
those function implementations in the header will be compiled every time the header is included, which means compiled hundreds of times throughout VAPOR, greatly increasing the compilation time and binary size.
separation between headers and implementations complies with the coding convention.

This has nearly been completed as of 3.6. I think we should defer further action until we have a more extensive testing framework, enabled by the upcoming Python API