haskell: Remove lib directory if no active libraries

Motivation for this change
I want to compile statically linked Haskell binaries with small closures so that I can put them into Docker images.
Things done

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

[ ] NixOS
[ ] OS X
[ ] Linux

[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Fits CONTRIBUTING.md.

Currently we can't compile static executables if the .cabal file
contains a section for both executables and libraries.
This is because we do a "./Setup install" which copies the libraries,
even when "isLibrary = false' is specified. It doesn't seem like Cabal
provides a way to specify installing only libraries - so this just
removes the default directory which Cabal installs into.
This expression shows the problem:
haskell.lib.overrideCabal haskellPackages.palindromes (p: {
isLibrary = false;
enableSharedExecutables = false;
})
The above will have references to GHC in its closure because the lib
directory had references to GHC.
After this change, the closure should only contain base things like GMP
and glibc.

@peti @shajra

The isLibrary attribute is something we inherited from earlier versions of Nixpkgs and it should probably be removed. Therefore, I'm reluctant to add logic into the generic builder that depends on it.
I would prefer that "rm" to go into the postInstall attribute by way of override when you need it.


Main page is repeating phenopackets across the top

Every time I open a phenopacket from the list, it adds it AGAIN across the top. I have only two phenopackets, note the multiples. I think it is doing it every time I make a change to the phenopacket and then go back to the list.

Fixed


Pod carousel does not pause if you don't leave the carousel area after clicking an arrow

If you use one of the navigation arrows on the pod carousel and then leave your cursor inside the pod carousel, it will move automatically after a short time instead of pausing.
Moving the cursor outside the pod and back in then gives the desired behaviour of pausing the carousel

Cannot reproduce. There is a 10 second delay between movements when the cursor is outside the carousel, and a pause when the cursor is inside. @mjc209 are you still seeing this problem? If so, which browser are you using?

I'm still seeing this problem using Chrome, Version 41.0.2272.89 m

Fixed


ROCm 5.3 rocprim support

Part of https://github.com/cupy/cupy/issues/7092
cc: @takagi , @AdrianAbeyta

I think this is the same than #7079 ?
Thanks for the PR!

/test jenkins

I think this is the same than #7079 ? Thanks for the PR!

Yes, I forgot @hubertlu-tw  opened the other PR. Found that ROCm5.3 also needs this on Friday.


Leaner requirements for Py>3.8

Since the following libraries are present only when installed with python <3.9, you can adopt the following requirements spec:
[project]
dependencies = [
    ...
    "importlib_resources>=5.10.2; python_version < \"3.9\"",
    "typing_extensions>=4.4.0; python_version < \"3.9\"",
    ...
]

In this way, they will only be installed for Python <3.9.

I have read the CLA Document and I hereby sign the CLA

Want to replace those with importlib.resources and typing if the Python version is >= 3.9?

OMG I did not check, super sorry about that. Will fix this!

Thanks! Let me know if you get stuck on anything.

@akshayka I should have it fixed :) I ran the tests and the only ones that do not pass are these ones:
SKIPPED [1] marimo/_ast/test_visitor.py:563: condition: sys.version_info < (3, 12)
SKIPPED [1] marimo/_ast/test_visitor.py:574: condition: sys.version_info < (3, 12)
FAILED marimo/_cli/test_cli.py::test_cli_edit - assert None is not None
FAILED marimo/_cli/test_cli.py::test_cli_run - assert None is not None
FAILED marimo/_cli/test_cli.py::test_cli_tutorial - assert None is not None
FAILED marimo/_cli/test_envinfo.py::test_get_pip_list - json.decoder.JSONDecodeError: Expecting value: line 1 column 1 (char 0)
FAILED marimo/_cli/test_envinfo.py::test_get_system_info - json.decoder.JSONDecodeError: Expecting value: line 1 column 1 (char 0)

But I think this is not up to me. Happy to correct if it's me :)

@baggiponte I made two small suggestions and committed them using the Github UI, I hope you don't mind! Let's see if these fix the remaining CI issues.


last-resort: 14.000 -> 15.000

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://repology.org/metapackage/last-resort/versions.
meta.description for last-resort is: Fallback font of last resort
meta.homepage for last-resort is: https://github.com/unicode-org/last-resort-font
Updates performed

Version update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 15.000 in filename of file in /nix/store/60gw395r7i30zlrh9qixz4rdpnp8pan0-last-resort-15.000

Rebuild report (if merged into master) (click to expand)

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
last-resort

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/60gw395r7i30zlrh9qixz4rdpnp8pan0-last-resort-15.000 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A last-resort https://github.com/r-ryantm/nixpkgs/archive/f53e25064015506788f4d8d54e619e5d536be464.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/60gw395r7i30zlrh9qixz4rdpnp8pan0-last-resort-15.000
ls -la /nix/store/60gw395r7i30zlrh9qixz4rdpnp8pan0-last-resort-15.000/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

last-resort

Maintainer pings
cc @deviant for testing.

@ofborg build last-resort last-resort.passthru.tests


Snack down 21 problem 1

Hey thanks for your contribution ❗. Keep up the good job and leave a star if you liked the repo .
Happy hacking 😉


feat(OCPADVISOR-132): workloads recommendations expanded details

https://issues.redhat.com/browse/OCPADVISOR-132
First prototype of the expanded details.
Some of the data is missing from the API response and I had to "improvise" to add it
Also, according to the Shayna I don't need to reduce the width of the objects table in the expanded details.

Codecov Report
Attention: 70 lines in your changes are missing coverage. Please review.

Comparison is base (a88dfea) 82.24% compared to head (66efc1c) 6.52%.
Report is 5 commits behind head on master.

Files
Patch %
Lines

src/Components/WorkloadRules/WorkloadRules.js
0.00%
46 Missing :warning:

src/Utilities/TemplateProcessor.js
0.00%
18 Missing :warning:

...ts/ExpandedRulesDetails.js/ExpandedRulesDetails.js
0.00%
6 Missing :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #645       +/-   ##
==========================================
- Coverage   82.24%   6.52%   -75.73%
==========================================
  Files          37      66       +29
  Lines        1521    3220     +1699
  Branches      599     705      +106
==========================================
- Hits         1251     210     -1041
- Misses        270    3010     +2740

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

:tada: This PR is included in version 1.32.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Add spectral density computation using lanczos vectors

Implement Stochastic Lanczos Quadrature using scipy or a custom Lanczos implementation. Integrates the approach from this repo.

Hi @noahgolmant,
I would like to work on this issue. Do you have some starting points?
Thanks

Hello! Algorithm 1 from this paper has a mathematical description of the approach, and the authors then implemented it in the repo I linked above. I think the jax implementation is nice and clear. Most of the jax code should be one-to-one with PyTorch. The main steps would be:

Port the lanczos implementation to PyTorch. I am currently using scipy's built-in Lanczos method, but this only gives access to the final eigenvalues and eigenvectors, not the tridiagonal matrices.
Port the code for density estimation from tridiagonal matrices.
Hook up the code to use the existing HVPOperator class with this to go straight from model/dataloader/loss to eigenvalue density.
Add some basic precision tests with random matrices.

Would love to see this happen!


spec: built-in 'dir' function should return a sorted list

The Python spec for the dir built-in function https://docs.python.org/2/library/functions.html#dir says "the resulting list is sorted alphabetically". Neither the Java nor Go implementations match this behavior. I think the spec and both implementations should be changed to match Python. Given that the current order is unspecified, and given typical uses of the dir function, this should be a safe change.

Yes please. See also: https://github.com/google/starlark-rust/issues/64

In the Java implementation, the output of dir has always been sorted (https://github.com/bazelbuild/bazel/blob/973c4e7406a7d397f796414cdbc54dd7e082b68d/src/main/java/com/google/devtools/build/lib/syntax/MethodLibrary.java#L743).

So it has! Not sure how I missed that. Too much time in Go perhaps, which has no sorted sets.

The Go and Java implementations of dir both sort, and the Go docs specify it. This PR updates the spec:
https://github.com/adonovan/starlark/pull/new/sorted-dir


Update docs to not reference SYS1 PDS members

Is your request for enhancement related to a problem? Please describe.
The docs mention SYS1.PROCLIB and SYS1.PARMLIB in a number of places for the location to install Zowe JCL members into.
Describe the solution you'd like
We should not ask users to update SYS1 as this is a system library.  Instead we should tell them how to get the parmlib and proclib concatenation lists and to select user PDS members
Related doc pages
https://docs.zowe.org/stable/user-guide/scripted-configure-server.html#cross-memory-and-auxiliary-server
...

Hi @nannanli  and @Joe-Winchester i would like to work on this issue .Please assign me this.

Hi @nannanli and @Joe-Winchester can one of you please elaborate more so that i can have a good understanding about he bug.

An example of where we ask users to update SYS1.PARMLIB is where in
https://docs.zowe.org/stable/user-guide/scripted-configure-server/
we say
Executing the command zowe-install-xmem.sh -d MYUSERID.ZWE -a SYS1.PARMLIB -r USER.PROCLIB copies:
   the PARMLIB member MYUSERID.ZWE.SZWESAMP(ZWESIP00) to SYS1.PARMLIB(ZWESIP00)

The customer who was following these instructions had pointed out to me that SYS1.PARMLIB should be considered read only, i.e. at no point should we ask people to update it.  This is because it's a system library and can have its contents overwritten.  The preferred approach would be to say:
Executing the command zowe-install-xmem.sh -d MYUSERID.ZWE -a USER.PARMLIB -r USER.PROCLIB copies:
   the PARMLIB member MYUSERID.ZWE.SZWESAMP(ZWESIP00) to USER.PARMLIB(ZWESIP00)

Hi @nannanli  and @Joe-Winchester can you please check my PR. correct me if i am wrong.


[HOLD] Inconsistency between the behavior of the "next" button and the scrollability of steps 1 and 2.

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Action Performed:

Navigate to the "Settings" section.
Click the "Security" option.
Select the "Two-factor authentication" feature.
Noted that in step 1, the "next" button can be scrolled, and a scrollbar is visible from top to bottom.
In step 2, enter an invalid magic code to trigger the appearance of a scrollbar on the right.
Observe that the "next" button remains fixed at the bottom, and the scrollbar is positioned from the top to just above the button.

Expected Result:
There should be consistency between both steps, meaning that the behavior of the "next" button should be the same in both cases. It should either be fixed or scrollable in both steps.
Actual Result:
The observed behavior is inconsistent. In step 1, the "next" button can be scrolled, while in step 2, the "next" button remains fixed.
Workaround:
Can the user still use Expensify without this being fixed? Have you informed them of the workaround?
Platforms:

Which of our officially supported platforms is this issue occurring on?

[ ] Android / native
[ ] Android / Chrome
[ ] iOS / native
[ ] iOS / Safari
[x] MacOS / Chrome / Safari
[ ] MacOS / Desktop

Version Number: 1.3.25-4
Reproducible in staging?: y
Reproducible in production?: y
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
https://github.com/Expensify/App/assets/43996225/66cfe083-5284-43a6-a4e1-85e3027f7794
https://github.com/Expensify/App/assets/43996225/7f245a81-3c16-48d5-b4d8-8b70db2e3e6f
Expensify/Expensify Issue URL:
Issue reported by: @usmantariq96
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1685944613089219
View all open jobs on GitHub

@strepanier03 this is pretty much dupe of #20138, please add hold for now, thanks!

@strepanier03 Do I get the reporting bonus considering these are two different bugs?

@Santhosh-Sellavel - Thank you I will do that.
Is this GH you linked enough of a dupe that this GH should be closed or do you think it is different enough that the reporting bonus should be paid?

@strepanier03
This is just a dupe of the other one, only explained differently. So we can close this one straight away!


Memory leak when assigning to a bytearray via subscripting.

See the following code, we slice a bytearray object via subscripting, ironpython crashes. This test works well on CPython 3.9.0.
test.py

class X:
    def __index__(self):
        del b[0:0x10000]
        return 1

b = bytearray(b"A"*0x100)
b[::X()] = bytearray(b"B"*0x8000)

Error message:
>>'ironpython/ironpython3/bin/Debug/net6.0/ipy' test.py
.........
  at Microsoft.Scripting.Hosting.Shell.CommandLine.Run(ScriptEngine engine, IConsole console, ConsoleOptions options) in /home/xxm/Desktop/IFuzzer/experiment_on_different_interpreter/ironpython/ironpython3/Src/DLR/Src/Microsoft.Dynamic/Hosting/Shell/CommandLine.cs:line 100
   at Microsoft.Scripting.Hosting.Shell.ConsoleHost.RunCommandLine() in /home/xxm/Desktop/IFuzzer/experiment_on_different_interpreter/ironpython/ironpython3/Src/DLR/Src/Microsoft.Dynamic/Hosting/Shell/ConsoleHost.cs:line 386
   at Microsoft.Scripting.Hosting.Shell.ConsoleHost.ExecuteInternal() in /home/xxm/Desktop/IFuzzer/experiment_on_different_interpreter/ironpython/ironpython3/Src/DLR/Src/Microsoft.Dynamic/Hosting/Shell/ConsoleHost.cs:line 321
   at PythonConsoleHost.ExecuteInternal() in /home/xxm/Desktop/IFuzzer/experiment_on_different_interpreter/ironpython/ironpython3/Src/IronPythonConsole/Console.cs:line 165
   at Microsoft.Scripting.Hosting.Shell.ConsoleHost.Execute() in /home/xxm/Desktop/IFuzzer/experiment_on_different_interpreter/ironpython/ironpython3/Src/DLR/Src/Microsoft.Dynamic/Hosting/Shell/ConsoleHost.cs:line 299
   at Microsoft.Scripting.Hosting.Shell.ConsoleHost.Run(String[] args) in /home/xxm/Desktop/IFuzzer/experiment_on_different_interpreter/ironpython/ironpython3/Src/DLR/Src/Microsoft.Dynamic/Hosting/Shell/ConsoleHost.cs:line 197
   at PythonConsoleHost.Main(String[] args) in /home/xxm/Desktop/IFuzzer/experiment_on_different_interpreter/ironpython/ironpython3/Src/IronPythonConsole/Console.cs:line 199
Aborted (core dumped)

System info:
ironpython3-debug(main branch):IronPython 3.4.0b1 DEBUG (<IP_ADDRESS>0)[.NETCoreApp,Version=v6.0 on .NET 6.0.11 (64-bit)] on linux
operating system: Ubuntu 18.04.6 LTS

This one is interesting, I guess would need to break up our slice evaluation into two steps. CPython introduced C APIs to do this in 3.6.1 (PySlice_Unpack and PySlice_AdjustIndices). I'm guessing to fix this very issue?
Interestingly we can crash the latest CPython with a variation...
class X:
    def __index__(self):
        b.extend(b"aaa")
        return 2

b = bytearray(b"A"*10)
b[::X()] = b"c" * 8
b


Additional properties are not allowed ('functionCall' was unexpected)

Package Name: @azure/openai
Package Version: "1.0.0-beta.5"
Operating system: macos
[ ] nodejs

version:

[ ] browser

name/version:

[ ] typescript

version:

Is the bug related to documentation in

[ ] README.md
[ ] source code documentation
[ ] SDK API docs on https://docs.microsoft.com

Describe the bug
Using function calls with Azure Open AI:
Model name: gpt-35-turbo
Model version: 0613
Setting apiVersion to 2023-07-01-preview, can't send a message with functionCall param:
      // Step 4: send the info on the function call and function response to GPT
      const newMessages = [...messages];
      newMessages.push(responseMessage); // extend conversation with assistant's reply
      newMessages.push({
        role: "function",
        name: functionName,
        content: functionResponse,
      }); // extend conversation with function response
      const secondResponse = await client.getChatCompletions(
        "gpt-35-turbo",
        newMessages,
        {
          functions,
          functionCall: "auto",
        }
      );

Additional context
Full error message:
Error generating completion:  {
  message: "Additional properties are not allowed ('functionCall' was unexpected) - 'messages.1'",
  type: 'invalid_request_error',
  param: null,
  code: null
}

It works if you replace with function_name but then you get a typescript error

Hi @austintackaberry, thanks for opening this report! I tried the same scenario using the following sample and it works:
const { OpenAIClient, AzureKeyCredential } = require("@azure/openai");
require("dotenv").config();

const endpoint = process.env["ENDPOINT"] || "<endpoint>";
const azureApiKey = process.env["AZURE_API_KEY"] || "<api key>";

const messages = [{ role: "user", content: "What's the weather like in Boston?" }];

const getCurrentWeather = {
  name: "get_current_weather",
  description: "Get the current weather in a given location",
  parameters: {
    type: "object",
    properties: {
      location: {
        type: "string",
        description: "The city and state, e.g. San Francisco, CA",
      },
      unit: {
        type: "string",
        enum: ["celsius", "fahrenheit"],
      },
    },
    required: ["location"],
  },
};

async function main() {
  console.log("== Chat Completions Sample With Functions ==");

  const client = new OpenAIClient(endpoint, new AzureKeyCredential(azureApiKey), {
    apiVersion: "2023-07-01-preview"
  });
  const deploymentId = "gpt-35-turbo-0613";
  const result = await client.getChatCompletions(deploymentId, messages, {
    functions: [getCurrentWeather],
    functionCall: "auto"
  });

  for (const choice of result.choices) {
    console.log(choice.message?.functionCall);
  }
}

main();

And here is some logs from the run (by setting the environment variable AZURE_LOG_LEVEL to verbose):
== Chat Completions Sample With Functions ==
request body:
{
  messages: [ { role: 'user', content: "What's the weather like in Boston?" } ],
  functions: [
    {
      name: 'get_current_weather',
      description: 'Get the current weather in a given location',
      parameters: [Object]
    }
  ],
  function_call: 'auto',
  max_tokens: undefined,
  temperature: undefined,
  top_p: undefined,
  logit_bias: undefined,
  user: undefined,
  n: undefined,
  stop: undefined,
  presence_penalty: undefined,
  frequency_penalty: undefined,
  stream: undefined,
  model: undefined,
  dataSources: undefined
}
azure:core-rest-pipeline retryPolicy:info Retry 0: Attempting to send request 9acf5371-9ebb-4c41-b96b-340583fc1f7d
azure:openai:info Request: {
  "url": "https://sdk8428.openai.azure.com/openai/deployments/gpt-35-turbo-0613/chat/completions?api-version=2023-07-01-preview",
  "headers": {
    "accept": "application/json",
    "content-type": "application/json; charset=UTF-8",
    "accept-encoding": "gzip,deflate",
    "user-agent": "azsdk-js-openai-rest/1.0.0-beta.5 core-rest-pipeline/1.12.0 Node/v19.9.0 OS/(x64-Linux-5.15.0-1041-azure)",
    "x-ms-client-request-id": "9acf5371-9ebb-4c41-b96b-340583fc1f7d",
    "api-key": "REDACTED"
  },
  "method": "POST",
  "timeout": 0,
  "disableKeepAlive": false,
  "withCredentials": false,
  "requestId": "9acf5371-9ebb-4c41-b96b-340583fc1f7d",
  "allowInsecureConnection": false,
  "enableBrowserStreams": true
}
azure:core-rest-pipeline:info No cached TLS Agent exist, creating a new Agent
azure:openai:info Response status code: 200
azure:openai:info Headers: {
  "cache-control": "no-cache, must-revalidate",
  "content-length": "659",
  "content-type": "application/json",
  "access-control-allow-origin": "*",
  "apim-request-id": "5e1f0753-5c3f-48d0-b08d-eb4962b2f9e6",
  "openai-model": "gpt-35-turbo",
  "x-content-type-options": "nosniff",
  "openai-processing-ms": "1224.0593",
  "x-ms-region": "East US",
  "x-accel-buffering": "no",
  "x-request-id": "aa70e886-e27d-4a2e-8b3a-e8d63a4f182b",
  "x-ms-client-request-id": "9acf5371-9ebb-4c41-b96b-340583fc1f7d",
  "strict-transport-security": "max-age=31536000; includeSubDomains; preload",
  "azureml-model-session": "turbo-669058-1",
  "azureml-model-group": "online",
  "date": "Thu, 07 Sep 2023 18:19:19 GMT"
}
azure:core-rest-pipeline retryPolicy:info Retry 0: Received a response from request 9acf5371-9ebb-4c41-b96b-340583fc1f7d
azure:core-rest-pipeline retryPolicy:info Retry 0: Processing 2 retry strategies.
azure:core-rest-pipeline retryPolicy:info Retry 0: Processing retry strategy throttlingRetryStrategy.
azure:core-rest-pipeline retryPolicy:info Retry 0: Skipped.
azure:core-rest-pipeline retryPolicy:info Retry 0: Processing retry strategy exponentialRetryStrategy.
azure:core-rest-pipeline retryPolicy:info Retry 0: Skipped.
azure:core-rest-pipeline retryPolicy:info None of the retry strategies could work with the received response. Returning it.
{
  name: 'get_current_weather',
  arguments: '{\n  "location": "Boston, MA"\n}'
}

Could you double check you're using<EMAIL_ADDRESS>

Yeah so making the initial call with functions in your example works as expected but if you follow the rest of the example and you try to pass in the function call response generated from openai back into the messages array for another completion, that's when you get the failure.
From openai docs:
import OpenAI from "openai";
const openai = new OpenAI();

// Example dummy function hard coded to return the same weather
// In production, this could be your backend API or an external API
function getCurrentWeather(location, unit = "fahrenheit") {
    const weatherInfo = {
        "location": location,
        "temperature": "72",
        "unit": unit,
        "forecast": ["sunny", "windy"],
    };
    return JSON.stringify(weatherInfo);
}

async function runConversation() {
    // Step 1: send the conversation and available functions to GPT
    const messages = [{"role": "user", "content": "What's the weather like in Boston?"}];
    const functions = [
        {
            "name": "get_current_weather",
            "description": "Get the current weather in a given location",
            "parameters": {
                "type": "object",
                "properties": {
                    "location": {
                        "type": "string",
                        "description": "The city and state, e.g. San Francisco, CA",
                    },
                    "unit": {"type": "string", "enum": ["celsius", "fahrenheit"]},
                },
                "required": ["location"],
            },
        }
    ];

    const response = await openai.chat.completions.create({
        model: "gpt-3.5-turbo",
        messages: messages,
        functions: functions,
        function_call: "auto",  // auto is default, but we'll be explicit
    });
    const responseMessage = response.choices[0].message;
    // Step 2: check if GPT wanted to call a function

^^^ that works fine
This breaks at Step 4:
    if (responseMessage.function_call) {
        // Step 3: call the function
        // Note: the JSON response may not always be valid; be sure to handle errors
        const availableFunctions = {
            get_current_weather: getCurrentWeather,
        };  // only one function in this example, but you can have multiple
        const functionName = responseMessage.function_call.name;
        const functionToCall = availableFunctions[functionName];
        const functionArgs = JSON.parse(responseMessage.function_call.arguments);
        const functionResponse = functionToCall(
            functionArgs.location,
            functionArgs.unit,
        );

        // Step 4: send the info on the function call and function response to GPT
        messages.push(responseMessage);  // extend conversation with assistant's reply
        messages.push({
            "role": "function",
            "name": functionName,
            "content": functionResponse,
        });  // extend conversation with function response
        const secondResponse = await openai.chat.completions.create({
            model: "gpt-3.5-turbo",
            messages: messages,
        });  // get a new response from GPT where it can see the function response
        return secondResponse;
    }
}

runConversation().then(console.log).catch(console.error);

This is the problematic code:
        messages.push(responseMessage);  // extend conversation with assistant's reply

The responseMessage includes a property functionCall which is not supported

I second this bug report.
Expectation is that you can push the assistant's response message onto your message history array and pass it back unmodified. This pairs the original request for a function call from the assistant with the results of that call.
Reality is that you have to change the field name from functionCall to function_call, which requires you to ignore the typescript error which results from the mismatch of the type and the actual parameter name.

Thank you so much for raising this bug report. We are working on a fix and will ship it in the next release.

Thank you for raising the issue. It has been addressed in <EMAIL_ADDRESS>


GOBBLIN-559: Implement FlowGraph as a concurrent data structure.

GOBBLIN-559: Implement FlowGraph as a concurrent data structure.
Dear Gobblin maintainers,
Please accept this PR. I understand that it will not be reviewed until I have checked off all the steps below!
JIRA

[x] My PR addresses the following Gobblin JIRA issues and references them in the PR title. For example, "[GOBBLIN-XXX] My Gobblin PR"

https://issues.apache.org/jira/browse/GOBBLIN-559.

Description

[x] Here are some details about my PR, including screenshots (if applicable):
Current implementation of FlowGraph only allows a single-threaded access to the data structure for reads and writes. We improve the implementation by allowing the readers to not block on other readers, and only block on a write.

Tests

[x] My PR adds the following unit tests OR does not need testing for this extremely good reason:
Existing unit tests in MultiHopFlowCompilerTest class.

Commits

[x] My commits all reference JIRA issues in their subject lines, and I have squashed multiple commits if they address the same issue. In addition, my commits follow the guidelines from "How to write a good git commit message":

Subject is separated from body by a blank line
Subject is limited to 50 characters
Subject does not end with a period
Subject uses the imperative mood ("add", not "adding")
Body wraps at 72 characters
Body explains "what" and "why", not "how"

@zxcware Please review this PR.


tabBar: Pass borderless from props to TouchableItem.

Before always true was passed.

Motivation
We are using this and was facing issue of extended ripple effect. Because always true was passed. So pass it from the the props, giving control to the user.

Test plan
Pass borderless: false, or borderless: true, in the tabBarOptions.

Updated!
Now checks are passing.
@satya164 please review it, when you get some time :)

Hey, I just released a new alpha 2.0.0-alpha.0 of the library. It's rewritten using react-native-gesture-handler and react-native-reanimated addresses a many platform specific bugs and performance problems. It's gone through a major refactor and there will be conflicts with your PR.
Please try the new version and see if it addresses your issue. If not, please open a new PR with the change.

I'm still seeing the issue (in the same project as @jainkuniya) with the rewritten version, react-native-tab-view 2.15.1. However, I don't see it in the react-native-tab-view example app. I could repeat this PR, but I'm thinking there may be pretty substantial changes with the new Pressable API with RN v0.63, and it might naturally be solved at that point. In the meantime, we have a workaround, setting borderWidth and elevation to zero (see zulip/zulip-mobile@29d1da6c28fb5e31426c3602acc186d6f9fe8e0e).


Travis CI Won't Run Tests Requiring MS SQL Server

We are ignoring the Include tests because they require MS SQL Server, which is not supported directly on Linux.

One idea is to use SQL Server for Linux in order to run those tests in CI.

Deferring this for now.


[Bug]: The traceid slice stored in memory was not properly initialized

What happened?
The traceid slice stored in memory was not properly initialized，ids slices should only have capacity
Steps to reproduce
look here
func newTenant(cfg config.Configuration) *Tenant {
return &Tenant{
ids:        make([]*model.TraceID, cfg.MaxTraces),
traces:     map[model.TraceID]*model.Trace{},
services:   map[string]struct{}{},
operations: map[string]map[spanstore.Operation]struct{}{},
deduper:    adjuster.SpanIDDeduper(),
config:     cfg,
}
}
Expected behavior
ids slices should only have capacity
Relevant log output
No response
Screenshot
func newTenant(cfg config.Configuration) *Tenant {
return &Tenant{
ids:        make([]*model.TraceID, cfg.MaxTraces),
traces:     map[model.TraceID]*model.Trace{},
services:   map[string]struct{}{},
operations: map[string]map[spanstore.Operation]struct{}{},
deduper:    adjuster.SpanIDDeduper(),
config:     cfg,
}
}
Additional context
No response
Jaeger backend version
No response
SDK
No response
Pipeline
No response
Stogage backend
No response
Operating system
os
Deployment model
No response
Deployment configs
No response

/

not sure what this is about. Needs more details.


[DCOS-2435] Improve error message when an app is locked by another deployment.

One naming change but otherwise looks good, thanks for fixing this!

Great, merge after successful build please.


[rtlsdr] Add new port

What does your PR fix? Fixes issue #
Nothing, this adds a new port for rtl-sdr - software that turns a Realtek RTL2832 based DVB dongle into a SDR receiver (https://osmocom.org/projects/rtl-sdr)

Which triplets are supported/not supported?  Not yet.

Does your PR follow the maintainer guide?
Yes.

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.:x: echesakov sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.

Hi @echesakov, thanks for this PR!
Please sign CLA first.
Thanks.

Please sign CLA first.

@JackBoosY I did, can you please re-run the ci?

/azp run

fatal: unable to connect to git.osmocom.org:
git.osmocom.org[0: <IP_ADDRESS>]: errno=Unknown error

/azp run

We firewall the git protocol at the moment (we allow only HTTP because we don't want a rogue PR to be able to turn our test fleet into spambots and that sort of thing). Is there an HTTP endpoint we can use or do we have to open that up?

#10980 will open that port

I have update the PR to use GitHub mirror https://github.com/osmocom/rtl-sdr

/azp run

/azp run

/azp run

Alright, this looks good! Thanks @echesakov :)


[GHSA-49gw-vxvf-fc2g] The various Is methods (IsPrivate, IsLoopback, etc) did...

Updates

Affected products
Summary

Comments
The package name and affected versions are not showing on the github advisory. I believe this might be the reason for which dependabot did not flag it.
I'm also not sure how to specify that the issue affects go 1.22.0 through 1.22.4

👋 Hi @salv-orlando, unfortunately, we can't review GHSA-49gw-vxvf-fc2g and issue alerts because GHSA-49gw-vxvf-fc2g concerns an issue in the Go standard library, rather than a package in pkg.go.dev. However, downstream users of the library are welcome to publish advisories about how CVE-2024-24790 affects their packages, as the maintainers of Traefik did in GHSA-7jmw-8259-q9jx, and send alerts to users that way.
Thank you for your interest in GHSA-49gw-vxvf-fc2g and have a good week!

Got it @shelbyc - thanks for reviewing the suggestion!


Howto mTLS

I don't know how to consume a OData-Service that is protected by mTLS.
I already have an Bearer-Token. Now I have to use mTLS using the DefaultHttpDestination.Builder.
We are using the httpClient 4 !
Where can I find some documentation?
Here my fist approach:
private Destination buildDestinationWithMTLSAuth(String name) throws Exception {
        String token =   ... fetch bearer token form token endpoint
        LocalDateTime validUntil =  ... served by token endpoint

        String url = ...// https://xxxxxx.yyyy.apimanagement.eu20.hana.ondemand.com:443
        String keyStorePassword = ... the keystore password;
        String applicationKey = ... the api key;

        DefaultHttpDestination.Builder destinationBuilder = DefaultHttpDestination
                .builder(url)
                .authenticationType(AuthenticationType.CLIENT_CERTIFICATE_AUTHENTICATION)
                // this enables mTLS
                .securityConfiguration(SecurityConfigurationStrategy.FROM_PLATFORM)
                .keyStore(getClientKeyStore(keyStorePassword)) // exactly one client-cert in KeyStore
                .keyStorePassword(keyStorePassword) // really needed once more?
                .trustAllCertificates() // should be removed later, setting the trustStore ...
                .property(VALID_DATE_UNTIL, validUntil) // Bearer token valid until
                .header("apikey", applicationKey)
                .header("Authorization", "Bearer " + token);
        HttpDestination destination = destinationBuilder.build();
        return destination;
    }

    private KeyStore getClientKeyStore(String keyStorePassword) throws Exception {
        String pathToKeyStore = getProperty("PathToKeyStore");

        final KeyStore keyStore = KeyStore.getInstance("PKCS12");
        keyStore.load(new FileInputStream(pathToKeyStore), keyStorePassword.toCharArray());
        return keyStore;
    }

Sadly currently the server-certificate is expired, so I cannot test accordingly.
I just want to be assure, that the above code works as expected.
I think I somehow have to use CF_INSTANCE_CERT and CF_INSTANCE_KEY, but I would like to deal without system properties.
Please also add an example for trustStore to validate the server-certificate.
How do I put the server-cert into the trustStore (I can fetch the certificate via Browser).
Impact / Priority
Must be solved before end of august 2024.
Maven
        <!-- SAP Cloud SDK -->
        <dependency>
            <groupId>com.sap.cloud.sdk.datamodel</groupId>
            <artifactId>odata-core</artifactId>
            <version>5.10.0</version>
        </dependency>
        <dependency>
            <groupId>com.sap.cloud.sdk.datamodel</groupId>
            <artifactId>odata-v4-core</artifactId>
            <version>5.10.0</version>
        </dependency>
        <dependency>
            <groupId>org.projectlombok</groupId>
            <artifactId>lombok</artifactId>
            <version>1.18.32</version>
            <scope>provided</scope>
        </dependency>
        <dependency>
            <!-- generated sources-->
            <groupId>com.sap.cloud.sdk.datamodel</groupId>
            <artifactId>odata-generator</artifactId>
            <version>5.10.0</version>
        </dependency>
        <dependency>
            <!-- oauth -->
            <groupId>com.sap.cloud.sdk.cloudplatform</groupId>
            <artifactId>connectivity-oauth</artifactId>
            <version>5.10.0</version>
        </dependency>
        <dependency>
            <groupId>jakarta.xml.bind</groupId>
            <artifactId>jakarta.xml.bind-api</artifactId>
            <version>4.0.2</version>
        </dependency>

Hi Oliver, please clarify which certificate you want to use as client certificate for the communication, and where this certificate is coming from.
Using SecurityConfigurationStrategy.FROM_PLATFORM) together with .keyStore(..) doesn't make sense. The former will use the Cloud Foundry Instance Identity while the latter will use the provided certificate. If you give both, the latter will be ignored completely.
Furthermore, the Cloud SDK is first and foremost designed to use destinations from the BTP destination service. In case you are not using the destination service, where do the credentials, including the certificate, come from?
Finally, the trust store should rarely require changes. If you are connecting to any SAP-managed server this should not be required. In case you do need changes, the Cloud SDK respects the default trust store that is installed on the machine your application is running on. Alternatively, you can use the API to manually set the trust store. Please refer to the official documentation for java and other online material on how to work with trust stores (and keystores, for that matter).

Hi Matthias, our OData-Client is executed from inside a WebApp deployed on a Tomcat-Instance.
The Tomcat-Instance works on-prem.
We managed to authenticate with OAuth2 yet. But our customer wants to switch to mTLS.
I searched around several hours, but I could not find an example.
The sourcecode of cloud-sdk-java does not contain a JUnit-Test that helped me.
Please provide an example! My first approach is just an helpless first shot.
The client certificate is provided by our customer. It is contained in a Java keystore.

I'm sorry, but you'll have to better explain your setup. From your description I can't tell what you are trying to do.

I just need to have a destination that supports mTLS.

DefaultHttpDestination does that. Nothing more I can say without knowing what you want to do.


multiple commands

do this allow running multiple commands, if so, how?
thanks,

i meant, say, run ls -lah before caddy is started. can you currently do that when the Dockerfile has:
ENTRYPOINT ["/bin/parent", "caddy"]
CMD ["--conf", "/etc/Caddyfile", "--log", "stdout", "--agree=$ACME_AGREE"]

so, i'd like say ls -alh /root/.caddy && caddy.

In that case, you'd override the entrypoint. e.g.
docker run -it --entrypoint sh abiosoft/caddy -c "ls -lah /root/.caddy && caddy"

that's not ideal, but thanks. i was hoping you might add support for it in the Dockerfile.


RustConnection::poll_for_event never reads from the socket

Currently, there is no way to get events in a non-blocking way. Only wait_for_event() actually reads from the socket.
I guess poll_for_event() has to somehow check if there is currently a reader and if not, try to read. No idea how this could fit into the Read trait.

I'm afraid this would require non-blocking reading...

Which is still compatible with the Read trait (and thus also with ReadFD), I think. We'd just have to require a non-blocking behaviour of the implementation.

How important is to have RustConnection generic over BufReadFD and BufWriteFD instead of just using Stream?

I am not quite sure what you are asking here. For the sake of argument, let's say that all generic arguments should be removed. What exactly would you propose to do? (I guess one can still add back generics to any solution we come up with)
Right now, the only solution that I see would be to require non-blocking reading. Then, we could add another generic something to RustConnection that abstract over poll. This "something" could then also be implemented on Windows (is there something similarly nice to the nix crate for windows? A quick Google search led me to https://docs.rs/ws2_32-sys/0.2.1/i686-pc-windows-gnu/ws2_32/fn.WSAPoll.html which is unsafe...)

I am not quite sure what you are asking here. For the sake of argument, let's say that all generic arguments should be removed. What exactly would you propose to do? (I guess one can still add back generics to any solution we come up with)

I just thought it would be easier implement without generics (e.g., using set_nonblocking). Of course, we could create a trait that provides that method.

This "something" could then also be implemented on Windows (is there something similarly nice to the nix crate for windows? A quick Google search led me to https://docs.rs/ws2_32-sys/0.2.1/i686-pc-windows-gnu/ws2_32/fn.WSAPoll.html which is unsafe...)

That's why I had proposed to use mio, which solves the portability issue (but would likely require to remove the generic parameters).

You should look at how smol does this
(https://github.com/stjepang/smol). iirc on Unix systems, it suffices
for the Read object to implement AsRawFd, and you can set the
nonblocking mode on the fd; on Windows systems it is more complicated.

Thinking better about it, no matter what library we use, RustConnection can be kept generic with a trait like this one...
trait NonBlockingStream: ReadFD + WriteFD {
    fn poll(&mut self, block: bool) -> io::Result<PollEvents>;
}

Then, we could add another generic something to RustConnection that abstract over poll.

... which I believe that is what you meant here. But in this case it joins the reader and writer into a single object.

Thanks for the ideas.

iirc on Unix systems, it suffices
for the Read object to implement AsRawFd, and you can set the
nonblocking mode on the fd; on Windows systems it is more complicated.

Seems like it. The relevant magic is here, I would say:
https://github.com/stjepang/smol/blob/2071f2a481e64459a99173b384afb33f12c18068/src/reactor.rs#L103-L114
I was thinking about something like this (very rough sketch)
trait Poll<T, R, W> {
    // This should be an abstraction over AsRawFd and AsRawSocket, with T being either RawFd or Socket
    fn extract_token(&self, either_R_or_W) -> T;
    // This would be the actual poll() call
    fn poll(&self, read: &[T], write: &[T]) -> dunno;
}

RustConnection would then also get an implementation of the above trait. (I guess T needs to be Send + Sync and perhaps even 'static? Perhaps it also makes more sense to split this up into two traits and what not, but this is the general idea that I have floating around so far).
Perhaps it makes more sense to just reinvent the whole PollFd thing as a trait instead and hope for the compiler to optimise the extra level of abstraction away: https://docs.rs/nix/0.17.0/nix/poll/fn.poll.html
Perhaps more realistic:
/// An abstraction over `AsRawFd` and `AsRawSocket`
trait AsToken {
    type Token;
    fn as_token(&self) -> Token;
}
/// An abstraction over `poll()` and `WSAPoll()`.
trait Poll {
    type Token;
    fn poll(&self, our-own-copy-of-the-poll-API) -> poll-return;
}

RustConnection would then end up very complicated:
struct RustConnection<R, W, P, T>
where
    R: ReadFd,
    R: AsToken<Token=T>,
    W: WriteFd,
    R: AsToken<Token=T>,
    P: Poll<Token=T>,
    T: Send + Sync + 'static // or whatever is needed for it

Oh and the API docs would then just mention that R and W have to behave non-blocking-ly.
For Windows, one could then use WSAPoll for basically the same result.

After some more thinking... I am inclined to agree that all the generic stuff could be removed. It would actually simplify lots of things. I am a bit sad about it (since it e.g. means that the reading test needs to be removed), but perhaps it is better this way.
What I just noticed: #222 cannot really be sanely fixed without integrating the reading and writing parts a lot more. At that point, a lot of the genericity would be lost.
Perhaps it is best to remove the generic arguments for now, fix this issue and #222 and only afterwards start thinking about how a generic version of this could look like.

I hacked together a quick demo on how things could work like: https://gist.github.com/psychon/8c12a9b201b56c72c44a2a832ae1dce4
This uses poll via the nix crate. It does not have the problem of #222: It always poll()s for reading whenever it wants to write.
After having written this... well. Actionable items would be: Stop using APIs like write_all and write_vectored_all. They just cannot work in non-blocking mode.
We want multi-thread support, so writing should be separate from reading. Currently, we achieve this with TcpSocket::try_clone(). This gives us one handle for reading and one for writing.
At the same time, #222 should be fixed, so writing also needs to be able to read. So... I have no idea how integrate this with Rust. Who borrows what when? I imagine the API for writing gets the read half as an argument... or something.
I guess I'll experiment with a unix-only API that uses RawFd instead of something fancy like TcpStream.

Thanks to https://github.com/BurntSushi/winapi-util/pull/4, I had the following idea: Just add a set_nonblocking() API to ReadFD. Normally, the reader is in blocking mode and only during poll_for_event() is it configured to be non-blocking.
That should be a lot simpler to implement than what I hacked up in #419.


special case rpb1

http://dev.pombase.kmr.nz/gene_protein_features/SPBC28F2.12
we have special allele naming syntax for the rpb1  (and another protein?)
@manulera  can poit you to this. These will be 'conjoined' alleles
(not necessary for release)

we have special allele naming syntax for the rpb1 (and another protein?)

The allele parsing for amino_acid_mutation is very simple at the moment.  There are quite a few that are ignored.  I've been meaning to chat about it on a call.
rpb1 has descriptions like these that are ignored:

https://www.pombase.org/genotype/rpb1-CTD-S5.S5A-(CTD)S5A(alternating)-amino_acid_mutation-expression-not_assayed_or_wild_type
https://www.pombase.org/genotype/rpb1-CTD-S2A-(CTD)S2A-amino_acid_mutation-expression-not_assayed_or_wild_type
https://www.pombase.org/genotype/rpb1-CTD-S2A(5-16),CTDdelta(17-29)-CTD-S2A(5-16),CTDdelta(17-29)-amino_acid_deletion_and_mutation-expression-not_assayed_or_wild_type

spt5 also has some that are ignored:

https://www.pombase.org/genotype/spt5-(CTD)T1A(7)-(CTD)T1A(7)-amino_acid_mutation-expression-not_assayed_or_wild_type
https://www.pombase.org/genotype/spt5-(T1A)7-CTD-T1A-amino_acid_mutation-expression-not_assayed_or_wild_type

Yes, rpb1 and spt5 are the CTD ones, for now those should be ignored.

Is this covered by?:

pombase/pombase-chado#1141

Similar, this ticket is for the alleles ...

do not do until
https://github.com/pombase/pombase-chado/issues/1192

The seems a bit more challenging than the modifications because of alleles like this:

rpb1-CTD-S2A(5-16),CTDΔ(17-29)(CTD-S2A(r5-r16),delta(r17-r29))

OK, we should have a chat about this sometime and decide which of these to show:
https://www.pombase.org/gene_alleles/SPBC28F2.12


Casbin dashboard UI inconsistency in the running port and authentication problem

What is supposed to be running port ?
It starts on 3000, but when authentication against github is performed it redirects to http://localhost:3030/callback
When I changed all references of 3030 to 3000 it seems that it works fine, however there is a problem in the authorization flow
It seems that https://api.github.com/user/emails requires that token is in the authorization header

Authentication problem will be resolved with https://github.com/casbin/casbin-dashboard/pull/139
However ports are not consistent in the web app.

@mteodor If you are not using Windows, you probably start with wrong port: 3000. In this case, you need to change the following code to use port: 3030
https://github.com/casbin/casbin-dashboard/blob/6a0b240226ff173030c632594adad8f134e14f4f/web/package.json#L18-L24


Typo on http://www.ponyos.org/ - "works many different machines."

Hi!
There's a typo on http://www.ponyos.org/ :
"works many different machines." should be "works on many different machines".
Please correct it.
Regards,
-- @shlomif

Thanks for correcting it. ♥!


C/C++ Standard Lib for contracts (part 1)

Not all of it works so far but the basic framing is there and all of our current contracts compile and tests run.
will continue to improve

Build error on Travis:
ninja: error: '../contracts/eosiolib/eosiolib.bc', needed by 'contracts/eosio.system/eosio.system.bc', missing and no known rule to make it

Something seems wrong in the cmake rules. A make -j8 will fail the first couple of times but will finally make it through after a few failed builds.

the new changes from @moskvanaft have resolved the build issues I've seen locally.   Waiting on travis and/or @spoonincode and/or @heifner  to confirm

Worked For Me


[Fix #11117] Fix a false positive for Style/BlockDelimiters

Fixes #11117.
This PR fixes a false positive for Style/BlockDelimiters when specifying EnforcedStyle: semantic and using a single line block with {} followed by a safe navigation method call.

Before submitting the PR make sure the following are checked:

[x] The PR relates to only one subject with a clear title and description in grammatically correct, complete sentences.
[x] Wrote good commit messages.
[x] Commit message starts with [Fix #issue-number] (if the related issue exists).
[x] Feature branch is up-to-date with master (if not - rebase it).
[x] Squashed related commits together.
[x] Added tests.
[x] Ran bundle exec rake default. It executes all tests and runs RuboCop on its own code.
[x] Added an entry (file) to the changelog folder named {change_type}_{change_description}.md if the new code introduces user-observable changes. See changelog entry format for details.

Thanks!


Upgrade to Java 11

Java 8 is deprecated and Java 11 is the latest long-term support version. The new Wicket release will use it by default. So this project should update its Java version as well.
The following aspects need to be considered:

[x] TravisCI
[x] Development environment
[x] Gradle build and test
[x] IDE
[x] README file

Done.

The way how it was done is described here:
https://dev.to/janux_de/report-about-updating-a-java-project-from-jdk-8-to-jdk-11-2klf


Edit notebook title by clicking it

Please allow for editing the notebook title by clicking the the tile beside "Beaker" in the top header.

That's an interesting idea.  It's not just a title though it's actually the file name, so if you change it, that would mean saving the notebook to a new file, is that what you want?  If you click it, we could just bring up the save as dialog...

Hm, I see. For me, I'm not thinking about the notebooks as files -- they're their own entity. Really I would rather Beaker abstract the file handling from me initially -- e.g., self manage files in a .beaker folder or something, and then when I want to export a file, let me save it as whatever I wanted to call it.

I appreciate what you are saying and we might get there some day, but currently we are sticking with files.


Rewrite everything to include epochs logic.

Disclaimer: There's more work to be done, especially on tests.


Draw.MolToFile not working (still)

The issue about Draw.MolToFile failing in 2019.03.01 (#3046) still appears in 2019.03.06

The issue was fixed in 2020.09.1b1


Added helpful comments to readme for how to get and use glide on Windows

Coverage remained the same at 91.331% when pulling 5ec74848450dd6e197c80efc50cc7cfc0d16cf73 on LispyAriaro:master into<PHONE_NUMBER>379acbdbe433c41f48e110c7e7b0fe on qiangxue:master.

Thanks for the contribution! This PR no longer applies because the new version is using go mod now.


fixes bad jp2 filenames

The filenames in the ALTO files are wrong, so the code that derives the TEI  section is generating the wrong filename for the  elements. This patch corrects the filenames in the TEI by truncating the 8.3 to the 4.3 format that matches the actual files.

Merged the pull request. This seems like a reasonable way to work.


Fix metadata update after dataset transformation

The metadata was not correctly updated after a dataset transformation.

Pitch dimensions were set to None when transforming the orientation
The coordinate system was not updated

Thanks for this PR! It would be great if you can add or adjust a test that breaks with previous implementation and is successful with your fix

I did that. See my changes to kloppy/tests/test_helpers.py where I extended the existing tests to make sure the added metadata is correct.
Or do you want me to split up the PR in two commits, one containing the failing tests and one containing the fix?


Merging of inline styles not always picked up

Describe the bug
I am trying to create a custom Text component with restricted styles.
It seems like nativewind doesn't like the merging of inline styles. When I make changes to the props, it doesn't always apply the new style. A refresh of the app sometimes works. A full restart with react-native start --reset-cache sometimes works but not always either. Replacing the variables with hard-coded values with a restart of the app always works and the styles gets picked up.
It seems the caching is very strong, or the transpiler is confused with the merged inline styles and they aren't always picked up.
With the following component
To Reproduce
This is my custom component:
import React from 'react';
import { Text as RNText, TextProps } from 'react-native';

type Props = Omit<TextProps, 'style'> & {
  size?: 'base' | 'sm' | 'lg';
  weight?: 'regular' | 'medium' | 'bold';
  color?: 'grey-600' | 'orange-500';
  className?: never;
};

const Text = ({
  size = 'base',
  color = 'orange-500',
  weight = 'regular',
  ...props
}: Props) => (
  <RNText
    className={`text-${size} text-${color} font-family-${weight}`}
    {...props}
  />
);

export default Text;

And this is how I use it:
import Text from './Text'

export default () => (
  <Text size="lg">Hello world</Text>
)

Expected behavior
Merged inline styles should always be picked up without needing to restart react native bundler.

+1 experiencing this issue as well

This is because tailwind does not allow "Dynamic class names", as documented here.

@langtube is correct - Tailwind CLI, the tool that compiles the styles does not allow dynamic styles this manner.
You can achieve this if you create a prop map, eg:
const textSizes = {
  base: "text-base",
  sm: "text-sm"
  ...
}

However, writing components like that goes against the advantages of Tailwind. Eg what happens when you need a text with arbitrary size?
A better way to write this component would be
const MyText = styled(Text, "text-base text-orange-500 font-regular")

Then just apply the overrides when you need to
<MyText className="text-medium" />
<MyText className="text-grey-500" />

When writing a component library with strict rules, you don't want arbitrary values but only want to allow specific values.  Which is what I've tried to do above with typescript. This is to prevent developers from diverging from the design system.
I'll give a try at using the prop map 👍  thanks


ddev import-files does not extract symlinks

Is there an existing issue for this?

[X] I have searched the existing issues

Output of ddev debug test
Expand `ddev debug test` diagnostic information
======= Existing project config =========
These config files were loaded for project xxx-typo3: [/home/mustermann/code/xxx/xxx-typo3/.ddev/config.yaml]
name: xxx-typo3
type: typo3
docroot: .
php_version: 7.2
webserver_type: nginx-fpm
webimage: ddev/ddev-webserver:v1.22.6
additional_hostnames: [xxx.xxx-typo3 xxx.xxx-typo3 xxx.xxx-typo3]
additional_fqdns: []
database: {mariadb 10.4}
project_tld: ddev.site
use_dns_when_possible: true
composer_version: 2
nodejs_version: 18
default_container_timeout: 120
======= Creating dummy project named  tryddevproject-8540 in ../tryddevproject-8540 =========
OS Information: Linux station 6.6.6-76060606-generic #202312111032~1702306143~22.04~d28ffec SMP PREEMPT_DYNAMIC Mon D x86_64 x86_64 x86_64 GNU/Linux
User information: uid=1000(mustermann) gid=1000(mustermann) Gruppen=1000(mustermann),4(adm),27(sudo),124(lpadmin),138(libvirt),142(wireshark),999(docker)
DDEV version:  ITEM             VALUE
 DDEV version     v1.22.6
 architecture     amd64
 db               ddev/ddev-dbserver-mariadb-10.4:v1.22.6
 ddev-ssh-agent   ddev/ddev-ssh-agent:v1.22.6
 docker           24.0.7
 docker-compose   v2.23.3
 docker-platform  linux-docker
 mutagen          0.17.2
 os               linux
 router           ddev/ddev-traefik-router:v1.22.6
 web              ddev/ddev-webserver:v1.22.6
PROXY settings: HTTP_PROXY='' HTTPS_PROXY='' http_proxy='' NO_PROXY='localhost,<IP_ADDRESS>/8,::1'
======= DDEV global info =========
Global configuration:
instrumentation-opt-in=true
omit-containers=[]
performance-mode=none
router-bind-all-interfaces=false
internet-detection-timeout-ms=3000
disable-http2=false
use-letsencrypt=false
letsencrypt-email=
table-style=default
simple-formatting=false
use-hardened-images=false
fail-on-hook-fail=false
required-docker-compose-version=v2.23.3
use-docker-compose-from-path=false
project-tld=
xdebug-ide-location=
no-bind-mounts=false
router=traefik
wsl2-no-windows-hosts-mgt=false
router-http-port=80
router-https-port=443
mailpit-http-port=8025
mailpit-https-port=8026
traefik-monitor-port=10999

======= DOCKER info =========
docker location: -rwxr-xr-x 1 root root 35939040 Okt 26 11:07 /usr/bin/docker
docker version:
Client: Docker Engine - Community
 Version:           24.0.7
 API version:       1.43
 Go version:        go1.20.10
 Git commit:        afdd53b
 Built:             Thu Oct 26 09:07:41 2023
 OS/Arch:           linux/amd64
 Context:           default

Server: Docker Engine - Community
 Engine:
  Version:          24.0.7
  API version:      1.43 (minimum version 1.12)
  Go version:       go1.20.10
  Git commit:       311b9ff
  Built:            Thu Oct 26 09:07:41 2023
  OS/Arch:          linux/amd64
  Experimental:     false
 containerd:
  Version:          1.6.27
  GitCommit:        a1496014c916f9e62104b33d1bb5bd03b0858e59
 runc:
  Version:          1.1.11
  GitCommit:        v1.1.11-0-g4bccb38
 docker-init:
  Version:          0.19.0
  GitCommit:        de40ad0
DOCKER_DEFAULT_PLATFORM=notset

Expected Behavior
The command should import the contents of the given tar.gz
Actual Behavior
The command imports files, but not symlinks
Steps To Reproduce

Start TYPO3 project
Import tar.gz file that only contains a symlink
The path fileadmin/ ends up empty

Anything else?
I thought this was a good way to go around the limitation in https://github.com/ddev/ddev/issues/5710, but in the end, this is also a no-go.

Linking the relevant code base:
https://github.com/ddev/ddev/blob/94d77509a582d92b5d975be97e84c66e0ad0a6de/pkg/archive/archive.go#L237-L238

I don't think symlinks are always supported in tarballs anyway. As suggested in

#5710

I think you'll want to just make your own custom command, either global or project.


 tag improvement

Currently, the D tag uses an attribute 'n', so <D n="67" a="3384"> identifes that the translation following is for saying 67.

The 'a' attribute is (I think!) this saying is numbered as 3384 in the first edition of Indische Spruch.  Recall our document is baed on the 2nd edition of I.S.

Also, in the 151 entries that contain multiple sayings, there are multiple translation D elements, e.g.
<D n="67" a="3384">
<D n="68" a="3385">
<D n="69" a="3386">

but the sayings themselves are not similarly identified separately, e.g.
<info L="67,68,69" page="1.13" gtypes="S,D,D,D,F"/>
<S>
agnistejo mahalloke gUDhastiSThati dAruSu |
na copayuGkte taddAru yAvannoddIpyate paraiH ||
sa eva khalu dArubhyo yadA nirmathya dIpyate |
taddAru ca vanaM cAnyannirdahatyAzu tejasA ||
evameva kule jAtAH pAvakopamatejasaH |
kSamAvanto nirAkArAH kASThe 'gniriva zerate ||
</S>

By S-tag improvement we would rewrite this example as
<S n="67">
agnistejo mahalloke gUDhastiSThati dAruSu |
na copayuGkte taddAru yAvannoddIpyate paraiH ||
</S>
<S n="68">
sa eva khalu dArubhyo yadA nirmathya dIpyate |
taddAru ca vanaM cAnyannirdahatyAzu tejasA ||
</S>
<S n="69">
evameva kule jAtAH pAvakopamatejasaH |
kSamAvanto nirAkArAH kASThe 'gniriva zerate ||
</S>

Of course, in entries with only 1 saying, the only change is the added attribute. For instance,
OLD:
<entry>
<info L="1" page="1.1" gtypes="S,D,F,V1"/>
<S>
aMzo 'pi duSTadiSTAnAM pareSAM syAdvinAzakRt |
vAlalezo 'pi vyAghrANAM patsyAjjIvitahAnaye ||
</S>
...
NEW:
<entry>
<info L="1" page="1.1" gtypes="S,D,F,V1"/>
<S n="1">
aMzo 'pi duSTadiSTAnAM pareSAM syAdvinAzakRt |
vAlalezo 'pi vyAghrANAM patsyAjjIvitahAnaye ||
</S>
...

Note that for the multiple saying cases, 'gtypes' is also modified
e.g., in the 67,68,69 example:
OLD:
<info L="67,68,69" page="1.13" gtypes="S,D,D,D,F"/>
NEW:
<info L="67,68,69" page="1.13" gtypes="S,S,S,D,D,D,F"/>

The dtd will also need adjustment
OLD: (only 1 S)
<!ELEMENT entry ((info,HS+) | (info,S,D+,F,V1*,V2*,V3*,V4*,V5*))>
NEW: (1 or more S)
<!ELEMENT entry ((info,HS+) | (info,S+,D+,F,V1*,V2*,V3*,V4*,V5*))>

This DTD would validate different numbers of S and D  (e.g. 'S,S,D,D,D'), but this currently
does not occur -- i.e., currently there are the same number of 'S' elements as 'D' elements in each entry.

D element required
The DTD also says that there must be at least 1 S, at least 1 D, and at least 1 F element in each entry.
For the 'extra' sayings (7614-7865), there actually is no German (or Greek) translation given, we have  included a 'minimal' D element, even though it is absent in the print: e.g.,
<entry>
<info L="7865" page="5.249" gtypes="S,D,F"/>
<S>
grAsodgalitasikathasya kariNaH kiM gataM bhavet |
pipIlikA tu tenaiva sakuTumbopajIvati || 7865 ||
</S>
<D n="7865">
<b>7865.</b>
</D>
<F n="7865">
7865) PRASAN3GAR.
</F>
</entry>

The above commit (337a681) corrects a few things required for the S-tag revisions.

The 'S tag improvements' now made.  See commit ef22c73.
Note 1:  Each line of Sanskrit in the S elements now enclosed within <s> tag.
This is so all Sanskrit text, whether in S element or Footnotes or additions/corrections,
will be text within an <s> element.
<entry>
<info L="7865" page="5.249" gtypes="S,D,F"/>
<S n="7865">
<s>grAsodgalitasikathasya kariNaH kiM gataM bhavet |</s>
<s>pipIlikA tu tenaiva sakuTumbopajIvati || 7865 ||</s>
</S>

Note 2:  a 'Version number' is present in boesp.xml and also in boesp.dtd.  The
current version number is 1.3.
Note 3: boesp.all_ansi.xml  is no longer being updated.  The intention is that
boesp_utf8.xml will be the starting point for further revisions, and that the 'ansi' version
has served its purpose.
@thomasincambodia  PLEASE LET ME KNOW IF THIS IS PROBLEMATIC.  Since you can
view the utf8 forms via Oxygen or Notepad++, I hoped this choice would be ok for you.

No problem. Everything fine.


how to debug view engine EJS *.ejs file

look at the title

Not sure I understand... could you give some more details?

@petruisfan
Look two images above,that red block on the second image is not found on the first image.
So i can not debug EJS files with *.ejs suffix.
Can you help me?thank a lot.

Sorry my friend, but this has nothing to do with node-supervisor

@petruisfan
Thx.
Whether or not add this feature on road map of node-inspector ?

This is node-supervisor, not node-inspector

@petruisfan
So sorry.


Feature: Feature-parity of Golang action

@laurentsimon @azeemshaikh38 @naveensrinivasan -- Would you mind updating the issue description into the following sections:

feature parity w/ entrypoint.sh
new functionality

We should descope the Golang-based rollout to expressly the things that are required for feature parity with the current entrypoint and then take new functionality as follow-up actions.
Originally posted by @justaugustus in https://github.com/ossf/scorecard-action/issues/79#issuecomment-1062490828

I (mistakenly?) created https://github.com/ossf/scorecard-action/issues/156.

Closing this in favor of #156


Never ending loop when trying to compare 2 version strings

Hi fsaintjacques.
At first, I want to thank You for such a great utility.
Despite the fact, that You primary targeted this tool for project usage, it also have possibility to compare 2 strings (which is my case):

semver compare <ver1> <ver2>

In case of compare two version strings, it ends in never ending loop when trying to find version file, since I'm running semver as utility (no .version file anywhere).
Maybe you should consider to load vesion file in case if only 1 parameter is passed (here) instead of doing that for each case (here).
Thanks in advance

That is actually a bug. I'll fix it.

release 1.2.1 fixes this.

And release 2.0.0 completely review the way semver works :)


md- prefix conflicts with ember-paper

Hi,
I just wanted to let you know that the md- prefix that you use conflicts with https://github.com/miguelcobain/ember-paper 's md- prefix. Currently, it causes quite a few font issues. I fixed it on my local copy by doing a simple search & replace of md- to mdi-.

Hey, thanks for taking the time to let me know!
I need to update the icons anyway, so when I do I'll be sure to change the prefix from md- to mdi-.

Thanks! And by the way, thanks for maintaining this icon pack. :+1:

@zavoloklom did the hard part of creating the icons. All I have to do is keep the ember-cli wrapper up to date. :sunglasses:

I should be able to push out an update this evening or tomorrow, I'll keep you posted.

Published to npm as<EMAIL_ADDRESS>

Awesome. Thanks again!

: )


[hmac] wipe key instead of xor

fixes #8506

Signed-off-by: Timothy Chen<EMAIL_ADDRESS>

ah don't thank me eunchan... i think the amount of time we sank into
discussing is probably 5 orders of magnitude greater than the time to
actually do the change...
I still don't think we really need this change..but may as well do it to
move things along :)
On Thu, Nov 11, 2021 at 3:46 PM Eunchan Kim @.***>
wrote:

@.**** approved this pull request.
Thanks Tim.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/lowRISC/opentitan/pull/9185#pullrequestreview-804304928,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAH2RSXNNFR4MGRIUPLN523ULRIVVANCNFSM5H3P44GQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.


Files rename and missing contents added

@v-hearya,
Thanks for your contribution as a Microsoft full-time employee or intern. You do not need to sign a CLA.
Thanks,
Microsoft Pull Request Bot

:x: Validation status: errors

File
Status
Preview URL
Details

:x:Error

Details

[Error] Cannot sync git repo to specified state. Please make sure the pull request is mergeable. Details: Git merge results in conflict, repository remote url: https://github.com/MicrosoftDocs/patterns-practices, working directory: W:\xftt-s, target branch: master, target branch commit: 26f0d7da02366a289cce6cd767eeded617afb599, source branch: 9c8de949203dfebb9159a956b934ddf0/master, source branch commit: .

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

:x: Validation status: errors

File
Status
Preview URL
Details

:x:Error

Details

[Error] Cannot sync git repo to specified state. Please make sure the pull request is mergeable. Details: Git merge results in conflict, repository remote url: https://github.com/MicrosoftDocs/patterns-practices, working directory: W:\tjk3-s, target branch: master, target branch commit: 26f0d7da02366a289cce6cd767eeded617afb599, source branch: 9c8de949203dfebb9159a956b934ddf0/master, source branch commit: .

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

:x: Validation status: errors

File
Status
Preview URL
Details

:x:Error

Details

[Error] Cannot sync git repo to specified state. Please make sure the pull request is mergeable. Details: Git merge results in conflict, repository remote url: https://github.com/MicrosoftDocs/patterns-practices, working directory: W:\zlz1-s, target branch: master, target branch commit: 26f0d7da02366a289cce6cd767eeded617afb599, source branch: 9c8de949203dfebb9159a956b934ddf0/master, source branch commit: .

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

:x: Validation status: errors

File
Status
Preview URL
Details

:x:Error

Details

[Error] Cannot sync git repo to specified state. Please make sure the pull request is mergeable. Details: Git merge results in conflict, repository remote url: https://github.com/MicrosoftDocs/patterns-practices, working directory: W:\wkp1-s, target branch: master, target branch commit: 26f0d7da02366a289cce6cd767eeded617afb599, source branch: 9c8de949203dfebb9159a956b934ddf0/master, source branch commit: .

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

:x: Validation status: errors

File
Status
Preview URL
Details

:x:Error

Details

[Error] Cannot sync git repo to specified state. Please make sure the pull request is mergeable. Details: Git merge results in conflict, repository remote url: https://github.com/MicrosoftDocs/patterns-practices, working directory: W:\qf1s-s, target branch: master, target branch commit: 26f0d7da02366a289cce6cd767eeded617afb599, source branch: 9c8de949203dfebb9159a956b934ddf0/master, source branch commit: .

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

Closing the request as the files has conflict. will create in PR with the updates.


windows version

am trying implement windows version, but it runs failed without stacktrace.
https://github.com/bung87/discord_rpc_puppy/tree/win

(un)fortunately, i dont have a windows system to test it on.

does it works fine on linux or macos ?

i use it on my linux machine every day.

I can't confirm it works or not. it just show like that, even it runs without any error.


Navbar

What feature?
navbar text not changing in dark mode!
Add screenshots

Record

[X] I agree to follow this project's Code of Conduct
[X] I'm a GSSoC'23 contributor
[X] I want to work on this issue

also, try to add a valid issue title it helps is to quickly undestand your issue

also, try to add a valid issue title it helps is to quickly undestand your issue

this navbar issue happen when you are using dark mode!

also, try to add a valid issue title it helps is to quickly undestand your issue

this navbar issue happen when you are using dark mode!

I'm also, using dark mode

also, try to add a valid issue title it helps is to quickly undestand your issue

this navbar issue happen when you are using dark mode!

I'm also, using dark mode

I guess this only in MacOS

also, try to add a valid issue title it helps is to quickly undestand your issue

this navbar issue happen when you are using dark mode!

I'm also, using dark mode

I guess this only in MacOS

but how to solve only for macOS?

also, try to add a valid issue title it helps is to quickly undestand your issue

this navbar issue happen when you are using dark mode!

I'm also, using dark mode

I guess this only in MacOS

but how to solve only for macOS?

Yaa that i have to figure out!!

also, try to add a valid issue title it helps is to quickly undestand your issue

this navbar issue happen when you are using dark mode!

I'm also, using dark mode

I guess this only in MacOS

but how to solve only for macOS?

Yaa that i have to figure out!!

ok ok assigned to you


Fix: single choice returned an array instead of a single value

This fix comes from issue #66.
 set value(v: any) {
        if (typeof v === 'undefined' || v === null || v === '') {
            v = [];
        }
        else if (typeof v === 'string') {
            v = [v];
        }

        if (v !== this._value) {
            this._value = v;
            this.optionList.value = v;

            this.hasSelected = v.length > 0;
            this.placeholderView = this.hasSelected ? '' : this.placeholder;
            this.updateFilterWidth();

           // here, v is an array and it's returned to your FormControl below
            this.onChange(v);
        }
    }

this.onChange(v)returned an array, it should return this.onChange(this.value) after being parsed by get value().
Also it enables plunker link in README.md to test the library.

Thanks for the help @aboglioli, very much appreciated.

 We are here to help :+1:


Update to Use Laravel 10 Process Facade

This PR is submitted to use Laravel 10 process facade.
There are 3 steps to install renv package:

Config repository URL for getting renv package
Install renv package
Initialise renv package

I have installed a new Laravel 10 application, update composer.json to refer to this branch in my local computer, then run command "php artisan rsetup rmarkdown" successfully.

Just found that renv folder is placed in project home folder instead of project home folder/scripts/R.
I will make correction and give signal when it is ready for review...

Bug fix applied, it is now ready for review.
Thank you.


esp_zb_zcl_set_attribute_val is not working with PM2.5 cluster (TZ-461)

Answers checklist.

[X] I have read the documentation ESP Zigbee SDK Programming Guide and the issue is not addressed there.
[X] I have updated my IDF branch (master or release) and ESP Zigbee libs (esp-zboss-lib and esp-zigbee-lib) to the latest version and checked that the issue is present there.
[X] I have searched the issue tracker for a similar issue and not found a similar issue.

IDF version.
5.1.0
esp-zigbee-lib version.
1.0.5
esp-zboss-lib version.
1.0.5
Espressif SoC revision.
ESP32-H2
What is the expected behavior?
esp_zb_zcl_set_attribute_val() should work with PM2.5 cluster as it works for instance with the Temperature measurement cluster.
What is the actual behavior?
esp_zb_zcl_set_attribute_val() reports an error when used with PM2.5 cluster.
Steps to reproduce.
esp_err_t err;
float_t pm25_level = (float_t) pm25;
ESP_LOGI(TAG, "Request PM25 reporting, val=%u", pm25);
err = esp_zb_zcl_set_attribute_val(HA_ESP_MAIN_ENDPOINT, ESP_ZB_ZCL_CLUSTER_ID_PM2_5_MEASUREMENT,
ESP_ZB_ZCL_CLUSTER_SERVER_ROLE, ESP_ZB_ZCL_ATTR_PM2_5_MEASUREMENT_MEASURED_VALUE_ID, &pm25_level, false);
if (err != ESP_OK) {
ESP_LOGE(TAG, "esp_zb_zcl_set_attribute_val / PM25 error, code=%d (%s)", err, esp_err_to_name(err));
} else {
ESP_LOGI(TAG, "esp_zb_zcl_set_attribute_val / PM25 was OK");
}
More Information.
The above code reports:
esp_zb_zcl_set_attribute_val / PM25 error, code=135 (ERROR)
The workaround given into https://github.com/espressif/esp-zigbee-sdk/issues/147 works partially
((float_t) esp_zb_zcl_get_attribute(HA_ESP_MAIN_ENDPOINT, ESP_ZB_ZCL_CLUSTER_ID_PM2_5_MEASUREMENT,
ESP_ZB_ZCL_CLUSTER_SERVER_ROLE, ESP_ZB_ZCL_ATTR_PM2_5_MEASUREMENT_MEASURED_VALUE_ID)->data_p) = pm25_level;
See my comment in the aforementioned issue (147).
I think it is the same bug and perhaps due to a mishandling of the float type ("Volume" unit in Zigbee spec for those clusters) into esp_zb_zcl_set_attribute_val().

worked for me after I properly set the min and max attributes

Hi @g-chevalier ,

My understanding is that the ESP-Zigbee -SDK should not check against min max if those are set to ESP_ZB_ZCL_PM2_5_MEASUREMENT_MIN_MEASURED_VALUE_UNDEFINED / ESP_ZB_ZCL_PM2_5_MEASUREMENT_MAX_MEASURED_VALUE_UNDEFINED

Or the definition of those constants is wrong or not the one expected by the library?
I disagree with your perspective. The values ESP_ZB_ZCL_PM2_5_MEASUREMENT_MIN_MEASURED_VALUE_UNDEFINED and ESP_ZB_ZCL_PM2_5_MEASUREMENT_MAX_MEASURED_VALUE_UNDEFINED are solely initialized as invalid values, and the stack does not prevent their assignment. However, according to the specification, the measure value is required to fall within the range defined by MinMeasuredValue and MaxMeasuredValue. It is conflict issue that a valid value is set for measure value within the range of these invalid values. Therefore, I believe that implementing a check for the value setting is a more stringent and appropriate behavior.

Hi @xieqinan,
Thank you for your answer.
It's OK for me as it is working when setting min and max, and being sure to interpret correctly the Zigbee Standard documentation is beyond my skills.
I was just wondering why it is working with the temperature cluster (I mean I set min and max to unknown 0x8000 as per the standard § <IP_ADDRESS>.1.2, and the reports work)…

@g-chevalier
The temperature cluster will also verify whether the measured value falls within the specified range from the minimum to the maximum. I believe the measured value will not be set successfully if it is not equal to 0x8000 and exceeds the specified range of minimum and maximum values.

@g-chevalier Could you please close the issue if no futher questions?


How to invoke a language server from command line?

I would like to query a language server from a cli utility, but I am having a hard time making first contact. How can I go about doing this? Are there any examples of "standalone" lsp clients (not part of a editor extension)?

I would like to query a language server from a cli utility, but I am having a hard time making first contact. How can I go about doing this?

@Gauteab Do you know how the language server communicates with the client? Sockets or stdin/stdout or something else?

Are there any examples of "standalone" lsp clients (not part of a editor extension)?

If you are familiar with TypeScript I have a few (very rudimentary) examples in my language server's README.md file that may get you started.
You can also try looking at my server.test.ts file I suppose.

@rcjsuen Thank you! This was very helpful and I have enough to get started now :)


feat: Update Cloud SQL MySQL samples to include more connection methods

Please review this PR to bring this sample into compliance with the standards in go/csql-v2-connectivity-samples. This PR only covers MySQL; I'll update the other two samples once this one is reviewed.

[x] I have followed Sample Format Guide
[x] pom.xml parent set to latest shared-configuration
[x] Appropriate changes to README are included in PR
[ ] API's need to be enabled to test (tell us)
[ ] Environment Variables need to be set (ask us to set them)
[x] Tests pass:   mvn clean verify required
[x] Lint  passes: mvn -P lint checkstyle:check required
[ ] Static Analysis:  mvn -P lint clean compile pmd:cpd-check spotbugs:check advisory only
[ ] Please merge this PR for me once it is approved.

Are you going to add any tags? (SnippetBot is unhappy)

Are you going to add any tags? (SnippetBot is unhappy)

Yes, I'll need to change the tags in our documentation right after merging this.

@shubha-rajan Let's pin the tags in the docs. Once we're done with all the samples, we'll make a pass and unpin everything.


Bump version to 1.2.9.

@tristane0 @shoumikhin Does this look complete?

Love it!

Looks good, thank you!
We also reference the latest release here: https://github.com/google/promises/blob/master/g3doc/index.md
But maybe we could generalize that reference so that we don’t have to update that file for each new release?

Good idea, updated docs to reference latest minor version.

FYI encountered some errors during pod lib lint, still need to resolve before tagging and pushing to trunk.

@ykjchen
I investigated pod lib lint error.
I think #145 and #148 needs be merged to fix pod lib lint error about PromisesObjC.podspec at least.
Could you see that?
Then try pod lib lint PromisesObjC.podspec
Thanks in advance.

@shoumikhin @tristane0 I'm trying to push to trunk and seeing:
... are not allowed to push new versions for this pod. The owners of this pod are ...

Are either of you able to add me as an owner?
pod trunk add-owner ...

https://guides.cocoapods.org/making/getting-setup-with-trunk.html


SimpleRtspPlayer has no sound, what is the reason?

I pull the source demo code, and there is no sound when playing RTSP stream after compiling. But playing RTSP streams with VLC is correct

Hi, sound player is not implemented

Hi, sound player is not implemented

Hi，BogdanovKirill
But I did not receive the audio stream when debugging the source code.
`        private void OnFrameReceived(object sender, RawFrame rawFrame)
{
if (!(rawFrame is RawAudioFrame rawAudioFrame))
return;
        FFmpegAudioDecoder decoder = GetDecoderForFrame(rawAudioFrame);

        if (!decoder.TryDecode(rawAudioFrame))
            return;

        IDecodedAudioFrame decodedFrame = decoder.GetDecodedFrame(new AudioConversionParameters() {OutBitsPerSample = 16});

        FrameReceived?.Invoke(this, decodedFrame);
    }`

Hi, sound player is not implemented

Hi,BogdanovKirill
But I did not receive the audio stream when debugging the source code.
private void OnFrameReceived(object sender, RawFrame rawFrame)
{
if (!(rawFrame is RawAudioFrame rawAudioFrame))
return;
FFmpegAudioDecoder decoder = GetDecoderForFrame(rawAudioFrame);
if (!decoder.TryDecode(rawAudioFrame))
return;
IDecodedAudioFrame decodedFrame = decoder.GetDecodedFrame(new AudioConversionParameters() {OutBitsPerSample = 16});
        FrameReceived?.Invoke(this, decodedFrame);
    }

Hi, sound player is not implemented

Hi,BogdanovKirill
But I did not receive the audio stream when debugging the source code.
` private void OnFrameReceived(object sender, RawFrame rawFrame)
{
if (!(rawFrame is RawAudioFrame rawAudioFrame))
return;
        FFmpegAudioDecoder decoder = GetDecoderForFrame(rawAudioFrame);
        if (!decoder.TryDecode(rawAudioFrame))
            return;

        IDecodedAudioFrame decodedFrame = decoder.GetDecodedFrame(new AudioConversionParameters() {OutBitsPerSample = 16});
        FrameReceived?.Invoke(this, decodedFrame);
    }`

@Doyle-net
Please debug sources by yourself or give me access to your RTSP server

Hi,BogdanovKirill
I tested this address source (rtsp://<IP_ADDRESS>/vod/mp4:BigBuckBunny_115k.mov) and it has audio. But I did not receive the audio stream by setting up the RTSP stream with VLC, but it is normal for me to play with VLC.

In "ConnectAsync" "parser.Parse(describeResponse.ResponseBody)" was not received, parsed into audio

@Doyle-net
Thank you for report. Need to fix it.

Hello!
I have the same issue.
Is there any information when it can be fixed?
Thanks!

May be later, I don't have free time for now

Hi, were you able to fix the audio issue as of yet?


bower: Add support for https proxy

bower was not picking up the environment variable, so we need to pass the flag before calling it.

Thank you. LGTM.
I'll test and merge if ok.

Updated:

http/https support
export variables

With the other commit merged, can we get this one in as well?

Thanks!


cmd/go2go

What version of Go are you using (go version)?

$ go version

Does this issue reproduce with the latest release?
What operating system and processor architecture are you using (go env)?
go env Output
$ go env

What did you do?

What did you expect to see?
What did you see instead?

Hit enter by accident.


Small bug in documentation

https://github.com/Luthaf/rascaline/blob/0f9a65184179a6dcf9cb3eb4cbd8abc8f3cfa417/python/examples/compute-soap.py#L60-L61
Hey y'all -- I'm on the latest rascaline and equistore, and these lines result in
    descriptor = descriptor.keys_to_properties(["species_neighbor_1", "species_neighbor_2"])
AttributeError: 'NoneType' object has no attribute 'keys_to_properties'

because apparently the transformation is done in-place. Removing the equality fixes the issue, should I make a PR?

Transformations are no longer done in-place since https://github.com/lab-cosmo/equistore/pull/86, merged roughly 2 month ago. Are you sure you are using the latest version of everything?
I just updated pre-built wheels on https://github.com/Luthaf/temporary-wheels, so if you run this, do you get rascaline v0.1.0.dev373 and equistore v0.1.0.dev187?
pip uninstall rascaline equistore
pip cache purge
pip install --extra-index-url https://luthaf.fr/temporary-wheels/ rascaline

It turns out my equistore branch hadn't pulled correctly, but just so you know, the commands you gave result in:
Found existing installation: rascaline 0.1.0.dev264
Uninstalling rascaline-0.1.0.dev264:
  Would remove:
    /opt/miniconda3/lib/python3.9/site-packages/rascaline-0.1.0.dev264.dist-info/*
    /opt/miniconda3/lib/python3.9/site-packages/rascaline/*
Proceed (Y/n)? y
  Successfully uninstalled rascaline-0.1.0.dev264
WARNING: Skipping equistore as it is not installed.
Files removed: 8
Looking in indexes: https://pypi.org/simple, https://luthaf.fr/temporary-wheels/
Collecting rascaline
  Downloading https://luthaf.fr/temporary-wheels/rascaline/rascaline-0.1.0.dev308-py3-none-macosx_10_9_x86_64.whl (1.3 MB)
     ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ 1.3/1.3 MB 20.4 MB/s eta 0:00:00
Collecting equistore
  ERROR: HTTP error 404 while getting https://luthaf.fr/temporary-wheels/equistore/equistore-0.0.0.dev187-py3-none-macosx_10_9_x86_64.whl (from https://luthaf.fr/temporary-wheels/equistore/)
ERROR: Could not install requirement equistore from https://luthaf.fr/temporary-wheels/equistore/equistore-0.0.0.dev187-py3-none-macosx_10_9_x86_64.whl (from rascaline) because of HTTP error 404 Client Error: Not Found for url: https://luthaf.fr/temporary-wheels/equistore/equistore-0.0.0.dev187-py3-none-macosx_10_9_x86_64.whl for URL https://luthaf.fr/temporary-wheels/equistore/equistore-0.0.0.dev187-py3-none-macosx_10_9_x86_64.whl (from https://luthaf.fr/temporary-wheels/equistore/)

Ok, I'll have a look at this tomorrow!


Generic paramater method

Sonaruqube can be extended with a lot of plugins. Some of them use specific sonar keys to configure the analysis run (e.g. GitLab). Instead of trying to match them all, it would be nice to have a generic way, like and extension methods in MSBuild implementation.
var settings = new SonarBeginSettings {
            Key = projectId,
            Url = sonarUrl
    }
    .AddParameter("sonar.some.key", "some value");

Just use ArgumentCustomization from the SonarBeginSettings.

closing this as @Roemer's suggestion should suffice


Barcelona Local Holidays to be added in Spain (ES) holiday package

Hello,
Our company is based in Barcelona and we utilize Barcelona Local Holidays as public holidays in our calculations.
Could you include Barcelona Local Holidays as linked here, in the holidays package?
For example May 20 and September 24 is missing from Spain (ES) holidays package.
Maybe it could be added under regional category.
Thank you.
The code example
from datetime import datetime, timedelta, time
import holidays
import calendar
import pandas as pd

per1 = pd.date_range(start="2024-01-01", end="2024-12-31").tolist()

holiday_lists = {
    "ES": holidays.country_holidays("ES", subdiv="CT"),
}

def is_public_holiday(date, country):
    # Choose holiday list based on the country
    if country not in holiday_lists:
        return False
    return date in holiday_lists[country]

for val in per1:
    if is_public_holiday(val, "ES"):
        print(val, "ES", is_public_holiday(val, "ES"))

Hi @SimgeChetin,
thanks for requesting this. We don't treat the source you've referred to as an official one. I found this and this one that confirm the missing holidays.
Please add any official sources if you know any.

Hi @SimgeChetin, thanks for requesting this. We don't treat the source you've referred to as an official one. I found this and this one that confirm the missing holidays.
Please add any official sources if you know any.

Hello @arkid15r thanks for your quick response.
I can confirm this one can be taken into consideration for the missing holidays.
Also is it possible to add missing holidays for 2023 as well?
Those would be June 5th 2023 and September 25th 2023.
Thank you :)


[appletrailers] xml.etree.ElementTree.ParseError: not well-formed (invalid token): line 39, column 138

youtube-dl -F http://trailers.apple.com/trailers/independent/thelook/
[appletrailers] thelook: Downloading XML
Traceback (most recent call last):
  File "__main__.py", line 19, in <module>
    youtube_dl.main()
  File "/home/amine/youtube-dl/youtube_dl/__init__.py", line 410, in main
    _real_main(argv)
  File "/home/amine/youtube-dl/youtube_dl/__init__.py", line 400, in _real_main
    retcode = ydl.download(all_urls)
  File "/home/amine/youtube-dl/youtube_dl/YoutubeDL.py", line 1677, in download
    url, force_generic_extractor=self.params.get('force_generic_extractor', False))
  File "/home/amine/youtube-dl/youtube_dl/YoutubeDL.py", line 665, in extract_info
    ie_result = ie.extract(url)
  File "/home/amine/youtube-dl/youtube_dl/extractor/common.py", line 291, in extract
    return self._real_extract(url)
  File "/home/amine/youtube-dl/youtube_dl/extractor/appletrailers.py", line 92, in _real_extract
    doc = self._download_xml(playlist_url, movie, transform_source=fix_html)
  File "/home/amine/youtube-dl/youtube_dl/extractor/common.py", line 466, in _download_xml
    return compat_etree_fromstring(xml_string.encode('utf-8'))
  File "/home/amine/youtube-dl/youtube_dl/compat.py", line 248, in compat_etree_fromstring
    doc = _XML(text, parser=etree.XMLParser(target=etree.TreeBuilder(element_factory=_element_factory)))
  File "/home/amine/youtube-dl/youtube_dl/compat.py", line 237, in _XML
    parser.feed(text)
  File "/usr/lib/python2.7/xml/etree/ElementTree.py", line 1642, in feed
    self._raiseerror(v)
  File "/usr/lib/python2.7/xml/etree/ElementTree.py", line 1506, in _raiseerror
    raise err
xml.etree.ElementTree.ParseError: not well-formed (invalid token): line 39, column 138

I've fixed the xml error, but the files use a different structure. Do you have more examples?
I don't have too much time; if you want to fix it yourself, just say it.

http://trailers.apple.com/trailers/independent/drive/
http://trailers.apple.com/trailers/independent/therumdiary/

i think these custom pages are not widely used.
i tested the extractor with more than 300 urls and it fails only on 4 of them.

Note that the three movies  are from 2011, so maybe that's the old layout. If someone wants to download them, we can add support for them.


Duplicate server entries being programmed in haproxy config.

Server version - V0.29.0-rc1
Steps to reproduce the problem:
Create LB service with 2 listener ports - 80 and 81 with following targets:

When LB ip address - wget http://<IP_ADDRESS>:80/service1.html is accessed , I see that user is being forwarded to same server twice before switching to another service for roundrobin.
haproxy configs have duplicate server entries being programmed:

Tested with build from master:
Created LB services with similar host routing rules.
There is no duplicate server entries being programmed in haproxy config and when the urls for path - "service1" is accessed , the traffic is redirected in roundrobin mode between instances of the service it is tied with.


[IVA'S PR] Remove packages

Description

https://github.com/biomage-org/ui-old/pull/39
Details
URL to issue
N/A
Link to staging deployment URL (or set N/A)
TBA
Links to any PRs or resources related to this PR

Integration test branch
master

Merge checklist
Your changes will be ready for merging after all of the steps below have been completed.

Code updates
Have best practices and ongoing refactors being observed in this PR

[ ] Migrated any selector / reducer used to the new format.

Manual/unit testing

[ ] Tested changes using InfraMock locally or no tests required for change, e.g. Kubernetes chart updates.
[ ] Validated that current unit tests for code work as expected and are sufficient for code coverage or no unit tests required for change, e.g. documentation update.
[ ] Unit tests written or no unit tests required for change, e.g. documentation update.

Integration testing
You must check the box below to run integration tests on the latest commit on your PR branch.
Integration tests have to pass before the PR can be merged. Without checking the box, your PR
will not pass the required status checks for merging.

[ ] Started end-to-end tests on the latest commit.

Documentation updates

[ ] Relevant Github READMEs updated or no GitHub README updates required.
[ ] Relevant Wiki pages created/updated or no Wiki updates required.

Optional

[ ] Staging environment is unstaged before merging.
[ ] Photo of a cute animal attached to this PR.

closed, because a more up to date pr is raised here: https://github.com/hms-dbmi-cellenics/ui/pull/938


Browser specific popup windows not recognised

Originally submitted to Google Code by kasarla1... on 10 Feb 2009

Hi,
I am using the below setup
Robo :2.0.4
Python :2.5.2
Selenium library :2.1.1
My application uses browser specific popup window to show/save/download an
output file
We could not recognize the popup window(as this is browser specific)
We need to recognize this window and perform some action.
Attached please find the screen shot of the popup window
Can you please help us in this context?
WBR,
Aravind Reddy K

Originally submitted to Google Code by @yanne on 11 Feb 2009

As this is a native dialog and not a browser component, this dialog cannot be handled
with SeleniumLibrary. Unless the SUT supports some other way of downloading the file,
it seems to me that the only option you have is using some other tool such as AutoIt
to handle the dialog.


Add Fuse shell doc in POSIX API doc

Page
POSIX API page
Add documentation for Fuse shell in general
and the current available commands.
In the future, when there are more Fuse shell commands, consider moving to a separate page.
Summary

@bzheng888 PTAL and help add the Fuse shell doc, thanks!

WIll do!

Thanks @bzheng888 !