The GPU used for training will be as slow as the CPU as long as there are other programs running?

The program only takes up more than 1G of GPU memory, which is very fast when run alone. However, if there are other programs running on this GPU, it will also take up more than 1G of GPU memory, but the speed will become as slow as the CPU, why? If there are other programs that use the GPU's other memory while the program running, the speed will suddenly become very, very slow (about a few hundredths of a percent). What's going on?

I believe this might be a problem specific to your system/server. In any case, it would be helpful if you could give more info about your setup and problem.


kpm misses versioning commands

npm exposes a version subcommand that allows you bump version numbers of your package.json on the commandline.
https://docs.npmjs.com/cli/version
It would be nice if kpm was capable of the same for project.json files.

What are you trying to do exactly? Change the number from build to build or is it a one time thing?

I think he is talking about a bump versions command.

This issue is being closed because it has not been updated in 3 months.
We apologize if this causes any inconvenience. We ask that if you are still encountering this issue, please log a new issue with updated information and we will investigate.


Use compass-blueprint gem for Compass generator

The application.scss generated by padrino g project NAME -c compass
uses the Blueprint framework, but it has been removed from Compass.
It has moved to compass-blueprint gem.
We should use compass-blueprint gem, or don't use Blueprint framework
in the application.scss.
See:

https://github.com/padrino/padrino-framework/blob/master/padrino-gen/lib/padrino-gen/generators/components/stylesheets/compass/application.scss#L2
https://github.com/Compass/compass/blob/stable/compass-style.org/content/CHANGELOG.markdown#013alpha8-10212013
https://github.com/Compass/compass-blueprint

This change is reasonable. Thank you so much @myokoym !

Thanks for merging it! :-)


Use computed property for disabled.

Allows child component to override the disabled property.

Thanks @arekt  :+1:


Accept / Reject + Edit

Currently one can only Accept or Reject the proposed commit message - add an option to edit the message (using $EDITOR)

if you set oc hook set it will integrate into your pipeline using "$EDITOR" var. e.g. using VScode — (after hook is set) you run git commit and it suggests you OpenCommit mesasge. Or you can submit an empty commit message through Source Control tab and it will do the same


port: [#4432] Expired JWT token exception not being handled (#6572)

Fixes #4432
Description
This PR ports the changes in the PR#6572 to handle expired tokens and return a most accurate error message.
Specific Changes

Added catch block in jwtTokenExtractor for TokenExpiredErrors.
Added a unit test in jwtTokenExtractor.test.js to cover the new code.
Added a unit test in cloudAdapter.test.js to verify the exception is properly thrown.

Testing
These images show the exception being displayed to the user in Emulator and the new unit tests passing.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

3 of 3 (100.0%)  changed or added relevant lines in 1 file are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage increased (+0.1%) to 84.766%

Files with Coverage Reduction
New Missed Lines
%

libraries/botbuilder-lg/src/templateExtensions.ts
1
84.48%

Totals

Change from base Build<PHONE_NUMBER>:
0.1%

Covered Lines:
20034

Relevant Lines:
22407

💛  - Coveralls


Consider adding more data to ClickHouse so it's self-sufficient

Right now the data on ClickHouse requires the config file to reconstruct the original inference and so on. Maybe we should add enough data to ClickHouse to make entries self-sufficient.

Nik also wants this for ClickHouse

After some thought, I think the source of truth should remain the TOML file and it is the gateway's job on startup to make sure that the clickhouse tables are consistent with the TOML file.

Proposal:
modify ProviderInferenceResponse to contain:
#[derive(Clone, Debug, PartialEq, Serialize)]
pub struct ProviderInferenceResponse {
    pub id: Uuid,
    pub created: u64,
    pub content: Vec<ContentBlock>,
    pub system: Option<String>, // new
    pub messages: Vec<RequestMessage>, // new
    pub raw_request: String,
    pub raw_response: String,
    pub usage: Usage,
    pub latency: Latency,
}

modify ModelInferenceResponse to contain
#[derive(Clone, Debug, PartialEq, Serialize)]
pub struct ModelInferenceResponse<'a> {
    pub id: Uuid,
    pub created: u64,
    pub content: Vec<ContentBlock>,
    pub raw_request: String,
    pub raw_response: String,
    pub system: Option<String>, // new
    pub messages: Vec<RequestMessage>, // new
    pub usage: Usage,
    pub latency: Latency,
    pub model_provider_name: &'a str,
}

modify ModelInferenceRequestWithMetadata to contain
#[derive(Clone, Debug, PartialEq, Serialize)]
pub struct ModelInferenceResponseWithMetadata<'a> {
    pub id: Uuid,
    pub created: u64,
    pub content: Vec<ContentBlock>,
    pub raw_request: String,
    pub raw_response: String,
    pub system: Option<String>, // new
    pub messages: Vec<RequestMessage>, // new
    pub usage: Usage,
    pub latency: Latency,
    pub model_provider_name: &'a str,
    pub model_name: &'a str,
}

modify ModelInferenceDatabaseInsert to contain
#[derive(Clone, Debug, PartialEq, Serialize)]
pub struct ModelInferenceDatabaseInsert {
    pub id: Uuid,
    pub inference_id: Uuid,
    pub raw_request: String,
    pub raw_response: String,
    pub system: Option<String>, // new
    pub messages: Vec<RequestMessage>, // new
    pub content: Vec<ContentBlock>, // new
    pub input_tokens: u32,
    pub output_tokens: u32,
    pub response_time_ms: u32,
    pub model_name: String,
    pub model_provider_name: String,
    pub ttft_ms: Option<u32>,
}

to allow for this, we'll need to add String columns system, messages, and content. We should consider making system nullable if we need to distinguish Some("") from None (I am not sure that we do).
here's a possible ClickHouse command:
ALTER TABLE tensorzero.ModelInference
ADD COLUMN system String,
ADD COLUMN messages String,
ADD COLUMN content Nullable(String);

We'll also need to change the trait signature for infer_stream in ModelProvider to return a struct in place of the current string that includes raw_request, messages, and content for the bookkeeping (probably as owned data and will require a clone over messages, system). These could be as Rust types or strings really. I am inclined towards strings as the intermediate representation (with the clone / serialization happening in the struct's new() method).


GDCM warnings

When compiling ITK with Module_RTK, I have a bunch of warnings which are related to GDCM, see the list in the following dashboard results
https://my.cdash.org/viewBuildError.php?type=1&buildid=1652539
https://my.cdash.org/viewBuildError.php?type=1&buildid=1652545
https://my.cdash.org/viewBuildError.php?type=1&buildid=1652594
Should I (1) submit a patch to fix them in ITK, (2) disable warnings in ITK for GDCM with itk_module_warnings_disable (I'm not sure what to do with the "has no symbols" warnings then) or maybe @malaterre wants to fix them both in GDCM and ITK? Thanks!

@SimonRit If you write a patch I can either git cherry-pick / am in GDCM if needed. Thanks

Alright, I'll propose something. Can you explain empty files like gdcmItem.cxx which generate those "has no symbols" warnings? Are they necessary for some reason or can my patch remove them?

Well really the warning is annoying to me. Anyway I'd prefer you move the gdcm::Item::Clear() definition to the cxx file. This should silence the warning.


As a developer, I want to make sure block verification and application is reliable, so that the node doesn't get stuck

When connecting to the mainnet, the node generally functions well, but I have encountered a few issues:

Sometimes block verification gets stuck and never finishes. When this happens, the current block doesn't pass through the consensus filter and isn't applied. As a result, no other blocks are processed.

Removing the thread::spawn call that wraps the block verification call has prevented this issue from recurring. I suspect a deadlock. This is a temporary workaround.

With this workaround, a new issue arises, though it takes longer to reproduce. Sometimes block application gets stuck. A request is sent to the ledger manager thread, the block application begins, but no response is received from the main thread. As a result, no more blocks are processed. The last time this occurred, a panic during block application was caused by a staged ledger hash mismatch, but I don't think this was the case for the previous instances I observed (so there may be two different issues at play).

The plan right now is to obtain all the blocks since the mainnet hardfork from an archive node, then rebuild the genesis state and start applying all blocks so see if we can catch any failures. If we can make this fast enough it would be a good addition to the CI tests.

Just found a new issue that caused the node to get stuck.
)> curl -s http://localhost:3000/stats/sync\?limit=1 | jq ".[0].blocks[11]"
{
  "global_slot": 22844,
  "height": 372644,
  "hash": "3NLdBwz3wVJNXy9XKx6yuhuy97qeYwsFcRprDPjHeqZNQ3GNPCED",
  "pred_hash": "3NLVGcDtp16TXQD7oqRDwLLbyCpUgQDeA5hoeGoEnBSRRyNPyR5y",
  "status": "Applied",
  "fetch_start": null,
  "fetch_end": null,
  "apply_start": null,
  "apply_end":<PHONE_NUMBER>212553000
}

This block was never seen by the node (at least, it is not in the logs), but the next block after it is (and is also in fetched state):
> curl -s http://localhost:3000/stats/sync\?limit=1 | jq ".[0].blocks[9]"
{
  "global_slot": 22847,
  "height": 372646,
  "hash": "3NKEXKfPQdQ1r5pf2GUBskwxZzcqfkeK4hr24w1KAgUFex6NVuzV",
  "pred_hash": "3NLHsEPED4pFw7JGFVrUDB1BkCt7khjJ6TuPmAiVvSmKxQivUqLX",
  "status": "Fetched",
  "fetch_start": null,
  "fetch_end":<PHONE_NUMBER>190012400,
  "apply_start": null,
  "apply_end": null
}

It seems the node is able to trigger catchup but not fetch that block (missing details in the logs regarding reason), there are multiple instances of this error:
2024-07-22T14:21:35.969543Z  WARN openmina_core::log::action: time: "1721658095967972690", node_id: 2bCf2veGepQxkaeFuoY5qgNTf1mwPegNMxYgUxQRemJzqYHAGR3, kind: "TransitionFrontierSyncBlocksPeerQueryError"

Update: working on a tool to replay all blocks in the best chain from the post-hardfork genesis using the blocks data that is stored in google cloud storage. Archive node can be used to reconstruct the best chain but not the full blocks in a way that we can use them for application (proofs are not part of the data stored in the archive node and neither is the protocol state)

Update: still working on the block replayer tool. Unexpectedly being able to consume the JSON turned out to be one of the harder parts (serde and yojson have kinda different defaults for representing values), but except for completed work proofs (which require s-exp parsing) parsing of precomputed blocks works. Now working on the main logic.

Update: was able to successfully apply all 20k canonical blocks on mainnet (up until yesterday) obtaining the expected ledger root hash. Proof verification is disabled (same as node), and I am not getting the correct value for the pending coinbase stack hash and ledger aux hash, but the cause is probably an issue with the setup, debugging that now.

Update:

Implemented parsing for the JSON representation of proofs (needed afterall, otherwise our scan state hash will not match)
Implemented replayer logic in OCaml to be able to replay and compare failed block applications

Now finishing the dumping and loading of the failed block application context so that it can be replayed with the OCaml replayer.

Update: for the 20k blocks tested before the current version of the node gets a matching staged ledger hash every time. Our verifier is also able to handle all block poofs and all the completed works tested (which was not all of them because that would take a very long time). There was a known issue on devnet which Sebastien fixed, so currently there are no known issues related to block and transaction application.
Will mark this one as DONE and create a new task for anything related to block replayer improvements and its integration to CI.


updated the description to better match man page

[x] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)
[x] My pull request targets the master branch of freeCodeCamp.
[x] None of my changes are plagiarized from another source without proper attribution.
[x] My article does not contain shortened URLs or affiliate links.

If your pull request closes a GitHub issue, replace the XXXXX below with the issue number.

Thank you for your contribution to the page! 👍
We're happy to accept these changes, and look forward to future contributions. 🎉


More tests for player

Added tests for AllPlayersResource and additional tests to
PlayerResource.
These mostly check the methods do what we expect them to from a unit
test perspective, although we should drive some of these with functional
tests too.
Signed-off-by: Ozzy Osborne<EMAIL_ADDRESS>

This change is

Reviewed 3 of 3 files at r1.
Review status: all files reviewed at latest revision, all discussions resolved.

Comments from the review on Reviewable.io


Use deterministic mode when running PyTorch models on GPU

See https://pytorch.org/docs/stable/notes/randomness.html for details.

Addressed in #14.


Doesn't update cortical when change the position

https://github.com/user-attachments/assets/8173ee4b-57ab-4627-acb4-82a744e7614a

this should be rectified for single cortical area changes


⚠ Fox Sports has degraded performance

In 5dd7cae, Fox Sports (https://foxsports.com) experienced degraded performance:

HTTP code: 200
Response time: 1180 ms

Resolved: Fox Sports performance has improved in e399428.


Luigi ListParameter None breaks when running with dynamic dependencies.

Luigi breaks when I try to run a task with dynamic dependencies and ListParameter with default to None.
When debugging, we found out that the method load_task inside luigi.task_register uses task_cls.from_str_params and it breaks when this parameter is None because ListParameter will try to parse it:
Error:
TypeError: 'NoneType' object is not iterable
A simple fix would be to evaluate if the value is None before parsing inside ListParameter.parse. I will create a PR for that.
Simple example to reproduce the error:
import luigi
from luigi.util import inherits

class TaskA(luigi.Task):
    l = luigi.ListParameter(default=None)

    def run(self):
        with self.output().open('w') as f:
            f.write('OK')

    def output(self):
        return luigi.LocalTarget(path='taska.txt')

@inherits(TaskA)
class TaskB(luigi.Task):

    def run(self):
        tasks = [self.clone(TaskA)]
        yield tasks
        with self.output().open('w') as f:
            f.write('OK')

    def output(self):
        return luigi.LocalTarget(path='taskb.txt')

luigi.build([TaskB()], local_scheduler=True)

Quick question - is there a reason that you need to pass None as your default for the ListParameter rather than the empty list?
In the Luigi test suite a default list value is set: https://github.com/spotify/luigi/blob/91ff2f020a7f30c8337ac033cd1db5a422de0b2c/test/parameter_test.py#L977 ... so my guess is that default=[] would work (I don't think there would be any mutation issues), and this might already be the default value for ListParameters.

Hi @jamescooke, thanks for answering! Indeed it works, as Luigi uses tuples, which do not present mutability issues. However, when thinking about Lists as a parameter, I naturally relate to python's pattern. Additionally, it works like that when not using dynamic dependencies, so there are a lot of pipelines built that rely on that behavior style. Therefore it would be nice to have it working uniformly on both cases (whether allowing 'None' or not).

Thanks for the helpful reply @gabrielgonzaga :+1: - I forgot about Luigi using tuples!
Looking at the PR, it looks like you're going to need to add a test to hit this new line: https://github.com/spotify/luigi/pull/2970/files#diff-6d22afaf0832ec4dde7f9ad3477e8b4aR1059

Done it @jamescooke 😀


Doesn't refresh when switching in and out of read_only

refreshes on the next on_change, so not really a problem

I just found a strange bug with unpinned preview images when switching to Pan Mode:
https://github.com/user-attachments/assets/e99ebb94-ac88-45fe-8c39-50fd4013f367

Could you raise this as a new issue, thx
On Tue, 12 Nov 2024 at 8:41 PM, LukeG89 @.***> wrote:

I just found a strange bug with unpinned preview images when switching to
Pan Mode:
https://github.com/user-attachments/assets/e99ebb94-ac88-45fe-8c39-50fd4013f367
—
Reply to this email directly, view it on GitHub
https://github.com/chrisgoringe/cg-controller/issues/326#issuecomment-2470051642,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABBLMAZSMFNON6PN2JO3EP32AHENDAVCNFSM6AAAAABRSWCAQ2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDINZQGA2TCNRUGI
.
You are receiving this because you authored the thread.Message ID:
@.***>


QWOYN

Description

Checklist

Adding Chains

If adding a new chain, please ensure the following:

[ ] Chain is registered to the Cosmos Chain Registry.
Note:

Chain's registration must have staking defined, with at least one staking_token denom specified.
Chain's registration must have fees defined; at least one fee token has low, average, and high gas prices defined.

[ ] Add chain to bottom of zone_chains.json
Note:

RPC and REST must not have any CORS blocking of the Osmosis domain.
RCP node must have WSS enabled.

Adding Assets

If adding a new asset, please ensure the following:

[ ] Asset is registered to the Cosmos Chain Registry.

[ ] Add asset to bottom of zone_assets.json.
Note:

The IBC channel referenced in path must be registered to the Chain Registry.
osmosis_main defaults to false (or else cite the listing rule that justifies enlisting to the Main app)

On-chain liquidity
If adding a new asset, please provide the plan for on-chain liquidity of the asset: (choose one)

[ ] The submitting team will ensure a pool will be created soon. Please hold off until further communication, which will notify of the Pool ID. (Please set the PR's status to Draft until the pool has been created)

OR

[ ] A preview link is requested for the submitting team to be able to create a pool using the Osmosis Frontend.

OR

[ ] The token is, or will be, going through a StreamSwap stream; thus, the token should be listed without requiring on-chain liquidity. A Pool ID will be provided in this PR following the stream's completion once the team has had a chance to create a pool using the earned funds.

Validation Testing
If adding or updating a chain or asset, the changes must be validated from the updated build of the Osmosis Frontend. Testers will need a small amount of tokens to validate.

[ ] Validaters can buy a small amount of this asset from Pool ID: {PROVIDE POOL ID}.
[ ] Preview build has been validated.

Thanks @sunnya97

@JeremyParish69 I'm not sure I did this properly.  I was confused about the stableswap yet decided to do it regardless.  No there are 50 QWOYN and 174 axl.USDC in the pool.  When I go to swap, I'm seeing 1.39 USDC per QWOYN.  When I got to add liquidity it wants me to add 3.49 USDC per 1 QWOYN.  Maybe its just not properly working yet :man_shrugging:
Can you give me some insight into this?

@JeremyParish69 I'm not sure I did this properly. I was confused about the stableswap yet decided to do it regardless. Now there are 50 QWOYN and 174 axl.USDC in the pool. When I go to swap, I'm seeing 1.39 USDC per QWOYN. When I go to add liquidity it wants me to add 3.49 USDC per 1 QWOYN. Maybe its just not properly working yet 🤷♂
Can you give me some insight into this?

It sounds like Stableswap is working exactly as intended. It sacrifices liquidity balance for keeping a more balanced spot price. It is expected that adding liquidity asks for asymmetrical amounts of liquidity, because that's precisely what stableswap allows to change a lot. If you can balance it closer to where the scaling factor is set, the amounts of each token require when adding liquidity will be more balanced.

Also, looks like the low fee rate registered for QWOYN is below the chain minimum, so this is failing validation on Osmosis frontend:
transaction failed with code 13 (codespace: ). Log: insufficient fees; got: 1103uqwoyn required: 2757uqwoyn: insufficient fee

Damn yeah, I think I chose the wrong pool type.  As far as the fee rate, do you mean on the cosmos chain registry?  I will create a new weighted pool now that I know I did it wrong.  Sorry about the trouble.

@JeremyParish69 ok new pool is 1295

Damn yeah, I think I chose the wrong pool type. As far as the fee rate, do you mean on the cosmos chain registry? I will create a new weighted pool now that I know I did it wrong. Sorry about the trouble.

Yes, for the Gas prices, we actually require that the Chain Registry has low, average, and high gas rates defined.
https://github.com/cosmos/chain-registry/blob/cb187a9c5abe2980361605837bd44fae8ca2cfe0/qwoyn/chain.json#L20

Will make a PR in a bit for that. Thanks for the heads up
On Thu, Nov 16, 2023, 4:56 PM JeremyParish69 @.***>
wrote:

Damn yeah, I think I chose the wrong pool type. As far as the fee rate, do
you mean on the cosmos chain registry? I will create a new weighted pool
now that I know I did it wrong. Sorry about the trouble.
Yes, for the Gas prices, we actually require that the Chain Registry has
low, average, and high gas rates defined.
https://github.com/cosmos/chain-registry/blob/cb187a9c5abe2980361605837bd44fae8ca2cfe0/qwoyn/chain.json#L20
—
Reply to this email directly, view it on GitHub
https://github.com/osmosis-labs/assetlists/pull/776#issuecomment-1815514703,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACBIMSUXV2AG2JEC66FI34TYE2R4BAVCNFSM6AAAAAA7LQMGXSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQMJVGUYTINZQGM
.
You are receiving this because you commented.Message ID:
@.***>

@JeremyParish69 all set, thanks again for all your help.


Correction in the sample code

Example codes for the Refresh access tokens should be as followings:
identity = new CommunicationUser(identity.Id);
tokenResponse = await client.IssueTokenAsync(identity, scopes: new [] { CommunicationTokenScope.VoIP });

Also users need to add using Azure.Communication; to avoid compile errors.

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

ID: a29430c6-fd84-c423-db14-4f87853dc5d6
Version Independent ID: 99605c52-055e-8361-4540-2527fd0ad342
Content: Quickstart - Create and manage access tokens - An Azure Communication Services quickstart
Content Source: articles/communication-services/quickstarts/access-tokens.md
Service: azure-communication-services
GitHub Login: @tomaschladek
Microsoft Alias: tchladek

Hi @rahatur, thanks for your feedback! We will review and update this doc accordingly.

Thanks for the feedback @rahatur! We've updated the documentation to clarify the C# token refresh snippet. You should see these changes live within the next 24 hours. cc @Grace-MacJones-MSFT

Thanks for the update @mikben! We will now proceed to close this thread.


[AutoGist] Add an option to only listen to self

I was quite sure I had already addressed this before, but since it looks like I hadn't, I did it now.

I was quite sure I had already addressed this before, but since it looks like I hadn't, I did it now.


Cannot create plan with $api->plan->create() via api

Hi, i don't know if this feature is currently disabled but i was unable to create plan using  $api->plan->create() as i keep getting error

Okay, i was getting error as a result of subscription not enabled for the account. Thanks


How to use PWCoefficient in a domain integrator

Hi,
May I sincerely ask how to use PWCoefficient in a domain inergator:
PWCoefficient coeff;
mVarf->AddDomainIntegrator (new MassIntegrator(irho)).
Here, the coeff is a piecewise function on each tetrahedral,  depending on a tetrahedral diameter size h.  Is there
a way to realize this in MFEM?
Thanks a lot.

Hello, @YHuiDong,
The PWCoefficient uses the element attribute number to select an appropriate Coefficient object. The intended use is to model a handful of regions in a mesh which have different material properties. It may be possible to tag each tetrahedron with a separate attribute number and create Coefficient objects for each one but this would be impractical for most meshes.
I see two options. If you can use the approximate element size as computed from the Jacobian then you can write a custom Coefficient class with an Eval method that computes your coefficient value from the Jacobian which is available through the ElementTransformation object. If you need a more specific calculation for the diameter you may need to compute the diameters in a separate step and store the results in a GridFunction (L2 grid functions are handy for such element-wise storage) then write a custom Coefficient class, similar to a GridFunctionCoefficient, which computes your coefficient value from the pre-computed diameter value.
Best wishes,
Mark

Hi Mark @mlstowell
for my application, it would be sufficient to know approximately the diameter of an tetrahedron.  If I use your first method:
class MyCoefficient : public Coefficient
{
private:
public:
MyCoefficient(){ ; }
virtual double Eval(ElementTransformation &T, const IntegrationPoint &ip);
virtual ~MyCoefficient() { ; }
};
double MyCoefficient :: Eval(ElementTransformation &T, const IntegrationPoint &ip)
{
T.Jacobian()  ??
here,  how return approximately the size of an element  from   a Jacobian matrix, Thanks..

}

HI, @YHuiDong,
Have a look at Mesh::GetElementSize
which shows three different measures of the size.
Best wishes,
Mark

Hi @mlstowell ,
thanks a lot Mark.  it works now with in the  Mesh::GetElementSize
if (type == 0)
{
return pow(fabs(J.Weight()), 1./Dim);
}
Best wishes.


Add actuator_msgs to bridge.

🎉 New feature
Adds actuators message to the ROS side of the bridge allowing for control of most (all?) types of actuators in simulation.
https://github.com/gazebosim/gz-sim/pull/1952
https://github.com/gazebosim/gz-sim/pull/1953
https://github.com/gazebosim/gz-sim/pull/1954
As seen in the Gazebo community meeting:
https://vimeo.com/812911652#t=39m58s
Depends on release of: https://github.com/rudislabs/actuator_msgs
Checklist

[x] Signed all commits for DCO
[x] Added tests
[x] Added example and/or tutorial
[x] Updated documentation (as needed)
[x] Updated migration guide (as needed)
[x] Consider updating Python bindings (if the library has them)
[x] codecheck passed (See contributing)
[ ] All tests passed (See test coverage)
[x] While waiting for a review on your PR, please help review another open pull request to support the maintainers

Note to maintainers: Remember to use Squash-Merge and edit the commit message to match the pull request summary while retaining Signed-off-by messages.

Force pushed to rerun CI now that actuator_msgs is upstream in rosdistro for humble. Still waiting on https://github.com/ros/rosdistro/pull/36796 to be merged. (will most likely force push again once that happens)

Looks like I just need to give it a little more time for it to propagate through rolling rosdistro into the package release.

Appears to be up in rolling, so forced pushed to run in CI again.


Specification extensions link is broken in Markdown file

The Specification Extensions link is broken through the specification file:

This is the broken link
This is the correct link

It would nice for readability of the specification if this worked.

For me as well, all links in the Table of Contents in this document are broken:
https://github.com/OAI/OpenAPI-Specification/blob/main/versions/3.0.3.md

Hey! I noticed the same thing and performed a slight refactor to resolve the issue! Waiting on reviews currently. See https://github.com/OAI/OpenAPI-Specification/pull/3128

Noticed this issue as well & refactored the links https://github.com/OAI/OpenAPI-Specification/pull/3160
Noticed now that mine is duplicate work with @SethAngell's PR.

Consolidating all broken link issues into issue #3054 and PR #3408


Use external speedyj repo

Changes

Remove speedyj from the repo.
Use cmakes FetchContent for pulling Catch2, stx and speedyj.
CI build with zserio 2.3.0
Migrate tests from gtest to Catch2
Replace nullptr-casts with std::declval<>

Follow-up work

Steps to reproduce

Affected use-cases

Review Checklist

[x] The changes are understood.
[x] The changes are well documented.
[x] The changes have been tested.

 Some tests should be moved to speedyj I think.

Going to remove them, as they are already in speedyj.


Errors for wrong URLs

Hi @seanmarpo,
I started to test our app and had few strange errors in the resulting report. There is an excerpt of our plugin configuration:
"postInstallPage":{"key":"getting-started","url":"/atlassian-connect/eazybi-jira/getting_started?user_is_admin={condition.user_is_admin}","name":{"value":"Get started"}}

the PATH for the post install page is /atlassian-connect/eazybi-jira/getting_started but in requirement 5 we got an error for a slightly different PATH: /atlassian-connect/eazybi-jira/gettingstarted?useris_admin=true, the underscore is missing.
There is another similar issue:
"jiraDashboardItems":[{"key":"report-gadget","name":{"value":"eazyBI report"},"description":{"value":"Embed selected eazyBI report."},"url":"/atlassian-connect/eazybi-jira/jira_dashboard_items/report?jira_dashboard_id={dashboard.id}\u0026jira_dashboard_item_id={dashboardItem.id}\u0026view_type={dashboardItem.viewType}\u0026user_is_admin={condition.user_is_admin}","configurable":false,"thumbnailUrl":"/images/eazybi_gadget.png"},{"key":"dashboard-gadget","name":{"value":"eazyBI dashboard"},"description":{"value":"Embed selected eazyBI dashboard."},"url":"/atlassian-connect/eazybi-jira/jira_dashboard_items/dashboard?jira_dashboard_id={dashboard.id}\u0026jira_dashboard_item_id={dashboardItem.id}\u0026view_type={dashboardItem.viewType}\u0026user_is_admin={condition.user_is_admin}","configurable":false,"thumbnailUrl":"/images/eazybi_gadget.png"}]

so the PATH should be /atlassian-connect/eazybi-jira/jira_dashboard_items/dashboard, but we got error on this PATH /atlassian-connect/eazybi-jira/jiradashboarditems/report?jiradashboardid=test&userisadmin=true
One more thing, I see that some pages which are expecting GET requests are receiving POST requests instead and vice-versa - is this intended? This results in 404 and with different headers.
Thanks for your effort.

Hi @gacha, can you include your connect app's descriptor URL? Once I have that, I'll check it for you.

@seanmarpo sure, https://aod.eazybi.com/atlassian-connect/eazybi-jira/atlassian-connect.json

Thanks for sending that over.
You are seeing a bug in the markdown parser which converts surrounding underscores to italics. The link in the HTML report is still correct, but it is displayed wrong.
You can test this by right-clicking, and choosing "Copy Link Address". The resulting link is correct.

Ok, now this is clear.
One more thing, I see that some pages which are expecting POST requests like enabled/disabled/webhooks are receiving GET requests instead, and vice-versa - is this intended? This results in 404 as those are completely different routes.
Thanks.

The latest code should fix the HTML reporting issue you were experiencing.
As part of CSRT's analysis, we attempt to probe all the available endpoints that we find. This means, submitting a GET, POST, and a few additional requests with fake authentication tokens. It allows us to validate that your app is doing proper authentication checks.


create the festures folder and finish the bdd test for LIST

Fix all the environment settings for bdd
create the features folder and required files
add bdd tests for LIST operation

Codecov Report

Merging #76 (37cf36d) into main (1581e31) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main      #76   +/-   ##
=======================================
  Coverage   96.33%   96.33%
=======================================
  Files           5        5
  Lines         382      382
=======================================
  Hits          368      368
  Misses         14       14

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 1581e31...37cf36d. Read the comment docs.


CON-4790 Handle aws staging url in ConnectFactory

Coverage remained the same at 36.45% when pulling 95ee4783602151dcfd0b7fc3a471895f362e206a on CON-4790-2 into 1dd817fbdabd3c106f1137cbfca000946b779c22 on master.


style: add focus and active states

Fix RM-XYZ

🧰 Changes
Umm, not sure how to describe this other than the title :)
🧬 QA & Testing

Broken on production.
Working in this PR app.

This PR was released!
🚀 Changes included in v6.64.0


How do I make a strum line move another way?

Describe your problem here.

I don't want it going like
🡆
🡆
↑↑
...But rather like
⇉🡆 🡆
How would I do this? And I don't even care if it needs runHaxeCode() I just want it done
Are you modding a build from source or with Lua?
Lua
What is your build target?
Windows x64
Did you edit anything in this build? If so, mention or summarize your changes.
Nope, just created a mod

Use	noteTweenDirection(tag,note id,direction,time,ease)

Oh ok


Allow assignment of tasks to executors based on current executor availability/load

Is your feature request related to a problem? Please describe.
I would like to use the multi-executor feature of Parsl to scale a computation across multiple Slurm partitions as well as local CPUs. Some partitions I have available to me have more resources and/or longer queue times than others. I would like to efficiently and quickly use all of the resources available to me for computation.
To solve this problem, I am currently creating multiple HighThroughputExecutor each with a different SlurmProvider or LocalProvider. I am then defining a set of bash_app with all of these executors available to it for execution. This doesn't work as intended however because currently when more than one executor is available, tasks are assigned at random and uniformly over the executors available to the DataFlowKernel (https://github.com/Parsl/parsl/blob/2024.02.26/parsl/dataflow/dflow.py#L948). Say I have 300 tasks and 3 executors: HighThroughputExecutor and LocalProvider with 2 CPUs, SlurmProvider with a short wait queue partition and 40 CPUs, and SlurmProvider with a long wait queue partition but 1000+ CPUs. The current logic will assign ~100 tasks to each of these in 3 equal task queues. Since tasks are assigned to the long wait partition, the execution of the entire workflow is delayed until workers become available on the long wait partition to execute its task queue when those tasks could have been executed earlier using local or the short wait partition resources. Ideally executors would be associated with tasks queues of a fixed size, with a size set in proportion to their available resources. Tasks would then be assigned to queues that are not full and task assignment would wait until at least one queue is not full. If all resources were available at once, then instead I'd expect 2 tasks would be assigned to the LocalProvider, 40 assigned to the short wait SlurmProvider, and 258 assigned to the long wait SlurmProvider. If the workers from the long wait and/or short wait partitions never connect, then I'd expect the local executor to eventually execute all tasks in a queue of size 2.
Describe the solution you'd like
I would like tasks to be assigned to executors that have connected workers, with more tasks assigned as more workers become available for execution and no tasks assigned if no workers ever become available during workflow execution.
Describe alternatives you've considered
Alternatively, the SlurmProvider could be altered to allow one to specify multiple partitions/accounts--in essence a MultiSlurmProvider. Or multiple providers could be provided to an executor.
I've considered writing my own logic to delay creating bash_app futures until a worker is available to execute to, deciding at some point in the future what executor will be associated with each app I create. However, I'm not sure yet how to implement this (or if it would work) and it seems to me that this functionality should exist in Parsl internals anyway. Any guidance on implementing this logic would be helpful if this is the appropriate route.

To write your own provider:
subclass parsl.providers.base.ExecutionProvider.
implement submit and status methods - I think in your case, your class would have its own internal instances of one local provider and two slurm providers, and your submit/status methods would delegate to the submit/status methods of those three providers.

@benclifford thanks for your excellent suggestion and guidance. I ended up implementing my own MultiProvider that subclasses ExecutionProvider. For future readers, my working solution is this:
class MultiHighThroughputExecutor(HighThroughputExecutor):
    def __init__(self, *args, **kwargs):
        HighThroughputExecutor.__init__(self,
            "multi",
            provider=MultiProvider(),
        )
        script_dir = os.path.join(self.run_dir, "scripts", "None")
        os.makedirs(script_dir, exist_ok=True)
        self.provider.script_dir = script_dir
        signal.signal(signal.SIGTERM, lambda *args, **kwargs: self.shutdown())

    def shutdown(self):
        logger.info("Cancelling all provider resources")
        old_jobids = []
        new_jobids = []
        # it looks like blocks can get allocated during/after shutdown...try to mitigate this?
        while True:
            new_jobids = [j for j in self.provider.resources.keys() if j not in old_jobids]
            if len(new_jobids) > 0:
                logger.info("new jobs since last cancel %s", new_jobids)
                self.provider.cancel(new_jobids)
            else:
                logger.info("no new jobs since last cancel, resuming executor shutdown")
                break
            old_jobids = list(new_jobids)
            sleep(1)

        super().shutdown()

class MultiProvider(ExecutionProvider):
    def __init__(self, *args, **kwargs):
        self._label = "multi"
        self._status_polling_interval = 30
        self.min_blocks = 0
        self.init_blocks = 1
        self.nodes_per_block = 1
        self.parallelism = 0.25
        self.channel = LocalChannel()

        self.local_cores = int(os.environ.get("J", 1))
        os.environ['PARSL_CORES'] = str(self.local_cores)
        self.max_workers = dict(
            local=self.local_cores,
        )
        self.providers = dict(
            checkpoint=SlurmProvider(
                partition="ckpt",
                account="astro-ckpt",
                min_blocks=0,
                max_blocks=256,
                init_blocks=0,
                parallelism=self.parallelism,
                nodes_per_block=1,
                cores_per_node=8,
                mem_per_node=128,
                launcher=SrunLauncher(overrides="-K0 -k"),
                exclusive=False,
                walltime="1:00:00",
            ),
            astro=SlurmProvider(
                partition="compute-bigmem",
                account="astro",
                min_blocks=0,
                max_blocks=30,
                init_blocks=0,
                parallelism=self.parallelism,
                nodes_per_block=1,
                cores_per_node=1,
                mem_per_node=8,
                launcher=SrunLauncher(overrides="-K0 -k"),
                exclusive=False,
                walltime="24:00:00",
            ),
            escience_a40=SlurmProvider(
                partition="gpu-a40",
                account="escience",
                max_blocks=20,
                init_blocks=0,
                parallelism=self.parallelism,
                nodes_per_block=1,
                cores_per_node=1,
                mem_per_node=8,
                launcher=SrunLauncher(overrides="-K0 -k"),
                exclusive=False,
                walltime="24:00:00",
            ),
            escience_rtx6k=SlurmProvider(
                partition="gpu-rtx6k",
                account="escience",
                max_blocks=5,
                init_blocks=0,
                parallelism=self.parallelism,
                nodes_per_block=1,
                cores_per_node=1,
                mem_per_node=8,
                launcher=SrunLauncher(overrides="-K0 -k"),
                exclusive=False,
                walltime="24:00:00",
            ),
            local=LocalProvider(
                min_blocks=0,
                max_blocks=1,
                init_blocks=1,
                parallelism=self.parallelism,
            )
        )
        self.provider_priority = ["local", "astro", "escience_rtx6k", "escience_a40", "checkpoint"]
        self.max_blocks = sum([self.providers[provider].max_blocks for provider in self.providers])

    def _find_provider(self, job_id):
        for provider in self.providers:
            if job_id in self.providers[provider].resources:
                logger.info("found job %s in provider %s", job_id, provider)
                return provider

        logger.warning("did not find job %s!", job_id)
        return None

    def submit(self, command, tasks_per_node, job_name="parsl.multiprovider"):
        logger.info("[multi] got submit %s %s %s", command, tasks_per_node, job_name)
        # this is responsible for launching launch_cmd
        # we should launch in a prioritized order
        for provider in self.provider_priority:
            if provider in self.providers:
                if provider in self.max_workers:
                    _command = command + f" --max_workers {self.max_workers[provider]}"
                else:
                    _command = command

                if len(self.providers[provider].resources) < self.providers[provider].max_blocks: # update to check JobStatus.RUNNING
                    logger.info("[multi] submitting %s to %s", _command, provider)
                    job_id = self.providers[provider].submit(
                        _command, tasks_per_node, job_name
                    )

                    if job_id is None: # provider is at capacity
                        logger.info("[multi] provider %s is at capacity" % provider)
                        continue
                    else:
                        logger.info("[multi] provider %s accepted submit and returned %s" % (provider, job_id))
                        return job_id
                else:
                    continue

        return None

    def status(self, job_ids):
        statuses = []
        for job_id in job_ids:
            provider = self._find_provider(job_id)
            if provider is not None:
                statuses += self.providers[provider].status([job_id])
            else:
                raise ExecutionProviderException("job id %s not found in any provider" % job_id)

        return statuses

    def cancel(self, job_ids):
        statuses = []
        for job_id in job_ids:
            provider = self._find_provider(job_id)
            if provider is not None:
                logger.info("cancelling %s on provider %s", job_id, provider)
                statuses += self.providers[provider].cancel([job_id])
            else:
                raise ExecutionProviderException("job id %s not found in any provider" % job_id)

        return statuses

    @property
    def resources(self):
        _resources = {}
        for provider in self.providers:
            # assume non-conflicting job ids...
            _resources.update(self.providers[provider].resources)
        return _resources

    @property
    def label(self):
        return self._label

    @property
    def status_polling_interval(self):
        return self._status_polling_interval

    def get_script_dir(self):
        return self._script_dir

    def set_script_dir(self, value):
        self._script_dir = value
        for provider in self.providers:
            self.providers[provider].script_dir = self._script_dir
            self.providers[provider].channel.script_dir = self._script_dir

    script_dir = property(get_script_dir, set_script_dir)

This solution creates a ranking of the provider priorities so that blocks are requested from high priority providers first. This lets me request workers from the "immediate" pool of resources I have before requesting from the higher latency start up pool.
I ran into a few issues along the way though:

I ended up needing to subclass the HighThroughputExecutor to sync the script directories of the executor and the providers. Additionally, I needed to sync the script directories of each provider and its channel.
Additionally, there were issues when cancelling the provider resources on executor shutdown soon after executor start up. I had to write a loop that continuously cancels jobs from the provider before continuing with executor shutdown.
There was an inconsistency in how the LocalProvider and SlurmProvider interface with the HighThroughputExectutor. If using the LocalProvider with the HighThroughputExecutor, one will specify max_workers at the executor level like so to limit the number of cores used:

HighThroughputExecutor(
    "local",
    provider=LocalProvider(
        min_blocks=0,
        max_blocks=1,
    ),
    max_workers=cores
)

on the other hand, with the SlurmProvider, resources are limited through nodes_per_block, cores_per_node, and max_blocks specified at the provider level:
HighThroughputExecutor(
    "local",
    provider=SlurmProvider(
        min_blocks=0,
        max_blocks=1,
        nodes_per_block=1,
        cores_per_node=1,
    ),
)

To merge these two, I had to modify the launch script to specify --max-workers manually for the LocalProvider:
This is working well for now though.

ok, nice that it mostly worked as I imagined - thanks for the report on all the bits that went wrong

    # it looks like blocks can get allocated during/after shutdown...try to mitigate this?

That's a bug in Parsl, I expect: the scaling up/down thread and shutdown code probably don't interact properly in the shutdown sequence.


Feedback: Schema

From a perspective of a person that took part in sybil-seeking hackathon, being provided a schema + db like this would have been pretty nice. On the other hand - if you are looking at a single round, a flat .csv file with votes is probably a great low-effort starting point to jump into doing analysis.
Possible additions to existing schema:

For each transaction that carries a  vote it would be nice to know gas i.e. gas price + gas spent.
Likewise for each project that did  apply to round on-chain I would like to see hash of transaction used to do that (+ gas fee).
Reporting blockNumbers for vote is nice, would be even nicer if there was also approximate_timestamp for plotting timeseries.

Key friction for me usability wise, is that I want to get clean, processed data from authoritative source without having to re-run the pipelines myself.
Interesting external information about each voter/grant address: POAPs, ENS name history, Snapshot votes

Shared these with owocki on TG, but here's some additional schema feedback:

include payoutAddress for a projectInRound (this is the wallet address that receives grant funds) ... I noticed that grantAddress is included in the vote
include the githubUrl of the project/user who posted the projectInRound (this may be explicitly linked in the application or in other cases the Github username of the account leading the project)

note: these field may both be in the application metadata already, but even so would be preferable to have easily accessible

include 'chain' in the 'vote' table

agree with @DistributedDoge's comments about tx gas and timestamp

include passport yes/no and score for voters?? Not sure if this is possible / good idea

Seeing Carl's comment made me think that a table with metadata would be very useful.
Going through the JSON metadata is cumbersome.
I know some fields in the metadata are mandatory and common to all applications, a table for those would be nice.

Pushed an update that includes the Schema feedback provided by @DistributedDoge & @ccerv1 on #11 PR.
Expect this to be merged in the next 24 hours if no issues arise.


Convert Mobilenet v2 from tensorflow to pytorch

Thanks for the great library!
The correctness test passed for MobileNet v2 for tensorflow. I am wondering if you could provide an example of the conversion of MobileNet v2 from tensorflow to pytorch and vice versa?

Hi @andreydung, thanks for your effort! You could upload the model and then I put it on our server.

@rainLiuplus Sorry, I meant that from the documentation (attached), mobilenet_v2 conversion should work for tensorflow and pytorch. I am wondering if you could help provide an example, especially handling NCHW for pytorch (3, 244, 244) and NHWC for tensorflow (244, 244, 3)?

For Tensorflow, I recommend you to read this doc. Just let the input shape be 224,224,3
For PyTorch, I am afraid that now this tool does not support these models. If the version of your PyTorch model is 0.4.0, it may work.


About the version of torch, i.e. from torch.cuda import amp

firstly really appreciate for your excellent work!
I got an issue when trying to implement your code.
The required torch version is 1.6.0 cause one module "amp" will be used in ./fastreid/engine/train_loop.py.
However, on the server I have no other choice but only torch==1.4.0+cu100, which dose not contain "amp"..
Tried to replace torch.cuda.amp with apex.amp but still does not work (seems they are not equally)
Could you please tell me if it is possible to  find some substitutions so that I can run it with torch==1.4.0 or just omit torch.cuda.amp?
Thanks a lot

You can just remove codes related torch.amp
https://github.com/JDAI-CV/fast-reid/blob/df823da09a1b24befbafd3bd3a5b938b7ef6de3f/fastreid/engine/train_loop.py#L211
Then it will work.

many thx for your fast reply! I'll try it out


zabbixctl: init at unstable-2019-07-06

Motivation for this change
init at unstable-2019-07-06
Things done

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

[X] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[X] Tested compilation of all pkgs that depend on this change using nix-shell -p nix-review --run "nix-review wip"
[X] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[X] Fits CONTRIBUTING.md.

@mmahut It looks like you accidentally included some exiv related work in here.

@aanderse oops, sorry. Rebased.

Time to test out my new super powers and see if this build on darwin ...
@GrahamcOfBorg build zabbixctl


Link to the full installer is not mentioned

The link to the full installer https://www.microsoft.com/en-us/download/details.aspx?id=53127 is not mentioned here, which would remove the requirement for port 80.

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

ID: 5c51a10c-d90c-7690-0979-cd8ce37234c8
Version Independent ID: 2da65f4c-f128-40b8-1a2b-5ae72c4252d6
Content: Install an on-premises data gateway
Content Source: data-integration/gateway/service-gateway-install.md
Service: powerbi
Sub-service: powerbi-gateway
GitHub Login: @mgblythe
Microsoft Alias: mblythe

Removing because it is a duplicate of https://github.com/MicrosoftDocs/data-integration/issues/8.


Ignoring a rule on a variable declaration doesn't work

I tried to ignore a rule on a variable declaration like this:
struct Bar {}
struct Foo {
  // swift-format-ignore: DontRepeatTypeInStaticProperties
  var fooBar: Bar

However this didn't seem to work and I had to move the ignore to the struct declaration to work.

Synced to Apple’s issue tracker as rdar://131991753

To paraphrase our offline chat here: I suspect it's a bug in the way we visit the nodes. We visit the type/extension and then manually recurse into the member list to emit the violations instead of implementing visit methods directly for the members. That would bypass the swift-format-ignore logic. (The implementation of the rule preceded the implementation of ignore, and we never caught this.)

@allevato It is definitely the case that the issue is with how the nodes are visited. I was looking at converting the DontRepeatTypeInStaticProperties rule to use the visit function on VariableDeclSyntax nodes, which would require knowing the current type name in the visit method. I was going to use the existing visit methods for that purpose, but because of nested types it seems necessary to push/pop the current type name using visit/visitPost pairs. Unfortunately, although everything works fine in the linter.walk(...) flow in LintOrFormatRuleTestCase, the visitPost methods are not called in the pipeline.lint(...) flow. Is this known or expected behavior? Or is there a better way to get the name of the enclosing type from a syntax node?

Update: I changed the code generator to make sure visitPost gets called, but don't use that. I just climb the tree looking for a type name instead.

just bumped into this but the problem is the "struct declaration" is out of the package so we can't even suppress this warning?
@available(*, deprecated, message: "Deprecating conformance to `PostgresDataConvertible`, since it is deprecated.")
extension Data: PostgresDataConvertible {
    public static var postgresDataType: PostgresDataType {
        .bytea
    }

Sources/PostgresNIO/Data/PostgresData+Bytes.swift:26:23: warning: [DontRepeatTypeInStaticProperties] remove the suffix 'Data' from the name of the variable 'postgresDataType'


Parser: function namespace is not supported

e.g: https://github.com/theforeman/puppet-foreman/blob/master/manifests/params.pp#L171
foreman.dev: cannot parse ./modules/foreman/manifests/params.pp:64:24:
  $db_password = extlib::cache_data('foreman_cache_data', 'db_password', extlib::random_password(32))
                        ^
 unexpected ':'
expecting "!=", "!~", "<<", "<=", "==", "=~", ">=", ">>", "and", "in", "or", '*', '+', '-', '.', '/', '<', '>', '?', '[', or '}'
at ./modules/application/manifests/foreman.pp:14:10

[x] Parser: allow qualify function name
[ ] Parser: allow qualify function as parameter in function (see commented test)
[ ] Fix resolution of qualify name function


Remove duplicate max-width and make it as default

when put container in flex box. work well but not in xs size.
before : https://codepen.io/kimix/pen/VWNaXj
and remove duplicate max-width for each media query, just put width 100% as default
after: https://codepen.io/kimix/pen/NgmNYY

Your examples render the same to me in Safari and Chrome.

in small screen
https://ibb.co/cAAmxv

Merged #23100, so closing out.


can't type ~all in search text input of the web, but can only paste ~all

Problem Description
the search text input keeps deleting/emptying the user's input:

can't input ~all, it immediately becomes empty when starting from l
can't use ~q to filter out all requests like https://docs.mitmproxy.org/stable/mitmproxytutorial-interceptrequests/ said, only a tooltip with "has no response"
can't input !~a too, it becomes empty after typing the a
~s ~t ".*text/html.*" can't work as well.
typing ~a can be deleted sometimes too.

Can the input box stop delete the user's input when there's any error or even any interval between inputs?
Maybe the error filter is only glitched and what happened is the user just typo.
System Information
Mitmproxy: 9.0.1 binary
Python:    3.11.0
OpenSSL:   OpenSSL 3.0.7 1 Nov 2022
Platform:  Windows-10-10.0.17134-SP0

And typing ~s ~t "text" cause a crash:
mitmproxy has crashed.
TypeError: Cannot read properties of undefined (reading 'length')
    at MessageUtils.get_first_header (http://<IP_ADDRESS>:8081/static/app.js:62:5783)
    at RequestUtils.getContentType (http://<IP_ADDRESS>:8081/static/app.js:62:5482)
    at contentTypeFilter (http://<IP_ADDRESS>:8081/static/app.js:64:15917)
    at andFilter (http://<IP_ADDRESS>:8081/static/app.js:64:11223)
    at http://<IP_ADDRESS>:8081/static/app.js:65:50039
    at Array.map (<anonymous>)
    at t.render (http://<IP_ADDRESS>:8081/static/app.js:65:49968)
    at qi (http://<IP_ADDRESS>:8081/static/app.js:5:68270)
    at pi (http://<IP_ADDRESS>:8081/static/app.js:5:68058)
    at ck (http://<IP_ADDRESS>:8081/static/app.js:9:10018)

Component Stack:
    at AutoScrollWrapper (http://<IP_ADDRESS>:8081/static/app.js:65:46267)
    at ConnectFunction (http://<IP_ADDRESS>:8081/static/app.js:58:5505)
    at div
    at MainView (http://<IP_ADDRESS>:8081/static/app.js:65:52003)
    at div
    at ProxyAppMain (http://<IP_ADDRESS>:8081/static/app.js:69:3070)
    at ConnectFunction (http://<IP_ADDRESS>:8081/static/app.js:58:5505)
    at Provider (http://<IP_ADDRESS>:8081/static/app.js:58:2818)

can't input ~all, it immediately becomes empty when starting from l

I'm unable to reproduce this at all, can you please provide steps to reproduce?

it's just normal typing

Which browser is this? Do you see anything in the event log? How do you access mitmproxy, locally or over a network?

chrome version 114.
remember no.
localhost.

I run it by using mitmweb command on terminal, and it opens a local web page <IP_ADDRESS>:8081
It happens even I use on mac only.
Mac connects to a wifi, and sometimes I use mobile phone using proxy to mitm.
Mitmproxy: 9.0.1
Python:    3.11.2
OpenSSL:   OpenSSL 1.1.1t  7 Feb 2023
Platform:  macOS-13.3.1-arm64-arm-64bit
OS: MacOS Version 13.3.1
Browser: Chrome Version 115.0.5790.114 (Official Build) (arm64)
Console log:
DevTools failed to load source map: Could not load content for http://<IP_ADDRESS>:8081/static/app.js.map: HTTP error: status code 404, net::ERR_HTTP_RESPONSE_CODE_FAILURE
DevTools failed to load source map: Could not load content for http://<IP_ADDRESS>:8081/static/vendor.css.map: HTTP error: status code 404, net::ERR_HTTP_RESPONSE_CODE_FAILURE
DevTools failed to load source map: Could not load content for http://<IP_ADDRESS>:8081/static/app.css.map: HTTP error: status code 404, net::ERR_HTTP_RESPONSE_CODE_FAILURE

I just tried Firefox, it also has the issue.
The difference is it recovers the text box slower, so usually I can type the filter I want.

10.1.3 fixed this issue.
but it sometimes still happens when you first time type at the <text> element


Model with nested class

I have Model
package com.example.Model;

import com.bluelinelabs.logansquare.annotation.JsonField;
import com.bluelinelabs.logansquare.annotation.JsonObject;

@JsonObject
public class MDeposit extends MBase {
    @JsonField
    private ArrayList<Deposit> list;

    public ArrayList<Deposit> getList() {
        return list;
    }

    public void setList(ArrayList<Deposit> list) {
        this.list = list;
    }

    @JsonObject
    public class Deposit{
        @JsonField
        private int id;

        public int getId() {
            return id;
        }

        public void setId(int id) {
            this.id = id;
        }
    }
}

But I Deposit class cannot recognized and get this error in MDeposit$$JsonObjectMapper.java
Error:(7, 25) error: cannot find symbol class Model

Any solution with this problem?

@willyanto Your inner class should be static

@markrietveld Still same problem
Error on import com.example.Model in generated file MDeposit$$JsonObjectMapper.java

I can look into fixing this, but the issue is caused because your package name is invalid. According to Oracle's docs (http://docs.oracle.com/javase/tutorial/java/package/namingpkgs.html):

Package names are written in all lower case to avoid conflict with the names of classes or interfaces.

Is package naming the root cause of the issue? That's really weird.
I got a projects for support with established structure that I cannot easily update, and I added LoganSquare support for object and list of objects.
But now, I cannot get it to work with object i object (object as a field on another one).
I'm using v. 1.3.6 and getting compiler exceptions:
D:\Projects\blablabla\build\generated\source\apt\AppName\debug\com\clientname\mainSDK\Models\Contacts\ContactsGroup$$JsonObjectMapper.java
Error:(8, 28) error: cannot find symbol class Models
Error:(18, 50) error: package Models.Contacts does not exist

Class declaration with issue is as follows:

@JsonObject
public class ContactsGroup {

    @JsonField(name = "GroupPK")
    private long _id;
    @JsonField
    private long MainFK;
    @JsonField(name = "GroupName")
    private String _name;
    @JsonField
    private String GroupDescr;
    @JsonField(name = "Contacts")
    private ArrayList<PhoneBookContact> _contacts;

//...
}

PhoneBookContact class is annotated with @JsonObject and works fine as a single class and as an array.
Failed as nested in another class.
What are any steps? Can I handle this without lots of packages renaming?
Thanks

Okay, even this is the craziest thing, it got fixed by local renaming (still not committing it to source control).
On the other hand, the nested array list of PhoneBookContact did not parse automatically.
What is the correct approach for parsing of nested objects?
Can I access the JSON field in the middle of parsing process to map it properly?


[EKSv2] Disk size creates another disk instead of increasing root disk size

What kind of request is this (question/bug/enhancement/feature request):
bug
Steps to reproduce (least amount of steps as possible):

Create an EKS cluster in Rancher without specifying a launch template
Check the "Storage" tab for one of the instances that are created for the cluster

Result:
There are two storage devices (/dev/xvda and /dev/sdr).
Other details that may be helpful:
This is not the intended outcome. There should be one storage device with the size specified by the user in the Rancher UI. This one device will have name /dev/xvda, in this case.

Reproduced this issue on Rancher server rancher\rancher:v2.5-head version 03b8153
Steps:
Create an EKS cluster in Rancher without specifying a launch template.
Note the default volume size is 20 GiB.
Create cluster.
After cluster is becomes active, navigate to an instance in EKS console.
Navigate to storage tab:

There are two volumes with each with 20 GiB.

The bug fix is verified with Rancher server rancher\rancher:v2.5-head version b4be2b2
Steps:
Create an EKS cluster in Rancher without specifying a launch template.
Note the default volume size is 20 GiB.
Create cluster.
After cluster is becomes active, navigate to an instance in EKS console.
Navigate to storage tab.
Result:
There is only 1 volume with size 20 GiB.

EKS operator version on rancher\rancher:v2.5-head version b4be2b2: rancher/eks-operator:v1.0.6-rc4


APICalls: fix time include name for linux

In the esp8266 2.4.0 core the filename for time.h is lowercase, without that the compilation fail when using arduino on linux.
./packages/esp8266/hardware/esp8266/2.4.0/tools/sdk/libc/xtensa-lx106-elf/include/time.h

@nicolehe and @KacobJeith Please review and merge quickly because master will not compile without it.


 and

Hi!
Will this crawler pull out any CSS files included via  tags and also

It should include CSS, JS, and images files. Could you show me your example?

Hi!
https://dor.ky - plenty of CSS files,
node node_modules/assets-crawler/cli.js https://dor.ky > output.json
cat output.json | grep css

@lvarayut any ideas on this? :)

You could also put the debug flag to see errors while fetching, assets-crawler --debug true https://dor.ky > output.json. Most of the pages return 502 from the server. That's why it doesn't get any JS and CSS files.
assets-crawler normally gets those files, for example, assets-crawler https://facebook.github.io/relay, this will return many JS and CSS files.

I'll certainly do that, thank you for the help!
On Wed, May 3, 2017 at 3:00 PM, Varayut Lerdkanlayanawat <
<EMAIL_ADDRESS>wrote:

You could also put the debug flag to see errors while fetching, assets-crawler
--debug true https://dor.ky > output.json. Most of the pages return 502
from the server. That's why it doesn't get any JS and CSS files.
assets-crawler normally gets those files, for example, assets-crawler
https://facebook.github.io/relay, this will return many JS and CSS files.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/lvarayut/assets-crawler/issues/2#issuecomment-298919527,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAPjqWorzuag5nTAyQ5_hSZxaOHpdIB6ks5r2IiEgaJpZM4NOD_M
.

You're very welcome :)

Quite odd, not sure what is causing it to 502, if I mirror with wget it will happily crawl the entire site.
Secondly, there are css  and some


Update zh.go

Fixes Or Enhances
Make sure that you've checked the boxes below before you submit PR:

[x] Tests exist or have been written that cover this particular change.

@go-playground/validator-maintainers

Coverage increased (+0.06%) to 75.159% when pulling 3901d74fea7f6b701f2110606dd9b1d3f8773e2a on NgeKaworu:zhTagTrans into ce34f361cca51e25715399ab801c5e1c01d02d89 on go-playground:master.


Update support-matrix-move-region-azure-vm.md

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

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

Hi @Rajani-Janaki-Ram - This pull request was opened in the public repo. PMs should work in the private repo, per the Microsoft Docs contributor guide. Would you make this content update in the private repo instead? It likely will auto-merge. Thank you! We'll close this PR.

Hi @Rajani-Janaki-Ram - This pull request was opened in the public repo. PMs should work in the private repo, per the Microsoft Docs contributor guide. Would you make this content update in the private repo instead? It likely will auto-merge. Thank you! We'll close this PR.


Hello, I would like to know if there is a method to use "generate_until" to evaluate on the ceval or cmmlu dataset. I'm using a chat model, which adds a prompt template to make it answer questions. However, the model's answer choices (like A, B, C, D) may not necessarily be the first generated token.

Hi!
We don't currently have a generative counterpart for all loglikelihood / multiple choice tasks in a one-size-fits-all way.
However, for CEval or CMMLU, you could make a variant that is zero-shot CoT, following _mmlu_flan_cot_zeroshot_template_yaml in this PR: https://github.com/EleutherAI/lm-evaluation-harness/pull/1356/files#diff-77e4a56ffcde165e312239a2abcef4943035c61df2eb09249d8872ce91f29658


error: cannot find symbol DaggerMyAppComponent.builder()

I added in the "partial reflection" dependencies only and removed my previous dagger dependencies.
aka my dependencies looks like this
dependencies {
    kaptDebug 'com.jakewharton.dagger:dagger-reflect-compiler:0.2.0'
    debugImplementation 'com.jakewharton.dagger:dagger-reflect:0.2.0'
}

So you can see I completely remove the "if ide" check and everything else. I have a simple dagger implementation (simple single android app with a single android app module, only a single dagger component, 2 dagger modules, and basically just using constructor injection. Nothing crazy)
MyApp.java:207: error: cannot find symbol DaggerMyAppComponent.builder()
It looks like my builder() isn't available on the generated DaggerMyAppComponent. If I'm reading the docs correctly, I don't have to change anything with the partial reflection route. Is there something wrong I'm doing?

From what you explained I presume you're using the implicit builder from standard Dagger. i.e.
@Component(modules = [...])
interface MyAppComponent { ... }

where { ... } doesn't contain @Component.Builder interface Builder { ... }, nor a @Component.Factory.
From what I can see reading the sources and tests at https://github.com/JakeWharton/dagger-reflect/tree/master/reflect-compiler/src in that case you'll get a create method, but not a builder:
https://github.com/JakeWharton/dagger-reflect/blob/7820187d0ae56fc718eabed28aa8b1aed2a39fa7/reflect-compiler/src/main/java/dagger/reflect/compiler/DaggerReflectCompiler.java#L146-L151
https://github.com/JakeWharton/dagger-reflect/blob/7820187d0ae56fc718eabed28aa8b1aed2a39fa7/reflect-compiler/src/test/java/dagger/reflect/compiler/DaggerReflectCompilerTest.java#L66-L73
@JakeWharton I think this default builder generation is not implemented in reflect-compiler, see here:
https://dagger.dev/api/latest/dagger/Component.html#instantiation

If neither exists, Dagger will generate a builder type that has a method to set each of the modules() and component dependencies() named with the lower camel case version of the module or dependency type.
In either case, the Dagger-generated component type will have a static method, named either builder() or factory(), that returns a builder or factory instance.

@ColtonIdle if your @Modules don't have constructor parameters, then you can use .create() instead of .builder().build(), but if they do, you'll need to add a Builder as described below to trigger the .builder() method generation in reflect-compiler: https://dagger.dev/api/latest/dagger/Component.Builder.html#instantiation

@TWiStErRob thanks for the detailed response!

if your @Modules don't have constructor parameters

My modules do have a constructor parameter, so it seems like I need to add the builder myself?
This is what I currently have:
@Singleton
@Component(
        modules = {
                NetworkModule.class
        }
)

public interface MyAppComponent {
    void inject(MyActivity inject);
}

I updated this to
@Singleton
@Component(
        modules = {
                NetworkModule.class
        }
)

public interface MyAppComponent {
    @Component.Builder
    interface Builder {
        Builder networkModule(NetworkModule module);
        AppComponent build();
    }

    void inject(MyActivity inject);
}

and my app seems to compile just fine. I really don't understand two things though.

Why is this needed? I'm new to dagger, so should I have known that I should have been doing this for some reason?

I noticed in https://dagger.dev/api/latest/dagger/Component.Builder.html it includes       @BindsInstance Builder foo(Foo foo); As you can see above, I don't do that. Should I have a @BindInstance somewhere? How does this relate to @JakeWharton comment here https://github.com/JakeWharton/dagger-reflect/issues/4 "but in general you should refactor to @BindsInstance and stateless modules." Anyone know why I should be using BindInstance, and what's a stateless module?

Re 1) see the part I wrote for Jake above, Dagger and Dagger reflect behave differently, that's my best informed guess. Note: the spec/doc of Component is pretty dense, every word has meaning and implications.
Dagger offers many ways to achieve the same thing. Neither one is really wrong, but there are some best practices. I don't know where you would get them though, I don't think I've seen a comprehensive and authorative source yet. The Dagger doc on the website is pretty good though (except the infamous Thermosiphon), worth reading it once at least.
Re 2) @BindsInstance is more efficient and easier to see/maintain than stateful modules (instance fields). A stateless module is a module that you can make an object or an abstract class with only static methods, in the best case an interface with only @Binds methods. Meaning, you don't need an instance for it and in many cases you can't even have an instance of it.
If you go stateless, the line Builder networkModule(NetworkModule module); becomes unnecessary and you can very likely even use a Factory which has further benefits.

Apparently the new hotness is to use a factory which I didn't know because I'm new to dagger. Closing


1.5.0

We’re getting 500 errors from weather.gov in CI which I can’t reproduce locally, but I figure it’s fine to cache the weather forecast for the getting started tutorial anyway and remove the external dependency.


Check Cookies sent by Django integration

Summary
Session cookie values logged in Sentry issues, allowing for impersonation if injected into a cookie.
Cause
The Sentry Python SDK (specifically the Django Integration) sends the cookie values by default. In addition, Sentry SaaS / self-hosted is configured to send_default_pii.
Recommended Remediation
The Sentry Python SDK’s Django Integration should be patched to never send the value for SESSION_COOKIE_NAME or CSRF_COOKIE_NAME. These values should be scrubbed regardless of the send_default_pii setting.
Other web framework integrations should have similar scrubbing put in place.
The default self-hosted installation configuration should not default to True for send_default_pii. Users are unlikely to change this and it should be opt-in.

The Django integration does not send any cookies over the wire when defaultSendPII=False (this is the default)
So this is "by design" as it is now.
We will have to improve this design.


Theming does not work on specific pages

Genre lookup for radio
Radio queue

Fixed in Release 1.3.2


Not Outputting Final Image

Running Docker 19.03.5 on macOS 10.14.6.
INFO:    Starting build...
INFO:    Creating SIF file...
INFO:    Build complete: /tmp/brainkey_brainage-standalone_bk-brainage-preproc-2019-12-11-b1bd81913114.simg
(10/10) Moving the image to the output folder...
765,931,520 100%  247.94MB/s    0:00:02 (xfr#1, to-chk=0/1)
Final Size: 731MB`
A search through my full system doesn't turn up a .simg. What's the issue?

I ran into the same thing as @mkwbritton on Ubuntu when I bind the /output volume to a relative path rather than an absolute one. Changing my path to an absolute one fixed this for me. Just thought I'd chime in!

Thanks @LiamBindle ! I think I've run into this as well, but for different containers. E.g., let's say I have a folder "data" in my present working directory. I might start by doing:
docker run --volume data:/output <container>

But actually, for the above Docker might think we want a named volume "data" So I usually need to do something like:
docker run --volume $PWD/data:/output <container>

So it's obviously a path. You could also do:
docker run --volume ./data:/output <container>

The difference I would suspect is specifying a named volume vs. an actual path volume, an example is written out here https://nickjanetakis.com/blog/docker-tip-28-named-volumes-vs-path-based-volumes. For kicks and giggles, you might want to look at your Docker volumes after trying the command that doesn't seem to work, it could be that your final container is saved to a named volume after all.

Oh, interesting, I see what you're saying. Thanks for the info!


Update alias setup language to clarify usage

Why:
Current language makes it seem like you should add the lines provided into your config file rather than run the command once to append the generated alias to your profile for you.
Updated the language to make it clear you should run the command rather than append it.
Users who append by mistake will see things working but would add ~1sec to their powershell launch time, and would end up appending duplicate lines to their config file every time they launch their shell.
Closes:

What's being changed (if available, include any code snippets, screenshots, or gifs):

Check off the following:

[x] I have reviewed my changes in staging, available via the View deployment link in this PR's timeline (this link will be available after opening the PR).

For content changes, you will also see an automatically generated comment with links directly to pages you've modified. The comment won't appear if your PR only edits files in the data directory.

[x] For content changes, I have completed the self-review checklist.

@moriartyb Thanks so much for opening a PR! I'll get this triaged for review ✨


backport to 17: flush package cache after registering or unregistering with rhsm_register

Backport https://github.com/chef/chef/pull/12828 to chef-17

@jasonwbarnett can you rebase and try again please? We updated the certs and just merged the code to fix the windows certs issues.

Thank you for the backport! The tests will need a rebase to pass.

Sorry about that. My fault. NOW there is stuff to rebase onto that should fix the windows errors.

Sorry about that. My fault. NOW there is stuff to rebase onto that should fix the windows errors.

rebased.


Fix: Draco exporter issues with duplicate geometry

Any concerns regarding this PR, or can I merge it?

Well, there are no concerns from my side.

https://github.com/KhronosGroup/glTF-Blender-IO/issues/1306 is a good suggestion. It is much easier to review code if there are tests that confirm its behavior. I haven't been able to test this so far, but also have no concerns.


openssh{,-hpn}: 9.8p1 -> 9.9p1

Description of changes
https://www.openssh.com/releasenotes.html#9.9p1
Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[x] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
24.11 Release Notes (or backporting 23.11 and 24.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

Review per https://github.com/NixOS/nixpkgs/blob/ecbb8968002e9c1a516240e7333ffeed71cb91d6/pkgs/README.md#package-updates
Reviewed points

[x] package name fits guidelines
[x] package version fits guidelines
[x] package builds on x86-64
[x] executables tested on x86-64

Not relevant/investigated:

[ ] all depending packages build
[ ] patches have a comment describing either the upstream URL or a reason why the patch wasn't upstreamed
[ ] patches that are remotely available are fetched rather than vendored

Also checked:

[x] nixosTests.openssh succeeds
[x] works on some of my machines (used after merging with nixos-unstable)

Possible improvements
Should we add a hint in the release notes about the removal of DSA signature algorithm?
Comments
Looks good, I have no objections, can be merged (with or without addition to release notes).

It is definitely a problem. Please see CONTRIBUTING.md for how to rebase things correctly.

Since the builds were failing and given the number of reviewers I took over in #350699.


Spine function enhancement

SpineLoader: Add new ability to load spine animation with only  json & atlas two files.
SpineAnimation:  Fix components clone bug.


Local pull request success status testing

Docs Build status updates of commit e16816d:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

E2E_DocsBranch_Dynamic/index.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


feat(ci): Enable timestamp in logs

What does this PR do?
Print timestamps in Gitlab logs. As we moved our gitlab version recently we can enable this feature
Motivation
Having access to the timestamp is a really important feature for a CI provider, very useful for investigation
Describe how you validated your changes
This is a gitlab feature

Possible Drawbacks / Trade-offs
This PR activates timestamp on the whole config. It should be possible to enable it per job.
It seems not possible to change the timestamp format which is quite long
2024-12-18T10:30:09.950336Z

However the full precision can still be useful so I guess we can accept the display impact
Additional Notes

/merge


Doesn't recognize files on network drive on Windows

For example, when i tried to edit a file with pathname like \\network\workspace\a.txt, it somehow tries to p4 edit c:\network\workspace\a.txt then get an error complaining that the file is not under the workspace root which is \\network\workspace\.

Can you try to enable the realPath experimental setting ?
Add this in your user or workspace settings
"perforce.realpath": true,

Tried. Didn't help. Still got the same errors.

Can you leverage AltRoots in the client spec to map the windows network drive?  This is how the p4 client for windows or mac handles a unix based workspace.


[DOC] Dependency with poetry

With poetry 1.1.4, you need to do: poetry add "typer[all]" instead of poetry add typer[all] as suggested in the doc. Otherwise you have an error:
zsh: no matches found: typer[all].
This might be due to my bash being zsh, I don't know.

I get this error also (also using zsh). poetry add typer -E all also works.

Right, that's the zsh thing.
It is not related to typer or poetry directly.
Please, check this SO answer by Lukas Graf for an explanation and a workaround.

Right, that's the zsh thing.
It is not related to typer or poetry directly.
Please, check this SO answer by Lukas Graf for an explanation and a workaround.

Sure sure, but as more and more people are using zsh, it might be worth pointing this out or having a better error message.

it might be worth pointing this out or having a better error message.
Do you mean, it should be in the poetry documentation? I guess, pip should have this mentioned somewhere too.
Though this errors comes from the zsh directly, it does not even hit poetry / pip / typer.

it might be worth pointing this out or having a better error message.

Do you mean, it should be in the poetry documentation? I guess, pip should have this mentioned somewhere too.
Though this errors comes from the zsh directly, it does not even hit poetry / pip / typer.

Ah indeed you're right, I understand now what you meant by a zsh thing! Well then I would say that something in the doc would be a nice to have. I mean if with bash the "zsh way" would work, I would just change it to this then.
But feel free to close this as it is a bit out of scope I guess 😄

Thanks for the help here everyone! 👏  🙇
Thanks for reporting back and closing the issue 👍

Sorry for the long delay! 🙈  I wanted to personally address each issue/PR and they piled up through time, but now I'm checking each one in order.


Bumping gotests image version

New updated gotests image that contains controller-gen, helpgen and type-scaffold

/lgtm

/approved

/approve


Create demo.gif

Hello Insider Team,
I created a demo.gif where I demonstrate the Insider and I think this is useful for other users.

You can see how it looks here: https://github.com/GouveaHeitor/insider/tree/develop#demo

Thanks @GouveaHeitor


Remove SVG attributes from Html5 attributes

I checked the HTML 5.1 draft to see whether these were valid attributes. They appear to be only for SVG.

I added another commit to LumiGuide master that fixes the style attribute.
Previously the style attribute was of type:
style :: AttributeValue -> Attribute ev

This isn't correct in react since the value needs to be a Javascript object.
I fixed this by changing the type to:
style :: Json.Object -> Attribute ev

where a Json.Object is a HashMap Text Value.

@roelvandijk: Seems reasonable. We should probably make sure that Text.Blaze.Svg.Attributes contains all the attributes you removed from Text.Blaze.Html.Attributes.
@basvandijk: Yeah, this was a known issue (https://github.com/meiersi/blaze-react/issues/5) - thanks for fixing it. Is there a reason to use Aeson.Object rather than Map String String? I think the latter has some advantages:

It doesn't pull in aeson and attoparsec.
It prevents users from using arrays or objects as the values. For the use-case of the style attribute, this is clearly correct.
It prevents users from using numbers as the values. React allows you to use numeric values, but it does a [key-dependent stringification]
(http://facebook.github.io/react/tips/style-props-value-px.html) - yuck. I think it would be better to force users to do the string conversion explicitly.

What do you think? Eventually we'll probably want something like this:
myHtml = H.div ! A.style
    [ backgroundColor (RGB 27 32  70)
    , position Absolute
    , top 0
    , font ["helvetica", "ariel", "sans-serif"]
    ]

What do you think?

Yes, Map String String or Map Text Text definitely makes more sense. A proper CSS DSL (maybe even clay) would be even nicer.
I'm currently a bit pressed for time to fix this (big deadline for next Friday). @asayers, do you by any chance have time to improve upon this?

Sure. I'll do it tomorrow.
I had a quick look at Clay to see if we could use it. It's intended for generating stylesheets, rather than inline styles, so you bind styles to selectors, rather than just listing them. I'll have to look into how easily we can separate the style-definition part of the library from the selector-definition part.

I had a quick look at Clay to see if we could use it. It's intended for generating stylesheets, rather than inline styles, so you bind styles to selectors, rather than just listing them. I'll have to look into how easily we can separate the style-definition part of the library from the selector-definition part.

Yes I noticed that too. @sebastiaanvisser would it be hard to separate (type-wise) the style-definitions from the selector-definitions?

@basvandijk The obvious (with obvious I mean easy) thing to me seems to change the selector semigroup into a monoid and allow empty selectors that can be used inside style attributes.
Separating the types will be harde because style are, by design, hierarchical in Clay, meaning you can always nest them creating something that doesn't fit within a style attribute. Of course you can always add a new module to Clay for inline styles that is just a flat list (or writer monad for neater syntax) of properties without hierarchy (Key (), Value).
The renderInline function would just be something like render . addEmptySelector.

For our purposes we would also need a version of render which returns something like HashMap Text Text, rather than just Text. It would be really cool though.

@roelvandijk I'm resistant to adding offsetX and offsetY. These values are inconsistent across browsers, and not even supported on firefox. I haven't tested it, but I expect that after this patch programs which use onMouseMove will throw errors on Firefox.

@asayers Yes, I just noticed a crash when testing on Firefox. It didn't mean for this commit to end up in a pull request. Tomorrow I'll test a variant that should work equally well on different browsers.
I'm a bit stressed for time and needed the mouse position relative to the targeted container. This only seems to work in Chrome, which is not very satisfactory.

@asayers That should be even easier I guess, no need to go into the final render step. Feel free to give it a try if you like :)
What is the reason for having the Hashmap instead of fully rendering it to Text, you want to do further processing?

@sebastiaanvisser We want to serialise the HashMap as a javascript object and pass it to ReactJS, which expects inline styles to be expressed in this way. I guess they think it makes a nicer API.

@basvandijk @roelvandijk I merged in the first two commits only and made some of my own changes. This means that we've now diverged, but hopefully it shouldn't be a problem for you to rebase on top of meiersi:master.

Thanks Alex that looks good.
Op 15 jan. 2015 03:17 schreef "Alex Sayers"<EMAIL_ADDRESS>
@basvandijk https://github.com/basvandijk @roelvandijk
https://github.com/roelvandijk I merged in the first two commits only
and made some of my own changes. This means that we've now diverged, but
hopefully it shouldn't be a problem for you to rebase on top of
meiersi:master.
—
Reply to this email directly or view it on GitHub
https://github.com/meiersi/blaze-react/pull/13#issuecomment-70029913.


Feature request for file quasiquoter

Could a quasiquoter that loads in a file be added? The end result for a user would be something like this:
let myStr = $(interpolatedFile "my_templates/email_body.template")

Or something like that. It would be very useful for quasiquoted strings that start getting really long.

Nevermind. I just realized that shakespeare already has something like this.


help i cant load a character

Describe your problem here.
i am working on getting a character into the game. the icon works just fine but when it tries to load the "character" itself it just crashes
Are you modding a build from source or with Lua?
Source
What is your build target?
Windows x64
Did you edit anything in this build? If so, mention or summarize your changes.
no

error says
C:\HaxeToolkit\haxe\std/haxe/xml/Parser.hx (Line 217)
C:\HaxeToolkit\haxe\std/haxe/xml/Parser.hx (Line 117)
C:\HaxeToolkit\haxe\std/haxe/Xml.hx (Line 122)
flixel/graphics/frames/FlxAtlasFrames.hx (Line 246)
Character.hx (Line 164)
PlayState.hx (Line 940)
flixel/FlxGame.hx (Line 637)
flixel/FlxGame.hx (Line 724)
flixel/FlxGame.hx (Line 677)
flixel/FlxGame.hx (Line 551)
openfl/events/EventDispatcher.hx (Line 402)
openfl/display/DisplayObject.hx (Line 1399)
Uncaught error: haxe.xml.XmlParserException: Expected node name at line 2 char 2.
Please report this error to the Github page: (link)

Certainly due to XML file, can you show it ?

here it is

Try add a prefix before animation name like "BF up0000"

I found the problem
Edit the first line to change "UTF-16" by "UTF-8"
And when you save, choose UTF-8 instead of UTF-16

THANK YOU it works (i spent a long time on this lol)


1.29 province mappings update

As discussed on Discord, this is too many changes in too many regions at once. It needs to be broken up into smaller pull requests.


Add contrib importance pie chart

This PR is part 2 of addressing issue #114 by making the following changes:

Creates a pie chart depicting contributor importance
Updates queries/contributors_query.py to query for contributor alias emails and returns the dataframe with an email column included
Updates chaoss/chaoss.py

Changes made:

Addressed no data issue for PR Review and Issue Open
Search for potential bots in login rather than email - this is mostly just a temp fix
Revert contributors_query.py to the most recent update in the dev branch

@christinaexyou is this meant to be closed?

The following changes were made:

Corrected original comment about isin for removing bots - it looks for exact matches between the login and the elements in patterns NOT substring matches
Updated graph info with a note about contributors that are ID'ed as None

@cdolfi @JamesKunstle
Changes made:

Added functionality user to create new keywords to filter out contributors by
Create dynamically changing title
Updated Graph Info description
Edited Pandas logic


Terraform Task Deprecation Warning

Noticed this deprecation warning when using the TerraformTaskV1@0 task, not sure what it means:
##[warning]This task uses Node 6 execution handler, which will be deprecated soon. If you are the developer of the task - please consider the migration guideline to Node 10 handler - https://aka.ms/migrateTaskNode10. If you are the user - feel free to reach out to the owners of this task to proceed on migration.

Hi, I will look into upgrading to node 10 or higher in one of the coming releases.

Any updates regarding this?  Is there any date to fix this warnings?
I'm using Microsoft-hosted agents and  every terraform tasks from the MicrosftDevLabs are getting this warning:
##[warning]This task uses Node 6 execution handler, which will be deprecated soon. If you are the developer of the task - please consider the migration guideline to Node 10 handler - https://aka.ms/migrateTaskNode10. If you are the user - feel free to reach out to the owners of this task to proceed on migration.
thanks!

I am aiming for the next release. No exactly sure when it will be. Soon


Background Color missing on PDF Export with "Crop" Option selected

Environment
draw.io Online Edition v12.1.0 / draw.io Desktop 11.3.0
Issue
When exporting a drawing to a PDF File with the "Crop" option selected, the solid Background Color is  omitted in the rendered PDF File.
However, when the "Crop" is not selected, the background Color is present as expected.

Thanks for the report, this will be fixed in the next release.


Proposal to change this.props inference based on props and defaultProps.

Search Terms
defaultProps, react, jsx

Suggestion
Currently while defining defaultProps we define our component something like this:
interface ABCProps {
  requiredProp: number;
  notRequiredProp: number; // Need to remember not to add ?
}

class ABC extends React.PureComponent<ABCProps> {
  static defaultProps = {
    notRequiredProp: 1
  }
  render () {
    // By not addiing ? in Props we don't need to check for undefined.
    const { notRequiredProp } = this.props
    return (

{notRequiredProp + 1}

    )
  }
}

But if we need to use ABCProps somewhere else, maybe as a part of parent component or if we create an object of type ABCProps, we will need to define the notRequiredProp as well.
const a: ABCProps = {
  requiredProp: 1,
  notRequiredProp: 2 // This is needed.
}

We can have two options:

Define ABCProps' optional props with ? and have component instances this.props be derived from ABCProps and defaultProps. (Breaking change)

interface ABCProps {
  requiredProp: number;
  notRequiredProp?: number; // ? added
}

class ABC extends React.PureComponent<ABCProps> {
  static defaultProps = {
    notRequiredProp: 1
  }
  render () {
    // this.props here !== ABCProps as is
    // typeof this.props === {
    //   [k in keyof ABCProps]: ABC.defaultProps[k] ? NonUndefined<ABCProps[k]> :  ABCProps[k]
    // }
    const { notRequiredProp } = this.props
    return (

{notRequiredProp + 1}

    )
  }
}

Define ABCProps' optional props without ? but have ABC["props"] be a modified version of ABCProps such that it makes the keys present in defaultProps as optional.

interface ABCProps {
  requiredProp: number;
  notRequiredProp: number; // Need to remember not to add ?
}

class ABC extends React.PureComponent<ABCProps> {
  static defaultProps = {
    notRequiredProp: 1
  }
}

// typeof ABC["props"] === {
//   [k in keyof ABCProps]: ABC.defaultProps[k] ? NonUndefined<ABCProps[k]> :  ABCProps[k]
// }

// typeof ABC["props"] !== ABCProps as is

const a: ABC["props"] = {
  requiredProp: 1
}

Use Cases & Examples
Should be able to reuse a Component's Props without having to define optional props as well.
Checklist
My first option would be a breaking change as people have started following advice from https://github.com/sw-yx/react-typescript-cheatsheet#typing-defaultprops.
My suggestion meets these guidelines:

[ ] This wouldn't be a breaking change in existing TypeScript/JavaScript code
[ ] This wouldn't change the runtime behavior of existing JavaScript code
[ ] This could be implemented without emitting different JS based on the types of the expressions
[ ] This isn't a runtime feature (e.g. library functionality, non-ECMAScript syntax with JavaScript output, etc.)
[ ] This feature would agree with the rest of TypeScript's Design Goals.

Maybe related: https://github.com/Microsoft/TypeScript/issues/28954
cc: @Kovensky because you are awesome.

If you're looking for a way to name your component's external props interface without repeating yourself, you want to do something like:
import React from "react";

interface ABCPropsInternal { // Shape of the props used within the component
    requiredProp: number;
    notRequiredProp: number;
}

class ABC extends React.PureComponent<ABCPropsInternal> {
    static defaultProps = {
        notRequiredProp: 1
    }
    render() {
        // By not addiing ? in Props we don't need to check for undefined.
        const { notRequiredProp } = this.props
        return (

{ notRequiredProp + 1}

      )
    }
  }

type ABCProps = JSX.LibraryManagedAttributes<typeof ABC, ABCPropsInternal>; // Calculates external props shape

const props: ABCProps = {
    requiredProp: 0
};

const component = <ABC {...props} />;


Feat/ownership

change ownership to @snyk/os-managed
add secrets scanning

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


Dev

Supply mule receives a new player station builder mode. In this mode, filling the building storage with materials is smoother. To activate the mode, you need to choose allow the build storage only. Not used to supply NPC stations.

@arshiba Everything looks fine to me, but can you go into a little more detail about the problem you were trying to solve? What you were seeing before and how your changes improve it?

Problem is:
step 1 plan a huge station;
step 2 assign a fleet to supply build storage;
step 3 wait for half a hour;
step 4 construction doesn't even start.
WTF?
I look at the storage and see something like:
advanced electronics 3k/7k
claytronics 5k/25k
field coils 3k/10k
hull parts 0/100k (!!!)
... etc
All because there are a plenty of electronics and coils and very little of hull parts and turret components in NPC stations. In this case the first ones will make more profit than the second ones all the time. It's not good.

Ah, yeah, I see. By prioritizing the lowest supply it ensures that constructions is always able to continue instead of getting stuck at a module or something. That seems good to me. Thanks for the contribution!


How to add two Actions to a Button ?

Version
Tell us which versions you are using:

react-native-router-flux v3.38.0
react-native v0.42.0

I want to add tow Actions to my button onPress handler, but I can't seem to figure how to do this. Could anyone assist me ?
I tried something like this but not surprising it didn't work.
<Button    onPress={Actions.Main1 && Actions.Tab2}    style={{ width: 90, height: 90 }} >       <Text>Show</Text> </Button>
Thank you !

This is not an issue. You should use StackOverflow for this type of question.