Shape color filling want to add borders and fill them at the same time,It also provides transparent options for shape filling borders

Shape color fill border displays black

Hi nh-live,
Thanks for your feedback
Which version,
can you share your xml graph please?
BR
Arnaud

use draw.io default template:
Untitled Diagram.xml.gz

version:4.0-snapshot

Very nice diagram

Hi nh-live,
MxGraph libs changes fill color in auto like draw.io if it not defined,
Test it on draw.io and you'll see that fill border changed.
The solution is simple but not friendly in current version :

Copy rule with the button
Change "color mode" to stroke in the copy of rule.

I think i'all add a new mode "Fill and Stroke" in color mode for this use case.
BR
Arnaud
PS : thx for your star

Screen capture

Thank you for creating a great plug-in and your quick response.

@nh-live
Can i share your graph in gallery and tweeter ? i dont see sensible datas
BR

It's not made by me. It's a default template for draw.io. I don't think it's going to restrict you from sharing it.


Background is called twice before Scenario Outline

1.1.0.RC1
I have following settings in the Background part:
Feature: Title of the feature
    Background:
      * configure printEnabled = true
      * configure continueOnStepFailure = true
      * url 'https://example.com/'

And then Scenario outline part:
 Scenario Outline: Background should be called once
    Given path 'testing/background'
    * request read('justtesting.json')
    When method post
    Then status 200
    * match response.value == "test

    Examples:
    | read('this:test.csv') |

When launching scenario, the Background part is being executed twice. This is how the report looks like:

@ptrthomas this is related with https://github.com/intuit/karate/pull/1562
With that change I changed the behavior of background (can't find the trail of the discussion). It now evaluates background to calculate the Examples table and then proceeds to execute the Dynamic Scenario Outline like any other normal Scenario Outline in which the background is re-executed with each run.
I guess I forgot to check the report, not sure how you tackled this here for normal Scenario Outline to show up nicely.
Btw this was due to some graal issues (https://github.com/intuit/karate/issues/1560 I think) which might now be fixed with the latest change you did but uncertain. To be honest I'd just fix the reporting of the background as the behavior above is not different than normal Scenario Outline (with the exception that there is one extra iteration to evaluate the Examples table).
The culprit line of code is the ScenarioRuntime.java:352 at https://github.com/intuit/karate/pull/1562/files .

@joelpramos so if I just hide the 1st background for this case we are good right

I think so. The ScenarioRuntime should already have a property/method indicating that it's a background execution.
@intest to confirm - are you only seeing those extra steps in the first iteration of your Scenario Outline?

@joelpramos no, it is in every iteration

hmmm ok so I'd need to open the code to confirm but @ptrthomas should be hiding the steps but I was expecting this to show up only on the first iteration. Maybe there is logic from before to pull the steps to each execution so that the reports looked similar to the other Scenario Outlines as the previous workflow only executed background once

@joelpramos should be fixed, seemed simple enough

1.1.0 released


Objective-C

This would settle an office debate :)

:+1: haha same here

Have fun!

:tada:

:+1:


Missing middle-ware id in Common/PluginsDefines.php

Expected Behavior
Add missing middle-ware ID in Plugins/Common/PluginsDefines.php  for:

MSSQL_SERVER => 2200
ORACLE => 2300
CUBRID => 2400
POSTGRESQL => 2500
CASSANDRA => 2600
MONGODB => 2651
RABBITMQ => 8300
ACTIVEMQ => 8310
ARCUS => 8100

Current Behavior
In version 4.0.0 of plugin package the previously listed middle-ware are not present. Wondering also if it doesn't make sence to rename define("PP_HEADER_NGINX_PROXY","Pinpoint-ProxyNginx"); to define("PP_HEADER_NGINX_PROXY","HTTP_PINPOINT_PROXYNGINX"); to make it consistent with others ?
Context

pinpoint-php: 0.4.0

@marty-macfly

ServerType ID

You can give up PR.

Pinpoint-ProxyNginx

I found an issue on https://github.com/pinpoint-apm/pinpoint/issues/3252. it could help you use Proxy Header feature.

Yep, I'll prepare that

Here is the PR


openssl s_client -crl_download doesn't work

Command openssl s_client -connect redhat.com:443 -crl_download -showcerts doesn't download CRL
Looks like crls_http_cb callback is installed only when -verifyCAfile or friends are specified but even then it is not called.

can you please set a break in check_cert and see if get_cert_delta is called

I put breakpoints to check_cert, get_cert_delta, and crls_http_cb and built openssl with -O0. None of the breakpoints were triggered :(

Looks like crls_http_cb callback is installed only when -verifyCAfile or friends are specified but even then it is not called.

It does get installed elsewhere - so specifying -verifyCAfile isn't necessary.
This could probably be called a documentation bug. The -crl_download option doesn't seem to do anything unless you also specify -crl_check
Adding that does cause the callback to be called. Unfortunately it then subsequently fails because:
807BE4F7FF7F0000:error:1E800075:HTTP routines:check_set_resp_len:max resp len exceeded:crypto/http/http_client.c:511:length=16236077, max=102400
807BE4F7FF7F0000:error:1E800067:HTTP routines:OSSL_HTTP_REQ_CTX_exchange:error receiving:crypto/http/http_client.c:1002:server=http://crl3.digicert.com:80

I agree about documentation bug, thanks.
I wonder if we can do smth to workaround the size, BTW

This function:
https://github.com/openssl/openssl/blob/4f48629c9d791c9105f2d68f4a4d0b0085e8e0ca/crypto/x509/x_all.c#L187-L191
Calls this one:
https://github.com/openssl/openssl/blob/4f48629c9d791c9105f2d68f4a4d0b0085e8e0ca/crypto/x509/x_all.c#L115-L131
Which uses OSSL_HTTP_DEFAULT_MAX_RESP_LEN (100k) for max response length. This sounds like it could be a bit too short for a CRL.


add base path option to helm chart

Also see: https://github.com/reactiveops/charts/pull/89/files

Helm chart has been moved to reactiveops/charts


Xbox version cannot see lan games

I was using this image for a couple months and it worked great.
However recently after I upgraded the client to 1.18 (xbox x game pass version), it can no longer show lan games.
Anybody has the same issue?

My android minecraft (latest version) can still see the server.

If you're familiar with tcpdump (on Linux) or Wireshark (Mac/Windows) then it might be interesting to watch the port 19132 traffic to see what happens with the multicast ping message that the client sends out.

Will try to do that. Before that I just want to quickly confirm if this feature was cut by Microsoft, if someone else also encounter this issue.

Not that I know of, but I do know the server discovery part is flaky. I have had times where restarting the iOS client got it to work.

Checked again. It worked again. Not sure if the issue was in Xbox OS, Minecraft Xbox version or bedrock server, but seems Microsoft found the bug and fixed it.


Remove goog.labs.userAgent

Not sure if there is a cleaner way of doing this so comments welcomed :)

@tschaub Changed to a function

Anyone able to review?

I added one more comment @nicholas-l. When you're able to address it, we can merge this. Thanks!

@ahocevar I have changed the global to this as I didn't want to add another goog :).

Are you sure that this will also work in a browserify context? We use goog.global for now everywhere else.

I believe it would work as in this is what goog.global is doing, maybe we should have an ol.global? http://google.github.io/closure-library/api/source/closure/goog/base.js.src.html#l48

ol.global would be a next step. This is why I'd prefer to use goog.global for now, because it will be easier and less error prone to change to ol.global later.

ok sounds good

Thanks @nicholas-l.

Have created #5178 to tackle global


Add cursor format to cursor inspector.

Update cursor inspector to display format information for the cursor.

I vote to abandon it.


Text.clone() wraps text in s

Cloning a Text node created using plain() (and thus having the form <text>myText</text>) results in the clone being of the form <text><tspan dy="...">myText</tspan></text>.
Example fiddle
I know this can be somehow avoided when using text.move() of group.translate() instead, but this breaks the spirit of svg, and I need to move the whole group, not single elements.  Plus, I think that cloning should, well, clone :)

I am with you. Cloning should just clone.
The bug occurs because the clone-method calls text() instead of plain() which then leads to a tspan in your text (as first line).
For now you can just go with the fix g2.add(t1.clone().move(0,0)) which moves the text back to its position. I will fix that later.

For now you can just go with the fix …
Thanks.

I think the core problem is that text.content is used. If there was one tspan with x attribute that is not zero, has a class set, ..., all that would just get lost (I assume. Not tested).  I think it should instead just recursivly clone all the tspan nodes.

Yes, cloning relies on the text.content-property which is not properly updated when e.g. using a function. Thats why cloning fails there.
svg.js has no deep-clone till now. Elements which are cloned are just recreated which leads to the problem with text since text is based on child-elements, too.
Basically it would just be needed to clone all children (including text-nodes), yes - still there is the problem with nested tspans... and so on...

this bug is solved in svg.js v2


fix json body-parser type check to work with content-type with charse…

fix json body-parser type check to work with content-type with charset (ex: application/json; charset=utf-8)
Thank you for helping out! ✨
Before submitting a pull request, please make sure the you checked the following:

[x] Forked the repository and created your branch from master
[x] Make sure the tests pass! npm it in the repository root
[ ] If you've fixed a bug or added code more tests are appreciated
[ ] If you're adding a new command, removing an old command, or changing how a command works, please update the relevant documentation for the command (under the doc/ folder)
[ ] Summarize your changes in the changelog
[x] If you haven't already please complete the CLA

Learn more about contributing: https://arc.codes/intro/community

Seems like github won't recognize my mail address.
Can't sign CLA properly.
Could you just copy the code and fix it?
Thank you for the good work.

Hey @hada-unlimited sorry but I cannot accept contributions without a CLA and copying would be bad faith to the project. I know its a pain but there are other methods to get a CLA to the foundation if you contact them here: https://js.foundation/contact (again sorry but its for the best for the project we stay very strict about accepting contributions as to ensure everything remains certifiably f/oss!)


Consider using foreman

Given there are multiple processes in this application, it might make sense to create a Procfile and use Forman.
A simple Procfile.dev would be:
web: cd web && npm run start:dev
worker: cd api && npm run start:dev

Then users could start the application (after having set up the necessary dependencies) by running nf start -j Procfile.dev from the root of the repository.
I'd be happy to put out a PR making this change if the Flow team supports this idea.

We have a 'consolidated' startup using concurrently in the root of the project. Procfiles could be a good road to a Heroku deployment.
Would love to make this deployable to Heroku. Any thoughts?

That would be fun.  I'd imagine the best way to do that would be with a Deploy To Heroku button (docs).  Happy to look into how to set this up if you like the idea.

That would be fun. I'd imagine the best way to do that would be with a Deploy To Heroku button (docs). Happy to look into how to set this up if you like the idea.

Love the idea. I have looked into it briefly. It looks like there might be some extra config because there are actually 2 separate apps in the project. (The server does not serve the html SPA).
I suppose we could set up the server to also handle the SPA.
It would be very cool to have a one-button deploy for this app.

OK, check this out: https://github.com/onflow/kitty-items/pull/54


Remove/move cmake/pio versions of the template

Since cargo-first is the recommended approach, what if we remove the cmake version from this template. It would simplify the repo, and/or it can be moved to another repository like esp-idf-cmake-template.
It is just an idea, I would like to hear any other opinions about this @ivmarkov @MabezDev @JurajSadel @georgik @Vollbrecht @bjoernQ

I'm strongly against this. What would it solve? Save one prompt? If you are anyway doing the two-prompts approach, that you mentioned elsewhere, what's the point?
As for cargo-pio - it is just a readme file.
I think we might be over-rotating on the simplification idea.

As you mentions, it does not save much from user perspective, the main advantage would be cleaning the repository, but if you are strongly against this I am more than happy to close it.

Thank you! I think the cmake code is not really that much in terms of code size or comexity. And more importantly - users do not see the repo code size anyway. Just the generated project, which is either cargo-first, or cmake, but not both.


Validate that subpath exists when deploying project from a monorepo

If you don't think this is worth doing now, feel free to add it to https://github.com/rilldata/rill/issues/5053

I can no longer reproduce it. May be it got fixed already or is reproducible under specific conditions only.
Closing issue for now.


TryValidateModel throws exception when unit tested

When attempting to run a unit test against an ApiController in ASP.NET Core, TryValidateModel throws an exception that ObjectValidator is null.  When trying to set the property, I can't seem to figure out what properties should be set to what.  I can inject the DefaultObjectValidator, but it always passes, even when I purposefully set bad values to my model.
There was a bug opened for this years ago that I found but the only solution seemed to be mocking, which I don't want to do because I want the test to legit fail should I pass it bad data to ensure everything is configured correctly in the controller.
public UserTests()
    {
        ICompositeMetadataDetailsProvider p = new DefaultCompositeMetadataDetailsProvider(new IMetadataDetailsProvider[]
        {
            new DefaultValidationMetadataProvider(),
            new DefaultBindingMetadataProvider()
        });
        IModelMetadataProvider metadataProvider = new DefaultModelMetadataProvider(p);
        var objectValidator = new DefaultObjectValidator(metadataProvider, new List<IModelValidatorProvider>
        {
            new DefaultModelValidatorProvider()
        });

        Controller = new UserController
        {
            ObjectValidator = objectValidator
        };
    }`

Setting up a controller should be much easier than this and object validation should be initialized by default. At the very least there should be some documentation somewhere regarding how to initialize this property outside of the full request pipeline.

I ended up creating my own IObjectModelValidator, which I'm not sure is the best approach, but it seems to work.
public class TestingObjectValidator : IObjectModelValidator
    {
        public ControllerBase Controller { get; set; }

        public void Validate(ActionContext actionContext, ValidationStateDictionary validationState, string prefix, object model)
        {
            var validationContext = new ValidationContext(model, null, null);
            var validationResults = new List<ValidationResult>();
            Validator.TryValidateObject(model, validationContext, validationResults, true);
            foreach (var validationResult in validationResults)
            {
                Controller.ModelState.AddModelError(validationResult.MemberNames.FirstOrDefault() ?? string.Empty, validationResult.ErrorMessage);
            }
        }
    }

public UserTests()
        {
            Controller = new UserController();
            var objectValidator = new TestingObjectValidator
            {
                Controller = Controller
            };
            Controller.ObjectValidator = objectValidator;
        }

I would still like to see a fix so this doesn't need to be done.  Who knows if I'm missing a configuration or edge case that is addressed in the full pipeline.

Thanks for contacting is, @mmulhearn.
@pranavkm, is the approach taken here one we'd recommend, or is there a better way to do this?

This issue was moved to aspnet/Mvc#8640


Update instructions for merged and released description repo

Since we've merged the melodic-devel-staging branch and released the description, we can update the docs here.

Oops, I've just realized, I've missed #6. @RobertWilbrandt I still think it would be a good idea to refer to binary installation of the driver (Which was not available when #6 was created).


Move "movex-react/movex-local" into it's own library

What?
At the moment the libs/movex-react contains the movex-local code, but this can be it's own separate library. Probably called something like movex-local-master or even 'movex-react-local-master'.
Why?
To reduce the movex-react library file size!
Open to discuss
Although one issue might arise once we support more frameworks, as then we need to add a movex-[framework]-local-master for each one! I think this deserves more thinking!

This was done a while ago!


Prevent drift within workspaces by importing all related resources

The action should be able to discover all related resources (e.g., variables, team access) and remove configuration that is not specified in the action inputs. This behavior should be optional, default TBD. This allows users to detect/revert drift within workspaces managed by this action. Drift is already prevented on the workspace settings themselves, but not for related resources.

If this doesn't make v1, we'd have to put this in v2 in order to enable it by default.

fixed by #72


Update Voting.sol: Gas Optimizations

🛠 Fixes Issue
Inefficient 'For' loops
👨💻 Changes proposed

Changed uint256 i = 0; -> uint256 i;, unsigned integers always have a default value of 0. Thus, it is a kind of waste of some gas to initialize variable to 0;

Efficient For loops -> Above point is maintained here. Plus i++ changed to ++i, does the same work with less gas used up

[x] My code follows the code style of this project.

[x] This PR does not contain plagiarized content.

[x] The title of my pull request is a short description of the requested changes.

LGTM. Thaks for this ssue.


Create ibis-ear-plugin for Maven

Reporter
Jaco de Groot
Is your feature request related to a problem? Please describe.
Ibis projects building their ear artifact with Maven need to have a copy of the was.policy and several other files in their project. These files are now copied from one project to another. When something needs to be added or changed to these files everyone needs to make this change (which in practice doesn't always happen).
Describe the solution you'd like
Have Maven add the files like was.policy to the ear file from a centralized location (no copies of the files in every project). Every project should only need to specify the use of the ibis-ear-plugin instead of the default ear plugin so the files will be added at build time.
Describe alternatives you've considered
-
Additional context
For building a war file with Maven the https://github.com/ibissource/ibis-war-plugin already exists. A similar plugin needs to be created for building an ear file. Features/files to be added to/by this plugin:

application.xml
was.policy (WebSphere)
ibm-application-bnd.xmi (WebSphere)
DisableIBMJAXWSEngine: true (WebSphere)
deployment.xml with classloader mode="PARENT_LAST" (WebSphere)
jboss-deployment-structure.xml (JBoss, see IJA_Graphviz)

No longer seems relevant?


hourly chime

i like to have a hourly chime in my clock but not so annoying as in original firmware, just a short beep. also i like functionality and logic of your firmware. so here is the patch for hourly chime support. chime display are right after alarm, both displays are with timeout when not adjusting. 12h/24h supported, dot1/dot3 are pm indicators for start/stop hours, dot2 - chime on/off, shown only at chime display.
works fine with my STC15W404AS.
of course, chime can be disabled with WITHOUT_CHIME at compile time.
diyclock_chime_1224.diff.txt

hi @venustrg, please submit any patches as a pull-request
thanks

did so

fixed with #43


feat: Minimally added focus-visible with postcss-focus-visible

Overview
Per the :focus-visible Polyfill RFC, brings in postcss-focus-visible to allow us to write :focus-visible in code and have it be polyfilled.
Does not bring in the focus-visible polyfill because that crashes in SSR. We'll need to move that into downstream repos.
PR Checklist

~[ ] Related to designs:~
[x] Related to JIRA ticket: WEB-1146
[x] I have run this code to verify it works
[x] This PR includes unit tests for the code change

Description
Sets it up in our postcss styles processing our webpack config to only rely on the [data-focus-visible-added] selector (instead of duplicating styles).
Testing Instructions
In each of a browser that supports :focus-visible (e.g. Chrome) and one that doesn't (e.g. Safari):

Visit the (deprecated) Button story in the styleguide
Click on a button with your mouse -> you should not see the focus outline
Tab to the button with your keyboard -> you should see the focus outline

Enabling keyboard navigation in Mac OS X Web browsers

📬Published Alpha Packages:@codecademy/gamut-labs@5.0.1-alpha.481a38.0@codecademy/gamut@17.0.1-alpha.481a38.0@codecademy/styleguide@18.0.1-alpha.481a38.0<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://5fa5bcedacc1ca29d2c69d69--gamut-preview.netlify.appDeploy Logs

📬Published Alpha Packages:@codecademy/gamut-labs@5.0.1-alpha.2c52dc.0@codecademy/gamut@17.0.1-alpha.2c52dc.0@codecademy/styleguide@18.0.1-alpha.2c52dc.0<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://5fa5c786d2a4cf2a7581dc86--gamut-preview.netlify.appDeploy Logs

📬Published Alpha Packages:@codecademy/gamut-labs@5.0.1-alpha.dad3db.0@codecademy/gamut@17.0.1-alpha.dad3db.0@codecademy/styleguide@18.0.1-alpha.dad3db.0<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://5fa5cf2fa2e01c3c021966a0--gamut-preview.netlify.appDeploy Logs

📬Published Alpha Packages:@codecademy/gamut-labs@5.0.1-alpha.d1d86f.0@codecademy/gamut@17.0.1-alpha.d1d86f.0@codecademy/styleguide@18.0.1-alpha.d1d86f.0<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://5fa5d01023d5693dd3f3ec0b--gamut-preview.netlify.appDeploy Logs

📬Published Alpha Packages:@codecademy/gamut-labs@5.0.1-alpha.097586.0@codecademy/gamut@17.0.1-alpha.097586.0@codecademy/styleguide@18.0.2-alpha.097586.0<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://5fa990bdc78cdc28c3c21321--gamut-preview.netlify.appDeploy Logs


seed_ctgs when continuing from aborted run

I have a run that aborted midway through the pipeline, when resuming I got this error:
============================== bin_reads ==============================
--> 0 chunks need to be run. Skipping...

============================== index_reads ==============================
--> 0 chunks need to be run. Skipping...

============================== assemble_bins ==============================
--> 0 chunks need to be run. Skipping...

============================== assemble_olc ==============================
1 chunks to run. Starting...
2018-09-30 13:41:25 - --starting logging AssembleOLCStep --
2018-09-30 13:41:25 - jointly assemble bins with OLC
2018-09-30 13:41:25 - merge input contigs
orig ctgs 13476
filtered ctgs 9755
2018-09-30 13:41:40 - ========== Exception ==========
2018-09-30 13:41:40 - Traceback (most recent call last):
2018-09-30 13:41:40 -   File "/ifs/scratch/bsiranos/miniconda3/envs/py27/lib/python2.7/site-packages/athena/pipeline.py", line 50, in _
run_chunk
2018-09-30 13:41:40 -     chunk.run()
2018-09-30 13:41:40 -   File "/ifs/scratch/bsiranos/miniconda3/envs/py27/lib/python2.7/site-packages/athena/stages/assemble_olc.py", li
ne 102, in run
2018-09-30 13:41:40 -     for ctg in seed_ctgs:
2018-09-30 13:41:40 - UnboundLocalError: local variable 'seed_ctgs' referenced before assignment

Adding this snippet of code (taken from from the above block) to assemble_olc.py solved the issue:
if not os.path.isfile(mergedfiltfa_path):
  filter_inputs(
    mergedbam_path,
    premergedfa_path,
    premergedfiltfa_path,
  )

  # append input seed contigs
  with open(seedsfa_path, 'w') as fout:
    fasta = pysam.FastaFile(self.options.ctgfasta_path)

   bins = util.load_pickle(self.options.bins_pickle_path)
   seed_ctgs = set()
   for (binid, seed_group) in bins:
     for ctg in seed_group:
       seed_ctgs.add(ctg)

    for ctg in seed_ctgs:
      seq = str(fasta.fetch(ctg).upper())
      for i in xrange(5):
      #for i in xrange(2):
        fout.write('>{}.{}\n'.format(ctg, i))
        fout.write(str(seq) + '\n')

  util.concat_files(
    [premergedfiltfa_path, seedsfa_path],
    mergedfiltfa_path,
  )
assert is_valid_fasta(mergedfiltfa_path), "merge FASTA not valid"

Hi,
Is this running with a particular version?  If not, what is the commit hash for the particular version you are running?  Line 102 in assemble_olc.py in HEAD of both 'main' and 'dev' branches does not match what you copied or what is present in the error message.

Yes, you're right what was in my anaconda environment was slightly different than the most recent commit in the dev branch. I reinstalled and hopefully won't have the error again, sorry about that.

No problem, thanks for letting me know.


Miecraft will crash when loaded

I dont understand any of it

This issue has been fixed. Please update to the newest version 9.2.0 for 1.20.1.


Also use the browser as entrypoint with HBL

Do not know if this is good...

confirmed new exploit?

No simply use the normal entry, like https://dukesrg.github.io/ for D9WIP,OTPH and other.

Uh, what?

Oh, I get what you're asking.
There's no reason to use the browser (which is unreliable) when we have menuhax (which is far less unreliable).


Add rowData to merged altexps

Part of #150
This PR adds the rowData slot into the merged altexp, in a manner than looks about like what you'd expect if Mr. Toad's Wild Ride were an R function. It works, but...I expect Reviews™ about strategy 😬
I added a function build_new_altexp_rowdata() which does what it says. The main reason this function is such a Wild Ride is because of code to handle the preserve_rowdata_cols argument; there will be some columns (notably, target_type) in the rowData which I figured we will not want to include for each altExp, since the values are the same. If we feel it's ok to have several columns of the {sce_name}-target_type flavor, a lot of this code can get simplified right off the bat!
Strategy, for now:

Pass in a list of rowDatas converted to data.frame() into the function
First, grab out the final.final version of each preserve_rowdata_cols we'd like to keep. The code is written flexibly enough to extend this default array in the future. The idea is, each rowData has the potential to have one of those columns, and we want to keep the comprehensive information from all of them "full joined." The list preserved_columns gets populated with the full columns to keep, and then turned into a dataframe for later cbinding.

Currently I rely on the argument to build_new_altexp_rowdata() to define these columns, but we could move this up the chain and pass into the crux merge_sce_list() function.

Then, I define new_rowdata, considering all columns except the ones I pulled out into preserved_columns.
Finally, add back in the preserved columns if they exist, and DataFrame() the rowdata to ultimately add it back into the merged altexp.

Finally, I added a test to make sure the rowData has the expected names, and it is passing, so that's something at least!
Please enjoy the 🎢 and give me all your thoughts on how to make it less of a 🎢 ...!

After some offline discussion, we're going to back up a few steps and come back to rowData soon. Closing this PR in favor of the different incoming strategies.


JSON to YAML

Script Description -
A Python Script to convert a file from JSON to YAML
Instructions

Create a new folder for your script and file/folder name should be appropriate.
Create a README.md in your folder for program Instructions
add requirements.txt if needed
add pytest/unittest if possible

Please add/delete options that are not relevant.

[x] Adding New Code
[x] Improving Documentation

Programming Language

[x] Python

:star2: Star it :fork_and_knife:Fork it :handshake: Contribute to it!
Discord server  - https://discord.gg/FXyh2S3
Happy Coding,

Hi, I can do this issue. Can you assign it to me?


[Q] End-of-text token disabled

Hello there. Thanks a bunch for the great work on this model. Its super cool! I just have a quick question and I wasn't able to find any other resources on this.
In the chat the endoftext token(0) is disabled and the chat flow seems to be controlled with newlines instead. Why is that? Is it just because thats how you have constructed the prompts or is there a more fundamental reasoning behind it?
Thanks again :)

May be you are useing RWKV?
In RWKV,model only output \n\n ,because token(0) is not used during training.

[0] is used to separate documents

That makes sense. Thanks 👍


raft: do not change RecentActive when resetState for progress

Fix #5879.
This reset is completely unnecessary. Most of time it is also wrong, since the state is usually changed when leader receives a message from follower, which indicates the follower is actually active.

/cc @siddontang

LGTM

/cc @bdarnell minor issues. i will just merge this. just want to let you know :) thanks.


egeloen/http-adapter is deprecated

I just noticed you guys use egeloen/http-adapter which has been deprecated and should be considered to be replaced by php-http/httplug

Dupe of #36


Allows Date and DateTime utc and fixes bugs around the same.

Fixes #718

Now Date() DateValue() accepts UTC time.
Fixes IsToday() to take time zone in the account.

✅ No public API change.

    private static TimeZoneInfo LocalTimeZone => TimeZoneInfo.Local;

Can we get rid of this and instead just set it once at runner.GetService ? That' s what we do for CultureInfo.
That avoids having to check at every usage, like:
var timeZoneInfo = runner.GetService() ?? LocalTimeZone;

Refers to: src/libraries/Microsoft.PowerFx.Interpreter/Functions/LibraryDate.cs:57 in 0e4cbb7. [](commit_id = 0e4cbb7af54796dff0606ed0d9f6e9223e7d24ef, deletion_comment = False)

✅ No public API change.

✅ No public API change.

✅ No public API change.

✅ No public API change.

        var timeUnit = ((StringValue)args[2]).Value.ToLowerInvariant();

this is great - we normalize to same timezone.
And ven better - we avoid conversions if it's not necessary (since conversions can be potentially lossly)
Can we add this assert:
Contract.Assert(start.Kind== end.Kind)

Refers to: src/libraries/Microsoft.PowerFx.Interpreter/Functions/LibraryDate.cs:290 in b03cf5d. [](commit_id = b03cf5d1cdb1e3e7aed09f165349235694eb04f8, deletion_comment = False)

        }

This pattern repeats so many times ...
Could we put behind a wrapper, like:
DateTime arg0 = GetNormalizedDateTime(irContext, args[0]);
For error handling, maybe throw a CustomFunctionErrorException? and

allow it to accept a ErrorValue
Call visitor will have to catch it.

Refers to: src/libraries/Microsoft.PowerFx.Interpreter/Functions/LibraryDate.cs:421 in b03cf5d. [](commit_id = b03cf5d1cdb1e3e7aed09f165349235694eb04f8, deletion_comment = False)

        }

And this helper could be GetNormalizedTimeSpan

Refers to: src/libraries/Microsoft.PowerFx.Interpreter/Functions/LibraryDate.cs:471 in b03cf5d. [](commit_id = b03cf5d1cdb1e3e7aed09f165349235694eb04f8, deletion_comment = False)

✅ No public API change.

✅ No public API change.

✅ No public API change.

✅ No public API change.

✅ No public API change.


Doc search text overlapping #359

Description

How and where has this been tested?

How this was tested:
Browser(s) and OS(s) this was tested with:

Screenshots

To-do list

[ ] If my change impacts other components, I have tested to make sure they don't break.
[ ] If my change impacts documentation, I have updated the documentation accordingly.
[ ] I have read the CONTRIBUTING document.

[ ] This pull request is ready to merge.

@rajitha302 this issue has to do with the order of the CSS being loaded, adding a left margin to every textfield will not fix this issue and will cause other errors down stream. @GarthDB is working on this issue. Thanks!


bare impls on same-crate different-mod imported types were useful and there is no alternative

 Issue by o11c
Monday Oct 13, 2014 at 22:09 GMT
For earlier discussion, see https://github.com/rust-lang/rust/issues/18016
This issue was labelled with: A-typesystem in the Rust repository

The following code used to work, but there is no longer any way to do this sort of thing.
Moving the impl into the mod can't be done because low-level modules need to remain similar to the C API they are wrapping, for example if they are generated; and unlike C it is not possible to reopen a module after it is closed.
Adding a meaningless trait does not work because trait methods cannot be called unless you import the trait.
Please bring back some way to do this (whispers of every crate being able to silently inject a prelude does not sound like a good idea at all).
pub mod really_a_crate
{
    pub use self::c::Foo;

    pub mod c
    {
        #[allow(non_camel_case_types)]
        #[repr(C)]
        pub enum Foo
        {
            FOO_A,
            FOO_B,
            FOO_C,
        }
    }

    impl c::Foo
    {
        pub fn frob(self)
        {
            println!("Hello, World!");
        }
    }
}

fn main()
{
    let foo: really_a_crate::Foo = really_a_crate::c::FOO_A;
    foo.frob();
}

See #735 for a likely solution.


[darwin-framework-tool] Matter.framework conflict and unknown behavior will occur.

When Matter.framework is updated in the macOS build, it will cause a conflict with the built Matter.framework for Darwin-framework-tool. We need a way to specifically target our built Matter.framework and ignore the one in the system.

Tested this issue by updating the framework and running the tool. Since RPATH takes precedence the issue does not occur.


Added warning for lack of browser support for the ALLOW-FROM option

See here: https://developer.mozilla.org/en-US/docs/Web/HTTP/X-Frame-Options#Browser_compatibility

Great addition. Could you add a link to that MDN article in your PR?

Done. I also added a link to OWASP's Clickjacking Defense Cheat Sheet that explicitly says that you will have no clickjacking defense if used in unsupported browser.

Other than my tiny nitpick, this is great and ready to go.

Nitpicking is good. I'm still new to this markdown thing. Fixed now.

Thanks!

I'm building a list of contributors for a new Helmet.js website. Do you have a name and/or website you want me to use to credit you?

Thanks! I do not have a personal website, so you can just use my name Eli Golding. If you want you can also attribute www.qualitybath.com. This contribution was made on their time.

Added you to the list! Stay tuned for the full website.

Thanks!
Can you use my github profile for my website? Sorry didn't think of that before. Thanks again!

Will do!


Reordering Items when dragged?

Currently, when dragging items, their list order does not change (only their display position), so if they are bound to a source, the items order in the source does not change.
Is there a way to do this with the current version?
If not, is there a chance it will be possible one day?

In a nutshell, no.  You have to think slightly differently, and use the PositionMonitor to monitor the order, and do what you want with that.  Due to the nature of WPF binding, if items get swapped, then UI elements get removed/added, and the whole dragging process goes down the toilet.  It's the nature of dragging tabs and the WPF beast...

You made the point! I'll think differently (I needed the order to save layout, but I'll bind and use positions, which is also fine).


merge package from arlac77/npm-package-template

package.json

chore(package<EMAIL_ADDRESS>

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Updating download link

All download links should just go to release page for now. In the near future, we will have a fancy download page, and it will automatically update with the latest release links.

@ph2708 soon I will create a page like this: https://www.etlegacy.com/download


Allow nil map updates in select

Reference
https://elixirforum.com/t/ecto-merge-select-or-select-merge-into-a-possibly-nil-value-left-join/57133
Issue
When performing a query like this, it will fail with BadMapError if one of the main rows doesn't have an associated row:
from(c in Comment, left_join: p in Post, on: c.post_id == p.id, select: %{p | temp: c.text}

Proposal
I think we should allow this and simply return nil. It seems like the intuitive thing to expect and also there is some precedent. We will return nil in this situation
from(c in Comment, left_join: p in Post, on: c.post_id == p.id, select: map(p, [:field1, :field2, ...])

@aiwaiwa You could give this a try :).

Coool. Wow!


Dynamically update status and navigation bars

When materials toolbar colours are changed (e.g. when using "colour from cover") need to see if can get Android and iOS colours to change. This might be possible by dynamically changing the "theme-color"
e.g. from https://www.amitmerchant.com/change-theme-color-meta-tag-dynamically-using-javascript/
const themeColor = document.querySelector('meta[name="theme-color"]');

function changeThemeColorMeta() {
    const isDarkTheme = window.matchMedia('(prefers-color-scheme: dark)').matches;

    const color = isDarkTheme ? '#0d1117' : '#4d4d4d';

    themeColor.setAttribute('content', color);
}

...hmmm, Material already updates "theme-color" :(

Hmm... Tried, and failed. Looks like if I change colour with the UI then it updates, but not from javascript. Perhaps its the CSS file load that forces a refresh? For now this 'from cover' feature is hidden for iOS.


feat(avm)!: cleanup CALL

(Static)CALL now returns just the success bit
Properly returns U1 instead of U8
Removed FunctionSelector

Fixes #8998. Part of #9061.

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

#9551  👈
#9475
master

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

Join @fcarreiro and the rest of your teammates on  Graphite

Merge activity

Oct 29, 1:57 PM EDT: A user merged this pull request with Graphite.


DR-965 Set Domain Default

Changes:

Add call to service.
Add content.
Add html for make default domain.

WIP:

Confirmation text or icon for setting the default Domain.

Nice work @mvillaseco !
What do you think about to add the tests before merge it?


ghc-8.2 HasField, Type & *

Is it ok to expose these for ghc-8.2?

Yes, this should be good now that we have GHC 8.2. :+1:


Jeiss FIBSEM DAT file specs

The Janelia-Zeiss (Jeiss) FIBSEB DAT files have a 1 kilobyte metadata header.
@clbarnes has compiled a tabular list of the format of that header here:
https://github.com/clbarnes/jeiss-specs
For example, the latest header specification is at the following URL.
https://github.com/clbarnes/jeiss-specs/blob/main/specs/v9.tsv

Hey Mark,
Apologies for my delayed response here. This is the part where I reveal my technical ignorance. Is the .dat format a TIFF variant? So, are these offset values indicating where the relevant bytes are located in the linear byte stream? We may need to meet and talk about this. It's different from the other formats I've been focusing on, which consist of chunked arrays + JSON metadata files. We should be documenting these TIFF-based formats, too, but I'm not sure of the best way to go about that. We can just describe them in manuals, but that feels very... manual.

Is the .dat format a TIFF variant?

No. It's its own thing completely. Basically, the first kibibyte of the file stores a bunch of numbers according to the table I linked above. After that, the metadata follows.
Some example Python code to load it is here:
https://github.com/janelia-cellmap/fibsem-tools/blob/main/src/fibsem_tools/io/dat.py

Here's another example from @clbarnes which uses that table above to parse the data:
https://github.com/clbarnes/jeiss-convert/blob/main/jeiss_convert/hdf5.py

The dtype codes are described here:
https://numpy.org/doc/stable/reference/arrays.dtypes.html

Interesting. I guess how we want to document formats like this one depends on our overall strategy for supporting them. I can certainly start by just creating a folder that points to @clbarnes' documentation. Ideally, long-term, we should have some kind of data model that both microscopists and developers can understand so that they can make choices together based on a common language. I would be curious to know if @axtimwalde has thoughts about how we should be documenting/modeling formats like this one.
This issue is now on my radar. I will talk to @axtimwalde, @bogovicj, @StephanPreibisch, and others, and report back here.

Just for everyone's information, I will shortly be moving on from the field and so my level of investment in the vagaries of the Jeiss format will take a nosedive very soon. I'm happy to stay as nominal maintainer and merge PRs to the spec repo in the future (although won't be able to verify them myself), or to donate that repo to an appropriate person/ organisation.

@clbarnes this community was lucky to have you - thanks for everything you've done and wishing you the best!

@clbarnes Thank you for letting us know. Should we create a Github organization around the various "Jeiss" packages so that we can share the burdens of maintainership.

the burdens of maintainership?
which are what, exactly? I feel like a github org is a bit overkill for this file format.

@d-v-b If you want to read it another way, I would like to decrease the bus factor for the canonical URL of the repository.

I wouldn't call it canonical for as long as the people writing the format aren't contributing to it :sweat_smile: But yes, by all means adopt them! I can either donate them to the org, or you can fork them and I'll archive mine. I guess either is a bit tricky with the internals of jeiss-convert which uses jeiss-specs as a submodule.

I will look into the details after lunch (in an hour). I've done repository transfers a few times in the past. I generally prefer transfers over forking when possible since Github will do automatic redirects and it is clear which repository is the primary one.
To initiate a transfer, I can think of two initial steps. One would be adding @clbarnes to a Github organization like JaneliaSciComp. The other would be adding someone in JaneliaSciComp as an owner of the repository.
https://docs.github.com/en/repositories/creating-and-managing-repositories/transferring-a-repository#transferring-a-repository-owned-by-your-personal-account
More in an hour. Thanks for discussing.

Hi @clbarnes , after looking over the various procedures, I think the easiest first step would be to add me as a collaborator on the repository and elevate my privileges to Admin.

Let me know if you have any questions, and thank you for considering this.


AttributeError: 'Pool' object has no attribute 'children'

Trying to run a simple example with apply_async:
with closing(Pool(2)) as pool:
    funclist = []

    for item in list_of_items:
        f = pool.apply_async(myfunc, [arg1, item, arg2])
        funclist.append(f)

    results = []
    for f in funclist:
        result = f.get(timeout=900)
        results.append(result)

    pool.close()
    pool.terminate()

This throws the error: AttributeError: 'Pool' object has no attribute 'children'.

How is this solved? Does something like:
    with mp.Pool(processes=cfg_prod["n_workers"]) as pool:
            pool.apply_async(...)

work?

What's the question?
This issue was closed a year ago. It's not clear to me what the problem is. @ogencoglu if you want help please provide a minimum working example. Some code I can copy-paste to get exactly the error you are facing.


Catch Event "onClick" of Sign In Options?

Is there a callback that I could hook into for any of the sign in options?
I would like to be able to set my UI to loading while the login and redirect process occurs on the private route switching example.

Idea: in uiConfig.callbacks add onAuthAttempt or something similar, a callback function that has the param choice or something similar available.


Setup Kanban Board

Setting up Kanban board for Recipe project.
Link to our Kanban board https://github.com/Theodoraldo/Recipe_app/projects/1
In this project we are two:
@Theodoraldo
@Zohra-Neda

Hi @Theodoraldo  & @Zohra-Neda ,
Your project is complete!  There is nothing else to say other than... it's time to merge it :shipit:
Congratulations! 🎉

Highlights

[ ] All cards are correctly assigned ✔
[ ] All the required columns are present ✔

Optional suggestions
Every comment with the [OPTIONAL] prefix won't stop the approval of this PR. However, I strongly recommend you to take them into account as they can make your code better. Some of them were simply missed by the previous reviewer and addressing them will really improve your application.
Cheers and Happy coding!👏👏👏
Feel free to leave any questions or comments in the PR thread if something is not 100% clear.
Please, remember to tag me in your question so I can receive the notification.

As described in the Code reviews limits policy  you have a limited number of reviews per project (check the exact number in your Dashboard). If you think that the code review was not fair, you can request a second opinion using this form.


Title not displayed from some sources

This is excellent, thanks for sharing!
When playing from some streaming sources, only the "artist" is being displayed but not the "title" (screenshot below) and I'm wondering how to fix that. The streaming source is TuneIn (New) and here's an example output of http://<IP_ADDRESS>:15005/kitchen/state
Please let me know if there's an easy fix. Thank you in advance!
{
"volume":9,
"mute":false,
"equalizer":{
"bass":2,
"treble":2,
"loudness":true
},
"currentTrack":{
"artist":"Classic Hits",
"title":"Lisa Lisa And Cult Jam - Head To Toe",
"albumArtUri":"/getaa?s=1&u=x-sonosapi-stream%3as123456%3fsid%3d333%26flags%3d1234%26sn%3d9",
"duration":0,
"uri":"x-sonosapi-stream:s123456?sid=333&flags=8232&sn=9",
"trackUri":"x-sonosapi-stream:s123456?sid=333&flags=8232&sn=9",
"type":"radio",
"stationName":"Classic Hits",
"absoluteAlbumArtUri":"http://<IP_ADDRESS>:1400/getaa?s=1&u=x-sonosapi-stream%3as123456%3fsid%3d333%26flags%3d1234%26sn%3d9"
},
"nextTrack":{
"artist":"",
"title":"",
"album":"",
"albumArtUri":"",
"duration":0,
"uri":""
},
"trackNo":1,
"elapsedTime":35,
"elapsedTimeFormatted":"00:00:35",
"playbackState":"PLAYING",
"playMode":{
"repeat":"none",
"shuffle":false,
"crossfade":false
}

Following up here to say I asked ChatGPT for a solution. I don't think it's worth including here because it only works with the formatting returned by TuneIn radio on Sonos. They include the station name as the artist and then combine the artist's name and song title as the song.


[Refactor]: change tagList to tags

Description
The plural format currently used for tag is tagList.
It should match the plural format of article and comment (ie articles and comments).
Solution
For consistency, the plural format for tag should be tags.

discontinued task


PTV-1868: Remove --insecure from curl command in Dockerfile

Presumably no longer needed with the newer base image. Tests pass locally

Hmm, the mmseqs download keeps failing but it worked for me locally, maybe it's just unreliable?

Yeah, I'm seeing it fail on my laptop as well

And I tried building the docker container again locally and it works

Hmm, I don't know what's going on. It's repeatedly failing here, but I can build the container locally (at least some of the time, it's not reliable there either).

I tried registering in in CI, first time failed, 2nd time succeeded. Seems like that site is just flaky...? Although I don't know why it fails all the time for GHA


Use the pending resolution when rendering

This is necessary for rootless compositors, where the output resolution
can be changed by a client at any time.  This requires wlr_scene to have
access to output_pending_resolution, which other renderers will need
as well.  Make that function a public API.

wlroots has migrated to gitlab.freedesktop.org. This pull request has been moved to:
https://gitlab.freedesktop.org/wlroots/wlroots/-/merge_requests/3292


Add TTY console support

Add console functions to inc/ext and src/ext
Add stb_sprintf.h to inc/ext/stb for optional use
Add console sample
Update doxyfile

The Codacy issues are false positives. There are some macros involved that are not correctly detected.


Crash when opening project converted from Godot 3.5.2

Godot version
4.1.1
System information
Godot v4.1.1.stable unknown - EndeavourOS #1 SMP PREEMPT_DYNAMIC Wed, 19 Jul 2023 19:19:38 +0000 - Vulkan (Mobile) - dedicated NVIDIA GeForce GTX 1070 (nvidia; 535.86.05) - Intel(R) Core(TM) i5-8400 CPU @ 2.80GHz (6 Threads)
Issue description
After automatically converting my project to godot 4.1.1, it crashes upon trying to open the project. It seems something specific with this project, but I don't know what.
This is what the terminal spits out when I try to open it:
Vulkan API 1.3.242 - Forward+ - Using Vulkan Device #0: NVIDIA - NVIDIA GeForce GTX 1070

WARNING: Blend file import is enabled in the project settings, but no Blender path is configured in the editor settings. Blend files will not be imported.
     at: _editor_init (modules/gltf/register_types.cpp:73)
ERROR: Condition "!preset.is_valid()" is true. Continuing.
   at: load_config (editor/export/editor_export.cpp:242)
SCRIPT ERROR: Parse Error: Member "velocity" redefined (original in native class 'CharacterBody2D')
          at: GDScript::reload (res://Enemies/av-stopper.gd:4)
ERROR: Cannot create tile. The tile is outside the texture or tiles are already present in the space the tile would cover.
   at: create_tile (scene/resources/tile_set.cpp:4138)
ERROR: The TileSetAtlasSource atlas has no tile at (0, 16).
   at: get_tile_data (scene/resources/tile_set.cpp:4540)

================================================================
handle_crash: Program crashed with signal 11
Engine version: Godot Engine v4.1.1.stable.arch_linux
Dumping the backtrace. Please include this when reporting the bug to the project developer.
[1] /usr/lib/libc.so.6(+0x39ab0) [0x7fa179162ab0] (??:0)
[2] godot(+0x3cb60ff) [0x564649a470ff] (??:?)
[3] godot(+0x3cc94d9) [0x564649a5a4d9] (??:?)
[4] godot(+0x3ccdf90) [0x564649a5ef90] (??:?)
[5] godot(+0x4c080bb) [0x56464a9990bb] (??:?)
[6] godot(+0x3b39d86) [0x5646498cad86] (??:?)
[7] godot(+0x3b3cfe0) [0x5646498cdfe0] (??:?)
[8] godot(+0x492482a) [0x56464a6b582a] (??:?)
[9] godot(+0x492bc83) [0x56464a6bcc83] (??:?)
[10] godot(+0x4971b81) [0x56464a702b81] (??:?)
[11] godot(+0x49729a8) [0x56464a7039a8] (??:?)
[12] godot(+0x4f1a7cd) [0x56464acab7cd] (??:?)
[13] godot(+0x1ff93d3) [0x564647d8a3d3] (??:?)
[14] godot(+0x4cd9461) [0x56464aa6a461] (??:?)
[15] godot(+0x1eede40) [0x564647c7ee40] (??:?)
[16] godot(+0x4b94e38) [0x56464a925e38] (??:?)
[17] godot(+0x306a41c) [0x564648dfb41c] (??:?)
[18] godot(+0x30afabd) [0x564648e40abd] (??:?)
[19] godot(+0x30b7717) [0x564648e48717] (??:?)
[20] godot(+0x11da0c7) [0x564646f6b0c7] (??:?)
[21] godot(+0x11758d5) [0x564646f068d5] (??:?)
[22] /usr/lib/libc.so.6(+0x23850) [0x7fa17914c850] (??:0)
[23] /usr/lib/libc.so.6(__libc_start_main+0x8a) [0x7fa17914c90a] (??:0)
[24] godot(+0x11843e5) [0x564646f153e5] (??:?)
-- END OF BACKTRACE --
================================================================
Aborted (core dumped)

Steps to reproduce
The included project files have already been converted, so just open it up in godot 4.1.1 and watch it crash.
Minimal reproduction project
Total-mal.zip

The crash happens here:
https://github.com/godotengine/godot/blob/6758a7f8c07d1f4c8ec4f052ded6d26402967ebe/scene/resources/tile_set.cpp#L5213-L5217
Seems the method is called on a nullpointer during the execution of _compatibility_conversion.
Could you provide the original project as well?

Anti-malfunction-wont-convert.zip
These are the project files before conversion. I used the Full Conversion option when attempting to convert.

The crash results from the region used for your tileset. (The one that is internally stored in main menu.tscn). There the texture region is larger than the texture. Due to the rewrite of the tileset system this crashes in godot 4, because no tile data can exist autside the texture in the new implementation. The large region does not seem to fullfill any purpose and looks like an accident to me, so you should be able to startup godot after setting the region to the texture size. (You will still need to do some work to fixup your tilemap e.g. the tilesize will default back to 16 in Godot 4. But this is beyond the scope of this issue.)
Not sure whether we need actual code changes here. Maybe tile sources could crop their region to the texture size on conversion. This would still break the tileset but would atleast prevent a crash which seems like a better user experience. But the case seems very rare so it might not be worth the extra code.


Removed AModel from models.py

Unless I'm missing something severe I can't see a reason for the AModel model included in models.py? It creates an unnecessary table when you run syncdb which, while not really an issue, is a small annoyance.
It isn't a requirement to have a model in the file so I have removed it and the tests still run fine (as they have their own models file); anyway, here's the commit if you wish to pull it.
Thanks for your work on this by the way, it's brilliant.
cheers, Darian

I'm following flupke , as he has added very useful support for taggit custom "through" custom model

Closing due to deadness.


SL7.2: stat /usr/sbin/init: no such file or directory, but CentOS 7 works

Hello @ringohub ,
I've setup SL6.8 and SL7.2 builds from docker using your repo and TravisCI here on github from one of my projects. I am using a script to setup the container for CentOS 6, CentOS 7, and now SL6.8 and SL7.2. The latter -- SL7.2 fails, e.g., here. It uses the same command between CentOS 7 and SL7, but only works for the former, but not the latter complaining /usr/sbin/init is not found:
docker run --privileged -d -ti -e container=docker -v /sys/fs/cgroup:/sys/fs/cgroup -v /home/travis/build/OpenISS/OpenISS:/OpenISS:rw ringo/scientific:7.2 /usr/sbin/init
4d822c0886ac5afd401fb88f3375e14aa55580149408941e14ea83e022c6051f
docker: Error response from daemon: oci runtime error: container_linux.go:265: starting container process caused "exec: \"/usr/sbin/init\": stat /usr/sbin/init: no such file or directory".

The command being using for both CentOS7 and SL7:

	docker run \
		--privileged -d -ti -e "container=docker"  \
		-v /sys/fs/cgroup:/sys/fs/cgroup \
		-v `pwd`:$DOCKERPATH:rw ${OS_TYPE}:${OS_VERSION_PREFIX}${OS_VERSION}${OS_VERSION_SUFFIX} /usr/sbin/init

where the variables resolve as illustrated in the error above. The image is pulled but not started. I am not sure why SL7 image does not have /usr/sbin/init and CentOS 7 does.
SL6 works properly (and so do CentOS 6 and CentOS 7). The complete docker setup script is here. It's invoked by Travis-CI as shown in here.
How do I go about resolving the missing /usr/sbin/init issue on your SL7.2 build? Also, is there SL7.3 planned?

Closing this as the originating issue is closed as well.


Instant zero elimination

closes #1639
Not really a change, but it works for now. :)

@popara @Anton-4 Are you still interested in this PR? (I'm just nagging stale PRs/branches, and this is almost a year old now.)

Thank you for this PR, but it will need to be reimplemented after the editor's rewrite. Happy to revisit!


<EMAIL_ADDRESS>

Implemented a Practical Example of Hexagonal Architecture in Java using a Spring Boot RESTful application.

Closing the final pull request


Hard linking for fs/copy

Is your feature request related to a problem? Please describe.
My business solution requires copies of really big directories to be made. This turns out to be a pretty big waste of disk space when a filesystem feature like hard links isn't being used.
Describe the solution you'd like
Add a hardlink: boolean; field to the options.
Describe alternatives you've considered
Currently I just copied the module and added this feature myself.

Can you please share a fork containing the code you came up with?


MINOR: move RecordReader from org.apache.kafka.tools (client module) to org.apache.kafka.tools.api (tools-api module)

from https://github.com/apache/kafka/pull/13393#issuecomment-1483899781
org.apache.kafka.tools is existent in both tools and client module, and it causes split packages (https://www.logicbig.com/tutorials/core-java-tutorial/modules/split-packages.html)
test aggregatedJavadoc and the output is shown below.

Committer Checklist (excluded from commit message)

[ ] Verify design and implementation
[ ] Verify test coverage and CI build status
[ ] Verify documentation (including upgrade notes)

@ijuma ping for review :)
I will update KIP if this change is acceptable

@ijuma @mimaison Could you take a look? move RecordReader to the same package as MessageFormatter

I have test docsJar, aggregatedJavadoc and releaseTarGz

Why does core depend on tools-api?

ConsoleProducer is still in core module. #13214 can remove the dependency after it moves ConsoleProducer from core to tool

https://cwiki.apache.org/confluence/pages/viewpage.action?pageId=158866569 has been updated


ValueError: numpy.ndarray has the wrong size

Hi,
When I execute line:
import lap
I got following error:
from ._lapjv import (
File "__init__.pxd", line 199, in init lap._lapjv
ValueError: numpy.ndarray has the wrong size, try recompiling. Expected 80, got 96
Environment:
python 3.8
numpy==1.19.5

After testing I found out that installing the lap version 0.5.0 is not on the pip. Running following command solved my issue:
pip install git+https://github.com/gatagat/lap.git
Maybe it could be also solved by adjusting numpy version. But I have noticed that upgrading it to the numpy 1.24 would yeld np.long error because np.long has been marked deprecated in the 1.20 release and removed in the 1.24. Source: https://stackoverflow.com/questions/76389395/attributeerror-module-numpy-has-no-attribute-long


Restructure middleware

Going to nix middleware and go with onCreate and onUpdate properties.
As part of this initiative, Provider.props.updates will be renamed. (Thinking "definition" instead of "updates")

Went with Provider.props.stateDef


Last used time not updating submitting via API

Expected Behavior
Last used time for a comfy backend should reflect when it was last used by any means
Actual Behavior
Last used is not updating
Steps to Reproduce
submit a job via api
Debug Logs
2024-10-01 13:55:43.880 [Info] Creating new admin session 'local' for <IP_ADDRESS> (forwarded-for: <IP_ADDRESS>)
2024-10-01 13:55:44.163 [Debug] Was not able to redirect Comfy backend direct prompt request
2024-10-01 13:55:44.166 [Debug] [ComfyUI-0/STDERR] got prompt
2024-10-01 13:55:44.180 [Debug] ComfyUI redirection failed (outsocket): System.Net.WebSockets.WebSocketException (0x80004005): The WebSocket is in an invalid state ('Closed') for this operation. Valid states are: 'Open, CloseReceived'
at System.Net.WebSockets.ManagedWebSocket.SendAsync(ReadOnlyMemory1 buffer, WebSocketMessageType messageType, WebSocketMessageFlags messageFlags, CancellationToken cancellationToken) --- End of stack trace from previous location ---    at SwarmUI.Builtin_ComfyUIBackend.ComfyUIRedirectHelper.<>c__DisplayClass7_3.<<ComfyBackendDirectHandler>b__6>d.MoveNext() in /swarm/StableSwarmUI/src/BuiltinExtensions/ComfyUIBackend/ComfyUIRedirectHelper.cs:line 297 2024-10-01 13:55:44.180 [Debug] ComfyUI redirection failed (outsocket): System.Net.WebSockets.WebSocketException (0x80004005): The WebSocket is in an invalid state ('Closed') for this operation. Valid states are: 'Open, CloseReceived'    at System.Net.WebSockets.ManagedWebSocket.SendAsync(ReadOnlyMemory1 buffer, WebSocketMessageType messageType, WebSocketMessageFlags messageFlags, CancellationToken cancellationToken)
--- End of stack trace from previous location ---
at SwarmUI.Builtin_ComfyUIBackend.ComfyUIRedirectHelper.<>c__DisplayClass7_3.<b__6>d.MoveNext() in /swarm/StableSwarmUI/src/BuiltinExtensions/ComfyUIBackend/ComfyUIRedirectHelper.cs:line 297
2024-10-01 13:55:44.543 [Debug] [ComfyUI-0/STDERR] model weight dtype torch.float16, manual cast: None
2024-10-01 13:55:44.549 [Debug] [ComfyUI-0/STDERR] model_type EPS
2024-10-01 13:55:45.279 [Debug] [ComfyUI-0/STDERR] Using pytorch attention in VAE
2024-10-01 13:55:45.281 [Debug] [ComfyUI-0/STDERR] Using pytorch attention in VAE
2024-10-01 13:55:46.300 [Debug] [ComfyUI-0/STDERR] Requested to load SDXLClipModel
2024-10-01 13:55:46.300 [Debug] [ComfyUI-0/STDERR] Loading 1 new model
2024-10-01 13:55:46.664 [Debug] [ComfyUI-0/STDERR] loaded completely 0.0 1560.802734375 True
2024-10-01 13:55:46.853 [Debug] [ComfyUI-0/STDERR] Requested to load SDXL
2024-10-01 13:55:46.853 [Debug] [ComfyUI-0/STDERR] Loading 1 new model
2024-10-01 13:55:47.842 [Debug] [ComfyUI-0/STDERR] loaded completely 0.0 4897.0483474731445 True
2024-10-01 13:55:47.863 [Debug] [ComfyUI-0/STDERR]
2024-10-01 13:55:48.009 [Debug] [ComfyUI-0/STDERR]   0%|          | 0/20 [00:00<?, ?it/s]
2024-10-01 13:55:48.143 [Debug] [ComfyUI-0/STDERR]   5%|▌         | 1/20 [00:00<00:02,  6.83it/s]
2024-10-01 13:55:48.291 [Debug] [ComfyUI-0/STDERR]  10%|█         | 2/20 [00:00<00:02,  7.20it/s]
2024-10-01 13:55:48.434 [Debug] [ComfyUI-0/STDERR]  15%|█▌        | 3/20 [00:00<00:02,  6.97it/s]
2024-10-01 13:55:48.572 [Debug] [ComfyUI-0/STDERR]  20%|██        | 4/20 [00:00<00:02,  7.00it/s]
2024-10-01 13:55:48.710 [Debug] [ComfyUI-0/STDERR]  25%|██▌       | 5/20 [00:00<00:02,  7.09it/s]
2024-10-01 13:55:48.847 [Debug] [ComfyUI-0/STDERR]  30%|███       | 6/20 [00:00<00:01,  7.13it/s]
2024-10-01 13:55:48.983 [Debug] [ComfyUI-0/STDERR]  35%|███▌      | 7/20 [00:00<00:01,  7.19it/s]
2024-10-01 13:55:49.122 [Debug] [ComfyUI-0/STDERR]  40%|████      | 8/20 [00:01<00:01,  7.23it/s]
2024-10-01 13:55:49.260 [Debug] [ComfyUI-0/STDERR]  45%|████▌     | 9/20 [00:01<00:01,  7.22it/s]
2024-10-01 13:55:49.395 [Debug] [ComfyUI-0/STDERR]  50%|█████     | 10/20 [00:01<00:01,  7.24it/s]
2024-10-01 13:55:49.531 [Debug] [ComfyUI-0/STDERR]  55%|█████▌    | 11/20 [00:01<00:01,  7.28it/s]
2024-10-01 13:55:49.666 [Debug] [ComfyUI-0/STDERR]  60%|██████    | 12/20 [00:01<00:01,  7.31it/s]
2024-10-01 13:55:49.804 [Debug] [ComfyUI-0/STDERR]  65%|██████▌   | 13/20 [00:01<00:00,  7.33it/s]
2024-10-01 13:55:49.940 [Debug] [ComfyUI-0/STDERR]  70%|███████   | 14/20 [00:01<00:00,  7.31it/s]
2024-10-01 13:55:50.078 [Debug] [ComfyUI-0/STDERR]  75%|███████▌  | 15/20 [00:02<00:00,  7.32it/s]
2024-10-01 13:55:50.216 [Debug] [ComfyUI-0/STDERR]  80%|████████  | 16/20 [00:02<00:00,  7.29it/s]
2024-10-01 13:55:50.350 [Debug] [ComfyUI-0/STDERR]  85%|████████▌ | 17/20 [00:02<00:00,  7.29it/s]
2024-10-01 13:55:50.486 [Debug] [ComfyUI-0/STDERR]  90%|█████████ | 18/20 [00:02<00:00,  7.33it/s]
2024-10-01 13:55:50.623 [Debug] [ComfyUI-0/STDERR]  95%|█████████▌| 19/20 [00:02<00:00,  7.34it/s]
2024-10-01 13:55:50.623 [Debug] [ComfyUI-0/STDERR] 100%|██████████| 20/20 [00:02<00:00,  7.34it/s]
2024-10-01 13:55:50.623 [Debug] [ComfyUI-0/STDERR] 100%|██████████| 20/20 [00:02<00:00,  7.25it/s]
2024-10-01 13:55:50.628 [Debug] [ComfyUI-0/STDERR] Requested to load AutoencoderKL
2024-10-01 13:55:50.628 [Debug] [ComfyUI-0/STDERR] Loading 1 new model
2024-10-01 13:55:50.657 [Debug] [ComfyUI-0/STDERR] loaded completely 0.0<PHONE_NUMBER>2253418 True
2024-10-01 13:55:51.001 [Debug] [ComfyUI-0/STDERR] Prompt executed in 6.83 seconds
Other
I believe this is happening in ComfyUIRedirectHelper.cs either its not finding the created user (in this case local) even though it created a new session, and or once it does the default redirect its not updating the necessary data

either its not finding the created user (in this case local)
You're bypassing Swarm's API entirely, so Swarm user is irrelevant - you're hitting the comfy API, and it's looking for a Comfy user, ie a websocket connection, which presumably you just don't have? I see in your log a failed websocket connection after the prompt, rather than before as you'd want. The websocket connection is where you set a client id (connect to /ws?clientId={id}) which you'd want to send along to the /prompt call as well, to link the socket and the request.
This is why you see in logs Was not able to redirect Comfy backend direct prompt request - you have no valid client id, so it has no idea how to link the request to anyone or anything.
The commit above should handle this case better

Ok, were I'm not establishing a WS connection - due to architecture constraints at the moment - and instead am polling for status, so persumably this issue will go away if I switch to WS instead of polling

A lot of things in Comfy core API work better if you use a websocket.
The above commit should fix the issue regardless.
If you want a non-WS non-polling API, Swarm's own API - well normally uses Websocket too, but also has a direct HTTP API that simple delays the HTTP reply until there's a generation output to give back


typescript-graphql-request generated code uses GraphQLError without importing it

Which packages are impacted by your issue?
@graphql-codegen/typescript-graphql-request
Describe the bug
Generated code uses GraphQLError without importing it.
Your Example Website or App
https://stackblitz.com/edit/github-jjt8y7?file=types.ts%3AL61
Steps to Reproduce the Bug or Issue
Please refer to the types.ts in the repro.
In line 62, you can see GraphQLError used but no import for it.
Expected behavior
There should be a import statement for GraphQLError
Screenshots or Videos

Platform

OS: macOS
NodeJS: 18.18.2
graphql version: 16.8.1
@graphql-codegen/tyepscript-graphql-request version: 6.0.0

Codegen Config File
import { CodegenConfig } from '@graphql-codegen/cli';

const config: CodegenConfig = {
  schema: 'schema.graphql',
  documents: 'document.graphql',
  generates: {
    'types.ts': {
      plugins: [
        'typescript',
        'typescript-operations',
        'typescript-graphql-request',
      ],
      config: {
        rawRequest: true,
      },
    },
  },
};

export default config;

Additional context
No response

In the meantime we can use add plugin to prepend the import statement.
      plugins: [
        'typescript-operations',
        'typescript-graphql-request',
        {
          add: {
            content: [
              '/* eslint-disable */',
              `import type { GraphQLError } from 'graphql'`,
            ],
          },
        },
      ],

<EMAIL_ADDRESS>


ReferenceError: htmlWebpackPlugin is not defined

Version
4.5.12
Reproduction link
https://github.com/xcgs123/vuecli-vue-demo
Environment info
"vue": "^3.0.11"
"@vue/cli-service": "~4.5.0"

Steps to reproduce
npm run dev
What is expected?
success
What is actually happening?
vue.config.js
it will be found when  add "templateParameters" in  pages
it may be htmlWebpackPlugin 'version wrong

Try to delete these codes.
If you want to use latest html-webpack-plugin, you can try alpha version of @vue/cli


Fix: Add dedicated integration tests targets for CI

What does this PR do?

This PR adds two new targets to the makefile:

One to run only the container integration tests (test-integration-container)
One to run only the host integration tests (test-integration-host)

Motivation

Release pressure on CI by having two targets that can be run in separate jobs.
More

[ ] ~Added/updated tests~
[ ] ~Added/updated documentation~

One test to rule them all, and in the darkness bind them?


i cant start colab

Traceback (most recent call last):
File "/content/gdrive/MyDrive/sd/stable-diffusion-webui/webui.py", line 13, in
initialize.imports()
File "/content/gdrive/MyDrive/sd/stable-diffusion-webui/modules/initialize.py", line 21, in imports
import gradio  # noqa: F401
File "/usr/local/lib/python3.10/dist-packages/gradio/init.py", line 3, in
import gradio.components as components
File "/usr/local/lib/python3.10/dist-packages/gradio/components.py", line 56, in
from gradio.blocks import Block, BlockContext
File "/usr/local/lib/python3.10/dist-packages/gradio/blocks.py", line 28, in
from gradio import (
ImportError: cannot import name 'wasm_utils' from partially initialized module 'gradio' (most likely due to a circular import) (/usr/local/lib/python3.10/dist-packages/gradio/init.py)

deleting sd and re downloading it solved the issue for me.

use the latest colab

I have the same problem. I've deleted sd folder and reran the latest colab, but the error is the same.


Improve unhandled exception logging in Cli.run

Only log the exception message if there is one and
always print the exception stack trace when Cli.run
receives an unhandled exception when invoking Command.run.
Fixes #2003

Coverage increased (+0.004%) to 84.5% when pulling cad43169657b0ba5352a1beac4fa5cd81a66d21c on anonyome:tbartley/improve-unhandled-application-exception-logging into 5677f8fa843575628b233f00f2fbc57b7c6f9321 on dropwizard:master.


[SPARK-28834][SQL] Support sessionInitStatement in dataframe jdbc write path

What changes were proposed in this pull request?

After each database session is opened to the remote DB and before starting to read data, this option sessionInitStatement  executes a custom SQL statement to do session initialization, this change enables this behavior for writing data.
Why are the changes needed?

Session initialization also need for writing data out of spark
Does this PR introduce any user-facing change?

no
How was this patch tested?

it

Test build #111953 has started for PR 26100 at commit 46e9081.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/16969/
Test PASSed.

Test build #111954 has started for PR 26100 at commit 828d1ee.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/16970/
Test PASSed.

Test build #111954 has finished for PR 26100 at commit 828d1ee.

This patch fails due to an unknown error code, -9.
This patch merges cleanly.
This patch adds no public classes.

Test build #111953 has finished for PR 26100 at commit 46e9081.

This patch fails due to an unknown error code, -9.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/111954/
Test FAILed.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/111953/
Test FAILed.

retest this please

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/16979/
Test PASSed.

Test build #111965 has started for PR 26100 at commit 828d1ee.

Test build #111965 has finished for PR 26100 at commit 828d1ee.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/111965/
Test PASSed.

Hi @yaooqinn , this is still a missing feature. Is there any reason not to continue with this pull request? (maybe more tests for more platform will be appreciated, I can help with that if it is needed)


Parsl profiling

We should add functionality for profiling parsl itself-- reporting how much of a script's runtime is spent on parsl-related overhead.

This should be added to monitoring. cc'ing @ZhuozhaoLi


Use @api3/eslint-plugin-commons for eslint

A number of new rules require parserServices to be generated which results in the error:
You must therefore provide a value for the "parserOptions.project" property for @typescript-eslint/parser

Currently it isn't specified:
https://github.com/api3dao/airnode/blob/c43d981905ae27562e2488c6211dda4304f2c5ed/.eslintrc.js#L9-L12
Specifying this, however, isn't as easy as other repos given this monorepo structure and, I believe, how the packages' tsconfig.json files use references, potentially confusing ESLint on how to find the files.
Disabling these few rules allows it to run, but with quite a number of errors:
✖ 2921 problems (2921 errors, 0 warnings)
399 errors and 0 warnings potentially fixable with the `--fix` option.

Yeah, I think for Airnode we should disable many of the non-fixable rules just to make the migration simpler. These can always be re-enabled later as separate issues.


world environment scrolls to top when i enable options

Tested versions
-happens in godot 4.3 stable, does not happen in 4.2 stable and 4.4 dev 3
System information
windows 10 - godot 4.3 stable  - vulkan forward+
Issue description
world environment scrolls to top when i enable ssr,ssao,ssil.. etc.
Steps to reproduce
add world environment, select environment, enable any option, then it scrolls to top
Minimal reproduction project (MRP)
i upload video
https://github.com/user-attachments/assets/1c57658d-d0d4-41ab-83d3-ef8049ef876b
https://youtu.be/4KWNeuoB_M8

Sounds like:

https://github.com/godotengine/godot/issues/96489

Which has been fixed and will be cherry picked for a future 4.3.x
Closing as, as you say, the same issue is fixed in 4.4 so should be the same issue and the same solution should solve that as well, when cherry picked


ARTEMIS-2451 eliminate knownDestinations cache

This is based on the work from #2796.  The connection tracking in that PR was removed as it's not necessary since we still track temporary queues.

+1

@jbertram Can I ask you to add a test with this PR?
One that shows users doing this anti pattern:
     int N = 100; // or any arbitrary number > 1
     for (int i = 0; i < N; i++) {
          Queue queue = session.createQueue("myQueue");
          MessageProducer prod = session.createProducer(queue);
          prod.send(....);
          prod.close();
     }

If an user did such anti-pattern, it should suffer the consequences of the extra roundtrip of course, and we should not optimize for it.
I just want to make sure it wouldn't fail for any reason.


Updating GCC High Support List

https://github.com/MicrosoftDocs/OfficeDocs-SkypeForBusiness/issues/3306

@nenonix @JohanFreelancer9 Copy editing is needed for this PR. Thanks.

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

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

@CarolynRowe @MicrosoftHeidi can you verify this PR for me?

@David-Rosenthal - in Heidi's absence, could you verify this GCC info for us please?


Patch release v1.6.4

What does this PR do?
As usual.

Need #13014 as well.

@kaushikb11 Inference mode is in 1.7 only: #12715

@Borda Quantization performance tests are failing in the LTS job. Do you think this is just flakiness?

@Borda Quantization performance tests are failing in the LTS job. Do you think this is just flakiness?

I do not see any reason for such failure so I believe it is just flakiness

is it still wIP?

The PR is ready but blocked by #13152 (which happens on master too)

@carmocca can you include this as well  #13067


Convert PointerEvent's toString to Diagnosticable

Rather than building a description string by hand, use Diagnosticable
properties. Simplifies output, since many fields have uninteresting
default values in many cases, and makes it easier to add more loggable
properties in subclasses.

I used my best judgment on when to use default values, and on some formatting specifics (e.g., DiagnosticsProperty for down instead of a FlagProperty), but someone who uses these log strings more might have suggestions for tweaks.

/cc @jacob314

@goderbauer Hixie suggested you for review, but feel free to redirect if there's a better candidate.

I'm on vacation this week so I will be a bit slow to respond. Great to see
more classes use diagnosticable. It helps a lot for our GUI debugging tools.
On Wed, Feb 13, 2019, 1:35 PM stuartmorgan<EMAIL_ADDRESS>wrote:

@goderbauer https://github.com/goderbauer Hixie suggested you for
review, but feel free to redirect if there's a better candidate.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/flutter/flutter/pull/27904#issuecomment-463420089,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABK4PNbGqvjBAPRxOyMUmVxz1apsHPm8ks5vNKEwgaJpZM4a6eem
.

Please follow the process at https://github.com/flutter/flutter/wiki/Tree-hygiene#handling-breaking-changes

Please follow the process [...]

Ah sorry, when you said not to worry about the breaking change because nobody would have done it I thought that meant it didn't need the full process. Will do when I have time.

I would be really surprised if any users are broken by refining toStringFull to not include irrelevant information. My guess would be that irrelevant information was previously included in it just because it is too inconvenient to filter out properties matching the default value when using a single large string interpolation template but someone more familiar with the API would have a better idea.

Updated; this should now cause only minor tweaks in terms of output behavior from calling toString and toStringFull. I believe all comments are now addressed.

The breaking change process is underway. Meanwhile, @goderbauer @Hixie any other comments/concerns with the current version of the patch?


Azure ML Team Management Sample Issue

Section link1, link2, link3:

Reason:
This sample demonstrates how to Deletes a project.
Suggestion:
Update function name and file name to projectDelete.
@joheredi, @achandmsft , @mayurid and @qiaozha  for notification.

Label prediction was below confidence level 0.6 for Model:ServiceLabels: 'Docs:0.23064412,Machine Learning:0.11325477,Azure.Core:0.050880525'

Hi @v-jiaodi,
This issue has been fixed.
Here is the package link<EMAIL_ADDRESS>

@v-jiaodi Can we close this issue now ?

Yes. I will close it.


fix-metabase-health-check-dns-value-prod

This PR is to fix the metabase health check DNS value in prod
Terraform plan output for prod
terraform plan -target=module.application_insights.azurerm_application_insights_web_test.metabase_test
  # module.application_insights.azurerm_application_insights_web_test.metabase_test will be updated in-place
  ~ resource "azurerm_application_insights_web_test" "metabase_test" {
      ~ configuration           = <<-EOT
            <WebTest Name="" Id="" Enabled="True" CssProjectStructure="" CssIteration="" Timeout="60" WorkItemIds="" xmlns="http://microsoft.com/schemas/VisualStudio/TeamTest/2010" Description="" CredentialUserName="" CredentialPassword="" PreAuthenticate="True" Proxy="default" StopOnError="False" RecordedResultFile="" ResultsLocale="">
                  <Items>
          -         <Request Method="GET" Version="1.1" Url="https://prod.prime.cdc.gov/metabase/api/health" ThinkTime="0" Timeout="60" ParseDependentRequests="False" FollowRedirects="True" RecordResult="True" Cache="False" ResponseTimeGoal="0" Encoding="utf-8" ExpectedHttpStatusCode="200" ExpectedResponseUrl="" ReportingName="" IgnoreHttpStatusCode="False"/>
          +         <Request Method="GET" Version="1.1" Url="https://prime.cdc.gov/metabase/api/health" ThinkTime="0" Timeout="60" ParseDependentRequests="False" FollowRedirects="True" RecordResult="True" Cache="False" ResponseTimeGoal="0" Encoding="utf-8" ExpectedHttpStatusCode="200" ExpectedResponseUrl="" ReportingName="" IgnoreHttpStatusCode="False"/>
                  </Items>
                </WebTest>
        EOT
        id                      = "/subscriptions/7d1e3999-6577-4cd5-b296-f518e5c8e677/resourceGroups/prime-data-hub-prod/providers/Microsoft.Insights/webTests/pdhprod-metabase-health-check"
        name                    = "pdhprod-metabase-health-check"
        tags                    = {
            "environment"
                     = "prod"
            "hidden-link:/subscriptions/7d1e3999-6577-4cd5-b296-f518e5c8e677/resourceGroups/prime-data-hub-prod/providers/Microsoft.Insights/components/pdhprod-appinsights" = "Resource"
        }
        # (10 unchanged attributes hidden)
    }

Plan: 0 to add, 1 to change, 0 to destroy.
Terraform plan output for staging
terraform plan -target=module.application_insights.azurerm_application_insights_web_test.metabase_test
No changes. Your infrastructure matches the configuration.
Terraform has compared your real infrastructure against your configuration and found no differences, so no changes are needed.
╷
│ Warning: Resource targeting is in effect
│
│ You are creating a plan with the -target option, which means that the result of this plan may not represent all of the changes requested by the
│ current configuration.
│
│ The -target option is not for routine use, and is provided only for exceptional situations such as recovering from errors or mistakes, or when
│ Terraform specifically suggests to use it as part of an error message.
╵

Please do an apply, thank you.


faucet not show to nightl wallet

*Describe the bug
for niglty wallet to claim faucet to mevm network
To Reproduce
Steps to reproduce the behavior:

Go to '...'
Click on '....'
Scroll down to '....'
See error

Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

OS: [e.g. iOS]
Browser [e.g. chrome, safari]
Version [e.g. 22]

Smartphone (please complete the following information):

Device: [e.g. iPhone6]
OS: [e.g. iOS8.1]
Browser [e.g. stock browser, safari]
Version [e.g. 22]

Additional context
Add any other context about the problem here.

Please reach out to Nightly team.


QG checks (Release 24.05)

QG checks
Please keep this issue open until QG is concluded and will be managed by the Issue Creator!
We will inform you about finding and proposals in separated issues, this issue here is for the Overview of the Checks!
Please keep this issue open until QG is concluded!
Product Owner: @jjeroch
Dev SPOC: @evegufy
Helm Chart Version: tbd
App Version: 1.0.0
Release Managemnet Reference Issue: eclipse-tractusx/sig-release#672
Check of Tractus-X Release Guidelines

Currently implemented automatic checks can be found under your product on our Release Guidelines Checks Board
This QG Check is depending on the mandatory information from our current Release Guidelines

TRG 1 Documentation

[ ] TRG 1.01 appropriate README.md
[ ] TRG 1.02 appropriate install instructions either INSTALL.md or in README.md
[ ] TRG 1.03 appropriate CHANGELOG.md
[ ] TRG 1.04 editable static files

TRG 2 Git

[ ] TRG 2.01 default branch is named main
[ ] TRG 2.03 repository structure
[ ] TRG 2.04 leading product repository
[ ] TRG 2.05 .tractusx metafile in a proper format

TRG 3 Kubernetes

[ ] TRG 3.02 persistent volume and persistent volume claim is used when needed

TRG 4 Container

[ ] TRG 4.01 semantic versioning and tagging
[ ] TRG 4.02 base image is agreed
[ ] TRG 4.03 image has USER command and Non Root Container
[ ] TRG 4.05 released image must be placed in DockerHub, remove GHCR references
[ ] TRG 4.06 separate notice file for DockerHub has all necessary information

TRG 5 Helm

[ ] TRG 5.01 Helm chart requirements
[ ] TRG 5.02 Helm chart location in /charts directory and correct structure
[ ] TRG 5.03 proper version strategy
[ ] TRG 5.04 CPU / MEM resource requests and limits and are properly set
[ ] TRG 5.06 Application must be configurable through the Helm chart
[ ] TRG 5.07 Dependencies are present and properly configured in the Chart.yaml
[ ] TRG 5.08 Product has a single deployable helm chart that contains all components
[ ] TRG 5.09 Helm Test running properly
[ ] TRG 5.10 Products need to support 3 versions at a time
[ ] TRG 5.11 Upgradeability

TRG 6 Released Helm Chart

[ ] TRG 6.01 Released Helm Chart

TRG 7 Open Source Governance

[ ] TRG 7.01 Legal Documentation
[ ] TRG 7.02 License and copyright header
[ ] TRG 7.03 IP checks for project content
[ ] TRG 7.04 IP checks for 3rd party content
[ ] TRG 7.05 Legal information for distributions
[ ] TRG 7.06 Legal information for end user content
[ ] TRG 7.07 Legal notice for documentation
[ ] TRG 7.08 Legal notice for KIT documentation

TRG 8 Security

[ ] TRG 8.01 Mitigate high and above findings in CodeQL
[ ] TRG 8.02 Mitigate high and above findings in KICS
[ ] TRG 8.03 Mitigate high and above findings in GitGuardian
[ ] TRG 8.04 Mitigate high and above findings in Trivy

Hints
Information Sharing

Hi @tom-rm-meyer-ISST the repo is ready for review
Hi @RoKrish14 could you please take over the security checks?

Review (additional things beside checklist)
QGate checklist lacks TRG 2.06, which is successfully fulfilled.
TRG 7.06 (frontend about license) is not applicable thus ticked.
Discussions
Dear co-pilot @carslen, I would like to have your opinion on the following topics (also feel free to check the findings). @evegufy feel free to also discuss - maybe we should schedule a meeting :)
TRG 3.02, TRG 5.06 PVC claims
From my understanding this is fulfilled (already ticked 3.02) as this is only applicable to the Postgres dependency, which comes with it by default.
The values.yaml by default does not make the PVC / StatefulSet configurable, which is OK from my point of view as I don't think I've currently seen it anywhere (and we also didn't do it).
Do you agree, that it's fine or do you think it's not following the TRG?
Dash Tool in the repository, but no notice
TRG 7.04 clarifies third party content while TRG 7.01  explains the Notice file.
This repository currently contains the Dashtool as a jar at ./scripts/download. I would expect to have a NOTICE.md beside it or better even remove it and add a step to download it to the respective point in workflows / development documentation. I'm also not fully sure about the license here (we're apache 2.0, dash is EPL-2.0). What do you think.
Also following the EF handbook, I noticed that the following excerpt of the NOTICE.md that should be used. It mentions the dash tool usage, but we don't have it mentioned in the TRG example. Any idea because I think no repo has it mentioned at all (we until now don't have it):
# Notices for Dash, Tools for Committers

This content is produced and maintained by the Eclipse Dash, Tools for
Committers project.

 * Project home: https://projects.eclipse.org/projects/technology.dash

TRG 5.05 expects digest in helm values
The digest stated in the should example is only a SHOULD and not a MUST, isn't it? If it's a must I would need to uncheck TRG 5.05.
Findings
Note: As a few findings are pretty small, I fixed them already in a PR.

[ ] #140 TRG 1.01: Readme lacks information about policy hub
[ ] #144 TRG 5.02: Prerequisites missing in helm README
[ ] #141 TRG 1.02 / 5.07: Documentation misses dependency update
[ ] #142 TRG 7.02: Missing license headers on a few files
[ ] #143 TRG 7.05: Docker image must contain distribution information

Recommendations

The repository's documentation files are inconsistently named in terms of space( ), dash(-), underscore (_). Please refactor the naming in the future because you currently can't link to files with spaces in the filename.
The helm chart's values file has a few application specific settings I can understand, but for which a short documentation may be beneficial (e.g. potential settings in logging if applicable): idp, dbConnection, dotnetEnvironment, policyhubmigrations.seeding and logging
Add CI workflow for QGate Checks (included in PR mentioned above)
For TRG 4.03 a uid and group user is used I don't know. Following the documentation of the asp.net image, you could use the user app with 1654:1654 since version 8 (that you use). I would go for it :)

Hi @tom-rm-meyer-ISST Thank you for performing the thorough review and especially for opening #145!
Regarding the findings: I've already commented some of the issues and I think only https://github.com/eclipse-tractusx/policy-hub/issues/142 is remaining, that one I'll implement today or tomorrow.
Regarding the Dash Tool: I'll just change to downloading at every run, I had that anyway on my TODO list but due to time constraints, I wasn't able to come around to.
Regarding the recommendations: thank you for the input and we might look into those in the future. Just letting you know that it's possible to link to files with spaces in the file name by using %20: https://github.com/eclipse-tractusx/policy-hub/blob/main/docs/technical-documentation/architecture/Development Concept.md

Hi @tom-rm-meyer-ISST the repo is ready for review Hi @RoKrish14 could you please take over the security checks?

Hi @RoKrish14 could you please take over the security checks and leave a comment on the following issue as well https://github.com/eclipse-tractusx/sig-release/issues/672? Thank you!

Hi @tom-rm-meyer-ISST Thank you for performing the thorough review and especially for opening #145! Regarding the findings: I've already commented some of the issues and I think only #142 is remaining, that one I'll implement today or tomorrow. Regarding the Dash Tool: I'll just change to downloading at every run, I had that anyway on my TODO list but due to time constraints, I wasn't able to come around to. Regarding the recommendations: thank you for the input and we might look into those in the future. Just letting you know that it's possible to link to files with spaces in the file name by using %20: https://github.com/eclipse-tractusx/policy-hub/blob/main/docs/technical-documentation/architecture/Development Concept.md

Thanks for the fast answers and clarifications! Also thanks for pointing out the encoding of spaces. Didn't think about that! :)

Hi @tom-rm-meyer-ISST the repo is ready for review Hi @RoKrish14 could you please take over the security checks?

Hi @RoKrish14 could you please take over the security checks and leave a comment on the following issue as well eclipse-tractusx/sig-release#672? Thank you!

Done 👍

Hi @tom-rm-meyer-ISST the repo is ready for review Hi @RoKrish14 could you please take over the security checks?

Also done 👍

Finally walked through closed issues and PRs. Everything fine from my perspective. Some issues were false positive, recommendations and discussion points are already resolved. Thanks for collaborating; it was fun!

Finally walked through closed issues and PRs. Everything fine from my perspective. Some issues were false positive, recommendations and discussion points are already resolved. Thanks for collaborating; it was fun!

Thank you for the great review!
The final version is released now: https://github.com/eclipse-tractusx/policy-hub/releases/tag/policy-hub-1.0.0


pin gke cluster version

upgrades take too long. We should just bump the version in spingo when we want to take the time to upgrade instead of having to manually pin the version every time we rerun the spinnaker directory terraform

Also could possibly look at new GKE stable release channel


[CHALLENGE SUBMISSION]

Please provide the following information for your submission:
What is your name (First, Last): Kris Schepers
Where is your GitHub Repo (with your challenge solution): https://github.com/TheZooPlayground/ColdStart-Challenge-2021/tree/main/challenges/challenge3
What Challenge is this for (challenge number between 1 and 6): 3
(Optional) Anything else we should know? e.g., language used, location, blog post?
This is a joint effort/team building by the developers at The Zoo!
Running result: https://zealous-desert-00c85bf03.azurestaticapps.net/

Thanks for participating in our coding challenges ❤! To claim your Raspberry PI device, please DM me on Twitter to get the link for providing your shipping details. https://twitter.com/nicktrog


Any plans to expose status events on component?

Things like onCanPlayThrough, onDurationChange, onLoadedMetadata etc would be nice to have exposed on the component level. Do you have any plans to add something like that?
I could give it a go if nothing else :)

Hi @rasmusfl0e thank you for raising an issue.
I went ahead and done this and released 1.2.6. Let me know if it does the trick!

Wow - thanks, that was quick!
Works like a charm :)


Update method and formmethod HTML attrs capitalization

As per the HTML standard (and the descriptions in vscode-html-languageservice, and common reference sites like w3schools and MDN, and the w3 validator and libraries like html-validate), the value of these should be in lowercase.

Also see https://github.com/microsoft/vscode-custom-data/pull/32

Requesting review from @aeschli

Thanks @domdomegg !


Verbs being changed

I'm only noticing this with verbs, but they'll often get deconjugated. ございません becomes ござる, なった becoming なる. This only happens in ttu so far with the extension, the text itself shows up normally in any other reader without the extension. It doesn't happen with all verbs, but enough that it became noticeable.

This bug is caused by the current fallback logic for words with no furigana.
The code assumes that using the word's spelling instead is fine. However, the spelling as reported through the jpdb API does not include any conjugations.
This will be fixed along with #8. Rather than waiting for the furigana customization settings as originally planned, I will prioritize just fixing these bugs for now.


chore(deps): upgrade configuration

Upgrades project dependencies. See details in workflow run.

Automatically created by projen via the "upgrade-configuration-k8s-26-main" workflow

⚪ Backport skipped
The pull request was not backported as there were no branches to backport to. If this is a mistake, please apply the desired version labels or run the backport tool manually.
Manual backport
To create the backport manually run:
backport --pr 3853

Questions ?
Please refer to the Backport tool documentation


Multiple HC-SR04 issue

Hello !
I've installed your HC-SR04 library.
Once I'm trying the multiple sensor way, with the following code :  HCSR04 hc(3,new int[2]{2,5},2);  (there are 2 sensors)
and
{ for (int i = 0; i < 2; i++ ) {Serial.println( hc.dist(i) );} }
I got the following error

no matching function for call to 'HCSR04::HCSR04(int, int*, int)'

Thank you!

Hello
are you on 2.0.1 version of the library ?
if you are on version 2.0.0 you need to use
HCSR04 hc (2, new int [6] {5,6,7,8,9,10});

probably your obstacle is too far or has a bad angle that prevents the impulse from returning to the sensor and therefore returns 0


Low Priority: For Carton on Swift 5.9 Toolchain

These changes are optional, very low priority, and are really meant to go in tandem with the below release of carton (if approved), but it would be nice to have some updated packages to sync with carton; and some better compatibility when targeting the Swift 5.9 toolchain as well as later versions of macOS/iOS.
The easiest way to test this since carton doesn't appear to support pre-release versions of Swift is to use this release of carton in the meantime.
If I fix-up the template for tokamak in the above carton release, would a PR for carton possibly be approved for a pre-release version? I do not see any prior pre-release versions of carton, and was not sure if this is something the carton developers would like to support.
Note: TimelineView has been temporarily disabled in the demo due to:
// cartonbeta dev -v --product TokamakDemo

CanvasDemo.swift:66:5: error: generic parameter 'Content' could not be inferred

The above error is strange, have you guys been able to compile the demo as-is on current toolchain versions?
Everything else appears to be in tact.
$ cartonbeta dev -v --product TokamakDemo
- checking Swift compiler path: /Users/furby/.carton/sdk/wasm-5.9-SNAPSHOT-2023-08-01-a/usr/bin/swift
- checking Swift compiler path: /Users/furby/.swiftenv/versions/wasm-5.9-SNAPSHOT-2023-08-01-a/usr/bin/swift
- checking Swift compiler path: /Users/furby/Library/Developer/Toolchains/swift-wasm-5.9-SNAPSHOT-2023-08-01-a.xctoolchain/usr/bin/swift
Inferring basic settings...
- swift executable: /Users/furby/Library/Developer/Toolchains/swift-wasm-5.9-SNAPSHOT-2023-08-01-a.xctoolchain/usr/bin/swift
SwiftWasm Swift version 5.9-dev (LLVM 8250a14f5d3ef6a, Swift 57acd8f2a758d01)
Target: arm64-apple-darwin23.0.0

Parsing package manifest:
[debug]: evaluating manifest for 'tokamak' v. unknown- development product: TokamakDemo
- development binary to serve: /Users/furby/dev/Atmos/Tokamak/.build/wasm32-unknown-wasi/debug/TokamakDemo.wasm

Building the project before spinning up a server...
Compiling...
/Users/furby/Library/Developer/Toolchains/swift-wasm-5.9-SNAPSHOT-2023-08-01-a.xctoolchain/usr/bin/swift build -c debug --product TokamakDemo --triple wasm32-unknown-wasi -Xswiftc -Xclang-linker -Xswiftc -mexec-model=reactor -Xlinker --export=main -Xlinker /Users/furby/.carton/static/so_sanitizer.wasm -Xlinker --stack-firstBuilding for debugging...
[0/1] Linking TokamakDemo.wasm
Build complete! (1.79s)

`swift` process finished successfully
`swift build` completed in 2.04 seconds
Binary transformation for Safari compatibility completed in 0.34 seconds

The carton changes to support the Swift 5.9 toolchain have been approved here. Looks like we should now bump the Swift tools version to 5.7 for Tokamak to reflect carton.


Show ******** when typing in password to login

Possible solution https://www.semicolonworld.com/question/46931/how-to-hide-password-in-the-nodejs-console.
https://github.com/nodejs/node/blob/507245436ba79ad3ca848d27f2e143f8debcfe7c/lib/readline.js#L291
Basically you would overwrite the builtin node.js _writeToOutput function on the readline interface.