--idea not usable with //INCLUDE

I really like the idea of having a dedicated utils.kts with common functions used by other scripts, and it's currently doable with the include directive, however if I then try to generate the --idea project, the file being opened is some temporary consolidated script, as opposed to a symlink to my original file as I get if I don't use includes.
I believe this can be made more useful by instead symlinking both the script and all it's included scripts recursively, while consolidating their dependencies in build.gradle. This should resolve the "red references" problem, as well as let modify both script and its dependencies all in one intellij project, making it smoother to move stuff back and forth.
I'm happy to implement and send PR if there's no objections to this approach

Indeed, temporary projects will be based on the pre-processed script.
That's a tricky one fix. Either we resolve them or good code will be red in the temporary intellij project. Resolving them temporary will work for some but won't for others. Eg consider:
//INCLUDE ../../../foo.kt
//INCLUDE https://raw.githubusercontent.com/holgerbrandl/kscript/master/test/resources/includes/include_3.kt

For those ones I don't see any sensible way to create symlinks. I think we need to actually resolve them to provide a temporary project context.
Because how would you symlink them without rewriting the user script?
I guess we could fix some cases, but an incomplete solution might be more confusing than helpful. What do you think?

For links, it should be ok to symlink to it's cached .kts file in ~/.kscript dir, like the one that gets created when calling kscript with that url directly.
For parent dirs in path - as long as we don't attempt to have the directory structure in src/ mimic that of the corresponding scripts we should be fine. The script and foo.kt would just be symlinked side by side in src/
So, for your example, the way I imagine it, the ~/.kscript would look something like this:
~/.kscript
├── kscript_tmp_project__url_cache_d8f22e323c8720a2.kts_1519196345122/
│   ├── build.gradle
│   ├── local.properties
│   └── src/
│       ├── actualScript.kts -> /tmp/1/2/3/actualScript.kts
│       ├── foo.kts -> /tmp/foo.kt
│       └── url_cache_d8f22e323c8720a2.kts -> ~/.kscript/url_cache_d8f22e323c8720a2.kts
└── url_cache_d8f22e323c8720a2.kts

With this approach if multiple dependencies have the same base file name, we'd have to rewrite the symlink name in some way, so that they can both be placed in src/, but otherwise I think it should be a pretty good way to handle it.
What do you think about this approach?

A potential issue when referencing URL cache copies is that kscript silently assumes URLs to be static. So changing the cached URL, will affect the local script behavior, but the user might forget to change the URL target. However, that's something we're not likely to solve, and the rest of your plan sounds solid.
I think I would ignore potential name-clashes for now.
So the actual user script would remain unchanged then? I think this would be important to avoid confusion on the user side. Intellij should resolve the dependencies internally nevertheless if all imports are located in the src directory.
It would be great if you could help implementing that feature.

Perhaps the stale url cache issue can be solved by kscript also maintaining a symlink mapping a url to it's last downloaded cache, like:
~/.kscript/urlcache/https/raw.githubusercontent.com/holgerbrandl/kscript/master/test/resources/includes/include_3.kt -> ~/.kscript/url_cache_d8f22e323c8720a2.kts. Then we could simply link to that instead when creating project, sparing the user from having to ever manually update it.

So the actual user script would remain unchanged then?

Yes, we won't be modifying the user script - that would indeed be confusing. Just symlinking to it and its includes from the src/ directory
I'll start working on this then

Not sure if I understand this correctly. How would this cache differ from the current one, where we download and cache an url the first time kscript detects it in an import statement, and keeps the cached copy until the user clears the cache with --clear-cache?
I'd suggest to keep the caching as it is for now and just point the URL caching behavior out in the docs. For truely robust dependencies users could and should use @file:DependsOn anyway.
Looking forward to your PR. :-)

Never mind the caching thing - I misunderstood the hash in the name to be of the contents instead of of the url itself.


Search

Fixed the issue where valid budgets were saying invalid. Added comments and cleaned up some ugly nested if statements.

All conditionals work and the program continues if a user searches for a budget they don't have.


Upgrade to<EMAIL_ADDRESS>

The config is different in v4: See updated README.
I also upgraded to the newest postmark 2.x.x library.
Best,
Moritz

Great! It would be great to have a release 4.0.0 so that people can install this version of the package.
Best,
Moritz

Hey @ScottAgirs, I'm in the process of migrating to strapi 4. Can you make a release out of this? Thanks mate

Any change @ScottAgirs ? If you are not interested in the package anymore, you could make me maintainer

Any change @ScottAgirs ? If you are not interested in the package anymore, you could make me maintainer

Hey @laurenskling sorry for the delays, have severely limited bandwidth atm, heh. Definitely still interested, but also more than happy to open doors for you as a maintainer (and anyone else for that matter)!

Thanks mate. Although to release 4.0.0 I would need access on NPM as well. I don't know how that works, is that something you can share?

@laurenskling let's move this to Twitter or Discord?

@ScottAgirs I tried messaging you on Discord, no response :)


patch for min_length, max_length, exact_length

I've fixed an issue with min max and exact length validators. They were testing a numeric value vs a string value. Adding parseInt fixes this.
https://gist.github.com/f58c2dc75cef5c51d6fa

Thanks would you mind submitting it as a pull request?

Also you may want to have it be parseInt(value, 10) to be safe?

Agreed, will add the radix to the parseInt call and submit a pull request.


Update React peerDependency to rc1

To enable smooth usage of React 0.14 RC1 without NPM errors on install.

Thanks!


Sanity when working with relative paths in config

What did you do?
Inserted a relative path in prometheus.yml (see #2246 for some context).
scrape_configs:
  - job_name: 'mdns-sd'
    file_sd_configs:
      - files:
        - "./mdns-sd.json"

What did you expect to see?
That it resolved relative to the path of prometheus.yml (element of least surprise and all), imported the service-definitions of the file and so on.
What did you see instead? Under which circumstances?
Nothing. No errors, no internal metrics to indicate no files were found/imported.
Given I was working in an unfamiliar environment (FreeBSD jail; I'm a linux-guy), I couldn't readily find a way to find out what the working directory of the running prometheus server was (/proc/$pid/cwd doesn't exist, nor strace. Some poking at ktrace and dtrace didn't look promising without investing a lot of time in learning them, ...)
Though I did expect the menu status → Runtime & Build information to show the CWD, it didn't.
Environment

System information:
FreeBSD 10.3-STABLE amd64

Prometheus version:
build user:
build date:
go version:       go1.7.3

For the record, I ended up just banging in the absolute path in the config. Ugly, but working.

Nothing. No errors, no internal metrics to indicate no files were found/imported.

That's intended behavior. We expect rule and target files to not necessarily exist at startup time.
Generally, just using absolute paths in a production config is generally more sane but might indeed be ugly for longer paths.
For rule files an API endpoint (and corresponding UI view) showing what files were discovered were could help. But exposing equally detailed runtime information on file SD etc. would be quite the maintenance burden.

Right. I can see how it'll be difficult to expose meaningful info in such corner-cases.
Wrt. the metrics, I did expect something to show up on how many targets each service-discovery mechanism coughs up with (might also be relevant in comparison w. number of scraped services).
Another brainfart - expand relative paths to absolute ones when rendering the config.file in the UI.  I have a hunch that'd also be a lot of work for very little gain, but not much more.

I'm not sure we should be munging paths in the config. I can imagine that catching users out where the base of the path is dynamic, and they want to do a straight config copy elsewhere.

Guys, do not go crazy, just resolve relative paths in config as relative to config file location - like everybody else does. Anything else is counter-intuitive and doesn't makes much sense.
Another option is ban relative paths at all. But resolving relative paths in config as relative to directory where prometheus binary was executed - is just plain wrong because may result in undefined behaviour depending on subtle environment change.
But, if you insist on current behaviour, then lifehack is:
cd /etc/prometheus && prometheus -config.file ./prometheus.yml …

So what's missing to resolve this is to apply the resolution of files paths relative to the config to file SD? (https://github.com/prometheus/prometheus/blob/dev-2.0/config/config.go#L248)
Seems like merely an oversight in the past but by now probably a breaking change.

That sounds about right to me. I think this should go in 2.0.


BAK-1195 Litigation data model

This implements the design as detailed in the database schema and in the data in the spreadsheets
Also it was noted that some of the tables would have a max of 32768 rows as they were using SmallInteger - this type has been removed from the schema with the migration. This has been tested by checking no results were returned when running:
make start
pg_dump -s navigator | grep -i small

This PR has been approved - but its merged into https://github.com/climatepolicyradar/navigator/pull/804
Reason is that there are some slight adjustments being made to the schema and so as to not generate other migrations and keep things simple it will get merged to main as one single PR /cc @joel-wright


vm-trace is broken

The vm-trace feature for the runtime is broken. It's related to separating the disassembler so it's getting an out of index issue. This happens in the VM mod file when compiling functions. Disabling that feature for now in the pipeline.

This might be related to the same bug affecting the diagnosis messages getting out of index errors. I think it's related to the offset used for parsing especially when VM::run is ran multiple times.


HHH-9962 Second level query cache returns stale data

Test in courtesy of Giambattista Bloisi
Includes HHH-10008 fix as well because without that one the testHitCacheInSameSession would fail.
https://hibernate.atlassian.net/browse/HHH-9962
https://hibernate.atlassian.net/browse/HHH-10008

@sebersole should probably look on this since we were discussing HHH-9962 in JIRA.

Applied upstream.  Thanks Radim!


Add new cacheBuildWheelFiles:boolean configuration to the habushu-maven-plugin that caches build wheel files.

In order to make mono-repository wheel files more available across multiple modules we want to cache them into a reliable location.
This change introduces a new habushu-maven-plugin configuration cacheBuildWheelFiles:boolean
When set to true will copy the build's wheel file into poetry's cache directory
{poetry-cache-dir}/cache/wheels/{artifactId}/*.whl
When set to false nothing will be cached.
Default is false.

PR  Merged and completed.

https://github.com/TechnologyBrewery/habushu/pull/75Merged and completed.


Ensemble parallel

This tutorial shows:

How to use two QPUs in an ensemble model on iris dataset
That the ensemble is better than the individual models
An example of using Rigetti and Qiskit devices together

:ice_cream:
Just one layout suggestion


Sema: Back out https://github.com/apple/swift/pull/36752 entirely [5.6]

Unfortunately, we can't make this optimization work internally for now.
However, we can revisit it later. One strategy would be to emit the
witness as a private declaration, so that clients call it through the
witness table.
Fixes rdar://problem/86861522.

@swift-ci Please test


[CSL-293] Rewrite wallet functions in a way they don't require full WorkMode

@flyingleafe
Now wallet functions (such as submitTx, getBalances, etc.) work in WorkMode constraint. This is clearly an overkill, because many WorkMode components are not needed for these functions and such restriction is really inconvenient.

@gromak
Analytics from sofa
I would add WalletMode constraint somewhere is Pos.Wallet.
End of analytics from sofa


Not working on react 18 !!!

It should work if if you remove StrictMode. Not ideal, but a quick fix until React 18 compatibility is added.


[Feature] Option to toggle converting tabs to spaces

Hi, after some grepping around your codebase I think this should be fairly simple to implement. From my brief testing, commenting out all calls to _Detab() in "core/built/assets/vendor.js" and "node_modules/showdown-ghost/src/showdown.js" will cause tabs to be left alone in both the live markdown editor and the published posts.

Hi there @opeik!
To request a feature for Ghost you can add your idea (or vote on a similar one) over on our wishlist. We use this list to help determine what goes into the roadmap and how to prioritise features. You can also access the wishlist any time from inside the Ghost admin:

As a general rule, we don't add options to the admin panel for minor features. We pick a sensible default that suits our core user base. The tab key is used for navigating around the app and is an important accessibility feature, which is why we do not hijack it within the editor.

Hi, thanks for the response.

The tab key is used for navigating around the app and is an important accessibility feature, which is why we do not hijack it within the editor.

Fair enough, but I was referring to pasting code into preformatted code blocks. I don't see how these things are related.

@opeik Sorry for the confusion, your meaning wasn't clear from what you wrote 😉  Nonetheless we still wouldn't add options for this kind of detail. There is a plan to overhaul the editor to be more configurable in the future, but it's not something we're going to be working on for a little while.

I look forward to it. 👌


Make playerround table reflect all relevant play information

The playerround table contains a lot of information, but not all. It can be made in such a way that all player decisions and changes in the round are reflected. In that sense, it should contain:

start situation for finance (savings, debt)
all financial mutations during the round (income, living costs, mortgage payment, improvement costs, tax, damage costs, etc.)
start situation for house (owned house at start of round, start mortgage)
mutations for house (selling price, buying price, owned house at end of round, new mortgage)
start situation for satisfaction
mutation for satisfaction (bought satisfaction, moving, flooding, measures, etc.)

For finance, start values (either debt, or savings -- positive or negative value of last round):

start_savings (was called savings) [NEW_ROUND]
start_debt (was called debt) [NEW_ROUND]

fixed numbers (will be added/subtracted during round):

round_income (was called income_per_round) [READ_BUDGET]
living_costs [READ_BUDGET]

cost incurred during round:

spent_savings_for_buying_house [ASSIGN_HOUSES]
paid_mortgage [ASSIGN_HOUSES]
cost_taxes [CALCULATE_TAXES]
cost_measures_bought [BUY_IMPROVEMENTS]
cost_satisfaction_bought [BUY_IMPROVEMENTS]
cost_fluvial_damage {ROLL_DICE]
cost_pluvial_damage [ROLL_DICE]

For satisfaction, start values:

start_satisfaction (was called satisfaction) [NEW_ROUND]

changes incurred during round:

satisfaction_move_penalty [ASSIGN_HOUSES]
satisfaction_house_rating_delta [ASSIGN_HOUSES]
satisfaction_house_measures [BUY_IMPROVEMENTS]
satisfaction_bought [BUY_IMPROVEMENTS]
satisfaction_fluvial_penalty [ROLL_DICE]
satisfaction_pluvial_penalty [ROLL_DICE]
satisfaction_debt_penalty [SUMMARY]

For house, start values:

start_house_id
start_mortgage

fixed values:

maximum_mortgage
preferred_house_rating

round specific values:

mortgage_left
house_price_sold
house_price_bought
new_house_id
moving_reason
new_mortgage

Flood related fields:

fluvial_damage
pluvial_damage

Fields not used anymore:

repaired_damage (= cost_pluvial_damage, cost_fluvial_damage)
satisfaction_cost_per_point (is a value in scenarioparameters table)
spendable_income (calculated and adapted during the round)

Of course the fixed fields id, player_id, player_state, create_time, groupround_id stay in the table.

Allowed two fields to be negative:

profit_sold_house
final_spendable_income (positive = savings in next round; negative = debt in next round)


remove obsolete --mary-era option in faucet article

Following the guide at Explore Cardano Wallet I get the following error when querying wallets for UTxOs
cardano-cli query utxo --mary-era --testnet-magic<PHONE_NUMBER> --address $(cat ~/cardano/keys/payment1.addr)

Invalid option `--mary-era'
Usage: cardano-cli query utxo [--shelley-mode | --byron-mode
                                [--epoch-slots NATURAL] |
                                --cardano-mode [--epoch-slots NATURAL]]
                              (--whole-utxo | (--address ADDRESS) |
                                (--tx-in TX-IN))
                              (--mainnet | --testnet-magic NATURAL)
                              [--out-file FILE]
  Get a portion of the current UTxO: by tx in, by address or the whole.

Works fine without the --mary-era option.

These era options are confusing me a bit... I had some issues with them when querying an unsynced chain before but that was regarding shelley or byron-mode if I recall correctly...  can someone explain what the --mary-era option used to do?

It should be OK to use without any flag.

Hi, Newbie here.
Could you let me know what you mean by flag?
"It should be OK to use without any flag."
Thanks,

Nevermind re: flag __ It's a line in terminal : }
Yes it does work without  --mary-era flag


WaybackProxy

For content better suited for 90s browsers compared to #16, adding support for WaybackProxy would be nice.

Good idea. Instead of adding it to the current monolithic container, we should probably run it as its own container.
Some host-side user-space tool (just a script or something more fancy) for selecting the services would be nice, so user could enable this and get it automatically set up with non-conflicting IP addresses.


Build TBB using bee and editor toolchain

Initial commit for building TBB 2020 U3 using bee and the editor toolchain.
Changes to TBB:

Fix building in debug mode (see https://github.com/oneapi-src/oneTBB/issues/207, fixed by switching to cmake in 2021)
Fix missing symbols with lld (see https://github.com/oneapi-src/oneTBB/issues/477, fixed in lld 10).

TBH this is not what I imagined when you said you were going to use bee to build oneTBB :joy:. Invoking make through bee actions is very messy. I propose 2 alternatives:

Ditch make. Make a list of source files that make builds (it outputs everything it builds to stdout, so should be easy to get by using find/replace on that output) and build all the code through bee. That's what I actually did for OpenSSL (https://ono.unity3d.com/unity-extra/openssl/files/b3a2551832fd48b21b54d7f38fa10f4444eda692/OpenSSL.bee.cs) and while that took half a day to do, the building itself was 10x nicer afterwards;
Don't invoke make through bee. Use a bee target to only download the SDKs you want, and invoke make directly from a batch/shell script afterwards.

@SimonBoorer

TBH this is not what I imagined when you said you were going to use bee to build oneTBB 😂. Invoking make through bee actions is very messy. I propose 2 alternatives:

Ditch make. Make a list of source files that make builds (it outputs everything it builds to stdout, so should be easy to get by using find/replace on that output) and build all the code through bee. That's what I actually did for OpenSSL (https://ono.unity3d.com/unity-extra/openssl/files/b3a2551832fd48b21b54d7f38fa10f4444eda692/OpenSSL.bee.cs) and while that took half a day to do, the building itself was 10x nicer afterwards;
Don't invoke make through bee. Use a bee target to only download the SDKs you want, and invoke make directly from a batch/shell script afterwards.

@mirceaispas did the first alternative for USD by ditching cmake and it took much longer than half a day, introduced new issues, and there was no noticeable improvement to building. It also ends up relying on a similar approach to this PR to bootstrap cmake to get the list of source files. Perhaps we can schedule a catchup to share our experiences?
I'm not sure I see how the second alternative would be any cleaner. Instead of having one build script, we would then need one for Windows and one for Mac/Linux, or another cross-platform scripting language (e.g. Perl, Python) introducing another build dependency. We would also lose the Bee Sdk API and its knowledge of how the SDK artifacts are structured (e.g. compiler paths, include and library directories, etc.) and would need to reimplement this in our language of choice. Most of the mess I see here is intrinsic to building for multiple platforms and architectures, and not to invoking make through bee. Happy to discuss this further with you offline.


Tenancy: Implement __FALLBACK tenant and __FALLBACK privilege

This change solved two problems in this issue https://github.com/vmware/docker-volume-vsphere/issues/621.

support __FALLBACK tenant
support __FALLBACK privilege
The remain problem "support default_datastore" will be addressed in my next PR. In my next PR, I plan to  address "support default_datastore" and privilege changes (https://github.com/vmware/docker-volume-vsphere/issues/709)

This change also include necessary unit test to support __FALLBACK tenant and __FALLBACK privilege.

@msterin @shaominchen

@msterin Please review this change.

@pdhamdhere
__FALLBACK tenant and __FALLBACK privilege are created by system. Admin can  remove the __FALLBACK tenant. Admin can also remove or modify __FALLBACK privilege.

I have adressed all comments. Please take a look.

CI test failed in "VmdkClone" test. I have added more debug info in the change to debug the CI failure.
Please ignore the change related to that.
All other comments have been addressed. Please review it.
@msterin @pdhamdhere


Not all intents are added

Summary
there's a lot of missing intents
Reproduction Steps
None
Minimal Reproducible Code
No response
Expected Results
None
Actual Results
None
Intents
none
System Information
None
Checklist

[X] I have searched the open issues for duplicates.
[X] I have shown the entire traceback, if possible.
[X] I have removed my token from display, if visible.

Additional Context
No response

Seems like I misread the docs.


Incorrect Partitioning

I am trying to run the following circuit with and without partitioning:
import QuantumCircuit from 'quantum-circuit';
import fs from 'fs';

function getGates(circuit) {
  var gates  = {}
  circuit.forEach(qubit => {
    qubit.forEach(gate => {
      if (gate) {
        gates[gate.name] = gates[gate.name] ? gates[gate.name] + 1 : 1;
      }
    })
  });
  return gates;
}
function loadQASMCircuit(file) {
  let data = fs.readFileSync(file, 'utf-8');
  if (data) {
    const circuit = new QuantumCircuit();
    circuit.importQASM(data, function(errors) {
      if (errors) {
        console.log("Errors:",errors);
      }
    });
    return circuit;
  } else {
    console.error('Error reading file:', err);
    return null;
  }
}

const neewss = loadQASMCircuit("mycircuit.qasm")
/*
// mycircuit.qasm
OPENQASM 2.0;
include "qelib1.inc";
qreg q[3];
creg c[2];
h q[0];
h q[1];
h q[2];
s q[2];
s q[2];
h q[2];
h q[2];
cx q[0],q[2];
cx q[1],q[2];
h q[0];
h q[1];
h q[1];
h q[0];
cx q[1],q[2];
cx q[0],q[2];
h q[2];
x q[2];
h q[1];
h q[0];
*/

for(let circuit of [neewss]) {
  const shots = 100;
  const results = {};
  console.log("********".repeat(10))
  let initalValues = []
  for(let i =0; i<circuit.numQubits; i++) {
    initalValues.push(1);
  }

  console.log(initalValues)
  for (let i = 0; i < shots; i++) {
      circuit.run(initalValues, {partitioning: false});
      const measuredState = circuit.measureAll();
      const bitstring = measuredState.map(bit => bit ? '1' : '0').join('');
      if (!results[bitstring]) {
          results[bitstring] = 0;
      }
      results[bitstring]++;
  }
  console.log(getGates(circuit.gates))
  console.log(results)
}

The expected result is state 111 for all 100 shots i.e: { '111': 100 }
But with partitioning the result is not as expected it is {'010': 100}
What is partitioning meant to do, i see that documentation is lacking for this...

@glanzz partitioning is meant to be used to split circuit into smaller "partitions" when circuit is separable. For example, if 30-qubit circuit doesn't have any connection between first 15 qubits and last 15 qubits (there is no multi-qubit gates between any of first 15 qubits and any of second 15 qubits), then circuit is separable into two 15-qubit circuits. That is two "partitions" which can be executed separately and their results combined into one 30-qubit state vector.
However, this idea is abandoned early, and partitioning code is not maintained, but is still present in the source code.
I advise you: do not use it.

How about other options provided by the compiler ?


Add First Contributions

I want to add First contributions to this project.
First contributions helps beginners get started with contributing to open source projects and it helped me personally as a beginner and first timer to make my first contribution here  It was so straight forward and hands-on you can contribute in less than ten minutes with suggestions on what to work on next and a slack community to help/ mentor beginners.
I would love to make a pull request to add it to your list of resources that helps beginners and first timers to contribute because it is a very good tutorial to get started.

Thanks for the suggestion! We'll accept a PR to add this.


Multiple IDs for getCharacteristicData

I am wondering if it is possible to change getCharacteristicData so it can accept multiple feature IDs rather than making several requests, one ID at a time. There could be a limit, for example 10 IDs or something like that.

This has been overcome by events and will need to be rethought. I think it may make the most sense to create a pygeoapi process for characteristics that uses HyRiver. For the python rewrite of the NLDI, I think we should probably deprecate the characteristics end point.


Harvester crawl doesn't seem to be working?

Hi there,
Trying to get this setup for work we've got quite a few projects, and the harvester crawl seemed to be perfect, however I can't seem to get it working with more than one numerated log:
See below my docker-composer instance:
  logioweb:
    image: 'quay.io/blacklabelops/logio:latest'
    # container_name: logio
    environment:
      - 'LOGIO_ADMIN_USER='
      - 'LOGIO_ADMIN_PASSWORD='
      - LOGIO_CERTIFICATE_DNAME=
    ports:
      - '28778:28778'
    volumes:
      - '/opt/server'
    networks:
      air:
        ipv4_address: <IP_ADDRESS>

  harvester:
    image: 'quay.io/blacklabelops/logio:latest'
    environment:
      - 'DELAYED_START=10'
      - 'LOGIO_HARVESTER_NODENAME=DockerLogs'
      - 'LOGIO_HARVESTER_STREAMNAME=UnifiedLogs'
      - 'LOGIO_HARVESTER_LOGFILES='
      - 'LOGS_DIRECTORIES=/var/lib/docker'
      - 'LOG_FILE_PATTERN=*.log'
      - 'LOGIO_HARVESTER1STREAMNAME=Phoenix_Reborn'
      - 'LOGIO_HARVESTER1LOGSTREAMS=/var/log/eagleeye/phoenix-reborn/'
      - 'LOGIO_HARVESTER1FILEPATTERN=*.log'
      - 'LOGIO_HARVESTER2STREAMNAME=Medusa'                     # When I remove the second addition the first one now then works, but when i add this second one, I get no nodes show up in log.io
      - 'LOGIO_HARVESTER2LOGSTREAMS=/var/log/eagleeye/medusa/'
      - 'LOGIO_HARVESTER2FILEPATTERN=*.log'
    volumes:
      - '/var/lib/docker:/var/lib/docker'
      - './log/:/var/log/eagleeye/'

Any idea what could cause this?
Cheers!

After speaking with @blacklabelops I had bad config, however unfortunately, I still get the issue - however I'll close this.

There is a bug inside the loop variables. Fixed it.


Added support for ActiveDirectoryInteractive authentication

Add new type to authentication property. ie `authentication=ActiveDirectoryInteractive". Driver will attempt to silently authenticate if access token has been cached or will be presented with a browser popup to authenticate interactively.


feat(catalog): always use search for UrlReaderProcessor

Hey, I just made a Pull Request!

BREAKING: The UrlReaderProccessor now always calls the search method of the UrlReaders. Previous behavior was to call the search method only if the parsed Git URL's filename contained a wildcard and use readUrl otherwise. UrlReaderService must implement this logic in the search method instead.
This allows each UrlReaderService implementation to check whether it's a search URL (that contains a wildcard pattern) or not using logic that is specific to each provider.
In the different UrlReadersService, the search method have been updated to use the readUrl if the given URL doesn't contain a pattern.
For UrlReaders that didn't implement the search method, readUrl is called internally.

See comment by @Rugvip for the reasoning: https://github.com/backstage/backstage/pull/27054#discussion_r1801439398
Closes https://github.com/backstage/backstage/issues/22504
Unblocks https://github.com/backstage/backstage/pull/25704
:heavy_check_mark: Checklist

[X] A changeset describing the change and affected packages. (more info)
[X] ~Added or updated documentation~
[X] Tests for new functionality and regression tests for bug fixes
[X] ~Screenshots attached (for UI changes)~
[X] All your commits have a Signed-off-by line in the message. (more info)

Hi @Rugvip 👋
I've finished the implementation for other readers. Let me know if there's something to change or a flag to make the change smooth 👍

Not stale!


remove uninitialized _dataAsRow flag from LDA

see issue #5318
the flag never gets initialized, and is obsolete, since the whole LDA expects row-aligned data

if i remove _dataAsRow it does not build
leaving it in, just for the ABI is bad as well.

can someone tell me, if i understood it right ?
LDA is one of the rare classes, that can be constructed directly (not like an interface/factory),
so, the abi check complains about the change in memory layout ?
if so, does it really matter (given any use case) ?
(keeping the unused flag in silences the bot, but that's definitely not an option)

unfortunately, removing the flag from the LDA class breaks the ABI check in linux buildbot.

You could rename this flag to _unused instead of removing or add explicit flag initialization to constructor with a brief comment.

ok thanks for the hint, i will just keep it, initialize, and remove the usage.

:+1:

found same problem already in version 2.4.9


Easier flatExtracting usage

For example, instead having to implements Extractor<BasketBallPlayer, List<BasketBallPlayer>>  to be able to call:
// teamMatesExtractor() returns a new Extractor<BasketBallPlayer, List<BasketBallPlayer>>
assertThat(list(noah, james)).flatExtracting(teamMatesExtractor()).contains(irving, rose);

I just want to write (assuming BasketBallPlayer has a teamMate property):
assertThat(list(noah, james)).flatExtracting("teamMates").contains(irving, rose);

or the type safe version:
assertThat(list(noah, james)).flatExtracting("teamMates", BasketBallPlayer.class)
                             .contains(irving, rose);

Implemented the non-typesafe variant but i have the feeling that it was too easy. Basically i was overloading AbstractIterableAssert.flatExtracting/AbstractObjectArrayAssert.flatExtracting and used internally Extractors.byName.
After a short feedback i would create also the typesyfe variant.

Thanks that was quick  ! :)
Can you create a PR ? It's just easier to comment code.
Can you check that you have the correct formatting preferences, there were so much diff due to reformatting that it is hard to see what has really changed.
From what I saw, we should (if possible) probably reuse the existing flatExtracting your code seems to duplicate it, I also don't understand the NPE.

I have refactored and reformatted the code and created a PR. I also removed the exception wrapping which was a result of a manually created extractor of the test classes.

Thanks, integrated with few improvements.


Query: difference between chaperone terms

Hi
please could someone explain the difference between these two terms and then improve the definitions so that it is easier to choose which one to use:
GO:0140318 protein transporter activity
Molecular Function
Definition: Directly binding to a specific protein and delivering it to a specific cellular location.  PMID:18706423

Synonym
Type

protein transport chaperone
related

protein carrier activity
related

GO:0140597 protein carrier chaperone
Molecular Function
Definition:
Binding to and carrying a protein between two different cellular components by moving along with the target protein. PMID:7628437
Synonyms are alternative words or phrases closely related in meaning to the term name, with indication of the relationship between the name and synonym given by the synonym scope.
Synonym 	Type
protein carrier activity 	exact
protein chaperone broad
Thanks
Ruth

The definition of "carrier" chaperone is a bit strange considering its child terms? I wouldn't say histone chaperones were "carrying a protein between two different cellular components".  They are preventing the histone from being lost from a location and making inappropriate interactions.

I agree this is not very clear.
Actually I had proposed to obsolete 'GO:0008565 protein transporter activity' but it seems I didn't :
https://github.com/geneontology/go-ontology/issues/17071
This is the ticket where I created GO:0140318 protein carrier activity
https://github.com/geneontology/go-ontology/issues/28445
In a way protein transporter activity serves as a useful (?) grouping term but then we should move GO:0140597 protein carrier chaperone under it?
Thanks, Pascale

We can't move protein carrier under "protein transporter activity" because histone chaperone is not a transporter.
Probably the main issue is that chaperones and carriers are conflated?
Histone chaperones should be defined along the lines:
proteins that assist in the assembly, disassembly, and exchange of histones on DNA.
and this would apply to many assembly/dissassembly chaperones?


[incubator-kie-issues#1719] Enable onEntry/onExit scripts in embedded nodes

Closes https://github.com/apache/incubator-kie-issues/issues/1719

PR job #1 was: UNSTABLE
Possible explanation: This should be test failures

Reproducer
build-chain build full_downstream  -f 'https://raw.githubusercontent.com/${AUTHOR:apache}/incubator-kie-kogito-pipelines/${BRANCH:main}/.ci/buildchain-config-pr-cdb.yaml' -o 'bc' -p apache/incubator-kie-kogito-runtimes -u https://github.com/apache/incubator-kie-kogito-runtimes/pull/3815 --skipParallelCheckout
NOTE: To install the build-chain tool, please refer to https://github.com/kiegroup/github-action-build-chain#local-execution

Please look here: https://ci-builds.apache.org/job/KIE/job/kogito/job/main/job/pullrequest_jobs/job/kogito-runtimes-pr/job/PR-3815/1/display/redirect
Test results:

PASSED: 3515
FAILED: 1

Those are the test failures:

org.jbpm.bpmn2.IntermediateEventTest.testMultiInstanceLoopCharacteristicsTaskWithOutputCmpCondSequential
Expected size: 1 but was: 0 in:[]

@jomarko thanks for looking into this issue, and for your input. This is very much appreciated, as you bring in some new perspectives and raise valid points!
Regarding your question on embedded nodes and user tasks, what is used in some of the tests here are multi-instance tasks (similar to MI subprocesses, but just in a single task). For the engine, this task is then embedded in a ForEachNode. I hope this helps to clarify.


The response I got for the SP creation was different from the documentation

My JSON had only appId, displayName, name, password and tenant.
Not any of the others.

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

ID: cc8d39c7-f55b-70e2-d99e-6fccb3b5ad34
Version Independent ID: f6714300-fd05-f95b-6afe-182e250fd180
Content: Quickstart – Azure Key Vault Python client library – manage secrets
Content Source: articles/key-vault/secrets/quick-create-python.md
Service: key-vault
Sub-service: secrets
GitHub Login: @msmbaldwin
Microsoft Alias: mbaldwin

@yuvalm2
I just wanted to check in and see if you had a chance to review my previous post.
I look forward to your response.
Thank you for your time!


Add the decoration manager to the IRContext.

To make the decoration manger available everywhere, and to reduce the
number of times it needs to be build, I add one the IRContext.
As the same time, I move code that modifies decoration instruction into
the IRContext from mempass and the decoration manager.  This will make
it easier to keep everything up to date.
This should take care of issue #928.

On Tue, Nov 14, 2017 at 3:16 PM, Steven Perron<EMAIL_ADDRESS>wrote:

// Phase 5: Ensure the import and export have the same types and decorations.
res = CheckImportExportCompatibility(consumer, linkings_to_do,
*linked_context.get_def_use_mgr(),

                                  decoration_manager);

                                  *linked_context.get_decoration_mgr());

No. linked_context is not a pointer. The * is dereferencing the pointer
returned by "get_decoration_mgr" so that it passed the decoration manager
by reference.

Ugh.  Right.

I made the suggested changes.  Let me know if more are needed.  The only thing that I think needs to be done is to add test for KillNameAndDecorates to make sure it deletes the OpMemberNames.  Existing test are covering the removal of the decorations and OpName.

Test are added, should be good for another look.


Add link button doesn't work if you set textbox value with javascript

Hi,
thnks and congratulation for this fantastic work!
When I try to set textbox content (for data-edit="createLink") with javascript, the Add button doesn't work.

What can I do to make it work ?
Thanks!
Nicola

Hi Nicola,
Thanks for reporting you're findings.  I was able to replicate your results so I'll have to look and see what's causing the exact issue. Hopefully if all goes well I hope to be able to have a PR for you to test this afternoon. (I'm in CST timezone)

How's the status on the fix for this?
I've been looking at your way of inserting links and can see that IE doesn't really support the createLink method. Even the MSDN demo fail to work in IE11.
Should an alternate method be made for IE, or should the feature just be listed as "not working" for now?

For now, I'd probably say it's not working.
This has been pretty much been out on hold so I can work on the OOP and unit test implementation.


Dependency cycle in CamundaBpmAutoConfiguration

In some circumstances spring is not able to detect the right bean creation order with our auto configuration. The only way to reproduce I found is:
Running org.camunda.bpm.spring.boot.example.dmn.rest.DmnRestApplication results in
***************************
APPLICATION FAILED TO START
***************************

Description:

The dependencies of some of the beans in the application context form a cycle:

┌─────┐
|  dataSource defined in class path resource [org/springframework/boot/autoconfigure/jdbc/DataSourceConfiguration$Dbcp.class]
↑     ↓
|  processEngineFactoryBean defined in class path resource [org/camunda/bpm/spring/boot/starter/CamundaBpmAutoConfiguration.class]
↑     ↓
|  processEngineConfigurationImpl defined in class path resource [org/camunda/bpm/spring/boot/starter/CamundaBpmConfiguration.class]
↑     ↓
|  camundaDatasourceConfiguration (field protected org.springframework.transaction.PlatformTransactionManager org.camunda.bpm.spring.boot.starter.configuration.impl.DefaultDatasourceConfiguration.transactionManager)
↑     ↓
|  org.springframework.boot.autoconfigure.jdbc.DataSourceTransactionManagerAutoConfiguration$DataSourceTransactionManagerConfiguration
└─────┘

org.camunda.bpm.spring.boot.example.dmn.rest.DmnRestApplicationTest is still stable.

To give spring a more clean hint I separated the ProcessEngineConfigurationImpl depending beans in a custom @Configuration


use cache for both poetry workflow runs

by caching poetry dependencies, we should hopefully significantly speed up the testing and the typing runs.
Theoretically the "fastest" option is to install pyright/pytest on their own, but then we have two places where dependencies are stored, instead of the single-source-of-truth being poetry.

...first cache hit, looks like the net saving is ~5 seconds? Poetry accounts for the vast majority of that setup time

I hate to be that person, but there's a much simpler way: https://github.com/actions/setup-python#caching-packages-dependencies
Imo if you're looking for improving testing run times, I think that combining the static check + unit test actions would be better. After all, they are both essentially testing the code itself. Black can remain separate, as it can run in parallel to these and doesn't require poetry.

There was smething about unable to pass certain parameters into poetry when using setup-python's cache, but I'll give it a look. If we use setup-python's cache it might even cache the actual poetry install, which would be significantly faster.

Yeah OK so I can make it cache the actual poetry install, which will be the fastest option (saves ~19s of poetry install on cache hit) but completely locks us out of using setup-python's autocaching because it requires us actually caching the .poetry folder... which is the opposite of your suggestion :skull:
I'd prefer to keep tests & typing separate so its easier to see what actually failed on a particular PR, although we don't really have enough tests for that to be a massive problem (yet).

Alright, we're now successfully caching poetry and the poetry dependencies.
It looks like the test workflow has gone from ~40s to ~20s, and the typecheck workflow has gone from ~40s to ~30s. (I'm unable to work out how the heck those numbers work, given we've eliminated the same steps from both workflows, but whatever.)
I don't believe it's possible to cache anything else, or if its even worth it considering the setup is now 13s of basically just "download the cache files".


Node::select is very slow

Node::select is significantly slower than Node::traverse(_inclusive) through all of the nodes and manually filtering!

My bad, this was not the actual issue.


Redundant method call

String.valueOf(String) returns exactly the same String object. Removed the redundant method call.

Thanks!


'scipy.signal.windows' periodicity

Hi,
In testing the hann function (and a few others), odd length windows are the same for both symmetrical and periodic types. For example:
from scipy.signal import hann
w1 = hann(5,True) #array([ 0. ,  0.5,  1. ,  0.5,  0. ])
w2 = hann(5, False) #array([ 0. ,  0.5,  1. ,  0.5,  0. ])

I think the cause is the N-1 denominator which is not adjusted for odd lengths when periodic.
I believe this issue was brought up here

I read the discussion and partly the paper that was mentioned. The paper can be found online. There's also a discussion about this topic on wikipedia.
There was also a discussion on the Octave mailing list and bug tracker.
Hamming
In the case of the Hamming window, Scipy 0.15.1 hamming function gives the expected result for an even amount of points (this example was brought up on the mailing list):
>>> hamming(8, False)
array([ 0.08      ,  0.21473088,  0.54      ,  0.86526912,  1.        ,
        0.86526912,  0.54      ,  0.21473088])

which is different from the symmetric case.
>>> hamming(8, True)
array([ 0.08      ,  0.25319469,  0.64235963,  0.95444568,  0.95444568,
        0.64235963,  0.25319469,  0.08      ])

Hann
When we look at odd length windows, then indeed symmetric and periodic cases windows are the same
>>> hamming(7, True)
array([ 0.08,  0.31,  0.77,  1.  ,  0.77,  0.31,  0.08])
>>> hamming(7, False)
array([ 0.08,  0.31,  0.77,  1.  ,  0.77,  0.31,  0.08])

Let's now have a look at the Hann window
>>> hann(6, True)
array([ 0.       ,  0.3454915,  0.9045085,  0.9045085,  0.3454915,  0.       ])
>>> hann(6, False)
array([ 0.  ,  0.25,  0.75,  1.  ,  0.75,  0.25])
>>> hann(5, True)
array([ 0. ,  0.5,  1. ,  0.5,  0. ])
>>> hann(5, False)
array([ 0. ,  0.5,  1. ,  0.5,  0. ])

Cause
The reason the symmetric and periodic windows are the same for even length is because of the lines
    if not sym and not odd:
        M = M + 1
    ...
    if not sym and not odd:
        w = w[:-1]

If the and not odd condition would be removed then symmetric and periodic windows would be different.
Octave behaves similarly to Scipy. Matlab behaves differently though,
hann(5, 'symmetric')
         0
    0.5000
    1.0000
    0.5000
         0

hann(5, 'periodic')
      0
    0.3455
    0.9045
    0.9045
    0.3455

What's correct?
The question is now, what is the correct behavior?

Duplicate of https://github.com/scipy/scipy/issues/2190 ?

Indeed, thanks @endolith - closing this as a duplicate. Have linked to this issue from gh-2190, because the links here are useful.

The discussion on the Octave bug tracker proposes to just add all possible options and document them properly. Might be a reasonable way to go here as well.

I think this demonstrates that the current behavior is incorrect:
In [19]: abs(fft(hann(8, sym=False)))
Out[19]: array([ 4.,  2.,  0.,  0.,  0.,  0.,  0.,  2.])

In [20]: abs(fft(hann(7, sym=False)))
Out[20]:
array([ 3.        ,  1.82397424,  0.21573417,  0.0397084 ,  0.0397084 ,
        0.21573417,  1.82397424])

If the window is periodic, and raised cosine (=Hann), then its frequency spectrum should only have DC + one frequency component.  hann(8, sym=False) looks this way, but hann(7, sym=False) does not.

just add all possible options

So that would be "MATLAB symmetric", "MATLAB periodic", and "DFT even", as described in Wikipedia?
But hann(8, sym=True) doesn't match any of those plots.  It looks like the "MATLAB symmetric" plot, except it goes to 0 at the ends.
hann(8, sym=False) matches the "DFT even" plot.

@endolith I think you're right with the test.
I removed the and not odd requirement
np.abs(np.fft.fft(hann_fix(7, False)))
array([  3.50000000e+00,   1.75000000e+00,   2.30026015e-16,
         1.70955155e-16,   1.70955155e-16,   2.30026015e-16,
         1.75000000e+00])

and all that remains is the DC component and the single frequency with its conjugate.
This is the same result as you would get in Matlab.

This was added Nov 2001 in https://github.com/scipy/scipy/commit/c910a765560a2edce76e1be8b2a9eead5b434b0d by @teoliphant .  I don't see any explanation of why it was done that way.
Octave only added this 2 months ago?

Yes, octave-signal has periodic since 1.3.1. However, while they mention in their changelog they have now this Matlab-compatible option, the results do not match with Matlab.


[WIP] Fix #64 prompting error

This can fix issues #64.
It seems that first argument of done() expects error object, and second one expects return value.

I think I could find the cause of this error.
At 20 hours ago, yeoman-environment upgraded to use inquirer v1.0.0,
https://github.com/yeoman/environment/releases/tag/v1.6.0
and inquirer v1.0.0 has changed some its interfaces.
According to the README:
https://github.com/SBoudrias/Inquirer.js/compare/v0.12.0...v1.0.0#diff-04c6e90faac2675aa89e2176d2eec7d8R115
{
  /* Preferred way: with promise */
  filter: function () {
    return new Promise(/* etc... */);
  },

  /* Legacy way: with this.async */
  validate: function (input) {
    // Declare function as asynchronous, and save the done callback
    var done = this.async();

    // Do async stuff
    setTimeout(function () {
      if (typeof input !== 'number') {
        // Pass the return value in the done callback
        done('You need to provide a number');
        return;
      }
      // Pass the return value in the done callback
 -    done(true);
 +    done(null, true);
    }, 3000);
  }
}

Thank you for digging into this! It's a pain that yeoman-environment changing it's inquirer dependency causes downstream breakage 😓  I'm not sure there's anything we can do for existing generator-hubot aside from releasing an update with this.
I'm going to review and test, and hopefully get this out soon.

I wonder if we could check the arity of done, ie if (done.length == 2).

Tested and this works ✨  We can leave a potential backwards compat fix ie checking arity of done, and get this out so new users don't have a bad times 😓

This is released in<EMAIL_ADDRESS>Anyone running into this should npm install -g generator-hubot to get the latest.


How to clean gpu-operator value which is installed last time

I installed the gpu-operator and want to install again.
When I run
sudo helm install --wait --generate-name      -n gpu-operator --create-namespace   --set driver.enabled=false nvidia/gpu-operator

and get the error
Error: INSTALLATION FAILED: Unable to continue with install: ClusterRoleBinding "gpu-operator" in namespace "" exists and cannot be imported into the current release: invalid ownership metadata; annotation validation error: key "meta.helm.sh/release-name" must equal "gpu-operator-1702439910": current value is "gpu-operator-1702434888"

It seems i have not clean the value clearly. How to clean it?
Output of helm version:
version.BuildInfo{Version:"v3.13.2", GitCommit:"2a2fb3b98829f1e0be6fb18af2f6599e0f4e8243", GitTreeState:"clean", GoVersion:"go1.20.10"}

Output of kubectl version:
Client Version: v1.28.4
Kustomize Version: v5.0.4-0.20230601165947-6ce0bf390ce3
Server Version: v1.26.0

Cloud Provider/Platform (AKS, GKE, Minikube etc.):
kind cluster

https://github.com/NVIDIA/gpu-operator/blob/fd2b1587d5a8a7cd5a3b28afbf2be80d67d0d3d5/deployments/gpu-operator/templates/rolebinding.yaml#L4C15-L4C15
you can see the chart code, the name of ClusterRoleBinding is fixed, and this is the the key issue. different release managed the same ClusterRoleBinding named gpu-operator. it will conflict.

From the looks of it, the chart installs a ClusterRoleBinding named gpu-operator. You can't install another chart (or the same chart twice) which also attempts to re-use this name.
You will need to follow up with the chart owners, likely to make the name of thisClusterRoleBinding dependent on the chart release name.

I delete the ClusterRoleBinding and clusterroles related to gpu-operator with kubectl delete ClusterRoleBinding/clusterroles name, and install gpu-operator again. Still the same error.

I 've resolve this problem by referring to this article.
Wish it helps !!

Solved.
helm ls -n gpu-operator
helm uninstall gpu-operator-xxx -n gpu-operator


fix: share function

Summary by CodeRabbit

New Features

Added functionality to generate and copy short links using the Dub.co API.
Introduced state management and dynamic icons to indicate link generation status (loading, copied, error).

Enhancements

Updated the package to include the "axios" library for making HTTP requests.

Sorry. I forgot it. I will update it.

 Create DubCo Share Thread Feature


[Multitenancy] functionality of webhooks, labels and image_url per wallet

Description:
The webhook functionality is an essential feature, because the controller should receive messages from the aca-py server. This PR enables the functionality of webhooks, labels and image_url per wallet. Also, it contains useful other features. See updates and APIs changes below.
Terms:
Base wallet: Wallet created when aca-py is launched.
Updates:

webhook_urls works per wallet
label works per wallet
image_url works per wallet
on demand wallet loading (reduce boot-up time)
wallet mapping (connection_id or key -> wallet_name) is stored in the storage (preparation for scale-out aca-py)
When registering nym, optional wallet_name is used to register the did of another wallet
When creating local did, optional seed is used

APIs changes:

POST /wallet : Add new wallet (added webhook_urls, label, image_url) - public
GET /wallet : Get information of wallets with query - base wallet only
PUT /wallet/{wallet_id} : Update information of a wallet - base wallet only
DELETE /wallet/{wallet_id} : Remove a wallet - base wallet only
GET /wallet/me : Get information of my wallet - user wallet
PUT /wallet/me : Update information of my wallet - user wallet
DELETE /wallet/me : Remove my wallet - user wallet
/wallet/did/create : Create a local DID (added seed)
/ledger/register-nym : Send a NYM registration to the ledger (added wallet_name to register the did of another wallet)

Demo changes:

The --multitenant flag enables multitenancy support. With this, Faber and Alice will use a single Base agent. Please see the detailed flows in README-multitenant.md.

TODO

Test code updates (Several send_webhook functions included context as a parameter to make webhook works per wallet)

Can you provide some info on how to test this code?
It's tough to asses without any unit tests or a working demo ...
Thx

@ianco I pushed the additional commits to updates faber/alice demo. Also, I updated the description for the demo changes. Thanks!

I restored additional options for creating a new wallet in faber/alice demo. I agree that these options are effective in demonstrating the features of multitenancy.


Include py.typed

Tested with
$ python setup.py bdist_wheel | grep typed
...
adding 'runez/py.typed'

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

Project coverage is 99.78%. Comparing base (422d761) to head (ea4c844).

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##             main      #27      +/-   ##
==========================================
- Coverage   99.95%   99.78%   -0.17%
==========================================
  Files          25       25
  Lines        6655     6655
==========================================
- Hits         6652     6641      -11
- Misses          3       14      +11

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


[Chip]: onRemove runs twice

Describe the bug
onRemove runs twice
To Reproduce
Steps to reproduce the behavior:

Go to fusion Chip react component in Examples
Click on removable and set it to true
Open console
See error running twice in console when clicking on x button

Expected behavior
onRemove should run only once
Desktop (please complete the following information):

OS: MacOS
Browser: Microsoft Edge
Version 113.0

solved in web-components


Seems there is a double "/" in the file name in cache folder...

Hello,
First of all thanks a lot for this really cool config file!
I just tried it and it seems that it generate a double / here https://github.com/maximejobin/rocket-nginx/blob/master/rocket-nginx.conf#L60
Here is a debug header:
x-rocket-nginx-file:/var/www/mysite.com/wp-content/cache/wp-rocket/mysite.com//2016/01/article-slug//index.html_gzip

The problem is in mysite.com//2016 for me.

Does it cause a problem ?  If so, what problem ?  Do you have a working example to reproduce the problem ?
The fact that there are 2 / should not change anything.  It is used to get the right cache file to display.  It has nothing to do with the URL.  If a problem occurs in the URL, please report it.

Sure it seems to be working, but that sounds like a possible little doable optimization no?

No, it wouldn't be an optimization.  It may look like it but, depending on how the HTTP request is built, it could cause problems.  The fact that you see 2 / is because not all parts of the request are used. Nginx's configuration does not handle a lot functions to "trim" and validate strings.
A lot of code would need to be added to "solve" the double slash. In the end, it would add more confusion.
That said, glad you take a look at what's under the hood! :-) :+1:

OK nevermind then... Thanks for your answers.


Fix Chat permissions mapping

Requests sent to TG API with values as "CanSend*" are ignored.
Only correct values like can_send_* are processed.

Consider adding test-cases for these changes into types_test.go, @Alexkurd.

@OvyFlash please look


TestNG run inherited method twice

Hi,
TestNG will run the inherited method twice in this case:
Image we have two classes
For parent, we have method a and b
For child, we have method c and d.
<test name="RstTest">
    <classes>
        <class name="Parent">
            <methods>
                <include name="a" />
                <include name="b" />
            </methods>
        </class>
        <class name="Child">
            <methods>
                <include name="c" />
                <include name="d" />
            </methods>
        </class>

Instead of running a, b, c and d
TestNG will run a b b a c d
After checking the code, I found this special case confused the testng init,
TestNG will use a map m_includedMethods in XmlMethodSelector to save the relationship between class full name and the includes methods, when it parse to Child, firstly, it will load all of methods (Even includes the method from parent class), then compare if the method is included or not, due to method a and b will use the same full class name as the one in parent class (During this time, method a and b have been injected into that map), so that's why it run more than once.
Thanks very much.
Br,
Tim

@wutingbupt I think your analyse is good. Maybe you can propose a test case that shows the issue, and a fix?

Maybe related to #949
See my comments on https://github.com/cbeust/testng/pull/950#issuecomment-179829948

I will try if I can fix it.
2016年2月9日 下午5:32，"Julien Herr"<EMAIL_ADDRESS>
@wutingbupt https://github.com/wutingbupt I think your analyse is good.
Maybe you can propose a test case that shows the issue, and a fix?
—
Reply to this email directly or view it on GitHub
https://github.com/cbeust/testng/issues/980#issuecomment-181944593.

The test case is a good start ;)


fix: User defined preset dashboards are ignored

What this PR does / Why we need it:
Preset Dashboards that are "pre-installed" and owned by Dynatrace are ignored during a download using Monaco, as these Dashboards cannot be modified by the user at all and re-deploying them will result in an error.
However, we should not ignore preset dashboards defined by the user.
This PR changes the filter for persisting dashboards to skip dashboards only if they are presets AND owned by "Dynatrace"
Special notes for your reviewer:
Does this PR introduce a user-facing change?

Users are able to download dashboards marked as "preset" if they were created ba themselves

E2E test failing because of current issues in test setup wrt. automation resources


Bazelisk is missing some incompatible flags

There's a mismatch with the CI: https://buildkite.com/bazel/bazelisk-plus-incompatible-flags/builds/37#6dc0f493-7763-47c2-9ff0-5d9ba95e86e5
For example, notice that the CI lists incompatible_no_transitive_loads in the list of flags, but Bazelisk doesn't use it when calling Bazel.
The list of flags we use come from: https://api.github.com/search/issues?q=repo:bazelbuild/bazel+label:migration-0.23
It surprinsingly doesn't include incompatible_no_transitive_loads (https://github.com/bazelbuild/bazel/issues/5636)
cc @meteorcloudy

I'm confused by GitHub search.
The 2nd request has results that the 1st one doesn't have, such as incompatible_no_transitive_loads:
https://api.github.com/search/issues?q=repo:bazelbuild/bazel+label:migration-0.23
https://api.github.com/search/issues?q=repo:bazelbuild/bazel+label:migration-0.23+state:open

https://developer.github.com/v3/#pagination

Requests that return multiple items will be paginated to 30 items by default.


Put cursor at the end of the line

My cursor is positioned at the start of the line instead of the end of the line when I press the up or down key. I found that the plugin changes the default bindings of the up and down keys to do a history substring search.
https://github.com/softmoth/zsh-vim-mode/blob/d0a53168fb448894b7a33befa30e489ef77c81d2/zsh-vim-mode.plugin.zsh#L179-L182
Can you make the cursor go to the end after pressing up or down? If that's not possible, can you add an option to get back the default behavior of the up and down keys?

Also, this doesn't happen when the zsh-history-substring-search plugin is loaded.
https://github.com/softmoth/zsh-vim-mode/blob/d0a53168fb448894b7a33befa30e489ef77c81d2/zsh-vim-mode.plugin.zsh#L173-L177
So, this part of the code is working fine and the cursor is placed at the end appropriately.

Thanks for the bug report. I had missed this before now. Yes, it should be possible to get the cursor to the end of the line. Or else restore the default binding, and rely on history-substring-search for added functionality.

I've fixed this by restoring the default binding. As you said, the zsh-history-substring-search plugin is a good alternative for people who want a fancier Up-arrow experience.
Fixed in 1fd2124 .


API endpoint for deploy#find_app_id seems to have been incorrect or old ...

...(and was causing a zat update to fail), so I updated it to '/apps/owned.json' per the documentation

Strange, as far as I know, /api/v2/apps.json should work however it will list both public apps and the private apps you've uploaded, compared to /api/v2/apps/owned.json.
/cc @zendesk/quokka

Please reopen if the problem is still there. We believe that it's not a problem. I personally use zat update fairly often.

Sorry, meant to update this. The /api/v2/apps.json endpoint is working for me again; it must have been down for some reason when I submitted this.

Good to know, thanks!
On 01/04/2015 8:51 am, "Joe Palmieri"<EMAIL_ADDRESS>wrote:

Sorry, meant to update this. The /api/v2/apps.json endpoint is working
for me again; it must have been down for some reason when I submitted this.
—
Reply to this email directly or view it on GitHub
https://github.com/zendesk/zendesk_apps_tools/pull/75#issuecomment-88261608
.


Enable CodeQL scanning

Changes

Adds codeql configuration. See See https://docs.github.com/en/code-security/code-scanning/automatically-scanning-your-code-for-vulnerabilities-and-errors/about-code-scanning-with-codeql

Submitter Checklist
As the author of this PR, please check off the items in this checklist:

[x] Has Docs included if any changes are user facing
[x] Has Tests included if any functionality added or changed
[x] Follows the commit message standard
[x] Meets the Tekton contributor standards (including
functionality, content, code)
[x] Release notes block below has been updated with any user facing changes (API changes, bug fixes, changes requiring upgrade notices or deprecation warnings)
[x] Release notes contains the string "action required" if the change requires additional action from users switching to the new release

Release Notes
NONE

I see this check detected two failures under third_party. Should we configure it to ignore that path?

There didn't seem to be an easy way to ignore the path for Go :(
Looking at this directory though, it seems very stale since we don't have any automation ensuring this stays up to date. AFAICT the top level licenses and source code (for things like MPL) are already included vendor, so I don't see a good reason to keep this around - removed!

/hold We need to tweak how we're handling licenses for publishing. This will need a bit more work. >_>

jk we tar up the entire vendor folder as part of releases - we're good to go!
/hold cancel


Fix search if there is no good solution

Currently the top solution gets 50% of the next generation. We need a quick if statement to check if the best is above a threshold. If it is then randomly select the entire next generation.

Patch included. Needs testing.

Patch seems to be working well.


Conflict with Shatur/neovim-session-manager: The coroutine failed  with this message ... Invalid buffer id: 1

Bug description
I'm not sure which plugin is the guilty here, so I'll open the same issue for both: https://github.com/Shatur/neovim-session-manager/issues/36
VGit was working well when I was using neovim 0.6.1 version but when I updated to 0.7.0 I start getting this issue only when I use Shatur/neovim-session-manager with VGit. for simplicity and make sure that I am right, I've used ~/.config/nvim/simple-init.vim:
How to start Neovim with simple configuration file:
nvim -u ~/.config/nvim/simple-init.vim

The simple configuration:
let g:plug_home = '~/.config/nvim/plugins'

call plug#begin(plug_home)
Plug 'nvim-lua/plenary.nvim'
Plug 'tanvirtin/vgit.nvim'
Plug 'Shatur/neovim-session-manager'
call plug#end()

lua << EOF
require('vgit').setup()
EOF

The error message:
Error executing vim.schedule lua callback: ...ig/nvim/plugins/plenary.nvim/lua/plenary/async/async.lua:14: The coroutine failed
 with this message: .../nvim/plugins/vgit.nvim/lua/vgit/features/LiveGutter.lua:30: Invalid buffer id: 1
stack traceback:
        [C]: in function 'error'
        ...ig/nvim/plugins/plenary.nvim/lua/plenary/async/async.lua:14: in function 'callback_or_next'
        ...ig/nvim/plugins/plenary.nvim/lua/plenary/async/async.lua:40: in function <...ig/nvim/plugins/plenary.nvim/lua/plenar
y/async/async.lua:39>

Environment

nvim --version output: NVIM v0.7.0
Operating system: Arch Linux x86_64 [kernel: 5.17.1-arch1-1]
Desktop environment: Suckless DWM
Plugins commit hash:

neovim-session-manager: f0b0501
vgit: 9a2ef0b

I got something similar:
Error executing vim.schedule lua callback: ...ck/packer/start/plenary.nvim/lua/plenary/async/async.lua:14: The coroutine failed with this message: .../packer/start/vgit.nvim/lua/vgit/features/LiveGutter.lua:30:
 Invalid buffer id: 1
stack traceback:
        [C]: in function 'error'
        ...ck/packer/start/plenary.nvim/lua/plenary/async/async.lua:14: in function 'callback_or_next'
        ...ck/packer/start/plenary.nvim/lua/plenary/async/async.lua:40: in function <...ck/packer/start/plenary.nvim/lua/plenary/async/async.lua:39>

🤖

This issue is not happening now. May be a NeoVim bug fix or a fix in the other plugin.


Pump Priming Funding: Scope possibility of application for KCL mini project for collab with KCL heat experts/analysts

[ ] Meeting with Alisha/Scott/Katrina on 15 Nov
[ ] Share with Mark/Gavin
[ ] Convo with Mat Corbett (Harry & Ruth)
[ ] Decide whether or not to apply

Timeline

Released 13 Nov 2023
Due in January 2024

Open Questions

When would this work start?
Can the funding be used for staff costs? (e.g. someone from Turing/someone from King's)
Is there the possibility of matched funding from Turing (15k + 15k)?

9 Nov Ideas

Where are older/vulnerable people located in future heatwaves?
How can we quantify individual risk based on data available in SPC?
Ultimate aim is to enable larger grant funding (e.g. funding postdoc/analyst time)
Deliverables: quantifying individual risk using SPC
Funding for people would have to remain within King's (e.g. Bowen would have to be added to King's talent bank); could also be access to King's HPC

Latest from @RuthBowyer is this is no longer a good fit so closing this issue for now


Fix CVE-2020-15257 for v1.2

What is the problem you're trying to solve
Fix CVE-2020-15257 for v1.2
Describe the solution you'd like
Backport the soluction of fix CVE-2020-15257 to v1.2.x
Additional context
CVE-2020-15257: https://github.com/containerd/containerd/security/advisories/GHSA-36xw-fx78-c5r4
1.3 backport: https://github.com/containerd/containerd/commit/ea765aba0d05254012b0b9e595e995c09186427f
1.4 backport: https://github.com/containerd/containerd/commit/269548fa27e0089a8b8278fc4fc781d7f65a939b

v1.2 reached EOL on October 15, 2020: https://github.com/containerd/containerd/blob/master/RELEASES.md
So, we are unlikely to ship a new v1.2.x release officially, but distro vendors may backport the patch to their v1.2.x packages.

👌

There was a patch created and verified, we did pull that onto the 1.2 release branch https://github.com/containerd/containerd/commit/3b3e9d5f62a114153829f9fbe2781d27b0a2ddac
As Akihiro said, there will be no release with this code. You are welcome to build yourself or rely on vendor patched version.


Add a more minimal panic message and update cortex-m

This PR adds a new feature called min-panic. This version of the panic message only prints out the line number at which the panic occurred if it exists. If it does not exist, the only thing that's printed out is that a panic occured.
This feature may help in situations with very small size constraints, as it means that the most important part of the error (i.e. the line number) is printed first, instead of the panic message.
While they could be considered equally important, knowing the line number generally helps more than (part of) the panic message.
Also updates cortex-m to the latest version

Thanks!

Included in the v0.3.0 release!


break apart argonaut

purescript-argonaut-core — Definition of Json, foldJson, and Eq, Show, Ord, etc., instances for Json, together with printing and parsing (which are native methods!). This will be a very small library with minimal dependencies, which other libraries can use to agree on what Json is.
purescript-argonaut-combinators — All the combinators and operators that make using Argonaut easy. These, in turn, are largely defined in terms of the fold methods, but some may be special-cased for performance reasons.
purescript-argonaut-serialization — EncodeJson / DecodeJson and instances for the world.
purescript-argonaut-zipper — For things like JCursor, JZipper, etc. Maybe there's a better name for this one or maybe it belongs somewhere else.

Thoughts?

Shall we make this repo aggregation like purescript-base?

Yes, this one should re-export the other modules in Data.Argonaut, and of course depend on them all in bower.


Artificially high time estimate for uninstalling

If the "turbo MSI" stuff doesn't fix this, I don't think there's much else we can do on our side. Gonna close it, since the "turbo MSI" stuff is merged in omnibus.


Regression: multiple createProcedure or sql changes in one changeSet with different dbms values causes "not a supported DB" error

Search first

[X] I searched and no similar issues were found

Description
We have multiple changeSets that worked in Liquibase 3.5.1 that contain three createProcedure changes, each with a unique dbms value (mssql, oracle, postgresql). When run with 4.25.0, they produce errors like below:
Unexpected error running Liquibase: Validation Failed:
2 changes have validation failures
oracle is not a supported DB, src/test/resources/TestChangelogOld.xml::100::test
mssql is not a supported DB, src/test/resources/TestChangelogOld.xml::100::test
The error message is misleading as oracle and mssql are valid values for dbms.
Splitting the changeSet into three changeSets, each with the unique dbms value at the changeSet level works around the problem.
TestChangelogs.zip
Steps To Reproduce
Unzip the attached zip file. Edit TestCreateProcedure.sh as needed for jar file paths, then execute.
The first Liquibase execution will fail with the above errors. The second execution should succeed.
Expected/Desired Behavior
A changeSet with multiple createProcedure changes should be supported, especially if each createProcedure element has a unique dbms value.
Liquibase Version
4.25.0
Database Vendor & Version
No response
Liquibase Integration
CLI
Liquibase Extensions
No response
OS and/or Infrastructure Type/Provider
Ubuntu 23.10
Additional Context
PostgreSQL
15.5 (Ubuntu 15.5-0ubuntu<IP_ADDRESS>)
PostgreSQL JDBC Driver
42.2.5
Are you willing to submit a PR?

[ ] I'm willing to submit a PR (Thank you!)

Hello @dhsmith1001! We were trying to reproduce it using your example and i failed as expected.
But we noticed that we can remove run parameter --databaseClass=liquibase.database.core.PostgresDatabase and then it will start working. In fact database class is not required/necessary anymore, so you can get rid of that. Would it work for you?
Anyway, we acknowledge that when dbms value is used within createProcedure it has a different behavior than when used in the changeSet element and we should fix this.

Same error with multiple  changes in one changeSet.

Same error with multiple changes in one changeSet.

That was wrong, it's when a changeSet has dbms with > 1 value:  dbms="postgresql,mssql". Error is mssql is "mssql is not a supported DB"

See https://github.com/liquibase/liquibase/pull/5397

Fixed by https://github.com/liquibase/liquibase/pull/5397


{% include [] %}

Could we have a possibility to use include with an array of templates as it is the case for the php twig?
http://twig.sensiolabs.org/doc/2.x/tags/include.html

It will be very useful too me too
Any plans on this ?

+1

+1


[Feature Request] A subcmd note to record something

Is your feature request related to a problem? Please describe.

No, it's a real new feature.

Describe the solution you'd like

I would like to use it like:
$ tweedle note
Notes:
	...
	...
	...

$ tweedle note --current-workspace
Is a python workspace ? [Yes]/No:<CR>
Notes(Python-related):
	CLI:
		pipx --designed for installing executable applications in python
		...
	Other4Dev:
		yapf --Format code
		isort --imports sort
		...

As you can see, it maybe can detect workspace type automatically..

Notes

Maybe it will support a web view and CLI original view.

Inspired

starship/starship: How to detect cwd state and type...

Enhancement Proposal:

Record cmds section

# tweedle record <Your want to run and record cmds>
  tweedle record poetry config virtualenvs.in-project true
[Exec <Your want to run and record cmds> ...]
need edit? Yes/[No]
save? [Yes]/No


App overrides should be field-granular

App overrides are currently coarse-grained - overridden registries or environments take precedence in their entirety over those defined in the primate app configuration.
This can lead to surprising behavior when packages are scoped to an environment, and the user intended only to override a namespace, for instance.
We can improve on this by merging environments defined in app.yaml with those defined in app.yaml.override. Only fields actually overridden should take effect.

Fixed in #708.


No Results from Query on Indexed Ulid Property

I have a model with a secondary index on another Ulid:
[Document(StorageType = StorageType.Json)]
public class Song
{
    [RedisIdField]
    public Ulid Id { get; set; }

    [Indexed]
    public Ulid AlbumId { get; set; }
}

When I try and query all songs matching an album the response comes back empty.
var result = _collection.Where(x => x.AlbumId == albumId);

However if I first enumerate the collection and then filter, I get the data as expected.
var result = _collection.ToList().Where(x => x.AlbumId == albumId);

In my main Program.cs I am calling await redisConnectionProvider.Connection.CreateIndexAsync on my models to build the indexes.
I feel like I'm missing something obvious here but have been debugging this for a while to no avail.

@rpf3 - As it stands now, Redis OM is only set up to index/search numerics, strings, and GeoLocs. When you ask it to index a ULID, it doesn't know how to interpret the ULID for index creation so it ignores it. Try changing AlbumId -> a string and you'll see it work as expected.
A broader effort to index/search other value types (enums, ULIDs, GUIDs, etc. . .) is something that is probably worth adding at some point down the lin.

@slorello89 thank you, converting the property to a string worked as you described.  I agree that indexing Ulid would be especially useful given that Redis OM generates them as IDs by default.

Going to roll this into #128


Fix Crash due to Matrix Validation in non Matrix Pipeline

Fixes #8086
Changes

Submitter Checklist
As the author of this PR, please check off the items in this checklist:

[ ] Has Docs if any changes are user facing, including updates to minimum requirements e.g. Kubernetes version bumps
[ ] Has Tests included if any functionality added or changed
[ ] pre-commit Passed
[x] Follows the commit message standard
[ ] Meets the Tekton contributor standards (including functionality, content, code)
[ ] Has a kind label. You can add one by adding a comment on this PR that contains /kind <type>. Valid types are bug, cleanup, design, documentation, feature, flake, misc, question, tep
[ ] Release notes block below has been updated with any user facing changes (API changes, bug fixes, changes requiring upgrade notices or deprecation warnings). See some examples of good release notes.
[ ] Release notes contains the string "action required" if the change requires additional action from users switching to the new release

Release Notes
NONE

/retest

Are tests failing due to WIP?

/retest

@vdemeester Sure. I am adding that.

@vdemeester I have added one regression test.

@vdemeester Examples aren't run as upgrade tests. We need a more complex pipelines for upgrade tests.

#8086 is marked as closed by #8089, but this is meant to fix the same issue. Do we need to reopen #8086, was the fix incomplete?

@afrittoli I would say it's a partial fix. Error can happen in some other flow also.

@vdemeester Examples aren't run as upgrade tests. We need a more complex pipeline for upgrade tests.

I think @vdemeester question was about why did you modify the upgrade test instead of the regular integration test.


[feature request] Filter "today and unread"

Filtering is not very user friendly.
I would suggest at least adding some options like "Today and unread" due to the fact, filter is shared among all subscriptions.
Better solution would be to add some buttons to fast filter unread/all/today ... instead you have to click on filter option, move your mouse to select one, click again.

@spenat28 are the default smart folders not accomplishing the same task?

Smart folders aggregate all feeds. I guess that @spenat28 wants to navigate between feeds.
But what would be the benefit of "Today and unread" over "Unread"? What is the strategy regarding backlog articles?


make extension work for web

gz#11869

Hi, Tabnine is not supported on vscode.dev.
I will pass this request to the team. If we'd see enough interest from the community, we will make sure such a plugin exists :)

thx

The TabNine extension can work with VSCode Web, all you have to do is port forward the ports used by the hub and tabnine-deep from localhost (where you are accessing VSCode Web) and the code-server (where the tabnine engines are running).

it says the extension isn't available in marketplace when searching for extension on https://vscode.dev - not sure how that approach resolves this @ksylvan ?

it says the extension isn't available in marketplace when searching for extension on https://vscode.dev - not sure how that approach resolves this @ksylvan ?

Oh I see, vscode.dev is the ready to use version. I’m using my self-hosted version:
See https://github.com/ksylvan/remote-coding/blob/main/README.md#a-note-on-using-tabnine-and-vscode-web


help: Update "private message" -> "direct message".

Updates help center documentation to match new direct message (DM) term in Zulip UI.
CZO discussion.
Screenshots and screen captures:

https://zulip.com/help/private-messages

https://zulip.com/help/reading-pms

https://zulip.com/help/restrict-private-messages

https://zulip.com/help/pm-mention-alert-notifications

Self-review checklist

[x] Self-reviewed the changes for clarity and maintainability

Communicate decisions, questions, and potential concerns.

[x] Highlights technical choices and bugs encountered.
[x] Calls out remaining decisions and concerns.
[x] Automated tests.

Individual commits are ready for review (see commit discipline).

[x] Each commit is a coherent idea.
[x] Commit message(s) explain reasoning and motivation for changes.

Completed manual review and testing of the following:

[x] Visual appearance of the changes.
[x] Links.

Thanks, those screenshots look good to me!
@timabbott This is WIP, because not all affected pages have been updated yet.

@alya This PR is ready for you.

OK. When this is reviewed, we may want to just have @laurynmm append it to the PR doing the renaming, so we can rebase/merge them together as a single action.

Hm, there's a failing test, but it doesn't seem related to this PR:

The Puppeteer frontend tests failed! The failing test was:
./tools/test-js-with-puppeteer user-deactivation.test.ts

I think we shouldn't change the mobile instructions, because the tab is still called "private messages" on a tablet, right? In any case, having those instructions have a mix of "private messages" and "direct messages" seems a bit.
Otherwise looks good to me! I have not attempted to verify whether the changes cover everything.

Just do a rebase to clear the test failure -- it's a nondeterminstic flake.

I think we shouldn't change the mobile instructions, because the tab is still called "private messages" on a tablet, right? In any case, having those instructions have a mix of "private messages" and "direct messages" seems a bit.
Otherwise looks good to me! I have not attempted to verify whether the changes cover everything.

I thought maybe the mobile app labels would be updated by the time this got merged.
I changed the mobile instructions back to "Private message" and rebased/updated the PR .

Sounds good, thanks! @chrisbobbe @gnprice let's coordinate on documentation changes when we update the mobile UI to rename PMs -> DMs.

@timabbott please take a look.

Merged, thanks @drrosa!


missing assets folder

Hello
saw in part of the code relation to assets folder but cannot find it in the zip file. Am I doing something wrong?
Kind regards
//neph

Hi @nephilim75 can you be more specific I'm not sure if I understood your question, also de lib does
not have any assets folder

@nephilim75 I just notice that you might refer to the tooltips on the site examples,
in the website we are using the Github Primer project to do the CSS + icon part there,
you can check more here https://github.com/zenorocha/clipboard.js#tooltips

Closed due to inactivity


ARTEMIS-3477 Upgrade selenium-java to 3.141.59

Seems ok, I think you should just push it.
(We should really use properties and in the parent pom to govern versions, but theres a wider change needed here, can do it later all together)


feat: Workflow Semaphore Support

Checklist:

[ ] Either (a) I've created an enhancement proposal and discussed it with the community, (b) this is a bug fix, or (c) this is a chore.
[ ] The title of the PR is (a) conventional, (b) states what changed, and (c) suffixes the related issues number. E.g. "fix(controller): Updates such and such. Fixes #1234".  #2550
[ ] I've signed the CLA.
[ ] I have written unit and/or e2e tests for my change. PRs without these are unlikely to be merged.
[ ] My builds are green. Try syncing with master if they are not.
[ ] My organization is added to USERS.md.

Did you add configmap GET RBAC privileges to the controller?
configmap RBAC is already there for Controller.

- apiGroups:
  - ""
  resources:
  - configmaps
  verbs:
  - get
  - watch
  - list

Should we wait to merge this until after the 2.9 GA?

yes, we can merge after 2.9 GA to avoid the cherry-picking

This has introduced a double save of workflows that 2x the API throughput. Needs to be fixed and fix needs to be backported.

@alexec Can you please provide more details?


Low potassium/phosphate diets

Phillipa (one of our supporters) runs RenalRecipies on Instagram. We can use this as a base.
I’m thinking an information page with a sample recipe and links would be the best place to start here

Should also mention baby formula of renastart and renastep


Download Progress not showing during installation in Terminal

Type: Bug
Whenever I am downloading any library, the progress and text that used to show while downloading are not showing up, instead only a spinner comes and only after everything is downloaded, text comes up.
VS Code version: Code 1.90.2 (5437499feb04f7a586f677b155b039bc2b3669eb, 2024-06-18T22:34:26.404Z)
OS version: Windows_NT x64 10.0.22621
Modes:

System Info

Item
Value

CPUs
13th Gen Intel(R) Core(TM) i7-13700HX (24 x 2304)

GPU Status
2d_canvas: enabledcanvas_oop_rasterization: enabled_ondirect_rendering_display_compositor: disabled_off_okgpu_compositing: enabledmultiple_raster_threads: enabled_onopengl: enabled_onrasterization: enabledraw_draw: disabled_off_okskia_graphite: disabled_offvideo_decode: enabledvideo_encode: enabledvulkan: disabled_offwebgl: enabledwebgl2: enabledwebgpu: enabled

Load (avg)
undefined

Memory (System)
15.70GB (6.06GB free)

Process Argv
--crash-reporter-id 236ad759-0032-40c6-afd1-5fe3e056cdfc

Screen Reader
no

VM
0%

Extensions: none
A/B Experiments
vsliv368:30146709
vspor879:30202332
vspor708:30202333
vspor363:30204092
vscorecescf:30445987
vscod805:30301674
binariesv615:30325510
vsaa593cf:30376535
py29gd2263:31024239
c4g48928:30535728
azure-dev_surveyone:30548225
962ge761:30959799
pythongtdpath:30769146
welcomedialogc:30910334
pythonnoceb:30805159
asynctok:30898717
pythonregdiag2:30936856
pythonmypyd1:30879173
h48ei257:31000450
pythontbext0:30879054
accentitlementsc:30995553
dsvsc016:30899300
dsvsc017:30899301
dsvsc018:30899302
cppperfnew:31000557
dsvsc020:30976470
pythonait:31006305
jchc7451:31067544
chatpanelc:31048052
dsvsc021:30996838
g316j359:31013175
pythoncenvpt:31062603
a69g1124:31058053
dvdeprecation:31068756
dwnewjupyter:31046869
legacy_priority:31082724

https://github.com/microsoft/vscode/assets/148696092/59ccd092-353c-4bb5-ad3f-8d236da00057

https://github.com/microsoft/vscode/assets/148696092/63da97ed-7e4b-426d-8d02-2c8f1096cb0f

thanks for the gif, I'm not sure what you mean - could you be more specific or provide a gif where they were working in prior versions?

https://stackoverflow.com/questions/78693097/terminal-not-displaying-installation-progress-properly
I have posted this on StackOverflow, it has the links to reference videos.

looks like you're in a different working directory when you ran in the new vs old

Yes, because these video are of 2 different devices

The different environments are likely the cause, unrelated to vs code

Okay, but do you know the fix, any suggestions?


Define a JointTrajectoryController for franka_gazebo

Replacement for #186, at least defining a joint trajectory controller at all.

Thanks @rhaschke for this, but we were working on this already internally. See https://github.com/frankaemika/franka_ros/commit/e899240365797de6d53c222d6512b74c3a40268e for details. This also adds an effort_joint_trajectory_controller to franka_control with the gains from @rickstaa in https://github.com/frankaemika/franka_ros/pull/186#issuecomment-956144658.


Read connection data by lines

I try to read the logcat lines from the open connection following the example. Using connection.read(1024) read the number of characters but it is possible to read the logcat by lines?
Thanks.

Maybe you can try to convert the socket to the file object by makefile
from adb.client import Client

client = Client()
device = client.device("emulator-5554")

def dump_logcat_by_line(connect):
    file_obj = connect.socket.makefile()
    for index in range(0, 10):
        print("Line {}: {}".format(index, file_obj.readline().strip()))

    file_obj.close()
    connect.close()

device.shell("logcat", handler=dump_logcat_by_line)


Hamburger Menu Overlapping Title Text On Narrow Screens

I'm opening way too many issues today, aren't I?
Anyway, see this twitter thread.
Reproducible by opening https://blog.alfiepates.me on a 360px-wide "mobile device" using the Google Chrome developer tools.

 I’ll put something permanent in place shortly.


Add new eyebrow banner component

Summary
Part of https://github.com/github/primer/issues/2779
Adds new eyebrow banner component.
🔗 Documentation
🔗 Storybook
Todo in follow up PR:

[ ] Test coverage
[ ] Integration tests

List of notable changes:

added eyebrow banner component
added  new eyebrow slot to Hero component, to enable placement of an eyebrow banner (amongst other things)

What should reviewers focus on?

Need additional middle step values for gradient colors
Review of light mode, for which we don't have any prior art

Steps to test:

Use this storybook link and review the individual stories and playground
Use this documentation link to read through public docs

Contributor checklist:

[x] All new and existing CI checks pass
[ ] Tests prove that the feature works and covers both happy and unhappy paths
[x] Any drop in coverage, breaking changes or regressions have been documented above
[x] New visual snapshots have been generated / updated for any UI changes
[x] All developer debugging and non-functional logging has been removed
[x] Related issues have been referenced in the PR description

Reviewer checklist:

[ ] Check that pull request and proposed changes adhere to our contribution guidelines and code of conduct
[ ] Check that tests prove the feature works and covers both happy and unhappy paths
[ ] Check that there aren't other open Pull Requests for the same update/change

Screenshots:
Playground walkthrough:
https://github.com/primer/brand/assets/13340707/37864018-1df3-4210-9054-4cb97e12a809
Hero placement:

Looks good @rezrah.
Couple changes suggestions.

Let's remove the linear gradient we have and substitute it with #0D1117 at 60% opacity since it creates legibility issues.
On mobile, let's make the border radius 16px to avoid the weird shape it's getting.
Also on Mobile, can we make the font-size font-size: var(--brand-text-size-100);? I think 200 it's too big in comparison to the text.

Thatss all.

Thanks @jesussandreas. All feedback addressed...
https://github.com/primer/brand/assets/13340707/38fd35e5-6158-451b-8f95-3fba067900ba


Add missing build command

Motivation
I got someone commenting in Backstage that Humanitec integration was not working.
We are not creating bundles before publishing packages as a result dist files are missing.
Approach
Make sure that we build files before publishing packages using INSTALL_SCRIPT

📣 NOTIFICATION
You are receiving this message because we did not publish any packages.
Generated by @thefrontside/actions


Added TimingInput react component

https://user-images.githubusercontent.com/749094/180838100-789b51d0-426c-4580-a525-c47b7176fecf.mp4

Coverage decreased (-0.02%) to 94.634% when pulling 4f82307a5a6dddc028eb58fffc7bb0f29ae959b8 on cody-timing-input into 7268d8e7c5e3c415cc9727ef9bfdcb86bfb8d2c2 on main.


Parameters/Statistics for HypreAME

Hello Everyone,
I am using HypreAME for a Maxwell eigenvalue problem (pretty much example 13 on a hemispherical domain), and I am having issues with computing higher-order eigenvalues. Usually, I can get between 110 and 115 eigenvalues, and beyond that the solver fails with "Error in LOBPCG: Bad initial vectors: orthonormalization failed".

Is it possible to check if the solver converged? So that I am not saving "solutions" if it didn't
I think LOBPCG allows for more parameters than I can readily set from MFEM, like the block size. Is it possible to get access to those? Maybe I will be able to find a combination that works for my problem, like computing the eigenvalues in blocks of 50 instea of all at once.
Is it possible to compute eigenvalues in the neighbourhood of a given value or above a given value? (Simple shifting does not work for that.)
Is it possible to run it without the preconditioner?

Good questions, @AntonIatcenko. I am afraid some of them may not have good answers.

There is no lobpcg function that I am aware of for this. You can print and monitor the convergence with HypreAME::SetPrintLevel and potentially check the norm of $\| A x - \lambda M x \|$ for eigenpairs before safint them.

The call to LOBPCG in hypre's src/parcsr_ls/ame.c is below. I don't really see much more parameters that will be useful to provide. Note that the lobpcg block size is set by HypreAME::SetNumModes.

   blap_fn.dsygv  = hypre_dsygv;
   blap_fn.dpotrf = hypre_dpotrf;
   lobpcg_tol.relative = ame_data -> rtol;
   lobpcg_tol.absolute = ame_data -> atol;
   residuals = hypre_TAlloc(HYPRE_Real,  ame_data -> block_size, HYPRE_MEMORY_HOST);

   lobpcg_solve((mv_MultiVectorPtr) ame_data -> eigenvectors,
                esolver, hypre_AMEMultiOperatorA,
                esolver, hypre_AMEMultiOperatorM,
                esolver, hypre_AMEMultiOperatorB,
                NULL, blap_fn, lobpcg_tol, ame_data -> maxit,
                ame_data -> print_level, &nit,
                ame_data -> eigenvalues,
                NULL, ame_data -> block_size,
                residuals,
                NULL, ame_data -> block_size);

I am not aware of a simple way to do that, sorry.

You can in principle provide an IdentityOperator to HypreAME::SetPreconditioner but I don't think that is a good idea. First, AME has really been designed to work with AMS and may not even converge. Second, even if it does converge, it will take many iterations for large problems.

Two additional comments:

The error you listed means that the initial LOBPCG vectors were not random enough (i.e. they were linearly dependent). This randomization should be easier if the vector size is larger relative to the number of eigenvectors. In hypre, it is done by the call to mv_MultiVectorSetRandom (eigenvectors, seed); in hypre_AMESetup, there are similar calls in MFEM's HypreLOBPCG. Maybe a better seed or random number generator can help?

I was wrong yesterday -- @mlstowell wrote the HypreAME wrapper in MFEM and may have additional comments and suggestions for you.

Thank you for your response @tzanio!

This is very strange... I guess I'll try to "intercept" what the function prints to the output and process it. Will use your suggestion for checking the norm if that does not work, thanks!
Here is what it says in the documentation for hypre's LOBPCG: "The LOBCPG also allows one to impose constraints on the eigenvectors of the form $x^T B y_i = 0$ for a set of vectors $y_i$ given to LOBPCG as input parameters. This makes it possible to compute, e.g., 50 eigenpairs by 5 subsequent calls to LOBPCG with the blockSize=10, using deflation." It sounds like the block size can be the same as the number of eigenvalues, but it doesn't have to be. Currently, the deflation happens automatically as the iterations progress, but in the cases when the solver fails it happens before any deflation. If the MFEM could take the maximum block size $mb$ (an array of block sizes?) and then run the LOBPCG $\lceil nev/mb \rceil$ times, making use of the constraints feature (I suspect one of the NULLs you are passing to lobpcg_solve is for ${y_i}$), then the user would have better control over deflation, which might fix my problem. Your additional comment also makes it seem like a good idea, as we would be computing fewer eigenvectors at a time and thus being sufficiently "random" would be easier.

Do I understand correctly that MFEM's HypreAME is a wrapper around ame.c, which is itself a wrapper around
lobpcg written by Andrew Knyazev? From looking at ame.c, it seems like the changes I require would have to be made there rather than in MFEM code.
Seems like quite a rabbit hole...

Do I understand correctly that MFEM's HypreAME is a wrapper around ame.c, which is itself a wrapper around
lobpcg written by Andrew Knyazev?

This is correct, but AME is a special eigensolver. As discussed in the research note for it, in addition to providing a preconditioner for the curl-curl operator (for AME this is AMS), one also needs to enforce a divergence-free constraint for the LOBPCG subspace in each iteration.
That being said, the AME code is 16 years old at this point and maybe could be updated for new LOBPCG features (I don't recall the constraint option being available in hypre before).


Anca/chain init tx

Closes: #371 #372 #373 #374
Description

CLIs for channel open handshake messages
building and relaying of channel open handshake messages

[will revert the Cargo.toml changes for tendermint dependencies after more testing]

For contributor use:

[ ] Updated the Unreleased section of CHANGELOG.md with the issue.
[ ] If applicable: Unit tests written, added test to CI.
[ ] Linked to Github issue with discussion and accepted design OR link to spec that describes this work.
[ ] Updated relevant documentation (docs/) and code comments.
[ ] Re-reviewed Files changed in the Github PR explorer.

Codecov Report

Merging #403 (2496f7a) into master (b1b37f5) will increase coverage by 19.7%.
The diff coverage is 65.1%.

@@            Coverage Diff            @@
##           master    #403      +/-   ##
=========================================
+ Coverage    13.6%   33.4%   +19.7%
=========================================
  Files          69     154      +85
  Lines        3752   10297    +6545
  Branches     1374    3782    +2408
=========================================
+ Hits          513    3443    +2930
- Misses       2618    6612    +3994
+ Partials      621     242     -379

Impacted Files
Coverage Δ

modules/src/events.rs
0.0% <0.0%> (ø)

modules/src/ics02_client/events.rs
0.0% <ø> (ø)

modules/src/ics02_client/msgs.rs
0.0% <0.0%> (ø)

modules/src/ics02_client/raw.rs
0.0% <0.0%> (ø)

modules/src/ics03_connection/error.rs
13.6% <0.0%> (-19.7%)
:arrow_down:

modules/src/ics04_channel/error.rs
75.0% <ø> (+50.0%)
:arrow_up:

modules/src/ics04_channel/msgs/acknowledgement.rs
0.0% <0.0%> (ø)

modules/src/ics04_channel/msgs/recv_packet.rs
0.0% <0.0%> (ø)

modules/src/ics04_channel/msgs/timeout.rs
0.0% <0.0%> (ø)

modules/src/ics04_channel/packet.rs
0.0% <0.0%> (ø)

... and 287 more

Continue to review full report at Codecov.

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


2<IP_ADDRESS> stabilization merge into master

See release notes.
https://github.com/Microsoft/MixedRealityToolkit-Unity/releases/tag/untagged-5c4c9fd0226decf3707a

I didn't get a chance to review.
We had one Compiler warning we could have got rid of.