DevOps Agent with private IP address <IP_ADDRESS>

We enabled firewall settings for Storage account to allow public IP CIDR for Azure Devops to access it. However during testing we periodically see 403 error, after digging into storage account log we found that somehow the traffic from Devops agent to storage account changed to <IP_ADDRESS> which is an unknown internal IP address and therefore could not pass firewall rule set on the storage account, would you please suggest how to address this?

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

ID: dc5a4cfa-aa84-9f0f-55f4-71ec4861c070
Version Independent ID: d621620a-c924-03b1-8b43-f99c33e58bb8
Content: Add IP addresses and URLs to allow list - Azure DevOps
Content Source: docs/organizations/security/allow-list-ip-url.md
Product: devops
Technology: devops-security
GitHub Login: @chcomley
Microsoft Alias: ChComley

I have the same problem !

We can see similar issue <IP_ADDRESS> getting assigned to agents

We can see similar issue <IP_ADDRESS> getting assigned to Azure Devops agents which is getting rejected

@CharmanderJieniJieni Thank you for question, we apologize for the issue.
You can see if this post resolves the problem, else you should try one of the following support options:

Azure DevOps community
Azure DevOps support

For any other documentation concerns, please reference and add comments below.

This still happens, for anyone else finding this article.
Seems to happen (sometimes) when the devops agent is in the same region as the storage account.

This is still not resolved. The AzureDevOps MS Hosted agents are not trusted in Azure and They make the private connenction when the resources are in same region. This make the upload messed up

We have the same problem. One workaround was to set the "Network Routing" in the storage account to "Internet Routing" instead of "Microsoft network routing". Atleast the deployment works now and internet routing is sufficient in our case.

@nsoeth but who does Storage account and MS agent communicate with Private IP

We have the same problem. One workaround was to set the "Network Routing" in the storage account to "Internet Routing" instead of "Microsoft network routing". Atleast the deployment works now and internet routing is sufficient in our case.

This didnt work for me at all; the only way I can get it to work is making it fully publicly accessible

@danpetitt have you seen this workaround? You might change the firewall rule to allow public inbound connections after the pipeline revert it back

In your pipeline, use Azure CLI to update the network ruleset for your Azure Storage account right before you access storage, and then restore the previous ruleset.

@communauto-asato yeah that's what I'm having to do and sort of works ok for now thx


The comment reply box has no border

It's not clear where to write the comment. Put border: 1px solid black; in the css.

This isn't as easy to fix as it first looks. It seems to be a problem with the library we're using.
Considering moving away from markdown and into something more WYSIWYG (since it's now aimed more broadly than just programmers) so we'll see if this is needed.

fixed


Automatically fix merged toggle mods

Story
For merged mods, previously users would have to go run the program in each individual mod, then make a custom mapping function in the merged.ini to each individual generated RaidenShogunRemapBlend.buf files
As a quality of life feature change, now the user can just run the program where the merged.ini is located.
This change is also compatible with mods that are not merged.
Demo
(IF THE LINKS DO NOT WORK OR HAVE EXPIRED, THE DEMO VIDEOS ARE ALSO IN THE ZIP FILE)
Links
Fixing Merged Mod
Compatible Fixing of Unmerged Mod
Zip File
videos_of_basic_tests.zip
Notes

Most of the new code uses the original code's model, then wrapped the model with lots of infrastructure and software design for robustness.

Although it would be better practice to put the individual classes into separate files, we stick all our classes into one file to keep the simple UI of the user copying the program into their desired mod and running the program. For better organization and software design, it would be better to separate out the classes, but a new UI may need to be introduced.

thank you for the best work 😍


Hull fails with a single node member

G6 Version:
Platform:
Mini Showcase(like screenshots):
CodePen Link:

The latest version of G6 has a bug when a hull has only a single member. For instance, if you change the example from here:
https://g6.antv.vision/en/examples/interaction/hull/API
...to include only a single member in the hull, and rendering the graph just fails.

Thanks, we will fix it as soon as possible

It has been fixed by 4.1.0-beta.1

Thanks!

v4.1.0 is released

v4.1.0 is released


Chunk large satellite payloads

To prevent the 100MB payload limit:
[Symbol(kError)]: RangeError: Max payload size exceeded
      at Receiver.haveLength (/Users/rob/src/electric-sql/electric/node_modules/.pnpm/ws@8.17.0/node_modules/ws/lib/receiver.js:419:28)
      at Receiver.getPayloadLength64 (/Users/rob/src/electric-sql/electric/node_modules/.pnpm/ws@8.17.0/node_modules/ws/lib/receiver.js:406:10)
      at Receiver.startLoop (/Users/rob/src/electric-sql/electric/node_modules/.pnpm/ws@8.17.0/node_modules/ws/lib/receiver.js:161:16)
      at Receiver._write (/Users/rob/src/electric-sql/electric/node_modules/.pnpm/ws@8.17.0/node_modules/ws/lib/receiver.js:94:10)
      at writeOrBuffer (node:internal/streams/writable:564:12)
      at _write (node:internal/streams/writable:493:10)
      at Writable.write (node:internal/streams/writable:502:10)
      at Socket.socketOnData (/Users/rob/src/electric-sql/electric/node_modules/.pnpm/ws@8.17.0/node_modules/ws/lib/websocket.js:1303:35)
      at Socket.emit (node:events:519:28)
      at addChunk (node:internal/streams/readable:559:12) {
    code: 'WS_ERR_UNSUPPORTED_MESSAGE_LENGTH',
    [Symbol(status-code)]: 1009
    ```

@robacourt Hey, I don't know if there was a discussion behind this that I missed, but this change doesn't address the issue stated in the PR description. For one, a single INSERT or UPDATE may be quite large, so the limit of 100 ops per SatOpLog does not in itself prevent it from overflowing.
More importantly, a single SatOpLog is treated by the client as a transaction. This change effectively splits a single PG transaction into multiple client-side transactions. This might not be an issue in the current implementation due to the fact that transactions are processed on the client in the same order as they are sent by the server, but any addition of concurrency to client-side processing further down the road could lead to subtle bugs because of this.
A proper solution to the payload size problem requires changing the Satellite protocol to support splitting a single transaction into multiple protocol messages.

test/satellite/client.test.ts

I think we have one already: https://github.com/electric-sql/electric/blob/31ffde3b0912d3b4e16ee38e5b44ae032b77cb48/clients/typescript/test/satellite/client.test.ts#L275
Is that enough?


Properly detect the availability of pthread_setname_np()

This commit adds a CMake check for the availability of
pthread_setname_np(), and only uses it on Linux when available.
Indeed, some C libraries, such as uClibc, do not provide this
non-POSIX function in all cases.
Signed-off-by: Thomas Petazzoni<EMAIL_ADDRESS>

Thank you for your pull request. Before we can look at your contribution, we need to ensure all contributors are covered by a Contributor License Agreement.
After the following items are addressed, please respond with a new comment here, and the automated system will re-verify.

User @tpetazzoni isn't covered by a CLA. They will need to complete the form at https://identity.linuxfoundation.org/projects/cncf

Regards,
CLA GitHub bot

@tpetazzoni Please sign the CLA first, thanks!

I already signed the CLA back on December 31. Github says "cla/linuxfoundation ** — tpetazzoni authorized", so I'm not sure why you think I haven't signed the CLA.

Oh, the test check below shows you are already authorized. But the last comment from @thelinuxfoundation shows that you are not. I thought one has to add a comment to the PR after signing CLA to trigger the update.. But I'm wrong..
Looks like there is some conflict to resolve before test can run.


Recompilation of file takes into account previous partial movie files

When running manim on a file twice with the same scene, if the second version of the file has fewer animations than the first version, the old animations that had a higher indexes than the second version are kept.
For example by running twice this file with manim test_issue_partial.py -pl :
Version 1
from manim import *

class TestPartialsScene(Scene):
    def construct(self):
        square = Square()
        self.play(FadeIn(square))
        self.wait()
        self.play(FadeOut(square))

Version 2
from manim import *

class TestPartialsScene(Scene):
    def construct(self):
        triangle = Triangle()
        self.play(FadeIn(triangle))
        self.wait()

Normally, three partial movie files should be created for the first version :

square FadeIn
wait
square FadeOut

Of course deleting the partial movies solves the issue but I am guessing this operation should not be left to the user.
And two for the second version :

triangle FadeIn
wait

However, the current result is that the third partial movie of the first version is kept and the clip goes as follow :

triangle FadeIn
wait
square FadeOut

Of course deleting the partial movies solves the issue, but I am guessing it is not an operation that should be left to the user.

In theory there is such a functionnality within manim.
In get_sorted_integer_files (L.31, file_ops.py), when remove_indices_greater_than is set partial movies files with an index to greater than the number of animations are deleted. I don't really know why is it broken.
Anyway, #166 will fix that.


The activate script adds a new entry for fermitools everytime it runs

... even when the entry is already there.
This is done in the ${CONDA_PREFIX}/etc/root/system.rootrc, in the initial section like:
Unix.*.Root.DynamicPath:    .:/Users/cr/miniconda2/envs/st_test/lib/root/cint/cint/stl:/Users/cr/miniconda2/envs/st_test/lib/root/cint/cint/include::/Users/cr/miniconda2/envs/st_test/lib/fermitools/:/Users/cr/miniconda2/envs/st_test/lib/fermitools/:/Users/cr/miniconda2/envs/st_test/lib/fermitools/:/Users/cr/miniconda2/envs/st_test/lib/fermitools/:/Users/cr/miniconda2/envs/st_test/lib/fermitools/:/Users/cr/miniconda2/envs/st_test/lib/fermitools/
The deactivate script takes one entry out when run when the user does source deactivate, but users rarely do so. If they just close the terminal then the path is not removed. Next time a new shell is opened and a new environment activate, a new entry will be added again. So the number of entries will grow indefinitely.
We need to fix this by making the activate script check whether an entry is there already, and only if it isn't it should insert it.

Which variable do we need to check? $PATH or  "Unix.*.Root.DynamicPath"?

That checks the $PATH. I'm not sure yet about "Unix.*.Root.DynamicPath"


background take picture

Hi
How to take picture on background

This is an issue tracker, not a support board. The documentation has instructions on how to get support.
Assuming that "take picture on background" means "take a picture from a background process", this library does not support that.


Can't get SAN the same way anymore

From comment/question #27 you had details on how to get the SAN from a certificate. With PowerShell 7 and the new .NET it doesn't work anymore. It seems x509subjectalternativenamesextension is gone and X509SubjectAlternativeNameExtension remains.
Here is what you had:
$row = Get-CA ca01.company.com | Get-DatabaseRow -Table Extension -RowID <rowid> -Filter "ExtensionName -eq <IP_ADDRESS>" $rawBytes = [convert]::frombase64string($row.ExtensionRawValue) $asn = new-object security.cryptography.asnencodeddata @(,$rawBytes) $san = new-object security.cryptography.x509certificates.x509subjectalternativenamesextension $asn,0
Now it only throws errors on the multiple previously working scripts I had. The closest I get to it still throws this error:
Cannot find an overload for "X509SubjectAlternativeNameExtension" and the argument count: "2".
I constructed it similar to that original one but the SAN line looks like this and is in a Try-Catch:
$san = New-Object Security.Cryptography.X509Certificates.X509SubjectAlternativeNameExtension $asn,$false
Any idea how I can get this working again? I feel like it is interpreting some data wrong now but I can't get it to read it correctly.

It seems x509subjectalternativenamesextension is gone

it isn't gone, it remains, just moved. Latest release brings a lot of resource move out from Microsoft namespaces (System.Security.Cryptography.*) and moving to my own. You can see a list of moved types in the end of release notes: https://www.pkisolutions.com/tools/pspki/release-notes-for-pspki-v400/. The X509SubjectAlternativeNameExtension` (singular) is Microsoft implementation which was added in .NET 7 and yet doesn't allow you to get everything from SAN extension comparing to my implementation.
so now, if you want to use my implementation of X509SubjectAlternativeNamesExtension, you must use new namespace:
$san = new-object SysadminsLV.PKI.Cryptography.X509Certificates.X509SubjectAlternativeNamesExtension $asn,0

@darkrhyes was your issue solved with my previous comment?

SysadminsLV.PKI.Cryptography.X509Certificates.X509SubjectAlternativeNamesExtension

Yes


how to train muti-label 3d ct image in sam

I would like to ask how the author trained and tested the Synapse multi-organ CT(BTCV) dataset, and did he have to convert each multi-label label to the onehot format one by one? thanks！

同问


dataset_info.json and features.json for long_horizon dataset

Would you please add dataset_info.json and features.json for long_horizon dataset? Otherwise, the dataset is unusable.

The same question!!! The long_horizon dataset cannot be parsed without dataset_info.json and features.json.


[Bug]: Embeddings based query does not returns documents

What happened?
Here are my steps:
Step 1) Create embedding for document-1(a small document with 100 chunks) and save in chroma-db vector store
After Step 1) making any query relevant to document-1 returns correct document chunks with high similarity distance eg. 0.8
Step 2) Create embedding for document-2(a very large document with 100000 nodes) and save in chroma-db vector store
After step 2) making any query relevant to document-2 does not returns correct document chunks instead it always returns all document chunks from document-2 only and that too with quite low similarity distance eg. 0.55
Step 3) Create embedding for document-1(same small document used in step-1) and save in chroma-db vector store with different doc_id as meta data than used in step-1
After step 3) making any query relevant to document-1 returns correct document chunks with same high similarity distance as found in step-2 eg. 0.8
Step 4) Delete document-1 uploaded in step-3 using corresponding doc_id

After step 4) Again embedding based search does not returns correct document chunks and returns chunks similar to step 2)
But if we make a search based on document context using where_document clause then it returns document chunks from Document-1 uploaded in step-1

I am able to replicate these steps multiple times.
Following are my questions:

Does a large volume document upload creates any impact on similarity search ?

Versions
chromdb==0.4.24
Relevant log output
No response

@flamelite, your search accuracy is expected to fall if you add many documents to Chroma. Fine-tuning the HNSW index is one strategy to improve recall at the expense of memory/storage. Have a look here for some explanation of the parameters you can configure - https://cookbook.chromadb.dev/core/configuration/#hnsw-configuration

@tazarov This is also what i thought but then i wonder why after step-3 it returns correct response wherein we add same document again as used in step-1?

@flamelite this is likely to do with HNSW graph construction. I could help debug in more depth if you provide your data (even as just embs and metadata so we can see which embs belong to which doc), but in the short term, could you try creating the collection with a higher ef_search parameter in the metadata:

create_collection( ... metadata={"hnsw:search_ef": 50 ... } )

This should improve recall in your case.


Custom Global Default Config

Description
I'm trying to gauge if there would be much interest from the maintainers and or the community in enabling more global default configs.
Currently the global defaults are set in the Client and some of those can be configured globally by the users while others can't.

The PROXY related configs can all be set via environment variables
The redirect configs are all on a public static property and can therefore also be globally configured as defaults by the user.
Everything else is unreachable

Example: User-Agent
In our specific case, what we're trying to do is to get a default user agent for each service. The UA header is usually captured in traces and logs and can be invaluable when debugging and tracking issues.
Working on larger / older systems, there are many places where new Guzzle clients are created without being given a specific configs. Some of those can be very non-trivial to identify.
Sometimes, the client is created in a library that is not in the control of the app maintainers. This means many of those older apps all introduce themselves as GuzzleHttp/7 by default.
Since finding and updating all the places where a Guzzle client is being created isn't practical, we're looking for approaches that would allow setting a default within a given app that would differ from GuzzleHttp/7.
What could this look like
The configuration that is currently done via env vars is done so for historical reason and it doesn't seem reasonable to pursue that as the mechanism for more flexibility in global configuration.
So it would probably look like
\GuzzleHttp\Client::setDefaultConfig($config); // with a config array / object
// or
\GuzzleHttp\Client::setDefaultConfig($key, $value);
\GuzzleHttp\Client::setDefaultConfig('headers.User-Agent', env('HTTP_USER_AGENT'));
// or
\GuzzleHttp\Client::$defaults = $config;

Or something else.
What next
If there is interest from the maintainers / community in enabling something like this, I'd be happy to start working on a PR for it.

Thanks for getting in touch. My feeling is that this static global state is a bad practice, and I'm not particularly interested in perusing this in the library core. Frameworks like Laravel have a way of providing syntax that looks like that, but is actually not that, which could be a way to achive what you want with configuring defaults, or making a factory class. I indeed have such a package https://github.com/grahamCampbell/guzzle-factory. ;)

I agree the pointers you give work fine for building things moving forward.
I don't think they work as well when you're trying to update things for a large existing code base with disparate usage of Guzzle clients. Or am I missing something?


Difference with SQL Analytics

Can you please share info how this SQL Insights inside Azure Monitor is different from SQL analytics.
All these have same DMVs underlying , so why we have duplicate products..Can you please share some insights?
[Enter feedback here]

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

ID: 124d1b14-660e-d0df-71e5-4eb3bc943288
Version Independent ID: 3eb3b7fb-644a-bbf1-186d-96bc43d279dc
Content: Monitor your SQL deployments with SQL insights (preview) - Azure Monitor
Content Source: articles/azure-monitor/insights/sql-insights-overview.md
Service: azure-monitor
Sub-service: insights
GitHub Login: @bwren
Microsoft Alias: bwren

@chaisen2006  Thanks for the question! We are investigating and will update you shortly.

At this point it appears that there's no edits or updates needed for the referred Azure documentation so closing this issue for now. If there are further questions regarding this matter or feedback on the referred Azure documentation, please do let us know and we will gladly continue the discussion.


URL by icon is now available in matrix

URL by icon is now available in matrix, the docs should be updated :)
Glad I tried it, after my initial disappointment after reading the docs that say it's not available.

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

ID: 5fbaf2eb-b0c1-6631-727f-f99d0ce28a97
Version Independent ID: 23ede21d-e41a-9121-e23c-b52409198e31
Content: Add hyperlinks (URLs) to a table or matrix - Power BI
Content Source: powerbi-docs/create-reports/power-bi-hyperlinks-in-tables.md
Service: powerbi
Sub-service: pbi-reports-dashboards
GitHub Login: @maggiesMSFT
Microsoft Alias: maggies

Hi, @mothman11 -- thanks for pointing that out!
Question: When I add a URL to a matrix and turn on the URL icon, I still see the URL itself if the URL field is in Rows, but I see the icon if the URL is in Columns. Are you seeing the same?

Hi @mothman11, moving this issue to our work item backlog for better tracking, and closing it here here now. Thanks for providing your feedback!


Tidied up page titles

Capitalization and formatting of content title ([name] | [type]) can be dropped if preferred.
Fixes #50.

I'd prefer if this used lodash.capitalize, I know it's a bit pedantic but there's no reason for us to own that code.
Otherwise LGTM.

TIL lodash.capitalize is a thing!

https://devdocs.io/lodash/ is a wonderful, valuable resource.

Oh, I love lodash, but didn't realise that they had a util function for this. Guess it'll have to become the first place I check from now on.

Yay thanks :smile_cat:


feat: Include both UNSET and None static types for nullable or optional query params

Based on #421 by @forest-benchling. Includes a bit of refactor in e2e tests as I slowly work toward organizing them a bit better.
Closes #421, #380.

@forest-benchling I swapped out the strings in the new e2e test for datetimes instead so we can see serialization.

Refactored a bit of tests in here to make resolving conflicts with main easier.


Error: Couldn't get a file descriptor referring to the console

I was able to install xhtml2pdf just fine (inside a virtualenv).  However, it seems something went wrong, because when I try to run the "hello world" from https://github.com/chrisglass/xhtml2pdf/blob/master/doc/usage.rst I get the following error:

Couldn't get a file descriptor referring to the console

Is there any way to fix this?

+1

Hi @machineghost - are you using the latest prerelease or a stable, and which one?

Eh sorry, I was a bit quick.. didn't see it was a 3 year old issue being bumped. Question goes the same for you, @aidan2129 :)

Haha think i am using the stable but i just did the command xhtml2pdf [template_name] without adding startview -s and it seemed to work the example in the docs just didnt work for me

You mean xhtml2pdf test.html --start-viewer works and xhtml2pdf test.html does not? And what's the exact output?

opposite way xhtml2pdf test.html works for me

would xhtml2pdf test.html --start-viewer not be for a url rather than a template ?

Alright, but what's the output?

Sorry i see the problem i was using a ubuntu virtual box server to generate the template and -s starts the default viewer which the virtual box does not have as i am just ssh into it from windows the output is the same as the title of this issue the problem is that there is no default pdf viewer for me

Alright thanks, closing this, since the issue is confusing and mis-leading then :)


remove fsapi (broken KBFS CI)

seems to be a circular CI dependency (client -> kbfs -> client)?
https://github.com/keybase/kbfs/pull/788
anyway, fsapi is to be replaced with SimpleFS

As long as this doesn't break mobile, so it's up to @MarcoPolo  and @gabriel

it is too messy, leaving it in for now, thanks!


Add the .sln files for sample projects

Hi there. Love your examples & youtube videos for IdS.
Could you add the .sln files for the various sample please?
E.g. When I just want to open and run the authenticating-a-dotnet-console-application-using-identityserver for example.
Optionally, if there's a really simple way to accomplish this with Rider, it would be good to call that out.

Hi @GFoley83, thank you for your positive feedback, I will aim to add a '.sln' file to each of the samples by the end of this week.

Additional request, there's a couple of other things that make the solution tricky to work with when you initially check it out.

The IdS clients that you add in your videos; the code to add those clients isn't checked into the solution so when you run the project you have no clients and no clear instructions to add them. Ideally this could be a seed class like shown here:
https://github.com/jasontaylordev/CleanArchitecture/blob/main/src/WebUI/Program.cs
https://github.com/jasontaylordev/CleanArchitecture/blob/main/src/Infrastructure/Persistence/ApplicationDbContextSeed.cs
The .sln files as mentioned previously.

Unrelated question: there are various aspects of the multiple sample projects that I personally, would love to see together in an all-in-one "final" type solution that I think the community would really benefit from
e.g. following the API / Console / SPA / IdS project structure with:

.NET Core Identity
JWT support
Policy / Roles
Swagger / OpenAPI
Entity Framework Core

Feel free to ignore all of this course. There's a big gap out there of a proper, useable, starter project for the new Duende IdentityServer (most are next to useless hello world type things) and I think if you combine some of your existing sample projects into another repo, you would have exactly that.
Also I really have to commend you on your YouTube videos; your very thorough breakdown & explanation of various parts of the solution e.g. the make up of the docker files and what the various parts mean, should be videos in their own right. You should keep it up!

Thank you very much for your kind words, I'm glad that you've found the videos helpful. Would you mind opening a separate issue for adding IdentityServer seed logic to the samples? That way I can track the changes independently of each other. Also, if you are happy to open another issue for adding a sample that combines everything together, I'm sure I will be able to take a look!

Hi @GFoley83, could I get you to checkout chore/24 and test the solution files (and optionally the run configurations)?

I've tested all the sample solutions and can confirm that they all build fine which is awesome but I'm unable to run them with docker compose currently because of the issue described in #27.

Cheers for testing the solution files, I'll merge that branch into main, and investigate docker compose separately in #27.


RDS Tables and Migrationclean-up

Is your idea related to a problem? Please describe.
I have seen many tables in the migration file that are unused and it is difficult to track the versions in the migration conducted by alembic in the DB migration step.
Describe the solution you'd like
I would like a clean init_database script that matches the versions in the releases
P.S. Don't attach files. Please, prefer add code snippets directly in the message body.

Included in #185


entql Mutation results inconsitent with edge resulsts

Create, Update mutations return an Entity result, not a EntityEdge result
Changing signature to EntityEdge in the mutation.graphql leaves incompatible resolvers.

[ ] The issue is present in the latest release.
[ ] I have searched the issues of this repository and believe that this is not a duplicate.

Current Behavior 😯
    createPost(input: CreatePostInput!): Post
    updatePost(id:ID!, input: UpdatePostInput!): Post

// CreatePost is the resolver for the createPost field.
func (r *mutationResolver) CreatePost(ctx context.Context, input ent.CreatePostInput) (*ent.Post, error) {
	client := ent.FromContext(ctx)
	return client.Post.Create().SetInput(input).Save(ctx)
}

// UpdatePost is the resolver for the updatePost field.
func (r *mutationResolver) UpdatePost(ctx context.Context, id uuid.UUID, input ent.UpdatePostInput) (*ent.Post, error) {
	client := ent.FromContext(ctx)
	return client.Post.UpdateOneID(id).SetInput(input).Save(ctx)
}

change it to
    createPost(input: CreatePostInput!): PostEdge
    updatePost(id:ID!, input: UpdatePostInput!): PostEdge

// CreatePost is the resolver for the createPost field.
func (r *mutationResolver) CreatePost(ctx context.Context, input ent.CreatePostInput) (*ent.PostEdge, error) {
	client := ent.FromContext(ctx)
	return client.Post.Create().SetInput(input).Save(ctx)
}

// UpdatePost is the resolver for the updatePost field.
func (r *mutationResolver) UpdatePost(ctx context.Context, id uuid.UUID, input ent.UpdatePostInput) (*ent.PostEdge, error) {
	client := ent.FromContext(ctx)
	return client.Post.UpdateOneID(id).SetInput(input).Save(ctx)
}

validation failed: packages.Load: resolvers.go:30:9: cannot use client.Post.Create().SetInput(input).Save(ctx) (value of type *"ent".Post) as *"ent".PostEdge value in return statement
resolvers.go:36:9: cannot use client.Post.UpdateOneID(id).SetInput(input).Save(ctx) (value of type *"ent".Post) as *"ent".PostEdge value in return statement

Why?
A graphql API is typically consumed by an apollo client in the JS world.
When you create a post, you expect the return type  of a CreatePost to be the same as the query result type, aka edge nodes.
The result then is you have to fiddle with the JS Object to have just.the.exact.type that is returned from CreatePost
aka
    const {mutate: createPost, onDone} = useMutation(CREATE_POST,
      () => ({
        variables: {
          body: postModel.value.body
        },
        update: (cache, {data: {createPost}}) => {
          let data = cache.readQuery({query: POSTS_PAGINATED})
          let postEdge = {
            node: createPost,
            __typename: 'PostsEdge'
          }
          data = {
            ...data,
            posts: {
              totalCount: data.posts.totalCount+1,
              edges: [
                postEdge,
                ...data.posts.edges
              ],
              pageInfo: data.posts.pageInfo,
              __typename: data.posts.__typename,
            }
          }
          cache.writeQuery({query: POSTS_PAGINATED, data})
        }
      })
    )

this is ludicrous.
You'd have to update the object every time the query model changes.
export const POSTS_PAGINATED = gql`
query postsPaginated($cursor: Cursor) {
  posts(first: 10, after: $cursor, orderBy: {direction: DESC, field:UPDATED_AT}) {
    totalCount
    edges {
      node {
        id
        createdAt
        updateTime
        sub
        body
      }
    }
    pageInfo {
      endCursor
      hasNextPage
    }
  }
}`

What if I'd like posts with comments and profile names of posts and comments, also number of followers etc?
I'd have to manually update the whole object that is placed on top.
You might say it's not your problem. But it is your problem.
That is the single thing that's keeping me from using graphql or rather ent/gqlgen.
On one hand you have edge nodes returned from queries on the other hand you don't support edge node results.
Expected Behavior 🤔
EntityEdge or edge node mutation results returned
Steps to Reproduce 🕹
Steps:

Your Environment 🌎

irrelevant

Hello @idc77, I also faced the same problem as you a year ago. So, I open a PR to add ToEdge on every entity so we can convert a node to an edge. But the problem is, the edge in EntGQL require an order to generate correct ID. So, you have to pass the order of the current connection via the mutation to generate the edge. Let's take the createPost example.
// CreatePost is the resolver for the createPost field.
func (r *mutationResolver) CreatePost(ctx context.Context, input ent.CreatePostInput) (*ent.PostEdge, error) {
	client := ent.FromContext(ctx)
	p, err := client.Post.Create().SetInput(input).Save(ctx)
+	if err != nil {
+		return nil, err
+	}
+	return p.ToEdge(ent.DefaultPostOrder), nil // Use default sorting order or via the mutation argument.
}

You can see more detail here. https://github.com/ent/ent/issues/1034
I will close this issue, but if something is unclear. Just let me known. I happy to help

Thanks for the quick support and your kindness, @giautm ❤


[#1946] Fix: warning msg for exceeding remaining seats

Calculation in some specific cases would result in a warning message
even though the remaining number of seats was >=1.

Merging since I want to release this today.


About the PR

Angered combat NPC now can unbuckle, unpull and escape from containers.
Why / Balance

No robust.
Technical details

Idk.
Media

https://github.com/space-wizards/space-station-14/assets/60344369/148e508d-eaf3-4e20-b8ff-3c65780af421

[X] I have added screenshots/videos to this PR showcasing its changes ingame, or this PR does not require an ingame showcase

Breaking changes

Changelog

:cl: VigersRay

fix: Angered NPC now can unbuckle, unpull and escape from containers.

Finally! No salvage robust

fix: Anrgy NPC now can unbucke, unpull and escape from containers.

Angered mobs*
unbuckle*

As a side note to my previous comment, there should probably be a “disable hostility” button on the admin/debug menu in the future

This would utterly nuke at least two triggers for xenoarch artifacts at the moment (blood and death) as scientists already lack tools to actually safely murder the monkeys/kobolds they use for these triggers, so "tie it to a chair" is the best option.

Normalize giving science laser rifles and cannons, clearly.

I'd assume science can still cuff monkeys without too much trouble to take care of this trigger. Also, I don't think we should continue supporting clearly wrong behavior because the meta of the game evolved around it.

This would utterly nuke at least two triggers for xenoarch artifacts at the moment (blood and death) as scientists already lack tools to actually safely murder the monkeys/kobolds they use for these triggers, so "tie it to a chair" is the best option.

Blood you just syringe yourself, put in a cup, and splash the artifact.

The problem isn't getting robust, the problem is that the monkey is still going to damage you whilst you beat it senseless, and from memory the best weapon to do so with is either a toolbox or the sometimes-mapped circular saw. You end up dragging yourself off to med when you should be doing xenoarch.

The problem isn't getting robust, the problem is that the monkey is still going to damage you whilst you beat it senseless, and from memory the best weapon to do so with is either a toolbox or the sometimes-mapped circular saw. You end up dragging yourself off to med when you should be doing xenoarch.

Use nitrous oxide from robotics vendors

All those options are currently worse than the meta of office chair + violence.

These changes are good.

I think there being a procedure to killing the monkey for testing purposes (say, handcuffing it, sedating it with chemicals) would be an interesting change, though might merit making the trigger rarer.

Guys just space the monkeys, let the artifact chamber doors do that for you

I usually take care of monkeys by spacing the testing chamber.
I've always found it kind of weird how sitting a monkey in an office chair somehow restrains it. It makes sense that you'd have to do a little more to keep them still. Maybe you could use some cables on them in the chair to tie them up? (But at that point why not just craft some makeshift cuffs?) Perhaps mappers could provide some convenient LV cable coils near artifact testing rooms?

I usually take care of monkeys by spacing the testing chamber, so whatever.
I've always found it kind of weird how sitting a monkey in an office chair somehow restrains it. It makes sense that you'd have to do a little more to keep them still. Maybe you could use some cables on them in the chair to tie them up? (But at that point why not just craft some makeshift cuffs?) Perhaps mappers could provide some convenient LV cable coils near artifact testing rooms?

In the NPC logic, you need to add a check of items in hand and their damage in order to prioritize between throwing away an item and attacking with your hands or picking up a more formidable weapon.

This would utterly nuke at least two triggers for xenoarch artifacts at the moment (blood and death) as scientists already lack tools to actually safely murder the monkeys/kobolds they use for these triggers, so "tie it to a chair" is the best option.

Im fairly sure monkeys can be handcuffed.

passive/neutral NPCS also do this which is kind of really annoying and terrible

Please for the love of god revert this change or improve it

Please for the love of god revert this change or improve it, more was lost than gained

Can you make an in-depth bug report of things lost at #27460 because this comment helps literally no one


how to responds two different method in one CallData

in official async server demo, I make some change. I register two functions in CREATE state. in PROCESS state, I want't respond to SayHello and SayAnotherHello separately. Has there any method to figure out which function is called now? Or are there any other method to respond more than one functions in on CallData class.
service Greeter {
  // Sends a greeting
  rpc SayHello (HelloRequest) returns (HelloReply) {}
  rpc SayAnotherHello (HelloRequest) returns (HelloReply) {}
}

  class CallData {
   public:
    CallData(Greeter::AsyncService* service, ServerCompletionQueue* cq)
        : service_(service), cq_(cq), responder_(&ctx_), status_(CREATE) {
      Proceed();
    }

    void Proceed() {
      if (status_ == CREATE) {
        status_ = PROCESS;
        service_->RequestSayHello(&ctx_, &request_, &responder_, cq_, cq_,
                                  this);
        //register another function
        service_->RequestSayAnotherHello(&ctx_, &request_, &responder_, cq_, cq_,
                                  this);
      } else if (status_ == PROCESS) {
        // I want to  figure out which function called here. but how?

        new CallData(service_, cq_);
        std::string prefix("Hello ");
        reply_.set_message(prefix + request_.name());
        status_ = FINISH;
        responder_.Finish(reply_, Status::OK, this);
      } else {
        GPR_ASSERT(status_ == FINISH);
        delete this;
      }
    }

This is mixing two separate RPC methods in one state object. It would be best to keep them completely separate.


[EAGLE-76] hide stream operation if role is not admin

User don't have permission to add/update/delete metadata stream.

LGTM.

Please rename title as EAGLE-76 hide stream operation if role is not admin

Commit with following message
 EAGLE-76 hide stream operation if role is not admin

 https://issues.apache.org/jira/browse/EAGLE-76

 Author: @zombieJ
 Reviewer: @haoch

 Closes #31.

Please resolve the JIRA ticket as well: https://issues.apache.org/jira/browse/EAGLE-76


Ticket embed args aren't right

Describe the bug
The ticket embed command only have to args, the title and the description
Cog Version*
Latest at 19/07/2024
To Reproduce
Just try the default command
Expected behavior
Is expected, acording to the docs to be able to change to change the color and channel of the embed message
Screenshots

Thats because you have to include the actual text in those placeholders.
For example: !tickets embed gold #open-a-ticket "Click the button below to open a ticket"
For future questions about cog usage, please ask in the 3rd party support Discord for Red Here


failOnError option not working

Describe the bug
Build doesn't fail if non-compliant files are found.
To Reproduce
<plugin>
  <groupId>com.spotify.fmt</groupId>
  <artifactId>fmt-maven-plugin</artifactId>
  <version>2.19</version>
  <configuration>
    <failOnError>true</failOnError>
  </configuration>
  <executions>
    <execution>
      <goals>
        <goal>format</goal>
      </goals>
    </execution>
  </executions>
</plugin>

mvn fmt:format

Expected behavior
Return a non-zero error code.

My bad, only works on check goal.

My bad, only works in check goal.


Update to CE v3.5.0-RC2, FS2 v3.7.0-RC1

Further propagates the CE v3.5.0 release candidate. This release brings major changes which required some fixes in FS2 (also bumped in this PR) and may require changes in other downstream libraries (e.g. see https://github.com/http4s/blaze/issues/772, https://github.com/http4s/http4s-dom/pull/253).
To help canary these RCs, we should consider releasing an http4s RC, or at least a snapshot. Thoughts? Not sure if we want to merge this PR into mainline yet, in which case we would have to release off of a branch.

I don't mind an RC.  An RC dependency on a series branch would be a nuisance to regular releases or security patches until these dependencies go final.  But we can always revert it or branch around it if another release needs to surpass it, so I'm fine keeping it simple.


Soundbar constantly becomes unresponsive to api calls

My SB36512-F6 soundbar loses wifi constantly.  At best it will stay connected for a few days, but usually it drops the connection in less than a day.  It still gets audio from HDMI-ARC and/or Bluetooth, but it becomes unavailable in Home Assistant and the SmartCast android app until I unplug the power and plug it back in.  I tried using ethernet instead (with a wifi bridge) but I had the same issue.  ECO mode is off on the soundbar.  It is connected to a TCL Roku TV.  On the Roku TV I have enabled 'Fast TV Start' and disabled the power-saving shut off (after 4 hours idle).  The soundbar has a reserved IP address on my router.  I've had this issue for a while and I tried a factory reset a while back.  I searched online but haven't found anyone with the same issue.  The only thing I can think of now is to put it on a smart plug and make an automation to reboot it whenever it becomes unavailable in HA.  I'd rather fix the issue though.  I was wondering if pyvizio or the HA vizio integration was somehow contributing to this problem.
Any suggestions would be appreciated.

To be honest, I have the same problem. I don't think the device is going off the network, I think the API gets into a weird state and becomes unresponsive. I suspect it's the Home Assistant integration and have tried many things, like reducing the number of calls made on each update and increasing the time between updates but haven't been able to resolve it.
Your best bet is to remove the integration from HA, do a power reset, and then try to connect to it via the SmartCast app every once in a while. Assuming it continues to work, that means it is the HA integration.
By the way, you are the first person to report this aside from me. From conversations with other people, this does not happen to all sound bars which makes it even harder to troubleshoot.

You're right that it hasn't actually been disconnecting from wifi.  I checked in my router when it became unavailable and it was still an "attached device" and I was able to ping the IP address.  Well, I should have just done what you said, but I wanted to try a factory reset (since the last reset was a while back) first.  Now it is much worse, but ironically pyvizio and the vizio HA integration are working.
After the factory reset I could not pair the soundbar to my smartcast app.  It just would not show up.  Turns out that while most settings were reset, the network settings are still there.  It still connects to my wifi, but doesn't appear as a device in smartcast.  I tried using the remote to trigger a "WifiLink" (I think that's what it calls it on the remote) but I still don't see a vizio hotspot and the smartcast app can't find it.  Bluetooth pairing does not work either for any bluetooth device.  The vizio HA integration finds it, but with a generic "Vizio SmartCast" name instead of what I had named it before the reset.  I tried updating firmware using usb (nothing happened, probably because it doesn't need updating), and tried multiple resets using the remote and the button combination. I tried turning off my router so it wouldn't have a wifi to connect to.  No progress.  Chromecast function was working at one point, but not this morning.
Maybe a future firmware update will help.  I should get in touch with Vizio support I suppose.

I have two soundbars and one of my soundbars is in the exact same state you are describing (different model though). Their software is pretty terrible as far as I can tell, which is unfortunate because the price point is great for what you get.

My vizio soundbar is connect to power through a mechanical timer, which shuts it down everyday around 3:30am when I'm most likely  not using it. Since I've done that (about a year ago) I never had that 'limbo' state again. Although I have to say the soundbar still shutdowns for no reason sometimes while I'm watching something, but I think that is another diferrent issue maybe related to the Hdmi-arc.

I am also experiencing this with some Crave 360 speakers.  My setup is 2 of the SB3651-E6B 5.1 sound bars, and 3 Crave 360 (SP50-D5) spread throughout the house.
I have them integrated into Google Home and working with cast groups, as well as into home assistant.
Every few days, one of the Crave 360 speakers will go unavailable in home assistant (and in the official SmartCast app).
That led me to start fiddling around with pyvizio.
When I run curl -k -H "Content-Type: application/json" -X GET https://device_name:9000/state/device/power_mode I get a 503 error from the API.
However, I'm still able to cast to it while it's doing this, I just have no control from the app or HA.
The only thing that fixes it is doing a short jab with a paperclip to the reset button (but don't hold or it will factory reset).  Just tapping it seems to reboot it and then all is well.
Since these operate on a battery I can't just throw them on a smart switch to power cycle them, so this is super annoying.
These were a steal at $36 renewed on Amazon.  They sound great and really helped to complete the "whole home audio" setup.
But their software is clearly a buggy mess.  It took probably 20 tries to get them to pair in the first place since their app is so glitchy, and now it seems that applies to the internal API as well.  I also have a really hard time casting to a single SmartCast speaker sometimes.  It just won't do it, but then it will work if it's in a cast group reliably every time.


Get live feed from RPI camera

Have the camera take a picture every couple of seconds and delete it once a process has been done on it.


[feature][byob] Rename setup-token Action

We may call it delagator/generic-init or delagator/generic-setup

@ianlewis wdut? Make the change or close the issue?

I don't really have an opinion. I guess technically having generic in the name is better since it matches with the corresponding delegator workflow.


(fix #370)Added Edmnton Wordpress meetup to the list of meetups

What issue is this referencing?
This PR is referencing https://github.com/devedmonton/DES-Website/issues/370, which adds WordPress to the list of meetups.
Do these code changes work locally and have you tested that they fix the issue yourself?
yes!
Does the following command run without warnings or errors?
yes! npm run pr-checks
Have you taken a look at our contributing guidelines?
yes!
My node version matches the one suggested when running nvm use?
yes!

@allcontributors add @shrutiijain26 for code

Thank you so much @shrutiijain26


pubsub over serial - Raspberry Pi / Arduino

Hello,
I would to subscribe from a Raspberry Pi to events that occured on an Arduino board (such a rising or falling edge). I would also like Arduino acquire data from analog digital converter and publish values on "topic".
I noticed that pubsubclient only support ethernet / wireless based Arduino board but maybe you have some tips to do this pub sub over USB serial.
Kind regards

The library supports anything that provides an implementation of the arduino platform's Client.h interface. If you were to create a Serial implementation that adheres to this interface, then you could make use of the client. On the RPI end of the serial link you'd also need something that forwards the USB comms to a TCP connection to an MQTT Broker.

thanks for your quick reply

That is really helpful. I'm working on a similar project. What are possible ways to forward the USB comms to a TCP connection?


Why this was broken????

0.5.3 ----> listening is broken
0.5.4,0.5.5 -----> java null pointer exception...
plz fix it
Only 0.5.2 works fine, what did you do??

I just published a new update, can you check this one?


Windows: symbolic links not created correctly with relative paths

SwiftPM creates a symbolic link in the .build directory to a sibling directory for backwards compatibility reasons. That link is currently in 6.0 not created correctly on Windows.
For Windows we check whether the destPath is a directory to add the SYMBOLIC_LINK_FLAG_DIRECTORY flag when creating the link. However, we don't check whether it's a relative path and just call fileExists to fetch the isDirectory boolean. That assumes the path is relative to the current directory not the symbolic link path.
I notice in corelibs-foundation, the path is resolved correctly before checking if it's a directory:
                let resolvedDest =
                  destPath.isAbsolutePath ? destPath
                                          : joinPath(prefix: path.deletingLastPathComponent,
                                                     suffix: destPath)

Original SwiftPM issue here: https://github.com/swiftlang/swift-package-manager/issues/7965

Awesome, it worked. Thanks Jeremy!


Adding a new JS project of GitHub Finder

Describe the project you want to add with tech stack

I would like to add a project of Github Finder to this repo. I will be using HTML, CSS, and JavaScript for building this.
Expected behavior

In this project, you can search for Github users and visit their repositories easily and simply.
Screenshots (optional)

I would like to add this project to this repo. I have already made this kind of project by using ReactJS and to fetch data I have used axios.
But according to this, I think you are expecting to make this project in Javascript, So I will use simple HTML, CSS and JavaScript.
And also it is fully responsive.
Here is the live link of this similar project which I have made.
https://github-user-finder-theanushka-v18.vercel.app/

I would like to add this project under GSSoC' 23. Please assign this task to me.
I have already commented the description.


No documentation about Data Length Extension

There's currently no documentation about Data Length Extension.
It is silly to list a feature on the front page of documentation that isn't actually documented in any normal way.

Yes, it is indeed not easy to find. Take a look at nimble/controller/syscfg.yml:
    BLE_LL_CFG_FEAT_DATA_LEN_EXT:
        description: >
            This option enables/disables the data length update procedure in
            the controller. If enabled, the controller is allowed to change the
            size of tx/rx pdu's used in a connection. This option has only
            minor impact on code size and non on RAM.
        value: '1'

And from my own experience, looking at

BLE_LL_MAX_PKT_SIZE
BLE_LL_CONN_INIT_MAX_TX_BYTES,
BLE_LL_SUPP_MAX_RX_BYTES
BLE_LL_SUPP_MAX_TX_BYTES

will add to the picture and hopefully achieve the desired behavior?!

Will investigate and test out, thanks @haukepetersen.

how was your investigation? any chances you could send PR with documentation update?


Deal with peole in non-existent syndicates

All  transcribers should be in a syndicate that reflects their transcribing. A rake task to identify those in:
Any Questions? Ask Us
Any County and Year
Technical
Wales Syndicate (?)
Yorkshire Syndicate (?)
And all old syndicates that are no longer in the dropdown list on Registration page has identified who is not: @DeniseColbert to remediate.

@DeniseColbert I bet that I'm missing from all syndicates. I've transcribed a number of Pieces in the past, most recently for a number of SCT counties that were coordinated by Rena Mitchell, but am no longer active as a transcriber.


Android: value is null, expected an object at drawText

Description
While using Text component getting this error. I have check x, y and text, none of them are null.
This is happening for android only. For IOS there is no such issue.
Version
0.1.123
Steps to reproduce
render list of Text component
Snack, code example, screenshot, or link to a repository

can you share a code snippet to reproduce the issue?

create array of Text :
const renderXText = () => {
for (let i = minHour + xRange; i <= maxHour; i = i + xRange) {
      const extraPixelToAdd = 4 * xRange * pixelPerReading; // 4 * 15 = 1hr

      const x = lastReadingPixel + extraPixelToAdd;

      xCoordinatePlot.push(
        <Text
          key={i}
          x={x} //check width of this and /2
          y={yCord}
          text={formatXLabel(i)}
          font={font}
          color={theme.palette.neutral.lightGrey}
          size={fontSize}
        />,
      );

      lastReadingPixel = x;
    }

    return xCoordinatePlot;
}

This is the return from the main function:
  return (
    <Group>
      {xPoints && xPoints.length > 0 && <Group>{renderXText()}</Group>}
    </Group>
  );

data set which u can use (array of any number):
[10, 11, 12, 13, 14]
let me know if u need anything else.

@wcandillon the reason iam using "0.1.123" because path.toCmds() in new version doesnt return array of 8 rather it returns 6. Basically needed for getYForX method.
export const selectCurve = (cmds, x) => {
const curve = cmds.find(cmd => {
if (cmd === null) {
return false;
}
const isCubicCurve = cmd[0] === PathVerb.Cubic;

const to = vec(cmd[1], cmd[2]);
const from = vec(cmd[7], cmd[8]);

const isXbetweenCurve = x <= from.x && x >= to.x;

return isCubicCurve && isXbetweenCurve;

});
if (curve) {
const to = vec(curve[1], curve[2]);
const from = vec(curve[7], curve[8]);
const midPoint1 = vec(curve[3], curve[4]);
const midPoint2 = vec(curve[5], curve[6]);

return {
  from: from,
  midPoint1: midPoint1,
  midPoint2: midPoint2,
  to: to,
};

}
return undefined;
};
so iam stuck at "0.1.123" if this issue is solved in latest version/or get fixed in upcoming version. Can you provide any alternative for this too.
Thanks in advance!

@wcandillon when i render with hardcoded values:
  <Text
    key={1}
    x={10}
    y={10}
    text={'Target Zone'}
    font={font}
    color={theme.palette.neutral.lightGrey}
    size={fontSize}
  />

  iam getting this error.

Based on the error it looks like the error is the font object being null (not finished to be loaded).
You also use a combination of font and size which is incorrect since size `is contained in font.
Unfortunately, we don't offer documentation yet for older version of Skia so these small details are hard to spot.
We strongly encourage to upgrade to the latest version.
About the toCmds() format being update, you can work around it, see this function for instance: https://github.com/Shopify/react-native-skia/blob/dbf8846da8b23646434d1b47e83e01a98d201892/example/src/Examples/Wallet/Math.ts#L149
I am closing this issue since there doesn't seem to be a bug here. If you think there is, please feel free to reopen.

Happy Hacking!


[WIP] Add tracing-error support

For SpanTrace to work, the tracing subscriber
needs an additional ErrorLayer.
See #2

Thanks for the pull request!
I haven't looked at it in close detail, but here are a few first thoughts:

test-env-log is meant to be a wrapper around existing functionality already used in a crate. From what I can tell with this pull request we violate that, because now we pull in tracing-error, which may or may not be used by the functionality under test. So I would think this functionality needs a new feature.
I find the commit message pretty nondescript. In my opinion it should state what it does and why it does it, at the least.
I'd think that this addition would require a CHANGELOG entry and also adjustment of the README and doc comments.
Can you please also make sure to mirror existing style?

Pull request does not have seen any forward progress. Closing.


Process contracts originated from migrations

Improvements:

Support Granadanet migration originations

Bugfixes:

Fix type imports not being wrapped with DipDupError
Fix missing template variable check

 FormattedLogger not working as expected

 Block hash mismatch


NUTCH-3013 Employ commons-lang3's StopWatch to simplify timing logic

Addresses https://issues.apache.org/jira/browse/NUTCH-3013

If this gets merged I'll squash all of those commits.
Note, this does change the logging slightly. The whole LOG.info("CrawlDb update: starting at {}", sdf.format(start)); is redundant because we already capture the datetime stamp in the log output anyway.
I simplified this to LOG.info("CrawlDb update: starting");.

Passes local unit tests.


LCFS - Export transfer data from TFRS to LCFS

Describe the task
Using Apache Nifi, build a reusable script to export Transfer data from TFRS (Transportation Fuel Reporting System) to LCFS (Low Carbon Fuel System). Transfers in the following statuses should be exported: Approved, Declined, Deleted, Draft, Recorded, Refused, Rescinded, Sent, Submitted.
Purpose
The task is required to automate the export of transfer data from TFRS to LCFS while ensuring the correct data mapping and transformation. This will streamline the transfer process and reduce manual efforts.
Acceptance Criteria

[ ] Export Transfer data with the following statuses: Approved, Declined, Deleted, Draft, Recorded, Refused, Rescinded, Sent, Submitted.
[ ] Ensure the following data field mappings between TFRS and LCFS:

TFRS-Organization (initiating) → LCFS-Organization (initiating)
TFRS-Quantity → LCFS-Quantity
TFRS-Not for consideration flag → Drop, no corresponding LCFS field
TFRS-Organization (receiving) → LCFS-Organization (receiving)
TFRS-Value (or price) → LCFS-Value (or price)
TFRS-Agreement date → LCFS-Agreement date
TFRS-Comment (Initiator) → LCFS-Comment (Initiator)
TFRS-Comment (Receiver) → LCFS-Comment (Receiver)
TFRS-Initiator sign-off → LCFS-Initiator sign-off
TFRS-Receiver sign-off → LCFS-Receiver sign-off
TFRS-Gov Comment (Public) → LCFS-Gov Comment to organization
TFRS-Gov Comment (Internal) → LCFS-Gov internal comments
TFRS-Category → LCFS-Category
TFRS-Status → LCFS-Status
TFRS-Recorded date → LCFS-Recorded date
TFRS-Effective date → LCFS-Effective date

[ ] Verify that the exported data conforms to LCFS specifications.
[ ] Ensure error handling for failed transfers with appropriate logging.

Additional context

This task will require a reusable flow in Apache Nifi to export data from TFRS to LCFS.
Error handling and logging mechanisms should be implemented to handle failures and retries.
The solution should maintain the integrity of the GUID and/or unique ID

wait for prod environment


how can I fix the headers?

I'm submitting a ...  (check one with "x")
[ ] bug report => search github for a similar issue or PR before submitting
[ ] feature request
[x ] support request => Please do not submit support request here

Current behavior

May I ask which attribute I can use to fix the table header? I checked the demo code as well as the documentation, and  I can not find out any configuration that can make the table header fixed. Now, when I scroll vertically, the whole table scroll up.
Thanks
Expected behavior

Reproduction of the problem

What is the motivation / use case for changing the behavior?

Please tell us about your environment:

Table version: 0.7.x

Angular version: 2.0.x

Browser: [all | Chrome XX | Firefox XX | IE XX | Safari XX | Mobile Chrome XX | Android X.X Web Browser | iOS XX Safari | iOS XX UIWebView | iOS XX WKWebView ]

Language: [all | TypeScript X.X | ES6/7 | ES5]

are you scrolling the page or the grid?
If you look at the 10k example:

if you scroll the grid the header stays at the top
if you scroll the page the whole grid scrolls as you say.

So you might make sure you are using vertical scroll [true].

This is my code:
@Component( {     selector: "jnpr-x-datatable",     template:

<ngx-datatable
class="material"
[rows]="tableRows"
[columns]="tableColumns"
[columnMode]="'force'"
[headerHeight]="50"
[footerHeight]="0"
[rowHeight]="50"
[scrollbarV]="true"
[scrollbarH]="true"
[limit]=10
>
      </ngx-datatable>
    </div>
  `

})
export class NgxDataTableComponent implements OnChanges, OnInit {
@Input() config: any;
@Input() dataList: any;
@Input() appId: string;
@Input() accountList;

tableColumns:any;
tableRows:any;

ngOnInit(){
    this.tableColumns = [
        { prop: 'name' }
    ];
    for(var i=0; i<20;i++)
        this.tableColumns.push( {name: 'name'+i} );

    this.tableRows = [];
    for(var j=0; j<40; j++){
        var row={ name: 'Austin'};
        for(var i=0; i<20;i++){
            row['name'+i] = 'value'+i;
        }
        this.tableRows.push(row);
    }
}

}`
anything wrong? I found the whole page is scrolling instead of the grid only, and head is moving with grid together.

the first thing that comes to my mind is did you give the grid a height?
I personally used flex layouts to make it fit my area, but my guess is without a given height it just overflows which causes the screen to scroll.

it is not the height of the grid, if I gives it the height, the table
become shorter, but not scrollable.
There may be some tricks to make the header fixed here, but where
require('@swimlane/ngx-datatable/release/index.css');
require('@swimlane/ngx-datatable/release/themes/material.css');
require('@swimlane/ngx-datatable/release/assets/icons.css');
@Component( {
selector: "jnpr-x-datatable",

template: `

    <div>

      <ngx-datatable

          class="material"

          [rows]="tableRows"

          [columns]="tableColumns"

          [columnMode]="'force'"

          [headerHeight]="50"

          [footerHeight]="0"

          [rowHeight]="50"

          [scrollbarV]="true"

          [scrollbarH]="true"

          [limit]=10

            >

      </ngx-datatable>

    </div>

  `

})
On Thu, Mar 23, 2017 at 4:37 PM, Tim Burnell<EMAIL_ADDRESS>wrote:

the first thing that comes to my mind is did you give the grid a height?
I personally used flex layouts to make it fit my area, but my guess is
without a given height it just overflows which causes the screen to scroll.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/swimlane/ngx-datatable/issues/624#issuecomment-288892444,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAN4c4QGp3RxpHHnhWYWykH7Lj9d8Qxdks5rowIsgaJpZM4MmUK8
.

I did have to upgrade my CLI to the latest to do the imports ... other than that i dont know.

See this demo: https://github.com/swimlane/ngx-datatable/blob/master/demo/basic/fullscreen.ts
when vertical scrolling is turned on, the heads will stick. If you want your table to make heads always top, you need to position your table like i did there.

I need to set the headers but without activating the virtual scroll, since I'm implementing a drag and drop, is this possible?

Just as an FYI, I had the same need for having a fixed header without the virtual paging, but still using server side pagination.  I modified the code for the component to make this work by implementing two new component attributes: [fixedHeader]="true" (defaults to false) and [fixedHeaderBodyHeight]="enter your numerical value here" which fixes the height of the body container for the rows.  It works very nicely.  I am ready to donate back the code changes, but am not certain what the procedure is for doing that.  Meanwhile, if you want the code changes, please reach out to me and let me know.

Hey @swidom-exparency, I would like to know how did you implemented it! Can you share your code?

Hi Rafael,
Sure.  Attached is the entire code base with my changes.  I apologize - it's been so long since I wrote those code changes and I didn't take any notes about what I did.  I do remember that the code changes were quite minimal once you figure out where they need to be placed.  Attached is a zip file, and I guess you can do a diff to figure out what changes I made.
Here is an example of how I am using it...
<ngx-datatable class="material scroll-vertical"        [fixedHeader]="true"        [fixedHeaderBodyHeight]="275"                           [rows]="rows"                           [columnMode]="'force'"                           [headerHeight]="50"                           [footerHeight]="50"                           [rowHeight]="50"                           [externalPaging]="true"                           [externalSorting]="true"                           [count]="count"                           [offset]="offset"                           [limit]="limit"                           (sort)="onSort($event)"                           (page)='onPage($event)'                           [selected]="selected"              [selectionType]="'single'"              (select)='onSelect($event)'...
You can see that I have added to new @Input members: fixedHeader and fixedHeaderBodyHeight.  The latter attribute determines the height of the grid you want to use, and you will see that the header will remain fixed above that grid, which is what we wanted in the first place.
Good luck!  Please let me know if this helps you.  My preference would be for swimlane to roll these changes into their baseline so that it will keep up with their new versions.  Maybe, with enough folks asking for it, they will comply.
-Regards, Steve
  From: Rafael Trostli Costella<EMAIL_ADDRESS>
To: swimlane/ngx-datatable<EMAIL_ADDRESS>Cc: Steve Widom<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Sent: Saturday, April 28, 2018 10:11 AM
Subject: Re: [swimlane/ngx-datatable] how can I fix the headers? (#624)
Hey @swidom-exparency, I would like to know how did you implemented it! Can you share your code?—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

Hi Rafael,
I should add that I have the following in my app.module.ts:
import {NgxDatatableModule} from './ngx-datatable/datatable.module';  (change this if you put the code in a different folder)
with NgxDatatableModule in my imports section of that file.  Also, naturally I am no longer declaring ngx-datatable as a dependency in my package.json file.  Again, I hope this helps.
-Steve
  From: Rafael Trostli Costella<EMAIL_ADDRESS>
To: swimlane/ngx-datatable<EMAIL_ADDRESS>Cc: Steve Widom<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Sent: Saturday, April 28, 2018 10:11 AM
Subject: Re: [swimlane/ngx-datatable] how can I fix the headers? (#624)
Hey @swidom-exparency, I would like to know how did you implemented it! Can you share your code?—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

Helps a lot!
Thanks mate!
Cheers

On 28 Apr 2018, at 14:15, Steve Widom<EMAIL_ADDRESS>wrote:
Hi Rafael,
I should add that I have the following in my app.module.ts:
import {NgxDatatableModule} from './ngx-datatable/datatable.module';  (change this if you put the code in a different folder)
with NgxDatatableModule in my imports section of that file.  Also, naturally I am no longer declaring ngx-datatable as a dependency in my package.json file.  Again, I hope this helps.
-Steve
From: Rafael Trostli Costella<EMAIL_ADDRESS>To: swimlane/ngx-datatable<EMAIL_ADDRESS>Cc: Steve Widom<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Sent: Saturday, April 28, 2018 10:11 AM
Subject: Re: [swimlane/ngx-datatable] how can I fix the headers? (#624)
Hey @swidom-exparency, I would like to know how did you implemented it! Can you share your code?—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub https://github.com/swimlane/ngx-datatable/issues/624#issuecomment-385191673, or mute the thread https://github.com/notifications/unsubscribe-auth/AGqzY4N5mhNXD_hiGbynRuXnqoRKFSc_ks5ttKPDgaJpZM4MmUK8.

Hello again!
My current config is:
`
<ngx-datatable class="material"
[loadingIndicator]="isLoading"
[headerHeight]="50"
[footerHeight]="50"
[rowHeight]="50"
[columnMode]="'force'"
      [scrollbarH]="true"
      [scrollbarV]="true"
      [rows]="rows"
      [columns]="filteredColumns"

      [externalSorting]="true"
      (sort)="onSort($event)"

      [selected]="selected"
      [selectionType]="'multi'"
      (select)="onSelect($event)"
      (activate)="onActivate($event)"

      (reorder)='saveOrder($event)'

      [externalPaging]="true"
      [count]="page.totalElements"
      [offset]="page.pageNumber"
      [limit]="page.size"
      (page)='setPage($event)'>

`
These are the pagination params:
page.totalElements == 6517
page.pageNumber == 0
page.size == 10
Now my wrapper ngx-datatable have a height that is 6517 * 50, apart from the fact that ScrollbarV is set to true. What is hapening?

Hi @swidom-exparency, I would like to know how did you implemented the fix header functionality. Could you please share your code? Thanks

Just as an FYI, I had the same need for having a fixed header without the virtual paging, but still using server side pagination. I modified the code for the component to make this work by implementing two new component attributes: [fixedHeader]="true" (defaults to false) and [fixedHeaderBodyHeight]="enter your numerical value here" which fixes the height of the body container for the rows. It works very nicely. I am ready to donate back the code changes, but am not certain what the procedure is for doing that. Meanwhile, if you want the code changes, please reach out to me and let me know.

Hello marvasten,
I apologize - I am just seeing this now.  I don't know why I never got an email letting me know about your inquiry.  If you still want the code, please let me know.  I literally just sent a zip file to another person asking for the same.   You can email me at<EMAIL_ADDRESS>-Regards,
Steve

Hi Steve,
It's two years and a couple of releases later and this is still an issue. I too have a situation where I cannot turn scrollbarV on, but still want my headers to be fixed. Would you be able to put your fix in the latest version and post a link to it so that everyone can benefit from it?  Not sure why it wasn't yet baked into the code base.
Thanks so much,
Shmuel Tannenbaum

To Anyone Interested,
I have rolled my original changes that date back to Angular 2 into (what appears to be) the latest release of Swimlane's ngx-datatable module, which at the time of writing this is 16.0.2.  My changes are actually quite limited (three files only, with only a few lines per file.  Again, if you are interested I will send you a zip file (7z) with the entire code base.
You will need to remove your dependency from the Swimlane component in your package.json, and copy the code base into your src/app Angular project, and add the import into your app.module.ts.
Please note a couple of things.  The first is that I have done very minimal testing on this - call it a smoke test and no more.  Secondly, I would really appreciate it if someone, somehow, would get Swimlane to roll these changes into their baseline and/or provide an alternative that meets our needs.   I am still in the process of putting together my own website which is why I cannot post a link to the zip file.
-Steve

Per my post above, I keep forgetting to give out my email address.  It is<EMAIL_ADDRESS>-Steve

That's awesome Steve, thank you so much for that!!!
I would like to make it into an NPM package, but at the moment i'm under pressure to meet a deadline. I'll touch base when that clears up, god willing...

Hi Rafael, Sure.  Attached is the entire code base with my changes.  I apologize - it's been so long since I wrote those code changes and I didn't take any notes about what I did.  I do remember that the code changes were quite minimal once you figure out where they need to be placed.  Attached is a zip file, and I guess you can do a diff to figure out what changes I made. Here is an example of how I am using it... <ngx-datatable class="material scroll-vertical"       [fixedHeader]="true"       [fixedHeaderBodyHeight]="275"                           [rows]="rows"                           [columnMode]="'force'"                           [headerHeight]="50"                           [footerHeight]="50"                           [rowHeight]="50"                           [externalPaging]="true"                           [externalSorting]="true"                           [count]="count"                           [offset]="offset"                           [limit]="limit"                           (sort)="onSort($event)"                           (page)='onPage($event)'                           [selected]="selected"             [selectionType]="'single'"             (select)='onSelect($event)'... You can see that I have added to new @input members: fixedHeader and fixedHeaderBodyHeight.  The latter attribute determines the height of the grid you want to use, and you will see that the header will remain fixed above that grid, which is what we wanted in the first place. Good luck!  Please let me know if this helps you.  My preference would be for swimlane to roll these changes into their baseline so that it will keep up with their new versions.  Maybe, with enough folks asking for it, they will comply. -Regards, Steve From: Rafael Trostli Costella<EMAIL_ADDRESS>To: swimlane/ngx-datatable<EMAIL_ADDRESS>Cc: Steve Widom<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Sent: Saturday, April 28, 2018 10:11 AM Subject: Re: [swimlane/ngx-datatable] how can I fix the headers? (#624) Hey @swidom-exparency, I would like to know how did you implemented it! Can you share your code?— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub, or mute the thread.

It's realy good suggestion and also usefull. My ngx-datatable's version is 18.0.0 & also need fixed header functionality in my project. Thanks in advance


Enhancement: HELPFUL LINKS should open in new tab

Links mentioned in HELPFUL LINKS should open in new Tabs on click instead of same tab.

Leave issue at : https://github.com/covid19india/covid19india-react


How to mock store in components

I've being looking around how to mock components in order to be tested, do you have an idea? https://github.com/rintoj/statex/blob/master/examples/todo-ng-ts/src/app/list/list.component.ts
Since you are doing a new instance of the store, it isn't quite easy to test it.
Thanks!

This is how I did it but I'm really not confortable with, if you have a better way please let us know :).
Love Statex man!!
Implementation:
private loadRooms(rooms: Room[], roomsNumber?: number) {
    if (!isUndefined(roomsNumber)) {
      if (roomsNumber > rooms.length) {
        this.dispatchedAction = new AddRoomsAction(roomsNumber);
      } else if (roomsNumber < rooms.length) {
        this.dispatchedAction = new RemoveRoomsAction(roomsNumber);
      }

      this.dispatchedAction.dispatch();
    }
  }

Unit test:
    it('should remove rooms', () => {
      const numberOfRoomsControl = {
        value: 0
      };

      component.rooms = [
        {
          id: '',
          name: '',
          ordinal: 3,
          isNew: true
        }
      ];

      component.onNumberOfRoomsChange(numberOfRoomsControl);
      expect(
        component.dispatchedAction instanceof RemoveRoomsAction
      ).toBeTruthy();
    });

@blopez2010, give me couple of hours. I will upload a test case for list component

@blopez2010, StateX is configured for detached store and view. Therefore when you unit test a view you must only test for appropriate actions being dispatched. What happens when an action is trigged must be tested under store's test case, not here.
This example should help you understand how to test that.
https://github.com/rintoj/statex/blob/master/examples/todo-ng-ts/src/app/list/list.component.spec.ts

Thanks @rintoj for the example!

I have some problems with the example above. What I forgot is to reset the state before each test:
initialize( {} )
an other possibility is the jasmin spy returns a correct state. example:
let toggleTodoActionCall = jasmine.createSpy('spy').and.returnValue(new ReplaceableState({}));


[Feature]: Sort by smoothed metric

Description
Sometimes the measurement of a metric can be a quite noisy estimate of the true value. In the plots this can be accounted for using smoothing. However, if you want to find the best run, it's not possible to sort by a smoothed metric.
Suggested Solution
I would suggest to add a smoothing button in the sort menu that allows for sorting by a smoothed version of that metric.
Alternatives
The alternative would be to manually calculate the smoothed metric during the run or later on by loading the data from the API. However, this is a significant amount of manual effort and it would be nicer to have this integrated right into W&B.
Additional Context
No response

Hi @JGameCreation,
Thank you for your feature request! I'll share this with our engineering team and I will keep you posted on the status of this request.
Thanks,
Ramit

Bump for this feature
Sometimes evaluating a model enough times to smooth out the results is infeasible during training. What @JGameCreation  suggested would solve this issue entirely.


Deprecate pre-NATS Avena services

Currently there are services such as cell_logger, can_logger, and gps2tsdb whose functions are going to be taken over by NATS-based services such as cell_nats, gps_nats, and j1939d, which better fit the Avena model for ISOBlue, and when used in conjunction with a Jetstream to long term storage backend (file-based or DB storage) will perform the same function of these pre-NATS services. In the same way, services such as socketcand and oada_upload which are also part of other pre-NATS servers should be archived with the pre-NATS services.

I propose to archive all pre-NATS services and their dependencies into a legacy subfolder in the services folder, and services that currently have no maintainer into an archive subfolder. @abalmos  Any thoughts on this?

Done


Abstract USB device implementation

I've extracted the platform independent code from the rp2040 device impl (see ZigEmbeddedGroup/raspberrypi-rp2040#40).
One can create a concrete implementation by calling Usb(F), where F is a struct that contains a set of functions to handle the low level details (register access etc.).
Check out the usb examples in the rp2040 repo!
@MasterQ32 @mattnite

🤌

In general I would prefer removing  Usb from the beginning of the identifiers since they're in a usb namespace. That being said, I'm also for keeping them as is since I know that's how the USB standard refers to them. What do you think?

Just write everything down you dislike, I'll take care of it

The only other change I'd like to see is to ensure that the unit tests run when you execute zig build test, once that's done with the removal of Usb we're good to merge :)

@r4gus is this ready to merge?

@mattnite yes. Sending multiple packets between Host and Client seems to Work fine (with the rp2040). Adding Support for other Chips will probably reveal some shortcomings.

Awesome


add captureParamMaybe, queryParamMaybe, formParamMaybe

closes #251 closes #137

@fumieval  any objections to merging this?


Remove default constructor for PublicKey

The root of the problem is PublicKey is default constructable, and I don't think it should be. Anything using a default constructed public key should be changed to use a boost::optional<PublicKey> and any constructed PublicKey should be valid. This removes the empty_ junk from public key and makes explicit the assumptions about how the key is used at the site.
Originally posted by @seelabs in https://github.com/ripple/rippled/pull/2774

If we do this we should also remove the default constructor for SecretKey.  And, yeah, I think this is a really good idea.


General Refactor

Update rubocop to 2.3 parser

we updated to ruby 2.3 5 months ago!
turned off the Style/FrozenStringLiteralComment as it errored on every single page and is due to be implemented in ruby 3
made code-climate happy

Removed OUTER from the LEFT JOINS

Updated the recent SQL refactor to use the new, indenting, HEREDOC format

👍 looking good


Don't send a sync XHR in onbeforeunload handler

Browsers and versions affected
All
Description
When the browser tab goes away, the onbeforeunload event causes Call hangups to send a sync XHR.
This appears to be deliberate, and is explicitly called out here: https://github.com/webrtc/apprtc/blob/6a83865f0e74a5d2c1ac197eefc613827835e138/src/web_app/js/call.js#L146
This is pretty bad for performance, especially on slow networks, as the main thread for the content is blocked waiting for the XHR to complete.
It might be possible to do the same work using sendBeacon in supporting browsers: https://developer.mozilla.org/en-US/docs/Web/API/Navigator/sendBeacon
Or perhaps a ServiceWorker would be useful here instead.
Steps to reproduce
Open a call on a bad network connection. Close the browser tab.
Expected results
There shouldn't be a sync XHR in the onbeforeunload event.
Actual results
There's a sync XHR.

This was originally filed as a bug in Mozilla's bug tracker at https://bugzilla.mozilla.org/show_bug.cgi?id=1250088

Yes its bad for performance but at the same time we need to cleanup in the backend. Interestingly it works fine in Chrome and in earlier Firefox versions hence it's a clear regression in Firefox.
Having said (written) that, we are ofc open to better ways of doing things here. But not sure when.

Closing due to age.


Add README and LICENSE

WHAT
Add README.md and LICENSE.

@spesnova Please review this :bow:

:+1: :star:


Unreviewed commit on master!

Unreviewed commit(s) COMPARE was merged to master by irustandi-cb.
Comment with appropriate number of :+1: to approve.

Reviewed requirements satisfied? No
Ancestors reviewed? Yes

Minimum Reviews
0/1

More info

:+1:

:+1:

Review Error for yifan-cb @ 2022-12-20 16:00:28 UTC
User failed mfa authentication, public email is not set on your github profile. see go/mfa-help

https://github.com/yifan-cb

:+1:


Internal server error (500) on page https://docs.docker.com/opensource/workflow/make-a-contribution/

Internal server error (500) on page https://docs.docker.com/opensource/workflow/make-a-contribution/

Output of docker version:
(paste your output here)

Output of docker info:
(paste your output here)

Additional environment details (AWS, VirtualBox, physical, etc.):
Steps to reproduce the issue:
1.
2.
3.
Describe the results you received:
Describe the results you expected:
Additional information you deem important (e.g. issue happens only occasionally):

Works for me. You still seeing it?

Nope. Its fine now.


Tech debt: Remove AWS spot version check

When spot instance support was added for MachinePools, it wasn't yet available in all then-supported versions of OpenShift (4.5+). Now it is. Remove the version check, which is now dead code.

/assign @dlom

/lgtm


[Hint addon] Custom hints[2]

CreatorVilius
Is there anyway to make custom hints? I want to create custom hints. What I mean by custom hints, is add our own hints.

marijnh
Yes, there is—start with the docs for the show-hint plugin, or look at css-hint.js for a relatively simple example.

I know there is manual for show hint addon, but I need clean example. Taking look at JavaScript's hints, makes me confused. I know what does what, but there are bunch of random functions put together which makes me slightly confused.

Please stop opening new issues for questions. Start a thread on the forum if you want, but this is a bug tracker.
Anyway, I pointed you at the css-hint file for an example, which is simpler. But yeah, reading code is going to require figuring out why the 'random functions' are there, that's an effort you'll have to make.


feat: Schema builder

#5  👈
main

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

Join @tazarov and the rest of your teammates on  Graphite


Add more wandb options

  # Project name for wandb logging
  wandb_project_name: Optional[str] = None
  # Team name for wandb logging
  wandb_entity_name: Optional[str] = None
  # Model tag for wandb labeling
  model_tag: Optional[str] = None

In case users have multiple active wandb projects/teams

Addressed in #43


"git cm" fails if invoked from a directory other than git repository root

I have a project checked out into /home/maksym/Work/project. I'm trying to run git cm from /home/maksym/Work/project/rust/src/ directory. The command fails with the following error:
thread 'main' panicked at 'Failed to open git repository:
Error { code: -3, klass: 6, message: "could not find repository from \'.\'" }',
/home/maksym/.cargo/registry/src/github.com-1ecc6299db9ec823/git-cm-0.1.2/src/git.rs:106:57
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace

When the same command is run from /home/maksym/Work/project, it works flawlessly.

You can pass a path as an input value to the command: git cm ../../. It's undocumented but it exists :)

True, I discovered that. At the same time, git commit does not require that and finds the repository from anywhere in its tree - which is what the fix can do. :)


Splatoon Ultrawide Fixes

Fix Squid Sisters TV render
Before:

After:

Closing so that it doesn't take double the time to compile each commit. Will reopen once all the changes are done.


new waiting flag for buttons. plumb that through for tracker windows

@keybase/react-hackers

can we use black75 for the spinner on secondary buttons?
It's invisible otherwise:

:+1:


Update change your language doc

Update http://localhost:9991/help/change-your-language to follow the same structure as the following examples:

http://localhost:9991/help/change-your-name
http://localhost:9991/help/change-your-avatar

Delete unnecessary screen shots.

https://github.com/zulip/zulip/pull/3066

I guess this is already fixed by a reviewer

https://github.com/zulip/zulip/pull/3105

Yep, looks good, closing.  In the future @synicalsyntax you can add Fixes: 3013 to your commit message to cause GitHub to auto-close issues when a commit is merged.


Pandoc could not be spawned error

Description
After installing pandoc and opening an RST file to edit, Atom will show the following error:
'pandoc' could not be spawned. Is it installed and on your path? If so please open an issue on the package spawning the process.
Steps to Reproduce

Install pandoc via the Atom Settings
Open RST file
Edit

Expected behavior: [What you expect to happen]
No error
Actual behavior: [What actually happens]
Error is produced:
'pandoc' could not be spawned. Is it installed and on your path? If so please open an issue on the package spawning the process.
Reproduces how often: [What percentage of the time does it reproduce?]
Constantly
Versions
Atom Version: 1.26.1
OS: Windows 10 Professional 64-bit

Thanks for contributing!
It appears that this is being caused by the pandoc package, not Atom itself. It may be that there is still a bug in Atom. If so, we'll need the maintainer of the package that is causing the problem to help track that down. You should file this issue on their repository so they can either fix the problem or tell us where the problem is coming from. The Atom FAQ has information on how to contact the maintainers of any Atom community package or theme.


Consider renaming TagContexts class.

It is a little confusing to have both TagContext and TagContexts, especially since TagContexts isn't a utility class.  I can't think of a perfect name for the class, so I figured I would open an issue for discussion.  Here are some ideas:
Tagger
TagContextFactory
TagContextManager
TagContexts (current name)
The purpose of the class is to allow creating and manipulating TagContexts for implicit and explicit tag propagation.
See https://github.com/census-instrumentation/opencensus-java/pull/498#pullrequestreview-55777822.

+1 for Tagger


Can I use it without blade support? (e.g. with Twig)

I want to use this package in octobercms. Although this CMS is written based on laravel but it has no support of blade and works totally based on Twig!
I just want to know if i can use it there.

I'm not quite sure I never try it actually


Interstitial code allows for responsive sizes

Updated code to account for variable-sized interstitial banners.

+1


Fix Git hooks not created with trailing newlines

These files could create spurious diffs if someone were to open them with an editor that automatically fixes new lines, or when running with something to automatically fix missing new lines, e.g. a pre-commit hook[1]
[1] https://github.com/pre-commit/pre-commit-hooks?tab=readme-ov-file#end-of-file-fixer

I figured it was about as much effort to review an issue as it was to review this PR
I'm not a rust dev so it's likely the added code could be more idiomatic

@matthewhughes934 Looks good, just need to run just bump patch and commit it.

@matthewhughes934 Looks good, just need to run just bump patch and commit it.

:+1: done, I've just squashed everything into d798bc4


US Virgin Islands flag

It appears that the British Virgin Islands are in there twice. ''vi.svg" should be the US Virgin Islands flag.

Confirmed. Currently both vi.svg and vg.svg look like this:

But vi.svg should look like this:

It appears wikipedia is using an SVG - can we just use that?


XmlGraphTest.java:43-45: In #413 we evolved graph...

The puzzle 413-b264bb98 from #413 has to be resolved:
https://github.com/yegor256/jpeek/blob/27d7bb11de1e6576f6c27b75635517c442451fe6/src/test/java/org/jpeek/graph/XmlGraphTest.java#L43-L45
The puzzle was created by Hamdi Douss on 05-Apr-20.
Estimate: 30 minutes,  role: DEV.
If you have any technical questions, don't ask me, submit new tickets instead. The task will be "done" when the problem is fixed and the text of the puzzle is removed from the source code. Here is more about PDD and about me.

@paulodamaso/z please, pay attention to this issue

The job #441 assigned to @victornoel/z, here is why; the budget is 30 minutes, see §4; please, read §8 and §9; if the task is not clear, read this and this; there will be a monetary reward for this job

@0crat refuse

@0crat refuse (here)

@victornoel The user @victornoel/z resigned from #441, please stop working. Reason for job resignation: Order was cancelled

@0crat refuse (here)

@victornoel Job refused in 22 hours - no penalty, see §6

The job #441 assigned to @dgarus/z, here is why; the budget is 30 minutes, see §4; please, read §8 and §9; if the task is not clear, read this and this; there will be a monetary reward for this job

@0pdd the puzzle #445 is still not solved.

@dgarus/z this job was assigned to you 5days ago. It will be taken away from you soon, unless you close it, see §8. Read this and this, please.

The user @dgarus/z resigned from #441, please stop working. Reason for job resignation: It is older than 10 days, see §8

The job #441 assigned to @marceloamadeu/z, here is why; the budget is 30 minutes, see §4; please, read §8 and §9; if the task is not clear, read this and this; there will be a monetary reward for this job

@marceloamadeu/z this job was assigned to you 5days ago. It will be taken away from you soon, unless you close it, see §8. Read this and this, please.

@0crat wait

@0pdd 2 puzzles #472, #473 are still not solved; solved: #445.

@0crat refuse

@0crat refuse (here)

@marceloamadeu The user @marceloamadeu/z resigned from #441, please stop working. Reason for job resignation: Order was cancelled

The job #441 assigned to @dfrau32/z, here is why; the budget is 30 minutes, see §4; please, read §8 and §9; if the task is not clear, read this and this; there will be a monetary reward for this job

@0crat refuse

@0crat refuse  (here)

@dfrau32 The user @dfrau32/z resigned from #441, please stop working. Reason for job resignation: Order was cancelled

@0crat refuse  (here)

@dfrau32 Job refused in 7 hours - no penalty, see §6

The job #441 assigned to @iiylll/z, here is why; the budget is 30 minutes, see §4; please, read §8 and §9; if the task is not clear, read this and this; there will be no monetary reward for this job

@iiylll/z this job was assigned to you 5days ago. It will be taken away from you soon, unless you close it, see §8. Read this and this, please.

The user @iiylll/z resigned from #441, please stop working. Reason for job resignation: It is older than 10 days, see §8

The job #441 assigned to @vzurauskas/z, here is why; the budget is 30 minutes, see §4; please, read §8 and §9; if the task is not clear, read this and this; there will be a monetary reward for this job

@0crat wait for #441

The puzzle 413-b264bb98 has disappeared from the source code, that's why I closed this issue.

@0pdd the puzzle #473 is still not solved; solved: #445, #472.

@sereshqua/z please review this job completed by @vzurauskas/z, as in §30; the job will be fully closed and all payments will be made when the quality review is completed

@0crat quality good

@0crat quality good (here)

@sereshqua The project doesn't have enough funds, can't make a payment

@0crat quality good (here)

@sereshqua The project doesn't have enough funds, can't make a payment

@0pdd the puzzle #484 is still not solved; solved: #445, #472, #473.


Update smoke tests to auto-generate dependency checks for all t2 SDKs

The goal for smoke tests is to catch and block on dependency issues before release, as opposed to samples tests which are currently a superset of this functionality.

Pull in all track 2 packages to a single project and install/compile (nightly versions, GA/beta versions).
Run the project and instantiate clients from each package, where useful (for detecting lazy loaded runtime dependency issues).

Smoke tests have been disabled globally for now, future work is tracked here: https://github.com/Azure/azure-sdk-for-js/issues/24440


Try changing qunit dependency to more up-to-date fork

I didn't look closely to see whether you had other important changes in node-qunit, but since @yomexzo's version seems to be working for me to get more recent QUnit API support, I'm suggesting this PR.
If you need to integrate your own changes to node-qunit, I suggest abandoning this PR and updating your own node-qunit to take into account the likes of @yomexzo's changes (as I demoed for you in another PR just now).

And maybe submit your changes back to @axemclion

Thanks, I'll try to realign with the latest original master!
Thanks,
Luciano


Expose error kind enum for matching?

Currently, the inner kind of the error type is unexposed which makes mapping into an SDKs error types.
https://github.com/hyperium/http/blob/master/src/error.rs#L25-L35
This works well for displaying the description but if I want to map the result to a different error to make it consistent I cannot. For example, if I'm using both hyper and http (this may work better in the future) I cannot match a Uri parse error in hyper to that of http with ease and combine them into a single BadUri enum. In fact, it's deeper than that in that the inner types for the Uri errors actually point to another struct and error kind.
I also acknowledge that I'm still new to rust and this means that I might be missing a design pattern that I should use.

I don't think we would expose the ErrorKind enum publicly, because it prevents adding new variants, or changing them around, and thus makes improving the Error internal design a lot harder.
If you had access to the ErrorKind, what kind of code would you hope to write? Maybe we can find a solution.

Hmm. I think that since individual functions appear to each implement their own error kinds (and those structs are public) I can properly map them. However, I'm currently implementing both a From the InvalidUri and the Error types:
https://github.com/kbacha/stellar-sdk/pull/39/files#diff-045ef8f78ba0431706a905bb35c78758R56
This enables me to map specifically for invalid errors but also catch-all when a builder replies with the generic error. Perhaps it works out the box anyhow?

You could change from using the builders to doing the conversions yourself, thus knowing exactly what operation the error came from.

Ah, that could work. And that's what I'm doing with the URI in this case.

Is this issue resolved?

I find myself with the same question @kbacha had, but I don't understand the solution @seanmonstar offered. I too am new to Rust, which may be the issue.
I'm trying to upgrade a library from hyper 0.10 to hyper 0.12. The library has a domain-specific Error enum, some variants of which correspond to variants of ErrorKind. I'd prefer to map the ErrorKind variants to my existing variants, rather than creating a catch-all for all instances of http::Error.
What's the best way to accomplish that?

Now that we have non_exhaustive (added in stable 1.40.0) this is possibly something to reconsider. Does this crate specify a minimum supported version less than 1.40?

MSRV is currently called out as 1.49, so this should be possible at this point. Could this issue be reopened?
I think this would just be:

Delete the current Error
Rename ErrorKind to Error
Mark this non_exhaustive

This should be a nonbreaking change IIUC. The debug impl could be adjusted or changed to a derive as well.

Please re-open this issue, to expose uri:InvalidUri(ErrorKind).  We should now be able to use the non exhaustive annotation.  It should be non-breaking because the previous API was private.
My use case is: I am trying to port the reqwest proxy implementation for hyper, and switch it to use uri instead of url, and there are tons of test cases that check the kind of error.