Refactor Builder classes for request objects

This pull request updates the naming convention for builder classes related to project requests. The classes CreateProjectRequest, LeaveProjectRequest, ListProjectRequest, LoginRequest, and ProjectNameRequest have been renamed to follow a consistent format, such as CreateProjectRequestBuilder.
Additionally, each builder class now implements the Builder<T> interface with the corresponding request type, ensuring type safety and clarity. For example, the CreateProjectRequestBuilder now implements Builder<CreateProjectRequest>.
These changes enhance code readability and maintainability by providing a clearer structure for the builder pattern used in the project request types.
cc. @namkyu1999

@jemlog @seedspirit @inpyu @somln Can you please take a look?

Hi @jongwooo
Thank you for the PR! I noticed that you’ve introduced a Builder interface for the builder pattern implementation. While I can see how this approach might be beneficial in larger, more complex projects, I have a few concerns and questions regarding its necessity in our current project context.
Why did you decide to separate the builder into an interface?
Given the current size of our project, it seems like the added complexity of an interface may not provide substantial benefits. Could you elaborate on the specific advantages this change brings in our case?
What practical benefits do you see from introducing this interface?
Our project is still relatively small, and I’m wondering how this interface could offer real-world value at this stage. In larger projects, it could help with standardization or testing, but is that something we anticipate needing soon?
Naming and readability concerns
By enforcing a Builder interface, we now have to create distinct builder class names (e.g., ListProjectRequestBuilder), which could lead to long and cumbersome names. I worry this could impact readability and make the code more difficult to maintain. Do you think this trade-off is worth it?
I would appreciate it if you could provide your thoughts on this!
Thank you 😊

Hi @suyeon-jung-dev
Thank you for your thoughtful feedback! Here are my thoughts:

Clear Contract for Builder
I bellieve that having a standardized interface promotes consistency in how objects are constructed across the project. The Builder<T> interface establishes a clear contract for any builder we create. This promotes consistency in how builders are defined and used throughout the codebase, making it easier for team members to understand and implement them. This can help team members follow the same patterns, making the code easier to read and maintain.

Flexibility for Future Growth
Even if our project is small now, we may need to add new features or object configurations later. An interface allows us to easily create new builder implementations that guide us in maintaining a consistent and organized approach to object construction

Clarity vs. Brevity
While longer names can be less concise, they also provide clarity about the purpose of the builder. I think it is important to balance clarity with brevity to avoid confusion.

I believe the interface provides a solid foundation that can benefit us in the long run, even if it seems like added complexity now. That said, I'm open to discussing this further and considering alternative approaches if they align better with our immediate needs. Your feedback is valuable, and I appreciate the opportunity to clarify these points!

Hi @jongwooo
I appreciate the effort put into this refactoring. While I have some reservations about the necessity of introducing the Builder interface at this stage, I recognize that this change can improve the consistency and maintainability of our codebase.
The enhancements in potential future scalability are valuable. I believe this refactoring will make it easier for this project to implement new features down the line.
Thank you for your PR! 🥰


publish to npm

need to publish this to npm

published... chromahash was available! find it at https://npmjs.org/package/chromahash


Crash when renaming animation StateMachine inside of an StateMachine created with the Group button

Godot version
4.0.beta10.mono
System information
Windows 11
Issue description
There is weird behaviour with a crash sometimes when renaming an AnimationNodeStateMachine that lives inside another StateMachine and was previously created using the Group Selected Node(s) button.

Steps to reproduce

Add a AnimationTree with a AnimationNodeStateMachine as the root.
Add some AnimationNodes to that StateMachine.
Select these Nodes and press the Group Selected Node(s) button.
Rename the resulting StateMachine.

Minimal reproduction project
N/A

Duplicate of https://github.com/godotengine/godot/issues/58822.

Ok, I for some reason don't get a crash anymore, instead this happens.

@Calinou Does this and #58822 crashes are caused by the same reason? I am puzzled as I cannot reproduce #58822...


Create_mini-ImageNet-LT.ipynb missing the txt file

all_classes.txt, equal_classes.txt, unequal_classes.txt are missing, can not generate the mini-imagenet-lt dataset

can you share your mini-imagenet-lt dataset?  Thanks.

@tcmyxc Sorry about the delayed reply. I am out of town. I will get back to you in a day.

Hello !  Can you share all_classes.txt, equal_classes.txt and unequal_classes.txt? Thanks

Hi @rahulvigneswaran,
I am facing a similar as @tcmyxc, can you help with it?

Hi Rahul,  Could you plz share all_classes.txt, equal_classes.txt and unequal_classes.txt?
Thanks

Hi Rahul,  Could you plz share all_classes.txt, equal_classes.txt and unequal_classes.txt?
Thanks

Hi guys, just wondering if anyone has fixed that issue.

@tcmyxc @Angelina1996 @xiaomabufei @vpavankalyan Sorry about the super delayed response. I have added the required missing assets under https://github.com/rahulvigneswaran/TailCalibX/tree/master/Notebooks/mini-imagenet-assets . Kindly remember to change the directory in the notebook accordingly.


Make baseurl description in api.rst more verbose

Following #432 I tried to make the description for baseurl a bit more precise.

Current coverage is 85.45%

Merging #434 into master will not affect coverage as of e1cb17d

@@            master    #434   diff @@
======================================
  Files           39      39
  Stmts         4628    4628
  Branches       632     632
  Methods          0       0
======================================
  Hit           3955    3955
  Partial        160     160
  Missed         513     513

Review entire Coverage Diff as of e1cb17d

Powered by Codecov. Updated on successful CI builds.

Thanks @joorei!


Brazilian Portuguese

Translate the index page, spec page, and errors into Brazilian Portuguese (pt-BR).

Hello! I can help with this one :D


Aryan work

I believe the merge is ready, as there are no conflicts.

Yes it is good for merging.

i think its good enough to merge


editormd.css中editormd-logo.eot引入错误

src: url(".../fonts/editormd-logo.eot?#iefix-5y8q6h")
表示上级目录那里写了三个点,webpack打包时会报错

@pauky  ./../ 或者 ../ 这样才是表示上级目录。

工程里这个css/editormd.css里存在这个src: url(".../fonts/editormd-logo.eot?#iefix-5y8q6h")引入错误

@pauky 不好意思，是我自己写错了，谢谢你的提醒。

@pandao how come you never fixed this? I'm grateful that you created this package, but not grateful that you left this simple and obvious mistake in it which wastes hours of peoples' time solving - which it could all be avoided if you just pushed an update.


vscode-extensions.aaron-bond.better-comments: init at 3.0.0

Description of changes
Added vscode-extensions.aaron-bond.better-comments
Things done

Built on platform(s)

[x] 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/)
22.05 Release Notes (or backporting 21.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
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

Can you complete the missing meta fields? (downloadPage => marketplace, homepage => github)
Also consider adding yourself as maintainer for testing on updates.

@schuelermine Please report back on status here.

Sorry for leaving this one behind. I will take a look again tomorrow.

I no longer use this extension.


Return nullTrace and nullSpan if nil context.Context provided

A little safety against nil context.Context objects.

Codecov Report

Merging #84 into master will increase coverage by 0.08%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master      #84      +/-   ##
==========================================
+ Coverage   84.09%   84.17%   +0.08%
==========================================
  Files          32       32
  Lines        3275     3279       +4
==========================================
+ Hits         2754     2760       +6
+ Misses        440      438       -2
  Partials       81       81

Impacted Files
Coverage Δ

v1/ao/context.go
100% <100%> (ø)
:arrow_up:

v1/ao/internal/reporter/reporter_grpc.go
66.32% <0%> (+0.34%)
:arrow_up:

Continue to review full report at Codecov.

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


Add new system events

API change check
APIView has identified API level changes in this PR and created following API reviews.
Azure.Messaging.EventGrid


Union types: Primitive type value (like number) can be assigned to interface type

TypeScript Version:  3.1.3
Search Terms: union types narrowing
Code
interface IComplexSize {
    size: ISizeConstraint;
}

function setSize(element: ISimpleSize | IComplexSize, width: number) {
    element.size = width;
}

var simpleSize: ISimpleSize = {
    size: 10
}
setSize(simpleSize, 100);
console.log('simpleSize', simpleSize.size); // logs '100'

var complexSize: IComplexSize = {
    size: {
        value: 10
    }
}
setSize(complexSize, 100);
console.log('complexSize', complexSize.size); // logs '100'

Expected behavior: Code should not compile
Actual behavior: Code compiles and allows to assign number to interface type. Function 'setSize' should compile because ISImpleSize / IComplexSize have nothing in common.
Playground Link: https://bit.ly/2J1eCeL

This is an intentional complexity/soundness trade-off -- property reads and writes are treated equally

I am sorry, maybe am not too good in TypeScript design principles... but I got a runtime error in my program due to this compiler behavior and spent a hour debugging this problem. I have strictly typed program without usage of 'any' type and object with property like 'IComplexSize' but after some code refactoring got run time error and 'number' type in it.
So I thought that TypeScript type system advantage over JavaScript is to protect user from such situations. For example, from my C# experience, I never was able to write program that able to assign number to incompatible type leading to run time error.
To be more clear, I just expect if I write class like
class MyClass  {
   public size: IComplexSize
}

I will never get a number in MyClass.size at run time, because TypeScript is strongly typed language.


[gyb] Add gyb doctest to validation-test

WARNING: DO NOT MERGE BEFORE #2978
What's in this pull request?

Added gyb doctest to validation test as validation-test/Python/gyb.swift

Removed --test option from normal gyb invocation in CMakeLists.txt.
While I agree with the comment:

Run gyb's self-tests whenever we use it.  They're cheap enough and it keeps us honest.

Do we really need it?

Resolved bug number: (SR-)

Before merging this pull request to apple/swift repository:

[ ] Test pull request on Swift continuous integration.

Triggering Swift CI
The swift-ci is triggered by writing a comment on this PR addressed to the GitHub user @swift-ci. Different tests will run depending on the specific comment that you use. The currently available comments are:
Smoke Testing
    Platform     | Comment
    ------------ | -------------
    All supported platforms     | @swift-ci Please smoke test
    All supported platforms     | @swift-ci Please smoke test and merge
    OS X platform               | @swift-ci Please smoke test OS X platform
    Linux platform              | @swift-ci Please smoke test Linux platform

Validation Testing
    Platform     | Comment
    ------------ | -------------
    All supported platforms     | @swift-ci Please test
    All supported platforms     | @swift-ci Please test and merge
    OS X platform               | @swift-ci Please test OS X platform
    OS X platform               | @swift-ci Please benchmark
    Linux platform              | @swift-ci Please test Linux platform

Lint Testing
    Language     | Comment
    ------------ | -------------
    Python       | @swift-ci Please Python lint

Note: Only members of the Apple organization can trigger swift-ci.

Confirming failure before #2978 merged.
@swift-ci Please test

@swift-ci Please smoke test

@swift-ci Please smoke test and merge


creation of instances from abstract class

The code below defines an abstract class (vehicle), then generates an instance from this class, and calls a setter & a getter for its speed attribute. Executing it in the transpiler leads to an output of 8 (the value of the instance's speed attribute) - i.e., the instance was successfully created, despite the fact that its class is abstract, so instance creation from it must not be allowed.
CLASS vehicle DEFINITION ABSTRACT.
  PUBLIC SECTION.
    METHODS:
      drive ABSTRACT,
      set_speed
        IMPORTING speed TYPE i,
      get_speed
        RETURNING VALUE(result) TYPE i.

  PROTECTED SECTION.
    DATA:
      speed TYPE i.
ENDCLASS.

CLASS vehicle IMPLEMENTATION.
  METHOD set_speed.
    me->speed = speed.
  ENDMETHOD.

  METHOD get_speed.
    result = me->speed.
  ENDMETHOD.
ENDCLASS.

DATA my_car TYPE REF TO vehicle.
CREATE OBJECT my_car TYPE vehicle.

my_car->set_speed( 8 ).
WRITE my_car->get_speed( ).

yea, its more a abaplint issue, it should give a syntax error
from a transpiler perspective the generated code is okay, just remove the abstract keyword, assuming the syntax is correct

Yes, this is true, it should be a syntax error because of the abstract keyword (so linter thing), without it it would be valid code.

@s7oev try again, 1.2.25 published

I tested with the transpiler (where nothing happens, unlike before), and the linter (where I get "/zfoobar.prog.abap[29, 1]: vehicle is abstract, cannot be instantiated(check_syntax)"), so it seems perfect 👍


Post-condition for contracts wrong

The following post condition
 makeContractSTXPostCondition(
            address,
            name,
            FungibleConditionCode.GreaterEqual,
            new BN(0)
          )

resulted in a post condition for address instead of address.name
See
https://explorer.stacks.co/txid/0x18bdc2b484566fdd22b15e26fdacb5cd8a7c200e65342dd6061caba87ceb11e0?chain=testnet

Can you rephrase? it sounds like this is expected

I expect the pc for the contract principal, not for the standard principal (updated the issue)

@aulneau I believe you mentioned during the call today that we need to investigate whether there's a collision somewhere in how address values are handled?

Thank you


Force SHA update to verify upgrade test

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

Test name
Commit
Details
Rerun command

prow/daily-perf-tests.sh
adcb1c785351fe1ba02441bd9f9bc694c44de294
link
/test daily-perf-tests

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


Introduce different lights on timers for summer (DST) and winter (standard) times

Introduce different lights on timers for summer (DST) and winter (standard) times. Both timers shall be possible to configure in ini file. Selection what timer to use shall be done based on DST flag.

implemented in main a7fe6d7


feat: make base dropdown based components accesible

Accessibility Check view has BaseColumnPickerDropdown and SortDropdown.
Check they fulfil accessibility requirements, as per this and this documents.
Please check:

https://www.w3.org/WAI/ARIA/apg/patterns/combobox/
https://www.w3.org/WAI/ARIA/apg/patterns/listbox/

You'll see a label for the BaseColumnPickerDropdown. It's there just to show how aria-label can be replaced

This is a link to Empathy Platform Docs Portal preview in a development environment.
⚠ Only for Empathy members. We’re sorry for the inconvenience!
Check out EPDocs preview


Implement F# Consumer

I don't think this is being actively worked on. Putting back to up for gabs


Make (select ...) compatible with Dune 2.0

In Dune 2.0, filenames in (select ...) must have format foo.{choice}.ext.
See https://github.com/ocaml/dune/commit/ede5676aefb3fe3b9265e3e6932532907c28e26b.
This commit doesn't actually change dune-project to require Dune 2.0. It just makes it easier to vendor Gluten in projects that use (subdir ...).
Note that I plan to eventually unvendor Gluten anyway (after a release with #20), so it is not necessary to merge this for my purposes.

Thanks!


Terraform Cloud integration

This PR introduces Terraform Cloud integration as default. Configure Terraform Cloud with the tfe {} block. See ./hashicorp/tfe/variables.tf for input variables.
New required environment variables are TFE_TOKEN and TFE_ORGANIZATION.
Example usage:
module "station-tfe" {
  [ . . . ]
  tfe = {
    project_name          = "Station TFE Development" # This project must exist before running
    workspace_name        = "station-tfe-testing" # Linked to above project
    workspace_description = "This workspace is for testing Station's TFE integration"
    vcs_repo = {
      identifier                 = "kimfy/tfe-testing"
# GitHub
      github_app_installation_id = var.github_app_installation_id
# BitBucket Cloud
      oauth_token_id = var.VCS_OAUTH_TOKEN_ID
    }
  }
}

This fixes #10


[Very WIP] Fix/refactor scss

Note: do not merge without making sure the stuff from add/rspec-page-regression is cleared out first.

In practice this has been split out into a lot of other PRs


browser lib rebuild

Please run your browser-rebuild.sh and commit new version of library. I used the outdated version from this repo and took me some time to figure out what I was doing wrong with my browser client.

Done.


Issue: error on downlink

Hello lorawan experts,
Could someone have a look at the following issue, please ?
https://os.mbed.com/teams/mbed-os-examples/code/mbed-os-example-lorawan/issues/6
cc @cdupaty
Thanks in advance

@hasnainvirk @AnttiKauppila

@adustm The issue is that they are sending on a Port which is not being listened at.
They are probably using an incarnation of receive() API which listens to a specific Port with specific flags.
If they wish to receive on any port and any flags, use this api:
int16_t LoRaWANInterface::receive(uint8_t *data, uint16_t length, uint8_t &port, int &flags)

@adustm Please close the ticket if you believe the issue is resolved, otherwise let us know how we can assist you ?

Closing issue because of inactivity, answer was provided to the reporter.


Unable to compress hidden files with Compress-Archive

The following code demonstrates two issues:

When passing a directory to the -Path parameter of Compress-Archive, any hidden files are silently ignored; they are neither present in the output file nor mentioned in the -Verbose output.  (This behavior is also not mentioned in the documentation.)
When passing hidden files to the -Path parameter of Compress-Archive...

...in PowerShell the first hidden file encountered causes errors to be thrown and output file creation to fail.
...in PowerShell Core output file creation succeeds but each hidden file encountered causes an error to be thrown and the corresponding compressed file entry has its modification timestamp set to 1980-01-01.

Code to reproduce:
$directoryToCompress = New-Item -ItemType Directory -Path 'Input';
$filesToCompress = @();

foreach ($id in 1..5)
{
    $file = New-Item -ItemType File -Path $directoryToCompress -Name "$id.txt" -Value "Hello, World!  This is file #$id.";
    if ($id % 2 -eq 0)
    {
        $file.Attributes = $file.Attributes -bor [System.IO.FileAttributes]::Hidden;
    }

    $filesToCompress += $file;
}

$PSVersionTable;

Compress-Archive -Path $directoryToCompress -DestinationPath 'FromDirectory.zip' -Verbose;
Compress-Archive -Path $filesToCompress     -DestinationPath 'FromFiles.zip'     -Verbose;

PowerShell console output:
Name                           Value
----                           -----
PSVersion                      5.1.17763.134
PSEdition                      Desktop
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0...}
BuildVersion                   10.0.17763.134
CLRVersion                     4.0.30319.42000
WSManStackVersion              3.0
PSRemotingProtocolVersion      2.3
SerializationVersion           <IP_ADDRESS>
VERBOSE: Preparing to compress...
VERBOSE: Performing the operation "Compress-Archive" on target "C:\Compress-Archive test\Input".
VERBOSE: Adding 'C:\Compress-Archive test\Input\1.txt'.
VERBOSE: Adding 'C:\Compress-Archive test\Input\3.txt'.
VERBOSE: Adding 'C:\Compress-Archive test\Input\5.txt'.
VERBOSE: Preparing to compress...
VERBOSE: Performing the operation "Compress-Archive" on target "
C:\Compress-Archive test\Input\1.txt
C:\Compress-Archive test\Input\2.txt
C:\Compress-Archive test\Input\3.txt
C:\Compress-Archive test\Input\4.txt
C:\Compress-Archive test\Input\5.txt".
VERBOSE: Adding 'C:\Compress-Archive test\Input\1.txt'.
Get-Item : Could not find item C:\Compress-Archive test\Input\2.txt.
At C:\WINDOWS\system32\WindowsPowerShell\v1.0\Modules\Microsoft.PowerShell.Archive\Microsoft.PowerShell.Archive.psm1:814 char:63
+ ... Entry.LastWriteTime = (Get-Item -LiteralPath $currentFilePath).LastWr ...
+                            ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
	+ CategoryInfo          : ObjectNotFound: (C:\Compress-Archive test\Input\2.txt:String) [Get-Item], IOException
	+ FullyQualifiedErrorId : ItemNotFound,Microsoft.PowerShell.Commands.GetItemCommand

VERBOSE: The partially created archive file 'C:\Compress-Archive test\FromFiles.zip' is deleted as it is not usable.
Exception setting "LastWriteTime": "Cannot convert null to type "System.DateTimeOffset"."
At C:\WINDOWS\system32\WindowsPowerShell\v1.0\Modules\Microsoft.PowerShell.Archive\Microsoft.PowerShell.Archive.psm1:814 char:25
+ ...             $currentArchiveEntry.LastWriteTime = (Get-Item -LiteralPa ...
+                 ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
	+ CategoryInfo          : NotSpecified: (:) [], SetValueInvocationException
	+ FullyQualifiedErrorId : ExceptionWhenSetting

PowerShell Core console output:
Name                           Value
----                           -----
PSVersion                      6.1.1
PSEdition                      Core
GitCommitId                    6.1.1
OS                             Microsoft Windows 10.0.17763
Platform                       Win32NT
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0...}
PSRemotingProtocolVersion      2.3
SerializationVersion           <IP_ADDRESS>
WSManStackVersion              3.0
VERBOSE: Preparing to compress...
VERBOSE: Performing the operation "Compress-Archive" on target "C:\Compress-Archive test\Input".
VERBOSE: Adding 'C:\Compress-Archive test\Input\1.txt'.
VERBOSE: Adding 'C:\Compress-Archive test\Input\3.txt'.
VERBOSE: Adding 'C:\Compress-Archive test\Input\5.txt'.
VERBOSE: Preparing to compress...
VERBOSE: Performing the operation "Compress-Archive" on target "
C:\Compress-Archive test\Input\1.txt
C:\Compress-Archive test\Input\2.txt
C:\Compress-Archive test\Input\3.txt
C:\Compress-Archive test\Input\4.txt
C:\Compress-Archive test\Input\5.txt".
VERBOSE: Adding 'C:\Compress-Archive test\Input\1.txt'.
Get-Item : Could not find item C:\Compress-Archive test\Input\2.txt.
At C:\program files\powershell\6\Modules\Microsoft.PowerShell.Archive\Microsoft.PowerShell.Archive.psm1:831 char:43
+ ...      $lastWriteTime = (Get-Item -LiteralPath $currentFilePath).LastWr ...
+                            ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
+ CategoryInfo          : ObjectNotFound: (C:\Compress-Archive test\Input\2.txt:String) [Get-Item], IOException
+ FullyQualifiedErrorId : ItemNotFound,Microsoft.PowerShell.Commands.GetItemCommand

WARNING: 'C:\Compress-Archive test\Input\2.txt' has LastWriteTime earlier than 1980. Compress-Archive will store any files with LastWriteTime values earlier than 1980 as 1/1/1980 00:00.
VERBOSE: Adding 'C:\Compress-Archive test\Input\2.txt'.
VERBOSE: Adding 'C:\Compress-Archive test\Input\3.txt'.
Get-Item : Could not find item C:\Compress-Archive test\Input\4.txt.
At C:\program files\powershell\6\Modules\Microsoft.PowerShell.Archive\Microsoft.PowerShell.Archive.psm1:831 char:43
+ ...      $lastWriteTime = (Get-Item -LiteralPath $currentFilePath).LastWr ...
+                            ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
+ CategoryInfo          : ObjectNotFound: (C:\Compress-Archive test\Input\4.txt:String) [Get-Item], IOException
+ FullyQualifiedErrorId : ItemNotFound,Microsoft.PowerShell.Commands.GetItemCommand

WARNING: 'C:\Compress-Archive test\Input\4.txt' has LastWriteTime earlier than 1980. Compress-Archive will store any files with LastWriteTime values earlier than 1980 as 1/1/1980 00:00.
VERBOSE: Adding 'C:\Compress-Archive test\Input\4.txt'.
VERBOSE: Adding 'C:\Compress-Archive test\Input\5.txt'.

The Get-Item errors reference the following lines in the ZipArchiveHelper function:
# Updating  the File Creation time so that the same timestamp would be retained after expanding the compressed file.
# At this point we are sure that Get-ChildItem would succeed.
$lastWriteTime = (Get-Item -LiteralPath $currentFilePath).LastWriteTime

A problem for zipping up git repos...

I'm seeing platform disparity for this too with regards to unix dotfiles e.g. .env where PowerShell 7 on Windows will happily include them but on Linux it will silently omit them from the archive. Had to workaround for now by renaming it to something else and renaming it back on the target.
Repro: PowerShell 7 on Ubuntu

Create a folder /tmp/compressme
Put an .env file inside the folder
Open pwsh and use Compress-Archive on the folder
The resulting zip file does not contain the .env file

Same Problem here. I'm trying to update a SharePoint Framework package (a .sppkg file which is in fact a ZIP archive) in a DevOps release on Linux. My process is as follows:

Uncompress the files using the Expand-Archive cmdlet
Update a package file
Build an updated version of the package with Compress-Archive.

The problem is that the package contains a _rels/.rels file which gets lost in the process. Thus the resulting package is invalid and its installation fails.

@ayousuf23 is there a plan to release v2? 🤞🏻

Indeed, it would be good to know:

when this fix will land
as well as what future PowerShell version the fix will be bundled with.

The currently available preview version does seem to have the fix (confusingly, the release notes only talk about empty directories, but those are already included in the latest stable version).

https://github.com/PowerShell/Microsoft.PowerShell.Archive/releases/tag/v2.0.1-preview1

However, the PowerShell Gallery package for this preview release has been unlisted, suggesting there was a problem with it.

https://www.powershellgallery.com/packages/Microsoft.PowerShell.Archive/2.0.1-preview2

That said, if you're adventurous and  know the exact version, you can still install it:
Install-Module -Name Microsoft.PowerShell.Archive -RequiredVersion 2.0.1-preview2 -AllowPrerelease


file:/mmdet/ may lose the version.py

I have tested the latest version, I found When run train.py in line8, it report can not find attribute 'version'

sorry, I just git it, I forgot to setup..._


Let's stop calling the package beta!

Locust has been widely used for many years now, it deserves to be considered production ready in PyPi!
It will always have bugs, and even (rarely) breaking changes, but that is part of normal development (and handled by versioning, not project status in pypi anyway).

Sounds good!


[GRIDFIELD] GridField search doesn't update pagination

Affected Version
SS 4.3
Description
The new Search doesn't update the GridField pagination after returning results.

Steps to Reproduce

Create a project with frameworktest install.
Access the Test ModelAdmin.
Filter your results with search criteria.
Pagination is not updated.

To be fair this might be an issue with GridField. GridFieldPaginator currently has to be added after GridFieldFilterHeader for the pagination to work correctly.
I'd say the fix would be to make that particular GridField component always apply after all other GridField_DataManipulators.

That sounds difficult to fix consistently. Would we want to add a way to give priorities to these GridField components?

This has only been fixed for ModelAdmin search forms, all other GridFields are still affected by this 😞


In ProfileCardScreen, expand the tap area of the button to be edited

Overview
As the title says, the button to edit should be in the same tap area as the button for sharing one above.

Figma
https://www.figma.com/design/XUk8WMbKCeIdWD5cz9P9JC/DroidKaigi-2024-App-UI?node-id=55644-39120&t=mTxTzXFOMIuOie4o-4
Course of events
Conversations in Figma below
https://www.figma.com/design/XUk8WMbKCeIdWD5cz9P9JC?node-id=55644-39120#923780086

🙋


Document custom maverick module

eg. how to create a custom module to add a second SITL instance

And is there somewhere outside of ~/software/maverick that a custom module can be placed?  ~/software/maverick is goodrobots git repo, either a custom module directory can be .gitignored, or probably better a separate area in ~/code should be setup.

Add two locations for custom modules:

~/software/maverick/manifests/custom-modules
~/code/maverick/custom-modules

Interface stats added to stats module.

Oops, wrong repo

Added support for the new module locations.  Just need to document.

https://github.com/rgreenosborne/maverick/blob/master/docs/next/modules/custom.md
@fnoop Am I doing this right?  do you have a code block for an actual second SITL module? Git is so damned intimidating.


add flex grid

@mikeyrayvon a test page would be perfect. As we discussed, I think the best approach is to have a test-page.php that WP could use as template for a Test page.
what do u think?

ok im working on page-test.php

actually i'll do a page template called grid-test.php, and we select it from dropdown

Im happy for the merge. @mixcas can you ok?


Updated to reflect output of dotnet new

dotnet new seems to produce different output now, this is how it looks from a fresh install this AM

Hi @Lewiscowles1986, I'm your friendly neighborhood .NET Foundation Pull Request Bot (You can call me DNFBOT). Thanks for your contribution!
You've already signed the contribution license agreement. Thanks!
We will now validate the agreement and then real humans will evaluate your PR.
TTYL, DNFBOT;

@rachelappel  please double check this. emphasis will need to change. Merge when you're finished.

I'll commit the emphasis as I'm pretty sure I know how it works


How can we insert new custom variable in pdf

My order has one attribute.How can i insert that new attribute in pdf.I am using EaDesgin Module.What changes i have to add in coding .that custom attribute depends on the customer who ordered the item.

This is not a simple thing and cannot be explained in a issue.


[Bug] Synchronization doesn't work with high number of labels with long strings

Community Note

Please use a 👍 reaction to provide a +1/vote. This helps the community and maintainers prioritize this request.
If you are interested in working on this issue or have submitted a pull request, please leave a comment.

Describe the bug
PG-Gremlin sync not working when large number of labels and label names are long. When doing a sync under these conditions the process fails with a "Unable to connect to database" error. Cannot see any additional detail

OS: Amazon Linux 2 (Any)
Browser: Any
Graph Explorer Version: 1.4.0
Graph Database & Version: Neptune <IP_ADDRESS> (I suspect 1.2.1 might not be affected because of the summary API)

To Reproduce
Steps to reproduce the behavior:

Use the python snipped code below to upload a csv to s3 bucket

import pandas as pd
import uuid
import string
import random
import awswrangler as wr
df = pd.DataFrame(columns=['~id','~label','name'])

max_rows = 100

for x in range(0, max_rows):
    uuid_str = uuid.uuid4()
    label_str = str(uuid_str) + ''.join(random.choices(string.ascii_lowercase +
                                        string.digits, k=100))
    df = pd.concat([pd.DataFrame([[uuid_str,label_str, "Some Random name"]],columns=df.columns), df], ignore_index=True)

wr.s3.to_csv(df = df, path = 's3://some_bucket/fail_load/bulk.csv', sep=',', index = False)

Do bulk loading into an empty Neptune cluster
Open Graph explorer and connect to the Neptune cluster. Notice the sync process will fail

Expected behavior
Correct synchronization of the connection

This seems to be a blocker, should be prioritized

this is a big block for semantic knowledge graphs using RDF.

Issue resolved with #206.


[beta] "ES6: Mutate an Array Declared with const" passes test 3 without the array being sorted

Challenge Name
ES6: Mutate an Array Declared with const
Issue Description
To begin with, I added "use strict"; to the top of the code, and commented out the second assignment of s so the code would compile.  I expected the third test (Test s is sorted) to fail, but it passed.
Browser Information

Browser Name, Version: Chrome 56.0.2924.87 (Official Build) (64-bit)
Operating System: Mac OS X El Capitan 10.11.6
Mobile, Desktop, or Tablet: Desktop

Your Code

"use strict";

const s = [ 5, 7, 2 ];
// change code below this line

//s = [2, 5, 7];
// change code above this line
// Test your code
console.log(s);

Screenshot

@elrolfe Thanks for reporting this issue!
We currently have some issues testing the transpiled ES6 code, and because of that the ES6 challenges don't work properly. In this case though, there are no tests - just comments explaining what we should test for once the test framework is updated.
But despite that, feel free to work through the challenges and give feedback! :blush:

Hello, i write it:
"use strict";
const s = [5, 7, 2];
// change code below this line
s[0] = 2;
s[1] = 5;
s[2] = 7;
// change code above this line
console.log(s);
But it doesn't work


Setup Nuget package

The result is available here:
https://www.nuget.org/packages/LibUsbDfu/
The next thing will be to integrate this with github:
https://docs.github.com/en/free-pro-team@latest/packages/guides/configuring-dotnet-cli-for-use-with-github-packages
but let's leave it as a separate task if the need arises.

The result is available here:
https://www.nuget.org/packages/LibUsbDfu/
The next thing will be to integrate this with github:
https://docs.github.com/en/free-pro-team@latest/packages/guides/configuring-dotnet-cli-for-use-with-github-packages
but let's leave it as a separate task if the need arises.


How to use --watch and --autotest w/o any addtional option?

I am attempting to use the command line: jasmine-node --watch test/lib/ --autotest test/unit/ Doing so give the following output:
USAGE: jasmine-node [--color|--noColor] [--verbose] [--coffee] directory

Options:
  --autotest         - rerun automatically the specs when a file changes
  --watch PATH       - when used with --autotest, watches the given path(s) and runs all tests if a change is detected
  --color            - use color coding for output
  --noColor          - do not use color coding for output
  -m, --match REGEXP - load only specs containing "REGEXPspec"
  --matchall         - relax requirement of "spec" in spec file names
  --verbose          - print extra information per each test run
  --coffee           - load coffee-script which allows execution .coffee files
  --junitreport      - export tests results as junitreport xml format
  --output           - defines the output folder for junitreport files
  --teamcity         - converts all console output to teamcity custom test runner commands. (Normally auto detected.)
  --growl            - display test run summary in a growl notification (in addition to other outputs)
  --runWithRequireJs - loads all specs using requirejs instead of node's native require method
  --requireJsSetup   - file run before specs to include and configure RequireJS
  --test-dir         - the absolute root directory path where tests are located
  --nohelpers        - does not load helpers.
  --forceexit        - force exit once tests complete.
  --captureExceptions- listen to global exceptions, report them and exit (interferes with Domains)
  --config NAME VALUE- set a global variable in process.env
  --noStack          - suppress the stack trace generated from a test failure
  --version          - show the current version
  -h, --help         - display this help and exit
Watching for changes in /Users/kbaltrinic/dev/oss/http-backend-protractor-proxy/test/unit/
Watching for changes in test/lib/

I am putting --autotest between the --watch folder list and the spec folder-list as a delimiter since it is the only other option I really want.  However, jasmine-node does not seem to like this.  It oddly does start watching the correct folders as the last two lines of the output suggest. (Changes in either of them result in the above output being spewed forth again!) Yet no tests actually seem to be run--at least there is no output from them being displayed.
Using some other option as a delimiter and putting --autotest back in the first position seems to work just fine.  jasmine-node --autotest --watch test/lib/ --color test/unit/ gives me:
.

Finished in 0.004 seconds
1 test, 1 assertion, 0 failures, 0 skipped

Watching for changes in test/lib/
Watching for changes in /Users/kbaltrinic/dev/oss/http-backend-protractor-proxy/test/unit/

Thankfully I found this post. I also did not want to add an unused parameter before my spec folder, but had to in order to work around this bug.


Incompatible with Auto-Entities and Apexcharts?

I'm trying to create a graph over the CPU usage for all my docker containers.
Here's a minimal configuration for once container:
  - type: custom:apexcharts-card
    update_interval: 30s
    graph_span: 24h
    series:
      - name: Test Container
        entity: sensor.docker_test_container #Changeme
        attribute: memory
        unit: '%'
    all_series_config:
      stroke_width: 2
      float_precision: 2
    apex_config:
      chart:
        height: 300px

For some reason the graph fails to render with no errors whatsoever. The same works for any other entity other then those created by this integration. Anyone encountered the same issue?

My bad, I had sensor.docker_test_* excluded from the recorder


swap backups to have resources run last

Signed-off-by: Valentina Birsan<EMAIL_ADDRESS>https://github.com/stolostron/backlog/issues/25596
related to https://github.com/stolostron/cluster-backup-operator/pull/313
With the number of backups reduced to 5, the swap should be updated

/test unit-tests


responseWhitelist.push('body') crashes

I'm trying to log the body of the response and apparently the lib crashes when trying to get it from the headers.
Simple example app:
var express = require('express');
var expressWinston = require('express-winston');
var winston = require('winston');
var app = express();

expressWinston.responseWhitelist.push('body');
app.use(expressWinston.logger({
    transports: [
        new winston.transports.Console({})
    ]
}));

app.get('/', function (req, res) {
    res.send('Hello World!')
});

var server = app.listen(3000, function () {

    var host = server.address().address;
    var port = server.address().port;

    console.log('Example app listening at http://%s:%s', host, port)

});

Accessing the defined route results in a crash
> node app.js
Example app listening at http://<IP_ADDRESS>:3000
TypeError: Cannot call method 'indexOf' of undefined
    at ServerResponse.res.end (/home/arnau/code/myapp/node_modules/express-winston/index.js:201:64)
    at ServerResponse.res.send (/home/arnau/code/myapp/node_modules/express/lib/response.js:154:8)
    at app.listen.host (/home/arnau/code/myapp/app.js:14:9)
    at callbacks (/home/arnau/code/myapp/node_modules/express/lib/router/index.js:164:37)
    at param (/home/arnau/code/myapp/node_modules/express/lib/router/index.js:138:11)
    at pass (/home/arnau/code/myapp/node_modules/express/lib/router/index.js:145:5)
    at Router._dispatch (/home/arnau/code/myapp/node_modules/express/lib/router/index.js:173:5)
    at Object.router (/home/arnau/code/myapp/node_modules/express/lib/router/index.js:33:10)
    at next (/home/arnau/code/myapp/node_modules/express/node_modules/connect/lib/proto.js:193:15)
    at Object.handle (/home/arnau/code/myapp/node_modules/express-winston/index.js:231:9)

What version of winston-express are you using?

I just created this example from scratch to check that the issue was not fixed yet, so I used the 0.2.9.
In another project I was using 0.2.8 and I had the same problem.

Yup.  I believe that my PR will fix it, so it's up to @bithavoc to pull the trigger on the merge.

Publishes to NPM as 0.2.11

0.2.11 corrects the crash, but the body field is empty

can't see the content type of the request @arnaugm
see this: https://github.com/itagenten/express-winston/commit/c2f2ba944269bb2a9d3ba1e8a18916ee0c6fe038

Sorry @bithavoc I don't understand your question.

@arnaugm I don't see the content type of the request, is it there in the headers?

Not sure what you mean. My point is that with the example above I think it should work, Am I wrong? If the content type is not specified, the default one should be taken into account and in any case return the content of the body.

This problem should be fixed in 2.11 (newest version) - the behaviour you describe is what now happens.

Seems to work now, great!

Cool. :)


issue in installing or deploying onefuzz

Information

Onefuzz version:1.6.0
OS:Ubuntu 18.04

Facing below issue can anyone suggests how to resolve this as i am getting one fuzz:command not found .

That error is coming from Azure Functions Core Tools, which we use to deploy the underlying API service.

Does it fail if you rerun the deploy script?
Can you try deploying the function code directly?  From the directory with the deployment tools, run:
func azure functionapp publish YOUR_INSTANCE_NAME_HERE --python --no-build

Thanks for reply @bmc-msft

Does it fail if you rerun the deploy script?
yes it fails

Can you try deploying the function code directly? From the directory with the deployment tools, run:
func azure functionapp publish YOUR_INSTANCE_NAME_HERE --python --no-build

i execute in and attached is reponse

but still while connecting to instance using
onefuzz config --endpoint "my-app-end-point"
it gives error
onefuzz: command not found
can you please guide

Any suggestion by any Contributors of OneFuzz????
it would be really helpful...as i am not able to deploy onefuzz

@nchandna As you were able to use the func azure command to publish the service API, I believe the error you experienced earlier was a transient error in Azure Functions.  We have experienced a handful of these ourselves.
Please run the deploy script such that the tasks that run after func azure ... can execute as well.
Note, that deploying onefuzz does not install the SDK or CLI.  You will need to do that as a follow-on sequence of commands.

@nchandna As you were able to use the func azure command to publish the service API, I believe the error you experienced earlier was a transient error in Azure Functions. We have experienced a handful of these ourselves.
Please run the deploy script such that the tasks that run after func azure ... can execute as well.
Note, that deploying onefuzz does not install the SDK or CLI. You will need to do that as a follow-on sequence of commands.

Hi @bmc-msft ,
Thanks for your reply.
if i am not wrong you are informing that i should run the next steps of deploy.py after deploy app stage.
But i am not a python geek, i generally do code in Java.
it would be helpful if you provide me info on next steps such as export_appinsights and update registration i.e how to run them

@nchandna We are looking to build a docker container with the deployment tools pre-installed and packaged in the very near future.  This should remove the issues you're experiencing in deployment.

You can track #276 for the deployment docker container.

You can track #276 for the deployment docker container.

Thanks a lot @bmc-msft  for the information.
Yes it would be better as per latest trends in IT


Load model callbacks

Summary
Call set_model on callbacks during load_model
The load_model function builds the model from the saved object, and
implicitly sets the model in the backed when the weights are set.
Callbacks that require set_model to be called before the model is
set on the backend can fail to operate on loaded models as a result.
This commit adds the ability to pass callbacks to the load_model function.### Summary
Related Issues
Fixes #11279
PR Overview

[ Y ] This PR requires new unit tests [y/n] (make sure tests are included)
[ N ] This PR requires to update the documentation [y/n] (make sure the docs are up-to-date)
[ Y ] This PR is backwards compatible [y/n]
[ Y ] This PR changes the current API [y/n] (all API changes need to be approved by fchollet)

Hello and thanks for the PR. For any change in the API, a google doc explaining the motivation should first be done and reviewed. Only after that can someone make a PR implementing this API change. See https://github.com/keras-team/keras/blob/master/CONTRIBUTING.md#pull-requests for more info on how to do that. I'll close the PR for now. It'll be reopened if/when the google doc get approved (add a message in this thread and I'll reopen).
Thank you for having invested your time into this.


New package: RCWA v1.0.0

Registering package: RCWA
Repository: https://github.com/jonschlipf/RCWA.jl
Created by: @jonschlipf
Version: v1.0.0
Commit: 45d98054df36d5855ca03e2cc6425db633b1e3b3
Reviewed by: @jonschlipf
Reference: https://github.com/jonschlipf/RCWA.jl/commit/45d98054df36d5855ca03e2cc6425db633b1e3b3#commitcomment-48084423

I renamed the package, pull request is at:
https://github.com/JuliaRegistries/General/pull/31662
This can be closed.


How to add formatting option in HtmlViewer provided in this library?

I came across this API and I am using HTML Viewer. I am displaying HTML using it and now I want to allow user to edit (i.e format data) is it possible to achieve using this library?

The HtmlViewer control was designed to render/display html data only and apply simple styles formatting.
However, it's based on WebView control. Maybe this control could adjust better in you are trying to do.
By the way, in our DEV branch, we are working in a new control HtmlBlock  for html rendering not based on WebView. It works much more better for html rendering...

Thanks.
So, you are suggesting that I can use HtmlViewer to allow simple formatting options? Could you provide some sample links from where I can learn to add formatting options in HtmlViewer?
Also, can I use HtmlBlock right now in my project to achieve my requirement?

The HtmlViewer control only allows you to customize the foreground color, background color and font size. On the other hand, HtmlBlock allows you to apply styles to each of the html tags supported.
If you cannot wait until our next release, you can fork/clone, checkout the DEV branch and add the project to your solution (remember remove the reference to nugget).
You can also run the sample app from DEV branch and see how HtmlBlock control works.
One important thing is HtmlBlock is under development, so you could find a few bugs...

Thanks @javitosanchez  I need to allow the user to customize any word to bold,italic,underline. Add bullet list, add number list, change heading styles , change foreground colors.
So, as you said I think for this, HtmlBlock will be useful right?

Right!

Thanks a lot.
Hope Microsoft keep adding useful controls like HtmlBlock in future to its SDK.

Hi, when you will release HtmlBlock control for this API?

We are planning to merge to master branch next week and then release the first beta of 1.5 version. We will keep you updated.

Ok thanks. I require this control as there is not HTMLEditor available in Windows.

hi, I checked the Dev project, in HTMLBlock it just displayed content. What I need to do, if I add a button in App bar for example B for Bold, and when the user clicks on that the selected word or sentence must be changed to Bold?

Please also check Carousel Control. It shows which image is selected in Indicator in Desktop App but not in Mobile app. I downloaded mobile app from Windows Store.

Hi, you can assign a specific style to each supported html tag in HtmlBlock control, and you can do it in runtime. The control has a DocumentStyle property composed by a lot of properties (one for each tag). You can change those with styles or directly in c# (all or 1 specific). Please, review the sample page where there is a combo changing the styles in runtime.
From the carousel control, we are preparing a new release. It should be solved in this one. Thanks for alert us.

Ok thanks. So, if I select any word then I can change that to Bold or any other HTML format right?
Also,please update when you release the new version in Nuget. These are really helpful controls for any Windows Developer. Thanks for them.

Not really. You can assign styles to tag elements. For example, you can change the font color of all bold elements in the document (in design time and runtime), but you cannot apply styles to a particular word.
Yesterday we merged to master branch, and also we released a beta version in Nuget.

Hi, Thanks for the beta version. I was using Carousel control with the indicator but the indicator is not displaying. My XAML and C# code are below
XAML Code
<StackPanel Grid.Row="1">             <appStudio:Carousel                 x:Name="evidenceFlipView"                 ItemsSource="{Binding}"                 ContentTemplate="{StaticResource MobileEvidenceFilpViewDataTemplate}"                 MaxItems="1"                 Height="500"                 Margin="5"                 AspectRatio="1.5"                 AlignmentX="Center"                 GradientOpacity="0.3"/>             <appStudio:Indicator x:Name="indicator"                                   Margin="0,9"                                  ItemsSource="{Binding ItemsSource, ElementName=evidenceFlipView, Mode=TwoWay}"                                  SelectedIndex="{Binding SelectedIndex, ElementName=evidenceFlipView, Mode=TwoWay}"                                  ShapeMode="Ellipse" Background="WhiteSmoke" SelectedBackground="DimGray" PressedBackground="Gray" />         </StackPanel>
C# Code
evidenceFlipView.ItemsSource = fileList;
Can you suggest what I have missed?

If I remove Height from Carousel then it works.

Thanks for reporting this. We'd fix it and should be solved in the current version.

Released in the current version


Enable using podman instead docker

Context
I'm trying to use podman to build the image using this plugin. A ln is set and podman is used instead of the docker command to build the image.
The build command fails because the podman don't log out the sha digest in the same way as docker.
In podman the digest is in one line without the "Digest:" part.
Current behaviour
If there is no ":" in the build output an exception is thrown:
[7:22:45 AM] [semantic-release] › ✖  An error occurred while running semantic-release: TypeError: Cannot read properties of undefined (reading 'substring')     at Image.build (.../node_modules/@codedependant/semantic-release-docker/lib/docker/image.js:123:20)
Expected behaviour
If no sha is parsed, it shall not throws an exception and continue.

I think that is outside the scope of this plugin. That needs to be its own plugin


I want Pull Requests to be squashed when they get merged

As an AaC repository maintainer, I want Pull Requests to be squashed when they get merged so that the repository is always in a working state and the commit history is clean.
AC

[ ] When I complete a Pull Request, it is accepted, and I click the merge button in the GitHub interface, the commits are squashed and the merge is performed.

Resources

Configuring commit squashing for pull requests

Good call! I was just thinking about doing this after playing with the release process.

I think the only person who currently has the necessary permissions to complete this is @jondavid-black.


How to erase past contents in history.

I want to know how to erase history once in history.
I found such a function in prosemirror, can this be used with tiptap?
https://prosemirror.net/docs/ref/#history.closeHistory
As a premise, can the history function be used to erase the history?

https://github.com/scrumpy/tiptap/issues/491


Obfuscate search_url

Need to obfuscate search_url in /py/search.py

completed

lol how the hell do you close an issue as "done". ah here we go


Update 1.0.0-alpha7 to serde ^1.0

I'm interested in trying the endian_choice module, but is blocked by this.

1.0.0-alpha7 doesn't exist anymore. Use https://crates.io/crates/bincode/0.8.0.


KnowYourCustomer r1.3 release review (Patch release with kyc-match v0.2.1, revoking r1.2)

Release PR or issue to review

https://github.com/camaraproject/KnowYourCustomer/pull/149
Review actions
Assign this issue to yourself and follow the below list.
For any review action, review the file(s) in the issue/PR listed above.
Put comments in the above issue or PR if they concern non-changed files/text.
Put a short summary of the main review result here into the review issue.

[x] API definition files (YAML) (check version in info & servers objects)
[x] test file(s) availability
[x] changelog updated
[x] readme updated (enforce correct release number / API version naming)
[x] API readiness checklist(s)

Release actions
Assign this issue to yourself or another RM team member and follow the below list.
When done, tick the box in this issue.

[x] API release tracker available on wiki
[x] Review comments addressed (by Sub Project)
[ ] Release PR approved (on behalf of Release Management)
[ ] PR merged (by Sub Project codeowner)
[ ] Release created within GitHub (by Sub Project codeowner)
[ ] Release Tracker updated (with creation date of the release and the release tag link)

Additional comments

For r1.2 the CHANGELOG.md contains the sentence:

NOTE: Release r1.2 has been revoked due to a bug found in kyc-match v0.2.0 code. It is fixed in kyc-match v0.2.1 as part of r1.3 release.

It is planned to delete the release r1.2 from the repository.

Hi @hdamker, we ahve already created the release. So the rest of the checks can be completed.
Thanks!

@fernandopradocabrillo thanks - great teamwork!


[trivial] remove benchmark dependencies from v3

Currently, importing github.com/cep21/circuit/v3 requires transitive dependencies on every circuit-breaking library that is used in the benchmarking package, even though they are not actually used by the real library. By creating a separate go.mod/go.sum in benchmarking/, we can remove these transitive dependencies.

@cep21 let me know what you think – seems like just an oversight to include these dependencies in the main v3 package

I like this.  Let me think about it today and get back to you.


New Version: Pulsar-Edit.Pulsar v1.106.0

[ ] Have you signed the Contributor License Agreement?
[X] Have you checked that there aren't other open pull requests for the same manifest update/change?

[X] Does your manifest conform to the 1.4 schema?

Note: <path> is the name of the directory containing the manifest you're submitting.

Microsoft Reviewers: codeflow:open?pullrequest=https://github.com/microsoft/winget-pkgs/pull/110643&drop=dogfoodAlpha

@microsoft-github-policy-service agree


Feat/dev wallet/write down recovery key

Add functionality for Backup recovery key - writing down
Related Issue/Asana ticket: https://app.asana.com/0/1205838799963624/1206840333553537/f
Steps to improve this PR:

move NavigationDialog to a separate component
move recovery phrase view to a separate component
remove password input on writing down recovery phrase page

@javadkh2


docs: stream operator throttle typo

found something typo and missing by reviewing in resolves https://github.com/apache/pekko-persistence-jdbc/pull/199

failure was the link-validator timing out connecting to http://www.gnu.org/licenses/old-licenses/lgpl-2.1.html . Rebuilding, #1309


add poll module in Android

poll functions are defined in Android as well.
libc is missing some constant, but once rust-lang/libc#663 is merged, it'll be good to merge here.

Good find.  Could you also enable the poll tests for Android in test/test.rs ?

Actually poll() and related constants are defined for all unix platforms. So we can just unconditionally enable that. But ppoll is only for android, linux, and freebsd/dragonfly, though the freebsd/dragonfly support is missing from ppoll, could you change that as well @ndusart?

Also you'll want to change Cargo.toml to point to libc git for these tests to pass once rust-lang/libc#663 lands.

I included the module and the test unconditionnaly.
Could only test for linux and android though. Waiting to see the CI results.

There is an issue on FreeBSD, seem that I have the same symbols in rust-lang/libc#663 should be added to FreeBSD and DragonFly. I'll update that.

Should be good to merge now.

@ndusart We're waiting for nix to release before merging this. Either that or a new libc release happens before and then we can merge this in.
In the mean time can you add a CHANGELOG entry for this? I think putting it under the Added section makes sense since this is just an exposure of more functionality on some platforms, versus a change in an actual API.

This needs a rebase.
I'd also like to ask if you could, as part of this PR, whip up some documentation for this module now that it's supported for all platforms. It'd be much appreciated!

Thanks for the docs, it looks a lot better! There's some minor things to clean up here, but nothing major.

The requested changes have been made.

@Susurrus does this PR seem ok ?

@Susurrus : done, thanks for your feedback !

@ndusart Thanks for your hard work on this, I know it took a while to work through my rigorous review process, and to deal with upstream, but it's contributions like this that make nix great!
cc @berkowski
bors r+

@Susurrus you wouldn't be a good reviewer if you wasn't a bit nitpicking about PR :p
It didn't take too long, it's just that the different timezones make that small successive revisions takes more time unfortunately.


Summary Command

Request
A command that will display only the messages sent to and from the user contacting the ModMail bot.
Think of this as a way to filter staff conversation on a thread and just dump the information that was sent/received.
Would be useful in a scenario where a discussion was in the thread between staff and someone wanted to see the conversation without reading all the chat between.

This already exists as an option for the loglink command: https://github.com/Dragory/modmailbot/blob/master/docs/commands.md#loglink--s
For other log links you'd have to manually add the simple=1 query, I believe.


Multiply with var_Color in lightMapping after u_AlphaThreshold check

Fixes an issue creating a void effect, where vertex + color/diffuse map produces different results in depth pre-pass generic and lightMapping shaders, due to different check/multiplication order.
Fixes #119. I've also tried moving the multiplication before the check in generic shader, but it ended up making surfaces disappear based on distance to them.

So what is "alphagen vertex" used for?
Also why does q3map2 even write values other than 1 to the alpha of the vertex color? Seems weird

I suppose it's still used for blending.

Also why does q3map2 even write values other than 1 to the alpha of the vertex color? Seems weird

It is used for terrain blending, for example in a single shader you have a rock and and a sand stage, and one of the stage receives its alpha channel from the vertex, this makes possible to blend two terrains per surface.

Also why does q3map2 even write values other than 1 to the alpha of the vertex color? Seems weird

It is used for terrain blending, for example in a single shader you have a rock and a sand stage, and one of the stage receives its alpha channel from the vertex, this makes possible to blend two terrains per surface.

Do we have an example of such a shader?

The garden in station15 (grass/mud), the outdoor on thunder (rock/sand).

It modifies the way the grass/mud floor is rendered in the garden:
Before:

After:

Now it's possible that is the map that is wrong. I had to edit it to workaround a q3map2 bug, but I remember I struggled to get the effect in the first screenshot, as I didn't seen much correlation between what I did in editor and the result I was getting.

This is how it looks in editor:

There is no place where the grass is 0% (mud 100%).
So maybe this branch is right and the map source is wrong.

If I set some 0%, I see the grass surfacing where it is 0%:

If I add more grass (less mud):

I get more grass:

With this branch, what I do in editor is what I get in game.

There was a discussion in IRC in which SomaZ said that actually the generic shader is the "correct" one, in terms of matching q3 behavior.

There was a discussion in IRC in which SomaZ said that actually the generic shader is the "correct" one, in terms of matching q3 behavior.

The generic shader actually does multiplication after discard.

Changing vertex alpha multiplication to be performed after the alpha discard is wrong (in terms of compatibility). Changing it to be performed before the discard in the other shaders like the generic shader would be the correct fix in terms of compatibility. This should also fix the incorrect void effect illwieckz encountered on station15 with the trees.

I tried that, and the small plants on the ground in station15 in the same area started disappearing based on distance. Although perhaps that is another map bug that was trying to workaround an engine issue.

I tried that, and the small plants on the ground in station15 in the same area started disappearing based on distance. Although perhaps that is another map bug that was trying to workaround an engine issue.

Probably it's the same issue as the trees, with the alpha volumes, which are intended to affect only the ground, doing collateral damage to the plant models? If there are a lot of bad alpha volumes, maybe it would be easier to put alphaGen identity in the plant shaders rather than moving all of them.
Changes like this which break the current version of our assets can be targeted to the for-0.56.0/sync branch.

Actually multiplying makes sense if one wants to combine alphaGen vertex with alphaFunc GE128, which is what does the textures/station15_custom/ter_mudmoss1 shader.
This may look weird, but it allows to do alpha blending with a strong cutoff: instead of linearly blending from moss to mud, only values above 50% makes mud appear, and everything elle displays full moss.
The fact I didn't felt something was meaningful between what was done in the editor and what was gotten in game was because I did not notice that cut-off. Now that I know about it, what I get in game is what I do in editor.

I tried that, and the small plants on the ground in station15 in the same area started disappearing based on distance. Although perhaps that is another map bug that was trying to workaround an engine issue.

Probably it's the same issue as the trees, with the alpha volumes, which are intended to affect only the ground, doing collateral damage to the plant models.

Yes, it's likely the same problem, as we see on above screenshot some plants are in some alpha volumes. It can be worked around by having smaller alpha volume that only hits the brushes intersections to mark them. Doing a large alpha brush is just a lazy way to mark all brushes intersections from a  whole area.
Also for the tree, the alpha brush can just be made less high, but that would not work for plants.
Actually I'm happy we start grasping what's going on after all those years.

I've added a commit that reverts the change to lightMapping and changes the generic shader instead, but now the plant isn't rendered. Maybe you could look at thr map if it's a map bug.


Issue in Using with ReactNative

I am trying to integrate STOMP on my reactNative Project...
When I open my debug mode everything works fine but when i close my debug mode and run it normally it is not able to fetch data//
alert('Component did mount');
client.configure({
brokerURL: 'ws://<IP_ADDRESS>:8012/stomp',
onConnect: () => {
alert('onConnect');
    client.subscribe('/random/city', message => {
      console.log('show msg...',message);
      console.lo('show msg body...',message.body);
      this.setState({city: message.body});
    });

    client.subscribe('/random/state', message => {
      console.log(message);
      this.setState({country: message.body});
    });

  },

});

client.activate();

Please enable console log for the library and attach it.


Fix for SemanticVersionPreReleaseTag Number parsing

Fix for SemanticVersionPreReleaseTag Number parsing so that it can work with "number" group that is larger than MaxInt.

Looping in @JakeGinnivan as I'm using the SemVer code from https://github.com/GitTools/GitVersion
Jake, what do you think?

Looks fine to me, if it is that big then it is not going to be an incrementing number so thats fine to lose it.
On a side note I was thinking about pulling the SemVer classes from GitVersion out into a source only nuget package. Thoughts on that?

A separate SemVer package would work for me (I'm ILMerge'ing everything in the project anyway) and it'd keep things more tidy.

Thanks!


Not working in the latest version of Chrome

I have been using this library for a project of mine. Today I noticed the P2P isn't detecting new peers. Even if you open it in a new tab. It seems it still works on my other computer and I have not updated Chrome on it. Any ideas what changes in Chrome have resulted in this breaking?

I have been doing my own debugging looking through the code and such. The issue is related to this update: https://groups.google.com/g/discuss-webrtc/c/L0qDWipd7VE?pli=1
The fix is to remove query options in the STUN URL in p2p-media-loader-core.js .  I can't find the corresponding code in this repo though from a quick look.
In other words: stun:global.stun.twilio.com:3478?transport=udp becomes  stun:global.stun.twilio.com:3478

Thank you for providing more info and the fix. It did work for me as well.

Thank you for providing more info and the fix. It did work for me as well.

Yeah I have had my fair share of going to a thread where they say "all god figured it out" or never come back. So I make my best effort to come back haha

Your are my real life super hero bro!
Thank you.  It help me saving cost so much.


Loading pkcs12 der formatted bytes fails on macOS

This change:
https://github.com/sfackler/rust-native-tls/commit/94a150379d42b9ee6b2d2dbbadde02cde88e5159
To get rid of the temp file, appears to have caused an error in my builds. Rolling back this change fixes my issues. I'm running into the unwrap() on
https://github.com/sfackler/rust-native-tls/commit/94a150379d42b9ee6b2d2dbbadde02cde88e5159#diff-9cef9dd6c1ba7275f2ad10014dd7697fR82
I was not running into this error before the change.
FYI: @BlameOmar

I just pushed a test showing this issue here:
https://github.com/bluejekyll/rust-native-tls/tree/bfry/working-pk12
the working test:
https://github.com/bluejekyll/rust-native-tls/blob/bfry/working-pk12/src/test.rs#L180
the broken test (calls into the new logic):
https://github.com/bluejekyll/rust-native-tls/blob/bfry/working-pk12/src/test.rs#L189
both use a newly generated cert each time, the first one passes every time for me.

The broken test fails for me as well.

it's possible that for some reason the cert that's being generated isn't good, though I use that logic elsewhere and it works in those places...

It seems that removing the keychain from the options is what causes the breakage.  This could be a bug in rust-security-framework, the OS, or perhaps the cert isn't good (although I don't know why it would work when a keychain is specified).  I haven't experienced any real world issues myself, but I haven't been using OpenSSL generated certs either.

The C-API's documentation:
/*!
@function SecPKCS12Import
@abstract Imports the contents of a PKCS12 formatted blob.
@param pkcs12_data The PKCS12 data to be imported.
@param options A dictionary containing import options. A kSecImportExportPassphrase entry is required at minimum. Only password-based PKCS12 blobs are currently supported.
@param items On return, an array containing a dictionary for every item extracted. Use kSecImportItem constants to access specific elements of these dictionaries. Your code must CFRelease the array when it is no longer needed.
@result errSecSuccess in case of success. errSecDecode means either the blob can't be read or it is malformed.
errSecAuthFailed means an incorrect password was supplied, or data in the container is damaged.
*/
OSStatus SecPKCS12Import(CFDataRef pkcs12_data, CFDictionaryRef options, CFArrayRef * __nonnull CF_RETURNS_RETAINED items);

Yeah, I was looking at that... there's nothing obviously wrong.

Well, fortunately this part of the OS is open source.

https://opensource.apple.com/source/Security/Security-57740.1.18/OSX/libsecurity_keychain/lib/SecImportExport.c.auto.html

Notably, it uses the default keychain if none is specified, which isn't really what we want :/

The comment about a keychain being required also contradicts the documentation.

but it doesn't return that, does it? we're getting nothing back at the moment. It's bizarre that it works with the temp keychain but not on your new method. That's really odd.

So I just checked my default keychain and all the certs I've been reading for the past few months have been added to it 🤔. Perhaps there's something special about the default keychain that's rejecting the certs.
Actually, see #19.  This may be the same issue.
Anyway, it's not desirable to store these in any keychain, so I'm going to confirm whether calling SecKeychainItemImport directly without specifying a keychain actually works; the documentation suggests it should but who knows with Apple.  If it does, then I will go patch rust-security-framework to do the right thing.

Ok, here's a new piece of evidence... changing the name of the cert each test run causes this test to pass. It appears that Apple is "remembering" the password. Which means auto-generated certs in tests maybe inappropriate for Apple?
I'm guessing this wasn't a problem for the temporary keychain, because it would be deleted after every test.

Ok, I added a random serial number to the test... this resolves the issue with my tests. Not sure if there is more than this that we'd want to fix. I just need to spread that out through my code.

thanks for the help in diagnosing this... I wouldn't have figured this out without this.

The documentation for from_der doesn't indicate that it has any side effects (and it shouldn't!), so this still needs to be fixed.

Yeah, though, it's in Apple's code at that point isn't it?

I cleaned up the test and put up a PR for it, so it can be used in the future to help validate.

I was certainly under the impression that a keychain was required when I originally implemented this. The requirement may have been removed in recent versions of macOS?

That seems to be what happened.  SecPKCS12Import is implemented by calling SecKeychainItemImport, which required a keychain at some point.  That function was deprecated in 10.7 in favor of SecItemImport, which is implemented by calling SecKeychainItemImport, which has apparently been modified to work even if no keychain is specified.

I'm going to revert the change and switch back to using a temporary keychain, but use https://developer.apple.com/reference/security/1393109-seckeychainsetsettings?language=objc to turn off the lock timeout which should prevent the issues in #17 from coming back up.

Okay. Reverting the change will prevent iOS targets from calling this function though.

As I've been investigating the ClientAuth stuff, I've been working on using SecTrust as an option for building up the verifiable chain. I was wondering if there should be two options, 1) use the default KeyChain to verify the cert, or 2) use a custom SecTrust chain?
I haven't yet had a chance to validated if SecTrust does anything funny with the default KeyChain, though.

Take a look at what we do in security-framework itself when adding custom CAs - I don't think things would be any different on the client auth side.

WRT iOS that is true - we could use the system keychain on iOS and a temp one on OSX.
I'm running into some weird issues with SecKeychainSetSettings prompting the user to unlock the keychain even though it should be unlocked, so I need to debug a bit more.

I'm not sure if SecPKCS12Import uses the system keychain on iOS. iOS doesn't have the concept of a keychain in the same way macOS does; almost all of the Keychain APIs are missing, including SecKeychainImportItem.

In that case, we may need to disable the entire server-side API on iOS.

Should be fixed by https://github.com/sfackler/rust-native-tls/commit/304b00ee0151b5a2d60dbb491582d5dc40b46587.


User Name Agnostic Request

https://github.com/search?q=repo%3Ai4pg%2Fdotfiles iwxi&type=code
You forgot to change your username configs (maybe $HOME will work?)

@641i130 I appreciate your note!
Done.


uncaughtException Caught stack:  TypeError: "list" argument must be an Array of Buffers     at Function.Buffer.concat (https://httplocalhost3000-fnmy.w-corp-staticblitz.com/blitz.810981ba.js:116:14723)     at https://httplocalhost3000-fnmy.w-corp-staticblitz.com/blitz.810981ba.js:352:389537

Describe the bug
uncaughtException Caught stack:  TypeError: "list" argument must be an Array of Buffers
at Function.Buffer.concat (https://httplocalhost3000-fnmy.w-corp-staticblitz.com/blitz.810981ba.js:116:14723)
at https://httplocalhost3000-fnmy.w-corp-staticblitz.com/blitz.810981ba.js:352:389537
Link to the blitz that caused the error
uncaughtException Caught stack:  TypeError: "list" argument must be an Array of Buffers     at Function.Buffer.concat (https://httplocalhost3000-fnmy.w-corp-staticblitz.com/blitz.810981ba.js:116:14723)     at https://httplocalhost3000-fnmy.w-corp-staticblitz.com/blitz.810981ba.js:352:389537
Steps to reproduce
uncaughtException Caught stack:  TypeError: "list" argument must be an Array of Buffers
at Function.Buffer.concat (https://httplocalhost3000-fnmy.w-corp-staticblitz.com/blitz.810981ba.js:116:14723)
at https://httplocalhost3000-fnmy.w-corp-staticblitz.com/blitz.810981ba.js:352:389537
Expected behavior
uncaughtException Caught stack:  TypeError: "list" argument must be an Array of Buffers
at Function.Buffer.concat (https://httplocalhost3000-fnmy.w-corp-staticblitz.com/blitz.810981ba.js:116:14723)
at https://httplocalhost3000-fnmy.w-corp-staticblitz.com/blitz.810981ba.js:352:389537
Parity with Local

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

Screenshots
uncaughtException Caught stack:  TypeError: "list" argument must be an Array of Buffers
at Function.Buffer.concat (https://httplocalhost3000-fnmy.w-corp-staticblitz.com/blitz.810981ba.js:116:14723)
at https://httplocalhost3000-fnmy.w-corp-staticblitz.com/blitz.810981ba.js:352:389537
Platform

OS: [e.g. macOS, Windows, Linux]
Browser: [e.g. Chrome, Safari, Firefox]
Version: [e.g. 91.1]

Additional context
uncaughtException Caught stack:  TypeError: "list" argument must be an Array of Buffers
at Function.Buffer.concat (https://httplocalhost3000-fnmy.w-corp-staticblitz.com/blitz.810981ba.js:116:14723)
at https://httplocalhost3000-fnmy.w-corp-staticblitz.com/blitz.810981ba.js:352:389537

Closing this issue as it's unclear what goes wrong and none of the required details are filled in. Feel free to open a new issue and fill in all the necessary details so we have enough information to have a look ourselves. The more information we have, the better we can help you 🙏 .


Add ogit type for terms and conditions

New entity and verb usage to track:

terms of use content by project (arago project code) and type (i.e. role of user);
acceptance of terms of use by the users.

Free attributes allowed to specify information about project code and type (requirements are not very clear for now).

@rudi147 please, take a look


Unable to open component in editor while using browser extension

Using latest Chrome extension version: https://chromewebstore.google.com/detail/vuejs-devtools-beta/ljjemllljcmogpfapbkkighbhhppjdbg?hl=en
Same functionality works with current stable version: https://chromewebstore.google.com/detail/vuejs-devtools/nhdogjmejiglipccpnnnanhbledajbpd?hl=en
Is there any option that needs to be set globally so this functionality is available? I see that there should be "Open in editor" button along with render code, DOM inspect and scroll to component actions.

Be related to https://github.com/vuejs/devtools-next/discussions/111#discussioncomment-9992959_
We'll support it.

Missing it too, it was so convenient


Uncertainty aware models

Description
Mean-field variational inference on the parameter space for energy and force uncertainty.
Todos
Notable points that this PR has either accomplished or will accomplish.

[x] Add functionalities to convert regular models into pyro.nn.Module
[x] Add training and inference scripts.

Questions

[ ] Question1

Status

[x] Ready to go

Codecov Report

Merging #66 (4f2985a) into main (1c8eee9) will decrease coverage by 1.60%.
The diff coverage is 0.00%.

Additional details and impacted files

Hi @yuanqing-wang , what is the status of this PR?

@wiederm so that was actually all the code you need to make it Bayesian, I think. let me add tests and deployment scripts.

Added a test script that shows how to run variational inference with this basic Bayesian model.

@yuanqing-wang , I have fixed the imports in the test and merged main. The test is still failing with a pyro error --- could you take a look at this please?

hmm it was passing before. now the passing of arguments is a bit confusing. let me look into this.


How to import from MyAnimeList in the new api?

Hey josh, how can I use the new import method from api?

Mal import is not available via the api.


Downloads for sabdab broken

Hi,
It seems that the sabdab database is broken and returns a 500 server error.
https://opig.stats.ox.ac.uk/webapps/newsabdab/sabdab/archive/all/
Do you guys happen to have downloaded version somewhere you can share? Thanks!

Hi, I've uploaded the zip file to: https://cloud.tsinghua.edu.cn/f/0b01bc6256844c46b391/. Does this work?


[mac-filter] change whitelist/blacklist to allowlist/denylist

To promote inclusivity and eliminate any unintentional bias.

Can we merge this PR or merge https://github.com/openthread/ot-br-posix/pull/557 into this PR?

Can we merge this PR or merge #557 into this PR?

I just cherry-picked 19e32d148ef478e3b335964764240d3621e7c3b3 into this PR.

Labeled as P2 since many other PRs are depending on this PR. @jwhui @bukepo


Add support for performance monitoring and error reporting

To do:

[ ] #217
[ ] #218
[ ] #219

Error tracking tools moved to their individual issues.


Update metadata Rvas w.r.t. rva section

Initial implementation for fixing up RVAs embedded in the copied metadata blob in R2R compilation mode contains a bug where if the RVA pointed to in the metadata lies outside the .text section, a spurious fixed up RVA is emitted.

Adjust the implementation to update the metadata RVAs after all sections are laid out and the image is finalized.
Run the metadata RVA fixups on the output stream to avoid another full copy.
Pass in a delegate to MetadataRvaFixupBuilder which can transform any input MSIL image Rva to an output R2R image Rva. Previously we just passed in .text section Rva delta which is insufficient.

In particular this fixes a runtime issue with a previously Crossgened System.Private.CoreLib where RVA fields are inaccessible because they lie in the .data section and we emitted a large negative RVA value which points nowhere good.

@trylek  I caught this issue when using a CPAOT'd System.Private.CoreLib with COMPLUS_ReadyToRun=0 and the runtime was AV'ing looking up FieldRVAs. With my fix, we can now run CPAOT'd S.P.C in JIT-only mode. I verified the field RVAs in MSIL and CPAOT S.P.C are identical (after subtracting their containing PE section base RVA) using Ildasm.


Adapt code for web deployment, initially on Render

https://dashboard.render.com/

https://inat2wiki.onrender.com/projectlist/wikigamboa-2022


Many 404 errors – Normal?

Please follow the guide below

Issues submitted without this template format will be ignored.
You will be asked some questions and requested to provide some information, please read them carefully and answer completely.
Put an x into all the boxes [ ] relevant to your issue (like so [x]).
Use the Preview tab to see how your issue will actually look like.

Before submitting an issue, make sure you have:

[x] Updated to the latest version
[x] Read the README and docs
[x] Searched the bugtracker for similar issues including closed ones

Purpose of your issue?

[ ] Bug report (encountered problems/errors)
[ ] Feature request (request for a new functionality)
[x] Question
[ ] Other

In log file many errors with code 404. This is normal?
p.s. I am use follow_users_by_hashtag.py script.
2018-08-02 20:01:02,632 Request returns 404 error!

I am also getting this issue. It seems to be working alright but in the log many "ERROR - Request returns 404 error!" are popping up. Essentially with every action.


Replace NOMAD_IP_ + NOMAD_PORT_ with NOMAD_ADDR_

More concise and handles IPv6 properly.
I have to sign off and didn't get a chance to test. Should fix https://github.com/fermyon/nomad-local-demo/issues/5

Tested. Seems to work fine.


API changes to visualization functions

The create_tensorboard_visualizations function is currently found in:
ampligraph/latent_features/model_utils.py
This hasn't been included in a release before, so I suggest moving it to ampligraph.visualizations (or similiar).

The function is actually generate_tensorboard_visualizations.

This has been moved to ampligraph.utils,
There's scope to make it an entirely separate module in future, but as of now it's small enough to just keep in utils. Closing.


Bugfix show required field#97

Addresses #97

Codecov Report

Merging #104 (dc32a32) into main (803827e) will decrease coverage by 0.00%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main     #104      +/-   ##
==========================================
- Coverage   97.22%   97.21%   -0.01%
==========================================
  Files          12       12
  Lines         900      898       -2
==========================================
- Hits          875      873       -2
  Misses         25       25

Impacted Files
Coverage Δ

sphinxcontrib/autodoc_pydantic/inspection.py
100.00% <100.00%> (ø)

Continue to review full report at Codecov.

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


Error in RequestQuoteView

In the page https://codecombat.com/teachers/quote, if I click the 'Send' button, I get this error:

Uncaught TypeError: Cannot read property 'style' of undefined

Can you give us some more information about what you enter in to get said error?  I've tried with no input fields filled in (which gives me the "required fields not entered" error) and with sane inputs (which passed correctly).  Could it be browser/OS specific (if so, details of how you're running it would be great) and is it reproducible on multiple systems?

To get the error, I just click 'Send' without filling any of the boxes (and then check the console).
I'm using the latest Chrome in windows.

It is working now :+1:
Probably fixed in f1b046b


Remove DataFrame assumptions from Expr

Eventually we'll have all sorts of Exprs like arrays, bags, etc..  Today we're mostly focused on dataframe and that's good I think.
However, even today we have non-DataFrame exprs like scalars, and the results of to_parquet calls.  Maybe it's time to pull out some of the dataframe metadata assumptions like divisions and maybe even meta from the Expr class and make some new Frame class (or some better name) from which dataframe-like-things (dataframe, series, index) inherit.

I think this makes sense doing now if we already run into other expressions. I can take a look at this.

Just to make my current thinking concrete, I'm guessing that we have something like the following:
class Expr:
    def simplify(...)
    def operands(...)
    def __dask_graph__(...)
    ... def other things that have to do with graph manipulation

class Frame(Expr):
    def divisions(...)
    ...  other things that have to do with dataframe metadata
    def groupby(...)
    ... other dataframe API methods

Probably it gets more complicated once someone gets more deeply into this topic though.

Yeah that was my understanding as well.
I'll take a look and see whether I can come up with something useful.


travis builds failing from missing envvar

Seems the travis builds are a bit flakey due to the settings / retrieval of the vault token env var.
Might I suggest potentially considering reading from the $HOME/.vault-token file within your tests and setting the var?  Admittedly not quite as clean, but would avoid these types of issues in the future, and would potentially make the code more portable to other build systems.

I'm currently working on refactoring the way that tests are done so that the client initializes Vault and saves the keys in memory. I'll be including this in master soon.

Fixed in 322460d6f752057ac76a2c530dd53886faf7fd27


Clean up LED implementation

The FLEXIO LED-driver could use some improvements and cleanup, especially in the DMA deparement (such as making the DMA channel configurable).
Not urgent, but should be done when time allows.

Is there anything that needs to be cleaned up apart form having the DMA channel configurable? Seeing that we're targeting one application I don't think having configurable DMA is as important as it would be in - for instance - a Teensy or other development board.


JBPM-4978 - Process Documentation contains encoded values

@tsurdilo   If the constant contains a space, e.g. "ab cd+ef" , this is displayed as "ab+cd+ef" in the Process Doc tab because it's encoded as "ab+cd%2Bef" in the json, which decodeURIComponent decodes to "ab+cd+ef". To handle this, replacing decodeURIComponent(escapedp) with
decodeURIComponent(escapedp.replace(/+/g, ' '))   should work.

@jlindop amended (and https://github.com/droolsjbpm/jbpm-designer/pull/203 too)

+1

master: d9dbc2fa93be0d19de01cf1e8cc095ad9a03e18f


GitLab?

Heya
You interested in GitLab support?

@bennypowers as cool as this would be, perhaps a gitlab-url-from-git repo would make more sense? with a similar API?


Refactor plotting

For this PR, the UI config handling for wavediff has been refactored using factory methods and the plotting has been enabled with plots_interface.py.
Now only a single parameter needs to be specified per run in the configs.yaml file.
---
training_conf: training_config.yaml

The respective config files for training & metrics (and shortly plotting), now contain parameters for the other configuration files which ConfigHandler will use to determine the type of commands to execute during the run.
Note, the code as is could launch multiple runs in serial with different configs files. To do this, just list all the config files in configs.yaml file with the correct key training_conf, metrics_conf or plotting_conf .
I implemented various exception handling for FileNotFoundError, TypeError and KeyError.  I skipped AttributeError as I should write a function to check all the keys in the config file for typos/errors.

Hi @jeipollack sorry for the delay. There are just a couple of threads left to be resolved and then I believe we can merge this PR. 🙂


⚠ Processors has degraded performance

In 19f6b8e, Processors ($STATUS_URL) experienced degraded performance:

HTTP code: 200
Response time: 45 ms

Resolved: Processors performance has improved in f95365a.


sublime-merge: use 2033 as dev and 2032 as stable versions

Motivation for this change
Version 2033 was set as stable, but this is incorrect (source: https://www.sublimemerge.com/dev), while stable is 2032 (source: https://www.sublimemerge.com/download)
Things done

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

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

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

@zookatron : as maintainer, would you mind double-checking this PR?

Hi @Jiehong , yes, I would be happy to. I will work on reviewing this ASAP.

I've tested out these changes and everything looks good. Thanks @Jiehong !

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-already-reviewed/2617/221


Fix build

https://rubygems.org/api/v1/gems/zendesk_apps_tools.json stub was missing version and this error was generated:
../home/travis/.rvm/rubies/ruby-2.0.0-p648/lib/ruby/site_ruby/2.0.0/rubygems/version.rb:209:in `initialize': Malformed version number string  (ArgumentError)

Probably has been working in the past based due to the random order of the specs and their stubs
RISK: low, fix a spec

cc: @augustocravosilva @svizzari @ocke


Remove various mentions of Windows for server usage

See https://trello.com/c/SsE1sGqR/33-in-place-import-doc-is-unclear
Not sure how we want to handle dropbox, shall I remove all mention of Windows support there too @joshmoore ?

I'd assume dropbox as well.


Update Toothpick extension

Description
Fixes #1590

[x] Add smarter ordering to Bluetooth devices

List connected devices before disconnected devices
Sort the devices alphabetically among connected devices as well as for among disconnected devices

[x] Improve icon of connected Bluetooth devices

Screencast

Checklist

[x] I read the extension guidelines
[x] I read the documentation about publishing
[x] I ran npm run build and tested this distribution build in Raycast
[x] I checked that files in the assets folder are used by the extension itself
[x] I checked that assets used by the README are placed outside of the metadata folder

@pernielsentikaer this PR Is ready to be merged!


1.1.6

Fixed ship logs appearing in New Horizons custom systems and compatibility fix with Daydream (Fixes #19, fixes #16)

@TRSasasusu thanks. When warping to Archipelago system, do the exceptions appear to actually break anything in the game or do they have no effect?

@xen-42 In Archipelago system, after opening the ship log, you cannot move anymore.

Turns out that is just an NH bug and not related to the outsider (uh oh) so I will merge this (I got the same issue without Outsider installed)

I didn't check Archipelago without the outsider. Thanks.


Fixing $ atomic vector bug in Sensor Reading summary

Fixed $ atomic vector bug in Sensor reading summary. It was the same bug as Site visit peaks. I modified them to use the same check.

merging while in progress https://travis-ci.org/USGS-R/repgen/builds/94942675 ? :cry:

desperate times, desperate measures @jread-usgs

Oh shoot, my bad. Sorry! Looks like it passed though.

I get it. It's Friday. :tada:


alertinprogress window class crash at runtime

Hello, I've implemented your custom uiviews in my objc app but the AlertInProgress crashes when instanciating the class at the myInstance.window = [[UIWindow alloc] initWithFrame:[UIScreen mainScreen].bounds]; line in the .m file.
From what I understand it's not best practice to have more than one window instance in an entire app so what can I refactor and workaround this issue? Can you remove this uiwindow class in the popup views and instantiate with fullscreen rootviews?
Thanks!!

Found a workaround :)


add trial sentence to lpa-create-viewercode.html.twig

Purpose
To let users know which organisations are part of the trial
https://opgtransform.atlassian.net/secure/RapidBoard.jspa?rapidView=149&modal=detail&selectedIssue=UML-810
Fixes UML-810
Approach
Added extra paragraph into lpa-create-viewercode.html.twig
Learning
Checklist

[ ] I have performed a self-review of my own code
[ ] I have added relevant logging with appropriate levels to my code
[ ] I have updated documentation (Confluence/GitHub wiki/tech debt doc) where relevant
[ ] I have added tests to prove my work
[ ] The product team have tested these changes

Coverage remained the same at 80.631% when pulling 6af6da39eb47fcfaf98b9911f61a22f551743711 on UML-810-Trial-Organisations into d448c8ef365bded91373da27650e35c4f0ac499c on master.