XML Issues while running the power shell command to install the responsive javascript or css for Sharepoint2013 OnPremises

Apply-SPOProvisioningTemplate : The Provisioning Template URI 01-Responsive.UI.Infrastructure.xml is not valid.
At line:49 char:5

Apply-SPOProvisioningTemplate -Path .\01-Responsive.UI.Infrastructure.xml -H ...

  + CategoryInfo          : WriteError: (:) [Apply-SPOProvisioningTemplate], ApplicationException
  + FullyQualifiedErrorId : EXCEPTION,OfficeDevPnP.PowerShell.Commands.Branding.ApplyProvisioningTemplate

Apply-SPOProvisioningTemplate : The Provisioning Template URI 02-Responsive.UI.Template.xml is not valid.
At line:52 char:5

Apply-SPOProvisioningTemplate -Path .\02-Responsive.UI.Template.xml -Handler ...

  + CategoryInfo          : WriteError: (:) [Apply-SPOProvisioningTemplate], ApplicationException
  + FullyQualifiedErrorId : EXCEPTION,OfficeDevPnP.PowerShell.Commands.Branding.ApplyProvisioningTemplate

I am also facing same issues, could you pls help on this.. Advance thank you

Hi,
Thanks for using the Responsive.UI sample. Unfortunately I cannot reproduce the issue. I've just tested it against an on-premises SP2013. Just to double-check, do you have the latest version of the PowerShell cmdlets? Thanks.

HI,
I think i have the lastes verison of the powerSheel cmdlets.
Name             : Windows PowerShell ISE Host
Version          : 4.0
InstanceId       : e44de9ba-6d8a-4a09-81b2-985b2b1e4252
UI               : System.Management.Automation.Internal.Host.InternalHostUserInterface
CurrentCulture   : en-US
CurrentUICulture : en-US
PrivateData      : Microsoft.PowerShell.Host.ISE.ISEOptions
IsRunspacePushed : False
Runspace         : System.Management.Automation.Runspaces.LocalRunspace
Please do let me know if i  am missing anything.
Thanks

are the specific XML files in the same directory as the powershell scripts? Go to that directory via command line and execute the ps1 script.

To see the version of the PnP PowerShell extensions please do that:
Connect-SPOnline "targetURL" -Verbose

HI @forrahul123 and @PandaHaz,
is this issue already resolved for you and all good - or still experiencing challenges? We did do recently updates on the process around this and it should be all good. Will close up the issue for now, but you can reopen it, if there's still some things causing exceptions. thx for reporting these.


possibility to change insert pattern according to file extension ?

it would be nice to be able to have multiple insert patterns ( basename, prefix etc) according to html markdown or latex files.
Is it possible to implement this feature ?
Tks for the great plugin !

making the extension language dependent (at least for the 'insertPattern' option) would be great.
for instance for the asciidoc language:
  "[asciidoc]": {
    "pasteImage.insertPattern": "${imageSyntaxPrefix}${imageFileName}${imageSyntaxSuffix}"
  }

and for the markdown language:
  "[markdown]": {
    "pasteImage.insertPattern": "${imageSyntaxPrefix}${imageFilePath}${imageSyntaxSuffix}"
  }

Difference is that asciidoc already has a default image location specified in the configuration (so adding the 'imageFileName' is sufficient in your plugin), but for markdown I want to add the full relative path 'imageFilePath'
Great extension btw!


Add MySQL db recreate script

What issue is this solving?
Closes #15
You can wipe the MySQL database and re-create all tables from scratch.
To use this new script, run the following command:
$ npm run db-recreate

Changes

Adds new script to drop and recreate all current tables in the db.

This is mostly for dev/testing purposes and getting new contributors
up and running.

Adds new script command and dateformat dependency in package.json
Uses new MySQL datetime format in sendcode.js

This script is for MySQL, but we may change over to Mongo in the future.
Whichever database we choose, it may be wise to switch over to an ORM
for database interaction. For MySQL, we could use Sequelize, for Mongo
we could use Mongoose.

Additional Information

Any new dependencies to install (npm install)? YES
Any special requirements to test? YES

Install the new deps
You need to have DB_USER, DB_HOST, DB_DATABASE and DB_PASSWORD for a MySQL db in your .env file
Your DB User must have perms to drop and create tables

One small thing - can you change the invalid on the infractions table to valid?

Done, sorry about that! Got sidetracked looking up ORMs for Node and forgot to change that one!


pg_dump: not found - for cli container to connect to postgresql

It seems that pg_dump is missing on the cli container
[xxx]cli-drupal:/app/web$ drush sql-sync @self @xxxx -y
You will destroy data in api-lagoon-master.lagoon.ch.amazee.io/drupal and replace with data from drupal.
Do you really want to continue? (y/n): y
Starting to dump database on Source.                                                                                [ok]
sh: pg_dump: not found
Database dump saved to /home/drush-backups/drupal/20180222134406/xxx.sql.gz                                                                                [success]
Starting to discover temporary files directory on Destination.                                                                                [ok]
You will delete files in xxx-xxxx@api-lagoon-master.lagoon.ch.amazee.io:/tmp/xxx.sql.gz and replace with data from /home/drush-backups/drupal/20180222134406/xxx.sql.gz
Do you really want to continue? (y/n): y
Copying dump file from Source to Destination.                                                                                [ok]
Starting to import dump file onto Destination database.

As a workaround this can be fixed by adding following line to Dockerfile.cli

RUN apk add --no-cache postgresql

It seems that pg_dump is not existing in a seperate package https://stackoverflow.com/questions/40628472/can-i-install-pg-dump-without-installing-postgres-on-alpine

Mhh we could install postgresql copy out the pg_dump and remove postgresql again, with that we would save space

mhh no wait it is part of https://pkgs.alpinelinux.org/contents?file=pg_dump&path=&name=&branch=edge

Another problem is that we can't use the postgresql package as it has a different version than postgres:alpine

We're trying to multistage the amazeeio/postgres-drupal image inside our Dockerfile.cli
and to copy the pg_dump from there.
FROM amazeeio/postgres-drupal:latest as amazeeio-postgres

FROM amazeeio/php:7.1-cli-drupal

# copy the pg_dump binary
COPY --from=amazeeio-postgres /usr/local/bin/pg_dump /usr/local/bin/pg_dump

@dasrecht
but the postgresql-client is also 10.2-r0 ? https://pkgs.alpinelinux.org/package/edge/main/x86_64/postgresql-client

For alpine:3.7 this is correct, the cli image runs on alpine:3.4 which only has 9.5.11 (https://pkgs.alpinelinux.org/package/v3.4/main/x86_64/postgresql-client)

Ok as discussed during standup we'll install the edge version of postgresql-client

Ok as discussed during standup we'll install the edge version of postgresql-client


Tekton Pipelines Topical Ownership

Changes

We identified topic areas in Tekton Pipelines and established explicit
ownership on specific areas of the codebase in doc
Topical owners are responsible for triaging bugs, reviewing TEPs and
moving along any work in their topic areas
Next step would be automating tagging topical owners on issues, PRs
and TEPs on their topic areas
/kind misc
Submitter Checklist
These are the criteria that every PR should meet, please check them off as you
review them:

[x] Commit messages follow commit message best practices
[x] Release notes block has been filled in or deleted (only if no user facing changes)

See the contribution guide for more details.
Double check this list of stuff that's easy to miss:

If you are adding a new binary/image to the cmd dir, please update
the release Task to build and release this image.

Reviewer Notes
If API changes are included, additive changes must be approved by at least two OWNERS and backwards incompatible changes must be approved by more than 50% of the OWNERS, and they must first be added in a backwards compatible way.
Release Notes

NONE

Question, do we also want to add some OWNERS file in some packages ? (note that can be done in follow-ups)

Question, do we also want to add some OWNERS file in some packages ? (note that can be done in follow-ups)

Question, do we also want to add some OWNERS file in some packages ? (note that can be done in follow-ups)

sure, will add that to the next steps

Question, do we also want to add some OWNERS file in some packages ? (note that can be done in follow-ups)

sure, will add that to the next steps


[IDEA] Auto queue

Auto queue will useful when you was kicked or boosting rank.
It will offer which gamemode will auto queue.

@danielkrupinski Concept:


Object prototype may only be an object or null

util.js:556
ctor.prototype = Object.create(superCtor.prototype, {
^
TypeError: Object prototype may only be an Object or null
at Function.create (native)
at exports.inherits (util.js:556:27)
at Object. (/home/username/.node_modules/lib/node_modules/webtorrent/node_modules/render-media/node_modules/videostream/mp4-remuxer.js:20:1)
at Module._compile (module.js:456:26)
at Object.Module._extensions..js (module.js:474:10)
at Module.load (module.js:356:32)
at Function.Module._load (module.js:312:12)
at Module.require (module.js:364:17)
at require (module.js:380:17)
at Object. (/home/username/.node_modules/lib/node_modules/webtorrent/node_modules/render-media/node_modules/videostream/videostream.js:6:18)
[1]+  Exit 8                  webtorrent magnet:?xt=urn:btih:73A3E1F9492BA6B0FA5AFB95FEEBA59C001ABF62
what i did is
webtorrent magnet_link

This is not an issue, sorry i was using an old node version.
after i installed the latest, i reinstall webtorrent, but now it stuck on fetching torrent metadata from 0 peers and then it exit.
am using the command as above

Don't use WebTorrent for piracy. We cannot help with such requests.


widget key pass through

I want to have an ability to directly refer by key to underlying widget in tests

Hi @vasilich6107,
As far as I understand given the description of the PR, you want to access underlying widgets by KEY just for Testing purposes.
Do you think it is weird to have a widget with 2 different keys (key and widgetKey)?

Do you know about any other widget (library or package) that has the same behavior? My concern is to confuse the users of the widgets.


chore: mention Sepolia changes in Docker Compose files

Make it clear what changes need to be done in order to run a Reth+Lighthouse setup on Sepolia using the provided Docker Compose files.

might be a good idea to add --network mainnet to the Lighthouse boot command to make it clearer that it's running on mainnet
The lighthouse docs also recommend to not rely on the default for the network flag, but to always specify a value

Yes. This would have helped me. I searched for mainnet and changed the two or three places I found, but if the existing yaml files had had --chain mainnet (for the docker-compose.yml) and --network mainnet for the lighthouse.yml that would have helped.
Maybe a very tight hint in the Reth book, "See the comments in the docker-compose files for information on running other chains" would have been helpful too.


Design feeds/notification for 'interesting events of the day'

Some things like meteor showers or ISS transit times should be easy to do.  Others might need some kind of feed.

To add to this issue, a general feed option of interesting things to see would be nice. An example that does that presently is the APOD app from NASA.

+1 to that.
On Sat, Apr 9, 2022 at 7:46 AM B. K. Oxley (binkley) <
@.***> wrote:

To add to this issue, a general feed option of interesting things to see
would be nice. An example that does that presently is the APOD app from
NASA.
—
Reply to this email directly, view it on GitHub
https://github.com/sky-map-team/stardroid/issues/468#issuecomment-1093950301,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AD5LWC5GVEDFFXSOIFANOFDVEFUYZANCNFSM5OZFJPVA
.
You are receiving this because you authored the thread.Message ID:
@.***>


Something's wrong

OS
Windows 10 x64
Godot Version
3.3.3
Plugin Version
1.0 (lastest)
Issue description

My code:
func _ready():
	# Creates a new DialogNode instance
	var dialog_node = DialogNode.instance()
	# Add the node as child
	add_child(dialog_node)
	# Shows the dialog node
	dialog_node.show()
	# Show an string. BBCode works too!
	dialog_node.show_text("Hello world")`

Steps to reproduce
1 create an empty project
2 add textalog as plugin
3 new scene
4 add code
5 run
Workaround
No response
Bug squad: Message edited to add a proper code format

Hey @CyberRedX did you tried to add DialogNode as child of a Node2D?
If that was the case, you need to:

Add it as child of a canvas layer
or
Resize it after adding it as child

Control and Node2D nodes are not compatible types, and mixing them usually leads this kinds of problems

I had the exact same issue. Your solution did work. I think this should be mentioned in the readme. Thank you!

After adding this to canvas it's working correctly, thanks!
And yeah, I too think this should be mentioned in the readme

@CyberRedX Please close this issue if its solved. If you like, can open an issue proposal to add this information in the readme (or leave it as is, I'll add it later in the new release)


Update README and docs for unified tagline

Use the unified tagline "A NumPy-compatible array library accelerated by CUDA" everywhere
Add link to Gitter chat room
Cleanup README

Jenkins, test this please.

Jenkins CI test (for commit d3224aa1701178a8c274f9aae74a52a4cfd919e1, target branch master) succeeded!

LGTM.

@asi1024 This pull-request is marked as to-be-backported, but the corresponding backport PR could not be found. Could you check?


1.10.4 Inconsistent

Thanks for all the work on 1.10.4 and the new Gmail. However, I am finding that it shows trimmed content on some of my emails but I still get the "show trimmed content" on some. What is even more confusing is that sometimes if I close and reopen the later emails, all the content is shown.

Thanks for the explanation. But, I have a fast cable internet connection and a decent speed computer. Further, before Gmail changed and using the prior version of Trimless I did not have this issue.


Support decorators +  other ES2016 syntax

Many people are already using babel and since it now supports decorators, class properties and more it would be great to support that.

Thanks for the request - this is similar to #1344. #1270. I got it working with babel by passing a version of it (modified babel-eslint) into the esprima option but it still has errors from what I've found so maybe soon.

See the esnext

Last time I checked, esnext option didn't include the ES2016 (ES7) new features like decorators or class properties.

If esprima doesn't support it, we can't too.
@hzoo could create a specific issue for your work?

Ok #1353


Allow AchilleEmbedded to be usable into docker

It seems that AchillesCassandraConfig configure AchilleEmbedded on localhost address.
This prevent using it into a docker container (with docker-compose).
What do you think of making these kind of parameters configurable?

Ok, I'll expose the listen_address & rpc_address as config param


Calling quit should always close the connection

Pull Request check-list

[x] Does npm test pass with this change (including linting)?
[x] Is the new or changed code fully tested?
[x] Is a documentation update included (if this change modifies existing APIs, or introduces new ones)?

Description of change
Calling quit should always end the connection and not end up reconnecting if called while the connection is down.
Fixes #212

Coverage decreased (-0.4%) to 99.291% when pulling 5b06caa946b7934827ac8b8acb5971301218754a on properly-quit into bf568b6df7560782215c2f91731761bb7151d10e on master.

Coverage remained the same at 100.0% when pulling 50fb343eeefdf3d9505b7e89d6c4d946319f1eb6 on properly-quit into 50f1663ba5a0632bfd9c47f25afd206738ca238b on master.

Coverage remained the same at 100.0% when pulling 8a8288cc8cad7003d99f1d236222b5ffed2e70be on properly-quit into 2e68a7a270b8bffe96fac296ee5e52438c783608 on master.

Coverage remained the same at 100.0% when pulling 9daef80b7bcd190cc02e4ba67bbae6dba4acdc12 on properly-quit into 3704ebdd9daa6e0bcb31ba9f5816c97b5edc71f7 on master.

Coverage remained the same at 100.0% when pulling 48481552c951a9edaf30a720df0950588cabd848 on properly-quit into 3704ebdd9daa6e0bcb31ba9f5816c97b5edc71f7 on master.


[QUESTION] TDLib local storage

@levlam I was just wondering if we can set the amount of data that TDLib caches/stores before it starts deleting old data? If yes, would you be able to point me on where I can do that? I know that we can enable the storage optimizer through the tdlibParameters or through TDLib options(use_storage_optimizer) which will delete "old" data automatically, but the Docs doesn't mention how big the data is before TDLib starts removing stuff. Reading from the C++ source code I think TDLib schedule its own GC to remove stuff everyday or at least GC runs with a day of interval. Is that right?
Thanks

@chairus Yes, GC is automatially run once a day using default parameters, which can be found there and not after some limit expires. If you need more strict control over the storage, you can from time to time use getStorageStatisticsFast to quickly get size of TDLib's cache and run the method optimizeStorage if needed.

@levlam Thanks!


mulaw encdoing

Hi bshall,
I have doubt about the mu-law encoding function. I wondered why the  function mulaw_encode()  returns np.floor((fx + 1) / 2 * mu + 0.5) instead of fx directly.

Hi @maximilianchen,
Sorry about the delay. If you look at the definition on wikipedia you'll see that the range of fx is between -1 and 1. So what we are actually doing is mu-law encoding and then quantizing the result. That's where the next step comes from. I hope that helps.


transform into bundle (concept)

This pull request is a concept, how to make wsse plugin optional in guzzle bundle.
Read more: https://github.com/8p/GuzzleBundle/pull/129
Todo:

[ ] add GuzzleBundle in requirements in composer.json
[ ] rename GuzzleWsseBundle into EightPointsGuzzleWsseBundle
[ ] tests
[ ] travis build

I think we should split that. So the plugin can stay and will be integrated by a new EightPointsGuzzleWsseBundle. Because than the plugin can also be used directly by a plain Guzzle integration. What do you think, @gregurco?

I'm agree with you but I had only one source to do a PR and I did it here :) So I think you can create new repository and I will do PR to new repository. Did I correctly understand you?

True. What do you think about creating this repo by your user/organization. It doesn't have to be in @8p. Just an idea. But you put so much time in this features maybe this would be nice for you to have this "hosted" by your own. If not, I can also create a new repo, sure. Just tell me what you prefer 👍

Yep, I can create and maintain it. As soon as I will create it, I will post the link here.
Thank you for this possibility :)

@florianpreusner hello! Yesterday I made a commit in GuzzleBundle. I added interface and made some additional changes. Also I created new repository with plugin and implementation of newly created interface. You can see code there: https://github.com/gregurco/EightPointsGuzzleWssePlugin/tree/init
Do you have any remarks or suggestions? For example about repository name, namespace, implementation or something another?
If all is ok, then I can continue with testing of new implementation, writing phpunit tests and documentation :)
Thanks.

Hi @gregurco! Nice!

[ ] Why not calling it gregurco/guzzle-bundle-wsse-plugin? I think we should make it clear that this is a plugin for a bundle and not a plugin for guzzle itself. The name of the repo should also change of course. But this is just a suggestion.

[ ] In composer.json some urls are wrong: "https://github.com/gregurco/EightPointsGuzzleWssePlugin/contributors", ...

👍  I like it a lot :)

@florianpreusner thank you, it's good idea :+1: just renamed repository, package name and fixed links. Tomorrow I will start to test and I will write when it will be ready :)

I closed this PR. Let's continue discussion in https://github.com/8p/GuzzleBundle/pull/129


Unintuitive styling of fields after Binder setBean

When Binder setBean is called with an empty bean while validatorsDisabled  is true:

If the field is a TextField bound asRequired, it will be cleared and it will be styled as valid.
If the field is a Select bound asRequired it will be cleared and it will be styled as invalid.

Even while Binder ignores the validation (because of validatorsDisabled), vaadin-select will do its own client-side validation and style the field as invalid, while TextField won't.
Example code
@Route
public class HelloWorldView extends HorizontalLayout {
    private TextField name = new TextField();
    private Select<QueueType> type = new Select<>(QueueType.values());
    final Binder<MyBean> binder = new Binder<>(MyBean.class);

    public HelloWorldView() {

        binder.forField(name).asRequired().bind("name");
        binder.forField(type).asRequired().bind("type");
        binder.setBean(new MyBean());

        Button clear = new Button("Clear");
        clear.addClickListener(event -> {
            binder.setValidatorsDisabled(true);
            binder.setBean(new MyBean());
            binder.setValidatorsDisabled(false);
        });

        VerticalLayout layout = new VerticalLayout();
        layout.add(name);
        layout.add(type);
        layout.add(clear);

        add(layout);
    }
}

** Instructions to reproduce and observed behavior **

Navigate to the view and fill values in the fields

Then press "Clear"

Now press "Clear" again

Expected behavior
After pressing clear once it should have displayed (3)
Workaround
Explicitely mark the field as valid after setting the value.
binder.setBean(new MyBean());
type.getElement().executeJs("this.invalid=false");

Fixed by https://github.com/vaadin/flow-components/pull/973


CodeFlow: unable to work with npm package esm

Describe the bug
I use the esm package to support running ES module in lower versions of Node.js. It seems that this package doesn't play well with WebContainer.
Link to the blitz that caused the error
https://stackblitz.com/~/github.com/Justineo/codeflow-esm-repro
Steps to reproduce

Go to the reproduction link (CodeFlow).
An error occurred when running pnpm install (I have a prepare script so that it will run automatically after install).

Expected behavior
No error occurred.
Parity with Local

[X] I have run the project in my local machine and I could not reproduce the issue.

Screenshots
No response
Platform
Version = 1.67.0
Hash = 35b869fee6c097380f8b2e33563ab4a16c08f57b
WebContainer = ea2f6b48b77806aba8e6fe3b6311530cc7674f64

Browser name  = Chrome
Full version  = <IP_ADDRESS>
Major version = 106
navigator.appName = Netscape
navigator.userAgent = Mozilla/5.0 (Macintosh; Intel Mac OS X 10_15_7) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/<IP_ADDRESS> Safari/537.36
performance.memory = {
  "totalJSHeapSize": 81512630,
  "usedJSHeapSize": 77705602,
  "jsHeapSizeLimit":<PHONE_NUMBER>
}

Additional context
No response

Hey 👋 Thanks for the report. There are indeed some issues with the esm package that we are aware of and that yet need to be fixed. There's a similar issue here #691.


Checking registration data for existing names

While WCA live is working great overall, WRT is experiencing more potential false newcomers compared to cubecomps times. A likely reason for this is that cubecomps used to have a way to check for existing names before or during a competition that allowed delegates and organizers to clarify possible matches right at the competition. But with WCA live, this check only happens during result submission when it is too late to simply approach people at the competition.
One way to improve this situation (at least for competitions with WCA registration) is enabling delegates and organizers to perform existing name checks on the registration data of their competitions.

@dunkOnIT I think this belongs to the main repo. The name check makes more sense in the registration system, so it can be addressed early. I don't think there's anything actionable on WCA Live side.

re-sync to WCA live

@timreyn I don't think syncing in WCA Live is necessary, because the results don't change. Let me know if I miss something. Obviously I agree that it is too many steps, eventually we should automate the whole results upload, since we effectively control all the related tooling.


[Bug]: LLM.generate() collapse with some padding side

Your current environment
Versions of relevant libraries:
[pip3] numpy==1.26.4
[pip3] nvidia-nccl-cu12==2.20.5
[pip3] torch==2.3.0
[pip3] triton==2.3.0
[pip3] vllm_nccl_cu12==<IP_ADDRESS>.4.0
[conda] Could not collectROCM Version: Could not collect
Neuron SDK Version: N/A
vLLM Version: 0.4.2

🐛 Describe the bug
Padding the input with some side seems to causes the generation by the LLM to break down. In the examples, I have only run this for one model, but the same phenomenon occurs with other models as well.
Reasonable generation results can be obtained by not padding the input, but I would greatly appreciate it if you could explain the cause of this issue.
from __future__ import annotations

from contextlib import contextmanager

from transformers import AutoTokenizer, PreTrainedTokenizer
from vllm import LLM, SamplingParams

model_name = "Qwen/Qwen1.5-7B-Chat"
tokenizer: PreTrainedTokenizer = AutoTokenizer.from_pretrained(model_name)
llm = LLM(model_name)
print("Original padding side: ", tokenizer.padding_side)

prompt1 = "Give me a short introduction to large language model."
# prompt2 is short, so padded.
prompt2 = "Hello"
messages = [
    [{"role": "system", "content": "You are a helpful assistant."}, {"role": "user", "content": prompt1}],
    [{"role": "system", "content": "You are a helpful assistant."}, {"role": "user", "content": prompt2}],
]
text = tokenizer.apply_chat_template(messages, tokenize=False, add_generation_prompt=True)

@contextmanager
def set_temporal_padding_side(tokenizer: PreTrainedTokenizer, padding_side: str):
    """Temporarily set the padding side of the tokenizer.

    Useful when the padding side of the tokenizer is unknown or needs to be overridden.
    """
    original_padding_side = tokenizer.padding_side
    tokenizer.padding_side = padding_side
    try:
        yield
    finally:
        tokenizer.padding_side = original_padding_side

# generation with padding on given side.
def generate_with_padding_side_specified(side):
    with set_temporal_padding_side(tokenizer, side):
        model_inputs = tokenizer(
            text,
            return_tensors="pt",
            padding=True,
        )

    vllm_outputs = llm.generate(
        prompt_token_ids=model_inputs.input_ids.tolist(),
        sampling_params=SamplingParams(),
        use_tqdm=False,
    )
    return tokenizer.decode(vllm_outputs[1].outputs[0].token_ids)

left_pad_generated_texts = generate_with_padding_side_specified("left")
right_pad_generated_texts = generate_with_padding_side_specified("right")

print("Output with left padding:", left_pad_generated_texts)
print("Output with right padding:", right_pad_generated_texts)

I got the following output:
Original padding side:  right
Output with left padding: Hello! How can I assist you today?<|im_end|>
Output with right padding: 圜 diplomácii byly posíleny každoročn

Hi @kevin3314 it is not necessary to pad the inputs because vLLM can handle batches with different lengths with continuous batching. When you are producing padding on the right-side and passing in the prompt_token_ids directly, there is no context for vLLM that the token_ids have padding and thus should have an edited attention_mask. So it will simply try to continue generating using those padding tokens as being part of the prompt.
Because of this, I would recommend simply passing a list of tokenized prompts without them being padded. vLLM will be able to batch them together.


Translation for Portuguese [Translate new messages]

DO NOT REMOVE THIS
Please make sure your pull request complies with these guidelines:

[ ] Use same formatting

[ ] Must test on PMMP

[ ] PLEASE don't use the GitHub Web Editor

[ ] Have a detailed title like "Fix players are being kicked randomly"

If you are translating, make sure you translation complies with these guidelines:

[ ] File must be named lang_iso. ISO Codes for languages can be found here.

[ ] PLEASE don't use Google Translate (or any other translators...)

[ ] Translate messages but keep tags like {player} or {pin} the same

[ ] Make a Pull Request named Translation for Language after you're finished

What does the PR change?
Translates the new messages added in by the last commit.
Extra Information
The pull request is ready to merge.

Thank you!


Add concurrency option

Similar to p-map, it would be nice to have this built-in rather than have to use p-limit for instance.

How would this even work? Reduce depends on the last value. If it doesn't, then why do you need reduce?

How would this even work? Reduce depends on the last value. If it doesn't, then why do you need reduce?

One potential use-case would be when the values can be determined asynchronously and independent from each other. Currently this can mostly be archived with  pMapIterable, however, that requires a for loop and thus cannot easily be used in e.g. an expression.

Or in other words: pReduce should be able to accept not only Iterable<Promise|any> but also AsyncIterator<any>. This way one could write pReduce(pMapIterable(values, mapper), reducer, initialVal)


*: update some dependencies

Signed-off-by: koushiro<EMAIL_ADDRESS>What problem does this PR solve?
update some dependencies
Firstly, waiting for tikv/yatp#42 to be merged
(I will use the merged commit instead of my fork version later)
Related changes
Check List
Tests

No code

Side effects
Release note

/run-all-tests

/run-all-tests

/run-all-tests

/run-all-tests

/run-all-tests

/run-all-tests

/cc @zhouqiang-cl

/merge


Add utxos-(chaincode-decoded)-episode-5-of-the-chaincode-podcast to chaincode-labs/chaincode-podcast

This PR adds UTXOs (Chaincode Decoded) - Episode 5 of the Chaincode Podcast transcript to the chaincode-labs/chaincode-podcast directory.

did feedback on https://cryptpad.fr


Redundant call to builder.Services.AddOptions()

Description
The section Expose the authentication state as a cascading parameter mentions client-side apps should include a call to builder.Services.AddOptions() in the Program file. This seems unnecessary since builder.Services.AddAuthorizationCore() already includes a call to AddOptions().
Page URL
https://learn.microsoft.com/en-us/aspnet/core/blazor/security/?view=aspnetcore-8.0
Content source URL
https://github.com/dotnet/AspNetCore.Docs/blob/main/aspnetcore/blazor/security/index.md
Document ID
1247ee64-fbc9-ce29-0710-f6e59380f5ee
Article author
@guardrex

Thanks @BassFaceIV ... I'll take a look early next week, hopefully on Monday.

Hello @BassFaceIV ... Thanks for the report. It seems to be a point-in-time situation ...

3.1: ❌ Not included by default
5.0 or later: ✔ Included by default

I'll version it out for 5.0 or later.


Using Yarn 2 (Berry): "Error: Query root type must be provided."

yarn set version berry && yarn install && yarn graphql-codegen fails. I don't have a more specific repro right now but I'd be surprised if it worked for anyone. Ubuntu 20.10, latest versions of graphql-code-generator packages.

I'm going to close this and reopen once I have a better understanding of what's going on. I think I'm mingling two different bugs here.


Problem running docker "open config.yaml: no such file or directory"

When using the command:
docker run --rm
-e EGRESS_CONFIG_FILE=/out/config.yaml
-v ~/livekit-egress:/out
livekit/egress
I get the error:
"+ mkdir -p /tmp/lkegress

exec egress
open /config.yaml: no such file or directory"

For that reason I can't use or test egress

I could not reproduce the issue above:
docker run --rm \
-e EGRESS_CONFIG_FILE=/out/config.yaml \
-v ~/go/src/github.com/livekit/egress/test:/out \
livekit/egress
+ export TMPDIR=/tmp/lkegress
+ TMPDIR=/tmp/lkegress
+ pulseaudio -D --verbose --exit-idle-time=-1 --system --disallow-exit
W: [pulseaudio] main.c: Running in system mode, but --disallow-module-loading not set.
N: [pulseaudio] main.c: Running in system mode, forcibly disabling SHM mode.
I: [pulseaudio] main.c: Daemon startup successful.
+ '[' -z /tmp/lkegress ']'
+ mkdir -p /tmp/lkegress
+ exec egress
{"level":"info","ts":1665016206.7505462,"logger":"egress","caller":"redis/redis.go:53","msg":"connecting to redis","nodeID":"NE_mYWBmTNqsiC7","sentinel":false,"addr":"<IP_ADDRESS>:6379"}
{"level":"debug","ts":1665016206.7529986,"logger":"egress","caller":"service/service.go:68","msg":"starting service","nodeID":"NE_mYWBmTNqsiC7"}
{"level":"debug","ts":1665016206.7538588,"logger":"egress","caller":"service/service.go:93","msg":"service ready","nodeID":"NE_mYWBmTNqsiC7"}
^C{"level":"info","ts":1665016208.4425173,"logger":"egress","caller":"server/main.go:103","msg":"exit requested, stopping recording and shutting down","nodeID":"NE_mYWBmTNqsiC7","signal":"interrupt"}
{"level":"info","ts":1665016208.4427583,"logger":"egress","caller":"service/service.go:98","msg":"shutting down","nodeID":"NE_mYWBmTNqsiC7"}

open open /config.yaml is unexpected here. What version of the server are you using?

I am using ubuntu 18, it is blank, it has nothing.

I am using the command:
"docker run --rm -e EGRESS_CONFIG_BODY="$(cat config.yaml)" -v /root/record:/out livekit/egress"
However I get the following error:

export TMPDIR=/tmp/lkegress
TMPDIR=/tmp/lkegress
pulseaudio -D --verbose --exit-idle-time=-1 --system --disallow-exit
W: [pulsaudio] main.c: Running in system mode, but --disallow-module-loading not set.
N: [pulsaudio] main.c: Running in system mode, forcibly disabling SHM mode.
I: [pulsaudio] main.c: Daemon startup successful.
'[' -z /tmp/lkegress ']'
mkdir -p /tmp/lkegress
exec egress
{"level":"info","ts":1665045471.3235445,"logger":"egress","caller":"redis/redis.go:53","msg":"connecting to redis","nodeID" :"NE_bjdCZqenEuLW","sentinel":false,"addr":"localhost:6379"}
unable to connect to redis: dial tcp <IP_ADDRESS>:6379: connect: connection refused

It does not connect to redis-server, I have tested redis with:
redis-cli -h localhost -p 6379 ping
The result is:
PONG
So redis is working perfectly.

This is because the redis server runs on the host and not the container, so is not accessible over localhost for the egress server.
https://github.com/livekit/egress/blob/main/README.md has further documentation on how to configure redis and egress when running locally. Note that <IP_ADDRESS> should be used on linux instead of <IP_ADDRESS>.

Configure Redis to work with ip address <IP_ADDRESS> and restart the service.
Then connect the Egress with <IP_ADDRESS> (local) or use the IP if you have Redis on the same network share.
I hope it helps you :)


fix: unexpected return in uploadAssets

Ah damn, I'm blind 🤣

@FabianLars Can you release a patch version for it? https://github.com/tauri-apps/tauri-action/pull/979

i'm back tomorrow, will do it then.


support different Pinecone initialization depending on the version

Added changes to support the new version of pinecone-client.
The initialization process has changed from the module level to the class level.
Note: Do not merge this PR

LGTM!

@DosticJelena plz hold off on merging this if possible -- I changed a few things around because of how we have to unit test the dynamic import of pinecone in the PineconeVectorStore class + removed some unnecessary if/then statements (IMO). Check out my draft PR here: https://github.com/aulorbe/llama_index/pull/2
That draft PR ^ also has unit tests for this change ✅


Custom game path ending with <storeUserId> includes <storeUserId>'s parent directory

Ludusavi version
v0.19.0
Operating system
Windows
Installation method
Scoop
Description
I've added a single path for a custom game, a game that doesn't exist in the manifest (i.e. my custom game entry doesn't override anything), and the path I've defined ends in .../<storeUserId>. The path I'm trying to add ends in a folder that is 17 numeric characters, so I assume that this is a <storeUserId>.
When previewing this path in the Backup Mode tab, the files found are those under <storeUserId> and those in <storeUserId>'s parent (i.e. items at the same hierarchy level as <storeUserId>). Only files under <storeUserId> should be detected by the preview.
Changing the custom path to be the game data's literal path (i.e. with the actual 17 numbers), then Ludusavi correctly detects files within that path, and only that path.
Thanks.
Logs
No response

Related discussion: https://github.com/mtkennerly/ludusavi/issues/39
Right now, <storeUserId> is regrettably just interpreted as *, because I'm not sure if it's feasible to account for all possible ID patterns. For example, one comment in the other ticket noted that, within a Steam root, GTA 4 used some other pattern for its ID rather than a typical Steam pattern.
Maybe we could have an option to enable stricter matching of known patterns, but there will be a risk of missing saves. For the default configuration, I think it's better to find too many files than too few, since at least it gives the user a chance to deselect them instead of miss them entirely.

Right now, <storeUserId> is regrettably just interpreted as *

This shouldn't mean that the files outside of the path should be matched, right?
For a hierarchy of /a/b/ with custom path /a/<storeUserId>/ i.e. /a/*/, only items in /a/b/ should be matched. Currently items inside /a are being matched. I see the issue now:  I was presuming a/*/ with a trailing slash would only match folders, but it probably does make sense to match any items, not just folders, with or without a trailing slash.

Maybe we could have an option to enable stricter matching of known patterns, but there will be a risk of missing saves.

Maybe the solution to varying <storeUserId> values would be to have a strict set of known patterns by default, and when a pattern can't be unambiguously matched, allow the user to pick which folder it is on backup and for Ludusavi to remember that value going forward. If the tracked value changes, ask the user to pick again. If a value was needed to be tracked by Ludusavi, then on restore, if the path already exists on the system, use that, else ask to pick a different folder as the stand-in for <storeUserId>. If this idea would better as alternate behaviour, have a toggle to enable strict behaviour for those that want to backup exactly as specified by the manifest. This was just my first idea, maybe there's a better way.

I see the issue now: I was presuming a/*/ with a trailing slash would only match folders, but it probably does make sense to match any items, not just folders, with or without a trailing slash.

Yeah, Ludusavi strips trailing slashes, mainly for the non-wildcard case since /a and /a/ will normally find the same results. I could see an argument that it shouldn't strip the slash when preceded by *, but if you really do just want to match in subfolders, then you can specify /a/*/* instead.
I should probably add a README section that covers this.

when a pattern can't be unambiguously matched, allow the user to pick which folder it is

The tricky thing is that, if we can't find the path, is it because we have the wrong ID pattern or just because the game isn't installed?  We could narrow it down to cases where we found some saves for a game but no matches for a <storeUserId> path, but then we also need to be very careful about not warning for potentially irrelevant paths (e.g., for a different OS or store). We might also have false positives if a game removes a <storeUserId> file on uninstall, but leaves some others.
It's hard 😅 I can definitely add an opt-in set of known patterns, but the alerting is much more complex. Also, because of how, for example, Steam games can require other launchers or account systems with their own user IDs, it will probably need to check all patterns for all root types (excluding other, which would continue using *).


Use lookup table instead of switch for selecting byte parser.

While Go may compile a packed switch into a lookup table, it doesn't seem to here maybe because it's too many cases, or because of the non-singular case statements.
Just small in the large scope of things but removed some noise from the cpu profile
After
goos: darwin
goarch: arm64
pkg: github.com/corazawaf/libinjection-go
BenchmarkSQLiDriver
BenchmarkSQLiDriver/sqli
BenchmarkSQLiDriver/sqli-10         	   20707	     56244 ns/op
BenchmarkSQLiDriver/folding
BenchmarkSQLiDriver/folding-10      	    9721	    118566 ns/op
BenchmarkSQLiDriver/tokens
BenchmarkSQLiDriver/tokens-10       	    5677	    204501 ns/op
PASS

Before
goos: darwin
goarch: arm64
pkg: github.com/corazawaf/libinjection-go
BenchmarkSQLiDriver
BenchmarkSQLiDriver/sqli
BenchmarkSQLiDriver/sqli-10         	   19716	     57947 ns/op
BenchmarkSQLiDriver/folding
BenchmarkSQLiDriver/folding-10      	    9536	    118126 ns/op
BenchmarkSQLiDriver/tokens
BenchmarkSQLiDriver/tokens-10       	    5713	    206581 ns/op
PASS

It looks good to me. My only question would be if itsn't better to generate such map with a go script and run the go generate to have this statically.

I agree with @jcchavezs but we should handle that in other issue
Also we could run some automatic optimization, like:
switch
case parse >= 0 && parse < 33:
  return parseWhite(s)


Failed to build bookie server

Describe the bug
Bookie pod CrashLoopBackOff and cannot restart
To Reproduce
Steps to reproduce the behavior:

04:04:26.014 [main] ERROR org.apache.bookkeeper.server.Main - Failed to build bookie server
org.apache.bookkeeper.bookie.BookieException$InvalidCookieException: Cookie [4
bookieHost: "<IP_ADDRESS>:3181"
journalDir: "data/bookkeeper/journal"
ledgerDirs: "1\tdata/bookkeeper/ledgers"
instanceId: "7940c93b-5da8-4fa7-941e-d254d678fb1c"
] is not matching with [4
bookieHost: "<IP_ADDRESS>:3181"
journalDir: "data/bookkeeper/journal"
ledgerDirs: "1\tdata/bookkeeper/ledgers"
instanceId: "7940c93b-5da8-4fa7-941e-d254d678fb1c"
]
at org.apache.bookkeeper.bookie.Cookie.verifyInternal(Cookie.java:136) ~[org.apache.bookkeeper-bookkeeper-server-4.10.0.jar:4.10.0]
at org.apache.bookkeeper.bookie.Cookie.verify(Cookie.java:147) ~[org.apache.bookkeeper-bookkeeper-server-4.10.0.jar:4.10.0]
at org.apache.bookkeeper.bookie.Bookie.verifyAndGetMissingDirs(Bookie.java:369) ~[org.apache.bookkeeper-bookkeeper-server-4.10.0.jar:4.10.0]
at org.apache.bookkeeper.bookie.Bookie.checkEnvironmentWithStorageExpansion(Bookie.java:432) ~[org.apache.bookkeeper-bookkeeper-server-4.10.0.jar:4.10.0]
at org.apache.bookkeeper.bookie.Bookie.checkEnvironment(Bookie.java:250) ~[org.apache.bookkeeper-bookkeeper-server-4.10.0.jar:4.10.0]
at org.apache.bookkeeper.bookie.Bookie.(Bookie.java:688) ~[org.apache.bookkeeper-bookkeeper-server-4.10.0.jar:4.10.0]
at org.apache.bookkeeper.proto.BookieServer.newBookie(BookieServer.java:136) ~[org.apache.bookkeeper-bookkeeper-server-4.10.0.jar:4.10.0]
at org.apache.bookkeeper.proto.BookieServer.(BookieServer.java:105) ~[org.apache.bookkeeper-bookkeeper-server-4.10.0.jar:4.10.0]
at org.apache.bookkeeper.server.service.BookieService.(BookieService.java:41) ~[org.apache.bookkeeper-bookkeeper-server-4.10.0.jar:4.10.0]
at org.apache.bookkeeper.server.Main.buildBookieServer(Main.java:301) ~[org.apache.bookkeeper-bookkeeper-server-4.10.0.jar:4.10.0]
at org.apache.bookkeeper.server.Main.doMain(Main.java:221) [org.apache.bookkeeper-bookkeeper-server-4.10.0.jar:4.10.0]
at org.apache.bookkeeper.server.Main.main(Main.java:203) [org.apache.bookkeeper-bookkeeper-server-4.10.0.jar:4.10.0]
Expected behavior
A clear and concise description of what you expected to happen.
Desktop (please complete the following information):

OS:
No LSB modules are available.
Distributor ID:	Ubuntu
Description:	Ubuntu 18.04 LTS
Release:	18.04
Codename:	bionic

pulsar version: 2.5.2

@bleacha870721 How did you deploy the Pulsar cluster? Bookies require a stable network identifier for bookie id. So you need to set useHostNameAsBookieID to be true.
See Pulsar Helm Chart: https://github.com/apache/pulsar-helm-chart/blob/master/charts/pulsar/templates/_bookkeeper.tpl#L84

I think this issue is fixed in v2.6.0
The condition check PF_ENV_PREFIX is missed when update the env values in env config files. Now it has fixed v2.6.0
https://github.com/apache/pulsar/blob/master/docker/pulsar/scripts/apply-config-from-env.py

I agree with @sijie, this is a configuration problem and not a bug.

Closed as stale and answered.
Feel free to open a new issue if it's still relevant to the maintained versions.

same issue here, after a crash of a node, with a destroyed pvc ....
had to delete the pvc and the pv, and this leads to this issue (at least here)
how to solve?
2024-07-17T10:10:23,047+0000 [main] ERROR org.apache.bookkeeper.bookie.LegacyCookieValidation - There are directories without a cookie, and this is neither a new environment, nor is storage expansion enabled. Empty directories are [/pulsar/data/bookkeeper/journal/current, /pulsar/data/bookkeeper/ledgers/current] 2024-07-17T10:10:23,048+0000 [main] ERROR org.apache.bookkeeper.server.Main - Failed to build bookie server org.apache.bookkeeper.bookie.BookieException$InvalidCookieException: 	at org.apache.bookkeeper.bookie.LegacyCookieValidation.checkCookies(LegacyCookieValidation.java:113) ~[org.apache.bookkeeper-bookkeeper-server-4.16.5.jar:4.16.5] 	at org.apache.bookkeeper.server.EmbeddedServer$Builder.build(EmbeddedServer.java:408) ~[org.apache.bookkeeper-bookkeeper-server-4.16.5.jar:4.16.5] 	at org.apache.bookkeeper.server.Main.buildBookieServer(Main.java:277) ~[org.apache.bookkeeper-bookkeeper-server-4.16.5.jar:4.16.5] 	at org.apache.bookkeeper.server.Main.doMain(Main.java:216) ~[org.apache.bookkeeper-bookkeeper-server-4.16.5.jar:4.16.5] 	at org.apache.bookkeeper.server.Main.main(Main.java:199) ~[org.apache.bookkeeper-bookkeeper-server-4.16.5.jar:4.16.5]


Set TypeScript composite: false

The composite and project references settings in TypeScript are used, among other things, to speed up build times when the project has already been built previously. Unfortunately, the generated tsbuildinfo file is either incorrect or outdated and causing subsequent builds to fail. The only way to get ourselves out of this state is to delete the tsbuildinfo file, which defeats the purpose of the composite/project references settings.
This PR turns off composite and project references to address the local build issues.

Codecov Report

Merging #6494 (e623a06) into dev (81d34b4) will decrease coverage by 2.11%.
Report is 322 commits behind head on dev.
The diff coverage is 60.52%.

Flag
Coverage Δ

*Carryforward flag

msal-angular
?

msal-browser
85.34% <ø> (-1.13%)
:arrow_down:
Carriedforward from bc3840d

msal-common
83.73% <ø> (-0.81%)
:arrow_down:

msal-core
?

msal-node
80.24% <ø> (-3.15%)
:arrow_down:

msal-node-extensions
68.25% <60.26%> (-7.39%)
:arrow_down:
Carriedforward from bc3840d

msal-react
94.24% <ø> (-0.45%)
:arrow_down:
Carriedforward from bc3840d

node-token-validation
?

*This pull request uses carry forward flags. Click here to find out more.

Files Changed
Coverage Δ

...ions/msal-node-extensions/src/utils/Environment.ts
20.51% <0.00%> (-22.08%)
:arrow_down:

...b/msal-browser/src/app/IPublicClientApplication.ts
36.66% <ø> (-7.09%)
:arrow_down:

...ib/msal-browser/src/app/PublicClientApplication.ts
65.90% <ø> (-29.42%)
:arrow_down:

...er/src/broker/nativeBroker/NativeMessageHandler.ts
86.29% <ø> (+0.33%)
:arrow_up:

lib/msal-browser/src/cache/AsyncMemoryStorage.ts
80.76% <ø> (+2.99%)
:arrow_up:

lib/msal-browser/src/cache/BrowserCacheManager.ts
89.47% <ø> (-2.25%)
:arrow_down:

lib/msal-browser/src/cache/BrowserStorage.ts
90.00% <ø> (-0.91%)
:arrow_down:

lib/msal-browser/src/cache/CryptoKeyStore.ts
80.00% <ø> (-3.34%)
:arrow_down:

lib/msal-browser/src/cache/DatabaseStorage.ts
34.00% <ø> (-4.54%)
:arrow_down:

lib/msal-browser/src/cache/MemoryStorage.ts
100.00% <ø> (ø)

... and 55 more

... and 180 files with indirect coverage changes


HHH-6475 Making sure that embeddable attributes will be available in …

…metamodel of mapped super classes

Applied upstream


Enabling filters feels clumsy on mobile

Description

The UX flow of enabling filters on mobile needs to improve. The action of Applying filters needs to be more explicit.
First reported by @panchovm
Reproduction

Go to search results, open the filters, tap on some filters, close the menu.

That UX flow doesn't feel solid as the "applying filters" action is not very explicit

Additional context

Resolution

[ ] 🙋 I would be interested in resolving this bug.

This report needs a design issue. I will add the needs design label to create the design issue.

Removing the aspect: design label so we can create a design issue to tackle filter area improvements.

Removing needs design label as #1669 address this problem.


[release-1.3] Cherry-pick memory leak fixes into 1.3

For release-1.3
As mentioned in #4271 and in the cert-manager biweekly meeting on 28/07, this PR cherrypicks two commits from #4233 which fix memory leaks.
The two commits are taken from:

https://github.com/jetstack/cert-manager/commit/eb947f98eaea83cc5b660f805daba2b6f109502b
https://github.com/jetstack/cert-manager/commit/8e872632f4a1ab7536ade7163ff9d9c21ee01ed1

/kind bug
Fixed a goroutine leak that was causing the controller's memory usage to grow with time

/lgtm


Undefined method error for the tracked owner in a has-many-through association

I've setup a has-many-through association between an authors and books. When a new book is created under the ownership of the artist it should be tracked. The problem occurs when trying to setup the tracked owner to the author. In a one-to-many this obviously would work. But I'm assuming since the :author is now more than one an error is thrown. Does PublicActivity support has-many-through associations? If so what would be the best approach?
class Author < ActiveRecord::Base
    has_many :authorships
    has_many :books, through: :authorships
  end

class Book < ActiveRecord::Base
  include PublicActivity::Model
  tracked owner: :author

  has_many :authorships
  has_many :authors, through: :authorships
end

Not a bug in p_a. Answered in your SO question: http://stackoverflow.com/questions/29419173/using-a-many-to-many-relationship-with-the-public-activity-gem-in-rails/29421554#29421554


Always emit comments in declaration files

With compiler options "declaration": true and "removeComments": true, the emitted .d.ts is currently stripped of comments.
I'd like to propose .d.ts files always be emitted with comments.
Since .d.ts files are important for documentation purposes only, and since file-size is of no concern, I believe there is no meaningful use-case for stripping the documentation from the emitted file.
Alternatively, a new compiler option "removeDeclarationComments" could be introduced, and should default to true - in the marginal case where somebody does want an uncommented .d.ts, this would still be possible, although, personally, I can't think of any use-case.
Note that the current behavior leads to production build scripts, in which we need to run the compiler twice - first with "declaration": false, "removeComments": true for the actual code, and then again with "declaration": true, "removeComments": false for the .d.ts file.

Duplicate of #14619.


Attribute label changes may break imported searches

The migration from ELK/Logsearch to OpenSearch included a change in the backend log aggregator, and some field labels have changed. For example,logmessage.message_type is now @level.  This has broken some imported searches and visualizations, as show below.

We will be updating this issue with identified differences. Many of these will not be fixed between the two systems, as they're inherent to the updated technology.

I don't think this is a case of a label change.
I think logmessage.message_type and logmessage.event_type are fields that existed in Kibana when logs were ingested from the Loggregator Firehose, but no longer exist when ingesting logs into Kibana from an aggregate drain.
The @level field exists in both systems, though interestingly, seems to have been set incorrectly for error logs in Kibana, while in OpenSearch @level: ERROR for logs with errors.
If anything, we may just want to note that the logmessage.* fields no longer exist

Yeah, a raw log from Kibana looks like:
<6> 2024-12-10T03:17:01Z c0ddec80-0c93-4566-9e6c-2e61bf65f5b6 doppler[29]: {"cf_app_id":"abc123","cf_app_name":"fake-app","cf_org_id":"abc123","cf_org_name":"org-
1","cf_origin":"firehose","cf_space_id":"abc123","cf_space_name":"space-1","deployment":"cf-
production","event_type":"LogMessage","ip":"<redacted>","job":"diego-
cell","job_index":"abc123","level":"info","message_type":"ERR","msg":"ERROR - {\"app_metadata\":{\"b3\":\"abc123-
abc123\",\"x_b3_traceid\":\"abc123\",\"x_b3_spanid\":\"abc123\"},\"app_message\":\"fake app
error\"}","origin":"rep","source_instance":"3","source_type":"APP/PROC/WEB","time":"2024-12-
10T03:17:01Z","timestamp":1733800620813807013}

Notice the event_type and message_type fields.
And a raw log from OpenSearch looks like:
957 <11>1 2024-12-10T03:17:00.813807+00:00 org-1.fake-app.fake-app abc123 [APP/PROC/WEB/3] - [tags@47450
app_id="abc123" app_name="fake-app" deployment="cf-production" index="abc123" instance_id="3" ip="<redacted>"
job="diego-cell" organization_id="abc123" organization_name="org-1" origin="rep" process_id="abc123"
process_instance_id="abc123" process_type="web" source_id="abc123" source_type="APP/PROC/WEB"
space_id="abc123" space_name="space-1"] ERROR - {"app_metadata":{"b3":"abc123-
abc123","x_b3_traceid":"abc123","x_b3_spanid":"abc123"},"app_message":"fake app error"}

Notice that there is no event_type or message_type fields on the raw message.
Thus, the logmessage.event_type and logmessage.message_type fields simply do not exist in OpenSearch because the source data no longer exists on the raw messages that OpenSearch is ingesting.


change to match GitHub org rename

changing StediInc > Stedi, the rest is just my linter

It's very opinionated.


Motion tweaking

Several small issues with Motion/Control components

Fix bugs with NewDirection (and angular version)

Fixes a rounding issue when calculating the sign of negative fractions
Triggers NewDirection in the frame where the direction changes, instead of the next frame
Renames NewRotation to NewAngularDirection and fixes the same issues
Adds/fixes related tests

Sync motion with clock rather than number of frames

Instead of adjusting the speed based on the frame rate, try to saty in sync with the game clock by using frame.dt.  Note that speeds are in pixels per second.
This is a breaking change, because it changes how we interpret speeds.  (Pixels per second instead of pixels per frame.)

Since it's a breaking change, I needed to fix several tests.  I found the existing tests very hard to follow, so I added a lot of clarification, and removed some that seemed redundant.
Change Jumpway -> Jumper
Reword motion docs

As mentioned, this does have a breaking change: it changes speeds from pixels per frame to pixels per second.
I think this is a more natural definition in most cases -- I often had to choose very small speeds or accelerations with the old units.

A quick note about tests: lots of tests are certainly way better than none!  But I wanted to give a few recommendations.  (I'm not trying to pick on the tests for Motion, they just happened to be what I happened to interface with on this patch!  And the thorough testing definitely let me feel good about making the changes!  :)  )

If tests don't have a description string, it becomes a bit difficult to discover which specific test is failing.
Ideally, each test() block should test one set of things in reasonable isolation.
There's no need to test every aspect of a behavior.  (For instance, a lot of the tests for multiway checked that the velocity was set correctly and that the entities moved the correct distance -- the latter is actually testing "Motion", not "Multiway".)

I have looked over the PR, good job!
Specifying speed in pixels per second is the most groundbreaking change I have witnessed so far. It's the right thing to do, but this will probably break all existing games!
Nice job identifying the problem with the Math.sign solution from SO.
The only thing I would like to nitpick is the order of the NewDirection event. The motivation behind it was that NewDirection should be fired just before the velocity/acceleration was applied. If user has custom EnterFrame callback that changes velocity/acceleration, he can not be sure whether it is applied before/after Motion changes the velocity/acceleration. By binding to the NewDirection event he can make sure what combined velocity/accel will be applied and still change it to suit his needs (e.g. he can prevent moving left).
Initially I had the NewDirection event fire after Motion changed it (as it's natural), but maybe there is a corner case later where the paragraph above will be relevant. We can leave it as proposed in this PR for now.

Initially I had the NewDirection event fire after Motion changed it (as it's natural), but maybe there is a corner case later where the paragraph above will be relevant. We can leave it as proposed in this PR for now.

Another possibility would just be to trigger this when the velocity is actually changed.  (So in the setter for the velocity properties.)  That way it wouldn't matter how the velocity was changed.

Specifying speed in pixels per second is the most groundbreaking change I have witnessed so far. It's the right thing to do, but this will probably break all existing games! That's why I initially proposed we stick to dt * 0,05.

I think there are enough breaking changes since the last version that pretty much any game will require a bit of work to update, so I'm not too worried about this.  I find it makes it easier to reason about how speeds and accelerations, so it should be worth it.

I find it makes it easier to reason about how speeds and accelerations, so it should be worth it.

It also plays nicely with tweening.
Ok then, I suggest you readd some test checks, as well as update the default values for movement and their respective documentation (_speed in Multiway and _jumpSpeed in Jumper).
After that we can merge this bad boy :)

Another possibility would just be to trigger this when the velocity is actually changed. (So in the setter for the velocity properties.) That way it wouldn't matter how the velocity was changed.

Yes, that would be great! Note that it will introduce some overhead probably.

Ok, I re-added those tests.  (A better fix would be to break up that long litany of tests into several isolated ones, and then check that each event was called individually, but I think this is fine for now.)

Great, I noticed some minor things that should be updated in documentation.

Another possibility would just be to trigger this when the velocity is actually changed. (So in the setter for the velocity properties.) That way it wouldn't matter how the velocity was changed.

Should this be in current or seperate PR?

Good catch with the examples.

Should this be in current or seperate PR?

If we do that, it'll be a separate PR.  Since there's an event any time the velocity changes, I'm not sure how important it is to have a separate one that occurs for a particular type of change.

Ok, all the examples should be updated now.

I just noticed this since I use the nightlies:
http://neothemachine.github.io/hungrymonkey/
I think to make the transition easier it would good to provide some rough formula to get from pixels per frame to pixels per second, assuming some common framerates like 60fps.

To go from old [px / frame] to new [px / s] you multiply it by the target frame rate, which defaults to 50, e.g.:

old jumpSpeed = 6 [px / frame] -> new jumpSpeed = 300 [px / s]
old gravityConst = 0.2 [px² / frame] -> new gravityConst = 0.2 * 50^2 = 500 [px² / s]

Good idea, we should probably add this to the changelog

Yep, that worked. Thanks


chore(main): release 0.1.0

:robot: I have created a release beep boop
0.1.0 (2023-04-02)
Features

adds main tab and status text (36d59b1)

Bug Fixes

handle password and validation (95d2837)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/others-git/watch-dawg/releases/tag/v0.1.0 :sunflower:


Implement the ingestion of the Performance hub metadata

As discussed in the spike an automated / API solution may be possible in future but for now we will ingest in a similar way to ESDAs - writing a script to scan a CSV and push catalogue entries via the DataHub API.
Ingestion

Metrics can be assigned to a new "database" container called "HMPPS Performance Hub"
Title - I suggest using ShortTitle. "Report title" is fairly legacy so can be ignored (in most cases the short title is the same).
Description - there aren't more detailed descriptions. Suggest constructing one from "Performance Hub metric $shortTitle ($metricRef). Sourced from $dataSource.". E.g. "Performance Hub metric: Approved premises RoSH level (AP01 MI001). Sourced from nDelius extract." (Some users will know metric ref - e.g. probation performance is published with this, so being searchable is good)
Subject area - see below
Data owner - see below
Request access - hard code to "Please contact the data owner"
Ask a question - hard code to "Please contact the data owner" (there are teams channels and a support email, but I'd rather not overhwlem those at this stage)
There aren't any instances now, but just in case, don't ingest any records where Active = false

Subject areas

If EntityType = Prison, assign it to the prison subject area
Else probation (that includes, BASS, EM and approved premises)

Data owners

If a metric has been assigned to the probation subject area, add<EMAIL_ADDRESS>as the owner
For prison  subject area metrics:

If DataSource is "Finds Upload", "Incidents Upload" "Incidents Upload (IRS)" or "Hub upload (assaults & referrals)" assign<EMAIL_ADDRESS>as the owner
If DataSource is "Alere Tox lab data", assign<EMAIL_ADDRESS>as the owner ("MDT" is mandatory drug testing in prisons)
If DataSource is "Programmes return" or "HII programme uploads" OR the MetricRef starts with "CUP", assign<EMAIL_ADDRESS>as the owner
Otherwise assign<EMAIL_ADDRESS>as the owner

Owners will almost certainly need tweaking later, but this is a reasonable start.

The spike ticket says the result will likely look like a chart but I'll need to check whether we can actually link to the chart like we did with with Justice Data.
Both charts and datasets can be specialised with SubType. Setting this to a value like "PerformanceMetric" preserves the option of handling metrics differently to other kinds of charts.

This is what the ownership logic was - we're not implementing this

If a metric has been assigned to the probation subject area, add<EMAIL_ADDRESS>as the owner
For prison  subject area metrics:

If DataSource is "Finds Upload", "Incidents Upload" "Incidents Upload (IRS)" or "Hub upload (assaults & referrals)" assign<EMAIL_ADDRESS>as the owner
If DataSource is "Alere Tox lab data", assign<EMAIL_ADDRESS>as the owner ("MDT" is mandatory drug testing in prisons)
If DataSource is "Programmes return" or "HII programme uploads" OR the MetricRef starts with "CUP", assign<EMAIL_ADDRESS>as the owner
Otherwise assign<EMAIL_ADDRESS>as the owner

Work in progress, but have something ingested
https://dev.find-moj-data.service.justice.gov.uk/details/dashboard/urn:li:dashboard:(performance-hub,performance-hub)
We're going to need to adjust the templates in Find MoJ data

Charts have hardcoded links to Justice data - this should be based on platform
There are two many metrics to display on one page - we really need some kind of pagination or search
Subject area is missing for dashboard

Chart/dashboard is not actually a very good description of what these metrics are, so I'm going to change it to Dataset/Container, and introduce a subtype for Metrics.
We can then hide the schema part from the template.


fix: small UX/debugging improvements

What this PR does / why we need it
This PR adds SkipReason (which is already in the function signature, but not used) in debug logs, and static log debugging. I also fixed #157.

Jira ID
DT-2902

Notes for your reviewers

Coverage decreased (-0.3%) to 46.131% when pulling 36c15d1304fac71997be932a94e0f1c9bedb8d8c on jaredallard/feat/small-debug-improvments into edfa44df93dd71595fd1052ca9fd092487faf282 on main.


nvmlErrorString error

i import nvml in my code，but when run code，have this error

@klueska

Which version of go-nvml are you building against and how do you build your application?

go-nvml is  v0.12.0-1

build command is : go build main.go

i use nvidia-mig-parted also have this problem

@elezar  @klueska

Note that when building applications that use the bindings, one has to add the following go build flags:
go build -ldflags="-extldflags=-Wl,-z,lazy" <files>.go

as per https://github.com/NVIDIA/go-nvml/issues/36#issuecomment-1471594094 for example.
Since you mentioned mig-parted, I will confirm that we're applying the correct flags there too.

i tested this command, but i get the same error

my go version is

@elezar  i update go version to 1.20.8, and try again. but i get the same error

Just as a sanity check -- are you able to run nvidia-smi from whatever environment you are in here? Is libnvidia-ml.so.1 in your library path?

Ignore my compiled program for now, I'll test nvidia-mig-manager first，i have the same error

there is something about my environment.

this is a k8s cluster, one node is gpu node.

nvidia-smi can run on this gpu node

libnvidia-ml.so.1 in my library path

mig-manager pod is running

i use this yaml and image https://github.com/NVIDIA/mig-parted/blob/main/deployments/gpu-operator/nvidia-mig-manager-example.yaml

but when i exec mig-manager pod, i can not run nvidia-smi

i think may be volume mount caught this ,but i don't know why

What else do I need to check? Thank you.  @klueska @elezar

We currently don't support running the mig-manger outside of the GPU Operator. Meaning these examples are likely out of date and probably need some tweaking to get them to work (though it's not recommended).
That said, I'm guessing the reason you are having issues is that the mig-manager you are starting doesn't have GPU support injected into it.
This is normally done either through a runtime class called nvidia or through making the nvidia runtime the default runtime in containerd. Which method are you using?

I think i use the second one.

i install  nvidia-container-runtime on my gpu node

modify /etc/docker/daemon.json and set nvidia-container-runtime as the default runtime

That looks like a docker demon.json, not a containerd config

yes，it‘s  docker demon.json.  docker will call containerd.
how can mig-manager have GPU support injected into it, Could you give me some advice?

Unless you are using a very old version of kubernetes or have explicitly selected docker to be your shim layer in kubernetes, I would imagine that docker is not the container runtime ou are using to launch containers in k8s (containerd is the default).
My question before still stands:

This is normally done either through a runtime class called nvidia or through making the nvidia runtime the default runtime in containerd. Which method are you using?

Only once I know the answer to this can I help you further.

I faced the same problem: ./app: symbol lookup error: ./app: undefined symbol: nvmlErrorString
Error exists in code example:
ret := nvml.Init()
if ret != nvml.SUCCESS {
    log.Fatalf("Unable to initialize NVML: %v", nvml.ErrorString(ret))
}

If nvml.Init() is not success just does not use nvml function nvml.ErrorString(ret)

@jgivc how are you building the application?
With regards to:

If nvml.Init() is not success just does not use nvml function nvml.ErrorString(ret)

This is not entirely true. At present, nvml.Init() also loads the dynamic library and if this fails we the symbol is invalid. If the underlying call to nvmlInit reutrns an error, calling nvml.ErrorString is valid.
We have a workaround for this in another set of libraries we use: https://github.com/NVIDIA/go-nvlib/blob/486ed3f0c8139174a97565985fd48664b3048ad6/pkg/nvml/nvml.go#L38-L69 and we may consider doing something similar here.

how are you building the application?

Just go build. And it work fine on my server. But on my computer that library does not exists and the example code gave me an error.

Not quite on topic, but also a problem with 'undefined symbol'. My application was working fine for several days and suddenly crashed with the error "undefined symbol: nvmlDeviceGetComputeRunningProcesses_v3", although before that there were several successful calls to nvml library methods. Is it possible to somehow intercept such errors so that the application does not crash?


Duplicate Icon

I'm getting mutiple icons appearing for my toastr.
I have Font awesome 5 included with my project. Could that be an issue?

wrong git, embarrassing.


removed $wait parameter from the obtainLetsEncrypt function description

Sorry, didn't spot this the first time around


flashlight test Error

when I run the demo of Usage with Maestro
error thrown
Error: Invalid line: 28825 (AWCN Worker(M)1) S 367 367 0 0 -1<PHONE_NUMBER> 15 0 0 0 0 0 0 0 20 0 31 0 12492622<PHONE_NUMBER>4 35945<PHONE_NUMBER>3709551615 1 1 0 0 0 0 4608 1<PHONE_NUMBER> 0 0 0 -1 3 0 0 0 0 0 0 0 0 0 0 0 0 0
/snapshot/flashlight-cloud/flashlight-open-source/packages/platforms/android/dist/src/commands/cpu/getCpuStatsByProcess.js:24:19
suggest changing the regex in line 22 of getCpuStatsByProcess.js to "/^(\d+) ((.)) (.)/"

Hi @Lokizjj29, thanks for reporting this, and good catch!
Would you like to submit a pull request by any chance?
You'd have to:

Modify https://github.com/bamlab/flashlight/blob/main/packages/platforms/android/src/commands/cpu/getCpuStatsByProcess.ts#L28
Add a test similar to https://github.com/bamlab/flashlight/blob/main/packages/platforms/android/src/commands/cpu/tests/getCpuStatsByProcess.test.ts#L40

Check it works with:
yarn
yarn jest getCpuStatsByProcess


issue about gmm_loss

In the file models/mdrnn.py, why minus max_log_probs first (line 38) and then add max_log_probs.squeeze() (line 43) ?
batch = batch.unsqueeze(-2)
normal_dist = Normal(mus, sigmas)
g_log_probs = normal_dist.log_prob(batch)
g_log_probs = logpi + torch.sum(g_log_probs, dim=-1)
max_log_probs = torch.max(g_log_probs, dim=-1, keepdim=True)[0]
g_log_probs = g_log_probs - max_log_probs
g_probs = torch.exp(g_log_probs)
probs = torch.sum(g_probs, dim=-1)

log_prob = max_log_probs.squeeze() + torch.log(probs)
if reduce:
    return - torch.mean(log_prob)
return - log_prob

Is there a corresponding mathematical formula？

The two are equivalent.

I also came across this and found it odd to have seemingly no effect. But I think this is done for numerical stability and also known as the "Log-Sum-Exp" Trick, nicely explained here: https://gregorygundersen.com/blog/2020/02/09/log-sum-exp/
I quickly tested it and made a variant not using this trick and for one value I got -inf while with the trick I got a valid value


Add Discord Mass Role Assigner by Canter#0548

This project adds a simple utility script which adds a specified role to all members of a guild (who do not currently have it). It is meant for server administrators to be able to assign "autoroles" retroactively (rather than just applying them to new guild members on join).
See README.md for remaining details.

I don't get it, is this a full bot? Code looks good otherwise.

Yes, it's meant to be run as a standalone utility.


cilium, daemon: remove arping lib dependency

The j-keck/arping library which we currently pull in via cilium/arping is a mess, implement something properly and get rid of this dependency.

I'd like to try to help with this.

I'm happy to review any code or discuss this, reach out on slack or here. It was a pretty ugly bit of work I did here.

This is fixed

@nathanjsweet I think it was closed by a mistake (I don't see any PR which replaced the arping lib)?

Cool. Going to get back to this this week if there has not been a PR with the fix (will check that first)

@brb sorry I was under the impression that the arping library itself was pulling in the suspect library, not that it was a clone of that library.

@vladdy Hey, are you still working on this issue?

@brb, sorry for the luck of the updates. Yes, I'm. I'll send an update by monday.

@vladdy any updates?

I'll have an update to the PR by Wednesday. Sorry about the delay.

I need a little more time. Will try to provide an update tomorrow.

Although this got closed, I wanted to confirm with @borkmann on his thoughts about IPv6 from the initial posting. I don't think it uses ARP and the code around calls to arping operates with IPv4 addresses. Should there still be a task for IPv6?

There is the issue for it: https://github.com/cilium/cilium/issues/14429


Handling of Categorical Variables

[ ] Build around formulaic's C(..., levels = ...) option.
[ ] Need to pass levels information to formula.
[ ] Rename levels to "ref" so that users can only provide a reference str instead of a full list.
[ ] Is there a need for a dedicated i() option to interact with a categorical variable? The key advantage of the i() method seems custom tooling around it, e.g. iplot. But that could be handled with plotting method with decent string regex for coef names (i.e. for variables included in i()). Maybe not 100% error prone?

Yes, i() needed for setting levels of interacted fixed effects.

Should be accomplished with #30 . (Still not possible to pass levels to C(), but possible to pass to i()).


Fix type hint in example sqlalchemy_async.py

PR Checklist

[x] Have you followed the guidelines in CONTRIBUTING.md?
[x] Have you got 100% test coverage on new code?
[x] Have you updated the prose documentation?
[x] Have you updated the reference documentation?

I found a wrong type hint in the SQLAlchemyPlugin example.

Nice!


feat: cli did create --method=key command

New command added to the CLI:
waltid did create --method=key --key=<filepath>
Which will create a did:key based on the key provided in --key

@alegomes the waltid alias for development is not working. the normal CLI script is working fine.

Weird failing build message. However, it works for me. 🤔

the tests fail for me when running ..gradlew build clean, however I can use both  CLI scripts fine

@alegomes the tests fail for me when running ..gradlew build clean, however I can use both CLI scripts fine Using windows OS !

Hummm. Can you give me more details about this failing test? All tests are passing in my local environment and in the Github's checks.


Tab Navigation Snack is Missing Dependencies

In the "tab navigation" guide, within the subsection titled "A stack navigator for each tab," the snack that's linked is missing three required dependencies and is thus unable to run. This can be resolved with the addition of the following lines to the package.json file:
"@react-navigation/stack": "^5.7.1",
"@react-navigation/bottom-tabs": "^5.7.2",
"react-native-screens": "^2.9.0"
Here is a link to a working snack for reference.
Edit: Here is a link to the relevant section of the documentation.

Hey @MichaelKovich, could you check it again? seems working fine now.


add initial docker push workflow

This adds a workflow to push images to dockerhub.
You need to create two variables in the repo for the username and token:
          username: ${{ secrets.DOCKERHUB_USERNAME }}
          password: ${{ secrets.DOCKERHUB_TOKEN }}

This pushes two tags: latest and the git-tag.
I couldn't test it but hope that it will mostly work and you can then iterate on it.

So I got it to work.
Here's a successful run: https://github.com/rndmh3ro/allero/actions/runs/3197224646/jobs/5220160129
Here are the images:  https://hub.docker.com/r/rndmh3ro/allero-cli/tags


Ignore trailing dot in hostname for ssl certificate hostname verification

Can the team add a verification flag to openssl to handle the trailing dot (if it exists) in hostname appropriately (for the hostname check). This is needed because sometimes we need to have the trailing dot in the hostname for DNS resolution to work properly (and for security), but that would cause the certificate SAN fields to not match the hostname (since we have the trailing hostname).
So for example, let's say that I am contacting
"example.com." but my certificate has "example.com" on it, proposed change (with flag enabled) would match the certificate correctly with the hostname and allow for a connection.
This enhancement would allow other projects (such as python) to allow for more customizability through the use of this flag.

I do not see a reason why the trailing dot should not be ignored unconditionally, i.e. no additional verification flag should be needed.

When doing hostname verification against an X.509 certificate, a trailing dot (period) in the hostname is matched against the certificate.  But the trailing dot should only be applied to the DNS lookup not the certificate match.
Conversely, a certificate that has a trailing dot in its commonName (probably rare but allowed) should match a hostname without the trailing dot.  As the ssl library is written now, both cases fail.
Web browsers allow a trailing dot in the URI and will accept a certificate even when the certificate doesn't have a trailing dot.  For example, visit Google with trailing dot (https://www.google.com./) from a browser of your choice and check certificate.  It should check out as valid.  -from another post online related to the topic

Also, contacting a URL without a trailing dot can be ambiguous (ie. ai.university vs ai.unversity.edu). That is why we may want to contact a url using a trailing dot. But we do not typically put hostname with trailing dot in a certificate, we just put the hostname. Let me give you an example where this will be a problem.
In Python for example, if we contact a url with a trailing dot, the hostname verification directly using openssl API only uses that url with the trailing dot for verification (so we would compare "hostname.com." with "hostname.com" in the certificate which would not match.
Now, you may say that can just be fixed on the python side, but the python developers said they would not touch the ssl verification stuff (which is why they switched to directly calling the openssl API for this). Let me know your thoughts on this. How do you think this can practically be fixed? On OpenSSL? on Python? somewhere else?

Maybe you misunderstood my comment - I am basically saying this should be fixed on OpenSSL side and there is IMO no need to add a flag for that.

What I am not sure about is whether this can be considered simply a bug or it is rather a feature.

Google

Ok, thanks @t8m I did misunderstand your comment

I think it would be considered a bug because expected behavior is to ignore trailing dot in hostname that you are comparing against the ones in the certificate but at the same time this is an important part of ssl verification (and so would be a feature).

Perhaps @vdukhovni has an opinion on this?

To be clear, if OpenSSL is calling SSL_set1_host() because we're establishing the connection to the host via some form of BIO_connect type call that takes hostnames, and uses SSL_set1_host() and sets the SNI name internally, ... then we're responsible for stripping off any trailing . in the hostname before using it as an SNI name or passing it to SSL_set1_host() (but not when using it in getaddrinfo(3)).

This is the case for cpython, see code here: https://github.com/python/cpython/blob/d5bd32f/Modules/_ssl.c#L778 (main branch as of today)
We'd like to use FQDNs to avoid unnecessary DNS queries but with the current OpenSSL implementation we can't.

Nothing in OpenSSL prevents the application from passing a dot-stripped name to SSL_set1_host() while separately using a trailing dot with getaddrinfo(3).  If the question is about higher level BIO APIs that take a name and do all the connection management, the dot stripping prior to calling SSL_set1_host() can be done there.
There's no need to second guess intent at the lower layers of the library.

There's no need to second-guess intent at the lower layers of the library.

I disagree. I keep running in to issues where various applications have broken implementations because this is so ambiguous. It also doesn't help that the SNI standard expects it but the SAN doesn't.
Adding the trailing dot is not expected in the SAN per RFC 5280 <IP_ADDRESS> paragraph 6:
When the subjectAltName extension contains a domain name system
label, the domain name MUST be stored in the dNSName (an IA5String).
The name MUST be in the "preferred name syntax", as specified by
Section 3.5 of [RFC1034] and as modified by Section 2.1 of
[RFC1123]. Note that while uppercase and lowercase letters are
allowed in domain names, no significance is attached to the case. In
addition, while the string " " is a legal domain name, subjectAltName
extensions with a dNSName of " " MUST NOT be used. Finally, the use
of the DNS representation for Internet mail addresses
(subscriber.example.com instead of [subscriber@example.com](mailto:subscriber@example.com)) MUST NOT
be used; such identities are to be encoded as rfc822Name. Rules for
encoding internationalized domain names are specified in Section 7.2.

an IA5String is defined in RFC 1034 3.5 makes no mention of a trailing dot and the examples do not show one:
The following syntax will result in fewer problems with many
applications that use domain names (e.g., mail, TELNET).

<domain> ::= <subdomain> | " "

<subdomain> ::= <label> | <subdomain> "." <label>

<label> ::= <letter> [ [ <ldh-str> ] <let-dig> ]

<ldh-str> ::= <let-dig-hyp> | <let-dig-hyp> <ldh-str>

<let-dig-hyp> ::= <let-dig> | "-"

<let-dig> ::= <letter> | <digit>

<letter> ::= any one of the 52 alphabetic characters A through Z in
upper case and a through z in lower case

<digit> ::= any one of the ten digits 0 through 9

The verbiage of this section is very non-committal but I think it's fair for people to read this and expect that a SAN should not end in a dot.

it seems like this still has some activity around it
@joekohlsdorf  @joejulian @brysonli12
could one of you please add a comment with a proposed change to openssl so that it can be considered?  What I'm reading above sounds to me like the ask is that domain names for the purposes of certificate parsing/validation/etc have any trailing dots ignored, but I'd like to be sure

You are correct. Taking dots should be ignored.


os.system function is invisible

Describe the bug
os.system function won't run and is invisible to compiler - due to https://github.com/kokonut27/zsq/commit/f65b3c7f74bb6fe359644cf05eea5d21be9afa1a
Expected behavior
os.system function is supposed to be visible and run.

I just saw this, but I believe you can capture the output through subprocess for example:
def shell(st):
    x = []

    proc = subprocess.Popen(st.split(' '), stdout=subprocess.PIPE,shell=True)

    if proc.stderr != None and proc.stderr.read().decode("utf-8") != "":
        return proc.stderr.read().decode("utf-8")
    else:

        result = proc.stdout.read()

        proc.stdout.close()

        return result.decode("utf-8")

and then print the returned result if you wish to show it


Create Cloud API endpoint to support chooser box for fields.

For implementing #84

@phillip-hopper I'm guessing you need a list of fields for each dictionary, e.g. GET /dictionary/{dictionaryId}/searchFields, which returns something like ["mainheadword", "headword", "gloss", ...]
I'm guessing we'll need some kind of hard coded conversion.
Plus, we'll need search to support these fields.


Develop update

Update develop branch to include new samples (and README formatting changes) ahead of a separate PR for a new sample being pushed to develop branch.

deleting in favour of copying master to develop on release


Add support TypeScript.

I added a filter to support TypeScript.
configuration of the sample to be added to the [asset_compress.ini]
[filter_TypeScript]
typescript = /usr/bin/tsc

It seems to use spaces instead of tabs for indentation - and the . need spaces around them as per coding standards.


⚠ Spryr has degraded performance

In bb43928, Spryr (https://spryr.com) experienced degraded performance:

HTTP code: 200
Response time: 3606 ms

Resolved: Spryr performance has improved in e5cc7df after 8 minutes.


[1.2 -> master] fix cpu counting (#14468)

This change is already in "master" for the installer.

Cherry pick from  (#14468)
(cherry picked from commit 00236e4ad82c07a2fd22649bd16b18caa5a037a4)
This was accidentally merged into release-1.2 without being in master first

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: howardjohn
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

install/OWNERS [howardjohn]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@howardjohn: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/integ-galley-k8s-presubmit-tests.sh
2f12978391e4cf0c935463456c6c5cf077d239a1
link
/test integ-galley-k8s-presubmit-tests-master

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

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: howardjohn, mandarjog
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

install/OWNERS [howardjohn,mandarjog]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

/test integ-galley-k8s-presubmit-tests-master

@howardjohn: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/integ-galley-k8s-presubmit-tests.sh
2f12978391e4cf0c935463456c6c5cf077d239a1
link
/test integ-galley-k8s-presubmit-tests-master

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

/test integ-galley-k8s-presubmit-tests-master

@howardjohn: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/integ-pilot-k8s-presubmit-tests.sh
2f12978391e4cf0c935463456c6c5cf077d239a1
link
/test integ-pilot-k8s-presubmit-tests-master

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

@howardjohn: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/integ-pilot-k8s-presubmit-tests.sh
2f12978391e4cf0c935463456c6c5cf077d239a1
link
/test integ-pilot-k8s-presubmit-tests-master

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

@howardjohn: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/integ-pilot-k8s-presubmit-tests.sh
2f12978391e4cf0c935463456c6c5cf077d239a1
link
/test integ-pilot-k8s-presubmit-tests-master

prow/istio-pilot-e2e-envoyv2-v1alpha3.sh
2f12978391e4cf0c935463456c6c5cf077d239a1
link
/test istio-pilot-e2e-envoyv2-v1alpha3-master

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

@howardjohn: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/integ-pilot-k8s-presubmit-tests.sh
2f12978391e4cf0c935463456c6c5cf077d239a1
link
/test integ-pilot-k8s-presubmit-tests-master

prow/istio-pilot-e2e-envoyv2-v1alpha3.sh
2f12978391e4cf0c935463456c6c5cf077d239a1
link
/test istio-pilot-e2e-envoyv2-v1alpha3-master

prow/integ-security-k8s-presubmit-tests.sh
2f12978391e4cf0c935463456c6c5cf077d239a1
link
/test integ-security-k8s-presubmit-tests-master

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

/test all

@howardjohn: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-pilot-e2e-envoyv2-v1alpha3.sh
2f12978391e4cf0c935463456c6c5cf077d239a1
link
/test istio-pilot-e2e-envoyv2-v1alpha3-master

prow/integ-security-k8s-presubmit-tests.sh
2f12978391e4cf0c935463456c6c5cf077d239a1
link
/test integ-security-k8s-presubmit-tests-master

prow/integ-pilot-k8s-presubmit-tests.sh
2f12978391e4cf0c935463456c6c5cf077d239a1
link
/test integ-pilot-k8s-presubmit-tests-master

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

@howardjohn: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/integ-security-k8s-presubmit-tests.sh
2f12978391e4cf0c935463456c6c5cf077d239a1
link
/test integ-security-k8s-presubmit-tests-master

prow/integ-pilot-k8s-presubmit-tests.sh
2f12978391e4cf0c935463456c6c5cf077d239a1
link
/test integ-pilot-k8s-presubmit-tests-master

prow/integ-telemetry-k8s-presubmit-tests.sh
2f12978391e4cf0c935463456c6c5cf077d239a1
link
/test integ-telemetry-k8s-presubmit-tests-master

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

@howardjohn: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/integ-security-k8s-presubmit-tests.sh
2f12978391e4cf0c935463456c6c5cf077d239a1
link
/test integ-security-k8s-presubmit-tests-master

prow/integ-telemetry-k8s-presubmit-tests.sh
2f12978391e4cf0c935463456c6c5cf077d239a1
link
/test integ-telemetry-k8s-presubmit-tests-master

prow/integ-pilot-k8s-presubmit-tests.sh
2f12978391e4cf0c935463456c6c5cf077d239a1
link
/test integ-pilot-k8s-presubmit-tests-master

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

@howardjohn: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/integ-security-k8s-presubmit-tests.sh
2f12978391e4cf0c935463456c6c5cf077d239a1
link
/test integ-security-k8s-presubmit-tests-master

prow/integ-telemetry-k8s-presubmit-tests.sh
2f12978391e4cf0c935463456c6c5cf077d239a1
link
/test integ-telemetry-k8s-presubmit-tests-master

prow/integ-pilot-k8s-presubmit-tests.sh
2f12978391e4cf0c935463456c6c5cf077d239a1
link
/test integ-pilot-k8s-presubmit-tests-master

prow/istio-pilot-multicluster-e2e.sh
2f12978391e4cf0c935463456c6c5cf077d239a1
link
/test istio-pilot-multicluster-e2e-master

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

@howardjohn: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/integ-security-k8s-presubmit-tests.sh
2f12978391e4cf0c935463456c6c5cf077d239a1
link
/test integ-security-k8s-presubmit-tests-master

prow/integ-telemetry-k8s-presubmit-tests.sh
2f12978391e4cf0c935463456c6c5cf077d239a1
link
/test integ-telemetry-k8s-presubmit-tests-master

prow/integ-pilot-k8s-presubmit-tests.sh
2f12978391e4cf0c935463456c6c5cf077d239a1
link
/test integ-pilot-k8s-presubmit-tests-master

prow/istio-pilot-multicluster-e2e.sh
2f12978391e4cf0c935463456c6c5cf077d239a1
link
/test istio-pilot-multicluster-e2e-master

prow/e2e-bookInfoTests-trustdomain.sh
2f12978391e4cf0c935463456c6c5cf077d239a1
link
/test e2e-bookInfoTests-trustdomain-master

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

@howardjohn: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/integ-security-k8s-presubmit-tests.sh
2f12978391e4cf0c935463456c6c5cf077d239a1
link
/test integ-security-k8s-presubmit-tests-master

prow/integ-telemetry-k8s-presubmit-tests.sh
2f12978391e4cf0c935463456c6c5cf077d239a1
link
/test integ-telemetry-k8s-presubmit-tests-master

prow/integ-pilot-k8s-presubmit-tests.sh
2f12978391e4cf0c935463456c6c5cf077d239a1
link
/test integ-pilot-k8s-presubmit-tests-master

prow/istio-pilot-multicluster-e2e.sh
2f12978391e4cf0c935463456c6c5cf077d239a1
link
/test istio-pilot-multicluster-e2e-master

prow/e2e-bookInfoTests-trustdomain.sh
2f12978391e4cf0c935463456c6c5cf077d239a1
link
/test e2e-bookInfoTests-trustdomain-master

prow/istio-pilot-e2e-envoyv2-v1alpha3.sh
2f12978391e4cf0c935463456c6c5cf077d239a1
link
/test istio-pilot-e2e-envoyv2-v1alpha3-master

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

@howardjohn: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/integ-telemetry-k8s-presubmit-tests.sh
2f12978391e4cf0c935463456c6c5cf077d239a1
link
/test integ-telemetry-k8s-presubmit-tests-master

prow/integ-pilot-k8s-presubmit-tests.sh
2f12978391e4cf0c935463456c6c5cf077d239a1
link
/test integ-pilot-k8s-presubmit-tests-master

prow/istio-pilot-multicluster-e2e.sh
2f12978391e4cf0c935463456c6c5cf077d239a1
link
/test istio-pilot-multicluster-e2e-master

prow/e2e-bookInfoTests-trustdomain.sh
2f12978391e4cf0c935463456c6c5cf077d239a1
link
/test e2e-bookInfoTests-trustdomain-master

prow/istio-pilot-e2e-envoyv2-v1alpha3.sh
2f12978391e4cf0c935463456c6c5cf077d239a1
link
/test istio-pilot-e2e-envoyv2-v1alpha3-master

prow/integ-security-k8s-presubmit-tests.sh
2f12978391e4cf0c935463456c6c5cf077d239a1
link
/test integ-security-k8s-presubmit-tests-master

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

/test istio-pilot-e2e-envoyv2-v1alpha3-master

@howardjohn: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-pilot-multicluster-e2e.sh
2f12978391e4cf0c935463456c6c5cf077d239a1
link
/test istio-pilot-multicluster-e2e-master

prow/e2e-bookInfoTests-trustdomain.sh
2f12978391e4cf0c935463456c6c5cf077d239a1
link
/test e2e-bookInfoTests-trustdomain-master

prow/integ-security-k8s-presubmit-tests.sh
2f12978391e4cf0c935463456c6c5cf077d239a1
link
/test integ-security-k8s-presubmit-tests-master

prow/integ-pilot-k8s-presubmit-tests.sh
2f12978391e4cf0c935463456c6c5cf077d239a1
link
/test integ-pilot-k8s-presubmit-tests-master

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

@howardjohn: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/integ-pilot-k8s-presubmit-tests.sh
2f12978391e4cf0c935463456c6c5cf077d239a1
link
/test integ-pilot-k8s-presubmit-tests-master

prow/integ-security-k8s-presubmit-tests.sh
2f12978391e4cf0c935463456c6c5cf077d239a1
link
/test integ-security-k8s-presubmit-tests-master

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


Functional call notification is empty

STR:

Create a Functional call proposal

ER:
Functional call notification is not empty
AR:
Functional call notification is empty

0

Verified on dev. Fixed


fix(template): remove /... from generated child routes

fixes #861

Could you please squash the commits into one?

@cironunes I sure could

yup got myself into some git trouble. give me a few 😄

yeah. got myself into some rebase issues closing this PR and opening a clean one


Clearly define the meaning of the different flags (accepted, committed, finalized etc)

we should define that in the documentation and also check correctness in the code.

linked to #701


nested 的路由回退问题？

Version
3.0.3
Reproduction link
http://sample.com
Steps to reproduce
路由跳转：/jobs/ -> /posts/ -> /posts/1/ ，然后点击浏览器的返回按钮，页面会返回到 /jobs/ 而不是 /posts/。好像返回时，nested 的路由会被直接跳过？
What is expected?
期望是返回到 /posts/ 页面
What is actually happening?
返回到了 /jobs/ 页面

That's not a valid repro. Also please, write the issue in English.
Before opening a new issue, make sure to check  existing  issues as well :)