9.5 doesn't seem to allow querying into noscript

I'm trying to use jsdom to munge with an html file to prepare it for a CDN. However, I'm not able to select into an element inside of a noscript. I have ProcessExternalResources set to false.

var document = jsdom('path', {features: {ProcessExternalResources: false}});
var els = document.querySelectorAll('[href]');
els.length === 0;

From my testing in FF this does indeed seem like a bug.

For future reference, this looks to be due to the same reason as issue #1802 and can be resolved when https://github.com/inikulin/parse5/issues/192 has been merged.


Please add go back and forward action for tabs.

Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]
Describe the solution you'd like
A clear and concise description of what you want to happen.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.

It's not built that way . Please add xpath of next input field to perform action on next field


percentage_match rule is sending false alert rules

I would like to trigger an alert when Nginx logs have more than 3% of HTTP 500. However, I receive the notification even when, according to my filter, the threshold is not reached. Here is my rule:
name: Too many HTTP 500
type: percentage_match

index: journalbeat-*
description: "No more than 3% of all http requests should respond HTTP 500"

filter:
- query:
    query_string:
      query: "not http.request_url: '/v1/test' AND container.name: *nginx"

buffer_time:
  minutes: 3

realert:
  minutes: 0

doc_type: _doc

match_bucket_filter:
- query_string:
    query: "http.status_code: 500"

max_percentage: 3

alert:
  - "googlechat"

alert_text: |
  There were more than 3% of HTTP 500 responses in this period ({0}% of all {1} HTTP requests).
alert_text_args:
  - percentage
  - denominator
alert_text_type: alert_text_only

I receive this false alert at each of the 3-minute interval, for instance:
There were more than 3% of HTTP 500 responses in this period (5.555555555555555% of all 108 HTTP requests).
However, when I run the query in Kibana, I don't have any results, even during the last 30 minutes.

We also recommend asking questions on the Gitter channel
https://gitter.im/Yelp/elastalert

Yelp/elastalert is no longer maintained. Please use jertel/elastalert. Questions to the discussion below
https://github.com/jertel/elastalert/discussions


dynamic import() chunk name

Using import() with a module seems to just give it an id as a name. It would be nice to either be able to name the chunk, or to use a name derived from the name of the import.

The ES6 spec doesn't support this and webpacks System.import implemenation should be comply to the spec.
Why do you need a chunk name?

The chunk name can be really useful for debugging to figure out the filename that an error is being thrown from, for example.
I understand that this would be non-spec based on the current whatwg loader spec.

@sokra @kesne I agree webpack should not deviate from the loader spec. Given we only need the chunk name at build time, what about using configuration comments to achieve the same thing?
System.import('./foo') // webpack chunk-name bar

Obviously, you can't dynamically name chunks but AFAIK that wasn't supported in Webpack 1 either.

I also worry about assets caching. Given a name has a pattern of [chunkhash].[name].js, we get something like f4621a974f1f255ce2bf.0.js, which has greater chances of name collision with another chunk in the future than, say, f4621a974f1f255ce2bf.foo.js.
Do my words have sense?

...plus we won't optimize them by CommonChunkPlugin with unknown chunk names. or I somthing missed?

+1

@bardt what collision do you expect based on hash value? The reason I would like to have meaningful names for chunks (not just numbers) is more easier debug/inspection of build result while dev and event production.
@sokra
Just asked this question on SO http://stackoverflow.com/questions/39564726/webpack2-meaningful-names-of-chunks

Another benefit to chunk naming is the ability to save a trip when requesting initial assets for a SPA.
https://github.com/webpack/docs/wiki/optimization#single-page-app

+1
I just took a shot at migrating from webpack 1.x to webpack 2.x and losing the ability to name the chunks means it's a lot harder to debug/inspect what's going on with the code splitting

@troutowicz I was able to do it using a solution combining webpack stats, __filename and line of code.

@cvle you could show it

@whitecolor @troutowicz I refactored my code and put the relevant code into a separate repository and published to npm.
Here is what I use to include initial chunks to the server response:
https://github.com/wikiwi/webpack-initial-chunks.

Here it is: https://github.com/wikiwi/webpack-content-chunks

@cvle Looks like an interesting approach! But I am no longer lazy loading chunks due to perf issues.

It would be really great to have a proper way to name the chunks, it would help a lot debugging things and also knowing what needs to be improved right from the webpack output.
Right now.. We just see that the chunk 0, 1, 2, 3 have  a filesize of X :/ No idea why I need to change to improve the file size for example (without further investigation on what are the chunks obviously).

Could we support something like this?
import('/Home?chunk=home');

The problem is that if you're using output.path, the ID is added not before chunk's name, but before the path, which leads to creating very weird directories.
For example with config like this:
output:  {
    path: './../../development/app/dist/js/', // this is added after ID
    chunkFilename: '[name].[id].js', // this is ignored
    publicPath: './'
}

something really strange appears in webpack's directory:
[0..] // yes, this is a directory name
--[development]
----[app]
------[dist]
--------[js]
-----------analytics.js
[1..]
--[development]
----[app]
------[dist]
--------[js]
	-----------analytics.js
[2..]
--[development]
----[app]
------[dist]
--------[js]
	-----------analytics.js
[node_modules]
webpack.config.js

I wonder if that's how you're supposed to use https://github.com/webpack/bundle-loader

@sokra Given now that the System.import function was renamed to import because it wasn't complying with the spec anyway, could we reconsider adding an optional parameter to support naming chunks?
My personal usecase is using code splitting in combination with service-worker runtime caching.
Using sw-toolbox you can specify cahcing rules based on a regex of the name of the asset. I'd like to cache some chunks, and not cache others depending on how "rare" they are, from user's usage perspective.
For that I need to be able to name chunks.

Related/duplicates: #2369, #2682, #3132.

Ionic Framework team would greatly like named chunks. It would make our lives easier for implementing a complicated build process and it would surely make debugging easier for our users utilizing code splitting.
Thanks,
Dan

How about adding a new [module] variable in chunkFilename config?

My current understanding is that this could be implemented through a comment. Example:
import('path/to/module' /* demoName */).then(doStuff)

This way would not interfere with the specification in any way in case they add new parameters etc. and would allow to keep this information specific to webpack.

Also in great need of this feature. I've pushed a PR #4573 to make it work with specific comment:
import('./foo' /* webpackChunkName = "myChunkName" */).then(function(foo){
    //...
});

@sokra @bebraw @simon04 Looking forward to your review and feedback. :)

glad there is a PR, but the comment theme seems weird compared to this comment. It seems like there should be a way to just tell webpack to always do this and use the filename (instead of meaningless numbers)

@quinn Through the configuration may not flexible. Assume there are two entries that need import moduleA both, the one entry want to put it into chunk-module-a, but another want to put it into chunk-tiny-modules, then we have to make two versions of configuration. There are much difference with require.ensure's useage and implementation.

@mc-zone thanks, I figured it was due to my limited understanding of webpack

I don't quite understand why the name can't be computed from the full path of the file.  So
import(./foo)
Might create, say,
contacts-details-util-foo.js
Could be a useful default to have, so we wouldn't need to add a webpack-specific comment on every import to have meaningful chunk names.

@arackaf What if you refactor and rename a folder or move things around, you'd need to update refs to your chunk names?
I'm also not a big fan of the comments though, which seems to have made it into webpack. I'm still in favour of the query parameter option ./Settings?chunk=settings. Seems at home with webpack's import style.

@mikestead I don't think so - async chunks that webpack makes usually have dynamic, names that are created on the fly; you're not normally able to reference those directly because you have no idea what webpack will call them.
My comments were just suggesting that webpack instead give them more friendly names by default, derived from the file path, just so we can debug a bit more easily, ie have a better idea where the a problem is if its in contacts-utilities.js as opposed to 16-chunk.js.
But alas it seems I'm in a minority here :(

I agree with @arackaf that more human-readable default names would be nice. However, I'd still like to be able to give chunks a name that I specify, just like require.ensure did

Yeah, I agree that having better names (and module ids) by default would be a step forward.

@Rhionin ftr, I agree.  I'm glad the team added the comment naming to import().  I'd just like to see it coupled with some sane default naming scheme to get the best of both worlds.

Using require.ensure I could pass an array of modules to bundle in a separate chunk.
Is it possible to use several import() calls with the same webpackChunkName to bundle them in the same chunk or I'll have to use a proxy module?

Is it possible to use several import() calls with the same webpackChunkName to bundle them in the same chunk?

yes

@romulof @sokra

Is it possible to use several import() calls with the same webpackChunkName to bundle them in the same chunk?

yes

Can anyone provide a working example of this? I cannot get a common chunk created with multiple files using the same webpackChunkName value.
Promise.all([
    import('path/1/file1' /* webpackChunkName:"subload" */), //appears in subload.js
    import('path/2/file2')                     //appears in [id].js
])

Promise.all([
    import('path/1/file1' /* webpackChunkName:"subload" */), //appears in [id].js
    import('path/2/file2' /* webpackChunkName:"subload" */) //appears in subload.js
])

Is it a problem to have the source files at different paths?

@chrisdeely this is working for me:
async function loadIntl() {
  await import(/* webpackChunkName: "polyfill-intl" */ 'intl');
  await import(/* webpackChunkName: "polyfill-intl" */ 'intl/locale-data/jsonp/en.js');
  warning(false, 'Intl polyfill enabled');
}

Thanks @romulof working now, but not sure what I did wrong before

For the record - the solution as described here and mentioned above doesn't work when comments are stripped along the way - e.g. setting comments to false in .babelrc

Could it be possible to have a dynamic chunk name for a dynamic import? E.g.
(relativeModulePath, chunkName) =>
import(
  /* webpackChunkName: `chunk-${chunkName}` */ `./${relativeModulePath}`
)

Hardcoded chunk names limit dynamic chunking abilities.

Agree with @simshanith, that would be very useful when having large amount of dynamic imports. It would be even cooler to be able to define that in webpack config, like:
output: {
    filename:      'js/[name].js',
    chunkFilename: 'js/[filename].js' // so that import('module') creates module.js
    chunkMode:     'lazy-once' // so I can override default `lazy` option once and for all
}

Use NamedChunksPlugin can custom your dynamic import chunk name
...
// used by code below;
paths.assets = 'path/to/project/resource/assets';

new webpack.NamedChunksPlugin(function(chunk) {
    // if it have name, and return name
    if (chunk.name) {
        return chunk.name;
    }

    const regex = new RegExp(`${paths.assets}/pages`);

    // every chunk have _modules, and iterate it.
    for (var m of chunk._modules) {
        // module context is the module path of webpack resolve for you.
        // such as require('react')
        // context will be /path/to/project/node_modules/react
        // because I want to locate the dynamic import url rather than other module.
        // and I know the dynamic import path have the prefix like /path/to/project/`resource/assets/pages`
        // so I can locate it use regex.
        if (regex.test(m.context)) {
            // return whatever name you defined
            return path
                .relative(paths.assets, m.context)
                .split('/')
                .slice(1)
                .join('_');
        }
    }

    return null;
})

@monkindey Thank you. One thing for future people using that to note is that path.assets should be the path to your source and you would need to bring in both webpack and path modules for this to work.

yes, you are right. I omit it. Thanks.

As a follow up to my dynamic chunk name request, looks like there have been dynamic placeholders since 2.6:

webpackChunkName: A name for the new chunk. Since webpack 2.6.0, the placeholders [index] and [request] are supported within the given string to an incremented number or the actual resolved filename respectively.

https://webpack.js.org/api/module-methods/#import-
I have yet to try it, and don't know if it can be composed with other strings, but guessing it would look something like this:
(relativeModulePath) =>
import(
  /* webpackChunkName: chunk-[request][index] */ `./${relativeModulePath}`
)

This was inspired by monkindey
    const regex = new RegExp(`${sourcePath}`);
    new webpack.NamedChunksPlugin(function(chunk) {
        if (chunk.name) return chunk.name;
        for (var m of chunk._modules) {
            if (regex.test(m.context)) {
                return path.basename(m.rawRequest);
            }
        }
    })

Why not works fine in production?

It somehow runs into errors with chunk names and styles. The app will still run but you end up with split code with weird names.

Maybe some different environment between development and production ?

Yes, due to different environments and, chunk names are in different paths so my logic above doesn't work. IMO, it's better to fall back to standard IDs because named chunks really only matter for debugging in development.

For React with typescript user, adding 'removeComments': false to ts-loader options is working rather than adding 'removeComments': false to tsconfig.json.

Hi, I wrote a babel plugin to name dynamic chunk:
importName('./a.js', 'name-a')

and it will be transformed to
import( /*webpackChunkName: 'name-a'*/'./a.js');

https://www.npmjs.com/package/babel-plugin-webpack-async-module-name

you can try https://gist.github.com/hjzheng/969c47661f4636d15a0234b3ce1edc72

I think this caters to more use-cases: https://github.com/bloody-ux/babel-plugin-path-chunk-name


Add microbadger badges.

See: https://microbadger.com/#/

Fixed by
microservices-demo/front-end#48
microservices-demo/queue-master#12


[ASCII] Update Go version to 1.21.11

Buildimages update
This PR updates the current buildimages (v34815905-ae40295e ) to v35901572-a9447181, here is the full changelog between:
https://github.com/DataDog/datadog-agent-buildimages/compare/ae40295e...a9447181
Golang update
This PR updates the current Golang version (1.21.10) to 1.21.11.

/merge


Game doesn't appear.

I did the getting started documentation, but the app doesn't appear.
Does the Switch need to be modded?

Here is my file structure:
microSD:
Nintendo:
[nintendo switch data stuff]
switch:
nxjs.nro
nxjs.js

Yes, you Switch needs to be modded to run homebrew applications. Once that is set up, then the game would appear in the Album menu homebrew launcher.

Nooooooooooooooo

😅 Ya, sorry, there's no way around that. I've updated the instructions in the README to clarify, so that this issue can be closed.


"delete_on_terminate" param is not present "rackspace computeV2 create_server.rb"

File name:"fog/rackspace/requests/compute_v2/create_server.rb"
"delete_on_terminate":true  parameter should be added  under data['server']['block_device_mapping_v2'] .

On creating nextgen rackspace server,storage volume will also be created for this .
But on deleting this server,storage volume will not be deleted if this parameter is missing,

Original issue opened by @cppatil at fog/fog#3814

@cppatil where is this information located? Closing due to age of original issue; please re-open if necessary.


additional parent for "meiotic centromeric cohesion protection"

meiotic centromeric cohesion protection
has a parent to
GO:0045144    JSON
meiotic sister chromatid segregation
but centromeric cohesion protection (i.e maintenance of cohesion at the centromere during meiosis I) is  also  part of
homologous chromosome segregation (keeps sister chromatids together during equational division at meiosis I)

In fact to make this absolutely clear
add exact synonym : protection of centromeric cohesion during meiotic anaphase I.


[docs] Information about Contributing

Supersedes #998.
@lirantal I made your suggested changes from the previous PR.

LGTM


LimitOrderHook improvements

Infra improvements

way more tests

Contract improvements

consider simplifying the epoch/key data structures (https://github.com/Uniswap/core-next/pull/93#discussion_r888166349)
consider renaming epochs -> orders (https://github.com/Uniswap/core-next/pull/93#discussion_r889726019)
consider a bitmap to track initialized ticks (https://github.com/Uniswap/core-next/pull/93#discussion_r889730737)
consider a single call to fill per swap (https://github.com/Uniswap/core-next/pull/93#discussion_r889739044)

UX-level changes

consider taking amountIn instead of liquidity (https://github.com/Uniswap/core-next/pull/93#discussion_r888143083)
consider issuing erc721/erc1155/erc20 representations of limit orders

closing this issue for now, since example hooks do not live in the main branch anymore.
i've added a revisit tag as a reminder


Cropping produces no output on some videos

Hi,
I might be doing this wrong. I'm asking the program to crop a 5 seconds video I recorded on a phone.
python crop-video.py --inp sup-mat\IMG_1130.mp4
I have a batch of videos of me talking in froont of the camera (short 5-10 seconds videos, 1920x1080). But some of those just don't produce anything when i try to crop them. No error message, no ffmpeg command. The script just stops.
it keeps happening to hose failed videos no matter how many times I try

It is hard to say without a videos. Crop-video.py is only give a suggestion, so there are a lot of heuristics there. The face should stay pretty close to the initial frame for 150 frames, closeness is measured using iou between initial face bbox and cureent face bbox. So you could either decrease min_frames or decrease iou_with_initial or record videos where head moves near initial position.


Bug OCPBUGS-1123: UPSTREAM: : server/etcdmain: add configurable cipher list to gRPC proxy listener

Carrying upstream fix for https://issues.redhat.com/browse/OCPBUGS-1123

/lgtm

/jira refresh

/jira refresh

/jira refresh

/retest-required

/jira refresh

/jira refresh

We have no 4.14 yet so going to override the invalid jira label for this one.

@hasbro17 not sure tide actually caught up on the labels, but https://prow.ci.openshift.org/pr?query=is%3Apr+repo%3Aopenshift%2Fetcd+author%3Adusk125+head%3Aopenshift-4.13
reports the bugzilla/valid-bug label to be missing. Even though it reports to be in the merge queue here...


Allow characters to return an empty string.

If the next element is an end tag, return an empty String from characters. Otherwise continue with the previous behavior of attempting to read in characters and return a String.
I believe this resolves #349.

Wrote a test:
let mut snapshots = Vec::new();
    snapshots.push("snap-0c9ec95b".to_owned());
    let mut mah_req = DescribeSnapshotsRequest::default();
    mah_req.snapshot_ids = Some(snapshots);

    match ec2.describe_snapshots(&mah_req) {
        Ok(desc_results) => {
            println!("describe_snapshots {:?}", desc_results);
        },
        Err(error) => {
// leftover, codepath not taken, meh
            assert!(error.description().contains("<Message>The instance IDs 'i-00000000, i-00000001' do not exist</Message>"), "Missing error message");
        }
    }

Saw it work: cargo test --features ec2 -vv -- --nocapture:
describe_snapshots DescribeSnapshotsResult { next_token: None, snapshots: Some([Snapshot { data_encryption_key_id: None, description: Some(""), encrypted: Some(false), kms_key_id: None, owner_alias: None, owner_id: Some("xxxx"), progress: Some("100%"), snapshot_id: Some("snap-0c9ec95b"), start_time: Some("2016-09-23T01:55:56.000Z"), state: Some("completed"), state_message: None, tags: Some([Tag { key: Some("Name"), value: Some("testsnapshot") }]), volume_id: Some("vol-40c762c8"), volume_size: Some(8) }]) }

Ideally we'd have the deserializer return None instead of Some("") but the service definition says it's a string.  We assume it's got something in there since our codegen wraps the describe deserializer results with Some(return value) anyways.
👍


Monotonic array added

Closes #996
🛠 Issue (996)
👨💻 Changes proposed
Added a program Monotonic Array in C, C++, Python, Java
✔ Check List

[x] My code follows the code style of this project.
[x] This PR does not contain plagiarized content.
[x] The title of my pull request is a short description of the requested changes.

📄 Note to reviewers
Simple Program to check Monotonicity.
📷 Screenshots

@Kumar-laxmi @zugzwang03 I have made the required changes . kindly review them.


Error 502.5 - ANCM Out-Of-Process Startup

Describe the bug
To Reproduce
Steps to reproduce the behavior:

Using this version of ASP.NET Core 2.2.*
Upload code to azure
x64

HTTP Error 502.5 - ANCM Out-Of-Process Startup Failure
in log
Unhandled Exception: System.IO.FileNotFoundException: Error reading the D:\home\site\wwwroot\ directory.
at System.IO.FileSystemWatcher.StartRaisingEvents()
at Microsoft.Extensions.FileProviders.Physical.PhysicalFilesWatcher.TryEnableFileSystemWatcher()
at Microsoft.Extensions.FileProviders.Physical.PhysicalFilesWatcher.CreateFileChangeToken(String filter)
at Microsoft.Extensions.Primitives.ChangeToken.OnChange(Func`1 changeTokenProducer, Action changeTokenConsumer)
at Microsoft.Extensions.Configuration.FileConfigurationProvider..ctor(FileConfigurationSource source)
at Microsoft.Extensions.Configuration.Json.JsonConfigurationSource.Build(IConfigurationBuilder builder)
at Microsoft.Extensions.Configuration.ConfigurationBuilder.Build()
at Microsoft.AspNetCore.Hosting.WebHostBuilder.BuildCommonServices(AggregateException& hostingStartupErrors)
at Microsoft.AspNetCore.Hosting.WebHostBuilder.Build()
at Innoline.Program.Main(String[] args) in \Mac\Home\Desktop\new\Innoline.Web\Program.cs:line 11

`using Innoline.Config;
using Microsoft.AspNetCore;
using Microsoft.AspNetCore.Hosting;
namespace Innoline
{
public class Program
{
public static void Main(string[] args)
{
var webHost = BuildWebHost(args);
webHost.Run();
}
    public static IWebHost BuildWebHost(string[] args)
    {
        return WebHost.CreateDefaultBuilder(args)
            .UseKestrel()
            .UseStartup<Startup>()
            .UseIISIntegration()
            .Build();
    }
}

}
`

Does this happen consistently? I don't think this is a permission issue, but it could be.

   <TargetFramework>netcoreapp2.2</TargetFramework>     <AspNetCoreModuleName>AspNetCoreModule</AspNetCoreModuleName>     <AspNetCoreHostingModel>OutOfProcess</AspNetCoreHostingModel>
worked in azure
   <TargetFramework>netcoreapp2.2</TargetFramework>     <AspNetCoreHostingModel>OutOfProcess</AspNetCoreHostingModel>
not worked in azure

@KANekT are you still seeing this issue?

@anurse I will check this as soon as possible

  <handlers>
    <add name="aspNetCore" path="*" verb="*" modules="AspNetCoreModuleV2" resourceType="Unspecified" />
  </handlers>
  <aspNetCore processPath=".\Innoline.exe" stdoutLogEnabled="true" stdoutLogFile=".\logs\stdout" hostingModel="InProcess" />

HTTP Error 502.5 - ANCM Out-Of-Process Startup Failure
Unhandled Exception: System.IO.FileNotFoundException: Error reading the D:\home\site\wwwroot\ directory.
at System.IO.FileSystemWatcher.StartRaisingEvents()
at Microsoft.Extensions.FileProviders.Physical.PhysicalFilesWatcher.TryEnableFileSystemWatcher()
at Microsoft.Extensions.FileProviders.Physical.PhysicalFilesWatcher.CreateFileChangeToken(String filter)
at Microsoft.Extensions.Primitives.ChangeToken.OnChange(Func`1 changeTokenProducer, Action changeTokenConsumer)
at Microsoft.Extensions.Configuration.FileConfigurationProvider..ctor(FileConfigurationSource source)
at Microsoft.Extensions.Configuration.Json.JsonConfigurationSource.Build(IConfigurationBuilder builder)
at Microsoft.Extensions.Configuration.ConfigurationBuilder.Build()
at Microsoft.AspNetCore.Hosting.WebHostBuilder.BuildCommonServices(AggregateException& hostingStartupErrors)
at Microsoft.AspNetCore.Hosting.WebHostBuilder.Build()
at Innoline.Program.Main(String[] args) in D:\OhMyBoss\server-main\Innoline.Web\Program.cs:line 12

D:\OhMyBoss\server-main\Innoline.Web\ is my local path - why is use in azure ?

That path is just the source file path that is compiled in when you built the DLL. It's used by debuggers to help resolve the source file and for your own reference to know where the line of code is.

Unhandled Exception: System.IO.FileNotFoundException: Error reading the D:\home\site\wwwroot\ directory.

Seems like something is locking or otherwise restricting your wwwroot directory. Could you create a sample project that reproduces this problem? It seems like this is more related to file watching issues on Azure App Service and the ANCM error is just how it is manifesting.
@pranavkm @natemcmaster I see your names in the history for the PhysicalFilesWatcher. Is this something we've seen before? Just curious if there's something known here.

Nope, haven't seen this before. Would be good to confirm first that D:\home\site\wwwroot\ exists and is readable before we assume its a problem with System.IO.FileSystemWatcher

Cool, just thought I'd check that first.
@KANekT Does this directory exist on the App Service server? You can check this by navigating to the "Advanced Tools" tab in the Azure Portal for your App Service. In there you can open the Debug Console and explore the file system.

@anurse directory is exits in root:

i find https://github.com/dotnet/core/blob/master/release-notes/2.2/2.2-known-issues.md
but it Note this has been resolved as of January 2019.

I wonder if something is broken on your Azure App Service instance. Does this repro if you try temporarily deploying to a new App Service? If it does, can you provide a sample project that reproduces the issue?

Closing this as we haven't heard from you and generally close issues with no response after ~7 days. Please feel free to comment if you're able to get the information we're looking for and we can reopen the issue to investigate further!


[GSSoC'23] Responsive Candies Website

Describe the project you want to add with tech stack
Title - Candies Website
Tech using - HTML, CSS and JS
Expected behavior

A responsive website of selling candies, chocalates, etc.
Adding cards effect on reviews
Adding feedback form
Creating a FAQ section and attractive and fixed navbar
Attractive user interface with best user experience.

Screenshots (optional)

Additional context (optional )

@MohitGupta121 @Saurabh-kumar09 @pranjay-poddar sir, please review this pull request


Removing/Replacing edits

How do you replace or remove the edit you made?
There seems to be no command like /messageeditor delete (edit code) and then it would go back to its' original message.
When I try to replace the edit with the previous message or a newer edit, it doesn't let me because "There already is a edit that uses a file with the name of x".
Why not just let the user rewrite the edit or is this a bug?
I also don't understand what "override mode" and "replace mode" are supposed to be. "override mode" for me just replaces the message with the edit and "replace mode" just does nothing.
Thanks

There seems to be no command like /messageeditor delete (edit code) and then it would go back to its' original message.

There is no such command at the moment and that is a mistake on my part, it will be added in a future update. Currently, to delete a message edit you can either prefix its file name with '#' or delete the file completely and then reload the plugin by using /message-editor reload command.

Why not just let the user rewrite the edit or is this a bug?

A message can only be edited once. If you want to re-edit it, you have to completely delete its file, reload the plugin and then edit it one more time. I admit that the process can be improved and will be in a future update.

I also don't understand what "override mode" and "replace mode" are supposed to be. "override mode" for me just replaces the message with the edit and "replace mode" just does nothing.

The main difference is that override mode replaces the entire message with the new one and replace mode only changes parts of it. I myself use override mode most of the times and I believe that replace mode was actually originally created just to correct some mistakes in the messages.
If it was not for the sake of trying to retain as much old functionalities as possible, I probably would have completely removed replace mode by now.
Before I close this issue, do let me know if there is further need to explain something.

Thank you for the answer and I hope to see the process being improved in the future updates.
I already knew that was one of the ways to re-edit messages, just figured if there was a faster way to do it.
I am loving the plugin itself and It does pretty much everything I needed it to.


Add env value to etcd

Allow to set custom env via the etcd chart values which allows to pass
custom etcd options
What this PR does / why we need it:
This adds env to the values that will be appended to the container env values. This allows us to configure etcd better (for example auto compactation) in the values.

/assign @prydonius

/ok-to-test
/assign

/lgtm


Incorrect height and scrolling when in landscape

When the device rotates into landscape mode, the picker dialog gets too high and draws a bit off screen towards the top.
Also, because the entire height can't be displayed, scrolling kicks in. However, the color grid area absorbs all scrolling motions, so the user can't scroll up or down to the close button or the hex color entry button. Possible solution would be: when a user, for example, is scrolling up on the color grid selector, when they reach the top row, then it starts scrolling the entire frame up to the title & close button, and the same for the bottom. I've also noticed that the scroll position doesn't always default to the top in this case, which is a bit confusing.
Thanks, and let me know if you need more detail or a video of this happening.

Sorry for the delay, but it's now fixed.


VPC peering connections (#158)

Adds the ability to configure VPC peering connections in a route table. Fixes issue #158

Great job. Thanks for the fix Dave.


MySql types not support

MySql types such as longtext or BigInt are unkown.
For MySql the SqlPojoMemoDB is currently useless.

Following MySql Types for table column are supported;
INTEGER, INT, SMALLINT, TINYINT, MEDIUMINT, BIGINT, DECIMAL, NUMERIC, FLOAT, DOUBLE
BIT, VARCHAR, VARCHAR2, TEXT, TINYTEXT, MEDIUMTEXT, LONGTEXT


The column order in the sql queries is significant

As an example, I have the following simple class:
case class Student(firstName: String, lastName: String)derives sq.Reader

This is the function that reads the students:
    def readStudents(cn: Connection, id: Option[Int]): List[Student] = {
        sq.run(cn) {
            var sql = if (id.isEmpty)
                sql"""select last_name, first_name
              from student"""
            else
                sql"select last_name, first_name  from student where student_id =${id.get}"

            sq.read[Student](sql)
        }
    }

Note that in both cases, the last_name field appears on the first position in the select statements. When the data is read, the last_name field populates the firstName member and the first_name populates the firstName member. This makes the code brittle ,imo.
I think it would be better to use a mapping between member names and column names, whether by attributes or convention (case class member names should match exactly the names of the columns, or add a configuration parameter that drives a name transformation).
I really like your library - it is the simplest one I could find among all the scala libraries, the next step would be just to write jdbc code by hand.
Note: the run(jsql.Connection) method doesn't exist in the code, I added it to be able to use the library with the Play framework.

I reworked this library a bit, mostly focusing on making the syntax lighter. However, I also added support for using columns by name, so this should be fixed in > 0.3.0


how to control puma5

Today I upgraded to Puma5. When I entered mina puma:start, it appeared:
-----> Starting Puma...
       bundler: failed to load command: puma (/home/ruby/demo1/releases/4/vendor/bundle/ruby/2.7.0/bin/puma)
       OptionParser::AmbiguousOption: ambiguous option: -d
         /home/ruby/demo1/shared/vendor/bundle/ruby/2.7.0/gems/puma-5.1.1/lib/puma/cli.rb:50:in `initialize'
         /home/ruby/demo1/shared/vendor/bundle/ruby/2.7.0/gems/puma-5.1.1/bin/puma:8:in `new'
         /home/ruby/demo1/shared/vendor/bundle/ruby/2.7.0/gems/puma-5.1.1/bin/puma:8:in `<top (required)>'
         /home/ruby/demo1/releases/4/vendor/bundle/ruby/2.7.0/bin/puma:23:in `load'
         /home/ruby/demo1/releases/4/vendor/bundle/ruby/2.7.0/bin/puma:23:in `<top (required)>'

I have bundle update but it still doesn't work.
How can I solve this problem?

same problem

same problem

Put together a gem to manage systemd + puma5. Welcome to fork it.
https://github.com/pipihosting/mina-systemd-puma

@Winter-Silence Using that gem generates the same error for me -- I just installed the gem, anything else I need to configure?


fix issue 2903

兼容jdk8 LocalDateTime 可以解析时间戳字符串

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

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

Codecov Report

Merging #2906 into master will decrease coverage by 0.03%.
The diff coverage is 62.5%.

@@             Coverage Diff              @@
##             master    #2906      +/-   ##
============================================
- Coverage     80.47%   80.43%   -0.04%
+ Complexity     7195     7190       -5
============================================
  Files           145      145
  Lines         27049    27053       +4
  Branches       6468     6469       +1
============================================
- Hits          21768    21761       -7
- Misses         3411     3417       +6
- Partials       1870     1875       +5

Impacted Files
Coverage Δ
Complexity Δ

...ba/fastjson/parser/deserializer/Jdk8DateCodec.java
76.14% <62.5%> (-0.64%)
83 <0> (ø)

...com/alibaba/fastjson/serializer/SerialContext.java
72.91% <0%> (-4.17%)
19% <0%> (ø)

...com/alibaba/fastjson/parser/JSONReaderScanner.java
72.61% <0%> (-1.28%)
44% <0%> (-1%)

...ava/com/alibaba/fastjson/serializer/DateCodec.java
87.05% <0%> (-1.18%)
40% <0%> (-1%)

.../java/com/alibaba/fastjson/parser/JSONScanner.java
76.11% <0%> (-0.06%)
506% <0%> (-1%)

...ava/com/alibaba/fastjson/parser/JSONLexerBase.java
71.35% <0%> (-0.04%)
827% <0%> (-1%)

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 c2e8165...639e089. Read the comment docs.

 All committers have signed the CLA.

 All committers have signed the CLA.


Not installable on 11ty 3.0

npm i @photogabble/eleventy-plugin-interlinker

npm error code ERESOLVE
npm error ERESOLVE could not resolve
npm error
npm error While resolving<EMAIL_ADDRESS>npm error Found<EMAIL_ADDRESS>npm error node_modules/@11ty/eleventy
npm error   dev<EMAIL_ADDRESS>from the root project
npm error
npm error Could not resolve dependency:
npm error peerOptional<EMAIL_ADDRESS>from<EMAIL_ADDRESS>npm error node_modules/@photogabble/eleventy-plugin-interlinker
npm error   @photogabble/eleventy-plugin-interlinker@"*" from the root project
npm error
npm error Conflicting peer dependency<EMAIL_ADDRESS>npm error node_modules/@11ty/eleventy
npm error   peerOptional<EMAIL_ADDRESS>from<EMAIL_ADDRESS>npm error   node_modules/@photogabble/eleventy-plugin-interlinker
npm error     @photogabble/eleventy-plugin-interlinker@"*" from the root project
npm error
npm error Fix the upstream dependency conflict, or retry
npm error this command with --force or --legacy-peer-deps
npm error to accept an incorrect (and potentially broken) dependency resolution.
npm error
npm error
npm error For a full report see:
npm error /Users/mattwilcox/.npm/_logs/2024-05-23T19_14_53_962Z-eresolve-report.txt

Just adding this here as I'd love to see a 3.0 compatible version (guess who's learning 11ty with 3.0 because he can just use ESM and not have to learn all the old-school node rubbish?)

As of now, I can install the plugin, but when I try to start the server I get these errors:
[11ty] Eleventy Error (CLI):
[11ty] 1. Error in your Eleventy config file 'eleventy.config.js'. (via EleventyConfigError)
[11ty] 2. `require("@11ty/eleventy")` is incompatible with Eleventy v3 and this version of Node. You have a few options:
[11ty]    1. (Easiest) Change the `require` to use a dynamic import inside of an asynchronous CommonJS configuration
[11ty]       callback, for example:
[11ty]
[11ty]       module.exports = async function {
[11ty]         const {EleventyRenderPlugin, EleventyI18nPlugin, EleventyHtmlBasePlugin} = await import("@11ty/eleventy");
[11ty]       }
[11ty]
[11ty]    2. (Easier) Update the JavaScript syntax in your configuration file from CommonJS to ESM (change `require`
[11ty]       to use `import` and rename the file to have an `.mjs` file extension).
[11ty]
[11ty]    3. (More work) Change your project to use ESM-first by adding `"type": "module"` to your package.json. Any
[11ty]       `.js` will need to be ported to use ESM syntax (or renamed to `.cjs`.)
[11ty]
[11ty]    4. (Short term workaround) Use the --experimental-require-module flag to enable this behavior. Read
[11ty]       more: https://nodejs.org/api/modules.html#loading-ecmascript-modules-using-require It is possible that the
[11ty]       newest version of Node has this enabled by default—you can try upgrading your version of Node.js.

I'm guessing the plugin hasn't been fully updated for Eleventy V3 yet.

Oh I'm getting that error also... don't know what to do at that point.

Hey, yes this plugin has yet to be updated for V3. It's actually something I am working on this week as I have updated my website to V3 in order to test it with.

@ZeFish @MattWilcox thank you for reporting. I will be working on this today, with a view of getting an RC release containing a fix ASAP.
I believe it to be a case of using a tool like duel in order to build the plugin as a dual package meaning it can be installed by people running CommonJS or ESM imports. Then I can update the 11ty version dependency to include version three.

Hi all, I have just released version v1.1.0-rc5 containing a bugfix for this issue and have that version running on my own website with Eleventy v3.0.0.


Update Best practice image

Design tweaks

Tugboat has finished building the preview for this pull request!
Link:

https://pr129-ujblzmygtlvgmu1wb5aottz8plsfzwed.tugboat.qa

Dashboard:

https://dashboard.tugboat.qa/5fcfcb9d224cb781ff656ffa

Tugboat has finished building the preview for this pull request!
Link:

https://pr129-ujblzmygtlvgmu1wb5aottz8plsfzwed.tugboat.qa

Dashboard:

https://dashboard.tugboat.qa/5fcfcb9d224cb781ff656ffa

Closing as I pulled this into another branch and merged.

Closing as I pulled this into another branch and merged.


Theming - remove unused css

As part of theming, we've noticed a few usages of css classes + variables that we'd like to get rid of.

Bootstrap $low and $high variables. These are provided by _variables.scss, so we currently just set them to be explicit. See https://getbootstrap.com/docs/5.0/customize/sass/#remove-from-map

bootstrap_overrides.scss

set $low and $high to ''.
Remove $mid var and mid from $semantic-colors

bootstrap_imports.scss - set $theme-colors: map-remove($theme-colors, 'dark', 'light'); after mixin imports
Verify there aren't any consumers
ThemeExports - remove dark, light, and mid

.console-creator .custom-select selector in BaseStyleSheet.scss is defined in DHC. Console creator only exists in DHE. We should confirm whether the css is actually used and either remove it or move it to DHE

Note we'll need to update enterprise as well

@bmingles should this ticket be closed?

@mofojed I think this was mostly taken care of. I do still see .console-creator styles in BaseStyleSheet.scss, so technically this may be in the wrong spot. Not sure how much it matters.

Actually looks like most of the cleanup items still remain


Add documentation to README for how Process Service Checks can be configured.

Title

doc update
feature request

Check the systemd status code

Add documentation for process service checks for windows
Add new feature for *nix Oses that is similar to the process check for windows.
We should check the status of a systemd file and if it is active/running and if it exited unclean (not 0)

Re: https://github.com/Notifiarr/notifiarr/pull/58


Out of CUDA memory on A100 40GB when trying to run the LORA example

I run out of CUDA memory on A100 40GB when trying to run the LORA example on a Mistral-7B model:

  'python scripts/download.py --repo_id mistralai/Mistral-7B-Instruct-v0.1 \
  	--checkpoint_dir /pretrained/checkpoints',

  'python scripts/convert_hf_checkpoint.py --checkpoint_dir \
  	/pretrained/checkpoints/mistralai/Mistral-7B-Instruct-v0.1',

  'python scripts/prepare_alpaca.py --checkpoint_dir \
  	/pretrained/checkpoints/mistralai/Mistral-7B-Instruct-v0.1 \
  	--destination_path /pretrained/data/alpaca',

  'python finetune/lora.py \
		--checkpoint_dir /pretrained/checkpoints/mistralai/Mistral-7B-Instruct-v0.1/ \
		--data_dir /pretrained/data/alpaca \
		--out_dir /results/lit \
  		--precision bf16-true'

I guess this is not to be expected, what might be the reason?

Same here, any ideas?

I guess this is not to be expected,

Sorta expected, though, unfortunately, I don't have access to A100, so cannot validate that exact model.
From the resource table you can see that LlaMa2-7b with microbatch size of 4 (the default value) the model doesn't fit into a single A100 40GB. With Mistral-7B it should be the same. With microbatch size of 2 it fits without problem. So try to play with it.
https://github.com/Lightning-AI/lit-gpt/blob/6b3a32c51f53461ab9cb4a28fcfc09e4821ce654/finetune/lora.py#L40


NavigationView: Fix regression where navigating to the different pane items via the Tab key would crash the app

Description
This PR fixes a recently introduced WinUI 2.5 preview regression where putting a NavigationViewItem inside the NavigationView.PaneFooter area as the last item and then moving focus to the NavigationView's in-built Settings item via the Tab key causes the app to crash. A menu item has to be selected at the time of tabbing to the Settings item from the pane footer.
Motivation and Context
Fixes https://github.com/microsoft/microsoft-ui-xaml/issues/3445
How Has This Been Tested?
Tested manually and modified interaction tests. The modified test TabNavigationTest() now verifies that this tabbing scenario does not crash the app any longer and now works as intended. The test has also been modified to cover the different tab focus behaviors for the menu items list: If a selected menu item exists and focus is set to the menu items list, focus will be set to the selected menu item and not the first/last menu item of the list. These different focus behaviors weren't being fully tested before from what I could tell looking through the existing interaction tests.

@ranjeshj lets make sure this makes it in the 2.5 release

/azp run

Sumary of the different test failures:

PaneBehaviorTests.VerifySettingsWidthOnLeftNavMediumMode():

Fails on RS3, RS4
Runs fine for me locally (also no MUXControlsTestApp crash)

EventTests.MenuItemInvokedTest():

Fails on RS4
Runs fine for me locally and no app crash

FocusBehaviorTests.EnsureTopSettingsRetainsFocusAfterOrientationChanges():

Fails on 19H1
Runs fine for me locally and no app crash

I'm not sure why these tests fail and unfortunately none of the Log.Comment() calls or the different Verify() fail messages are listed in Azure Pipelines.

@StephenLPeters can you help debug these failures ?

@Felix-Dev RS2 and RS5 are the flavors that we run the tests with a debug build, where rs3, rs4 and 19h1 we use the release build. Can you try running the tests locally with a release build of the mux controls test app?

@StephenLPeters I ran all three failing tests on a local release build and every test passed. EventViewer also showed no errors for the test app.
Perhaps the failing tests don't like me having added another item to the pane footer in the test UI, though lookign at the test code, I'm not sure why that would be the case.

@Felix-Dev  My guess is that the NavigationViewItem that you added to the test page caused layout to change and maybe some of the MenuItems got pushed out into the overflow menu. Now tests that assumed to see those items in the UIA tree are not seeing it anymore. a simple fix might be to make the footer item smaller in size so that items don't get pushed out.

/azp run

Looks like we are down to one test failing now (consistently across all runs).
VerifyNavigationViewItemContentPresenterMargin
Error log:
[Error]: AreEqual(0,0,20,0, 0,0,0,0) [File C:\a\1\s\dev\NavigationView\NavigationView_InteractionTests\CommonTests.cs Line: 1709]

@ranjeshj Thanks for fixing the previous test failured.
First observation about the new failing test: Runs fine for me locally.
This is getting midly annoying now. While I run the test on 20H2 and the test pipeline only tests up to 19H1, I have the suspicion the test pipeline would fail in 20H2 as well. Yet locally, I keep seeing "test passed", even if I run it multiple times (and the test fails consistently in the pipeline).

@ranjeshj Thanks for fixing the previous test failures.
First observation about the new failing test: Runs fine for me locally.
This is getting midly annoying now. While I run the test on 20H2 and the test pipeline only tests up to 19H1, I have the suspicion the test pipeline would fail on 20H2 as well. Yet locally, I keep seeing "test passed", even if I run it multiple times (and the test fails consistently in the pipeline).

The first issue seemed to be related to the resolution/size of the window. @chingucoding added the button to switch the app to test machine resolution. I'm curious if we are switching to that resolution when running locally. I agree, anytime there is a difference between the lab run and local runs, it is not ideal.

The first issue seemed to be related to the resolution/size of the window. @chingucoding added the button to switch the app to test machine resolution. I'm curious if we are switching to that resolution when running locally. I agree, anytime there is a difference between the lab run and local runs, it is not ideal.

@StephenLPeters Updated the test app to run all tests in CI dimensions. I know there was one test that had problems with that, I am not sure if we fixed that test or if we made that test an exception. So in CI runs and local runs shouldn't behave differently if we look at sizing of test content.

Yeah, I am seeing the correct CI dimensions being used locally by default, so that shouldn't be the issue.

Yeah, I am seeing the correct CI dimensions being used locally by default, so that shouldn't be the issue.

This might be a timing issue with clicking on the flyout and then reading the child (which is in the flyout) properties. If the flyout did not open in time, the test just no-ops and we will end up reading the previous value for the margin. Trying a minor fix to delay by a little bit to see if that is the case. If it is a timing issue, then it explains why you are not seeing it on your dev box which is probably more performant than the lab machines.

/azp run

Yeah, I am seeing the correct CI dimensions being used locally by default, so that shouldn't be the issue.

This might be a timing issue with clicking on the flyout and then reading the child (which is in the flyout) properties. If the flyout did not open in time, the test just no-ops and we will end up reading the previous value for the margin. Trying a minor fix to delay by a little bit to see if that is the case. If it is a timing issue, then it explains why you are not seeing it on your dev box which is probably more performant than the lab machines.

That didnt work. @ojhad any ideas what might be happening with this test ?

@ranjeshj Thanks for trying to fix the test failures.
Lookign at the code, I have two more ideas. I suspect the test might fail at this check: https://github.com/microsoft/microsoft-ui-xaml/blob/e90f6f8fa7b8283e0114856ce69891d908fc3eb4/dev/NavigationView/NavigationView_InteractionTests/CommonTests.cs#L1718
Running the test locally, I noticed the following:

The child flyout we opened immediately before is still open (we do not close it). This might interfere with some of the following tets code (like switching the pane display mode from Compact to Expanded, so that the pane remains in Compact mode and thus we get the wrong ContentPresenter margin).

I don't know if that is cause for the test failures we are still seeing but I'm a bit shooting in the dark here anyway so might be worth running the pipeline with that code change as well (we now close the flyout before attempting to switch the pane display mode).

/azp run

@ranjeshj Thanks for trying to fix the test failures.
The test could also fail at this check:
https://github.com/microsoft/microsoft-ui-xaml/blob/e90f6f8fa7b8283e0114856ce69891d908fc3eb4/dev/NavigationView/NavigationView_InteractionTests/CommonTests.cs#L1717
Running the test locally, I noticed the following:

The child flyout we opened immediately before is still open (we do not close it). This might interfere with some of the following tets code (like switching the pane display mode from Compact to Expanded, so that the pane remains in Compact mode and thus we get the wrong ContentPresenter margin for top-level items).

I don't know if that is cause for the test failures we are still seeing but I'm a bit shooting in the dark here anyway so might be worth running the pipeline with that code change as well (we now close the flyout before attempting to switch the pane display mode).

Its worth a try. I've kicked off a run. The NavView test page is getting pretty complex, perhaps there is opportunity to simplify the test page a bit. For starters, content gets clipped right now for smaller sizes.

So that didn't solve the test failure either....I didn't add a small timer delay after clicking to close the flyout. We can give that a try as well but that's just me running out of options here 😑

So that didn't solve the test failure either....I didn't add a small timer delay after clicking to close the flyout. We can give that a try as well but that's just me running out of options here 😑

I think the simplest option might be to move the test out of this page.

Since we about tried everything in the test implementation to fix the failing test, yet without success, this sounds reasonable to me. I will take a look at moving the test to another test page.

@ranjeshj I think I have found the reason for the test failures we were seeing. PR https://github.com/microsoft/microsoft-ui-xaml/pull/3602 changed the pane scrolling and causes the menu item HasChildItem to not be visible on the screen by the time we try to open its flyout. Since it isn't on the screen, the child menu flyout is not opened and we are seeing the test failure. I had yet to merge the above PR into my local branch so I was not yet testing against the updated behavior, whereas the test pipeline did.
Adding the second pane footer item as part of this PR caused the HasChildItem menu item to be pushed out of the viewport, thus we are seeing the test fail, whereas it succeeded as part of the test run for PR https://github.com/microsoft/microsoft-ui-xaml/pull/3602.
Instead of movign the failing test to another page, I will fix the failing test in an upcoming PR.

@ranjeshj You can start a test pipeline run again to see if test runs fine now.

/azp run


[bazel] Add support for different signing key domains

Description
Background
Currently Bazel uses a default DEFAULT_SIGNING_KEYS dictionary to configure the key used to sign any opentitan binary target.  In some cases the dictionary is iterated to generate binaries signed with all available keys.
PR #17316 adds support for a ROM_EXT key domain, which is used to manage signing and verification of the first Silicon Owner boot partition. This key domain will be available for OpenTitan configurations that don't have Ownership Transfer support available, and where ownership of the chip is explicitly assigned by the ROM_EXT configured in flash.
Minimum key domain requirements
This issue is used to track the work needed to add support for multiple key domains to the build system. At a minimum the following key domains need to be supported:

ROM key domain: Any binaries verified by the ROM, e.g. ROM_EXT
ROM_EXT key domain: Any binaries verified by the ROM_EXT, e.g. first Silicon Owner boot stage
Silicon Owner key domain: Any binaries verified by any intermediate Silicon Owner stage. This key domain is available mainly for reference implementation purposes.

CC @cfrantz @timothytrippel

CC @pamaury

This should be addressed by #19493


Build all commits inside a pull request

Can a setting be added to build all commits added to a pull request rather than just the last one?  We generally like to test all the commits in a PR.
I looked through the settings and GitHub issues and couldn't find any way to do this.

For your case why not just create a build that tests all commits using the regular source control triggers?

I think there's a misunderstanding. I want to build all the commits within a pull request, not just the latest commit. (bonus points for not rebuilding already built commits when the PR is updated)
I'm not sure how I can do this with regular build triggers without a plugin that understands pull requests.

Is it still not resolved ? We are experiencing the same issue, where in our PR usually there are many commits but when the Job triggers only the most latest commit will execute. Using the below config:
refspec = +refs/pull/:refs/remotes/origin/pr/
branch specifier = ${sha1}


Add temple

I moved the better half of the Temple's test to examples/temple/, the basic.janet, and templates I suited to test/suite8.janet. There I made one code change.
I put the whole readme before the Test part under the Temple heading.
The temple/temple.janet became spork/temple.janet.

How do you decide what's in the "standard library" (if that's how we view spork)?   100% in favor of basic testing utilties, etc being in spork.  But HTML templates?  I'm not so sure.  There are other approaches to html rendering that are in joy and janet-html.  (Not templates exactly.)  I definitely don't feel strongly about this either way but wanted to at least raise the issue so spork doesn't become too much of a mishmash of things.

How do you decide what's in the "standard library" (if that's how we view spork)?   100% in favor of basic testing utilties, etc being in spork.  But HTML templates?  I'm not so sure.  There are other approaches to html rendering that are in joy and janet-html.  (Not templates exactly.)  I definitely don't feel strongly about this either way but wanted to at least raise the issue so spork doesn't become too much of a mishmash of things.

The first issue here is to look at Temple as HTML templates. It could be used for any templating. Even Janet code files, as could be seen here: https://git.sr.ht/~pepe/chidi/tree/master/item/chidi/generate. Or let's say benchmark results?
The second issue is the imagined spork library size, where we all differ a lot. But from some past conversation and by the latest confirmation https://github.com/janet-lang/janet/discussions/597#discussioncomment-296984 it should be big. Temple is one hundred forty lines of code in one file if we talk about the code size.
The third is the value of the code in spork as an example or educational piece, which for any loader related problems, Temple is a treasure. Pun intended.
Finally, Calvin's opinion is the most important here, and it was positive to the merge.

The first issue here is to look at Temple as HTML templates. It could be used for any templating. Even Janet code files, as could be seen here: https://git.sr.ht/~pepe/chidi/tree/master/item/chidi/generate. Or let's say benchmark results?
The second issue is the imagined spork library size, where we all differ a lot. But from some past conversation and by the latest confirmation https://github.com/janet-lang/janet/discussions/597#discussioncomment-296984 it should be big. Temple is one hundred forty lines of code in one file if we talk about the code size.
The third is the value of the code in spork as an example or educational piece, which for any loader related problems, Temple is a treasure. Pun intended.
Finally, Calvin's opinion is the most important here, and it was positive to the merge.

@pepe this all makes sense.  I just now reread the doc and noticed the mention of non-html usage.  I'm for it now.

@pepe this all makes sense.  I just now reread the doc and noticed the mention of non-html usage.  I'm for it now.

it will become harder and harder to do versioning properly if any of the sub libraries need backwards incompatible changes, i am not sure i like the direction spork is headed.

it will become harder and harder to do versioning properly if any of the sub libraries need backwards incompatible changes, i am not sure i like the direction spork is headed.

Versioning is indeed an issue, but mostly for the future ourselves, and I believe we will come with a smart solution.
BTW @andrewchambers, would you agree with adding some of your small libraries into spork? URI comes to mind first, if @bakpakin would agree.

Versioning is indeed an issue, but mostly for the future ourselves, and I believe we will come with a smart solution.
BTW @andrewchambers, would you agree with adding some of your small libraries into spork? URI comes to mind first, if @bakpakin would agree.

Hmm, uri and also bigint are perhaps candidates, though they both require C code. (URI has a tiny bit of C for performance, that could maybe be removed, but i don't really want to)

Hmm, uri and also bigint are perhaps candidates, though they both require C code. (URI has a tiny bit of C for performance, that could maybe be removed, but i don't really want to)

Hmm, I guess spork should stay .janet only, but only personal opinion.

Hmm, I guess spork should stay .janet only, but only personal opinion.

perhaps if jpm could decide if you had a c compiler and fall back.

perhaps if jpm could decide if you had a c compiler and fall back.

I think it is rather useful. Also, spork libraries don't need to be completely generic in my opinion, and I think temple is unlikely to change much of at all.

I think it is rather useful. Also, spork libraries don't need to be completely generic in my opinion, and I think temple is unlikely to change much of at all.


Support Kapacitor tick script creation with template

Hi,
In the latest Kapacitor version, there is a new feature for creating tick script templates - https://docs.influxdata.com/kapacitor/v1.0/examples/template_tasks/.
Can you add support for defining tasks using a given template?
I'll be happy to submit a PR that implements this.

Hey,
that sounds great. If you have the time and will, sure, go ahead! I would be glad to accept a PR. :) If you need any help, let me know.
Also if it's not a problem, please do it on a new branch created from dev or on dev itself.

Implemented.


Add Payments New for better disbursement support

#201 Update Payments New Disbursement Support
Changelog

Add https://developers.xendit.co/api-reference/?php#payouts to SDK withPaymentsNew callable class
Replace documentation from Payments into Payment Links for better readability
Update phpunit configuration

Task

[x] PHPUnit Test Coverage
[X] Method Example
[X] PHPDoc

Hopefully this PR is merged immediately @ellennugroho

@xen-HendryZheng

I've bump up the composer.json file which includes changes below

[x] Bump php version into 7.4 and up for PHPUnit 9 better support
[x] Remove coverage on phpunit.xml to avoid Warning: No code coverage driver available caused by abandoned packages

@xen-HendryZheng


Rancher 2.6.3: Cluster Management not remove imported cluster

Rancher Server Setup

Rancher version: 2.6.3
Installation option (Docker install/Helm Chart): Helm Chart
If Helm Chart, Kubernetes Cluster and version (RKE1, RKE2, k3s, EKS, etc): Kubespray

Information about the Cluster

Kubernetes version: 1.21.1
Cluster Type (Local/Downstream): Downstream

Hosted = cloud vps
Infrastructure Provider = Hetzner

Describe the bug
Deleted cluster already exists in rancher dashboard and in the menu
To Reproduce

Setup cluster for Rancher
Add rancher in cluster
Setup additional cluster
Add additional cluster into rancher in cluster manager(type generic)
Delete additional cluster(remove vms)
Remove cluster in rancher

Result
Cluster removed only in Cluster Management -> Clusters
Expected Result
Cluster removed in all components, Dashboard, Menu
Screenshots

Additional context

This cluster already exist in tab 'Cluster Management -> Advanced -> Mgmt Clusters' but i can not remove item

I found some resources for the old cluster in the rancher
cat cluster-scan-alert_clusteralertgroups.management.cattle.io.yaml

apiVersion: management.cattle.io/v3
kind: ClusterAlertGroup
metadata:
  annotations:
    lifecycle.cattle.io/create.cluster-alert-group-lifecycle_c-m-xlp7prgz: "true"
  deletionGracePeriodSeconds: 0
  deletionTimestamp: "2021-12-24T08:33:42Z"
  labels:
    cattle.io/creator: norman
  name: cluster-scan-alert
  namespace: c-m-xlp7prgz
spec:
  clusterName: c-m-xlp7prgz
  description: Alert for Cluster Scans, both manual and scheduled
  displayName: A set of alerts for cluster scans
  groupIntervalSeconds: 180
  groupWaitSeconds: 180
  repeatIntervalSeconds: 3600

cat etcd-alert_clusteralertgroups.management.cattle.io.yaml

apiVersion: management.cattle.io/v3
kind: ClusterAlertGroup
metadata:
  annotations:
    lifecycle.cattle.io/create.cluster-alert-group-lifecycle_c-m-xlp7prgz: "true"
  deletionGracePeriodSeconds: 0
  deletionTimestamp: "2021-12-24T08:33:42Z"
  labels:
    cattle.io/creator: norman
  name: etcd-alert
  namespace: c-m-xlp7prgz
spec:
  clusterName: c-m-xlp7prgz
  description: Alert for etcd leader existence, db size
  displayName: A set of alerts for etcd
  groupIntervalSeconds: 180
  groupWaitSeconds: 180
  repeatIntervalSeconds: 3600

cat event-alert_clusteralertgroups.management.cattle.io.yaml

apiVersion: management.cattle.io/v3
kind: ClusterAlertGroup
metadata:
  annotations:
    lifecycle.cattle.io/create.cluster-alert-group-lifecycle_c-m-xlp7prgz: "true"
  deletionGracePeriodSeconds: 0
  deletionTimestamp: "2021-12-24T08:33:42Z"
  labels:
    cattle.io/creator: norman
  name: event-alert
  namespace: c-m-xlp7prgz
spec:
  clusterName: c-m-xlp7prgz
  description: Alert for receiving resource event
  displayName: A set of alerts when event happened
  groupIntervalSeconds: 180
  groupWaitSeconds: 180
  repeatIntervalSeconds: 3600

cat kube-components-alert_clusteralertgroups.management.cattle.io.yaml

apiVersion: management.cattle.io/v3
kind: ClusterAlertGroup
metadata:
  annotations:
    lifecycle.cattle.io/create.cluster-alert-group-lifecycle_c-m-xlp7prgz: "true"
  deletionGracePeriodSeconds: 0
  deletionTimestamp: "2021-12-24T08:33:42Z"
  labels:
    cattle.io/creator: norman
  name: kube-components-alert
  namespace: c-m-xlp7prgz
spec:
  clusterName: c-m-xlp7prgz
  description: Alert for kube components api server, scheduler, controller manager
  displayName: A set of alerts for kube components
  groupIntervalSeconds: 180
  groupWaitSeconds: 180
  repeatIntervalSeconds: 3600

cat node-alert_clusteralertgroups.management.cattle.io.yaml

apiVersion: management.cattle.io/v3
kind: ClusterAlertGroup
metadata:
  annotations:
    lifecycle.cattle.io/create.cluster-alert-group-lifecycle_c-m-xlp7prgz: "true"
  deletionGracePeriodSeconds: 0
  deletionTimestamp: "2021-12-24T08:33:42Z"
  labels:
    cattle.io/creator: norman
  name: node-alert
  namespace: c-m-xlp7prgz
spec:
  clusterName: c-m-xlp7prgz
  description: Alert for Node Memory, CPU, Disk Usage
  displayName: A set of alerts for node
  groupIntervalSeconds: 180
  groupWaitSeconds: 180
  repeatIntervalSeconds: 3600

Already reproduces on 2.6.8
022/09/25 07:33:17 [INFO] Stopping cluster agent for c-m-fbbfqs9h
2022/09/25 07:33:17 [INFO] Shutting down /v1, Kind=Namespace workers
2022/09/25 07:33:17 [INFO] Shutting down rbac.authorization.k8s.io/v1, Kind=Role workers
2022/09/25 07:33:17 [INFO] Shutting down /v1, Kind=Node workers
2022/09/25 07:33:17 [INFO] Shutting down rbac.authorization.k8s.io/v1, Kind=RoleBinding workers
2022/09/25 07:33:17 [INFO] Shutting down /v1, Kind=Secret workers
2022/09/25 07:33:17 [INFO] Shutting down /v1, Kind=LimitRange workers
2022/09/25 07:33:17 [INFO] Shutting down /v1, Kind=ResourceQuota workers
2022/09/25 07:33:17 [INFO] Shutting down rbac.authorization.k8s.io/v1, Kind=ClusterRole workers
2022/09/25 07:33:17 [INFO] Shutting down /v1, Kind=ServiceAccount workers
2022/09/25 07:33:17 [INFO] Shutting down apiregistration.k8s.io/v1, Kind=APIService workers
2022/09/25 07:33:17 [INFO] Shutting down rbac.authorization.k8s.io/v1, Kind=ClusterRoleBinding workers
2022/09/25 07:33:55 [ERROR] Error during subscribe websocket: close sent
2022/09/25 07:37:18 [ERROR] Unknown error: users.management.cattle.io "034g0dwd1kupj13" not found


Unexpected under bits applied to 0

Version 0.1.0
I would expect that ⍜f∘ (identity under an invertible function f) behaves the same as ∘ (identity), and that is the case that I found so far, but I have noticed that ⍜⋯∘0 (identity under bits applied to 0) returns [] instead of 0. That's because ⋯0 returns [] and ⍘⋯[] returns [] again (⍘⋯[0] returns 0 though).
Maybe ⍘⋯[] could return 0 instead, but I guess that there is an ambiguity regarding which rank [] should be inferred: 1 if it meant to be the binary representation of 0, or 2 if it meant to be an empty array of binary representations.
Should ⋯0 return [0] instead for avoiding this ambiguity with ⋯'s result?
Motivation
I wrote a function that computes the bitwise AND of two numbers as ⍘⋯⬚0×∩(⋯), and found an error when both arguments are 0 because it produced [] and the remaining program expected a number.

In f272bbef85ac, I've made ⍘⋯[] give 0, ⍘⋯[[]] give [0], etc

Thank you for the quick response! It sounds good.
But now ⍜⋯∘[] crashes, which returns [] in version 0.1.0, so I don't know if it relates to this solution. Now that I tried, there seem to be other inconsistencies with arrays that contain only 0s: ⍜⋯∘[0 0 2] returns [0 0 2], but ⍜⋯∘[0 0 0] crashes on main and returns a [0 0] shape array on 0.1.0.
Maybe I'm wrong, but I suspect that the ambiguity of [] being the binary representation of 0 and, at the same time, an empty arguments array (because bits is pervasive) is the root cause of these inconsistencies.

Sorry about that.
This should be fixed in 0732a1d21e72ac6bf.
I've also added some tests.


Add Shortcut Option

An option for configuring a hotkey to run the addon.
Hotkey should be customizable and removable.
Default hotkey should be "None."

I would also love this option.
Any chance this could be added in the near-future? Since its already been 2 years since the feature request

I would also love this option.
Any chance this could be added in the near-future? Since its already been 2 years since the feature request

already been 2 years since the feature request
Well, since I'm the one who added the enhancement issue even though I wouldn't use the hotkey, I never felt the need to add it.

If you have a use case for it, would you mind explaining it to me?
I predominantly use my mouse when browsing, so clicking the icon is much quicker than say ctrl-shift-r or something.

already been 2 years since the feature request
Well, since I'm the one who added the enhancement issue even though I wouldn't use the hotkey, I never felt the need to add it.

If you have a use case for it, would you mind explaining it to me?
I predominantly use my mouse when browsing, so clicking the icon is much quicker than say ctrl-shift-r or something.

theres at least 2 news sites i visit daily where i drag the tab to open in a new window, then select around 20 links to open in new tabs and use the keyboard (ctrl+tab, ctrl+w) to cycle through them. it often happens that either darkreader or stylish conflicts or i just feel like switching, or that i accidently wasnt logged in when i clicked the tabs. in that case it would be really convenient to have a single action keyboard hotkey instead of having to move my hand back to the mouse and do another 4 actions.
and yeah i predominantly use the keyboard.
cool extension btw, the only similar one i found would reload all tabs in all windows, which isnt always desirable.

theres at least 2 news sites i visit daily where i drag the tab to open in a new window, then select around 20 links to open in new tabs and use the keyboard (ctrl+tab, ctrl+w) to cycle through them. it often happens that either darkreader or stylish conflicts or i just feel like switching, or that i accidently wasnt logged in when i clicked the tabs. in that case it would be really convenient to have a single action keyboard hotkey instead of having to move my hand back to the mouse and do another 4 actions.
and yeah i predominantly use the keyboard.
cool extension btw, the only similar one i found would reload all tabs in all windows, which isnt always desirable.

interesting. i'm sure the hotkey code is fairly easy to do. i'll try to add it in when i can

interesting. i'm sure the hotkey code is fairly easy to do. i'll try to add it in when i can

well, this was the easiest addition in my life

well, this was the easiest addition in my life

Thanks for adding this. How simple it may be, it is a game changer. Left you a 5-star review.

Thanks for adding this. How simple it may be, it is a game changer. Left you a 5-star review.


Dotted notation handling

What happened:
The current code doesn't handle dotted notations from config section in a package spec file correctly. It treats the whole dotted notation as a key, while the official Helm code breaks it down and treats it as a nested map.
For example, expose.tls.certSource: auto results in the following:
Helm code:

Our code:

The discrepancy results in no value override in those dotted-notation fields, because Helm relies on keys for value coalescing. More details for value coalescing can be found here.
What you expected to happen:
Treat those dotted notations as nested maps, the same way Helm handles.
How to reproduce it (as minimally and precisely as possible):
Sample package spec:
apiVersion: packages.eks.amazonaws.com/v1alpha1
kind: Package
metadata:
  name: test
  namespace: eksa-packages
spec:
  config: |
    expose.tls.certSource: "auto"
    secretKey: "use-a-secret-key"
  packageName: harbor

Anything else we need to know?:
Environment:

EKS Anywhere Release:
EKS Distro Release:

maybe
apiVersion: packages.eks.amazonaws.com/v1alpha1
kind: Package
metadata:
  name: test
  namespace: eksa-packages
spec:
  config: |
    expose:
       tls:
         certSource: "auto"
    secretKey: "use-a-secret-key"
  packageName: harbor

If I change the package spec to the following, it works
apiVersion: packages.eks.amazonaws.com/v1alpha1
kind: Package
metadata:
  name: test
  namespace: eksa-packages
spec:
  config: |
    expose:
        tls:
          certSource: "auto"
    secretKey: "use-a-secret-key"
  packageName: harbor

Closing this as the issue is now with the CLI part that generates a malformed package spec file.


Consider integrating divnix/devos.

Once flakes are implemented (https://github.com/nix-gui/nix-gui/issues/55) we should consider piggybacking on devos. It would provide an opinionated approach to system configuration and allow for sane defaults and easier integrations.
One of the developers has suggested they'd be interested in helping with integration https://discourse.nixos.org/t/nix-gui-use-nixos-without-coding/15409/11 And the projects creator also seems interested in nix-gui.
More research needs to be done on tradeoffs, model compatibility, how it may be integrated, etc.

I'm also happy to help with this. I imagine the benefit of integrating with devos, is that there is a semi-standard folder structure. I think the profiles folder is likely the most important for nix-gui, since you could add a profile which nix-gui can edit. I'm not entirely sure where nix-gui sets configuration options. I think we could setup devos, so the flake exports information about its directory structure so the flake can be evaluated to tell you where folders are located. Bascially I'm thinking folder structure metadata.

That's great to hear, thanks for your interest.
Nix-Gui has quite a few changes necessary before this can take place though. Could I follow back with you in 2 to 4 months after https://github.com/nix-gui/nix-gui/milestone/2 is complete and https://github.com/nix-gui/nix-gui/milestone/3 has the necessary progress for this issue?


Use pipe attribute selector

Currently, Prism uses selectors like [class*="lang-"]. It works but it also selects classes like foolang-css.
That doesn’t usually matter, but there’s more correct selector which looks like this: [class|="lang"]. It selects the class lang-css but not foolang-css or even foo-lang-css!
Could we use it?

That won't work with Prism because there may be any number of classes before the lang-xxxx (or language-xxxx) class. With [class|="lang-"], elements like

 will not be selected.


Version conflict with, needs updating in python repositories.

An issue which was fixed needs to be updated in the python repositories.
from werkzeug import secure_filename,FileStorage
changed to
from werkzeug.utils import secure_filename
from werkzeug.datastructures import  FileStorage

Was already fixed in this repository, but not push to pip.
As a temporary solution i use -e git://github.com/maxcountryman/flask-uploads.git#egg=elasticutils in the requirements.txt file.
https://stackoverflow.com/questions/61628503/flask-uploads-importerror-cannot-import-name-secure-filename

Please see this related issue: https://github.com/maxcountryman/flask-uploads/issues/39

Note, that should likely be:
-e git://github.com/maxcountryman/flask-uploads.git#egg=Flask-Uploads

Note the difference in the #egg value.

Huh, pip freeze emitted it like this:
-e git://github.com/maxcountryman/flask-uploads.git#egg=Flask_Uploads


chore(deps): use node 20

Description
Update the action to use node 20
Related Issue(s)
https://github.com/redhat-actions/oc-login/issues/35
Checklist

[ ] This PR includes a documentation change
[X] This PR does not need a documentation change

[ ] This PR includes test changes
[X] This PR's changes are already tested

[ ] This change is not user-facing
[ ] This change is a patch change
[X] This change is a minor change
[ ] This change is a major (breaking) change

Changes made
Upgrade dependencies

Can someone merge this and release it? The NodeJS 16 warnings are pretty annoying

@sepulvedablanco thanks for your PR! 👍
I will look at it very soon. Apologies for the delays.

done @divyansh42

@sepulvedablanco could you please sign your commits as well?

done @divyansh42

@divyansh42 friendly reminder


fixes search position

Purpose

fixes search position
fix for https://github.com/wso2/docs-apim/issues/5395

Before

After

@tmkasun , when I added this config back locally, the position of the search got adjusted. Therefore, looks like this config does do something after all.


Change compass name and color

Could you make it one line?
player.getInventory().setItem(0, Items.createItem(Material.COMPASS, 1, (short) 0, ChatColor.BLUE + "" + ChatColor.BOLD + "Teleport Tool");

Why is this code in four places, also, localization

Hey @Vixo1, are you able to amend your PR down to one commit or would you like me to do that for you?

I maked it one commit (Thanks @Pablete1234) but I think that this will not be necessary if #810 is implemented.

Implemented in https://github.com/twizmwazin/CardinalPGM/commit/56b4c5b5896387ad579ea0de62f7f823b0fba813

As @TiTo-418 said has been implemented by #933


Fixing the unusual scroll bar

Based on @Amnish04 's comment, I confirm that after deploying I can see the unusual scroll bar triggered when opening menu
Refer #374

Which browser/OS are you using?  Is it due to the sidebar being open?

Interesting.  Bugs like this can be subtle, and finding the right set of steps to reproduce it, tricky.  I'm not able to reproduce this on macOS/Chrome, but that doesn't mean it's not there.  See if you can figure out what you're doing that I'm not.
It could be related to the content you're using in the chat, for example, or somehow be caused by an extension, etc.

@Rachit1313 what size is your screen/monitor?
I'm on Windows 10/Edge I don't encounter the bug when I open the menu on a maximized ChatCraft window (on a 27-inch monitor), even with the sidebar open:

For me, the horizontal bar will appear when I reduce the ChatCraft window to a certain screen width.

I can even see this right now 100% width on 13 inch macbook pro

Whatever is causing it, the issue is that we have an overflow-x on a parent element that shouldn't have it.  Do you want to see which one it is?


Convert to TypeScript

Currently blocked by https://github.com/googleapis/gapic-generator/issues/2054

Maybe you (maintainers) can contribute to the @types/google-cloud__datastore until https://github.com/googleapis/gapic-generator/issues/2054 is resolved? Keep it more up to date? It would be easier for you because you're working with the source code.

This is actually in pretty good shape :) We need to ship a new version, but the types are pretty complete at this stage.


add pods

Codecov Report

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

@@           Coverage Diff           @@
##           master      #32   +/-   ##
=======================================
  Coverage   90.57%   90.57%
=======================================
  Files          28       28
  Lines         658      658
=======================================
  Hits          596      596
  Misses         62       62

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 86238ca...23d068d. Read the comment docs.

closing temporarily


Use new token storage in Credential classes

The requested changes here are provided in https://github.com/okta/okta-mobile-kotlin/pull/292. Relevant sample changes are in https://github.com/okta/okta-mobile-kotlin/pull/293


Made with icons not showing in Light primary color toggle

Description
When light primary color is set, the made with icons are not visible in footer:
Light Theme:

Dark Theme:

The icons must be visible in light theme as well.
Acceptance criteria

The Made with icons should be visible in both light and dark primary colors

Fixed by #162 . Thansk @Charliiee


Can give the source about params of pool_list configuration?

So if I want a farmer to connect a pool by cli, I have to config pool_list manually in the config.yaml and I can not how to get the values of some params .  Can the official  give the source about the params of pool_list， or how I can use chia cli to get these params。

authentication_key_info_signature:
authentication_public_key:
authentication_public_key_timestamp:
owner_public_key:
target_signature:
singleton_genesis:
target_puzzle_hash:

you need to use the wallet to create a singleton (plot nft), the wallet will then fill the values into the config.


--no-print-algo doesn't work

Dear Zipline Maintainers,
Before I tell you about my issue, let me describe my environment:
Environment

Operating System: MacOS 10.14.1
Python Version: 3.5
How did you install Zipline: Macports
zipline 1.3.0

Now that you know a little about me, let me tell you about the issue I am
having:
Description of Issue

When using the --no-print-algo command line option with zipline run I didn't expect to the the performance results dataframe printed out at the end.
However, it does. It seems like unless you specify an outfile, the performance dataframe will get printed out.

Here is how you can reproduce this issue on your machine:
Reproduction Steps

Write a simple algo, like the dual moving average crossover.
Run it with zipline run and the --no-print-algo command line.
See the results dataframe get printed at the end.
...

What steps have you taken to resolve this already?
I write the results to a pkl file to suppress output.
...
Sincerely,
Brett Elliot

Ah that makes sense. Ill use  -o /dev/null
Thanks!
On Tue, Nov 27, 2018 at 11:27 AM Richard Frank<EMAIL_ADDRESS>wrote:

Hi @brettelliot https://github.com/brettelliot ,
I believe the --no-print-algo flag governs whether or not the simulator
should print the text
https://github.com/quantopian/zipline/blob/d708817e44e06ae2ff747f478e2e8f3c0a8750d1/zipline/utils/run_algo.py#L117-L126
of the strategy before simulating it, as opposed to printing the results of
the simulation.
@click.option(
'--print-algo/--no-print-algo',
is_flag=True,
default=False,
help='Print the algorithm to stdout.',
)
As you noted, the performance results of the simulation are written to the
--output location, i.e. via a separate option.
@click.option(
'-o',
'--output',
default='-',
metavar='FILENAME',
show_default=True,
help="The location to write the perf data. If this is '-' the perf will"
" be written to stdout.",
)
Internally, the zipline ipython magic uses the same underlying
functionality and disables printing the perf data by passing os.devnull
as the output location. If you pass the value of that attribute (looks like
'/dev/null' on posix envs or 'nul' on windows), I think you'll see the
behavior you're looking for.
I could also imagine an enhancement to zipline by exposing an OS-agnostic
sentinel value to the click option to disable printing from the command
line.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/quantopian/zipline/issues/2370#issuecomment-442122610,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AJbQ8VNLhMLAcIYeyAv63oUQbdi9YwKhks5uzWfrgaJpZM4YvxpD
.

--
Brett Elliot


Remove console.log when using KHR_materials_transmission

Also pass extensionContext instead of context when loading extension data.
(I believe this part is not essential here but it makes implementation match that
for other extensions)

@bghgary is out so please bear with us as he is OOF for the week


time: Event::TimeSystem abstraction to make it feasible to inject time with simulated timers

Description: Unifies the TimeSource so that system and monotonic sources are always handled similarly (mocks, real-time, or in the future, simulated), and derives a new interface, Event::TimeSystem, which managers timers.  This is a pure refactor; none of the operation or semantics of any production or test code is changed with this PR.  This is another step in resolving #4160.
Risk Level: medium, just because PR is large and merge conflicts happen, but it should be pretty safe
Testing: //test/...
Docs Changes: n/a
Release Notes: n/a

Yes I think this is ready for review.

Added TODO for duration_cast hacking.

Assigned @htuch for senior maintainer.


BabyAI_MixedTrainLocal doesn't exist

I followed all the instructions but found the gym env doesn't exist.

Hi,
I fixed the typos in BabyAI-Text's readme related to your first issue.
Concerning the second one, could you confirm this PR fixes it, please?

OK, I have solved the problem. Thanks for your response.


Feature detection and cleanup (Ml-KEM/SHA3)

This PR

adds runtime detection for hash functions
cleans up the sha3 implementation

I wish we could have hidden the different implementations inside libcrux-sha3 rather than polluting hash_functions.rs but I don't think that's possible with our structure and optimization constraints.

We can, I'm not sure it's the right structure though. It's pushing that logic into the sha3 crate. I don't have a strong opinion, both is doable and has pros and cons.


rdt-discovery: update links to intel-cmt-cat source

This PR is side-track, discovered outdated links while I was preparing #120.
Source links were outdated: repo name has changed, cap.c file name changed.
This PR makes no functional changes.
repo seems relocated from 01org to intel,
GH redirects 01org for now, but lets have it corrected in source.

Please update your patch. Your latest patch that was just merged just added more references to 01org :) Dockerfile also points to the old repo

@okartau please rebase on latest master and we'll merge this

rebased, also changed repo name in Dockerfile


Chapter 3: check log files - fix grep command

Instead of
$ ansible multi -b -m shell -a "tail /var/log/messages | grep ansible-command | wc - l"

I think it should be
$ ansible multi -b -m shell -a "tail /var/log/messages | grep ansible.legacy.command | wc -l"

Correct, thanks for finding and reporting this!

Thanks for the book. Just started learning Ansible, really enjoying going through the book and doing the exercises.


Specify a element to give focus to when dialog is opened (Proposal)

Bug, feature request, or proposal:
Proposal
What is the current behavior?
Currently there is only the following options in the focus-trap to give focus:
focusInitialElementWhenReady()
focusFirstTabbableElementWhenReady()
focusLastTabbableElementWhenReady()
focusInitialElement()
focusFirstTabbableElement()
focusLastTabbableElement()

What I would like to do is add something like this to the focus-trap.ts
NOTE: naming is obviously subject to change and would make it whatever you guys would like
  /** Focuses the specified element passed in. */
  focusSpecifiedElement(focusElement: string) {
    let redirectToElement = this._element.querySelector(
      focusElement
    ) as HTMLElement;

    if (redirectToElement) {
      redirectToElement.focus();
    }
  }

So what this would allow for is in dialog-config for example to have a variable where the user can specify a element to give focus to:
  /** Specify the element that should receive focus on load */
  focusElement?: string;

Thoughts?
I have already implemented a version in a fork of mine and it seems to work really well. I would like to make it officially available in material2. Mainly wanted to get your guys opinion before I go through all the work to put up a PR and such on if you would even accept something like this.
Let me know!
Will put up a PR very soon if you guys would welcome a update like this.
What is the use-case or motivation for changing an existing behavior?
Would like to be able to give focus to a element I specify.

Adding the cdkFocusInitial attribute to the element accomplishes this same purpose.
https://material.angular.io/cdk/a11y/overview#regions

Gotcha, thanks will.
Closing this.


IoU smooth-l1 loss function

It seems that the implementation of IoU smooth-l1 loss mentioned in the paper is missing in this repo. While it is implemented in https://github.com/DetectionTeamUCAS/RetinaNet_Tensorflow_Rotation/blob/b03a7eafba21bfbb78dbffe1be53ab914080201e/libs/losses/losses.py#L139

The early paper was called R2CNN++, which did not include IoU-smooth loss. I built another repo to verify it separately. @yzheng97


500 Status Code for Layout Deletion

Deleting a Layout fails with a 500 status code if the layout is used in a room.
I think in an older slurk version it was possible to set the layout of a room to null. And if this was the case the default.json layout was used.

@TimDiekmann

That's not a problem pertinent to the Layout table alone. Same applies to deleting permissions while they are used in a token.

As I'd avoid cascading deletion in slurk this error should be caught and return a 409 error code (conflict).


To use Authorize

If you want to use [Authorize], you can use IOWebSocketChannel instead of WebSocketChannel to pass Header.
_webSocket = IOWebSocketChannel.connect(Uri.parse(url), headers: {
      'Authorization':
          'Bearer ${_accessTokenFactory != null ? await _accessTokenFactory!() : ''}'
    });

It is also possible to transmit the entire header by slightly modifying the code in part lib/web_socket_transport.dart.

If you want to use [Authorize], you can use IOWebSocketChannel instead of WebSocketChannel to pass Header.


0.55.1 Docker fails to start with "cryptography.hazmat.primitives.hashes" attribute error

Home Assistant release (hass --version):
0.55.1
Component/platform:
Docker
Description of problem:
Docker container fails to start
Traceback (if applicable):
Traceback (most recent call last):
  File "/usr/local/lib/python3.6/runpy.py", line 193, in _run_module_as_main
    "__main__", mod_spec)
  File "/usr/local/lib/python3.6/runpy.py", line 85, in _run_code
    exec(code, run_globals)
  File "/usr/src/app/homeassistant/__main__.py", line 374, in <module>
    sys.exit(main())
  File "/usr/src/app/homeassistant/__main__.py", line 349, in main
    args = get_arguments()
  File "/usr/src/app/homeassistant/__main__.py", line 90, in get_arguments
    import homeassistant.config as config_util
  File "/usr/src/app/homeassistant/config.py", line 26, in <module>
    from homeassistant.util.yaml import load_yaml, SECRET_YAML
  File "/usr/src/app/homeassistant/util/yaml.py", line 16, in <module>
    import credstash
  File "/usr/local/lib/python3.6/site-packages/credstash.py", line 55, in <module>
    'RIPEMD': hashes.RIPEMD160,
AttributeError: module 'cryptography.hazmat.primitives.hashes' has no attribute 'RIPEMD160'

Same problem here

+1 on this. Refuses to start now. Also on Docker.

+1 here, refusing to start on Docker.

Same with docker as well.  Reverting to 0.54 got me back up and running

+1, same here

Looks like maybe RIPEMD160 was removed from cryptography module v2.1 on 2017-07-10:
https://github.com/pyca/cryptography/blob/master/CHANGELOG.rst

I would like to add to what @cameronr said and point out that before #9844, credstash version 1.13.3 was part of the requirements_all.txt file and was installed when the docker image is built. I believe after that PR, the latest version of credstash is installed instead, using the gen_requirements_all.py file.

+1, hitting this problem too.

+1,I have watchtower updating all the containers, this morning I was like, did I broke my config? Have I been hacked?
From 1 to 10 how serious is the problem/time to fix?
If you need any logs or anything let me know if I can help :)

+1 seeing this problem as well.  Watchtower updated on 12-Oct on Dev channel and am seeing it on the non-dev channel as well.

+1

+1

+1

+1

+1

+1

+1

+1

+1

Instead of posting a +1 you can react with the same on the original post, and click the subscribe button. That way you will show that you're affected but not clutter the thread. Thanks!

+1

Removed all the crappy +1 posts. Will lock if spam continues.
Try removing <config dir>/deps/credstash and see if that works.

Upstream issue: https://github.com/fugue/credstash/issues/175

At least in my case I have no file or directory called credstash am I the only one?

@eduardoolivera - same here. My /config/deps dir is empty on my hass.io install

"Try removing /deps/credstash and see if that works."
Doesn't exist

My deps directory is empty, too in a Docker install.

You can rollback to 0.55 for now by SSH'ing to HASSIO and running the follwing command:
curl -d '{"version": "0.55.0"}' http://hassio/homeassistant/update

@dcshoes23 hassio is not a domain and if I try hass.io instead I get :
<head><title>405 Method Not Allowed</title></head>

@dcshoes23 I had to take the ".0" off the end of the version. This worked for me:
curl -d '{"version": "0.55"}' http://hassio/homeassistant/update

Changing known requirements file for Docker will fix the issue until upstream fix is inplace.
In requirements_all.txt add
credstash==1.13.2

cryptography==2.0.3

Fixed in release 0.55.2


Pin serde and re-export bech32

As per IRC discussion:
12:43 < andytoshi> oh, damn, i thought we re-exported bech32
12:45 < elichai2> no it was only hashes and secp iirc
12:47 < andytoshi> if someone is so inclined as to export bech32, that'd be cool..
12:47 < andytoshi> and bump to 0.19.1
12:49 < elichai2> We have a bigger problem
12:49 < elichai2> "Minimum supported rustc version is 1.31"
12:49 < andytoshi> where's that?
12:54 < elichai2> ok. apperantly it's serde-derive fault
12:54 < elichai2> he published a new minor version while bumping a major proc-macro2
12:54 < elichai2> https://github.com/serde-rs/serde/blob/v1.0.98/serde_derive/Cargo.toml
12:54 < elichai2> https://github.com/serde-rs/serde/blob/v1.0.99/serde_derive/Cargo.toml
13:01 < andytoshi> patch version even
13:02 < andytoshi> w/e we can just pi
13:02 < dongcarl> :man-facepalming:
13:02 < andytoshi> but
13:04 < elichai2> yep
13:04 < elichai2> we'll see what dtolnay will say but I can probably guess.... https://github.com/serde-rs/serde/issues/1602
13:06 < elichai2> i'll open a PR pinning, exporting and bump to 0.19.1. wanna yank 0.19.0?
13:07 < andytoshi> all old versions are affected

cc https://github.com/serde-rs/serde/issues/1602

Codecov Report

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

@@           Coverage Diff           @@
##           master     #316   +/-   ##
=======================================
  Coverage   81.85%   81.85%
=======================================
  Files          38       38
  Lines        6951     6951
=======================================
  Hits         5690     5690
  Misses       1261     1261

Impacted Files
Coverage Δ

src/lib.rs
100% <ø> (ø)
:arrow_up:

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 fbc8510...0153061. Read the comment docs.

What will happen to the serde version that users get when they use this PR and have other dependencies on serde?

Can we publicly export serde?

Hopefully no other dependency will pin a serde version and then it will resolve to the versions pinned here
Re exporting is mostly needed for multiple major versions

I don't think reexporting makes much sense in the case of serde. It's seldomly used by end users directly and if so, they just use the derive feature and I don't think you can derive from an exported crate out of ours...

Yeah, probably not


Web ui dependency tree

how to search the service dependency tree.Is the web ui the same as twitter use?

Same confused with me.

I'll answer the easy question: This is the same UI as Twitter uses.

I checkout the master code and start the zipkin-sample, I don't see any Overview Timeline or Dependencies tabs.The web UI differents from the screen shot whitch is yuchen99 post

FWIW, your experience is common. The first time setup and UI experiences
are lacking and contributors are actively brainstorming on how to address
high priority topics like this.

Closing due to lack of activity. The Dependency tab should be there now.


Preserve selection in tables with open typeahead menu on vertical arrow key press

This change prevents the table cell selection from changing on vertical arrow keys while a typeahead menu is open (Fix #5819).
Before
https://www.loom.com/share/92e75e1299434e62838bd40449b77083?sid=a82a72d8-03ea-42b9-ba46-d2431be5a2d2
After
https://www.loom.com/share/53579c647ae84429be68b264a44a50bf?sid=f7e29571-dfe1-45f7-bb02-b7ac2d211f33

@ivailop7 Can you take a look at this?