Feat/remote add

Aim: Add remote write capabilities to modo-api and cli
PR elements:
1. Bug fix to remove zarr element group itself in modo.element.remove(element_id):
Ops: this fix is unrelated to remote writing, but is a bug detected during the PR.
Before modo.element.remove(element_id) removed data files associated to the element that should be deleted and all links to other elements, but not the element's zarr group itself.
2. Fix modo.list_arrays() to correctly display file tree for local and remote objects:
list_arrays() calls the zarr.group.tree() function. For remote objects this functions does not use consollidated metadata (even, if zarr.consollidate_metadata() was called before) resulting in file trees with duplicated elements. Explicitly opening the consollidated zarr store fixes this issue and speeds up the process noteably!
3. Use s3fs filestore to handle remote file objects:
Remote files that are not written in zarr need to be accessed and handled via s3fs.  To add this all file handling functions had to be adapted by including conditionals on the filestore (local or remote). This affected api functions related to adding, removing and listing files as well as some of there internal functions.
Ops:: This resulted in multiple conditionals. We should restructure the code to use child classes managing the differences (see https://github.com/sdsc-ordes/modo-api/issues/29)
4. Add remote capabilities to cli:
So far cli functions did not comply with remote objects. By adding a new cli-argument (--s3-endpint, -s3) they can automatically use the modo.api remote capabilities.
5.Fix remote modo creation
As previously remote modos were read only, we did not differentiate between existing and non-existing modos in MODO() . To allow creating new modos remotely MODO() can now distinct between remote/local and existing/non-existing for each of them.

bug/enhancement: when creating a remote object that has an ID that already exists on the remote, modo will overwrite the object without letting you know there was one there already.

Very good point! The behavior was indeed different from local creation and non-transparent.
cli: In 6e1e0afc2646c7f1ed8e5e5bda1fa6e9c63c61a6 I added a check to raise a ValueError if the object_directory exists at the remote storage to modo create.
api: The same command (MODO()) can be used to create a new or load an existing object, depending on the path existance. To be more transparent, in 6e1e0afc2646c7f1ed8e5e5bda1fa6e9c63c61a6 I added a message, if an existing modo was loaded from local or remote:
f"Load existing MODO from remote path: {path}\n"  "NOTE: Additional arguments will be ignored."

Thanks a lot @supermaxiste for the review! You raised very good and useful points. I  tried to address all of them, but the 2 non-blocking enhancement points. For those I opened issues #34 and #33 and I will look into them in future.


Feature: Add torchserve custom server with pv storage

Torchserve custom model server with external pv storage
Fixes #1179
Special notes for your reviewer:

Please confirm that if this PR changes any image versions, then that's the sole change this PR makes.

Release note:
Add torchserve custom server with external pv storage

/lgtm
/approve


Separated means of invoking an EventHubs Sample

Hi @jsquire
Trying to think a bit outside the box.
What we would like is being able to differentiate samples by mean of invoking them.
In order to do that I created some coloured interfaces (different kinds of Azure) that would sit as placeholders for different ways of invoking samples.
That could be another alternative to all what we have been thinking so far.

Can one of the admins verify this patch?


Removing Edge conflicts with Windows updates?

Just for completeness ...
I had use RemoveEdge and now I am facing with an issue with Windows updates. After flagging the issue to Microsoft community, someone there responded that if one has uninstalled Edge this browser should be reinstalled...
What do you thing about this?

Thanks for the note!
I have not seen this issue.... yet. It's very possible that a new update may cause problems. I head about some "big" update coming out, but didn't look much into it.
If I do see this, I'll make a note in the Readme. I have links in the Readme to install Edge again.
This tool does not prevent Edge from re-installing. Install Edge, run the update, and remove Edge again.
I saw issues with updates back in 2021 for KB5006670

I will try and report..

Ok, with Edge installed the update installs.. Maybe we can close this..

Added to the Readme.
Thanks for the information!


maybe a bug

The text is stored in vuex, but when I refresh the page, the text disappears from the page unless I run the program again
In addition the text is obtained by sending a network request

The problem has been solved, you only need to do persistent storage for the text.


500 error publishing BumpSequenceOp to test-network

This transaction with a single BumpSequenceOperation gave the following response via the lab:
{
  "type": "https://stellar.org/horizon-errors/server_error",
  "title": "Internal Server Error",
  "status": 500,
  "detail": "An error occurred while processing this request.  This is usually due to a bug within the server software.  Trying this request again may succeed if the bug is transient, otherwise please report this issue to the issue tracker at: https://github.com/stellar/go/services/horizon/internal/issues. Please include this response in your issue."
}

Additionally, the error message asks that the issue be raised on a page that 404s.

Testnet has not been upgraded to V10 yet. It will probably happen later today.

Thanks. Obviously it downgrades the importance quite a bit, but what do you think about the HTTP response code 500? Is this a side-effect of deployment in progress?  Otherwise it should be a 4xx if the error is on the client side.

Yeah, this probably shouldn't be 500. Have you sent this transaction to horizon-testnet.stellar.org?

Yes, I sent the txn to horizon-testnet.stellar.org via the lab.
I note that the same txn now results in HTTP 400 with op_not_supported, so it's either fixed (quick work!) or transient.


Git commands from the command platte are unavailable during git operations

Type: Bug
Hi VS Code team,
For years, I've been using VS Code to execute git operations (from the amazing Command Platte).
I always explained this to my colleagues as "you can talk to this platte in plain english, and the right command would pop up before you finish... boosting your speed".
It did boost my performance, I was able to send commands through that Platte so fast, that they sort of got "piped" until the git executable would finish the first command.
However, in recent version of VS Code that ability has disappeared.
I really have to WAIT for the git operation to finish, for example wait 5s for the pull command, before I can issue the next command, for example checkout to new branch.
The command platte does not show the git commands anymore, while the git executable is running.
I hope you can fix it. It worked amazing for me in the past.
 VS Code does not have any git command orchestration mechanism hence each command is being executed immediately when initiated by the user, hence git commands can be executed in parallel.
In the past we have seen cases in which executing git commands in parallel can result in the git repository getting in an unintended state. To avoid this, out of all commands contributed by the git extension we have marked 5 commands as blocking: checkout, commit, pull, push, sync. When any of these commands are started, users have to wait until the complete before they can execute any other git commands in order.

Would you consider to solve this issue with a queuing mechanism? Instead of blocking the user?
Thanks for the response

At the moment we have no plans for implementing an orchestrator for commands in the git extension. I will leave this issue opened and mark it as "under discussion" so that I can use it to track additional user feedback.

Thanks. Last question; is it possible to somehow disable this protection? I admit I've never had issues the way I used it.

No, there is no way to disable this protection.


Tagging a new release

i came here looking for a fix for the issue detailed in https://github.com/weiboad/kafka-php/issues/268
I then noticed it is over 2 years (30 Oct 2017) since v<IP_ADDRESS> was released, including a lot of changes - https://github.com/weiboad/kafka-php/compare/v<IP_ADDRESS>...master but also there hasn't been any commits to master for nearly 2 years too...

Ping @nmred :)


Mixed-mode arithmetics for mpc

I think, we should implement mixed-mode rules for complex arithmetic.  Partially, this already acomplished: mpmath has special rules for mpf op mpc cases, for instance mpc_mul_mpf() to multiply a complex z by a real number r component-wise, z*r == mpc(z.real*r, z.imag*r).
But I'm not sure if that implementation is complete, and obviously it has many bugs.  A quick example:
>>> 1/mpc(inf, 1)
mpc(real='nan', imag='0.0')
>>> 1/complex('(inf+1j)')
0j

This case is worse than CPython, probably just due to using naive algorithm for complex division (instead of something like Smith's version).
Entirely missing component is an imaginary type, i.e. one that has only imaginary component and no real.  See draft CPython proposal: https://github.com/skirpichev/cpython/pull/1
It should be noted, that this issue depends on #167: presence of signed zeros provide more "corner cases" for analytic expressions (like asinh(z)=log(z + sqrt(1 + z**2))).  But regardless on this, we already have infinities, that will require special treatment.

I think we must support mixed-mode arithmetic at least for mpc and mpf.


Generalize SuggestedActions from taking IList to IEnumerable

Update MessageFactory.SuggestedActions to accept IEnumerable.
Add test against the updated MessageFactory output.

Fix for #625.


Remove preview for python SDK V2

Python SDK has GA'd in June.

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


add progress bar

used solution from: https://stackoverflow.com/questions/41920124/multiprocessing-use-tqdm-to-display-a-progress-bar


url_validate - updated method functionality

Wrote an improved function for blu/mst, want to refactor and replace the method in this package.

biggest change: added optional parameter to force (’//’ default, http://, https://) (should the default be https:// ?)
need to write a better regex (added $urlp = explode('//', $url); to fix an edge case, but there could be a better solution)
?? should we lowercase the protocol? (blu had someone who entered ‘HTTPS://’ and it rendered ‘https://HTTPS://’)

public function url_validate($url, $protocol = false){
  $o = null;
  if (isset($url) && $url != '' && strpos($url, '.') !== false){
    $url = trim($url);
    if (!preg_match('#^http(s)?://#', $url)) {
      $o = '';
      if ($protocol){
        $o .= $protocol;
      }else{
        $o .= '//';
      }
      if (strpos($url, '//') !== false){
        $urlp = explode('//', $url);
        $url = $urlp[1];
      }
      $o .= $url;
    }
    else{
      $o = '';
      if ($protocol){
        $urlp = explode('//', $url);
        $o =  $protocol . $urlp[1];
      }else{
        $o = $url;
      }
    }
  }
  return $o;
}

Added with 1.2.2


[Question] Where should complex business logic be in clean architecture?

Hi @amantinband !
First of all a huge thanks and bravo for the great work both here and on youtube.
The question in the title is pretty generic, but I wasn't sure what to write in so I'll try to give more details here.
I'll face soon a very complex project with a lot of functional requirement. As you know these bring a lot of business logic in the code and I'd love to use the clean architecture (along with CQRS) in the best and most powerful way.
In your example the CreateSubscriptionCommandHandler is in charge of checking whether an element already exists and create it.
In my case the CreateArticleCommandHandler should make the same verification with much more complexity (it's not just a matter of a field). If I try to figure out how to do it, I think to a Service working as "query/commands orchestrator" that contains all the BL, while the CreateArticleCommand is only in charge of writing the command to the DB.
What is your idea regarding this? How should I structure my project using CleanArchitecture?
Thanks!!

Hey @matteotrapani, (if you don't mind me answering ;) )
Could you give some information/examples on the BL you're talking about?
From a DDD's point-of-view you'd want to put most of the BL in the domain layer, i.e. a Rich Domain Model.
Checking whether an element already exists (in the database) is done in the application layer since the db is an external service - Infrastructure.
So deciding where you want to put your Business Logic depends on the Business Logic itself.
Also, since you're talking about a 'service', it might be worth looking into the difference between Application Services, and Domain Services.

Thank you for your return! Very appreciated!
I'll try to be as clear as possible regarding the application I'm working on, but it's quite hard to give all the functional details!
Figure out an online store like Amazon that sells many different kind of articles. Each article is defined by its set of properties and its category. The article is directly linked with the warehouse physical item, so it can't exist twice.
As you can imagine, the concept of "already existing" is not as simple as it seems because it actually means (sorry for the repetition ;) ) to look for an article, with the same set of properties and the same category.
At the same time the properties of the new articles must be "validated" and so on and so forth.
Before I try to propose an example of architecture, is the context clearer? Do you have any ideas?
Thanks!

Okay, so If I understand it correctly, you have an article that is defined by its properties + category, and before adding a new article to the database (in the CreateArticleCommandHandler) you need to check the following two things:

Check whether the new article is valid (according to your business rules), and
Check whether that new article doesn't already exist (I'm assuming in your database?).

As I mentioned earlier, you'd want to put as much of the Aggregate/Entity validation inside the entity itself. So validating whether the new entity is valid should be placed inside the Article class in your Domain Layer. You could have a static method like this (Using ErrorOr like Amichai, but you could also throw an exception or any Result object):
public class Article
{
    // properties

    public static ErrorOr<Article> Create(... parameters)
    {
        // validate whether the parameters are valid, and return a new instance of an Article
    }
}

Then you need to check whether an article with these properties already exists, this can be done in the CommandHandler in the Application layer, or you could write a function for this check in your Repository for example, so you can reuse it in different handlers.
So your handler could look something like this:

public async Task<ErrorOr<Article>> Handle(CreateArticleCommand command, CancellationToken cancellationToken)
    {
        // 1. Try to create a new Article, validation is done in the Article itself.
        var result = Article.Create(... properties);

        if (result .IsError)
        {
            return result.Errors;
        }

       // 2. Check whether the article already exists.
        if (await _articleRepository.ExistsAsync(result.Value, cancellationToken))
        {
            return Error.Failure(description: "Article already exists.")
        }

       // 3. Add article to the database, and save changes.
        _repository.Add(result.Value);
        await _unitOfWork.SaveChangesAsync();

        return result.Value;
    }

Please let me know if this is something you can use, and just want to mention that I'm not an expert on this. This is just how I would do it. 🙂


Wrong game scale in iPhone when game is embedded in iframe.

Version

Phaser Version: v3.20.0

Operating system: iOS 12.4.4

Browser: Safari
Device: iPhone6

Description

I tried load the game inside the iframe in portrait mode,
the iframe size is full width, and height is about 1/2, but scaleManager doesn't scale down the game  size to fit the iframe. As a result, I can only see partial of the game.
The behavior is correct in desktop, and other android browsers.
It seems only occurs in almost all iPhone devices, I also tried Chrome and still no luck.
I also tried:

initialize the game after a delay. - not working
call this.game.scale. setGameSize() every few seconds. - not working

EXPECTED RESULT: The game should exactly fill the iframe(container).
Example Test Code

The Game Scale Config:
const config: Phaser.Types.Core.GameConfig = {
	type: Phaser.AUTO,
	parent: "canvas",
	width: size.width,
	height: size.height,
	backgroundColor: 0,
	transparent: true,
	scale: {
		mode: Phaser.Scale.FIT,
		autoCenter: Phaser.Scale.CENTER_BOTH
	},
	dom: {
		createContainer: true,
		behindCanvas: false
	}
};

Testing html:
<!doctype html>

	<meta name="viewport"
		content="width=device-width,initial-scale=1.0,minimum-scale=1.0,maximum-scale=1.0,user-scalable=no,shrink-to-fit=no">
	<% if(htmlWebpackPlugin.options.weinreScript) { %>

	<% } %>

Additional Information

Screenshot?
What are the contents of http://localhost:8080/index.html?

game html

CORRECT

iPhone

Finally, I can make it correct by modify the scaleManager as below. However, it should have more test.
//Method getParent()
if (config.expandParent && config.scaleMode !== CONST.SCALE_MODE.NONE)
{
	var DOMRect = this.parent.getBoundingClientRect();

	if (this.parentIsWindow || DOMRect.height === 0)
	{
		document.documentElement.style.height = '100%';
		document.body.style.height = '100%';

		DOMRect = this.parent.getBoundingClientRect();

		//  The parent STILL has no height, clearly no CSS
		//  has been set on it even though we fixed the body :(
		if (!this.parentIsWindow && DOMRect.height === 0)
		{

			this.parent.style.overflow = 'hidden';

			if(this.game.device.os.iOS && window.self !== window.top) {// iPhone, in iFrame
				this.parent.style.width = window.innerWidth + 'px';
				this.parent.style.height = window.innerHeight + 'px';
			} else {
				this.parent.style.width = '100%';
				this.parent.style.height = '100%';
			}
		}
	}
}

What are size.width and size.height?

I tried the original code, as best as I could reconstruct it, and it displayed correctly in iPhone 8 Simulator.

I would advise:

In game.html, add a doctype.
In game.html, is the canvas selector supposed to be #canvas'? You probably shouldn't style canvas`.
I'm not sure you need absolute positioning at all.


python310Packages.flake8-length: adjust build system

Description of changes
Fix build (https://hydra.nixos.org/build/209307153)

Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

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

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

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

[ ] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 217997 run on x86_64-linux 1

2 packages built:

python310Packages.flake8-length
python311Packages.flake8-length


Terraform required_version to be changed to 1.0.0 in sync with other modules like bastion,operator etc..

Community Note

Please vote on this issue by adding a 👍 reaction to the original issue to help the community and maintainers prioritize this request
Please do not leave "+1" or "me too" comments, they generate extra noise for issue followers and do not help prioritize the request
If you are interested in working on this issue or have submitted a pull request, please leave a comment

Description

New or Affected Resource(s)

Potential Terraform Configuration

# Copy-paste any Terraform configurations for how the requested feature may be used.

References

Let's schedule this change to the next major version.


C5477: Broken hyperlink due to duplicated reference

Hello, @MandiOhlinger,
Verified that reported content is duplicated and needs to be removed in order to avoid missing hyperlink in localized versions. Could you help to review this and merge this PR?
Many thanks in advance.

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

biztalk/technical-guides/checklist-performing-monthly-maintenance-checks.md
:white_check_mark:Succeeded
View

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

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

biztalk/technical-guides/checklist-performing-monthly-maintenance-checks.md
:white_check_mark:Succeeded
View

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

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

biztalk/technical-guides/checklist-performing-monthly-maintenance-checks.md
:white_check_mark:Succeeded
View

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


Migrate terra application doc

Summary
The swallow may fly south with the sun or the house martin or the plumber may seek warmer climes in winter yet these are not strangers to our land.
Thanks for contributing to Terra.
@cerner/terra

Closing this until we have a doc template.


[Doc Improvement] [Link updated]

Acrolinx Scorecards
We currently enforce a minimum score of 80.
Click the scorecard links for each article to review the Acrolinx feedback on grammar, spelling, punctuation, writing style, and terminology:

Article
Score
Issues
Scorecard
Processed

msteams-platform/concepts/build-and-test/deep-links.md
92
3
link
:white_check_mark:

More info about Acrolinx
Use the Acrolinx extension, or sidebar, in Visual Studio Code to check spelling, grammar, style, tone, clarity, and key terminology when you're creating or updating content. For more information, see Use the Visual Studio Code extension to run Acrolinx locally.


Error returns wrong starting year

Bug description
The error message for invalid headers prints year-1 - year as suggestion
Code snippet to reproduce the bug
python /validate_headers.py Pyronear 2022 --license Apache-2.0 --folders src,client,.github --ignore-files version.py,init.py --ignore-folders client/tests/,src/tests/ --license-notice in the repository https://github.com/pyronear/pyro-storage
Error traceback
The error message will be :
Your header should look like:

# Copyright (C) 2023-2024, Pyronear.

# This program is licensed under the Apache License 2.0.

However, the message should be :
 Your header should look like:

# Copyright (C) 2022-2024, Pyronear.

# This program is licensed under the Apache License 2.0.

as the starting year is 2022
Environment
Ran in a github action, irrelevant

Considering this message is common to all the detected files, would this suffice?
 Your header should look like:

# Copyright (C) <FILE_CREATION_YEAR>-2024, Pyronear.

# This program is licensed under the Apache License 2.0.

Because this isn't necessarily uniform in the codebase

Yep this would work :)


Allow backtracking

Backtracking, that is undoing sector crossings would allow to

undo an erroneous turn from one sector into another or
undo choosing the wrong sector to start with

without interrupting the input by lifting one's finger.

Alternative: Make backtracks meaningful, e.g. gesture as far as the desired base character, and then change direction to access the accented/decorated variants of that character.
I should probably catch up other issues' discussion around accented characters.

If I read https://github.com/flide/8VIM/issues/159#issuecomment-848730986 correctly, this is the current situation. I was not aware of this, partly because the F-Droid build comes without accents currently. I'm not sure how this can be reconciled generically with a way to undo turns without becoming terribly confusing.
Maybe we could establish that backtracking in the absense of "variants" (e.g. accented characters) undoes turns? That could be generalized to the point where we say that as soon as a backtrack does not match anything (think "variants of variants" by a 2nd backtrack were possible), it acts as "undo". However, this generalization seems to be half-baked as there would be no simple way to undo characters with variants.

I reconsidered with that new information and I am no longer in favour of this. The current meaning of backtracking to access variants (or more generally nested topologies) is a much better choice in my opinion. In analogy to how 8Vim attempts to replicate a natural flow, a small mistake (i.e. entering the wrong sector) should be immediately amendable (#240) much like you'd amend writing cursive. Larger mistakes (i.e. turning into the wrong direction) on the other hand are analogous to party writing a wrong letter, it necessarily comes with the penalty of explicit erasure.
I think the behaviour of backtracking when no variants match still needs to be defined (ignore?), but as far as I am concerned this FR should be rejected.

@ManDay, You have arrived at the conclusion that I would have voted for. Thanks :)


Remotely disable chat reporting on a whole server - client-only

Would it be possible to remotely disable chat reporting for the whole server, using a client-only mod? Perhaps by spamming DMs?

yeah totally


Show 6-player tables instead of empties

#136

[x] Draft log view shows 6 seats instead of 8 with empties.
[x] Draft log view shows team colors if teamDraft=true.
[x] Team draft logs save with teamDraft=true.

@Senryoku Review App, please :P

Looks good, thanks!


Sycl blas update

Re-organised the main README and made minor changes to the sample code to ensure it builds and runs correctly.
Deleted the samples README and consolidated this into the main README file to make it clear how users can run some sample code that uses the library.
Re-organised and updated the main README file to make the structure clearer for users, and to adopt a common format for READMEs across all our GitHub library projects.
Updated the sample code CMake file because as it was it could not be built separately, also changed the types of some variables based on help from @mehdi-goli because this was causing the samples  to fail to build.
Added a CONTRIBUTIONS.md file that will be used across our GitHub projects.

Hey @rodburns are you planning to continue with this change? I'm tempted to close the MR but we can always come back to it later.

Hey @rodburns are you planning to continue with this change? I'm tempted to close the MR but we can always come back to it later.

It slipped off my radar. I do think we should adopt some consistent template for our open source projects but perhaps we can re-visit it and create a new PR.

Alright I'm closing it and we can re-visit if this becomes a priority again.


[RFC] dotnet-xunit support

This is initial support for the dotnet-xunit tool provided by the XUnit framework for C#.

I added a few spec tests and I think that for the initial implementatino it may be enough. The problems arise when trying to handle some cases when the design of the test classes is not ideal, but I guess we can think about it at some later point.
Could someone review this and let me know what else I should add in order to get this merged? :) Thanks :)

Thank you so much for your contribution, very much appreciated. Could you also update doc/test.txt with the test command please.

@codeinabox, I rebased and squashed on top of the latest master and created an issue.


TLS inspector should not be installed by default in gateway

/cc @rshriram
Bug description
Affected product area (please put an X in all that apply)
[ ] Configuration Infrastructure
[ ] Docs
[ ] Installation
[x] Networking
[ ] Performance and Scalability
[ ] Policies and Telemetry
[ ] Security
[ ] Test and Release
[ ] User Experience
[ ] Developer Infrastructure
Expected behavior
Steps to reproduce the bug
Version (include the output of istioctl version --remote and kubectl version)
How was Istio installed?
Environment where bug was observed (cloud vendor, OS, etc)
Additionally, please consider attaching a cluster state archive by attaching
the dump file to this issue.

ignore.. its installed for all filter chains using tls. so nothing bad


Auth: open alexa app does not give me a notification and previous version redirect is incorrect

I'm running node-red on docker so I set "this ip" to <IP_ADDRESS> so I could access it.
The site loads but when it opens my Alexa app, no notification.
I click "previous version" but that redirects to <IP_ADDRESS> instead of the ip address I'm accessing it with.
Can you please fix the redirect to not use a host or use the ip address it is currently accessed by?
thanks!

Bump

Bump

bump


Plotly trisurface broken in 1.12.12

Following exception:
_trisurf() takes at least 7 arguments (7 given)
plotly\chart3d\init_graph.py

Fixed in #1099.


Design a process.status metric

Area(s)
area:system
Is your change request related to a problem? Please describe.
In https://github.com/open-telemetry/opentelemetry-collector-contrib/pull/33731 we received a proposal to add process.status as a resource attribute. Due to the changing nature of the process status within a process run, I believe it does not make sense as part of a process resource attribute. I'd like to engage the System Semantic Conventions working group to design it instead as a metric.
Describe the solution you'd like
I am not sure. :slightly_smiling_face:
Process Status is generally a string result, and on Linux at least is a restricted set of options: running, uninterruptible sleep, interruptable sleep, stopped, zombie. Metric design isn't my strong suit, so I would like to hear from someone with a good idea for this, or if anyone is able to point to a similar enum-style metric.
Describe alternatives you've considered
No response
Additional context
No response

cc @open-telemetry/semconv-system-approvers

I do see system.process.status in the semantic conventions. https://github.com/open-telemetry/semantic-conventions/blob/v1.26.0/docs/attributes-registry/system.md#system-process-attributes

Thank's @braydonk!
Regarding the immutability concern: we are facing this in several other places like at https://github.com/open-telemetry/semantic-conventions/pull/997#issuecomment-2129538490 and https://github.com/open-telemetry/semantic-conventions/issues/1160#issuecomment-2176190366. This seems to be another one.
Regarding the process.status, do you propose another attribute different than https://github.com/open-telemetry/semantic-conventions/blob/v1.26.0/docs/attributes-registry/system.md#system-process-attributes which @ishleenk17 also mentioned?

@tigrannajaryan This is another case for the need of mutable resource attributes.
Is there already some central place where the Entity WG is tracking these cases? If not would it make sense to have single issue to reference them all?

Is there already some central place where the Entity WG is tracking these cases? If not would it make sense to have single issue to reference them all?

This repository is the best place to track them. Entity SIG is not currently working on specific cases. We are working on generic data model. The work on specific cases for concrete entity types will happen in this repo.

The system.process.status attribute is used for the overall system process count metrics. It does cover the same concept as what we'd want to see here, however way you'd use it for an individual process couldn't be as a resource attribute due to the mutability issue. So in that case I am not sure where to use it.

Discussed in System Semantic Conventions meeting June 27. The only effective way to use process.status under the current data model is as a resource attribute, despite the mutability problem.
Recording the decisions:

I'm going to accept process.status as a resource attribute upstream in the hostmetricsreceiver.
I think system.process.status in the current semantic conventions spec should move to process.status to live with the other process attributes. I will add it as a resource attribute in semantic conventions as well after that's done.
When Entity data model starts to take shape, this will be adapted to that instead.

I will use this issue to attach to moving system.process.status, and I can either keep this issue open or create a new one to track how process is going to adapt to the upcoming Entity data model.

I'm going to accept process.status as a resource attribute upstream in the hostmetricsreceiver.

If we have decided to proceed with it, can we proceed with this PR: https://github.com/open-telemetry/opentelemetry-collector-contrib/pull/33731 ?

Yes that is the plan. I was going to finish my PR to semantic conventions
to add it as a resource attribute so we can follow it in that PR. I will
give it a review once I have done that.
On Fri, Jun 28, 2024, 1:10 AM Ishleen Kaur @.***> wrote:

I'm going to accept process.status as a resource attribute upstream
in the hostmetricsreceiver.

If we have decided to proceed with it, can we proceed with this PR:
open-telemetry/opentelemetry-collector-contrib#33731
https://github.com/open-telemetry/opentelemetry-collector-contrib/pull/33731
?
—
Reply to this email directly, view it on GitHub
https://github.com/open-telemetry/semantic-conventions/issues/1181#issuecomment-2196154477,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AWJXJSECKE3MDWP4ZTRNZNTZJTV3HAVCNFSM6AAAAABJ4SBQAGVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCOJWGE2TINBXG4
.
You are receiving this because you were mentioned.Message ID:
@.***>

Resource CANNOT change over the lifetime.  A few reasons:

The SDK specified resources are immutable which we rely on for many features.
Metrics identity is tied to resource.  If a resource attribute changes, you have disjoint metrics/timeseries.
Resource attributes are used as identifying attributes for bundling purposes.  Again, mutable attributes (or super high cardinality ones) can cause issues in the collector's handling/joining of data and on backends interpreting the data.
Mutable resource attributes are being considered for instances where the identity of the resource can change during lifetime of the SDK.  That does not mean a process changing status, but it DOES mean a browser creating a new client session, where want the resource to be the client session, not the browser.

Entities is a WIP and does not exist yet.  The goal of this group is find a way to allow descriptive attributes of resource which can change to be described/reported.  The discussion is still TBD - but it is NOT guaranteed (or likely in my opinion) that descriptive attributes, particularly problematic ones like process.status would end up in resource, for all the reasons mentioned above.
Having process status in the registry (https://github.com/open-telemetry/semantic-conventions/blob/v1.26.0/docs/attributes-registry/system.md#system-process-attributes) does NOT mean it is for resource.  This just means we have a consistent name for it regardless of where it shows up.
I am against adding process.status to Resource.  This would break so many requirements of the opentelemetry data model.
I would also reject this request in the collector.

Thank's for the feedback @jsuereth! I'm a bit confused though with what is the generic guidance here. Reading https://github.com/open-telemetry/semantic-conventions/issues/1160#issuecomment-2217998698 regarding the k8s.pod.ip attribute, seems to be in contrast with your comment 🤔? Would that be possible to clarify this mutable VS immutable topic horizontally for SemConv to ensure that we don't accidentally violate any ruling in any of the ongoing work-streams?

yes - Working on that, going to discuss with TC to make sure we have alignment, but for now let's stick to the guidance from this thread.

host.ip is an interesting conundrum.  I'd say in many cases, it's stable enough to be acceptable in resource.  In instances where it's unstable, and unstable frequently, we should not be using it and prefer a more stable id.  I expect decisions around that to fall out in Entity WG work - Specifically around allowing Entities to have identifying and descriptive attributes.  We're still working on core modelling.


feat!: Add sigstore verification to kwctl {pull, run}

kwctl {pull, run, verify} now accept new flags. The flags are similar to cosign et al flags, and multiple are allowed:

-a, --verification-annotation: also -a in cosign.
-k, --verification-key: corresponds to  -key in cosign. Clap only supports POSIX style short flags, not short flags that are more than 1 character.

Now:

kwctl pull -k key.pub -a env=prod ...  will verify prior to pulling, and then verify the local checksum against the manifest.
kwctl run -k key.pub -a env=prod ...  will verify prior to running the policy.
kwctl verify -k key.pub -a env=prod ...  will verify the policy.

Breaking change: kwctl verify flags for keys and verification annotations have also changed for consistency.
Note: kwctl {pul, run, verify} only works for registry:// protocol if passing keys, as the rest of verification features.
Note:  When missing annotations on verification, we fail with a bogus string. It should return instead: Annotation not satisfied missing_annotation="stable". This needs to be changed in sigstore-rs.
For that, we need to consume sigstore-rs-v0.1.0, which expects a rekor key for building the client. I wonder if we want to ask the users to pull a rekor key for verifying something, regarless of rekor functionality. I suggest gating rekor functionality under an experimental env var, as done in cosign. This can go in a follow-up PR.
Note: Refactoring calls to verify() so it uses references instead of cloning objects is not done yet, as it needs changes in sigstore-rs for annotations, and then to consume them in policy-fetcher.
Depends on https://github.com/kubewarden/policy-fetcher/pull/25
Fixes: https://github.com/kubewarden/kwctl/issues/81
Fixes: https://github.com/kubewarden/kwctl/issues/80

Right now the e2e tests for verification work against the image in ghcr.io/viccuad/pod-privileged:viccuad (see e2e-tests/sigstore/README.md). I suppose it would be a good idea to create and use a namespace under the Kubewarden org, such as  ghcr.io/kubewarden/tests/pod-privileged:v0.1.9. But I lack rights for that.

As a curiosity, one can see the signed manifest of the image under test on GH.

BTW, I'm about to push out a new iteration of sigstore-rs that will support verification of artifacts signed in keyless mode. This will require the end-users to provide not only rekor's public key, but also fulcio's certificate.
cosign has two ways to obtain these information:

Rely on some hard coded data
Fetch the data from Sigstore's TUF repository

Unfortunately the 2nd option is not feasible yet, because of a bug I'm currently working on (see https://github.com/sigstore/sigstore-rs/issues/9).
In the meantime I would go ahead and implement the 1st solution

Updated policy-fetcher to released v0.1.9 instead of my fork. Will rebase autosquash when it's approved.

rebased, ready to merge.


Can't pick image version to pull.

In my case, I want to create a container with the Odoo Version 8.
Currently, I only can add the latest version of an image.
Is there a way to select a different version.

Hi @GeorgSteyn. Do you sure that you use official version. Because I have a full list of versions.

@igor-lemon
Thank You for the quick response. I did not know that was possible.
Can there maybe an option at pull to select version?
Because currently, I start to pull then cancel it. Then select the version.

Huh. What is image name?
Is it https://hub.docker.com/_/odoo/ ?
I think that it doesn't have any relations with cancel of pulling image. Because info about versions gets from Docker Hub via image name, that gets from compose config.

Sorry, maybe I must rewind a little.
I create a new project => Then in the Scheme view. I drag and drop any image of any DockerHub repo and DockStation automatically pulls the latest version.
In my case wanting an older version of the image. I must start the pull of the image => Then cancel the pull and then change the version of the container image.
This is the current method I am using after you have shown me you can change the version.
Maybe I am using your tool wrong or this is a feature that needs to be added.
The feature being, when you are requesting to pull any image to first ask for which version to pull.

I understand. Yes, it has sense.


Flycheck fails to install

Installing flycheck seems to fail on OSX Yosemite.
Emacs is installed under Homebrew.
Emacs version: 24.4.1
Makeinfo version: 4.8
Error:
flycheck.texi:6: warning: unrecognized encoding name `UTF-8'.
flycheck.texi:86: @pxref expected braces.
flycheck.texi:86: ` {Installation}
and @ref{Quickstart} res...' is too long for expansion; not expanded.
flycheck.texi:86: First argument to cross-reference may not be empty.
/Users/james/.emacs.d/el-get/flycheck/doc//flycheck.texi:75: Next reference to nonexistent node `Introduction' (perhaps incorrect sectioning?).
/Users/james/.emacs.d/el-get/flycheck/doc//flycheck.texi:85: Cross reference to nonexistent node `Introduction' (perhaps incorrect sectioning?).
makeinfo: Removing output file `/Users/james/.emacs.d/el-get/flycheck/doc/flycheck.info' due to errors; use --force to preserve.

The error seemed to be related to #47, but this is the latest version of el-get.

How about  brew link texinfo --force and try again?

Great! That works!
Thanks for all your hard work.

You are welcome. Check https://github.com/xiaohanyu/oh-my-emacs/blob/c664894e2f1c1cb0f95a9f2da88d41b00f190856/core/ome-basic.org#homebrew and https://github.com/xiaohanyu/oh-my-emacs/issues/85 for technical details, if you're interested.

I did as you said but still failed.
The error info is the same with jamiees2

I did as you said but still failed.
The error info is the same with jamiees2

What's the output of which texinfo and texinfo --version?

I'm running on OS X 10.10
I have installed texinfo from homebrew and link them to my path.
However I don't have texinfo in my path, but I have info, makeinfo, etc.
info --version return 5.2.

Same here by the way :smile:

Hi jamiees,
your installation is successful now?

Yes, it is. Been running fine for 5 days.

@marksix did you take a look at #47 that I linked aswell? Might be something there aswell.

Also, what Emacs version are you using? The default OSX one isn't compatible. What is the output of emacs -version ?

I installed emacs from homebrew and built with imagmagick, gnutls.
emacs -version returned <IP_ADDRESS>

Hmm, then I don't know :tongue:

I run emacs in gui application and installation failed.
Now I run it within terminal and it succeeded.
I don't understand how it happens but it's working now.
Thank for your kindly help, friends.

This is really weird, I also run Emacs on Mac OS X 10.10 and I met no problems. I'll keep an eye on this problem.

Could it be that you forgot to run brew linkapps? The GUI version might still running the default OSX version, while emacs is the brew version.

Things on Mac OS X is a little complicated due to:

Mac has different package managers, macports/homebrew
Even homebrew has two ways to install emacs, one way is to install emacs by brew install emacs, and another way is to install emacs by brew cask install emacs
The GUI apps on Mac OS X has different $PATH with terminal(bash/zsh)

So it's not easy to track all these kind of problems in a simple way. The best way I can figure out is to provide binary distribution of oh-my-emacs, however, I still didn't get enough time to implement this.

I think that might be a little drastic. I would simply post a setup FAQ in the wiki on how to get emacs + oh-my-emacs running properly with OSX and homebrew.

I didn't know about the brew cask setup, but it might be better, as I had some issues with the Homebrew setup. I'll probably reinstall Emacs later using cask instead.

I do have written a section about Mac OS X installation: https://github.com/xiaohanyu/oh-my-emacs/blob/master/core/ome-basic.org#mac-os-x-compatibilities .
However, trying to build an .emacs.d from source(instead of binary, I mean, download and extract) may have some unexpected problems during installation. For instance, one man install oh-my-emacs successfully today but it may failed to install oh-my-emacs again. Oh-my-emacs depends on el-get, and el-get will download lots of things from internet, different packages have different dependencies, different package versions and may have incompatibility problems between Emacs 24.3 and 24.4 and 25.0.
So I think a better way to workaround this problem is https://github.com/xiaohanyu/oh-my-emacs/issues/99 . But I didn't got enough time to implement this for now, forgive me.

makeinfo 5.2 is working version for overwhelming the flycheck issue.(for osx Ver. 10.9.5-mavericks)
To get it work:
update your texinfo package with brew update && brew install texinfo
after installation complete you need to check which version of makeinfo is used.At my situataion it was still 4.8 version.(which makeinfo && makeinfo --version)
for standart homebrew installation all packeges are put under /usr/local/Cellar after brew install ...
so before moving the orginal makeinfo file go to /usr/local/Cellar and check if there is texinfo/5.2 folder or not.if it isnt there it is probably at the homebrew's user defined custom package folder.
sudo mv /usr/bin/makeinfo /usr/bin/mknfo.bkp
sudo ln -s /usr/local/Cellar/texinfo/5.2/bin/make /usr/bin/makeinfo
after this, run emacs and wait it finishes it job.(oh-my-emacs installation stuck at flycheck.)
meanwhile if you get any kind of "let-alist" error, install it from elpa repos: https://elpa.gnu.org/packages/let-alist.html (M-x install-package RET let-alist RET) this would solve the problem.


Log full back trace of error in unit tests

GraphQL catches exceptions and shows very limited amount of information from the error. This makes debugging unit tests almost impossible. It would be much better to log full back traces from errors.
For example following error message does not help much to understand the actual problem:
___________________________________________________ test_uuid ____________________________________________________

    def test_uuid():
        @strawberry.type
        class Query:
            @strawberry.field
            def example_uuid_out(self) -> uuid.UUID:
                return uuid.NAMESPACE_DNS

        schema = strawberry.Schema(Query)

        result = schema.execute_sync("{ exampleUuidOut }")

>       assert not result.errors
E       AssertionError: assert not [GraphQLError('', locations=[SourceLocation(line=1, column=3)], path=['exampleUuidOut'])]
E        +  where [GraphQLError('', locations=[SourceLocation(line=1, column=3)], path=['exampleUuidOut'])] = ExecutionResult(data=None, errors=[GraphQLError('', locations=[SourceLocation(line=1, column=3)], path=['exampleUuidOut'])], extensions={}).errors

tests/schema/types/test_uuid.py:17: AssertionError

The error message above was cause by this
--- a/strawberry/field.py
+++ b/strawberry/field.py
@@ -244,6 +244,7 @@ class StrawberryField(dataclasses.Field):
             return result

         def _strawberry_info_from_graphql(info: GraphQLResolveInfo) -> Info:
+            raise Exception()
             return Info(
                 field_name=info.field_name,
                 field_nodes=info.field_nodes,

Implementing the solution explained in #840 should fix this problem and make debugging much more pleasant.
I agree with you that right now debugging errors in GraphQL is like finding a needle in a haystack

The solution in #840 is related to the views, might not help our tests :)

We could use the same solution that is in #840 but instead of doing it in the process_result, we could do it in execute_sync (or whatever is the first point where we get the errors from GraphQL) so that it's a generic/unique solution that solves the problem with errors everywhere? Is it possible? 🤔

yes, that's probably a better idea, I wonder if there might be some downsides as you won't be able to  easily customise the behaviour, maybe we can do it on the schema class

We subclassed GraphQL to get this working:

class GraphQL(BaseGraphQL):

    async def process_result(self, request: Request, result: ExecutionResult) -> GraphQLHTTPResponse:
        data = await super().process_result(request, result)

        if result.errors and settings.TESTING:
            for error in result.errors:
                _error: Union[Exception, GraphQLError] = error
                if isinstance(error, GraphQLError) and error.original_error:
                    _error = error.original_error
                if settings.TESTING:
                    raise _error
                else:
                    # TODO This is where we would do explicit Sentry logging
                    logging.error(error, exc_info=error)
        return data

@la4de this PR should resolve this: #851

@la4de this PR should resolve this: #851

Excellent. Good job! :)


UITextView instance does not select or edit.

Dear WinObjC Engineers,
The attached project (scrollerror.zip) makes a UITextView instance. That text view instance can edit, select, become a responder, respond to key commands like copy and paste, etc. on an iPad, but not on WinObjC.
Since UITextView is a primary way to input text the keyboard processing (textual editing functions) should be implemented.
Sincerely,
Ed
scrollerror.zip

Thanks for reporting this, @edvv.
This is a known limitation with the current UITextView implementation (which is more of a 'placeholder' than a fully-implemented control.  We plan to port UITextView to a Xaml-backed control (likely RichEditBox) in the ~near future.  We haven't done an exhaustive comparison of UITextView vs. RichEditBox yet, but we're pretty sure that leveraging RichEditBox will prevent us from implementing two features:

Custom-rendered text (especially in an editable UITextView, but we'll likely start without any support for custom-rendered text in UITextViews)

Attributed text (although we may be able to find a way to add attributed text support at some point in the future)

We think the benefits to building on RichEditBox clearly outweigh the drawbacks, but any more details you can provide on your usage patterns would help guide us in the correct implementation strategy.
It would also be great to know how many of your scenarios this is blocking, to help with our bug/feature prioritization.

It's also worth noting that depending on your needs, you may be able to work around this by adding a RichEditBox to your scene and plumbing its events through the UIResponder chain yourself.  For example:
#import <UWP/WindowsUIXaml.h>
#import <UWP/WindowsUIXamlControls.h>
...
CGRect parentFrame = [(UIView*)parentView frame];
CGRect editBoxFrame = CGRectMake(0, 0, parentFrame.size.width, parentFrame.size.height);
[parentView addSubview:[[UIView alloc] initWithFrame:editBoxFrame xamlElement:[WXCRichEditBox make]]];

That code frag is helpful, I'll give that a try. This type of textual editing has been replaced except for one non-blocking use case. That use case is [non-attributed] plain text. I'll try out your RichEditBox frag hopefully next week (the codebase here is frozen now in a QA cycle and I need to wait for that to be over).

Great; let me know if you run into any issues with that approach.  A third option for non-attributed plain text entry would be to use a single or multi-line UITextField (which we've already backed by a Xaml TextBox).


JSON marshal on DomainEventLifecycle

I'm always getting Detail: 0 when I try to marshal DomainEventLifecycle into JSON. When I Printf the variable, detail is displayed ok:
Domain event="shutdown" detail="finished"
{"Event":6,"Detail":0}
Domain event="stopped" detail="shutdown"
{"Event":5,"Detail":0}

It looks like it has a String method which is what you see when you print:
func (e DomainEventLifecycle) String() string {

And inside it:
361:	DOMAIN_EVENT_SHUTDOWN_FINISHED = DomainEventShutdownDetailType(C.VIR_DOMAIN_EVENT_SHUTDOWN_FINISHED)
So maybe that value is actually zero??

detail=="finished" is string repr of  libvirt.DOMAIN_EVENT_SHUTDOWN_FINISHED which has value 0.
detail=="shutdown" is string repr of libvirt.DOMAIN_EVENT_STOPPED_SHUTDOWN which again has value 0.
So everything's working as intended.

tfw when "machine" Daniel Berrange beats you to the answer by 10 seconds :P

OK guys, sorry for that, my bad.


Update http links

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

#sign-off

@ecfan: I'm sorry - only the author of this article, @hedidin, can sign off on your changes. But we do have an exception process - if you are on the Microsoft content or product team for this product area, you can ask the PR review team to review and merge it by sending mail to the techdocprs alias.

I will review the changes
On 1/4/20, 11:04 AM, "Esther Fan"<EMAIL_ADDRESS>wrote:
#sign-off
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/MicrosoftDocs/azure-docs/pull/45477?email_source=notifications&email_token=ABUG4AYK34GNKBZZCU7P66DQ4C6SVA5CNFSM4KCDZCY2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEIC34BY#issuecomment-570801671, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ABUG4A7QWUYO3H4NSGUU4QDQ4C6SVANCNFSM4KCDZCYQ.

Only Microsoft employees can sign off on PRs. Your PR will be routed for review.

Merging per @ecfan


Feature/issue 1024 refactor header component

@bensinjin Yes for every new screen we will have to define a Header.


⚠ Bofrost.nl has degraded performance

In 9825d05, Bofrost.nl (https://www.bofrost.nl) experienced degraded performance:

HTTP code: 200
Response time: 1146 ms

Resolved: Bofrost.nl performance has improved in 9b80102.


Fix error in navigating directly to '/create' urls

Fixes error when navigating to /create urls without UI buttons.

Note to self: Probably avoid using useLocation and/or history state in the future...


Migrate away from assert-failures

Assertions aren't the best option for terminating processes, since they don't provide much in the way of logs. We should investigate alternatives (e.g. C++ exceptions) and migrate away from assert() calls.
This also affects our tests, where EXPECT_DEATH doesn't play nice with multithreading.

Resolved by PR #204


(refactor): Lerna

Investigating whether Lerna / Nx will fix #600 and also provide other benefits.

Should we consider Turborepo as well? The OpenJS Foundation has a partnership with Vercel so we could also get the paid version of it.

Should we consider Turborepo as well? The OpenJS Foundation has a partnership with Vercel so we could also get the paid version of it.

I already implemented Turborepo in the previous PR.  I'm revisiting Lerna / Nx for the comparison and also to see if it fixes #600.

I applied some of the things I tried here back in Turborepo land and I now think refining the approach there is the way to go.


Method name is not logged

Any idea why method name is not logged? I have used the following config but no luck.
Config:

Output:
DEBUG 2019-06-11 11:47:41,335   287ms ? Banana                 - Test log: Message from the main program

Hmm yes, something is wrong with this project, maybe I was distracted idk
I'll check this and update the repo when I've some free time. Thanks for the bug reporting! (I completely forgot that I had this repository lol)

Actually, I checked and I understand why it does not show the method name. It's because... It wasn't in the scope of my blog articles.
Anyway, I'll add this on my free time and surely update my blog posts as this is something more than useful (and I obviously use this every work day).
Thanks again! 😄

Hi again @kabilanvk,
I updated the repo and added a blog post (see the README.md file), you can now fetch the latest version and you should have the method name added into the log file.


onLogMessage is not merging state.context object

Reason: https://github.com/octoherd/octokit/pull/5#discussion_r589073758


fix: add space after prompt to look nicer

add space after prompt for fzf backend to look good
Context
Using fzf as backend, without this extra space, the prompt gets mixed with the ones we are typing making it unable to read like in the below screenshot

Even after disabling trim_prompts it looks like this which is little bit unclear to read

Description
It adds an extra space to the prompt making it look cleaner and seperates the input from prompt making it readable like in the screenshot below
Without trim prompts enabled

With trim prompts enabled

Test Plan
bootstrap_paq({
  	{
		"junegunn/fzf",
		build = ":call fzf#install()",
	},
	"junegunn/fzf.vim",
	"nomnivore/ollama.nvim",
	"stevearc/dressing.nvim",
})

require("ollama").setup({
	model = "codellama",
	url = "http://<IP_ADDRESS>:11434",
	-- serve = {
	--	on_start = false,
	--	command = "ollama",
	--	args = { "serve" },
	--	stop_command = "pkill",
	--	stop_args = { "-SIGTERM", "ollama" },
	-- },
	prompts = {
		Sample_Prompt = {
			prompt = "This is a sample prompt that receives $input and $sel(ection), among others.",
			input_label = "> ",
			model = "codellama",
			action = "display",
		},
	},
})

require("dressing").setup({
	select = {
		backend = { "fzf", "nui", "builtin" },
		trim_prompt = false,
	},
})

Thanks for the PR!


CV: 2020-2021 Aided a known fraudster to create his pump and dump scheme.

https://tinyurl.com/3xaej452 (changed link to removeddit since /r/monero mod removed to try to cover it up - maybe fluffy himself?)

Spam or truth? There is only one way to find out. Look into the sources yourself.

tari-project/tari#2928

tari-project/tari#2924

tari-project/tari#2922

tari-project/tari#2920

tari-project/tari#2927

tari-project/tari#2923

tari-project/tari#2921

tari-project/tari#2919


Correction for raw affiliation Department of Psychology, King's University College at Western University, London, Canada

Correction needed for raw affiliation Department of Psychology, King's University College at Western University, London, Canada
raw_affiliation_name: Department of Psychology, King's University College at Western University, London, Canada
new_rors: 02grkyz14;
previous_rors: 02grkyz14;048f5b183
works_examples: W4386407042
contact: 76a0b72bad86ef514c030bc6af1502a8:079b62bff3386c6e @ uwo.ca

This issue was accepted and ingested by the OpenAlex team on 2024-12-21. The new affiliations should be visible within the next 7 days.


Use cacheKey if no font name provided

prevents fonts without a name property from overwriting each other in the cahce.

Fixed in a slightly different way in d35f7cc but just not saving the font to the cache at all if there is no name. We already checked by cache key above (line 34).


Composer Install issue

Warning: Ambiguous class resolution, "App\Services\User\SignUpService" was found in both "C:/Desktop/project/react-laravel-boilerplate/app\Services\User\SignUpService.php" and "C:/Desktop/project/react-laravel-boilerplate/app\Services\User\UpdateUserService.php", the first will be used.

Sorry yes this is a fault of mine, I keep on developing in the master branch, I guess with more eyes on the project I should stop doing that, if you pull the latest master it should be fine now.


Have two JFlex editor colour panes

One with correct colours, other with wrong colours. Idea uses the wrong set of colours.

This is due to Grammar-Kit also supplying support for JFlex. I have uninstalled this plugin for now.


VSCode TreeDataProvider does not reload icons with same URI after they change on disk and a refresh call is made.

I'm developing a View extensions with the TreeDataProvider. Users are able to right click on an item in the tree, opening the showOpenDialog window. There they can pick an svg or png image file to replace the current icon used for the selected Tree Item. Once chosen the new image overwrites the current icon file with the same name in the same path (same URI). Then the TreeDataProvider is refreshed using the onDidChangeTreeData event. On refresh the new icon does not load. Icons with different URI's will reload. The new icon only appears after a window reload. Am I missing a call in the API to force reload the URI, otherwise this would seem like a bug.
Version: 1.41.0-insider (user setup)
Commit: a747d264d668a947eeacd3492b60df6ce8dee6b4
Date: 2019-11-23T11:09:42.834Z
Electron: 6.1.5
Chrome: 76.0.3809.146
Node.js: 12.4.0
V8: 7.6.303.31-electron.0
OS: Windows_NT x64 10.0.19025
Steps to Reproduce:

Using the TreeDataProvider API load a custom View into Explorer with icons
Change out one of the icons on disk with another icon of the same name and path.
Refresh the TreeDataProvider without a window reload.
Note the icon with same URI will not change.

Does this issue occur when all extensions are disabled?:
This issue relates to an extension feature, therefore could not be tested without extensions.

@tomsaunders-code do you have some sample code a repo I could clone to make testing/verifying easier?

@alexr00 absolutely, checkout master on the extension repo:
https://gitlab.com/tomsaunders-tools/vscode-workspace-explorer
The file doing the heavy lifting:

/src/workspaceTreeItem.js - Tree Item and functions to find images on treeDataProvider refresh event.

To reproduce you can do the following in this extension:

Set the workspaceExplorer.workspaceStorageDirectory configuration
Add/save a workspace to the directory specified in the configuration
You should now have a .code-workspace file in your  workspaceExplorer.workspaceStorageDirectory directory
Refresh the extension's explorer view
Right click on the workspace you created
Choose change icon and pick a png or svg. Do not reload the window.
The extension's explorer view should refresh the tree and you'll see the icon next to your workspace name
Right click and change the icon again or open up the workspaceExplorer.workspaceStorageDirectory and save a different image file as the same name as your workspace (overwriting the first image you saved with the right click menu)
Hit the extensions explorer view refresh button
Note the icon for your workspace does not change as expected

Thanks!

I did a bit of learning, and this is actually intentional(and normal for browsers). If we reloaded icons from disk ever time we rendered something, we could take a performance hit depending on how many icons there are. However, there is a work around for you.
Your extension returns strings for the light and dark properties of the iconPath. If instead you return a Uri and have a query that changes when you know that your icon changed, then it will be loaded from disk again. As a very silly example:
let x = 0;

const getCustomWorkspaceIcons = function getCustomWorkspaceIcons(
  workspaceFileNameAndFilePath,
  collapsableState,
  extensionConfig,
) {
  ...
    return {
      light: vscode.Uri.file(svg).with({query: `x=${x++}`}),
      dark: vscode.Uri.file(svg).with({query: `x=${x++}`}),
    };
  }

Clearly you wouldn't want to change the query every time (otherwise your extension could perform worse), only when you know the icon changed.


text needs to be changed  for Endpoint Hight Traffic

probably need to be fixed also in  Endpoint Low Traffic
change the Duration column to Calls

Closed as duplicate of #641


fix(ROX-15587): expose metrics on different port to rest of server

Moved metrics to a different port, so they aren't world-readable anymore.
Verify with the cluster linked in the comment below:

kubectl port-forward deploy/infra-server-deployment -n infra 9101:9101
Access https://localhost:9101/metrics

Thanks for taking care of this!


Fix application modal and routing

Issue Ticket Number
closes #784, #788
Description

Automatic Modal Display: When a superuser navigates to /application?id=, the modal will automatically be displayed.

Button State Management: If the application has already been accepted or rejected, the corresponding action button will be disabled.

Dynamic URL Updates: The URL will update dynamically based on the filters applied, reflecting the selected filter parameters.

API Limit Adjustment: The API has been updated to fetch 6 applications at a time, ensuring that all available space in the application cards is utilized effectively.

Enable applications button on dashboard page

Documentation Updated?

[ ] Yes
[ ] No

Under Feature Flag

[ ] Yes
[ ] No

Database Changes

[ ] Yes
[x] No

Breaking Changes

[ ] Yes
[x] No

Development Tested?

[x] Yes
[ ] No

Screenshots
https://github.com/user-attachments/assets/3188cec4-1299-4359-b65b-8c33df94d3aa

For test fix and enhancement here is the issue ticket
https://github.com/Real-Dev-Squad/website-dashboard/issues/791


Create LICENSE.md using Apache 2.0 license

Ticket
Related to https://wicmtdp.atlassian.net/browse/WMDP-83
Changes

What was added, updated, or removed in this PR.

Adds the Apache 2.0 license as LICENSE.md

Context for reviewers

Testing instructions, background context, more in-depth details of the implementation, and anything else you'd like to call out or ask reviewers. Explain how the changes were verified.

We want to release this work under an open source license. We have selected Apache 2.0 to maximize ease of adoptability.
Also approved in https://github.com/navapbc/wic-mt-demo-project-eligibility-screener/pull/15
Testing

Screenshots, GIF demos, code examples or output to help show the changes working as expected. ProTip: you can drag and drop or paste images into this textbox.

N/A

@aplybeah Do you mind approving this one too? It's the same as the other.


reimplement overdue-section in timed agendas

The old morgue had it. We should implement it again, some of the infrastructure is already present.

This is implemented in morgue versions from <IP_ADDRESS> onwards.


Lwm2m data cache api

New API for enable Historical data storage for selected LwM2M resource.
Data cache is only supported at resource which resource size is static and well known:
• Signed and unsigned 8-64-bit resources
• Float
• Boolean
Requires time() to provide current Unix timestamp (CONFIG_POSIX_CLOCKS)
Object Writer Update
Extend output writer API for write cached data timestamp.
SenML-CBOR Encoder and CDDTL update
Added support for historical data encode by adding base time (bt) and time (t) label.
New labels are needed for Encoder so Decoder is not regenerated.
Added support for SenML-CBOR to write time series data.
Use "bt" base time and "t" timestamp labels for data cache.
SenML-JSON support for Historical data
Extend SenML-JSON content format for handle cached data timestamp API for base time and timestamp label's.
Added support for write historical data for static resource size's: Float (v) and Boolean (vb).

@SeppoTakalo , @jarlamsa @VeijoPesonen Please review.

@VeijoPesonen and @jheiskan81 I have fix your comments

@rlubos thanks for comment's and good findings.

rebased withmain branch

@jarlamsa @SeppoTakalo @rlubos Please review. I start use Zephyr Ring buffer API to data cache. If You think is it good idea I can squash latest commit to first one. ring buffer structure takes few 32-bit parameter more than my own implementation. Now I need to to cast byte pointer to structure internally.

@SeppoTakalo I fix your comment's. @jarlamsa I fix lenght typo's

Dropped ringbuffer integration away for avoid structure aligment's.

@SeppoTakalo @jarlamsa I change a cache read and write API and also put Zephyr ring buffer integration. I have tested cache now with boolean and float cache write and read. If you are happy with I squash commit to 1st commit.

fixed compliance issues.

Rebased with main branch

I squash Ring buffer integration for replace own ring buffer. I added to requirement for CONFIG_RING_BUFFER to documentation also.
@rlubos @SeppoTakalo @jarlamsa @VeijoPesonen Are we now ready for merge?

rebased with main branch


License

Since this code is on github, I assume that you'd allow other people to use it.
Could you add a License? (Or tell me where I can find it, in case I missed it).

I will update license shortly.

Added in cfbc4a16


Support for DOUBLECLICK

Hi friends.
Is your ANE compatible with DOUBLECLICK FOR DEVELOPERS??

I don't families with DOUBLE-CLICK. You means by Google SDK? Or?


Crash with looking up JEI item usage

Issue Description:
When I look at an items uses, like a wooden slab for example, I get a crash that I suspect is because of the painter as the crash log seems to show, it appears to error with a liquid from tech reborn. (I will most likely make an issue report on their issue tracker aswell)
Unsure if this is the same as this issue as the crashes seem to originate from the painter, but seem to be for different reasons.
I have already tried to set:
# When enabled blocks with tile entities (e.g. machines) can be used as paint targets. (synced from server) [default: true]
        B:allowTileEntitiesAsPaintSource=false

and
# If true, only a handful of sample painter recipes will be shown in JEI. Enable this if you have timing problems starting a world or logging into a server. [default: true]
    B:jeiUseShortenedPainterRecipes=false

to false for a possible fix to no avail.
What happens:
The games crashes right upon trying to look as uses of an item with JEI
What you expected to happen:
To show me uses without crashing
Steps to reproduce:

have tech reborn installed along with JEI and ender io
Open up JEI
Look for uses on a wooded slab (or others, haven't tested on others as of yet)

Affected Versions:

EnderIO: 5.0.29
EnderCore: 0.5.35
Minecraft: 1.12.2
Forge: <IP_ADDRESS>39
SpongeForge? no
Optifine? yes
Single Player and/or Server? Both

Your most recent log file where the issue was present:
Latest Log

Dulpicate #4744
You were close

Read the error message and don't shoot the messenger.

java.lang.RuntimeException: Fluid nc.fluid.FluidCoolant@6d120ce3 (redstone_nak, class nc.fluid.FluidCoolant) from item 1xitem.techreborn.cell@0 returned an unregistered block from getBlock(). This is a major bug in the mod that fluid belongs to.

You can fix this by going into the nuclearcraft config and enabling fluid blocks... but backup your world first...  the next update of nuclearcraft should have this fixed whenever it releases.


[Python][Packaging] In python3.10, report GCC_7.0.0 not found

Describe the bug, including details regarding any error messages, version, and platform.

Component(s)
Python

Could you share which version of pyarrow are you using and how did you install, please?

Could you share which version of pyarrow are you using and how did you install, please?

The version is pyarrow-16.0.0.

Could you share which version of pyarrow are you using and how did you install, please?

BTW, I try to compile pyarrow to use in other docker。But, I get a error like next. Package comman is python setup.py build_ext --build-type=Release --bundle-arrow-cpp bdist_wheel.

Can you share a minimal Dockerfile to reproduce? I see you are using redhat with GCC 7 but is hard to tell how you are building Arrow CPP, installing dependencies, etcetera to reproduce.
Is this happening only with Arrow 16.0.0 or was happening before? We did release Arrow 16.0.0 a couple weeks ago and I want to discard any issues there. Thanks

Can you share a minimal Dockerfile to reproduce? I see you are using redhat with GCC 7 but is hard to tell how you are building Arrow CPP, installing dependencies, etcetera to reproduce. Is this happening only with Arrow 16.0.0 or was happening before? We did release Arrow 16.0.0 a couple weeks ago and I want to discard any issues there. Thanks

Thank you for your attention. I understand now. It seems the error was due to an incorrect version of GLIBC in your Docker image.


SwiftIfConfig: A library to evaluate #if conditionals within a Swift syntax tree.

Swift provides the ability to conditionally compile parts of a source file based on various built-time conditions, including information about the target (operating system, processor architecture, environment), information about the compiler (version, supported attributes and features), and user-supplied conditions specified as part of the build (e.g., DEBUG), which we collectively refer to as the build configuration. These conditions can occur within a #if in the source code, e.g.,
func f() {
#if DEBUG
  log("called f")
#endif

#if os(Linux)
  // use Linux API
#elseif os(iOS) || os(macOS)
  // use iOS/macOS API
#else
  #error("unsupported platform")
#endif
}

The syntax tree and its parser do not reason about the build configuration. Rather, the syntax tree produced by parsing this code will include IfConfigDeclSyntax nodes wherever there is a #if, and each such node contains the a list of clauses, each with a condition to check (e.g., os(Linux)) and a list of syntax nodes that are conditionally part of the program. Therefore, the syntax tree captures all the information needed to process the source file for any build configuration.
The SwiftIfConfig library provides utilities to determine which syntax nodes are part of a particular build configuration. Each utility requires that one provide a specific build configuration (i.e., an instance of a type that conforms to the doc:BuildConfiguration protocol), and provides a different view on essentially the same information:

doc:ActiveSyntaxVisitor and doc:ActiveSyntaxAnyVisitor are visitor types that only visit the syntax nodes that are included ("active") for a given build configuration, implicitly skipping any nodes within inactive #if clauses.
SyntaxProtocol.removingInactive(in:) produces a syntax node that removes all inactive regions (and their corresponding IfConfigDeclSyntax nodes) from the given syntax tree, returning a new tree that is free of #if conditions.
IfConfigDeclSyntax.activeClause(in:) determines which of the clauses of an #if is active for the given build configuration, returning the active clause.
SyntaxProtocol.isActive(in:) determines whether the given syntax node is active for the given build configuration.

There are a few things I'd still like to do before we can call this "complete":

[ ] Create a build configuration that's populated by swift-driver, so that a client with command-line arguments can get mostly-accurate #if information by only using swift-driver and swift-syntax. (#if canImport(...) effectively requires a compiler).
[ ] Create a build configuration in the compiler itself, so we can use these utilities from the compiler in lieu of it's C++ implementation.
[ ] Add a "fixed" or "simple" build configuration instance based on the host, which clients can use if they truly have no information to go on (e.g., because they don't have build settings).
[ ] Improve error handling and figure out how to deal with warnings. Right now, we either throw if there's an error, or swallow the error and treat that region as inactive.
[ ] Improve documentation with more examples.
[ ] Update the SourceKit query for active ranges to use these utilities rather than walking the compiler's semantic AST.
[ ] Teach the various checks to distinguish "inactive" from "unparsed" regions.

@swift-ci please test

I would love to use this in the part of the compiler that strips inactive #if conditions out of module interfaces

@swift-ci please test

@swift-ci please test Windows

@swift-ci please test

Alright, major update for this library today, including:

Introduce complete handling for diagnostics produced via #if condition evaluation.
Add nice test harnesses for all of the APIs.
Distinguish between "inactive" and "unparsed" regions so we know where to emit syntactic diagnostics.

@swift-ci please test

@swift-ci please test

I forgot about PostfixIfConfigExprSyntax, so I'll have to fix that before this becomes mergable.

Turns out that PostfixIfConfigExprSyntax works. I've renamed IfConfigState to ConfiguredRegionState, which I think I like better. Although it's pretty long.

@swift-ci please test

@swift-ci please test

More improvements:

Improve handling of ill-formed code so that we treat an ill-formed condition as "unparsed" consistently and aligns with the compiler
Cleaned up file organization
Added support for the new _hasAtomicBitWidth in the compiler

@swift-ci please test

@swift-ci please test Windows

@swift-ci please test Windows

Addressed most of these comments in https://github.com/swiftlang/swift-syntax/pull/2759

Thanks for addressing the comments. I’m addressing a few more minor ones in https://github.com/swiftlang/swift-syntax/pull/2762.
The following discussions are still open. Composed this list partially for myself to keep track.

[ ] Using a single pass in ActiveSyntaxRewriter

https://github.com/swiftlang/swift-syntax/pull/1816#discussion_r1681126213

[ ] Emitting diagnostics in case of errors instead of ignoring errors / trapping

https://github.com/swiftlang/swift-syntax/pull/1816#discussion_r1681142474
https://github.com/swiftlang/swift-syntax/pull/1816#discussion_r1681151598
https://github.com/swiftlang/swift-syntax/pull/1816#discussion_r1681155777

[ ] Renaming versioned to eg. syntaxErrorsAllowed

https://github.com/swiftlang/swift-syntax/pull/1816#discussion_r1681211064

[ ] Document how #if regions are handled if BuildConfiguration throws

https://github.com/swiftlang/swift-syntax/pull/1816#discussion_r1681274837

[ ] Consider the use of Identifier or a similar concept

https://github.com/swiftlang/swift-syntax/pull/1816#discussion_r1693914616
https://github.com/swiftlang/swift-syntax/pull/1816#discussion_r1693879981

[ ] Support canImport with build numbers

https://github.com/swiftlang/swift-syntax/pull/1816#discussion_r1681365980

[ ] Figure out where 9223371 is coming from

https://github.com/swiftlang/swift-syntax/pull/1816#discussion_r1681366653


Update “Version Date Range” to “Publication Date Range”

Describe the solution you'd like
Update “Version Date Range” to “Publication Date Range” depending on the context of which project is selected.
Some projects 'Version Date Range' is appropriate, and for some it's better to show 'Publication Date Range'.
-Figure out how to determine which label to use, based on what project is selected.

The "version" attribute is in date format for most projects.  Eg for E3SM - need to change the search widget to not use a calendar.
For "publication date" - for E3SM we need to add &from=&to=  and convert to the proper Solr ISO dates.  This could be a separate issue.

what we need is a conditional for projects that don't use the vYYYYMMDD format.  eg E3SM = false, CMIP6, etc = true


Filter weather alerts by using SAME codes?

Is it possible to have the weather alerts filtered by SAME codes (up to 3 or 5)?
Weather stations broadcast on a vast territory and alerts are often only for a specific county.
From https://www.weather.gov/iln/nwr_same:
At the beginning of the product transmission, 3 bursts of digital static are heard (which contains 6 digit SAME code, amongst other things)
Followed by 10 second 1050hz alarm tone
Followed by the actual audio message.
Has anyone ever looked into this? Would it even be feasable...I have doubts because of the "digital static" part possibly couldn't be demodulated? To my knowledge, this would make the Quansheng radios the only ham radios to have this functionality.

NOAA is standard NOT enabled in this firmware.
There is no use for it in Europa.
See README.md
ENABLE_NOAA                   := 0       everything NOAA (only of any use in the USA)

From the datasheet it looks like the RF chip supports the NOAA SAME code decoding, but I'm from Europe where there is no NOAA system, I have no motivation to work on that.


Mysql parser error with CASE operator and IF function

Which version and edition of Flyway are you using?
Flyway Community Edition 6.5.5 by Redgate
If this is not the latest version, can you reproduce the issue with the latest one as well?
(Many bugs are fixed in newer releases and upgrading will often resolve the issue)
Which client are you using? (Command-line, Java API, Maven plugin, Gradle plugin)
Command-line and Java API
Which database are you using (type & version)?
mysql 5.7.31 (installed from homebrew)
Which operating system are you using?
Mac OS 10.15.6
What did you do?
(Please include the content causing the issue, any relevant configuration settings, the SQL statement that failed (if relevant) and the command you ran.)
We recently upgraded the Flyway version we use via the Java API from 4.1.2 to 6.5.2 which caused an existing migration script to fail when initialising a new database. Confirmed that the issue is still present in 6.5.5 with both the Java API and command-line versions.
Sample script that generates a parse error:
CREATE TABLE `test_table` (
    `id` bigint(20) unsigned NOT NULL AUTO_INCREMENT,
    `field` varchar(64) DEFAULT NULL,
    PRIMARY KEY (`id`)
) ENGINE=InnoDB DEFAULT CHARSET=utf8;

CREATE TABLE `test_join` (
    `id` bigint(20) unsigned NOT NULL AUTO_INCREMENT,
    `test_table_id` bigint(20) unsigned NOT NULL,
    `field1` varchar(64) NOT NULL DEFAULT '',
    `field2` varchar(64) NOT NULL DEFAULT '',
    PRIMARY KEY (`id`)
) ENGINE=InnoDB DEFAULT CHARSET=utf8;

UPDATE `test_table` t
LEFT JOIN test_join j
    ON t.id=j.test_table_id and j.field1 in ('VALUE_1', 'VALUE_2', 'VALUE_3')
SET t.field =
    CASE j.field1
        WHEN 'VALUE_1' THEN IF(STRCMP(j.field2, 'A')=0, 'VALUE_A1', 'VALUE_A2')
        WHEN 'VALUE_2' THEN 'VALUE_B'
        WHEN 'VALUE_3' THEN 'VALUE_C'
        ELSE NULL
    END;

What did you expect to see?
Database migration script successfully applied.
What did you see instead?
Parse error:
ERROR: Unable to parse statement in /Users/amccarthy/Temp/flyway-parse/V002__Test.sql at line 15 col 1. See https://flywaydb.org/documentation/knownparserlimitations for more information: Incomplete statement at line 15 col 1: UPDATE `test_table` t
LEFT JOIN test_join j
    ON t.id=j.test_table_id and j.field1 in ('VALUE_1', 'VALUE_2', 'VALUE_3')
SET t.field =
    CASE j.field1
        WHEN 'VALUE_1' THEN IF(STRCMP(j.field2, 'A')=0, 'VALUE_A1', 'VALUE_A2')
        WHEN 'VALUE_2' THEN 'VALUE_B'
        WHEN 'VALUE_3' THEN 'VALUE_C'
        ELSE NULL
    END;

Caused by: Incomplete statement at line 15 col 1: UPDATE `test_table` t
LEFT JOIN test_join j
    ON t.id=j.test_table_id and j.field1 in ('VALUE_1', 'VALUE_2', 'VALUE_3')
SET t.field =
    CASE j.field1
        WHEN 'VALUE_1' THEN IF(STRCMP(j.field2, 'A')=0, 'VALUE_A1', 'VALUE_A2')
        WHEN 'VALUE_2' THEN 'VALUE_B'
        WHEN 'VALUE_3' THEN 'VALUE_C'
        ELSE NULL
    END;

The script is valid mysql sql. The parse error can be worked around by surrounding the CASE operator with parentheses, but this is not a solution as that would cause checksum errors on existing deployments.
CREATE TABLE `test_table` (
    `id` bigint(20) unsigned NOT NULL AUTO_INCREMENT,
    `field` varchar(64) DEFAULT NULL,
    PRIMARY KEY (`id`)
) ENGINE=InnoDB DEFAULT CHARSET=utf8;

CREATE TABLE `test_join` (
    `id` bigint(20) unsigned NOT NULL AUTO_INCREMENT,
    `test_table_id` bigint(20) unsigned NOT NULL,
    `field1` varchar(64) NOT NULL DEFAULT '',
    `field2` varchar(64) NOT NULL DEFAULT '',
    PRIMARY KEY (`id`)
) ENGINE=InnoDB DEFAULT CHARSET=utf8;

UPDATE `test_table` t
LEFT JOIN test_join j
    ON t.id=j.test_table_id and j.field1 in ('VALUE_1', 'VALUE_2', 'VALUE_3')
SET t.field =
    (CASE j.field1
        WHEN 'VALUE_1' THEN IF(STRCMP(j.field2, 'A')=0, 'VALUE_A1', 'VALUE_A2')
        WHEN 'VALUE_2' THEN 'VALUE_B'
        WHEN 'VALUE_3' THEN 'VALUE_C'
        ELSE NULL
    END);

Fixed in Flyway 6.5.6

Hi @Lyeeedar,
This doesn't appear to be fixed with the command-line version. Attempting to apply the first script to an empty database still produces the error.
Additionally other scripts that use the CASE operator are no also failing. These scripts were parsed and applied successfully in version 6.5.5. For example this script which create a stored procedure with an ORDER BY CASE. As with the previous example the parser error does not happen if the CASE operator is wrapped in parentheses.
DELIMITER //
CREATE PROCEDURE test_procedure(in startTime dateTime, in endTime dateTime, in showTotals enum('Y','N'))
BEGIN
    IF showTotals = 'Y' THEN
        SELECT * FROM test_table
        ORDER BY CASE WHEN `name` is null then 1 else 0 end, `name`;
    ELSE
        SELECT * from test_table;
    END IF;
END //
DELIMITER ;

The error:
ERROR: Unable to parse statement in /Users/amccarthy/Temp/flyway-parse/V003__Test.sql at line 2 col 1. See https://flywaydb.org/documentation/knownparserlimitations for more information: Delimiter changed inside statement at line 2 col 1: CREATE PROCEDURE test_procedure(in startTime dateTime, in endTime dateTime, in showTotals enum('Y','N'))
BEGIN
    IF showTotals = 'Y' THEN
        SELECT * FROM test_table
        ORDER BY CASE WHEN `name` is null then 1 else 0 end, `name`;
    ELSE
        SELECT * from test_table;
    END IF;
END //

Caused by: Delimiter changed inside statement at line 2 col 1: CREATE PROCEDURE test_procedure(in startTime dateTime, in endTime dateTime, in showTotals enum('Y','N'))
BEGIN
    IF showTotals = 'Y' THEN
        SELECT * FROM test_table
        ORDER BY CASE WHEN `name` is null then 1 else 0 end, `name`;
    ELSE
        SELECT * from test_table;
    END IF;
END //

We are very sorry that the release did not fix your issue, we will look into a more comprehensive fix for 6.5.7. To help with our tests, would you be interested in providing more sql for us to test our parsing against? The more we have the better we can make the parser.

@Lyeeedar we are failing on the same issue, here is our example:
UPDATE notification_subscriber t
SET
    t.customer_id = CASE
        WHEN project_id = 1 THEN 1
        WHEN project_id = 2 THEN 2
        WHEN project_id = 3 THEN 1
        WHEN project_id = 4 THEN 3
        WHEN project_id = 5 THEN 4
        WHEN project_id = 9 THEN 8
        WHEN project_id = 10 THEN 9
    END;

DELETE FROM notification_subscriber where project_id = 1;

Hi @Lyeeedar,
I have just been through all of our flyway migration scripts and the only scripts that we are currently experiencing parse failures (for version 6.5.6) are those mentioned above.
Scenario 1 - Use of CASE operator in SET clause of UPDATE query
Scenario 1 is failing in version 6.5.5 and still in version 6.5.6.
CREATE TABLE `test_table` (
    `id` bigint(20) unsigned NOT NULL AUTO_INCREMENT,
    `field` varchar(64) DEFAULT NULL,
    PRIMARY KEY (`id`)
) ENGINE=InnoDB DEFAULT CHARSET=utf8;

CREATE TABLE `test_join` (
    `id` bigint(20) unsigned NOT NULL AUTO_INCREMENT,
    `test_table_id` bigint(20) unsigned NOT NULL,
    `field1` varchar(64) NOT NULL DEFAULT '',
    `field2` varchar(64) NOT NULL DEFAULT '',
    PRIMARY KEY (`id`)
) ENGINE=InnoDB DEFAULT CHARSET=utf8;

UPDATE `test_table` t
LEFT JOIN test_join j
    ON t.id=j.test_table_id and j.field1 in ('VALUE_1', 'VALUE_2', 'VALUE_3')
SET t.field =
    CASE j.field1
        WHEN 'VALUE_1' THEN IF(STRCMP(j.field2, 'A')=0, 'VALUE_A1', 'VALUE_A2')
        WHEN 'VALUE_2' THEN 'VALUE_B'
        WHEN 'VALUE_3' THEN 'VALUE_C'
        ELSE NULL
    END;

Scenario 2 - Use of CASE operator in ORDER BY clause
Scenario 2 only started failing in version 6.5.6.
Our migration scripts use this when creating a stored procedure:
DELIMITER //
CREATE PROCEDURE test_procedure(in startTime dateTime, in endTime dateTime, in showTotals enum('Y','N'))
BEGIN
    IF showTotals = 'Y' THEN
        SELECT * FROM test_table
        ORDER BY CASE WHEN `name` is null then 1 else 0 end, `name`;
    ELSE
        SELECT * from test_table;
    END IF;
END //
DELIMITER ;

But the same parse error occurs when directly executing the query in the migration script:
CREATE TABLE `test_table_2` (
    `id` bigint(20) unsigned NOT NULL AUTO_INCREMENT,
    `name` varchar(64) DEFAULT NULL,
    PRIMARY KEY (`id`)
) ENGINE=InnoDB DEFAULT CHARSET=utf8;

SELECT *
FROM test_table_2
ORDER BY CASE WHEN `name` is null then 1 else 0 end, `name`;

Situations where use of the CASE operator parses correctly
We use the CASE operator in the following situations that are currently being parsed correctly.
Part of the SELECT clause
SELECT CASE WHEN `name` is null THEN 1 ELSE 0 END as value
FROM test_table_2;

Inside an aggregate function
SELECT SUM(CASE WHEN `name` is null THEN 1 ELSE 0 END) as value
FROM test_table_2;

@yuvalprtn @amccarthy Thank you for these examples, I will make sure that our next fix fully supports these cases. If you are happy to share more of your scripts, you can post them here or email them to me at<EMAIL_ADDRESS>


bgfx.cpp does not compile

It fails on this line
https://github.com/bkaradzic/bgfx/blob/07aae82d16d742efd36f4023f30ec2d77c390bee/src/bgfx.cpp#L511
../../../../../3rdparty/bgfx/src/bgfx.cpp: In function 'uint8_t bgfx::parseAttrTo(char*&, char, uint8_t)':
../../../../../3rdparty/bgfx/src/bgfx.cpp:511:4: error: 'strlncpy' is not a member of 'bx'
    bx::strlncpy(tmp, sizeof(tmp), _ptr, len);
    ^

CI build status disagrees...
https://ci.appveyor.com/project/bkaradzic/bgfx
https://travis-ci.org/bkaradzic/bgfx

it must be GCC playing up then.

I checked locally with GCC before I checked CI. CI also uses GCC:
https://travis-ci.org/bkaradzic/bgfx/jobs/186470884
I kicked another build just in case, but probably your setup got messed up. Check do you have any changes in bx.

It happened after the update. I'm building it with no changes to bgfx atm. I've touched all the bgfx files, but to avoid some leftovers i'll have to rebuild the whole project i suppose. it will take 2h :/


Consider changing names of computing statistics on samples to: compute_<statistics_name>

In Samples class we have, for example, methods:

compute_rhat
compute_ess

We can consider renaming all the methods that achieves computation to be of the format compute_<statistics_name>.
There is one drawback, however: this will break consistency with CUQIarray names (e.g. ca.mean(), ca.std()), where ca is object of type CUQIarray. The method names of CUQIarray are inherited from ndarray.

@nabriis, thinking about this issue if consistency with CUQIarray method names, I thought not to include methods name changing in PR #11 for the time being.


Modal form submit in IE9 workaround - add'l info for #566

A modal form consisting of modal-header, modal-body and modal footer.
form fields in modal-body, submit button in modal-footer. Works in FF, Chrome, Safari - does not work in IE9
Workaround #1: place submit button in modal-body (ugh)
Workaround#2: change submit button to type=button and add an onclick event to document.modalform.submit()

I'm not quite sure why you are having this issue, but I am doing a form post from a modal and it works fine with IE9.  Here's a snippet of one of mine that works:

    ×

      Title

  <form action="/post/url" class="form-stacked" method="post" style="margin: 0; padding: 0;">

      Name
      <input autofocus="autofocus" id="Name" name="Name" type="text" value="" />

      <input type="submit" value="Add " class="btn primary" />

  </form>

```

It was user error. I had the beginning form tag just inside the div for modal-body, after I followed your example and wrapped modal-body and modal-footer with the form tags it worked like a charm.  Thanks so much.

The same problem but thanks to mkchandler for providing us a correct way. However, it makes more sense to me with BizSherpa way when I did just like BizSherpa from the first time.


Fix Unmarshal's handling of optional byte fields

Backports upstream fix https://github.com/gogo/protobuf/pull/35

Why?

we've recently ran into an issue where we accidentally share pointers when setting repeated messages in a loop.  This would have been prevented if the generated code supports proper mutator api

Oh sorry I was confused.


add run-constraint on clang

Since llvm 15 started depending on libxml2 (which itself depends on a few libs), conda twists itself into a knot when trying to rebuild one of those dependencies. Even though this feedstock pins llvm to an exact version, which in turn exports libllvm<same_major>, the actual clang-implementation doesn't pin cctools.
It seems it can then happen that a wrong cctools gets used, as the whole thing shouldn't have been an issue until we're on llvm 15, but it did affect some packages. So add a run-contraint on the right libllvm to avoid that this can happen.
PTAL @isuruf

From my POV this is good to go, but I'll leave the merging up to you @isuruf.
We'll probably also have to mark the _12 for llvm builds as broken, I can do that once this PR is merged.

@isuruf, should we merge this now to help out those feedstocks? Any other inputs?

Thanks. I opened a PR to mark the _12 builds as broken on osx.


feat(audio): Add audio for each keypress

read out the letter pressed

I've been using to review the alphabet in Spanish with my daughter. I love the idea, but this would add a little friction to that.

sound was added, so this isn't needed


[Feature] WSE Minimap Icon

A guildie of mine requested this. Not sure what it would entail

I've just merged; https://github.com/wowsims/exporter/pull/23
Maybe this is what makes this request redundant? As this also opens the exporter window.


Killed While generating SMIL Files

How can I prevent from being killed while generating SMIL Files.
size= 9707045kB time=15:39:15.42 bitrate=1411.0kbits/s speed= 232x
video:0kB audio:9707045kB subtitle:0kB other streams:0kB global headers:0kB muxing overhead: 0.000001%
Killed

Please re-open if reproduced.


[BEAM-6713] Add withMaxNumWritersPerBundle from WriteFiles to FileIO …

…and TextIO.
The WriteFiles.DEFAULT_MAX_NUM_WRITERS_PER_BUNDLE is not ideal for my use case and is difficult to change when using FileIO and TextIO. Adding pass through values to these classes to make this configuration more accessible.
@lukecwik

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Choose reviewer(s) and mention them in a comment (R: @username).
[ ] Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue, if applicable. This will automatically link the pull request to the issue.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

Post-Commit Tests Status (on master branch)

Lang
SDK
Apex
Dataflow
Flink
Gearpump
Samza
Spark

Go

---
---
---
---
---
---

Java

Python

---

---
---
---

See .test-infra/jenkins/README for trigger phrase, status and link of all Jenkins jobs.

R: @lukecwik

Build is failing due to FileIO.Write#defaultNaming not having a javadoc. I did not make a change to this method and I am unsure of what to write for its javadoc. Also there is another overloaded defaultNaming method and a relativeFileNaming method that may also require javadocs.

Should this also be added to AvroIO and TFRecordIO? Or conversely removed from TextIO?

Run Java PreCommit

@chamikaramj or @pabloem can some of you guys PTAL at this one sicne @lukecwik is in holidays (I am too).

Closing this PR to avoid exposing a knob. Additionally you can remove the maxNumWritersPerBundle limit with the added methods from #7929 *IO.Write#withNoSpilling.


[$250] [Crash] Android: SurfaceMountingManager.getViewState

Potentially corresponding reproduction steps: https://github.com/Expensify/App/issues/50463
Looks like this has been happening for some time, creating this issue so we can keep track of it
com.facebook.react.bridge.RetryableMountingLayerException - Unable to find viewState for tag 1112. Surface stopped: false
com.expensify.chat_issue_4f8a216f0a769e684d009b28ef9bd7d4_crash_session_670538A401B000015610788664F8096F_DNE_0_v2_stacktrace.txt
com.expensify.chat_issue_4f8a216f0a769e684d009b28ef9bd7d4_crash_session_670538F603E5000135E901B5CB3CC679_DNE_0_v2_stacktrace.txt
com.expensify.chat_issue_4f8a216f0a769e684d009b28ef9bd7d4_crash_session_67053A24023800014E02F8EA97CEAAD4_DNE_0_v2_stacktrace.txt
com.expensify.chat_issue_4f8a216f0a769e684d009b28ef9bd7d4_crash_session_670575BC01CD00014CC8B5D670B1D2AF_DNE_0_v2_stacktrace.txt
com.expensify.chat_issue_4f8a216f0a769e684d009b28ef9bd7d4_crash_session_6705473302F5000179E748FD60E92734_DNE_0_v2_stacktrace.txt
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~021843746971003676610
Upwork Job ID:<PHONE_NUMBER>003676610
Last Price Increase: 2024-11-26

Issue OwnerCurrent Issue Owner: @paultsimura

This might be related to https://github.com/software-mansion/react-native-screens/issues/2341 (it has a similar stack trace) but it's hard to reproduce. Are there any additional details such as which page was open?

Looking for updates in https://github.com/Expensify/App/issues/50463

Yes, going to close this one out and I'll drop a comment in the other issue referencing this.

Hey @shubham1206agra, sorry was actually going to close this issue out in favor of https://github.com/Expensify/App/issues/50463 as commented here. But I did assign you in the other issue.


Add travis CI or Appveyor CI automatic testing

Repo will add CI for automatic testing when have new pull request ?

I would like to do this.