Aggregate Class in Global Namespace

It seems that a dependency (aggregate) is defining a class on the global namespace (namely Aggregate). If you have a model or some other class defined as Aggregate in your project the version included by this gem may cause conflicts and lead to some confusion, especially with ruby's ability to reopen classes later.
It seems that aggregate is included by librato-metrics, so this may not be an issue that can be fixed in this repo.

@iterion This is true. I don't know of a way to require a gem in ruby such that it can't import things into the global namespace. The gem in question is an important dependency so removing it isn't likely.
The same is true of our other dependencies, faraday, multi_json, etc. It is just less likely you'll reuse those names and collide with them.
Is namespacing classes in your project an option?

Namespacing is an option, but not really preferred (dev effort to rename > integrating with another service). Aggregate seems like it could exist in many rails projects, especially as a name for a model. I'd really like to get Librato working, but the conflict is unfortunate and means we won't be able to move forward without additional effort.
I looked around at the code to see what it would take to remove the dependency and it appears that it's only referenced by Librato::Metrics::Aggregator. Is it possible that Aggregate could be forked by librato and namespaced under Librato's namespace (Librato::Metrics::Aggregate)? I'm not sure how many other projects are using the aggregate gem so I'm not sure how responsive they would be to namespacing within that gem and I'm not even sure what a good namespace would be for it.

I would second this one (if this is still an issue.) Would this problem also appear when I am not using client side aggregates at all?


prepare-release: Fix ImportError

Fixes the following error:
  File "/mnt/workspace/jenkins/working/_cd-builds_build_prepare-release/pyartcd/pyartcd/__main__.py", line 4, in <module>
    from pyartcd.pipelines import (prepare_release, promote, rebuild,
  File "/mnt/workspace/jenkins/working/_cd-builds_build_prepare-release/pyartcd/pyartcd/pipelines/prepare_release.py", line 27, in <module>
    from pyartcd.util import (get_assembly_basis, get_assembly_type,
ImportError: cannot import name 'go_suffix_for_arch'

It turns out go_suffix_for_arch was removed from pyartcd.util by this commit: https://github.com/openshift/aos-cd-jobs/commit/1a2e45c4641212cd98f965e7e7ff422a6b0d4622#diff-6a68c2ae2c26daa692da9c2b42a8e7a6d88d9e73f7b505e4afddbb8c0dfc673eR8-R10
When I worked on this PR https://github.com/openshift/aos-cd-jobs/pull/3028, I didn’t have that above commit pulled.
This PR fixes the issue by importing go_suffix_for_arch directly from
doozerlib.util.

/lgtm


Allow customisations of the OIDC discovery settings

This PR allows more customisations on the OidcServerDiscoverySettingsFactory class.

Backported to the 6.6.x stream: https://github.com/apereo/cas/pull/5914


host: Store logs in /var/log/flynn rather than /tmp

/tmp is commonly a tmpfs so rebooting a node can lose logs which are helpful for debugging purposes.

This requires a change to CI: https://github.com/flynn/flynn/blob/c0a5adc07ae7cdc1cab2f9712405d1b0de941ce8/test/cluster/cluster.go#L630

@titanous comments addressed

LGTM


Are any of these npm install warnings important?

When I run npm install after glocing the map repo, I get the warnings below.  Are any of them important?
rsignell@igsagiegwsrsil0:~/map$ npm install
npm WARN package.json<EMAIL_ADDRESS>No license field.
npm WARN engine<EMAIL_ADDRESS>wanted: {"node":">=0.6.x","npm":"1.1.x"} (current: {"node":"0.12.5","npm":"2.11.2"})
npm WARN peerDependencies The peer dependency stream-browserify@* included from bl will no
npm WARN peerDependencies longer be automatically installed to fulfill the peerDependency
npm WARN peerDependencies in npm 3+. Your application will need to depend on it explicitly.
npm WARN optional dep failed, continuing<EMAIL_ADDRESS>npm WARN optional dep failed, continuing<EMAIL_ADDRESS>

No, those should all be harmless.  And I think only the first is under our control.


Forum Link missing from readme.md

The readme.md has a section that states "To suggest new features, please visit the YNAB Forum thread here and comment" however I'm not sure where "here" and the forum is.  Can you update the link?

Thank you very much for letting us know. I'll fix that today.


Add/improve type declarations for PHP 8.1

cakephp-codesniffer 5 is meant to be paired with cakephp 5 code. One of the key changes is requiring native type declarations where possible.

@MasterOdin You can override the header settings for phinx from the cakephpcs defaults now if you want. I don't think there will be any large-scale changes after this.


Remove the Contributors part or hide it in a Menu

Are you going to keep the top menu of the app window? If you do, maybe we can add the credits under "Help/About".

In general, we are planning to re-think the whole UI.
So please do not take any current UI decisions as long term decisions.
But my suggestion is to think about components.
Even if we are thinking to change the UI, we still can decide if we want an area for "About", so you can start to create this "area" as component and put it somewhere. After the UI change, we will just put it in another place.
But since this is your first contribution, maybe it will be better to just remove the Contributors.
@brafdlog Need you here.

In general, we are planning to re-think the whole UI.
So please do not take any current UI decisions as long term decisions.
But my suggestion is to think about components.
Even if we are thinking to change the UI, we still can decide if we want an area for "About", so you can start to create this "area" as component and put it somewhere. After the UI change, we will just put it in another place.
But since this is your first contribution, maybe it will be better to just remove the Contributors.
@brafdlog Need you here.

Sure I can remove.
I don't know electron yet, but my guess is that the top menus are not part of the web content. So if we decide to keep the menus and add the info there, there's no point in creating a web component.

@giladtaase welcome 🙂
As @baruchiro wrote this ui will undergo a significant change.
Currently our main goal is to have a functioning app that we can merge and release as a beta version. We will iterate on further improvements after we have a working version.
Since this a small chunk of ui that would be easy to create again in a newer design, I think we should just remove it at this stage.

No problem I'll remove it.


Auto-publish VS Code extension via CI/CD

Currently, the VS Code extension is only published manually by me when I get around to it, and I noticed the syntax highlighting in the published extension is currently out of date with respect to what is on the master branch.
We should set it up so that the extension is auto-published on each commit by a CI/CD process.

Since we are using CircleCI, using this Orb might be a good approach: https://circleci.com/orbs/registry/orb/uraway/vsce

I'll work on this.

This is done, and the latest version is published as 0.4.1
All we have to do is bump the version in tooling/vscode/package.json, and CircleCI will do the rest.
The publish token will need to be renewed after 90 days.


no need to publish DD agent container port to the host

Thanks for the PR Ingrid. I reworked how it all works together, I think it should be ok to cancel this.


Use both: keypoints + refinementMLP ?

I can see that match_xfeat and match_xfeat_star use different strategies for fine grained feature matching:
match_xfeat
On one hand, match_xfeat uses the model keypoints output (fine-grained heatmap) to first extract the mkpts and then use those mkpts to bicubic interpolate the descriptors (aka feats) to the correct offest.
match_xfeat_star
On the other hand match_xfeat_star, totally ignores the model keypoints output. match_xfeat_star get the correct offsets by using the refinementMLP on the raw model feats (no feats interpolation).
My idea: Use both for better performance ?

Use keypoints and interpolate feats
Use refinementMLP to offest mkpts

match_xfeat
✅
❌

match_xfeat_star
❌
✅

Use both
✅
✅

My idea to (maybe) achieve a better accuracy is to use the refinementMLP on the bicubic-interpolated feats to further correct the offset.

Hi @javiabellan, thanks for the suggestions! I'm not sure if the refinement module will work seamlessly with bicubically interpolated features to produce sub-pixel correspondences, but it would be interesting to try! I might attempt this in the near future, but feel free to try it yourself and share the results here!

For me match_xfeat_star have some problems.
I can't undestand how "refinementMLP to offest mkpts" works: when the refine calculate the offsets, they are multiply for the scales and add to the kpts, but only for mkpts0. If you try to execute match_xfeat_star on identical images the output of this function gives different keypoints, which doesn't make sense, right? Or am I wrong?

Hi @ve-cattozzi, I'm not sure if I understood correctly.
Do you mean that matching the same image will produce 'different keypoints' relative to what?
Best,
Guilherme.

Hi @guipotje, exactly!
If you call match_xfeat_star with im_set1 equal to im_set2 during the self.refine_matches it calls
mkpts_0 += offsets* (sc0[:,None]) #*0.9 #* (sc0[:,None])
and offsets are not all zeros (what I m expect if the images are the same) modify the position of "mkpts_0".

@ve-cattozzi  I think I understand now! Indeed, predictions from image1 to image2 (where image1 == image2) likely won't have a non-zero offset because the network learned offsets that lead to better correction on average. Since we are trying to recover offsets at a much coarser resolution (1/8), it is difficult for the network to perfectly predict the correct offsets even if the images are the same.

@guipotje ty!

Hi, I would like to ask about match_xfeat_star. It is by using refinementMLP on original Model feats to get the correct offset. After looking at the code, rough feature point extraction and matching are two images based on the upper left pixel of each 8×8 mesh slice. So why is the resulting refinement offset only added on pts1 and not pts2? Why is the supervised (true) value of the offset refinement not the relative offset of the two matching points on their respective 8×8 grids?


Set Correct Platform Tag in Wheels on Mac OS with Python 3.10

Intended to fix https://github.com/grpc/grpc/issues/28387
After much experimentation (seriously, look at the commit history), I found that the "ARCHFLAGS" approach just wasn't working. Instead, I simply rename the artifact. This has been tested to work on an M1 mac running with Rosetta.
This is a hack. Long term, we'll need to figure out why distutils is not generating the proper platform and fix that. We'll also want to generate both arm64 artifacts and a proper universal2 artifact containing both x64 and arm64 artifacts. However, this will unbreak M1 users in the short term.
Side note: This PR also updates the Python 3.10 patch version installed on MacOS.

@gnossen , should we expect a new release of grpcio (and the other grpc libraries) in order to pick up the change in https://github.com/grpc/grpc/pull/29857?

Yes. I'm expecting expecting a release candidate later today (pacific time) and 1.47.0 will follow in a couple of days.


Simple example

Where can I find a simple example using this PGP crate, encrypting something simple like "Hello, World!" and then decrypting that again?
That would be of great help for newbies!
Thanks!

@wimvelzeboer take a look at this one: https://github.com/granitDev/rPGP-Working-Examples

This is a simplified version of code that is working for me to decrypt a message and check its signature:
n decrypt_and_verify(
    sender_key: &SignedPublicKey,
    receiver_key: &SignedSecretKey,
    receiver_passphrase: Option<&str>,
    ciphertext: &[u8],
) -> Vec<u8> {
    let encrypted_message = pgp::Message::from_bytes(Cursor::new(ciphertext)).unwrap();
    let (mut decrypter, _recipients) = encrypted_message
        .decrypt(
            || receiver_passphrase.unwrap_or_default().to_string(),
            &[receiver_key],
        )
        .unwrap();
    let decrypted_message = decrypter.next().unwrap().unwrap();
    decrypted_message.verify(sender_key).unwrap();
    decrypted_message.get_content().unwrap().unwrap()
}


Split out rewards from protected positions

Recently saw a lot of people confused on Discord about the fully protected and claimable values on the "My Protected Positions" table.  (See for example this latest thread https://discord.com/channels/476133894043729930/764907781462425680/812612527103934466)
I had some issues with it myself, mostly:

When you have multiple positions in the same pool, the sum does not add up, because on the top level the rewards are included.
Pending rewards being shown as a part of the "My Stake" summary, while those rewards are not staked and not accruing yield.
ROI drops after claiming fees.

This PR is my attempt at making the UI more intuitive, each topic has its separate commit and the commit messages go more in depth in the reasoning behind the change.
I am happy to contribute to this project, hope you'll consider this PR and I am of course open to feedback!

I am gonna merge this into a feature branch on our repo for further testing and valuation


Joining keywords in trend data

Joining keywords up in groups up to 5 in order to decrease number of requests sent (and thus decrease the likelihood of a quota limit increase) ⚡

Coverage increased (+1.6%) to 77.941% when pulling 4eafdcb0cefad0da1c9f2dbabc93d63d76d7a162 on joiningKeywordsInTrendData into 9bdadb04aba0d0a0b15d84c2ace34401bcf20489 on master.

Coverage increased (+1.6%) to 77.941% when pulling 4eafdcb0cefad0da1c9f2dbabc93d63d76d7a162 on joiningKeywordsInTrendData into 9bdadb04aba0d0a0b15d84c2ace34401bcf20489 on master.


[bug] signOutOnNetworkChange: false causes hang on network switch in injected MetaMask

Link to minimal reproducible example
http://localhost:3000/library/wagmi-all/
Summary
Using the signOutOnNetworkChange: false option and injected MetaMask, if you switch chains and approve in MetaMask the modal will hang on "Approve in wallet" after approving the network switch in MetaMask.
Can reproduce in lab by changing this setting. Video reproduction
List of related npm package versions
latest

This issue seems to be solve in the latest packages.


[FEATURE] Use native instruction optimized CRC32C to improve batch.ensureValid()

Describe the solution you'd like
Use Crc32cLongChecksum from bookkeeper, instead of Crc32C.compute. the Crc32cLongChecksum uses SSE 4.2 to optimize the CRC32C algorithm, it might have 6 times better performance.

I do regular OMB tests of Pulsar clusters including with a version of KoP. We tend to get high end to end latency which appears to be more a function of the message size than it is the throughput. I looked at the flamegraphs from a flight recording and i noticed that a lot of time (over 20%) is spent in java.util.zip.CRC32C.update(). I'm using OpenJDK 17.

@Demogorgon314 I'd like to upvote this "feature"!


Add driver-provider version check

fixes https://github.com/deislabs/secrets-store-csi-driver/issues/130

cc @malnick @kalafut

I've opened a PR in azure provider to support --version flag. We need a similar change in the vault provider too.
cc @ritazh @kalafut

I think the azure.bat needs to be updated now


Fetching local pull requests failed: this.repository.getRefs is not a function

Extension version: v0.99.2024102213 (pre-release)
VSCode Version: 1.95.0-insider
OS: Web
Repository Clone Configuration (single repository/fork of an upstream repository): https://insiders.vscode.dev/github/microsoft/vscode
Github Product (Github.com/Github Enterprise version x.x.x): github.com

Steps to Reproduce:

Open the PR viewlet.
The error notification "Fetching local pull requests failed: this.repository.getRefs is not a function" shows up.

Wow, no one has tried to view local PR branches on vscode.dev ever.


Load more than 1500 annotation in graphic tool

My actions before raising this issue

[x] Read/searched the docs
[x] Searched past issues

I have a dataset of 1500 images with the matching .json file containing COCO annotations. Loading the images works fine and I can go through them. When loading the annotations nothing happens. No error message, but also no annotations loaded. When I use the same annotations format with a smaller dataset (200 images) it works fine.
Expected Behaviour

The annotations should load and be shown on the images
Current Behaviour

The images are loaded but no annotations are loaded. I can draw new annotations and save them, but the previously made ones are not loaded.
Possible Solution

Maybe if it just takes longer to load (I waited around 10-15min) a loading bar would be nice to show, that the tool is still working fine.
Steps to Reproduce (for bugs)

Get a large dataset of images
Have a COCO file > 30Mb
Load the images
Load the annotations

Context

I have a large dataset where I have to look through the annotations and correct them as there are probably some mistakes in them. Without the ability to load the annotations I cannot do that.

Operating System and version (e.g. Linux, Windows, MacOS): WSL2 with Ubuntu 20.04
Code example or link to GitHub repo or gist to reproduce problem:
Other diagnostic information / logs:
Logs from `cvat` container

Next steps
You may join our Gitter channel for community support.

Found out that the data is crowd and hence the issues


Queries involving bigquery do not evaluate all predicates correctly

Some queries don't seem to be evaluated properly if they involve tables in BigQuery.  Tested on presto v339 and v344.  I've encountered this problem when using a query generated from a template that needs to accommodate missing variables.
I have a table with the following row:
presto> select * from "mycatalog"."myschema"."mytable" limit 1 \G
-[ RECORD 1 ]------+---------
startp     | 1280
endp       | 1281
ref    | C
alt   | A

(startp and endp are bigint, ref and alt are varchar)
This query (Q1) returns false when it should return true:
 WITH  pv AS (
    SELECT  1  FROM  "mycatalog"."myschema"."mytable" bv  WHERE
    ((1280 IS NULL) OR bv.startp = 1280) AND
    ((NULL IS NULL) OR bv.startp >= NULL) AND
    ((NULL IS NULL) OR bv.startp < NULL) AND
    ((NULL IS NULL) OR bv.endp >= NULL) AND
    ((NULL IS NULL) OR bv.endp < NULL) AND
    bv.ref = 'C'  AND
    bv.alt = 'A')
SELECT  EXISTS (select 1 from pv) as "exists" ;

 exists
--------
 false
(1 row)

If I remove the third and fourth to last OR predicates, then it returns true:
 WITH  pv AS (
    SELECT  1  FROM  "mycatalog"."myschema"."mytable" bv  WHERE
    ((1280 IS NULL) OR bv.startp = 1280) AND
    ((NULL IS NULL) OR bv.startp >= NULL) AND
    ((NULL IS NULL) OR bv.startp < NULL) AND
    bv.ref = 'C'  AND
    bv.alt = 'A')
SELECT  EXISTS (select 1 from pv) as "exists" ;

 exists
--------
 true

Finally, if I copy this bigquery table to postgres, the above Q1 query works correctly and returns true.  It also works fine if I copy the database in-memory (via the in-memory connector).

Relevant slack conversation: https://prestosql.slack.com/archives/CGB0QHWSW/p1603220016037300

cc: @davidrabinowitz

If I remove the third and fourth to last OR predicates, then it returns true:
 WITH  pv AS (
    SELECT  1  FROM  "mycatalog"."myschema"."mytable" bv  WHERE
    ((1280 IS NULL) OR bv.startp = 1280) AND
    ((NULL IS NULL) OR bv.startp >= NULL) AND
    ((NULL IS NULL) OR bv.startp < NULL) AND
    bv.ref = 'C'  AND
    bv.alt = 'A')
SELECT  EXISTS (select 1 from pv) as "exists" ;

 exists
--------
 true

@jvlasblom This is really fishy. As the part of predicate you are removing constant-folds totrue. And should be eliminated during optimization, before query is run, and before any interaction with connector is made.
Is this for sure the behaviour you are observing?
Could you share EXPLAIN plans for both queries (with shorter and longer predicate)?

Attached is a log of what I do to reproduce the problem from the CLI, and the output of the two explain queries.   Four commands are run: the two queries with and without the predicate involving endp (in the log the column is called end_position), and the two corresponding explain queries.
Here are the steps I took:
create table in BQ
Using BigQuery web interface
CREATE TABLE test.variants (
    Start_Position INT64,
    End_Position INT64,
    Reference_Bases STRING,
    Alternate_Bases STRING
);

populate the data
with variants.csv
1280,1281,C,A

run
bq load test.variants variants.csv

verify data in Presto
presto:test> show create table variants;
             Create Table
---------------------------------------
 CREATE TABLE bigquery.test.variants (
    start_position bigint,
    end_position bigint,
    reference_bases varchar,
    alternate_bases varchar
 )

presto:test> select * from variants \G

-[ RECORD 1 ]---+-----
start_position  | 1280
end_position    | 1281
reference_bases | C
alternate_bases | A

run problematic query
[granit:~/workspace/tmp] [] ⌘ /Users/lukaszos/workspace/repos/prestosql/presto/presto-cli/target/presto-cli-*-executable.jar  --catalog bigquery --schema test --execute "WITH  passing_variants AS ( \
    SELECT  1  FROM  \"variants\" bv  WHERE \
    ( bv.start_position = 1280) AND \
    ((NULL IS NULL) OR bv.start_position >= NULL) AND \
    ((NULL IS NULL) OR bv.start_position < NULL) AND \
    ((NULL IS NULL) OR bv.end_position >= NULL) AND \
    ((NULL IS NULL) OR bv.end_position < NULL) AND \
    bv.reference_bases = 'C'  AND \
    bv.alternate_bases = 'A') \
SELECT  EXISTS (select 1 from passing_variants) as \"exists\" "

"true"

run explain query
[granit:~/workspace/tmp] [] ⌘ /Users/lukaszos/workspace/repos/prestosql/presto/presto-cli/target/presto-cli-*-executable.jar  --catalog bigquery --schema test --execute "EXPLAIN WITH  passing_variants AS ( \
    SELECT  1  FROM  \"variants\" bv  WHERE \
    ( bv.start_position = 1280) AND \
    ((NULL IS NULL) OR bv.start_position >= NULL) AND \
    ((NULL IS NULL) OR bv.start_position < NULL) AND \
    ((NULL IS NULL) OR bv.end_position >= NULL) AND \
    ((NULL IS NULL) OR bv.end_position < NULL) AND \
    bv.reference_bases = 'C'  AND \
    bv.alternate_bases = 'A') \
SELECT  EXISTS (select 1 from passing_variants) as \"exists\" "
"Fragment 0 [SINGLE]
    Output layout: [exists]
    Output partitioning: SINGLE []
    Stage Execution Strategy: UNGROUPED_EXECUTION
    Output[exists]
    │   Layout: [exists:boolean]
    │   Estimates: {rows: ? (?), cpu: ?, memory: ?, network: ?}
    └─ Project[]
       │   Layout: [exists:boolean]
       │   Estimates: {rows: ? (?), cpu: ?, memory: ?, network: ?}
       │   exists := (""count"" > BIGINT '0')
       └─ Aggregate(FINAL)
          │   Layout: [count:bigint]
          │   Estimates: {rows: ? (?), cpu: ?, memory: ?, network: ?}
          │   count := count(""count_1"")
          └─ LocalExchange[SINGLE] ()
             │   Layout: [count_1:bigint]
             │   Estimates: {rows: ? (?), cpu: ?, memory: ?, network: ?}
             └─ RemoteSource[1]
                    Layout: [count_1:bigint]

Fragment 1 [SOURCE]
    Output layout: [count_1]
    Output partitioning: SINGLE []
    Stage Execution Strategy: UNGROUPED_EXECUTION
    Aggregate(PARTIAL)
    │   Layout: [count_1:bigint]
    │   count_1 := count(*)
    └─ ScanFilterProject[table = bigquery:BigQueryTableHandle{projectId=losipiuk-bq-test, schemaName=test, tableName=variants, type=TABLE, constraint=TupleDomain{...}, projectedColumns=Optional[[BigQueryColumnHandle{name=Start_Position, type=INTEGER, mode=NULLABLE, subColumns=[], description=null}, BigQueryColumnHandle{name=Reference_Bases, type=STRING, mode=NULLABLE, subColumns=[], description=null}, BigQueryColumnHandle{name=Alternate_Bases, type=STRING, mode=NULLABLE, subColumns=[], description=null}]], limit=OptionalLong.empty}, grouped = false, filterPredicate = (((""start_position"" = BIGINT '1280') AND (""reference_bases"" = CAST('C' AS varchar))) AND (""alternate_bases"" = CAST('A' AS varchar)))]
           Layout: []
           Estimates: {rows: ? (0B), cpu: ?, memory: 0B, network: 0B}/{rows: ? (0B), cpu: ?, memory: 0B, network: 0B}/{rows: ? (0B), cpu: ?, memory: 0B, network: 0B}
           start_position := BigQueryColumnHandle{name=Start_Position, type=INTEGER, mode=NULLABLE, subColumns=[], description=null}
           alternate_bases := BigQueryColumnHandle{name=Alternate_Bases, type=STRING, mode=NULLABLE, subColumns=[], description=null}
           reference_bases := BigQueryColumnHandle{name=Reference_Bases, type=STRING, mode=NULLABLE, subColumns=[], description=null}
"

@jvlasblom
In the explain output of my repro-attempt you may notice that there are only 3 columns in projectedColumns= list. There is end_position missing. Which is expected at it should have be optimised out during planning.
In your explain output there are all 4 columns listed which is very unexpected. I cannot think of anything specific, but it looks like Presto is executing different query that what it looks like is content of --execute param. The query where value end_position is actually meaningful.
I kinda run out of ideas.
Does my reproduce steps make sense to you? Or maybe your setup is actually different?

This is very odd.  So my actual table has five columns.  The final one, sequence_accession, is a varchar/string and is not used in this query and didn't seem relevant.  However, if I create a table WITHOUT this column, it returns true.  If I create a table WITH this column, it returns false.  The values in the column don't matter (same result if i set them to null).
More precisely, something similar to what you said above (but with more rows) WORKS for me:
$ cat fourColumns.csv
1280,1281,C,A
1280,1281,C,T
1280,1281,C,T
1280,1281,C,Y
1280,1281,C,Y

but something with an extra varchar column does NOT work, and returns false.
$ cat fiveColumns.csv
1280,1281,C,A,MT825091
1280,1281,C,T,MT434757
1280,1281,C,T,MT509959
1280,1281,C,Y,LR860897
1280,1281,C,Y,LR824219

The schema table for the five column version was created with:
CREATE TABLE myProject.covid_test_beacon.fivecolumns (
    Start_Position INT64,
    End_Position INT64,
    Reference_Bases STRING,
    Alternate_Bases STRING,
    Sequence_Accession STRING
);

Data was loaded using bq load from one of the above CSVs, just as you did.

Thanks. With 5 columns it reproduces in my env. I will dig into that.


Prompt to verify in case of no Pipfile at context

I've been using pipenv for a while now, and I've grown accustomed to deleting unnecessary mistakably created Pipfiles.
Whenever running pipenv shell / install / (even) check - in case of no Pipfile at context an empty one will be generated, and in almost every case for me this was a result of simply running the pipenv command in the wrong directory.
I suggest prompting the user in such cases to notify that no Pipfile was found, an empty one will be created, and allowing the user to reject at that stage.
In my opinion it will improve usability.

Easy fix, large gain, would love to see this.

@bendem I agree, but when attempting to approach this I've realised that since this is a behavioural type of change it has a wide effect on the entire CI testing schema (which fails due to basing on top of the current convention where a Pipfile is auto generated), so it might require more work then anticipated.


Run chcp 65001 when building

Hi,
On Windows, I got the following error when executing jekyll build:
Liquid Exception: incompatible character encodings: CP850 and UTF-8 in feed-articles.xml
It works if I execute chcp 65001 before, as mentionned in http://jekyll-windows.juthilo.com/5-running-jekyll/
With Octopress 2, the command was automatically executed on Windows when doing rake generate:
C:\Users\TiTi\Desktop\octo2>rake generate
## Set the codepage to 65001 for Windows machines
## Generating Site with Jekyll
    write source/stylesheets/screen.css
Configuration file: C:/Users/TiTi/Desktop/octopress/_config.yml
            Source: source
       Destination: public
      Generating...
                    done.
 Auto-regeneration: disabled. Use --watch to enable.

C:\Users\TiTi\Desktop\octo2>

Could it be possible to also execute this command with Octopress 3, when using jekyll build?
I suppose this is more a Jekyll issue, but it could be nice to have that included rather than using a .bat to build the site...

I have the same problem on Windows only it says its codepage IBM437.

I don't think it's something we'll do automatically, but that's eventually up to @imathis. I'd recommend setting up Vagrant with a Linux OS. Ruby in general just isn't super great on Windows, so using a Linux OS for your development is highly recommended. I'll recommend the latest version of Ubuntu to get you started. Vagrant and VirtualBox are free and are easy to set up. :)

Nope we're staying out of the rake task business. If you need to do something like that, I suggest you create your own Rakefile to automate it.


Dockerfile exits right after load

Running into issue where docker file exits immediatley after loading. Both the Dockerfile and the pre-built image on dockerhub

.

@mikefoundation due to the age of the reported issue, we are closing it out.  Please open another issues if this remains unresolved and include the Looker + Deployer versions you're using.  Thanks!


High CPU Utilization and All File Watchers Consumed

Environment data

Language Server version: 2023.3.40
OS and version: Ubuntu 22.10
Python version (& distribution if applicable, e.g. Anaconda): 3.11.2 CPython installed via asdf

Code Snippet
N/A
The issue occurs in all of our repos, (e.g., ScribeMD/template-repo) regardless of whether they contain any *.py files. Running poetry install --sync to install our Python dependencies or, alternatively, rm -rf .venv to remove them has no effect on the bug.
Repro Steps

Clone any ScribeMD repository.
Enable the Pylance extension in VS Code.
Open the repository in VS Code.
Additional development environment setup steps are documented here if needed.

Expected behavior
Utilization doesn't remain at 100% on any CPU for more than a minute. No VS Code warning appears after a few minutes.
Actual behavior
The CPU utilization remains high upon launching VS Code until returning to baseline after about 80 seconds, precisely when the following warning appears:  "Unable to watch for file changes in this large workspace folder. Please follow the instructions link to resolve this issue." More specifically, 1 CPU is pegged to 100% utilization throughout these 80 seconds, while the others remain at baseline. The CPU in use cycles. Repeatedly running the following Bash script inspired by this Stack Overflow answer proves that a single VS Code process abruptly (within approximately a second) consumed all available file watchers right around that 80 second mark, explaining why the CPU utilization returns to baseline.
find /proc/*/fd -lname anon_inode:inotify -printf '%hinfo/%f\n' 2>/dev/null |
  xargs grep --count '^inotify' |
  sort --numeric-sort --field-separator=: --key=2 --reverse |
  head
cat /proc/sys/fs/inotify/max_user_watches

Logs
[Info  - 10:15:10 PM] (98316) Pylance language server 2023.3.40 (pyright 04471971) starting
[Info  - 10:15:10 PM] (98316) Server root directory: /home/kurt/.vscode/extensions/ms-python.vscode-pylance-2023.3.40/dist
[Info  - 10:15:10 PM] (98316) Starting service instance "template-repo"
[Info  - 10:15:10 PM] (98316) Auto-indent enabled
[Info  - 10:15:10 PM] (98316) No configuration file found.
[Info  - 10:15:10 PM] (98316) pyproject.toml file found at /home/kurt/Documents/ScribeMD/template-repo.
[Info  - 10:15:10 PM] (98316) Setting pythonPath for service "template-repo": "/home/kurt/Documents/ScribeMD/template-repo/.venv/bin/python"
[Info  - 10:15:10 PM] (98316) Loading pyproject.toml file at /home/kurt/Documents/ScribeMD/template-repo/pyproject.toml
[Error - 10:15:10 PM] (98316) Pyproject file "/home/kurt/Documents/ScribeMD/template-repo/pyproject.toml" is missing "[tool.pyright]" section.
[Warn  - 10:15:10 PM] (98316) stubPath /home/kurt/Documents/ScribeMD/template-repo/typings is not a valid directory.
[Info  - 10:15:10 PM] (98316) Assuming Python version 3.11
[Info  - 10:15:10 PM] (98316) Assuming Python platform Linux
[Info  - 10:15:10 PM] (98316) Search paths for /home/kurt/Documents/ScribeMD/template-repo
[Info  - 10:15:10 PM] (98316)   /home/kurt/.vscode/extensions/ms-python.vscode-pylance-2023.3.40/dist/typeshed-fallback/stdlib
[Info  - 10:15:10 PM] (98316)   /home/kurt/Documents/ScribeMD/template-repo
[Info  - 10:15:10 PM] (98316)   /home/kurt/Documents/
[Info  - 10:15:10 PM] (98316)   /home/kurt/Documents/ScribeMD/template-repo/typings
[Info  - 10:15:10 PM] (98316)   /home/kurt/.vscode/extensions/ms-python.vscode-pylance-2023.3.40/dist/typeshed-fallback/stubs/...
[Info  - 10:15:10 PM] (98316)   /home/kurt/.vscode/extensions/ms-python.vscode-pylance-2023.3.40/dist/bundled/stubs
[Info  - 10:15:10 PM] (98316)   /home/kurt/Documents
[Info  - 10:15:10 PM] (98316)   /home/kurt/.asdf/installs/python/3.11.2/lib/python3.11
[Info  - 10:15:10 PM] (98316)   /home/kurt/.asdf/installs/python/3.11.2/lib/python3.11/lib-dynload
[Info  - 10:15:10 PM] (98316)   /home/kurt/Documents/ScribeMD/template-repo/.venv/lib/python3.11/site-packages
[Info  - 10:15:10 PM] (98316) Adding fs watcher for library directories:
 /home/kurt/Documents
/home/kurt/.asdf/installs/python/3.11.2/lib/python3.11
/home/kurt/.asdf/installs/python/3.11.2/lib/python3.11/lib-dynload
/home/kurt/Documents/ScribeMD/template-repo/.venv/lib/python3.11/site-packages
[Info  - 10:15:10 PM] (98316) Adding fs watcher for directories:
 /home/kurt/Documents/ScribeMD/template-repo
[Info  - 10:15:10 PM] (98316) Searching for source files
[Info  - 10:15:10 PM] (98316) No source files found.
[Info  - 10:15:10 PM] (98316) Background analysis(1) root directory: /home/kurt/.vscode/extensions/ms-python.vscode-pylance-2023.3.40/dist
[Info  - 10:15:10 PM] (98316) Background analysis(1) started
(98316) Background analysis message: setConfigOptions
(98316) Background analysis message: setImportResolver
(98316) Background analysis message: ensurePartialStubPackages
(98316) Background analysis message: setConfigOptions
(98316) Background analysis message: setTrackedFiles
(98316) Background analysis message: markAllFilesDirty
(98316) Background analysis message: analyze

Thank you for taking the time and adding the logs.
from the logs I see file watchers on /home/kurt/Documents/ because it is added to your search path.
did you add it to extrapaths? or is that in your PYTHONPATH environment variable?
I haven't tried repoing the problem yet though

Ah hah! Yeah, my PYTHONPATH was set to /home/kurt/Documents, and deleting that environment variable dropped the CPU utilization back down to a very reasonable level and duration where launching VS Code only spikes one CPU for a few seconds as it should. Very few file watchers are consumed now, and the warning no longer appears. I didn't touch my PYTHONPATH for a long time prior to this, so I'm guessing the enhancement that causes the Pylance extension to watch everything in PYTHONPATH was released within the past month or two?

I don't think we would be modifying it. but i'm glad its fixed for you!

Interesting, yeah from #3468, I infer the indexing feature was added at least 6 months ago. Could be the behavior of a transitive dependency changed, but I agree it doesn't seem worth getting to the bottom of this.


This does not play nicely with wildcard reads.  Need to fix ZAP to

put it in the metadata, or fix wildcard expansion to add it.

https://github.com/project-chip/connectedhomeip/blob/f0f5de0d79ae1c51d7fddbf70900ebac0acd899a/src/app/util/ember-compatibility-functions.cpp#L307-L312

This issue was generated by todo-issue based on a TODO comment in f0f5de0d79ae1c51d7fddbf70900ebac0acd899a.

Hi Boris, it looks like you added this TODO. Is this needed for 1.0? If not please remove the label. Thanks!

This is a spec-compliance issue, so yes, needed.  That said, we should be able to fix this without too much trouble.  But we should merge https://github.com/project-chip/connectedhomeip/pull/14299 first and then fix all three attributes together.


Tabs: account updateHistory only if deepLink:true.

The advent of deep links feature to F6 Tabs was a real boon to the component.
I couldn't refrain from manually updating my local copy of foundation.tabs.js just to face a new issue; regardless whether the deepLink option is set true/false, tab hashes are always being appended to the URL.
I honestly don't think that was the intended outcome from the work of @ahebrank . I think it makes sense to restrict accounting for updateHistory if and only if deepLink:true, otherwise never bother and continue to function like it used to.
P.S. I've also fixed docs depicting tabs with deepLink attribute. Wasn't aware that it should go to the master branch.

It wasn't originally my intent to have the hash always appear in the browser history, but I'm OK with it since I think it's generally good accessibility and/or UX to have the browser history reflect the state of visible content on the page.  But... that would probably mean that deepLink should be on by default as well, so you're right that the current setup is a bit asymmetrical.
If this goes in, I'd take out the else branch of the conditional since I'm not sure what it's doing, and just have:
if (this.options.deepLink) {
    var anchor = $target.find('a').attr('href');
    if (this.options.updateHistory) {
        history.pushState({}, '', anchor);
    } else {
        history.replaceState({}, '', anchor);
    }
}

Right, the else block intention was to use tabs without no hash at all, just like it used to function prior to your PR, whilst the block inside the condition when deepLink:true will append a hash at all times, with or without history. In a nutshell there are three different situations, and that aims to account all of them.

I believe prior behavior was to not touch history at all, so unless I'm totally misunderstanding I'd agree with @ahebrank about removing that 2nd else branch

@kball you're right, it didn't. The issue I brought up here is to avoid have #hash being appended to the URL at all times, regardless of the deep-link value and this is a side-effect that arose with @ahebrank 's deep-link work.
I don't wanna sound insisting or anything cuz I have the impression we're not on the same page when referencing the problem here, tho my problem is with the anchor value being passed to replaceState().
If so badly you want to ditch the else block, then could we please have line 244 of foundation.tabs.js updated to this history.replaceState({}, '', window .location.pathname);. With that we can even remove the if conditional I wrapped around the history block.

@xhezairi I want to make sure I'm understanding what your desired outcome is... in the case that deepLink is false, shouldn't we not touch the history api at all?

Now I'm wondering why I didn't use your simple line to make my point.
But basically yes, tho I also thought for situations when we want to use deepLink (add #anchors to URL) but not alter history, hence I saw adding the else block you're not fond off handy, to offer the same Tabs functionality as before.
via Newton Mail [https://cloudmagic.com/k/d/mailapp?ct=pi&cv=9.2.5&pv=10.1.1&source=email_footer_2]
On Mon, Nov 28, 2016 at 11:24 PM, Kevin Ball<EMAIL_ADDRESS>wrote:
@xhezairi [https://github.com/xhezairi] I want to make sure I'm understanding what your desired outcome is... in the case that deepLink is false, shouldn't we not touch the history api at all?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub [https://github.com/zurb/foundation-sites/pull/9407#issuecomment-263414057] , or mute the thread [https://github.com/notifications/unsubscribe-auth/AAcQ940TvEFS2136ySqXyJ4PPutAUtRdks5rC1S3gaJpZM4K8D2v] .

@xhezairi I think we may be talking about different else blocks... my suggestion would be to keep the inner one, just can the outer one... so the resulting code would look like:
if (this.options.deepLink) {
    var anchor = $target.find('a').attr('href');
    if (this.options.updateHistory) {
        history.pushState({}, '', anchor);
    } else {
        history.replaceState({}, '', anchor);
    }
}

rather than
if (this.options.deepLink) {
 var anchor = $target.find('a').attr('href');

  if (this.options.updateHistory) {
    history.pushState({}, '', anchor);
  } else {
    history.replaceState({}, '', anchor);
  }
} else {
  history.replaceState({}, "", anchor);
}

I'm sure we're talking about the additional else block following (if
this.options.deepLink), but please see my second commit I've followed to
this PR, that should clarify my point. Thanks.

Sorry I feel like I'm being dense... what is the goal of manipulating the path/history at all in the case this component is not using it? (to be clear, I think the current state of things is a bug - I'm not advocating we don't merge your PR, just trying to make sure we get to the best solution)
If we manipulate it in a case where we're not using it (or for that matter, even if deepLink is true if the anchor isn't a part of this set of tabs), won't that cause us to interfere with other potentially legitimate uses of the anchor?

@kball I know you're aiming for the best solutions. I'm also aware there are shortcomings to the different scenarios I could think of. Here's how I see it, in user stories:
1) User doesn't want #tabIDs being shown on the URL. This is the way F6 tabs used to work until recently when @ahebrank made deepLink. This however doesn't make any one tab active even if the user lands directly to an URL like this http://localhost:3000/tabs.html#panel2
2) User wants anchors but doesn't want them stored in browser history, thus clicking the browser back button will take him to straight to the previous page disregarding all the tab navigation he did.
3) User wants anchors appended to URL and have them alter browser history, therefore clicking the back button will go through the tab panes until it reaches previous page.

I'm sure there can be more than this, but these are the ones I've thought of.
In all cases tho I think that browser history should be altered only if deepLink's are enabled, otherwise it'll look a little mysterious if the back button kept the user on the same page whilst switching tabs somewhere in the document.

Ok, so took another look through this again with fresh eyes... here's my thoughts:
Case 2 and 3 are handled by your change, moving all of the behavior inside of the if for deepLink.  I'm 100% on board with this change.
Case 1, the big outstanding question for me is, if someone has an anchor in their URL, then clicks on a tab that does NOT have deeplink enabled, what should happen?
In the code as written, the anchor will be removed via replaceState.
If we remove the 2nd else, nothing would happen to anchor.
Previous versions of F6 have the second behavior, and I think this is "more correct".  My reasoning is - the developer may wish to use anchor tags in other ways, and if they don't tell us they want tabs to manipulate them, we should just leave them alone completely.
I'm inclined to make this change directly and merge the PR... @xhezairi anything I'm missing in this reasoning?

Merged!  I did end up removing the else based on my above comment, but @xhezairi do please keep advocating for your case if you believe this was wrong - we can get to a shared understanding.
Thanks much for trying things out ahead of the release and catching this bug.


[Fix #4633] Make metrics cops aware of define_method

Problem
AbcSize, CyclomaticComplexity and PerceivedComplexity cops ignore define_method.
See #4633
Solution
Update MethodComplexity mix-in.
This change makes these cops aware of define_method.

Before submitting the PR make sure the following are checked:

[x] Wrote good commit messages.
[x] Commit message starts with [Fix #issue-number] (if the related issue exists).
[x] Used the same coding conventions as the rest of the project.
[x] Feature branch is up-to-date with master (if not - rebase it).
[x] Squashed related commits together.
[x] Added tests.
[x] Added an entry to the Changelog if the new code introduces user-observable changes. See changelog entry format.
[x] All tests(rake spec) are passing.
[x] The new code doesn't generate RuboCop offenses that are checked by rake internal_investigation.
[x] The PR relates to only one subject with a clear title
and description in grammatically correct, complete sentences.
[x] Updated cop documentation with rake generate_cops_documentation (required only when you've added a new cop or changed the configuration/documentation of an existing cop).

👍


Fix Typo in "Who Can Access the Data?" Section

"Who Can Access the Data Section?" Section
Changed "Security Admininistrator" to "Security Administrator"

@cocallaw : Thanks for your contribution! The author(s) have been notified to review your proposed change.

Thanks. Typo fix confirmed and merged.


DF 2267 - Update Event macros to display content based on state.

Updated event macros to use Sheer 'when' function in order to display content based on state.
Changes

Updated src/events/_event-post-macros.html to use Sheer 'when' function.
Updated 'src/events/_event.html' to use Sheer 'when' function and display appropriate event_status content.
Updated 'src/events/_single.html' to use Sheer 'when' function and display appropriate event_status content.
Removed 'src/events/archive/_single.html' because all events will use the same _single.html file.

Testing
Future Event

Visit 'http://localhost:7000/events/fall-consumer-advisory-board-meeting-in-washington-dc/'

Archived Event

Visit 'http://localhost:7000/events/summer-consumer-advisory-board-meeting-in-omaha-ne/'

Screenshots

Notes

This PR doesn't address the video playing capability while an event is in progress. I will address that in a separate PR.
Currently using placeholder image for archived events. Will need to determine what properties to use to display the images.

:+1: LGTM

:+1: from me after you address @anselmbradford comments


Tweaks for doc site

Codecov Report

Merging #28 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master      #28   +/-   ##
=======================================
  Coverage   91.77%   91.77%
=======================================
  Files           4        4
  Lines         158      158
  Branches       56       56
=======================================
  Hits          145      145
  Misses         11       11
  Partials        2        2

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 025ae79...3221c9d. Read the comment docs.


Avoid creating invalid rsa pss params

salt length -3 means use maximized salt length on signing and verification
salt lenght -2 means adjust to the salt length from the signature on verification, which is not
exactly representable by the RSA_PSS_PARAMS.
Checklist

[ ] documentation is added or updated
[x] tests are added or updated

Merged to master as 491360e, and 1.1.1 as d8ceb24.
Thanks!


How to change the entry text cursor color in dark mode

Hi,
Excellent theme! Quick Question: I have been trying to change the color of the text cursor color but I'm not able to do it.
I already tried thing like foreground="white" or insertbackground="white" also I was checking the tcl files but I haven't be able to figure it out.
Any clue can help! Thanks!

Add -insertcolor $colors(-fg) \ in the Settings (below line 31) of the azure-dark.tcl file.

@andreasfloros  Gracias very much! It is working perfectly!


Yubico Authenticator window should minimize once copy to clipboard icon clicked

Improved UX if window minimises automatically, rather than user having to do it before pasting auth code into the required application - e.g. GitHub, via a browser.
Using: Yubico Authenticator 2.3.0, Mac OS X 10.10.5

This is already what happens if you double-click on the code itself rather than the copy button.

Aha, thanks for that tip!


Automator: update<EMAIL_ADDRESS>dependency in<EMAIL_ADDRESS>

Generated by Automator - 2021-05-20T01:21:08+00:00

@istio-testing: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

integ-security-multicluster-tests_istio_release-1.10
d7330f3fe00b7626a960e39e17f723cbda4d94ab
link
/test integ-security-multicluster-tests_istio_release-1.10

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/test integ-security-multicluster-tests_istio_release-1.10


Please provide a --no-color command line argument

That would be useful for e.g. Neomake to disable the color setting from a user's config (Ref: https://github.com/neomake/neomake/issues/442).
Alternatively, it would be nice to use colors automatically, e.g. by detecting if stdin is connected etc.

Hi, the --no-color option is now available.
Please update Vint and try it.

Awesome, thanks a lot!
From looking at the code I believe that it works.

Works, thanks!


🐛 leading spaces prevent wallet connect

Description: It is very easy to copy and paste a leading or trailing space into the stake address of a representative, however doing so will lead the representative to be prevented from signing in.
User Story: As a delegate or alternate, I want the voting app to remove any leading or trailing spaces from my stake address when it is stored, so that I am not prevented from signing in.

Can Verify that this is fixed in latest QA deployment from main.


sound: Add Direction enum

Summary of the PR
We use input/output direction values in many places, yet we lose Rust's exhaustive static checks when matching for a stream's direction. This commits adds a Direction enum and From/Into implementations for audio backend respective direction types.
Requirements
Before submitting your PR, please make sure you addressed the following
requirements:

[x] All commits in this PR are signed (with git commit -s), and the commit
message has max 60 characters for the summary and max 75 characters for each
description line.
[x] All added/changed functionality has a corresponding unit/integration
test.
[x] All added/changed public-facing functionality has entries in the "Upcoming
Release" section of CHANGELOG.md (if no such section exists, please create one).
[x] Any newly added unsafe code is properly documented.

Rebased on main


Title of some references appear in lowercase

Hi, I want to use the Thesis template of the National University of Singapore (NUS). However I saw that for most of the references, for example the one that is included in the template:
@phdthesis{PHD:Pavlo,
title       = {On Scalable Transaction Execution in Partitioned Main Memory Database Management Systems},
author      = {Andrew Pavlo},
school      = {Brown University},
year        = {2014},
The title in the final doccument appears like "On scalable transaction execution in partitioned main memorydatabase management system", so all letters in lowercase. How can I fix this problem? Thanks in advance.

This is actually not a problem, but a feature of BibTeX. This FAQ explains the details as well as the solution for this issue. Basically, you should enclose the words or letters whose capitalization BibTeX should not touch in braces.
You may refer to the reference entries of ARIES, STREAM and TPC-C as examples.
@article{tods92:Mohan,
  author = {Mohan, C. and Haderle, Don and Lindsay, Bruce and Pirahesh, Hamid and Schwarz, Peter},
  title = {{ARIES}: A Transaction Recovery Method Supporting Fine-Granularity Locking and Partial Rollbacks Using Write-Ahead Logging},
  ...
}

@techreport{TR04:Arasu,
  title = {{STREAM}: The Stanford Data Stream Management System},
  ...
}

@misc{tpc-c,
  title = {The {TPC-C} Benchmark},
  howpublished = {\url{http://www.tpc.org/tpcc/}},
}


Install osascript instructions not clear enough

running thyme dep on osx gives instruction
npm install --save osascript'
running which prints error
npm WARN enoent ENOENT: no such file or directory, open '/Users/<username>/Project/thyme/bin/package.json'
npm WARN bin No description
npm WARN bin No repository field.
npm WARN bin No README data
npm WARN bin No license field.```

maybe tell user (like me) who doesn't knows much about npm much to run npm init first and fill with this repo's data.

+1

npm install --global osascript

Should be in the docs, rather than --save

Actually.. this is wrong, too.
You shouldn't have to install osascript on a mac at all..

I mistakenly thought some people might due to https://github.com/sourcegraph/thyme/issues/11. Thanks for the revert!


How can i implement with MM/yyyy

Need to implement month and year picker. i have implement as follows but it will give me date view also.
$('#example').datepicker({
startView: 1,
inputFormat: 'MM/yyyy',
outputFormat: 'MM/yyyy',
titleFormat: 'MM/yyyy',
theme: 'bootstrap',
});

I have the same headhacke.


Consider adding an example to keepers on how to best pass parameters to performUpkeep

Per this: https://stackoverflow.com/a/70103227/11969592
It seems like this is a common use case, some examples like what is shown in the answer here would be nice.

Tutorial ongoing - under review @rgottleber @dwightjl


autoImport: false is not working

Environment

Operating System: Darwin
Node Version:     v20.1.0
Nuxt Version:     3.6.2
Nitro Version:    2.5.2
Package Manager<EMAIL_ADDRESS>Builder:          vite
User Config:      imports
Runtime Modules:  -
Build Modules:    -

Reproduction
https://stackblitz.com/edit/github-jp4xw2?file=components%2FHelloWorld.vue
Describe the bug
I tested disabling the auto import feature, but the component still being imported
Additional context
No response
Logs
No response

imports.autoImport is how to disable the importing of functions from your composables directory.
You can disable the components module by setting components.dirs to an empty array.

Thanks @danielroe!
It was my first time using nuxt (as a 7 yo React developer) and I was struggling with disabling it. I've created a PR explaining this on the docs


Unable to connect to ACME server

A simple Windows ACMEv2 client (WACS)
Software version <IP_ADDRESS>9 (release, pluggable, standalone, 64-bit)
Connecting to https://acme-v02.api.letsencrypt.org/...
Initial connection failed, retrying with TLS 1.2 forced
Unable to connect to ACME server
Scheduled task looks healthy
Please report issues at https://github.com/win-acme/win-acme
It was working fine but now I am getting this error.

There must have been some change on your end then, as new bugs don't suddenly pop up in 3 year old software 😄. I'll convert this issue into a discussion.


logs "Checking licenses..." when -Dlicense.skip=true

When running license:check with the -Dlicense.skip=true flag set, the logging outputs:  "Checking licenses..." even though licenses are not being checked.
I think the issue is in the class:  LicenseCheckMojo - on line 41
https://github.com/mycila/license-maven-plugin/blob/1a08ad3af8c1543e397b9d5c4bfac0c7b13f49c3/license-maven-plugin/src/main/java/com/mycila/maven/plugin/license/LicenseCheckMojo.java
Thanks!

I need to review also other mojo to avoid logging when skip is set to true


Add airport table and data for worldwide airports

This patch fixes #7 and includes the following:

Creates a new airport table
Fills the data using seeders, when running the db:seed command - This works by supplying a .sql-file which gets loaded into the table.
Adds an airport model with datatypes
Changes the references of airports to the respective foreign keys

@Konrad00 This is the PR for #7

I will go ahead and merge this, since this does not really break anything, that we have right now.


Disabling Installer Validation Stage

Disabling Installer Validation Stage.


The expiration/validity of the consent is not working properly

Steps

Create travelocity as a SP
Log into travelocity app by user1
Give consent to all the claims
Login to dashboard of user1
Change the validity until the same day.
eg: the user gets authorized with the app on 1st and validity also ends on 1st.
On 2nd login to the travelocity app by user1

Issue
The consent will not be asked again from the user.

fixed with https://github.com/wso2/carbon-identity-framework/pull/1419


[refactor] #1210: Split event stream

Description of the Change

Split EventSocketMessage into separate enums for transmitting / receiving
Issue

Benefits

Possible Drawbacks

Usage Examples or Tests [optional]

Alternate Designs [optional]

This shares a lot with #1724.

This shares a lot with #1724.

I know, the other one depends on this. It will be sorted out when this is merged and I'll remove draft label from the other


Next release Metafacture core 5.3.1

Prepare next release 5.3.1.
First, this is about testing the Metafacture core 5.3.1rc.
From #428:

This is the branch for testing metafacture-core release candidate 5.3.1.
It follows up #397.
Based on the experience we've made with the release of metafacture-core 5.3.0 (which is deprecated resp. not recommended for usage) it was discussed with @blackwinter the need to better test release candidates before actually releasing them (this is also reflected now in the Maintainer Guidelines).
@blackwinter found the dependency upgrade of log4j:1.2.12' to log4j-core:2.14. problematic, so this is reverted in 36ed969.
I have tested 5.3.0 against lobid-resources and found an incompatibility which I reverted in 7c1ea04.
Furthermore, I found out that if you are using a metamorph.xsd of your own and make use of FileMap you have to also update your locally metamorph.xsd (like the one at https://github.com/hbz/lobid-resources/blob/master/src/main/resources/schemata/metamorph.xsd):
-      <attribute name="separator" type="string" use="optional" default="\t">
+      <attribute name="separator" type="string" use="optional" default="	">

As this will occur only quite rarely (is there anybody out there using a local duplicate of metamorph.xsd ?) and this is the result of a bug fix (d528ac9) (in effect the default separator defined (falsely) in metamorph.xsd has had no effect at all) this "breaking" fix will be part of the release (or is anyone against this?) but the release would come with the here mentioned note.

For testing the release candidate artifacts as artifacts in maven or gradle.follow the instructions at https://github.com/metafacture/metafacture-core/wiki/Maintainer-Guidelines#making-a-release-candidate -> 9.  ("candidate is available to be consumed").
The standalone can be tested by downloading https://github.com/metafacture/metafacture-core/releases/tag/untagged-8ac55fdd758acd7035b6.

@hagbeck and @UBmakla: you are also encouraged to test this release candidate.

:+1: Tested successfully with Limetrans.

+1, tested with the metafacture-fix, metafacture-playground and oersi-etl builds, updated Wiki instructions for Leiningen builds (used in metafacture-playground).

Released, see https://repo1.maven.org/maven2/org/metafacture/metamorph/5.3.1/ and https://github.com/metafacture/metafacture-core/releases/tag/metafacture-core-5.3.1.
Closing.


[Feature request] Autocomplete Abstract, Protocol and method override

Hi,
Thank you for this amazing product !

Python support Abstract class (it is stable since 3.4) and Protocol class (New in version 3.8).
Abstract class
It would be a great feature if, when implementing a concrete class that inherit from an abstract one, all the methods decorated by @abstractmethod will be immediately copy with the same signature (and maybe a raise NotImplementedError in the body ?), with the same decorator and with the same docstring
Example:
from abc import ABC, abstractmethod

class BaseAbc(ABC):
    def my_concrete_method(self, a, b, c):
        """My docstring my_concrete_method"""
        pass

    @abstractmethod
    def my_abstract_method(self, a, b, c):
        """My docstring my_abstract_method"""
        pass

    @classmethod
    @abstractmethod
    def my_abstract_classmethod(cls, a):
        """My docstring my_abstract_classmethod"""
        pass

    @staticmethod
    @abstractmethod
    def my_abstract_staticmethod(b, c):
        """My docstring my_abstract_staticmethod"""
        pass

    @property
    @abstractmethod
    def my_abstract_property(self):
        """My docstring my_abstract_property"""
        pass

And than whenever we start a new class and inherit from BaseAbc, after the : and we press enter, we would have this auto-completion:
from my_abc_module import BaseAbc

class ConcreteClass(BaseAbc):
    def my_abstract_method(self, a, b, c):
        """My docstring my_abstract_method"""
        raise NotImplementedError

    @classmethod
    def my_abstract_classmethod(cls, a):
        """My docstring my_abstract_classmethod"""
        raise NotImplementedError

    @staticmethod
    def my_abstract_staticmethod(b, c):
        """My docstring my_abstract_staticmethod"""
        raise NotImplementedError

    @property
    def my_abstract_property(self):
        """My docstring my_abstract_property"""
        raise NotImplementedError

Protocol
I did not use it yet but it sounds useful if I can import a module with Protocols inside and than vscode would propose an "implement protocol ..." in the "right click menu" and/or in the command palette when the cursor is inside a class.
It would autocomplete the same way it does with abstract class described here.

Anyone is welcome to improve further this suggestion.

Thanks for the suggestion @GilShoshan94. Pylance already implements a part of what you're suggesting. If you create a subclass of an ABC, you can start to type the name of an abstract method, and Pylance will offer to fill in the full signature for that method. It doesn't currently copy any decorators (like @property or @classmethod), and it doesn't copy the docstring. It also doesn't currently work for protocols. Those are all good extensions to the current behavior.
Here are screen shots that show how it works currently:

@erictraut
I see Pylance offers the full signature, with the Type Hinting, excellent ! I forgot to add type hints in my example, but indeed I would love to have them too as well in the child class.
That's a good start, I still think the autocomplete I am talking about would be very useful since you anyway has to implement every abstract methods, and it seems error-prone when you have more than 2,3 abstract method to do since as more as you "fill" your class with stuff, you can forget about them and won't see it clearly in the suggestion list.
It is not critical of course since Python will throw an error if an abstract method is missing.
I just remember how comfortable that was when I was programming in C++ in Visual Studio.
It has an "Implement Interface" action :

(https://www.youtube.com/watch?v=jV4_VhOCMuU)

Pylance offers a couple of related features that you may find useful. These are off by default, but you can enable them is you want.
First, if you enable the type checker ("python.analysis.typeCheckingMode": "basic"), Pylance will flag any attempt to instantiate a class that has abstract methods that haven't been overridden.

Second, if you enable the diagnostic rule "reportIncompatibleMethodOverride", Pylance will report any case where a subclass overrides a method with a signature that differs from its parent class. This applies to abstract and non-abstract methods.

We are working on a project where we have several abstract classes for different purposes and they have a bunch of abstract methods as well. We then create classes that inherit from this abstract classes we need to go 1 by 1 implementing the different abstract methods which is tedious and it is rather common that we try instantiating one in the same file just to check the error raised to see which method we have forgotten.
Having this type of autocomplete would be great because you are forced to implement those methods and giving that the method signature is already autocompleted it would be great to be able to select from autocompletion pop up an option that autocompletes every method one needs to write.
Is this something you plan on adding? It would make it sooo much simpler.

As something related, I like the Alt+Enter shortcut in PyCharm which, when applied on a class which inherits from an abstract class, allows to select the methods to implement. It is really convenient because I don't have to look at the abstract class definition to remember the name of the methods.
It would be really nice to have something similar in VS Code 🙂

Moving this issue to discussion as an enhancement request for comments and upvotes.


Splited tests and added coverage report preperation

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@4bc90b6). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master      #11   +/-   ##
=========================================
  Coverage          ?   27.07%
=========================================
  Files             ?       33
  Lines             ?      628
  Branches          ?        0
=========================================
  Hits              ?      170
  Misses            ?      458
  Partials          ?        0

Flag
Coverage Δ

unit
27.07% <0.00%> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


[Feature]: change the layout of project's page

Is your feature request related to a problem? Please describe.
for reference
screen-capture.webm
Describe the solution you'd like
No response
Describe alternatives you've considered
No response
Additional context
No response

Hey, I would like to work on this. Can you please assign this to me?

assign this issue to me

Hey @mnnkhndlwl , I have kept the pull request can you please check it and say if there is any issue?

please asign this issue to me under OSEN and Hacktoberfest


import {ElDatePicker} from 'element3' error

when use element3 as
import {ElDatePicker} from 'element3'
runtime error
Uncaught SyntaxError: The requested module '/node_modules/.vite/element3.js?v=48ac6f6f' does not provide an export named 'ElDatePicker'

Closing this issue because the issue template was not respected. Please create a new one that includes a minimal reproduction so it is easier for others to debug the problem. Also please respect the Before submitting the issue, please make sure you do the following and tick the checkboxes so we know that you read the contribution guidelines and further documentation.


Stop using the search-summary pattern for our search summary

While this is admittedly a retrograde move, I think it's fine
right now.

the buyer app is using two different search summary templates,
none of which are from the toolkit
this app is effectively doing its own search summary and them
passing it into a 3-line template
the template has nothing in it

It seems like the real thing to do if we think this is a pattern
(I think it is) is to rewrite it so that it works for all cases
and doesn't just echo an unescaped message.
This achieves the short-term goal of being able to remove the
safe filters from the front-end toolkit, so that seems worth doing first.

Does the search summary pattern just wrap things in <p class="search-summary">? Or am I missing something?
:+1: anyways - looks fine (identical HTML I think?) with the change.

Yes, that's all it's doing, Kev.  Wraps the message with some paragraph tags and a classname.


CSS clean does too much cleaning

Make css_clean function less greedy. It removes empty classes that are required for the nav drawer on mobile. See issue #77.
I have removed it from the build process. @nathakits - please fix this using RegExp and not long lists for the ignore list and make a PR after testing.

resolved with https://github.com/pupil-labs/pupil-labs-website/commit/28c52238c8e41ee8ed52f92a0c6cf573fa57ae2f


Bump floe to v0.4.1 for task output fixes

Pull in floe v0.4.1 for fixes in task output and combined stdout/stderr on podman/docker
https://github.com/ManageIQ/floe/releases/tag/v0.4.1

Backported to quinteros in commit 85154b698a6b705b97fcae1e114fa29dd667c185.
commit 85154b698a6b705b97fcae1e114fa29dd667c185
Author: Jason Frey<EMAIL_ADDRESS>Date:   Fri Oct 6 14:01:24 2023 -0400

    Merge pull request #54 from agrare/bump_floe_0_4_1

    Bump floe to v0.4.1 for task output fixes

    (cherry picked from commit bc28c48dcecf7e29b8242cdf061d713d0e996384)


For custom dataset, we generate anchors from original images (shape 4000x3200x3) or from scaled images (shape 416x416x3) in kmeans.py?

For test, I am training three models from one custom dataset. First model employed the default anchors, and second model used the kmeans anchors from original images, and the final one trained with the anchors generated from scaled images(416x416x3)

For test, I am training three models from one custom dataset. First model employed the default anchors, and second model used the kmeans anchors from original images, and the final one trained with the anchors generated from scaled images(416x416x3)

so ?  the performance is ?


Laravel 7.0 compatibility

Love this repository but would it be possible to make it compatible with Laravel 7.*.
Thanks

Hi.
Please update to V2.0

HI,
Where do I find V2.0?

https://packagist.org/packages/parsidev/tcp-socket#v2.0


beam search pick - error

Hi:
Training and standard sampling working perfectly in tensorflow 12.1 on 64 bit ubuntu with gpu.
Problem occurs when choosing "beam search pick" for sampling. After 15 minutes of trying to generate default 200 words the following warning appears:
/home/aaron/Desktop/word-rnn-tensorflow/beam.py:26: RuntimeWarning: divide by zero encountered in log
cand_scores = np.array(live_scores)[:, None] - np.log(self.probs)
25 minutes still no terminal output or redirect to text file.
Any ideas how I can fix this?
Thanks

That part is not fully tested. Could you try with a very small dataset?
Sung
On Wed, Jan 18, 2017 at 8:48 PM, GenTxt<EMAIL_ADDRESS>wrote:

Hi:
Training and standard sampling working perfectly in tensorflow 12.1 on 64
bit ubuntu with gpu.
Problem occurs when choosing "beam search pick" for sampling. After 15
minutes of trying to generate default 200 words the following warning
appears:
/home/aaron/Desktop/word-rnn-tensorflow/beam.py:26: RuntimeWarning:
divide by zero encountered in log
cand_scores = np.array(live_scores)[:, None] - np.log(self.probs)
25 minutes still no terminal output or redirect to text file.
Any ideas how I can fix this?
Thanks
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/hunkim/word-rnn-tensorflow/issues/49, or mute the
thread
https://github.com/notifications/unsubscribe-auth/AA3DV8NsbfBUUfG4ksofLLOqi5L1sIBJks5rTgohgaJpZM4Lm2Fj
.

Original corpus was 6.3 megs. Trained with tinyshakespeare 1.1 megs.
Script generated random prime word (which did not happen before) but 20 minutes later there was still no terminal output for 200 words.
Previous warning message did not appear.

Original reference:
https://github.com/hunkim/word-rnn-tensorflow/pull/29

I've attempted a solution in #51.

I've pulled @victorkwan solution into master with latest update. Please check and open new issue if problems persist.


feat(@kampus/email): add new email templates

Describe the feature you would like to see added
We merged #465, which enabled us to write emails using react-email (see #456). Now we wanna add more emails to cover our use cases. Let's use this issue to figure out what we need to prepare.
Emails
Pasaport - auth

[x] Magic link - login with email link flow
[ ] Signup - confirmation
[ ] Signup - welcome (after confirmation)
[ ] Email changed
[ ] Password updated
[ ] Login from an ip for the first time

Pano

[ ] Reply to a comment/post
[ ] Reported content

Gonna update this list when we have more ideas.

Please comment if you wanna contribute to this so we can assign you individual email templates.

Closing this as we are now using clerk to do this


Add Support For RuinedFooocus metadata

Is your feature request related to a problem? Please describe.
Nope
Describe the solution you'd like
Implement the metadata
Describe alternatives you've considered
No alternatives
Additional context
Example metadata:
{"Prompt": " ffserenity,  Grit, Rust, Stains, concept art a spaceship inside an abandoned barn, covered in rust and cobwebs, digital artwork, illustrative, painterly, matte painting, highly detailed, ", "Negative": "photo, photorealistic, realism, ugly, ,  photo, photorealistic,realism, ", "steps": 30, "cfg": 5, "width": 1344, "height": 768, "seed":<PHONE_NUMBER>, "sampler_name": "dpmpp_2m_sde_gpu", "scheduler": "karras", "base_model_name": "juggernautXL_version6Rundiffusion.safetensors", "base_model_hash": "7d723a7b", "loras": "Loras:<Objects\\Serenity_SDXL.safetensors \ud83d\uddd2\ufe0f:1>,<Detailers\\Grit-SDXL.safetensors \ud83d\uddd2\ufe0f:0.7>,<None:0.5>,<None:0.5>,<None:0.5>", "start_step": 0, "denoise": null, "software": "RuinedFooocus"}
Link: RuinedFooocus

Cab you attach some sample images? I"ll need to find what PNG tags it falls under.

There we go dude!

nice pics! thanks, I'll get the metadata in

oh no. No no no no I shouldn't have updated. You did something yesterday ! xD
I can tell because the Show Info changed.
Now I can no longer see the Lora weights, so I can't compare the images parameters anymore using Diffusion Toolkit. And I was doing just that a few hours ago... :'(

Can you give a sample of how it was before (v1.5) and how it is now?
I don't think I made any other changes than changing how it is displayed and adding separate seed/cfg/size/sampler


fix final_fit

Issue #, if available:
https://github.com/awslabs/autogluon/issues/1279
Description of changes:
This issue was caused by final_fit not being properly passed to ImagePredictor. ImagePredictor will retrieve the best model so far if final_fit is true. Passed final_fit as True to ImagePredictor's train_fn.
Updating the args won't really update what's being used by the train_fn as the scheduler is actually doing a deepcopy of the kwvars and convert that as the args passed to train_fn. Therefore, an update to the kwvars is needed. I updated the args as well as an indication of what the code is doing.
This is only a quick fix. We should consider refactor base_task instead later on
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@Innixma Please review the pr.

Job PR-1282-1 is done.
Docs are uploaded to http://autogluon-staging.s3-website-us-west-2.amazonaws.com/PR-1282/1/index.html


Migrate to PlatformWidgets

First I would like to thank for putting the time to create this library. It really has helped me speed up the completion of one of my assignments
After playing around with the example and going through your code I saw that it would be relatively easy to switch form using native Cupertino widget to using Platform Widgets and make this a generic settings library for both Android and iOS
https://github.com/aqwert/flutter_platform_widgets
I am be willing to contribute and pull the changes myself but that will happen either mid april or after the middle of may.

Yes, that sounds like a good idea. I'll keep this in mind, when I have time again to work on this project.


Update hacktoberfest-guide.md

My redirection to my github page was changed from '/' to '.' !

done


C# tuple types: Example doesn't correspondence to text

In text

A Student object now has two accessible Deconstruct methods: the extension method declared for Student types, and the member of the Person type. Both are in scope, and that enables a Student to be deconstructed into either two variables or three. If you assign a student to three variables, the first name, last name, and GPA are all returned. If you assign a student to two variables, only the first name and the last name are returned.

In code below
var s1 = new Student("Cary", "Totten", 4.5);
var (fName, lName, gpa) = s1;`

I didn't see the code corresponding to

If you assign a student to two variables, only the first name and the last name are returned.

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

ID: 3bf15268-9d34-cf10-97b9-0b94022c3305
Version Independent ID: 17728168-48fa-0f5f-0c19-31ae4ce6fdb9
Content: Tuple types - C# Guide
Content Source: docs/csharp/tuples.md
Product: dotnet-csharp
Technology: csharp-fundamentals
GitHub Login: @BillWagner
Microsoft Alias: wiwagn

Thanks for pointing this out @DSivtsov
It looks like the code and the article have gotten out of sync.  I've added this to the backlog to get addressed.


[Bug]: v3 does not exit with exit code <> 0 if any check fails (like in v2)

What happened?
If a prowler check fails the prowler executable still returns a exit code of 0. Prowler v2 was returning a different exit code in that case. This is very important to use be able to use prowler in pipelines.
How to reproduce it
Steps to reproduce the behavior:
prowler aws --allowlist-file prowler-allowlist.yaml --region eu-central-1 --excluded-checks accessanalyzer_enabled_without_findings
exit code = 0
Expected behavior
exit code <> 0
From where are you running Prowler?
Please, complete the following information:

Resource: workstation and docker
OS: Ubuntu / Alpine
Prowler Version [./prowler --version]: 3.1.1
Python version [python --version]: Python 3.10.6
Pip version [pip --version]:  pip 22.0.2 from /usr/lib/python3/dist-packages/pip (python 3.10)
Installation method (Are you running it from pip package or cloning the github repo?): pip
Others:

Additional context
Add any other context about the problem here.

Hi @jansepke we are going to review this take in to account https://github.com/prowler-cloud/prowler/issues/1759
Thanks for using Prowler !!

We will add a flag to give exit 1 when a FAIL is found like in v2. Coming in the next weeks.


Flutter doctor not recognizing "IntelliJ IDEA 2.app" on mac OSX

Subject: Flutter doctor not recognizing IntelliJ on mac OSX
Question: why does flutter doctor not show anything about IntelliJ on mac OSX, despite my being able to launch and debug even the Android emulator?
(I'm pretty sure it worked at some point in the last few weeks)
Thanks!
~/recipes_app>flutter doctor
Doctor summary (to see all details, run flutter doctor -v):
[✓] Flutter (Channel master, v1.10.15-pre.368, on Mac OS X 10.14.6 18G87, locale en-US)
[✓] Android toolchain - develop for Android devices (Android SDK version 29.0.2)
[✓] Xcode - develop for iOS and macOS (Xcode 11.1)
[✓] Chrome - develop for the web
[✓] Android Studio (version 3.5)
[✓] VS Code (version 1.39.0)
[✓] Connected device (3 available)
• No issues found!
~/recipes_app>

Hi @hhkk
can you please provide your flutter doctor -v?
Did you try with a fresh installation following the documentation?
Thank you

Where is IntelliJ installed?
I see it on my machine:
$ flutter doctor
Doctor summary (to see all details, run flutter doctor -v):
[✓] Flutter (Channel master, v1.10.15-pre.382, on Mac OS X 10.14.6 18G87, locale en-US)
[✓] Android toolchain - develop for Android devices (Android SDK version 28.0.3)
[✓] Xcode - develop for iOS and macOS (Xcode 11.2)
[✓] Android Studio (version 3.5)
[✓] IntelliJ IDEA Ultimate Edition (version 2019.2.4)
[✓] VS Code (version 1.39.2)
[✓] Connected device (2 available)

• No issues found!

In /Applications/IntelliJ IDEA 2.app
i wonder if installing flutter again would help.
~/utd/a191020FlutterRecipe/recipes_app>flutter -v doctor
[✓] Flutter (Channel master, v1.10.15-pre.368, on Mac OS X 10.14.6 18G87, locale en-US)
• Flutter version 1.10.15-pre.368 at /Users/hkon/dev/flutter
• Framework revision 3cd8c3142c (23 hours ago), 2019-10-31 19:35:52 -0700
• Engine revision 8ea19b1c76
• Dart version 2.6.0 (build 2.6.0-dev.8.2 bbe2ac28c9)
[✓] Android toolchain - develop for Android devices (Android SDK version 29.0.2)
• Android SDK at /Users/hkon/Library/Android/sdk
• Android NDK location not configured (optional; useful for native profiling support)
• Platform android-29, build-tools 29.0.2
• Java binary at: /Applications/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
• Java version OpenJDK Runtime Environment (build 1.8.0_202-release-1483-b49-5587405)
• All Android licenses accepted.
[✓] Xcode - develop for iOS and macOS (Xcode 11.1)
• Xcode at /Applications/Xcode.app/Contents/Developer
• Xcode 11.1, Build version 11A1027
• CocoaPods version 1.7.5
[✓] Chrome - develop for the web
• Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome
[✓] Android Studio (version 3.5)
• Android Studio at /Applications/Android Studio.app/Contents
• Flutter plugin version 40.2.2
• Dart plugin version 191.8593
• Java version OpenJDK Runtime Environment (build 1.8.0_202-release-1483-b49-5587405)
[✓] VS Code (version 1.39.0)
• VS Code at /Applications/Visual Studio Code.app/Contents
• Flutter extension version 3.5.1
[✓] Connected device (2 available)
• Chrome     • chrome     • web-javascript • Google Chrome 78.0.3904.87
• Web Server • web-server • web-javascript • Flutter Tools
• No issues found!
~/utd/a191020FlutterRecipe/recipes_app>

@hhkk Did you mean to close this?  If so, how did you fix it?

I did not mean to close.  It is still an issue for me.   How to unclose?
Thanks
On Mon, Nov 4, 2019, 1:32 PM Jenn Magder<EMAIL_ADDRESS>wrote:

@hhkk https://github.com/hhkk Did you mean to close this? If so, how
did you fix it?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/flutter/flutter/issues/43964?email_source=notifications&email_token=AAF34OHX257G47OB3SLHNKLQSBTCPA5CNFSM4JHXDJEKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEDAH5GA#issuecomment-549486232,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAF34OG3NELEEPB43BUIHWTQSBTCPANCNFSM4JHXDJEA
.

@hhkk we discover installations of IntelliJ by checking for the following app names:
  817   static final Map<String, String> _dirNameToId = <String, String>{
  818   ⎸ 'IntelliJ IDEA.app': 'IntelliJIdea',
  819   ⎸ 'IntelliJ IDEA Ultimate.app': 'IntelliJIdea',
  820   ⎸ 'IntelliJ IDEA CE.app': 'IdeaIC',
  821   };

Thus it is not detecting IntelliJ IDEA 2.app. This should be fixed, but if you are still in this broken state, renaming the app to IntelliJ IDEA.app is a workaround.


[Enhancement] - Unclear "How to get Swags" guidelines on the Hacktoberfest page

Description
Hi, currently the guidelines on "How to get Swags" seem a bit unclear. Also, I would like to suggest some improvements.

The "If your PR got merged" text looks a bit too small and inconsistent with the design.
I would like to elaborate on the instructions a bit more and add more points if required. ( We can discuss what points can be added and which points can be elaborated)
Add a "Copy your profile URL from the address bar" point.
The "Comment your profile URL" point is very unclear since it does not tell the contributor where to comment.
The placement of the image makes the whole section look like it's kind left aligned. I would suggest adding proper spacing to make it more uniform for the design.

I would like to work on this section, particularly on the above-mentioned issues under Hacktoberfest. Please assign me this issue if approved and I will start my work at the earliest.
Thank you,
Abir Santra
Screenshots

Additional information
No response

@AbirSantra Go for it.

@AbirSantra Go for it.

Ok sure. I'll get started! Also, can you add the hacktoberfest labels to this issue?

okay👍

@AbirSantra What's the progress?

@AbirSantra What's the progress?

I'll get started today. I had an exam. I'll share the new instructions with you today before implementing and we can discuss it.

@AbirSantra Sure

@AbirSantra  can i also work on this?

@AbirSantra  can i also work on this?

Yeah definately but after I make my contribution. If you feel like you can improve it even further and the project admins approve of it, then feel free to raise an issue. 😃

@Debajyoti14 What do you think of this content?
Section Header: How to win SWAGS?
Section Details: If your PR got successfully merged, you can earn points and have a chance to win awesome Swags! Make sure to follow the steps below carefully so that we can track your profile.
1. Star the Repository that you contributed to.
2. SignIn to your Clueless account. If you don't have an account yet, SignUp at clueless.tech (this will be a link) for free!
3. Go to your profile page from the menu at the top-right corner.
4. Copy your profile URL from the browser address bar.
5. Go back to your PR and comment your profile URL tagging the person who merged your PR.
6. You're all set!

Also, do you think we can implement a "Copy Profile URL" button on the user's profile page to make step 4 even better?
Let me know what you think.

@AbirSantra Great work. Go with the changes you specified.
But for now no need to add a copy button for Profile. One can simply copy it from Addressbar.
But make the changes you mentioned in the content and also do some changes in the ui so that it looks good

@AbirSantra Great work. Go with the changes you specified.
But for now no need to add a copy button for Profile. One can simply copy it from Addressbar.
But make the changes you mentioned in the content and also do some changes in the ui so that it looks good

Okay sure! I'll work on the UI and keep you updated!

@Debajyoti14 What do you think of this?

@Debajyoti14  What do you think of this?
I used a card with the same theme as the main website and changed it to a black background to match the theme of the Hacktoberfest page. I think this looks better than the previous design.


ROX-25526: Fix OCP Internal Registry Scan Regressions

Description
The annotation used to associate a pull secret with a service account has changed in OCP 4.16 from:
kubernetes.io/service-account.name: default

to
openshift.io/internal-registry-auth-token.service-account: default

OCP 4.15
$ oc version
Server Version: 4.15.22
Kubernetes Version: v1.28.11+add48d0
...

$ oc get secret default-dockercfg-k4wgc -o yaml | yq '.metadata.annotations'
kubernetes.io/service-account.name: default
...

$ oc get secret deployer-dockercfg-pb5r9 -o yaml | yq '.metadata.annotations'
kubernetes.io/service-account.name: deployer
...

OCP 4.16
$ oc version
Server Version: 4.16.3
Kubernetes Version: v1.29.6+aba1e8d
...

$ oc get secret default-dockercfg-f99p5 -o yaml | yq '.metadata.annotations'
openshift.io/internal-registry-auth-token.service-account: default
...

$ oc get secret builder-dockercfg-hq4dl -o yaml | yq '.metadata.annotations'
openshift.io/internal-registry-auth-token.service-account: builder
...

Impact:
In environments w/ multiple clusters, this may break scanning of images from the OCP internal registry.  This also results in auto-generated image integrations being created in Central for the internal registries (previously suppressed).
As a workaround before this fix is in place, adding the OCP internal registry hosts to the delegated scanning config will allow for images to be scanned.  CLI/API scan requests delegated to a secured cluster are NOT impacted.
User-facing documentation

[x] CHANGELOG is updated OR update is not needed

Testing

[ ] inspected CI results

Automated testing

[x] modified existing tests

How I validated my change
Spun up OCP 4.16 and 4.15 secured clusters, and verified that image scans are working on both for images from the OCP internal registry, also verified that the auto-generated integrations are no longer created for internal registry.

/test ocp-4-12-nongroovy-e2e-tests


feat: Add support to alias to DockerHost to be used by the ComposeRunners

Supported added, now it can be referenced the owner of the compose-resource:
apiVersion: tool.6zacode-toolbox.github.io/v1
kind: DockerComposeRunner
metadata:
  name: dockercomposerunner-redis-fake
spec:
  resourceOwner: dockerhost-fake
  composeFile: docker-compose.yaml
  repoAddress: github.com/6zacode-toolbox/docker-compose-demo.git

This will use connection credentials for such host.


Update reactive docs.

In 9.0.0 release we deprecate both reactive providers. However, in our Readme.md we still encourage to use them. We need to update Readme.md to use classic MoyaProvider and both provider.rx.request & provider.reactive.request instead.
We also forgot about updating docs/RxSwift and docs/ReactiveSwift. I think we don't need to remove the reactive providers usage in the docs (besides Readme), but we should label it with DEPRECATED sign. And add both requests usage (with and without progress) at the top of the file maybe.
When we cover all of the above, we can remove both providers docs from  examples (docs/Examples/RxSwift, docs/examples/ReactiveSwift). Right now I feel they do not provide any value. We just need to make sure we got everything in the docs part and it should be alright.

I think these have been lost with merging somewhere? Pretty sure those were updated. 🤔

Yup those were, but unfortunately only examples 😢


Please update Highlight.js library from 8.0 to current 8.4 version

Highlight.js is currently stuck at version 8.0, which is a year old.
The current 8.4 version adds support for a lot more languages, among which are some quite popular ones such as LESS.
Also, it would be nice if the minification process were adapted so that the properties used for the language definitions would NOT be minified, as they are now.
Having them retained would enable users to use the Custom extension feature in StackEdit to register custom language definitions without having the predict and match the minified property names. E.g. right now I'm stuck registering a custom language extension for the EJS template language like so:
userCustom.onReady = function() {
  var hljs = require( "highlightjs" );
  hljs.registerLanguage( "ejs", function( hljs ) {
    return {
      cI              : true,
      sL              : 'xml',
      subLanguageMode : 'continuous',
      c               : [{
        cN : 'comment',
        b  : '<%#',
        e  : '%>',
      }, {
        b  : '<%={0,2}',
        e  : '%>',
        sL : 'javascript',
        eB : true,
        eE : true
      }]
    };
  });
};

Not very nice...
(Actually; it'd be pretty cool if we could get a dedicated extension point in the editor where we could append custom Highlight.js language definitions, instead of having to rely on userCustom.onReady like I'm doing in the above example.)

+1

:+1: +100000


How to set the data on an ImageData struct?

I know I can create image data by doing:
let image_data = context.create_image_data(width, height)?;

And applying it on the context with:
context.put_image_data(image_data, 0.0, 0.0)?;

But how can I actually change the values of the pixels of the image_data object? No functions for it are exposed and the .data field is private.

It seems we're missing the data getter. Could you send a pull request for that?

Is this good? #377


BBR is a moving target

Placeholder issue that we need to track BBR versions and consider loading tcp_bbr that is newer than the CoreOS base kernel.
Re the BBR behavior mentioned in ndt-server/#132
Neal Cardwell says:

BBRv1 does have the property that it will only exit "STARTUP" mode once it has had 3 non-application-limited bandwidth samples that fail to push up the estimated bw by 25%. This means that application pauses (within a single round trip time), which "invalidate" bw samples by marking them as application-limited, prevent BBRv1 from exiting STARTUP.

This is not fixed by the January 2019 BBR patches.  It should be fixed in BBRv2.

Speaking of BBRv2 - https://github.com/google/bbr/blob/v2alpha/net/ipv4/tcp_bbr2.c


Update rules

Is there a way to update rules?
I would like to deploy the rules from one instance/server to another. I know how to get the rules, but can I also update them?

Hi hermie64,
I did it once in same instance. (Probably will work different instances)
This coding will help you;
##Rule Update
#Get SourceCube rule
SourceRule = SourceCube.rules
TargetCube = tm1_target.cubes.get("_Sales")
#Write Source to Target Rule
TargetCube.rules = SourceRule
#Update the Target Rule
tm1_target.cubes.update(TargetCube)

Create an instance of Cube by using the get method on the source TM1 connection
Pass that Cube instance to the update method using the target TM1 connection.

from TM1py.Services import TM1Service

# Establish source TM1 connection
tm1_source = TM1Service()

# Establish Target TM1 connection
tm1_target = TM1Service()

# Get Cube metadata (this includes rules) from the source instance
SourceCubeWithRules = tm1_source.cubes.get('cube_name')

# Update the cube in target instance
tm1_target.cubes.update(SourceCubeWithRules)

That worked, thanks @AliUgurGures and @rkvinoth


GitHub thinks every method needs authorization

<< BTools`External`
GitHub["ListRepositories", "b3m2a1"]

It will ask you to login, you can login provide wrong credentials or cancel and it will return ListRepositories result anyway. It should not ask I guess.

@kubaPod this is actually a design decision that I feel iffy about. I must have asked for the $GitHubUsername somewhere in the call and that automatically asks the user for both username and password. Now, I partially think this is an oversight as there is no need to ask for that, but I also need to think about whether asking for the password when I get the username is smart or whether I should just ask for that later

What do you need the name for if the author name is provided as an argument? (Makes sense for one argument case but can be missleading as you can ask to listrepositories of someone else) Or am I missing the point?

It turned out I was requesting it for every query as an optimization and I just didn’t notice it because I use the request with my keychain so I never actually authenticate. I fixed that up and it ought to work with the most recent paclet on my server.

Works! Thanks.


add sphere annotator

progress on sphere annotator and todo:

[x] ability to generate sphere data from points with radius in features
[x] ability to configure radii interactively
[x] ability to configure position interactively

okay @kevinyamauchi this is in a working state and ready for review

the example runs but the GUI isn't finished, I'd like to add that in a followup PR
alt-click to add/recenter selected sphere
'n' to go into 'add new point' mode which deselects an existing sphere
'r' to modify the radius from mouse position

tests are passing, and examples are fixed up - going to merge and iterate from here

Nice! Thanks for this, @alisterburt . Sorry that the review fell through the cracks.


Any way to load original yolov5 series model ?

Hi,
Just wonder if we can load original custom pretrained yolov5(s~x) model using your work?
or any way? if so, I suggest updating an easy example.
Thanks your great work !

Hi, thanks for your attention.
Original yolov5 models can't be loaded directly now.
If you want to load, you must rename parameters in yolov5 model.
We will consider adding yolov5 head in the future.


Use Host header to populate serverName and serverPort, like Tomcat does.

I realized that the previous fix wasn't really a fix, and that my intuition of headers not including port information was wrong - of course the Host header existed.
Found the code in tomcat that populates serverName/port by host header, so mimicing it here was the correct solution.
https://github.com/apache/tomcat/blob/trunk/java/org/apache/coyote/http11/Http11Processor.java#L1670

@trustin Apologies for the formatting

Current coverage is 63.80% (diff: 40.00%)

Merging #214 into master will increase coverage by 0.12%

@@             master       #214   diff @@
==========================================
  Files           274        274
  Lines         10506      10519    +13
  Methods           0          0
  Messages          0          0
  Branches       1518       1521     +3
==========================================
+ Hits           6691       6712    +21
+ Misses         3083       3074     -9
- Partials        732        733     +1

Powered by Codecov. Last update c1d6a94...896bb37


Simplify output of '--i18n LANG'

@esdoppio Is there a reason you closed this PR without it being merged?

@danascheider I don't know. My branch conflicts with the master now; now I have to resolve the conflicts so it can be merged, but I don't want to. And no one seems to care about the changes.
Anyway, you can always reopen it if you want to :metal:

I dropped the ball on these. I very much appreciate your contribution @esdoppio, I like these changes, and we'll definitely get them merged ASAP. I'm really sorry we gave you the impression we didn't care.

Also, I don't know why the for review tag got removed 7 hours ago! I didn't do that. 😕

Anyway, you can always reopen it if you want to

Reopen button is grey for me: "The repository that submitted this request has been deleted" 😞

I can go ahead and create new PRs with the changes.

@mattwynne I would say that the for review tag was removed automatically when the PR was closed (how can I closed PR be "for review"?). Probably a GitHub rule (or are you having a Waffle board connected to this repo?).


Remove redundant clickCloseContainer function

As the functionality was removed in #670 this function is no longer used and can be removed

You can preview this change here:
Built with commit d13733a3558deff372a093f165e6282edbd78c4d
https://deploy-preview-785--govuk-design-system-preview.netlify.com

Nice thanks Jani :)