[GitHub actions] KeyError: 'service_job_id'

Hey,
thanks for this very useful client. I use it in almost all of my open source projects, but it recently stopped working. At first, I thought it was related to the new 2.2.0 release, but since I ran an older (previously successful) pipeline using 2.1.2 again and it failed, I think the issue lies somewhere else.
I use GitHub actions and integrated it like this (full workflow):
...
test:
      ....
      - name: Install dependencies
        run: |
          python -m pip install --upgrade pip
          pip install tox tox-gh-actions coveralls  # or coveralls==2.1.2
      ...
      - name: coveralls
        run: coveralls  # or `coveralls debug` for debugging
        env:
          COVERALLS_PARALLEL: true
          GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
...
  coveralls:
    needs: [test]
    runs-on: ubuntu-latest
    steps:
      - name: Set up Python
        uses: actions/setup-python@v2
      - name: Install dependencies
        run: |
          python -m pip install --upgrade pip
          pip install coveralls
      - name: coveralls
        run: coveralls --finish  # or `coveralls debug --finish` for debugging
        env:
          GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}

When running the pipeline, this is the error I am getting:

To debug this, I created a new branch based on a failing branch:

Initial run failed, version 2.2.0
Using coverall debug made the pipeline pass, version 2.2.0
Reverting the commit and the pipeline fails again, version 2.2.0
Switching to version 2.1.2 fails the pipeline as well
Using coveralls debug with version 2.1.2 works again

I am a bit lost now; looks like it working fine when using the debug mode, but doesn't without it. Does anybody have an idea if that's a bug or a configuration issue on my side?

The payload sent to coveralls.io from a Github Actions job must contain {"service_job_id":null, "service_name": "github"}. By default, the coveralls library omits service_job_id and uses github-actions.

I worked around the issue like this: https://github.com/linux-system-roles/tox-lsr/pull/7

I'm running into the same problem. It's because the service_job_id never get set here:
https://github.com/coveralls-clients/coveralls-python/blob/a9b36299ce9ba3bb6858700781881029d82e545d/coveralls/api.py#L56-L57
because jobs is None from
https://github.com/coveralls-clients/coveralls-python/blob/a9b36299ce9ba3bb6858700781881029d82e545d/coveralls/api.py#L109
which then blows up in
https://github.com/coveralls-clients/coveralls-python/blob/a9b36299ce9ba3bb6858700781881029d82e545d/coveralls/api.py#L224-L225
Can't we just change that to the following?
new_id = '{}-{}'.format(
    self.config.get('service_job_id', None), random.randint(0, sys.maxsize))

I found a weird way to fix this. I initially set up my workflow with the COVERALLS_FLAG_NAME variable set and got this error. Even after I removed that from the workflow, I would still get this error. I then deleted my repo on coveralls.io and reran the jobs without the flag name and they succeeded. Links here:

Initial run with COVERALLS_FLAG_NAME set
Run without COVERALLS_FLAG_NAME

I'm encountering the same issue in a different context and I'm afraid the above discussion did not really clarify the problem for me.
I don't know whether my issue is related or if I should open a new issue, so I'll first expose the problem here.
I used to use Travis to run tests but I'm now moving to SourceHut, I configured the new setting (using repo token etc) and everything seems to run fine except for that service_job_id error...
Does this mean additional variables should be set in .coveralls.yml or as environment variables?
I did not see any change setting COVERALLS_FLAG_NAME to a specific value but debug indeed works fine.
So far I only set
service_name: sourcehut
repo_token: mytoken

I'm encountering the same issue in a different context and I'm afraid the above discussion did not really clarify the problem for me.
I don't know whether my issue is related or if I should open a new issue, so I'll first expose the problem here.
I used to use Travis to run tests but I'm now moving to SourceHut, I configured the new setting (using repo token etc) and everything seems to run fine except for that service_job_id error...
Does this mean additional variables should be set in .coveralls.yml or as environment variables?
I did not see any change setting COVERALLS_FLAG_NAME to a specific value but debug indeed works fine.
So far I only set
service_name: sourcehut
repo_token: mytoken

@escaped looks like your original issue here is related to what @richm suggested -- setting the service_job_id to null and the service_name to github-actions should hopefully fix up your usecase. I am in progress releasing coveralls==3.0.0 which will fix these issues; please take a look and let me know if that fixes your issue!
Note that v3 will change the precedence order for config files -- user-provided configuration will now override the CI values. If you are still having issues with v3 you can, for example, try setting export COVERALLS_SERVICE_NAME=github to override the value we're sending (we send github-actions by default). If I'm reading your issue correctly, the default behaviour in v3 should work for you? To be honest, the Github Actions API seems to keep changing under me, or at least I'm not understanding what it's expecting properly, so YMMV.
With respect to the missing service_job_id key, it's worth noting that only occurs during the retry -- you shouldn't need to retry at all once the fix is in place. That said, thanks for pointing it out; I'll add a fix for that as well!
@richm thanks for the instructions and the link to your PR! You should be able to make use of the config values in v3 to avoid needing that custom script, if you'd like -- just set COVERALLS_SERVICE_NAME=github (or use the .coveralls.yml or CLI flags, if you'd prefer those).
@goerz @J-M0 @Silmathoron I suspect the issues y'all are running into will also be solved with these v3 fixes.

@escaped looks like your original issue here is related to what @richm suggested -- setting the service_job_id to null and the service_name to github-actions should hopefully fix up your usecase. I am in progress releasing coveralls==3.0.0 which will fix these issues; please take a look and let me know if that fixes your issue!
Note that v3 will change the precedence order for config files -- user-provided configuration will now override the CI values. If you are still having issues with v3 you can, for example, try setting export COVERALLS_SERVICE_NAME=github to override the value we're sending (we send github-actions by default). If I'm reading your issue correctly, the default behaviour in v3 should work for you? To be honest, the Github Actions API seems to keep changing under me, or at least I'm not understanding what it's expecting properly, so YMMV.
With respect to the missing service_job_id key, it's worth noting that only occurs during the retry -- you shouldn't need to retry at all once the fix is in place. That said, thanks for pointing it out; I'll add a fix for that as well!
@richm thanks for the instructions and the link to your PR! You should be able to make use of the config values in v3 to avoid needing that custom script, if you'd like -- just set COVERALLS_SERVICE_NAME=github (or use the .coveralls.yml or CLI flags, if you'd prefer those).
@goerz @J-M0 @Silmathoron I suspect the issues y'all are running into will also be solved with these v3 fixes.


save_network_settings

When i go in wp-admin/network/settings.php with back office and then i save setting, this message appears :
'wpCloud\StatelessMedia\Settings' does not have a method 'save_network_settings'
I think, it is  on wp-stateless/lib/classes/class-settings.php => the method save_network_settings is not exist in this class !!

@mjezegou Thanks for reporting. We will fix this in next release.
In the mean time if this causing trouble you can just delete this line class-settings.php#L77.


Test case for IDENTITY-7233

Hi @madurangasiriwardena ,
Can you get the requested changes done at earliest possible?

Closing this since the change should be merged to master branch. New PR https://github.com/wso2/product-is/pull/3032.


Wildcards change for every rule

Snakemake version
5.10
Describe the bug
During pipeline execution Snakemake reports different wildcards for each rule. Example: Expected wildcard sample: 0053_P2017BB3S20R_S2, becomes: 0053_P2017BB3S20R_S2_R or 0053_P2017BB3S20R_S2_ depending on he rule.
Minimal example
I construct the input/output names from the samples table, a read and a fastq_file_extension list like this:
samples = pd.read_csv('samples.tsv', sep='\t').set_index('samples', drop=False)
reads = ['R1', 'R2']
fastq_file_extension = _001.fastq.gz

Two rules from my pipeline are:
rule fastqc:
    input:
        os.path.join(config['Directories']['fastq_raw_dir'], '{sample}_{read}' + fastq_file_extension)
    output:
        html=os.path.join(config['Directories']['fastqc_dir'], '{sample}_{read}_fastqc.html'),
        zip=os.path.join(config['Directories']['fastqc_dir'], '{sample}_{read}_fastqc.zip')
    wrapper:
        "0.49.0/bio/fastqc"

and
rule trim_galore_pe:
    input:
        [os.path.join(config['Directories']['fastq_raw_dir'], '{sample}' + read + fastq_file_extension) for read in reads]
    output:
        fastq_read1 = os.path.join(config['Directories']['fastq_trimmed_dir'], '{sample}.1_val_1.fq.gz'),
        fastq_read2 = os.path.join(config['Directories']['fastq_trimmed_dir'], '{sample}.2_val_2.fq.gz'),
        report_read1 = os.path.join(config['Directories']['trim_galore_qc_dir'], '{sample}.1.fastq.gz_trimming_report.txt'),
        report_read2 = os.path.join(config['Directories']['trim_galore_qc_dir'], '{sample}.2.fastq.gz_trimming_report.txt')
    params:
        extra="--retain_unpaired"
    wrapper:
        "0.49.0/bio/trim_galore/pe"

During execution both rules report different wildcards and so the process goes wrong, the output filenames are not as expected:
For the fastqc rule you can see that it pulled the R to the sample wildcard and added an _.
[Fri Feb 21 12:39:57 2020]
rule fastqc:
input: fastq_raw/0053_P2017BB3S20R_S2_R1_001.fastq.gz
output: qc/fastqc/0053_P2017BB3S20R_S2_R1_fastqc.html, qc/fastqc/0053_P2017BB3S20R_S2_R1_fastqc.zip
jobid: 10
wildcards: sample=0053_P2017BB3S20R_S2_R, read=1

For the trim_galore_pe rule you can see that it added and extra _ to the sample wildcard
[Fri Feb 21 12:39:57 2020]
rule trim_galore_pe:
input: fastq_raw/0053_P2017BB3S20R_S2_R1_001.fastq.gz, fastq_raw/0053_P2017BB3S20R_S2_R2_001.fastq.gz
output: fastq_trimmed/0053_P2017BB3S20R_S2_.1_val_1.fq.gz, fastq_trimmed/0053_P2017BB3S20R_S2_.2_val_2.fq.gz, qc/trim_galore/0053_P2017BB3S20R_S2_.1.fastq.gz_trimming_report.txt, qc/trim_galore/0053_P2017BB3S20R_S2_.2.fastq.gz_trimming_report..txt
jobid: 7
wildcards: sample=0053_P2017BB3S20R_S2_

I also tried to add the _ to the read parameter, like so: reads = ['_R1', '_R2'], actually I prefer this because some files do not use _ as a coupling between the sample name and read indicator and I can change it in 1 place.
But then Snakemake will also add another extra _ behind the sample wildcard (notice the two _'s after S1) and the pipeline ends immediately:
Building DAG of jobs...
MissingInputException in line 1 of /home/nlv24077/temp/RNASeqPairedEnd/rules/trim_galore_pe.smk:
Missing input files for rule trim_galore_pe:
fastq_raw/0053_P2017BB3S19R_S1__R2_001.fastq.gz
fastq_raw/0053_P2017BB3S19R_S1__R1_001.fastq.gz

Is there a way to prevent Snakemake from altering the wildcards in any way? Why does it do it anyway? Is there something I should change?
Oh and one more thing, is this the primary home for Snakemake or bitbucket? Where do you want me to post issues? Thanx for the great software by the way! I'm very happy with it.

Welcome!  Github is the primary home for snakemake, though for questions like this stackoverflow is a better place to post.
The issue with your pipeline is that you aren't quite matching the inputs of trim_galore with the outputs of fastqc.
input:
        [os.path.join(config['Directories']['fastq_raw_dir'], '{sample}' + read + fastq_file_extension) for read in reads]
# becomes
    'dir/{sample}R1_001.fastq.gz', 'dir/{sample}R2_001.fastq.gz',

Which is missing the underscore between sample and read.  I find it easier use a variable for each file and then reuse that between rules to eliminate these kind of errors.  If you place those in your config yaml then all the filenames are in one place.
fastq_file = os.path.join(config['Directories']['fastq_raw_dir'], '{sample}_{read}' + fastq_file_extension)

rule fastqc:
    input:
        fastq_file
    output:
        html=os.path.join(config['Directories']['fastqc_dir'], '{sample}_{read}_fastqc.html'),
        zip=os.path.join(config['Directories']['fastqc_dir'], '{sample}_{read}_fastqc.zip')
    wrapper:
        "0.49.0/bio/fastqc"

rule trim_galore_pe:
    input:
        expand(fastq_file, read=reads, allow_missing=True)  # allow missing keeps sample as {sample}
    output:
        fastq_read1 = os.path.join(config['Directories']['fastq_trimmed_dir'], '{sample}.1_val_1.fq.gz'),
        fastq_read2 = os.path.join(config['Directories']['fastq_trimmed_dir'], '{sample}.2_val_2.fq.gz'),
        report_read1 = os.path.join(config['Directories']['trim_galore_qc_dir'], '{sample}.1.fastq.gz_trimming_report.txt'),
        report_read2 = os.path.join(config['Directories']['trim_galore_qc_dir'], '{sample}.2.fastq.gz_trimming_report.txt')
    params:
        extra="--retain_unpaired"
    wrapper:
        "0.49.0/bio/trim_galore/pe"

Or just change your input:
input:
        [os.path.join(config['Directories']['fastq_raw_dir'], '{sample}_' + read + fastq_file_extension) for read in reads]
# becomes
    'dir/{sample}_R1_001.fastq.gz', 'dir/{sample}_R2_001.fastq.gz',

Hi thanx for your response, I did what you said:
fastq_file = os.path.join(config['Directories']['fastq_raw_dir'], '{sample}_{read}' + fastq_file_extension)

rule trim_galore_pe:
    input:
        expand(fastq_file, read=reads, allow_missing=True)
    output:
        fastq_read1 = os.path.join(config['Directories']['fastq_trimmed_dir'], '{sample}.1_val_1.fq.gz'),
        fastq_read2 = os.path.join(config['Directories']['fastq_trimmed_dir'], '{sample}.2_val_2.fq.gz'),
        report_read1 = os.path.join(config['Directories']['trim_galore_qc_dir'], '{sample}.1.fastq.gz_trimming_report.txt'),
        report_read2 = os.path.join(config['Directories']['trim_galore_qc_dir'], '{sample}.2.fastq.gz_trimming_report.txt')
    params:
        extra="--retain_unpaired"
    wrapper:
        "0.49.0/bio/trim_galore/pe"

It still crashes:
Building DAG of jobs...
MissingInputException in line 1 of /home/nlv24077/temp/RNASeqPairedEnd/rules/trim_galore_pe.smk:
Missing input files for rule trim_galore_pe:
fastq_raw/0053_P2017BB3S19R_S1__R2_001.fastq.gz
fastq_raw/0053_P2017BB3S19R_S1__R1_001.fastq.gz

There is still an extra _ coming from somewhere, right??

Are you still doing this:

I also tried to add the _ to the read parameter, like so: reads = ['_R1', '_R2'], actually I prefer this because some files do not use _ as a coupling between the sample name and read indicator and I can change it in 1 place.
If that's how you want your reads then remove the underscore from the first rule.

Are you still doing this:

I also tried to add the _ to the read parameter, like so: reads = ['_R1', '_R2'], actually I prefer this because some files do not use _ as a coupling between the sample name and read indicator and I can change it in 1 place.

If that's how you want your reads then remove the underscore from the first rule.

No, I removed it from there. Somewhere Snakemake adds it again. Is there a way to see what all wildcards at every step? Is there way to just say to Snakemake: Please don't change my wildcards, just use whatever is provided in the samples.tsv and the snakefile? I feel that there is a lot going on "under the hood" regarding in and output, which I have a hard time grasping.

Try running with --verbose and --debug-dag.  Snakemake doesn't just change wildcards, but the regular expression and directive code can be hard to grasp.  Using rules.trim_galore_pe.output.read1 is fine and I think the rules you shared look ok.
From your initial outputs:
# rule fastqc
wildcards: sample=0053_P2017BB3S20R_S2_R, read=1
# rule trim_galore_pe
wildcards: sample=0053_P2017BB3S20R_S2_

so already the sample and read aren't being parsed the way you think.  It could be a rule upstream is also ill-formed and the effect is cascading to these rules at the start of your pipeline.
If debug and verbose don't help I can look at your Snakefile to provide more feedback.  If you can put it on github we can move the conversation to that repo.

Try running with --verbose and --debug-dag. Snakemake doesn't just change wildcards, but the regular expression and directive code can be hard to grasp. Using rules.trim_galore_pe.output.read1 is fine and I think the rules you shared look ok.
From your initial outputs:
# rule fastqc
wildcards: sample=0053_P2017BB3S20R_S2_R, read=1
# rule trim_galore_pe
wildcards: sample=0053_P2017BB3S20R_S2_

so already the sample and read aren't being parsed the way you think. It could be a rule upstream is also ill-formed and the effect is cascading to these rules at the start of your pipeline.
If debug and verbose don't help I can look at your Snakefile to provide more feedback. If you can put it on github we can move the conversation to that repo.

I have gotten it to work, indeed by cleanly separating the wildcards. I believe at least some of my grief was caused by that. The rest was by my wrong definition in sample.tsv.
This can be closed and I thank you for your help.

I'll close this issue since @freekvh has said so.
Possibly a little more hand-holding for users when debugging a DAG like this could be added, as a future enhancement to Snakemake, but I don't think that is a reason to keep this particular bug live.


V1 to v2 apps migration (Release_2) + Threat Intel

Purpose of this pull request
This pull request is to update the app installation steps as per V2. Also, add updating and deleting the app information.
Select the type of change

[ ] Minor Changes - Typos, formatting, slight revisions
[x] Update Content - Revisions, updating sections
[ ] New Content - New features, sections, pages, tutorials
[ ] Site and Tools - .clabot, version updates, maintenance, dependencies, new packages for the site (Docusaurus, Gatsby, React, etc.)

Ticket (if applicable)
Doc changes for the migrated V2 apps which are using Threat Intel

@kimsauce Thanks Kim for your review.

@JV0812, the threat intel GA release date is currently targeted for January 16, 2025.


Error: Coverage output file not found. (file "report.json" not found)

Describe a bug
in the github actions , error

Expected behavior
no error
Details
This is my yaml file：
https://github.com/lixiaofa/fast-plus/blob/dev/.github/workflows/test-coverage.yml

Hello @lixiaofa :wave:,
As I can see, you're using vitest. This action supports only jest, although some people were able to use it with vitest.


Handle 72game

Add 7-2 Bounty Hand Skipping and Reporting
Summary
This PR modifies the Game class to skip 7-2 bounty hands during processing and report the number of skipped hands in the output. This change addresses the issue of 7-2 bounty hands causing problems in the conversion process. This fixes the following issue.
Changes

Added logic in Game.__init__ to detect and skip 7-2 bounty hands
Implemented a counter for skipped 7-2 bounty hands
Added a brief message, per your recommendation, on the hands / amount of hands skipped.

Please review and let me know if any further changes or clarifications are needed.
Unrelatedly, I submitted the issue for this a while ago...just sat down to tackle it and that took roughly 2minutes, so jeez -- sorry for the wait.

I just pushed a separate push. Essentially, consider the following file:
ramp.csv
In such a log format, you'll find that the parser fails, because the raises for user Hiroto fail. Frankly, I don't know WHY the change works (it's a singular line), but perhaps you can shine some light on that. The irony is that it ONLY FAILS for this user, and no other users...sigh.

Hey Rudy,
Thanks very much for the PR, I will check it out and see if I can work out
what's going on with that user when I next get a chance.
No problem at all about the delay - I was meaning to look at this as well
for a while but something always comes up! :)
Cheers,
Charles
On Thu, Jul 11, 2024 at 3:49 AM Rudy Roggio @.***>
wrote:

I just pushed a separate push. Essentially, consider the following file:
ramp.csv https://github.com/user-attachments/files/16170849/ramp.csv
In such a log format, you'll find that the parser fails, because the
raises for user Hiroto fail. Frankly, I don't know WHY the change works
(it's a singular line), but perhaps you can shine some light on that. The
irony is that it ONLY FAILS for this user, and no other users...sigh.
—
Reply to this email directly, view it on GitHub
https://github.com/charlestudor/PokerNowLogConverter/pull/17#issuecomment-2221886634,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABS3SRDJRK6XWEVCVH5KVNTZLXXC7AVCNFSM6AAAAABKWAFBMSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDEMRRHA4DMNRTGQ
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

hey, any bump on this?

Hey Rudy! Sorry again for the long delay!
Your change looks good to me. The reason we are seeing the error with the "raises" is actually because of the "to" in "Hiroto"'s name. As I think I mentioned before there are a few places this has tripped me up, if I was writing the tool again from scratch I would definitly implement a "tokenization" stage at the start before parsing.
For now, I would suggest making two changes to your PR:

Specify an escaped quotation in the "to" line:  raise_amount = float(line.split("\" raises to ")[1])
Could you please also update the "state not considered list" to remove the warnings about "forced the player to away mode" and "rejected the seat request". These are two new states I hadn't seen before and look safe to ignore.

Thanks!

thank you. added your suggestions, worked with test file as well. let me know if there's anything else you want.

Brilliant, thanks Rudy! Merged. I'll create a new release.


Migrating Tenant Configurations from the Registry

Migrating tenant configurations from registry and support managing through the admin portal
Currently, Admin has to log in to the carbon console and use the Registry to add/update/delete configurations for the following scenarios, which cannot be considered user-friendly.
This feature provides a method to add/update/delete configurations for these scenarios through the admin portal and use RDBMS as the persistent storage instead of the Registry.

[x] #361
[x] Documentation - Migration of Self Sign up Config
[ ] #363
[ ] Add integration tests for Publishing to Multiple External Developer Portals
[ ] #288
[ ] #362
[x] #255
[x] #520
[x] #314
[x] #364

Hi all,
Will resume the following subtasks
Phase 1

[ ] https://github.com/wso2/api-manager/issues/363
[ ] https://github.com/wso2/api-manager/issues/362
[ ] https://github.com/wso2/api-manager/issues/2201

Progress Update (26/27 Oct 2023)

Conducted the design review and finalised the design for workflow, GA config migrations.
Decided not to proceed with external API store migration and deprecate the feature.
Started implementation on workflow migration.

Work Log (30 Oct - 3 Nov)
Completed

Changed the workflow config fetching way from registry to Advanced Configuration JSON file.
Facilitated default workflow config when the JSON schema is not provided in the Advanced Configuration.
Used new Workflows config format & facilitated different config options.
Removed of all uses of the existing workflow-extensions.xml file in carbon-apimgt.
Defined the new Workflows schema in tenant-config-schema.json.
Wrote/Modified tests related to workflow configs in carbon-apimgt.

Ongoing/Upcoming

Writing/Modifying integration tests related to workflow configs in product-apim.
Creating migration artefacts.

Work Log (6 Nov - 10 Nov)
Completed

Modified integration tests in product-apim to support new workflow configs.
Modified tenant-config-schema.json to support proper validation for workflow config.
Started work on APIM 5.x.x migration-client to support registry migration.

Ongoing/Upcoming

Completed the work on migration resources.
Conduct the code review.

Work Log (13 Nov - 17 Nov)
Completed

Completed work on migration resources.
Tested the migration to new workflow configuration from old xml configuration.
Fixed several minor issues identified in the implementation.
Refactored the code.
Scheduled the code review.

Ongoing/Upcoming

Conduct the code review.
Do requested changes.

WorkLog ( 21 Nov -23 Nov )
Completed

Conducted code review
Completed requested changes

Remove the enabled flag from workflow approvals. If someone wants to disable, they can remove the json element. If approval workflows, the value of the json field will be {}.
Remove the XML parsing for the property value. Assume XML values are not used as property values in migrated setups.
Make properties to allow only primitives. Restrict it from the schema as well.
Support resolving secrets using securevault is similar to what we have done for notification endpoints in the IS KM connector. We cannot use the same function for XML.
Make the default StateList properties hardcoded and change only if someone adds the property.

Did required documentation changes
Opened draft PRs as discussed in code review.

Ongoing/ Upcoming

Start work on GA Config Migration

Work Log (11 Dec - 15 Dec)
Started work on GA config Migration
Completed:

Examined the existing GA configuration and observed its behavior.
Finalized the JSON format for the new GA configuration.
Eliminated the use of the ga-config.xml registry file.
Implemented the DTO class.
Adjusted the GaConfigApiServiceImpl to accommodate the new configuration format.
Updated the APIMgtGoogleAnalyticsTrackingHandler class to support the new configuration format.
Initiated testing with the modified codebase.

Ongoing/Upcoming:

Continue testing and implement necessary changes to the codebase.
Resolve issues related to detecting configuration changes when the server is running.
Write/Modify unit/integration tests.

Hi all,
Migrations of workflow config and GA config will not be included in the upcoming 4.x releases. Instead, they will be incorporated in future 5.x release.


Fix CLI output -> toolset input versioning

CLI produces version number 3.0.100-alpha1-00000001-01 in a dev build.  We should change this to match what toolset is actually expecting to remove toolset patch 0005 but this is causing problems right now because toolset finds the official version before our source-built version.  3.0.100-alpha1-00000001-01 has no colliding official version and is safe to use until I dig into this more.

Working on this as part of patch removal in master.

Fixed in #719.


Execute tests with blockifier state

Closes #475
Introduced changes

tests are now executed on blockifier state, specifically test becomes a contract now

Breaking changes
There is no breaking changes
Checklist

[ ] Linked relevant issue
[ ] Updated relevant documentation
[ ] Added relevant tests
[ ] Performed self-review of the code
[ ] Added changes to CHANGELOG.md

Copying my comment from here: https://github.com/foundry-rs/starknet-foundry/pull/570#discussion_r1314738123
Should we still be allowing the patter of declare & deploy and then using dispatchers if this change is introduced?  If so, should the contract-test state be applied to this calls? E.g. should the caller address be that of whatever it was set in the test itself (if we allow it)?
code example
#[test]
fn test() {
  cheatcode_that_changes_the_caller_address_in_test(1234);
  assert(get_caller_address() == 1234, 'this passes');
  // ...
  other_dispatcher.function();
}

// ...
#[starknet::contract]
mod Other {
  // ...
  fn function() {
    // should get_caller_address here be whatever it was set in the test above i.e. 1234?
    get_caller_address() == ???
  }
}

Other case:
Should the test itself have an address? If so, should contracts be allowed to interact with it?

I covered basic part of the requested changes and test cases. I created a follow up task for adding more tests. This one does not include any breaking change so there is no point of keeping it open too long as it might casue a lot of merge conflicts.

Task for adding docs https://github.com/foundry-rs/starknet-foundry/issues/611

Related milestone https://github.com/foundry-rs/starknet-foundry/milestone/8


Added a fix to keep the facet height after performing various operations

Fixes #5606
Fixes the Regression from: https://github.com/OpenRefine/OpenRefine/pull/5514
The initial height is set when a facet is added.  The facet height is not adjusted in the following cases:

When other facet is added.
When other facet is deleted.
Include/Exclude Choice
Edit Choice -> Apply
Sort by: name count
Refresh
Cluster Choices -> Merge selected & Close
Resize/Min/Max Browser Window
Show/Hide Left Panel
Min/Max a facet

@elebitzero Tested and now the Facet height is kept in place if user adjusts it. Thanks!
However, while testing I noticed a few things that ultimately need to be improved at some point for leftPanelDiv elements.
I will open some new issues for those.


User agent library version is not being tracked correctly.

User-agent does not have library version:
User-Agent: gcloud-java/ Google-API-Java-Client/1.30.10 Google-HTTP-Java-Client/1.36.0 (gzip)
x-goog-api-client: gl-java/11.0.7 gdcl/1.30.10 linux/5.6.14
Content-Type: application/json; charset=UTF-8
Content-Encoding: gzip

Should be:
User-Agent: gcloud-java/1.xx.x Google-API-Java-Client/1.30.10 Google-HTTP-Java-Client/1.36.0 (gzip)

@dmitry-fa do you have cycles to look at this issue?

@frankyn I managed to reproduce the problem if the Storage client is running from IDE. My question: how do you run the Storage client which issues incorrect header?

@frankyn I think this is not a bug.
The following nano application prints the string to be inserted into the UserAgent header:
import com.google.cloud.storage.StorageOptions;

public class UserAgent {
  public static void main(String... args) throws Exception {
    System.out.println(StorageOptions.newBuilder().build().getApplicationName());
  }
}

Being inserted into the Storage project and run from Intellij Idea  this code prints:
gcloud-java/
The same code being inserted into another project depending on the Storage prints:
gcloud-java/1.110.0
The library version is exracted by the following code:
  public static String getLibraryVersion(Class<?> libraryClass) {
    String version = libraryClass.getPackage().getImplementationVersion();
    return version != null ? version : "";
  }

libraryClass.getPackage().getImplementationVersion() reads the version from the manifest stored in the the library jar file. Being run from an IDE the classes of the project are loaded not from the jar, but from the classes/ dir where there is no manifest file,  as the result no version.
Classes for dependencies are loaded from jars and they have the version available.

Gotcha, I was seeing this in Intellij, thanks for digging this up. It's working as expected in this case. Definitely got confused on my end.

@frankyn you are always welcome :) nice bug, I love such :)


Add contributing guidelines

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

Project coverage is 100.00%. Comparing base (b6245f9) to head (96cb664).

Additional details and impacted files
@@            Coverage Diff            @@
##              main      #128   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files           38        38
  Lines          817       817
  Branches        57        57
=========================================
  Hits           817       817

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


docs/wip: add work-in-progress doc for ingress

Description: This PR creates a new ingress doc, which consolidates the existing ingress documentation with additional information as well as a sample demo.

Affected area:

New Functionality      [ ]
Documentation          [X]
Install                [ ]
Control Plane          [ ]
CLI Tool               [ ]
Certificate Management [ ]
Networking             [ ]
Metrics                [ ]
SMI Policy             [ ]
Security               [ ]
Tests                  [ ]
CI System              [ ]
Demo                   [ ]
Performance            [ ]
Other                  [ ]

Please answer the following questions with yes/no.

Does this change contain code from or inspired by another project? If so, did you notify the maintainers and provide attribution? No

Part of #2871


Document using IPython.display for manual repr display

Thanks for the reminder @grovduck!

You're nothing if not responsive!  Thanks for the tip!


There are updates available for this pull request while master branch is checked out

Extension version: 0.22.0
VS Code version: Code 1.52.1 (ea3859d4ba2f3e577a159bc91e3074c5d85c0523, 2020-12-16T16:34:46.910Z)
OS version: Windows_NT x64 10.0.19041
Steps to Reproduce:

Check out master branch
Have remote changes that are not pulled

Edit: This might be related to the following info I found in the log:
[Info] Review> current branch master is associated with pull request #1335
[Info] Review> Resolving pull request
[Info] Review> Fetching pull request data
[Info] Review> register comments provider
[Info] Review> display pull request status bar indicator and refresh pull request tree view.

If I try to open it from the status bar , I'm getting the error Cannot read property 'commit' of undefined

Upon further investigation: This is the mentioned PR: https://github.com/zwave-js/node-zwave-js/pull/1335
I'm not sure what went wrong, but Github says that it was merged into master without any commits:

I would help fix this (since this is really annoying), but I have no clue where to even start.

Ping - How can I help get this fixed?

So, I figured this out. At some point this extension has associated the master branch with the broken PR under the key github-pr-owner-number in the local .git/config. Since that entry never got cleaned up, it kept showing that the master branch was associated with the PR.
I removed the entry manually and the problem is resolved.

Good find! That sort of worked for me. I had to manually change the record in .config then switch branches then back to the branch my PR was open on. I wonder if it has anything to do with protected branches? Perhaps the plug-in is expecting the branch to be deleted?

So, I figured this out. At some point this extension has associated the master branch with the broken PR under the key github-pr-owner-number in the local .git/config. Since that entry never got cleaned up, it kept showing that the master branch was associated with the PR.
[branch "master"]
	remote = origin
	merge = refs/heads/master
	github-pr-owner-number = "zwave-js#node-zwave-js#1335"

I removed the entry manually and the problem is resolved.

I'm having the same issue, but deleting the entry manually didn't seem to help.  I even tried to delete the entire repo and re-clone it, but the pull request still came back. Any hints?


Add istio-proxy and allow watching on different label

Adds a new container type, and also allows watching different labels than nais' default.
Also makes the README somewhat more generic, and adds an explanation for the Label Selector.

If @Reasonable-Solutions gives his blessing, I think this can be merged :slightly_smiling_face:


Does not work in Chrome 61 beta

I stopped working on all sites. Incognito mode is selected, without extensions.

It hasn't been working since long time ago with the Chrome experimental features too.
Eventually, I've moved to W3C spec from this (https://developer.mozilla.org/en-US/docs/Web/API/Intersection_Observer_API).

The PR by @pkostadinov-2create has been merged and a new version is available: 1.8.8. That one will probably fix this too.


Celerycam doesn't work with Django 1.10 when error log level is set

(env)$ python manage.py celerycam -lINFO
Traceback (most recent call last):
  File "manage.py", line 22, in <module>
    execute_from_command_line(sys.argv)
  File "env/lib/python2.7/site-packages/django/core/management/__init__.py", line 367, in execute_from_command_line
    utility.execute()
  File "env/lib/python2.7/site-packages/django/core/management/__init__.py", line 359, in execute
    self.fetch_command(subcommand).run_from_argv(self.argv)
  File "env/lib/python2.7/site-packages/djcelery/management/base.py", line 103, in run_from_argv
    return super(CeleryCommand, self).run_from_argv(argv)
  File "env/lib/python2.7/site-packages/django/core/management/base.py", line 294, in run_from_argv
    self.execute(*args, **cmd_options)
  File "env/lib/python2.7/site-packages/djcelery/management/base.py", line 96, in execute
    super(CeleryCommand, self).execute(*args, **options)
  File "env/lib/python2.7/site-packages/django/core/management/base.py", line 345, in execute
    output = self.handle(*args, **options)
  File "env/lib/python2.7/site-packages/djcelery/management/commands/celerycam.py", line 26, in handle
    ev.run(*args, **options)
  File "env/lib/python2.7/site-packages/celery/bin/events.py", line 86, in run
    detach=detach)
  File "env/lib/python2.7/site-packages/celery/bin/events.py", line 113, in run_evcam
    return cam()
  File "env/lib/python2.7/site-packages/celery/events/snapshot.py", line 97, in evcam
    app.log.setup_logging_subsystem(loglevel, logfile)
  File "env/lib/python2.7/site-packages/celery/app/log.py", line 97, in setup_logging_subsystem
    loglevel = mlevel(loglevel or self.loglevel)
  File "env/lib/python2.7/site-packages/celery/utils/log.py", line 119, in mlevel
    return LOG_LEVELS[level.upper()]
AttributeError: 'list' object has no attribute 'upper'

This looks like the same problem as #467.


[SPARK-49031] Implement validation for the TransformWithStateExec operator using OperatorStateMetadataV2

What changes were proposed in this pull request?

Implementing validation for the TransformWithStateExec operator, so that it can't restart with a different TimeMode and OutputMode, or invalid State Variable transformations.
Why are the changes needed?

If there is an invalid change to the query after a restart, we want the query to fail.
Does this PR introduce any user-facing change?

No
How was this patch tested?

Unit tests
Was this patch authored or co-authored using generative AI tooling?

No

@HeartSaVioR PTAL when you have a chance

Thanks! Merging to master.


the version for windows

Hi, could you build the new version of tensorflow for windows? the cuda version just tensorflow 1.1.

Version 1.4.0 of the tensorflow-gpu package is available from PyPI and should be installed if you enter the command pip install tensorflow-gpu.


FIX: Make changes for recent Arsenal updates

PR has been updated. Reviewers, please be cautious.

Closing this because fixes are to be added in https://github.com/scality/S3/pull/1129.


Feat/Live-2356

Upgrade dependencies:
@types/jest (dev) 27.5.0 ❯ 27.5.1
eslint-plugin-react (dev) 7.29.4 ❯ 7.30.0
glob (dev) 7.2.0 ❯ 7.2.3
stellar-sdk 10.1.0 ❯ 10.1.1
ts-jest (dev) 27.1.4 ❯ 27.1.5
ts-node (dev) 10.7.0 ❯ 10.8.0
xstate 4.32.0 ❯ 4.32.1

Hey @alexalouit,
Thank you for your contribution 🙇.
Unfortunately:

You did not fill in the PR template
You did not provide any reasons for upgrading dependencies

I know these are minor / patch bumps, but we have had issues in the past even with these kind of updates so we do handle most dependencies upgrades in-house until further notice.


Sqpackage 14.0 cannot deploy dacpac

First of all, Sqlpackage 15 handles dacpac-file properly. The issue comes to play only with sqlpackage version 14 privisioned by msi or SSMS 2017.
I have stumbled upon the following error during running deploy with dacpac file:
The Element or Annotation class SqlDatabaseOptions does not contain the Property class CatalogCollation.

My researches have led me to the reason: model.xml file contains <Property Name="CatalogCollation" Value="0" /> line. After removing it (with hash recalculation in origin.xml) and making dacpac again deploy finishes successfully.
Nothing with configuration has helped me. I tried to override KnownModelProperties property on csproj without problem property - no result.
I compared model.xml file from original dacpac (produced from sqlproj) and didn't see there CatalogCollation property. Also I noticed that SchemaVersion is used by version 2.9 in contrast to 2.4 in dacpac from csproj file.
Versions:
MSBuild.Sdk.SqlProj 1.15.0
SqlServerVersion property 140
DotNet Sdk 3.1

@iremezoff Thanks for reporting this. This is actually the first time I've heard of any compatibility issues between this project and SqlPackage.exe. In fact, we are happily deploying .dacpac's build by this project using SqlPackage.exe 14. Would you mind sharing the exact version number of SqlPackage.exe?

Yes, of course. I installed dacfx by choco package manager (https://community.chocolatey.org/packages/sql2017-dacframework). The version is 14.0.3757.2. By the way,  I saw comments for the same issue for sqlproj caused by buggy release of sqlpackge. The recommendation was to update build of the tool. Of course, in my case it wasn't feasible.

Can you share a link to those comments?

https://dba.stackexchange.com/questions/208775/bacpac-import-fails-with-error-the-element-or-annotation-class-sqldatabaseoptio
https://techcommunity.microsoft.com/t5/azure-database-support-blog/what-will-happen-with-catalog-collation-and-case-sensitive-vs/ba-p/368978

Right, so looking at the release notes for SqlPackage.exe it seems that 17.8 is the latest version which has a version number of 14.0.4079.2. That seems to be a newer version than the one you are using. Have you tried upgrading SqlPackage.exe to the latest version by upgrading the Microsoft® SQL Server® Data-Tier Application Framework to the latest version?

Nope, because of using certain version provided by choco repository. I will try and report here

No problem. Glad it worked out!


only one GridMap is supported at a time

GridMap is defined as an object at import time, then initialized using the init() function.  This prevents multiple GridMaps from being used at once.  Rework GridMap into a constructor such that the a client can create multiple GridMaps, such as
map1 = new GridMap('selector1', size, options);
map2 = new GridMap('selector2', size, options);

Issue is being tracked and worked on as https://vulcan.atlassian.net/browse/SAU-1532

Fixed in 6f1e8c4186aa9a7d7ecb5dbb44fd06511b7218a6


Support for Windscribe Pro accounts (more proxy locations)

Hi, are there any plans to add support for Pro accounts in the future?

Closed as there is no problem with Pro accounts. There may be a problem with special characters in the password (% for example), but I'm not sure and for now I've solved it by changing the password.


TextEditor.pixelPositionForScreenPosition is deprecated.

This method is deprecated on the model layer. Use TextEditorElement::pixelPositionForScreenPosition instead
TextEditor.pixelPositionForScreenPosition (/Applications/Atom.app/Contents/Resources/app/src/text-editor.js:2636:9)
ruler-view.update (/Users/gokmen/.atom/packages/rulerz/lib/ruler-view.coffee:35:41)

Aw, jeeze - I can't get a break with this code! ;-)


toggle subfloor must be toggled to to show new disposal pipes added

Description
 yet another known but unreported issue effecting mappers
Screenshots

https://user-images.githubusercontent.com/95458399/152843794-589baa14-2226-440d-a7c2-7165fbf541ba.mp4
Additional context

Something fixed this.


updated version is also not working for google pay

when use google pay payment is success fully done but upi india final response
UPI_INDIA_FINAL_RESPONSE: user_canceled

Duplicate of #5

Can you try any other plugin like Upi Pay and comment what happens in that case?


Getting { Promise {  } } downloading many attachment from a mail server

Hi, I am encountering hanging with the error below when downloading many attachments from a mail server. Please how can I make the processing of bulk attachment from a mail very fast without hanging? Thanks
{ Promise { <pending> } }
{Promise { <pending> } }
{  Promise { <pending> } }
{  Promise { <pending> } }
{ Promise { <pending> } }
{  Promise { <pending> } }
{ Promise { <pending> } }
{ Promise { <pending> } }
{ Promise { <pending> } }
{  Promise { <pending> } }
{ Promise { <pending> } }
{ Promise { <pending> } }
{Promise { <pending> } }
{ Promise { <pending> } }
{ Promise { <pending> } }

Those aren't errors, those are pending promises. Consider the use of Promise.all in [this example] (https://github.com/chadxz/imap-simple/blob/master/README.md#download-all-attachments-from-all-unread-email-since-yesterday)

I used Promise.all(). If I am downloading a few attachments, I will not experience any problem but when downloading a large attachment say 1000 attachment, I will experience that error. After increasing my Nodejs server timeout, It still waits so long with { Promise {  } }.This is my code below. Thanks
const getAttachmentsInOpenBox = async (messages, imapConnection) => {
  let attachmentPromise = [];
  const subjects = messages.map(message => {
    let subject = '### NIL ###';
    let date = '### NIL ###';
    let sender = '### NIL ###';
    let uid = message.attributes.id;

    message.parts.map(async part => {
      subject = part.body.subject[0];
      date = part.body.date[0];
      sender = part.body.from[0];
    });

    let attachmentParts = getParts(message.attributes.struct);
    attachmentPromise = attachmentPromise.concat(
      attachmentParts
        .filter(function(attachmentPart) {
          return attachmentPart.disposition && attachmentPart.disposition.type.toUpperCase() === 'ATTACHMENT';
        })
        .map(async function(part) {
          // retrieve the attachments only of the messages with attachments
          return await imapConnection.getPartData(message, part).then(function(partData) {
            return {
              id: part.partID,
              uid: uid,
              filename: part.disposition.params.filename,
              data: partData,
              sender: sender,
              subject: subject,
              date: date,
            };
          });
        }),
    );
    console.log({ attachmentPromise });
  });
  let attachmentPromises = Promise.all(
    attachmentPromise.map(attachment => {
      console.log({ attachmentPromise: attachment });
      //   return attachment;
      return new Promise((resolve, reject) => {
        resolve(attachment);
      });
    }),
  );

  console.log({ attachmentPromises });
  return attachmentPromises;
};

You need to practice using promises some with simpler code. Your use of await mixed with then illustrates that you don't quite understand these constructs. Once you do that, you'll be better suited to fix your problem.

Ok, I will look into that
On Sat, Jun 20, 2020 at 3:33 PM Chad McElligott<EMAIL_ADDRESS>wrote:

You need to practice using promises some with simpler code. Your use of
await mixed with then illustrates that you don't quite understand these
constructs. Once you do that, you'll be better suited to fix your problem.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/chadxz/imap-simple/issues/77#issuecomment-647003303,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABGIHQBLGU2TJ5GIEOXGWJTRXTCFFANCNFSM4ODKMEDQ
.

After refining my code, I was getting the results. How can I resolve the Promise inside the data
{ id: '2',
       filename: 'EBELE CV.docx',
       data: [Promise],
       sender: 'Ebelechukwu Akpofe<EMAIL_ADDRESS>       subject: 'OFFICE MANAGER: CV',
       date: 'Fri, 12 Jun 2020 07:33:23 +0000 (UTC)' }


Difficult to zoom in and navigate on mobile devices

So I'm using photostream to host a little gallery of my astrophotography; its difficult to zoom into an image on mobile devices because any touch on the screen causes a transition to the previous photo, the next photo, or the gallery view. Is there a way to make the... button areas... be maybe 10%  of the window size so that one can easily pan/zoom on mobile?

Thanks for bringing this up. I know this issue has been first, but let's use #50 to cover this please. Feel free to discuss ideas on the UI side with the other people.


requirePaddingNewlinesBeforeKeywords: Modify special scenarios

Make special scenario for "elseif" perform index searching against a special case object.
Allow "while" that is part of a "dowhile" to succeed
Fixes #1092
@mikesherov Please let me know if you think this is suitable.

Thank you!


Beautiful Async Module

# -*- coding: utf-8 -*-

import requests
from requests import async

urls = [
    'http://www.readability.com',
    'http://tablib.org',
    'http://httpbin.org',
    'http://python-requests.org',
    'http://kennethreitz.com'
]

urls = urls + urls

Standard (14.125s)
for url in urls:
     r = requests.get(url)
     r.content

Standard + Gevent (1.589s)
>>> rs = [async.get(u) for u in urls]
>>> async.map(rs)
[<Response [200]>, <Response [200]>, ...]

Hmm, I think that's about it.


rfc_stune: Add the cgroup module back

4e2673a38945 ("libs/utils/env: move modules definition into board
specific section") trimmed down the list of modules defined for each
testsuite.  For STune, it trimmed it too much, removing the cgroups
module which is needed by the test.  Add it back.

I noticed the same error me to, but not pushed a patch.
Thanks for pushing that update.


Typescript types haven't been generated in the last release

I get the classic error Could not find a declaration file for module 'react-daisyui'
Version 2.4.4 works fine.

I have this same issue. Projects that worked in 2.4.4 are now showing no typing information for react-daisyui, breaking builds.

Thanks for finding this @mattiaz9 and @cefn. I'll take a look at all the commits since 2.4.4 and try to identify the issue!

Hi,
I created a PR 247 to fix this issue introduced by the setup of jest-dom library used by testing in the most recent release.
@benjitrosch do you mind adding a code-owner approval if this looks good?

Fixed by @xaamin's PR. Thanks!


Numbers of Books

Hi, @jigsaw2212 and @muggin
Thanks for sharing the scripts for reproducing the BookSum dataset.
I'm still running the get_works.py and get_summaries.py as it took some time to complete.
I have a few things to confirm:

Meanwhile I read on your paper it says that the BookSum Full contains 436 documents, but in the Gutenberg project Zipped file, it contains only 269 folders (I assume it is corresponding with the number of books). Can you explain the count mismatch?

Could you explain the relationship among numbers of paragraphs, chapters, and books? Do all the paragraphs belong to chapters, and the chapters belong to books?

Cheers

I think I understand now by reading the alignments *.jsonl files.

@geeraay we have multiple book summaries from different sources, for the same book. And you're right about the relationship between the paragraphs, chapters and books.


Add SqlConnection constructor for SqlConnectionStringBuilder directly

Microsoft Connect has been retired so I file my request here:
For me, the SqlConnection constructors are missing the SqlConnectionStringBuilder.
I have to provide new SqlConnection(builder.ConnectionString) which is really weird and not straight forward?
Just allow new SqlConnection(builder) directly.

Please follow https://github.com/dotnet/corefx/blob/master/Documentation/project-docs/api-review-process.md for new API suggestions

Thanks, done
https://github.com/dotnet/corefx/issues/37582


Unable to get Chardata

Authenticating xxx [OK]
Loading characters...                                                                     [OK]
Loading Standard Stash Tab 1...
Loading Standard Stash Tab 1...                                                           [OK][OK]
Loading Standard Stash Tab 2...
Loading Standard Stash Tab 2...                                                           [OK][OK]
Loading Standard Stash Tab 3...
Loading Standard Stash Tab 3...                                                           [OK][OK]
Loading Standard Stash Tab 4...
Loading Standard Stash Tab 4...                                                           [OK][OK]
Loading Standard Stash Tab 5...
Loading Standard Stash Tab 5...                                                           [OK][OK]
Loading Standard Stash Tab 6...
Loading Standard Stash Tab 6...                                                           [OK][OK]
Loading Standard Stash Tab 7...
Loading Standard Stash Tab 7...                                                           [OK][OK]
Loading Standard Stash Tab 8...
Loading Standard Stash Tab 8...                                                           [OK][OK]
Loading Standard Stash Tab 9...
Loading Standard Stash Tab 9...                                                           [OK][OK]
Loading Standard Stash Tab 10...
Loading Standard Stash Tab 10...                                                          [OK][OK]
Loading Standard Stash Tab 11...
Loading Standard Stash Tab 11...                                                          [OK][OK]
Loading Standard Stash Tab 12...
Loading Standard Stash Tab 12...                                                          [OK][OK]
Loading Standard Stash Tab 13...
Loading Standard Stash Tab 13...                                                          [OK][OK]
Loading Standard Stash Tab 14...
Loading Standard Stash Tab 14...                                                          [OK][OK]
Loading Standard Stash Tab 15...
Loading Standard Stash Tab 15...                                                          [OK][OK]
Loading Standard Stash Tab 16...
Loading Standard Stash Tab 16...                                                          [OK][OK]
Loading Standard Stash Tab 17...
Loading Standard Stash Tab 17...                                                          [OK][OK]
Loading xxx inventory...
[Error] Error reading character data for xxx, if you are in offline mode you will need to login and update. Make sure that your account name is entered correctly and with the correct casing. If you received this error while logging in, the authenticated session may have expired or bad data has been returned by GGG or a network issue may have occurred - Please try again.
i don't know what going wrong since i am using session id and forced it to create a new to be sure its not a issue with an old id
i am using the prog for the first time and using win10 x64

Most likely your account name is incorrect, note that it is case sensitive.

i doublechecked that it isnt my name tats wrong

Don't be, it's a common mistake !


Add Collection.clone()

This was discussed the other day. I didn't need it for my earlier pr, so I made it a separate pr.

why not return new Collection(this.entries())

You can also return new this.constructor(this); ... But to be consistent with existing code, (concat) it is written this way.
It was a conscious decision to keep it consistent.

concat has to do that since it iterates in one collection from multiple. new this.constructor(this) would actually be much more efficient, and that is what we want.


XXXX, defined at XXXXX.ts:5, is referenced by XXX but not included in the documentation.

Search terms
"XXXX, defined at XXXXX.ts:5, is referenced by XXX but not included in the documentation."

Question
I have the warning :
Warning: IController, defined at src\IController.ts:5, is referenced by Controller but not included in the documentation.

The file Controller is here : https://github.com/thib3113/unifi-client/blob/main/src/Controller.ts
And I doesn't understand why ...
The interface is correctly exported (not default), correctly imported in Controller ... And finally seems to be included in documentation : https://thib3113.github.io/unifi-client/interfaces/IController.html . ( the documentation page of the Controller : https://thib3113.github.io/unifi-client/classes/Controller.html ) .

I fully thought this was a bug for a few minutes!
You do have an export called IController, but it isn't the IController which Controller implements.
The IController that is documented is defined at src/WebSockets/Interfaces.ts:104
The IController that the Controller class is referencing is defined at src/IController.ts:6
I'm not sure what the proper fix for this one is going to be for your library, if you export the IController from src/IController, then you'll get a different warning due to ISystemEvent.apps, which references the other IController

ohhhhh .... I'm sorry for this ...
I search for a long time without understanding ... Thank you .
For the solution, I've my idea :)

ohhhhh .... I'm sorry for this ...
I search for a long time without understanding ... Thank you .
For the solution, I've my idea :)

hi, i have the same problem now, how do u solve it ? thanks

did u use implements to solve it ?

@carltian in my case, I have two interfaces with the same name, and only one is exported at the root level ... So I rename one of the two, and export the two ...


Add "no" and "razed" to highway filter

Refers to #937

Thanks @nikita03565 this looks good. Could you add "no" and "razed" similarly to the other built-in filters? Then I'll do a final review and merge.

 Can you correct that? Thanks!


Unexpected rendering of ます as 〼 (Japanese)

What is happening
When displaying or entering Japanese text, the Hiragana characters ます (masu) are rendered as the character 〼 (masu mark).
Example:
Actual text
上書きできない項目があります

Rendered text
上書きできない項目があり〼

Additional information
Codepoint: U+303C
https://unicode-explorer.com/c/303C
The usage of this character as a replacement for ます is not, strictly speaking, incorrect. This behaviour, however, is unexpected and undesirable. In cases where the 〼 is desired, it is entered explicitly via the input method.
When entering text in a sequence such as あります, replacement occurs after adding the final す。
e.g.

あ
あり
ありま
あり〼

Deleting the final character す will return the text to render ありま.
Other Instances
I haven't looked hard yet, but I suspect there are other cases of this happening.
Examples

キロ is rendered as ㌔, U+3314 SQUARE KIRO
キログラム is rendered as ㌕  , U+3315 SQUARE KIROGRAM
平成 is rendered as ㍻
株式会社 as ㍿

I suspect all the other characters in the CJK Compatibility Block between U+3300 and U+33FF are behaving in the same way.
Note that MASU 〼 does not belong in the CJK Compatibility Block, and rather comes from the CJK Symbols and Punctuation block.
Environment
This behaviour occurs irrespective of application

neovim
fish
through utilities like rg cat etc

Input Method: the default Japanese IME on Mac OS, using romaji input (not kana)
OS: Mac OS Sonoma 14.2.1
Version: 4f8aa0e3
Build: 5953

@mitchellh

This is a bug (presumably) in the BIZ UDMincho font.

Not a bug. This kind of ligature, which is relatively useless in daily file though, is installed in super decent Japanese fonts, e.g. Yu Gothic/Mincho and Source Han Sans/Serif.
All Ghostty have to do is to designate liga instead of dlig by default. dlig enables uncommon kinds of ligatures. Why has it adopted such a radical option instead?

https://developer.mozilla.org/en-US/docs/Web/CSS/font-variant-ligatures
https://www.adobe.com/uk/creativecloud/illustration/discover/typography/ligatures.html

Other source about dlig:
https://learn.microsoft.com/en-us/typography/opentype/spec/features_ae#dlig

This feature covers those ligatures which may be used for special effect, at the user’s preference.

UI suggestion: This feature should be off by default.

https://github.com/search?q=repo%3Aghostty-org%2Fghostty dlig&type=code

UI suggestion: This feature should be off by default.

Right.

@mitchellh

This is a bug (presumably) in the BIZ UDMincho font.

Not a bug. This kind of ligature, which is relatively useless in daily life though, is installed in super decent Japanese fonts, e.g. Yu Gothic/Mincho and Source Han Sans/Serif.
All Ghostty has to do is to designate liga instead of dlig by default. dlig enables uncommon kinds of ligatures. Why has it adopted such a radical option instead?

In reply to this. I stumbled upon this myself just now, and yeah like you said essentially the fix is to just do the following (which preserves ligatures in codings while disabling Japanese ligatures):
font-feature = -dlig
font-feature = liga

I agree that this should be what Ghostty do instead, but maybe there's other usecase for dlig that is important instead of liga I don't know of.


quiver step not parallizing

Hi there,
I am running the quiver step with the latest version of falcon falcon-2018.08.08-21.41-py2.7-ucs4-beta.tar.gz.
In the cfg file is set the quiver section as follows:
[job.step.unzip.quiver]
njobs=40
NPROC=24
MB=20000
It appears that this is parsed out proper as i can see the following when running fc_unzip.py
options={'JOB_QUEUE': 'hugemem.q', 'job_queue': 'hugemem.q', 'pwatcher_type': 'fs_based', 'use_tmpdir': False, 'MB': '20000', 'job_type': 'SGE', 'submit': 'qsub -S /bin/bash -sync y -V  \\n-q ${JOB_QUEUE}     \\n-N ${JOB_NAME}        \\n-o "${JOB_STDOUT}" \\n-e "${JOB_STDERR}" \\n-pe threads ${NPROC}    \\n-l virtual_free=${MB}M,h_vmem=${MB}M     \\n"${JOB_SCRIPT}"', 'NPROC': '24', 'njobs': '40'})
Yet the pipeline only submits one job at a time waiting for the previous to finish. Any pointers how to fix this?

Please avoid pwatcher_type=fs_based. It's very difficult to maintain. pwatcher_type=blocking is much simpler.

Yet the pipeline only submits one job at a time waiting for the previous to finish.

This was fixed July 31. What version of FALCON are you using? It should be printed at the start, and also at the start of stderr for each task. (Look in the task directories.)
You should see lines like this:
[INFO]Setting max_jobs to 1; was 8

Whenever it gets set to 1, the following section of job-submissions will be sequential. In your case, you should see 40 before the variantCaller jobs are submitted. If you don't you have a software-integration problem. (This is unrelated to the pwatcher_type.)
For now, NPROC is hard-coded at 24 for Arrow. We'll fix that soon. But njobs should work. It works for me.

Thanks. I seemed I found the issue, I think, with people hugging our hugemem queue with only one node left :). I was running the latest 0808 flacon. Should have checked.


clipr not supported in docker image

@clifflun please see the error message from docker image
Error in error_interactive: To run write_clip() in non-interactive mode, either call write_clip() with allow_non_interactive = TRUE, or set the environment variable CLIPR_ALLOW=TRUE

updated, can you try again

https://github.com/bugaevc/wl-clipboard
probably need to install this in the image


feat: generate swagger api

Signed-off-by: Gaius<EMAIL_ADDRESS>
Description

Generate swagger api.

Related Issue

Motivation and Context

Screenshots (if appropriate):
Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] Documentation Update (if none of the other choices apply)

Checklist:

[x] Code compiles correctly.
[x] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[x] I have read the CONTRIBUTING document.
[x] I have added tests to cover my changes.
[x] All new and existing tests passed.

Codecov Report

Merging #557 (bb66887) into main (4ca37e7) will decrease coverage by 0.19%.
The diff coverage is n/a.

:exclamation: Current head bb66887 differs from pull request most recent head 320d0e4. Consider uploading reports for the commit 320d0e4 to get more accurate results

@@            Coverage Diff             @@
##             main     #557      +/-   ##
==========================================
- Coverage   39.53%   39.33%   -0.20%
==========================================
  Files          86       86
  Lines        6556     6556
==========================================
- Hits         2592     2579      -13
- Misses       3593     3607      +14
+ Partials      371      370       -1

Flag
Coverage Δ

e2etests
?

unittests
39.33% <ø> (-0.20%)
:arrow_down:

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

Impacted Files
Coverage Δ

client/daemon/peer/peertask_stream.go
55.05% <0.00%> (-2.44%)
:arrow_down:

client/daemon/peer/peertask_base.go
34.11% <0.00%> (-1.28%)
:arrow_down:


docs: storybook: mdx only

It's surprising to me that --exit-once-uploaded in 091858b does not work out, given its description but that does seem to be subject to interpretation.
Feel free to edit.
Bug: https://phabricator.wikimedia.org/T259694

This is a dated attempt at a - now - solved problem.


Tensorrt

when i use the onnx model by tensorRT,get the error like follows,some suggestions。
While parsing node number 346 [Upsample]:
ERROR: builtin_op_importers.cpp:2026 In function importUpsample:
[8] Assertion failed: mode == "nearest"
[E] Failure while parsing ONNX file
段错误(吐核)

This is not the place for issues with someone else's code. Message them directly, because I have no idea what's going on there (and they're likely not going to see this issue).

Hello @toyal I had the same problem. It is caused because TensorRT does not support the mode 'bilinear' for the Upsample layer, only 'nearest' mode works.
To solve it just change the interpolation mode:

In 'yolact.py' line 577 : proto_downsampled = F.interpolate(proto_downsampled, size=outs[idx].size()[2:], mode='nearest')
In 'config.py' line 363: 'interpolation_mode': 'nearest',

This is not the place for issues with someone else's code. Message them directly, because I have no idea what's going on there (and they're likely not going to see this issu

This is not the place for issues with someone else's code. Message them directly, because I have no idea what's going on there (and they're likely not going to see this issue).

sorry，i don‘t express the issues clearly，i transform the yolact model to onnx and then using
it by tensorrt. i solve the issues now.thx any more

Hello @toyal I had the same problem. It is caused because TensorRT does not support the mode 'bilinear' for the Upsample layer, only 'nearest' mode works.
To solve it just change the interpolation mode:

In 'yolact.py' line 577 : proto_downsampled = F.interpolate(proto_downsampled, size=outs[idx].size()[2:], mode='nearest')
In 'config.py' line 363: 'interpolation_mode': 'nearest',

thanks,.but if i using "near" mode instead of "bilinear" then the accuracy  will be changed.what should i do for this.

That's true, but I suggest you to try to change that configuration and see the results. It does not seem to degrade so much the outputs.
If you find another solution please write me.

@toyal  can yo please let me know how did u convert yolact to onnx format ??
@rpaner  what is the fps you achieved when you use it using tensort ??

@dbolya @toyal @rpaner  guys r u able to share the process of the conversion you have done , it would be helpful

Sorry for not answering before @abhigoku10 I just converted the network to ONNX format thanks to https://github.com/dbolya/yolact/issues/74#issuecomment-513795638 repo changing the "bilinear" modes of upsample layer by "nearest" as I mentioned in a previous comment.
In terms of speed, I achieved 5 fps for resnet-50 architecture with an image input size of 550x550 in the Nvidia Jetson TX2.

@rpaner  i am not able to generate the onnx model would you be able to share it for me for testing and as mentioned in #74 any other procedure you have followed

@abhigoku10 @rpaner can you share conversion process to onnx, or at least, weights for pretrained model on coco/pascal sbd?
Any help will be much appreciated

@poltavski can you share the link where i can upload the onnx model for you i shall share it sorry for the delayed response

@abhigoku10 thank you for reply, could you use https://dropmefiles.com/ for this reason. I re-upload it on google drive, if somebody else needs it
Anything on conversion process, btw? I can convert model to onnx, but accuracy falls

@poltavski  same observation the accuracy falls and there the confidence score of the predicted value is also low compared to without conversion

@poltavski try this https://github.com/simutisernestas/yolact

@simutisernestas what is the FPS on the jetson's (nano & xavier) ?

@sdimantsd nano only ~3fps. Yesterday I've found this: https://github.com/haotian-liu/yolact_edge. It claims 30fps on Xavier :)

@simutisernestas  Wow!!
I will check it, thanks!

@toyal    hi, I could convert the yolact to onnx, but I am afraid of the post process in c++ code,
can you share the code ,  really thankful for it.


Compiling only ts into src/tmp seems to conflict with modularity practices

Issue Overview
Compiling into src/tmp breaks modularity using moduleId: module-id
Issue Description
One emerging school of thought on component modularity seems to be to use
@Component({
    moduleId: module.id,
   ...
})

to allow relative paths to templateUrls. Because this starter compiles into src/tmp,
that practice doesn't work. The absolute path formed by the module.id is in src/app but the code looking for it is in src/tmp and the HTML files cannot be found.
Reproducables
Insert any component using the aforementioned modularity design.
Information

Operating System
Ubuntu

Node version
5.12.0

NPM Version
3.8.6

Environment
Browser

I'm still pretty much a n00b in the Angular 2 space, so I'm not asserting that you should fix it, only that it seems like a design pattern that might be useful.

Since I had to separate transpiled files (along with sourcemaps) so that it didn't get messy with the original TypeScript files, I had to save the output to different directory.
You can try https://github.com/antonybudianto/angular-webpack-starter for relative path (and more features too)

Ok. I'll take a look. I wonder if the build process couldn't simply copy the HTML files into src/tmp as well as the transpiled files.

Sure you can copy them as separate gulp process, I'd love to accept a PR for that

Ok. I'll take that as a challenge :-); I couldn't get the angular-webpack-starter code to run at all.

Well, that turned into a #fail. Configuring html and watch-html tasks was reasonably straightforward. It fixes the bug I reported, but the relative path issue runs deeper than that. Unit tests and build environments  don't work with the moduleId: module.id approach and I fear fixing those problems is well beyond my current skill level. C'est la vie.

@ndw , can you push your work? so I can check it out

Sure, it's in the copyhtml branch here: https://github.com/ndw/angular2-starter/tree/copyhtml
I modified the todolist component to use the moduleId approach. Running it in develop works fine but attempting to build an actual release version fail.

@ndw  I've cherry-picked your commit to my branch, it's now on #177, please try and test it out, thanks!

Yes, that seems to work. Thanks!

I pulled both feat-relativepath and feat-relative-path-1, FYI.

@ndw Ok, I need to improve the copying code first (copy only changed file), then merge it soon


Godot 4 on Android potential memory leak

Godot version
4.0-beta1 on tuxfamily
System information
Galaxy Tab S4, Android 10 (API 29), Adreno 540, Vulkan 1.1.87, RAM 4GB
Issue description
After creating a new Project in the Project-Manager, pressing `Edit' to go into editor.
Selecting Vulkan for Mobile.
My RAM monitor increasing fast and inbetween 5-8 seconds in the 3d view the App crashes.
Propably a memory leak.
Steps to reproduce
See description above.
An empty starter-project works for reproducing.
Minimal reproduction project
No response

I have not be able to reproduce the issue in the latest version of the Godot Android Editor 4.
@PriNova please reopen if the issue still occurs for you.


added check in identify for encrypted office files

added a check to identify encrypted office files prior to sending to extract + added msoffcrypto module to setup.py

Codecov Report

Merging #59 (3805a1b) into master (fd6e4e2) will decrease coverage by 0.29%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##           master      #59      +/-   ##
==========================================
- Coverage   61.48%   61.19%   -0.30%
==========================================
  Files         115      115
  Lines        9893     9841      -52
==========================================
- Hits         6083     6022      -61
- Misses       3810     3819       +9

Impacted Files
Coverage Δ

assemblyline/common/identify.py
0.00% <0.00%> (ø)

assemblyline/filestore/transport/sftp.py
26.60% <0.00%> (-5.02%)
:arrow_down:

assemblyline/filestore/transport/ftp.py
18.47% <0.00%> (-3.96%)
:arrow_down:

assemblyline/datasource/alert.py
88.46% <0.00%> (-3.85%)
:arrow_down:

assemblyline/datastore/__init__.py
59.20% <0.00%> (-1.56%)
:arrow_down:

assemblyline/datastore/helper.py
24.80% <0.00%> (-1.27%)
:arrow_down:

assemblyline/run/cli.py
23.24% <0.00%> (-1.27%)
:arrow_down:

assemblyline/datasource/common.py
70.37% <0.00%> (-1.06%)
:arrow_down:

assemblyline/filestore/transport/http.py
64.55% <0.00%> (-0.45%)
:arrow_down:

assemblyline/datastore/stores/es_store.py
70.89% <0.00%> (-0.17%)
:arrow_down:

... and 4 more

Continue to review full report at Codecov.

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

Codecov Report

Merging #59 (3805a1b) into master (fd6e4e2) will decrease coverage by 0.29%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##           master      #59      +/-   ##
==========================================
- Coverage   61.48%   61.19%   -0.30%
==========================================
  Files         115      115
  Lines        9893     9841      -52
==========================================
- Hits         6083     6022      -61
- Misses       3810     3819       +9

Impacted Files
Coverage Δ

assemblyline/common/identify.py
0.00% <0.00%> (ø)

assemblyline/filestore/transport/sftp.py
26.60% <0.00%> (-5.02%)
:arrow_down:

assemblyline/filestore/transport/ftp.py
18.47% <0.00%> (-3.96%)
:arrow_down:

assemblyline/datasource/alert.py
88.46% <0.00%> (-3.85%)
:arrow_down:

assemblyline/datastore/__init__.py
59.20% <0.00%> (-1.56%)
:arrow_down:

assemblyline/datastore/helper.py
24.80% <0.00%> (-1.27%)
:arrow_down:

assemblyline/run/cli.py
23.24% <0.00%> (-1.27%)
:arrow_down:

assemblyline/datasource/common.py
70.37% <0.00%> (-1.06%)
:arrow_down:

assemblyline/filestore/transport/http.py
64.55% <0.00%> (-0.45%)
:arrow_down:

assemblyline/datastore/stores/es_store.py
70.89% <0.00%> (-0.17%)
:arrow_down:

... and 4 more

Continue to review full report at Codecov.

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


Env variable for default key buffer size

Continuation from: https://github.com/spulec/moto/pull/3994

Codecov Report

Merging #4001 (1f9178b) into master (b670962) will increase coverage by 0.01%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master    #4001      +/-   ##
==========================================
+ Coverage   94.88%   94.89%   +0.01%
==========================================
  Files         454      454
  Lines       49513    49695     +182
==========================================
+ Hits        46982    47160     +178
- Misses       2531     2535       +4

Flag
Coverage Δ

servertests
55.72% <100.00%> (+0.03%)
:arrow_up:

unittests
94.77% <100.00%> (+0.01%)
:arrow_up:

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

Impacted Files
Coverage Δ

moto/s3/models.py
95.72% <100.00%> (-0.01%)
:arrow_down:

moto/settings.py
100.00% <100.00%> (ø)

moto/logs/responses.py
93.61% <0.00%> (-0.14%)
:arrow_down:

moto/elbv2/models.py
89.05% <0.00%> (-0.01%)
:arrow_down:

moto/elbv2/responses.py
94.83% <0.00%> (ø)

moto/organizations/utils.py
100.00% <0.00%> (ø)

moto/ssm/responses.py
99.01% <0.00%> (+0.01%)
:arrow_up:

moto/organizations/models.py
98.27% <0.00%> (+0.02%)
:arrow_up:

moto/ssm/models.py
94.01% <0.00%> (+0.15%)
:arrow_up:

... and 5 more

Continue to review full report at Codecov.

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


Fixes #116: Reuse OkHttpClient Instance

As pointed out in #116, OkHttpClient instances should be reused. With the @ApplicationScope annotation, I think OkHttpClient etc instances will be automatically reused by Dagger, not certain on that though

All OkHttpClient, Retrofit and Gson dependencies are now provided by Dagger
Removed all other occurrences of OkHttpClient creation
Minor refactor I missed in #100

Not sure if what I've done with the classes is considered best practice - for example, for LiveWikiRepository, I moved the API and response types into a Wiki class, so the API is accessed with Wiki.Api and response types with Wiki.SomeResponseType

That's quite common, but I think it should be done the other way around, e.g we have a WikiApi , the token is a WikiApi.Token because the token is specific to this api. Similar to "package by feature".

Also unsure if I should continue to use @ApplicationScope for the API instances

The annotation causes the provided instance to last as long as the corresponding dagger component  stays alive, with ApplicationScope this usually refers to the dagger component referenced from App. Because the api instances never change, we can keep this annotated as we don't need different instances during the lifetime of the app.

@d4rken if this looks good I'll merge?

lgtm


cmd/compile: performance regression compiling crypto/internal/edwards25519/field.feMulGeneric on PPC64

The change to support carry flag optimization on ARM64 (go.dev/cl/432275) caused a large regression in the crypto/ed25519 benchmarks. I missed this regression when reviewing the patch.
The optimization created by using the CarryChainTail priority fails when schedulable memory ops get intertwined with schedulable CarryChainTail ops like ADDZ on PPC64. This leads to many avoidable CA bit clobbers.
What did you expect to see?
No difference in benchmark tests.
What did you see instead?
name              old time/op  new time/op  delta
KeyGeneration-2   27.2µs ± 0%  29.6µs ± 0%   +9.05%  (p=1.000 n=1+1)
NewKeyFromSeed-2  23.9µs ± 0%  29.4µs ± 0%  +23.13%  (p=1.000 n=1+1)
Signing-2         33.7µs ± 0%  37.2µs ± 0%  +10.36%  (p=1.000 n=1+1)
Verification-2    68.3µs ± 0%  88.8µs ± 0%  +30.06%  (p=1.000 n=1+1)

In triage, context from @cherrymui, @pmur is already looking into it so assigning it to you.


Run ignition-disks earlier to avoid race condition

Make sure to run before the file system checks, as sgdisk will trigger
udev events, potentially resulting in race conditions due to disappearing
devices. Executing earlier will avoid this and hopefully similar problems.
Fixes #114.

Tested and confirmed this works. @cgwalters How does this affect your safe/restore root work?

@cgwalters How does this affect your safe/restore root work?

I think it should still work fine offhand - the save work just needs to run Before=ignition-disks.service, and should be OK ordering even earlier.


Resource files issue

Hi, I add *.resx file to my project, it also create *.Designer.cs. And analyzer don't skip this file
File:
//------------------------------------------------------------------------------
// <auto-generated>
//     This code was generated by a tool.
//     Runtime Version:4.0.30319.42000
//
//     Changes to this file may cause incorrect behavior and will be lost if
//     the code is regenerated.
// </auto-generated>
//------------------------------------------------------------------------------

namespace TempProj {
    using System;

    /// <summary>
    ///   A strongly-typed resource class, for looking up localized strings, etc.
    /// </summary>
    // This class was auto-generated by the StronglyTypedResourceBuilder
    // class via a tool like ResGen or Visual Studio.
    // To add or remove a member, edit your .ResX file then rerun ResGen
    // with the /str option, or rebuild your VS project.
    [global::System.CodeDom.Compiler.GeneratedCodeAttribute("System.Resources.Tools.StronglyTypedResourceBuilder", "<IP_ADDRESS>")]
    [global::System.Diagnostics.DebuggerNonUserCodeAttribute()]
    [global::System.Runtime.CompilerServices.CompilerGeneratedAttribute()]
    internal class Data {

        private static global::System.Resources.ResourceManager resourceMan;

        private static global::System.Globalization.CultureInfo resourceCulture;

        [global::System.Diagnostics.CodeAnalysis.SuppressMessageAttribute("Microsoft.Performance", "CA1811:AvoidUncalledPrivateCode")]
        internal Data() {
        }

        /// <summary>
        ///   Returns the cached ResourceManager instance used by this class.
        /// </summary>
        [global::System.ComponentModel.EditorBrowsableAttribute(global::System.ComponentModel.EditorBrowsableState.Advanced)]
        internal static global::System.Resources.ResourceManager ResourceManager {
            get {
                if (object.ReferenceEquals(resourceMan, null)) {
                    global::System.Resources.ResourceManager temp = new global::System.Resources.ResourceManager("TempProj.Data", typeof(Data).Assembly);
                    resourceMan = temp;
                }
                return resourceMan;
            }
        }

        /// <summary>
        ///   Overrides the current thread's CurrentUICulture property for all
        ///   resource lookups using this strongly typed resource class.
        /// </summary>
        [global::System.ComponentModel.EditorBrowsableAttribute(global::System.ComponentModel.EditorBrowsableState.Advanced)]
        internal static global::System.Globalization.CultureInfo Culture {
            get {
                return resourceCulture;
            }
            set {
                resourceCulture = value;
            }
        }

        internal static string SomeText {
            get {
                return ResourceManager.GetString("SomeText", resourceCulture);
            }
        }
    }
}

Do You have some ideas?

In most cases where this gets reported, it's not actually StyleCop Analyzers which is reporting warnings but some other analyzer.

Diagnostics with IDs starting with SA and SX are probably StyleCop Analyzers
Diagnostics starting with IDE are from analyzers built in to the Visual Studio IDE
Diagnostics starting with CS are typically reported by Roslyn

Other diagnostics are possible as well. Do you have any more information about the specific diagnostics which you are seeing in the file?

I have several issues:

CS8019 Unnecessary using directive
IDE0005 Using directive is unnecessary
IDE0001 Name can be simplified

@AndrosovIgor Yes, all of those are reported by other projects. :smile: / :frowning:
If any SA issues show up in the file, please feel free to reopen this issue!


movetofirst flag and startsection

Would be nice to have the following flags/arguments:
 movetofirst: 1,
//the first loaded section upon page load is moved to with an animation and fadein. This way before:function() is also called for the first loaded section.
startsection: "#test1",
//set the starting section. Could be a number or the sectionName.

It would be great if author can implement this feature, but for the alternative, you can call $.scrollify.move(number); when the DOM is ready.


e2e/ci: check failure in some RKE2 upgrade tests

Some tests are failing while trying to check/sync ManagedOSVersionChannel:

https://github.com/rancher/elemental/actions/runs/7088838627/job/19294109808
https://github.com/rancher/elemental/actions/runs/7095355252/job/19312144574

Operator issue, follow here: https://github.com/rancher/elemental-operator/issues/577.
So this one can be closed.


Upgrade couchbase libraries

Upgrade couchbase-client to 2.5.8
Upgrade couchbase-testcontainers to 1.2 to resolve issues with fixed ports
Fixes https://github.com/jhipster/generator-jhipster/issues/7654#issuecomment-391986523

Release 2.0.8 is done, let's wait for Maven Central to sync

@tchlyah couchbase-client 2.5.9 is released but I dare not upgrade without checking with you first...

I've just test it, everything is working well. You can launch the upgrade! Thank you.


The task role arn is not getting updated

I am getting the following error while running terraform apply
Plan: 1 to add, 1 to change, 1 to destroy. module.ecs.module.ecs.module.service["worker"].aws_ecs_task_definition.this[0]: Creating... ╷ │ Error: creating ECS Task Definition (worker): ClientException: Container.image should not be null or empty. │  │   with module.ecs.module.ecs.module.service["worker"].aws_ecs_task_definition.this[0], │   on .terraform/modules/ecs.ecs/modules/service/main.tf line 609, in resource "aws_ecs_task_definition" "this": │  609: resource "aws_ecs_task_definition" "this" { │  ╵
Code:
`module "ecs" {
source  = "terraform-aws-modules/ecs/aws"
version = "5.2.2"
cluster_name = "terraform"
cluster_configuration = {
execute_command_configuration = {
logging = "OVERRIDE"
log_configuration = {
cloud_watch_log_group_name = "/aws/ecs/aws-ecs"
}
}
}
fargate_capacity_providers = {
FARGATE = {
default_capacity_provider_strategy = {
weight = 50
}
}
FARGATE_SPOT = {
default_capacity_provider_strategy = {
weight = 50
}
}
}
services = {
worker = {

  enable_execute_command = true

  cpu        = var.worker_cpu
  memory     = var.worker_memory

  container_definitions = {

    tasks_iam_role_arn = "arn:aws:iam::411531107052:role/ecsTaskExecutionRole"

    worker = {
      cpu        = var.worker_container_cpu
      memory     = var.worker_container_memory
      essential  = true
      image      = "311231107042.dkr.ecr.us-east-1.amazonaws.com/worker:322"
      port_mappings = [
        {
          containerPort = 80
          protocol      = "tcp"
        }
      ]

      entrypoint = ["/usr/src/app/common/CICD/init_worker.sh", "laravel-workers"]

      readonly_root_filesystem = false

      log_configuration = {
        logDriver = "awslogs"
        options = {
            awslogs-group = "/ecs/workers"
            awslogs-region = var.region
            awslogs-stream-prefix = "ecs"
        }
      }

    }

  }

  load_balancer = {
    service = {
      target_group_arn = var.target_group_arn
      container_name   = "worker"
      container_port   = 80
    }
  }

  subnet_ids = var.subnet_ids
  assign_public_ip = true

  security_group_ids = var.security_group_ids

}

}
}
module "ecs_service" {
source = "terraform-aws-modules/ecs/aws//modules/service"
create = false
create_iam_role = false
create_task_definition = false
create_task_exec_iam_role = false
create_task_exec_policy = false
create_tasks_iam_role = false
create_security_group = false
}`
Could anyone help me please?

I also see an issue with task role arn is not getting used.
module "ecs_service_definition_2" {
  source  = "terraform-aws-modules/ecs/aws//modules/service"
  version = "~> 5.2.2"
  create_iam_role        = false
  tasks_iam_role_arn     =  aws_iam_role.task.arn
---
}

I expect the already existing task ARN to be used.  But a new task Role is created with empty IAM permissions.

I see.
Thank you for letting me know.


No way to change order of icons in FA::stack()

In trying to re-create the cameras banned example at Font Awesome Examples I've found that yii2-fontawesome doesn't offer a way to change the order of the items in an icon stack.
This image shows my results...

The code from the examples page is...

& gives the first icon in the image.
If I assemble my own stack with...

		<?= FA::i('camera')->addCssClass('fa-stack-1x') ?>
		<?= FA::i('ban')->addCssClass('fa-stack-2x')->addCssClass('text-danger') ?>

I get the 2nd icon in the image.
Using...
	<?php
		$camera = 'camera';
		$ban = FA::i('ban')->addCssClass('text-danger');
		$options = ['class' => 'fa-lg'];
		echo FA::stack($options)->on($ban)->icon($camera);
	?>

or...
	<?php
		$camera = 'camera';
		$ban = FA::i('ban')->addCssClass('text-danger');
		$options = ['class' => 'fa-lg'];
		echo FA::stack($options)->icon($camera)->on($ban);
	?>

gives HTML...

& the 3rd icon in the image.
The difference is only the order of the 2 icons. In the Font Awesome example, 'camera' with class 'fa-stack-1x' is the first icon, & 'ban' with class 'fa-stack-2x' is the 2nd (top, & also larger) icon.

I apologize for the long wait.
Your problem is solved in version 2.15.1
$camera = 'camera';
$ban = FA::i('ban')->addCssClass('text-danger');
$options = ['class' => 'fa-lg', 'template' => '{front}{back}'];
echo FA::stack($options)->icon($camera)->on($ban);

Beaut & thanks. I'll now go update my own work :)


[Fix #11628] Add new Style/DirEmpty cop

Fix #11628
# bad
Dir.entries('path/to/dir').size == 2
Dir.children('path/to/dir').empty?
Dir.children('path/to/dir').size == 0
Dir.each_child('path/to/dir').none?

# good
Dir.empty?('path/to/dir')

Before submitting the PR make sure the following are checked:

[x] The PR relates to only one subject with a clear title and description in grammatically correct, complete sentences.
[x] Wrote good commit messages.
[x] Commit message starts with [Fix #issue-number] (if the related issue exists).
[x] Feature branch is up-to-date with master (if not - rebase it).
[x] Squashed related commits together.
[x] Added tests.
[x] Ran bundle exec rake default. It executes all tests and runs RuboCop on its own code.
[x] Added an entry (file) to the changelog folder named {change_type}_{change_description}.md if the new code introduces user-observable changes. See changelog entry format for details.

Thanks!


[Go][CI] Nightly verification tasks are failing with syntax error on thrift

Describe the bug, including details regarding any error messages, version, and platform.
Several nightly verification tasks:

verify-rc-source-go-linux-almalinux-8-amd64
verify-rc-source-go-linux-conda-latest-amd64
verify-rc-source-go-linux-ubuntu-20.04-amd64
verify-rc-source-go-linux-ubuntu-22.04-amd64
verify-rc-source-go-macos-arm64
verify-rc-source-integration-linux-almalinux-8-amd64
verify-rc-source-integration-linux-conda-latest-amd64
verify-rc-source-integration-linux-ubuntu-20.04-amd64
verify-rc-source-integration-linux-ubuntu-22.04-amd64
verify-rc-source-integration-macos-arm64
verify-rc-source-integration-macos-conda-amd64

are failing with:
 # github.com/apache/thrift/lib/go/thrift
/tmp/arrow-HEAD.5MzaX/go/gopath/pkg/mod/github.com/apache/thrift@v0.17.0/lib/go/thrift/deserializer.go:83:12: syntax error: unexpected [, expecting semicolon or newline or }
/tmp/arrow-HEAD.5MzaX/go/gopath/pkg/mod/github.com/apache/thrift@v0.17.0/lib/go/thrift/pointerize.go:24:13: syntax error: unexpected [, expecting (
/tmp/arrow-HEAD.5MzaX/go/gopath/pkg/mod/github.com/apache/thrift@v0.17.0/lib/go/thrift/pool.go:28:13: syntax error: unexpected any, expecting ]
/tmp/arrow-HEAD.5MzaX/go/gopath/pkg/mod/github.com/apache/thrift@v0.17.0/lib/go/thrift/pool.go:30:16: syntax error: unexpected newline, expecting name or (
/tmp/arrow-HEAD.5MzaX/go/gopath/pkg/mod/github.com/apache/thrift@v0.17.0/lib/go/thrift/pool.go:31:1: syntax error: non-declaration statement outside function body
/tmp/arrow-HEAD.5MzaX/go/gopath/pkg/mod/github.com/apache/thrift@v0.17.0/lib/go/thrift/pool.go:38:13: syntax error: unexpected [, expecting (
/tmp/arrow-HEAD.5MzaX/go/gopath/pkg/mod/github.com/apache/thrift@v0.17.0/lib/go/thrift/pool.go:38:37: method has no receiver
/tmp/arrow-HEAD.5MzaX/go/gopath/pkg/mod/github.com/apache/thrift@v0.17.0/lib/go/thrift/pool.go:40:21: method has no receiver
/tmp/arrow-HEAD.5MzaX/go/gopath/pkg/mod/github.com/apache/thrift@v0.17.0/lib/go/thrift/pool.go:40:21: syntax error: unexpected *, expecting name or (
/tmp/arrow-HEAD.5MzaX/go/gopath/pkg/mod/github.com/apache/thrift@v0.17.0/lib/go/thrift/pool.go:46:16: method has no receiver
/tmp/arrow-HEAD.5MzaX/go/gopath/pkg/mod/github.com/apache/thrift@v0.17.0/lib/go/thrift/pool.go:46:16: too many errors
note: module requires Go 1.18
github.com/golang/snappy
github.com/klauspost/compress/flate
github.com/klauspost/compress/gzip
github.com/JohnCGriffin/overflow
github.com/apache/arrow/go/v14/parquet/internal/bmi
golang.org/x/exp/mmap
github.com/docopt/docopt-go
golang.org/x/sync/errgroup
github.com/davecgh/go-spew/spew
github.com/pmezard/go-difflib/difflib
github.com/stretchr/objx
gopkg.in/yaml.v3
github.com/stretchr/testify/assert
github.com/stretchr/testify/mock
Failed to verify release candidate. See /tmp/arrow-HEAD.5MzaX for details.

Component(s)
Continuous Integration, Go

cc @zeroshade

This seems to fail since we bumped the minimum go version: https://github.com/apache/arrow/commit/2a56d4598a3a401447c91645448853b940e86fbe

@zeroshade if it's not a quick fix for you I can try to investigate next week.

@raulcd ugh, it looks like it's still installing Go 1.17 instead of 1.19, I'll see if i can fix the verification scripts to get the right version


fix: 修复登出失败，登录token未做unquote

修复登出失败，登录token未做unquote

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


Twitter agents suddenly not pulling tweets using list searching

Hey team,
Not sure where to start. I have two Huginn installs running on two Ubuntu VMs (one in Rackspace and one in VMWare locally) with several Twitter agents that are posting to Discord webhooks. About 24 hours ago, they just stopped pulling new items. There are no errors in the logs. The agent seems to run fine but nothing is generated. API keys are correct and I just updated to the latest release. The Rackspace instance is quite large with around 200 agents configured.
I haven't really had to touch anything on either deployment for months so I know nothing has changed on my end.
What can I provide to help troubleshoot this?
Update: I am able to successfully pull tweets from a single account (from:username). List searching seems to have stopped functioning. (i.e. list:username/listname).

It's been 12 days. Is Huginn still supported here?

I'm not sure what changed or even what happened but the agents I have set up for list searching suddenly started pulling again. It's been 24 hours since this started and they seem to be stable. Hopefully it is a temporary issue that has since been resolved. Perhaps Twitter API?

Could be.  None of me have heard anything about it, but birbsite isn't the greatest about status reports that don't involve complete system failures.

Sorry I don't know, we didn't change anything and I don't use twitter lists. Let's hope it was just a glitch in the twitter API.

Sorry I don't know, we didn't change anything and I don't use twitter lists. Let's hope it was just a glitch in the twitter API.

Yeah most likely. Still working fine today. Thanks for the response.