Bump to Snakemake v8

[ ] Install the latest Snakemake version (document which exactly)
[ ] Run the tutorial locally with it
[ ] Get to run the tutorial using SLURM (executor plugin) on the CeMM cluster -> there seem to be 3 different plugins available, find the one that suits our purposes best
[ ] Document what to do to get it to run

[ ] in general for any executor plugin (akin to here and here)
[ ] specifically SLURM at CeMM (will replace this repo/section of README)

[ ] (Get to) Run the unsupervised_analysis pipeline with test data using SLURM on CeMM cluster

[ ] change the partition from params to resources

[ ] Document the necessary changes
[ ] Create issue for each MR.P module to address the change to bump it to v8 or one central in this repo with a list of all modules like here https://github.com/epigen/mr.pareto/issues/2

importantly, doc in every module the exact Snakemake version it  was tested in, at the same point in the README (add first to the README template)

[ ] <add above created issue(s) here>
[ ] adapt mr.pareto README accordingly to reflect Snakemake v8 usage in all regards

@burtonjake great progress! Please find out and document how to...

...use one central cluster profile per user so you do not have to install or copy it every time for each workflow you want to use
...parameterize the cluster profile from a workflow config either directly or via the resources specified within each rule but not within the cluster profile ie how to map config-/rule-based resources in the configfile and Snakefiles to the cluster profile.

here it seems like you can name and store it wherever you want, as long as you define the global variable SNAKEMAKE_PROFILE accordingly. Maybe I am missing someting.
https://snakemake.github.io/snakemake-plugin-catalog/plugins/executor/slurm.html#using-profiles

in the main docs, they explain profiles (a concept new to me as I have been developing without them and my latest Snakemake version is 7.15.2): https://snakemake.readthedocs.io/en/stable/executing/cli.html#profiles

with exact versions! (that's a MR.P requirement to increase reproducibility)

is this also a problem for full Snakemake installations? If no, what are the advantages of minimal installations?

[ ] if the provided resources (however we provide them) do not fit the user's setup we point them to look at workflow-specific-profiles

[ ] check if workflow-specific-profiles overwrite rule-resources

[ ] add a Wiki page for the migration from 7 to 8 (optional: with a dedicated CeMMie section)

Instructions
# install Snakemake 8.20.1
conda create -c conda-forge -c bioconda -n snakemake8_20_1 snakemake

# install SLURM executor plugin in Snakemake environment
# https://snakemake.github.io/snakemake-plugin-catalog/plugins/executor/slurm.html#
conda install snakemake-executor-plugin-slurm

CeMM SLURM repo:
v3.0.0 supports all Snakemake versions, below the relevant config files:

for Snakeamke version >=8: config.v8+.yaml
for Snakeamke version <8: config.yaml

LOG of Snakemake 8 bump:

works with fixed partition and qos in config.v8+.yaml
works with workflow-specific profile in workflow/profiles/default

tasks

[ ] think through workflow-specific profiles and if "partition" shouldn;t be moved there -> not compatible with previous snakemake version anymore?
[ ] job names should not be random
[ ] logs & error files should exist
[ ] MR.P repo: remove tips or docs with "-reason" as its deprecated


Can i train efficientvit on a single A100 or V100?

Hi , I want to fully retrain the efficientvit model, can you tell me if i can train this model just using the A100 or V100?

It would be great to have more info about the training!


Unhandled Exception: SocketException: Failed to create datagram socket (OS Error: Too many open files, errno = 24), address = , port = 5353

Hi
I am getting these in Flutter ios app.
[ERROR:flutter/runtime/dart_vm_initializer.cc(41)] Unhandled Exception: SocketException: Failed to create datagram socket (OS Error: Too many open files, errno = 24), address = , port = 5353
#0      _NativeSocket.bindDatagram (dart:io-patch/socket_patch.dart:1051:7)
<asynchronous suspension>
#1      _RawDatagramSocket.bind.<anonymous closure> (dart:io-patch/socket_patch.dart:2558:15)
<asynchronous suspension>
#2      MDnsClient.start (package:multicast_dns/multicast_dns.dart:135:40)
<asynchronous suspension>
#3      MdnsScanner.findingMdnsWithAddress (package:network_tools/src/mdns_scanner/mdns_scanner.dart:73:5)

How do I resolve it?

Can you tell us how did you get this error? what values for range you used for subnet

I am trying to do a Host Scan
1 -254

Please attach a reproducible example so that we can test on our machine

It is working now after a Flutter upgrade


Update Minitest dependencies to latest

Update minitest related dependencies to latest version.

@glebm What do you think of just loosening these to ~> 5.6 and ~> 1.0?

@thomas-mcdonald Personally, I agree with that, but I was trying to keep things as they had been. If you'd like I'll loosen it and push that change up.


Configure dispatcher with application-specific type conversions

Background
What's the best way to enforce application-specific type constraints? All action parameters must be simple aggregate types, and therefore constraints cannot be specified during construction. If the constraints were simply checked in the action definition, then there would be redundant checks in every action that used that object type, and subsequently redundant unit tests ensuring that the checks were in place. Therefore, action definitions are not the right place either. It was previously discussed that fracpack would look for a specially-named method on objects, and call it - but then a data serializer is dealing with application-specific constraints, and furthermore it forces all JSON/ABIs to specify an additional property for each object, for example:
// Credit action with wrapper types
"credit": {
    "tokenId": 0,
    "receiver": "",
    "amount": {
        "value": "0"
    },
    "memo": {
        "contents": ""
    }
}

vs.
// Credit action without wrapper types
"credit": {
    "tokenId": 0,
    "receiver": "",
    "amount": "0",
    "memo":  "",
},

Solution proposal

Some way to indicate to the dispatcher that Type A in the function signature means Type B in JSON, fracpack, and ABI. The dispatcher would convert from A to B. This leave the entire set of serializers untouched and get rid of {"content": "..."}

What's the best way to indicate such a mapping to the dispatcher?

CC @tbfleming
I added this to organize any further discussion and so we don't lose the idea

serialization can just forward instead of using reflection:
template<typename T, void (*V)(const T&)>
struct checked
{
   T value;
};

template<typename T, void (*V)(const T&), typename S>
void from_json(checked<T, V>& obj, S& stream)
{
   from_json(obj, stream);
   V(obj.value);
}

Forwarding fracpack serialization is hard to get right and easy to get wrong; it confuses the internal if constexpr logic.

e.g. option<vector<...>> and option<string> are special cases that would have to detect forwarding.


Example algorithms look for .tiff and not .tif

Small typo, should be fixed

I think that .tiff is generally also acceptable as an extension for the format. Better to cover both, rather than only one.
Seems like rse-panimg outputs .tif in reality but suggests it's outputting .tiff: https://github.com/DIAGNijmegen/rse-panimg

Better update things: https://github.com/DIAGNijmegen/rse-panimg/pull/113


Prerequisite checks for auto approval do not insert claim events with failure reason if length is too long

This is because we can only store 250 characters in the ClaimEvent Note column.
The note I was trying to store was 327 characters long. We should increase the length of the Note column, perhaps to the max that knex migration will allow (2000)

Integration test fails - there is a function that is trying to retrieve the PaymentAmount using the old column name.


heat_m dataset

Hello,
Your code looks great and is very helpful! Sorry if I miss any details, but could you please explain the heat_m dataset?
Thank you in advance for replying!

Hi,
The heat_m is dataset is a modified version of the heat dataset where heat is not diffused linearly. In this version, the heat diffusion is slower on the edges. You can find more details in the journal version of the paper.


GetItemCount fails for ender eyes

Steps to Reproduce (as best as you can)
Expected Behavior
Bot makes blaze powder
Bot makes eyes of ender
Actual Behavior (What lead to the bot getting permanently stuck)
Bot swaps between the tasks as it thinks that blaze powder is achieved before it actually is
Crashlogs and Screenshots (if applicable)
(you may drag + drop text logs and images here, thanks GitHub)
` private Task getEyesOfEnderTask(AltoClef mod, int targetEyes) {
if (mod.getEntityTracker().itemDropped(Items.ENDER_EYE)) {
setDebugState("Picking up Dropped Eyes");
return new PickupDroppedItemTask(Items.ENDER_EYE, targetEyes);
}
    int eyeCount = mod.getItemStorage().getItemCount(Items.ENDER_EYE);

    int blazePowderCount = mod.getItemStorage().getItemCount(Items.BLAZE_POWDER); // THIS RETURNS 14 BEFORE THE CRAFTING IS COMPLETE
    int blazeRodCount = mod.getItemStorage().getItemCount(Items.BLAZE_ROD);
    int blazeRodTarget = (int)Math.ceil(((double)targetEyes - eyeCount - blazePowderCount) / 2.0);
    int enderPearlTarget = targetEyes - eyeCount;
    boolean needsBlazeRods = blazeRodCount < blazeRodTarget;
    boolean needsBlazePowder = eyeCount + blazePowderCount < targetEyes;
    boolean needsEnderPearls = mod.getItemStorage().getItemCount(Items.ENDER_PEARL) < enderPearlTarget;

    if (needsBlazePowder && !needsBlazeRods) {
        // We have enough blaze rods.
        setDebugState("Crafting blaze powder");
        return TaskCatalogue.getItemTask(Items.BLAZE_POWDER, targetEyes - eyeCount);
    }

    if (!needsBlazePowder && !needsEnderPearls) {
        // Craft ender eyes
        setDebugState("Crafting Ender Eyes");
        return TaskCatalogue.getItemTask(Items.ENDER_EYE, targetEyes);
    }//CAUSES CYCLING BETWEEN THESE TWO IF BLOCKS`

Issues are written in commetns in all caps

Attempted fix num 1)
Duct Tape solution:
Make sure that the make powder task is fully done before moving on. Hopefully, this will get it out of the inventory before
something else takes over.
In the future, we should figure out why the 2x2 grid is counting potential results as inventory.

attempted fix #2:
check if we are making eyes and then don't stop until we have them.

attempted fix 3)
Swap crafting ender eyes with blaze powder. i.e, make eyes before powder.

Renamed as the get items count is not the issue

Attempted fix 4)
Try crafting eyes first. This will force that task to finish before worrying about rods

                Observation:
                If I provide the bot all the items it needs, such as iron swords, it moves to "collecting 11 beds".
                There may not actually be an issue here, but the order of the tasks may be broken. Meaning, if the
                bot does not have 2 iron picks and 1 sword by the time its making eyes, it will break.

I think that the "Ensure 2x2 crafting grid" is being called somewhere errantly
as we narrowed it down to simply being stuck clearing the grid

Observation
we discovered that blaze powder count is wrong

crafting eneder eyes manually (placing ingrediants in a chest, telling alto to make eyes) works. Error is certainly located in BM2

When blaze powder is in the crafting grid, the count is 0

attempted fix 4)
Check specifically for the crafting slots. Not sure why these were not included before

the crafting fails because "got powder" relies on the current count of blaze powder, which because of the structure of beat minecraft, is obtained each frame during crafting. Because we intentionally do not consider items in the crafting grid as "obtained", the count for blaze powder goes to 0 once the powder is added as part of the recipe. This changes the "got powder" boolean to false when it should be true resulting in the bot clearing its inventory to "get the powder". Once it has the powder, the loop restarts.
This problem is located in the inventory sub tracker
   private static boolean shouldIgnoreSlotForContainer(Slot slot) {         // IMPORTANT NOTE:!!!!         // Ignore crafting table output when calculating container slots.         //         // Why?         // Because we don't want the bot to think we "have" an item if it's in our output slot. Otherwise it will         // softlock because it will assume we're all good (we got the item!) when in reality we need to grab that item.         //         // We also don't want our bot to think we "have" an item if it's in our armor/crafting/shield slots. That's annoying to work with. //TODO re-consider this         if (slot instanceof CraftingTableSlot && slot.equals(CraftingTableSlot.OUTPUT_SLOT))             return true;         if (slot instanceof PlayerSlot) {             // Ignore non-normal inventory slots             int window = slot.getWindowSlot();             return window < 9 || window > 44;         }         return false;     }

Changing the above may cause issues. In short, by not counting these items, we cannot perform logical steps in between crafting phases. However, it isn't an issue with crafting table slots. Notice that for crafting tables, we care if it is the output slot, but not input slots.
Since the bot no longer carries items in its 2x2 crafting grid, we should count items in there.

Partial Progress
Changing the inventory sub tracker to include the count of items in the player crafting slot fixes the crafting issue, but results in the items being stuck in the slot

Solution write up
In order to craft properly in @gamer we have to ensure our count is correct throughout the execution. This means that we had to count items in the crafting grid (2x2). However, this stood in conflict with the estabolished schema of "don't count items in the crafting grid". As a result, when this was changed, the bot made attempts to store items in the crafting grid as it normally would. A few updates ago, it was decided that we should not use the 2x2 grid as item storage and wrote an interface ITaskRequiresCrafting which would clear the crafting grid anytime the grid was not specifically being used for crafting. This caused a loop as Alto would store an item in the crafting grid, pick it up to clear the grid, and then store it there again. A simple change was made to inventory tracker that prohibits alto from considering the 2x2 grid as a storage solution. This resolved the issue and improved run time across the board as altoclef spends considerably less time clearing the grid now since items are never placed there unless specifically for crafting.


[bug] direct exec "explorer.exe /select,"{filePath}""

await $"explorer /select,\"{FilePath}\""
    .CreateCli()
    .ExecuteAsync();


Add Asynchronous Requests

Look at adding asynchronous support to the SDK. This can be done by implementing an Async version of each of the service operations. For example,
// Synchronous request.
$response = $service->getSingleItem($request);
$item = $result->Item;
print("$item->Title\n");

// Asynchronous request.
$response = $service->getSingleItemAsync($request);
$response->then(function ($result) {
    $item = $result->Item;
    print("$item->Title\n");
  })->otherwise(function ($reason) {
    echo 'Encountered an error: ' . $reason->getMessage();
});

This feature will result in a breaking change due to changes to the Handler configuration option. More information can be found in the notes for the upcoming Sophie release


Currency Converter Application

Description
Description
Domain
C/CPP
Type of Contribution
Documentation
Code of Conduct

[X] I follow Contributing Guidelines & Code of conduct of this project.

/assign

/assign


Revert Mystery Box Changes

🚀 Revert Mystery Box Changes
The recent mystery box changes have been confusing and unnecessary. The joy of a mystery box has now been tarnished as there is no excitement for an untradeable, or even a pet. This update also takes away from the fun grind of grinding pets.
The complete mystery of a mystery box was what made them compelling, but now having multiple different types, without any real reason, has tarnished this.
Please revert the mystery box changes to add back the true mystery.

The different mystery boxes was embraced by the BSO community and is now being normally traded over and used.


Exporter should handle the Recorder convertion logic.

Currently a Recorder is converted to a String via Into<String> trait. I think a better approach would be to provide a trait method to iterate over the contents of the Recorder so that an Exporter can implement the conversion logic directly. So the Into<String> trait would not be required.

Recorders are intended to define the formatting, and exporters are intended to define the transport.
The biggest reason that I chose to go with conversion traits is that not every recorder will output a string, and not every exporter will want a string.  I'm not sure it's a wise idea to expose the internal state of a recorder by fiat via the Recorder trait.
For one, it exposes internal state, potentially intermediate state, where the From<T> implementations are meant to signal a finalized conversion into their output format.  Secondly, if we encode this ability into a trait, it means that recorders can only output themselves in one format.
With From/Into, a recorder can render itself into multiple output formats.
As a final example: TextRecorder defines a particular format for the metrics it renders... how they're indented, how they're structured, etc.  The recorder itself, as it's recording, is doing this formatting work.  How would an exporter implement the formatting logic without essentially becoming the formatter itself?

Makes sense.


Why hypervolume goes down

I guess that it could drop due to the changes in diversity

Hi @irecsys,
This is not a bug of pymoo. Many algorithms do not guarantee strict convergence and may choose individuals whose hypervolume contribution is lower than others. You can look at the work on HypE (https://doi.org/10.1162/EVCO_a_00009) for strict hypervolume convergence.
If you are interested, please open a thread under: https://github.com/anyoptimization/pymoo/discussions

Not only that, most algorithms cannot guarantee that the next population is not dominated by a previous one. See: https://lopez-ibanez.eu/publications#LopKnoLau2011emo (the title mentions "archiving", but the conclusions apply to any bounded-size or fixed-size multi-objective solution-set)

The solution is to use an unbounded external archive or a hypervolume-based one.


Fixes #93: Travis build error fixed [WIP]

Fixes #93
Changes: The changes I have made are:

[ ] Downgrade lib

@mariobehling @CloudyPadmal I'll need some time to fix this. Not sure why it is breaking.


App crashes after splash screen

Greeting,
I downloaded this project, and tested with this project original Unity version and the latest Unity version.
All compiled app crashes after the splash screen. And I did follow the instructions that was given.
Anyone faced this issue before?

OK, now I managed to get it work by making the Scripting Backend to IL2CPP.


apptainer: 1.3.1 -> 1.3.2

Description of changes
Changelog: https://github.com/apptainer/apptainer/releases/tag/v1.3.2
cc @ShamrockLee
Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] 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

[x] 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/)
24.11 Release Notes (or backporting 23.11 and 24.05 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

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

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

2 packages built:

apptainer
apptainer-overriden-nixos

Result of nixpkgs-review pr 315423 run on aarch64-linux 1

2 packages built:

apptainer
apptainer-overriden-nixos

apptainer.tests builds fine on both x86_64-linux and aarch64-linux.

CVE -2024-3727 affects apptainer on Nixpkgs 23.11 (Apptainer 1.2.5), as the latter uses an affected version (5.25.0) of go module github.com/containers/image/v5 .

Backport failed for release-23.11, because it was unable to cherry-pick the commit(s).

Quite a few non-trivial changes are applied to apptainer and singularity packages between Nixpkgs 23.11 and 24.05. Automatic backport by cherry-picking is expected to fail.

Please cherry-pick the changes locally and resolve any conflicts.

Instead of cherry-picking, it makes more sense to do a version bump manually based on release-23.11 and pick changes necessary for the update from the master branch.

Is it nontrivial to bump the dependency in Apptainer 1.2.5?

Is it nontrivial to bump the dependency in Apptainer 1.2.5?

That depends on the test result after bumping. Everything will be alright if 'apptainerandapptainer.testsbuilds, andnix run .#apptainer exec "$(nix build --no-link --print-out-paths .#apptainer.tests.image-hello-cowsay)" hello` runs.

Is it nontrivial to bump the dependency in Apptainer 1.2.5?

It'll be trivial, as most updates I could recall were simple version bumps. The actual case depends on the test result after bumping.

The only nontrivial apptainer-specific change between Nixpkgs 23.11 and Nixpkgs 24.05 is #280076. Let's just bump Apptainer to 1.2.5 on Nixpkgs 23.11.

Is it nontrivial to bump the dependency in Apptainer 1.2.5?

Oops! I misunderstood what you were saying and thought you would like to bump Apptainer.
I haven't tried manually " bumping" the dependency version before, but it sounds hacky. Maybe it would be easier to bump the Apptainer version.

I made a backport PR targeting Nixpkgs 23.11 (#319649). Please take a look.


[ClangImporter] Remove importer-based NS stripping.

As @parkera puts it, in the end we wound up with more Foundation declarations imported as members or keeping "NS" than those that dropped it, and any further decisions will be made on a case-by-case basis. Move all of the existing cases of prefix-stripping into Foundation's API notes and drop the logic from the compiler.
Tested by dumping the generated interface for Foundation and its submodules for both macOS and the iOS simulator, and comparing the results. A few cases did slip through here because of the interaction between SwiftName and Availability: nonswift.
rdar://problem/26880017

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

A smoke test on macOS does the following:

Builds the compiler incrementally.
Builds the standard library only for macOS. Simulator standard libraries and
device standard libraries are not built.
lldb is not built.
The test and validation-test targets are run only for macOS. The optimized
version of these tests are not run.

A smoke test on Linux does the following:

Builds the compiler incrementally.
Builds the standard library incrementally.
lldb is built incrementally.
The swift test and validation-test targets are run. The optimized version of these
tests are not run.
lldb is tested.

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.

@DougGregor, mind checking the compiler parts of this?
@swift-ci Please test OS X platform

Cool.

Compiler bits look great, thanks @jrose-apple !

@swift-ci Please test OS X platform

Over to Ted to merge. We only ever stripped from modules named "ObjectiveC" and "Foundation", so this should have no effect on Linux.


unlock doesn't work

request new unlock :
https://rails4-starterkit.herokuapp.com/a/unlock/new
mailer:
https://rails4-starterkit.herokuapp.com/a/unlock?unlock_token=MESrgbKRe-9bg-xxx
404 page not found

same issue here..

changed to class Users::UnlocksController < Devise::UnlocksController fixed it.


Header logo tweaks

What is the context of this PR?
Fixes: #1266
How to review

Check that when the window is at the xs breakpont (300px) the census header logo is the same size with or without the menu button
Check that the stacked ONS header logo is the same size for both Welsh and English

 The width of the census logo should be the same regardless of presence of mobile menu

 The width of the census logo should be the same regardless of presence of mobile menu


breaking changes

execution of the statements sais breaking changes in
WARNING: Breaking changes in the cmdlet 'New-AzVirtualNetworkSubnetConfig' :
WARNING:  - Update Property Name
WARNING: Cmdlet invocation changes :
Old Way : -ResourceId
New Way : -NatGatewayId
WARNING:  - Update Property Name
WARNING: Cmdlet invocation changes :
Old Way : -InputObject
New Way : -NatGateway
[Enter feedback here]

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

ID: bc5b0105-258e-4bcd-f75d-1ad10086cec2
Version Independent ID: 80bd27e2-638b-0982-bd4b-d285ec4cedf8
Content: Create a Bastion host using Azure Powershell
Content Source: articles/bastion/bastion-create-host-powershell.md
Service: bastion
GitHub Login: @cherylmc
Microsoft Alias: cherylmc

@jtrich This warning is about a upcoming change in the New-AzVirtualNetworkSubnetConfig cmdlet, and can be ignored for the purpose of this lab. It only applies if a NAT Gateway is applied to a subnet.
Previously, NAT Gateways were added to a subnet using the -ResourceID parameter, which is not very descriptive of what it is adding. This parameter is being changed to -NatGatewayId.
We will now proceed to close this thread. If there are further questions regarding this matter, please tag me in your reply. We will gladly continue the discussion and we will reopen the issue.


[Bug]: Missing link to "Terraform Style Guide" in codex

Module Name
N/A
Description
Missing link to "Terraform Style Guide" in codex:

Src: https://github.com/Azure/terraform-azure-modules/blob/main/codex/general_rules.md

Code
N/A

Relevant log output
No response
What was your expected output?
No response
Other information
No response

Thanks @jtracey93 for opening this issue, that's because HashiCorp hasn't published this document yet so we cannot obtain a link. We'll update the document once HashiCorp has made that document public.


[Fix] Enable logging of metrics from Callbacks to ConsoleLogging

What does this PR do?
What issue(s) does this change relate to?
fix CO- 1636

Discussed offline, we're going to remove the trainer arg and that arg to the ConsoleLogger

defer to @abhi-mosaic for approval to verify this solves his issue, but lgtm

I sent Abhi a console log output from a mosaicml/examples GPT run and he confirmed over DM that it looks good.


provider.server_port is required

For consistency with all other F5 Ansible modules, provider.server_port should not be required and should default to 443 (or even env var F5_SERVER_PORT).  But atc_deploy throws an error when not provided:
The task includes an option with an undefined variable. The error was: 'dict object' has no attribute 'server_port'

Thank you for the feedback. Tracking

I have worked up the changes I believe are needed to handle default provider values.
Due to variable order of precedence, a set_fact used within a role cannot overwrite the provider included in a playbook as a role parameter. To work around this I set a default_provider in defaults/main.yml, created a role_provider dictionary in tasks/main.yml using the combine filter (to combine the defaults with the role parameters set in the playbook), and updated all of the references from provider.xxx to role_provider.xxx. I'm open to other ideas/methods.
https://github.com/tabernarious/ansible-role-f5_atc_deploy_declaration/tree/default_server_port
I will create a Pull Request for this, but have another Pull in play for something else...


Allowing cors headers at view level

I want only few of my views to support cors and not all. How can I achieve that?

You can use CORS_URLS_REGEX to restrict the URL's CORS is applied to.

P.S. I know this isn't quite the same as a view decorator, I plan to one day rebuild the library and support a decorator.

ok thanks.


Add mypy and check python type annotations

Fix #1023
Builds on #1029

Even with the 13 error codes we ignore here, this is protecting us against a number of other possible errors: I think there are 47 mypy error codes total:
wget https://raw.githubusercontent.com/python/mypy/master/docs/source/error_code_list.rst
grep '.. _code-' error_code_list.rst | perl -pne 's/.*_code-//; s/://' | sort | wc -l
      47

If this much is good, can file a follow-up issue to address the errors in the ignore list: They might be the result of misusing the tool and not real errors at all, for all I know.

Notes on some errors it reports, if I comment out individual lines in the ini:
src/opendp/mod.py:278: error: Too many arguments for "__call__"  [call-arg]

        if isinstance(other, PartialConstructor):
            return self >> other(self.output_domain, self.output_metric)

Haven't been able to make a minimal reproduce: This seems analogous, but doesn't produce a mypy error:
from typing import Union, Any

class MyCallable(object):
    def __call__(self, a, b):
        print(a, b)

def demo(mystery_obj: Union["MyCallable", Any]):
    if isinstance(mystery_obj, MyCallable):
        mystery_obj("foo", "bar")


message-editing: toggle topic-edit-pencil according to message editin…

…g setting.

Fixes: #5946

Notes:

it displays/hides the icon on the home stream (or any current stream selected when changing
the setting), but it doesn't re-render all the streams without re-loading the page
I noticed when closing the org settings view and re-opening it seems like a new event handler is
attached to the view / ’update' button every time the settings view is opened and it’s not being
destroyed when the view is closed.
This results in the functions being called one additional time, every time the view gets closed
and reopened. I found this issue to be existing in the master branch as well.
This seemed out of the scope for this PR, should this be added as an issue or do we think it’s
fine if the user needs to refresh the page?

@franziskagoltz that existing bug would have been fine to just fix in a second commit in this PR.  This change is a clear improvement, though, so I'm going to merge this now; can you work on that follow-up issue?
For the re-rendering issue, I've started work on that in #6223.

Merged, thanks @franziskagoltz!  BTW, is it intentional that you have your GIT_AUTHOR_NAME set to your username rather than your name?

Hi @timabbott,
sorry for the delayed response, I was traveling last week. Yes, I can work on the follow-up issue (the follow up issue being that the icon should re-render across all streams, right?)
And no, my git user name being my GitHub user name was not intentional, thanks!

No worries; I hope you had a great trip!
The follow-up issue is this:
I noticed when closing the org settings view and re-opening it seems like a new event handler is
attached to the view / ’update' button every time the settings view is opened and it’s not being
destroyed when the view is closed.
This results in the functions being called one additional time, every time the view gets closed
and reopened. I found this issue to be existing in the master branch as well.
This seemed out of the scope for this PR, should this be added as an issue or do we think it’s
fine if the user needs to refresh the page?

(The re-rendering issue I actually fixed in #6223, though it'd be great if you re-tested that!).


Address field type - fixes #43

Was a nicety, but a super basic feature that could be added easily. As
it happens in a few weeks we’re starting a new app that will need basic
address lookups!
usage
$this->crud->addField([
            'name' => ‘address’,
            'type' => 'address',
            'label' => 'Address Lookup'
        ]);

This looks and works great! It's really beautiful field.
One question: does Algolia also provide the GPS coordinates, city name/id, country name/id, postcode, zipcode or other information? I would store everything in the same "address" field, as JSON (with attribute casting to array or object).
My reasoning is this:

right now, Algolia helps to write the address faster; which is nice;
but if we also use Algolia to store extra information about that address (lat, lng, postcode, city, country) developers could use one field instead of multiple AND have the coordinates to place pins on a map, if necessary; that would be a huge help in some cases;

I did consider doing this!
Algolia provides back "the data that it has" - from my testing, the data-set is actually inconsistent.
For example when searching some postcodes, it returned no data, but searching the address returned it, some address searches returned back the postcode, some searches didn't return a postcode.
So I was thinking of adding an extra param similar to address_full => true/false - which would save the full response back, but the developers need to be aware that the data returned from Algolia wont be 100% reliable.

the GitHub app went weird, showed I was on a different branch but still pushed to this branch! so accidently pushed other changes into this branch, they're reverting now!

Ok, that sounds good. I've changed the naming to store_as_json, though, so it would be more intuitive. Merging into master!
Another great work, @OwenMelbz  :-) Thank you!


missing textures

Describe the Bug
i have some missing textures for few items,
its missing for: water wheel, large water wheel, crushing wheel, blaze burner, etc..

Reproduction Steps
i just open world and dont have the textures
Expected Result
its either issue of the mod or issue of my minecraft
Screenshots and Videos
No response
Crash Report or Log
No response
Operating System
Windows 10
Mod Version
0.5.1d
Minecraft Version
1.20.1
Other Mods
No response
Additional Context
No response

either needs indium or is an upstream issue.


Resolves #363 Visual glitch/flash on refreshContentView() while cell is swiped open

Specifically, when refreshContentView() is called, a new UIImage snapshot of the cell's content view is created using the older - (BOOL)drawViewHierarchyInRect:(CGRect)rect afterScreenUpdates:(BOOL)afterUpdates with UIGraphicsBeginImageContextWithOptions.  In order to cause the new content to be rendered, afterUpdates must be YES, which causes the user's screen to redraw in order to make the latest content available.
Changed - (UIImage *)imageFromView:(UIView *)view cropSize:(CGSize)cropSize to use UIGraphicsRenderer and - (void)renderInContext:(CGContextRef)ctx instead, eliminating the need to draw a view update to the screen.
Resolves https://github.com/MortimerGoro/MGSwipeTableCell/issues/363

Thanks for looking into this. I have tested it and LGTM!


Adding to leaflet plugins

We could get this plugin listed on https://leafletjs.com/plugins.html - what do you think @boldtrn ?

We could add you as the maintainer.

Sounds good, that's a great idea :).

@boldtrn let's see ;-) https://github.com/Leaflet/Leaflet/pull/6731

Awesome, thanks @TimMcCauley!

listed in
https://github.com/Leaflet/Leaflet/pull/6731


GH-4886 problem with use of @TestFactory and @TempDir

GitHub issue resolved: #4886
Briefly describe the changes proposed in this PR:

PR Author Checklist (see the contributor guidelines for more details):

[ ] my pull request is self-contained
[ ] I've added tests for the changes I made
[ ] I've applied code formatting (you can use mvn process-resources to format from the command line)
[ ] I've squashed my commits where necessary
[ ] every commit message starts with the issue number (GH-xxxx) followed by a meaningful description of the change

@JervenBolleman I've run into a more general issue with your rewrite of the SPARQLQueryComplianceTest class et al. because the JUnit 5 @TestFactory annotation doesn't work very well with the @TempDir annotation. In fact @TestFactory doesn't work well with any of the lifecycle annotations, including before each. Essentially the combination of @TestFactory and @TempDir means that we use the same temporary directory for all the tests in a class, instead of a new sail and repository with a new temporary directory for each and every test.
This issue has been masked by the test code catching exceptions. Pretty much none of the NativeStore tests have been running. I managed to get the NativeStore tests working now, but the LmdbStore doesn't seem so happy about being continuously re-instantiated on top of the directory for an existing LmdbStore. @kenwenzel maybe you could take a look at why the LmdbStore gets upset after a while when being shutdown and then recreated on the same directory. You should be able to reproduce the issues by running LmdbPARQL11UpdateComplianceTest.

Oh no, I am so sorry for this!

"Aldri så galt at det ikke er godt for noe" - Norwegian way to say that even when bad things happen they still bring something good. The ExtensibleStore test failing led to discovering the issue with the tests here, and that led to discovering the issue with the LmdbStore :)
Maybe a quick fix is to add a File or Path param to the creeateRepo method and manually handle the creating and deletion of temporary directories.
Another approach would be to try to use the ParamaterizedTest annotation with the method source option. I do this with the ShaclSail where I read the directory structure to generate the tests at runtime.
It's still really nice to be able to run just a single test from IntelliJ though! Instead of always having to run every single one like I had to before.

I tried to create a dir per dynamic test. Still have the failing extensiblestore test. Native and Lmdb tests seem to run?
The code is in pull:4888

Yeah, I think the ExtensibleStore failure is unrelated. It uses a ConcurrentHashMap as the main data structure when running the tests, so maybe there are some changes to that class in jdk 21 that is causing the issue. Not sure what it could be, and it's particularly annoying that I can't reproduce it in IntelliJ.

@hmottestad I could imagine that something is not correctly closed when unsing the LmdbStore. Have you checked that everything gets closed after running the tests?

From what I could see everything is closed correctly, but I haven't done anything more than look over the code.

Is the store instantiated with an empty directory or are there any artifacts left from the previous run?
It might also be the case that the store is not correctly closed due to some issue like
https://github.com/eclipse-rdf4j/rdf4j/issues/4806
I'll try to take a look within the next days.

Is the store instantiated with an empty directory or are there any artifacts left from the previous run?
It might also be the case that the store is not correctly closed due to some issue like
https://github.com/eclipse-rdf4j/rdf4j/issues/4806
I'll try to take a look within the next days.

You can checkout this commit here: 5d825520e1c268222d6cb0163724c26ffa94f539
The issues occurs when the store is shutdown() and then recreated with the same directory. It doesn't happen immediately but after ~6 times.

Is the store instantiated with an empty directory or are there any artifacts left from the previous run?
It might also be the case that the store is not correctly closed due to some issue like
#4806
I'll try to take a look within the next days.

You can checkout this commit here: 5d82552
The issues occurs when the store is shutdown() and then recreated with the same directory. It doesn't happen immediately but after ~6 times.

@hmottestad I nailed this down to an issue with the LMDB ValueStore and its recycling of IDs. I hope that I can fix this within the next days.


[BDW][Media][Decode/Encode]some relevant package for build libyami should be detailed expecially when adding the parameter --enable-gles

1.Testing environment:
Libva: (master)4763db1c2133d4e6b92355938ecb6f23a7767b6b
Libva_intel_driver: (master)4a1c4d21f3428b08ef765d7f7de75b97006514ac
libyami: (master)5c64c7237e06e41d0c64b4bc97c5d74900531786
2.Test command
(1) the steps of build for libyami
git clean -dxf && ./autogen.sh --prefix=$LIBYAMI_PREFIX --enable-tests --enable-tests-gles && make -j8 && make install
(2)check the build result
/home/wenxin/libyami/libyami/tests/decodeoutput.cpp:486: undefined reference to glEGLImageTargetTexture2DOES' /home/wenxin/libyami/libyami/tests/decodeoutput.cpp:491: undefined reference to eglDestroyImageKHR'
gles2_help.o: In function createTextureFromPixmap': /home/wenxin/libyami/libyami/tests/./egl/gles2_help.c:85: undefined reference to eglCreateImageKHR'
/home/wenxin/libyami/libyami/tests/./egl/gles2_help.c:88: undefined reference to glEGLImageTargetTexture2DOES' egl_util.o: In function createEglImageFromDrmBuffer':
/home/wenxin/libyami/libyami/tests/../egl/egl_util.c:47: undefined reference to eglCreateImageKHR' collect2: error: ld returned 1 exit status make[2]: *** [yamidecode] Error 1 make[2]: *** Waiting for unfinished jobs.... mv -f .deps/vpp.Tpo .deps/vpp.Po make[2]: Leaving directory /home/wenxin/libyami/libyami/tests'
make[1]: *** [all-recursive] Error 1
make[1]: Leaving directory `/home/wenxin/libyami/libyami'
make: *** [all] Error 2

wenxin, maybe your libyami pkg-config find the egl.pc and glesv2.pc.
But the egl and glesv2 library location is not right.
Pls check again.

Newer versions of Mesa don't have these defined in their symbol table... we need to use eglGetProceAddress instead.  I have a patch in the pipline...

#252 can fix this issue, @xuguangxin will you add this patch into libyami?


[BUG] Terraform Provider can't apply user data

Describe the bug
When creating VM by terraform provider, cloudinit.user_data will not be applied to the VM.
To Reproduce
Steps to reproduce the behavior:

Create file vm.tf with following snippet

resource "harvester_image" "issue-2255" {
  name      = "issue-2255"
  namespace = "harvester-public"

  display_name = "issue-2255"
  source_type  = "download"
  url          = "http://cloud-images.ubuntu.com/releases/focal/release/ubuntu-20.04-server-cloudimg-amd64.img"
}

resource "harvester_virtualmachine" "issue-2255" {
  name      = "issue-2255"
  namespace = "harvester-public"
  description = "test issue-2255"
  restart_after_update = false

  cpu    = 2
  memory = "2Gi"

  run_strategy = "RerunOnFailure"
  hostname     = "issue-2255"
  machine_type = "q35"

  network_interface {
    name         = "default"
  }

  disk {
    name       = "rootdisk"
    type       = "disk"
    size       = "10Gi"
    bus        = "virtio"
    boot_order = 1

    image       = harvester_image.issue-2255.id
    auto_delete = true
  }

  cloudinit {
    user_data    = <<-EOF
      #cloud-config
      user: ubuntu
      password: root
      chpasswd:
        expire: false
      ssh_pwauth: true
      package_update: true
      packages:
        - qemu-guest-agent
      runcmd:
        - - systemctl
          - enable
          - '--now'
          - qemu-guest-agent
      EOF
    network_data = ""
  }
}

execute terraform apply
Image will be created, VM will be created with empty User Data

Expected behavior
User Data should be applied
Environment:

Harvester ISO version: v1.0-416db269-head
Underlying Infrastructure (e.g. Baremetal with Dell PowerEdge R630): qemu/KVM single node

Additional context
https://user-images.githubusercontent.com/5169694/178966790-9a8d8126-d5a8-47ec-a75a-7a1f1d97dc12.mp4
issue discovered when testing #2452

terraform provider harvester support three methods to add userdata like kubevirt http://kubevirt.io/api-reference/v0.49.0/definitions.html#_v1_cloudinitnocloudsource

user_data
user_data_base64
user_data_secret_name

Harvester UI only use secret to apply userdata
If you check the vm yaml, you will see the user_data is applied
just Harvester UI didn't show the user data content configured by the first way

can we default to use user_data_secret_name as it will be consistent with the UI behavior, thanks?

This bug more like a integration case, when user deployed VMs via terraform, and they should still able to modify User Data via dashboard UI.

Verified this bug has been fixed.

Test Information

Environment: qemu/KVM single node
Harvester Version: master-dc6a3089-head
ui-source Option: Auto
Harvester Terraform Provider Version: v0.5.1

Verify Steps:

Follow Steps to reproduce in https://github.com/harvester/harvester/issues/2503#issue-1304616932


Not responsive on an iPad

I’m on an iPad Pro iOS 11.2.6. When I try to navigate to another component, I need to touch a link on the sidebar twice to actually starts the navigation.
I’m not sure if this is related to this repo or the angular itself.

I think it might possibly be iOS and a CSS quirk it has documented here: https://css-tricks.com/annoying-mobile-double-tap-link-issue/
I could repo on my iPad, but getting a dev environment up will take a bit more work.

So I did narrow this down... It is because the ActiveItemMixin adds mouseenter and mouseleave events which then makes the browser not fire a click event. I couldn't find any documentation on this explicitly for webkit, but the exact behavior for webkit is not well documented.
The reason ActiveItemMixin has this code is for a Firefox bug that was causing list items from popups to scroll weirdly since we were adding scroll into view to ensure the list items were changing on keyboard events. So I don't think I can just remove the events.
I'm not sure exactly what I am going to do to mitigate it, but wanted to preserve the hours worth of debugging I already did.

This particular instance should be fixed in the next release. Migrated away from ActiveItemMixin for most use cases including the one in the gallery.

Hey @TedSander.
I have the iOS double tap issue throughout my app. I wondered if you have some insight in to it as you've spent some time debugging it?
It happens consistently on my tab elements & I'm using these mixings to style them:
@include tab-strip-color('.material-header', $brand-fg, $accent);
@include tab-strip-selected-tab-color('.material-header', $brand-fg);
@include tab-strip-accent-color('.material-header', $accent);

Hey @TedSander.
I have the iOS double tap issue throughout my app. I wondered if you have some insight in to it as you've spent some time debugging it?
It happens consistently on my angular tab & button components & I'm using these mixings to style my tabs:
@include tab-strip-color('.material-header', $brand-fg, $accent);
@include tab-strip-selected-tab-color('.material-header', $brand-fg);
@include tab-strip-accent-color('.material-header', $accent);

When you touch a tab, it does highlight it but doesn't activate it,  I wondered if it was caused what I'm doing here:
::ng-deep body {
-webkit-tap-highlight-color : transparentize($accent, 0.7);
}

This also features in the general angular material mixin:
@mixin mdc-ripple-surface() {
  -webkit-tap-highlight-color: rgba(0, 0, 0, 0);

Both the tab components and button components use the ripple surface so I wonder if this is the cause?

This could be related?
https://css-tricks.com/annoying-mobile-double-tap-link-issue/

I've just read the blog post that you mentioned earlier.  If this is the cause then it's to do with the css :hover selector which should apparently be used only in conjunction with @media (pointer: fine) { in order to avoid this issue.
I've just checked and I don't have any :hover selectors in my project but there are 42 mentions of it in the angular_components project that may potentially be the cause of the issue? Just quickly skimming over them the material_button or mdc_web/ripple mixins look to be the most likely suspects as they're general purpose and shared between components.
I could look at implementing a fix but not sure where you would want the media query to be implemented?

Please leave separate bug reports with simple repos for each instance if possible.
As you can see in the above debugging it wasn't quite always clear that the hover was the problem, or what/where the fix is all the time. I don't see us going through and ripping out one pattern especially when it doesn't always fix the underlying issue.
For instance the hover styles were not the problem with the menu instance above it was certain events.
That being said most of our products aren't used on tablets very often so the priority of fixing said bugs will probably not be too high, but we will see what we can do.

Okay I'll try to put together some simple repos showing the issue. It's not just iPad though, it's mobile Safari so all iPhones too.


Document "how to overwrite/undefine macros"

Description
We discussed in a copr PR there's a way how to override macros, namely:
spec = Specfile("python-specfile.spec", macros=[("fedora", "39"), ("rhel", "")])

Is this documented somewhere?  Is there a way to "undefine" (e.g. %rhel or %fedora macro), not replace with an emtpy string?
Benefit
I checked the README.md, and wasn't able to quickly find this topic. Sorry if I'm just blind.
Importance
No response
Workaround

[ ] There is an existing workaround that can be used until this feature is implemented.

Participation

[ ] I am willing to submit a pull request for this issue. (Packit team is happy to help!)

No, unfortunately it's not documented anywhere, except for the docstring.
It's not possible to undefine a macro, but that could be easily implemented.


Can I specify multiple outgoing ip addresses when I have multiple ethernet cards?

Can I specify multiple outgoing ip addresses when I have multiple ethernet cards?
I found a really nice java proxy.
It would be really cool to have this feature.

Do you want to use a specific ethernet card when the proxy server sends requests? You can do it directly by configuring the routing table.


(PieChart) onPress event is not returning data properly

Description
When I tap on small area than its data is coming wrong (Expected value is 'Andhra Pradesh' , Getting values is 'Delhi'. (refer video duration 0 - 10 secs)
Also it changes its value when click on same area multiple times.(refer video duration 0.14 secs)
Please refer to video
My Json I am using in PieChart
PieChartData.json
https://github.com/user-attachments/assets/3111ea4a-2aa4-483a-ae90-b35810727a6d
Steps to reproduce

Render pie chart with multiple values.
Click on same area multiple times (we will get different values on different taps)
Click on small area (we will not get actual value of that area)

Snack or a link to a repository
No response
version of react-native-gifted-charts
^1.4.29
React Native version
0.74.3
Platforms
Android
Workflow
React Native

Hi @aabidDevit 👋
This has been fixed from versions 1.4.39 onwards. 🎉
Please use the latest version of the library.


diagnostics: meaningful error when graph node has wrong number of inputs

Objective
Currently, providing the wrong number of inputs to a render graph node triggers this assertion:
thread 'main' panicked at 'assertion failed: `(left == right)`
  left: `1`,
 right: `2`', /[redacted]/bevy/crates/bevy_render/src/renderer/graph_runner.rs:164:13
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace

This does not provide the user any context.
Solution
Add a new RenderGraphRunnerError variant to handle this case. The new message looks like this:
ERROR bevy_render::renderer: Error running render graph:
ERROR bevy_render::renderer: > node (name: 'Some("outline_pass")') has 2 input slots, but was provided 1 values

Changelog
Changed
RenderGraphRunnerError now has a new variant, MismatchedInputCount.
Migration Guide
Exhaustive matches on RenderGraphRunnerError will need to add a branch to handle the new MismatchedInputCount variant.

bors r+


Jackson support

Jackson serializer/deserializer interceptor

per application / per routing entry configuration
send objects through response.send()
get objects through request.content.get
nulls support

open questions:

RequestJsonContent vs TypedJsonContent
is there way to not add send and request.content interceptors for every request?

@cy6erGn0m
Why hasn't this p-r merged yet? I want to use JSON serialization with ktor~

Perhaps usurped by #138

This PR is obsolete.


canvas_item shader_type

Feature/enhancement description:
For 2d textures in Godot, the shader_type is canvas_item, and output goes to COLOR= in the vertex() function (rather than fragment()).  There should be a new type of material, called something like "Dynamic 2D material" to support this.

material_2d.txt
Something like this -- but I have no idea what to do for Unity or Unreal, so ignore those parts.

I guess this could just be an export option of the dynamic unlit material.


Documentation refers to deprecated annotation EnableEurekaClient

The official documentation example for Eureka (Registering with Eureka) under URL http://projects.spring.io/spring-cloud/spring-cloud.html still refers to the @EnableEurekaClient annotation, which was changed to @EnableDiscoveryClient few revisions back.

It was re-introduced (in RC1 I think) as a way for the user to say he wants Eureka discovery specifically. I added a sentence to the docs.


Segfault with SelectGene function

Hi,
I'm trying to get a gene list, but I get a segmentation fault. I couldn't install on windows so I istalled on WSL linux. The error I get is presented below:
`*** caught segfault ***
address (nil), cause 'memory not mapped'
Traceback:
1: DsGene(expr, labels_in, as_df, k, 0, n_threads)
2: SelectGene(data_mat, k = 50)
`
Thanks

We are now developing a pure R implementation of SciBet for Windows users, which will be released in around 2 weeks.

Hi, I met the same problem, this happen on linux not windows,  how to solve it on linux?

Could you please provide more detailed information including OS version, R version or session info for us to debug? In current situation, please use the pure R implementation temporarily (https://github.com/zwj-tina/scibetR/blob/master/R/SciBetR.R)

expr last column should be "label",add this information,


Changed cpu-z rule to use https

What happened to the updater? Last update was 2 days ago.

There hasn't been any updates: https://ci.appveyor.com/project/geek1011/just-install-updater-go/builds/33908263#L906.

Ah. I see. The Empoche and Handbrake PRs are still waiting. That's why the entries weren't updated. 😀


i'm unable to push to my ios devices...

the .p12 certificate that i have,
If used on onesignal i'm able to push notifications on ios devices, but not on this module please help me my whole work is pending because of this i think the issue is with the .pem file that is being generated.
can you please help me it's a bit urgent.
the error that i get is ...
Application modified : PlaySports
Removing connection to PlaySports
Adding connection to APNS for  PlaySports
::ffff:<IP_ADDRESS> - POST /admin/application/edit/56580f4bbf77f7a571dddb39 HTTP/1.1 302 92 - 1289.899 ms
APNS : Error while initializing the connection to the feedback platform for PlaySports[ios] : Error: error:0906D06C:PEM routines:PEM_read_bio:no start line
[Error: error:0906D06C:PEM routines:PEM_read_bio:no start line]

i'm really unable to understand what it is can you please help me....
i guess the issue is with .pem file can you please please help me...

This error typically occurs if a cert is used as a key, and vice-versa.
Instructions for generating compatible key & cert for pushserver (which uses node-apn for iOS push) are available here : https://github.com/argon/node-apn/wiki/Preparing-Certificates

ok thanks.
i'll look into it and try it ..

hi
this is teammate of dipesh
thanks a lot, the certificates are working perfectly fine.
but now i'm getting a different error
APNS : Unable to connect to APN for application PlaySports[ios] : TypeError: msecs must be a number
[TypeError: msecs must be a number]

can you please look into it...

hi @fredericbarrau i think this is issue of ssh connection
is this the same issues that we are facing .. ?
https://github.com/jcoglan/vault/issues/29

I'll need further information in order to help.
which version of node are you using ? on which system ?

hey,
below are my system specifications
nodejs = v0.12.7
npm  = 2.11.3
Ubuntu 32bit 14.10
Please help.

Seems like a node 0.12 issue, but I haven't been able to reproduce it with node 0.12.9 on Mac OS
Quick inspection of the code does not give me any hint either.
I'll give it a try on a Ubuntu box, and let you know if I find anything.


Don't separate concerns as much

Right now, the package is separated into core and common for some reason. This is a bad practice, just flatten the folder structure a little.

I just make this simple Stracture for the future , if for some reason we decided to add it as a core Module , but I don't think so , It will soon have a breacking changes , so it's not stable at all , as a README saying , it's under construction 😃

haha 👍


Remove wrapper function in srcset check

There are no variables declared here so it seems pointless to have a function.

I also saw this and I have removed it in 3.0. The new test looks a little bit crazy now, thanks to the crazy browser world: https://github.com/scottjehl/picturefill/blob/3.0/src/picturefill.js#L887-L898.


[Bug]: Add AmazonSSMManagedInstanceCore policy to agents.

Describe the bug
AmazonSSMManagedInstanceCore is  missing to agents config file.
To reproduce
AmazonSSMManagedInstanceCore is nor part of agent-node-config.ts
Expected behavior
No response
Screenshots
If applicable, add screenshots to help explain your problem.
Host / Environment
No response
Additional context
No response
Relevant log output
No response

PR https://github.com/opensearch-project/opensearch-ci/pull/124 merged.


Iconfont support

Is iconfont currently supported ?  http://www.iconfont.cn/
Is there a point in time ?

#825 is working in progress. Please check news out there. This issue will be closed.
Thanks.


Add "restart" to Chat workflow

Currently, one has to restart/refresh the propup (or exported files) to clear a chat, but the "restart" button is only visible in behaviors other than chat. This means folks have to refresh the page using something other than a button.

This should also include adding the restart button to instances where the user is asked if they want to continue generating text.


Unmet peer dependency even with request installed

I'm sorry to bring this up yet again but hear me out. I saw many other issues in this repo related but the answer was always "Install request first". I have done that but I'm still getting the error. I have even tried installing<EMAIL_ADDRESS>specifically with the same result. The package that is installed is<EMAIL_ADDRESS>however. I'm guessing this could have something to do with it. Otherwise I'm lost!
I am using the Ubuntu WSL on Windows 10 if that makes a difference. I can't test on a real Ubuntu install so maybe this is unique to WSL since I'm not seeing this issue from others?
Below are the commands I have run exactly and their output.
akeenan / ~ > npm install -g request
/home/akeenan/.node_modules/lib
└─┬<EMAIL_ADDRESS>  ├──<EMAIL_ADDRESS>  ├──<EMAIL_ADDRESS>  ├──<EMAIL_ADDRESS>  ├─┬<EMAIL_ADDRESS>  │ └──<EMAIL_ADDRESS>  ├──<EMAIL_ADDRESS>  ├──<EMAIL_ADDRESS>  ├─┬<EMAIL_ADDRESS>  │ └──<EMAIL_ADDRESS>  ├─┬<EMAIL_ADDRESS>  │ ├─┬<EMAIL_ADDRESS>  │ │ ├──<EMAIL_ADDRESS>  │ │ ├──<EMAIL_ADDRESS>  │ │ ├──<EMAIL_ADDRESS>  │ │ └─┬<EMAIL_ADDRESS>  │ │   └──<EMAIL_ADDRESS>  │ └──<EMAIL_ADDRESS>  ├─┬<EMAIL_ADDRESS>  │ ├──<EMAIL_ADDRESS>  │ ├─┬<EMAIL_ADDRESS>  │ │ ├──<EMAIL_ADDRESS>  │ │ ├──<EMAIL_ADDRESS>  │ │ └─┬<EMAIL_ADDRESS>  │ │   └──<EMAIL_ADDRESS>  │ └─┬<EMAIL_ADDRESS>  │   ├──<EMAIL_ADDRESS>  │   ├──<EMAIL_ADDRESS>  │   ├──<EMAIL_ADDRESS>  │   ├──<EMAIL_ADDRESS>  │   ├──<EMAIL_ADDRESS>  │   ├──<EMAIL_ADDRESS>  │   ├──<EMAIL_ADDRESS>  │   └──<EMAIL_ADDRESS>  ├──<EMAIL_ADDRESS>  ├──<EMAIL_ADDRESS>  ├──<EMAIL_ADDRESS>  ├─┬<EMAIL_ADDRESS>  │ └──<EMAIL_ADDRESS>  ├──<EMAIL_ADDRESS>  ├──<EMAIL_ADDRESS>  ├──<EMAIL_ADDRESS>  ├──<EMAIL_ADDRESS>  ├─┬<EMAIL_ADDRESS>  │ ├──<EMAIL_ADDRESS>  │ └──<EMAIL_ADDRESS>  ├──<EMAIL_ADDRESS>  └──<EMAIL_ADDRESS>
akeenan / ~ > npm install -g request-promise-native
/home/akeenan/.node_modules/lib
├── UNMET PEER DEPENDENCY<EMAIL_ADDRESS>└─┬<EMAIL_ADDRESS>  ├── UNMET PEER DEPENDENCY<EMAIL_ADDRESS>  ├─┬<EMAIL_ADDRESS>  │ └──<EMAIL_ADDRESS>  ├──<EMAIL_ADDRESS>  └─┬<EMAIL_ADDRESS>    ├──<EMAIL_ADDRESS>    └──<EMAIL_ADDRESS>
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none was installed.
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none was installed.
akeenan / ~ >

I'm having a similar as well @adamk33n3r with<EMAIL_ADDRESS>Although I'm using yarn instead of npm.

I'm having a similar as well

Please help as this is a blocker for me


TYP: update supported Mypy version from 1.0.0 to 1.10.0

This is pretty straightforward; needed to support Python 3.12

Does this fix the typing issues in #20593?

so if CI doesn't care and it is just me seeing that fair enough.

Yep. I am seeing a few other errors too locally, but it's essentially impossible to make Mypy happy in multiple configs. So I'd say that if CI is green and the changes make sense, everything else can be ignored.

Does this fix the typing issues in #20593?

No, but I started to work on those, which triggered this PR. I want to keep a Mypy upgrade separate from changes to array API support.


fix subscription when enable monitor

What is the purpose of the change
same fix on monitor in 2.6.x
because there is bug which willl block shutdown progress in 2.5.x and it was fixed in 2.6.3.
related: #2110
Follow this checklist to help us incorporate your contribution quickly and easily:

[x] Make sure there is a GITHUB_issue filed for the change (usually before you start working on it). Trivial changes like typos do not require a GITHUB issue. Your pull request should address just this issue, without pulling in other changes - one PR resolves one issue.
[ ] Format the pull request title like [Dubbo-XXX] Fix UnknownException when host config not exist #XXX. Each commit in the pull request should have a meaningful subject line and body.
[ ] Write a pull request description that is detailed enough to understand what the pull request does, how, and why.
[ ] Write necessary unit-test to verify your logic correction, more mock a little better when cross module dependency exist. If the new feature or significant change is committed, please remember to add integration-test in test module.
[ ] Run mvn clean install -DskipTests & mvn clean test-compile failsafe:integration-test to make sure unit-test and integration-test pass.
[ ] If this contribution is large, please follow the Software Donation Guide.

Codecov Report

Merging #2166 into 2.6.x will increase coverage by 0.01%.
The diff coverage is 57.14%.

@@             Coverage Diff             @@
##             2.6.x    #2166      +/-   ##
===========================================
+ Coverage     47.3%   47.32%   +0.01%
- Complexity    4266     4267       +1
===========================================
  Files          557      557
  Lines        24390    24397       +7
  Branches      4340     4342       +2
===========================================
+ Hits         11538    11545       +7
  Misses       11054    11054
  Partials      1798     1798

Impacted Files
Coverage Δ
Complexity Δ

.../alibaba/dubbo/config/AbstractInterfaceConfig.java
80.78% <57.14%> (-0.75%)
81 <0> (ø)

...onfig/spring/extension/SpringExtensionFactory.java
73.07% <0%> (-11.54%)
8% <0%> (ø)

...a/dubbo/remoting/transport/netty/NettyChannel.java
61.25% <0%> (-5%)
20% <0%> (-1%)

...rpc/protocol/dubbo/telnet/InvokeTelnetHandler.java
51.89% <0%> (-2.54%)
11% <0%> (-1%)

...bo/remoting/transport/netty/NettyCodecAdapter.java
54.68% <0%> (+1.56%)
3% <0%> (ø)
:arrow_down:

...ba/dubbo/remoting/transport/netty/NettyServer.java
71.42% <0%> (+3.57%)
9% <0%> (+1%)
:arrow_up:

...aba/dubbo/remoting/transport/mina/MinaChannel.java
54.92% <0%> (+12.67%)
17% <0%> (+2%)
: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 79859aa...95893f6. Read the comment docs.


should not be able to shift two vertically adjacent panels

see related issue #340
this might be resolved by fixing #340 and then also not allowing a panel to swap into empty space directly under a "hovering" panel.
Note: I would not consider panels that has transformed from garbage and is waiting for the rest of its garbage transformation group to be done as "hovering"

this seems to be a rather simpler fix will look tomorow

cant recreate this at all anymore, maybe this was fixed by #352


Issue when display chinese character , it will keep repeat in the same line?

Issue when display chinese character , it will keep repeat in the same line?

Could you try #13 to see if it solves the issue for Chinese characters?


doc generation seems to be trimming the first character when concatenating lines.

generated doc is
Contains inferred type information about specific instance of ‘metric’ template. This is passed to nfrastructure backends during configuration-time through InfrastructureBackend.CreateSession.
Notice the spelling of nfrastructure
for proto that looks like:
/* Contains inferred type information about specific instance of 'checknothing' template. This is passed to
infrastructure backends during configuration-time through [InfrastructureBackend.CreateSession][TODO: Link to this fragment]. */
message Type {

}

Seems to be an issue only when using multi line comments.

I'll look into this, but the usual form for such comments in protos would be using // comments. Evidence suggests (looking at istio.io) that // comments work correctly and don't cause truncation.

Fixed by this commit.

@geeknoid thanks and you are right in protos // is the common form of comment.


Use a 3 minutes timeout for /package/describe

The default 5 seconds during cluster setup shown some failures in CI:
https://jenkins.mesosphere.com/service/jenkins/blue/organizations/jenkins/public-dcos-cluster-ops%2Fmesosphere-dcos-cli%2Fintegration-tests/detail/integration-tests/178/pipeline/48/

@mesosphere-ci run integration tests


feat: add disable trait

Adds a disable trait to disallow any user input on the admin interface
Proposed Changes

disable trait
should be used on all fields (TODO)

Great work @zangab! I didn't even know about the disabled attribute existed. Do you know where I can find documentation on the disabled and readonly settings?

Just tried this locally and seems to work fine. If you want to, please add a second pull request for the readonly attribute with ReadOnly trait.

@vinkla
don't know if this is officially documented somewhere, but it's in some stackoverflow threads and it was quite handy sometimes. the disable should work on all fields. sure, i will do another PR for the readonly. 🤓

It seems they aren't supported on all fields and they are working on updates for the next release:
https://twitter.com/wp_acf/status/1288358105080713217?s=20

Before we add this feature we need to figure out which fields actually supports the disabled attribute.
Maybe we could as @elliotcondon?

Hi @vinkla 👋
The "disabled" setting is not yet fully supported across all our Field Types, however, I can confirm that this is something I am hoping to address in an upcoming release - most likely 5.10.
The Field Types that currently support the "disabled" property are:

Checkbox
Date Picker
Email
Number
Radio Buttons
Range
Select
Text
Textarea
URL

Thanks for getting back @elliotcondon. Looking forward to read the 5.10 beta changelog 🙌
Then we'll add the supported fields for now and revisit this once 5.10 beta is realsed.

This has now been merged. Thanks @zangab and @elliotcondon!


Error when config doesn't exist, issue #118

I'd love to see this merged in, are you waiting on anything in particular to merge in and create a release?

@pjfreeze might be just requires thumbs up from me or you. :-) https://github.com/witoldsz/angular-http-auth/issues/118#issuecomment-155941049
Wanna test this? I'm on holidays.

@simison is right, I mean it would be great if you could confirm the new code fixes the issue.

@simison @witoldsz Looks good to me. I've included 2 links, let me know if you have any questions or issues with that test case
Not working with version angular-http-auth#1.2.2
http://codepen.io/anon/pen/zvQBom
Working with angular-http-auth#issue-118
http://codepen.io/anon/pen/epazgK
I borrowed the test case from a comment on the related issue simison linked to from #118:

rcollette commented on Jul 17
Just to show where this scenario can occur without any other interceptors defined.
I had a case where a configured URL was not specified resulting in a null url parameter being passed to $http.post. The result is an infinite loading bar.
See example here:
http://plnkr.co/edit/YhsWp7G4L1Gh93v1fsTJ?p=preview

Thanks @witoldsz & @pjfreeze


Unify logging in sesman

Follow-on pull request to #1633 which migrates all logging in sesman to use the LOG() and LOG_DEVEL() macros.

Hi @aquesnel
Thanks very much for this. I was half expecting it it show up soon!
I've just run a few textual modifications, and I'm pretty sure that the vast majority of this PR, with the exception of chansrv/smartcard_pcsc.c is just replacing log_message function calls with LOG macro invocations. Is that correct?
Assuming it is, we can focus the discussion on chansrv/smartcard_pcsc.c. Comments are:-

The majority of the LLOGLN(0, calls you've replaced (rather than the 10, variety) are generally reporting errors, and so should probably be replaced with LOG(LOG_LEVEL_ERROR,. Exceptions are the calls at lines 1703, 1747, 1751, 1755, 1842 and 1885 which look like they should be LOG_DEVEL.
There's a call to g_hexdump() at line 491 which is part of the error reporting / debugging mechanism. We could add similar functionality to the logging module, but given this won't be needed a lot, the simplest thing might be to comment out this line for now (as as been done elsewhere, I've just noticed).

@matt335672 ,
Yes you are correct, this commit was created by running a find and replace over the code. I created a shell script to handle the weirdness of the double parenthesis of some of the old macros. I can post this script or add it to the pull request if you think it's worth it.
For this change I assumed that the previous log levels were correct, and I'm happy to fix them up.
For the hex dump, I've already written a log hex dump function and macro, but I didn't include it in pull request since I hadn't seen a hex dump used in any of the files. I'll just include the function in this pull request.

I don't think we need the script - this particular file is easy enough to review manually in any case.
The hex dump was a surprise to me too - I've not seen it before but I can see where it would be useful.

Hi @matt335672 ,
I've updated the code to include the LOG_DEVEL_HEXDUMP function and I've fixed up the log levels for chansrv/smartcard_pcsc.c. Please let me know if you have more feedback.

Hi @aquesnel,
That's great - thanks.
A couple of general points:-

I like where you're going with this. think it makes sense to concentrate on your changes here and leave some other sesman changes (#1650 #1711) until we've got this one merged - they'll probably need tweaking to fit in with this one.
Predating your involvement in this project, there's a lot of private stuff in log.h which could be moved to log.c. I'd like to get this fixed at some stage, but I think it should be done as a separate PR later on. I'd be happy to work with you on getting that done. Does that sound reasonable to you?

Also, with reference to log.c, could I ask you to look at the astyle utility? It's mentioned in the XRDP [coding standard] (/neutrinolabs/xrdp/wiki/Coding-Style) and it's very useful at tidying up shared code. As part of this PR, it would be good to get log.c formatted with it. The way it could be used here is:-
define ARTISTIC_STYLE_PROJECT_OPTIONS=astyle_config.as
astyle log.c\*

The wildcard looks a bit odd, but is necessary with a single file. One hint I'll give is around line 729 where you've got dump_line_length =, you might need to add braces () around the expression or it will line everything up on the left.
I'll get on with a review of the code now. I might be a bit pressed for time over the next couple of days with family duties.

Hi @matt335672 ,
I've been trying to get astyle to format the log.c file nicely but I haven't figured out a non-hacky way to get the split dump_line_length = line to line up indented.
Diff pre- vs post- astyle formatting:
     dump_line_length = 4 + 3             /* = 4 offset + 3 space */
-                       + ((2 + 1) * 16)  /* + (2 hex + 1 space) per byte for 16 bytes */
-                       + 2 + 16 + 1;     /* + 2 space + 16 bytes as chars + 1 newline */
+    + ((2 + 1) * 16)  /* + (2 hex + 1 space) per byte for 16 bytes */
+    + 2 + 16 + 1;     /* + 2 space + 16 bytes as chars + 1 newline */

I've been able to trace this down to having the word "enum" in the return type of the function that is being formatted. When I replace the enum with a typedef or macro for that enum, then astyle formats the split line with the expected indentation. This looks like a bug in astyle mis-identifying the function as an enum or something. The only thing I have found in the is a reference in the astyle release notes for 2.05.1 for "Fix error in identifying an enum return type as an enumeration"
I've tried running with astyle version 2.06 and with 3.1. The 3.1 version does a better formatting job overall, but it still lines up the split dump_line_length line to the left.
My plan is to make a typedef for enum log_config then just use that everywhere. I'll also add a note to the Coding-Style wiki page about this issue.
Do you have a different suggestion for how to workaround this astyle issue?

Hi @aquesnel
Yes I do. I'd mentioned it above but re-reading my comment I could have been clearer. If you add parentheses around the whole expression astyle will behave differently.
Try this (adjust to taste):-
    dump_line_length = (4 + 3 +           /* = 4 offset + 3 space */
                        ((2 + 1) * 16) + /* + (2 hex + 1 space) per byte for 16 bytes */
                        2 + 16 + 1);     /* + 2 space + 16 bytes as chars + 1 newline */

I'm using astyle 3.1.
Try that and let me know how you get on.

Hi @matt335672 ,
I had originally tried adding braces as you suggested around the dump_line_length line, but apparently astyle 2.06 has a bug that causes it to not indent the braced split line when a function returns an enum. This bug is fixed in astyle 3.1 so I'll add a note to the Coding-Style wiki page to say that astyle 3.1 or above should be used.
I've also run astyle on all of the .c files in sesman/* since most of the formatting errors are caused by the logging changes. I skipped the following files because:

sesman/chansrv/chansrv_fuse.c: the non-logging formatting changes were non-trivial and you have recently been working on fuse, so I didn't want to introduce a bunch of changes.
sesman/chansrv/pcsc/wrapper/winscard.c: no logging changes were made to this file
sesman/chansrv/pcsc/xrdp_pcsc.c: no logging changes were made to this file

Please let me know if you have any concerns with the additional code formatting.

Hi @aquesnel
Thanks for that.
I've approved the changes I've requested, and I've run your hexdump code through a test rig. It all looks fine to me.
One small change you might like to consider is adding a call for the g_malloc() failing. We had a security advisory CVE-2020-4044 last year which exposed a number of ways in which sesman could be caused to crash. This can then be exploited to harvest passwords.
As it stands, an attacker managing to get a large value into the log_hexdump_with_location() could cause a memory overflow and a crash. That's not a major consideration at the moment as your code is currently for debugging only. However, since it's clearly a useful function, I can see it over time being used for non-debug logging purposes, and then we might have a problem.
Personally I'm happy that you've run the rest of the sesman code through astyle. It's a good time to do it as all the logging macros are changing too, so I can't see it causing any more upheaval than this PR would anyway. It makes checking code before commit a lot easier in the future,
@metalefty - are you happy with asyle being run on sesman in this way?

Hi @metalefty ,
Do you have any concerns or feedback for this pull request that you think should be addressed before this can be merged?

Let's merge.


provide url and icon for desktop team contacts

[x] to Anjillan Firdous and
[x] Bradley MacDonald

sent information


Document ability to drag-and-drop anywhere to upload a file

We should document the functionality introduced in #14579 / https://github.com/zulip/zulip/pull/26041.
I would probably replace the "Via Markdown" tab with "Via drag-and-drop", and modify the instructions to explain that you can drag anywhere, whether or not the compose box is open.

@zulipbot claim

@zulipbot claim

@zulipbot claim


Change image format from png to svg for travis-ci badge

svg appears to be the standard badge format today, but this repo is still using png, which was the preferred option at the time the badge was added in d2c432e16c5f347488e24875ce61db36cf3b18e0.
svg will look sharper on high-resolution displays.

cc @dlwh
I totally understand if PRs are lingering simply due to not having much time to devote to the project.

yeah, i'm sorry, it's been a crazy year


Update README.md

Fixed broken links to ossec documentation
Description
URLs to official ossec documentation needed to be updated in order to work
Issues Resolved
Fixes #84 - All documentation either in the readme or in the official documentation looks to be enough to configure agents.
Check List

[ ] All tests pass. See https://github.com/chef-brigade/sysctl/blob/master/TESTING.md
[ ] New functionality includes testing.
[ ] New functionality has been documented in the README if applicable

Thanks @hafrei


Adds test for script double eval bug.

Adding a test to show how script tags are evaluated twice during partial replacements.

Thanks for the test. I'll look into this in the coming weeks.

Fixed by 27318d2f0ab37e0abfee405e1bb85dde7d90f226. Thanks again!


Print Username Arn as Cause event

Support riboseinc/aws-codecommit-trigger-plugin/issues/46

Thank you @phuonghuynh , looks great!


Call MixPanel with isDemoSite = True instead of isDemoSite = undefined

Right now the isDemoSite parameter we pass to Mixpanel can be one of two values: false and undefined.

I assume undefined means "true", but obv. it should actually be true.

undefined === Staging site I believe

Okay, so DEMO_SITE should be false for staging. I can confirm that the config var DEMO_SITE is not set on staging right now, so it makes sense that would be undefined.
I assume that false means production right now. Maybe let's report staging, production, demo instead of true or false?
I'm wondering if undefined really is only staging. Mixpanel reports 44 unique undefineds in the past few days -- where would we be getting all those unique visitors to staging?

I assume that false means production right now. Maybe let's report staging, production, demo instead of true or false?

👍  👍  👍

I'm wondering if undefined really is only staging. Mixpanel reports 44 unique undefineds in the past few days -- where would we be getting all those unique visitors to staging?

Hm. I wonder how Mixpanel defines a unique visitor. I also wonder if some of the undefineds could possibly coming from folks running Student Insights on local machines?

Closed by https://github.com/studentinsights/studentinsights/commit/fd39d8319028f0990fca61dd86d704c96ab13423.


Problems with connect to database using the same configuration that can connect from other PC

Description
I´m using the same configuration as my college. Appears the error:
Communications link failure
The last packet sent successfully to the server was 0 milliseconds ago. The driver has not received any packets from the server.
Connect timed out
Connect timed out

Have you any idea how to check and solve this?
Regards
DBeaver Version
<IP_ADDRESS>403241413
Operating System
Windows 11
Database and driver
MySQL
Steps to reproduce

I try connect to db
I push double click
the error appears after 20-30 seconds

Additional context
No response

Please, provide the log files. You can find instructions here https://dbeaver.com/docs/wiki/Log-files/.

It's been a while since no update here. If the issue is still actual please let me know and provide additional information.


Allowing duplicate tags

Looking for a way to have a user add multiple tags that could have the same text. Read about the "key-property" and don't really understand how you are suppose to use it to get the desired effect. Anyone use this property successfully? Thanks in advance.

I use index as id and works for me below
<tags-input ng-model="tags" key-property="id" display-property="text" on-tag-added="tagAdded($tag)"></tags-input>

$scope.tags=[];
$scope.tagAdded = function($tag) {
      $scope.tags = $scope.tags.map(function(tag,$index){
             return {
                  text: tag.text,
                  id: $index
             }
      })
}

But when paste duplicate tags still have some error like Error: [ngRepeat:dupes]

This issue was selected to be spring-cleaned by a script. More information can be found here.
If anyone thinks this was a mistake and this issue should be reopened, please leave a message below explaining why. Before doing so, please consider reading the CONTRIBUTING file.

If array contains duplicate entries(same text but different ids) drop down does not appears. How to allow such duplicate entries? I have tried with "key-property='id'" suggested by the above solution but does not solve my problem. I have gone through similar issues with keyword duplicate but could not help me out.

array(
{
id:0,
name:xyz
<EMAIL_ADDRESS>}
{
id:1,
name:xyz
<EMAIL_ADDRESS>}
)


Add 'untriaged' label workflow

Description
Add 'untriaged' label workflow to identify new issues created which would help to 'triage' and followup with the issues.
Issues Resolved
List any issues this PR will resolve, e.g. Closes [...].
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

@idanl21 @swoehrl-mw @dbason merging this PR as it would be easy to track new issue with untriaged label.


Update FreeNAS dashboard

I made some tweaks to the FreeNAS dashboard that makes it a little easier for others to use and tweaked some of the display to make it look a little nicer.

Stack network values
Only show CPU Temp in CPU Temp
Add HDD Temp
Allow user to select the mount point shown in chart

Remove the alert since the disk usage chart is now using a variable

Fix some chart labeling

Thanks for your contribution!
I believe the df_resource variable is missing in your pr so the disk graphs is broken

Implemented in commit 24a0fa26878ac024afbd81fe3cd7df4ea1601354


Set stopped flag to allow PollingQueueConsumer object reuse

ClusterRpcProxy objects can only be used once because the PollingQueueConsumer object sets it's stopped flag to False in the init() method and to True in the unregister_provider() method.  If the proxy is used a second time, the stopped flag is not reset to False when register_provider() is called, so the ConsumerEvent wait() method thinks the service has stopped, even though the message was sent and a response would have been received.
This PR adds a test for this case and sets stopped to True in the PollingQueueConsumer init() method, since register_provider() has not been called yet.  The stopped flag is set to False in register_provider() after the consumer has been set up.
This allows a single proxy instance to be used for multiple service invocations.

@bobh66 thanks for the PR!
This looks good to me, although I can't remember why we protect against a stopped PollingQueueConsumer. @davidszotten do you recall?

no, i was hoping you would :p. is the issue that stopping and starting again ends up re-doing all the work anyway and so we wanted to make this explicit?

@davidszotten perhaps that was the reasoning. Perhaps the consumer setup logic was different back then too, and restarting was not an option?
I don't think it matters really, unless you can think of a scenario where this won't work? The test @bobh66 has added suggests it does!

i guess my only worry would be if we make it seem like we're re-using machinery but are in fact re-setting everything up (with performance implications)

You have to leave and reenter the context manager though, which usually implies some kind of work.

fair

@bobh66 as discussed i'm happy to accept this. thanks again for the report and the pr!
could you please add a small note to the changelog, and (if you want) add yourself to the contributors list?

@davidszotten will do - thanks!

@davidszotten - should I add an entry to the CHANGESfile with no associated version, and assume the version will be added when it is released?  Thanks

yep, that's our normal process. just mark it 'unreleased'. thanks


RubricController sometimes breaks when attempting to parse a grade.

Lab4 (lab1.ipynb) breaks rubric controller halfway.

Currently investigating

Identified as a malformed rubric.
Offending line:
rubric={accuracy=3,reasoning:3}

which should be:
rubric={accuracy:3,reasoning:3}

Current behavior is to stop parsing the rubric and halt; possibility to include an error message when detecting this issue.


OSError: Can't load tokenizer for 'baffo32/decapoda-research-llama-7B-hf'.

bash scripts/llama_prune.sh
[START] - Start Pruning Model
Traceback (most recent call last):
File "/home/iotsc01/xinpengq/LLM-Pruner-main/hf_prune.py", line 314, in
main(args)
File "/home/iotsc01/xinpengq/LLM-Pruner-main/hf_prune.py", line 39, in main
tokenizer = LlamaTokenizer.from_pretrained(args.base_model)
File "/home/iotsc01/anaconda3/envs/xinpengq_env/lib/python3.10/site-packages/transformers/tokenization_utils_base.py", line 2012, in from_pretrained
raise EnvironmentError(
OSError: Can't load tokenizer for 'baffo32/decapoda-research-llama-7B-hf'. If you were trying to load it from 'https://huggingface.co/models', make sure you don't have a local directory with the same name. Otherwise, make sure 'baffo32/decapoda-research-llama-7B-hf' is the correct path to a directory containing all relevant files for a LlamaTokenizer tokenizer.
[FINISH] - Finish Pruning Model
[START] - Start Tuning
Traceback (most recent call last):
File "/home/iotsc01/xinpengq/LLM-Pruner-main/post_training.py", line 262, in
main(args)
File "/home/iotsc01/xinpengq/LLM-Pruner-main/post_training.py", line 33, in main
pruned_dict = torch.load(args.prune_model, map_location='cpu')
File "/home/iotsc01/anaconda3/envs/xinpengq_env/lib/python3.10/site-packages/torch/serialization.py", line 986, in load
with _open_file_like(f, 'rb') as opened_file:
File "/home/iotsc01/anaconda3/envs/xinpengq_env/lib/python3.10/site-packages/torch/serialization.py", line 435, in _open_file_like
return _open_file(name_or_buffer, mode)
File "/home/iotsc01/anaconda3/envs/xinpengq_env/lib/python3.10/site-packages/torch/serialization.py", line 416, in init
super().init(open(name, mode))
FileNotFoundError: [Errno 2] No such file or directory: 'prune_log/llama_prune/pytorch_model.bin'
[FINISH] - Finish Prune and Post-Training.
[INFO] - The pruned model is at {prune_log/llama_prune/pytorch_model.bin}, and the recovery weight is at {tune_log/llama_0.2}/
You can use the command:
python generate.py --model_type tune_prune_LLM --ckpt prune_log/llama_prune/pytorch_model.bin --lora_ckpt tune_log/llama_0.2
to use the pruned model

@horseee 您知道这是什么原因造成的吗，我在llama_pune.sh中添加上了--base_model baffo32/decapoda-research-llama-7B-hf这个参数，但是还是报错

一样的问题，请问您解决了吗

原来你也是uestcer


Add dcc meter flag to dashboard meter

Could update the spec/factories/meter_factory.rb file as well so the attribute can be used within tests?
And maybe a simple test for Dashboard::Meter even if its just a stub so we can slowly improve coverage?

:+1:


Move shared process to a browser window

Moving the shared process to a browser window will enable it to use Chrome's network stack. This will affect mostly the extension viewlet.
Related to #18103

Wow, that took a lot of commits. Thanks for the help, @bpasero !

@joaomoreno is the merge already in the latest insider build?
If yes,  i can't confirm working proxy yet.


How does the overtraining detector work?

As of recently I've noticed you guys added an overtraining detector thingy. I'd like to know how it works and what graphs and values it takes into account.

You can check this commit for more info https://github.com/IAHispano/Applio/commit/5593d183ed44d24639667ecb385fe14088b71693
In each epoch it stores the loss gen all of that point and from there it makes the comparisons.

So that's how it finds the global minimum? If so that really isnt all that accurate, its better for the model to look at loss/g/total rather than loss/gen/all. Looking at loss/g/total gives u a much better representation of how the generator is doing with the training data whilst also taking into account all the other factors such as mel, kl and fm.

I'll try testing it out


Refactor tests

They're a bit of a mess - it's actually my fault for splitting 'v0.1.0' and 'v0.2.0' apis into sections way back - so I'll clean it up when I can - just wanted to make note of it here.  Now that there's coverage checking, this can finally be done without missing anything.

The structure of the tests - the describes and its - should better follow the structure of the module with a describe for each exports-ed item;
All the functionality should be explicitly tested for, as some is currently just 'coincidentally' tested;
Most of the setup code can probably be refactored and better reused;

this would be awesome. I looked at the tests for the first time today. Found them quite hard to follow

@floatingLomas I'm interested in refactoring the tests and think I could probably get it done well—except for the separation between the v0.1.x and v0.2.x API tests. I've looked over them and don't understand why they're separated like that. If both APIs/behaviors are actively supported in the current version, then it seems like the tests should be able to all be lumped together. Does that make sense? I'm sure I'm probably missing some detail here.

also @rosston please add yourself as contributor in the Readme

Cool.  They're separated like that because I wanted to make sure the new code still passed all the previous tests when I bumped versions - and because I was really new at all this back then.  It was done way back in 5e2f0f6beffdbf3196233370aca7bf840e415906.
So it's an ancient artifact, one that you are free to kill in your pursuit of a less hideous test suite - at least as far as I'm concerned.

Wow, that's a long time ago! And cool, that makes things easier. I think your original 3 points at the top of this thread are a rough outline of what I plan to do.
For cleaning up the setup code, I plan to move most of the existing helper functions towards promises, as that should make it easier to chain assertions off of the async code without needing to always use a before block. Which should in turn make it easier to consolidate/eliminate some of the describe blocks.


Fix uses of default(omit) with string concatenation

Port of #3584 to master.
When {{omit}} is concatenated with another string, it expands to something
like __omit_place_holder__63eea0d96dd6ed867b95405e11d87dddf61f448d.
However, in these use-cases we need an empty string.
Regression introduced in d53f55e807e.
Signed-off-by: Leah Neukirchen<EMAIL_ADDRESS>

@chneukirchen Thanks!


It appears I have to be on another branch other than master?

I wasn't able to push because I was on the master branch

yes, because the base branch which the PR will be opened will be master on bitcointranscripts

Tried changing the y2btc branch named test with the same result.

Then realized you must mean that I need to cd into the cloned bitcointranscripts dir and change that branch which I did.
I got a little farther but it still failed:


🚀 Feature: add --no-install to backstage-cli version:bump

🔖 Feature description
Added the parameter --no-install to backstage-cli version:bump command
🎤 Context
I have a CI pipeline that semi updates backstage that uses the command backstage-cli version:bump.
However it can take a long time to install the packages. I really don't need these packages to installed as apart of the pipeline it would be nice to have an option to skip the install of these packages. As just need the package.json files and yarn.lock file to be updated. So can I commit these files to git,
This option would speed up the pipeline by several minutes.
✌ Possible Implementation
on line https://github.com/backstage/backstage/blob/master/packages/cli/src/commands/versions/bump.ts#L258 we could create an if statement something like the below
if (!opts.noInstall) {
   await runYarnInstall();
}

👀 Have you spent some time to check if this feature request has been raised before?

[X] I checked and didn't find similar issue

🏢 Have you read the Code of Conduct?

[X] I have read the Code of Conduct

Are you willing to submit PR?
No, but I'm happy to collaborate on a PR with someone else

We're also using the command in the CI, to check if versions are correct. In other words, the pipeline fails if version:bump <current version> would introduce any changes. It would definitely be helpful if we could avoid the yarn install :)

Thanks for the suggestion, I think it's a good one. We should make sure that it is --skip-install to align with other commands that offer the same functionality. PR's welcome! 🙏


Refactor actions to allow for async start and completion.

These changes refactor the collection rule actions to have separate start and completion notions as well as use the factory pattern for creating the actions. This will allow (in a future change) to asynchronously start actions in the action list and synchronously wait for actions to complete when necessary.

I will merge and refactor https://github.com/dotnet/dotnet-monitor/pull/883 once it is completed.


Caching

In order to save CPU time and serve requests faster, it could be useful to implement a simple caching scheme. We could create a file for every request, containing the response, and serve that if the request reoccurs. We could furthermore touch the file when it is used, and keep only the newest n files in the cache. To generate the filenames, we could use base64 after jsonEncode. It would be best to do this in the Clean backend, because then also requests from elsewhere talking to Clean directly are cached.
Ideas are welcome.

this can also be done with a caching http proxy like squid. This has to be on webserver level

This can even be used by default in the docker images:
https://hub.docker.com/r/sameersbn/squid/

While that would definitely be the easiest way, it would mess up both long term and live statistics, since cached requests would be no longer passed on to the PHP wrapper or the Cloogle server.