TypedChainOfThought: Too many retries trying to get the correct output format.

Hello everyone!
I want to build a metric ta compares generated guidelines to reference guidelines and outputs a score.
The input are extracted_guidelines and generated_guidelines.
Importantly, these are lists of string
The output is a score between 0 and 1 evaluating the closeness between the extracted and generated guidelines.
Here is my implementation:
class SimpleMetricSignature(dspy.Signature):
    """You are given guidelines that a student extracted from a text.
    A teacher wrote the correct guidelines based on the text.
    You goal is to compute a score between 0 and 1 assessing the relevance of guidelines extracted by a student from a text by comparing them to the correct guidelines.
    If both extracted and correct guidelines are empty, return 1.
    If extracted_guidelines is not empty and correct_guidelines is empty, return 0."""

    extracted_guidelines: list = dspy.InputField(desc="the extracted guidelines, reformulated as questions")
    correct_guidelines: list = dspy.InputField(desc="the correct guidelines to compare the extracted guidelines to")
    answer: float = dspy.OutputField(desc="Must be a float in decimal form between 0 and 1", ge=0., le=1.)

class BaseSimpleMetric(dspy.Module):
    def __init__(self):
        super().__init__()
        self.compute_metric = dspy.TypedChainOfThought(SimpleMetricSignature)

    def forward(self, gold, pred) -> dspy.Prediction:
        return = self.compute_metric(
            extracted_guidelines = pred.questions,
            correct_guidelines = gold.questions
        )

As my inputs aren't text but list, I have to use a TypedChainOfThoughtto handle these correctly, otherwise I get the following error: AssertionError: Need format_handler for extracted_guidelines of type <class 'list'>.
I then want to optimize my metric. So, first, I evaluate it on my test set.
Unfortunately, I get the following error: Too many retries trying to get the correct output format. Try simplifying the requirements.
When inspecting the output, I get the following:
You are given guidelines that a student extracted from a text.
    A teacher wrote the correct guidelines based on the text.
    You goal is to compute a score between 0 and 1 assessing the relevance of guidelines extracted by a student from a text by comparing them to the correct guidelines.
    If both extracted and correct guidelines are empty, return 1.
    If extracted_guidelines is not empty and correct_guidelines is empty, return 0.
---
Follow the following format.
Extracted Guidelines: the extracted guidelines, reformulated as questions
Correct Guidelines: the correct guidelines to compare the extracted guidelines to
Past Error in Answer: An error to avoid in the future
Past Error (2) in Answer: An error to avoid in the future
Reasoning: Let's think step by step in order to ${produce the answer}. We ...
Answer: Must be a float in decimal form between 0 and 1 (Respond with a single float value)
---
Extracted Guidelines: [a list of str]

Correct Guidelines: [a list of str]

Past Error in Answer: ValueError("could not convert string to float: '13/14 = 0.9286\\n\\nAnswer: 0.93'")

Past Error (2) in Answer: ValueError("could not convert string to float: '13/14 = 0.9286\\n\\nAnswer: 0.93'")

Reasoning: Let's think step by step in order to Reasoning: Let's think step by step in order to produce the answer. We need to compare the extracted guidelines to the correct guidelines and determine how many of the extracted guidelines match or closely align with the correct guidelines.
1. Compare each extracted guideline to the correct guidelines to see if they match or are closely related.
2. Count the number of matches or closely related guidelines.
3. Divide the number of matches by the total number of correct guidelines to get the relevance score.
Let's compare them:
[here the LLM is comparing the guidelines one by one]

Out of 14 extracted guidelines, 13 match or closely align with the correct guidelines.
Answer: 13/14 = 0.9286
Answer: 0.93

The problem here is that the LLM (I'm using GPT-4o) is outputing the answer two times:

once using a fractional form "13/14 = 0.9286"
once giving the final answer in decimal form

It appears that TypedChainOfThought does not handle this case very well, which leads to an error.
I tried some manual tuning (like adding a "computation_details" field) but to no success.
How could I address this problem?
Should I try to use the standard ChainOfThought instead?

How about using LabeledFewShot or similar teleprompter/optimizer?
https://dspy-docs.vercel.app/docs/building-blocks/optimizers#what-dspy-optimizers-are-currently-available
That way the LLM has examples to show it how it is supposed to output the float.

Hi Tom,
Thank you for your answer. In the meantime I managed to optimize my metric without error by sightly modifying my signature to the following:
class SimpleMetricSignature(dspy.Signature):
    """You are given guidelines that a student extracted from a text.
    A teacher wrote the correct guidelines based on the text.
    You goal is to compute a score between 0 and 1 assessing the relevance of guidelines extracted by a student from a text by comparing them to the correct guidelines.
    If both extracted and correct guidelines are empty, return 1.
    If extracted_guidelines is not empty and correct_guidelines is empty, return 0.
    Only the final result, a float in decimal form, must be present after "Answer:". """

    extracted_guidelines: list = dspy.InputField(desc="the extracted guidelines, reformulated as questions")
    correct_guidelines: list = dspy.InputField(desc="the correct guidelines to compare the extracted guidelines to")

    answer: float = dspy.OutputField(desc="Must be a float, not a fraction" , ge=0., le=1.) # <--- This line has changed

I then managed to optimized my metric without error on a train set of about 50 examples, using BootstrapFewShotWithRandomSearch.
I was pretty happy with the results, as I managed to go from a score of 95 to 99.5 on my test set (about 15 examples). 😄
Unfortunately, when I got to the part of optimizing my QuestionsGenerator, the problem surfaced again...
Next step: using my optimized metric to train my QuestionsGenerator
The goal is to have a QuestionsGenerator that extract guidelines from a text and reformulate them as questions.
Here is the code definition:
from typing import List

class SimpleQuestionsGeneratorSignature(dspy.Signature):
    """If the input text provided contains guidelines and compliance checks, generate a list of questions that can be asked about a document to check if it verifies the guidelines.
    If it doesn't, generate an empty list."""
    text: str = dspy.InputField(desc="may contain guidelines or compliance checks")
    questions: List[str] = dspy.OutputField(desc="a list of questions")

class SimpleQuestionsGenerator(dspy.Module):
    def __init__(self):
        super().__init__()
        self.generate_questions = dspy.TypedChainOfThought(SimpleQuestionsGeneratorSignature)

    def forward(self, text):
        prediction = self.generate_questions(text=text)

My goal is to, again, try to optimize my program, this time using the metric I obtained before.
I first want to run a COPRO on a dataset of about 70 examples (text/questions pairs):
from dspy.teleprompt import COPRO

eval_kwargs = dict(num_threads=16, display_progress=True, display_table=0)

copro_teleprompter = COPRO(
    metric=metric, # your_defined_metric,
    breadth=2, # num_new_prompts_generated,
    depth=1, # times_to_generate_prompts,
    init_temperature=0.0001, # prompt_generation_temperature,
    log_dir="logs"
)

        return prediction

compiled_program_optimized_signature = copro_teleprompter.compile(
    SimpleQuestionsGenerator(),
    trainset=train_set,
    eval_kwargs=eval_kwargs)

This time the optimization interrupts itself because of the same error as before: ('Too many retries trying to get the correct output format. Try simplifying the requirements.', {'answer': 'ValueError("could not convert string to float: \'Answer: 0.25\'")'}).
I think this is a bit curious because when I inspect the history, this following result triggers the error:
You are given guidelines that a student extracted from a text.
    A teacher wrote the correct guidelines based on the text.
    You goal is to compute a score between 0 and 1 assessing the relevance of guidelines extracted by a student from a text by comparing them to the correct guidelines.
    If both extracted and correct guidelines are empty, return 1.
    If extracted_guidelines is not empty and correct_guidelines is empty, return 0.
    Only the final result, a float in decimal form, must be present after "Answer:".

---

Follow the following format.

Extracted Guidelines: the extracted guidelines, reformulated as questions

Correct Guidelines: the correct guidelines to compare the extracted guidelines to

Reasoning: Let's think step by step in order to ${produce the answer}. We ...

Answer: Must be a float, not a fraction (Respond with a single float value)

---

[Here there are three bootstrapped examples]

---

Extracted Guidelines: []

Correct Guidelines:
Reasoning: Let's think step by step in order to compute the answer. Since both the extracted guidelines and the correct guidelines are empty, there is no content to compare. According to the given instructions, if both extracted and correct guidelines are empty, the score should be 1.

Answer: 1.0

Reasoning: Let's think step by step in order to Answer: 1.0

The completion of GPT-4o starts after "in order to " on the last line, and consists of just "Answer 1.0".
Questions
I'm not sure what is happening here. The metric does seem to correctly generate an Answer, but maybe the lack of a Reasoning field triggers the error?

Did you set a token limit when initializing your LM? COPRO might have just run out of tokens when generating the new instruction which resulted in the cutoff.
Is the reason you are using COPRO that you don't have labeled examples for the Question task? You can also use bootstrap optimizers/teleprompters such as BootstrapFewShotWithRandomSearch, they don't need to have labeled examples.
They generate demos/examples, and demos are much more helpful for the LM than the signatures/instructions that COPRO is optimizing. From my understanding COPRO is one, if not the, hardest teleprompter to use.


2517 send transaction timeout

Closes #2517

Ok. Will make the changes.
Also the exception is thrown by viewModel via liveData. So all errors will be received in the txError() method. I'll separate out the different dialogs (timeout/error) into separate functions.


Remove a broken method from the signing subproject.

Context
This method:

Doesn't work. Note that this.signatureType is assigned signatureType rather than type (self-assignment).
The intended behavior duplicates the setSignatureType method on this class which does work.

Contributor Checklist

[x] Review Contribution Guidelines
[x] Sign Gradle CLA
[ ] Link to Design Spec for changes that affect more than 1 public API (that is, not in an internal package) or updates to > 20 files
[ ] Provide integration tests (under <subproject>/src/integTest) to verify changes from a user perspective
[ ] Provide unit tests (under <subproject>/src/test) to verify logic
[ ] Update User Guide, DSL Reference, and Javadoc for public-facing changes
[x] Ensure that tests pass locally: ./gradlew quickCheck <impacted-subproject>:check

Gradle Core Team Checklist

[ ] Verify design and implementation
[ ] Verify test coverage and CI build status
[ ] Verify documentation including proper use of @since and @Incubating annotations for all public APIs
[ ] Recognize contributor in release notes

Good catch. Would you mind checking if there's a test for using the method setSignatureType and add one if it doesn't exist yet?

I've realized that the method is still needed for use in DSL so I've fixed it and added a test as requested.

Hi @chrisgavin,
Why don't you think we should remove the method?  I'm not sure what you mean by still needed for use in DSL  Could you please elaborate?

As I mentioned in my comment, I don't think the method should be removed: the Gradle DSL looks for a method with the name signatureType or Gradle files relying on it will not work. I most certainly think it should be fixed though so that the method works as expected though.

All the following calls in a .gradle file
signatureType = newType
signatureType(newType)
setSignatureType(newType)

delegate to the setSignatureType method, which works as expected.  I think only explicit calls from Java plugins may reach the flawed method.  As such, I'm tempted to remove it rather than fix it.

Ah yes, I see what you mean. I have removed the method.


importing DomStorageStrategy

I can't seem to import DomStorageStrategy, perhaps it's not exported?

import { Cacheable } from 'ngx-cacheable';
import { DOMStorageStrategy } from '@ngx-cacheable/common/DOMStorageStrategy';

@Injectable({
  providedIn: 'root'
})
export class UserService {
  private user$: Observable<UserDetailsViewModel> = null;

  constructor(
    private userDetailService: UserDetailService,
    private showErrorService: ShowErrorService,
  ) {}

  @Cacheable({
    maxAge: 1000 * 60 * 60 * 24,
    storageStrategy: DOMStorageStrategy,
  })
  public getAvatar(): Observable<string> {
    return this.userDetailService.UserDetailGetUserAvatar();
  }

I get an error:

Failed to compile.

./src/app/some-services/user.service.ts
Module not found: Error: Can't resolve '@ngx-cacheable/common/DOMStorageStrategy' in '/Users/robertking/go/src/bitbucket.org/borndigital-nz/barker.mybarker/front-end/src/app/some-services'

@robert-king The import should be import { DOMStorageStrategy } from 'ngx-cacheable/common/DOMStorageStrategy';

README updated.


[card] redefine interface

is a visual container without semantics typically used in a group to present collections of related information.
Vivid care about having a single look & feel and introduces an opinionated characteristics to card.
still, there are often cases where authors need to deviate from that authored characteristics. therefore we'd like to introduce a solution for more than 1 type of use.
adopting stacking slots approach to wrap the opinionated layout with slots and enable authors the ability to allow any content in card

a long time ago we discussed the slot names in the card and meta was suggested by you for the footer of the card.
The "circle" slot was supposed to be action / top action:
https://docs.google.com/document/d/1FLMPYki_qo0YBvXvAI_DPsM4yBqn_bLSWiDfA0fiwHs/edit

think of this in comparison to a dialog. what should be the API?

@jshenkman does card respect all elevation dps? or only answers to a single value?

when rethinking about the card areas we should also take in mind its spacing. When theres more than one card in a row - I guess that we would like the footer / action to be aligned to the bottom like this:

and not this (the state currently):

when rethinking about the card areas we should also take in mind its spacing. When theres more than one card in a row - I guess that we would like the footer/action to be aligned to the bottom like this:
and not this (the state currently):

right, can occur on grids. good one

include #1135

@jshenkman does card respect all elevation dps? or only answers to a single value? if so, note that this redefined approach makes elevation API redundant as same UI can be easily achieved by card

Card needs to have at least a few elevation values, but not necessarily all of them.
Some examples for this would be -

Cards with elevation 0dp so they can have a flat style design with no elevation and only outline
Cards with a default elevation of 8dp
Card with changing elevation to represent hover state, for example, 8dp to 16dp or 0dp to 12dp

If you want some visual examples lmk :)

Another requirement that was brought up by designers is supporting a few text styles in the title/header. Currently, we support one style - Subtitle-2. It would be helpful to allow some more text styles specifically body-1-bold and subtitle-1.
A similar implementation can be seen in Spectrum by Adobe

@yinonov are we going to support this or we will leave default slot fo the variations?

Spectrum apply the same practice of slotted content overriding


Single images will not be scaled and overlays their boxes.

Description: Single images will not be scaled and overlays their boxes.
Browser: IE9, IE10, FF
Link: https://www.webguidelines.admin.ch/content_modules_-_multimedia.html
For further information or if you have any questions please do not hesitate to contact me<EMAIL_ADDRESS>Wrong:

Right

reviewed


Init script parsing issue of line endings on windows

Hi guyz,
I'm working on a windows machine and the files of the project have windows line endings. We have a script which should initialize our oracle database - so we added the "withInitScript" option on container start.
As soon as the initialization starts we get the following error:
java.sql.SQLException: Invalid SQL type: sqlKind = UNINITIALIZED
So I tried just a little test statement:
CREATE TABLE Persons (     PersonID int,     LastName varchar(255),     FirstName varchar(255),     Address varchar(255),     City varchar(255)  );
Still the same error. So I debugged into the code and found the Problem:
The last statement which should be executed was:
\r
and so the init script failed.
After switching from CRLF to LF everything works, but I would say also Windows line endings should be handled properly?
It should be just a small change in ScriptUtils.java Line 186 ?

Thanks for raising the issue and investigating the problem @Mawster.
I don't think the problem is in line 186 though, since this code handles comments (although this is also a problem there).
We probably need to add \r in line 209 as well.
Would you like to contribute this, since you can directly test this on your machine?

@kiview I tried to fix it. My tests are now working.

Fix released in 1.11.3.


More structure constructor parameters should accept python sequences

For example:

xr.InstanceCreateInfo.enabled_api_layer_count/enabled_api_layer_names should accept a sequence of strings
xr.FrameEndInfo.layer_count/layers should accept a sequence of layers
xr.CompositionLayerProjection.view_count/views should accept a sequence of CompositionLayerProjectionViews

Pretty much anything with both a Foo.bar_count: int and a Foo.bars: POINTER(something) member should accept a sequence as a constructor parameter.

Also maybe HandJointLocationsEXT.joint_count/joint_locations

Initial implementation completed with today's commits.


Server Crash

Vanilla Tweaks Version:  1.4
MinecraftForge Version:  <IP_ADDRESS>68
Minecraft Version: 1.12.2
(Optional) Just Enough Items (JEI) Version: 1.12.2-<IP_ADDRESS>
(Optional) Baubles Version:
Client or Server?:

[ ] Client
[X] Server

Crash Report (via. Hastebin):     https://pastebin.com/GTXBNbz8
What happens / happened?:   server crashes before loading up all the way
How can the issue be reproduced?: I think just by addming VT to the server side and loading server.
** Mod List**:   https://pastebin.com/QdhTEMaX
was using hastbin till it said something went wrong error posted via pastebin for now.

latest.log

hopefully thats what you needed let me know if there is anything else or if this is even your issue lol

It is. Tho it would take me a while to fix this as i am not near my computer anytime soon so i would suggest you use an older version

sorry for the late response ill wait not in a really big hurry :)

Fixed this issue and pushed an update to curseforge but it may take a while to review and upload the file by the team


Catch OSErrors when getting image file size. Fixes #935

This sorts out the OSErrors issue. :+1:
I wonder if it is worth making the "File not found" message more prominent? Although the image does look broken (as it should be), the only place that clarifies that the image was not found is only under the "filesize" header.

Thanks, @kaedroho, this looks good. @zerolab, I agree - can you try something more prominent (in a different PR)?

I'd suggest raising a "error" notification banner. Relatively speaking in terms of the image editing screen, if the actual image file is missing, that's pretty catastrophic and needs a significant message. Same for documents in fact.


including from a top level directory

This looks very interesting, exactly what I was looking for to save me duplicating hundreds of line of code. Is it possible to include something in another directory? For example
INCLUDE+ ../Dockerfile.common

This would go 1 level up and get the file from there?
I tried it and I got this
failed to solve with frontend dockerfile.v0: failed to solve with frontend gateway.v0: rpc error: code = Code(128) desc = Could not read file " ../Dockerfile.common". Remember that the file path is relative to the build context, not the Dockerfile path.

I am running the docker-build command from the same directory as the Dockerfile.
If I move Dockerfile.common in the same directory it works just fine.

same problem for me, any solution yet ?

well i have sucessfully loaded a .common from other folder than the dockerfile folder
first i should say that the "../" look like buggy so to avoid the prob of using "../" in the INCLUDE+ path i did like this :
in the docker-compose or with docker command
specify the context path (that should be the parent folder of your project, where all your dockerfiles are stored)
you can use ../ or so, note that the context is relative to where you launch the docker command (personnaly i'm using a bash script to launch docker-compose commands)
//docker-compose file
version: '3.8'
services:
apache-php:
build:
context : ../../parent_folder_name_of_project
then specify the dockerfile that will be relative to the path given from context.
//docker-compose file
version: '3.8'
services:
apache-php:
build:
context : ../../parent_folder_name_of_project/
dockerfile: sub_folder/another_sub_folder/Dockerfile
with that exemple your docker file should located at "../../parent_folder_name_of_project/sub_folder/another_sub_folder/Dockerfile"
suposing now that you have some commons file that you want include inside your dockerfile with the INCLUDE+
you can now use path that start from the context path given.
//dockerfile exemple
ARG PHP_VERSION=""
FROM php:${PHP_VERSION}-apache
INCLUDE+ alpine/php/commons/Dockerfile.cakephp_requirements.common
the include will work, and load the common from "CONTEXT_FOLDER_SPECIFIED_IN_COMPOSE/alpine/php/commons/Dockerfile.cakephp_requirements.common"


btc_dm_remove_ble_bonding_keys() called for default action (IDFGH-7323)

https://github.com/espressif/esp-idf/blob/master/components/bt/host/bluedroid/btc/core/btc_dm.c#L280
I understand that delete bonding for default action is the safest route.
But, I have been getting classified as default on a connection timeout SMP_CONN_TOUT because there is no other case statement for timeout.
So, my question is... Are there any reasons why a connection timeout would be treated as a threat?

@rimai-azena I don't think that the SMP_CONN_TOUT  is in default.

Thanks for reporting, feel free to reopen.


ARM64: Features -> gardener -> containerd

Meta Issue: #1028
Features -> gardener -> containerd
We need to ensure that all required services for the gardener feature will work. This is mandatory for a full Gardener support. Therefore, we need to ensure this also works on AARCH64 architecture. Therefore, we need to validate the functionality of containerd. Afterwards, we need to create a Pytest based unit test for this service.

● containerd.service - containerd container runtime
     Loaded: loaded (/lib/systemd/system/containerd.service; disabled; vendor p>
    Drop-In: /etc/systemd/system/containerd.service.d
             └─override.conf
     Active: active (running) since Wed 2022-06-29 09:54:10 UTC; 17min ago
       Docs: https://containerd.io
    Process: 647 ExecStartPre=/sbin/modprobe overlay (code=exited, status=0/SUC>
   Main PID: 649 (containerd)
      Tasks: 9
     Memory: 47.3M
     CGroup: /system.slice/containerd.service
             └─649 /usr/bin/containerd

Jun 29 09:54:10 localhost containerd[649]: time="2022-06-29T09:54:10.086733724Z>
Jun 29 09:54:10 localhost containerd[649]: time="2022-06-29T09:54:10.086796084Z>
Jun 29 09:54:10 localhost containerd[649]: time="2022-06-29T09:54:10.086863124Z>
Jun 29 09:54:10 localhost containerd[649]: time="2022-06-29T09:54:10.086895835Z>
Jun 29 09:54:10 localhost containerd[649]: time="2022-06-29T09:54:10.086905340Z>
Jun 29 09:54:10 localhost containerd[649]: time="2022-06-29T09:54:10.086912794Z>
Jun 29 09:54:10 localhost containerd[649]: time="2022-06-29T09:54:10.087094031Z>
Jun 29 09:54:10 localhost containerd[649]: time="2022-06-29T09:54:10.087202146Z>
Jun 29 09:54:10 localhost containerd[649]: time="2022-06-29T09:54:10.087309965Z>
Jun 29 09:54:10 localhost systemd[1]: Started containerd container runtime.

no issues at all.


Spotify cannot add source for local files

Describe the bug
When adding a local directory to spotify client, it always fails due to missing gsettings schema(s) and spotify crashes.
This issue is the same as the one from 2021 https://github.com/NixOS/nixpkgs/issues/120070.
Works as expected on arch based distros indicating that this an issue with nixpkgs
Steps To Reproduce
Steps to reproduce the behavior:

run spotify
open Settings
turn on Show Local Files
click ADD A SOURCE button

Expected behavior
Open a file chooser to select which directory to be added.
Additional context
/home/leon/.nix-profile/bin/spotify: /nix/store/pqlfvq6nm4vq3cpd0f3wdl43wdvj958f-curl-8.8.0/lib/libcurl-gnutls.so.4: no version information available (required by /home/leon/.nix-profile/bin/spotify)
/nix/store/902fsnn62xgj3ljh092i9f9zb7k3j5jv-spicetify-Dribbblish/share/spotify/.spotify-wrapped: /nix/store/pqlfvq6nm4vq3cpd0f3wdl43wdvj958f-curl-8.8.0/lib/libcurl-gnutls.so.4: no version information available (required by /nix/store/902fsnn62xgj3ljh092i9f9zb7k3j5jv-spicetify-Dribbblish/share/spotify/.spotify-wrapped)
/nix/store/902fsnn62xgj3ljh092i9f9zb7k3j5jv-spicetify-Dribbblish/share/spotify/.spotify-wrapped: /nix/store/pqlfvq6nm4vq3cpd0f3wdl43wdvj958f-curl-8.8.0/lib/libcurl-gnutls.so.4: no version information available (required by /nix/store/902fsnn62xgj3ljh092i9f9zb7k3j5jv-spicetify-Dribbblish/share/spotify/.spotify-wrapped)

(spotify:132796): Gtk-WARNING **: 01:33:24.189: Could not load a pixbuf from icon theme.
This may indicate that pixbuf loaders or the mime database could not be found.

(spotify:132796): GLib-GIO-CRITICAL **: 01:33:24.223: g_settings_schema_source_lookup: assertion 'source != NULL' failed
/proc/self/exe: /nix/store/pqlfvq6nm4vq3cpd0f3wdl43wdvj958f-curl-8.8.0/lib/libcurl-gnutls.so.4: no version information available (required by /proc/self/exe)

(spotify:132796): GLib-GIO-ERROR **: 01:33:54.117: No GSettings schemas are installed on the system
[1]    132796 trace trap (core dumped)  spotify

Notify maintainers
@eelco @ftrvxmtrx @sheenobu @mudri @timokau @Ma27 @jtojnar @freyacodes @BlankParticle @Gerg-L @mjoerg @Artturin

Metadata
Please run nix-shell -p nix-info --run "nix-info -m" and paste the result.
[user@system:~]$ nix-shell -p nix-info --run "nix-info -m"
 - system: `"x86_64-linux"`
 - host os: `Linux 6.9.5, NixOS, 24.11 (Vicuna), 24.11.20240618.c00d587`
 - multi-user?: `yes`
 - sandbox: `yes`
 - version: `nix-env (Nix) 2.18.2`
 - channels(root): `"nixos-23.11"`
 - nixpkgs: `/nix/store/pfqbxdicyksnqda6h5xv285sx7vamwzd-source`

Add a :+1: reaction to issues you find important.

I can't actually reproduce the issue, but I do see some inherent packaging issues that I attempted to address. Can you test the linked PR?

I also cannot reproduce the issue. However, I have never actually had local files appear after I have added their directories (without crashing).

@eclairevoyant Yes your PR solves the issue!!! Thanks a lot for the fast reply :)
I would really like to learn more about packaging programs for nixpkgs so I can contribute more myself.
Do you have any recommendations on how to learn the different options for mkDerivation, the different phases, wrapping programs and libraries, etc.? I feel like the documentation is basically non-existend.
If anything comes to mind (including workshops, etc.) feel free to just send me an email<EMAIL_ADDRESS>(this issue is probably the wrong place to discuss this)


Improvements needed: code switfy, readability, more customizations

More swifty
1. Naming
All the static vars in NVActivityIndicatorView are named in FULL_UPPERCASE, such as:
public static var DEFAULT_TYPE: NVActivityIndicatorType = .ballSpinFadeLoader

According to API Design Guidelines, a better way can be:
public static var defaultType: NVActivityIndicatorType = .ballSpinFadeLoader

2. Positions
Many of static vars in NVActivityIndicatorView.swift are not used by NVActivityIndicatorView  itself, such as:
public static var DEFAULT_BLOCKER_SIZE = CGSize(width: 60, height: 60)

Move these vars to NVActivityIndicatorPresenter.swiftor NVActivityIndicatorViewable.swift can improve readability.
Potential issue
At line 118 and line 154 in NVActivityIndicatorShape.swift :
lineWidth is set to 2, instead of using variable lineWidth declared earlier in the scope.
More customization supports

Users should be able to customize builtin shapes' specific properties, such as lineWidth, inner paddings.
Users should be able to customize builtin animations' specific properties, such as speed, orientation.
Users should be able to associate their custom shapes with builtin animations, or custom animations with builtin shapes, or both to leverage this library's viewable and presenter functionalities.

For above improvements, PRs can be sent by me if the owner is willing to do so

Thanks for your suggestions and thoughts. Feel free to create PR regarding Swift style and code refactor.
As for customization, I'm a bit conservative about it. The reason is that all current animations are preset with specs like speed, stroke width, and look good with these values. Moreover, it's the essence of this library, simple, good looking animations out of the box, and worry free about customization. Giving users too much freedom and choices often confuses them rather than helps. Possible customizable properties like color, padding are already supported. You can suggest anything else without touching too much animation specs.
That being said, it doesn't mean there won't be room for customization. One direction I already think of is making a framework to ease creating new animations, and it might be a complement library to this one, or as a creator of this one.

Thanks for your response. Fully understand👌. I'll only focus on code improvements. Closing the issue.


Can't resolve 'module'  ‘fs’

I updated my webpack version to v4.1.1 node v9.5.0 ,but I encounter a problem that current extract-text-webpack-plugin does not resolve 'module'  'fs'.

TL;DR Use mini-css-extract-plugin instead :)
The webpack >= v4.0.0 'support' for extract-text-webpack-plugin was moved to mini-css-extract-plugin as more lightweight approach for webpack >= v4.0.0 and ETWP entered maintenance mode. We are sorry for the inconvenience here, but ETWP reached a point where maintaining it become too much of a burden and it's not the first time upgrading a major webpack version was complicated and cumbersome due to issues with ETWP. The mid-term plan is integrate CSS support directly into webpack to avoid this kind of issue in the future


make ceylon.bat work on 64-bit windows from a 32-bit process

It looks like the command REG QUERY queries against a fake 32-bit compatible view of the registry when running from a 32 bit process on a 64 bit Windows installation. This prevents ceylon.bat from locating the Java installation.
On a 64 bit machine, adding /reg:64 to the REG QUERY command is a workaround to Microsoft's "Registry Redirector", but I imagine this would break 32 bit installations.
So, somehow, what's needed is for the registry redirector to be disabled for our REG QUERY command.
See also https://msdn.microsoft.com/en-us/library/windows/desktop/aa384232(v=vs.85).aspx

MS's 32bit to 64bit transition is hilariously bad: http://www.samlogic.net/articles/sysnative-folder-64-bit-windows.htm

in a 64-bit Windows the System32 folder is intended for 64-bit binary files

and that's just the starting point.
It works to use %Windir%/sysnative/reg.exe, but sysnative appears to only be available on newer 64 bit versions of windows when running 32 bit code.

So I don't know what the best solution is, but adding these to sections to the bat file works for me:
After the first "get the current version" attempt:
if "%ValueValue%" NEQ "" (
    set "JAVA_CURRENT=%KEY_NAME%\%ValueValue%"
) else (
    rem Try again for 64bit systems from a 32-bit process

    FOR /F "usebackq skip=2 tokens=3" %%A IN (`REG QUERY "%KEY_NAME%" /v CurrentVersion /reg:64 2^>nul`) DO (
        set "JAVA_CURRENT=%KEY_NAME%\%%A"
    )
)

and after the "get the javahome" attempt:
if "%JAVA_HOME%" EQU "" (
    rem Try again for 64bit systems from a 32-bit process
    FOR /F "usebackq skip=2 tokens=3*" %%A IN (`REG QUERY "%JAVA_CURRENT%" /v JavaHome /reg:64 2^>nul`) DO (
        set "JAVA_HOME=%%A %%B"
    )
)

@quintesse I've taken this about as far as I can. Making the changes above works for me, but I don't know enough about Windows to be sure they are correct.

Yeah, and the problem is: how to test? :/

For the "this works" case, the VSCode extension can be used. But for the "this breaks everything else" case, no idea...
It seems the risk would be older versions of windows and 32-bit windows, which may not have the /reg:64 option.

Indeed, and that's exactly what I mean with "how to test?" heh
I have no idea where to get a 32 bit Windows from...

So here's the relevant section of the modified ceylon.bat I tested on win XP 32-bit
:: get the current version
FOR /F "usebackq skip=2 tokens=3" %%A IN (`REG QUERY "%KEY_NAME%" /v CurrentVersion 2^>nul`) DO (
    set "ValueValue=%%A"
)

if "%ValueValue%" NEQ "" (
    set "JAVA_CURRENT=%KEY_NAME%\%ValueValue%"
) else (
    rem Try again for 64bit systems

    FOR /F "usebackq skip=2 tokens=3" %%A IN (`REG QUERY "%KEY_NAME2%" /v CurrentVersion 2^>nul`) DO (
        set "JAVA_CURRENT=%KEY_NAME2%\%%A"
    )
)

if "%ValueValue%" NEQ "" (
    set "JAVA_CURRENT=%KEY_NAME%\%ValueValue%"
) else (
    rem Try again for 64bit systems from a 32-bit process

    FOR /F "usebackq skip=2 tokens=3" %%A IN (`REG QUERY "%KEY_NAME%" /v CurrentVersion /reg:64 2^>nul`) DO (
        set "JAVA_CURRENT=%KEY_NAME%\%%A"
    )
)

if "%JAVA_CURRENT%" == "" (
    @echo Java not found, you must install Java in order to compile and run Ceylon programs
    @echo Go to http://www.java.com/getjava/ to download the latest version of Java
    exit /b 1
)

:: get the javahome
FOR /F "usebackq skip=2 tokens=3*" %%A IN (`REG QUERY "%JAVA_CURRENT%" /v JavaHome 2^>nul`) DO (
    set "JAVA_HOME=%%A %%B"
)

if "%JAVA_HOME%" EQU "" (
    rem Try again for 64bit systems from a 32-bit process
    FOR /F "usebackq skip=2 tokens=3*" %%A IN (`REG QUERY "%JAVA_CURRENT%" /v JavaHome /reg:64 2^>nul`) DO (
        set "JAVA_HOME=%%A %%B"
    )
)

:javaend

Thanks @chochos!

Fixed. Thanks @jvasileff !


install gpu-operator fail

Environmental Info:
K3s Version:
(base) [root@localhost docker-package]# k3s -v
k3s version v1.27.9+k3s1 (2c249a39)
go version go1.20.12

root@corp:~/gpu-operator/gpu-operator-24.6.2/deployments# helm version
version.BuildInfo{Version:"v3.16.2", GitCommit:"13654a52f7c70a143b1dd51416d633e1071faffb", GitTreeState:"clean", GoVersion:"go1.22.7"}

Node(s) CPU architecture, OS, and Version:
(base) [root@localhost docker-package]# cat /etc/redhat-release
CentOS Linux release 8.5.2111

Cluster Configuration:
root@corp:~/gpu-operator/gpu-operator-24.6.2/deployments# kubectl get node -o wide
NAME                    STATUS   ROLES                  AGE    VERSION        INTERNAL-IP      EXTERNAL-IP      OS-IMAGE             KERNEL-VERSION                CONTAINER-RUNTIME
corp                    Ready    control-plane,master   235d   v1.27.9+k3s1   <IP_ADDRESS>   <IP_ADDRESS>   Ubuntu 20.04.6 LTS   5.4.0-177-generic             containerd://1.7.11-k3s2.27
localhost.localdomain   Ready    <none>                 23h    v1.27.9+k3s1   <IP_ADDRESS>   <IP_ADDRESS>   CentOS Linux 8       4.18.0-348.7.1.el8_5.x86_64   containerd://1.7.11-k3s2.27

Describe the bug:
The cluster is provisioned with Helm. There are a Nvidia 4090 GPU card in node localhost.localdomain.
When I try to install gpu-operator 24.6.2 with helm-chart, i get following error
root@corp:~/gpu-operator/gpu-operator-24.6.2# helm install nvidiagpu -n gpu-operator --create-namespace --set toolkit.env[0].name=CONTAINERD_CONFIG --set toolkit.env[0].value=/var/lib/rancher/k3s/agent/etc/containerd/config.toml --set toolkit.env[1].name=CONTAINERD_SOCKET --set toolkit.env[1].value=/run/k3s/containerd/containerd.sock --set toolkit.env[2].name=CONTAINERD_RUNTIME_CLASS --set toolkit.env[2].value=nvidia --set toolkit.env[3].name=CONTAINERD_SET_AS_DEFAULT --set-string toolkit.env[3].value=true nvidia/gpu-operator
Error: INSTALLATION FAILED: failed to install CRD crds/nvidia.com_clusterpolicies_crd.yaml: resource mapping not found for name: "clusterpolicies.nvidia.com" namespace: "" from "": no matches for kind "CustomResourceDefinition" in version "apiextensions.k8s.io/v1"
ensure CRDs are installed first

Expected behavior:
helm install successfully.
Actual behavior:
helm install fail

This doesn't look like a k3s issue. What version of Helm are you using?

I don't believe this is an issue with k3s. This looks like a problem with the chart or the values you're passing when installing the chart.


Add snow analyzers and collectors in support bundle

Issue #, if available:
Description of changes:
Add snow crd, controller analyzers and collectors in the support bundle.
Testing (if applicable):
Unit tests
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

/lgtm
/approve


[BitFlyer] Partial Implementation

Can you resolve the conflict? You'll need to update your upstream with the timmolter fork and merge conflicts. The re-push.


Diagnostic UI

Please provide user acceptance criteria/requirements in the description of this ticket when you have time so as we are click testing it we know what to pay attention to.


chore(version): bump to 4.6.2

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (3003e32) 89.54% compared to head (93e7430) 89.54%.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@           Coverage Diff           @@
##           master    #1508   +/-   ##
=======================================
  Coverage   89.54%   89.54%
=======================================
  Files         194      194
  Lines        6341     6342    +1
=======================================
+ Hits         5678     5679    +1
  Misses        663      663

Flag
Coverage Δ

py3.8-ubuntu-22.04-pandas
89.54% <100.00%> (+<0.01%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


fix(client): replace lookbehind in transformEditorLink

Checklist:

[x] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)
[x] My pull request targets the main branch of freeCodeCamp.
[x] All the files I changed are in the same world language, for example: only English changes, or only Chinese changes, etc.

Closes #41884
This is a quick fix, which may just become the permanent fix until Safari get on board with other browsers. However, as I do not have a Mac, I am unable to confirm if this is the only needed change.

Although, I can't get to any of the challenges in Safari from a separate issue.

Is that reported already and isolated to Safari only


Multi-block blob transfer in Storage client

The 1.3.0 version preview provides the Storage client for sending blobs to the Azure Storage service. However, this client allows only for sending one HTTP request with the whole blob content as the request body. Let's take the situation where the microcontroller device is acquiring lots of data from sensors (e.g. 10 seconds of 16 kHz, 3-axis accelerometer = ~2MB) and saving those data in the external flash memory. The device has only ~100 kB of free RAM. Without the multi-block blob transfer, the only solution I see with this Storage client is to create over a dozen files for one data acquisition instead of one, which is of course possible, but very inconvenient for future processing.
Are there any plans to implement a multi-block transfer similar to the one in the Azure IoT C SDK?

We could implement this; we've been waiting to see if any customers really needed the behavior.
We could implement uploading multiple blocks to a block blob or we could implement appending blocks to an append blob.
Do you have a preference? Seems like the append blob is perfect for your scenario.

Support for the append blob sounds good. It will be great if the library supports sending MD5 or CRC64 hash of the append block content to verify the integrity of the block during transport.

Don't you think that just using HTTP(S) is good enough for verifying the integrity ?

Most probably, yes. I just saw this option in the append block documentation: https://docs.microsoft.com/en-us/rest/api/storageservices/append-block

It will take our team a while to get to this request but we'd love to do it. It also wouldn't be difficult for someone to copy/paste this code and tweak it to create a new az_storage_blobs_appendblob_appendBlock function which wraps this operation. Another az_storage_blobs_appendblob_create function is required that wraps this operation.
If you wrote these 2 functions, you'd be up and running quickly and, of course, we'd love you to make a PR to this repo :)

Hi there, I've made a pull request with the code related to this issue. As different blob types are useful for different use cases I've decided to implement both multi-block and append blobs functionalities. I've also implemented sending blobs using IoT Hub, but the code relies on this PR, so I will make another contribution with this functionality after this PR will be merged with the feature branch.
Regards,
Szymon Stanik

Hi @szymsta,
Thank you for the PR.
We could take  az_storage_blobs_appendblob_create() and az_storage_blobs_appendblob_append_block(), but not az_storage_blobs_multiblock_blob_upload() as it is.

With current design, it is not clear what should be done if error occurs in the middle of upload.
With 2 functions, user can wrap them to essentially do what az_storage_blobs_multiblock_blob_upload() currently does: it could iterate over blocks, and then commit the result, if there were no errors (or with some errors it thinks are acceptable).
BTW, I saw that you are doing snprintf() of block id to perform "itoa". That is not necessary. The block id is a binary-value int64 that is Base64-encoded.

The block ID is a 64 BYTE (not bit) value that is base-64 encoded. Please see https://learn.microsoft.com/en-us/rest/api/storageservices/put-block. Then, this function needs the list of base-64 strings after all the blocks have been uploaded: https://learn.microsoft.com/en-us/rest/api/storageservices/put-block-list.
We're basically OK with replicating these function from our Go SDK; use these as a model:
https://github.com/Azure/azure-sdk-for-go/blob/main/sdk/storage/azblob/blockblob/client.go#L170
https://github.com/Azure/azure-sdk-for-go/blob/main/sdk/storage/azblob/blockblob/client.go#L202

We are not currently planning to add this to the C SDK.  If you are able to use C++ the Storage SDK in C++ has many of these features.


Paradise playtest: some screen elements never disappear

I got into a box then popped out to scare someone, somehow this also scared me and gave us both the red exclamation alert, but it never faded either.

Are these handled by client.images? Possible dupe of https://github.com/OpenDreamProject/OpenDream/issues/1422

The box displays the image using flick_overlay(I,viewing,8)
/proc/remove_images_from_clients(image/I, list/show_to)
	for(var/client/C in show_to)
		C.images -= I

/proc/flick_overlay(image/I, list/show_to, duration)
	for(var/client/C in show_to)
		C.images += I
	addtimer(CALLBACK(GLOBAL_PROC, GLOBAL_PROC_REF(remove_images_from_clients), I, show_to), duration, TIMER_CLIENT_TIME)

Something about the timer or remove_images_from_clients is not working.

oh update, i was wearing the pet chicken on my head at hte time I popped out of the box, not sure if that was related.

I can't seem to reproduce this. I think the client.images fix fixed this?

I cannot replicate this anymore either.


series flag doesn't work

I added the series flag to my post like categories and tags but it doesn't appear in the series section is it a bug or I made a mistake somewhere?

How did you set it in the front-matter?

I did it too, this is my latest posts front-matter in the below link as you can see the series flag is there
photoshop.md

I found that you have the wrong config params.
[taxonomies]
  category = "categories"
  tag = "tags"
  series = "series"

for the search, in your config.toml file
[outputs]
  page = ["HTML", "SearchIndex"]
  home = ["HTML", "RSS", "SearchIndex"]
  section = ["HTML", "RSS", "SearchIndex"]
  taxonomyTerm = ["HTML", "RSS", "SearchIndex"]
  taxonomy = ["HTML", "RSS", "SearchIndex"]

[outputFormats]
  [outputFormats.SearchIndex]
    mediaType = "application/json"
    baseName = "index"
    isPlainText = true
    notAlternative = true

In your params.toml file
searchResultPosition = "side" # side, main

thank you so much problem solved :1st_place_medal:


StackOverflowException with LinkedList

I have simple data class.
public class Test
{
	private Question currentQuestion;
	public event Action<Question> OnNext;
	public event Action OnFinished;
	public LinkedList<Question> Questions { get; }
}

When i create my service with method
[JsonRpcMethod]
		public Test GetTest(int questionsCount)
		{
			....
		}

VS lock and after some times throw StackOverflowException
If i change Questions type to List<> exception not throw

Possibly fixed with https://github.com/Astn/JSON-RPC.NET/commit/b9f435a38e051550aea12da79b3857740d3c8c04
Can you test?

Possibly nuget package is outdate


luhn: tweak canonical tests

I was also tempted to update the description for this test to make it more clear that this is the only test that will fail if the "wrong" set of alternating digits are manipulated. Thoughts on that?
    {
      "description": "simple valid sin",
      "input": " 5 9 ",
      "expected": true
    }

Good call; I'll update that now!

@petertseng updated with a few changes:

removed doubled "valid Canadian SIN" test cases and simplified invalid cases to all be variants of the remaining valid SIN;
added early test using "059" and following valid test using "59";
updated the multiple-zeros test case to reduce possibility for misinterpretation.

@petertseng see also #549 which I believe addresses another potential source of confusion for implementors.

These changes look good to me. The descriptions and the test cases make sense and there doesn't seem to be enough overlap between them where they could be condensed down without sacrificing something. I don't have merge rights here but 👍   :shipit:

Great work, thanks! :tada:


[BUG] Table coloring messed up for darkmode

On My Courses -> Table of courses are a bit broken in darkmode.
I have included the exact CSS variable that needs to be changed in the image

Fixed in next release: https://github.com/PhilipFlyvholm/learnit-plus-plus/commit/7f505710d18c8b70ce935ceb6220f57ae4bc391d


903 - Update Test Bucket to 2024-09-10

Issue Number
Closes https://github.com/AlexsLemonade/scpca-portal/issues/903
Purpose/Implementation Notes

This PR updates the test bucket from scpca-portal-public-test-inputs/2024-07-19/ to scpca-portal-public-test-inputs/2024-09-10/.
In order to accommodate the change in how the Project attribute diagnoses is handled, a PROJECT_METADATA_VALUES_TRANSFORMS constant and transform_values helper function were added to metadata_file.
Values in the expected_values/ files were also updated in order to match any changed values in the metadata files in the new test bucket.
Types of changes
What types of changes does your code introduce?

Refactor (addresses code organization and design mentioned in corresponding issue)

Functional tests
N/A
Checklist

[x] Lint and unit tests pass locally with my changes

Screenshots
N/A

@davidsmejia
This PR is not very controversial, the one thing to discuss is the handling of the way that we update Project.diagnoses.
In light of https://github.com/AlexsLemonade/ScPCA-admin/pull/958/files, the diagnoses value in the projects_metadata.csv file is now correct. As such, we can now use it to populate Project.diagnoses, instead of what we used to do, which was to throw that value away and to collect the value from the aggregation of sample diagnosis values.
I proposed transforming the semi-colon separated list which we get in the project_metadatas.csv file to our intended comma (and space) separated output inside of metadata_file, where we're already doing key transformations, but we can discuss other options as well. The direction should be that this value should be updated in Project::get_from_dict, like all other values pulled from the projects_metadata.csv, and not to be saved in Project::update_project_aggregate_properties at the end.

This is not a problem and only a concern so I will open an issue to discuss this along with some other changes to Project::update_project_aggregate_properties for improving code reliability / consistency.

Sounds good. I opened up an issue related to aggregate value updating a few weeks ago to have as a place holder until we would devise a plan of attack for this. I'll attach it here for reference.
https://github.com/AlexsLemonade/scpca-portal/issues/880


Fix for #3 - secret_key generator

Shell script to create local_settings.py(if it doesn't exist) file and add SECRET_KEY

Rebase it. Not merge to prevent creating a merge commit.


IPS-735: Add name to loadbalancer

Proposed changes

Add name to load balancer as this continuously changes

What changed

Add name to load balancer

Why did it change

Load balancer name changes everytime when deploy a stacks.
This breaks the load balancer called on the common infra data source

Issue tracking

IPS-735

Checklists
Environment variables or secrets

[ ] Documented in the README
[ ] Added screenshots to show the implementation is working
[ ] Ran cfn-lint on any SAM templates

Other considerations

Changing load balancer name is not a solution for the WAF association. Instead the WAF association could be added on the Frontend template rather than doing it with terraform


init-pants - getting "gh: Not Found (HTTP 404)"

Tried to figure out why init-pants action does not work for me... According to https://stackoverflow.com/a/73011420/3142796, there should not be / in front of repos. Could this be the problem?
The result I'm getting: https://github.com/robofit/arcor2/actions/runs/4022974520/jobs/6913303946#step:6:96

Oh, interesting, good find!
@cognifloyd , I assume you tested the CACHECOMMIT stuff with the URL as-is and it worked? Maybe this is sensitive to gh version or something.
@ZdenekM would you be able to test your hypothesis by copying that gh api command into a step directly in your CI (early enough that it's before this failure, of course, or you can temporarily remove the init-pants step, since we only care about running that gh api command, everything after it can fail) verifying that it fails, and then verifying that it succeeds if you remove the slash?

Yes. I've used the initial / for awhile in a variety of projects. That's good to know we can drop it. I'm surprised it didn't work.

Ok, it seems that the problem is somewhere else. Sorry for the noise :-)

Hmm, maybe some issue with GITHUB_TOKEN permissions? I think I haven't changed anything from the defaults...

read permissions should be enough, so I doubt that's it.

Your main branch is master not main, so you need to pass that in the parameter: base-branch=master

@cognifloyd Yep, that was it! Thanks...


Broken results while navigating under FD

Can be reproduced if fd is installed. for i in * works well.

@AethanFoot could you please help me with this? These lines are probably the issue. Thanks!

Can you run the fd -H -d 1 in the folder and an ls to compare the output

The outputs are the same:

Note that I can also reproduced this problem on all directories with files. And the thumbnails generated from image files are not showing/missing, while the icons from icon pack (tela/papirus/etc.) aren't. There's also problem with first whitespace again. I tried fixing it but I'm really at my wits end haha.


REASON_INVALID_OFFERS optional retry

Hi Team,
singularity Version: 0.14.1
mesos version: 1.2.0
We seem to have a lot of cases of "Task launched with invalid offers: Offer 1b7d8a83-8b6b-47cd-b6a6-fddf4f92eed2-O250453 is no longer valid"
Is it possible to optionally configure a task retry incase of a task  lost due to "REASON_INVALID_OFFERS"
We are in a process to investigate why these happen, but task lost due to those reasons are almost successful in a retry.

REASON_INVALID_OFFERS generally appears because Singularity and/or the mesos master are too slow to process the tasks being launched. We noticed in versions <15 that our zk usage was slowing down status update processing enough that, with a high count of tasks, the pipe to and from the mesos master was essentially getting clogged. So, Singularity would receive the offer, but that would be delayed behind a bunch of status updates (the mesos java library restricts the processing of events from the mesos master to a single thread). By the time that offer was accepted by singularity and made it back to the mesos master, the master had already expired the offer (expiration time is configurable).
As a short term fix, you could increase the offer expiration timeout on the mesos master. Longer term, we have more permanent fixes for this in 0.15.0 which dramatically speed up status update and offer processing by moving more of the data needed into memory instead of relying on hitting zk each time.
I'm going to close this since the underlying issue has a fix in newer release, feel free to reopen if you are still running into the issue.

Thanks @ssalinas.
There were suggestions to upgrade to a newer version of singularity/mesos but I did not see any previous ticket or new test case addressing this problem.
Anyways thanks for explaining the issue.
Appreciate your help.


Roadmap

Is there a roadmap set? I'm looking for effective ways of contributing to the project.

@Irio Currently we don't have roadmap, feel free to suggest/work on any topic you find useful.


Consider removing some warnings.

as title.
use make all that you will get the following warning message
error: boolean to int conversion using if
   --> src/date.rs:181:26
    |
181 |         let leap_extra = if is_leap_year(self.year) { 1 } else { 0 };
    |                          ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: replace with from: `u16::from(is_leap_year(self.year))`
    |
    = note: `-D clippy::bool-to-int-with-if` implied by `-D warnings`
    = note: `is_leap_year(self.year) as u16` or `is_leap_year(self.year).into()` can also be valid options
    = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#bool_to_int_with_if

error: single-character string constant used as pattern
  --> src/datetime.rs:45:81
   |
45 |             f.write_str(std::str::from_utf8(&buf[..]).unwrap().trim_end_matches("0"))?;
   |                                                                                ^^^ help: try using a `char` instead: `'0'`
   |
   = note: `-D clippy::single-char-pattern` implied by `-D warnings`
   = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#single_char_pattern

error: single-character string constant used as pattern
  --> src/time.rs:47:81
   |
47 |             f.write_str(std::str::from_utf8(&buf[..]).unwrap().trim_end_matches("0"))
   |                                                                                ^^^ help: try using a `char` instead: `'0'`
   |
   = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#single_char_pattern

error: could not compile `speedate` due to 3 previous errors

This is just because clippy has added more warnings.
PR welcome to add an ignore to the first one and fix the second two.


fix: cwd() in Windows

this is a small fix for #2
file.history[0] is using forward slashes but in Windows process.cwd() returns a path with backslashes.

oops, good catch & thanks for the fix!


⚠ SceneTime has degraded performance

In 8d4e8fa, SceneTime ($ST) experienced degraded performance:

HTTP code: 200
Response time: 1233 ms

Resolved: SceneTime performance has improved in 1176135 after 6 minutes.


Training halts after some iteration in first epoch

After some iteration in first epoch of training, the process halts with following output.
Epoch 1/10
/home/deepesh/Documents/ssd_traffic/ssd_utils.py:119: RuntimeWarning: divide by zero encountered in log
assigned_priors_wh)
2017-10-14 15:53:09.118394: W tensorflow/core/common_runtime/bfc_allocator.cc:217] Allocator (GPU_0_bfc) ran out of memory trying to allocate 1.54GiB. The caller indicates that this is not a failure, but may mean that there could be performance gains if more memory is available.
2017-10-14 15:53:17.147597: W tensorflow/core/common_runtime/bfc_allocator.cc:217] Allocator (GPU_0_bfc) ran out of memory trying to allocate 1.14GiB. The caller indicates that this is not a failure, but may mean that there could be performance gains if more memory is available.
2017-10-14 15:53:18.300263: W tensorflow/core/common_runtime/bfc_allocator.cc:217] Allocator (GPU_0_bfc) ran out of memory trying to allocate 2.17GiB. The caller indicates that this is not a failure, but may mean that there could be performance gains if more memory is available.
2017-10-14 15:53:18.300311: W tensorflow/core/common_runtime/bfc_allocator.cc:217] Allocator (GPU_0_bfc) ran out of memory trying to allocate 1.10GiB. The caller indicates that this is not a failure, but may mean that there could be performance gains if more memory is available.
2017-10-14 15:53:19.177374: W tensorflow/core/common_runtime/bfc_allocator.cc:217] Allocator (GPU_0_bfc) ran out of memory trying to allocate 2.26GiB. The caller indicates that this is not a failure, but may mean that there could be performance gains if more memory is available.
2017-10-14 15:53:20.211245: W tensorflow/core/common_runtime/bfc_allocator.cc:217] Allocator (GPU_0_bfc) ran out of memory trying to allocate 1.46GiB. The caller indicates that this is not a failure, but may mean that there could be performance gains if more memory is available.
2017-10-14 15:53:20.211321: W tensorflow/core/common_runtime/bfc_allocator.cc:217] Allocator (GPU_0_bfc) ran out of memory trying to allocate 3.94GiB. The caller indicates that this is not a failure, but may mean that there could be performance gains if more memory is available.
2017-10-14 15:53:23.378445: W tensorflow/core/common_runtime/bfc_allocator.cc:217] Allocator (GPU_0_bfc) ran out of memory trying to allocate 2.27GiB. The caller indicates that this is not a failure, but may mean that there could be performance gains if more memory is available.
2017-10-14 15:53:23.378500: W tensorflow/core/common_runtime/bfc_allocator.cc:217] Allocator (GPU_0_bfc) ran out of memory trying to allocate 1.15GiB. The caller indicates that this is not a failure, but may mean that there could be performance gains if more memory is available.
2017-10-14 15:53:25.365196: W tensorflow/core/common_runtime/bfc_allocator.cc:217] Allocator (GPU_0_bfc) ran out of memory trying to allocate 2.13GiB. The caller indicates that this is not a failure, but may mean that there could be performance gains if more memory is available.
13/14 [==========================>...] - ETA: 8s - loss: 2.9593
No output/error after the above line. What may be causing this problem???

you could lower your batchsize


windows 10 incompatibility

". : File B:\Programy\cmder\vendor\profile.ps1 cannot be loaded because running scripts is disabled on this sys tem. For more information, see about_Execution_Policies at http://go.microsoft.com/fwlink/?LinkID=135170.
At line:1 char:3

. 'B:\Programy\cmder\vendor\conemu-maximus5..\profile.ps1'

+ CategoryInfo          : SecurityError: (:) [], PSSecurityException
+ FullyQualifiedErrorId : UnauthorizedAccess

"
This is the welcome message when i turn cmder on windows 10.
Also i can't install bower. After install grunt he don't see the "grunt" at all.
In cmd everything ok. I'm waiting for updates for windows 10:)

To fix that, run Powershell as Administrator und execute the command Set-ExecutionPolicy RemoteSigned.
To change it back, execute: Set-ExecutionPolicy Restricted

RemoteSigned didn't work for me, I had to choose the "Bypass" execution policy.

I had it installed, before I moved to Windows 10, so after the upgrade was done everything works fine. My only issue is that the current git branch, isn't displayed, instead I get a {git}. The lamba sign is also replaced with {lamb}.

Update to v1.2 and overwrite the config files.

I had the same thing as @crisbeto: Updated to Win10. So I installed from fresh, updated ConEmu from source and ran the command suggested by @r15ch13.
It is working find apart from the following warning, which is only a very minor annoyance.

WARNING: Enable-GitColors is Obsolete and will be removed in a future version of posh-git.

Same as @crisbeto Updated to Win10 too, so I decided to delete all and start from fresh.
Same as @pavelkouril, "Bypass" saved me.
I still have an issue with zsh, but can't find out how to fix it, so if you have any advice / tips.

I actually fixed it by downloading again and pasting everything into the old folder. The only other issue I found is that {hg} gets added after the path., although I can live with that.

that happens when you updated to 1.2 without deleting the old folder. Compare the files in the zip with what is in your folder and delete the ones that are extra.

My prompt color issue came from posh-git who enable colors by default now.
In this commit eeree found and solved my issue with colors. :+1:

the "Bypass" execution policy worked for me

Set-ExecutionPolicy RemoteSigned in Windows 10 Enterprise worked for me when running powershell as Admin.


Implement Delayed Rejection HMC

Presentation of the new sampler
https://arxiv.org/abs/2110.00610
How does it compare to other algorithms in blackjax?
Authors claim up to 5x more ESS / gradient evaluation.
Where does it fit in blackjax?
In the mcmc section. The delayed rejection part is not specific to HMC and should be relatively general.
Are you willing to open a PR?
Yes.

More infos:
Paper (pre-print): https://projecteuclid.org/journals/bayesian-analysis/advance-publication/Delayed-rejection-Hamiltonian-Monte-Carlo-for-sampling-multiscale-distributions/10.1214/23-BA1360.full
Talk: https://www.youtube.com/watch?v=p32zThyo9kY
Slides: https://statmodeling.stat.columbia.edu/wp-content/uploads/2023/09/carpenter-sciml-webinar-2023.pdf

Delayed rejection is a fairly old idea, there may be some room to make it a meta-algorithm rather than an instance of HMC. See https://www.jstor.org/stable/2673700

I'm working with the STAN team to implement / benchmark Delayed Rejection Generalized HMC. I'd be happy to answer any questions, if helpful.
Right now the algorithm is still under testing, but can update later when it is finalized.


Update artifact id of dependency for spring-cloud-deployer-kubernetes

Resolves spring-cloud/spring-cloud-dataflow-server-kubernetes#64
Resolves spring-cloud/spring-cloud-dataflow-server-kubernetes#65
Depends on spring-cloud/spring-cloud-deployer-kubernetes#7

Squashed and pushed as https://github.com/spring-cloud/spring-cloud-dataflow-server-kubernetes/commit/117d38e7a672ddcd24967deac3b015cc1ba32f1b


[Backport 2.x] [Refactor] ClusterInfo to use j.util.Map instead of ImmutableOpenMap (#7126)

Backport https://github.com/opensearch-project/OpenSearch/commit/ee4c8a13c380f7cc2fb45dfc18ce22a30ca4c8bc from #7126.

oye.. flaky test bonanza! refiring

More flaky tests that won't reproduce. It seems 2.x is less stable than main.

merged with #8241


CVE fixes for v1.3.13 release

CVE fixes for v1.3.13 release, upgrade the grpc protobuf and netty dependency to 1.56.0
Check List

[ ] New functionality includes testing.

[x] All tests pass

[ ] New functionality has been documented.

[ ] New functionality has javadoc added

[x] Commits are signed per the DCO using --signoff

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.

Mend Security check is currently failing for CVE-2023-34462: https://github.com/opensearch-project/performance-analyzer-rca/pull/488/checks?check_run_id=16695434293
This dependency version is picked from core and the failure will be resolved once the core has CVE fix added.


vector/aspect scales length of vectors incorrectly

Reported by ansley b manke on 28 Nov 2011 19:37 UTC
Russ Fiedler reports,
http://www.pmel.noaa.gov/maillists/tmap/ferret_users/fu_2011/msg00874.html
that the vector lengths are incorrect when /ASPECT is specified.  In his example, the W component of the vectors has a constant magnitude, so he's thinking that the vertical length should remain constant.
The documentation for vector/aspect says that we're intentionally keeping the lengths as drawn on the page the same as they would have been without /asp.
In Russ's example, the length of the vectors at latitude y=0 is
(1.0e-3*1.0e-3)^0.5 = 0.001

and the length of the vectors at y=5 is
(5.001*5.001 + 1.0e-3*1.0e-3)^0.5 = 5.001

Do we want to show the varying lengths graphically? Or do we want to show the constant w component graphically?
Migrated-From: http://dunkel.pmel.noaa.gov/trac/ferret/ticket/1900

Comment by ansley.b.manke on 29 Nov 2011 17:34 UTC
Capturing Russ's script here, for convenience,
yes? def axis/z=5:95:10/units=meters/depth zax
yes? def axis/y=-5:5:1/units=degrees_n yax
yes? def grid/y=yax/z=zax myg
yes? let w=1e-3+0*y[             ! Constant 1mm/s
yes? let v=y[g=myg](g=myg]+0*z[g=myg])+0*z[g=myg]                     ! Vary with latitude

yes? vec/asp v,w           ! w varies with lat  ! Wrong?

Re-thinking this more, the intention /ASPECT is to visualize a vector by giving the correct magnitude and direction, whereas removing /ASPECT correctly represents the individual vector components faithfully.  When /ASPECT is specified there is no way to document the scaling of the individual X and Y components, as they are not preserved uniformly.
We might add a third option, say, "/ASPECT=by_rescaling" (or something) that automates just what Russ suggests - to rebalance component lengths based on the different axis scalings. In this case rescaling w to w*9174, taking the value from the label on the vector/aspect plot: "adjusting for differing X/Y scales: 9174".
The 9174 is based on axis lengths and scale, so that if you do
yes? set win/asp=1
yes? vector/aspect v,w

then the label is "adjusting for differing X/Y scales: 11223". There must be something more consistent to use in setting a scaling based on the axis range and units, not on their lengths?


stage1: force dependent libraries to be installed for stage1

Sometimes necessary libraries become missing in stage1.aci, which
makes binaries broken at run-time. That's because usr_from_coreos.mk
does not require dependent libraries to be installed by default,
in contrast to usr_from_src.mk. As workaround, individual manifest
files have been updated to contain necessary libraries.
This can be easily fixed by turning on STAGE1_COPY_SO_DEPS. Then
in stage1/stage1.mk, it could invoke a pre-defined STAGE1_FSD_STAMP
target defined in find-so-deps.mk.
Fixes: https://github.com/coreos/rkt/issues/1190

I did not opted-in for getting libraries from host for coreos flavor purposefully - updating the manifest files in stage1/usr_from_coreos/manifest directory is the proper way to do it.
Also, it does not fix #1190 - for this issue I wanted to have the build system to actually bail out if ldd reports that some required library is missing from ACI rootfs. The message would say that "this and that library is missing, please update the manifest".
I'll comment more in the issue #1190.

@krnowak should we close this out then?

@krnowak Yeah, I see. Reading the issue #1190, I'm not sure I'm following. Apart from that, I agree that we need to do smarter anyway.


Cargofy the project

I'm by far not a Cargo expert, but here's an initial attempt that at least builds the library and executable.

@engstad You should remove the .gitattributes and the Cargo.lock file.
Also, the Cargo.toml should have as many keys as possible from the Package metadata requirements for crates.io.
That means you should remove the tags field, and add at least a readme, license, repository, and keywords field.

One thing missing with Cargo is the windres compilation of the resource file. Now, I can't actually test this, since I don't have it in my path (and I need to update my msys2 setup), so I will let someone else take care of that. However, I believe we'll need to make use of a build.rs script, such as the following:
#![allow(unstable)]

use std::io::{stderr, Command};
use std::os;

fn main() {
    let out_dir = os::getenv("OUT_DIR").unwrap();

    let status = Command::new("windres").arg("src/hello.rc")
        .arg(format!("{}/hello.rc.o", out_dir))
        .status();

    match status {
        Ok(_) => println!("cargo:rustc-flags=-C link-args={}/hello.rc.o", out_dir),
        Err(e) => stderr().write(format!("warning: failed to run windres: {}", e).as_bytes()).unwrap()
    }
}

Feel free to experiment with it (add build = "build.rs" to Cargo.toml).

@alexchandel Thanks, I will let the maintainer take care of that!

@engstad The additional Cargo.toml file in the examples directory is unnecessary. As the Cargo Project Layout guide details, files in the examples/ directory are automatically considered binaries and built as tests. Any other integration-like tests should go in tests/. The repository as a whole shouldn't have more than a single Cargo.toml, and a library shouldn't have a Cargo.lock anywhere.
Can I suggest squashing these commits too? Since a few files were added and subsequently deleted.

@alexchandel Cargo.toml is needed because the example needs to have an additional build step (see above).
My git-fu is not very strong. How would I squash them and push it upstream?

My bad. Are you sure the library and example build with just cargo build and cargo test?
Your branch hasn't been pulled yet so you're free to rewrite history.  You'll have to force push the branch to your GitHub repo.

Thanks! I'll do that.
Yes, it compiles and runs fine (cargo clean && cargo build && cargo run). The only problem is that the menu is not working, because of the windres issue.

@alexchandel Squashed!

Looks good to me.
I'd still think examples with their own Cargo are strange, since its toml doesn't build it any better than the master one.

@klutzy ready to merge?


Delete draft translation

When editing translations, I noticed that sometimes multiple drafts get saved, even while you've published one. What I miss is a way to clean up unwanted drafts by deleting them. I could not find a delete button,
please add.

I think this is an enhacement proposal 😄

Duplicate of #81.


Replaced Remove-AzureADUser with Remove-MgUser

Replaced the Cmdlet Remove-AzureADUser with Remove-MgUser
improved the script formatting for better readability
added #Requires statement to prevent running a script unless the required PowerShell modules are met.

@csmulligan
FYI:
I used the same GUID as used in the examples of the MS Learn documentation of "Remove-MgUser".
So may update is required there as well.

@csmulligan
Thanks for reviewing the proposed changes.
When the changes are ready for publication, add a #sign-off comment to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

#sign-off


fixed FileAccessError

Thanks for the PR, @fewspider! We'll need you to accept our Contributor License Agreement before we can merge your contributions.
Cheers,
The robots and humans at Realm

This is incorrect. RLMError.FileAccess is the right value: https://github.com/realm/realm-cocoa/blob/7d97479d49ea896859eb3b32d5caf18b5ce1b9ba/Realm/RLMConstants.h#L68