try to fix issue#741

Sorry for the ugly English.
I tried to fix the issue #741 .
This issue can happen frequently, so consider merging.
Cause
There are two cause for this issue:

In changeCurrentIndexProgressive in didSelectItemAt, OldCell is nil, because OldCell is off the screen.

cellForItemAt is called when an OldCell outside the screen enters the screen with moveTo.
However, the currentIndex has not been updated, the OldCell is determined to be a NewCell.

Amendment policy
The fix was applied only inside the buttonBar related sources. Because of the following two reasons

We also considered modifying PagerTabStripViewController, but the scope of influence is too wide.

changeCurrentIndexProgressive exists only in buttonBar.

I'm sorry.
I found an omission, so please wait for a review.


After using snowflake_tag_association to set tag on account, subsequent terraform plan/apply fail with "error listing tag associations".

Provider Version
0.64.0
Terraform Version
1.3.9
Describe the bug
The first attempt to set tags on the account using snowflake_tag_association succeeds. However, afterwards any attempt to run terraform plan or apply fail with error listing tag associations, because the GET_TAG function is invoked using account name (e.g  SANDPIT01_AWS_EW1_DEV instead of account locator (e.g. KP44023).
Example of failing SQL query:
SELECT SYSTEM$GET_TAG('"LIBRARY_FEATURE_PAALDEV"."TAGS"."ACCOUNT_TYPE"', '"SANDPIT01_AWS_EW1_DEV"', 'ACCOUNT') TAG_VALUE WHERE TAG_VALUE IS NOT NULL;

The above query succeeds if SANDPIT01_AWS_EW1_DEV is replaced with KP44023.
It is not an option to use account locator in object_identifier, since the SQL generated to set the tag explicitly mentions the account:
ALTER ACCOUNT "SANDPIT01_AWS_EW1_DEV" SET TAG "LIBRARY_FEATURE_PAALDEV"."TAGS"."ACCOUNT_GROUP" = 'SANDPIT01';

(The Snowflake documentation specifies a syntax without account name for 'ALTER ACCOUNT SET TAG', and even though it works
with account name, the same query fails when replacing account name with account locator.)
Expected behavior
When listing tag associations on the account (during plan or apply), the provider should use a query that correctly retrieves the tag.
Code samples and commands
Please add code examples and commands that were run to cause the problem.
Additional context
Add any other context about the problem here.

@pstDnb thank you for reporting this issue. We will look into it and get back to you.

We have same error using snowflake_tag_association to set tag on VIEW. It's working fine on TABLE.

Hey @jia-ying-lin, @pstDnb. We will address this issue as part of https://github.com/Snowflake-Labs/terraform-provider-snowflake/blob/main/ROADMAP.md#preparing-essential-ga-objects-for-the-provider-v1.

Hi @pstDnb  👋
We've just released v0.100.0 (release, migration guide) in which we reworked tag associations. Tags can be assigned to accounts, like
resource "snowflake_tag_association" "account_association" {
  object_identifiers = ["\"ORGANIZATION_NAME\".\"ACCOUNT_NAME\""]
  object_type        = "ACCOUNT"
  tag_id             = snowflake_tag.test.fully_qualified_name
  tag_value          = "engineering"
}

Please upgrade to this version and let us know if you have any issues.


Fix Issue #4056 - Prevent cargo order quantities larger than the maximum

About the PR
Should fix issue #4056
Cargo console now plays an error sound when the order could not be added.
Adds a counter for the unapproved orders:

Changelog

:cl: TheDracheX

fix: Fixed cargo points overflowing with large order quantities
tweak: The cargo console now limits the amount of orders (default 20). An error sound will play if a crate could not be ordered.

I decided to add two UI/UX changes in the cargo order console for this change so players will see how many orders they have left. A sound will play if a order could not be added.

If you're interested in fixing some more small bugs with cargo I compiled a thread here. #2717.

The check should be in ApproveOrder unless I coded the console weird where it doesn't go through approve.

Git moment


Archi 4.9 and DataBase export plugin

Version of Archi
4.9
Archi Plug-ins
org.archicontribs.database_2.2.19
org.archicontribs.modelrepository_<IP_ADDRESS>102021056
Operating System
Windows 11
Expected Behaviour
Test connection ok. Save and start exporting
Actual Behaviour
Database export plugging giver error while initializing. Test Connection is ok when closing windows give null reference
Steps to Reproduce the Behaviour

Choose an existing connection (eg: neo4j)
Save
Select connection and stat exporting

The database plugin is not maintained by us (Phil and me) but by Herve. To my knowledge, Herve's plugins are still not updated to work with Archi 4.9, but you can open an issue on its plugin page: https://github.com/archi-contribs/database-plugin/issues


Undeclared dependency on charset normalizer?

Hi, I use normality for my project and the tests in the Dependabot pull request failed.  Does the dependency on charset_normalizer need to be added to a setup.py?

Thanks.  Will there be a release with this change soon?

Yes sorry I hotfixed this after half a bottle of wine so no commentary, but there's 2.3.1 on PyPI now.

Thanks.  I guess Dependabot will pick it up eventually.  I hadn't seen anything, so I was wondering.

FWIW, Dependabot has caught up.  Thanks for the quick fix.


feat: Update from default-net to rebranded netdev

Description

Upgrades from default-net v0.20 to netdev v0.25, which is simply a rebrand of the original default-net
Allows depending on system-configuration version 0.6 instead of 0.5.1 downstream. This may help iOS compliation.

Breaking Changes
Would need to check if/how much the default-net types were exposed in the API.
Notes & open questions
Not sure yet if it fixes our iOS problems.
Change checklist

[x] Self-review.
[x] Documentation updates if relevant. (no mention of default_net in docs)
[ ] Tests if relevant.
[ ] All breaking changes documented.

Changed items in the public API
===============================
-pub iroh_net::net::interfaces::IpNet::V4(netdev::ip::Ipv4Net)
+pub iroh_net::net::interfaces::IpNet::V4(default_net::ip::Ipv4Net)
-pub iroh_net::net::interfaces::IpNet::V6(netdev::ip::Ipv6Net)
+pub iroh_net::net::interfaces::IpNet::V6(default_net::ip::Ipv6Net)

interfaces should probably not even be exposed as part of the public api, but in any case, here is what changed

This fixes our iOS build together with the instructions from here: https://iroh.computer/docs/examples/ios-starter#add-the-system-configuration-framework

It seems like it's been resolved and was released in version 0.16.
iroh-net already depends on version 0.20 at the moment, so should the macOS logic be adjusted, or is the original issue not actually fixed?

yeah, I think this just never got rechecked after that fix was done

I'd be happy to contribute a change that removes the logic that's possibly subsumed by the upstream fix, but don't have a macOS device to test on :confused:


Contact form validation

I have implemented the contact form validation for emails
I created a container under the submit button in the index.html of the contact from

I styled the container for text color, border color appropriately.
I referenced the element Ids, for the email and form in a form-validaton.js
I created a listening function and a email checker function
The functions check for uppercase in email and returns an error message.
If the email has no uppercase the form submits, else it does not.

I tested it locally and it runs well
I also checked for html, css and js linter errors and all passed.

Thank you for the good review.


superagent support IE6+?

this page say that superagent is now support IE6+，is it true?

Please use our documentation and not some random website: https://github.com/visionmedia/superagent#supported-browsers-and-node-versions


Support postman Oauth 1.0 and Oauth 2.0 config

This PR adds support for importing as much configuration as we can
from Postman v2.0 and v2.1 exports with Oauth 1.0 and 2.0
authentication.
Note: For Oauth2.0 this means setting authorization to Oauth2 but
generating a disabled config, since postman does not export anything we
can use to fill our config values.
Fixes #1240

Closing to combine with my other auth PRs


seems  to create an infinite loop?

Challenge http://www.freecodecamp.com/challenges/waypoint-understand-boolean-values has an issue. Please describe how to reproduce it, and include links to screenshots if possible.

user notices only 2 challenges and the first challenge is complete
user copies and pasts the second challenge as:

assert(welcomeToBooleans() === true
to the code and then adding ); at the end. after clicking run the code it seemed to create more of the same challenge.

user comments out line 5 "return false". after clicking run the code user receives error in 2nd pic.

This is a recursive function
function foo() {
  foo();
}

It will call itself infinitely until the browser runs out of room in the call stack. This is what your code is doing.

Your code is also incorrect. Remove the self calling and you should be fine.
Happy Coding!


improve scripts/ develop github actons for updating user_examples and test_models

Currently, there are R scripts for updating the models in user-examples and test-models upon each release. These are run locally, but the scripts require some manual changes and take local computing power to run. I feel it takes me longer than it should to run this because I have to refamiliarize myself with the script each time.
Another issue to updating is not well documented, because the script requires some manual changes and doesn't provide good instructions on how to run it. This is not helpful if anyone besides me needs to do the updating.
It may be nicer to fully automate these in github actions. The steps would be:

Download the latest SS3 executable
Run a streamlined version of the script that doesn't require manual input changes. This would update model input and output files.
Delete any files that shouldn't be committed
Open a pull request with the changes to the repository for review.

A tag would also need to be added once it is merged in, but this step may need to be left as manual.

Another reason to fully automate updating the test results on each release are that the compiler/os is then different, which can lead to some small differences in model results. It would be cleaner to run the model updates on ubuntu since that is the system used to run the stock synthesis regression tests.

These scripts are linked in https://github.com/nmfs-stock-synthesis/stock-synthesis/issues/352#issue-1308326395

There are now github actions for updating these the user-example models and the test models in the respective repositories.
user-example models github action
test models github action


[feature] Add transaction history page

This could be a third page between wallet and settings. Other than that that's a pretty straightforward page to do. The necessary data to construct it is already included in the wallet data that we keep in storage.

closed by #25


[1.20.1] Crash with Tech Reborn and REI

Minecraft Version
1.20.1
Fabric Version
0.89.3+1.20.1
Hephaestus Version
1.20.1-<IP_ADDRESS>
Describe your issue
REI crashes upon trying to render (supposedly) Tech Reborn cells filled with Hephaestus potions.
This crash requires Hephaestus, Tech Reborn and REI to be installed.
Crash Report
https://pastebin.com/qAz4WK3Q
Other mods

architectury-9.1.12-fabric
cloth-config-11.1.106-fabric
fabric-api-0.89.3+1.20.1
fabric-language-kotlin-1.10.10+kotlin.1.9.10
Hephaestus-1.20.1-<IP_ADDRESS>
indium-1.0.27+mc1.20.1
NoChatReports-FABRIC-1.20.1-v2.2.2
RebornCore-5.8.7
RoughlyEnoughItems-12.0.665
sodium-fabric-mc1.20.1-0.5.3
TechReborn-5.8.7

Tried reproducing with just Tinkers?
No
Performance Enchancers
Sodium
Searched for known issues?
Searched open issues, Searched closed issues

Edit:
It seems that Hephaestus potion fluid is bugged in general:
Filling a Hephaestus tank with a Hephaestus potion crashed the client instantly:
https://pastebin.com/BsRY6bq5
(no other mods needed)


[Dockerfile] FROM fedora:31

Will merge once https://github.com/packit-service/packit/pull/638 is merged & built


Develop > Blockchains > Maintenance > Runtime Upgrades

This PR:

moves content from previously merged PR #19
moves and updates path for code snippet

LGTM pending Luca's suggestion

Fixed on 40756b5


Region settings display stripe payment provider as fulfillment provider

Define a region, add Stripe as payment provider to the region. It is displayed as both payment provider (which is ok) but also fulfillment provider (which is not ok), as seen on the screenshot:

Nice catch! Will be fixed in upcoming release 💪


game freeze bug

Describe your bug here.
it was letting me do the game stuff, like playing the audio, but it was frozen. but then it completely froze. ill try to get an another recording of it when it happens again
Command Prompt/Terminal logs (if existing)
No response
Are you modding a build from source or with Lua?
Lua
What is your build target?
Windows
Did you edit anything in this build? If so, mention or summarize your changes.
nuh uh
If you use 1.12.0 or earlier, did you have Optimized Chart Loading turned on?
None

make sure you have your audio files in the game for the current song as "Inst.ogg" and or "Voices.ogg", and make sure you didn't just rename the extension, otherwise they will not work


Implement IsRegExp operation

Algorithm is based on ECMA-262 v6, <IP_ADDRESS>.1
JerryScript-DCO-1.0-Signed-off-by: Adam Szilagyi<EMAIL_ADDRESS>

Please rebase the PR to the latest master.

The Appveyor build failed due to an internal resource unavailable error in one job which is unrelated to the content of the PR.


fix: transpile element-plus on build

See https://github.com/nuxt/nuxt/issues/18490#issuecomment-1402805760

You can move it to nuxtOptions.ts

You can move it to nuxtOptions.ts

Done.


404 at /docker-for-mac/%29

I found a broken link : https://docs.docker.com/docker-for-mac/)
I arrived on this page through: https://www.youtube.com/

That URL looks to be invalid, and needs to be fixed on YouTube. Unfortunately, not something we can fix (the correct url is https://docs.docker.com/docker-for-mac/)
Do you know which video on YouTube mentioned that link?
I'll close this issue here, but feel free to continue the conversation

That URL looks to be invalid, and needs to be fixed on YouTube. Unfortunately, not something we can fix (the correct url is https://docs.docker.com/docker-for-mac/)
Do you know which video on YouTube mentioned that link?
I'll close this issue here, but feel free to continue the conversation


The pre-commit hook does not follow coding-style

https://github.com/01org/libyami/wiki/Coding-Style states that libyami uses the Webkit coding style.  However, the pre-commit hook does a very poor job at formatting the code according to the Webkit coding-style guidelines.  We should either fix the pre-commit hook to follow the coding style or stop asking users to use the hook and remove it from the source tree.

Seems  the hook do bad things for c++ file. and we did not find a right tool for them. Remove it is good


Feature request - allow user to select which redshifts to make animations from

Hi Cèdric:
Frequently when I use the redshift detection feature, there is one outlier with a particularly large value. It would be nice to have a way in the redshift tab, to select which redshifts to make animations from, instead of automatically doing all five. The animation assembly can take several minutes, so selecting just the one(s) of interest, can save the user considerable time.
Thanks,
Steve

Works great Cèdric - thanks.
steve

On Jul 14, 2024, at 06:31, Cédric Champeau @.***> wrote:
Closed #357 https://github.com/melix/astro4j/issues/357 as completed via #359 https://github.com/melix/astro4j/pull/359.
—
Reply to this email directly, view it on GitHub https://github.com/melix/astro4j/issues/357#event-13497956416, or unsubscribe https://github.com/notifications/unsubscribe-auth/AG4JHEFJHTAHRHZFUI645A3ZMJ4RTAVCNFSM6AAAAABK2MNE7WVHI2DSMVQWIX3LMV45UABCJFZXG5LFIV3GK3TUJZXXI2LGNFRWC5DJN5XDWMJTGQ4TOOJVGY2DCNQ.
You are receiving this because you authored the thread.


Allow multi-line arguments with space-between-parens rule

This PR fixes the false positive that would arise when it came across multi-line arguments #260.
DCO 1.1 Signed-off-by: Ben Griffith<EMAIL_ADDRESS>

:+1:
Fixes #260


PyCall does not precompile

Dear all,
I'm on
Julia Version 0.4.0-rc1+66
Commit 88e55be* (2015-09-16 08:37 UTC)
Platform Info:
  System: Darwin (x86_64-apple-darwin14.5.0)
  CPU: Intel(R) Core(TM) i7-4870HQ CPU @ 2.50GHz
  WORD_SIZE: 64
  BLAS: libopenblas (USE64BITINT DYNAMIC_ARCH NO_AFFINITY Haswell)
  LAPACK: libopenblas
  LIBM: libopenlibm
  LLVM: libLLVM-3.3

using PyPlot

returns me lots of warnings and eventually
ERROR: LoadError: InitError: cfunction: return type of pyjlwrap_hash does not match
 in __init__ at /Users/pagnani/.julia/v0.4/PyCall/src/pyinit.jl:108
 in _require_from_serialized at loading.jl:84
 in _require_from_serialized at ./loading.jl:109
 in require at ./loading.jl:219
 in include at ./boot.jl:260
 in include_from_node1 at ./loading.jl:304
 [inlined code] from none:2
 in anonymous at no file:0
 in process_options at ./client.jl:284
 in _start at ./client.jl:411
during initialization of module PyCall
while loading /Users/pagnani/.julia/v0.4/PyPlot/src/PyPlot.jl, in expression starting on line 5
ERROR: Failed to precompile PyPlot to /Users/pagnani/.julia/lib/v0.4/PyPlot.ji
 in error at ./error.jl:21
 in compilecache at loading.jl:383
 in require at ./loading.jl:250

It's probably worth pointing out that on 0.5 it PyPlot (and PyCall) work
Thanks for your really precious work!
Andrea

It works fine for me with 0.4.0-rc1 on MacOS 10.10.   Are you sure you have the latest PyPlot?  Try Pkg.update().

I did a
Pkg.update()

but also a more radical thing. I manually deleted in my .julia/v0.4/ both PyCall and PyPlot and did again a
the update.
The error is always there

What is the output of versioninfo(true)?

Julia Version 0.4.0-rc1+66
Commit 88e55be* (2015-09-16 08:37 UTC)
Platform Info:
  System: Darwin (x86_64-apple-darwin14.5.0)
  CPU: Intel(R) Core(TM) i7-4870HQ CPU @ 2.50GHz
  WORD_SIZE: 64
  uname: Darwin 14.5.0 Darwin Kernel Version 14.5.0: Wed Jul 29 02:26:53 PDT 2015; root:xnu-2782.40.9~1/RELEASE_X86_64 x86_64 i386
Memory: 16.0 GB (6744.03515625 MB free)
Uptime: 253578.0 sec
Load Avg:  1.44921875  1.724609375  1.8056640625
Intel(R) Core(TM) i7-4870HQ CPU @ 2.50GHz:
       speed         user         nice          sys         idle          irq
#1  2500 MHz      91375 s          0 s      63494 s    1144432 s          0 s
#2  2500 MHz      18154 s          0 s       5892 s    1275186 s          0 s
#3  2500 MHz      79621 s          0 s      33789 s    1185854 s          0 s
#4  2500 MHz      18510 s          0 s       6289 s    1274425 s          0 s
#5  2500 MHz      79880 s          0 s      33932 s    1185440 s          0 s
#6  2500 MHz      18487 s          0 s       6293 s    1274436 s          0 s
#7  2500 MHz      79581 s          0 s      33861 s    1185800 s          0 s
#8  2500 MHz      18494 s          0 s       6308 s    1274406 s          0 s

  BLAS: libopenblas (USE64BITINT DYNAMIC_ARCH NO_AFFINITY Haswell)
  LAPACK: libopenblas
  LIBM: libopenlibm
  LLVM: libLLVM-3.3
Environment:
  MANPATH = /opt/local/share/man:
  TERM = xterm-256color
  PATH = /usr/local/ncbi/blast/bin/:/Developer/NVIDIA/CUDA-6.5/bin:/usr/local/bin/:/opt/local/bin:/opt/local/sbin:/opt/local/libexec/gnubin:/usr/bin:/bin:/usr/sbin:/sbin:/Users/pagnani/bin:/opt/local/lib/mysql56/bin
  XPC_FLAGS = 0x0
  HOME = /Users/pagnani
  DYLD_LIBRARY_PATH = /Users/pagnani/lib/yeppp-1.0.0/binaries/macosx/x86_64:/usr/local/cuda/lib:/Developer/NVIDIA/CUDA-6.5/lib:
  JULIA_EDITOR = /Users/pagnani/bin/xem

Package Directory: /Users/pagnani/.julia/v0.4
15 required packages:
 - CUDA                          0.1.0
 - Docile                        0.5.18
 - Gadfly                        0.3.16
 - Gurobi                        0.1.29
 - HDF5                          0.5.5
 - Jewel                         1.0.6
 - JuMP                          0.10.1
 - LsqFit                        0.0.1
 - MAT                           0.2.12
 - Markdown                      0.3.0
 - MathProgBase                  0.3.17
 - NLsolve                       0.4.0              master
 - ProfileView                   0.1.0
 - PyCall                        1.0.0
 - PyPlot                        2.1.0
57 additional packages:
 - ArrayViews                    0.6.3
 - BinDeps                       0.3.15
 - Blosc                         0.1.4
 - Cairo                         0.2.30
 - Calculus                      0.1.10
 - Codecs                        0.1.4
 - ColorTypes                    0.1.4
 - Colors                        0.5.3
 - Compat                        0.7.0
 - Compose                       0.3.15
 - Contour                       0.0.7
 - DataArrays                    0.2.18
 - DataFrames                    0.6.9
 - DataStructures                0.3.12
 - Dates                         0.4.4
 - Distances                     0.2.0
 - Distributions                 0.8.6
 - DualNumbers                   0.1.3
 - FactCheck                     0.4.0
 - FastaIO                       0.1.3
 - FixedPointNumbers             0.0.10
 - GZip                          0.2.17
 - GaussDCA                      0.0.0-             master (unregistered)
 - Graphics                      0.1.3
 - Grid                          0.3.10
 - Gtk                           0.8.6
 - GtkUtilities                  0.0.5
 - Hexagons                      0.0.4
 - Homebrew                      0.1.16
 - HttpCommon                    0.2.3
 - ImmutableArrays               0.0.10
 - Iterators                     0.1.8
 - JSON                          0.4.5
 - JuliaParser                   0.6.2
 - KernelDensity                 0.1.1
 - LNR                           0.0.1
 - LaTeXStrings                  0.1.4
 - Lazy                          0.10.0
 - Loess                         0.0.4
 - MacroTools                    0.2.0
 - MacroUtils                    0.0.0-             master (unregistered)
 - NLopt                         0.2.2
 - NaNMath                       0.1.0
 - Optim                         0.4.2
 - PDMats                        0.3.5
 - PlmDCA                        0.0.0-             master (unregistered)
 - Reexport                      0.0.3
 - Requires                      0.2.0
 - ReverseDiffSparse             0.2.10
 - SHA                           0.1.1
 - Showoff                       0.0.5
 - SortingAlgorithms             0.0.5
 - StatsBase                     0.7.2
 - StatsFuns                     0.1.3
 - URIParser                     0.1.0
 - WoodburyMatrices              0.1.1
 - Zlib                          0.1.9

Tried on another MacOS 10.10 machine with 0.4-rc1 and Anaconda, and I still can't reproduce.  :-(

I'm finding this too, and it started after https://github.com/JuliaLang/julia/commit/1b6efc7ab297b7b8aea4d7cb0b52fb589392042b, which added binding deprecations like Uint => UInt64 (the warning that shows up just before the error on my system).

Sorry, scratch that - I can no longer reproduce this on either release-0.4 or 0.5 after checking I was using PyCall master.

Ah, sounds like it was fixed by 2ff91cff447fb99b0359ec7f32af96925f9bab61.  I'll tag a new release.

Works for me too.
Thanks!!!

Also worked for me! #192


smaller docker images

Problem
Hosh image is about 197MB.
Currently we are using docker-maven-plugin + Dockerfile.
Solution
Explore jlink to custom runtime image and package them with docker. May require more work
regarding module-info.java and splitting maven modules.
Benefit
Smaller docker images with only what is strictly required.

Did you already tried to base your build with this image:
https://hub.docker.com/_/microsoft-java-jre-headless
Alpine configuration is very lightweight:
FROM mcr.microsoft.com/java/jre-headless:11-zulu-alpine

Image size is down to 159MB now, thanks!
$ docker image ls
mcr.microsoft.com/java/jre-headless   11-zulu-alpine        247051b3aea5        13 days ago         159MB


Grammar and Fragmentation Fix

Confusing comment on line 12, not beginner friendly.
Fixed, so the specific comment means what was originally intended with greater clarity.

CLA Signed

recheck


Make Auto Injection Optional

Because of the current implementation of Auto-injection I think we should provide a way to turn it off. It is implemented by going through all properties of each object being created including super class ones because of that big projects with large number of dependencies and objects with many properties tend to take time to resolve such object graphs because of auto-injection overhead.
I would suggest you can have a way to completely turn it off like:
let container = DependencyContainer(autoInjectionEnabledByDefault: false)

Users can still override the default behavior for individual registrations.
container.register(.unique, autoInjectionEnabled: .enabled) { Client() }

Edit:
The initializer would default to true
class DependencyContainer {
    init(autoInjectionEnabledByDefault: Bool = true) { }
}

And the register would use an enum, defaulting to useDefault:
enum AutoInjectionEnabled {
  case useDefault
  case enabled
  case disabled
}

func register<T>(_ scope: ComponentScope = .shared, type: T.Type = T.self, tag: DependencyTagConvertible? = nil, autoInjectionEnabled: AutoInjectionEnabled = .useDefault, factory: @escaping (()) throws -> T) -> Definition<T, ()> {
    let definition = DefinitionBuilder<T, ()>

@mohamede1945 that's a good point, but this will be a breaking change for current behavior, so I'd say the default should be true.

@mohamede1945 There is now a parameter in container configuration and a method to override it on a single definition, i.e.:
let container = DependencyContainer(autoInjectProperties: false)
container.register { ServerImp() as Server }
container.register { ClientImp() as Client }
  .autoInjectingProperties(true)

That's perfect. Thanks for implementing it.


[Workflow] Update Ubuntu version to 22.04 for docker integration

Workflow issues
Update the ubuntu version in integrations/docker/images/base/chip-build and integrations/docker/images/base/chip-build-minimal to 22.04.
This should get rid of errors due to outdated packages, such as #28435 etc.
It would also allow to run e.g. Zephyr twister based tooling using the container without the need to separate containers (e.g. for the nrfconnect-sdk, Zephyr is used, so this container can also be seen as kinda superset of the Zephyr container).
Platform
all
Anything else?
No response

Update also comes with the security issue of outdated OpenSSL: https://github.com/project-chip/connectedhomeip/issues/25688

Ubuntu 24.04 is now available.
Since I didn't set up the original Dockerfile I have no complete overview what is todo, from a quick glance one may also update gcc, clang and make to recent versions.


refactor: remove params from config and introduce app context

Description
Main purpose of linked issue is to remove runtime params from config.
Simple split of these params and config resulted in one more argument being passed around from top level (called ZetaCoreContext). However, since config and these params are still related, it makes sense to have them together in top level struct (called AppContext).
I also added masterLogger to AppContext as illustration and proposal that these common application context specific things can be grouped together and passed around and accessible as needed in convenient way.
This is opened as draft PR to get some feedback on this solution and then will be converted in proper PR and tested.
Closes: #1608
Type of change

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

How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Include instructions and any relevant details so others can reproduce.
Still in draft phase.

[ ] Tested CCTX in localnet
[ ] Tested in development environment
[ ] Go unit tests
[ ] Go integration tests
[ ] Tested via GitHub Actions

Checklist:

[ ] I have added unit tests that prove my fix feature works

@skosito I see PR comments have been addressed, is it ready to be reviewed?


Require documentation of unions

Closes #516 .
This PR:

Exempts discriminator unions from requiring documentation
Requires documentation of the union itself
Requires documentation of named union variants, but not symbolic variants

:x: There is undocummented changes. Run chronus add to add a changeset or click here.
The following packages have changes but are not documented.

:x:@azure-tools/typespec-azure-core

Show changes

You can try these changes at https://cadlplayground.z22.web.core.windows.net/typespec-azure/prs/740/
Check the website changes at https://tspwebsitepr.z22.web.core.windows.net/typespec-azure/prs/740/

For reviewers, this currently is the minimal passing behavior:
/** Describes the kind of Pet. */
union PetStyle1 {
  /** Kitty */
  Cat: "cat",
  /** Doggy */
  Dog: "dog",
  string
}

/** Describes the kind of Pet */
union PetStyle2 {
  "cat",
  "dog",
  string
}

The logic only requires documentation for named variants. string doesn't require it because it's just a symbol, and any documentation added is ignored by the Autorest emitter anyhow. However, in the second style, the emitter basically converts this to the same Swagger as the first, so should documentation be required here as well?


Allow including the application name in the title

Cheers.

Changes Unknown when pulling da5e6dd1684aff3c94f526ee9734daf5b3916a8b on twe4ked:include-application-name into ** on calebthompson:master**.

Fixing the hound errors will make the style inconsistent with the rest of the file. Thoughts?

Is this likely to be merged?


use :delayed_write option to reduce number of event triggers

Currently each write is flushed, and so the number of result events triggered are numerous. This affects the responsivenes of neovim greatly when running the entire test suite on large projects. Using the :delayed_write option (doc) improves this greatly.
Obviously with streaming this will reduce the live update a bit, but it's not very noticeable IMO.
The size and delay values are configurable. Sticking to default right now (as per docs it's 64kb, 2 seconds)
Before the change

After the change

I agree, it's not necessary to trigger an update every single time a test finishes.
2 seconds seems a good default. If it starts bothering me, I'll just add an option to set the delay.


Add instrumentation to sample app

Followup to https://github.com/slackhq/circuit/pull/23 to add instrumentation tests for sample app. This is part of https://github.com/slackhq/circuit/issues/9

Once #63 is merged, it'd be good to run it in UI tests too: https://square.github.io/leakcanary/ui-tests/

Nice work!


Fix the build

Fixing the build to proceed

Hi @d-led,
Thanks for your making your first contribution to Cucumber, and welcome to the Cucumber committers team! You can now push directly to this repo and all other repos under the cucumber organization! 🍾
In return for this generous offer we hope you will:

✅ Continue to use branches and pull requests. When someone on the core team approves a pull request (yours or someone else's), you're welcome to merge it yourself.
💚 Commit to setting a good example by following and upholding our code of conduct in your interactions with other collaborators and users.
💬 Join the community Slack channel to meet the rest of the team and make yourself at home.
ℹ Don't feel obliged to help, just do what you can if you have the time and the energy.
🙋 Ask if you need anything. We're looking for feedback about how to make the project more welcoming, so please tell us!

On behalf of the Cucumber core team,
Aslak Hellesøy
Creator of Cucumber

Hi @aslakhellesoy,
thanks a lot! Very appreciated!


ARROW-1916: [Java] Include java/dev/checkstyle in git archive for source releases

cc @BryanCutler

+1


Question: How do you get messages as they come in?

I wish to get new messages as they arrive in a chat.
I see types:::UpdateNewMessage but no update_new_message() function..
What do I use UpdateNewMessage with?

You can get new messages with mpsc::channel if you pass sender to ClientBuilder::with_updates_sender. See this for more details.

Thanks!


Add sidecar containers

Adding the option to have sidecar containers on KCISS.

Hello Philippe, just 2 minor comments and this one should be good to go !


error gething Your session has expired. Please login to continue. on adding new users

Hello , i managed to install hospital-server buth each time i want to add new user i get the error
Your session has expired. Please login to continue.
is this noraml !

@qasmaoui Sorry we haven't gotten back, if this is still an issue, comment here adn we'll reopen!

i Have same issue. please help


Make waveforms exclude x-axis (restore #478)

Short description of changes:
Waveform outlines no longer include 0 sample (x-axis). This is particularly important for highly zoomed-in views. Before this change, a highly-zoomed-in waveform might look like:

(where the amplitude=0 point is included at every time). With this change, it looks more like:

(where the actual waveform is drawn).
Thanks to<EMAIL_ADDRESS>for drawing my attention to this problem.
Breaking in the external API:
None.  Zoomed-in or offset waveforms appear differently, as in image above.
Breaking changes in the internal API:
None.
Todos/Notes:
None.
Related Issues and other PRs:
I added the ability to follow the detail in zoomed-in waveforms in katspaugh#478. This was regressed by @nathansnider soon afterwards in katspaugh@46e6edd#diff-268e184f68ebebc7a4772c6e3a47c2ab , but I'm not clear if the change was actually breaking something. I believe a display that doesn't arbitrarily render the waveform as connected to zero is preferable, but I don't understand why nathansnider made the change (and he's no longer active on GitHub).

Coverage remained the same at 80.62% when pulling b88e1189ff20f256a0d124f7ddb29dd1d6b413c5 on dpwe:master into f33cfc84465af406edae7100952a39ada37bed95 on katspaugh:master.

Nice! Thanks a lot!
Does it cause any change in how silence looks like?

No.  If the waveform is already all zero, then the max and min peaks are also all zero, both before and after this change.
I modified the "zoom" example in the dist to allow the pixelsPerSecond to go up to 1000, then zoomed in to get comparison screenshots on a familiar example.  Before:

After:

This segment begins with near-silence, so you can see that it is little changed.

Perfect, thanks! 👍

I added an inline comment to describe the intention of the initialization.
I added a description to CHANGES.md.
I included the modification to the max zoom in examples/zoom to showcase the effect.

thanks. the comment indentation looks a little off though.

Sorry!  Apparently untabify was not running in my editor.


Invalid Loss error when training on the GPU

What could be the reason for the Invalid Loss error to be present in GPU training and not in the CPU training?
I've successfully trained the WTTE-RNN algorithm on a CPU using a GRU RNN on the C-MAPSS dataset. However, when doing the same on a Nvidia GPU with CuDNNGRU I am getting the Invalid Loss error at around 20/100 epochs.
I am using Keras with Tensorflow backend. And the WTTE-RNN version is 1.1.1.

Great update. I don't have access to GPU right now so I haven't run the unit-tests for CUDA for a while.
One initial theory is that I think cuda-batchnorm is (must) be different than Keras-cpu-batchnorm, since the latter accepts a mask and as far as I know, cuda-batchnorm unfortunately doesn't. I'm not sure if keras calls the cuda-batchnorm primitives though.
Another is that the machine-epsilon is different on CPU/GPU. I recommend setting keras.backend.set_epsilon(1e-07), but I'm not sure whether GPU respects this.
As a general recommendation, I recommend clipping log-likelihood using
loss_fun = wtte.loss(kind='discrete',reduce_loss=False,clip_prob=1e-5).loss_function

@ragulpr Thanks for your reply. I am not using batchnorm and you are correct about the CuDNN not accepting the masking.
I will try the epsilon and log-likelihood clipping and will let you know how it goes.

If you find anything inside WTTE not working properly with GPU it would be very good to know thanks alot for raising issue. For general NaN-avoidance there are many other git-issues with recommendations. Some top-of the list remedies for further reference;

Doublecheck mask working properly
Pretrain outputlayer as seen in https://github.com/ragulpr/wtte-rnn-examples
Clip log-likelihood through clip_prob=... flag

Another idea I forgot; I've had problems getting GPU to respect the random seed I set for it, but that might be a pytorch problem. If you repeat experiment using different seeds on CPU maybe you get the same NaN-failures?

Hi,
@dongishan called my attention to this post recently, and it just came to my mind today while working with the GPU. I have also observed numerical instabilities in the loss function when using it (we use the same cluster). I have observed this for WTTE-RNN, but also for an extension of it that I wrote for a gaussian-based loss function.
I had not commented anything until now because my main hypothesis was that those stabilities were due to my data being contaminated (I use real industrial data). But today I started comparing the GPU and the CPU and initial results show that the loss is much more stable for the CPU.
My architecture is quite simple, with a large batch size, two stacked 50 neuron LSTM's with regularisation, and a Timedistributed 100 neuron dense layer. I use tanh everywhere as an activation function.
With regards to numerical instability in the wtte-rnn case, I have usually been successful avoiding it by normalizing the times to event and by using the continuous log-lik.  For some reason, in my data-sets the discrete mode was more prone to numerical instability. I prefer that to clipping.

There may be a lot of reasons for numerical instability as pointed out, so would be very helpful if we can find a GPU/CPU reproducible example. Can it have anything to do with content of your keras.json-file? Maybe GPU is float32 and CPU is float64 or similar?
ps.
Since your final layer is dim 100, after dense(2) it will be approximately Normal(0,100) so the variance is high. I usually scale this as below
model.add(Lambda(wtte.output_lambda, arguments={"init_alpha":init_alpha,
                                                "max_beta_value":2.0,
                                                # Stability heuristic: scale by log-number of pre-output layer inputs
                                                "scalefactor":1/np.log(100),
                                               }))


Flex infobar for Firefox 69 [Irene]

User story
As a web developer, I want to easily identify flex items on my pages.
Acceptance criteria

[x] Flexbox Indicator in picker infobar 1521188

Tasks

[x] Flexbox indicator documented

[x] Images showing the flexbox indicator added to documentation

[x] Flexbox indicator added to the 69 release notes

Added content about adding flex information to the infobar to How to Examine Flex items In the infobar

I have removed the sub-grid items in this issue because they already exist in another issue: Sprints #1913

New text reviewed and approved by Patrick Brosset.


Node.JS console app arguments

Hi:
I'm having a problem finding ANY documentation on how to configure debug arguments for node.js console apps.
I can successfully run the node.js console app by right-clicking and saying "set as startup item" and configuring a launch.vs.json file. For example, here's two different configurations, one a typical Node.JS server instance, the other cloned from Mean-Expert's Loopback generator command line project:
{
  "version": "0.2.1",
  "defaults": {},
  "configurations": [
    {
      "type": "nodeDbg",
      "name": "server.js",
      "project": "server\\server.js"
    },
    {
      "type": "nodeDbg",
      "name": "lb-cl-sdk.js",
      "project": "bin\\MyModelGen\\bin\\lb-cl-sdk.js"
    }
  ]
}

Yet, when I run the app, the console window flashes up so quickly as to be useless, and I need to pass command arguments to the runtime process.
If I try adding:
       "nodeArgs": "${workspaceFolder}..\\..\\..\\server\\server.js ${workspaceFolder}..\\..\\..\\client\\src\\app\\shared\\sdk\\ -w 'enabled'"
I then get an error:
One or more errors occurred.
Failed to launch debug adapter. Additional information may be available in the output window.
Cannot connect to runtime process, timeout after 10000 ms - (reason: Cannot connect to the target: connect ECONNREFUSED 126.0/0/1:5858).
I believe this is because adding the NodeArgs means that the typical --inspect-brk=5858 argument doesn't get added? I'd add myself, but don't know how to configure VS2019 to use a specfiic debug port.
I miss the C++ project parameters window.
Oddly, I have the same problem with Visual Studio Code, and it DOES add the argument.
I've searched for this information on the web and haven't gotten many hits, oddly (other than VS Code hits, where oddly I get the same problem).
    == John ==

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

ID: dbb9adca-2c5c-edff-2c4e-4645579fb907
Version Independent ID: 85cade65-f1b0-61d4-c69e-fb8ca94fefed
Content: Debug a JavaScript or TypeScript app - Visual Studio
Content Source: docs/javascript/debug-nodejs.md
Product: visual-studio-windows
Technology: vs-javascript
GitHub Login: @Mikejo5000
Microsoft Alias: mikejo

@jgwinner This topic describes an "args" array for passing arguments to the debugger, though it's not clear if this will work for Node.js. @armanio123 for comment on what might or might not work here.

@Mikejo5000 Thanks Mike - basically, I need to pass arguments to the Node.JS program. It's a CLI app that 'does stuff' with Loopback. Without arguments, the program just prints the help screen and exits - not too useful.
Any help you can provide would be great - and update the docs. In VSXXXX it was pretty easy. I see Visual Studio is going the route of Visual Studio Code ... everything is hidden in JSON files instead of dialog boxes, and you've got to try to intellisense your way into meaningful JSON. The problem with that as a UI strategy is that it's frequently difficult to get any feedback on what an item is for, just it's name.

Sorry, forgot the link:
https://docs.microsoft.com/en-us/visualstudio/ide/customize-build-and-debug-tasks-in-visual-studio?view=vs-2019#configure-debugging-with-launchvsjson

Hmm ... I'm still having problems getting that to work. (which implies service request, but I think how to do this common operation should be documented).
I used 'args' but nothing gets passed through. The app reports no args, see screenshot.
If I use 'NodeArgs' the args DO get passed through, but before the 'options' statement that has the debugger port, which has to be passed to Node (I wouldn't know that ahead of time, unless I can force a port ...hmm).
It shouldn't be this hard!
Not a 'hello world' example, but you can see 'args' is filled in, yet the app thinks it has 0 args

If I use 'NodeArgs' I get a connection refused, because the debugger port is passed in after the arguments, so node ignores the options.

From the node debugger, you can see the out of place options:
[22:19:41.558 UTC] To client: "runInTerminal"({"kind":"external","title":"Node Debug Console","cwd":"C:\c\sanitized","args":["C:\Program Files\nodejs\node.exe","bin\MyModelGen\bin\lb-cl-sdk.js",".\server\server.js","client\src\app\shared\sdk\","-w",""enabled"","--inspect-brk=5858","bin\MyModelGen\bin\lb-cl-sdk.js"]}), timeout: 5000

Ah! That worked. My fix was:
  "debuggerPort": "1234",
  "nodeArgs": "--inspect-brk=1234 bin\\MyModelGen\\bin\\lb-cl-sdk.js .\\server\\server.js client\\src\\app\\shared\\sdk\\ -w \"enabled\""

The framework might (will) pass on a second --inspect-brk but it would be ignored, so the one we 'hard coded' will be the one that worked. Doing this I was able to hit "F10" and step into my code with VS2019! So long VS Code!
    == John ==

Thanks for the follow up! You are using an "Open Folder" solution, it looks like? I think that is required for launch.vs.jon.

I had originally, but there is a .sln from way back. Looks like it's folder open, I tried the .SLN and it didn't work.
With the above, rather frustratingly, it also doesn't leave the console window open. I checked both the debugger options and the Node.JS options; both are set to leave the windows open.
I'm able to get it to work more or less. VS2019 locks up occasionally when debugging, and the PC has to be hard reset (restart never works), but it might be Outlook related, not my above fix. (it's a dual CPU system ... which I think isn't designed to use this software).


Fix switch chain

Problem
@Fpinto29 reported that while paying an invoice, if you click switch network and try to approve, it does not work on Polygon and Ethereum(you need to close the invoice and open it again). It works well only in Sepolia.
@sstefdev could not reproduce.
Root Cause
Unknown.

@Fpinto29 can you share a video of the problem?

@sstefdev is unable to reproduce.

@Fpinto29 to try again. Close if cannot reproduce.

According to @Fpinto29 It does not happen anymore because the page refreshes now after switching networks

Closing as Can't Reproduce.


TTTAttributedLabel doesn't support Natural alignment form R to L aligned languages

My app supports Right to Left aligned languages but TTTAttributedLabel doesn't support that, when I change the region R to L aligned language should start from right side but it is displayed on the left side only, Can you tell me how can I fix this ?

@ZubairShah
let text = "..."
let lang = CFStringTokenizerCopyBestStringLanguage(text as CFString, CFRange(location: 0, length: text.characters.count))

if let lang = lang {
    let direction = NSLocale.characterDirection(forLanguage: lang as String)

    if direction == .rightToLeft {
        textView.textAlignment = .right
    }
    else {
        textView.textAlignment = .left
    }
}

Source: https://stackoverflow.com/a/46933457


The gradients in self-supervised loss.

Hello, I cannot find the process of updating the parameters in self-supervised loss. On line 125 of the ''savn_train.py'' file, the gradients in self-supervised loss are None, so could you please tell me how to update the parameters? Thank you!

Do you solve this problem? I have the same question.


Add basic Zemu tests to this repo

this is related to #7

In order to use Zemu for basic test this would require minimalist wasm code.
I have started one which can also served as an example of how ledger-rs can be used in wasm code.


Add planning API and DB schema

This change is

Current dependencies on/for this PR:

main

PR #1228   👈

This comment was auto-generated by Graphite.


<EMAIL_ADDRESS>hash check failed

Updating 'teamviewer-np' (15.2.2756 -> 15.3.2682)
Downloading new version
TeamViewer_Setup.exe (25.7 MB) [==========================================================================================================] 100% Checking hash of TeamViewer_Setup.exe ... ERROR Hash check failed!
App: nonportable/teamviewer-np
URL: https://dl.teamviewer.com/download/TeamViewer_Setup.exe#/setup.exe
First bytes: 4D 5A 90 00 03 00 00 00
Expected: df578d88e38a3f7ff81e27f11db25664d56fb082a5c9588395ec8d06eada7646
Actual: a8bb637ca9d4995de4e105f49fa162a92126197fe40de6c6c12fb08367b23c0c
Please try again or create a new issue by using the following link and paste your console output:
https://github.com/oltolm/scoop-nonportable/issues/new?title=teamviewer-np%4015.3.2682%3A+hash+check+failed

Still the same problem.
And now I don't even have TeamViewer installed...

The site is not blocked, and I don't think it is because of the other two reasons either.


Possible incompatibility with mod Cubic Chunks

I have tried running a server with Cubic Chunks while joining using this mod, but when I do I see the world and then immediately get booted out of the server with the mod stating the game crashed. This has never happened until I installed vanillafix...
crash.txt

java.lang.ArrayIndexOutOfBoundsException: 4
    at net.optifine.util.RenderChunkUtils.getCountBlocks(RenderChunkUtils.java:25)
    at net.optifine.util.RenderChunkUtils.getRelativeBufferSize(RenderChunkUtils.java:37)
    ...

seems like optifine to me

It works fine without VanillaFix, and VanillaFix works with Optifine
(without Cubic Chunks installed).
On Thu, Aug 16, 2018 at 2:51 AM, temp1011<EMAIL_ADDRESS>wrote:

java.lang.ArrayIndexOutOfBoundsException: 4
at net.optifine.util.RenderChunkUtils.getCountBlocks(RenderChunkUtils.java:25)
at net.optifine.util.RenderChunkUtils.getRelativeBufferSize(RenderChunkUtils.java:37)
...
seems like optifine to me
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/DimensionalDevelopment/VanillaFix/issues/106#issuecomment-413489568,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AQFAo72t5n-1ZUusWknVBcfIq7tt8RbLks5uRUC-gaJpZM4V5CdJ
.


Question, what are the recommended flags for OpenJ9?

I am sure it has some things that would help the game a ton, just not in the default configuration.

My tests show it hits server throughput by approximately 40%, and their own description claims its optimized for startup time, not throughput...
If you have any suggestions, I'd be happy to test.

My tests show it hits server throughput by approximately 40%, and their own description claims its optimized for startup time, not throughput... That's quite a hit to overcome with flag changes.
If you have any suggestions, I'd be happy to test!

Thing is, it's optimized for low RAM usage. If we could somehow turn the tables and turn the optimization to throughput performance, that would be great

Hmm, there are some interesting flags documented here: https://www.eclipse.org/openj9/docs/x_jvm_commands/

I was recommended these flags for low memory usage, and they seem to do just that: https://gist.github.com/FluffyFoxUwU/69f8f156feefae3d826ad0d15c694002

Some of the flags are ignored by OpenJ9 or don't work with certain features in OpenJ9. As well, by default, it defaults to the gencon policy which is a throughput collector. Referencing OpenJ9's blog post indicates gencon by itself is similar to ParallelGC which means fps would tank severely due to GC.
What would be better is either balanced policy, or gencon with concurrent scavenge. As well, OpenJ9 has adaptive GC threading in the gencon policy, meaning that you should let the VM figure out how many threads to use.
The following flags are garbage and will be ignored by OpenJ9:
-XX:InitialTenuringThreshold=n
-XX:InitiatingHeapOccupancyPercent=n
XX:ReservedCodeCacheSize=70M
-XX:G1NewSizePercent=20
-XX:G1ReservePercent=20
-XX:+OptimizeStringConcat
-XX:+TieredCompilation

The gist above returns 404 for me, what were the flags?

Nvm, Wayback Machine


Life lost during fast forward though MEM Chip caught attacker

Attackers keep moving while fading out when caught by a MEM chip.
In the (not so) special scenario that

the last chip before the CPU is a MEM chip
the distance between the MEM Chip and the CPU is short
the attacker is fast
fast forward is activated

the attacker can reach the CPU and take a life while simultaniously being caught by the MEM Chip.

Very similar to what happened with #89 and #114 :/ althwugh they had been solvel, it seems not to be enough.
Which level were you playing ? was it in late Endless levels ?

Yes, endless levels > 50

I made a test to check if other blocks are affected:

I saved this exact moment (just before wave 42/69) in my endless level 65 and let it run to the end over and over again, without interfering, in different speeds.
In regular I lost only one life in one run, the other 9 runs were without life loss.
In regular fast forward I lost 2-3 lives per run.
In fast fast forward I died immediately every time.
This led me to believe, that SHR and SUB are also affected, BUT:

Once I deleted the ACC blocks and left the slots open I lost one life in fast fast forward at max.
This means, the ACC blocks affect the enemies in a way, that they are then not handled by the following blocks anymore and reach the CPU.
Hope, this helps.

I have found the same with men chips.  I always make sure I don't have acc or men before I use FF


Add valid unicode for swift classes and types

Umlauts and several other characters are now also highlighted (correctly, corresponding to this document to fix such highlighting issues:

Thanks for the update!


Add clustermgmt flag to create kafka

Updates the CLI to display cluster ID and Name for register-cluster, deregister-cluster
adds the cluster-mgmt-api-url and access-token flags to kafka create
fixes typos and implements better wording
addresses some comments brought up in the sprint review demo

Verification Steps

Register an OpenShift cluster
Create Kafka clusters interactively
Deregister Hybrid OpenShift clusters

Type of change

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Documentation change
[ ] Other (please specify)

Also maybe worth to squash some of these commits, I don't normally care but 13 is a lot

dedicated list needs clustermgmt flags

Also we have a convention in the CLI that packages that contain commands need to match the name of the command and filename. Right now dedicated list has a package listclusters which has the command list. So the package needs to be renamed to list


pkg/gadgets: Add cifs support to trace fsslower.

Hi.
In this PR, I added support to cifs in trace fsslower:
root@vm-amd64:~# ./share/kinvolk/inspektor-gadget/ig trace fsslower --host -f cifs -m 0 > /tmp/cifs &
...
root@vm-amd64:~# more /tmp/cifs
RUNTIME.CONTAINERNAME          PID              COMM             T      BYTES
  OFFSET        LAT FILE
                               338              cat              O          0
       0        962 perf.data
                               339              cat              O          0
       0       1467 perf.data
                               340              cat              O          0
       0        987 perf.data

Best regards.

It seems cifs uses different functions depending on the mount options:
https://github.com/torvalds/linux/blob/9f8413c4a66f2fb776d3dc3c9ed20bf435eb305e/fs/smb/client/inode.c#L37-L51
Can you try with various options?

mount.cifs -o cache=none|loose|strict
mount.cifs -o nobrl

I think you have to either attach the ebpf programs to additional functions, or to document the restrictions in the documentation.

It seems cifs uses different functions depending on the mount options: https://github.com/torvalds/linux/blob/9f8413c4a66f2fb776d3dc3c9ed20bf435eb305e/fs/smb/client/inode.c#L37-L51
Can you try with various options?
* `mount.cifs -o cache=none|loose|strict`

* `mount.cifs -o nobrl`

I think you have to either attach the ebpf programs to additional functions, or to document the restrictions in the documentation.

I saw this. Sadly, the only way to support this would be to have several definition for cifs, like cifs-none, cifs-loose, etc.. I will add them, so we cover everything.

With this user experience I don't think this would be useful. A user shouldn't have to know what mount option was specified to trace vfs operations. This shows that a cifs specific tool is needed.

With this user experience I don't think this would be useful. A user shouldn't have to know what mount option was specified to trace vfs operations. This shows that a cifs specific tool is needed.

I agree. This could be solved if the code defines a list of functions instead of a single set:
-fsConfMap = map[string]fsConf
+fsConfMap = map[string][]fsConf


Generic SFC component inside render function is missing type hint

I have created a SFC component Test.vue:
<template>
  <div>
    {{ animal }}
  </div>
</template>

<script setup lang="ts" generic="T extends Animal">
import { PropType } from "vue";

export interface Animal {
  name: string;
}
defineProps({
  animal: {
    type: Object as PropType<T>,
    required: true,
  },
});
</script>

In my other component I try to use: h(Test, { }) but the type hint to use animal inside object is missing.

Could also be related to this issue

Please check if this issue has been resolved in the latest 2.x. If not, feel free to create a new one.


.net8

perhaps we should include .net 8 so it is more straightforward for new users to contribute and it is the latest LTS version now.

Hi @nathangmahler1987. As the dotnet core progresses the support for the library should advance. There is no need absolute need to upgrade the dotnet TargetFrameworks moniker though. Since the library does not depend on 3rd party dependencies and does not use any new framework features of dotnet 8 the TargetFrameworks can stay the same. Keep in mind that dotnet 6.0 would be a subset of dotnet 8.0.
What I would do to support dotnet 8.0 would be

drop 6.0 altogether in favor of 8.0 (which is the latest LTS)
Use the latest dotnet 8.0 SDK (8.0.300 I believe) in the global.json
Update the appveyor.yaml to reflect to this change so that the build does not break.
Ensure that all tests complete successfuly.


PSProviderDefaultParameterValue should not be applied to parameters

I'll try to make my case for this again.  The function body does not need to provide values for its parameters.  That is the job of the caller.  The scripter does need to make sure that values are provided and if not, handle that situation appropriately.  This could include generating an error (either terminating or not-terminating) or proceeding on with a default value.  The scripter can also use PowerShell Validation attributes to help.  From what I'm seeing, the analyzer does not take into account these validation attributes e.g. ValidateNotNull.  And even if it did, I might be working with an API where $null is a valid value i.e. is a sentinel value.
Finally, even C# and FxCop, do not require a method's parameters to be set to a default value.  This C# code:
class Program
{
    static void Main(string[] args)
    {
        Foo("foo");
    }

    static void Foo(string name)
    {
        Console.WriteLine(name);
    }
}

If I turn on CA with the "All Rules" ruleset:

I get no errors and the only warning I get is that args in the Main method is not used.  So I think PSScriptAnalyzer is going way too far in applying this rule to function parameters.

Hi Keith.
We have reworked this rule:
pls see:
https://github.com/PowerShell/PSScriptAnalyzer/pull/382
The rule (renamed) is fired only when default values are supplied for Mandatory parameters
-Raghu


Improve the build-figma-plugin CLI output when run with the --typecheck flag

Improve user feedback that a compilation error was fixed

Shipped in 1.2.0


Add the new client side web parts in enum of 1st party web parts

https://techcommunity.microsoft.com/t5/Microsoft-SharePoint-Blog/SharePoint-web-part-updates-Yammer-Conversations-My-Documents/ba-p/303719

This is in sync at schema level.


Fixed #26498 -- Prevented data loss of microseconds on MySQL and SQLite

Thanks adamchainz for reporting this issue.
https://code.djangoproject.com/ticket/26498

Buildbot, add to whitelist.

Please add a release note in docs/releases/1.8.13.txt and docs/releases/1.9.6.txt as this will backported.

Hi @charettes, I don't see these docs/releases/1.8.13.txt and docs/releases/1.9.6.txt files on master. Shall I make a separate pull request for each of them?

Both should be there now (ad3c72118fd79d27c9f958f41d354b59d883d403). If you don't see them make sure to rebase your branch on top of the latest master.

Just to clarify the release notes, I'm not certain there is a data loss issue on SQLite. I don't see a usage of typecast_time() in that backend on 1.8. In 1.9 it's added in 2dc93bb10ab1c2af1df9977f1b51d47cadab7b21 but I think that's more likely to affect search queries rather than cause data corruption.

merged in d3c87a2425b30400c3e6ea76585a9a537b6d0386, thanks!


BibSched: documentation improvement

Originally on 2014-01-22
BibSched has been improved a lot in recent master. Documentation in e.g. [https://inspirehep.net/help/admin/bibsched-admin-guide] need to be updated to be in sync with new features.
Additionally a link to the new read-only web interface must be added in:
[https://inspirehep.net/help/admin/]

@kaplun Do you have some updates in the INSPIRE OPS driven branch?

Nothing was done in that regard. Everything bibsched is already integrated.

Since bibsched is deprecated I am closing this ticket.


Could module JDY-40 replace module LC12S ?

JDY-40 is based on the same chip BK2461 as the LC12S and seems to be designed for the same purpose.

Thanks

After some analyse I answer myself, response is no.
LC12S configuration is done by sending a specific config array of 18 bytes length and JDY-40 config is done by AT commands.
Maybe it can replace the LS12S but the code must be rework to use AT commands.
I will maybe try if this can be done when I receive my modules, I ordered both types but the JDY-40 is normaly faster to get.
Second problem is that the JDY-40 seems to not have a transmitting status LED, so for the channel search it can be a problem.
I must also check if a led can be added somewhere on this module when using it in serial port transparent transmission mode.
Or find another technique to detect the correct channel.


feat(ingest): powerbi config improvements modified_since, extract_dataset_schema and many more

Checklist

[x] The PR conforms to DataHub's Contributing Guideline (particularly Commit Message Format)
[ ] Links to related issues (if applicable)
[x] Tests for the changes have been added/updated (if applicable)
[x] Docs related to the changes have been added/updated (if applicable). If a new feature has been added a Usage Guide has been added for the same.
[ ] For any breaking change/potential downtime/deprecation/big changes an entry has been made in Updating DataHub

modified_since config ingestion which ingest only modified workspaces (which also improve stateful ingestion to be per checkpoint per workspace)
ownership config to have more fine grain control of how the corp user is built and also what criteria in term of authority over the powerbi asset
extract_datasets_to_containers config wrap powerbi tables (datahub dataset) under 1 powerbi dataset (datahub container)
extract_only_matched_endorsed_dataset config to extract only dataset that have matching endorsement
extract_dashboards config whether to ingest powerbi dashboards
extract_dataset_schema to extract powerbi dataset table schema to datahub dataset schema

I think the main improvement will be modified_since, which leave a interesting problem for stateful.
I tried my best to solve this in the most elegant way as compared to my previous PR which got rejected.
As much as I am catering for backward compatibility, those extra config are entirely optional and will not impact current users. :)

@hsheth2 , @mohdsiddique and @jjoyce0510  . Can you all please help to take a look?

@mohdsiddique  hihi, please help me to review again. I have changed accordingly to your comments. thanks!

@hsheth2  and @mohdsiddique  please help to review again. thanks!

Hi @hsheth2 , can we get this reviewed again? thanks

@hsheth2  and @mohdsiddique please help to review again. I hope we can close this soon as the default recipe setting is already backward compatible.

Codecov Report
Patch coverage: 87.05% and project coverage change: -11.12 :warning:

Comparison is base (c7d35ff) 74.90% compared to head (b4366d7) 63.78%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
Additional details and impacted files
@@             Coverage Diff             @@
##           master    #7519       +/-   ##
===========================================
- Coverage   74.90%   63.78%   -11.12%
===========================================
  Files         353      353
  Lines       35395    35586      +191
===========================================
- Hits        26511    22699     -3812
- Misses       8884    12887     +4003

Flag
Coverage Δ

pytest-testIntegration
?

pytest-testIntegrationBatch1
?

pytest-testQuick
63.78% <87.05%> (+0.20%)
:arrow_up:

pytest-testSlowIntegration
?

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

Impacted Files
Coverage Δ

...tahub/ingestion/source/powerbi/m_query/resolver.py
78.76% <22.72%> (-4.07%)
:arrow_down:

...on/src/datahub/ingestion/source/powerbi/powerbi.py
94.14% <89.28%> (-2.08%)
:arrow_down:

...n/source/powerbi/rest_api_wrapper/data_resolver.py
85.86% <92.30%> (+0.15%)
:arrow_up:

...ion/source/powerbi/rest_api_wrapper/powerbi_api.py
88.71% <94.59%> (+0.85%)
:arrow_up:

...on/source/powerbi/rest_api_wrapper/data_classes.py
92.72% <97.50%> (+2.72%)
:arrow_up:

...on/src/datahub/ingestion/source/common/subtypes.py
100.00% <100.00%> (ø)

...ion/src/datahub/ingestion/source/powerbi/config.py
97.47% <100.00%> (-0.26%)
:arrow_down:

...stion/source/state/stale_entity_removal_handler.py
93.49% <100.00%> (-0.68%)
:arrow_down:

... and 90 files with indirect coverage changes
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

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

@hsheth2  can u help to review again? we should close this soon. Can u help to resolve the conflict if all is good?

@aezomz in the future, it'd be much easier to review these changes if they were broken up into multiple smaller PRs

Sorry for this big PR. Thank you and Mohd so much for reviewing this as well.
In the future, I will separate them into smaller PR so it will be more manageable.
I have made the necessary changes, please review again. Thanks @hsheth2 !

@hsheth2 please help to review again, thanks.

Can we merge already? @hsheth2  @mohdsiddique .

Hihi @hsheth2 , lets merge this?

@mohdsiddique

We can remove expression from custom properties and add it in viewProperties, please refer https://github.com/datahub-project/datahub/blob/master/metadata-ingestion/src/datahub/ingestion/source/looker/lookml_source.py#L935.
The DataHub display the code in view definition section on Portal. (https://demo.datahubproject.io/dataset/urn:li:dataset:(urn:li:dataPlatform:looker,long_tail_companions.view.customer_focused,PROD)/View Definition?is_lineage_mode=false)
Could you please confirm your user related changes are backward compatible ?

I have added view properties, and view subtype to enable the view definition.
I have fixed the user mcps ordering issues. The golden json ordering will be similar to the OSS now for corpuser.
@hsheth2 please review and lets close this. thanks

Hi @aezomz
Could you please update the concept mapping https://github.com/aezomz/datahub/blob/powerbi-improvements/metadata-ingestion/docs/sources/powerbi/powerbi_pre.md#concept-mapping,

Hi @mohdsiddique , i think its already done in this PR which is approved by Joyce.
https://github.com/datahub-project/datahub/pull/7835

Once again, please close this soon. thanks

@aezomz I'm going to merge this in tonight, assuming CI turns green
One thing I'd like to call out - the changes to stateful ingestion in this PR are definitely experimental, and I can't guarantee that we won't be making changes to stateful ingestion in the future that may mess with the implementation here.


Bring back wells, thumbnails, and panels

I suggest make wells, thumbnails, and panels deprecated. Because it may break the UI when updated to v4 if those are widely used in the UI and forget to update at some places. Plus I even have custom styling for those in my UI, others may have it too. So it will be better to have them for now beside cards.

Approximate duplicate of #17240.


Inline setting of variable is incorrectly translated

Setting a variable and then using its result on one line does not get converted properly, see the following excerpt from the code I tested:
import static org.lwjgl.glfw.GLFW.*;

import org.lwjgl.glfw.GLFWKeyCallback;

public class Main {

    private static GLFWKeyCallback keyCallback;

    public static void main(String args[]) {

        glfwSetKeyCallback(window, keyCallback = new Input());

    }
}

This converts to
import org.lwjgl.glfw.GLFW._
import org.lwjgl.glfw.GLFWKeyCallback
//remove if not needed
import scala.collection.JavaConversions._

object Main {

  private var keyCallback: GLFWKeyCallback = _

  def main(args: Array[String]) {
    glfwSetKeyCallback(window, keyCallback = new Input())
  }
}

The problem is on line 11, it should really become the following:
    glfwSetKeyCallback(window, {keyCallback = new Input(); keyCallback})

Bugs should be reported to scalagen library. This is just a wrapper project.


Clarify consequences and security risks of installing iot-inspector-client

IoT-inspector does not warn of the risks of installing the tool prior to installation, and does not follow best practices for installation and client security.
The instructions should start by showing the disclaimer and warnings, so users know what they're getting in to before installing it. If, after learning of the risks, they aren't going to consent, it is a waste of time to install it, and might even cause the user various Python configuration and security problems, since the un-installation procetures don't remove everything that was installed.
E.g. the Linux installation scripts do not warn the user about what would seem to be a variety of security and system configuration risks:

On the download page, instructions to click for doing the installation are placed above the material that people are warned to read before installation. The installation scripts themselves provide no warning. The app itself says simply Please run as root if not run with full system permissions. All these scripts should include links to clear warnings and ask the user for permission after they've read the warnings.
The Installation shell code is run as root, risking bugs or malicious code in the installer and in all the installers for all the third-party packages that are installed. The user should at least be warned to look at the code and package lists. Signatures for the code should be provided. This is a big and growing problem throughout the world of software development: Malware in PyPI Code Shows Supply Chain Risks
If even security tools don't help users learn good hygiene, where are they going to learn it?
A variety of Python packages are installed as root into the system's Python2 library, rather than being installed via venv or into the individual user's Python package space. No provision is made for keeping these packages up-to-date, though bugs in them are likely to surface over time.
The whole process runs as root. Normally daemon processes run only essential code as root, in order to minimize the attack surface, rather than opening up the whole system to any flaw in any of the project code or Python libraries.
The procedures to uninstall IoT Inspector don't remove the Python packages that it installed. Using a Python virtual environment as noted above would make it easy to do a clean removal.
The code is written in Python2. which will not have any security updates as of 2010-01-01, less than 3 months from now. See Python 2 End Of Life Threatens Security

In addition, as noted in #21, the tool runs persistently, regardless of who is logged in, and thus may send information that others may find sensitive without warning them.
Finally, readers may be under the assumption, given the unqualified use of "we" in the consent agreement and FAQ, that the Principle Investigators are the only ones who will have access to the data. They would have to dig pretty far to notice that "We are likely to release the full dataset by the end of 2019"  in the Frequently Asked Questions. So it seems that a variety of approved researchers from a variety of institutions will have access to the data. This should be made more clear to users (research subjects).

Thank you @nealmcb! We will address these issues in the next release!


Issue with Unwrapped SpeechRequest in SpeechController Causing Nil Issue in iOS 17

In the SpeechController module, there is an explicit unwrapping of the speechRequest, which is causing a nil issue, particularly in iOS 17. This nil issue arises when tapping the screen while recording, which previously resulted in displaying existing text but is now malfunctioning due to the nil issue with the speechRequest. And, does it affect the usage of AVAudioPlayer?

This is also happening with me on iOS 17 and the app is crashing on production, is there any solution??

This is also happening with me on iOS 17 and iOS 18 now.


Access to title (menuTitle) UILabel

Thanks for a great framework.
Is it possible to open an access to menuTitle: UILabel property so it would be easier to customize it? Or give some interface functions for customizations the font and text. Would be great if it'd allow to assign attributed text.
Right now, I'm forced to do such kind of workaround
dropdown.didSelectItemAtIndexHandler = { [weak self] index in
            for view in dropdown.subviews.first!.subviews {
                if view is UILabel {
                    (view as! UILabel).text = FamilyNavigationBarBuilder.calendarTitle(subcategory: items[index])
                }
            }
        }

I would be glad to issue a pull request if there are no reasons you won't consider merging it.

I need this as well.  Thanks for the workaround in the meantime!


Remove global lock from shared scheduler

Currently the shared scheduler is implemented behind locks, see:
https://github.com/Thomasdezeeuw/heph/blob/40984a97301706e32e7237a1535a7d06f5e157e7/src/rt/shared/scheduler/mod.rs#L102
And:
https://github.com/Thomasdezeeuw/heph/blob/40984a97301706e32e7237a1535a7d06f5e157e7/src/rt/shared/scheduler/runqueue.rs#L16
This was done in commit 3c59598 to fix #288. Remove this global lock.

Pr #429 removes the lock from the Inactive tree, now the RunQueue is left.


Profile page is not displaying completed solutions correctly

The profile page isn't showing the completed Algorithms and Challenges correctly. It appears that there are Challenges (Old Waypoints) mixed in with the Algorithm (Old Bonfire) solutions.
Please see the screenshot below:

Any update?

Note: When I sign out of my profile it shows something completely different from what I see when signed in:

Also, the basic Challenges and Checkpoints are still mixed in with the Algorithm challenges.

@revisualize I think you've locked your profile that's why your challenges are hidden when you're logged out (if this is what you're trying to show on that screenshot).

I'm going to ping @BerkeleyTrue about the mixed challenges.

Look, I know that all of my challenges are hidden. That isn't the issue. I'm trying to show that I have over 30days of continuous activity.
Signed in I am currently seeing:
Longest Streak: 34 days   Current Streak: 34 days
However, signed out only shows:
Longest Streak: 12 days   Current Streak: 9 days
That is current as of 2016-02-18.

@revisualize Can you confirm that you still have the mixed challenges?
About the different streaks we have #7468 in discussion. Please feel free to carry it there.

I can confirm that basic Challenges are still mixed in with the Algorithms on my profile page.

At this time the issue is not resolved.

@BerkeleyTrue @raisedadead This issue is not resolved and should not have been closed.

@revisualize @raisedadead have you had a chance to look at this recently? This is an issue we still want to fix. Would either of you be interested in taking a crack at it?

Nope. Unfortunately this is open. Tagging as Help Wanted. I might take a look once I get some time, but if any one is interested, please have a look.

I'd be happy to take this one this week.

@petrgazarov thanks. do checkout the contributing guide and reach out on the contributors chat if you need any assisstance

Is this still in issue in staging? I changed the way the profile page pulls the meta information for a challenge.


Use html page for showing magic code

Provide a default html page or view
Allow the html page to be overridden by the API consumer.
{
staticDirectory : __dirname + "/public",
magicPage : "magic.html"
}

Completed.  See Evernote example for a branded html page showing the magic code


feat: upgrade Tiltfile hot reloading speed

Hah I think I could load the result of the kustomization into a variable then do a string replace.

Ah! This https://docs.tilt.dev/templating.html alloys definitely improve things.

Ah! This https://docs.tilt.dev/templating.html alloys definitely improve things.

Yep. This is exactly what I was looking for. Patching the deployment before we apply it. Nice. I'll have an update on this later on.

yaaaay this worked. :)))


Fedora/29

Besides whatever fedora-review finds, I think it looks good now!
One last thing, we could put the duplicated description text box in a %global macro:
%global description HPX ist great bla bla

For some packages we have one additional sentence, like This package contains development headers and libraries for hpx.
How could I add it to the %global description?

I would just use it for the duplicated block and leave the individual last sentences where they are. See this example here: lammps.spec

[Patrick@thinpad review]$ fedora-review --rpm-spec --prebuilt --name hpx
INFO: Processing local files: hpx
INFO: Getting .spec and .srpm Urls from : Local files in /home/Patrick/review
INFO:   --> SRPM url: file:///home/Patrick/review/hpx-1.2.0-0.1.rc1.src.rpm
INFO: Using review directory: /home/Patrick/review/hpx
WARNING: No disttag found in prebuilt packages

Added Release:        0.1.rc1%{?dist}

https://bugzilla.redhat.com/show_bug.cgi?id=1649874

https://koji.fedoraproject.org/koji/taskinfo?taskID=30886771

Hmm, i686 has a real problem:
In file included from /builddir/build/BUILD/hpx_1.2.0/hpx/lcos/local/packaged_continuation.hpp:22,
                 from /builddir/build/BUILD/hpx_1.2.0/hpx/lcos/future.hpp:1798,
                 from /builddir/build/BUILD/hpx_1.2.0/hpx/parallel/executors/default_executor.hpp:12,
                 from /builddir/build/BUILD/hpx_1.2.0/hpx/parallel/executors.hpp:11,
                 from /builddir/build/BUILD/hpx_1.2.0/hpx/include/parallel_executors.hpp:9,
                 from /builddir/build/BUILD/hpx_1.2.0/examples/resource_partitioner/async_customization.cpp:6:
/builddir/build/BUILD/hpx_1.2.0/hpx/util/deferred_call.hpp:160:5: error: 'hpx::util::detail::deferred<typename std::decay<_Tp>::type, typename std::decay<_Args>::type ...> hpx::util::deferred_call(F&&, Ts&& ...) [with F = const test(Executor&) [with Executor = hpx::threads::executors::guided_pool_executor_shim<hpx::threads::executors::pool_numa_hint<dummy_tag> >]::<lambda(hpx::lcos::future<hpx::util::tuple<hpx::lcos::future<long long unsigned int>, hpx::lcos::shared_future<float> > >&&)>; Ts = {hpx::lcos::future<hpx::util::tuple<hpx::lcos::future<long long unsigned int>, hpx::lcos::shared_future<float> > >}; typename std::decay<_Tp>::type = test(Executor&) [with Executor = hpx::threads::executors::guided_pool_executor_shim<hpx::threads::executors::pool_numa_hint<dummy_tag> >]::<lambda(hpx::lcos::future<hpx::util::tuple<hpx::lcos::future<long long unsigned int>, hpx::lcos::shared_future<float> > >&&)>]', declared using local type 'const test(Executor&) [with Executor = hpx::threads::executors::guided_pool_executor_shim<hpx::threads::executors::pool_numa_hint<dummy_tag> >]::<lambda(hpx::lcos::future<hpx::util::tuple<hpx::lcos::future<long long unsigned int>, hpx::lcos::shared_future<float> > >&&)>', is used but never defined [-fpermissive]
     deferred_call(F&& f, Ts&&... vs)
     ^~~~~~~~~~~~~
In file included from /builddir/build/BUILD/hpx_1.2.0/hpx/lcos/local/packaged_continuation.hpp:22,
                 from /builddir/build/BUILD/hpx_1.2.0/hpx/lcos/future.hpp:1798,
                 from /builddir/build/BUILD/hpx_1.2.0/hpx/parallel/executors/default_executor.hpp:12,
                 from /builddir/build/BUILD/hpx_1.2.0/hpx/parallel/executors.hpp:11,
                 from /builddir/build/BUILD/hpx_1.2.0/hpx/include/parallel_executors.hpp:9,
                 from /builddir/build/BUILD/hpx_1.2.0/examples/resource_partitioner/async_customization.cpp:6:
/builddir/build/BUILD/hpx_1.2.0/hpx/util/deferred_call.hpp:160:5: error: 'hpx::util::detail::deferred<typename std::decay<_Tp>::type, typename std::decay<_Args>::type ...> hpx::util::deferred_call(F&&, Ts&& ...) [with F = const test(Executor&) [with Executor = hpx::threads::executors::guided_pool_executor_shim<hpx::threads::executors::pool_numa_hint<dummy_tag> >]::<lambda(hpx::lcos::future<hpx::util::tuple<hpx::lcos::future<long long unsigned int>, hpx::lcos::shared_future<float> > >&&)>; Ts = {hpx::lcos::future<hpx::util::tuple<hpx::lcos::future<long long unsigned int>, hpx::lcos::shared_future<float> > >}; typename std::decay<_Tp>::type = test(Executor&) [with Executor = hpx::threads::executors::guided_pool_executor_shim<hpx::threads::executors::pool_numa_hint<dummy_tag> >]::<lambda(hpx::lcos::future<hpx::util::tuple<hpx::lcos::future<long long unsigned int>, hpx::lcos::shared_future<float> > >&&)>]', declared using local type 'const test(Executor&) [with Executor = hpx::threads::executors::guided_pool_executor_shim<hpx::threads::executors::pool_numa_hint<dummy_tag> >]::<lambda(hpx::lcos::future<hpx::util::tuple<hpx::lcos::future<long long unsigned int>, hpx::lcos::shared_future<float> > >&&)>', is used but never defined [-fpermissive]
     deferred_call(F&& f, Ts&&... vs)
     ^~~~~~~~~~~~~

Let me know when you have a patch!

OpenSuse has the same problem on i586: https://build.opensuse.org/package/show/home:cjunghans:branches:science:HPC/hpx

Ok I added the patch from https://github.com/STEllAR-GROUP/hpx/pull/3551 in https://koji.fedoraproject.org/koji/taskinfo?taskID=30901371

And https://build.opensuse.org/package/show/home:cjunghans:branches:science:HPC/hpx

@junghans Is it possible to add a License header and our both names to the spec file, as for the open sude one?

It sure is possible, but by the end of the day hpx.spec will live in the git repo for the distribution, so I would worry about that too much.

I tried that in the past with other projects, but in Fedora the spec gets bumped from time to time by release maintenaners anyhow, so you will have to merge things manually anyhow.

https://pagure.io/releng/fedora-scm-requests/issue/8897

Nice, can you add me to this repo?

https://pagure.io/releng/fedora-scm-requests/issue/8902
https://pagure.io/releng/fedora-scm-requests/issue/8903
https://pagure.io/releng/fedora-scm-requests/issue/8904

Once the repo is there, you will need an https://src.fedoraproject.org account!

Already had it (https://src.fedoraproject.org/user/diehlpk)

https://koji.fedoraproject.org/koji/taskinfo?taskID=30997586
https://koji.fedoraproject.org/koji/taskinfo?taskID=30997599
https://koji.fedoraproject.org/koji/taskinfo?taskID=30997601
https://koji.fedoraproject.org/koji/taskinfo?taskID=30997603

Already had it (https://src.fedoraproject.org/user/diehlpk)

There is an error: "This user must be in one of the following groups to be allowed to be added to this project: packager"

https://koji.fedoraproject.org/koji/getfile?taskID=30997630&volume=DEFAULT&name=build.log&offset=-4000
The epl7 build failed somehow

https://koji.fedoraproject.org/koji/getfile?taskID=30997630&volume=DEFAULT&name=build.log&offset=-4000
The epl7 build failed somehow

Yup, the gcc is too old, see root.log.

https://koji.fedoraproject.org/koji/getfile?taskID=30997630&volume=DEFAULT&name=build.log&offset=-4000
The epl7 build failed somehow

Yup, the gcc is too old, see root.log.

Ok, so we just do not provide the epl7?

Already had it (https://src.fedoraproject.org/user/diehlpk)

There is an error: "This user must be in one of the following groups to be allowed to be added to this project: packager"

I reported the issue at the ticket in bugzilla.

https://koji.fedoraproject.org/koji/getfile?taskID=30997630&volume=DEFAULT&name=build.log&offset=-4000
The epl7 build failed somehow

Yup, the gcc is too old, see root.log.

Ok, so we just do not provide the epl7?

<EMAIL_ADDRESS>seems related.

https://bodhi.fedoraproject.org/updates/FEDORA-2018-8367209bd8
https://bodhi.fedoraproject.org/updates/FEDORA-2018-a9b53adbdc


@conda-forge-admin, rerender

Solution to issue cannot be found in the documentation.

[X] I checked the documentation.

Issue
a
Installed packages
a

Environment info
a

Hi! This is the friendly automated conda-forge-webservice.
I just wanted to let you know that I rerendered the recipe in conda-forge/cypari2-feedstock#39.


Listen for session data filtering

Session data set here: https://github.com/tombruijn/appsignal-elixir_js_plug/blob/3e2c7a6e15110ccf6faecceeff392e5544efe065/lib/appsignal_js_plug.ex#L125
Should be filtered like so: https://github.com/appsignal/appsignal-elixir/blob/26f21efd7e5ae4af8b9f889b4ce3c1a6c5ceedeb/lib/appsignal/transaction.ex#L416-L424
For the documentation see:
https://docs.appsignal.com/elixir/configuration/session-data-filtering.html

⚠  This Plug is now deprecated. Please use AppSignal's JavaScript integration instead.
Closing issue.


Update migrations

Update and squash migrations for hs_swat and hs_model_program
Update development database

@hyi - update to migrations files and development db. If you wouldn't mind merging this in once you've looked it over it would be appreciated. That way I'm not merging my own commits without having them vetted by someone other than myself.

@mjstealey I have merged in this update-migrations branch into latest develop branch and rebuilt everything locally using deploy-hs script, and everything runs smoothly without any issues. I also tested a few resource types and did not find any issues. Everything looks good to me. +1


”Gogoșari”, ”Drăghiceanu”, ”Izvoru”, ”Rălești” are listed in the drop-down menu

Description:
When user writes ”Gogoșari”, ”Drăghiceanu”, ”Izvoru”, ”Rălești” the locations appear in the drop-down menu next to the comune and the county witch is a part according to the law.
Precondition:
The website is up an running.

Step 1
Write in the search bar "Gogoșari"

Expected results
The location appears in the drop-down menu as "Sat Gogoșari, GIURGIU (Gogoșari)".

Step 2
Press "x" button.

Expected results
The location was deleted from the search bar.

Step 3
Write in the search bar "Drăghiceanu"

Expected results
The location appears in the drop-down menu as "Sat Drăghiceanu, GIURGIU (Gogoșari)".

Step 4
Press "x" button.

Expected results
The location was deleted from the search bar.

Step 5
Write in the search bar "Izvoru"

Expected results
The location appears in the drop-down menu as "Sat Izvoru, GIURGIU (Gogoșari)".

Step 7
Write in the search bar "Rălești"

Expected results
The location appears in the drop-down menu as "Sat Rălești, GIURGIU (Gogoșari)".


Cannot download and open file in Chrome - pending trying download_location()

ift can not open

@GrowUpXin can you try if changing the download location can help? See here for syntax - https://github.com/tebelorg/RPA-Python/issues/279#issuecomment-877749880

(closing issue for now, but please feel free to open or share any thoughts or questions)


[terra-clinical-onset-picker] Clicking on the year drop down displays a scrool bar in IE but not in Chrome

Bug Report
Description
Clicking on the year drop down displays a scrool bar on IE and the view is distorted but on chrome their is no scrool bar and the view of the drop down with the years is good.
IE drop down with scrool bar:

Chrome drop down with no scrool bar:

Steps to Reproduce

Open patient chart in solgm for a patient with health concerns. (Schreur,Grace)
Open OnePlan Plus Button mpage tab
Click a healthconcern in the list to open detail view.
Click the modify button(top right) to open the modify view.
Scroll down to "Onset date", select year and click on tthe year dropdown.

Expected Behavior
The year drop down should display as in chrome or else if it displays a scrool bar in IE then the view should be set to be displayed as in chrome and also to be consistent both chrome and IE should display the scrool bar.
Environment
Component Name and Version: Plan-js 1.2.1
Browser Name and Version: IE11 11.737.17763.0
Powerchart: 2018.01 - solgm
Mpages: 6.12 - http://ip85wodr12a/mpage-content/solgm.ip.devcerner.net/MPAGES_6_12

I think the scrollbar is something we can't avoid.  However, we should make sure the year can be seen without wrapping

Changing min-width to 5.5em in this line can solve the wrapping issue.
I am seeing that the scrollbar appears on hover in chrome on mac. But it displays always on IE and Chrome in Windows in Form-select which is being used here. I suggest displaying scrollbar on-hover for all Browser-OS combinations.
cc @neilpfeiffer @ryanthemanuel

@saket2403 The scrollbar should show only when there is overflow as mentioned in dropdown . so setting the display of scroll on hover might display it even there is no overflow.  this issue seems to be browser specific and as @ryanthemanuel  mentioned in the above comment we cannot avoid this . so i think setting the min-width here in onset picker for year would be the correct approach to fix this.


docs:  Variance theming docs [WEB-1416]

Overview
Main description from: https://github.com/Codecademy/client-modules/pull/1611

Adds variance docs for theme customization

PR Checklist

[ ] Related to designs:
[x] Related to JIRA ticket: https://codecademy.atlassian.net/browse/WEB-1416
[x] I have run this code to verify it works
[ ] This PR includes unit tests for the code change

📬Published Alpha Packages:@codecademy/gamut-illustrations@0.10.1-alpha.7e882c.0@codecademy/gamut-kit@0.3.33-alpha.7e882c.0@codecademy/gamut-labs@13.5.10-alpha.7e882c.0@codecademy/gamut-styles@9.5.2-alpha.7e882c.0@codecademy/gamut-tests@2.3.13-alpha.7e882c.0@codecademy/gamut@31.0.4-alpha.7e882c.0@codecademy/konamimojisplosion@0.2.13-alpha.7e882c.0@codecademy/markdown-overrides@0.5.25-alpha.7e882c.0@codecademy/styleguide@35.0.7-alpha.7e882c.0<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://6081e42b52cb70144916dce8--gamut-preview.netlify.appDeploy Logs


EIP-3770: Fix inputs playing nicely with QR codes

What it solves
Resolves #3055
How this PR fixes it
First of all, partially reverts #2954.
AddressBookInput
Just accepts both prefixed and un-prefixed addresses. Validates them and becomes a static EthHashInfo on blur.
(Used in SendFunds and SendCollectibles)
AddressInput
This component is used in all other places where we allow typing an address.
The input accepts both prefixed and un-prefixed addresses. Prefix is validated against the current chain.
The value is sanitized and passed to the form (parent component). The prefix is NOT displayed at this point. ☹
Todo:

[x] Fix AddressInput tests (commented out some)

How to test it
Follow the scenarios in the linked bug report.

It was a complete revert on the validation of the fields we did before it seems. I want to showcase the current status of the inputs so we know where we are standing with it:
If it is an input that remains editable after entering an address with prefix by QR code or copy/paste (Editable inputs like AB add entry form) The prefix is not shown at all as long the address is valid.

Only those inputs that become "read only" show the prefix (or not, depending on Settings > Appearance configuration)

Example
Add Entry in AB: I add an address that is valid-> The address show no prefix regardless of how the settings are configured
Add Entry in AB: I try to add an address that already exists -> The prefix is shown, giving the error of already existing owner
Send funds: I enter an address by QR, copy/paste or selecting it from the suggested list -> the prefix is shown/not shown depending on Settings > Appearance configuration, because this is one of the fields that becomes "read only"

Issue:
Prefixes are not being validated in different networks.
You can copy/paste or use a QR code with a "rin:" prefix, and it will be valid in the xdai network for example and replaced by an "xdai"
Expected:
The system should show the prefix in the input and imply that the input is not from the correct network:
A rinkeby address of a safe:
rin:0x9913B9180C20C6b0F21B6480c84422F6ebc4B808
And the QR code with prefix for the same address:

Gif:

@francovenica should be fixed now ✅

Please use the epic branch, not this one.

Ok, tested here https://pr3015--safereact.review-safe.gnosisdev.com/app/
Issues:
The PNG I used, a QR code for a Rink safe:

1 - The network validation is still not working for Spending limits and send collectibles:

2 - The Send funds network validation works fine, but it shows the input empty after trying a QR code with the wrong network:

@francovenica please use the epic PR for these comments.
And please send me over that QR code.


Faster bike speeds or switching from walk to bike causes accessibility decreases

Increasing the bike speed on the request can cause decreases in accessibility, clearly a bug:

Additionally, switching from walk to transit to bike to transit can cause decreases in accessibility. It may be that we're using a slower walk speed when walking a bike, or because we're taking into account turn costs even in the walk search when doing the bike search.

It may be that we're not properly combining the non-transit search with the transit search.

One bug: we were always calculating travel time based on walk distance in bike-to-transit mode, which obviously is nonsensical.

OK, fixed that bug, but the issue remains:

Looks a lot more like random noise now though.

This should be fixed. I'll run another multipoint job to confirm.

Confirmed fixed.


Memory problem in class relationship

Hi,
First of all, thank you for your time and for this amazing package.
However, when I use #[LodataRelationship] between two classes a memory issue is generated.
Error
PHP Fatal error:  Allowed memory size of 536870912 bytes exhausted (tried to allocate 32768 bytes) in C:\Projects\oData\vendor\laravel\framework\src\Illuminate\Filesystem\Filesystem.php on line 77
PHP Fatal error:  Allowed memory size of 536870912 bytes exhausted (tried to allocate 32768 bytes) in C:\Projects\oData\vendor\symfony\error-handler\Error\FatalError.php on line 1

Classes
namespace App\Models\PAD;

use App\Models\ACD\Student;
use Flat3\Lodata\Attributes\LodataIdentifier;
use Flat3\Lodata\Attributes\LodataRelationship;
use Illuminate\Database\Eloquent\Model;

#[LodataIdentifier('persons')]
class Person extends Model
{
    protected $table = 'person';
    protected $primaryKey = 'idperson';
    public $timestamps = false;

    protected $fillable = [
        'idperson',
        'firstname',
	'lastname',
	'email'
    ];

    #[LodataRelationship]
    public function student()
    {
        return $this->hasMany(Student::class, 'idperson', 'idperson');
    }
}

namespace App\Models\ACD;

use App\Models\PAD\Person;
use Flat3\Lodata\Attributes\LodataIdentifier;
use Flat3\Lodata\Attributes\LodataRelationship;
use Illuminate\Database\Eloquent\Model;

#[LodataIdentifier('students')]
class Student extends Model
{
    protected $table = 'student';
    protected $primaryKey = 'idstudent';
    public $timestamps = false;

    protected $fillable = [
        'idstudent',
        'idperson',
	'idschool',
	'username',
	'password'
    ];

    #[LodataRelationship]
    public function person()
    {
        return $this->belongsTo(Person::class, 'idperson', 'idperson');
    }
}

Request
http://localhost:8050/odata
Version
"flat3/lodata": "^5.17"
If I comment the relationship person() in Student class the error isn't generated.
Thanks !

Thanks for the report @marciotheis, this was due to the combination of the two-way relationship and setting a specific identifier for the set, causing a loop in the discovery code which made PHP run out of memory.
Resolved in https://github.com/flat3/lodata/releases/tag/v5.17.1


Prototype Indexing

Indexing is a highly requested feature (#1205) that we currently don't have an answer to. This PR implements one possible version of indexing, by eagerly updating the index to ensure it's always valid.

Closing, might reimplement in a separate PR once #2254 lands.


Is it possible to use Retry in NodeRed?

Checklist

[X] I have filled out the template to the best of my ability.
[X] This only contains 1 feature request (if you have multiple feature requests, open one feature request for each feature request).
[X] This issue is not a duplicate feature request of previous feature requests.

Is your feature request related to a problem? Please describe.
I was wondering if it is possible to use Retry in NodeRed?
Describe the solution you'd like
I would like to be able to use Retry in NodeRed.
Describe alternatives you've considered
I have all my automations in NodeRed. I would have to migrate everything to HA to use your code.
Additional context
No additional context.

[Disclaimer: I'm not an expert (or even a user) of NodeRed.]
It seems that NodeRed supports call service.
Can you elaborate on the scenario? What is not working when calling retry.call from NodeRed?

For anyone else trying to figure this out, here is an example of how I'm using this in Node Red.
The service needs to be "action", not "call" (not sure why, but this works).

Something I'm noticing, though, is that in Node Red the payload is not getting passed along. I'm guessing this has something to do with how the service call is getting wrapped. This example works with the normal tts.google_translate_say service call where {{payload}} is a string passed from the previous node, but when using retry.action (the screenshot below) it errors out.

@amitfin, since you're not a Node Red user I don't expect a resolution, just wanted to point it out.


Make authenticator and authorizer configurable

C* and DSE DTESTs sometimes use specific authenticator and authorizer settings in cassandra.yaml. Stargate currently does not allow configuring these settings, therefore running some of the DTESTS against Stargate is not meaningful (i.e. the tests always fail with something like "unexpected error message").

Looks like @tomekl007 got half way there with https://github.com/stargate/stargate/pull/725

No specific current plans, closing: if needed in future let's re-file with concrete plan, use case(s).

won't fix