[Bug] The initial value of property couldn't display correctly on xct:BadgeView  when using data binding.

Description
The initial value of the binding property couldn't display correctly on xct:BadgeView  when using data binding. But after changing the value of the binding property, the UI could update correctly on  xct:BadgeView.
Stack Trace

Link to Reproduction Sample

Steps to Reproduce

create a  xamarin forms demo
copy the following code to this demo:

 public class MyViewModel: INotifyPropertyChanged
   {
       public ICommand TestCommmand => new Command(TestMethod);

       private void TestMethod(object obj)
       {
           NotificationsNumber = 168;
       }

       private int _notificationsNumber = 0;

       public int NotificationsNumber
       {
           get => _notificationsNumber;
           set => SetProperty(ref _notificationsNumber, value);
       }

       public MyViewModel() {

           NotificationsNumber = 0;
       }

       bool SetProperty<T>(ref T storage, T value, [CallerMemberName] string propertyName = null)
       {
           if (Object.Equals(storage, value))

               return false;

           storage = value;

           OnPropertyChanged(propertyName);

           return true;

       }

       protected void OnPropertyChanged([CallerMemberName] string propertyName = null)
       {
           PropertyChanged?.Invoke(this, new PropertyChangedEventArgs(propertyName));
       }

       public event PropertyChangedEventHandler PropertyChanged;
   }

MainPage.xaml
  <?xml version="1.0" encoding="utf-8" ?>
<ContentPage xmlns="http://xamarin.com/schemas/2014/forms"
             xmlns:x="http://schemas.microsoft.com/winfx/2009/xaml"
             xmlns:xct="http://xamarin.com/schemas/2020/toolkit" xmlns:formapp826="clr-namespace:FormApp826"
             x:Class="FormApp826.MainPage">

    <ContentPage.BindingContext>
        <formapp826:MyViewModel></formapp826:MyViewModel>
    </ContentPage.BindingContext>

    <StackLayout Padding="10">
        <Label  Text="{Binding NotificationsNumber}" HorizontalOptions="Center" FontSize="Header"   />

        <xct:BadgeView

            Margin="10"
            BackgroundColor="Red"
            FontAttributes="Bold"
            FontSize="Medium"
            TextColor="White"
            Text="{Binding NotificationsNumber}">
            <Label
                Text="This label has a badge in the top-right"/>
        </xct:BadgeView>

        <Button  Text="change value" Command="{Binding TestCommmand}"  />

    </StackLayout>

</ContentPage>

deploy the app to a android device, the result is as follows:

We could find that the first Label could   display NotificationsNumber correctly(0) ,but for the   xct:BadgeView  ,the value of NotificationsNumber  can not display correctly.
After that, we can try to click button change value,  here  we change  the value  of  NotificationsNumber, we can find that the value of NotificationsNumber could update correctly for both label and  xct:BadgeView .

Expected Behavior
The initial value of property could display correctly on xct:BadgeView  when using data binding.
Actual Behavior
The initial value of property couldn't display correctly on xct:BadgeView  when using data binding.
Basic Information

Version with issue:
Last known good version:
IDE:
Platform Target Frameworks:

iOS:
Android:
UWP:

Android Support Library Version:
Nuget Packages:
Affected Devices:

Workaround

Reproduction imagery

Have you tried with the property "AutoHide"?
As per documentation "Gets or sets if the badge should be hidden automatically when the value is "0". This is a bindable property."
If it doesn't work with that property it is probably due to a bug. I have a similar problem.

I am experiencing the exact same thing.
Making BadgeView useless. :/

I have implemented my own version based on the xamarin community badge view toolkit to solve this type of bug. I will post the code when I have time.


Auto mark passes through other options to mark

Part of https://github.com/observablehq/plot/issues/1252

Stale, too naive an approach, I’m closing; tracked by https://github.com/observablehq/plot/issues/1631.


Set up FixedPlayer and toggle button UI when clicking

It seems that Saas is not used now. But it may be useful in the future.


Add aggregateintervals function

Add new command to aggregate and export statistics over intervals with
access to a 'variants' aggregator.  Takes an interval list as input,
takes an export command (see exportvariants), and an output path.
Exposed Interval in the expr language, which has start, end,
and contains (all locus-based).
Reworked property-based testing for annotation impexes.

Resubmitted as: https://github.com/broadinstitute/hail/pull/564.


Introduce incremental structural Tests

Is your feature request related to a problem? Please describe.
Whenever you have an exercise which tasks the student to create lots of classes and may even include multiple levels of inheritance etc, you would usually use sequential testing to make this easier. This works perfectly fine, however this is most often quite annoying for the student since they first have to create all the classes and method stubs and so on. Ideally you should be able to test the structure incrementally for each class e.g. first test structure for class A then run the behaviour tests for class A.
Describe the solution you'd like
I do not want to change the way sequential tests are run, they work exactly as they should. What I'd ideally want is to expand the existing {Method | Class | ... }TestProviders. The new method would then return a Map<String, DynamicTest>, that maps the standard test method names to the actual test, which then can be run whenever you need it. You could then run the structural test per class by just putting it in the respective @BeforeEach method.
Describe alternatives you've considered
You can achieve the same by using the Dynamic{Method | Class |....} and then set boolean flags to then fail behavioural tests, but that doesn't really scale well.
Ares-Version you are using
Most recent one.
I'm up for implementing this, but I would like your opinion and feedback if this is desired or if I'm maybe overlooking something.

This would probably look something like this:
public Map<String, DynamicTest> generateTests() {
        Map<String, DynamicTest> tests = new HashMap<>();

	if (structureOracleJSON == null) {
		throw failure("The MethodTest test can only run if the structural oracle (test.json) is present. If you do not provide it, delete MethodTest.java!"); //$NON-NLS-1$
	}

	for (var i = 0; i < structureOracleJSON.length(); i++) {
		var expectedClassJSON = structureOracleJSON.getJSONObject(i);
		// Only test the classes that have methods defined in the structure oracle.
		if (expectedClassJSON.has(JSON_PROPERTY_CLASS) && expectedClassJSON.has(JSON_PROPERTY_METHODS)) {
			var expectedClassPropertiesJSON = expectedClassJSON.getJSONObject(JSON_PROPERTY_CLASS);
			var expectedClassName = expectedClassPropertiesJSON.getString(JSON_PROPERTY_NAME);
			var expectedPackageName = expectedClassPropertiesJSON.getString(JSON_PROPERTY_PACKAGE);
			var expectedClassStructure = new ExpectedClassStructure(expectedClassName, expectedPackageName, expectedClassJSON);

			var displayName = "testMethods[" + expectedClassName + "]";
			tests.put(displayName, dynamicTest(displayName, () -> testMethods(expectedClassStructure)));
		}
	}

	if (tests.isEmpty()) {
		throw failure("No tests for methods available in the structural oracle (test.json). Either provide attributes information or delete MethodTest.java!"); //$NON-NLS-1$
	}

	return tests;
}


Update library to compile on upcoming 0.14.0 PS release

For context, see https://discourse.purescript.org/t/updating-the-ecosystem-for-upcoming-0-14-0-release/1144

remind me to look at this, since i'm not an admin of this repo


chore(package): update eslint-config-pretty to version 2.1.1

Codecov Report

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

@@          Coverage Diff          @@
##           master     #9   +/-   ##
=====================================
  Coverage     100%   100%
=====================================
  Files           6      6
  Lines          59     59
=====================================
  Hits           59     59

Continue to review full report at Codecov.

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


Extract API namespace from router at gen command

WHY
When user executes apig gen --all at application repository, API namespace always rewrite as "" (empty string).
WHAT
Extract namespace from existing router/router.go and rewrite with it.
This is another and better approach of #53 .

Great!! LGTM 🎉 🎉 🎉


Monitoring v2 - should be prevented from installing if monitoring v1 is installed

What kind of request is this (question/bug/enhancement/feature request):
bug master-head cb9e8a
Steps to reproduce (least amount of steps as possible):

enable Monitoring v1 in the downstream cluster
install Monitoring v2 via the dashboard in the same cluster ignoring the warning message on the installation page

Result:
Both Monitoring v2 and v1 are installed, but neither of them can work properly.
Monitoring v2 breaks the running monitoring v1, which is not acceptable.
Instead of just a warning, we should stop users from installing monitoring v2 if monitoring v1 is installed in the cluster.

in master-head 5a60e25 with the latest UI
here is the monitoring v2 installation page When the monitoring v1 is installed in the cluster:

after disabling the monitoring v1, the installation page is back to normal.


Still Beta?

Does the Beta notice in the README still apply? This seems to be public now and no longer relevant.

We still consider this a "beta" project where we'd like to collect community feedback before marking it ready-for-production-use.
If you have any feedback after using the Docker images, please reach out.

Wondering if this still applies: "Beta Service is confidential and you may not discuss the features, functionality, or documentation related to the Beta Service with any parties that are not authorized by AWS"


feature request: split_search by % identities

we have an option to request max_pvalue, max_evalue and min_prob. It would be nice to also have an option to request minimum pair-wise sequence identity (min_ident) which can be directly read from HHsearch output, e.g.:
No 1
>1FP3_B N-ACYL-D-GLUCOSAMINE 2-EPIMERASE FROM PORCINE KIDNEY; ALPHA/ALPHA-BARREL, N-acyl-D-glucosamine 2-epimerase, ISOMERASE; 2.0A {Sus scrofa} SCOP: a.102.1.3
Probab=96.16  E-value=0.00015  Score=60.96  Aligned_cols=290  **Identities=16%**  Similarity=0.102  Sum_probs=145.1  Template_Neff=10.900

identities highlighted by "**" in quote above

let me work on that as soon as we solved #71

Good that I have foreseen that sometimes someone might need to operate on those information that are only accessable in the alignments and not in the leading table. Thus, this feature request should be easy. PR #74 should address it.


Zoom Setting not saved between chats

Operating system: Win10 Home Vers 1607 OS Build 14393.222
Messenger for Desktop version: 2.0.1-beta

I zoom out on my chats and whenever I select a new conversation, the zoom resets to default.  Weirdly, the app seems to remember what my setting WAS, because when I hit zoom out one time on the new, default sized chat, it zooms out to one step FURTHER than where I had it set manually on the previous chat.

Same issue on Ubuntu 16.04 and messenger version: 2.0.4-beta

Should be fixed in Electron 1.6+, we need to update.


Enable OSX ARM builds again for cross compilation

Checklist

[ ] Used a personal fork of the feedstock to propose changes
[ ] Bumped the build number (if the version is unchanged)
[ ] Reset the build number to 0 (if the version changed)
[ ] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[ ] Ensured the license file is being packaged.

@conda-forge-admin please rerender

@conda-forge-admin please rerender

@conda-forge-admin please rerender

@conda-forge-admin please rerender

Thank you @Thoemi09 for looking into this!


"Method Not Allowed" for snapshot request of resolution HMDVideoResolutionType320x240

I receive the following error in my console.app from the homed process when opening the home app:
[<redacted home>/<redacted device name>/<redacted uuid>]
Accessory sent Error Domain=HMErrorDomain Code=58
"Failed with OSStatus code 200405 (Method Not Allowed)"
UserInfo={NSLocalizedDescription=Failed with OSStatus code 200405 (Method Not Allowed),
NSUnderlyingError=0x104bd6d30 {Error Domain=NSOSStatusErrorDomain
Code=200405 "Method Not Allowed"
UserInfo={NSLocalizedDescription=Method Not Allowed}}}
for snapshot request of resolution HMDVideoResolutionType320x240

And
Image snapshot returned HTTP client error:
Error Domain=HMErrorDomain Code=58 "Failed with OSStatus code 200405 (Method Not Allowed)"
UserInfo={NSLocalizedDescription=Failed with OSStatus code 200405 (Method Not Allowed),
NSUnderlyingError=0x106f387c0
{Error Domain=NSOSStatusErrorDomain Code=200405 "Method Not Allowed"
UserInfo={NSLocalizedDescription=Method Not Allowed}}}

Interestingly enough, video streaming works fine and it uses the last captured frame of the video stream as snapshot after dismissing the stream but the initial snapshot when opening the home App doesn't load.

So I'm digging around in HAPServer.js... It looks like something fails to register a listener for request-resource.
It exits _handleResource at HAPServer.js Line 1005 and I assume this means that the plugin did not register as a listener for the request-resource event.

Nevermind, I fixed it by correctly creating a CameraControl()...


Add support for MSVC 2015

This adds support for Visual Studio 2015:

Don't redefine type traits, since they're implemented in VS2015's stdlib
Change ASSERTED_EXPRESSION macro to be compiler-agnostic
Work around a constexpr initialization bug in the VS2015 beta

Interesting. Does VS 2015 support constexpr member functions? Do the tests from test_optional.hpp all pass?

@akrzemi1 It does not pass the test in the latest VS2015 RC. It throws ~80 errors instead, since it doesn't fully support constexpr yet.
http://webcompiler.cloudapp.net/ has a newer VS compiler though (which will be in the VS2015 RTM later on). Using this website it passes all but one test:
/*
 * main.cpp(1373): error C2131: expression did not evaluate to a constant
 * main.cpp(1373): note: failure was caused by non-constant arguments or reference to a non-constant symbol
 * main.cpp(1373): note: see usage of 'gi'
*/
static_assert(&gri == &*gori, "WTF");

My initial guess would be that taking the address of an object is not guaranteed to be a compile time operation which is why the static_assert isn't as well, or am I wrong? :smile:
Apart from that there are only a couple minor warnings:

warning C4456: declaration of 'XYZ' hides previous local declaration
warning C4127: conditional expression is constant
warning C4702: unreachable code

Yep, I get the same results as above with 2015 final. The problems isn't from &gi itself, it's that MSVC thinks that dereferencing a constexpr ptr-to-nonconstexpr is wrong, even if the result is only used for &.
I haven't been able to convince myself whether this is MSVC being too restrictive or gcc/clang being too permissive, so I started a thread here: https://groups.google.com/a/isocpp.org/forum/#!topic/std-discussion/gsdQVv4UOHk

I was wondering if this was going to get merged at some point. AFAIK Visual Studio 2015 Update 1 will still not have an implementation of  optional. I have rolled my own but would love to use this implementation instead.

Thanks for the interest in this implementation. I have implemented these changes (with modification). I still don't see how they help. Unit tests fail on VC++ 2015, most likely due to incomplete implementation of constexpr member functions.
If you do not mind the lack of constexpr operations why not use boost::optional?


Fix EnsembleForecaster and AutoEnsembleForecaster docstring example

@fkiraly pls approve and merge

Now merged - thanks everyone! :)


CAMEL-8584 Circuit breaker does not honour halfOpenAfter period

Updated unit test and fix for CAMEL-8584 Circuit breaker does not honour halfOpenAfter period

Thanks for this PR the code has been merged. Do you mind closing this?


Please add San Leandro CA PD

Email addresses<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>Subject line: SLPD Budget Reallocation 2020
Message:
Dear Mayor Cutter and City Council Members,
My name is [YOUR NAME] and I am a San Leandro (resident or business owner) writing to urge you to defund SLPD.
SLPD takes an enormous share of the city’s general fund, and that percentage has risen exponentially for the last two decades, taking away desperately needed resources from essential city programs and services. The City of San Leandro transferred $20 million to the SLPD headquarter fund in 2018-2019; money that could be used much more beneficially if inserted directly into the community. The investment in policing has not made us safer – SLPD remains an embarrassment to the city and a lethal threat to San Leandro’s Black and Brown communities, while increased police spending shows no correlation to decreasing crime levels. With current finances so tight, it is clear that we must defund the police.
As San Leandro residents, we therefore demand that you take immediate action to ensure the following:

Reduce SLPD’s allocation from the General Fund by 50%
Discontinue use of general fund dollars to pay for settlements due to police murder, misconduct, and negligence
Reallocate money from SLPD into housing, jobs, youth programs, restorative justice, and mental health workers to keep the community safe.
San Leandro can not wait any longer for a budget that meets the needs of its residents. The only way to achieve this is to take immediate steps to Defund SLPD.

Thank you,
[YOUR NAME]

If there is still availability to offer suggestions, these are mine:
Message:
Dear Mayor Pauline Russo Cutter and City Council Members,
My name is [YOUR NAME] and I am a San Leandro [RESIDENT/BUSINESS OWNER] writing to urge you to defund San Leandro PD.
SLPD takes an enormous share of the city’s general fund (35%), and that percentage has risen exponentially for the last two decades, taking away desperately needed resources from essential city programs and services. In the 2018-2019 Biennial Budget, the City of San Leandro allocated roughly $39 million dollars to our police system, an inordinate 34.7% of our total budget. This is compared to just $6.5 million allocated towards community development (5.7% of the budget) and recreation & human services (4.6% of the budget). The investment in policing has not made us safer – SLPD remains an embarrassment to the city and a lethal threat to San Leandro’s Black and Brown communities, while increased police spending has not returned a decrease in crime rate. With fiscal responsibility being a primary goal of this council, it is clear that you must defund the police and reallocate resources into community programs.
The projected budget for 2020-2021 for San Leandro PD still sits at 35% of the general budget, at $42,000,485, while community development is projected at 6% of the general fund, recreation & human services at 4% of the budget, and public works also at 6%.
Within the community development budget, $307,662 was spent on the City Innovation Division for the 2018-2019 budget (8% of budget) and has been slashed to a projected $3,434 for 2019-2020 & 2020-2021 (0.0005% of community development budget for 2020-2021). Additionally within the community development budget for 2019-2020, $326,264 was spent for housing services (4%), and $429,568 for code compliance (6%). The projected budgets for 2020-2021 come to $341,211 for housing services (5% of budget) and $458,203 for code compliance (7% of the budget).
As San Leandro residents, we therefore demand that you take immediate action to ensure the following:

Reduce SLPD’s allocation from the General Fund by 50%
Discontinue use of general fund dollars to pay for settlements due to police murder, misconduct, and negligence
Reallocate money from SLPD into housing, jobs, youth programs, restorative justice, and mental 4. health workers to keep the community safe.

San Leandro cannot wait any longer for a budget that meets the needs of its residents. The only way to achieve this is to take immediate steps to Defund SLPD.
Thank you,
[YOUR NAME]
[YOUR ADDRESS]
[YOUR EMAIL]
[YOUR PHONE]


[docs] Fix typo

Edit: "generates the following class names you that can override:" -> "generates the following class names that you can override:"

[ ] I have followed (at least) the PR section of the contributing guide.

@rick-software It's a great first pull request on Material-UI 👌🏻. Thank you for working on it!


fixed folder structure and gitattributes for asset library

I wanted to test out your addon and couldn't so I fixed the folder structure and gitattributes file for you as per godot documented guidelines on posting to the asset library:
https://docs.godotengine.org/en/stable/community/asset_library/submitting_to_assetlib.html#recommendations
If you accept and update your asset library submission, people will actually be able to use your addon without it destroying their projects now.

You're absolutely right. Thank you for bringing it to my attention.


Option to skip push in login

Useful if previous step modifies workdir but login to the machine restores it.

One problem though: If push never happened then login tries to cd into nonexistent directory. Currently there is no way of indicating that push has happened.

Marking as draft until I find workaround to sometimes missing tree directory.


GEODE-9229: Avoid possible duplicate events sent with group-transacti…

…on-events
When group-transaction-events is set to true
the SerialGatewaySenderQueue class stores those events
peeked from the queue to complete transactions
for a batch inside the extraPeekedIds member. This is done
so that, when new events are peeked from the queue
by peekAhead() to be put in a batch, the
events in extraPeekedIds are skipped. Otherwise
those events will be sent several times.
When the remove method of SerialGatewaySenderQueue
is called, the peekedIds member and the head key
of the queue are updated and events from extraPeekedIds
are removed if necessary.
Given that the peekAhead() method first gets the
currentKey (that depends on the peekedIds and the headkey)
and then checks if the event to be removed is in extraPeekedIds,
it could be that in between those two calls,
remove() is called from another thread, causing that
peekAhead() returns an event that
belonged to extraPeekedIds when the currentKey was
obtained but was deleted from extraPeekedIds by remove()
before it was checked that the event belonged to
extraPeekedIds.
This provokes that an event already peeked is returned
again by peekAhead() leading to duplicate events
sent by the gateway sender.
The solution proposed to solve this problem is to make
synchronized the peekAhead() function when group-transaction-events
is set given that remove() is already synchronized. This will
prevent the race condition described above.
Thank you for submitting a contribution to Apache Geode.
In order to streamline the review of the contribution we ask you
to ensure the following steps have been taken:
For all changes:

[ ] Is there a JIRA ticket associated with this PR? Is it referenced in the commit message?

[ ] Has your PR been rebased against the latest commit within the target branch (typically develop)?

[ ] Is your initial contribution a single, squashed commit?

[ ] Does gradlew build run cleanly?

[ ] Have you written or updated unit tests to verify your changes?

[ ] If adding new dependencies to the code, are these dependencies licensed in a way that is compatible for inclusion under ASF 2.0?

Note:
Please ensure that once the PR is submitted, check Concourse for build issues and
submit an update to your PR as soon as possible. If you need help, please send an
email to<EMAIL_ADDRESS>

Kind reminder, code owners reviews still pending.

The original solution for GEODE-8971 is reverted so this PR is not needed anymore given that it added the case for which a fix is provided in this PR.
Also, the solution proposed in this PR was not valid. Making peekAhead() synchronized would not fix the possible issue because what was required was that remove() was synchronized and that already was so.


Webpack config not finding nested node_modules

Bug Report or Feature Request (mark with an x)
- [x] bug report -> please search issues before submitting
- [ ] feature request

Versions.
@angular/cli: 1.0.2
node: 6.10.1
os: darwin x64
@angular/common: 4.1.1
@angular/compiler: 4.1.1
@angular/core: 4.1.1
@angular/forms: 4.1.1
@angular/http: 4.1.1
@angular/platform-browser: 4.1.1
@angular/platform-browser-dynamic: 4.1.1
@angular/router: 4.1.1
@angular/cli: 1.0.2
@angular/compiler-cli: 4.1.1

Repro steps.

Clone the repo: https://github.com/jshcrowthe/ng-cli-nested-node-modules.git
Enter the repo dir
Run npm install
Run ng serve
Observe error from ng serve command

The log given by the failure.
ERROR in ./~/firebase/app/shared_promise.js
Module not found: Error: Can't resolve 'promise-polyfill' in '$PATH_TO_REPO/nested-node-module/node_modules/firebase/app'
 @ ./~/firebase/app/shared_promise.js 22:35-62
 @ ./~/firebase/app/firebase_app.js
 @ ./~/firebase/app.js
 @ ./src/app/app.component.ts
 @ ./src/app/app.module.ts
 @ ./src/main.ts
 @ multi webpack-dev-server/client?http://localhost:4200 ./src/main.ts

Desired functionality.
Webpack should compile this module and it's nested children correctly
Mention any other details that might be useful.
n/a

Unfortunately, yes, it currently doesn't work but a solution is in progress.
Note that everything works with yarn as it will hoist the dependencies to the top level.
Also, I would highly suggest using https://github.com/angular/angularfire2 which makes integrating firebase into an Angular app quite quick and easy.

@hansl this is the problem we talked about yesterday with Firebase.

Enabling the use of the Node.js module resolution algorithm (a minor code change) will ensure the correct and proper module is used within an app.   As witnessed in this issue as well as #3023, this is a real and repeatable concern.
The change will cause issues in two main situations:

Angular libraries which do not properly use peer dependencies.
Linked libraries which have their peer dependencies installed via dev dependencies.

The first is essentially a non-issue as the libraries are improperly packaged.
The second is a concern and will cause trouble to those wishing to use linked Angular library packages in development scenarios.  However, ensuring an application can resolve and use the proper module (and allow the application to properly function) would generally outweigh the added developer inconvenience for a use case that while not uncommon is not extremely common either.  This point, however, is definitely one for discussion.

IIRC there are already some issues with linked libraries anyway, and there is a pretty good solution in https://github.com/angular/angular-cli/issues/3854#issuecomment-274344771.
So if those are the only two problematic cases, I'd say we should switch over to node module resolution.


Revert "Replace deprecated imp module with importlib"

This reverts commit db8d0b3a14de97ac9ecc571f75c1bc2991455b07.
Using importlib.find_spec() is not safe at runtime as it can import
modules which will cause side effects within environments.
From the docs:
https://docs.python.org/3/library/importlib.html#importlib.util.find_spec

If name is for a submodule (contains a dot), the parent module is
automatically imported.

This has caused issues in some projects, including python-ldap which
includes the word "setup.cfg" causing its "setup.py" file to be imported
which eventually raises a SystemExit error. A test was written to catch
this case.
A deprecation warning is preferable over system level side effects.

Should we release this today?

Yeah, I think that would be a good idea. The unintended side effects could be nasty in some environments. Thanks.

You should have permission to push a signed tag to trigger a release. Do you want to try it out and make sure it works? I'll have time in a few hours to do it, if you don't have time now.

I probably will not have time during the work day, so please feel free to go ahead. If the release isn't out by tonight, perhaps I'll give it a shot. Thanks.


scripts: add exception handler when parsing input for syscalls fails

If an input file fails to decode due to a unicode parsing error this will dump the offending file as part of the error message

Please review the Contribution Guidelines and re-format the commit messages.


[Regression]: NS_ERROR_NET_INTERRUPT errors with version 1.45.0 (no errors with 1.44.0)

Last Good Version
1.44.0
First Bad Version
1.45.0
Steps to reproduce

git clone https://github.com/BoD/repro-playwright-network-interrupt.git
./gradlew run

Expected behavior
With 1.44.0
The page that appears has a blue panel on the left and should look like this:

Also if you edit the url to remove everything except the domain name, and go to it, you should see this:

If you show the network calls while doing this, they all work normally:

Actual behavior
With 1.45.0
The page that appears doesn't have the blue left panel, and looks like this:

If you edit the url to remove everything except the domain name and go to it the page doesn't load and show a network error instead:

If you show the network calls while doing this, we can see a a NS_ERROR_NET_INTERRUPT error on the first request and no further requests:

Additional context

This also doesn't work with the latest Playwright version
I'm noticing v1.44.0 is using Firefox 125.0.1, whereas v1.45.0 is using Firefox 127.0
Trying to load these pages with the latest version of a "regular Firefox" (not within Playwright) works - you can try yourself to go to https://suiviconso.edf.fr/comprendre with Firefox, you should land on the page that looks like the first screenshot (blue left panel).

Environment

System: ARM Mac
OS: MacOS Sonoma 14.7.1

Looks like the web site serves different content to the browser controlled by automation. Do you have an example with the page whose source you control which exposes this behavior?

Thanks for having a look.

Do you have an example with the page whose source you control which exposes this behavior?

I don't. I'm only using Playwright to scrape that website.
Are you saying this site is detecting the presence of Playwright, but only with recent versions? Is Playwright setting a special User-Agent maybe? And if yes can I override it?

Are you saying this site is detecting the presence of Playwright, but only with recent versions?

Yes, that was my hunch.

Is Playwright setting a special User-Agent maybe? And if yes can I override it?

You can override user-agent in the context options: https://playwright.dev/java/docs/emulation#user-agent

I don't. I'm only using Playwright to scrape that website.

Playwright's primary focus is on testing scenarios, where you have some control over the web app under test. Given that this is a third-party app that exposes the issue and it's unclear if it's an intended behavior, we can't help with it. If you manage to narrow the problem down to a simple repro, feel free to open another request.

Actually, it looks like the site in question is broken with Firefox 127, independently of Playwright (no idea why).
I does work with the latest Firefox 134 though, so I'll just wait until Playwright uses a more recent version of Firefox before upgrading.


Add gear set sections

Add automatic gear set section breakout with a toggle to enable or disable it.

Also, I just did LFR on that toon and got some new pieces.  I set them up in my gear sets, but the groups would not show in the bag frame.  /reload did not help.  I had to log out and back in to get the groups to show up.

So it's a bit deceptive, but they are sorting correctly.  The columns sort left to right by the number of sections, so if you look at the first column, 2 sections down, and the second column, 2 sections down, you'll see they sort correctly (compaction non-withstanding). I do this because it keeps sorting deterministic -- if I did it based on the sizes of sections, then sections would dramatically shift based on the size of the section. The non-deterministic sorting is what AdiBags did, and was a chief complaint of folks from all sorts of feedback channels.
That being said, I do plan to add a tighter compaction algorithm that sorts things as closely as possible, however that still won't promise that like sections will sort together. I could spend more time on this, but N-dimensional sorting is a non-trivial bit of work that, while can be done, would take up more time than I care to spend right now. This is probably the single most complex topic of the addon, and I'll revisit it down the line, as there's a multitude of opinions to take into account here.
For your second point, things not showing up in a gear set -- are you saying you added a new item to a gear set, and it didn't show up in the inventory, or you made a new gear set, and it didn't show up? It not working without a full quit and restart is very, very odd -- a /reload completely eliminates all state and addons can't hold onto state in memory across reloads, so that's very surprising. If possible, a reproduction case would be extremely useful here.
Thanks!

This seems to be implemented as intended.


Keep internal logs of actions in converter objects

sneakers-the-rat's fork of the repo diverges in a number of ways. Though they used it for a different purpose, it's still worth bringing up as a point of discussion - to keep an internal log of what actions/results a NWBConverter object has undergone.
@luiztauffer Thoughts?
Is your feature request related to a problem? Please describe.
Mostly useful for debugging purposes, to be able to have the user give us the output of printing the internal class logs to see exactly how they were  using it through their conversion process.
Describe the solution you'd like
An example snippet of what this might look like
       def run_conversion(self, ..., **kwargs):
       ....

        # prepare dict for storage
        full_spec = dict(
            "metadata"=metadata,
            "save_to_file"=save_to_file,
            ...,
            'kwargs': dict(kwargs)
        )

        spec_key = "run_conversion"
        if spec_key not in self._spec.keys():
            self._spec[spec_key] = [full_spec]
        else:
            self._spec[spec_key].append(full_spec)

        ...

Describe alternatives you've considered
The alternative for helping a user debug their conversion is simply to have them share their code and possibly data as well.
Checklist

[X] Have you ensured the feature or change was not already reported?
[X] Have you included a brief and descriptive title?
[X] Have you included a clear description of the problem you are trying to solve?
[X] Have you included a minimal code snippet that reproduces the issue you are encountering?
[X] Have you checked our Contributing document?

I like that idea, we could use python logging for that.
Maybe even wrap run_conversion within a try/except to catch errors on the logger

@CodyCBakerPhD I think he's referring to Python's standard logging library

But with logging optional argument that we will inform users to set.


feat(2023-11-16): Update SDK to use API generated on 2023-11-16

:tada: This PR is included in version 0.18.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


The [pass] parameter to $timeout is not transferred to the callback function

The $timeout reference says it should pass the [pass] parameter to the callback function. This does not seem to be working.

Which version of angular are you using?

This feature was added in 1.4.0, maybe it's just an issue of looking at the incorrect version of the docs. Please clarify if this is not the case

Closing due to the lack of details.


ci: use setup-node for caching

This change: (check at least one)

[ ] Adds a new feature
[ ] Fixes a bug
[x] Improves maintainability
[ ] Improves documentation
[ ] Is a release activity

Is this a breaking change? (check one)

[ ] Yes
[x] No

Is the: (complete all)

[x] Title of this pull request clear, concise, and indicative of the issue number it addresses, if any?
[ ] Test suite(s) passing?
[ ] Code coverage maximal?
[ ] Changeset added?

What does this change address?
Use setup-node to cache dependencies.
How does this change work?

Update actions/setup-node to cache dependencies in place of actions/cache

This feature only caches the .yarn directory and not node_modules


Module always sends same IP address when behind varnish

Description
We have a setup of Magento running on Nginx, which is behind varnish cache. Since varnish cache will act as a proxy to Nginx it will register the IP address of the server as the IP address which has placed the order. The real IP address of the customer is forward and stored in Magento as "X-Forwarded-For".
The problem
The module gets the IP address from the order with $order->getIpAddress() this will always return the IP address of the server and not of the client.
The solution
Check of there's a value stored in $order->getXForwardedFor() first and send that value as IP address to Buckaroo when it's available. It stores the real client IP address. If no value is present fallback to the IP address stored in the order.

Hi @dverkade !
As we can see it's not a new behavior and it was from scratch. But you are right and this fix may be done. So we will consider to include it to further releases ASAP. Thanks for informing !

released in https://github.com/buckaroo-it/Magento2/releases/tag/v1.24.0-RC1

Full release 1.24.0 is done.


[SD Correction] Fuzz Testing

Provide the location of the issue
Section A.1.4
What is the correction request for the cPP? Please describe.
Per RfI202200, Fuzz testing should be made optional.

Text relating to fuzz testing updated as per Rfi202200

Issues resolved with PR #209


asyncValidationErrors and getValidationResult are inconsistent

asyncValidationErrors will resolve if there is no error, and the error will show up in the error fn or a catch block of the promise,
where as getValidationResult will return the error in the then() block, not in the error fn or the catch block of the promise.
I assume that getValidationResult will return results, other than errors? if this is the case, wouldnt it make more sense to resolve the promise if validation passes, and reject the error in the error handler or the catch block?
also asyncValidationErrors will reject the errors into the error function or the catch block, which makes sense, but it should also resolve passed in data successfully, instead of returning nothing.
alot of time these validation are used in a functional pipelines, where you would need to validate the params/body/query, and see if its valid, then pass the data to another function in the pipeline, by not returning anything from the validator function, this would forces the program to wrap the entire execting block inside the then resolve block, since you can use the param value until it passes the validation function.

Hi @replicat0r, thanks for using express-validator.
Please see duplicate issue #326 for related discussion on some of your points.


Upgrade to Tensorflow v2.2.0

Please upgrade from v2.2.0-rc2 to v2.2.0 final. I am hoping this might fix issue #219.
I may try this. If so I will submit a PR.

You should just be able to change the tag in the CMakeLists.txt file and it will change the version used.


confusing documentation

the new documentation doesn't seem to have the most important parts of a mod api documentation page: methods that the modder can call, and methods the modder can create to be called on certain events

I'm not sure I follow. The documentation has information on all the hooks we've made. Everything vanilla related is easily inferred from the method signatures Visual Studio will autocomplete for you. There is also a section in the wiki about useful vanilla methods.

I'm not sure what you mean either, @Tyfyter, the new documentation contains all methods a modder can call. You can use the search bar to find what you're looking for. Some documentation may still be missing at this point in time, but methods should still be there.


Path required in cmfe filenames?

cqid: VisIt00008047cqsubmitter: Brad Whitlockcqsubmitdate: 05/18/07 The cmfe expressions seem to need the database path. VisIt stores filenames using the entire path internally so perhaps the lack of path in the filename prevents cmfe from finding the right database.  Open A**.silo database2. A_minus_B = pressure - pos_cmfe(<B**.silo database:pressure>, quadmesh, 1.)3. Create Pseudocolor of A_minus_B4. Draw Pseudocolor: ()viewer: The file "B*.silo database" does not exist. This expression works and is what we want. It must need the path.pressure - pos_cmfe(</home/whitlocb/data/richard/B*.silo database:pressure>, quadmesh, 1.)
-----------------------REDMINE MIGRATION-----------------------
This ticket was migrated from Redmine. As such, not all
information was able to be captured in the transition. Below is
a complete record of the original redmine ticket.
Ticket number: 34
Status: Pending
Project: VisIt
Tracker: Bug
Priority: Normal
Subject: Path required in cmfe filenames?
Assigned to: -
Category: -
Target version: -
Author: Cyrus Harrison
Start:
Due date:
% Done: 0%
Estimated time:
Created: 06/21/2010 07:15 pm
Updated:
Likelihood: 4 - Common
Severity: 3 - Major Irritation
Found in version: < 1.12
Impact:
Expected Use:
OS: All
Support Group: DOE/ASC
Description:
cqid: VisIt00008047cqsubmitter: Brad Whitlockcqsubmitdate: 05/18/07 The cmfe expressions seem to need the database path. VisIt stores filenames using the entire path internally so perhaps the lack of path in the filename prevents cmfe from finding the right database.  Open A**.silo database2. A_minus_B = pressure - pos_cmfe(<B**.silo database:pressure>, quadmesh, 1.)3. Create Pseudocolor of A_minus_B4. Draw Pseudocolor: ()viewer: The file "B*.silo database" does not exist. This expression works and is what we want. It must need the path.pressure - pos_cmfe(</home/whitlocb/data/richard/B*.silo database:pressure>, quadmesh, 1.)
Comments:

reduced likelihood and impact because wizard takes care of this issue.
@BradWhitlock ... should we close?


unable to access internet from ueransim-ue pod

I used free5gc-ulcl deployment
the association setup was successful between the smf and up1,up2,upf-b
also pdu session establishment and modification was successful

but i am unable to ping google or <IP_ADDRESS> using the ue's ip

Hi @Satyam-git-hub ,
You have to UPFs in chain in this case. Did you figured out which UPF doesn't forward ping packets?
Could you share eUPF tracepipe logs?
And as a blind hit. Routing between 2 eUPF may be not possible without manual pinging from I-UPF to UPF in order to update ARP table. See #465 for solution.
-- BR, Alex


include errata in jendl5

This PR adresses this issue, the problem with jendl-5.0 generation was that the downloaded data did not include errata published here by JAEA.
This PR adds errata URLs to the urls.py file, these errata are all downloaded, extracted, and the resulting endf files moved to the correct location when running generate_jendl.py -r  5.0.
NJOY treatment then continues on correctly to create the hdf5 data files.

Nice fix many thanks for this. I shall merge and make a new release


glfs_init ()  blocks?

I created it with gluster-block and it will block here.

I think you might have opened this by mistake or forgot to add the information about the bug. There just seems to be a sentence about something blocking but no info on where.
Closing this now, but reopen if you have additional info about what the issue you are hitting is.


[WIP]output quickstart when show config template

Signed-off-by: Thor-wl<EMAIL_ADDRESS>Pre-Checklist
Note: please complete ALL items in the following checklist.

[ ] I have read through the CONTRIBUTING.md documentation.
[ ] My code has the necessary comments and documentation (if needed).
[ ] I have added relevant tests

Description

Related Issues

New Behavior (screenshots if needed)

Hi @Thor-wl, thanks for your contribution! Please join CNCF's Slack, and join our channel in CNCF Slack!
Here are a few useful things for you to continue your open-source journey:

If you are searching for something else to work on, please see "find an issue".
Check out our new contributor guide and the "Developer Guide".
Read our blog; they are on Medium, too.

DevStream community follows the conventional commits specification. Please read our doc and update your commit message.


2.1.0 NullPointerException when loading a large file (70 columns, 250,000 rows) with a large number of blank cells using sst cache.

@texhnolyzze would you be able to share your xlsx file?
I have a fork and I use a different SST implementation. If you can't share your xlsx, could you try it with my lib to see if it handles your file ok?


BUG: multiple warnings are displayed when running unit tests

Solution: make tests more strict. They should not allow wernings.

Fix them and install jest-fail-on-console
https://www.npmjs.com/package/jest-fail-on-console


kube-state-metrics publishes on 8080 by default, but datadog-agent looks on 8081

I'm using the DataDog agent (v6.1.0) and kube-state-metrics (v1.2.0) on GKE (v1.8.9). I'm using the Kubernetes manifests provided by both projects with minimal modifications.
I see this error in my logs:
[ AGENT ] 2018-03-26 13:14:23 UTC | ERROR | (runner.go:276 in work) | Error running check kubernetes_state: [{"message": "HTTPConnectionPool(host='<IP_ADDRESS>', port=8081): Max retries exceeded with url: /metrics (Caused by NewConnectionError('<urllib3.connection.HTTPConnection object at 0x7efcc08619d0>: Failed to establish a new connection: [Errno 111] Connection refused',))", "traceback": "Traceback (most recent call last):\n File \"/opt/datadog-agent/bin/agent/dist/checks/__init__.py\", line 332, in run\n self.check(copy.deepcopy(self.instances[0]))\n File \"/opt/datadog-agent/embedded/lib/python2.7/site-packages/datadog_checks/kubernetes_state/kubernetes_state.py\", line 196, in check\n self.process(endpoint, send_histograms_buckets=send_buckets, instance=instance)\n File \"/opt/datadog-agent/embedded/lib/python2.7/site-packages/datadog_checks/checks/prometheus/mixins.py\", line 350, in process\n for metric in self.scrape_metrics(endpoint):\n File \"/opt/datadog-agent/embedded/lib/python2.7/site-packages/datadog_checks/checks/prometheus/mixins.py\", line 314, in scrape_metrics\n response = self.poll(endpoint)\n File \"/opt/datadog-agent/embedded/lib/python2.7/site-packages/datadog_checks/checks/prometheus/mixins.py\", line 467, in poll\n response = requests.get(endpoint, headers=headers, stream=True, timeout=1, cert=cert, verify=verify)\n File \"/opt/datadog-agent/embedded/lib/python2.7/site-packages/requests/api.py\", line 72, in get\n return request('get', url, params=params, **kwargs)\n File \"/opt/datadog-agent/embedded/lib/python2.7/site-packages/requests/api.py\", line 58, in request\n return session.request(method=method, url=url, **kwargs)\n File \"/opt/datadog-agent/embedded/lib/python2.7/site-packages/requests/sessions.py\", line 508, in request\n resp = self.send(prep, **send_kwargs)\n File \"/opt/datadog-agent/embedded/lib/python2.7/site-packages/requests/sessions.py\", line 618, in send\n r = adapter.send(request, **kwargs)\n File \"/opt/datadog-agent/embedded/lib/python2.7/site-packages/requests/adapters.py\", line 508, in send\n raise ConnectionError(e, request=request)\nConnectionError: HTTPConnectionPool(host='<IP_ADDRESS>', port=8081): Max retries exceeded with url: /metrics (Caused by NewConnectionError('<urllib3.connection.HTTPConnection object at 0x7efcc08619d0>: Failed to establish a new connection: [Errno 111] Connection refused',))\n"}]

It looks like the check is trying to scrape the /metrics URL from port 8081. However kube-state-metrics exposes that URL on port 8080 by default. (The IP address of the container is correct.)
I've spent a while reading your docs and spelunking in a couple of your codebases and I can't work out how that default gets set or how to override it.
So I suppose I have a couple of questions:

How do I change the port that the check uses?
Is there some reason for it not to use the default port by default?

Thanks
-Ben

Sorry for spamming you today. :-)

@benbc thanks for the report, it seems that the KSM project recently added the 8081 port in the exposed port list and this messing up with the autodiscovery template, because in case of multiple exposed port, we'll take the highest port number.
I've made a PR to fix the official AD template https://github.com/DataDog/integrations-core/pull/1308 in the meanwhile you can either edit the AD template or try using annotations on the KSM pod

@mfpierre Thank you.
I'm sure I can use the annotation approach. The diff for your fix will help with that. I was unsure, reading your docs, which pod the annotation should go on (datadog-agent or kube-state-metrics), but your comment above makes that clear, so I now I have everything I need.
What is the easiest way for me to trace your linked fix in https://github.com/DataDog/integrations-core to a published version of the datadog/datadog-agent Docker image?

@benbc just merged the PR should go out with the next agent release


Fix IP leak on main interface when using eni IPAM

We noticed that the operator allocates IP on the main interface (index:0) even if first-interface is et to 1
After analyzing the logs and looking at the code, we noticed that the pointer always get the address of the last ENI in the map when iterating over it (and the last interface in the map can be the main one). Relevant code here: https://github.com/cilium/cilium/blob/06bce43ab84170ea281d6741f5adc31a91be6c46/pkg/aws/eni/node.go#L473
This patch simply uses the key in the map instead of a pointer. Initial testing on our side looks good without any leak after a few scale up events. We can use another approach if you prefer of course.
cc @tgraf

This change is

test-me-please

Coverage increased (+0.04%) to 45.523% when pulling e666a46b2eb2e7472fb1e3322dba0b326fa1c04c on DataDog:lbernail/fix-ip-leak into 19c4b54f1d40e3a7fd139be3c8624e2aa030c499 on cilium:master.


Create ERC20 Wrapper Factory and Wrappers for ParaAugurs

In order to have a consistent set of cost effective ERC20s for use in various pools we are going to add the machinery all ParaDeploys to be able to consistently create share tokens.

[x] Create WrappedShareToken that is simple ERC20
a. [x] wrap
b. [x] unwrap
c. [x] claim proceeds
[x] Create WrappedShareTokenFactory for creating ERC20s associated with a particular ParaDeploy
a. create WrappedShareTokens (for a tokenID/symbol pair)
b. wrap / unwrap / etc
c. helper functions for: publicBuyCompleteSet, publicSellCompleteSet that works on all the ERC20s?
c. Is trusted by WrappedSharedToken
[x] Integrate with AMM Factory
[ ] Update all the tests

closing


VCD: Add bastion host to terraform configs

What this PR does / why we need it:
This PR adds a bastion host for the VCD terraform provider.
Currently, we have a limitation "that there is only 1 public IP in our VDC and that cannot be used for e2e tests since it's attached to the edge gateway" as stated in https://github.com/kubermatic/kubeone/issues/2056. We have a single edge gateway that is shared within our organization on VCD. Hence, using this public IP to further expose resources like bastion VMs, kubeapi server, etc., using NAT is not ideal and can result in conflicts.
With that said, I'm not aware of the current state of our VCD infra as reportedly NSX-T has been enabled for our organization there. Having NSX-T should fix the limitation of using edge gateway's public IP and there should be some pool of public IPs that we can then use. Anyways, due to a lack of knowledge, I'll leave this up to the KubeOne maintainers, or whoever ends up working on this ticket to figure this out.
This should unblock KubeOne maintainers from continuing with https://github.com/kubermatic/kubeone/issues/3228, https://github.com/kubermatic/kubeone/issues/2056, and https://github.com/kubermatic/kubeone/issues/2057, etc., without being dependent on me.
I've also added a sample tfvars file that can be later used for e2e tests. This documents the values that can be used for this provider and can be later used in the E2E tests as well.
Which issue(s) this PR fixes:

Fixes #
What type of PR is this?

/kind feature
Special notes for your reviewer:
Does this PR introduce a user-facing change? Then add your Release Note here:

VCD: Add bastion host support to terraform configs

Documentation:

NONE

/approve
/lgtm

/cherrypick release/v1.8


[Audio]: Kevin Macleod - Movement Proposition

Check if this audio is already added to RetroStudio.

[X] The audio is not already in game.

Is this for the main-menu playlist?

[x] Add to main-menu playlist.

Is this song from an old Roblox era or a sound effect?

[X] This song is from old Roblox or is a sound effect.

A link to the audio on Roblox
https://www.roblox.com/library/12216709656/
A link to the audio file to download.
https://incompetech.com/music/royalty-free/mp3-royaltyfree/Movement Proposition.mp3
Notes
This is the boss music from Egg Hunt 2013.

12909903896


Create 0000-01-02-twingenious.md

I have made some changes to the project for learning purposes

Hi im learning to use Github for manage open source projects
The text above explains the reason why i am learning to use Github


removed root from group docker

Don't add root to the "docker" group since it's unnecessary and Proxmox doesn't like root in any other groups.

Thanks!


Help building miryoku for v1

Hi, I've been trying to build miryoku to be able to use QWERTY with it, but so far to no avail. I'm able to build the default cheapino firmware just fine, then I add the miryoku branch and merge tompi/cheapino. After that I run qmk compile -kb cheapino -km manna-harbour_miryoku but I get Invalid keymap argument., then I try make cheapino:manna-harbour_miryoku and get
QMK Firmware 0.20.6
Making cheapino with keymap default and target manna-harbour_miryoku                           [ERRORS]
make[1]: *** No rule to make target 'manna-harbour_miryoku'.  Stop.
Making cheapino with keymap default_split_3x5_3 and target manna-harbour_miryoku               [ERRORS]
make[1]: *** No rule to make target 'manna-harbour_miryoku'.  Stop.
Making cheapino with keymap lars and target manna-harbour_miryoku                              [ERRORS]
make[1]: *** No rule to make target 'manna-harbour_miryoku'.  Stop.
Making cheapino with keymap tompi and target manna-harbour_miryoku                             [ERRORS]
make[1]: *** No rule to make target 'manna-harbour_miryoku'.  Stop.
Make finished with errors
make: *** [Makefile:415: cheapino:manna-harbour_miryoku] Error 1

Other than that I tried forking miryoku_qmk to use github actions and add tompi/qmk_firmware/cheapino to Merge QMK Branches, but that seems to fail because of merge conflicts. I'm new to qmk so I'm not sure what else to try. P.S. let me know if it's better to ask in the miryoku repo.

Hi, after some more research I was able to build miryoku successfully. I found https://github.com/manna-harbour/miryoku/discussions/287 and did the following:
# first follow the cheapino firmware guide
git remote add miryoku_qmk https://github.com/manna-harbour/miryoku_qmk
git fetch miryoku_qmk
git checkout miryoku_qmk/miryoku users/manna-harbour_miryoku
git checkout miryoku_qmk/miryoku layouts/community/split_3x5_3/manna-harbour_miryoku

after that, trying to build miryoku seems to work but I get an error about having to remove IGNORE_MOD_TAP_INTERRUPT since it is now the default. After doing that, the build is successful and everything works as expected.
I'll still wait to see if manna-harbour can confirm that this is the right way to go about it, but for now I'm happy of the result. :)

Hey, thats great!
Feel free to send a pull request against my qmk fork, otherwise I will replicate your approach and get it in.
P.S. Miryoku is colemak, right? Did you have to edit the keymap manually to change it to qwerty?

Error on my part, I meant to say you need to delete IGNORE_MOD_TAP_INTERRUPT from users/manna-harbour_miryoku/config.h (line 15), so it's not a problem with the cheapino qmk fork.

P.S. Miryoku is colemak, right? Did you have to edit the keymap manually to change it to qwerty?

There are some options you can set for the build command, for example to get the QWERTY layout you can use qmk compile -c -kb cheapino -km manna-harbour_miryoku -e MIRYOKU_ALPHAS=QWERTY https://github.com/manna-harbour/miryoku_qmk/tree/miryoku/users/manna-harbour_miryoku#qmk

Thank you @bunnken
Added Miryoku prebuilds to release section, and a miryoku branch in my qmk fork

The mess with https://github.com/manna-harbour/miryoku/discussions/287 will be sorted out eventually, then you'll be able to build cheapino using the workflow merge feature again. Thanks @bunnken and @tompi for fixing things here for now!


[Spaces] Update docker tutorial to be compatible with the Dev Mode

TL;DR
Update the tutorial for Docker Spaces to make them compatible with the Dev Mode requirements

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update.

@radames the important changes are:

Having the app live under the /app directory
Create a user with ID 1000 and a home directory, even for the "basic" image


Can the PrimaryButtons on the HamburgerMenu be filled via data binding?

Can the PrimaryButtons on the HamburgerMenu be filled via data binding?
From Shell,xaml
Controls:HamburgerMenu.PrimaryButtons
<Controls:NavigationButtonInfo PageType="views:MainPage" ClearHistory="True">

</Controls:NavigationButtonInfo>
</Controls:HamburgerMenu.PrimaryButtons>

Should be able to bind to an ObservableCollection<NavigationButtonInfo> and be off to the races!

Did that work?

That should work just fine.  I am about to try now.  I was looking for a way to define the DataTemplate for NavigtionButtonInfo in XAML.  Maybe an ItemTemplate property perhaps. I will either just do everything in code or try this approach since DataType has been removed. http://matthamilton.net/winrt-typetemplateselector
Thanks!

If my PrimaryButtons contain information from a database that need to load first.
Should this data be loaded in  MyHamburgerMenu.PaneOpen += MyHamburgerMenu_PaneOpen ?

Note: Just started Template10 and UWP.

I am not sure where that should be located but I did create a ShellViewMofel in my solution. I have not worked on it in a while. Maybe I can post something this weekend

would love to see a sample yes.
Lack of decent samples and documentation makes it hard for a beginner.

@wstaelens in @NoleHealth's post above, you'd simply use your DB information when inserting the HamburgerButtonInfo objects in to the PrimaryButtons collection, no?

Now I just do something like this in Shell.xaml.cs:
    public Shell(NavigationService navigationService)
    {
      Instance = this;
      InitializeComponent();
      _DataService = GalaSoft.MvvmLight.Ioc.SimpleIoc.Default.GetInstance<IDataService>();
      LoadGroupsInMenu();
      MyHamburgerMenu.NavigationService = navigationService;
    }

Where the LoadGroupsInMenu creates new HamburgerButtonInfo's, stores them in a collection in Shell.xaml.cs:
    private ObservableCollection<HamburgerButtonInfo> _NavButtonGroups = default(ObservableCollection<HamburgerButtonInfo>);
    public ObservableCollection<HamburgerButtonInfo> NavButtonGroups
    {
      get { return _NavButtonGroups; }
      set
      {
        if (Equals(_NavButtonGroups, value))
          return;

        _NavButtonGroups = value;
        Instance.PropertyChanged?.Invoke(Instance, new PropertyChangedEventArgs(nameof(NavButtonGroups)));
      }
    }

and Shell.xaml:
<Controls:HamburgerMenu x:Name="MyHamburgerMenu" IsFullScreen="False" PrimaryButtons="{x:Bind NavButtonGroups, Mode=OneWay}">

Is this a correct way of working?
extra question: how come Shell.xaml does not have a viewmodel to work in an MVVM way? For this single Page I now need to add code in codebehind while other pages have a view and viewmodel.


fix: At Success modal is a preview of iterated art (good), and on the listing is the default. (bad)

PR Type

[x] Bugfix
[ ] Feature
[ ] Refactoring

Context
fix until https://github.com/kodadot/private-workers/issues/87 is implemented and  "setMetadata" is done earlier  as @preschian  mentioned

But, I think it can solve the problem if we also put metadata in our minting step. Since it works on the "success modal". In https://github.com/kodadot/private-workers/issues/87#issuecomment-1997444473, I put "setMetadata" in the steps 2 and 6

[x] Closes #9804

Needs QA check

@kodadot/qa-guild please review

Did your issue had any of the "$" label on it?

[x] Fill up your DOT address: Payout

Screenshot 📸

[x] My fix has changed something on UI;

https://github.com/kodadot/nft-gallery/assets/44554284/1b84cd9e-203e-42af-acd5-72b796794fa8

cc @prury ?

😍 Perfect, I’ve sent the payout
💵 $50 @ 9.43 USD/DOT ~ 5.302 $DOT
🧗 16faLfsywwNATaEfbH2ah75dn6ZmctQWpMS5G4KFhbmj5hnD
🔗 0xb441bb54b0e3a450e2b0e45ce71631b21e250bae853cef542fa8b243ab971ef2
🪅 Let’s grab another issue and get rewarded!
🪄 github.com/kodadot/nft-gallery/issues


Creating custom Layer for NodeSetUpdate or EdgeSetUpdate

Hi, am trying to create my own custom layer that is passes as "edge_set_inputs"  where I get the following error, the purpose of the class is to take a transformation layer(EdgesToNodePoolingLayer) as input and before the input graph is passed through the layer make changes to the edges and edge attributes. I need help in the following 2 places:

Resolving the below error where the transformation layer without any change when attempting to call creating an issue
How to get edges source and target along with edge attributes inside the layer

The error faced is:

TypeError: outer_factory..inner_factory..tf__call() got an unexpected keyword argument 'edge_set_name'

For understanding the code better have attached the graph tensor spec, custom class created, and graph building code below.
Graph Tensor Spec:
graph_tensor_spec = tfgnn.GraphTensorSpec.from_piece_specs(
    context_spec=tfgnn.ContextSpec.from_field_specs(features_spec={
                  'label': tf.TensorSpec(shape=(1,), dtype=tf.int32)
    }),
    node_sets_spec={
        'my_node':
            tfgnn.NodeSetSpec.from_field_specs(
                features_spec={
                    tfgnn.HIDDEN_STATE:
                        tf.TensorSpec((None, 1), tf.float32)
                },
                sizes_spec=tf.TensorSpec((1,), tf.int32))
    },
    edge_sets_spec={
        'my_edge':
            tfgnn.EdgeSetSpec.from_field_specs(
                features_spec={
                    tfgnn.HIDDEN_STATE:
                        tf.TensorSpec((None, 1), tf.float32)
                },
                sizes_spec=tf.TensorSpec((1,), tf.int32),
                adjacency_spec=tfgnn.AdjacencySpec.from_incident_node_sets(
                    'my_node', 'my_node'))
    })

Custom Class Created:
class MyLayer(tf.keras.layers.Layer):
    def __init__(self, transformation, **kwargs):
        super().__init__(**kwargs)
        self._transformation_layer = transformation

    def get_config(self):
        return dict(transformation_layer=self._transformation_layer,
                    **super().get_config())

    def call(self, graph: tfgnn.GraphTensor):
        out = self._transformation_layer(graph)
        return out

graph model code:
model_input_graph_spec, label_spec = train_ds.element_spec

node_dim = 32
edge_dim = 32

def set_initial_node_states(node_set: tfgnn.NodeSet, *, node_set_name: str):
    return tf.keras.layers.Dense(node_dim)(node_set[tfgnn.HIDDEN_STATE])

def set_initial_edge_states(edge_set: tfgnn.EdgeSet, *, edge_set_name: str):
    return tf.keras.layers.Dense(edge_dim)(edge_set[tfgnn.HIDDEN_STATE])

l2_regularization = 6E-6
dropout_rate = 0.2
use_layer_normalization = True

inputs = tf.keras.layers.Input(type_spec=model_input_graph_spec)
graph = tfgnn.keras.layers.MapFeatures(
    node_sets_fn=set_initial_node_states, edge_sets_fn=set_initial_edge_states)(inputs)
graph = graph.merge_batch_to_components()
graph = tfgnn.keras.layers.GraphUpdate(node_sets=
{"my_node": tfgnn.keras.layers.NodeSetUpdate({
    "my_edge": MyLayer(SimpleConv(tf.keras.layers.Dense(32,activation='relu'), "sum", receiver_tag=tfgnn.TARGET))},
    tfgnn.keras.layers.NextStateFromConcat(tf.keras.layers.Dense(128)))},

    edge_sets={"my_edge": tfgnn.keras.layers.EdgeSetUpdate(
        tfgnn.keras.layers.NextStateFromConcat(tf.keras.layers.Dense(128)),
    )}
)(graph)

readout_features = tfgnn.keras.layers.Pool(tfgnn.CONTEXT, "mean", node_set_name="pixel")(graph)

out_layer = tf.keras.layers.Dense(10, activation='softmax')(readout_features)
model = tf.keras.Model(inputs, out_layer)

Any help will be greatly appreciated ! , Please do let me know if any more details is needed for the same.

Hi There,
NodeSetUpdate should receive a convolution layer with the following signature:
 new_edge_state = edge_set_update(graph, edge_set_name="...")
So MyLayer should have the edge_set_name in the input args for the call() function, but currently it only has graph as input, hence the failure.
You can read more on these api's here:
https://github.com/tensorflow/gnn/blob/main/tensorflow_gnn/docs/guide/gnn_modeling.md#using-edge-states and
https://github.com/tensorflow/gnn/blob/main/tensorflow_gnn/docs/guide/gnn_modeling.md#convolutions
Re: How to get the edge's source and target node in the layer along with edge attributes inside the layer;
After you have the edge_set_name you can access the source and target node id's as well as the edge attributes as below:
graph.edge_sets[<edge_set_name>].adjacency.source
graph.edge_sets[<edge_set_name>].adjacency.target
graph.edge_sets[<edge_set_name>][<edge_set_feature>]
Also I wonder why you'd need a separate MyLayer layer around SimpleConv, while SimpleConv already handles all the convolutions. SimpleConv will help you configure which sender/receiver and/or edge features you'd prefer to use and transform those for you.
Hope this helps!

Hey, thanks for the response!
By getting edge_set_name the above error is resolved, but we still need clarity on how to access edges and nodes and their attributes and properties. Currently in our ongoing research the edges has to be dropped dynamically and edge attributes have to be modified dynamically for each graph, but since the graphs are merged as components in a batch, we are finding it difficult to get the edges and nodes of each graph separately. We tried to print the graph along with edge source, targets and attributes, which gives output as follows.
GraphTensor(
  context=Context(features={}, sizes=Tensor("model/graph_update/ones_like:0", shape=(None,), dtype=int32), shape=(), indices_dtype=tf.int32),
  node_set_names=['my_node'],
  edge_set_names=['my_edge'])
Tensor("model/input.merge_batch_to_components/Add_1:0", shape=(None,), dtype=int32)
Tensor("model/input.merge_batch_to_components/Add_3:0", shape=(None,), dtype=int32)
Tensor("model/graph_update/edge_set_update/next_state_from_concat_1/dense_2/BiasAdd:0", shape=(None, 128), dtype=float32)

The code used:
class DroNo(tf.keras.layers.Layer):
    def __init__(self,transformation: tf.keras.layers.Layer, **kwargs):
        super().__init__(**kwargs)
        self._transformation = transformation

    def get_config(self):
        return dict(transformation=self._transformation,
                    **super().get_config())

    def call(self, graph: tfgnn.GraphTensor, edge_set_name):
        print(graph)
        print(graph.edge_sets['my_edge'].adjacency.source)
        print(graph.edge_sets['my_edge'].adjacency.target)
        print(graph.edge_sets['my_edge']['hidden_state'])
        return self._transformation(graph, edge_set_name=edge_set_name)

So kindly help us on how to drop edges  and nodes dynamically in each graph and alter edge attributes and node properties.

You can find the data structures that's used by the GraphTensor to keep the edge-sets from the following classes; Adjacency and HyperAdjacency.
What's the criteria you want to use for updating edges?
If let's say you want to drop an edge if its edge attribute(hidden_state) is all zeros, you could do something like below;
`
hidden_tensor_sum = tf.math.reduce_sum(graph.edge_sets['my_edge']['hidden_state'], axis=-1)
zeros_mask = tf.math.equal(hidden_tensor_sum, 0.0)
Note that this keeps the masked out edge information in _masked_my_edge, you could also drop this one later
gt = tfgnn.mask_edges(gt, 'my_edge', zeros_mask, '_masked')
`
After batching, if you still need to identify a particular graph within a merged batch that's somehow independent of its node-set and edge-set details/features then you could potentially add that graph specific information into the respective graph-pieces(nodes/edges etc) that could later be used.
To learn more about the TF-GNN you can find tutorials from various conferences here: https://github.com/tensorflow/gnn/tree/main/examples/tutorials, also particularly more about GraphTensors here in the 2nd part of the talk : https://www.youtube.com/watch?v=JqWROPYeqjA

Hey Neslihans, thanks for the help, in our scenario we are trying to randomly mask/drop 30% of edges or nodes or both for each graph during training. Currently we tried to randomly mask edges (without percentage) to check the working ,using the following code
class DroNo(tf.keras.layers.Layer):
    def __init__(self,transformation: tf.keras.layers.Layer, **kwargs):
        super().__init__(**kwargs)
        self._transformation = transformation

    def get_config(self):
        return dict(transformation=self._transformation,
                    **super().get_config())

    def call(self, graph: tfgnn.GraphTensor, edge_set_name):
        print(graph)
        print(graph.edge_sets['my_edge'].adjacency.source)
        print(graph.edge_sets['my_edge'].adjacency.target)
        print(graph.edge_sets['my_edge']['hidden_state'])
        hidden_tensor_sum = tf.math.reduce_sum(graph.edge_sets['my_edge']['hidden_state'], axis=-1)
        print(tf.random.uniform(hidden_tensor_sum.shape, minval=0, maxval=2, dtype=tf.int32))
        return self._transformation(graph, edge_set_name=edge_set_name)

and we are getting this error
ValueError: Cannot convert a partially known TensorShape (None,) to a Tensor.

Any help will be highly appreciated to obtain our objective which is to drop 20-30% edges and nodes randomly in each graph during training.

The error message
ValueError: Cannot convert a partially known TensorShape (None,) to a Tensor.

comes from this line of code, I suppose:
print(tf.random.uniform(hidden_tensor_sum.shape, minval=0, maxval=2, dtype=tf.int32))

...and presumably can be fixed by using tf.shape(hidden_tensor_sum) instead of hidden_tensor_sum.shape.
For more information, please visit tensorflow.org/guide/tensor and read the explanations around "You may run across not-fully-specified shapes." I suspect your code is run as part of tf.keras.Model.fit(), which uses tf.function under the hood unless you set model.compile(..., run_eagerly=True).
Those TensorFlow generalities aside, is there a particular defect or feature request you'd like to report for the TF-GNN library? If not, I'd like to close this issue.
For general "how-to" questions, please consult the respective StackOverflow tags [tensorflow] and [tensorflow-gnn].


Focus problem

i am using following code for set focus to editors when the enter key is pressed;
This code works fine on new XAF mobile theme, but not works on XAFBootstrap.
Code:
    private void CurrentRequestWindow_PagePreRender(object sender, EventArgs e)
    {
        var window = (WebWindow)sender;
        var script = IsSuitableView() ? @"window.onkeydown = function(e) {
                                              var key = e.keyCode ? e.keyCode : e.which;
                                              if (key == 13) {

                                                  RaiseXafCallback(globalCallbackControl, 'T171941', 'ENTER', '', false);
                                              }
                                           };" : "window.onkeydown=undefined;";
        window.RegisterStartupScript(GetType().FullName, script);
    }
    public void ProcessAction(string parameter)
    {
        if (IsSuitableView() && (Frame != null))
        {
            switch (parameter)
            {
                case "ENTER":
                    if (Convert.ToInt32(MiktarEdit.PropertyValue) > 0)
                    {
                        BakodEdit.Editor.Focus();
                    }
                    else
                    {
                        MiktarEdit.Editor.Focus();
                    }
                    break;
            }
        }
    }

Focus logic is not supported for now.

Do you have any plans to support focus logic?

I think yes, it's possible, so I'll mark this issue as enhancement

this feuture will be useful when ready, but I need a quick solution.
is there any way to catch Enter key and set fous to other editor? can you give some example?

For your task you can use WebWindow.CurrentRequestWindow.RegisterStartupScript in your controller and register logic you need to do with javascript.
For example, this script sets focus to input element with specified placeholder:

protected override void OnViewControlsCreated()
{
     base.OnViewControlsCreated();
     WebWindow.CurrentRequestWindow.RegisterStartupScript("SetFocusToElement", "$('[placeholder=Имя]').focus();");
}

Enhanced in release 15.2


MAN2-2955 again

Purpose
Add testing and typedefs to ReducerGenerator.

Coverage increased (+0.8%) to 46.612% when pulling 5206ab5b6de3c440c570dc0758d23205787ff5cc on rjstires:MAN2-2955-again into 32e1c296ce11c4b7c7f4527c08fa7f8018d0595a on linode:develop.


ERR_PACKAGE_PATH_NOT_EXPORTED: No "exports" main defined in helia/package.json

Error
Error [ERR_PACKAGE_PATH_NOT_EXPORTED]: No "exports" main defined in /<path>/node_modules/helia/package.json
    at new NodeError (node:internal/errors:399:5)
    at exportsNotFound (node:internal/modules/esm/resolve:361:10)
    at packageExportsResolve (node:internal/modules/esm/resolve:641:13)
    at resolveExports (node:internal/modules/cjs/loader:538:36)
    at Function.Module._findPath (node:internal/modules/cjs/loader:607:31)
    at Function.Module._resolveFilename (node:internal/modules/cjs/loader:1033:27)
    at Function.Module._resolveFilename.sharedData.moduleResolveFilenameHook.installedValue [as _resolveFilename] (/home/sxia/code/dasset/node_modules/@cspotcode/source-map-support/source-map-support.js:811:30)
    at Function.Module._load (node:internal/modules/cjs/loader:893:27)
    at Module.require (node:internal/modules/cjs/loader:1113:19)
    at require (node:internal/modules/cjs/helpers:103:18) {
  code: 'ERR_PACKAGE_PATH_NOT_EXPORTED'
}

main.ts
import { strings } from '@helia/strings';
import { createHelia } from 'helia';

(async () =>{
    const helia = await createHelia();
})()

dependencies:
"helia": "^1.3.8"
"@helia/strings": "^1.0.0",
When I just simply run the above code with ts-node main.ts

+1
this is very bad, and needs to be addressed as soon as possible.

I'm having same issue

Helia is an ESM-only module.  You cannot use require to load it, you must use import.
If you cannot use ESM for your project, you can use the dynamic import function to import it - please see the helia-cjs example for more.

I’m using import, on a nestjs project.
And obviously it doesn’t work, still same error described in this thread.

@tomachianura can you provide your source code or a reproducible example so we can dive deeper?

For others here, please check out https://github.com/ipfs/helia/issues/216#issuecomment-1669558901 and let us know if you continue to have issues, but confirm your build tool supports ESM.

There's also helia-ts-node example to show how to get it working, it needs a bit of extra config.

we work with CommonJS, and we can't change this.
Helia shall fully support CommonJS as well, right?

Yes, please see the helia-cjs example for how to use Helia from a CommonJS project.

even when looking at
https://github.com/ipfs-examples/helia-cjs
just doesn't work on an existing nestJS project using CommonJS.
can you kindly provide any solution to that please?
thanks in advance for your support and for being so reactive on this topic.

just doesn't work on an existing nestJS project using CommonJS.

I'm guessing that you have a NextJS project, you've set "module" to "CommonJS".
I'm guessing that you've done something like:
const { createHelia } = await import('helia');

and you've checked your output folder and seen it being transpiled to:
const { createHelia } = await Promise.resolve().then(() => require('helia'));

The solution here appears to be to change "moduleResolution" to "Node16" - this prevents the dynamic import from being transpiled to require.
If this does not solve your problem then you are going to have to provide a link to your source code or a reproducible example for us to assist further, because all this guess work has a high cost in time spent and a low probability of success.

it's not Next, it's Nest :)
changing moduleResolution to Node16 breaks the entire project as well.
our engine is totally closed source, but it's enough to start a new nestjs app (which by default is CommonJS) and install helia to make it crash with the ERR_PACKAGE_PATH_NOT_EXPORTED error.
Clearly, your suggestions described in the helia-nestjs doesn't work, cause changing CommonJS for something else is NOT a solution, considering it'll break a whole project.
Looking forward to have a valid solution, thanks again!
PS: let me know if you need a clean nestjs app with helia installed and crashing, i can create it in the next days once back home.
wish you all a great weekend ahead!

PS: let me know if you need a clean nestjs app with helia installed and crashing, i can create it in the next days once back home.

Yes please. We can provide much better help with reproducible code. Thanks!

TBH I'm not sure if that's going to add much value since there an example available of how to configure NestJS to work with Helia, so we know the default settings for NestJS are to build CJS, and we know the config changes needed to make it build ESM, but you can't apply them because it "breaks the entire project".
The root of the issue here is that ERR_PACKAGE_PATH_NOT_EXPORTED is caused when CJS code uses require to load an ES module.
To prevent his happening you have to import it, or await import if you have to stick on CJS.
Your TypeScript settings are (I think, please confirm) transpiling await import('helia') to await Promise.resolve().then(() => require('helia') which just isn't going to work.
This is not a problem specific to Helia, you'll have this problem with any module written in ESM that use an exports map.
I suggest fixing your project so it can work with ES modules.
If you won't do this then the only other thing you can do is try to defeat the transpilation of import to require by using eval:
const { createHelia } = await eval('import("helia")')

'Helia is an ESM-only module. You cannot use require to load it, you must use import."
This is absolutely insane. Probably a deal breaker for many developers. You might think the people behind Helia would care about adoption and give the community a product easy to use but obviously that's not the case.
IPFS was a good it for my Electron app project but unfortunately I cannot use it. That's just sad.

There is nothing stopping you from using Helia in Electron - please see the Running Helia in Electron example for how to get started.

OK That's encouraging! Thank you for the link.


Typo in page

Typo in Note at the end of page. Should be "Your system path variable must include access to your machine's Java installation."

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

ID: 02d8bdce-a2ad-80cb-efdb-74c0bb3ebb8c
Version Independent ID: b8d0b4a7-a6b0-42d6-aab3-58a9d8681502
Content: Preparing Espresso Tests for Upload - Visual Studio App Center
Content Source: docs/test-cloud/frameworks/espresso/index.md
Service: vs-appcenter
Sub-service: app-center-test
GitHub Login: @lucen-ms
Microsoft Alias: lucen

Thank you @iakdeniz64 for reporting the type - it is now fixed.


Support for k8s network policies

We should add support for k8s network policies for Rancher network:
http://kubernetes.io/docs/user-guide/networkpolicies/

Wondering if this will be prioritized in the near future?
As I see Cattle has network policies, so I suppose the basics are in place. Anyways, would be highly appreciated!

With the release of Rancher 2.0, development on v1.6 is only limited to critical bug fixes and security patches.
In 2.0, we've added the ability to turn on/off network policies for canal.


Self traffic rules

Description
Changes proposed in this pull request.
Test plan
Outline the test plan used to test this change before merging it.

Fixes #.

Trying to identify the issue here
Started with a wide rule using enforcer cgroup. trying to figure out whats wrong here will make the rule cgroup based again

/build - automatically fired by gogo with following PRs and commit SHAs v1.0.0
[
  {
    "project": "",
    "component": "enforcerd",
    "pr-id": "1130",
    "commit-sha": "34ad7f935f3d5a7bfe416a3692e4bbdcdd852c34"
  },
  {
    "project": "",
    "component": "trireme-lib",
    "pr-id": "780",
    "commit-sha": "12e6844d6a2a3971e4d30fa7cfbcb3203119f898"
  }
]

/build - automatically fired by gogo with following PRs and commit SHAs v1.0.0
[
  {
    "project": "",
    "component": "enforcerd",
    "pr-id": "1130",
    "commit-sha": "cc56dad1c9697c2f91094d6ce7004dab31d01ffe"
  },
  {
    "project": "",
    "component": "trireme-lib",
    "pr-id": "780",
    "commit-sha": "12e6844d6a2a3971e4d30fa7cfbcb3203119f898"
  }
]

/build - automatically fired by gogo with following PRs and commit SHAs v1.0.0
[
  {
    "project": "",
    "component": "enforcerd",
    "pr-id": "1130",
    "commit-sha": "357ecaf9b9b58ea877b91f1a43faf755511cc04d"
  },
  {
    "project": "",
    "component": "trireme-lib",
    "pr-id": "780",
    "commit-sha": "ac94a9afd87b5d29d427ac1b0125aa310dad97a3"
  }
]

/build - automatically fired by gogo with following PRs and commit SHAs v1.0.0
[
  {
    "project": "selftraffic",
    "component": "enforcerd",
    "pr-id": "1130",
    "commit-sha": "357ecaf9b9b58ea877b91f1a43faf755511cc04d"
  },
  {
    "project": "selftraffic",
    "component": "trireme-lib",
    "pr-id": "780",
    "commit-sha": "ac94a9afd87b5d29d427ac1b0125aa310dad97a3"
  }
]

/build - automatically fired by gogo with following PRs and commit SHAs v1.0.0
[
  {
    "project": "selftraffic",
    "component": "enforcerd",
    "pr-id": "1130",
    "commit-sha": "357ecaf9b9b58ea877b91f1a43faf755511cc04d"
  },
  {
    "project": "selftraffic",
    "component": "trireme-lib",
    "pr-id": "780",
    "commit-sha": "997f8cad89c8043fddff000c296927019a0ef4b1"
  }
]

/build - automatically fired by gogo with following PRs and commit SHAs v1.0.0
[
  {
    "project": "selftraffic",
    "component": "enforcerd",
    "pr-id": "1130",
    "commit-sha": "357ecaf9b9b58ea877b91f1a43faf755511cc04d"
  },
  {
    "project": "selftraffic",
    "component": "trireme-lib",
    "pr-id": "780",
    "commit-sha": "c65e8ff7d07f480ba3e9278ebbad7ab6a22a74f3"
  }
]

/build - automatically fired by gogo with following PRs and commit SHAs v1.0.0
[
  {
    "project": "selftraffic",
    "component": "enforcerd",
    "pr-id": "1130",
    "commit-sha": "6a57aee935731928b04db0864b2aa3ce72be0c67"
  },
  {
    "project": "selftraffic",
    "component": "trireme-lib",
    "pr-id": "780",
    "commit-sha": "c65e8ff7d07f480ba3e9278ebbad7ab6a22a74f3"
  }
]

/build - automatically fired by gogo with following PRs and commit SHAs v1.0.0
[
  {
    "project": "selftraffic",
    "component": "enforcerd",
    "pr-id": "1130",
    "commit-sha": "6dda4414c7e0f1cd9efa5404e5ae20c78b939f0b"
  },
  {
    "project": "selftraffic",
    "component": "trireme-lib",
    "pr-id": "780",
    "commit-sha": "c65e8ff7d07f480ba3e9278ebbad7ab6a22a74f3"
  }
]

/build - automatically fired by gogo with following PRs and commit SHAs v1.0.0
[
  {
    "project": "selftraffic",
    "component": "enforcerd",
    "pr-id": "1130",
    "commit-sha": "8c1e482c27182b2dc7a56386b27562f86108828a"
  },
  {
    "project": "selftraffic",
    "component": "trireme-lib",
    "pr-id": "780",
    "commit-sha": "6ed8c187e6f124f0297b3b9d36b41f5f830b51fd"
  }
]

/build - automatically fired by gogo with following PRs and commit SHAs v1.0.0
[
  {
    "project": "selftraffic",
    "component": "trireme-lib",
    "pr-id": "780",
    "commit-sha": "6ed8c187e6f124f0297b3b9d36b41f5f830b51fd"
  },
  {
    "project": "selftraffic",
    "component": "enforcerd",
    "pr-id": "1130",
    "commit-sha": "9683486274163b0c924eca3fb425fa6ac04eccc6"
  }
]

/build - automatically fired by gogo with following PRs and commit SHAs v1.0.0
[
  {
    "project": "selftraffic",
    "component": "enforcerd",
    "pr-id": "1130",
    "commit-sha": "adefab6a9f8eb63edf5b87495b670a9ecb0c8c51"
  },
  {
    "project": "selftraffic",
    "component": "trireme-lib",
    "pr-id": "780",
    "commit-sha": "5c78c41762f861fffb67c122b4ca82a40cc124fa"
  }
]


integrating with angularjs app

I need to use Marzipano within an angular js application, So is there some wrapper for marzipano to be used in client side app, and if not what would your advice be?

Why did you closed this issue? I'm interested to know also


[Bug] Duplicate row detected during DML action

Is this a new bug in dbt-core?

[X] I believe this is a new bug in dbt-core
[X] I have searched the existing issues, and I could not find an existing issue for this bug

Current Behavior
I have an incremental model pulling from a collection of ephemeral models. I have defined a unique key (surrogate key of several columns). Using the MERGE strategy on the unique key I have defined (I have confirmed this key is unique), I get the error Duplicate row detected during DML action.
This confuses me because I have confirmed my key is unique and this how are these duplicate rows when MERGing?
Expected Behavior
I expected my MERGE strategy to incrementally merge new records based on my unique surrogate key. Instead this model fails with the error "Duplicate row detected during DML action"
Steps To Reproduce

I am not sure how to instruct someone to replicate this as we have a 1,000+ model warehouse

Relevant log output
18:02:10
18:02:10  Completed with 1 error and 0 warnings:
18:02:10  Database Error in model fact_loan_histories (models/core_data_warehouse/fact_loan_histories/fact_loan_histories.sql)
18:02:10    100090 (42P18): Duplicate row detected during DML action
18:02:10    Row Values: ["e250a04d82a2ca0f5f7c37f8d0dd34b4", 2, 4, "b3bc5b1759a83c996cfa3e568ac2734c", "6fa2c7f681cc33eafa32c3caa8bf086a", 1658817, "e08dde133e154d7b7be707b407d917d1", 8, 20240709, "9b837f2b7ef9ce3dfa6121bafabceff2", 2, 0, 0, 0, 0, 0, 0, 0, 0, 0, 19000101, 0, 0, 0, 0, 0, 0, 20250314, 20240605, 0, 56, 0, 1, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 1, 0, 0, 0, 0, 0]

Environment
- OS: MacOS 13.4.1 (c)
- Python: using DBT cloud
- dbt:1.3

Which database adapter are you using with dbt?
snowflake
Additional Context
dbt-error-logs.odt

Thanks for reaching out @edwrand !
Did you confirm that the unique_key has unique values for both of the following?

The existing data in the table for the incremental model
The new data that is to be added during the next incremental run

yes both places are unique but I still see this issue

@edwrand Unfortunately, I am not able to reproduce this error without seeing your data and the details of your incremental model.
If you're able to share a simple model and example data that produces this error, that would be ideal. But barring that, I'd suggest that you find the SQL that is causing the "Duplicate row detected during DML action" and trying to troubleshoot it in the  Snowflake UI.
Here's how you can find that SQL:
Search within your target directory for a file named like this:
target/run/YOUR_PROJECT_NAME/models/.../fact_loan_histories.sql
You'll see some code that generally looks like this:
    merge into analytics_dev.dbt_dbeatty.fact_loan_histories as DBT_INTERNAL_DEST
        using analytics_dev.dbt_dbeatty.fact_loan_histories__dbt_tmp as DBT_INTERNAL_SOURCE
        on (
                DBT_INTERNAL_SOURCE.id = DBT_INTERNAL_DEST.id
            )

    when matched then update set
        "ID" = DBT_INTERNAL_SOURCE."ID","UPDATED_AT" = DBT_INTERNAL_SOURCE."UPDATED_AT"

    when not matched then insert
        ("ID", "UPDATED_AT")
    values
        ("ID", "UPDATED_AT")

;
    commit;

Then you can try running that SQL in the Snowflake UI. If it gives the "Duplicate row detected during DML action", then you can try to determine the source of the error. There should only be two possibilities:

There's a duplicate in the data somewhere
There's a bug in the SQL logic

Let us know what you find out!


Srcset does not work

Screenshot_1 https://monosnap.com/file/DSObCbH8kTqKfZyoptQK2rEPMlrGCs
Screenshot_2  https://monosnap.com/file/seckVU0D9wT2S7gf6ytGf0QuGqFnlG

Simply use:
![](woman.jpg)

or:
![](woman.jpg?sizes=80vw)

Don't actually put @2x in your image name, so Grav knows to create a @1x version or it will use it statically (as you are seeing).
BTW i've updated the docs to make this a bit clearer. https://learn.getgrav.org/content/media#higher-density-displays

As I know from the CSS vw and vh values means 80% of the device screen width or height.
So it is unclear how to use that. I would like to have srcset="woman.jpg 1x<EMAIL_ADDRESS>2x"
I read the documentation and I still confused. I am not sure I understand how it is should work.


Request

Please Add Quality Adjustment Button
Thank You.

Not something I would consider. Also you are not using the issue template.


Bug related to top_n in run_colocation

Hi guys,
Thanks for this nice package and very helpful tutorial. In run_colocation.py you save the top 10 ranking cell types for each factor to a dataframe using print_gene_loadings on L245:
https://github.com/BayraktarLab/cell2location/blob/master/cell2location/run_colocation.py#L245
This leads to an error when returning the dataframe whenever the user has fewer than 10 cell types because I see that the top_n variable is set to 10 by default in the source of print_gene_loadings:
https://cell2location.readthedocs.io/en/latest/_modules/cell2location/models/base_model.html#BaseModel.print_gene_loadings
Can I suggest that you add a parameter for top_n when run_colocation is called, so that top_n can only take integer values up to the total number of annotations being mapped with cell2location, and then call this top_n value at L245 in run_colocation.py.
Cheers!

Hi @vitkl apologies for the delay, pull request from my fork now submitted!

Thanks for contributing :)

Hi,
run_colocation uses all the available CPU of my server, how to limit the CPU number ? Or can we use GPU instead ?

It's not possible to use GPU for this step. Under the hood we are using sklearn NMF - please check how to restrict cpu use for that package.


Even more animator features

This is the cleaned up diff for sunpy/sunpy#3504

What does this do?

See the SunPy PR for a description.


🐛Tiles could be moved accidentally on mobiles

Fixes #214

Works perfectly! Also solves issue #202 ( #225 ) depending on the intended behavior.


Image Options Feature

Great plugin! I would like to see though a little more control over how the modal editor presents. More specifically in my case I would like to be able to set the stretch option of the image being edited. I could also see someone wanting to add a description/instructions to the modal editor page so maybe some kind of template feature maybe?

Adding whole template will be very difficult as the editing is done in the native environment and is not in the {N} runtime.


Bug plotting missing Shape Metrics e2 and R2

Closes #110 and #94
Refactored the ShapeMetricsPlotHandler to reduce duplicated lines, and altered the sequence of commands to produce plots for e1, e2, and R2 shape metrics (see plots below).
This PR also applies a fix raised in https://github.com/CosmoStat/wf-psf/issues/92 regarding skipped validation tests.
We want the validation tests to be skipped during CI, but to be ran manually with pytest by the developer. We applied the conditional pytest.mark.skipif decorator and will check if the CI environment variable was defined. If yes, the test is skipped, else the test is run.
Note, 28 files were changed due to rebase with the develop branch with updated package dependencies.
Below I show examples of the Shape metric plots for e1, e2 and R2/MeanR2.

@sfarrens I am ready for you to review this PR:

plotting/plot_interface.py
tests/metrics_test.py
tests/train_test.py


Need a $Deprecated utility type

Sometimes I'd like to indicate that a property is deprecated and is going to be removed soon.
There is a helper to do that if I used prop-types:
import deprecated from 'react-prop-types/lib/deprecated';
...
static propTypes = {
    myDeprecatedProp: deprecated(PropTypes.bool, 'Use `newProp` instead'),
    ...

But I don't want to use prop-types and flow types simultaneously.
So I find it handful if flow had a special utility class which throws a warning if such deprecated property is used in the code.

See: https://github.com/facebook/flow/issues/4646


Vcu update

Fix errors in Hindbrain

Hysteresis in system on Estop
Wrong variable sending cmds to roboclaw

Re-tuning of VCU control loops for velocity actuator

Code tested on tractor

Wow, I merged the stuff on the tractor once I saw these changes approved, but never finished this pull request. Sorry Nathan, my bad.