Fix a couple of broken error messages

flake8 caught these while I was browsing through JAX's source code

Nice! I bet we’ve got more...


add some wait before check liquid balance

Explain your changes:
This PR fixes the flakiness of hard fork integration. The flakiness is that when getting the liquid balance, we are actually using the global slot from the consensus state: https://github.com/MinaProtocol/mina/blob/605b8c874ff5b4957e602972a4c97ed15b68c21f/src/lib/mina_graphql/types.ml#L934
This means that if the account is about to cliff at that slot but a block is being produced before we check the balance, then the liquid balance we get is from previous slot. My fix is just to wait 30s before we check the balance so that if a block is being produced then waiting 30s would make sure that we can get this block included in the transition frontier.
Even with my fix, this test could still be flaky because it's not guaranteed that every slot would produce a block. If we are okay to change the semantics of liquid balance in graphql requests, then we can make them use the current time instead of latest consensus_state. This way we would really fix the flakiness.
Explain how you tested your changes:
*
Checklist:

[ ] Dependency versions are unchanged

Notify Velocity team if dependencies must change in CI

[ ] Modified the current draft of release notes with details on what is completed or incomplete within this project
[ ] Document code purpose, how to use it

Mention expected invariants, implicit constraints

[ ] Tests were added for the new behavior

Document test purpose, significance of failures
Test names should reflect their purpose

[ ] All tests pass (CI will check this if you didn't)
[ ] Serialized types are in stable-versioned modules
[ ] Does this close issues? List them

Closes #0000

!ci-nightly-me

coverage: 85.379%. first build
when pulling ad5df8c1c00bc34275d04a9677af4fbf55f058f3 on fix/hard-fork-integration-test
into 27ace0692c8df3ab1d365b6bde3cf78aed9b9426 on berkeley.

!ci-build-me

I would close this PR since the fix here only reduce the flakiness, but it didn't really fix the problem. This ticket is created to discuss how we could fix it for real: https://app.zenhub.com/workspaces/protocol-eng-64d12d77bff4a91f97f60205/issues/gh/minaprotocol/mina/14745

!ci-build-me

!ci-nightly-me

!ci-build-me

!ci-build-me

!ci-build-me

!ci-build-me

!ci-nightly-me

Nightly tests passed, except for version linter. If ci-build-me yields a "green" result, it's ok to merge this PR.

!ci-nightly-me

!approved-for-mainnet


fix incompatible api

simple.go cannot run properly because of incompatible api.

thanks, modify according to the latest samara api.


Consistent component names and CSS

Some component files are lowercase, some are uppercase. All should be CamelCase.
Some CSS classes are capitalised, they should all be lower case, hyphen-separated.

Also, CSS indentation should be 2.

Some places, missing spaces after CSS classes. It makes me think, maybe CSS linting would be a good idea too.


Python 3.10 wheels

Any chance of some python 3.10 wheels for this package? Having issues installing on 3.10 currently

#13


MODULES-434: handle proxy authentication in apt_key#source_to_file

Avoids "407 Proxy Authorization Required" errors while downloading keys.
Bug report: https://tickets.puppetlabs.com/browse/MODULES-434

@mhaskel Do we need to retain 1.8 compatibility?

@daenney we can drop 1.8 in our next major version bump :\

@edu-tsen @daenney so, I'd suggest either guarding against 1.8, or adding the backwards incompatible flag and we'll be able to merge it 'soon' when we actually start the breaking changes.

Mmm. I don't like the guard because then different Ruby versions but the same Puppet module will get you different results. It sort of goes against the 'least surprise' principle.

Hi @edu-tsen . Thank you for this contribution, but we haven't heard back from you in a while so I'm going to need to close this PR for now. If this is still functionality you'd like to see in the module and you have time to work on the issues from the comments please resubmit. I will let you know the upcoming 1.8.0 release of puppetlabs-apt will be the last feature-release on the 1.x series, after that all new features will be slated for 2.x, but we are unfortunately unable to drop ruby 1.8 support for that release. Thanks again for your contribution!

I'm confused. What's the status of this?  Can I use apt-key behind a proxy now?  https://tickets.puppetlabs.com/browse/MODULES-434 is open, but this issue here is closed.

The PR is closed, not merged, so that means it was not accepted.


[video_player] update periodic timer duration to be public

Use case
Currently, the code snippet:
Timer.periodic(
        const Duration(milliseconds: 500),

found at code, is configured with a periodic duration of 500 milliseconds. This introduces a delay when using internal captions, as the Timer only updates every 500 milliseconds.
In scenarios where subtitles are displayed for a short duration, such as 1 second, this delay can cause captions to either appear late or for an inconsistent duration (e.g., 500 milliseconds or less). This significantly impacts user experience, as subtitles may not align properly with the video, making it difficult for users to follow along.
Proposal
A potential solution is to expose a setter for the periodic duration. This setter would allow developers to adjust the timer's interval dynamically. When a new duration is provided, the setter would cancel the current timer and create a new one with the updated duration. This flexibility ensures that subtitle timing can be adjusted to match use cases requiring finer granularity, significantly improving the user experience by reducing delays and ensuring accurate subtitle display.

This is essentially a duplicate of https://github.com/flutter/flutter/issues/90080; see the last comment there for a link to discussion in a previous PR proposing this.


IE8 compatibility

Hi!
Object Array does not support a method IsArray in IE8.
For compatibility can be replaced at line 139 the Array.isArray(meta.options) to $.isArray(meta.options).

I am OK with not supporting IE8, PR is welcomed though. Closing.


added error raise when list field recieves a non list item.

Description
Includes a language error raise when a non-list is given to a command that expects a list.
Linked Items:
Closes/Fixes/Resolves #769
Added

_Language error raise when list type is expected but non-list is given.

Changed

Describe any changes in existing functionality.

Deprecated

Describe any deprecated features.

Removed

Describe any removed features.

Fixed

Describe any bug fixes.

Security

Describe any security-related changes.

Checklist:

[ ] I updated project documentation to reflect my changes.
[ ] My changes generate no new warnings.
[ ] I updated new and existing unit tests to account for my changes.
[ ] I linked the associated item(s) to be closed.
[ ] I bumped the version.
[ ] I added the labels corresponding to my changes.

Make sure you merge in main and bump version

Remember to update the project and milestone pieces for having it properly link up and display within the board.

Since we made this modification within the check_aac_impl, I think it would be worth adding to the check unit test for evaluating this handling. Especially since it doesn't have any bad path testing included yet.


Dev

asyncpg connection manager added.

Codecov Report
Attention: 25 lines in your changes are missing coverage. Please review.

Comparison is base (34b29f1) 89.09% compared to head (431f2b4) 87.37%.
Report is 11 commits behind head on master.

Files
Patch %
Lines

generic_connection_pool/contrib/asyncpg.py
0.00%
23 Missing :warning:

generic_connection_pool/contrib/unix.py
0.00%
1 Missing :warning:

generic_connection_pool/contrib/unix_async.py
0.00%
1 Missing :warning:

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##           master      #26      +/-   ##
==========================================
- Coverage   89.09%   87.37%   -1.72%
==========================================
  Files          16       17       +1
  Lines        1403     1426      +23
==========================================
- Hits         1250     1246       -4
- Misses        153      180      +27

Flag
Coverage Δ

unittests
87.37% <0.00%> (-1.72%)
:arrow_down:

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

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


branch-2.1 When Load ends, check memory tracker value returns is equal to 0

pick
#38960
#39908
#40043
#40092
#40016
#40439

run buildall

run buildall


Include missing files in sdist

Include missing files in sdist for #4
Tested with
++ mktemp -d
+ D=/tmp/tmp.Z7WdPbe1Ie
+ trap 'rm -rf /tmp/tmp.Z7WdPbe1Ie' EXIT
+ python -m venv /tmp/tmp.Z7WdPbe1Ie
+ python setup.py sdist -d /tmp/tmp.Z7WdPbe1Ie
running sdist
running egg_info
writing pipup.egg-info/PKG-INFO
writing dependency_links to pipup.egg-info/dependency_links.txt
writing entry points to pipup.egg-info/entry_points.txt
writing requirements to pipup.egg-info/requires.txt
writing top-level names to pipup.egg-info/top_level.txt
reading manifest file 'pipup.egg-info/SOURCES.txt'
reading manifest template 'MANIFEST.in'
writing manifest file 'pipup.egg-info/SOURCES.txt'
running check
creating pipup-0.3.0
creating pipup-0.3.0/pipup
creating pipup-0.3.0/pipup.egg-info
copying files to pipup-0.3.0...
copying .version -> pipup-0.3.0
copying MANIFEST.in -> pipup-0.3.0
copying README.md -> pipup-0.3.0
copying setup.cfg -> pipup-0.3.0
copying setup.py -> pipup-0.3.0
copying pipup/__init__.py -> pipup-0.3.0/pipup
copying pipup/exceptions.py -> pipup-0.3.0/pipup
copying pipup/freeze.py -> pipup-0.3.0/pipup
copying pipup/main.py -> pipup-0.3.0/pipup
copying pipup/req_files.py -> pipup-0.3.0/pipup
copying pipup.egg-info/PKG-INFO -> pipup-0.3.0/pipup.egg-info
copying pipup.egg-info/SOURCES.txt -> pipup-0.3.0/pipup.egg-info
copying pipup.egg-info/dependency_links.txt -> pipup-0.3.0/pipup.egg-info
copying pipup.egg-info/entry_points.txt -> pipup-0.3.0/pipup.egg-info
copying pipup.egg-info/requires.txt -> pipup-0.3.0/pipup.egg-info
copying pipup.egg-info/top_level.txt -> pipup-0.3.0/pipup.egg-info
Writing pipup-0.3.0/setup.cfg
Creating tar archive
removing 'pipup-0.3.0' (and everything under it)
+ cd /
+ /tmp/tmp.Z7WdPbe1Ie/bin/pip install /tmp/tmp.Z7WdPbe1Ie/pipup-0.3.0.tar.gz
Processing /tmp/tmp.Z7WdPbe1Ie/pipup-0.3.0.tar.gz
Collecting click==7.0 (from pipup==0.3.0)
  Using cached https://files.pythonhosted.org/packages/fa/37/45185cb5abbc30d7257104c434fe0b07e5a195a6847506c074527aa599ec/Click-7.0-py2.py3-none-any.whl
Installing collected packages: click, pipup
  Running setup.py install for pipup: started
    Running setup.py install for pipup: finished with status 'done'
Successfully installed click-7.0 pipup-0.3.0
WARNING: You are using pip version 19.2.3, however version 20.1.1 is available.
You should consider upgrading via the 'pip install --upgrade pip' command.
+ echo Success
Success
++ rm -rf /tmp/tmp.Z7WdPbe1Ie

Thanks!


Added DipDup indexer

What changed? Why?
Added DipDup indexer in the Data indexers section
Notes to reviewers
How has it been tested?
Started the project locally according to the instructions in README.md

🟡 Heimdall Review Status

Requirement
Status
More Info

Reviews

🟡
0/2

Denominator calculation

Show calculation

1 if user is bot
0

1 if user is external
0

From .codeflow.yml

1

Additional review requirements

Show calculation

Max
0

0

From CODEOWNERS

0

Global minimum
0

Max

1

1

1 if commit is unverified
1

Sum
2


Added Italtel to LMO list

Added Italtel to LMO list

@CarolynRowe
PR has been copyedited and is ready for final review, could you please check and merge? Thanks!

@CarolynRowe
PR has been copyedited and is ready for final review, could you please check and merge? Thanks!

Hello @scanum ...  This is the second PR I looked at this morning with conflicted branches.  Not sure how to resolve this. Is there something going on in the backend?

Hi @Carolyn Rowe hope you are doing well. Let me check internally and I will get back to you.
Thanks for your help and patience!


go-ethereum: v1.6.7 -> v1.7.0

Motivation for this change
New major version with e.g. protocol upgrades for the upcoming hard fork.
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option build-use-sandbox in nix.conf on non-NixOS)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] Linux

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[x] Fits CONTRIBUTING.md.

Sorry, duplicate of https://github.com/NixOS/nixpkgs/pull/29408.


Added payment failed screen

Added screen for failed payment.
This screen will be shown in the event of a failed payment instead of redirecting the user back to check your answers

CI: No security warnings found


[ACPI] Proposal

Is there any reason in this guide there are some useless SSDT that don't add any "feature" on macOS?
I'm talking about:

SSDT-ARTC (see also this)
SSDT-FWHD
SSDT-XSPI

I'd rather drop them rather than clogging uselessly ACPI tables as o*****a does...
Please read this PR also: https://github.com/acidanthera/OpenCorePkg/pull/121#issuecomment-696825376

As staded in chapter 01's summary, these tables are purely cosmetic and optional. To add them or not, is up to the user.
BTW, I think DMAC has to be considered cosmetic as well. I am not sure about XSPI, though.
If your proposal was to remove the tables: not happening. Even if one doesn't add any of this tables, one may still want to know what the device does.

Nobody forces users to use these SSDTs.
This spectacular guide is NOT to be reduced by information, but things must be added.
I am not discussing anything, because I have only added some useless things.
But you have already removed the variation of the awac and rtc method and it is nonsense to me.
I use these methods in every hack of my reposities and no one has ever had a problem.
Rebuilding devices where it is possible to change the state is for me the cleanest method.
Rebuilding devices is just an old and functional method. And they fill those removed parts with knowledge in my opinion.
Always endless compliments for this masterpiece, you are great

@Baio1977 I deleted the AWAC and RTC variants you mentioned because @dreamwhite said they were not ACPI conform and suggested to remove them:
https://github.com/5T33Z0/OC-Little-Translated/issues/22
And since I don't know enough ASL yet to really judge if a table conforms to ACPI specs or not, I acted based on the recommendation.

No problem, I'm not an ACPI Guru, I'm just an average user, but whoever taught me to understand ACPI would never have made me do patches that are not in line with the ACPI standard and in fact the creator of this Guide has already explained the possible usable variables.
So this is stated by my compatriot I do not know if it is really true, but I certainly do not have the skills to contradict it.
/*

Intel ACPI Component Architecture
AML/ASL+ Disassembler version 20200925 (64-bit version)
Copyright (c) 2000 - 2020 Intel Corporation

Disassembling to symbolic ASL+ operators

Disassembly of iASLLHvEzK.aml, Mon Apr 25 20:42:54 2022

Original Table Header:

Signature        "SSDT"

Length           0x0000048D (1165)

Revision         0x02

Checksum         0x88

OEM ID           "HACK"

OEM Table ID     "HackLife"

OEM Revision     0x00000000 (0)

Compiler ID      "INTL"

Compiler Version 0x20200925 (538970405)

*/
DefinitionBlock ("", "SSDT", 2, "HACK", "HackLife", 0x00000000)
{
External (SB.AWAC.STA, UnknownObj)
External (SB.PCI0, DeviceObj)
External (SB.PCI0.GFX0, DeviceObj)
External (SB.PCI0.LPCB, DeviceObj)
External (SB.PCI0.LPCB.ACAD, DeviceObj)
External (SB.PCI0.LPCB.RTC._STA, UnknownObj)
External (SB.PR00, ProcessorObj)
External (XPRW, MethodObj)    // 2 Arguments
Scope (\)
{
    If (_OSI ("Darwin"))
    {
        \_SB.AWAC._STA = Zero
        \_SB.PCI0.LPCB.RTC._STA = 0x0F
    }

}
MacBook Pro di lollo78.ioreg.zip
SSDT-HP 250G7

Hi and sorry for the late reply.
Basically the approach of @Baio1977 isn't bad at all BUT as already stated in issue #22, you can't assign a value to a function that returns a value. If _STA was a variable, then the proposed approach is fine, but to be ACPI 6.4 compliant, _STA is a method, not a variable, so assigning a value to it is just a hacky thing that works, for whatever reason, but it's not compliant...
To be more specific, if _STA was defined like Name (_STA, 0x0F) and not Method (_STA, 0, NotSerialized) the hacky approach would have been fine.
Maybe on old motherboards firmwares it's a variable, but from my whole experience it's not...
P.S. @Baio1977 for the HP 250 G7 you can just create a dummy RTC0 device and it will work :)
/*
 * Intel ACPI Component Architecture
 * AML/ASL+ Disassembler version 20200925 (64-bit version)
 * Copyright (c) 2000 - 2020 Intel Corporation
 *
 * Disassembling to symbolic ASL+ operators
 *
 * Disassembly of iASLn8hmD4.aml, Mon Apr 25 21:04:52 2022
 *
 * Original Table Header:
 *     Signature        "SSDT"
 *     Length           0x0000008E (142)
 *     Revision         0x02
 *     Checksum         0x97
 *     OEM ID           "ACDT"
 *     OEM Table ID     "RTC0"
 *     OEM Revision     0x00000000 (0)
 *     Compiler ID      "INTL"
 *     Compiler Version 0x20200528 (538969384)
 */
DefinitionBlock ("", "SSDT", 2, "ACDT", "RTC0", 0x00000000)
{
    External (_SB_.PCI0.LPCB, DeviceObj)

    Scope (_SB.PCI0.LPCB)
    {
        Device (RTC0)
        {
            Name (_HID, EisaId ("PNP0B00") /* AT Real-Time Clock */)  // _HID: Hardware ID
            Name (_CRS, ResourceTemplate ()  // _CRS: Current Resource Settings
            {
                IO (Decode16,
                    0x0070,             // Range Minimum
                    0x0070,             // Range Maximum
                    0x01,               // Alignment
                    0x08,               // Length
                    )
                IRQNoFlags ()
                    {8}
            })
            Method (_STA, 0, NotSerialized)  // _STA: Status
            {
                If (_OSI ("Darwin"))
                {
                    Return (0x0F)
                }
                Else
                {
                    Return (Zero)
                }
            }
        }
    }
}


Add archive/delete functionality

Related to #588 but going to separate out archive/delete functionality.
Problem: There is a need for either archiving or deleting campaigns. As a campaign manager, I might be creating a test campaign that I don't to use anymore, or I want to clean up some existing campaigns and so I would like to delete my campaign. Right now we don't expose any functionality to users for this. Current process is to delete campaign directly from the S3 bucket.
Potential Solution: When in Edit mode on a campaign, add a archive/delete function.
I say archive or delete because there might be a use case for only archiving (and thus removing from frontend) instead of permanently deleting. There doesn't seem to be a huge need to save the campaigns forever but there might be a use case for being able to access a very old campaign. Open for discussion.

@Eleonore9 this will require some front end changes as well obviously (see wireframe) but i think we can implement a function on the backend to delete an s3 campaign folder.  Additionally we may want to traverse user json to remove the deleted project from the project arrays.  We can wait on this until the user json is sorted out

Archiving sounds like an interesting feature, although I guess it would involve even more changes for retrieving the archived project.

archiving will likely not exist in favor of a full delete. the "inactive" state of a project will serve a similar function to archive in that it will set projects out of the active view

@Eleonore9 see the figma page for exact layout, but in general the delete will be available in the header that will exist across all the new "project" level pages.  Per the new user management file this will only be displayed to users with "manage" permission for a project:

The confirmation dialog is in the wireframe on figma for reference


doc/code-of-conduct.md: Add a community code of conduct

I propose this short and sweet code of conduct for the Snabb community. This is intended proactively to frame future discussions when problems come up.
The basic suggestions here are to be civil, for upstream to act as a resource for helping people to cooperate efficiently (even when their goals don't immediately align with the master branch), and to keep important discussions on Github so that watching the project here is sufficient to stay "in the loop."
I have found it frustrating in the past to contribute to projects where the upstream community are rude, don't care whether contributors achieve their own goals, or require unproductive rituals that disrespect contributors' time.
Let's not become one of those projects. Instead let's encourage contributors to whack us on the head with this document any time we accidentally step in that direction.
Just to repeat, this is based on past experience with other projects and not related to anything going on in the Snabb community today. I imagine it would be much harder to introduce a change like this in the middle of a heated discussion about project policies and that is why I like the idea of doing it now.

@eugeneia will you be upstream for this?
what do you think about renaming documentation-fixes to simply documentation and using that as the next-hop for general documentation changes that are not part of a specific program? (I suppose it would be a bit funny for you to be your own next-hop with documentation -> max-next and a simple solution would be for documentation to feed directly into next instead.)

(Could also be that maintaining both max-next and documentation separately is too much extra work. In that case another alternative would be to "optimize away" the merge to documentation and merge directly onto max-next instead, which I suppose is roughly the current practice. Even in this case I think it is valuable to have the specific branches well-defined and written up in branches.md because it does at least tell us which person is responsible for each area even if they may take a short-cut in terms of where the merge things. I feel like we are still feeling around to see how this will play out longer term.)

documentation-fixes is somewhat underused these days. I see value in a documentation branch based on the fact that it will only ever contain documentation changes and never changes to code. So you (next) could trust me to ensure that documentation never changes code which would potentially ease reviewing the changes.
I think replacing documentation-fixes with documentation ( a branch for all documentation changes without code changes) is a good move. I think the next hop should be next, and I would gladly maintain this branch.

That sounds like a good plan to me.

I would be interested in what others think about the “Contributor Covenant”. I don't think the text is trivial. Are we binding ourselves to follow it literally or is it more of a non-authoritative guideline?

I guess if "All checks have passed" it should be fine :)

It seems like a fairly common sense Code of Conduct - this is a friendly project, and "don't troll or throw insults around" isn't a culture change, happily. If Luke's intention is to actually enforce it if it ever gets violated, and the existing community all buys in, I'm in favour of it.

I would be interested in what others think about the “Contributor Covenant”. I don't think the text is trivial.

Thanks for reading and thinking about it. My take is that the Contributor Covenant rules seem sensible. They pass my basic litmus test of accepting basic Snabb upstream behavior and rejecting basic Linux kernel upstream behavior. This is important to me because I am often explaining that we use the same workflow as the Linux kernel but I don't want to imply that we should conduct ourselves like Linus Torvalds does.

Are we binding ourselves to follow it literally or is it more of a non-authoritative guideline?

This would be binding and enforced, just like other project policies and related rules that we are bound by such as the Github Terms of Service.

I think replacing documentation-fixes with documentation ... is a good move

Done on next: 06c27c655bf565d83a2234f2617cd6add46587dd.

FWIW, I'm aligned with this CoC and I think is positive to have one.


The hyperlinks of the second element is not working when PDF is generated with loop

Hi @eKoopmans ,
With my below code I am having trouble making the hyperlinks working:
var element1 = document.getElementsByClassName("hero10").length > 0 ? document.getElementsByClassName("hero10")[0] : "";
var element2 = document.getElementById('quizBox-variation1');
var element3 = document.getElementById('showResults');

var opt = {
    jsPDF: {
		unit: 'in',
		format: 'A2',
		orientation: 'portrait'
	},
	image: {
		type: 'jpeg',
		quality: 0.98
	},
    html2canvas: {
        logging: true,
    },
    imageType: 'image/jpeg',
    pagebreak: {
        mode: 'avoid-all',
        before: ['#survey-report-type-overall-report'],
        after: ['.tabs-panel']
    },
    enableLinks: true
};

var pages = [];
pages.push(element1);
pages.push(element2);
pages.push(element3);

let doc = html2pdf().set(opt).from(pages[0]).toPdf()
for (let j = 1; j < pages.length; j++) {
    doc = doc.get('pdf').then(
        pdf => {
            pdf.addPage()
        }
    ).from(pages[j]).toContainer().toCanvas().set(opt).toPdf()
}
doc.save();

The element 3 is having the links and when added after element 1 or 2, the PDF just rendering without anchor tag. But if I add the element3 on top the Links just works fine.
I have also tried using the parent div with single element, then also it doesn't generates the hyperlinks in PDF.
Do you see anything wrong here or if something you can help with? We are actually sending it back to server with datauristring.

Same problem. Any idea? Thanks. The hiperlinks is in first page


Sealed MixedRealityToolkit

Question. Why does this class need to be sealed? There was no description on this PR (and no referenced issue).
Thanks!

This is a pretty key & important class. Makes sense that we wouldn't want people to change its functionality.

Makes sense that we wouldn't want people to change its functionality.

Sealing means customers cannot extend it as well. Let's not make this change for beta and discuss pros/cons for RTW.
Seem fair?

Sealing means customers cannot extend it as well.

Exactly. This is critical infrastructure and no one should be extending it.

Can we discuss this post beta? There have been enterprise customer requests to provide their own implementation of the toolkit component and it seems prudent to consider all possibilities.
Thanks

If there's something missing from the implementation that is required, then they need to submit a PR to fulfill the requirements like any other contributor.
We've already provided a full suite of options for creating their own services and components that the Framework can ingest and utilize.

The customer request has been to enable custom Toolkit component implementations.
We need to ensure that we understand the customer request and do our due diligence. That feels like a post beta task.
Thanks.

If you're referring to the customer I think you are, that's an entirely different thing altogether.
They don't want to use this service locator pattern at all, which means likely it won't inherit or implement this class, which in turn makes your argument invalid, as they'd likely use their own service locator.

The request is to discuss this post beta. Can we please have an architecture discussion.
Thank you

Closing this PR. Has unrelated changes on the branch.


update login page

add desktop client links, make signup link to the registration page.  Point google analytics to agf account. and lint :).

BTW, usually my changes can be previewed at shin-beta.gokgs.com.


run containers as host user

Adding user: ${user} to a docker-compose service will allow you to run a service with the host's username. This allows files created within the container to have the same permissions as the host. This would resolve issues with not being able see the contents of mounted volumes.

Found this recommendation courtesy of the manim community:

For Linux users there might be permission problems when letting the user in the container write to the mounted volume. Add --user="$(id -u):$(id -g)" to the docker CLI arguments to prevent the creation of output files not belonging to your user.

doesn't work, because the containers themselves won't have the local user, have it a try and its a hot mess.

There is a way to do it but yeah... it would take a big lift...
Maybe a medium/long term goal...

https://github.com/RandyMcMillan/docker.shell

Do we still want to do this?

I am not super concerned with it, but it writing out files as root can be confusing for users that don't have firm understanding of permissions under a non-root under Linux, so it is still probably worth doing (tho would hate to bust any existing that upgrade and now the container cannot read the mapped files they originally were written as root on host system)
Seems the OSX defaults just give non-root users access to those files anyways so they never notice a difference.

I will mess around with a config that defaults to root but allows for a user=user_name option.

i know how to do this... i will take another look since I am doing some work on the playground...


[5.3] [Typechecker] Emit a specialised diagnostic for redeclaration errors when the declaration is synthesised

Cherry-pick of #31915

Explanation: We are not diagnosing redeclaration errors when the declarations involved are synthesised. This sometimes leads to compiler crashes. Upon investigation, this is a regression caused by #31037.
Scope: Affects situations where we have re-declarations.
SR Issue: SR-12839
Risk: Low. This fixes a compiler crash while also emitting improved diagnostics.
Testing: Added new tests which pass on Swift CI and updated some existing ones.
Reviewed by: @hborla

@swift-ci please test


Add OkHttp bot - very common we have seen in our test server logs

See github: https://github.com/WhichBrowser/Parser-PHP/issues/533

Coverage remained the same at 99.952% when pulling bc7da430925571af455dfc60d70ebc120013e42a on ayumi-cloud:OkHttp into 880b9fa797401d14b28956442944c3daa70240ff on WhichBrowser:master.

Fixed in 7e80a62c0b648309ad30bd0f21d0bddf5cf57fe1

Fixed in 7e80a62c0b648309ad30bd0f21d0bddf5cf57fe1


Argo Workflow MetricsError: Custom Metrics Unable to Resolve Output variables

Pre-requisites

[X] I have double-checked my configuration
[X] I have tested with the :latest image tag (i.e. quay.io/argoproj/workflow-controller:latest) and can confirm the issue still exists on :latest. If not, I have explained why, in detail, in my description below.
[X] I have searched existing issues and could not find a match for this bug
[ ] I'd like to contribute the fix myself (see contributing guide)

What happened/what did you expect to happen?
I am encountering an issue with my Argo Workflow template related to the metrics configuration. The error message indicates that it is unable to substitute parameters for the metric count and fails to resolve {{steps.generate.outputs.result}}.
Version
latest
Paste a small workflow that reproduces the issue. We must be able to run the workflow; don't enter a workflows that uses private images.
apiVersion: argoproj.io/v1alpha1
kind: Workflow
metadata:
  generateName: custom-metrics-
spec:
  entrypoint: output-parameter
  metrics:
    prometheus:
      - name: count
        labels:
          - key: app
            value: argo
        help: "Count of the generated number"
        gauge:
          value: "{{steps.generate.outputs.result}}"
  templates:
  - name: output-parameter
    steps:
    - - name: generate
        template: generator
    - - name: print
        template: printer
        arguments:
          parameters:
          - name: count
            value: "{{steps.generate.outputs.result}}"

  - name: generator
    container:
      image: alpine:latest
      command: [sh, -c]
      args: ["echo 10 | tee /tmp/count"]
    outputs:
      parameters:
      - name: count
        valueFrom:
          path: /tmp/count

  - name: printer
    inputs:
      parameters:
      - name: count
    container:
      image: alpine:latest
      command: [sh, -c]
      args: ["echo {{inputs.parameters.count}}"]

Logs from the workflow controller
# argo get custom-metrics-65qbp
Name:                custom-metrics-65qbp
Status:              Succeeded
Conditions:
 PodRunning          False
 Completed           True
✖ MetricsError unable to substitute parameters for metric 'count': failed to resolve {{steps.generate.outputs.result}}

# kubectl logs -n argo deploy/workflow-controller | grep custom-metrics-65qbp
{"level":"info","msg":"TaskSet Reconciliation","namespace":"maintenance","time":"2024-05-30T09:57:21.741Z","workflow":"custom-metrics-65qbp"}
{"level":"info","msg":"reconcileAgentPod","namespace":"maintenance","time":"2024-05-30T09:57:21.741Z","workflow":"custom-metrics-65qbp"}
{"level":"info","msg":"Updated phase Running -\u003e Succeeded","namespace":"maintenance","time":"2024-05-30T09:57:21.741Z","workflow":"custom-metrics-65qbp"}
{"level":"info","msg":"Marking workflow completed","namespace":"maintenance","time":"2024-05-30T09:57:21.741Z","workflow":"custom-metrics-65qbp"}
{"level":"error","msg":"unable to substitute parameters for metric 'count': failed to resolve {{steps.generate.outputs.result}}","namespace":"maintenance","time":"2024-05-30T09:57:21.742Z","workflow":"custom-metrics-65qbp"}
{"action":"deletePod","key":"maintenance/custom-metrics-65qbp-1340600742-agent/deletePod","level":"info","msg":"cleaning up pod","time":"2024-05-30T09:57:21.747Z"}
{"action":"deletePod","error":"pods \"custom-metrics-65qbp-1340600742-agent\" not found","key":"maintenance/custom-metrics-65qbp-1340600742-agent/deletePod","level":"warning","msg":"failed to clean-up pod","time":"2024-05-30T09:57:21.773Z"}
{"level":"warning","msg":"Non-transient error: pods \"custom-metrics-65qbp-1340600742-agent\" not found","time":"2024-05-30T09:57:21.773Z"}
{"level":"info","msg":"Workflow update successful","namespace":"maintenance","phase":"Succeeded","resourceVersion":"448906","time":"2024-05-30T09:57:21.774Z","workflow":"custom-metrics-65qbp"}

Logs from in your workflow's wait container
# kubectl logs -c wait -l workflows.argoproj.io/workflow=custom-metrics-65qbp,workflow.argoproj.io/phase!=Succeeded
{"deadline":"0001-01-01T00:00:00Z","includeScriptOutput":true,"level":"info","msg":"Executor initialized","namespace":"maintenance","podName":"custom-metrics-65qbp-generator-967939906","templateName":"generator","time":"2024-05-30T09:57:03.961Z","version":"\u0026Version{Version:untagged,BuildDate:2024-05-29T20:02:01Z,GitCommit:fce51dfd0d49c521102a2c1a8d661fab742578a5,GitTag:untagged,GitTreeState:clean,GoVersion:go1.21.10,Compiler:gc,Platform:linux/amd64,}"}
{"level":"info","msg":"Starting deadline monitor","time":"2024-05-30T09:57:04.067Z"}
{"error":null,"level":"info","msg":"Main container completed","time":"2024-05-30T09:57:05.067Z"}
{"level":"info","msg":"Capturing script output","time":"2024-05-30T09:57:05.067Z"}
{"level":"info","msg":"Saving output parameters","time":"2024-05-30T09:57:05.067Z"}
{"level":"info","msg":"Saving path output parameter: count","time":"2024-05-30T09:57:05.067Z"}
{"level":"info","msg":"Copying /tmp/count from base image layer","time":"2024-05-30T09:57:05.067Z"}
{"level":"info","msg":"Successfully saved output parameter: count","time":"2024-05-30T09:57:05.067Z"}
{"level":"info","msg":"No output artifacts","time":"2024-05-30T09:57:05.067Z"}
{"level":"info","msg":"Alloc=6542 TotalAlloc=12745 Sys=24165 NumGC=5 Goroutines=8","time":"2024-05-30T09:57:05.126Z"}
{"level":"info","msg":"Starting Workflow Executor","time":"2024-05-30T09:57:14.037Z","version":"untagged"}
{"Duration":1000000000,"Factor":1.6,"Jitter":0.5,"Steps":5,"level":"info","msg":"Using executor retry strategy","time":"2024-05-30T09:57:14.040Z"}
{"deadline":"0001-01-01T00:00:00Z","includeScriptOutput":false,"level":"info","msg":"Executor initialized","namespace":"maintenance","podName":"custom-metrics-65qbp-printer-671601449","templateName":"printer","time":"2024-05-30T09:57:14.040Z","version":"\u0026Version{Version:untagged,BuildDate:2024-05-29T20:02:01Z,GitCommit:fce51dfd0d49c521102a2c1a8d661fab742578a5,GitTag:untagged,GitTreeState:clean,GoVersion:go1.21.10,Compiler:gc,Platform:linux/amd64,}"}
{"level":"info","msg":"Starting deadline monitor","time":"2024-05-30T09:57:14.124Z"}
{"error":null,"level":"info","msg":"Main container completed","time":"2024-05-30T09:57:16.125Z"}
{"level":"info","msg":"No Script output reference in workflow. Capturing script output ignored","time":"2024-05-30T09:57:16.125Z"}
{"level":"info","msg":"No output parameters","time":"2024-05-30T09:57:16.125Z"}
{"level":"info","msg":"No output artifacts","time":"2024-05-30T09:57:16.125Z"}
{"level":"info","msg":"Alloc=7203 TotalAlloc=12720 Sys=20069 NumGC=4 Goroutines=8","time":"2024-05-30T09:57:16.179Z"}
{"level":"info","msg":"Deadline monitor stopped","time":"2024-05-30T09:57:16.205Z"}

see the below doc, it only support for the argo template level.
https://argo-workflows.readthedocs.io/en/stable/metrics/#metric-spec


Add poll timeout config for kafka consumer

Closes issue206

[ ] I incremented the version in the VERSION file
[ ] I updated CHANGELOG.md
[ ] I added test coverage for any changes

We are going close this out since #218 contains this and has some missing pieces.


WIP: Ensure prefix is picked up on Windows so pyrcc5 runs

Checklist

[x] Used a fork of the feedstock to propose changes
[x] Bumped the build number (if the version is unchanged)
[ ] 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.

xref: https://github.com/conda-forge/qgis-feedstock/pull/86
In pyqt 5.9, pyrcc5 is now a .bat file. The path to Python is written into it during the build, but because there is no space after the @ symbol (I think) conda-build isn't picking up that this needs to be replaced at install time like it does on Unix.

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

@conda-forge-admin, please rerender

Thanks @gillins!


Add logged warnings for elevated launch on windows

Resolve issue #5: Add warning when running as an elevated processes

Adds a warning when launched as admin on windows:

11:53:37 warn: Microsoft.Diagnostics.Tools.Monitor.Startup[21]
      The process was launched as an administrator level and will have access to all processes on the system. The dotnet-monitor process should be launched as the same user as the target dotnet process.

Adds warning when launched as admin on windows and NTLM auth is enabled:

11:53:37 warn: Microsoft.Diagnostics.Tools.Monitor.Startup[22]
      While running as administrator NTLM authentication is not possible, NTLM has been disabled.

@wiktork @shirhatti, did we want this for Preview 4 (the upcoming preview)? If so, this should target release/5.0 and then be backported to main.

I don't think it's necessary to re-target. Preview5 is fine


Add tests and add action to run them on PR

Initial setup and first test done in #28

More tests in #30

Closing this for now, more tests will come as we go.


Redirect URL - b2clogin.com/..../oauth2/authresp unavailable

When configuring the OIDC IdP with the specified redirect URI (as in https://docs.microsoft.com/en-us/azure/active-directory-b2c/openid-connect-technical-profile#redirect-uri), the error message "The resource you are looking for has been removed, had its name changed, or is temporarily unavailable." is shown. It appears that this URL is not configured. When adding ?p=, a valid response is given. Is the documentation not up to date or is there something wrong in B2C?

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

ID: 8cb586f8-c1ff-7676-08b3-915ee00383b0
Version Independent ID: f1770ad7-7ee4-090b-78f8-85924233b190
Content: Define an OpenID Connect technical profile in a custom policy - Azure AD B2C
Content Source: articles/active-directory-b2c/openid-connect-technical-profile.md
Service: active-directory
Sub-service: b2c
GitHub Login: @msmimart
Microsoft Alias: mimart

@TaekeK
Thanks for your feedback! We will investigate and update as appropriate.

@TaekeK
Thank you for your time and patience throughout this issue!
I replicated the error message that you're seeing on my end, and also tried the Redirect URLs from our Set redirect URLs to b2clogin.com for Azure Active Directory B2C documentation, and ran into the same issue. I was also not able to add a query string Redirect URI to my App, can you share what you used when it comes to "?p=" so I can try to see if it'll work on my end?

If you have any other questions, please let me know.
Thank you for your time and patience throughout this issue.

Hi Jamea,
I added ?p=B2C_1A_SIGNIN to the URL, which gave a proper response in the browser. I'm still waiting for the IdP to add this as a redirect URL, so I have not tested this end-to-end.
There is a setting in the B2C policy that configures the redirect behavior,  but that did not work as expected.
false indicates that a policy is not needed in the URL, but it doesn't work and ends up in the error described in the OP.
Regards,
Taeke

From: James Tran-MSFT @.***>
Sent: Monday, 12 July 2021 22:19
To: MicrosoftDocs/azure-docs
Cc: Taeke Kooiker; Mention
Subject: Re: [MicrosoftDocs/azure-docs] https://{your-tenant-name}.b2clogin.com/{your-tenant-name}.onmicrosoft.com/oauth2/authresp unavailable (#78145)
@TaekeKhttps://github.com/TaekeK
Thank you for your time and patience throughout this issue!
I replicated the error message that you're seeing on my end, and also tried the Redirect URLs from our Set redirect URLs to b2clogin.com for Azure Active Directory B2Chttps://docs.microsoft.com/en-us/azure/active-directory-b2c/b2clogin documentation, and ran into the same issue. I was also not able to add a query string Redirect URI to my App, can you share what you used when it comes to "?p=" so I can try to see if it'll work on my end?
[image]https://user-images.githubusercontent.com/36619112/125350087-233cc100-e313-11eb-9296-1e375289c731.png
[image]https://user-images.githubusercontent.com/36619112/125350262-61d27b80-e313-11eb-95b9-943223ae1b2a.png
If you have any other questions, please let me know.
Thank you for your time and patience throughout this issue.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/MicrosoftDocs/azure-docs/issues/78145#issuecomment-878567394, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ABBGZINSSZFBR7TXRP5TUD3TXNE5HANCNFSM5AC3I3MA.

@TaekeK
Thank you for the quick follow up on this!
When testing out your app, have to tried using the https://jwt.ms reply URL, to see if that lets you complete the sign-in?

If you have any other questions or would like to work closer with our support team on this, please let me know.
Thank you for your time and patience throughout this issue.

For this setup we make use of an external IdP that offers OIDC integration, for which we followed the documentation provided to integrate with Azure AD B2C. It's not an app in a tenant we control, so I cannot add a redirect to jwt.ms.
Sent from Ninehttp://www.9folders.com/

From: James Tran-MSFT @.***>
Sent: Monday, 12 July 2021 23:11
To: MicrosoftDocs/azure-docs
Cc: Taeke Kooiker; Mention
Subject: Re: [MicrosoftDocs/azure-docs] https://{your-tenant-name}.b2clogin.com/{your-tenant-name}.onmicrosoft.com/oauth2/authresp unavailable (#78145)
@TaekeKhttps://github.com/TaekeK
Thank you for the quick follow up on this!
When testing out your app, have to tried using the https://jwt.ms reply URL, to see if that lets you complete the sign-in?
[image]https://user-images.githubusercontent.com/36619112/125356281-d957d900-e31a-11eb-9a12-86bbf0cdf2f5.png
[image]https://user-images.githubusercontent.com/36619112/125356178-b5949300-e31a-11eb-899d-4057ba7e1b00.png
If you have any other questions or would like to work closer with our support team on this, please let me know.
Thank you for your time and patience throughout this issue.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/MicrosoftDocs/azure-docs/issues/78145#issuecomment-878598147, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ABBGZIMIYZ3BUIQDIOAPJ43TXNK6HANCNFSM5AC3I3MA.

@TaekeK
Thank you for the quick follow up on this! I've reached out to our engineering team for additional inputs, and will update as soon as possible.
If you have any other questions, please let me know.
Thank you for your time and patience throughout this issue.

@TaekeK
Thank you for your time and patience throughout this issue!
I worked with our engineering team and we found that the documentation is currently up to date, and the redirect URL is working as expected.
I used a test app, and realized that the returned JWT is within the actual URL when using the /oauth2/authresp URL:

And in the webpage when using the jwt.ms redirect URL:

When it comes to this issue, we think it's best that our support team takes a closer look into your environment, so we can gain a better understanding of your issue and help resolve this as soon as possible.
Can you please email me with the info below, I'll go ahead and enable a one-time free technical support request for your subscription so you can work with our support engineers to get this issue resolved.
Email<EMAIL_ADDRESS>Subject: ATTN - James Tran
Body:
Azure Subscription ID
Link to this issue
I'll go ahead and close this out and keep an eye out for your email.
Thank you again for your time and patience throughout this issue.

Hi James,
Thanks for your efforts. We don’t have the #id_token after the authresp, so that might be the issue.
Met vriendelijke groet,
Taeke Kooiker
@.***
Maatschap Fortigi
Gargouille 47  |  4007 RE  |  Tiel
KVK: 67045294  |  BTW: NL856807151B01  |  Bank: NL26 INGB 0007 9055 37
M: +31 6 49 135 975tel:+31649135975  |  T: +31 85 060 2215tel:+31850602215  |  E<EMAIL_ADDRESS>|  W: www.fortigi.nlhttp://www.fortigi.nl/
From: James Tran-MSFT @.>
Sent: Tuesday, July 20, 2021 22:14
To: MicrosoftDocs/azure-docs @.>
Cc: Taeke Kooiker @.>; Mention @.>
Subject: Re: [MicrosoftDocs/azure-docs] Redirect URL - b2clogin.com/..../oauth2/authresp unavailable (#78145)
@TaekeKhttps://github.com/TaekeK
Thank you for your time and patience throughout this issue!
I worked with our engineering team and we found that the documentation is currently up to date, and the redirect URL is working as expected.
I used a test app, and realized that the returned JWT is within the actual URL when using the /oauth2/authresp URL:
[image]https://user-images.githubusercontent.com/36619112/126388794-8c8aada3-e160-49a1-82c3-88c679bf5760.png
And in the webpage when using the jwt.ms redirect URL:
[image]https://user-images.githubusercontent.com/36619112/126388823-5692d1d1-b45e-4c2b-88ea-14c1f20965c5.png

When it comes to this issue, we think it's best that our support team takes a closer look into your environment, so we can gain a better understanding of your issue and help resolve this as soon as possible.
Can you please email me with the info below, I'll go ahead and enable a one-time free technical support request for your subscription so you can work with our support engineers to get this issue resolved.
Email<EMAIL_ADDRESS>Subject: ATTN - James Tran
Body:
Azure Subscription IDhttps://wmatthyssen.com/2019/09/06/azure-tip-how-to-quickly-find-your-azure-subscription-id-guid/
Link to this issue
I'll go ahead and close this out and keep an eye out for your email.
Thank you again for your time and patience throughout this issue.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/MicrosoftDocs/azure-docs/issues/78145#issuecomment-883665113, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ABBGZIL6LFOLST7MPQW5EE3TYXKJDANCNFSM5AC3I3MA.


12228 - BC Registry - Add Rural Property and Wills Registry to the platform

Issue #:
https://github.com/bcgov/entity/issues/12228
Add in RPT and VS.
Minor styling tweaks.

/gcbrun

Temporary Url for review:https://yfthig-dev--pr-76-ledpopz8.web.app

I'd add another reviewer if I could.. but it's disabled.

This project needs some other people with write permission :)
In the meantime, if you ask someone to review and they add a comment, they automatically become reviewers.

/gcbrun

Should be "Rural Property Tax Search" instead of "Rural Property Tax" - hopefully we can get this link/icon to still fit onto a single line. This header title update should be made to the "moved" Rural tile that's under BC Online as well.

You can Tracey's comment in context here: https://github.com/bcgov/bcregistry/commit/0caec674d7926821dbebccc67ece2dcb3e8018db

@tlebedovich
Seeing the thumbs up from you on the picture.


Privacy Manifest

New issue checklist

[x] I have reviewed the README and documentation
[x] I have searched existing issues and this is not a duplicate
[x] I have attempted to reproduce the issue and include an example project.

General information
From Apple docs:

Starting in spring 2024, you must include the privacy manifest for any SDK listed below when you submit new apps in App Store Connect that include those SDKs, or when you submit an app update that adds one of the listed SDKs as part of the update. Signatures are also required in these cases where the listed SDKs are used as binary dependencies.

Could you please add a privacy manifest for this library?

New issue checklist

[x]  I have reviewed the README and documentation
[x]  I have searched existing issues and this is not a duplicate
[x]  I have attempted to reproduce the issue and include an example project.

General information
From Apple docs:

Starting in spring 2024, you must include the privacy manifest for any SDK listed below when you submit new apps in App Store Connect that include those SDKs, or when you submit an app update that adds one of the listed SDKs as part of the update. Signatures are also required in these cases where the listed SDKs are used as binary dependencies.

Could you please add a privacy manifest for this library?

hello, if you on urgent need it you can pull from mine or use
pod 'IGListDiffKit', :git => 'https://github.com/cendolinside123/IGListKit', :branch => 'Jan/Privacy-Manifest-Support'
pod 'IGListKit', :git => 'https://github.com/cendolinside123/IGListKit', :branch => 'Jan/Privacy-Manifest-Support'

https://github.com/cendolinside123/IGListKit/tree/Jan/Privacy-Manifest-Support-Main-Branch
I create it from main branch

Thanks folks! I just landed a PR that adds the privacy manifest to the project and all subsequent dependency managers!

Thanks folks! I just landed a PR that adds the privacy manifest to the project and all subsequent dependency managers!

Hello, this will cause duplication of "PrivacyInfo.xcprivacy"，它应该像这样：
s.resource_bundles = { "#{s.name}_Privacy" => ['Source/PrivacyInfo.xcprivacy'] }


Can't find variable: self

First I had this error after importing a internal module
TypeError: undefined is not an object (evaluating 'GLOBAL.Text={
get defaultProps(){
throw getInvalidGlobalUseError('Text');}}')

Searching for a solution I found that removing es2015 from presets on .babelrc ( which in my case I use package.json )  would fix it but then I started to get this error.
Can't find variable: self

This is the file that is throwing the exception
// the whatwg-fetch polyfill installs the fetch() function
// on the global object (window or self)
//
// Return that as the export for use in Webpack, Browserify etc.
require('whatwg-fetch');
module.exports = self.fetch.bind(self);

I could't find any information about this so I decided to open I ticket.
Here are the package.json of my react-native project:
{
  "name": "xxxx",
  "version": "0.0.1",
  "private": true,
  "scripts": {
    "start": "node node_modules/react-native/local-cli/cli.js start"
  },
  "dependencies": {
    "client-core": "path/to/my/module/repository#myBrach",
    "immutable": "^3.7.6",
    "react-native": "^0.19.0",
    "react-native-swiper": "^1.4.3",
    "react-native-vector-icons": "^1.0.4",
    "react-redux": "^4.1.2",
    "react-timer-mixin": "^0.13.3"
  },
  "devDependencies": {
    "babel-preset-es2015": "^6.3.13",
    "babel-preset-react": "^6.3.13",
    "eslint-plugin-react": "^3.15.0"
  },
  "babel": {
    "presets": [
      "react"
    ]
  }
}

package.json of the module that I installed:
{
  "private": true,
  "version": "1.0.0",
  "description": "Client side logic core",
  "dependencies": {
    "isomorphic-fetch": "^2.2.0",
    "lodash": "^4.0.0",
    "moment": "^2.10.6",
    "nock": "^5.2.1",
    "ramda": "^0.19.0",
    "redux": "^3.0.4",
    "redux-logger": "^2.1.1",
    "redux-thunk": "^1.0.0"
  },
  "devDependencies": {
    "babel-core": "^6.3.15",
    "babel-plugin-transform-object-rest-spread": "^6.3.13",
    "babel-preset-es2015": "^6.3.13",
    "babel-preset-react": "^6.3.13",
    "chai": "^3.4.1",
    "chai-datetime": "^1.4.0",
    "deep-freeze": "0.0.1",
    "eslint": "^1.10.3",
    "eslint-plugin-mocha": "^1.1.0",
    "gulp": "^3.9.0",
    "gulp-babel": "^6.1.1",
    "gulp-babel-istanbul": "^0.11.0",
    "gulp-eslint": "^1.1.1",
    "gulp-mocha": "^2.2.0",
    "gulp-sequence": "^0.4.4",
    "isparta": "^4.0.0",
    "istanbul": "^0.4.2",
    "merge-stream": "^1.0.0",
    "mocha": "^2.3.4",
    "pre-commit": "^1.1.2",
    "sinon": "^1.17.2",
    "sinon-chai": "^2.8.0"
  },
  "scripts": {
    "test": "gulp",
    "test:watch": "mocha --compilers js:babel-core/register -w test/**/*.spec.js",
    "lint": "gulp lint"
  },
  "pre-commit": [
    "lint"
  ],
  "repository": {
    "type": "git",
    "url": "xxxxxx"
  },
  "author": "xxxx",
  "bugs": {
    "url": "xxxxxx"
  },
  "homepage": "xxxxx",
  "babel": {
    "presets": [
      "es2015",
      "react"
    ],
    "plugins": [
      "babel-plugin-transform-object-rest-spread"
    ]
  }
}

Closing this issue because it is related to:
https://github.com/github/fetch/issues/125
and
https://github.com/matthew-andrews/isomorphic-fetch/pull/63

In a way, I feel that this is something which needs to be addressed by React Native directly. If the code isn't being used, it shouldn't be loaded, and it shouldn't matter.

@billinghamj Do you think that I should reopen the issue?
I don't think that the discussion will be extended.

It is an ongoing issue with React Native. Even if it isn't likely to be addressed any time soon, it should probably still be thought about.

What is our current work around fo this issue ?
@billinghamj @billinghamj


Stack Implementation in C++

Would you like to work on the issue? Yes
Let us know if this issue should be assigned to you or tell us who you think could help to solve this issue.

I implement it to use the linked list :)


Issue 1456

Resolves #1456, but please see #1642 as it is relevant here.

Discussed on the OBI call 2023-01-09: Looks good.

@johnwjudkins Can you please double-check that I resolved the conflicts correctly?

@jamesaoverton Looks good, thanks!


Fix inconsistent command in the JSONPath Support page

Fixes: https://github.com/kubernetes/website/issues/42251

/lgtm

I confirmed that the proposed command change works along with the original command but the proposed command is in line with the previous example
We should note in the example that commands are used to retrieve pods with the prefix "test-", this can be used to retrieve pod names of a deployment called "test"

$ kubectl get pods -o json | jq -r '.items[] | select(.metadata.name | test("test-")).metadata.name'
test-c787ff6bc-d5jk4
test-c787ff6bc-v7ttz
$ kubectl get pods -o json | jq -r '.items[] | select(.metadata.name | test("test-")).spec.containers[].image'
nginx:1.7.9
nginx:1.7.9

/approve


add ability to plot a colour bar on level 4 animation

Proposed changes
Make nice colour bars for level 4 Land Cover Animations Possible
Previously they were a big mess. addition to the Landcover.py tools file makes them display correctly. Belle wasn't able to finish reviewing this for me before she left so hoping someone can have a look over it for me? I first opened this request with a separate fiction to generate level 4 colourbars but Belle requested I combine it with the existing function. I initially published our landcover animation code with-out the ability to put a colour bar on animations using level 4 (our full classification) because it is very unwieldy. I just want to give a tiny over view so when you look at this you have some context, hope that's ok?
The colour bars for animations have to be generated outside the FuncAnimation() function or a new one is generated with every frame, there fore they need to be generated not using matplotlibs standard colour bar generation procedure because i's won't automatically space it and the level 4 colour bar is very wide due to all the text, this is why I actually do it on a figure with 2 sub plots, so one can be left empty and the colour bar can go over it.
Our colour scheme for level 4 colour bars are different to what we use for the actual plots. why? because the full colour scheme has a number of classes we don't classify (yet?) and the numerical spacing between class numbers is uneven because it's categorical not continuous. Needed to make a different one for colour bars so the classes are even sizes.
Please let me know if you have any questions about anything.
Checklist (replace [ ] with [x] to check off)
[ X] Notebook created using the DEA-notebooks template
[X ] Remove any unused Python packages from Load packages
[ X] Remove any unused/empty code cells
[ X] Remove any guidance cells (e.g. General advice)
[ X] Ensure that all code cells follow the PEP8 standard for code. The jupyterlab_code_formatter tool can be used to format code cells to a consistent style: select each code cell, then click Edit and then one of the Apply X Formatter options (YAPF or Black are recommended).
Include relevant tags in the final notebook cell (refer to the DEA Tags Index, and re-use tags if possible)
[X ] Clear all outputs, run notebook from start to finish, and save the notebook in the state where all cells have been sequentially evaluated

Hey @eloise-b so the test that failed is because of the removal of C2 data, so tests run on the beginner notebooks now fail. shrug


Use method chaining for configuration definitions

Note: this PR is a good example case for #2723 :)

Please rebase

Rebased.

Thank you @julienfalque.


Implicit configuration and cabal files

A common use case is having a vanilla cabal project and getting hie-bios to pick up right ghc flags.
It doesn't work for me, presumably because my project doesn't compile yet, so cabal v2-repl doesn't give hie-bios what it needs.  It is fixable by adding an hie.yaml file, saying please use cabal, but that is clunky and I'm not sure why it tells hie-bios anything more than it could already work out for itself.

The implicit config is essentially looking for a *.cabal file and then using cabal repl File.hs to find the options. This is pretty much equivalent to a hie.yaml such as:
cradle:
  cabal:

What did you write in your hie.yaml?
Did it discover a different cradle or did it just fail to load?

Sure.  It works if I just use a hie.yaml cabal file like that in the readme here, but it doesn't seem to use the cabal fille if the hie.yaml isn't there.  I thought that the point was that it should but only if cabal v2-repl works, which it may not if the project doesn't compile.  But I'm speculating.

Please add some more details, what was the project you were trying to use? Did you try, e.g. hie-bios debug Lib.hs to see the details what cradle is selected, etc...
Why do you think it doesnt use cabal?

@jrp2014 what happens if you use implicit-hie generated hie.yaml?

With just a '.cabal' file and the following 'hie.yaml':
cradle: {cabal: {component: "lib:smuggler"}}

I get, for hie-bios debug
Root directory:      /Users/.../Haskell/smuggler
GHC options:         -fbuilding-cabal-package -O0 -outputdir ...
Config Location:     No explicit config found
Cradle:              Cradle {cradleRootDir = "/Users/.../Haskell", cradleOptsProg = CradleAction: Default}
Dependencies:        smuggler.cabal cabal.project

That output also contains some random characters, it seems, which makes it harder to cut & paste into here, but that's a different issue.
Without that hie-bios file I get
Root directory:      /Users/.../Haskell/smuggler
GHC options:
System libraries:    /usr/local/Cellar/ghc/8.8.3/lib/ghc-8.8.3
Config Location:     No explicit config found
Cradle:              Cradle {cradleRootDir = "/Users/.../Haskell", cradleOptsProg = CradleAction: Default}
Dependencies:

Please post your cabal file

@jrp2014 You should pass a filepath to hie-bios debug e.g. src/Lib.hs.

(I should say that my expectation was that the two outputs should be the same.)
Here's the cabal file.  (NB: It's not the same as the original smuggler one pointed to as the source repo' its a fork.)
cabal-version:   2.4
name:            smuggler
version:         <IP_ADDRESS>
synopsis:        GHC Source Plugin that helps to manage imports
description:
  == Usage
  .
  Add @smuggler@ to the dependencies of your project.
  .
  Then add the following to ghc-options: @-fplugin=Smuggler.Plugin@

homepage:        https://github.com/kowainik/smuggler
bug-reports:     https://github.com/kowainik/smuggler/issues
license:         MPL-2.0
license-file:    LICENSE
author:          Dmitrii Kovanikov, Veronika Romashkina
maintainer:      Kowainik<EMAIL_ADDRESS>copyright:       2018-2019 Kowainik
category:        Development, Refactoring
build-type:      Simple
extra-doc-files:
  README.md
  CHANGELOG.md

tested-with:     GHC ==8.8.3

flag debug
  description: Enable debugging support
  default:     False
  manual:      True

source-repository head
  type:     git
  location: git@github.com:kowainik/smuggler.git

common common-options
  build-depends:      base ^>=4.13
  ghc-options:
    -Wall -Wextra -Wincomplete-uni-patterns -Wincomplete-record-updates
    -Wcompat -Widentities -Wredundant-constraints -fhide-source-paths

  if flag(debug)
    ghc-options: -ddump-minimal-imports

  default-language:   Haskell2010
  default-extensions:
    DeriveGeneric
    GeneralizedNewtypeDeriving
    InstanceSigs
    LambdaCase
    OverloadedStrings
    RecordWildCards
    ScopedTypeVariables
    StandaloneDeriving
    TypeApplications

library
  import:          common-options
  hs-source-dirs:  src
  exposed-modules:
    Smuggler.Anns
    Smuggler.Export
    Smuggler.Import
    Smuggler.Loc
    Smuggler.Options
    Smuggler.Parser
    Smuggler.Plugin

  if flag(debug)
    exposed-modules:
      Smuggler.Debug
      Smuggler.Name

  -- Does not compile with 8.8.3      Smuggler.Import
  build-depends:
    , containers      >=0.5   && <0.7
    , filepath        ^>=1.4
    , ghc             ^>=8.8.0
    , ghc-exactprint  ^>=0.6
    , syb

  if flag(debug)
    build-depends:
      , fmt
      , pretty-simple
      , syb
      , text

executable play-smuggler
  import:         common-options
  hs-source-dirs: app
  main-is:        Main.hs
  ghc-options:    -threaded -rtsopts -with-rtsopts=-N

  if flag(debug)
    ghc-options: -fplugin=Smuggler.Plugin

  build-depends:  smuggler

common common-tests
  import:         common-options
  hs-source-dirs: test
  other-modules:
    --Test.BoolBoolUnused
    --Test.BoolBoolUsed
    --Test.BoolPartlyUsedFirst
    --Test.BoolPartlyUsedLast
    --Test.ConstructorsUnused
    --Test.Dummy
    --Test.ExportExplicit
    --Test.ExportImplicit
    Test.ImplicitImportUnused
    --Test.ImplicitImportUsed
    --Test.ImportEmpty
    --Test.MultilineUnused
    --Test.TypeConstructorUnused
    --Test.TypeConstructorUnusedComma
    --Test.TypeConstructorUsed
    --Test.TypeUnused
    --Test.TypeUsed
    --Test.WildcardUnused

  build-depends:
    , ansi-terminal
    , directory
    , filepath
    , smuggler

  ghc-options:
    -threaded -rtsopts -with-rtsopts=-N -fno-warn-unused-imports
    -fno-warn-missing-signatures -fplugin=Smuggler.Plugin
    -fplugin-opt=Smuggler.Plugin:test

test-suite smuggler-test
  import:  common-tests
  type:    exitcode-stdio-1.0
  main-is: Test.hs
  ghc-options:
    -fplugin-opt=Smuggler.Plugin:test
 --   -fplugin-opt=Smuggler.Plugin:NoExportProcessing
 --    -fplugin-opt=Smuggler.Plugin:MinimiseImports

test-suite smuggler-test-maximal
  import:  common-tests
  type:    exitcode-stdio-1.0
  main-is: Test.hs
  ghc-options:
    -fplugin-opt=Smuggler.Plugin:test
    -fplugin-opt=Smuggler.Plugin:MinimiseImports
--    -fplugin-opt=Smuggler.Plugin:ReplaceExports

@jrp2014 You should pass a filepath to hie-bios debug e.g. src/Lib.hs.

That seems to produce identical output without the hie.yaml file.

@jrp2014 Yeah it should and the CradleAction should be Cabal in both cases.
What are you trying to do and what fails exactly?

What happens if you use this hie.yaml
cradle:
  cabal:
    - path: "src"
      component: "lib:smuggler"
    - path: "play-smuggler"
      component: "exe:app"
    - path: "smuggler-test"
      component: "test:."
    - path: "smuggler-test-maximal"
      component: "test:."

What are you trying to do and what fails exactly?

What I am trying to do is fix this  https://github.com/dense-analysis/ale/blob/master/ale_linters/haskell/cabal_ghc.vim linter for ale, a vim plugin.
It seems to be broken because it doesn't supply a project root directory and even when you fix it to do that, ghc complains because it cannot find the hidden package smuggler, presumably because is trying to compile a temporary version version of the file you are editing (outside the project).  It is easy to fix for a specific project: just add -package smuggler for example, to the ghc command that it uses and it seems to work well.
I was hoping that hie-bios would generate the right information (project root, ghc flags) to make the linter go.  If I have to create a seemingly redundant hie.yaml file to get things going, I may as well just hack cabal_ghc.vim for each project and be done.
@Avi-D-coder The minimal hie.yaml file seems to work OK.  My object is not to have to create it, when hie-bios should just use the cabal file that the minimal hie.yaml file already points to.

Fixed by #221


Authenticated proxy feature and/or example

I am trying to connect to a authenticated proxy which has the below structure.
http_proxy=http://<username>@<domain>:<password>@<hostname>:<port>
I wasn't able to figure out how to add credentials to proxy, please provide some information on this. If this feature is not present I am willing to help on this, one of our components in project is blocked because of this.

I believe it uses the standard authentication configuration.
@dwclark can you confirm this?

Ah, the fun of dealing with the corporate proxy.
There are two ways of tackling this problem. Approach 1: add your proxy credentials directly to http builder ng. The user guide shows how to do this here:
https://http-builder-ng.github.io/http-builder-ng/asciidoc/html5/#_proxying_connections
For the native Java client, it uses this constructor of the java.net.Socket class:
https://docs.oracle.com/javase/8/docs/api/java/net/Socket.html#Socket-java.net.Proxy-
to proxy all traffic. Apache and OkHTTP have their own way of using proxies, which http builder ng supports with the same configuration. I'm guessing that if you look in the source code for both of those libraries they end up using the same mechanism for proxying, but I have not looked at the source code for those projects.
Approach 2: Force the JVM to proxy all connections at the socket level by specifying http proxy information as system properties. To do this set the following system properties at JVM startup time:
-Dhttp.proxyHost=<> -Dhttp.proxyPort=<> -Dhttp.proxyUser=<> -Dhttp.proxyPassword=<> -Dhttps.proxyHost=<> -Dhttps.proxyPort=<> -Dhttps.proxyUser=<> -Dhttps.proxyPassword=<>
Once you do this all sockets created in the JVM will go through the proxy you set (be sure to replace '<>' with the proper information). This again should work regardless of whether you are using native java, apache, or okhttp, the proxying mechanism is transparent to the library.

 OK, the second approach definitely supports that. Let me look and see if/how that is supported by the library level settings.

I just checked for java native and apache. If you set up basic authentication for the connection, it will use those credentials for authenticating with the proxy. If this is not working, we need to file a bug report and fix it.
We might also want to consider separating basic auth from proxy auth. There are valid cases where the user name and password might be different. However, supporting this is not going to be fun.

Same for ok http. Again, if it doesn't work, we need to file a bug report, fix, and release.

The second approach I suggested is the Java equivalent of setting http_proxy environmental variables, just setting Java system properties instead. It's good that at least that OkHttp works.

@dwclark I've tried approach 2 too, JVM doesn't understand proxyUser and proxyPassword so I've used Authenticator.setDefault() but doesn't work because of the issue described previously.

Weird, setting http.proxyUser and http.proxyPassword has worked for me in the past.

@dwclark See updated snippet from the previous comment. I am already passing the required properties. JVM seems to understand proxyHost and proxyPort but not username and password.

I wonder why that is. It may be that the basic authentication you are using is overwriting the credentials for the proxy. If I remember correctly, that part of java authentication is not that well designed. When I get some time I'll investigate.

For the java system properties I've got help from:

https://stackoverflow.com/questions/1626549/authenticated-http-proxy-with-java
https://blogs.oracle.com/wssfc/handling-proxy-server-authentication-requests-in-java
https://docs.oracle.com/javase/8/docs/technotes/guides/net/proxies.html

I will add this to the trouble-shooting documentation in our next release.

@cjstehno Since this only works with okhttp I feel this should be a feature/enhancement.


Pub/Sub: Subscriber Unary Ack/ModAck

Send Subscriber's Ack/ModAck messages using normal RCP API calls instead of using the GRPC stream. This will relieve some GRPC stream congestion. An empty message is pushed to the GRPC stream every 30 seconds to keep the GRPC stream alive if there are messages being processed.

@jba PTAL

cc @jba


Set specific module name when exporting a JAX model

Request description
I'm exporting several JAX functions through IREE so I can run them via the IREE runtime, using this function
def aot(function, static_argnums, *args, **options):
    """Traces and compiles a function, flattening the input args.

    This is intended to be a lower-level interface for compiling a JAX function to
    IREE without setting up the runtime bindings to use it within Python. A common
    usecase for this is compiling to Android (and similar targets).

    Args:
      function: The function to compile.
      args: The inputs to trace and compile the function for.
      **kwargs: Keyword args corresponding to xla.ImportOptions or CompilerOptions
    """
    xla_comp = jax.xla_computation(function, static_argnums=static_argnums)(*args)
    xla_comp = jax.jit(function, static_argnums=static_argnums).lower(*args)
    ir = str(xla_comp.compiler_ir())
    ir = re.sub(r'(module @).* {', 'module @module {', ir)  # TODO write why we did this
    return iree.compiler.tools.compile_str(ir, input_type=iree.compiler.InputType.XLA, **options)

The thing is that the initial aot function, ignoring ir = re.sub(r'(module @).* {', 'module @module {', ir), generates a random-ish name for the module
module @jit_f.3 {
  func.func public @main(%arg0: tensor<16x64x128xf16>, %arg1: tensor<16x128x3072xf16>) -> tensor<16x64x3072xf16> {
    %0 = "mhlo.dot_general"(%arg0, %arg1) {dot_dimension_numbers = #mhlo.dot<lhs_batching_dimensions = [0], rhs_batching_dimensions = [0], lhs_contracting_dimensions = [2], rhs_contracting_dimensions = [1]>, precision_config = [#mhlo<"precision DEFAULT">, #mhlo<"precision DEFAULT">]} : (tensor<16x64x128xf16>, tensor<16x128x3072xf16>) -> tensor<16x64x3072xf16>
    return %0 : tensor<16x64x3072xf16>
  }
}

This makes our function that loads functions in the runtime to not be able to be generic because of
IREE_RETURN_IF_ERROR(iree_runtime_call_initialize_by_name(
      session, iree_make_cstring_view("module.main"), &call));

Therefore, the regex substitution over there is used to replace the random name with the module name so we don't have to save the module name as well.
Is there a cleaner way to do this? Is there a parameter I can add somewhere in order to also provide a name for the module?
What component(s) does this issue relate to?
MLIR, Compiler
Additional context
Versions used:
jax==0.3.17
jaxlib==0.3.15
-f https://github.com/iree-org/iree/releases/expanded_assets/candidate-20220930.282
--extra-index-url https://github.com/iree-org/iree/releases/expanded_assets/candidate-20220930.282
iree-compiler
iree-runtime
iree-tools-tf
iree-tools-tflite
iree-tools-xla

@rsuderman Can you take a look at this and triage? Or pass along to another owner?


What is seed?

Hello,
I am new to few-shot object detection, I didn't understand the terminology “seed”. Could you please tell what exactly it is?

I found it out.


Make actor-persistent store of email senders

Players should be able to pivot from domain -> senders -> more domains


What are the labels training the depth estimator

Thank you for your great work!
Do you train (fine-tune) the depth network only with the depth values at the keypoint locations predicted by the 2D pose estimator? It puzzle me that only a number of pixels in the depth map are supervised during training can be successful (the training labels seem sparse).

The network is pretrained on a dataset which has dense labels. Then it is
finetuned for the pose estimation task using sparse labels only.
Jia Li @.***> ezt írta (időpont: 2021. dec. 21., Ke
5:54):

Thank you for your great work!
Do you train (fine-tune) the depth network only with the depth values at
the keypoint locations predicted by the 2D pose estimator? It puzzle me
that only a number of pixels in the depth map are supervised during
training can be successful (the training labels seem sparse).
—
Reply to this email directly, view it on GitHub
https://github.com/vegesm/depthpose/issues/3, or unsubscribe
https://github.com/notifications/unsubscribe-auth/ADNQKV7LMHKLIW5TWQ6645TUSAB7FANCNFSM5KPIEFGA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

Thank you! Thus, the depth network was fine-tuned by the values of Z coordinates of the keypoints.


How to avoid deadlock in fairlock?

Hello,
I'm using redission 3.17.7 and I want to using fair lock.
Normally it works perfectly
But, In case of unexpected application termination or deploy, the pending two key(redisson_lock_queue, redisson_lock_timeout)  remains.
So after deployment, even if you try to preempt the lock, you can't preempt it.
Is there any way to release for 2 keys (redisson_lock_queue, redisson_lock_timeout) ?
It happens in the following scenarios:
This is simple code in SpringBoot.
@RestController
public class TestController {
...
    @GetMapping("v1/test")
     public void peterTest() {
         String lockKey = "test";
         RLock lock = redissonClient.getFairLock(lockKey)
         lock.tryLock(10000, 2000, TimeUnit.MILLISECONDS);
         Thread.sleep(1000);
         lock.unlock();
   }
}

Application start and execute the send a request in multiple terminal window below.

curl -X 'GET' 'http://localhost:8080/v1/test'

Check 2 keys ( redisson_lock_queue, redisson_lock_timeout ). and application shut down.  You can check if the data is still there.
waiting a minute.
Application start.
Send a Request (http://localhost:8080/v1/test) but always false in tryLock()
2 keys (redisson_lock_queue, redisson_lock_timeout) still have values

How to acquire a lock in the following situations?
Thank you.

Hi, have you found any workaround for this by any chance? We are planning to user Redisson for synchronization using fair-locks but this issue prevents us from doing so, because it can break whole backend.

Fixed in https://github.com/redisson/redisson/issues/4153. Please upgrade to 3.31.0 or higher version


Group imports by their visiblities

Hi, this is a feature request.
I'm using group_imports: One and I found that it's difficult to distinguish re-exported imports at glance.
Unordered:
use chrono::Utc;
use super::update::convert_publish_payload;

pub use juniper::{FieldError, FieldResult};
use uuid::Uuid;
use alloc::alloc::Layout;

use std::sync::Arc;

use broker::database::PooledConnection;

use super::schema::{Context, Payload};
use core::f32;
use crate::models::Event;

Ordered:
use super::schema::{Context, Payload};
use super::update::convert_publish_payload;
use crate::models::Event;
use alloc::alloc::Layout;
use broker::database::PooledConnection;
use chrono::Utc;
use core::f32;
pub use juniper::{FieldError, FieldResult};
use std::sync::Arc;
use uuid::Uuid;

I'm proposing another variant Visiblity that will group them by their visibilities.
Unordered:
pub(super) use chrono::Utc;
pub use super::update::convert_publish_payload;

pub(crate) use juniper::{FieldError, FieldResult};
pub(in crate::models) use uuid::Uuid;
use alloc::alloc::Layout;

use std::sync::Arc;

pub(self) use broker::database::PooledConnection;

use super::schema::{Context, Payload};
use core::f32;
pub use crate::models::Event;

Ordered:
use super::schema::{Context, Payload};
use alloc::alloc::Layout;
use core::f32;
use std::sync::Arc;

pub(self) use broker::database::PooledConnection;

pub(super) use chrono::Utc;

pub(crate) use juniper::{FieldError, FieldResult};

pub(in crate::models) use uuid::Uuid;

pub use super::update::convert_publish_payload;
pub use crate::models::Event;

I've made a patch here.
What do you think?

@kafji Thanks for reaching out! This feels like a duplicate of #4070 so I'm going to close this issue.
PRs are always welcome, but I want to be up front and say that there are higher priorities on the teams backlog and it might be a while before we're able to give it a review.


US26: On the Menu screen, there are "Leaderboard", "Friends" and "Profile" tabs, to which the user can navigate.

Closed with commit: d5d91232ea0166433ed32c7a35821b5324763a72


Make SwiftValue hash/equality work the same as SwiftObject

TL;DR:  Update PR #68720 with lessons learned in reviewing #69464
Background:

SwiftValue can expose Swift value types (structs/enums) to ObjC by wrapping them in an Obj-C object on the heap

SwiftObject is the Obj-C type that Swift class objects all inherit from (when viewed from Obj-C).  This allows arbitrary Swift class objects to be passed into Obj-C.

History:

PR #4124 made Obj-C -hash and -isEqual: work for SwiftValue for Hashable Swift types

PR #68720 extended SwiftValue to also support Equatable Swift types

PR #69464 added similar support to SwiftObject

In the process of working through #69464, we found a better way to handle an ObjC request for -hash for a type that is Swift Equatable but not Hashable.  This PR updates SwiftValue to use the same approach.  The approach considers three cases:

A Hashable type can forward both -hash and -isEqual: to the Swift object.

A type that is neither Equatable nor Hashable can implement -isEqual: as the identity check and -hash as returning the address of the object in memory.

A type is that Equatable but not Hashable is more complex.

In this last case, we can easily forward -isEqual: to the Equatable conformance but ObjC also requires us to always provide a compatible -hash implementation. The only way to do this is to have -hash return a constant, but that is a very bad idea in general, so we're also including a log message whenever we see a request for -hash on a Swift value that is Equatable but not Hashable.
To limit performance problems from the logging itself, we emit the log message only once for each type.
Testing:  I copied the fairly extensive SwiftObjectNSObject test suite over as SwiftValueNSObject in order to verify the new behaviors and help guard against future regressions.

Yeah, @lorentey also suggested that in the discussion for #69464.  I opted for the warning approach since this isn't a fundamental safety problem.

@swift-ci please test

The overly-expansive SwiftValueNSObject test suite had a few cases that aren't really ready.  I commented those out.

@swift-ci Please test

@swift-ci Please test Windows platform

@swift-ci Please test Windows

@swift-ci Please test Windows


nixpkgs-21.05-darwin upgrade python 3.9.4 -> 3.9.6

@mweinelt, python update is needed, because jupyter is not working with current version on MacOS BigSur and you cannot change python as an "application developer" though overrides, because you end up with duplicated package error.
Python 3.10 is not supported by jupyter or its dependencies yet, so current state is pretty bad for python.

I am not a nix expert, but it might be a problem with binary cache.
It seems to me that 3.9.4 just need to be rebuild for MacOS from scratch, but it is loaded from cache.
nix-shell --check is not working on MacOS so it is hard to prove my assumption.

Superseded by https://github.com/NixOS/nixpkgs/pull/133252.


added dir support for artifacts clause

Addresses #48 . Now you can add a directory as the target for an artifacts clause and all contents are preserved. The downside is that most containers are running as (fake) root within the container so the UID/GID in the tar header will report as root/root, which could be undesirable? It seems that the common use case for the artifacts clause it to allow access across stages and not preserve the artifact after the image build is completed... so there should be no issue.

great stuff. looks good to me. i will merge, run some tests and will release as soon as possible


Move frontend, lambda & terraform sources to src/

Purpose
Per #96, we're moving sources to an src folder, leaving only repository, docker & docs related files in the root.
This PR starts this moving process by moving everything except backend files to src.
Proposal

[x] Move frontend files to src/frontend
[x] Move the terraform folder to src/terraform
[x] Move each lambda to its own folder eg src/lambda-encode

I would say presets belong more to terraform than to lambdas...
Actually I'm not entirely convinced yet of the split terraform//lambdas. Lambdas are a dependency of terraform and the configure lambda is closely related to terraform configuration because it is configuring what terraform could not...

Would it make more sense if we grouped them in an "aws" directory?
Should we just rename terraform to "aws" and keep the lambdas inside?

I don't think lambdas should be hidden away in aws/source/(foo|bar|baz) folders. The idea of this src folder was to put all the projects in it to clean up the repo structure but avoid nesting them too deeply in a series of folders.
Your idea to have an aws folder would work best if we did away with src and instead put frontend backend and aws directly in the root.
Less neat but probably more practical. WDYT @jmaupetit ?

@jmaupetit @sampaccoud Ok I think this is done and follows our latest agreement

Only one point: the Makefile that you put in src/backend should remain at the root of the project IMO. It is the Makefile that builds the container with the Dockerfile that is at the root of the project... etc. WDYT?

Ping @jmaupetit @sampaccoud


[Suggestion] Spearfishing keeps missing on higher ping

Please add some kind of slider to the config menu to adjust how early it fires the gig, I tried it on a fast fish, it gets them in the bottom 2 layers, the 3rd one it misses sometimes and the top layer it misses every single time

I'll try to improve this, its a very experimental feature and kinda hard to balance the timing wi the way i did

You can now customize when the gig is activated by selecting a new hitbox size, this could help mitigate the latancy problem by adjusting it.


inspec & ssh question

I am a noob with respect to inspec so I apologize in advance.  It seems like kitchen-terraform works well when you are provisioning EC2 infrastructure that you can login to and perform your tests.  Let's say I wanted to use Terraform to provision just a security group or SNS topic.  Is there a way I can tell inspec to not SSH into the infrastructure to test but just test from my host.  I know if I was using inspec natively I could achieve this.

Hi @pantocrator27:
Thanks for your interest in the project!
What you're describing will be possible with the implementation of #7, which I'm hoping will be included in the next release.


Malformed calls from JS: Field size is diferent

Hi @nandorojo,
I hope you are doing well. I'm writing this regarding the error that I am encountering in the Android platform when using Burnt.alert() function. It is doing pretty okay in iOS platform. However, when I am switching to Android (Java 22.0.1, Gradle 88) and when I load the Burnt.alert() function, it encounters an error given in the picture below:

Here is the information that I have took using npx react-native info:
System:
  OS: macOS 14.2.1
  CPU: (10) arm64 Apple M1 Pro
  Memory: 153.95 MB / 16.00 GB
  Shell:
    version: "5.9"
    path: /bin/zsh
Binaries:
  Node:
    version: 22.4.1
    path: /opt/homebrew/bin/node
  Yarn: Not Found
  npm:
    version: 10.8.1
    path: /opt/homebrew/bin/npm
  Watchman:
    version: 2<IP_ADDRESS>
    path: /opt/homebrew/bin/watchman
Managers:
  CocoaPods:
    version: 1.15.2
    path: /opt/homebrew/bin/pod
SDKs:
  iOS SDK:
    Platforms:
      - DriverKit 23.5
      - iOS 17.5
      - macOS 14.5
      - tvOS 17.5
      - visionOS 1.2
      - watchOS 10.5
  Android SDK:
    API Levels:
      - "34"
      - "34"
      - "34"
    Build Tools:
      - 30.0.3
      - 34.0.0
    System Images:
      - android-31 | Google Play ARM 64 v8a
      - android-34 | Google APIs ARM 64 v8a
    Android NDK: Not Found
IDEs:
  Android Studio: 2024.1 AI-241.15989.150.2411.11948838
  Xcode:
    version: 15.4/15F31d
    path: /usr/bin/xcodebuild
Languages:
  Java:
    version: 22.0.1
    path: /opt/homebrew/opt/openjdk/bin/javac
  Ruby:
    version: 3.2.1
    path: /Users/cobeo/.rubies/ruby-3.2.1/bin/ruby
npmPackages:
  "@react-native-community/cli": Not Found
  react:
    installed: 18.2.0
    wanted: 18.2.0
  react-native:
    installed: 0.74.3
    wanted: 0.74.3
  react-native-macos: Not Found
npmGlobalPackages:
  "*react-native*": Not Found
Android:
  hermesEnabled: true
  newArchEnabled: false
iOS:
  hermesEnabled: true
  newArchEnabled: false

In addition, here is the typescript code that I used for testing the burnt.alert().
 const onSignIn = async () => {
    await Burnt.alert({
      title: "Signing in",
      message: "Please wait...",
      duration: Number.MAX_VALUE,
      preset: "spinner",
    });
    const user = await handleSignIn(email, password);
    try {
      if (user) {
        setIsLoggedIn(true);
        setUser(user);
        router.navigate("/home");
        await Burnt.dismissAllAlerts();
        await Burnt.alert({
          title: "Signed in",
          message: "You are now signed in",
          preset: "done",
          duration: 1,
        });
      } else {
        await Burnt.dismissAllAlerts();
        await Burnt.alert({
          title: "Error",
          message: "Invalid credentials",
          preset: "error",
          duration: 3000,
        });
      }
    } catch (error) {
      await Burnt.dismissAllAlerts();
      await Burnt.alert({
        title: "Error",
        message: "Something went wrong",
        preset: "error",
        duration: 3000,
      });
    }
  };

Should you have any questions or any clarifications regarding the error, please let me know and I will try my best to deliver the informations to you. Thank you.
Best regards.

hmm not sure. what if you remove await

Hey @nandorojo, I have removed the await on all of my codes and I am still encounters with the error. I am thinking on my Gradle version is not compatible with Burnt (Gradle 88). Do you have any comments with this ?

it's possible. but the android code simply uses ToastAndroid from react-native so something is off there

That could be the thing, let me try downgrade the Gradle version to 86 and have a look at it !


Update stable version in repo from 0.0.15 to 0.0.22

Think about update stable version to 0.0.22
Version: stable=0.0.15 (85 days old) => latest=0.0.22 (15 days old)
Installs: stable=975 (55.46%), latest=382 (21.73%), total=1758
Click to edit

This issue seems to be outdated.
This issue suggests to update the stable version of this adapter to 0.0.22 but the current stable version is already 0.0.25.
So this issue should be closed.


get rid of cherrypySetup decorator

That decorator doesn't properly handle tearing down cherrypy, and worse doesn't have exception handling to even try if the test case blows up.

This commit is also related:
https://github.com/dmwm/WMCore/commit/de452acc0a54ccd8a8f27366a0a52d66d2c4f4b3
maybe @ericvaandering has some input here. I still need to ramp up on unit tests stuff :(

No idea. Probably leave it open for now.

@ericvaandering, still wants to leave it open. :-)?

Yeah. Maybe @amaltaro will fix this since he was just asking me about the "CherryPy still has 32 threads running" issue in the unit tests.

OK set the milestone and assign to Alan. :-)

Let's be honest. this is not going to be changed in the next month. No point in a close in milestone (or perhaps a milestone at all)

If we don't set the milestone we tend to lose track. It is just a reminder. so someday we will fix it. Other wise it will show up next annual clean up. 😄


[WIP]Workaround to vagrant up for freebsd

The vargrant version 4.3.4 updated 3 days ago is error. https://app.vagrantup.com/generic/boxes/freebsd13/versions/4.3.4
So roll the version back to 4.3.2 .
fix: https://github.com/containerd/nerdctl/issues/2596

FAIL - has whitespace errors. See `git show --check edb55157719adb9b31ed7d49c53261a88127d23c`.

https://github.com/containerd/nerdctl/actions/runs/6650164705/job/18069829411?pr=2597

Thank you, carried as:

https://github.com/containerd/nerdctl/pull/2606


[Bug]: AzureOpenAI : Sub Question Query Engine is not working

Bug Description
Was trying to do the Subquestion query engine
Got this error
InvalidRequestError: Unrecognized request argument supplied: functions
is it an issue from the Azure Openai itself? because the deployment is gpt-35-turbo-0301 not the 0613 model
Version
0.8.57
Steps to Reproduce
Follow the subquestion query engine with azure openai service
ps:it works with openai service
Relevant Logs/Tracbacks
No response

@vipinap98 you need to make sure you use the latest version of OPENAI_API_VERSION, I think the august preview added support for function calling


Failed to read config file On Android Device

IMPORTANT: Before you create an issue, please take a look at our Issue Reporting Guide.
Briefly summarize your issue:
I am trying to run AVS on android mobile, but facing an error.
the error is :

type=1400 audit(0.0:38): avc: denied { search } for name="alexaSDK" dev="mmcblk0p7" ino=212032 scontext=u:r:untrusted_app:s0:c512,c768 tcontext=u:object_r:app_data_file:s0 tclass=dir permissive=0
Failed to read config file /data/data/com.myapp.avs/alexaSDK/AlexaClientSDKConfig.json
Failed to create to SampleApplication!

What is the expected behavior?
Avs should run.
What behavior are you observing?

The file AlexaClientSDKConfig exists on the path and its contents are also correct.
AlexaClientSDKConfig file and  /data/data/com.myapp.avs/alexaSDK folder has open access (permission to read and write), not restricted for any user.
I have tried running on emulator and its running smoothly.

Provide the steps to reproduce the issue, if applicable:
I am trying to run avs from android app using JNI. The location where i am installing the avs sdk is, /data/data/com.myapp.avs this is the folder of my android app. I have kept avs sdk files in
/data/data/com.myapp.avs/alexaSDK.
Tell us about your environment:
Android 24.
What version of the AVS Device SDK are you using?
Latest One.
Tell us what hardware you're using:

[ ] Desktop / Laptop
[ ] Raspberry Pi
[x] Other - tell us more:

Tell us about your OS (Type & version):

[x] Linux
[ ] MacOS
[ ] Raspbian Stretch
[ ] Raspbian Jessy
[ ] Other - tell us more:

Hello @ajinkyaw thanks for posting.
For android API Levels 23+ you need to require runtime permissions in addition to having read write access.
https://developer.android.com/training/permissions/requesting.html

@womw : i have given all the permissions on runtime , one more thing i wanted to report that this error was resolved by using adb shell setenfoce 0 command.

OK, if this has resolved your issue then I will be closing here, feel free to reopen or open a new issue if problems still occur.


Disable the docker container builds on travisci

Due to (1) consistent docker container build problems on travisci,
and (2) container issues to be considered in issue #1442, disable
the docker container build.  Any updates to the docker hub image
will need to be manual until #1442 is resolved.

@sgratch , @rszwajko , @hstastna - I'd rather see travisci pass with just running the build by itself than keep on failing (most of the time at least) with the docker build included.

I have no opinion on that.

Note: travis-ci.org is no longer running builds, so this PR is a bit of a moot point.  If we want automated container builds, it'll need to be done in std-ci or migrate to travis-ci.com.

ci test

I'd rather see travisci pass with just running the build by itself than keep on failing (most of the time at least) with the docker build included.

@sjd78 note that now after travis-ci.org is no longer running builds, the travisci is totally dismissed on all PRs except here where it's always pass due to running the build only.
I'm actually not sure what is preferable for now. Let's wait after 4.4.7 is released, check with the community if docker container is ever used and decide.

Closing this PR because travisci.org is not running builds anymore so the change is not required.  Leave #1442 open to track future container deploy work.


Test prompt engineering for better responses

Adding information into the system prompt to increase the usefulness of the tool. eg. rubin was in the past referred to as LSST
System prompt "You are an astrophysics expert with a focus on the rubin telescope formerly known as the LSST or Large Synoptic Space Telescope"
Additionally need to add this context to the query to the vector DB.


Need to logo size and icon padding issue help

Need to logo size and icon padding issue help #

its not working for me


Package registration

This issue page is used to register new releases to the Julia General repository

@JuliaRegistrator register
Release notes:

First release. Support "flat ntuple"  write to a ROOT TTree.

Registration pull request created: JuliaRegistries/General/116260
Tagging
After the above pull request is merged, it is recommended that a tag is created on this repository for the registered package version.
This will be done automatically if the Julia TagBot GitHub Action is installed, or can be done manually through the github interface, or via:
git tag -a v0.1.0 -m "<description of version>" 709a7381c58d021914d918c231baeb902ea7362e
git push origin v0.1.0

@JuliaRegistrator register

Registration pull request updated: JuliaRegistries/General/116260
Tip: Release Notes
Did you know you can add release notes too? Just add markdown formatted text underneath the comment after the text
"Release notes:" and it will be added to the registry PR, and if TagBot is installed it will also be added to the
release that TagBot creates. i.e.
@JuliaRegistrator register

Release notes:

## Breaking changes

- blah

To add them here just re-invoke and the PR will be updated.
Tagging
After the above pull request is merged, it is recommended that a tag is created on this repository for the registered package version.
This will be done automatically if the Julia TagBot GitHub Action is installed, or can be done manually through the github interface, or via:
git tag -a v0.1.0 -m "<description of version>" 2593b7fa9a0545632187d7277ea5fb41617bd076
git push origin v0.1.0

@JuliaRegistrator register

Registration pull request updated: JuliaRegistries/General/116260
Tip: Release Notes
Did you know you can add release notes too? Just add markdown formatted text underneath the comment after the text
"Release notes:" and it will be added to the registry PR, and if TagBot is installed it will also be added to the
release that TagBot creates. i.e.
@JuliaRegistrator register

Release notes:

## Breaking changes

- blah

To add them here just re-invoke and the PR will be updated.
Tagging
After the above pull request is merged, it is recommended that a tag is created on this repository for the registered package version.
This will be done automatically if the Julia TagBot GitHub Action is installed, or can be done manually through the github interface, or via:
git tag -a v0.1.0 -m "<description of version>" 988b84f6e307ef200dce3f9c681c5fe632d9814e
git push origin v0.1.0