How to access the client in nuxt.config.js?

When generating a website, dynamic routes can be generated by calling an end-point using Axios.
Is there a way to get a reference to the Apollo client to do something equivalent? Someone raised the issue on the nuxt page.
My only alternative is to create webhooks on the GraphQL server and simply fetch them in a Restful matter.

This question is available on Nuxt.js community (#c69)

I got it done using Apollo-Fetch instead. It's good enough for this use-case.
const { createApolloFetch } = require('apollo-fetch');
...
{
        routes: function () {
            const staticRoutes = [
            ];

            const GRAPHCMS_API = 'URL';
            const apolloFetch = createApolloFetch({ uri: GRAPHCMS_API })
            const query = `
            query allPostIdentifiers {
                posts:allPosts {
                    id
                    slug
                }
                tags:allTags {
                    name
                }
            }
            `;

            return apolloFetch({ query }) // all apolloFetch arguments are optional
                .then(result => {

                    const { data } = result
                    const postRoutes = data.posts.map(post => `/posts/${post.slug}?id=${post.id}`);
                    const tagRoutes =   data.tags.map(tag => `/tags/${tag.name}`);

                    return staticRoutes.concat([...postRoutes, ...tagRoutes]);
                })
                .catch(error => {
                    console.log('got error')
                    console.log(error)
                })
        }
    }

I would still like to know if this is possible.

Me too 👍

This would be insanely helpful. Especially with the ability to pass payloads. I want to make all of my queries up front as I dynamically fetch the routes.
Ideally I want to keep the gql files I already have and pass everything through the same Apollo client. Has anyone done this?

bump


Create traits for pretty/canonical serialization

Maybe we should rename to_stdiowrite to write (and to_stdiowrite_canonical to write_canonical). Maybe to_json_pretty to to_json too?
I put the traits in src/serialize.rs, but I wonder if there's a better place.

Maybe we should rename to_stdiowrite to write (and to_stdiowrite_canonical to write_canonical).

The thing is, if you look at all the things called write in the standard library, I think it's beneficial to clarify.
There's even another trait called Write! (in std::fmt)

Maybe to_json_pretty to to_json too?

We are already playing with fire by emitting an underspecified, mutable, and text-editor-friendly representation like json as a canonical representation and expecting it to have a stable hash.
Ideally, we would write it as hex or something.
People will look at the canonical representation and think "oh, yeah, it's json", and that could lead to confusion, so I want the interface to be absolutely explicit about what it's giving you, especially when what it's giving you doesn't hash correctly.
Now that I think about it, maybe we even ought to consider inserting a few trailing space chars at random in the pretty output to reduce the likelihood that someone will rely on it for consistency.

I put the traits in src/serialize.rs, but I wonder if there's a better place.

I suggested serializable.rs instead to reflect its trait-ness, but that's minor.
Thanks again!

Now that I think about it, maybe we even ought to consider inserting a few trailing space chars at random in the pretty output to reduce the likelihood that someone will rely on it for consistency.

That strikes me as potentially excessive, but we would probably benefit from some loud warnings about it in the doc comments. Maybe we could move the pretty output functionality to a Debug implementation or something? Just a thought.

Now that I think about it, maybe we even ought to consider inserting a few trailing space chars at random in the pretty output to reduce the likelihood that someone will rely on it for consistency.

That strikes me as potentially excessive

I know it does.
But we need to think about more than just writing a correct program here. We are attempting to help build an entire ecosystem of interoperable programs.
Everyone knows that fixed structures should remain fixed, by definition.
What is less known is that varying structures need to be actively varied, otherwise what happens is the ecosystem becomes contaminated with components that silently rely on behaviors no one is aware of. Randomization surfaces hidden bugs at the beginning of the development process, which is golden, because EG needs a whole multi-vendor ecosystem of parts to work perfectly the first time, and every time.
But almost no one does this. When people's code breaks (I mean, when they learn that their code has always been broken) they will look for someone to blame, and you will be standing right there.
Probably you should not be the one to write this function. I will write it.

move the pretty output functionality to a Debug implementation or something? Just a thought.

Debug is a close analogy, but not quite the same as json.
I find it's really very valuable to use a trait to encode in the language's type system specifically what they do and (don't) guarantee in terms of specific functionality. They cost us almost nothing in terms of code size or execution time.


PHP Builtin Server + FPM?

Context
The pre-restructured PHP Web buildpack which includes support for FPM and the web servers (built-in, HTTPD, Nginx) does not have support for the PHP Builtin Server + FPM together. In Restructure RFC 0001 however, the PHP FPM buildpack is present in the PHP Builtin Server order group.
There are two use cases for FPM: The first is running the FPM process on the same container as the web server (which is what PHP Start does as of release v0.1.0). The second scenario is running FPM in a separate container from the server (not implemented, but tracked in an issue)
Issue
We should investigate if there is a use case to have FPM + builtin server, and if so we should implement functionality for these two to work together.
This may involve porting the PHP builtin server buildpack over to the PHP start buildpack in order to leverage the progmgr functionality there.
Outcome
If either use case is viable (running on one container, or in two separate ones), implement the feature and document the use case.
If it is not a viable feature, we should amend the restructure RFC to remove FPM from the builtin server use case.

Update: we can close this out since PHP runs the built-in server an internal dev-only HTTP server is used. FPM shouldn't be needed here, so I will send a PR into the RFC to update the order group.


docs:  [FC-0074] add glossary for openedx-events ecosystem

Description
This PR:

Creates a glossary document for Open edX Events terminology in the reference section.
Uses sphinx utilities to implement a glossary.
References glossary throughout all documents when needed.

@sarina: I completely forgot to let you know I addressed your comments 🫠. Thank you for the thorough review!

@felipemontoya @Ian2012: Could you quickly verify that everything is technically correct from a framework maintainer's perspective? Thanks!


test-branch/execute-job/quoted-authorization/715e9f54-bfbb-4c73-bd13-576bcc1a8036

PR for testing 44265904cd6733db7c6f9c2a449e06a5be2bef1d

/canonical/self-hosted-runners/run-workflows 44265904cd6733db7c6f9c2a449e06a5be2bef1d


Clean up the side buffer

Describe the problem
The ~ at the side buffer is kind of ugly, maybe remove this to make it more clean?

Hey, this should be the case already, do you load NNP before loading your theme? If so, it might override the highlight groups we set.

closing the issue as it have been stalled for some time, feel free to re-open if necessary!


The scraper is does not synchronize  the job list  (Loreal)

https:[//admin.peviitor.ro/]
Browser: Chrome
Device: PC
OS: Windows 10
STEPS TO REPRODUCE:
Enter URL in browser: [//admin.peviitor.ro/]
In go to https://admin.peviitor.ro/jobs/Loreal
Click on synchronize Job List
EXPECTED RESULTS:
The job list should synchronized with the jobs from the Loreal job list
ACTUAL RESULTS:
The job list is not synchronizing and one job is missing (Consultant Vanzari Kiehl's Promenada)

Fixed..


CPU quota reached.

Hi.
I have tried to restore a VM (OS disc) this way but my CPU quota was reached. So I asked for a quota extension, but the VM the backup was made from was on a VM that is not available any more. So the quota team asked me to use another type of VM. How do I do that? Because is not something I can choose.
So maybe If you request this as Azure you should write a guide about how to do this.

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

ID: 66081bde-1e22-dd9c-8357-a0485b23b0a7
Version Independent ID: eb2a52a5-d290-84fb-c3fe-2a5d8512669a
Content: Azure Backup: Restore virtual machines by using the Azure portal
Content Source: articles/backup/backup-azure-arm-restore-vms.md
Service: backup
GitHub Login: @markgalioto
Microsoft Alias: markgal

@YvonneArnoldus Thanks for the feedback! I have assigned the issue to the content author to evaluate and update as appropriate.

@Adam-Smith-MSFT, please assign this issue to GeethaLakshmiG

We can restore disk in this case. We can use these restored disks to create a VM of our choice from the variations available in compute

#please-close


Add Tonalztics spec to spec counter plugin

Adds tonalztics to the spec counter plugin, after testing the spec always applies the hitsplat 2t after speccing regardless of distance, which means it will never queue before a mage or range thrall (which will always be at least 2t later and if they're both 2t the npc action is always processed first) so the last hitsplat logic still works, except for melee thralls (which is also true for dorgeshuun crossbow and accursed scepter, which both additionally can be incorrect on mage and range thralls if you bow/cast from max distance)

Has this been looked at yet? It is currently used in CMs and this would be valuable information to have presented via the plugin. It will be similarly important after the changes next week.

It hasn't; I (nor the other devs, that I'm aware of) don't have this weapon to test/verify this change.

The logic has been reduced to what @Adam- suggested and getHit() has been pulled out of specialAttackHit()

The logic has been reduced to what @Adam- suggested and getHit() has been pulled out of specialAttackHit()

This is just not true because getHit() is still called from specialAttackHit()

Correctly removed getHit() from specialAttackHit this time and changed .equals per @Adam- request. Additionally cleaned up the commits.


Add setup_dataset

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 100.00%. Comparing base (f284076) to head (231e994).
Report is 2 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff            @@
##              main      #408   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files          115       115
  Lines         3418      3430   +12
  Branches       275       277    +2
=========================================
+ Hits          3418      3430   +12

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

🚨 Try these New Features:

Flaky Tests Detection - Detect and resolve failed and flaky tests


IndexError: list index out of range (ped file is tab seperated and matplotlib is also updated).

2017-11-17 11:46:21 l-1-01 peddy.cli[97880] INFO Running Peddy version 0.3.2
Traceback (most recent call last):
File "/home/ganesans/miniconda/envs/grinenv/bin/peddy", line 11, in
load_entry_point('peddy', 'console_scripts', 'peddy')()
File "/home/ganesans/miniconda/envs/grinenv/lib/python3.5/site-packages/click/core.py", line 716, in call
return self.main(*args, **kwargs)
File "/home/ganesans/miniconda/envs/grinenv/lib/python3.5/site-packages/click/core.py", line 696, in main
rv = self.invoke(ctx)
File "/home/ganesans/miniconda/envs/grinenv/lib/python3.5/site-packages/click/core.py", line 889, in invoke
return ctx.invoke(self.callback, **ctx.params)
File "/home/ganesans/miniconda/envs/grinenv/lib/python3.5/site-packages/click/core.py", line 534, in invoke
return callback(*args, **kwargs)
File "/mnt/isilon/helbig_lab/software/peddy/peddy/cli.py", line 159, in peddy
ped_obj = Ped(ped)
File "/mnt/isilon/helbig_lab/software/peddy/peddy/peddy.py", line 441, in init
self._parse(ped)
File "/mnt/isilon/helbig_lab/software/peddy/peddy/peddy.py", line 460, in _parse
sample = Sample.from_row(toks, header=self.header, warn=self.warn)
File "/mnt/isilon/helbig_lab/software/peddy/peddy/peddy.py", line 262, in from_row
return cls(row[0], row[1], row[2] or "-9", row[3] or "-9", row[4], row[5],
IndexError: list index out of range

sounds like you don't have 6 columns for all of the rows in your ped file.

Thanks!! Will look into it.

can you put the ped file here or email to me?

Fixed this! was using a different version of python!!! Thanks!


Update to newest available versions.

Updating the dependencies to enable git2 to handle emtpy timestamps.

This fixes an issue with empty commit timestamps. These were created in
git-lfs 1.
dialoguer::Confirmation was renamed to Confirm 2
Update was done automatically with cargo-edit's cargo upgrade.

Thanks. I thought about it beforehand and decided that it's fine. But good eye!


Fix email preview, use wkhtmltopdf for html to pdf conversion

What does this change?
Email preview is currently broken. This is due to some linux permissions issue when giant tries to call out to chromium-browser to generate the PDF preview.
It would definitely be possible to fix the permissions issue (giant would need write access to /home/pfi/snap I think) but @itsibitzi suggested abandoning chromium-browser and instead using wkthmltopdf.
This works round the permissions issue, and should somewhat speed up the preview process now that we're not spinning up an entire browser to generate the pdf.
Separately, I noticed that we're not properly url encoding the filename in the url we pass to PDF.js - this PR also fixes that.
How to test
Open any .eml file in giant - here's an example on playground https://playground.pfi.gutools.co.uk/viewer/CAGHe3Pmc37CiMwbWks-CVOiM=uYhnpvq2=Zg51dkESL80On94g@mail.gmail.com?details=summary&view=preview

Also I guess there's a corresponding Amigo change? Which I'm sure you've already done because you're an infra guru. Just always weirds me out that that stuff lives outside version control

Thanks Joe! Yep I've added whtmltopdf to amigo here - not removed chrome yet as the same AMI is used by playground and prod.
Here's a screenshot. I'm not sure how much better/worse than the old chromium renders it is.


Extracted home page into its own component

Added Home.tsx
Description
Added home.tsx to allow for future addition without risk to app.
Screenshot(s) (if applicable, e.g. style updates)

PR Readiness Checklist

General

[x] Verified all changes are related to feature described above (no scope creep)

Branch Stuff

[x] Checked target and base branch (should be feature branch merging INTO main)
[x] Merged latest main into feature branch (so it has latest code)
[ ] Resolved any conflicts between feature branch and main

Test Stuff

[ ] Added unit tests to cover modified / additional code

GitHub Stuff (on the right 👉)

[x] Updated labels
[x] Linked issue to which this PR relates
[x] Added assignee
[x] Added at least one (1) reviewer

This PR now has conflicts, since another PR got merged to main; I may get to resolving them on this feature branch tonight, but I may not; they'll need to be resolved before merge


CLI workload Subcommand

The Run functions are not correct as I do not believe the structs for creating a workload exist yet.  Currently agent structs are placeholders to ensure all the required fields are represented
This will need to be rebased once #384 is merged - putting in draft until then

Completed in #414


List open files

Hi,
is there currently a way, or is there a clean way to make a function, to list the open buffers?
Something like
<buffers
e1: .ebrc
e2: http://edbrowse.org
e99: .ebmarks

Sorry, that was intended for edbrowse obviously.


Update Lua

See  #780

This file has been deleted. Sorry for any inconvenience.
See: https://github.com/exercism/x-common/issues/780#issuecomment-300394704


Automate generation of client certificates.

I'd like to have an easy way to generate client certificates that a browser can use to connect to the UI. Creating an action to generate client certificates might be a good way to go?

@galgalesh - Those client certificates you call out would be used in the kube proxy command, not necessarily for client browser sessions as I don't like the idea of polluting the system keychain with self signed certs from K8s. I'm sure other security professionals would agree that this should be isolated to the use cases with kubectl.
But we do need a clean way to generate new client side certs for distribution/deprecation and is a larger bug that needs to be filed/triaged.
I'll leave this issue open as is so we can track progress, but its currently priority: low in terms of this cycle's todo list. I'll be happy to loop you in on any concept work we propose for this feature to ensure we meet your goals with this bug.

I could've sworn I opened an issue against layer-easyrsa on this. Apparently not, oops!
In case someone finds it helpful, here's a workaround I used when I needed to create server certs manually: easyrsa-remote.sh
I don't know exactly how it would differ for client certs though.

@chuckbutler - The main reason why I opened the original issue (on the old repo) was to have a way to connect to a Kubernetes cluster from Windows. From what I remember, kubectl didn't have official Windows support back then. This has changed since, kubectl is now officially supported on Windows so this issue isn't important for my anymore.
Feel free to close it, or keep it open as you wish.

Thanks for the update Galgalesh. Closing as your needs have changed.


Add WritableKeys & ReadonlyKeys types

Thanks! I am gonna write some additional docs and release it.

Released as 2.0.10


New release for Carthage header changes

Can you publish a new release that includes all of the changes to the header files and such that make the Carthage installation work?

Publish, where?

Here in Github. The last release is 2.4.0 from April 19th.

@chazmo03 me and some other people( @eliburke ) has made some of the Carthage support.
Since @m1entus  doesn't use Carthage himself. It is indeed a bit problematic with this "half"-support.
But you could use:
github "m1entus/MZFormSheetPresentationController" "master"
in your cartfile to get all updates from master.
Or you could specify a commit if you feel that's safer!

See #97 for another Carthage related fix. Hopefully Michał will merge it, but if he doesn't create a new release tag you will need to specify a commit in your carthage file like this:
github "eliburke/MZFormSheetPresentationController" "b34c93e"
(that's my repo, if the merge of #97 goes through then you should link against that)

I released 2.4.1 and i think i can close it


[Workflow] Update home dashboard styles

This PR:

brings the admin home dashboard panels more in line with the edit panels.
updates the panel heading text, and order
adds an unlock button for the locked pages
sorts the workflow-related items descending
simplifies the workflow-related columns for consistency
includes task progreesion for pages awaiting review and adds a visual indicator for any items that require changes

See screenshot

</details

Can we simplify the relative dates? e.g.
19 hours, 48 minutes ago -> 19 hours ago
1 week, 1 day ago -> 1 week ago
0 minutes ago -> just now

@tomdyson done in 45ca64a


polish #692 doc no correction

Describe what this PR does / why we need it
some doc no error correction.
Does this pull request fix one issue?

Describe how you did it
fixed some doc no error
Describe how to verify it
Special notes for reviews

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.panda seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


Make JWT signing secret readily available

Feature request
Within a NextJS app, I have some API endpoints where I manually validate the JWT to avoid a round-trip. This breaks when using supabase start because I don't know what the signing token is.
Describe the solution you'd like
Make the signing token static or available within the UI (or ideally, both). You could also make it configurable via the toml file as well I suppose.
Current JWT signing token within a deployed Supabase project

I found the signing token within the docker image GOTRUE_JWT_SECRET=super-secret-jwt-token-with-at-least-32-characters-long.
I still think it'd be nice if this was surfaced somewhere more accessible than within an ENV_VAR inside of the auth docker container.

Is it sufficient to output this when running the supabase start command?

Is it sufficient to output this when running the supabase start command?

Yeah, I think that could work as well.


sensor: bmp388 dirver has bug

https://github.com/zephyrproject-rtos/zephyr/blob/3a3cfbb652b29e9f39a38fe6febf29e1570d32bc/drivers/sensor/bmp388/bmp388.c#L600
when device is on spi bus, there is no i2c
https://github.com/zephyrproject-rtos/zephyr/issues/51873#tasklist-block-3e9cf6e6-2736-4b13-8365-aa9ec8b0e700

It is judged that the way the device attaches the bus is incorrect during operation

@xodus7 since you submitted this driver

Indeed this is a bug, and very suggestive of which hardware configuration I has to test with :-( . I see a now closed PR #51876 that was making changes to this driver. Is someone working to fix this? If not, I'd be happy to submit a simple patch.


askOpenFileWin32

It is not possible to pass alternative suffixes to the windows file opener. They are not actually passed to the function that calls the windows process.
It looks like someone has had a go but has hard coded the suffixes in the windows function instead of passing the original ones on.

Thank you for pointing this.
We'll have a look :smile:

Should be fixed for the next update
Now there's a fourth optional parameter to this function which is suffixes
It will look something like: ["zip", "docx"] and the name there would be shown in the file opener
Of course we can't promise the code will know what to do with those file, it will just show them to the user in the file opener
I hope it helps and thanks for the support :)

Thanks for working on this and keeping development ticking over.
I can't seem to get it to work. I am running the following
file = mcplatform.askOpenFile(title="Select Post-Commands text document", suffixes=["txt"])
and all that is appearing in the box is All and , am I doing something wrong?


image runs over into clarifai tags on image page

see https://ccsearch.creativecommons.org/photos/fe2b2152-2238-4d10-8980-d67f9f2fccde for example

Does this still reproduce @janeatcc

nope, closing


[Feature Request] customized Diff-Algorithm, needed for diff-editor.

Context

[X] This issue is not a bug report. (please use a different template for reporting a bug)
[X] This issue is not a duplicate of an existing issue. (please use the search to find existing issues)

Description
Is there a way to customized the diff-algorithm to achieve IgnoreCase, ignoreWhitespaceAtStart, ignoreWhitescapeAtEnd, IgnoreLineFeed APIs. The below code is not going to work when monaco version increase, so is there a way to acheive a permanent customized solution.
Monaco Editor Playground Link
No response
Monaco Editor Playground Code
const m1 = monaco.editor.createModel(`
1
2
3
4
5
6
7
8
9`);

const m2 = monaco.editor.createModel(`
1
2
x
4
5
6
8
9`);

const standaloneServices = require("vs/editor/standalone/browser/standaloneServices");
const WorkerBasedDocumentDiffProvider = require('vs/editor/browser/widget/workerBasedDocumentDiffProvider').WorkerBasedDocumentDiffProvider;
const provider = standaloneServices.StandaloneServices.initialize().createInstance(WorkerBasedDocumentDiffProvider, {});

const diffEditor = monaco.editor.createDiffEditor(document.getElementById('container'), {
	diffAlgorithm: {
		onDidChange: () => ({ dispose: () => {} }),
		computeDiff: async (original, modified, options) => {
			const result = await provider.computeDiff(original, modified, options);
			console.log("xyz ",result);
			result.changes.shift(); // Remove first diff
			return result;
		}
	},
});
diffEditor.setModel({
	original: m1,
	modified: m2,
});

See here

Is there a way to compare Contented of both the editor, instead of manupilating it on basis of line-change. because i need to tackle a tricky/teadious task of Masking Expression(example :  inside the file wherever this  present that whole line should be ignored. Sorry if i sound confusing.

See here

Is this work when there is any version upgradation.


Example using ARM to setup service-delegated MI?

I see that service-delegated subnet is the way forward for setting up route table/subnet correctly, but I am not seeing an example of how to do so via Azure Resource Manager (ARM). I rely on ARM to assert infrastructure (which means redeploying on existing infrastructure), and we've run into two issues in the past two months where the Network Intent Policy that SQLMI asserts on top of the subnet blocks deployments of SQL MI because it has changed without warning.
My team has manually updated the routes as we discover them, but we would prefer to use the service-aided option to prevent this constant back and forth.

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

ID: fc2dd547-3ff7-b32a-ba8a-c582a721a0ee
Version Independent ID: f083bdda-0c66-af32-0d27-f535e46c96dc
Content: Connectivity architecture for a managed instance - Azure SQL Database
Content Source: articles/sql-database/sql-database-managed-instance-connectivity-architecture.md
Service: sql-database
Sub-service: managed-instance
GitHub Login: @srdan-bozovic-msft
Microsoft Alias: srbozovi

@MaxFrost We are checking this and will get back to you.

@MaxFrost We are assigning this to our author for further review regarding such templates exist or not.
@srdan-bozovic-msft Please check the question from user and help.

I also faced similar issue with the UDR changes without any indication and also difficult to identify changes. is there a way where we can get the latest UDR(routes for SQL MI)

quick start templates were updated to use delegation
set of udrs is updated in the doc https://docs.microsoft.com/en-us/azure/azure-sql/managed-instance/connectivity-architecture-overview#mandatory-user-defined-routes-with-service-aided-subnet-configuration
#please-close


do not throw if value is null

Fixes #11.

Sorry, totally forgot about this one... I've changed my mind, I no longer think it makes sense to throw on accessor. Could you fix the merge conflict and only handle the other cases?

ping @stevemao

Hm... I was waiting for #17 to be merged first... Since I think this one is a bit tricker... I'll work on this if you think its better :)

What do you think @SamVerschueren?

Sorry, think I got lost somewhere in the conversation :). Where do you want my opinion on? To merge #17 first and after that this PR?

They all need to be merged. Just need +1s on these PRs

@sindresorhus I have added some comments.

LGTM

Sorry guys...  Can you please re-push the latest commits to master branch? Or is there a way for me to restore them (I don't have those commits in my local repo)? Sorry!!

No worries. Fixed. Btw, you can always use git reflog to undo stuff.

Thanks @stevemao. Glad we could finally get this merged :)


[FIX] Allow sphere maps to work when an envAtlas is set

Fixes #5253
Fixes #5364
I confirm I have read the contributing guidelines and signed the Contributor License Agreement.

Feels bit strange that when the sphereMap is set, it goes to the else branch where it seems to use constant ambient?

Sphere maps basically override/ignore the env atlas. So it's not that weird.


Tweak logging in publishing_api_republisher

Make it clear that the items haven't been republished, they've been queued to
be republished. This confused me because the data hadn't (yet) been changed in
content-store/router-api even though it seemed they had been republished
already.

@jamiecobbett: you've brought-up a good point. :+1: for merge once tests pass.
continuing on this point, queuing wasn't apparent to me till i saw publishing_api.rb. should we also rename publish => queue_for_publishing, republish => queue_for_republishing etc.?
what does everyone else think?

@vinayvinay thanks - merged. With regard to renaming - I'm not at home enough in Whitehall to have a strong opinion.


feat: make it easier to make changes to profile

Checklist:

[x] I have read and followed the contribution guidelines.
[x] I have read and followed the how to open a pull request guide.
[x] My pull request targets the main branch of freeCodeCamp.
[x] I have tested these changes either locally on my machine, or GitPod.

This makes it easier to make new changes to the profile for #48234 .
I have made a custom page for development in the pages directory called developmentuser. Now when clicking on the profile it will directly go to the profile page.
Let me know if this is something we want.

I thought about that, but I just wanted to make sure.
If we actually want this, the test need to be updated too.

the test need to be updated too

I thought about that - that some tests need to click that "preview" button, but none of them are failing?

What problem is this solving? Just that you need to click through the 404 page when specifically working on the /developmentuser profile? It doesn't seem worth creating a special case in the client for that.

It is not solving a problem, this makes the development environment mirror the production build. Instead of needing to go to a weird custom 404 page.
As stated a lot needs to be done before we can merge this.

It is not solving a problem, this makes the development environment mirror the production build.

Unfortunately that's not true or how it works.
The code is trying to override the override.
That is:

Gatsby, when in production uses the custom 404 - which redirect to user profiles.
Gatsby, when in development presents its own  debug 404 - before the custom 404 with the debug info and the button - clicking the button activates the custom 404 - the next thing in the lifecycle.

The code is essentially, preventing the debug 404.
It's not "mirroring" the production but "mocking" it – meaning I would not expect it to behave logically like the production routing does.
Example: It will obviously fail when you add another user into the development workflow, example lets say we create developmentuser2 with different dataset than developmentuser for say some debugging needs.
The point being:

It tries to hide the Gatsby Dev Servers behaviors.
It could surprise uninitiated devs (when the do see the debug 404) to believe something is not correct - since they are used to the /developmentuser page "just working"?
It has a potential need for "special" case handling in tests (like you said).
It adds complexity without solving "the mirroring" - because it is not?

Now, do we need the trick: Say as a convenient way for debugging? That's another story and discussion - but lets be clear on what the code is doing at the very least.
Hope this makes sense.

My original intention is indeed to make debugging the profile page a little less inconvenient.
You are correct, "mirroring" was a poor choice of words. Instead I should've used mocking. The Gatsby's debug 404 should still be available, just not for the profile page.

Example: It will obviously fail when you add another user into the development workflow, example lets say we create developmentuser2 with different dataset than developmentuser for say some debugging needs.

Yep this is correct, but for simplicity, let's keep it this way.
My suggestion is: Add a route for both seeded users, one being developmentuser and the other certifieduser.
My questions are:

It adds complexity without solving "the mirroring" - because it is not?

If we replace this word by mocking, does it make a difference?


move: read toolchain versioning from lock file

Description
Stacked on https://github.com/MystenLabs/sui/pull/14973.
Adds ToolchainVersioning::read to read versioning info from the lock file.
Test Plan
How did you test the new or updated feature?
Updated existing test to deserialize values with new function.

If your changes are not user-facing and not a breaking change, you can skip the following section. Otherwise, please indicate what changed, and then add to the Release Notes section as highlighted during the release process.
Type of Change (Check all that apply)

[ ] protocol change
[ ] user-visible impact
[ ] breaking change for a client SDKs
[ ] breaking change for FNs (FN binary must upgrade)
[ ] breaking change for validators or node operators (must upgrade binaries)
[ ] breaking change for on-chain data layout
[ ] necessitate either a data wipe or data migration

Release notes

FWIW I prefer [toolchain-version]

yep, sounds good to me too, will do a sweep to change this in follow up


[Feature Request] Expose get_server function

I'm trying to do something like this:
local lsp_installer = require('nvim-lsp-installer')
local _, server = lsp_installer.get_server("rust-analyzer")

if not server.is_installed() then
    server.install()
end

I want to do this so that I only install the language server if a specific plugin is loaded (in this case rust-tools.nvim).
But I'm unable to because the get_server function isn't exposed. Would it be possible to expose it?

Hi! I think it'd make sense to include such a Lua API for sure. I'm wondering whether such an API should be exported via the main module (as you've suggested) or if importing the server module directly would be neater. I can't make my mind up whether require'nvim-lsp-installer.servers.rust_analyzer' is reaching into internals or not.. wdyt?

Hmm as long as it's a documented part of the API I think it's fine.
The only thing I would say is that it's difficult for the user to handle potential failure.
Say you were to update the plugin and remove support for an LSP I was using, it wouldn't be the end of the world if I got a module not found error, or if I had to wrap the require call in pcall. But it would be nicer if the API handled it for me.

Although this isn't too bad tbh:
local server_name = 'rust_analyzer'

local ok, server = pcall(require, 'nvim-lsp-installer.servers.'..server_name)

if not ok then
    -- Show an error & throw
end
if not server.is_installed() then
    server.install()
end


added .gitignore file

solved #14
Added .gitignore file

@swapnilsparsh please check this one and inform me

Issue has already been resolved.
Check #26


Dns delete record

closes #1138

Coverage decreased (-0.6%) to 81.933% when pulling a6703946936fbda9ac92e1c0f0489eaec14ca002 on abemassry:dns-delete-record into 15d8d5e0bd15c2fbbb2f89fcfdf742a7907edaf5 on linode:master.

Coverage decreased (-0.3%) to 82.299% when pulling 0dcd772ab6f2c8c9b01865c7190e57b348a891c7 on abemassry:dns-delete-record into 15d8d5e0bd15c2fbbb2f89fcfdf742a7907edaf5 on linode:master.

Coverage decreased (-0.1%) to 82.442% when pulling 3f3b7de29e6d79fb5d437787c39519867e70e58a on abemassry:dns-delete-record into 15d8d5e0bd15c2fbbb2f89fcfdf742a7907edaf5 on linode:master.


Fix #845

See #845

I just want to say a big thank you for debugging this issue so extensively and collaborating with other users to do so. You have been very helpful of late and both me and the community is grateful.

Released.
https://www.nuget.org/packages/FluentFTP/37.0.2


Feature Request - Batch Encrypt/Decrypt operation in keyvault

We have a requirement for a customer where in course of a data processing flow, we need to encrypt/decrypt a list of items (let's say 100 in a batch) individually. Now if we do that from client side sending multiple requests to keyvault, there is a throttling from keyvault and also network latency. So, we need something for bulk operation at one shot. Let's say if the API can accept and array of key operation input and return an array of key operation result.

@krish-gh batch operations are not supported by Key Vault service as of now. To bypass the issue, you can send the requests at the same time to key vault and if throttled, you need to back off and wait for few seconds. Here is the throttling limit for Key Vault. Does it work for you?

We are adding your request to the requested features to review. :) There is no plan to support it yet.


[Question] Is it possible for bluetooth to work?

Hi, we are currently unable to connect bluetooth devices to the emulator.
With apple silicon is it possible to use bluetooth in the emulator?

Hi, we're working on BT support and will update you later with an ETA.

is there any way still to solve this

is there any way still to solve this


Bug: Marker extraction is not working

Is there an existing issue for this?

[X] I have searched the existing issues

Is this a regression?
Yes
Current behavior
Strings annotated with marker function aren't extracted. Looks like the code responsible is not aware that the package namespace has changed from @ngneat to @jsverse.

https://github.com/jsverse/transloco-keys-manager/blob/master/src/keys-builder/typescript/marker.extractor.ts#L11
https://github.com/jsverse/transloco-keys-manager/blob/master/src/keys-builder/typescript/index.ts#L35

Expected behavior
Marker extraction should work.
Please provide a link to a minimal reproduction of the bug
Any attempt to extract marker-annotated strings.
Transloco Config
No response
Debug Logs
No response
Please provide the environment you discovered this bug in
Transloco: 6.0.4
Transloco Keys Manager: 3.8.0
Angular: 17.1.0
Node: 18.18.0
Package Manager: 9.8.1
OS: macOS

Additional context
No response
I would like to make a pull request for this bug
Yes 🚀

I also created a PR about it yesterday. It's not just marker that's no longer working.

This should be resolved in v4.0.1, can you please verify 🙂

Feel free to reopen if not foxed 🙂


Can not install on mac

Installation using pip failed.
Traceback (most recent call last):
File "/opt/local/Library/Frameworks/Python.framework/Versions/3.4/lib/python3.4/site-packages/setuptools/sandbox.py", line 152, in save_modules
yield saved
File "/opt/local/Library/Frameworks/Python.framework/Versions/3.4/lib/python3.4/site-packages/setuptools/sandbox.py", line 193, in setup_context
yield
File "/opt/local/Library/Frameworks/Python.framework/Versions/3.4/lib/python3.4/site-packages/setuptools/sandbox.py", line 237, in run_setup
DirectorySandbox(setup_dir).run(runner)
File "/opt/local/Library/Frameworks/Python.framework/Versions/3.4/lib/python3.4/site-packages/setuptools/sandbox.py", line 267, in run
return func()
File "/opt/local/Library/Frameworks/Python.framework/Versions/3.4/lib/python3.4/site-packages/setuptools/sandbox.py", line 236, in runner
_execfile(setup_script, ns)
File "/opt/local/Library/Frameworks/Python.framework/Versions/3.4/lib/python3.4/site-packages/setuptools/sandbox.py", line 45, in _execfile
code = compile(script, filename, 'exec')
File "/tmp/easy_install-5tsdus5b/BeautifulSoup-3.2.1/setup.py", line 22
print "Unit tests have failed!"
^
SyntaxError: Missing parentheses in call to 'print'
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "setup.py", line 70, in
test_suite="tests"
File "/opt/local/Library/Frameworks/Python.framework/Versions/3.4/lib/python3.4/distutils/core.py", line 148, in setup
dist.run_commands()
File "/opt/local/Library/Frameworks/Python.framework/Versions/3.4/lib/python3.4/distutils/dist.py", line 955, in run_commands
self.run_command(cmd)
File "/opt/local/Library/Frameworks/Python.framework/Versions/3.4/lib/python3.4/distutils/dist.py", line 974, in run_command
cmd_obj.run()
File "/opt/local/Library/Frameworks/Python.framework/Versions/3.4/lib/python3.4/site-packages/setuptools/command/install.py", line 67, in run
self.do_egg_install()
File "/opt/local/Library/Frameworks/Python.framework/Versions/3.4/lib/python3.4/site-packages/setuptools/command/install.py", line 117, in do_egg_install
cmd.run()
File "/opt/local/Library/Frameworks/Python.framework/Versions/3.4/lib/python3.4/site-packages/setuptools/command/easy_install.py", line 380, in run
self.easy_install(spec, not self.no_deps)
File "/opt/local/Library/Frameworks/Python.framework/Versions/3.4/lib/python3.4/site-packages/setuptools/command/easy_install.py", line 610, in easy_install
return self.install_item(None, spec, tmpdir, deps, True)
File "/opt/local/Library/Frameworks/Python.framework/Versions/3.4/lib/python3.4/site-packages/setuptools/command/easy_install.py", line 661, in install_item
self.process_distribution(spec, dist, deps)
File "/opt/local/Library/Frameworks/Python.framework/Versions/3.4/lib/python3.4/site-packages/setuptools/command/easy_install.py", line 709, in process_distribution
[requirement], self.local_index, self.easy_install
File "/opt/local/Library/Frameworks/Python.framework/Versions/3.4/lib/python3.4/site-packages/pkg_resources/init.py", line 824, in resolve
dist = best[req.key] = env.best_match(req, ws, installer)
File "/opt/local/Library/Frameworks/Python.framework/Versions/3.4/lib/python3.4/site-packages/pkg_resources/init.py", line 1069, in best_match
return self.obtain(req, installer)
File "/opt/local/Library/Frameworks/Python.framework/Versions/3.4/lib/python3.4/site-packages/pkg_resources/init.py", line 1081, in obtain
return installer(requirement)
File "/opt/local/Library/Frameworks/Python.framework/Versions/3.4/lib/python3.4/site-packages/setuptools/command/easy_install.py", line 629, in easy_install
return self.install_item(spec, dist.location, tmpdir, deps)
File "/opt/local/Library/Frameworks/Python.framework/Versions/3.4/lib/python3.4/site-packages/setuptools/command/easy_install.py", line 659, in install_item
dists = self.install_eggs(spec, download, tmpdir)
File "/opt/local/Library/Frameworks/Python.framework/Versions/3.4/lib/python3.4/site-packages/setuptools/command/easy_install.py", line 842, in install_eggs
return self.build_and_install(setup_script, setup_base)
File "/opt/local/Library/Frameworks/Python.framework/Versions/3.4/lib/python3.4/site-packages/setuptools/command/easy_install.py", line 1070, in build_and_install
self.run_setup(setup_script, setup_base, args)
File "/opt/local/Library/Frameworks/Python.framework/Versions/3.4/lib/python3.4/site-packages/setuptools/command/easy_install.py", line 1056, in run_setup
run_setup(setup_script, args)
File "/opt/local/Library/Frameworks/Python.framework/Versions/3.4/lib/python3.4/site-packages/setuptools/sandbox.py", line 240, in run_setup
raise
File "/opt/local/Library/Frameworks/Python.framework/Versions/3.4/lib/python3.4/contextlib.py", line 77, in exit
self.gen.throw(type, value, traceback)
File "/opt/local/Library/Frameworks/Python.framework/Versions/3.4/lib/python3.4/site-packages/setuptools/sandbox.py", line 193, in setup_context
yield
File "/opt/local/Library/Frameworks/Python.framework/Versions/3.4/lib/python3.4/contextlib.py", line 77, in exit
self.gen.throw(type, value, traceback)
File "/opt/local/Library/Frameworks/Python.framework/Versions/3.4/lib/python3.4/site-packages/setuptools/sandbox.py", line 164, in save_modules
saved_exc.resume()
File "/opt/local/Library/Frameworks/Python.framework/Versions/3.4/lib/python3.4/site-packages/setuptools/sandbox.py", line 139, in resume
compat.reraise(type, exc, self._tb)
File "/opt/local/Library/Frameworks/Python.framework/Versions/3.4/lib/python3.4/site-packages/setuptools/compat.py", line 65, in reraise
raise value.with_traceback(tb)
File "/opt/local/Library/Frameworks/Python.framework/Versions/3.4/lib/python3.4/site-packages/setuptools/sandbox.py", line 152, in save_modules
yield saved
File "/opt/local/Library/Frameworks/Python.framework/Versions/3.4/lib/python3.4/site-packages/setuptools/sandbox.py", line 193, in setup_context
yield
File "/opt/local/Library/Frameworks/Python.framework/Versions/3.4/lib/python3.4/site-packages/setuptools/sandbox.py", line 237, in run_setup
DirectorySandbox(setup_dir).run(runner)
File "/opt/local/Library/Frameworks/Python.framework/Versions/3.4/lib/python3.4/site-packages/setuptools/sandbox.py", line 267, in run
return func()
File "/opt/local/Library/Frameworks/Python.framework/Versions/3.4/lib/python3.4/site-packages/setuptools/sandbox.py", line 236, in runner
_execfile(setup_script, ns)
File "/opt/local/Library/Frameworks/Python.framework/Versions/3.4/lib/python3.4/site-packages/setuptools/sandbox.py", line 45, in _execfile
code = compile(script, filename, 'exec')
File "/tmp/easy_install-5tsdus5b/BeautifulSoup-3.2.1/setup.py", line 22
print "Unit tests have failed!"
^
SyntaxError: Missing parentheses in call to 'print'

Looks like you're using Python 3.4 which Goose currently doesn't support, as far as I know...

OK, thanks!

Getting this error while trying to install as well. I'm confused as many comments seem to suggest goose-extractor is python 3 ready? Frustrating as goose works very well


aws-rotate-key: 1.0.4 -> 1.0.6

Motivation for this change
Update aws-rotate-key (https://github.com/Fullscreen/aws-rotate-key)  to 1.0.6 (https://github.com/Fullscreen/aws-rotate-key/releases/tag/v1.0.6)
Things done

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

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] 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 nix-review --run "nix-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

Notify maintainers
cc @mbode

@GrahamcOfBorg build aws-rotate-key


[i18n] Bengali 'ben-in' translation #304

Call for Translator Help
Facts needed Translating in Bengali for India location
Acceptance Criteria

[x] Create a new file ./src/models/localizations/ben-in.js
[x] Using the Template below, translate existing and even add new cat facts in the localized language.

Template
const facts = [
  "..."
];

const langName = "bengali",
	langISO = "ben",
	langLocale = "in",
	langLocaleName = "India";

module.exports = {
	langName,
	langISO,
	langLocale,
	langLocaleName,
	code: `${langISO}-${langLocale}`,
	codeName: `${langName} (${langLocaleName})`,
	facts: facts,
};

Inspired by  https://github.com/wh-iterabb-it/meowfacts/issues/318 from Main Issue: https://github.com/wh-iterabb-it/meowfacts/issues/175

Call for Translator Help
Facts needed Translating in Bengali for India location
Acceptance Criteria

[x]  Create a new file ./src/models/localizations/ben-in.js
[x]  Using the Template below, translate existing and even add new cat facts in the localized language.

Template
const facts = [
  "..."
];

const langName = "bengali",
	langISO = "ben",
	langLocale = "in",
	langLocaleName = "India";

module.exports = {
	langName,
	langISO,
	langLocale,
	langLocaleName,
	code: `${langISO}-${langLocale}`,
	codeName: `${langName} (${langLocaleName})`,
	facts: facts,
};

Inspired by #318 from Main Issue: #175

I am folowing this template and I will try to translate all of this asap

@dshaw0004 No worries, your work was  great! I merged it into a dev branch just to fix a linting error. I am merging it in now in https://github.com/wh-iterabb-it/meowfacts/pull/322 which completes your works merge into main.
Once again, thank you so much for your contribution, you have helped make this API available in a whole new language and local and we appreciate it!

:clap:
https://meowfacts.herokuapp.com/?lang=ben-in shows up for me now after deploy

I can also help you to add hindi language but i can not write hindi that efficiently it will took a long time.


Add new tutorial link

This is a tutorial I wrote that shows how to use the storybook-dark-mode addon to enable a dark mode toggle option for Storybook, and then how to connect this dark mode toggle to the React components being rendered in preview. So when Storybook is toggled to dark mode, the component being rendered in preview changes as well. The tutorial covers everything involved, from installing Storybook to configuring the React components to switch between dark and light mode. Any edits to the description or title I added in the README are welcome. And of course if this doesn’t fit in with the theme of what you’re going for here feel free to ignore — thanks!

Thanks @davidyeiser, well done!!!


Request - Allow RecordNumber to be not unique

Because in botany we often make duplicate (up to 5 or more) collections of the same organism (or population) under a single collector's number, different physical specimens at different herbaria will have different catalog numbers but the same record number. Currently in TW two records with the same RecordNumber is not allowed, requiring addition of, e.g.  @K or  at ALA to the Record number. This will 'dirty' the data for export.  Can we please allow records with non-unique RecordNumbers? Thanks

@camwebb understood! I think while it's a kind of "localIdentifier", hence the way it's currently modeled, indeed it has this nature you describe. Perhaps best imported into TaxonWorks as a Data Attribute of the CO, instead of a formalized identifier. Will have to wait and see what @mjy and @LocoDelAssembly have to say.

It's an Identifier, and needs to be modelled as such, we can remove the restriction.

we can remove the restriction

Thanks

@mjy can or cannot?

can , we can skip the validation callback I think, but I'll have to test if that actually works


Read-only account type

How horrified will our tests and linters be with me? Tbd.

going over controllers quickly -- do we want reader accounts to have block access -- I think yes? indexes currently require a mod to create but what about those? they specifically shouldn't have access to messages -- news creation is already locked -- tags should be?
...also we might want a better header for readers


New Token submission

ioTube Token submission
Please read the token submission guide here: https://docs.iotube.org/introduction/token-submission
Team / Developer name
...
Token Symbol
...
Token Decimals
...
Token Name in full
...
Link to the token icon
https://...
Short description of your project
Descrie your project here
What is the blockchain (source blockchain) that your token is issued on

[x] Ethereum
[x] Binance Smart Chain
[x] Polygon
[x] IoTeX

What blockchains (destination blockchain) would you like your token is bridged to

[x] Ethereum
[x] Binance Smart Chain
[x] Polygon
[x] IoTeX

(Please check https://docs.iotube.org/introduction/token-submission#integration-fee for integration fees of different blockchains)
Token native contract address explorer link
Address:
Your project websites

https://...

Your social channels

Telegram:
Twitter:
Discord:

Verification of the token
Please provide the verification of your token in one of the following ways.

[x] Token address on your website. Please specify the link.
[x] CMC / Coingecko profile link with token contract address and link to website/social

Send test tokens
Please send a few test tokens ($1-$10 worth) to our wallet address: 0x571e603e3B5c82Bd40998C538603dF4fb42bB051.
Integration fee
Please send the payment to 0x571e603e3B5c82Bd40998C538603dF4fb42bB051 or io12u0xq03mtjpt6sye33fcvq7lf76zhvz3yvt8q0 and post your txn link here.

Content is not filled.


WIP: internal/filetransfer: initial setup for handling incoming NOC/COR files

These files can give corrected data for an EntryDetail (which might allow us to fix a Depository's data) that was sent back after we uploaded a file. I'm not sure if we'll be able to automatically fixup any data, but these files do we need to be handled by an ACH originator.

Codecov Report

Merging #275 into master will decrease coverage by 0.63%.
The diff coverage is 0%.

@@            Coverage Diff             @@
##           master     #275      +/-   ##
==========================================
- Coverage   61.25%   60.62%   -0.64%
==========================================
  Files          43       44       +1
  Lines        4819     4863      +44
==========================================
- Hits         2952     2948       -4
- Misses       1312     1358      +46
- Partials      555      557       +2

Impacted Files
Coverage Δ

internal/filetransfer/incoming.go
31.42% <0%> (-1.41%)
:arrow_down:

internal/filetransfer/correction.go
0% <0%> (ø)

internal/accounts.go
78.08% <0%> (-2.74%)
:arrow_down:

internal/database/mysql.go
58.66% <0%> (-2.67%)
:arrow_down:

Continue to review full report at Codecov.

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


Identify module_name provider_meta for Equinix provider

This change supplies metadata that reflects the adoption of this module by Equinix users.

This change failed for terraform init, see https://github.com/rancherlabs/terraform-harvester-equinix/actions/runs/9321969024/job/25674337943?pr=70

Thanks for catching this, @dnoland1
provider_meta's parent is the terraform block. fixed
terraform init

Initializing the backend...

Initializing provider plugins...
- Finding latest version of hashicorp/http...
- Finding rancher/rancher2 versions matching "4.1.0"...
- Finding equinix/equinix versions matching "1.34.1"...
- Finding latest version of hashicorp/random...
- Finding latest version of hashicorp/local...
- Installing hashicorp/http v3.4.2...
- Installed hashicorp/http v3.4.2 (signed by HashiCorp)
- Installing rancher/rancher2 v4.1.0...
- Installed rancher/rancher2 v4.1.0 (signed by a HashiCorp partner, key ID 2EEB0F9AD44A135C)
- Installing equinix/equinix v1.34.1...
- Installed equinix/equinix v1.34.1 (signed by a HashiCorp partner, key ID 1A65631C7288685E)
- Installing hashicorp/random v3.6.2...
- Installed hashicorp/random v3.6.2 (signed by HashiCorp)
- Installing hashicorp/local v2.5.1...
- Installed hashicorp/local v2.5.1 (signed by HashiCorp)

Partner and community providers are signed by their developers.
If you'd like to know more about provider signing, you can read about it here:
https://www.terraform.io/docs/cli/plugins/signing.html

Terraform has created a lock file .terraform.lock.hcl to record the provider
selections it made above. Include this file in your version control repository
so that Terraform can guarantee to make the same selections by default when
you run "terraform init" in the future.

Terraform has been successfully initialized!

You may now begin working with Terraform. Try running "terraform plan" to see
any changes that are required for your infrastructure. All Terraform commands
should now work.

If you ever set or change modules or backend configuration for Terraform,
rerun this command to reinitialize your working directory. If you forget, other
commands will detect it and remind you to do so if necessary.

@displague I understand the use of provider_meta after reading the documentation here - https://developer.hashicorp.com/terraform/internals/provider-meta
Basically, it would parse in the metadata for the provider whenever Terraform calls the Equinix provider and it would be useful for you to get all the necessary details.

@displague Can you please share a screenshot of how you are using the metadata or how you are exactly processing the metadata?
Then I will approve the change and merge it.

@devenkulkarni The metadata can be surfaced in any tool that processes HTTP UserAgent, any web traffic analyizer.
Because these requests are from Equinix customers (otherwise they would not have valid user tokens), and the requests are currently being sent as "...Terraform... (along with the version of the equinix provider, equinix-sdk-go, and go", this information helps to understand how users are interacting with the service and identify where we may need to reach out to users about changes in the service or behavior. (If they are using a deprecated feature of the API or the Terraform provider, we can follow-up.)
This visualization represents an alphabetical list of specific API behaviors made by public Terraform modules on equinix-labs that include the module_name, scale omitted:

Two CI failures to address, although I'm not sure that this PR introduced them
Set output stderr = ╷
##[debug]│ Error: Attempt to get attribute from null value
##[debug]│
##[debug]│   on main.tf line 3, in locals:
##[debug]│    3:   metro      = var.use_cheapest_metro ? local.cheapest_metro_price.metro : lower(var.metro)
##[debug]│     ├────────────────
##[debug]│     │ local.cheapest_metro_price is null
##[debug]│
##[debug]│ This value is null, so it does not have any attributes.
##[debug]╵
##[debug]╷
##[debug]│ Error: Failed to load provider configuration
##[debug]│
##[debug]│   with provider["registry.terraform.io/equinix/equinix"],
##[debug]│   on provider.tf line 18, in provider "equinix":
##[debug]│   18: provider "equinix" {}
##[debug]│
##[debug]│ the provider needs to be configured with the proper credentials before it
##[debug]│ can be used.
##[debug]│
##[debug]│ One of pair "client_id" - "client_secret" or "token" must be set in the
##[debug]│ provider
##[debug]│ configuration to interact with Equinix Fabric and Network Edge services,
##[debug]│ and
##[debug]│ "auth_token" to interact with Equinix Metal. These can also be configured
##[debug]│ using
##[debug]│ environment variables.
##[debug]│
##[debug]│ Please note that while the authentication arguments are individually
##[debug]│ optional to allow
##[debug]│ interaction with the different services independently, trying to provision
##[debug]│ the resources
##[debug]│ of a service without the required credentials will return an API error
##[debug]│ referring to
##[debug]│ 'Invalid authentication token' or 'error when acquiring token'.
##[debug]│
##[debug]│ More information on the provider configuration can be found here:
##[debug]│ https://registry.terraform.io/providers/equinix/equinix/latest/docs
##[debug]╵

@displague approved and merged.
Ignore the CI failures those are expected since you forked the repo and created this PR and CI needs access to the repo directly and some specific perms.  I am closing this issue now.


Makes the deviceflow auth URL simplier

Why are the changes needed?

Currently, device flow requires one to copy over the user code:
To Authenticate, navigate in a browser to the following URL:
https://signin.mydomain/activate and enter code: THISISACODE

What changes were proposed in this pull request?

This changes the URL one click with device-flow:
To Authenticate, navigate in a browser to the following URL:
https://signin.mydomain/activate?user_code=THISISACODE

My concern is if using user_code= is generic enough.

background reading: https://datatracker.ietf.org/doc/html/rfc8628#section-3.3


[AutoPR azure-kusto/resource-manager] Kusto: Add Kusto swagger specification, documentation and examples

Created to sync https://github.com/Azure/azure-rest-api-specs/pull/3584

(message created by the CI based on PR content)

This PR has been merged into https://github.com/Azure/azure-sdk-for-python/pull/3082


Add testSlugHelper test

As title.

@RaggiTech, could you help me to add the hacktoberfest-accepted tag on this issue? Thanks.


Runner

This is WIP for https://github.com/remind101/empire/issues/550. Opening it up now for a round of feedback.
There are lot of little details that I want to refactor and cleanup, but for the most part, this actually works pretty nicely:
Non-Interactive
https://asciinema.org/a/d38flqwpw36aqxep4ltollc8s
Interactive
https://asciinema.org/a/5amhjeznqg1fxhl85hliuf6of

[x] Refactor to set EMPIRE_* environment variables inside the run.
[x] Return error if -d flag is provided (we should consider these as potentially long running and hand them off to ECS to run. Eventually, we should add a Runner implementation for detached runs).
[x] Handle signals sent by the client.
[ ] Fix occasional bug where stdout doesn't appear.
[x] Replace "\n" with "\r\n" in pull output.

Bottom!

Alright. Pretty happy with this. Tests are gonna fail until the CLI changes are merged.

Signals now get properly passed through via ANSI escape sequences (ctrl+c kills sleep x properly).
Container gets killed when ctrl+c or ctrl+d is used.
Run was moved to the service.Manager interface. Next step in this would be to implement Run method for the ECS implementation for detached runs.

Would be good to add some logging whenever someone launches a one-off container.  Would probably be useful for admins of Empire.

:+1:

Nice :+1:

Closing this and re-opening against master.


heroku deploy instruction

It could be really helpful to have a heroku deploy button or simple instructions

Wouldn't Heroku be overkill for a static site?

ok


Add program constructor that accepts a vector of strings

The official OpenCL C++ bindings provide a program constructor that accepts a vector of strings for sources. Presumably this would be used to load a list of files into the vector. As it stands in boost compute, the way to obtain the same behavior is making a really long concatenated string and pass that instead.

---
Want to back this issue? **[Post a bounty on it!](https://www.bountysource.com/issues/28669409-add-program-constructor-that-accepts-a-vector-of-strings?utm_campaign=plugin&utm_content=tracker%2F402515&utm_medium=issues&utm_source=github)** We accept bounties via [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F402515&utm_medium=issues&utm_source=github).

Yeah, that shouldn't be too hard. And like you said, the best way currently is to create a single concatenated string.

Implemented in PR #539.

Sweet!


Search: Accessibility issue when screen-reader reads out search results

It would be great if the React integration for Semantic-UI is more screen-reader-friendly, but we understand that this issue may be out of scope. Feel free to close this issue if you think this is the case.
Steps

Install a screen-reader application
Go to http://react.semantic-ui.com/modules/search
Click on any search component to focus on it
Type something to bring up some search results
Press down (or up) arrow key to select a different search result

Expected Result
The screen-reader application should be able to focus on each search result as you go up or down the search results. Once focused, the application will be able to read the text out loud from the search result.
Actual Result
The screen-reader application is unable to focus on the currently active search result, and consequently is unable to read out the text.
Version
0.67.2

We've had some contributions to make this library more accessible and would love more.  You can reference these other PRs for guidance:
#1006 fix(Dropdown): added role/aria attributes to Dropdown and DropdownItem
#1002 fix(Icon): added aria-hidden attribute to icon  ready for review
#1001 fix(List): Added aria roles for list/list-item  ready for review
#892 feat(Rating): Rating now keyboard navigable and visually impaired accessible needs author feedback

Hi all! I'm looking to make my first contribution to the project. Can I pick up this issue?

@thenorthstarblues please do, we'd be thrilled to have you. You can find helpful setup info in the CONTRIBUTING.md in the .github directory.

Sounds good! Thank you.

@levithomason I did a quick accessibility check on the doc site (using http://wave.webaim.org/) to see what other things might come up. There are a couple of small things like logos missing alt text. The screen reader issue might take a little bit, but I was wondering if it would be helpful to work on these other accessibility-related issues as well? Should I open new issues for them here? Let me know what you think is best...thanks!

Thanks to @rijk, we have improved how we handle stale issues, see #2761.   Reopening.

Hello all, is this still an issue that is open for contribution? Already looking over the CONTRIBUTING.md.


provide Funcs to convert between Id and String

Set <=> Set
Map<String, Object> <=> Map<Id, Object>

#9 Conversion between Set, List, Set and List


METAMODEL-1205: JDK >8 support in core and excel (with POI v 4.0.0)

This is a step along the way to get to resolution of METAMODEL-1205 (supporting builds on JDKs > 8).
The main change in this PR is that I upgraded Apache POI to version 4.0.0.

I think our tests should cover most of the stuff, but the one thing that I will admit was somewhat complicated was what I noted in the third commit:

I ran into an issue that reminded me of what we saw some time back [1]
with writing to a file-based workbook failing with message "user-mapped
section open". I found a reasonable workaround where workbooks are being
loaded based on their InputStream rather than their File reference
whenever we access the workbook in an update callback.
[1] https://bz.apache.org/bugzilla/show_bug.cgi?id=58480

Or in terms of testing ... It would be interesting to compare performance and ability to do multiple inserts into the same sheet. I have a small fear that this new approach is slower because it is instantiating the Workbook based on InputStream instead of File. But OTOH I don't know of an alternative, because the File based thing is creating file system locks that prevent those inserts from happening without having to do very silly sleeps or something like that.

I like it! Thanks a bunch for testing it.

Okay, made an even  stupider test, this time doing multiple inserts, from multiple threads (I think we had problems with that on Excel previously) :)
import java.io.File;
import java.util.Collections;
import java.util.List;
import java.util.concurrent.Callable;
import java.util.concurrent.ExecutionException;
import java.util.concurrent.ExecutorService;
import java.util.concurrent.Executors;

import org.apache.metamodel.excel.ExcelDataContext;
import org.apache.metamodel.schema.Column;
import org.apache.metamodel.schema.Schema;
import org.apache.metamodel.schema.Table;

public class ExcelTest {
    private final ExcelDataContext dataContext;

    private ExcelTest(ExcelDataContext dataContext) {
        this.dataContext = dataContext;
    }

    private void go() throws InterruptedException {
        long startTime = System.currentTimeMillis();
        ExecutorService executor = Executors.newFixedThreadPool(8);

        List<Callable<Long>> workers = Collections.nCopies(16, this::addStuff);

        try {
            final long combinedTime = executor.invokeAll(workers).stream().mapToLong(future -> {
                try {
                    return future.get();
                } catch (InterruptedException | ExecutionException e) {
                    throw new IllegalStateException(e);
                }
            }).sum();
            long endTime = System.currentTimeMillis();
            System.out.println(
                    "Combined execution time: " + combinedTime + "ms, real time: " + (endTime - startTime) + "ms");
        } finally {
            executor.shutdown();
        }

    }

    private long addStuff() {
        long startTime = System.currentTimeMillis();

        Schema schema = dataContext.getDefaultSchema();
        Table table = schema.getTable(0);
        Column column1 = table.getColumn(0);
        Column column2 = table.getColumn(1);
        Column column3 = table.getColumn(2);

        char[] chars = "abcdefghijklmnopqrstuvwxyz".toCharArray();
        dataContext.executeUpdate(updateCallback -> {
            for (int i = 0; i < 100; i++) {
                updateCallback.insertInto(table).value(column1, i).value(column2, i * 10)
                        .value(column3, chars[i % chars.length]).execute();
            }
        });

        long endTime = System.currentTimeMillis();

        final long time = endTime - startTime;
        System.out.println("Thread '" + Thread.currentThread().getName() + "' execution time: " + time + "ms");

        return time;
    }

    public static void main(String[] args) throws InterruptedException {
        ExcelTest excelTest = new ExcelTest(new ExcelDataContext(new File("C:\\Users\\Dennis\\Documents\\empty.xlsx")));
        excelTest.go();
    }
}

The test runs fine, at least I saw no issues over 10-15 runs. It's obvious that workers are blocked while others are writing, but that's fair, as long as nothing goes wrong.

I think the conclusion must be that in general it works as it should, and with quite a good boost on XLS speed, maybe at the cost of a little bit of writing speed on the XLXS side.

D'oh, of course they're blocking, I'm doing the loop inside an update script! Let me know if you want me to change it. :-)

No I think this is perfect. Thanks again for doing extra due diligence on the PR. I think we can merge ...
Did you by any chance look at the two other JDK9+ related PRs?

Yeah, I agree, everything looks good.
Nope, not really. I'll take a look in a moment.


zombie child processes

Zombie created with each authentication request.
i think something like this http://stackoverflow.com/questions/17015830/how-can-i-prevent-zombie-child-processes can help you solve problem

Should be fixed now. I simply ignored SIGCHLD signals. Thank you for pointing this out!

nope. I still getting horde of zombies :)

on version 0.0.3? what type of system are you running this on? I confirmed the bug in 0.0.2, applied the fix, and it is no longer leaving zombies hanging on 0.0.3

CentOs 3.10.0-514.2.2.el7.x86_64. Oh, i see problem: you add code in "if (daemonize == 1) {}
". But i don't daemonize it :)

Doh! Should be good now.


[ui] Feature Request: Add progress bar during export of Chart and Images

Is your feature request related to a problem?
Given that the export of a Chart and Images can be lengthy, we could add a progress bar for the user in the UI.

Solution you'd like
No response
Alternatives you've considered
No response
Anything else?
No response

@torchiaf in theis repo we don't have the bot that automatically re-opens issues that were closed by merging a linked PR. this needs to happen manually. can you reopen this one and put it in to test?

Sure thing!

epinio.webm
Tested today with ghcr.io/epinio/epinio-ui:latest-next image.
The progress bar is there but for some reason the last "Compressing Files" step (5:18) is not part of it and actually I believe the compression step itself started much earlier than announced. @torchiaf is that expected?

epinio.webm
Tested today with ghcr.io/epinio/epinio-ui:latest-next image.
The progress bar is there but for some reason the last "Compressing Files" step (5:18) is not part of it and actually I believe the compression step itself started much earlier than announced. @torchiaf is that expected?

Hi @thehejik , this is not the same behavior we have in dev; the "Compressing files" label should be shown after ~1 second. Let me try it after refreshing our backend instance.

epinio.webm
Tested today with ghcr.io/epinio/epinio-ui:latest-next image.
The progress bar is there but for some reason the last "Compressing Files" step (5:18) is not part of it and actually I believe the compression step itself started much earlier than announced. @torchiaf is that expected?

@thehejik the compression phase works as expected; there is an issue with progress bar calculation, I opened a new issue to fix it: #313 .
This will also include the compression phase as part of the progress bar.

epinio.webm
Tested today with ghcr.io/epinio/epinio-ui:latest-next image.
The progress bar is there but for some reason the last "Compressing Files" step (5:18) is not part of it and actually I believe the compression step itself started much earlier than announced. @torchiaf is that expected?

@thehejik the compression phase works as expected; there is an issue with progress bar calculation, I opened a new issue to fix it: #313 . This will also include the compression phase as part of the progress bar.

Hi @torchiaf, I will attempt to follow this issue as Tomas is on holiday this week.
So per my understanding, the current status on this particular ticket is that approximately 1-2 seconds after the progress bar is done, the compression phase, with the subsequent label should appear.  Nothing else correct? If this is so, it works as expected and I can close this, but I'd like to confirm with you first.
Here I could see how approximately at second 20 the progress bar is done and the next part with the compression occurs
https://github.com/epinio/ui/assets/37271841/8eaf724e-1759-44be-87d9-329f12ada866
I understand that @thehejik comments are taken care of later on the next pr (313) where the compression is taken care altogether within the progress bar. Correct?

epinio.webm
Tested today with ghcr.io/epinio/epinio-ui:latest-next image.
The progress bar is there but for some reason the last "Compressing Files" step (5:18) is not part of it and actually I believe the compression step itself started much earlier than announced. @torchiaf is that expected?

@thehejik the compression phase works as expected; there is an issue with progress bar calculation, I opened a new issue to fix it: #313 . This will also include the compression phase as part of the progress bar.

Hi @torchiaf, I will attempt to follow this issue as Tomas is on holiday this week. So per my understanding, the current status on this particular ticket is that approximately 1-2 seconds after the progress bar is done, the compression phase, with the subsequent label should appear. Nothing else correct? If this is so, it works as expected and I can close this, but I'd like to confirm with you first. Here I could see how approximately at second 20 the progress bar is done and the next part with the compression occurs
Screencast.from.21-08-<IP_ADDRESS>.mov
I understand that @thehejik comments are taken care of later on the next pr (313) where the compression is taken care altogether within the progress bar. Correct?

Hi @mmartin24 correct. In addition, #313 will also fix a small an issue with the progress bar filling.
The issue is more visible in the video from Thomas (probably because the image is larger in his test).
With 313, the bar should gradually fills up.

epinio.webm
Tested today with ghcr.io/epinio/epinio-ui:latest-next image.
The progress bar is there but for some reason the last "Compressing Files" step (5:18) is not part of it and actually I believe the compression step itself started much earlier than announced. @torchiaf is that expected?

@thehejik the compression phase works as expected; there is an issue with progress bar calculation, I opened a new issue to fix it: #313 . This will also include the compression phase as part of the progress bar.

Hi @torchiaf, I will attempt to follow this issue as Tomas is on holiday this week. So per my understanding, the current status on this particular ticket is that approximately 1-2 seconds after the progress bar is done, the compression phase, with the subsequent label should appear. Nothing else correct? If this is so, it works as expected and I can close this, but I'd like to confirm with you first. Here I could see how approximately at second 20 the progress bar is done and the next part with the compression occurs
Screencast.from.21-08-<IP_ADDRESS>.mov
I understand that @thehejik comments are taken care of later on the next pr (313) where the compression is taken care altogether within the progress bar. Correct?

Hi @mmartin24 correct. In addition, #313 will also fix a small an issue with the progress bar filling. The issue is more visible in the video from Thomas (probably because the image is larger in his test). With 313, the bar should gradually fills up.

Thanks @torchiaf for confirming.
Yes I saw the issue was clear on Tomas video taking longer times to appear the compression label. Mine simply shows the shorter time to appear the compression label. What I needed to know is if that was all the expected behavior for the moment. Since it seems to be, I will close the issue and later we can take a look at Tomas's comments on https://github.com/epinio/ui/pull/313

Verified in v1.10.0-rc1


pin microk8s version

microk8s just released 1.16 thus breaking us. we need to pin microk8s to avoid this in the future

clusterrolebinding.rbac.authorization.k8s.io/add-on-cluster-admin created
Creating /home/travis/.helm
Creating /home/travis/.helm/repository
Creating /home/travis/.helm/repository/cache
Creating /home/travis/.helm/repository/local
Creating /home/travis/.helm/plugins
Creating /home/travis/.helm/starters
Creating /home/travis/.helm/cache/archive
Creating /home/travis/.helm/repository/repositories.yaml
Adding stable repo with URL: https://kubernetes-charts.storage.googleapis.com
Adding local repo with URL: http://<IP_ADDRESS>:8879/charts
$HELM_HOME has been configured at /home/travis/.helm.
Error: error installing: the server could not find the requested resource


[SDK + Dashboard] BatchMintAddtionalSupplyTo

Problem solved
Short description of the bug fixed or feature added

PR-Codex overview
This PR introduces functionality for batch minting additional supply to NFTs in the erc1155 extension, enhancing the NFT drawer UI with a new tab for this feature.
Detailed summary

Added batchMintAdditionalSupplyTo function in erc1155 extension.
Introduced BatchMintAdditionalSupplyTab component in the NFT drawer.
Updated useNFTDrawerTabs to include the new batch minting tab.
Integrated form handling for batch minting in the new tab.

✨ Ask PR-Codex anything about this PR by commenting with /codex {your question}

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#4934  👈
#4932
#4931
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @kien-ngo and the rest of your teammates on  Graphite


Custom licenses not displayed since 8.3.0

About this issue
Since 8.3.0 the custom licenses are not being displayed. Downgrading to 8.2.0 fixes the issue.
(I checked 8.4.2 but the issue is still there)

can you please showcase in a sample app what you do and what is not working?
I know that for various users it's working

Thanks for the quick answer, I uploaded a sample app here with a single personalized library called Bunch that uses a custom license (see res/values/about_libraries.xml).
I setup version 8.2.0 so the Bunch license is correctly displayed. If I upgrade it to 8.3.0 the Bunch library will still be displayed but with an empty license.

Thank you. I'll try it out as soon as possible and let you know what's up

@awoisoak the change to support multiple licenses changed the specification from:

add support for custom license mapping to allow multiple licenses

Please change
<string name="library_xcom_tompee_bunch__annotations_licenseId" tools:ignore="UnusedResources">bunch</string>

To:
<string name="library_xcom_tompee_bunch__annotations_licenseIds" tools:ignore="UnusedResources">bunch</string>

Also please consider using the latest release of the library v8.4.2.
I will have a look and possibly adjust it to fallback to the licenseId if no licenseIds is specified

Thank you!


Fix link to the contribution guide in README.md

File names are case sensitive. Updated link to CONTRIBUTING.md in the READEME file

Thanks for the fix!


PR Workflow Test (Feature)

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (f9ee2c5) 87.23% compared to head (8980a9d) 87.23%.
Report is 6 commits behind head on release/pr-workflow-test.

Additional details and impacted files
@@                    Coverage Diff                    @@
##           release/pr-workflow-test     #221   +/-   ##
=========================================================
  Coverage                     87.23%   87.23%
=========================================================
  Files                             7        7
  Lines                            47       47
  Branches                         12       12
=========================================================
  Hits                             41       41
  Misses                            6        6

Flag
Coverage Δ

html-ui
93.10% <ø> (ø)

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.


Question: How do I listen to an activation of an Application? OnLaunched is not fired when calling AppInstance.RedirectActivationToAsync(activatedEventArgs).

Describe the bug
My App.xaml.cs looks like this (following the docs at https://learn.microsoft.com/en-us/windows/apps/windows-app-sdk/migrate-to-windows-app-sdk/guides/applifecycle):
public partial class App : Application
{
    private MainWindow m_window;

    public App()
    {
        this.InitializeComponent();
    }

    protected override async void OnLaunched(LaunchActivatedEventArgs args)
    {
        string debugPath = Path.Combine(Environment.GetFolderPath(Environment.SpecialFolder.ApplicationData), "TrackMixerv2", "debug.txt");
        var mainInstance = AppInstance.FindOrRegisterForKey("main");

        if (!mainInstance.IsCurrent)
        {
            // Redirect the activation (and args) to the "main" instance, and exit.
            var activatedEventArgs = AppInstance.GetCurrent().GetActivatedEventArgs();
            await mainInstance.RedirectActivationToAsync(activatedEventArgs);
            Process.GetCurrentProcess().Kill();
            return;
        }
        File.AppendAllText(debugPath, "Test");
        // Create the main window if it doesn't already exist
        if (m_window == null)
        {
            m_window = new MainWindow();
        }
        m_window.Activate();
        // Check if the activation is due to file launch
        AppActivationArguments arguments = AppInstance.GetCurrent().GetActivatedEventArgs();
        if (arguments.Kind == ExtendedActivationKind.File)
        {
            // Extract the file paths from the arguments
            var fileArgs = arguments.Data as Windows.ApplicationModel.Activation.FileActivatedEventArgs;
            if (fileArgs != null)
            {
                var files = fileArgs.Files.Select(file => file.Path).ToArray();
                // Call the AddNewTabs method on the existing main window
                m_window.AddNewTabs(files);
            }
        }
    }
}

But my test file does only append the string "Text" once on the main startup, but not on the activation from another instance, confirming that OnLaunched is not called.
Steps to reproduce the bug

Create an unpackaged Application
Override the OnLaunched to support single-instancing
Call RedirectActivationToAsync on the main instance
Verify OnLaunched not being called

Expected behavior
No response
Screenshots
No response
NuGet package version
WinUI 3 - Windows App SDK 1.4.1: 1.4.230913002
Windows version
Windows 11 (22H2): Build 22621
Additional context
No response

Oh, thank you very much! Yeah, I figured it was probably a problem on my end but I did not know under which tag to post other than bug.


feat: polished buttons

What has been done?
Allows polishing buttons

Requires level 3 agility might be a mistake here, would require 3 crafting :)

Oh! Ha, yea, thanks for spotting that :)


Feature Request: Will you add the Yolov7 model?

What should I do to add a new YOLO model?

Hi @kadirnar!
Other than adapting Yolov7 detections to be compatible with Norfair's Detection class, there shouldn't be any issues with using it.
If you're asking for a demo of Norfair using Yolov7, that can be an exciting addition. We can add it to our backlog.

@facundo-lezama  I tested the Norfair library yolov7 model. I sent PR.


Validate program type in ebpf_program_create

Description
__ebpf_program_parameters had program_type not set. Hence the ebpf program failed to load and attach.
Testing
_Do any existing tests cover this change? Yes
Are new tests needed? No
Manual testing was done from ebpf-for-windows-demo using ebpf-for-window's ebpfcore changes, and collected the traces to verify.
PS C:\ebpf-for-windows-demo\Debug>  .\cilium_based_agent.exe --bpf-lb-mode=snat --device=ethernet
Initializing daemon with mode = snat
Using interface:
name         = ethernet
mtu          = 1500
ifindex      = 8
IPv4 address = <IP_ADDRESS>
Cleaning up previous configuration ...
Compiling XDP eBPF program for SNAT ...
Verifying the program ...
Loading and attaching the program to XDP hook ...
bpf_object__load failed with error=-22
Failed to load and attach the XDP program, error = -22
PS C:\ebpf-for-windows-demo\Debug>
Logman trace:

{"Entry":"_ebpf_core_protocol_create_program","meta":{"provider":"EbpfForWindowsProvider","event":"EbpfGenericMessage","time":"2023-06-24T02:45:16.3382364Z","cpu":1,"pid":7184,"tid":7032,"channel":11,"level":5,"opcode":1,"keywords":"0x1"}}
[1]1C10.1B78::2023/06/23-19:45:16.338237300 [EbpfForWindowsProvider]{"Entry":"ebpf_program_create_and_initialize","meta":{"provider":"EbpfForWindowsProvider","event":"EbpfGenericMessage","time":"2023-06-24T02:45:16.3382373Z","cpu":1,"pid":7184,"tid":7032,"channel":11,"level":5,"opcode":1,"keywords":"0x1"}}
[1]1C10.1B78::2023/06/23-19:45:16.338238000 [EbpfForWindowsProvider]{"Entry":"ebpf_program_create","meta":{"provider":"EbpfForWindowsProvider","event":"EbpfGenericMessage","time":"2023-06-24T02:45:16.3382380Z","cpu":1,"pid":7184,"tid":7032,"channel":11,"level":5,"opcode":1,"keywords":"0x1"}}
[1]1C10.1B78::2023/06/23-19:45:16.338239400 [EbpfForWindowsProvider]{"Message":"Program type must be specified.","*guid":"{00000000-0000-0000-0000-000000000000}","meta":{"provider":"EbpfForWindowsProvider","event":"EbpfGenericMessage","time":"2023-06-24T02:45:16.3382394Z","cpu":1,"pid":7184,"tid":7032,"channel":11,"level":2,"keywords":"0x80"}}
[1]1C10.1B78::2023/06/23-19:45:16.338240300 [EbpfForWindowsProvider]{"ErrorMessage":"ebpf_program_create returned error","Error":6,"meta":{"provider":"EbpfForWindowsProvider","event":"EbpfGenericError","time":"2023-06-24T02:45:16.3382403Z","cpu":1,"pid":7184,"tid":7032,"channel":11,"level":2,"keywords":"0x2"}}
[1]1C10.1B78::2023/06/23-19:45:16.338245500 [EbpfForWindowsProvider]{"ErrorMessage":"ebpf_program_create_and_initialize returned error","Error":6,"meta":{"provider":"EbpfForWindowsProvider","event":"EbpfGenericError","time":"2023-06-24T02:45:16.3382455Z","cpu":1,"pid":7184,"tid":7032,"channel":11,"level":2,"keywords":"0x2"}}
[1]1C10.1B78::2023/06/23-19:45:16.338246200 [EbpfForWindowsProvider]{"ErrorMessage":"_ebpf_core_protocol_create_program returned error","Error":6,"meta":{"provider":"EbpfForWindowsProvider","event":"EbpfGenericError","time":"2023-06-24T02:45:16.3382462Z","cpu":1,"pid":7184,"tid":7032,"channel":11,"level":2,"keywords":"0x2"}}
[1]1C10.1B78::2023/06/23-19:45:16.338253800 [EbpfForWindowsProvider]{"Api":"\"ebpf_core_invoke_protocol_handler\"","status":"0xC000000D(NT=An invalid parameter was passed to a service or function.)","meta":{"provider":"EbpfForWindowsProvider","event":"EbpfApiError","time":"2023-06-24T02:45:16.3382538Z","cpu":1,"pid":7184,"tid":7032,"channel":11,"level":2,"keywords":"0x4"}}
[1]1C10.1B78::2023/06/23-19:45:16.338286400 [EbpfForWindowsProvider]{"Api":"DeviceIoControl","last_error":"87(WIN=The parameter is incorrect.)","meta":{"provider":"EbpfForWindowsProvider","event":"EbpfApiError","time":"2023-06-24T02:45:16.3382864Z","cpu":1,"pid":7184,"tid":7032,"channel":11,"level":2,"keywords":"0x100"}}
[1]1C10.1B78::2023/06/23-19:45:16.338287200 [EbpfForWindowsProvider]{"ErrorMessage":"invoke_ioctl returned error","Error":87,"meta":{"provider":"EbpfForWindowsProvider","event":"EbpfGenericError","time":"2023-06-24T02:45:16.3382872Z","cpu":1,"pid":7184,"tid":7032,"channel":11,"level":2,"keywords":"0x4"}}

Documentation
_Is there any documentation impact for this change? No

Description
__ebpf_program_parameters had program_type not set. Hence the ebpf program failed to load and attach.
Testing
_Do any existing tests cover this change? Are new tests needed? No

I don't follow why "Are new tests needed" is a no.
This seems trivial to add a unit test to cover.

Are new tests needed" is a no.

Thanks for your comment. There is an existing test case with program_type set to 0. Updated the description.
TEST_CASE("EBPF_OPERATION_QUERY_PROGRAM_INFO", "[execution_context][negative]")

There is an existing test case with program_type set to 0. Updated the description. TEST_CASE("EBPF_OPERATION_QUERY_PROGRAM_INFO", "[execution_context][negative]")

Were we seeing CI/CD failures due to this issue?  If so, why is there no issue linked to this PR?  Or if not, then why isn't a test update needed?

Were we seeing CI/CD failures due to this issue?

Yes, there was a CI/CD failure reported. And I have updated the test case EBPF_OPERATION_QUERY_PROGRAM_INFO in the latest diff.

Were we seeing CI/CD failures due to this issue?

Yes, there was a CI/CD failure reported.

Please link the issue # to the PR description.


Superfluous Debug message

Useful library, good work!
This debug message was probably left in by mistake. It screws usage of the library on windows, because it outputs to stdout (when it shouldn't output at all).

Thanks!


[INLONG][feature][audit] add audit-common module

Fixes #1880
Motivation
add audit-common module

Codecov Report

Merging #1904 (b391207) into master (cf771bc) will decrease coverage by 0.01%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master    #1904      +/-   ##
============================================
- Coverage     11.61%   11.59%   -0.02%
+ Complexity     1055     1053       -2
============================================
  Files           401      401
  Lines         34881    34881
  Branches       5525     5525
============================================
- Hits           4050     4045       -5
- Misses        30064    30070       +6
+ Partials        767      766       -1

Impacted Files
Coverage Δ

.../client/producer/qltystats/BrokerStatsItemSet.java
67.44% <0.00%> (-4.66%)
:arrow_down:

.../producer/qltystats/DefaultBrokerRcvQltyStats.java
44.14% <0.00%> (-1.18%)
:arrow_down:

Continue to review full report at Codecov.

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

@baomingyu Pull request should respect the PULL_REQUEST_TEMPLATE


__int128_t not supported in Visual Studio 2017 C++14

Need to devise a work-around to __int128_t not being supported in VS2017 C++14 since this breaks builds for the Windows platform.

Worked around this by ifndef'ing away code that relied on __int128_t being available. Changes finished at about commit c279d935dbc85a742557ad42bdd2bee05fd0cdd3.


Add support for swift caching with path remapping and builtin cache replay

Retry add swift caching support.

Add a workaround for swift package manager if the option to disable implicit concurrency is still required.

@swift-ci please test

Doing a cross-repo test from the swift repo can validate the previous failure, I think.

And/Or  from the swiftpm repo.

Cross repo here: https://github.com/apple/swift-package-manager/pull/7125

Cross repo testing passing now. Let's try again. Feel free to revert directly if anything else break.


bugfix on Jira comment store.

The Jira CommentStore fails to sync due to a wrong regex.

/cc @bradmwilliams

/lgtm


Pager url issues in category pages when blog is not root

Some of the values in PagerModel are not set correctly inside virtual pages. In the case I'm running into, the blog is a child of the home page, with the url "/blog." When I visit one of the category pages Model.NextUrl and Model.PreviousUrl in the Pager partial both start with "/blog/blog."

@jaandrews how did you manage to get the pager partial to include the numerical navigation links and include the actual links themselves to the respective pages?
I am struggling to identify the root Url for the blog to enable me to populate this using variables and then assigning them to the relevant links.

I have created a topic on this here: https://our.umbraco.org/projects/starter-kits/articulate/discussions/61107-Articulate-Pager-on-listcshtml-view,-show-current-page-number-but-be-a-link-to-respective-page-of-article


Packaging requests: hexo

Project description
Hexo is a fast, simple & powerful blog framework powered by Node.js.
Metadata

homepage URL: https://hexo.io/
source URL: https://github.com/hexojs/hexo
license: mit
platforms: linux

related: I've packaged hexo-cli in https://github.com/NixOS/nixpkgs/pull/254070


It should be possible to install CNI plugin through Kubernetes API

This would imply that that we need the router to run as a DaemonSet.

Could you expand a little bit?  What sequence of commands would you expect to run to achieve this?

I looked into this; currently kubelet will fail at startup unless there is at least one CNI network configured.
https://github.com/kubernetes/kubernetes/pull/28178 should fix that.

@bboreham what is the answer to your own question? i.e. post the fix you mention, what sequence of commands would we envisage?

I have now successfully tested with a DaemonSet using this manifest:
apiVersion: extensions/v1beta1
kind: DaemonSet
metadata:
  name: weave-net
spec:
  template:
    metadata:
      labels:
        name: weave-net
    spec:
      hostNetwork: true
      containers:
        - name: weave
          image: weaveworks/weave-kube:1.6.1
          command:
            - /home/weave/launch.sh
          env:
            - name: WEAVE_VERSION
              value: kube-test
          securityContext:
            privileged: true
          volumeMounts:
            - name: weavedb
              mountPath: /weavedb
            - name: cni-bin
              mountPath: /opt
            - name: cni-conf
              mountPath: /etc
          securityContext:
            privileged: true
      restartPolicy: Always
      volumes:
        - name: weavedb
          emptyDir: {}
        - name: cni-bin
          hostPath:
            path: /opt
        - name: cni-conf
          hostPath:
            path: /etc

This uses a container built off weaveexec, containing a program to fetch peer names via the Kubernetes API and this startup script:
#!/bin/sh

set -e

/home/weave/weave --local setup-cni
/home/weave/weave --local create-bridge

exec /home/weave/weaver --port=6783 --datapath=datapath \
     --http-addr=<IP_ADDRESS>:6784 --docker-api='' --no-dns \
     --ipalloc-range=<IP_ADDRESS>/14 \
     --name=$(cat /sys/class/net/weave/address) $(/home/weave/kube-peers)

I have put this in its own repo: https://github.com/weaveworks/weave-kube


bug: not compatible with other firebase community plugins

Plugin version:
0.4.2
Platform(s):
ios
Current behavior:
when adding other firebase plugins like @capacitor-community/firebase-crashlytics or @capacitor-community/firebase-analytics there seem to be versions conflicts in Podfile. When syncing with cap sync this results in:
[error] Analyzing dependencies
        [!] CocoaPods could not find compatible versions for pod "Firebase/CoreOnly":
        In Podfile:
        CapacitorCommunityFirebaseCrashlytics (from `../../node_modules/@capacitor-community/firebase-crashlytics`) was
        resolved to 1.2.1, which depends on
        Firebase/Crashlytics (= 8.4.0) was resolved to 8.4.0, which depends on
        Firebase/CoreOnly (= 8.4.0)

        RobingenzCapacitorFirebaseAuthentication (from
        `../../node_modules/@robingenz/capacitor-firebase-authentication`) was resolved to 0.4.2, which depends on
        Firebase/Auth (= 7.11.0) was resolved to 7.11.0, which depends on
        Firebase/CoreOnly (= 7.11.0)

Expected behavior:
no conflicts on Firebase/CoreOnly and able to run a sync without problems
Steps to reproduce:
add:
...
"@capacitor-community/firebase-crashlytics": "^0.3.0",
...

to your capacitor package.json and run npx cap sync
Capacitor doctor:
Latest Dependencies:

  @capacitor/cli: 3.4.3
  @capacitor/core: 3.4.3
  @capacitor/android: 3.4.3
  @capacitor/ios: 3.4.3

Installed Dependencies:

  @capacitor/cli: 3.4.2
  @capacitor/core: 3.4.2
  @capacitor/android: 3.4.2
  @capacitor/ios: 3.4.2

[success] iOS looking great! 👌
[success] Android looking great! 👌

Hi @florisvink,
thank you for your request.
I am aware of the problem. I am already working on a solution related to https://github.com/capacitor-community/proposals/issues/112.
I'll let you know as soon as I publish it.

@florisvink I have updated the pods and released new versions.
You can give it a try:
npm i<EMAIL_ADDRESS><EMAIL_ADDRESS>
I also updated https://github.com/robingenz/capacitor-firebase-app and https://github.com/robingenz/capacitor-firebase-performance.
Let me know if this helps.

I will close the issue for it seems resoled

Long term solution: https://github.com/robingenz/capacitor-firebase 🎉


try git first when detecting VCS

It is reasonable change because there are more git repositories than hg.

Reasonable enough, thanks!


Support parallel requests

Exokit hasn't supported (or really needed to support) parallelized dom load so far, but we've been seeing some timeout issues when resources like images and scripts load too slowly (serially) on the initial dom bootstrap.
This issue is to track adding support for parallel "running" of the html on load. This seems pretty simple; I think we just need to keep a refcount of the number of resources being loaded in parallel so we know when to emit DOMContentLoaded.
Note that things other than the initial dom load are already parallelized by node's I/O. This is just about the initial load.

This one has gotten worse for some script-juggling load engines after we implemented properly wait for the manual script attach case in https://github.com/webmixedreality/exokit/pull/317.