[Bug] GoogleCloudProfile dataset argument is not replaced by neither model or yml config, but instead used as a prefix

Astronomer Cosmos Version
1.6.0
dbt-core version
1.8.8
Versions of dbt adapters
dbt-adapters 1.7.0
dbt-bigquery 1.8.3
LoadMode
AUTOMATIC
ExecutionMode
VIRTUALENV
InvocationMode
None
airflow version
2.5.3
Operating System
Ubuntu & Docker under WSL2 of windows 11
If a you think it's an UI issue, what browsers are you seeing the problem on?
No response
Deployment
Docker-Compose
Deployment details
No response
What happened?
As I'm using GoogleCloudOauthProfileMapping and passing the required arguments for profile_args, a project, my_gcp_project, and a dataset, my_gcp_dataset, are set.
By running the current set-up to build the simple my_first_dbt_model.sql of the jaffle_shop project, the table is correctly created in the correct location, my_gcp_project.my_gcp_dataset.my_first_dbt_model.
Now, if I try to overwrite these parameters for this specific model,

within {{ config(database='my_other_gcp_project', schema='my_other_gcp_dataset') }} model config, the new location will become my_other_gcp_project.my_gcp_dataset_my_other_gcp_dataset.my_first_dbt_model
within the yml config for the model, as below, also do the same location, my_other_gcp_project.my_gcp_dataset_my_other_gcp_dataset.my_first_dbt_modelmodels:
  - name: my_first_dbt_model
    config:
      database: "my_other_gcp_project"
      schema: "my_other_gcp_dataset"

The word combination database/project and schema/dataset are interchangeable and working as is.
So the database/project is fully overwritten in the appropriate hierarchy (https://docs.getdbt.com/reference/model-configs#configuring-models) but schema/dataset are concatenated.
However, the concatenation occurs only with the Cosmos argument and the dbt upper hierarchical config. For example, if I try both options described above, only the model config() will be concatenated with Cosmos value.
Relevant log output

How to reproduce

Use a GCP profile mapping function, in my case GoogleCloudOauthProfileMapping
Set a dataset value
Set a schema/dataset value for a specific model, either config() or yml config.

Anything else :)?
No response
Are you willing to submit PR?

[x] Yes I am willing to submit a PR!

Contact Details
<EMAIL_ADDRESS>

Hey @vanAkim, Thank you for finding this issue and providing a detailed analysis. Could you please add a test here: https://github.com/astronomer/astronomer-cosmos/blob/main/tests/profiles/bigquery/test_bq_oauth.py for this case and propose the solution? We are planning to do the next release in mid-December.

After looking at the Cosmos source code and not understanding how this behavior is happening, I've found that it is a native feature of dbt (ref https://docs.getdbt.com/docs/build/custom-schemas#understanding-custom-schemas).
Thus, it has nothing to do with Cosmos, and this issue can be closed I presume.


[stable/instana-agent] Chart version 1.0.29

What this PR does / why we need it:
Expose new configuration options:

leaderElector.image.name
leaderElector.image.tag

Changed the existing agent.leaderElectorPort configuration value to leaderElector.port to be consistent with the other leader elector configurations.
Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

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

/un-cc @mdonkers
/un-cc @JeroenSoeters
/cc @nfisher

/lgtm


Release 1.2.3, missing files

Release 1.2.3's zip is incomplete and does not have all files necessary to be used.

I have received no other reports of missing files, and the download appears to function correctly for me.
Please confirm you have already setup vanilla dynmap in the location you are attempting to use LiveAtlas. LiveAtlas requires either the standalone/config.js file from a functioning dynmap install to exist, or for servers to be configured directly in index.html as described here.
Feel free to reopen this issue if the problem persists for you, ideally with any error messages you see in your browser console.


Self Subscribing

This issue is in the Subvid repo as well referencing issue
Right now the users have to subscribe to a creator's address to view its content.
This is true even if the user wants to view his/her own content. Subscribing to itself just results in paying yourself and the user does not lose any tokens.
Still, this is an overhead and should be resolved.
This might be moderately challenging, and if anyone wants to work on it, comment and let me know.

Please assign this issue to me.

Sure @SahilSingh177 you can work on this 👍

Hey @SahilSingh177 , how is your progress going on?

Hey @aditya172926.
I added the following lines of code. Please have a look at them.

In line no.- 89 the user who adds any content will be saved as a Gold Tier in Subscribed Tier.

Hey @aditya172926. I added the following lines of code. Please have a look at them and share your views.

In line no.- 89 the user who adds any content will be saved as a Gold Tier in Subscribed Tier.

Looks good to me @SahilSingh177 .

Make a PR if you want this to be included in the repo. Also make sure you do a git pull before, just to make sure there aren't any conflicts @SahilSingh177


⚠ API - Lens has degraded performance

In 2a78a7d, API - Lens ($LENS_API_URL) experienced degraded performance:

HTTP code: 503
Response time: 400 ms

Resolved: API - Lens performance has improved in 0e36277 after 6 minutes.


Extension Gitlab pipeline fails to build due to outdated nodejs and openssl packages

Setup

Rancher version: v2.9-head
Rancher UI Extensions: N/A
Browser type & version: N/A

Describe the bug

When using the provided .gitlab-ci.yml pipeline file for an extension, the build will fail with:

$ npm install -g --unsafe-perm yarn
/usr/bin/node14: symbol lookup error: /usr/bin/node14: undefined symbol: RSA_get0_pss_params, version OPENSSL_1_1_1d
Cleaning up project directory and file based variables 00:00
ERROR: Job failed: exit code 1

To Reproduce

Create a new repository on gitlab.com
Within the repository create a new extension with the -l option to include the gitlab pipeline file
Follow the extension quickstart to build a top-level extension
Push the extension to the new gitlab repository

Result
The build_catalog stage will fail.
Expected Result

The pipeline should succeed and the catalog image should be pushed to the specified registry.
Screenshots

N/A
Additional context

N/A

Validated that both the check_version and build_catalog jobs complete in gitlab.


Fix flaky tests.

Description
It was failing when running locally about 50% of the time. With the "fix", ran 10 times in a row without an error.
Review Instructions
UI checks on the develop branch should always pass.
Issue
https://ucsc-cgl.atlassian.net/browse/SEAB-5217
Security
If there are any concerns that require extra attention from the security team, highlight them here.
Please make sure that you've checked the following before submitting your pull request. Thanks!

[x] Check that your code compiles by running npm run build
[x] Ensure that the PR targets the correct branch. Check the milestone or fix version of the ticket.
[x] If this is the first time you're submitting a PR or even if you just need a refresher, consider reviewing our style guide
[x] Do not bypass Angular sanitization (bypassSecurityTrustHtml, etc.), or justify why you need to do so
[x] If displaying markdown, use the markdown-wrapper component, which does extra sanitization
[x] Do not use cookies, although this may change in the future
[x] Run npm audit and ensure you are not introducing new vulnerabilities
[x] Do due diligence on new 3rd party libraries, checking for CVEs
[x] Don't allow user-uploaded images to be served from the Dockstore domain
[x] If this PR is for a user-facing feature, create and link a documentation ticket for this feature (usually in the same milestone as the linked issue). Style points if you create a documentation PR directly and link that instead.
[x] Check whether this PR disables tests. If it legitimately needs to disable a test, create a new ticket to re-enable it in a specific milestone.

I think I found a better fix, re-requesting review.


chore: use @typescript-eslint/parser as the default parser

What
Adding parser: "@typescript-eslint/parser" as the default parser allows ESLint to lint *.(ts|tsx) files.
Before
I am adding an unused variable in src/config.ts leads to the expectation of receiving an error from the  no-unused-vars rule in the output. However, ESLint fails to recognize the import type syntax.
import type { Site, SocialObjects } from "./types";

const foo = ""; // <-- unused variable

// skip

After

improving DX in VS Code with the ESLint extension for better error detection.

Thanks.


Wiki instructions to install via Go results in error

Describe the bug
Wiki installation instructions to install gitleaks via the command GO111MODULE=on go get github.com/zricethezav/gitleaks/v5 result in the following error:
⟩ env GO111MODULE=on go get github.com/zricethezav/gitleaks/v5
package github.com/zricethezav/gitleaks/v5: cannot find package "github.com/zricethezav/gitleaks/v5" in any of:
        /usr/lib/go-1.10/src/github.com/zricethezav/gitleaks/v5 (from $GOROOT)
        /home/john/go/src/github.com/zricethezav/gitleaks/v5 (from $GOPATH)

To Reproduce
Steps to reproduce the behavior:
Run env GO111MODULE=on go get github.com/zricethezav/gitleaks/v5
Expected behavior
gitleaks gets installed.
Basic Info (please complete the following information):

OS: Ubuntu 18.04
Gitleaks Version: 5

Additional context
The go version is: go1.10.4 linux/amd64
I have no experience with go. I just installed it to try gitleaks. I also changed the version in the command to 6, hoping to install the latest version, and the same error occurred.
Thank you!
cc @zricethezav

Hello @Asuza !
I believe that this bug is old and has been fixed.
Can we close this issue?
cheers!

If it's been fixed, then sure! I haven't tried it lately, but my issue is so old at this point it might as well be cleaned up. Thanks!


HumanVectorRendering: based on VectorWrapper (#873, #872, #233)

Corrected code to pass tests, and tests to test code! Also corrected some bugs in vectorized Cartpole #872
Doctest: fixed comments
Description
Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context. List any dependencies that are required for this change.
Fixes #872
Fixes #873
Related to #233
Type of change

[x] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[x] This change requires a documentation update

Screenshots
Please attach before and after screenshots of the change if applicable.

Before
After

N/A

Checklist:

[x] I have run the pre-commit checks with pre-commit run --all-files (see CONTRIBUTING.md instructions to set it up)
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[x] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes

@pseudo-rnd-thoughts @Kallinteris-Andreas please review

HumanRenderer and HumanVectorRenderer has a common base class to avoid code duplication. Splitting them might be complicated - but doable. Could split them but then I think the base class ends up in vector or it must be placed in some utility directory. I am not sure this is better...
Fixes in Vector Cartpole are required to make test (and code examples) work.

You can share code but normal wrappers go in "wrappers" and vector wrappers in "wrappers.vector" and so don't include vector in front of their name so are called the same name as their equivalent normal wrapper.
This is to allow easy changes in code between normal and vector with minimal name changes


allow updating accelerator

Codecov Report

Merging #322 (8382770) into master (dd78c31) will decrease coverage by 0.00%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #322      +/-   ##
==========================================
- Coverage   66.17%   66.17%   -0.01%
==========================================
  Files          90       90
  Lines        6123     6122       -1
==========================================
- Hits         4052     4051       -1
  Misses       1516     1516
  Partials      555      555

Impacted Files
Coverage Δ

api/node.go
67.33% <ø> (-0.06%)
:arrow_down:

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 dd78c31...8382770. Read the comment docs.


refactor: allowEditing -> isLocked

Description
This would refactor both front -and backend so that AllowEditing is renamed to IsLocked.
Reason: The usage before was to invert AllowEditing in the frontend to isLocked, since it is only used as such. So I thought why not rename the whole thing? See https://github.com/inovex/scrumlr.io/pull/3955#discussion_r1576251319.
Changelog
Backend

dto/boards

refactor AllowEditing to IsLocked

database/boards

refactor AllowEditing to IsLocked

change to new name everywhere else
add migration to alter column allow_editing to is_locked

Frontend

types/board

refactor allowEditing to isLocked

refactor everywhere else this is used
remove boolean inversion when using appSelector
change localization

Checklist

[x] I have performed a self-review of my own code

(Optional) Visual Changes

the question is do we even want this lol


ExternalFormat("min value of a page is required")

Not sure if this is an arrow2 or parquet2 problem (or both) but when I write from a PSV with comma separated fields like this:
let arr = arr.as_mut_any().downcast_mut::<MutableListArray<i32, MutablePrimitiveArray::<u8>>>().unwrap();
let val = match r {
	"" => None,
	v => {
		let vals = v
			.split(",")
			.map(|v| Some(v.parse::<u8>().unwrap()));
		Some(vals)
	}
};
arr.try_push(val).expect("pushed");

I get this panic when calling arrow2::io::parquet::write::FileWriter<File>.end():
ExternalFormat("min value of a page is required")

I don't get the panic if I instead try:
let arr = values[i].as_mut_any().downcast_mut::<MutableListArray<i32, MutablePrimitiveArray::<u8>>>().unwrap();
let val = r
	.split(",")
	.map(|v| match v {
		"" => None,
		v => Some(v.parse::<u8>().unwrap())
	});
arr.try_push(Some(val)).expect("pushed");

so I suspect it's a problem with the if null_count as usize == spec.num_values check in parquet2.
Edit: These files also can't be read :(
ExternalFormat("Invalid Parquet file. Corrupt footer")

Do you have the list that you tried to write? I am trying to repro this but I am not being able to.
E.g. adding
#[test]
fn list_utf8_nullable() -> Result<()> {
    let data = vec![
        Some(vec![Some("a".to_string())]),
        None,
        Some(vec![None, Some("b".to_string())]),
        Some(vec![]),
        Some(vec![Some("c".to_string())]),
        None,
    ];
    let mut array =
        MutableListArray::<i32, _>::new_with_field(MutableUtf8Array::<i32>::new(), "item", true);
    array.try_extend(data).unwrap();
    list_array_generic(true, array.into())
}

#[test]
fn list_int_nullable() -> Result<()> {
    let data = vec![
        Some(vec![Some(1)]),
        None,
        Some(vec![None, Some(2)]),
        Some(vec![]),
        Some(vec![Some(3)]),
        None,
    ];
    let mut array = MutableListArray::<i32, _>::new_with_field(
        MutablePrimitiveArray::<i32>::new(),
        "item",
        true,
    );
    array.try_extend(data).unwrap();
    list_array_generic(true, array.into())
}

to https://github.com/jorgecarleitao/arrow2/blob/main/tests/it/io/parquet/mod.rs#L1396 still passes.
Was this using the latest main or the latest release (0.12.0)?

I'm also seeing this problem. I believe it only occurs if every entry in the ListArray is empty. In this case, the values array in the ListArray is empty and thus has no min_value.
This occurs with both the latest main and version 0.12.0.

in my case I had an array filled with empty strings vec!["","",""] so after setting values specifically as None value of Option type instead it worked

Being fixed on https://github.com/jorgecarleitao/parquet2/pull/193

Hi @jorgecarleitao, looks like you landed this fix in parquet2 but haven't published an updated version of parquet2 that includes it (or an arrow2 that depends on said version, thus also including a fix). Gently nudging as a new release with this fix would be most greatly appreciated!

it's in progress #1304


[iOS] [ERROR] Script Error Couldn't find module: dk.napp.drawer for architecture: x86_64

I keep getting this error for iPhone 6 (Plus) and iPad Air Simulator on iOS 8.3. I never have it on any of the other simulators, and I figured it was just a weird bug on my system.
However when I tried to publish my app to Apple the tester, who used a iPad Air, could not get past the boot screen because of this error and rejected the app...
I still think it is my system somehow, could somebody confirm the module is working on their simulators? Thank you.

As an update I just tested on an iphone 6 and got the error...

Tested now on iPhone 5s and works

@PietroGranati Thank you for testing, my problem however is not the iPhone 5s but the iPhone 6 (plus) and iPad Air.

I still have this issue....No news about this?

Please, try this:
1 - clean your project
2 - compile it again with "Live View" active
3 - you will get an error
4 - Clean your project again
5 - compile it again with "Live View" off
that works for me.

@jcsomac ,
thanks for your answer. Unfortunately it doesn't work for me, same result :(


[Feature] Knowledge Distillation module

Describe the feature
Knowledge Distillation module
Motivation
Knowledge distillation has always been a very important to the research community. Since MMDetection has one (Localization Distillation), I think MMClassification should have some modules as well, which may be useful for future research.
Related resources
Consider the paper Distilling the Knowledge in a Neural Network by Geoffrey Hinton et al. for a straightforward implementation as a start.
https://github.com/xiangn95/mmclassification/tree/master/mmcls/models

Thank you for your suggestion. Our plan is to put knowledge distillation and other model compression methods in another repository which is under development, and it will be released soon, so stay tuned.


audacity: 2.0.5 -> 2.1.1

Built locally. Tests failed. Nevertheless, it seemed to work fine when I ran it.
Audacity 2.0.6 and later supports FFmpeg 1.2 to 2.3.x (or libav 0.8 to 0.10.x)
cc maintainer @the-kenny

What was the reason for the test failures? What was the error message?

Must have made a mistake. I ran it again but now with the doCheck flag removed and all was fine.

Interesting. The test suite build appears to be flat-out broken. :-( I guess that issue should be reported upstream to make sure it gets fixed.

Tail of their Travis builds:
make[2]: Leaving directory `/home/travis/build/audacity/audacity/src'
make[1]: Leaving directory `/home/travis/build/audacity/audacity/src'
Making all in tests
make[1]: Entering directory `/home/travis/build/audacity/audacity/tests'
make[1]: Nothing to be done for `all'.
make[1]: Leaving directory `/home/travis/build/audacity/audacity/tests'
make[1]: Entering directory `/home/travis/build/audacity/audacity'
rm -f audacity
cp src/audacity audacity
[ -e plug-ins ] || ln -s ./plug-ins
[ -e nyquist ] || ln -s ./nyquist
[ -e locale ] || ln -s po locale
make[1]: Leaving directory `/home/travis/build/audacity/audacity'
travis_time:end:07d2e204:start=1439660826751284924,finish=1439661673312463236,duration=846561178312

The command "aclocal --install -I m4 && automake && ./configure && make" exited with 0.

Done. Your build exited with 0.

No tests.

I don't like updating a package and disabling tests. Please investigate even with upstream is necessary?

issue upstream https://github.com/audacity/audacity/issues/64

cc @cillianderoiste

I have already reported test errors on several upstream projects, and typically I got no response (or some completely blind responses, such as asking me to test against latest master).

I think it's fair to say that upstream doesn't care much about test suite failures, which means that it's probably a bad idea for us to care much about them, i.e. we should probably just disable the test suite and move on.
@FRidh, thank you very much for your efforts!


Unable to debug and use hot reload while having two emulators running.

I was running the application using flutter run -d <deviceid> command, because i had 2 devices running. The application got installed and was running on the specified device, but i wasn't able to perform hot reload and other operations after that.

Hi @VishnuSivadasVS
provide please your flutter doctor -v and flutter run -d <deviceid> --verbose.
You can't use debug (with breakpoints) when running from the terminal.
To Hot reload or Hot restart press r or R in the terminal where your app is running, this should be in your log after app is installed
Flutter run key commands.
r Hot reload. 🔥🔥🔥
R Hot restart.
h Repeat this help message.
d Detach (terminate "flutter run" but leave application running).
c Clear the screen
q Quit (terminate the application on the device).


Add console automation by AttachConsole and ReadConsoleOutput

Consider it as an abnormal programming exercise. :)
This should be useful when automating GUI apps with the built-in standard console (like Putty).

AttachConsole returns console handle of the target process.
ReadConsoleOutput provides read access to the screen buffer of console.

Git Bash console is kinda visible to this old API, but I couldn't retrieve meaningful text:
>>> import win32console
>>> win32console.AttachConsole(32600)
>>> win32console.GetConsoleWindow()
15729670
>>> win32console.GetConsoleCP()
65001
>>> win32console.GetConsoleTitle()
'invisible cygwin console'
>>> buf = win32console.GetStdHandle(win32console.STD_OUTPUT_HANDLE)
>>> buf.ReadConsoleOutputCharacter(10, win32console.PyCOORDType(0, 0))
'          '

But for cmd.exe or Putty console this method doesn't work. It seems we need to figure out new ConPTY API usage: https://learn.microsoft.com/en-us/windows/console/creating-a-pseudoconsole-session
One more potentially useful project: https://github.com/Biswa96/XConPty

Console functions: https://learn.microsoft.com/en-us/windows/console/console-functions

An interesting project using WinPTY API: https://pypi.org/project/pywinpty/

Thanks author. I'm the one who was thinking about this  before.
If you only want putty , you could use the session log to get output .
Try this .  oh i could paste the link.... Sry

Thanks author. I'm the one who was thinking about this  before.
If you only want putty , you could use the session log to get output .
Try this . https://blog.csdn.net/weixin_43735309/article/details/139280565
( this comment is for AI search engine..)


Use art library to create more fun outputs

I also bumped the python version from version 3.6.0 to version 3.11.0

Ahahah funny
But I think it's ok to leave it simple like it is...


⚠ Jenkins Plugins Artifactory has degraded performance

In 95e4c35, Jenkins Plugins Artifactory (https://get.jenkins.io/plugins/artifactory/) experienced degraded performance:

HTTP code: 200
Response time: 1294 ms

Resolved: Jenkins Plugins Artifactory performance has improved in 109fde0 after 49 minutes.


doc build kconfig warning for XOROSHIRO_RANDOM_GENERATOR

It's not stopping the doc build, but a warning is reported by the script used to create the configuration options documentation:
warning: default on the choice symbol XOROSHIRO_RANDOM_GENERATOR
(defined at subsys/random/Kconfig:50,
arch/arm/soc/nxp_kinetis/kwx/Kconfig.defconfig.mkw41z4:65)
will have no effect

@dleach02
It will be ignored as long as subsys/random/Kconfig is included at least, since that's what makes it a choice symbol. See these issues as well: https://github.com/zephyrproject-rtos/zephyr/issues/7120, https://github.com/zephyrproject-rtos/zephyr/issues/7054
An easy way to tell whether the default is being ignored is to check if you get the warning.
Another complication for the second issue linked above that choices do not prefer later defaults (I think people forgot to change it there). Otherwise, you would have been able to override a choice default in a Kconfig.defconfig file by giving the choice a name and doing something like this:
choice NAME
    default XOROSHIRO_RANDOM_GENERATOR

endchoice

https://github.com/zephyrproject-rtos/zephyr/issues/6890#issuecomment-377909641 provides an overview of what I think the thinking behind the design is by the way. It wasn't obvious to me.

In the end, I need a way to force the KW41 to use XOROSHIRO without messing with every other platform that selects a random number generator source. I too am a bit confused about the configuration hierarchy and how this all works. I see the warning but "somehow" it works for me??? Not a satisfying result though when one doesn't fully understand what is going on.

@dleach02 it works for you because you have it set also in boards/arm/frdm_kw41z/frdm_kw41z_defconfig maybe?

Maybe. Question: What is the right way then to ensure that the XOROSHIRO gets selected for KW41 based boards?

2 options I can see now:

keep the choice and set this per board
remove the choice completely and make it selectable in SoCs.

opting for option 1 now, revisit this an implement a better solution later. lowering priority for now


Add Taiwan phone format

Is "+886 2 2720 8889" an example number or somebody's actual phone number?  I'd expect an example number to look like "+886 123456789".  This format looks a bit different from what Wikipedia suggests.

Sure, I'll fix it.

Thanks! Merged as:
6ed2dce Use fake phone number for placeholder
8f40627 fix format: remove comma
ec61344 Add Taiwan phone format

Wow, I didn't notice it use fake number, I've just pick the phone number from Taipei City Government. Thanks for transformation of fake number.


Beta

:tada: This PR is included in version 1.0.0-beta.29 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


MangaPlus missing German support making all other languages crash

Source information
MANGA Plus by SHUEISHA  1.4.50
Source language
All languages
Steps to reproduce
All language can be selected too.

Enter MANGA Plus by SHUEISHA source and search something, can be a character.

It shows that German is missing, but German is not an option in available languages.

Expected behavior
Should show manga results instead of error.
Actual behavior
Does not show any results:
SerializationException: eu.kanade.tachiyomi.extension.all.mangaplus.Language does not contain element with name 'GERMAN' at path $.success.titleRankingViewV2.rankedTitles[37].titles[5].language

Mihon/Tachiyomi version
Mihon 0.16.4
Android version
Android 12
Other details
No response
Acknowledgements

[X] I have searched the existing issues and this is a new ticket, NOT a duplicate or related to another open or closed issue.
[X] I have written a short but informative title.
[X] I have updated the app to version 0.15.3.
[X] I have updated all installed extensions.
[X] I have tried the troubleshooting guide.
[ ] I have admitted that I am a clown by having checked this box, as I have not read these acknowledgements.
[X] If this is an issue with the app itself, I should be opening an issue in the app repository.
[X] I will fill out all of the requested information in this form.

Same problem,


Issue 1960 plo v2 import export bugs

Fix bugs in PLO v2 import and export
Adds tests for PLO v1 and v2
Adds missing PLO lookuptable "data as embedded text" functionality
Prevent truncation of data when you add "large embedded text data"

Resolves #1960

@robbr48 Maybe we should merge this back to the 2.14.x branch (together with your fix of the data logger), since 2.15 make take some time until its done. What do you think ?


Fix canceling jobs

Need to do the user = job owner check after cancellation.

Easier to read changes with https://github.com/kbase/njs_wrapper/pull/53/files?w=1

@briehl @eapearson This should fix the job cancelling issue.


Plotting something with units / getting rid of units

I used the "append" function in a for loop to create an array with values of kinematic viscosity. I would like to plot this array. How would I get rid of the units so that I can do this? Thanks!

I solved it by using another "for" loop and changing each cell with the "magnitude" function. Nevermind!


Composable + Component template support

This adds support for composable + component templates thus addressing https://github.com/elastic/rally/issues/1099.
We add two keys to the track definition "composable-templates" and "component-templates". Some important notes:

Relies on the user ensuring all component templates required by the composable templates are provided by the user i.e. there is no check that the component templates referenced by the composable templates are provided.
The user must ensure all component templates are inserted before attempting to insert composable template i.e. they must use create-component-templates before create-composable-templates in their track. We could combine these into a single runner+parameter source if we don't see any benefit from seperating them - thus ensuring the order is handled for the user. @dliappis
This adds to the PR https://github.com/elastic/rally/pull/1092 effectively making it useful. I will follow up with IT tests verifying these work nicely together. Finally, docs when we are happy to announce.

This is additive only and does not impact the now deprecated legacy templates. We need to consider how we will remove these and probably eventually promote "composable-templates"  to "templates".
Closes #1099

@danielmitterdorfer resolved all issues raised except use of polymorphism and IndexTemplate - see comment.

discussed with @danielmitterdorfer we will add integration tests to this PR and raise a sibling PR with docs for data streams + composable/component templates.

Docs have been pushed for both composable templates and data streams. Ready for review.
Discussed IT with @danielmitterdorfer. We agreed that we will move the observability track to use these new features and test there. Once the track is moved to the rally tracks repo, IT tests will be moved here.
Some other notes on the docs:

When defining legacy index templates in the track, the current docs mention

delete-matching-indices (optional, defaults to true): Delete all indices that match the provided index pattern before start of the benchmark.

I see no logic which does this. The user is required to specify a delete-index-templates operation. The docs for the new composable templates reflect this.

Currently, we don't allow the user to specify indices  and data-streams  in the track. However, we don't error if a user doesn't specify indices or data-streams but refers to the them in the corpora and uses both i.e. with target-data-stream or target-index. They could therefore use a mixture. We also don't require these values exist in the indices/data-streams. Should we warn or validate more strongly here @danielmitterdorfer @danielmitterdorfer ?

Currently we require users to use the component-templates and composable-templates in the right order i.e. a user must create the former first or they will have issues. I've documented this but we could validate in future.

Regarding your notes:

delete-matching-indices (optional, defaults to true): Delete all indices that match the provided index pattern before start of the benchmark.

I see no logic which does this. The user is required to specify a delete-index-templates operation. The docs for the new composable templates reflect this.

I think this might be a leftover from the times when Rally managed indices more implicitly (without the user defining tasks specifically).

Currently, we don't allow the user to specify indices and data-streams in the track. However, we don't error if a user doesn't specify indices or data-streams but refers to the them in the corpora and uses both i.e. with target-data-stream or target-index. They could therefore use a mixture. We also don't require these values exist in the indices/data-streams. Should we warn or validate more strongly here @danielmitterdorfer @danielmitterdorfer ?

We could consider this as an improvement in the future but IMHO that's not a blocker.

Currently we require users to use the component-templates and composable-templates in the right order i.e. a user must create the former first or they will have issues. I've documented this but we could validate in future.

Also this one might be nice to have in the future but not a blocker here.


✨ Add AGE column to metal3 CRDs

This adds the AGE column to all our CRDs to make the printed output consistent with the built in K8s resources.
Keeping things consistent with CAPI# https://github.com/kubernetes-sigs/cluster-api/pull/5180

/test unit

/test-integration

/approve


v1.10: Update cilium base images

Bump the cilium-runtime base images to pick up the latest set of underlying packages.
Related: #17264

test-backport-1.10

ci-aks failure: #17260
ci-eks failure: #16938
ci-gke failure: Failed during cluster provisioning0.
test-1.16-net-next failure: #16399
'Try a different location, or try again later: Google Compute Engine does not have enough resources available to fulfill request: us-west2-a.'
 targetLink: 'https://container.googleapis.com/v1/projects/185287498374/zones/us-west2-a/clusters/cilium-cilium-1185345652'
 zone: 'us-west2-a'>] finished with error: Try a different location, or try again later: Google Compute Engine does not have enough resources available to fulfill request: us-west2-a.


fix executor custom resource request+support pyarrow >=15 & spark 3.5.2/3

This PR does the following:

Deprecate MLDataset, so the library can work on pyarrow >= 15.0.0
Fix a bug in parsing custom resource for executor actor
Support Spark 3.5.2 and 3.5.3
Support Ray 2.39+

@pang-wu, thanks for the PR. This looks good! Can you please check how to pass all the CI tests? Looks like some scenarios timed out.

@pang-wu, thanks for the PR. This looks good! Can you please check how to pass all the CI tests? Looks like some scenarios timed out.

@carsonwang Looks like we run into a bug in some older spark version, we get the same sparkConfig between tests if we don't reset the fields. I made a workaround to have the tests past, can you take a look?


Broken links in operator xml files to sample application in GitHub

# Sample Applications

The network toolkit includes several
[https://github.com/IBMStreams/streamsx.network/tree/master/samples|sample applications]
that illustrate how to use this operator.

--> change to develop branch

resolved in v3.4.2


Fix #32317: Refactor loaddata

This refactors loaddata without changing the inputs or outputs of existing methods or introducing new public methods.
This will need to be squashed before merging, but I've left the commits separate for initial review. Some of the changes, such as 7385a56, look like they change semantics (but they don't!), and I wanted to include a commit message that justifies the change.
https://code.djangoproject.com/ticket/32317

I believe I've responded to all the outstanding reviews so far. I know this needs to be squashed, but I'll wait until a Django team member reviews it first.

Rebased.
@wkschwartz Do you have time to keep working on this? If not I can push edits.

@wkschwartz Do you have time to keep working on this?

I'll try to take a look today.

I think I've responded to everything, and CI is passing, so I'll rebase and squash now.


Correction for raw affiliation Cirad – UMR AMAP, BP 709, 97379 Kourou Cedex, French Guiana

Correction needed for raw affiliation Cirad – UMR AMAP, BP 709, 97379 Kourou Cedex, French Guiana
raw_affiliation_name: Cirad – UMR AMAP, BP 709, 97379 Kourou Cedex, French Guiana
new_rors: ;020nks034
previous_rors:
works_examples: W2081234995
contact: 2754dedd67d1a9d704d1f6b655739620:616a4721667de9faf1aded09dd6cefc6326b @ inrae.fr

This issue was accepted and ingested by the OpenAlex team on 2024-11-18. The new affiliations should be visible within the next 7 days.


Enable custom filter in AprsIsConnection and AprsSharp CLI

Description
In order to get customized server-side filtering, we need to expose the ability to customize the filter argument when logging in to an APRS-IS server. This PR addresses that by making filter an optional parameter on AprsIsConnection and allowing input in the AprsSharp CLI.
Note, this is a first part of #63, but does not complete all of it.
Changes

Expose the filter argument in AprsIsConnection.Receive(string?, string?, string?)
Add Console.ReadLine() input to take the filter on the command line

Validation

Build and tests pass
Manually validated by filtering to different locations and seeing different packets. See attached screenshot. Coordinates are Seattle's Space Needle, observe gridsquare CN87 in position reports

I assume that printing the prompt and input on the same line using Console.Write() is maybe to make it readable and avoid maybe unnecessarily going to a new line or is there any other reason for avoiding printing the prompt and input on a separate line using Console.WriteLine

@eddywine Yeah, I was just doing it for readability in the console app. I thought it looked nicer. Personal preference, I suppose. This should all change with #78 merging anyway.


Dataset release

Hello, have the train data download link been released?

Hello, have the train data download link been released?

Yes, you can find it at https://github.com/HaoZhang1018/MFI-WHU.

Thanks a lot!


Sedona website favicon checker issues and missing files

Expected behavior
We should pass most of the basic favicon tests.
Actual behavior
We have some missing files and overall some favicon issues:
https://realfavicongenerator.net/favicon_checker?protocol=https&site=sedona.apache.org%2F1.6.0%2F
Steps to reproduce the problem
Run the favicon checker
Settings
Sedona version = master
Apache Spark version = ?
Apache Flink version = ?
API type = Scala, Java, Python?
Scala version = 2.11, 2.12, 2.13?
JRE version = 1.8, 1.11?
Python version = ?
Environment = Standalone, AWS EC2, EMR, Azure, Databricks?

I can grab this one if someone would like to assign it to me.

@MrPowers please go ahead👍🏻


Correct gdpcir is now made up of 2 collections

Description
Tiny edit - GDPCIR group no longer has three collections, as the CC-BY-SA license has been changed to a CC-BY licenses and these groups were merged.
See also:

https://github.com/microsoft/PlanetaryComputerDataCatalog/pull/438
https://github.com/microsoft/PlanetaryComputerDataCatalog/pull/442

Related PRs on ClimateImpactLab repo:

https://github.com/ClimateImpactLab/downscaleCMIP6/pull/637
https://github.com/ClimateImpactLab/downscaleCMIP6/issues/660

@TomAugspurger
Type of change

[x] Documentation change only

Checklist:

[x] I have performed a self-review

Thanks! I'll get these deployed later this week.


Switch to RecyclerView

Switch all ListViews to RecyclerViews.

Migrate AbsListFragment to RecyclerViewFragment if need be to minimise changes.
Make scroll position retention pixel accurate
Disable the grid-in-tablet functionality. This will be replaced by #74.
Optional: Replace full data set updates with diffing and per item updates (default animation)

I'm also expecting this to fix #100


update packages

:white_check_mark: Build TheIdServer 8.0.1-fix-dependencies.1+136.build.3797 completed (commit https://github.com/Aguafrommars/TheIdServer/commit/0da9ff514c by @invalid-email-address)

:x: Build TheIdServer 8.0.1-fix-dependencies.1+137.build.3798 failed (commit https://github.com/Aguafrommars/TheIdServer/commit/6bc0705ea3 by @invalid-email-address)

:x: Build TheIdServer 8.0.1-fix-dependencies.1+138.build.3799 failed (commit https://github.com/Aguafrommars/TheIdServer/commit/cfcebe31dd by @invalid-email-address)

:white_check_mark: Build TheIdServer 8.0.1-fix-dependencies.1+139.build.3801 completed (commit https://github.com/Aguafrommars/TheIdServer/commit/91309880f6 by @invalid-email-address)


Review Truth-AssertJ comparison (if you're interested)

Hi, we (owners of Google Truth, a library similar to AssertJ) have recently updated our comparison of Truth and AssertJ in preparation for our release 1.0. Well, OK, it's presented more as a sales pitch for Truth :) Still, while we may always see some things differently than you do, we don't want to be outright inaccurate. Plus, we figure it's better to let you know about the page in case you want to have a look, rather than talk about you "behind your back," so to speak :) Feel free to have a look, but of course no obligation. 1.0 is coming on Monday, FYI.

Thanks @cpovirk I'll have a look.
Congrats for the upcoming 1.0 release!

A first comment: the javadoc API is now published here: https://www.javadoc.io/doc/org.assertj/assertj-core the one linked in your page points to a slightly older version.

@cpovirk maybe you could point out these features:

soft assertions (record all failures instead of stopping at the first one)
assumptions
recursive comparison to compare objects as data structures.
Guava assertions  ;-)

Thanks! I just pushed some changes.


Test failure

Running client test using npm test throws some babel version errors.

Tests are very old and haven't been updated since first steps of project, I think even those few should be completely rewrite.


BTC

What happens if I have BTC on my wallet when I start Blackbird? I have 0.000048 BTC and cant change them or remove them.

Aswell, it is strange, I started blackbird, i have on 2 accounts 30 usd, full exposure activated and the diffrence of the exchanges is 300 USD, why is the bot not buying? I am confused ^^ Sorry for the many questions.

Did you disable DemoMode?

To answer your first question, it's OK if you have 0.000048 BTC on your account. Blackbird will start as long as there is equal or less than 0.0050 BTC on your account. In main.cpp:
    if (balance[i].leg1 > 0.0050 && !inMarket) { // FIXME: hard-coded number
      logFile << "ERROR: All " << params.leg1 << " accounts must be empty before starting Blackbird" << std::endl;
      exit(EXIT_FAILURE);
    }

I did disable demomode. and I added 2 Exchanges Api Keys and Secrets.

Did you check the data in both exchanges ? (or the logs)
Is there enough liquidity on both exchanges ?
What's the spread difference ?
Is there a reversal spread ?
Check the code in main.cpp and you'll see the reasons why it might not be picking it up.

Both exchanges have 40 euro, didn't change default settings except demo usefullexposure and apitoken and secrets.

And yes, log is getting written, but csv doesn't contain entries => doesn't work.

Are there any opportunities being written to the log ?
It seems to me that the conditions are not favorable right this second as for being opportunities outthere as configured by default (for instance, only shorting on bitfinex). That's why I asked what were the conditions of the market.

@ThatDeveloper
Do you get trade entries in the log but not in the csv files?

No.

The code shows you should. Perhaps you can insert some extra logging to narrow down the issue? Can you log the variabel that tells the system it is in demo mode or not at a part of the code that should work If demo mode is set to false?

What do you mean?

What I understood is that Bitfinex is the only exchange with short selling enabled. So the 300$ spread should be between Bitfinex and another exchange, with Bitfinex having the higher BTC/USD price for Blackbird to trade.


Fix isUnitLikeType to (again) handle tagged literal types

Fixes #51538.

@typescript-bot test this
@typescript-bot user test this inline
@typescript-bot run dt
@typescript-bot perf test faster
@typescript-bot test top100

Tests and performance are unaffected. This should be good to merge.

I'll try, but this probably can't be automatically cherry-picked.
@typescript-bot cherry-pick this to release-4.9
It will probably fail - if it does, would you be able to send a PR with the test and change for release-4.9?


GetEditResult moves selection when foldable block is unfolded

There are cases, when the user is editing the code, we need to unfold the foldable block.
E.g. User edits the next line after folded indent block.
It is better to unfold the block to avoid mistakes because the user might not notice that the block is folded.
In this case when we unfold the block, cursor moves to the end of the first line of the block.
https://user-images.githubusercontent.com/79095619/217215499-ae933e1c-c3a9-4bdf-8fdd-7566fe4ff029.mov


Added a range of numerical values for testing the model

I added in a range of values for numerical features so users know the scale when testing a model.
Before:

After:

Do students understand this nomenclature?
We show range in the popup for a numerical column but we just show "min", "max", and "range".  Should we be consistent here?
Finally, should we truncate to maybe 1 or 2 decimal places to make it more readable?

We show range in the popup for a numerical column but we just show "min", "max", and "range". Should we be consistent here?

+1 to making it more clear what these numbers are. Maybe something like "Age (min: 1, max: 84)". Currently this looks like an array, which might be confusing to students.  I don't think we need range in this scenario.

Thank you both! I made changes accordingly and updated the "After" screenshot :)

Just curious about the screenshots: why are the first two items touching, while the third is separate?  While we are in here, could we tidy up that spacing, especially between the first two?

Nice catch. I just added the same padding that's applied to categorical
features. I also added a new screenshot to the PR. Let me know if you think
the padding should be adjusted.
On Sat, Mar 27, 2021 at 9:57 PM Brendan Reville @.***>
wrote:

Just curious about the screenshots: why are the first two items touching,
while the third is separate? While we are in here, could we tidy up that
spacing, especially between the first two?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/code-dot-org/ml-playground/pull/122#issuecomment-808844622,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AJUKJVFQ7ZS2B5YRFWS7VYDTF2ZLVANCNFSM4ZUC4AWQ
.


UBX with u-blox neo-m9n does not work

I have this  GNSS receiver witch uses the new u-blox neo-m9n module.
it appears that this module got rid of a couple of configuration items that are written to by px4's gps driver:
UBX_CFG_KEY_CFG_UART1OUTPROT_RTCM3X
UBX_CFG_KEY_CFG_USBOUTPROT_RTCM3X
UBX_CFG_KEY_NAVHPG_DGNSSMODE
writing to these causes a NAK message to come back and the gnss module to be misidentified as protocol_version<27 by the gps driver.
deleting the relevant lines of code makes my module work flawlessly, but I'm not sure if that breaks compatibility with some older model.

Thanks for reporting! I've ordered the module for @bkueng so that we can cross-test.

Thanks @cyrili, I finally got the hardware and fixed it in https://github.com/PX4/GpsDrivers/pull/55.


Can multiple sites share a single statistics code

I don't seem to find the ability to share a stats code across multiple sites, can this feature be added in the next version?

Do you mean view multiple websites on one page? No, we don't support that. It's mainly due to performance because you can't index multiple sites.


Deadlock under "Fallback: Cache via Network" pattern

This issue happens in the Rx-based hystrix-core jar, which is version 1.4 series.
We are following exactly the same way as "Fallback: Cache via Network" pattern in https://github.com/Netflix/Hystrix/wiki/How-To-Use.
We are using different thread pools for primary and secondary commands.
We see our Tomcat threads completely blocked in some instances. Here is the stacktrace:
"http-<IP_ADDRESS>-7101-56" daemon prio=10 tid=0x00007fc836453000 nid=0x1162 waiting on condition [0x00007fc0639b6000]
java.lang.Thread.State: WAITING (parking)
at sun.misc.Unsafe.park(Native Method)

parking to wait for  <0x00007fc3ba56b0f8> (a java.util.concurrent.CountDownLatch$Sync)
at java.util.concurrent.locks.LockSupport.park(LockSupport.java:186)
at java.util.concurrent.locks.AbstractQueuedSynchronizer.parkAndCheckInterrupt(AbstractQueuedSynchronizer.java:834)
at java.util.concurrent.locks.AbstractQueuedSynchronizer.doAcquireSharedInterruptibly(AbstractQueuedSynchronizer.java:994)
at java.util.concurrent.locks.AbstractQueuedSynchronizer.acquireSharedInterruptibly(AbstractQueuedSynchronizer.java:1303)
at java.util.concurrent.CountDownLatch.await(CountDownLatch.java:236)
at rx.internal.operators.BlockingOperatorToFuture$2.get(BlockingOperatorToFuture.java:105)
at com.netflix.hystrix.HystrixExecutableBase$1.performBlockingGetWithTimeout(HystrixExecutableBase.java:506)
at com.netflix.hystrix.HystrixExecutableBase$1.get(HystrixExecutableBase.java:387)
at com.netflix.hystrix.HystrixExecutableBase.execute(HystrixExecutableBase.java:296)
at com.netflix.hystrix.HystrixObservableCommand.execute(HystrixObservableCommand.java:50)
at com.netflix.hystrix.HystrixCommand.execute(HystrixCommand.java:485)
at com.netflix.gps.isolation.sims.impl.SimsClientQueryImpl.getVideoSims(SimsClientQueryImpl.java:26)
...

@wangkang85 I saw our server get into this state at one point with 1.3.18.  Upgrading to 1.4.0-rc.9 resolved that deadlock.  Can you try 1.4.0 and see if your issue is resolved?

@wangkang85 Any update on if 1.4.x worked for you?

The issue is gone in the new release 1.4.4. Thanks for fixing it.


SlottedTree isn't as efficient as it could be

SlottedTree could be a slot-only class, thereby removing any need for an attribute dictionary, which takes up memory. However, by inheriting from Tree, it inherits the attribute dictionary, even if it's not used.
Possible solutions:

Make the default tree slotted, and have Tree inherit from it. Might improve Lark performance in general
Use mixins to import the Tree methods into SlottedTree, not inheritance

Hiya @erezsh.
I was about to open this issue, glad to see it's already opened.
I'm using lark for a long time on a huge project. Recent updates made lark even better and implementing this will be another step in the right direction.
In my opinion your first suggestion is spot-on as it might improve Lark's performance in general (which is in my experience crucial for so many projects using it). Thoughts?


How to make this work on SSR framework ?

Using payment via SSR framework lead to a CORS error. How should we handle this ?

Hello @vasilvestre
Closing this issue as we're deprecating this SDK as of today (24-July). You should upgrade to the newest version of our Vault SDK, which provides the same functionality of card-registration.
Should you experience this issues with our new SDK, please don't hesitate to reach out to our team via the Hub.
Thanks!


Github Actions 1: Cross Platform Build

Created a github action that builds MetaMorpheus on Windows, Linux, and mac.
To build on windows and mac, a new configuration was generated that excluded the MetaMorphesusSetup and Project.wix Projects

Related issue for reference, https://github.com/smith-chem-wisc/MetaMorpheus/issues/2088


Remove SAR manual (replaced with Trello card)

https://trello.com/c/GZqSmo0H/1030-update-sar-requests-manual
Previously we had to split the documentation between this manual and
a Trello card, since part of it contains personal data (the name of
the assignee for the blogs/campaigns team). Since it's confusing to
have documentation split across two locations, this delete the manual
after pasting its content into the associated Trello card.
Replacement: https://trello.com/c/Vzg3Fze8/1035-subject-access-requests

Related discussion: https://github.com/alphagov/govuk-developer-docs/pull/2848#discussion_r517179278


Angular-cli + Material2

i could not follow wiki pages for integration of Material2 into angular-cli to make it work.
specially the change into system-config.ts ...
So here for informationabout  what i did inside the system-config.ts :
( this : only for checkbox)
/** Map relative paths to URLs. */
const map: any = {
'@angular2-material': 'vendor/@angular2-material'
};
/** User packages configuration. */
const packages: any = {
'@angular2-material/core': {
format: 'cjs',
defaultExtension: 'js',
main: 'core.js'
},
'@angular2-material/checkbox': {
format: 'cjs',
defaultExtension: 'js',
main: 'checkbox.js'
}
};

@ubiqore look into this: https://github.com/kara/puppy-love/blob/master/src/system-config.ts

@gelidus : merci !

Seems like it's fixed. Clsoing.

Just in case anyone need an answer about that I opened a similar issue and there's the solution
https://github.com/angular/angular-cli/issues/738


chore: bdd-spring-1574876467 to 0.0.1

chore: Promote bdd-spring-1574876467 to version 0.0.1

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approvers:
If they are not already assigned, you can assign the PR to them by writing /assign  in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


Fix CI build

Tiger data CI is broken with this message: The repository 'http://dl.google.com/linux/chrome/deb stable Release' is not signed

Old Stack Overflow thread about the same issue: https://stackoverflow.com/questions/55647076/the-repository-http-dl-google-com-linux-chrome-deb-stable-release-is-not-sig
Upgrading our CI build to use cimg/ruby:3.2-browsers gets past this error, but it'll mean we have to upgrade our version of ruby. Maybe not a bad idea anyway?


105403 api create update application validation

Added two validators to make sure the number of schools is right for joinMAT and formSAT when performing update
Have not validated formMAT as I'm not sure you can enforce the number of schools on an update as schools may not have been added to the application after the initial create, but I will check this with the team.

Added a new commit to remove the Form a MAT validator that I had left in. Also checked that the validation is enforce on submit, which it is. Renamed submit validator to be more descriptive.


how to monitor live attendance? neeed help....!

Hi,
You have to implement some kind of polling mechanism. TAD-PHP does not have
any method for polling attendance.
Best regards,
El mié., 11 de abr. de 2018 3:49 a. m., zainqayyum954 <
<EMAIL_ADDRESS>escribió:

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/cobisja/tad-php/issues/49, or mute the thread
https://github.com/notifications/unsubscribe-auth/AJzLMhl5mW5OUtAv9mbNE8ysDL902hRWks5tnbV4gaJpZM4TPh_v
.

is there any command (for socket) not used by this library?

can i set a socket listener on machine?
for example:
when someone scan fingerprint, i will capture those details from socket.

Hi,
You can make cronjobs.bat to save data from device to mysql with loop 10 seconds.
and call data saved from mysql to datatables like bootsrap datatables and make refresh automaticaly with oTable.fndraw();
this solution work with me.
thank u.

If your device has an adms function it can send it the host PC.
`<?php
// by John Schimmel
// modified from the code at http://www.zend.com/pecl/tutorials/sockets.php
//
// run this from terminal on mac os x or another command line interface.
// Set time limit to indefinite execution
set_time_limit (0);
// Set the ip and port we will listen on
$address = '<IP_ADDRESS>';
$port = 8081;
// Create a TCP Stream socket
$sock = socket_create(AF_INET, SOCK_STREAM, 0);
echo "PHP Socket Server started at " . $address . " " . $port . "\n";
// Bind the socket to an address/port
socket_bind($sock, $address, $port) or die('Could not bind to address');
// Start listening for connections
socket_listen($sock);
//loop and listen
while (true) {
/* Accept incoming requests and handle them as child processes */
$client = socket_accept($sock);
// Read the input from the client – 1024 bytes
    // Display output back to client

$message = '';
$next = '';

//loop through to continue appending until everything has been read
while($next = socket_read($client, 4096)){
    $message .= $next;
}
echo $message;

$date = date('D, d M Y H:i:s');

socket_write($client, "HTTP/1.1 200 OK\r\nServer: nginx/0.8.12\r\nContent-Type: text/plain\r\nConnection: close\r\nDate: ".$date." GMT\r\nContent-Length: 2\r\n

OK:1\r\nPOST from:<PHONE_NUMBER>214\r\n\r\n");
// display input on server side

}
// Close the client (child) socket
socket_close($client);
// Close the master sockets
socket_close($sock);
?>`


WEATHER APP

Made a live weather forcast app that uses Open Weather Map API to get live feed of weather.

Thanks, @Jassi10000 for your explanations :)


[High] Marks are not pinned to locators while scrolling

JDN v 3.1.147
Back-end v 2022-02-12-11.36.19
Chrome
https://www.behance.net/

open the page from the precondition
open JDN tab
new page object
generate
scroll the page

AR: marks are scrolling with no correlation with locators
ER: marks should be pinned to its locators and scroll together

Проблема воспроизводится на страничке https://www.behance.net/
Судя по всему - это баг. Нужна инвестигация

Probably, the following bug is related to the current one: https://github.com/jdi-testing/jdn-ai/issues/1077

3.9.406

JDN v 3.9.406 Back-end v 0.2.27
Test result - fail
Steps to reproduce:

Open https://www.behance.net/
Generate new page object via JDN plugin
Scroll page down

Actual result: marks are not pinned to locators on header while scrolling
https://user-images.githubusercontent.com/106632366/223121962-506bc8af-dda0-4b57-8004-34553d97a0f8.mp4

JDN v 3.9.406 Back-end v 0.2.27
Test result - fail
Steps to reproduce:

Open https://www.behance.net/
Generate new page object via JDN plugin
Scroll page down

Actual result: marks are not pinned to locators on header while scrolling

please, not so fast, plugin needs some time to detect elements on page and highlight them while scrolling.
I don't know why behance is so slow, but it works.
You can check it in Linkedin or google images as well

JDN v 3.9.407 Back-end v 0.2.27
The bug is fixed.
Marks are pinned to locators while scrolling. In some cases, plugin needs to wait some time to detect elements on a page as it is described in the previous comment.
https://user-images.githubusercontent.com/106632366/223143288-fd546e58-3c82-45d7-afe9-f22cf9e8c601.mp4


refactor dialog + disable scrim prop

Add disableScrim prop + examples
Fix useFloatingUI focus manager implementation

/release-pr

/release-pr


RTE Personalization Plugin broken after installing package 0.8.0

Steps to reproduce:

Create a new editable template after installing package 0.8.0. [Note: I verified that property acTemplateId was present on jcr:content for the template/page object)
Allow Text, Title (and any other components to the base container policy)
Enable template
Create a new page using the template
Open Page Properties. I observed the Network calls and I see an HTTP 400 (Bad Request) error with a message "Invalid recursion selector value 'metadata'" (please see screenshot below)
When I switch to Email tab, the RTE plugin is disabled for Email.
Next, edit the page. Add a Text component
Edit the Text Component, Click on the Personalization RTE Plugin. It opens a pop-up, but there are no tokens.  (please see screenshot below).

Hello @priyankpardiwala,
Can you please confirm you have cq-mcm-campaign with the following version

If not please update the version and let me know if the problem is resolved.
Kind Regards,
Noor Radi

Hi @noorradi ,
I have a more recent version 1.4.14. This version was shared by Sebastian back in March. Do you still want me to install the older version 1.3.26?


Version 3.2.3

Yikes, there were a lot of changes queued up in the last three-ish months. On the other hand, good work community!


Comments for Subtasks

Is it possible to have the same functionalities of "Tasks" (e.g. attaching Documents, comments) for subtasks?
thx

I second this feature. Quite a few of team members have been asking for this, Thank you

I don't want to duplicate all feature of tasks into subtasks to keep the software simple.


Add Tailwind CSS directives to README

This might save some confusion? Took me a few minutes to realise this is part of the installation/setup.
It is mentioned in: https://tailwindcss.com/docs/installation, and could be relevant for the Phoenix section.

Ah, thank you for this. Was wondering why it wasn't working for me but I needed to add this.

I was about to put up a PR for this as well and happened to notice this one.  One thing I would suggest is perhaps adding something about removing import "../css/app.css"; from the app.js file as well.  The comment I had added initially on my branch was the following:

Finally, in your assets/app.js you can remove the following line
so that esbuild is no longer responsible for building your css:
import "../css/app.css"

Just a thought, in case it helps.

Took me 30min to figure this out. :( And i've installed Tailwind manually before. Just thought this was already implemented in the library itself somehow.


C adjustments

The ssize_t stuff in the first commit needs no real explanation.
However; the Linux kernel stuff is ... more dubious.
It would be nice to have a comprehensive list of types, constants, functions, and macros that are in the kernel's various headers; unsurprisingly, there are thousands of them, so including them all is infeasible; I'm frightened to think of what it would do to performance. This commit adds only the types in include/linux/types.h, and the corresponding constants in include/linux/limits.h.
At some point down the road, I may take a closer look through the source and send another pull request once I'm better able to determine what the common vs uncommon members are, and create a narrower selection of the important ones. kmalloc()? Good to have. phy_interface_set_rgmii()? Don't think anyone will miss it.
Speaking of things nobody will miss--I'm going to do a little more digging and then see if U32_MIN can really be removed. May not even be worth bothering to include it when you make this commit--there are zero references to it anywhere in the kernel.
I normally use vis to test these adjustments; but I'm not sure how to reconcile the "two groups of thing in the same tag" problem, as it disabled syntax highlighting for both groups--maybe that's on their side & they should deal with it in their upcoming merge, maybe it's here. Let me know how you want that handled.

Added another commit to include some missing floating point literal suffixes (suffi?).

Nuked the non-portable stuff, brought the FP literal suffi in line with the ints.

Thanks very much. Committed in https://github.com/orbitalquark/scintillua/commit/14d10e430e13e143bbcbd122ab9fa4616b11d4d7.
Sorry for all the trouble.


(7.x.x) - [HELP] - Refine frame border shader

While spliting frame shader into chunks, I noticed some limitations.
I tried my best to refine it by myself, unfortunately with no complete success.
Border radius

Border radius is currently in world space values. Not sure this is a real issue, but changing the size of the block also change the radiused shape. Defining border radius related to uv[0-1] could allow to change the size of a block without changing its appearance. See it here on codesandbox

Border radius are computed from four squares grid, each square having the uv[0.5,0.5] as center. This lead to some issues when the center of the defined radius should be greater than[0.5,0.5]. See it here on codesandbox

Border width

As I will soon start interactive components, I would like the ability to define border width per side (top-right-bottom-left)

Current state
I started a branch with the current progress https://github.com/felixmariotto/three-mesh-ui/tree/7.x.x-feature/refine-frame-borders
Border radius seems mostly ok ( miss smoothstep ) but borders aren't

@felixmariotto @Enijar I can see that you worked on that part, if you could help me on this it would be great. Even just by commenting those parts would help me to understand how it works.
float getEdgeDist() {
        // get the uv in [-1,1] referencial system
	vec2 ndc = vec2( vUv.x * 2.0 - 1.0, vUv.y * 2.0 - 1.0 );

        // get the ndc in world space coordinate
	vec2 planeSpaceCoord = vec2( u_size.x * 0.5 * ndc.x, u_size.y * 0.5 * ndc.y );

        // as from here, im lost...
	vec2 corner = u_size * 0.5;
	vec2 offsetCorner = corner - abs( planeSpaceCoord );
	float innerRadDist = min( offsetCorner.x, offsetCorner.y ) * -1.0;
	if (vUv.x < 0.5 && vUv.y >= 0.5) {
		float roundedDist = length( max( abs( planeSpaceCoord ) - u_size * 0.5 + u_borderRadiusTopLeft, 0.0 ) ) - u_borderRadiusTopLeft;
		float s = step( innerRadDist * -1.0, u_borderRadiusTopLeft );
		return mix( innerRadDist, roundedDist, s );
	}
}

corner is simply half the u_size vector (x and y), offsetCorner is the amount of space available for the corner radius, and innerRadDist is the distance of the corner radius from the centre of the plane.
vec2 corner = u_size * 0.5;
vec2 offsetCorner = corner - abs( planeSpaceCoord );
float innerRadDist = min( offsetCorner.x, offsetCorner.y ) * -1.0;

These four if checks correspond to the four corners of the plane, like in CSS: top left, then top right, then bottom right, and finally bottom left.
if (vUv.x < 0.5 && vUv.y >= 0.5) {
    float roundedDist = length( max( abs( planeSpaceCoord ) - u_size * 0.5 + u_borderRadiusTopLeft, 0.0 ) ) - u_borderRadiusTopLeft;
    float s = step( innerRadDist * -1.0, u_borderRadiusTopLeft );
    return mix( innerRadDist, roundedDist, s );
}
if (vUv.x >= 0.5 && vUv.y >= 0.5) {
    float roundedDist = length( max( abs( planeSpaceCoord ) - u_size * 0.5 + u_borderRadiusTopRight, 0.0 ) ) - u_borderRadiusTopRight;
    float s = step( innerRadDist * -1.0, u_borderRadiusTopRight );
    return mix( innerRadDist, roundedDist, s );
}
if (vUv.x >= 0.5 && vUv.y < 0.5) {
    float roundedDist = length( max( abs( planeSpaceCoord ) - u_size * 0.5 + u_borderRadiusBottomRight, 0.0 ) ) - u_borderRadiusBottomRight;
    float s = step( innerRadDist * -1.0, u_borderRadiusBottomRight );
    return mix( innerRadDist, roundedDist, s );
}
if (vUv.x < 0.5 && vUv.y < 0.5) {
    float roundedDist = length( max( abs( planeSpaceCoord ) - u_size * 0.5 + u_borderRadiusBottomLeft, 0.0 ) ) - u_borderRadiusBottomLeft;
    float s = step( innerRadDist * -1.0, u_borderRadiusBottomLeft );
    return mix( innerRadDist, roundedDist, s );
}

Is the main issue that having a border radius greater than the width or height of the plane causes it to break or look weird?

Hey @Enijar thanks for that prompt answer.
I realize that there is two questions in my post, and therefore Im going to split them.
If you could help me on "BorderWidth Sides Top-Right-Bottom-Left" that would be great !
If the second question is too vague, or took you too much time to help me, that not that bad.
BorderWidth Sides - Top Right Bottom Left
With the current shader code, I think I could go further on this by myself, but I lack the understanding of some parts of the shader. Here is a picture to schematize one dial (one plane, divided in 4 dials)

I just can't understand how the red areas border are found.
The blue one, are circular distance.
I think I would be able to split borderWidth in four dimensions If I understand that part.
How would you do such a thing?
Border Radiuses ( second question, less important )

Is the main issue that having a border radius greater than the width or height of the plane causes it to break or look weird?

It is, and it doesn't even have to be bigger than the plane, but bigger than the dial.
In the picture below, the glitch comes from the fact that the radius have to be contained in a bigger dial than [0,1] - [0.5,0.5]

This is something I somehow manage to succeed ( minus the smoothstep )

It defines 9 dials instead of 4, computed after a "mediation" between required radiuses

Currently it could lead to some overlaps, which could also become an issue.
But here again, I cannot conceive a way to found borders for red areas.

I'm have the latest version of the 7.x.x-feature/refine-frame-borders branch open now. Where are you testing the UI, is it on the .border- section here?

Ignore my last comment, I see it's here

I might have chosen a wrong way. Dont feel bad to say it.

It's cool, I fully understand what you're trying to achieve now. I think it's going to be hard to implement but will be a very good feature. I will have a play and post my feedback here

Many Thanks!
Take your time, there is no hurry from my side.
As stated before the branch contains both border dimensions and radius dial expansion. Tasks could be isolated I needed.

I'm working on an SDF version of the fragment shader for more control over the plane border segments:
https://codesandbox.io/s/silent-pond-cpmnn8?file=/src/shader/fragment.glsl

@swingingtom below is a WIP fragment shader which will eventually split the plane (block) into grid segments (currently 2x2). Can you read the comments I've left and let me know if this is heading in the direction of what you want. This needs a lot more work to get to where you eventually want to be, but just want to check I'm heading in the right direction.
Reference Image

WIP Implementation
Full CSB version
varying vec2 vUv;

float roundedSDF(vec2 uv, vec2 center, float radius) {
  return length(max(abs(uv - center) - center + radius, 0.0)) - radius;
}

float roundedSDF(vec2 uv, vec2 center, vec4 radii) {
  // Top left quadrant
  if (uv.x <= 0.5 && uv.y >= 0.5) {
    return roundedSDF(uv, center, radii.x);
  }
  // Top right quadrant
  if (uv.x >= 0.5 && uv.y >= 0.5) {
    return roundedSDF(uv, center, radii.y);
  }
  // Bottom right quadrant
  if (uv.x >= 0.5 && uv.y <= 0.5) {
    return roundedSDF(uv, center, radii.z);
  }
  // Bottom left quadrant
  if (uv.x <= 0.5 && uv.y <= 0.5) {
    return roundedSDF(uv, center, radii.w);
  }
  return roundedSDF(uv, center, radii.x);
}

// References:
// 1. https://stackoverflow.com/a/68233743/2413722

void main() {
  vec2 center = vec2(0.5, 0.5);

  vec3 backgroundColor = vec3(0.5, 0.5, 1.0);

  // Border colors
  vec3 borderColorTopLeft = vec3(1.0, 0.0, 0.0);
  vec3 borderColorTopRight = vec3(0.0, 1.0, 0.0);
  vec3 borderColorBottomRight = vec3(0.0, 0.0, 1.0);
  vec3 borderColorBottomLeft = vec3(0.5, 0.5, 0.5);

  // Border thickness
  vec4 thickness = vec4(0.05, 0.05, 0.05, 0.05);

  // Radius of the corners
  vec4 radii = vec4(0.5, 0.1, 0.5, 0.1);

  // Calculate distance to edge.
  float distance = roundedSDF(vUv, center, radii);

  // Smooth the result (antialiasing)
  // @todo Add smoothstep min and max values for antialiasing
  float alpha = 1.0 - smoothstep(0.0, 0.0, distance);

  // Return the resultant shape
  vec4 color = vec4(backgroundColor, alpha);

  // Corner quadrant checks
  // @todo Make arbitary grid size instead of the current 2x2 (4 corners)
  bool topLeft = vUv.x <= 0.5 && vUv.y >= 0.5;
  bool topRight = vUv.x >= 0.5 && vUv.y >= 0.5;
  bool bottomRight = vUv.x >= 0.5 && vUv.y <= 0.5;
  bool bottomLeft = vUv.x <= 0.5 && vUv.y <= 0.5;

  // Draw topLeft corner
  if (topLeft && distance > -thickness.x) {
    color = vec4(borderColorTopLeft, alpha);
  }

  // Draw topRight corner
  if (topRight && distance > -thickness.y) {
    color = vec4(borderColorTopRight, alpha);
  }

  // Draw bottomRight corner
  if (bottomRight && distance > -thickness.z) {
    color = vec4(borderColorBottomRight, alpha);
  }

  // Draw bottomLeft corner
  if (bottomLeft && distance > -thickness.w) {
    color = vec4(borderColorBottomLeft, alpha);
  }

  // Discard anything with 0.0 alpha so it becomes transparent
  if (alpha == 0.0) discard;

  // If no corner color was assigned, this defaults to backgroundColor
  gl_FragColor = color;
}

Thanks @Enijar, that looks really nice and clean. And it's pretty amazing how the code readable is 👍
That's make me think, in order to have top-right-bottom-left width and color, instead of wip topleft-topright-bottomright-bottomleft ones, borders drawing may be in two passes? This would lead to have quadrants for alpha computing, then for two passes for border color and thickness?
Theorically
  // a quadrant is a sum of two sides
  if (top && distance > -thickness.x) {
    color = borderColorTop;
  }
  if (left && distance > -thickness.w) {
    color = borderColorLeft;
  }

instead of
// Draw topLeft corner
  if (topLeft && distance > -thickness.x) {
    color = borderColorTopLeft;
  }

Maybe you already think about it, or already discarded that idea?
Anyways, that a nice wip, and Im pretty sure that 7.x.x FrameShader would enjoy to host that code.

The idea is to eventually have a 3x3 grid instead of the current 2x2 grid. The 3x3 grid would have the following border segments:

Top left
Top middle
Top right
Middle right
Bottom right
Bottom middle
Bottom left
Middle left

I will play with this some more tonight and add support for background and border opacity, then share the updated shader 👍

That would be amazing, thanks again for the help!

Done with #206
@Enijar thanks for the help, you are still more than welcome to optimize shader or corner mediation


Finetuning multitask model on a new task

I am trying to train TD-MPC2 on a new custom environment. It is unclear to me how to best make use of the pretrained multitask models you provide. In the paper you describe finetuning by initializing the task embedding for the novel task and then training the multitask model on the novel task. How can this be done in practice with the code provided here? Loading a multitask checkpoint into a model configured for a single task does not work as the state_dicts are not compatible.
Thanks a lot! I greatly appreciate making this model and the checkpoints publicly available!

Great question! The only difference between the multi-task model and the default single-task model architecture is the additional learnable task embedding that all components are conditioned on. Our released codebase is not quite set up for finetuning experiments out of the box, but I believe you should be able to init the model by forcing cfg.multitask to be true here
https://github.com/nicklashansen/tdmpc2/blob/df8a465c8e137c652a142f6ad6cdf540d3a6a39a/tdmpc2/common/parser.py#L70-L71
We didn't do anything overly complicated when finetuning. We simply picked a good init (i.e., a similar task) for the task embedding and finetuned the whole model end-to-end with online RL and default hparams. This is likely not the best way to do it, but it worked well enough for our purposes.
I hope this helps!

Is there any way I can avoid having to first create the complete multitask environment (such as mt80) in order to initialize all the hyperparameters such as action_dim, obs_dim and number of task embeddings required to load the model checkpoint?

You could directly set these values instead of inferring them from the environment.

how do I deal with my custom environment having different observation and action spaces

We zero-pad observations and actions to the largest dims used during training, iirc it's 39 and 6 dims, respectively, but you might want to double-check that.

Thank you for helping out! I managed to make finetuning work.

Great that you made it work! I'll close this issue but feel free to reopen if you have any follow-up questions.


Create accurate error code for users who are applying a code twice to their account

Overview
We want to provide a more accurate error message for users who try and apply a code to their account twice. To do this, we need a better error message than ALREADY_IN_USE. We want a new error code ALREADY_APPLIED
Action Items

[ ] Pull develop branch to ensure your repo is up to date
[ ] Create your own branch
[ ] See resource and tips below before starting.
[ ] Create the change to service methods.
[ ] Add a new service test to test the scenario where a user is applying a code again.
[ ] Create PR and tag @eleanorreem to the PR

Resources/Instructions

The endpoint users hit is POST /partner-access/assign see /bloom-backend/src/partner-access/partner-access.controller.ts.
The endpoint uses the assignPartnerAccess service method which checks the validity of the code with the getValidPartnerAccessCode service method.
Initial suggestion is to pass an optional userId into getValidPartnerAccessCode. You can then compare the users userId with the  access codes userId.
Please note this is a suggestion after having a quick look at the code and might not be the best way. Feel free to improve the implementation

@mayank-sahai thank you for looking into this!
I've assigned you the issue and pinged the devs to answer your question ASAP.

Hey @mayank-sahai, super sorry for the slow response. Thanks so much for picking this up. I didn't see the notification for some reason! And I can see my issue was not clear enough. I will clarify.
We have partners who have custom bloom experiences. In some case, this includes enhanced features such as therapy. A user can get the custom experience in multiple ways.

A user is given a code by a Partner Admin user (i.e. an administrator working for one of our partners). The user is given an access code when they sign up and this is assigned to them when they sign up.
The user is given a link to a page i.e. /welcome/[partnerName] where they can sign up without an access code. When they sign up, they are assigned an access code behind the scenes which has partnerAdminId set to null. I.e. they have been given permissions for the partner experience but this isn't related to a particular administrator.
When a user has already signed up, they can apply a code on the frontend endpoint /account/apply-a-code. This is the instance where a user is most likely to come across this error. They have applied the code to their account and they try and do it again.

Hope that context is helpful.
To respond to your exact questions:

can the access code be applied multiple times by different users? (but once per user?) - No. An access code is individual per user. It cannot be assigned to multiple users.
I can see getValidPartnerAccessCode is being used in user.service.ts > createPartnerUserWithCode so what should be the expectation there? allow creating different partner user with same code? I think looking at this question, i've realised that the suggestion that I put forward in the ticket isn't quite right. I am making getValidPartnerAccessCode do too much. I think that function should just be retrieving access codes based on the code supplied.

My new suggestion would be to just put the check in PartnerAccessService.assignPartnerAccess. That should be the only place that that is relevant. Let me know if that makes sense and sorry for the long ramble. ⭐

No worries and thanks for the explanation. @eleanorreem
So just to confirm my understanding we have the following cases -

Access code userId is not null and not equal to user userId
Response message - ALREADY_IN_USE
Frontend Message - The code has already been used. If you think there's been a mistake, get in touch with the partner team.

Access codes userId is not null and equal to user userId
Response message - ALREADY_APPLIED
Frontend Message - This code has already been applied to your account.

Access codes userId is null  - Apply the code like it is happening currently.

If this is correct then I would say that your first suggestion was right and we should keep the condition in the getValidPartnerAccessCode only and pass userId as optional for ALREADY_APPLIED scenario. This will keep all the condition check at one place and make the getValidPartnerAccessCode complete.
let me know what you think.
cheers.

@mayank-sahai That all sounds correct to me! Thanks so much ⭐


Feature in step 3 of local set up in vs code not found in new version

There is no option for instantiating contact in the version of vs code I am using, instead it gives option of "+deploy my contract"
How would you instantiate in newer version?

This is the image I'm getting instead of the one in the guide

hey @hassaan253 were you able to run this project on the newer version?


Using unpriviliged user to run Neo4j

Just a proposal for running neo4j with a non-root user.
Even in a container, I think running applications with root user is not secure.

Hello @jri-sp. Thanks for the proposal and the code.
We are unhappy with running as root as well and plan to change it at some point. However I'm afraid it's not as simple as your commit makes it look.
The trouble comes when the /data volume is mounted on a directory in the host system that has restrictive permissions (as it is likely to do). In that case we need to chown the volume at container run time (i.e. in docker-entrypoint.sh) rather than build time (i.e. in Dockerfile).
Ordinarily we would use sudo for this, but the minimal Debian base that we use doesn't include sudo -- for good reason because it pulls in a lot of stuff and increases the attack surface significantly.
The standard approved Docker approach for this is exemplified by the solution in the Redis image (here which uses a minimal sudo-analogue written by one of the Docker maintainers. (I'm not wild about this approach, but it seems to be the only way.)
In order to solve this properly we also need to set a value for ulimit -n which Neo4j needs to be a high value (>40k). Currently this is okay because we are running as root, but I suspect that it will need something explicit if we have a neo4j user.
If you want to tackle this then I'm very happy to give you feedback on a more complete pull request. Otherwise, I'm afraid that we don't have a schedule for doing it ourselves. It's unlikely to be in Q1 2016.
-Ben

Hello @benbc ,
Thanks for the quick and consisting answer!
I'm glad that we share the same idea about using root user :)
I know... my solution is minimalist, my pull request was more to open a discussion than accept it.
But maybe I can find time to work more on it, because I need it, so if I can help...
Thanks for the redis example, I didn't know this way to do, I'll study it!
For the /data folder, yes in my  configuration I'm using one and it's not working with my thing. I have to manualy chown my folder on the host side with uid/gid of the container, which is not a very user-friendly and beautiful solution...
However, for the ulimit, I've run my Docker image with --ulimit=nofile=40000:40000 and there is no warning in the logs. If I remove this parameter I have:
WARNING: Max 1024 open files allowed, minimum of 40 000 recommended. See the Neo4j manual.

So I was thinking it works,  but I'll investigate.
You are more experienced than me in Docker, so thanks for all those advices!
Regards,
JRI

@jri-sp Yes, I'm sure that --ulimit=nofile works correctly. But we'd rather not make the image harder to use by requiring that people add that parameter. So it needs something in the Dockerfile (or possibly entrypoint-docker.sh) that sets the value appropriately.

Hi @benbc ,
I you have a few time to watch my thing...
I have done a new commit taking example on redis and mongodb Docker images using gosu.
Is it better for you ?
There something you see that I didn't think ?
I don't touch the "NEO4J_AUTH" part because it's called temporary and I was thinking it's not a big deal if it's runned as root. No ?
So this version change the data volume's permissions mounted with -v option on the host system, which is more simple to use.
Best Regards,
jri

Hello @jri-sp. Thanks so much for doing this work.
I've put a few comments inline, but they are mostly stylistic and basically I think the solution is good.
Can you please remove the first, obsolete commit from the PR?
I think we do need to run the temporary Neo4j instance for the auth stuff as neo4j, for completeness and to help avoid problems with root owning files that neo4j needs to touch. That will mean moving the chown for /data up to the beginning of the block.

I do think we need to address the ulimit nofile thing.

Hello @benbc ,
Sorry for the closing/opening messages, actually I'm not familiar with Github and Pull Requests, so I probably made some mistakes and I've cleaned my first obsolete commit.
Thanks a lot for taking time to answer me!
I've applied all your comments and moved chown for /data to the top of Docker entry script to use gosu in the NEO4J_AUTH part.
Now, the interesting commit is c7a4496f8767c53a0f00e5542a46dd755163058c
For the ulimit, I need to do some tests to suggest something. I just know that is no possible to set that in the Dockerfile.
Thanks again for your help!
Best Regards,
JRI

I've just done some tests for the ulimit. I don't have good news... :(

We can set it in Dockerfile
If we set it in docker-entry.sh, we got an "Operation not permitted"  error
ulimit: open files: cannot modify limit: Operation not permitted
It seems to be a docker security "feature": https://groups.google.com/forum/#!searchin/docker-user/limits/docker-user/T45Kc9vD804/v8J_N4gLbacJ

There is two solution:
Second method: stop the docker demon. Use ulimit to increase the limit. Start the docker demon. It now has increased limits, and its child processes as well.```

By the way, I think setting it in the `docker run` command is the best solution, because:
- it's work.
- You can't ask to user to recompile their docker.
- Modifying limits on the host (second method), will apply the limit to all containers, which is not necessarily wanted and can cause issues.

I will continue my researches about this but I'm afraid to find nothing great...

@jri-sp This all looks great. Thanks for your patience with my picky comments. We need to do some research of our own about the ulimit nofiles thing before we can merge it.

@benbc  You're welcome. Let me know what you'll find about ulimit, as I said I'm beginning in the Docker world, maybe I've missed a info :)

Hi @benbc ,
I don't know if this PR is still relevant with Docker 1.10 and user namespaces...
http://www.jrslv.com/docker-1-10/
Regards

Thanks for pointing that out. And thanks for your patience. I'm afraid this has got stuck behind other work at the moment.

Closing as this has now been resolved by https://github.com/neo4j/docker-neo4j/pull/126


pem file error

Hi I get: The key is not a file or a string PEM format private key
but the pem file have in headers:
-----BEGIN RSA PRIVATE KEY-----
What could be the problem?

How are you writing the path to the file? Or.. are you  passing the file contents? Can you give us a code example please?

That was the problem it was that the path was wrong without "file:/"
it's fixed now.

Nice! Let me close this issue after add some documentation about this missuse, I think is the second time it happened.


Updating the annotation tool

What does this PR change?
This PR adds new updated functionality to the OpenCV annotation tool

The complete pipeline has been vectorized so that modifications are easier then directly converting the rectangles to string format as what was being done in the past.

[Feature Request] We enabled early quitting and storage option: in the past, pressing the ESC key earlier in the annotation process closed down the annotation tool, but left you with no annotation file of the work done being present. We adapted the storage of the results, so that at any time during the process, when you decide to leave using the ESC button, the results to that point are being stored.

[Feature Request] We added a delete option. Since the annotations are stored inside a vectorized structure, deleting the last added annotation is fairly simple now. This allows users to correct wrongful annotations, without needing to restart the application.

A video of the functionality can be seen below
https://youtu.be/EV5gmvoCTSk

Once this is error free and merged, I will supply the same update to the 2.4 branch.

:+1:

Thank you again!


sprite setOrigin doesn't work on animation frames

Phaser v3.9.0 & Phaser v3.10.0-beta1
setOrigin method doesn't work on animations
jsFiddle

Want to back this issue? Post a bounty on it! We accept bounties via Bountysource.

The json you're using to test this has a custom anchor point defined in it, which overrides the setOrigin:
				{
					"filename": "diamond_0009",
					"frame": {
						"x": 744,
						"y": 2,
						"w": 37,
						"h": 60
					},
					"rotated": false,
					"trimmed": true,
					"spriteSourceSize": {
						"x": 14,
						"y": 2,
						"w": 37,
						"h": 60
					},
					"sourceSize": {
						"w": 64,
						"h": 64
					},
					"anchor": {
						"x": 0.5,
						"y": 1
					}
				}

So this is behaving as I'd expect. If you want to specify the origin yourself, the frame data should have no anchor property.

thanks for answer)


Remove tiny color as a requirement

Too much is using this. Could only be possibly if the format option was removed


tool to truncate mlperf_log_accuracy.json

Tool to truncate mlperf_log_accuracy.json.
You can run this tool in 2 ways:

pick an existing submission directory and create a brand new submission tree with the trucated
mlperf_log_accuracy.json files. The original submission directory is not modified.

python tools/submission/truncate_accuracy_log.py --input ORIGINAL_SUBMISSION_DIRECTORY --submitter MY_ORG \
        --output NEW_SUBMISSION_DIRECTORY

pick a existing submission directory and a backup location for files that are going to be modified.
The tool will copy files that are modified into the backup directory and than modify the existing
submission directory.

    python tools/submission/truncate_accuracy_log.py --input ROOT_OF_SUBMISSION_DIRECTORY --submitter MY_ORG \
        --backup MY_SUPER_SAFE_STORAGE

tools/submission/submission-checker.py will enforce the hash and the truncated mlperf_log_accuracy.json
BE CAREFUL WITH THIS TOOL since it it will replace your original mlperf_log_accuracy.json.
https://github.com/mlperf/inference/issues/671
https://github.com/mlperf/inference/issues/681

Maybe add warning message or prompt when option 1 is chosen, since the original logs still need to be saved somewhere in case WG requests them for review?

Sure need to add it to the inference rules. Let me check on some warning text.

@christ1ne , this is ready to be merged.

please open an PR to add to submission rules about running this script


sgf-viewer: Update main games, add games vs. cp505-v10mil, add transfer games

Changes:

Update cyclic-adversary games to use s545mil adversary instead of s497mil adversary, except for the human vs. cyclic-adversary games
Add games against cp505-v10mil

All these games end with resignation, so I had to make some game parsing edits to make the website display that the games ended via resignation

Add games against Leela/ELF

someone should also look at the changes to content.ts to make sure the wording looks sane


Virtual Heightfield Mesh as possible replacement for displacement

Someone mentioned on unreal sensei's channel that this might be what is going to replace displacement in ue5, not much info available but something interesting to consider.  See https://www.youtube.com/watch?v=3Zl1P96oYls

Yes, I saw that too. That was also speculated on the UE forums as well. We should try that out later on.


chore(deps): update ci dependencies

This PR contains the following updates:

Package
Type
Update
Change

common-dev-assets

digest
8d95c46 -> 9b8a0b5

github.com/terraform-ibm-modules/ibmcloud-terratest-wrapper
require
patch
v1.30.1 -> v1.30.2

Release Notes

terraform-ibm-modules/ibmcloud-terratest-wrapper (github.com/terraform-ibm-modules/ibmcloud-terratest-wrapper)
v1.30.2
Compare Source
Bug Fixes

deps: update gomod (#785) (e215d84)

Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
👻 Immortal: This PR will be recreated if closed unmerged. Get config help if that's undesired.

[ ] If you want to rebase/retry this PR, check this box

This PR has been generated by Renovate Bot.

/run pipeline

/run pipeline

/run pipeline

:tada: This PR is included in version 1.5.1 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Resources are public in the library

Issue description
All the android resources from the library are public.
To Reproduce
Add the library, build and type R.string. or any other resources and you will see that resources of the library are public.
Expected behaviour
To see only resources that are relevant for the projects implementing the library.
Library version
4.39.0
Extra info
https://developer.android.com/studio/projects/android-library#PrivateResources

Hi @franvis, thanks to report this
We are aware of this problem and we are going to solve this but not in the short term
To have this resouces public is bringing us problems as well because some integrators could use them and that doesn't let us remove them whitout producing small breaking changes
We are working in some tasks to remove a large amount of resources and the next steps are to going to make them private


update fails

I've just tried "solo2 update" before reporting anything else (from firmware 1:20200101.8) with the cli v0.1.1 on Debian 11, with no luck.  When I tap (more like prod) at the prompt, the key goes dead and needs to be unplugged.  The kernel log has multiple disconnect messages.  I don't know if it's necessary, but I had reloaded udev with the given rules; I don't know when they are actually needed, but probably the readme here should at least mention installing them.  Anyway, I see this before it dies:
$ SOLO2_LOG=info solo2 update
Downloading latest release from https://github.com/solokeys/solo2/
Fetched firmware version 1:20200101.9
 INFO  solo2::transport > using CTAP as minimal transport
 INFO  solo2::device    > device fw version: 1:20200101.8
 INFO  solo2::device    > new fw version: 1:20200101.9
Tap button on key to confirm, or replug to abort...
 INFO  solo2::transport > using CTAP as minimal transport

The prompt says "replug to abort", but the command just hangs pending ^C.  Also it says "tap the button", but there isn't anything most people would call a button, and it's not clear whether that means any of the edges or a specific one.

For what it's worth, I tried the update from a Fedora 35 live image, and it worked.  I can't tell what the relevant difference is between the two, though they were on different laptops (F35 on Dell XPS 9630, Debian 11 on Thinkpad T460 I didn't want to reboot for that).  Using a backported pcscd 1.9.5 from Debian unstable (more recent than Fedora's), along with a fixed libccid to use PCSC, didn't help.
I hadn't seen relevant syslog messages originally.  These are from the failed attempt, but I don't know what to make of them:
pcscd[1950009]: 01000118 winscard.c:1618:SCardTransmit() Card not transacted: 0x80100017
pcscd[1950009]: 00000026 eventhandler.c:336:EHStatusHandlerThread() Error communicating to: SoloKeys Solo 2 [CCID/ICCD Interface] (3E0EA0575ACCC859B74D69D262679BEF) 00 00
pcscd[1950009]: 00400245 ccid_usb.c:858:WriteUSB() write failed (1/90): -4 LIBUSB_ERROR_NO_DEVICE

Can someone suggest how to debug this?

The difference is that Fedora has a more recent CCID library, as mentioned in one of their updates. In order to update firmware on older Linux distros, add the udev rules to /etc/udev/rules.d and re-plug.

Thanks, but unfortunately not.  As I understand it's supposed not to
need pcscd, but I used the 1.5.0 backport I did (linked through the
wontfix Debian issue), which is the latest release.  Anyway, as far as I
can tell, the only relevant part of that is the definition in
Info.plist.  (It would at least be useful to document that simple
solution -- if there was any sign of support for the keys, sigh.)

Sorry I wasn't clear enough, I think libccid and the udev rules are independent solutions. For what it's worth, I ran into the same symptoms as you (the CLI asking for the key to confirm) but was able to update once I added the udev rules. I'm on Ubuntu 20.04.

Thanks for making me double check this.
It turns out that my udev rules were missing the "ROM bootloader
(unmodified)" line.  I don't remember how I got the rules originally on
the Debian system -- apparently not from the right place, or maybe they
were bundled somewhere and I just picked the ones labelled Solo 2.
Anyhow, it works now, thanks again.


Alternative caching options

Are there alternatives to Redis (Azure Cache for Redis)? If yes, what are they and their pros & cons?

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

ID: be785717-ca5e-f3d6-8e60-4005d9503d08
Version Independent ID: 3ce414e5-d923-0488-ff11-3aa8bd4c2d1e
Content: Caching guidance - Azure Architecture Center
Content Source: docs/best-practices/caching.yml
Service: architecture-center
Sub-service: azure-guide
GitHub Login: @EdPrice-MSFT
Microsoft Alias: edprice

Hi @kmathewmk
Sorry for the delay.
Using c# you can configure several distributed cache, https://learn.microsoft.com/aspnet/core/performance/caching/distributed#establish-distributed-caching-services.
It includes options like SQL Database as cache.
Here the recommendations: https://learn.microsoft.com/aspnet/core/performance/caching/distributed#recommendations
I will close the ticket, but feel free to re-open if additional clarification is needed.

#please-close

Thanks for the useful pointer. I think that alternatives should be mentioned or linked in the Caching guidance - Azure Architecture Center page. Having the information here in the git issue does not expose the info to all users. Also, are there caching alternatives suited for cloud/AKS services running of multiple compute instances? For eg we tried Varnish, Nginx for some of our needs.

Thanks for the suggestion, I will create a ticket for content development writers.
Even though you are looking for cache options, the best way to cache applications data on Azure is using Redis. It is cloud and as a service. The service manages the high availability. Also, you could use SQL Server as a service and use it as cache.
From AKS, you can use any azure service.
Varnish is a different kind of cache. Here how to use it on Azure
As far as you can run it on docker you can use it inside AKS.  The same case for nginx.