Try using scylla2

Hi,
I've quickly announced at the Scylla Summit 2023 that I wrote recently on a new Scylla Rust driver : https://github.com/wyfo/scylla2 – I explain in the README the reason and the differences.
I've especially worked on performance, and first benchmarks results are promising (CPU utilisation is reduced by 30-40%, without counting zero-copy deserialization as latte cannot support it). Of course, I've done most of them with a fork of latte. You may be interested by this new driver implementation, so here is the PR to use it.
The project is obviously still in development (I've started it 4 months ago), but it seems to work fine with Scylla and Cassandra (using protocol V5 btw). Don't hesitate to take a look at it.

Thank you, this looks interesting. Currently I don't have much spare time to work on this project, so I can't promise when that happens, but anyway wish you best luck. Good to have more drivers to chose from. Any chance to support DataStax Astra, btw? That's the biggest feature latte is missing now.

There is no point in keeping this PR open. I'm not planning to switch the driver now, as the Scylla one is well supported and fully open-source.  Please contribute improvements to the original scylla driver.


feat(dataStore_pref): show all the pref data and a sample

 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.Rishabh singh bisht seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


Use proper icons for close buttons

Currently it uses text, which is ugly and inconsistent across platforms (not so much when zoomed in):

I've already extracted the icons (from screenshots):

jspaint only needs the small x, but I'm planning on implementing this first in 98, with all the icons, and maybe implementing maximization while I'm at it, and then backporting it to jspaint. (jspaint and 98 share a windowing system... via copy and paste)

The Marlett font has these icons, and in my own personal half-assed CSS recreation of Windows 95, I got away with using Wine's version of Marlett.

Well I already cut out the icons, so I think I'll use those 😄
I don't need them to be scalable, altho it might be nice - if I could still have it pixel perfect for @1x
Maybe I could switch between png and svg based on the DPI...
@media
(-webkit-min-device-pixel-ratio: 2),
(min-resolution: 192dpi) {
    .titlebar-button {
        background-image: url(".../.svg");
    }
}

This is done. I got a whole library goin' now: http://os-gui.js.org/


Feature: Expose send text constant as a parameter

Is it possible to expose this as a parameter:
https://github.com/Ikuyadeu/vscode-R/blob/master/src/rTerminal.ts#L119
The current one feels a little slow.  Is that as fast as it goes without error?

Hi @jrounds, thank you for the suggestion. This could be exposed as a parameter. Because of changes to VS Code itself, I think sending text to VS Code's terminal has slowed down slightly since that line was added. For many users, the value could probably be safely decreased.
In the mean time, for much faster sending of text to the console we recommend radian.


FinRL_PortfolioOptimizationEnv_Demo.ipynb

"Hello, in the FinRL_PortfolioOptimizationEnv_Demo.ipynb file, I've been practicing and checking to reproduce the details mentioned in a paper. I've come across a section that appears to affect the neural network. I am not certain about the aspects it impacts. Is it appropriate to make such changes?"

I didn't understand you. pls describe more details


Make BrickProvider module (internal) visible

Fixes https://github.com/wayfair/brickkit-android/issues/121

Codecov Report

Merging #127 into master will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master     #127   +/-   ##
=========================================
  Coverage     86.91%   86.91%
  Complexity      368      368
=========================================
  Files            29       29
  Lines          1131     1131
  Branches        175      175
=========================================
  Hits            983      983
  Misses           76       76
  Partials         72       72

Continue to review full report at Codecov.

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


Automatic device transfer converts namedtuples into regular tuples

🐛 Bug
In the latest version of the utilities library, automatic device transfer converts namedtuples into regular tuples, causing loss of attribute access provided by the namedtuple.
To Reproduce
from collections import namedtuple
import torch
from pytorch_lightning import LightningModule, Trainer

Batch = namedtuple("Batch", ["features", "targets"])

class SimpleModule(LightningModule):
    def __init__(self):
        super().__init__()
        self.linear = torch.nn.Linear(5, 1)

    def forward(self, batch):
        print(type(batch))
        print(batch.features)
        return self.linear(batch.features)

    def training_step(self, batch, batch_idx):
        return self(batch)

    def train_dataloader(self):
        features = torch.randn(10, 5)
        targets = torch.randint(0, 1, (10,))
        dataset = [Batch(features[i], targets[i]) for i in range(10)]
        return torch.utils.data.DataLoader(dataset, batch_size=1)

    def configure_optimizers(self):
        return torch.optim.Adam(self.parameters(), lr=1e-3)

model = SimpleModule()
trainer = Trainer(fast_dev_run=True)
trainer.fit(model)

This script fails with attribute error AttributeError: 'tuple' object has no attribute 'features', when the library version is 0.10.0 and runs without errors on 0.9.0. The Pytorch lightning version was unchanged, 2.1.2 for both.
Additional context

Environment details

PyTorch Version (e.g., 1.0): 2.1.1+cu121
OS (e.g., Linux): Ubuntu
How you installed PyTorch (conda, pip, source): pip
Python version: 3.10.13
CUDA/cuDNN version: 12.2

This happened for both single GPU and multi GPU run with DDP. This was a bit of work to track down since the pinned pytorch_lightning version was unchanged.
This is the exact same as a previous lightning issue, although that one seems to have been due to PyTorch?

Hi @lakshmi-speak. Sorry for this.
I think it's best that we revert https://github.com/Lightning-AI/utilities/pull/160.
Would you like to open a PR doing that and adding your namedtuple as a test to https://github.com/Lightning-AI/utilities/blob/main/tests/unittests/core/test_apply_func.py?

@carmocca was this resolved in #210 too?


quick attribute on md-dialog

Description
A quick attribute:
<md-dialog quick>...</md-dialog>

to open/close the dialog without any animations.
One implementation solution would be to change:
https://github.com/material-components/material-web/blob/6589b2ee4b41c19f4683cbeb2c9457c97c1b434a/dialog/internal/dialog.ts#L76-L86
to
getOpenAnimation() {
  if (this.quick) {
    return {};
  }
  else {
    return DIALOG_DEFAULT_OPEN_ANIMATION;
  }
}

getCloseAnimation() {
  if (this.quick) {
    return {};
  }
  else {
    return DIALOG_DEFAULT_CLOSE_ANIMATION;
  }
}

(Waiting the green light)

I think a quick attribute similar to menus makes sense!
However, I would move it to animateDialog() so that clients who override getOpenAnimation/getCloseAnimation do not need to check for dialog.quick.
   private async animateDialog(animation: DialogAnimation) {
+    if (this.quick) {
+      return;
+    }

     const {dialog, scrim, container, headline, content, actions} = this;
     if (!dialog || !scrim || !container || !headline || !content || !actions) {
       return;
     }
     // ...
   }


Make masks respect the transformation of their target without having to add them to the target

This is a feature request, not a bug report. It's related to #1978.
Masks are a super powerful feature. Thanks a lot for that! That said, I got bitten by masks ignoring the transformation of their parent twice already (the only two times I used them so far).
In some cases it's possible (although hacky) to add the mask to the parent, as recommended in that other bug. The reason I say this is hacky is that it the intent is confusing. It's unclear whether the intent of some code having the mask as a child is to have the mask follow the transform of the parent, or just have it rendered as well. It can be mitigated with documentation, but with a different API, such documentation wouldn't be necessary.
In other cases, due to design choices, altering the children of the parent to include the mask is not possible or really hard to do.  Another example is a library that produces PIXI objects from another representation (e.g. ReactPixi) and fails when the mask is injected as a child because indices no longer match. These are not theoretical, I have both use cases.
The requested feature is a boolean property controlling this mask behavior, so it can be toggled to decide whether the mask should be applied with the parent transformation or without any transformation like it does today. All of this without altering the children list of the parent.
Thank you!

I found another workaround that does not require modifying the children of the target object:
Every time I suspect the transform of the target might have changed:
mask.transform = target.transform;
Still a bit hacky, since it requires hooking to all the ways in which the target transform can change, to copy it over to the mask (not a problem for my use cases), and prevents the mask itself from having a transformation "delta". But unblocks me until Pixi hopefully provides something better.

Another way is to do something like mask.transform.setFromMatrix(target.worldTransform.clone())

Thanks @SukantPal. What's the advantage of that approach?

The transform isn't supposed to be reassigned - with this approach you're merely copying data over into the mask.


Add library, update schema

Adds another library from the public repo
Updates the categories to accommodate a rename we did in the public repo
Updates the schema to allow multiple authors

Never mind, just going to add this to the other PR!


Typos and grammar suggestions

Hey, this commit contains a few typo fixes and grammar suggestions for you to consider.

 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.You have signed the CLA already but the status is still pending? Let us recheck it.


Improve documentation for Style/EvalWithLocation cop

This PR is to improve documentation for Style/EvalWithLocation cop.
There are no description and example for the case that a string variable is given as a code string like below.
code = <<-RUBY
def do_something
end
RUBY

eval code

I added description and example to make this case clear.

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.
[ ] 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).
[ ] Squashed related commits together.
[ ] 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.

Hmm, I wonder why we'd treat this case differently. Perhaps an oversight on our part and we should treat a variable param just a string literal. @dvandersluis what do you think?

Hmm, I wonder why we'd treat this case differently. Perhaps an oversight on our part and we should treat a variable param just a string literal. @dvandersluis what do you think?

I think we cannot know where the given string variable is defined so the cop does not check this case.

I think we cannot know where the given string variable is defined so the cop does not check this case.

This is true about a string that came externally, but for any other string parameter it's pretty much the same as for an explicit string literal IMO. E.g. in your example you can supply the location metadata.

This is true about a string that came externally, but for any other string parameter it's pretty much the same as for an explicit string literal IMO. E.g. in your example you can supply the location metadata.

Ah, I understand. You're thinking the cop should check as much as possible.
Do following examples match your thought?
Examples which the cop should check:
def eval_string
  code = <<~CODE
    def do_something
    end
  CODE
  eval(code) # report offense
end

def eval_string_from_file(file)
  code = File.read(file)
  eval(code, file, 1)
end

Examples which the cop does not check:
def eval_string(s)
  eval(s)
end

def eval_code
  eval(@code)
end

Ah, I understand. You're thinking the cop should check as much as possible.
Do following examples match your thought?
Examples which the cop should check:
def eval_string
  code = <<~CODE
    def do_something
    end
  CODE
  eval(code) # report offense
end

def eval_string_from_file(file)
  code = File.read(file)
  eval(code, file, 1)
end

Examples which the cop does not check:
def eval_string(s)
  eval(s)
end

def eval_code
  eval(@code)
end

@taichi-ishitani I think what @bbatsov is getting at is that your examples are still calls to eval without a file and location given.
In other words, your eval_string method would be better done as:
def eval_string
  code = <<~CODE
    def do_something
    end
  CODE
  eval(code, binding, __FILE__, __LINE__) # this line is changed
end

We don't need to know what the code is doing for this cop, just that eval is called without the proper arguments.
However, I'm not sure if we should autocorrect when not given a string, because the cop sets a value relative to __LINE__ and as you said we don't know where code comes from (unless we always just set it to __LINE__ with no offset when not given a string).

@taichi-ishitani I think what @bbatsov is getting at is that your examples are still calls to eval without a file and location given.
In other words, your eval_string method would be better done as:
def eval_string
  code = <<~CODE
    def do_something
    end
  CODE
  eval(code, binding, __FILE__, __LINE__) # this line is changed
end

We don't need to know what the code is doing for this cop, just that eval is called without the proper arguments.
However, I'm not sure if we should autocorrect when not given a string, because the cop sets a value relative to __LINE__ and as you said we don't know where code comes from (unless we always just set it to __LINE__ with no offset when not given a string).

Yeah, that's exactly what I was referring to. Thanks for explaining it much better than me! :D

Yeah, that's exactly what I was referring to. Thanks for explaining it much better than me! :D

def eval_string
  code = <<~S

    foo
  S
  instance_eval(code, __FILE__, __LINE__)
end
eval_string

We can get following back traces when we invoke the above code:
$ ruby with_location_info.rb
with_location_info.rb:9:in `eval_string': undefined local variable or method `foo' for main:Object (NameError)
        from with_location_info.rb:7:in `instance_eval'
        from with_location_info.rb:7:in `eval_string'
        from with_location_info.rb:9:in `<main>'

The 1st line of the back trace shows incorrect location.
Therefore, I think we should not set the location info if we cannot know where a string variable is defined correctly.

def eval_string
  code = <<~S

    foo
  S
  instance_eval(code, __FILE__, __LINE__)
end
eval_string

We can get following back traces when we invoke the above code:
$ ruby with_location_info.rb
with_location_info.rb:9:in `eval_string': undefined local variable or method `foo' for main:Object (NameError)
        from with_location_info.rb:7:in `instance_eval'
        from with_location_info.rb:7:in `eval_string'
        from with_location_info.rb:9:in `<main>'

The 1st line of the back trace shows incorrect location.
Therefore, I think we should not set the location info if we cannot know where a string variable is defined correctly.

Fair enough. I think updating the docs is the simplest thing we can do right now. Thanks!


Issue: Unable to properly parse output of custom chat agent.

Issue you'd like to raise.
pip list:

langchain                                0.0.314
langchain-core                           0.0.8

prompt_template = """
### [INST]

Assistant is a large language model trained by Mistral.

Assistant is designed to be able to assist with a wide range of tasks, from answering simple questions to providing in-depth explanations and discussions on a wide range of topics. As a language model, Assistant is able to generate human-like text based on the input it receives, allowing it to engage in natural-sounding conversations and provide responses that are coherent and relevant to the topic at hand.

Assistant is constantly learning and improving, and its capabilities are constantly evolving. It is able to process and understand large amounts of text, and can use this knowledge to provide accurate and informative responses to a wide range of questions. Additionally, Assistant is able to generate its own text based on the input it receives, allowing it to engage in discussions and provide explanations and descriptions on a wide range of topics.

Overall, Assistant is a powerful tool that can help with a wide range of tasks and provide valuable insights and information on a wide range of topics. Whether you need help with a specific question or just want to have a conversation about a particular topic, Assistant is here to assist.

Context:
------

Assistant has access to the following tools:

{tools}

To use a tool, please use the following format:

'''
Thought: Do I need to use a tool? Yes
Action: the action to take, should be one of [{tool_names}]
Action Input: the input to the action
Observation: the result of the action
'''

When you have a response to say to the Human, or if you do not need to use a tool, you MUST use the format:

'''
Thought: Do I need to use a tool? No
Final Answer: [your response here]
'''

Begin!

Previous conversation history:
{chat_history}

New input: {input}

Current Scratchpad:
{agent_scratchpad}

[/INST]
 """

# Create prompt from prompt template
prompt = PromptTemplate(
    input_variables=['agent_scratchpad', 'chat_history', 'input', 'tool_names', 'tools'],
    template=prompt_template,
)

prompt = prompt.partial(
    tools=render_text_description(tools),
    tool_names=", ".join([t.name for t in tools]),
)

# Create llm chain
llm_chain = LLMChain(llm=mistral_llm, prompt=prompt)

from langchain.agents import AgentOutputParser
from langchain_core.agents import AgentAction, AgentFinish
import re
from typing import List, Union

class CustomOutputParser(AgentOutputParser):
    def parse(self, output) -> Union[AgentAction, AgentFinish]:
        print(output)
        output = ' '.join(output.split())
        # Check if the output contains 'Final Answer:'
        if 'Final Answer:' in output:
            # Extract the final answer from the output
            final_answer = output.split('Final Answer:')[1].strip()
            print(final_answer)

            agent_finish = AgentFinish(
                # Return values is generally always a dictionary with a single `output` key
                # It is not recommended to try anything else at the moment :)
                return_values={"output": final_answer},
                log=output,
            )
            print(agent_finish)
            return agent_finish
        else:
            # Handle other cases or raise an error
            raise ValueError("Unexpected output format")

output_parser = CustomOutputParser()

# Create an agent with your LLMChain
agent = ConversationalAgent(llm_chain=llm_chain , output_parser=output_parser)

memory = ConversationBufferMemory(memory_key="chat_history")
agent_executor = AgentExecutor(agent=agent, tools=tools, verbose=True, memory=memory)

agent_executor.run("Hi, I'm Madhav?")

I get the following output:
> Entering new AgentExecutor chain...
Error in StdOutCallbackHandler.on_agent_action callback: 'tuple' object has no attribute 'log'
 Thought: Do I need to use a tool? No
  Final Answer: Hello, Madhav! How can I assist you today?
Hello, Madhav! How can I assist you today?
return_values={'output': 'Hello, Madhav! How can I assist you today?'} log='Thought: Do I need to use a tool? No Final Answer: Hello, Madhav! How can I assist you today?'

File /opt/conda/lib/python3.10/site-packages/langchain/agents/agent.py:1141, in AgentExecutor._call(self, inputs, run_manager)
   1139 # We now enter the agent loop (until it returns something).
   1140 while self._should_continue(iterations, time_elapsed):
-> 1141     next_step_output = self._take_next_step(
   1142         name_to_tool_map,
   1143         color_mapping,
   1144         inputs,
   1145         intermediate_steps,
   1146         run_manager=run_manager,
   1147     )
   1148     if isinstance(next_step_output, AgentFinish):
   1149         return self._return(
   1150             next_step_output, intermediate_steps, run_manager=run_manager
   1151         )

File /opt/conda/lib/python3.10/site-packages/langchain/agents/agent.py:983, in AgentExecutor._take_next_step(self, name_to_tool_map, color_mapping, inputs, intermediate_steps, run_manager)
    981     run_manager.on_agent_action(agent_action, color="green")
    982 # Otherwise we lookup the tool
--> 983 if agent_action.tool in name_to_tool_map:
    984     tool = name_to_tool_map[agent_action.tool]
    985     return_direct = tool.return_direct

AttributeError: 'tuple' object has no attribute 'tool'
...

I'm having trouble understading what is happening here. I'm certain my output contains Final Answer so I'd assume I need an ActionFinish as per this tutorial.
Any tips would be much appreciated!
Suggestion:
No response

Facing a similar error
agent_executor = AgentExecutor(agent=agent, tools = tools, type = AgentType.ZERO_SHOT_REACT_DESCRIPTION, verbose=True,     handle_parsing_errors=True, llm = llm)
On Calling agent_executor.invoke method,error:
...
-> 1009 if agent_action.tool in name_to_tool_map:
   1010     tool = name_to_tool_map[agent_action.tool]
   1011     return_direct = tool.return_direct

AttributeError: 'tuple' object has no attribute 'tool'

Try using langchain.schema instead of langchain_core.agents as the source from the AgentAction and AgentFinish imports. Don't fully understand what the difference between the two is but this solved it for me.

This solution worked for me:
class CustomOutputParser(ConvoOutputParser):
    def parse(self, text: str) -> Union[AgentAction, AgentFinish]:
        """Attempts to parse the given text into an AgentAction or AgentFinish.

        Raises:
             OutputParserException if parsing fails.
        """
        try:
            # If the response contains an 'action' and 'action_input'

            print(text)
            if "Action" in text or "Action Input" in text:
                # If the action indicates a final answer, return an AgentFinish
                if "Final Answer" in text:
                    return AgentFinish({"output": text.split('Final Answer:')[1]}, text)
                else:
                    # Otherwise, return an AgentAction with the specified action and
                    # input
                    return AgentAction(action, action_input, text)
            else:
                # If the necessary keys aren't present in the response, raise an
                # exception
                raise OutputParserException(
                    f"Missing 'action' or 'action_input' in LLM output: {text}"
                )
        except Exception as e:
            # If any other exception is raised during parsing, also raise an
            # OutputParserException
            raise OutputParserException(f"Could not parse LLM output: {text}") from e

action and action_input are not defined


COMP: Fix missing initialization braces warnings

Fix missing initialization braces warnings.
Fixes:
[CTest: warning matched]
/Users/builder/externalModules/Core/Common/test/itkSobelOperatorTest.cxx:52:9:
warning: suggest braces around initialization of subobject
[-Wmissing-braces]
      { -1.0, 0.0, 1.0, -2.0, 0.0, 2.0, -1.0, 0.0, 1.0 }
        ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
        {                                             }

and warnings of the same kind reported at:
https://open.cdash.org/viewBuildError.php?type=1&onlydeltap&buildid=8264373
PR Checklist

[X] No API changes were made (or the changes have been approved)
[X] No major design changes were made (or the changes have been approved)

/azp run ITK.macOS

For the record, this warning only seems to appear with older clang versions, as I tried at  https://godbolt.org/z/P9fqrTWG1  using clang 5.0.2 on the following code:
#include <array>

struct FixedArray
{
    explicit FixedArray(const std::array<double, 3>&);
};

struct Size
{
    int data[3];
};

int main()
{
  Size size{1, 2, 3};
  FixedArray fixedArray{{1, 2, 3}};
  (void)size;
  (void)fixedArray;
}

The warning does not appear anymore with clang 6.0.0 up to clang 15.0.0, as far as I can see.

Approved, with the note that the extra braces should not be necessary anymore once we have abandoned those old clang compilers (clang version < 6.0.0). Thanks Jon.

There are still a number of related warnings in the dashboards.


Handle int64 and uint64 in properties by wrapping it

Summary
We started using structpb.Value to hold the payload of our properties,
but since the Value type is expected to be directly serialized to a JSON
value, all numerical values are treated as float64.
Because we also want to store int64 values, we need to treat them
separately, by storing them as objects instead with the integer value
stringified.
Related: #4171
Change Type

[ ] Bug fix (resolves an issue without affecting existing features)
[x] Feature (adds new functionality without breaking changes)
[ ] Breaking change (may impact existing functionalities or require documentation updates)
[ ] Documentation (updates or additions to documentation)
[ ] Refactoring or test improvements (no bug fixes or new functionality)

Testing
mostly unit tests
Review Checklist:

[x] Reviewed my own code for quality and clarity.
[ ] Added comments to complex or tricky code sections.
[ ] Updated any affected documentation.
[x] Included tests that validate the fix or feature.
[ ] Checked that related changes are merged.

coverage: 53.838% (+0.1%) from 53.716%
when pulling 12988865b8765f91b60cd83794d00c30fc04d9d0 on jhrozek:prop_int64
into b30ca2efeb3ac2ae7f35bdf00f9b220e1be0f8e1 on stacklok:main.


fix: Missing renaming

This PR fixes a missing renaming in #3047
See this comment for more details: https://github.com/vuejs/vue-router/pull/3047#pullrequestreview-414419587

Thanks for the quick PR! This is the kind of things that are caught with TypeScript 😓

Or the Closure Compiler, if you're into that sort of thing. 😅


Support rescaling worker numbers in Kubernetes

Kubernetes support rescaling replication controllers or replica sets. The support can be added to Mars to facilitate adjusting cluster size.

Need implementations in oscar, thus reopen.


Android - Could not find com.google.android.exoplayer:exoplayer:2.11.4.

Hi, I am recently using this library in my RN project. It works fine in iOS devices, but when comes to android, I can't build success. Here is the error log.
* What went wrong:
Execution failed for task ':app:checkDebugAarMetadata'.
> Could not resolve all files for configuration ':app:debugRuntimeClasspath'.
   > Could not find com.google.android.exoplayer:exoplayer:2.11.4.
     Searched in the following locations:
       - https://repo.maven.apache.org/maven2/com/google/android/exoplayer/exoplayer/2.11.4/exoplayer-2.11.4.pom
       - file:/Users/teowenlong/.m2/repository/com/google/android/exoplayer/exoplayer/2.11.4/exoplayer-2.11.4.pom
       - file:/Users/teowenlong/rn-visitor-app/node_modules/react-native/android/com/google/android/exoplayer/exoplayer/2.11.4/exoplayer-2.11.4.pom
       - file:/Users/teowenlong/rn-visitor-app/node_modules/jsc-android/dist/com/google/android/exoplayer/exoplayer/2.11.4/exoplayer-2.11.4.pom
       - https://dl.google.com/dl/android/maven2/com/google/android/exoplayer/exoplayer/2.11.4/exoplayer-2.11.4.pom
       - https://www.jitpack.io/com/google/android/exoplayer/exoplayer/2.11.4/exoplayer-2.11.4.pom
     Required by:
         project :app
         project :app > project :react-native-video
   > Could not find com.google.android.exoplayer:extension-okhttp:2.11.4.
     Searched in the following locations:
       - https://repo.maven.apache.org/maven2/com/google/android/exoplayer/extension-okhttp/2.11.4/extension-okhttp-2.11.4.pom
       - file:/Users/teowenlong/.m2/repository/com/google/android/exoplayer/extension-okhttp/2.11.4/extension-okhttp-2.11.4.pom
       - file:/Users/teowenlong/rn-visitor-app/node_modules/react-native/android/com/google/android/exoplayer/extension-okhttp/2.11.4/extension-okhttp-2.11.4.pom
       - file:/Users/teowenlong/rn-visitor-app/node_modules/jsc-android/dist/com/google/android/exoplayer/extension-okhttp/2.11.4/extension-okhttp-2.11.4.pom
       - https://dl.google.com/dl/android/maven2/com/google/android/exoplayer/extension-okhttp/2.11.4/extension-okhttp-2.11.4.pom
       - https://www.jitpack.io/com/google/android/exoplayer/extension-okhttp/2.11.4/extension-okhttp-2.11.4.pom
     Required by:
         project :app > project :react-native-video

It keeps saying that exoplayer is not found. but I had follow exactly the installation steps stated in main page. What had I done wrongly?

Finally I found the solution.
In RN 0.65, jcenter() in Android was deleted.
Add it back to app/build.gradle and it works all fine :)
I get the solution from #2462. Any further question may ask in here.

This fix isn't working for me :( I'm using RN 0.66

Not working for me well RN 0.66
Could not find com.google.android.exoplayer:exoplayer:2.9.2.
Required by:
project :expo-av
Search in build.gradle files

same issue

I was able to fix this by adding the line jcenter() to **android**/build.gradle' (rather than android/app/build.gradle`). Here is the context:
allprojects {
    repositories {
        maven {
            // All of React Native (JS, Obj-C sources, Android binaries) is installed from npm
            url("$rootDir/../node_modules/react-native/android")
        }
        maven {
            // Android JSC is installed from npm
            url("$rootDir/../node_modules/jsc-android/dist")
        }
        mavenCentral {
            // We don't want to fetch react-native from Maven Central as there are
            // older versions over there.
            content {
                excludeGroup "com.facebook.react"
            }
        }
        google()
        **jcenter()**
        maven { url 'https://www.jitpack.io' }
        maven {
            // expo-camera bundles a custom com.google.android:cameraview
            url "$rootDir/../node_modules/expo-camera/android/maven"
        }
    }
}

Worked for me with RN 0.67.2.
I was able to fix this by adding the line jcenter() to android/build.gradle (rather than android/app/build.gradle). Here is the context:
allprojects {
    repositories {
        maven {
            // All of React Native (JS, Obj-C sources, Android binaries) is installed from npm
            url("$rootDir/../node_modules/react-native/android")
        }
        maven {
            // Android JSC is installed from npm
            url("$rootDir/../node_modules/jsc-android/dist")
        }
        mavenCentral {
            // We don't want to fetch react-native from Maven Central as there are
            // older versions over there.
            content {
                excludeGroup "com.facebook.react"
            }
        }
        google()
        // I ADDED THE NEXT LINE:
        jcenter()
        maven { url 'https://www.jitpack.io' }
        maven {
            // expo-camera bundles a custom com.google.android:cameraview
            url "$rootDir/../node_modules/expo-camera/android/maven"
        }
    }
}

its not app/build.gradle its android/build.gradle

Android has two build.gradle files, one located at android/build.gradle, the other at android/app/build.gradle. You have to modify the right one (android/build.gradle).

the problem is jcenter is deprecated now..

WARNING:: Please remove usages of jcenter() Maven repository from your build scripts and migrate your build to other Maven repositories.

not sure how to get the com.google.android.exoplayer:exoplayer:2.11.4 out of jcenter in future.

Finally I found the solution. In RN 0.65, jcenter() in Android was deleted. Add it back to app/build.gradle and it works all fine :)
I get the solution from #2462. Any further question may ask in here.

This workaround works but it causes the app to crash on devices with low width. For me it was Samsung Galaxy M 62. Does anyone know how to fix this?

The proper solution can be found at #2611, no jcenter() as it will be obsolete.


Launch Planning for Tuesday

Here a list of things we need to do for the hard launch of this project Tuesday:
Launch start: 11:00 CET
ToDOs:

[ ] Create oneliner for HN. ( 80 characters max )

[ ]  Find your open source contribution to climate change.
[ ]  Find the Open Source you need to help fight climate change.
[ ]  Contribute to open source focused on climate technology and sustainability.

[ ] Create social media image. The current AI generator image do not look really professional.
[ ] Create social media post for:

[x] Reddit
[x] LinkedIn
[ ] Slack
[x] SustainOSS
[x] Twitter
[x] Product Hunt

[x] Create Blog post for OpenSustain.tech and LF Energy.

[x]  https://docs.google.com/document/d/1JS0afkg0cGD5KNWO84uGw6oXV6Lrs33ZsXaJbLiEpTM/edit

[x] Add Launch message to ClimateTriage header
[x] Add Launch message to OpenSustain.tech header
[ ] Contact influencer in climate and sustainability if they want to share the website.

I think this can be closed now that it's launched?


Grammar Mistake

There is a Grammar mistake in the README file:
Prevent neutral mob hitting
Prevents you from [Missing Word] neutral mobs (as defined in the minecraft wiki, except Spiders and Endermen)

thanks for pointing that out, I'll fix it


20532B_LAB_AK_03.md

Assuming you want all of the steps to be up to date...
In 20532B_LAB_AK_03.md:

In the Database list, select Create a free 20 MB SQL database.
Now says "Create a free SQL database"

In the navigation page on the left side, click Websites.
In general, "Websites" are now "Web Apps".

then double-click Contoso.Events.sln
The default install of Windows does not display file extensions.
Error message: project requires SQL Server 2012 Express LocalDB, which is not installed on this computer

On the Debug menu, click Start Debugging.   (two places in the lab)
Due to the missing SQL Server, the app will fail: ProviderIncompatibleException.
(Maybe change the Start Debugging to just a Build.)

Got the local app to work by separately downloading/installing SQL Server 2014 Express with Tools with defaults (instance name SQLExpress) and changed the connection string to .\SQLExpress.

instructions for install added


enhancement add status code and error msg to audit log for proxy stmt

Proposed changes
Issue Number: close #xxx

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall

run buildall

run buildall


volume hostpath with subpath

Rancher versions:
rancher/server or rancher/rancher: 2.0.6
Infrastructure Stack versions:
kubernetes (if applicable):  v.1.10.5
Create a volume in a pod

According to document, https://kubernetes.io/docs/concepts/storage/volumes/#using-subpath
a Pod uses subPath to create a directory pod1 within the hostPath volume /var/log/pods, using the pod name from the Downward API. The host directory /var/log/pods/pod1 is mounted at /logs in the container.
Expecting directory /iag/unix create in node host and mount to pod as /unix
But got

view yaml

Not sure what I missed. No directory created in node host.

It's a known k8s issue. It will work if you leave subpath empty
Please check https://github.com/kubernetes/kubernetes/issues/61456

This is still an issue with Rancher installed K8s 1.11 and 1.12.
The issue for hostPath+subPath with containerized Kubelet has been fixed upstream since v1.11: https://github.com/kubernetes/kubernetes/pull/63143 first shipped in K8s v1.11.0.
It looks like there is some work todo on RKE side (like mounting the host rootfs to kubelet /rootfs mountpoint).

Steps to reproduce:

Create a cluster in Rancher using K8s v1.12.x
Deploy a workload that mounts a hostPath volume using a subPath (see manifest below)
Exec to the pod and create a new file under the mount point (e.g. uniquefile)
SSH to the host where the Pod is running.
Verify that the file was created under the expected hostPath/subPath in the rootfs

Result:
worker1:/# stat /site-data/nginx
stat: cannot stat '/site-data/nginx': No such file or directory

apiVersion: apps/v1
kind: Deployment
metadata:
  name: nginx-deployment
  labels:
    app: nginx
spec:
  replicas: 1
  selector:
    matchLabels:
      app: nginx
  template:
    metadata:
      labels:
        app: nginx
    spec:
      containers:
      - name: nginx
        image: nginx:latest
        volumeMounts:
        - mountPath: /volume/nginx
          name: site-data
          subPath: nginx
      volumes:
      - name: site-data
        hostPath:
          path: /site-data
          type: DirectoryOrCreate

Ran into that problem today and can also reproduce it with mosquitto and a sub path in the claimed volume.  Using v1.13.4-rancher1-1 and rancher-agent:v2.1.7

/cc @loganhz

Added into 2.3
https://github.com/kubernetes/kubernetes/pull/63143#issuecomment-401887153

Here's the workaround:

Edit cluster, Edit as YAML
Add the following flags for kubelet:

services:
  kubelet:
    extra_args:
      containerized: "true"
    extra_binds:
      - "/:/rootfs:rshared"

Click save and wait till the cluster is updated.

Notes:
The community is planning to deprecate the "--containerized" for kubelet(https://github.com/kubernetes/kubernetes/issues/74148).
But the flag is essential for the capability as there is no alternative at the moment.

@gitlawr does that mean you're mount-binding the whole root system of your Kubernetes host to the kubelet container? That can't be good for security, right?

Looks like an rke fix?

I realize @galal-hussein is out this week, but assigning to him at least temporarily

From https://github.com/rancher/rancher/issues/14836#issuecomment-491152114:

The community is planning to deprecate the "--containerized" for kubelet(kubernetes/kubernetes#74148).

This already got merged, so a containerized kubelet is already deprecated.

But the flag is essential for the capability as there is no alternative at the moment.

What's the plan for rancher/rke regarding hyperkube ? When does the containerized kubelet gets replaced ? This issue is really annoying because a lot of upstream charts break on a hostpath provider due to their use of subpath.

Any news here?

I created a feature request to deprecate the containerized kubelet.
Meanwhile I have two questions, maybe somebody from rancher could comment on:

What's the recommended workaround for this issue ? Is it bind-mounting the whole host rootdir into the kubelet container like suggested by @gitlawr  in above comment ?
If so, are there any security implications with this, or anything that could reduce potential risks ?

Is it bind-mounting the whole host rootdir into the kubelet container like suggested by @gitlawr in above comment ?

To be more precise, which host directories need to get additionally bind-mounted into the pod in order to make subpath work ?

Any news on this??? The ability to mount an specific file is very important to us.

any update on this , i have created a pv before i claim it using the volume claim template in values.yaml file for prometheus helm chart.

my goal is to take the tsdb snapshot and store the snapshot in host path
i am ablet create the snapshot, bound the pv but prometheus server pod is breaking with crate config error with error stating /data/prometheus: no such file or directory.

if i could attach the pv. i will use the snapshot path not to loose the metrics during upgrade
could some one help me with a work around to achieve my goals

Workaround provided by @gitlawr is not working anymore on kube 1.17.4 since https://github.com/kubernetes/kubernetes/issues/74148 .
Now I'm seeing this message :
FATA[0339] [workerPlane] Failed to bring up Worker Plane: [Failed to verify healthcheck: Failed to check http://localhost:10248/healthz for service [kubelet] on host [cpu3.store.local]
[server.go:156] unknown flag: --containerized

I'm afraid it will not be anymore enough to fix this.
AFAIK, the only other way to fix it is to apply some kubectl patch as told in https://github.com/rancher/local-path-provisioner/issues/4#issuecomment-578620842
Please make something fast :'(

I've done some further investigation on this issue and will summarize my findings below.
Problem
When using a hostPath volume with a subPath, the containerized kubelet is not able to create a mount back to the host filesystem.
Validate hostPath volume
In a cluster running a containerized kubelet you can perform the following steps to check that hostPath volumes work.
Create a deployment with a hostPath volume that has no subPath:
apiVersion: apps/v1
kind: Deployment
metadata:
  name: nginx-deployment
  labels:
    app: nginx
spec:
  replicas: 1
  selector:
    matchLabels:
      app: nginx
  template:
    metadata:
      labels:
        app: nginx
    spec:
      containers:
      - name: nginx
        image: nginx:latest
        volumeMounts:
        - mountPath: /volume/nginx
          name: site-data
      volumes:
      - name: site-data
        hostPath:
          path: /site-data
          type: DirectoryOrCreate

SSH into the worker node where the pod was scheduled and inspect the corresponding docker container to see bind mounts:
{
    "HostConfig": {
        "Binds": [
            "/site-data:/volume/nginx"
        ]
    }
}

This looks good and the /site-data directory is created on the host
Reproduce subPath issue with hostPath volume
Create a deployment with a hostPath volume that has a subPath:
apiVersion: apps/v1
kind: Deployment
metadata:
  name: nginx-deployment
  labels:
    app: nginx
spec:
  replicas: 1
  selector:
    matchLabels:
      app: nginx
  template:
    metadata:
      labels:
        app: nginx
    spec:
      containers:
      - name: nginx
        image: nginx:latest
        volumeMounts:
        - mountPath: /volume/nginx
          name: site-data
          subPath: nginx
      volumes:
      - name: site-data
        hostPath:
          path: /site-data
          type: DirectoryOrCreate

SSH into the worker node where the pod was scheduled and inspect the corresponding docker container to see bind mounts:
{
    "HostConfig": {
        "Binds": [
            "/var/lib/kubelet/pods/d0b79a40-6a69-47d9-a7e4-ef36331732e2/volume-subpaths/site-data/nginx/0:/volume/nginx"
        ]
    }
}

Check the mounts on the host with findmnt /var/lib/kubelet/pods/d0b79a40-6a69-47d9-a7e4-ef36331732e2/volume-subpaths/site-data/nginx/0 this will return:
TARGET                                                                                SOURCE                 FSTYPE OPTIONS
/var/lib/kubelet/pods/d0b79a40-6a69-47d9-a7e4-ef36331732e2/volume-subpaths/site-data/nginx/0 none[/site-data/nginx] aufs   rw,relatime,si=2fb3ccf5b8b9c475,dio,dirperm1

From this we can see the SOURCE has the correct <hostPath>/<subPath> but if we check the node for this path with: stat /site-data/nginx we get no such file or direcotry. So what's happening? If we exec into the kubelet we can get more information. Do the following:

docker exec -it $(docker ps -aq --filter "name=kubelet") /bin/sh
Once inside the kubelet, run: stat /site-data/nginx

We can see from the output that /site-data/nginx actually does exist in the kubelet, however, since the kubelet is containerized and does not share a filesystem with the node, /site-data/nginx does not exist on the node, where it should.
This gives us more information on the previous findmnt output. It tells us that a filesystem that lives in the kubelet at /site-data/nginx is being mounted to /var/lib/kubelet/pods/d0b79a40-6a69-47d9-a7e4-ef36331732e2/volume-subpaths/site-data/nginx/0 on the kubernetes node. It is able to be mounted to this path on the node because there is a bind mount from the node's /var/run/kubelet directory to the container's /var/run/kubelet directory, as can be seen from the kubelet container's binds:
{
    "HostConfig": {
        "Binds": [
            ...
            "/var/lib/kubelet:/var/lib/kubelet:shared,z",
            ...
        ]
}

This is not the desired functionality as there no presence of /<hostPath>/<subPath> on the host, only in the containerized kubelet.
If we wanted to confirm the above we could take some manual steps to make this work, i.e. get /site-data/nginx to be populated on the host. Here's what we can do:

Create the target directory on the host with: mkdir -p /site-data/nginx
Create a bindmount with: mount --bind /var/lib/kubelet/pods/d0b79a40-6a69-47d9-a7e4-ef36331732e2/volume-subpaths/site-data/nginx/0 /site-data/nginx

With these steps taken, we have the following:

Now, if we add to the hostPath/subPath volume in the nginx pod, we should actually see the files in the kubelet and on the host.

Exec into the nginx pod's corresponding docker container with: docker exec -it $(docker ps -aq --filter "label=io.kubernetes.container.name=nginx") /bin/sh
Create a file in the container at the mount point with: echo "hello" > /volume/nginx/f.txt
Exit container and check file on host with: cat /site-data/nginx/f.txt (this should output hello)
Exec into the kubelet with: docker exec -it $(docker ps -aq --filter "name=kubelet") /bin/sh
Validate in the kubelet container with: cat /site-data/nginx/f.txt (this should output hello)

Taking this one step further, we can setup a k8s cluster that does not use a containerized kubelet. This can be accomplished most easily with k3d or minikube. The output from the findmnt /var/lib/kubelet/pods/d0b79a40-6a69-47d9-a7e4-ef36331732e2/volume-subpaths/site-data/nginx/0 command here will look something like:
TARGET                                                                                SOURCE                  FSTYPE OPTIONS
/var/lib/kubelet/pods/d0b79a40-6a69-47d9-a7e4-ef36331732e2/volume-subpaths/site-data/nginx/0 ext4[/site-data/nginx] ext4  rw,relatime,size=3543256k

We can see that a filesystem was created at /site-data/nginx and mounted to /var/lib/kubelet/pods/d0b79a40-6a69-47d9-a7e4-ef36331732e2/volume-subpaths/site-data/nginx/0, which in turn is mounted as a volume to the corresponding container. The difference here is that, since the kubelet is running directly on the kubernetes node, the mount's source (ext4[/site-data/nginx]) actually lives on the host and not inside of a container. This allows for files written to /volume/nginx inside the container to be persisted back to the host filesystem.
Other k8s clusters which use a containerized kubelet will run into this same issue with hostPath volumes using subPath. I confirmed this by creating a Digital Ocean managed cluster and deploying the above nginx deployment which uses a subPath. Any files added to /volume/nginx in the pod's corresponding container were not populated on the host at /site-data/nginx.
Alternatives
Instead of using a hostPath volume with a subPath, multiple hostPath volumes without a subPath could be used.
Additional information
There was previously a workaround where the kubelet's container could be run with a volume -v /:/rootfs:rshared mounting the host's root filesystem inside the container at /rootfs, along with the kubelet process being given the --containerized flag which makes the kubelet aware of /rootfs. This allowed for hostPath volumes using a subPath to work as expected, but due to maintenance efforts by the upstream, this flag was deprecated in kubernetes version 1.14 and removed in version 1.16. As such, it is no longer an option.


Feature request: Cancel move action if file exists

Hi!
I would like to request a feature on the move action. I would like to be able to specify the move action to cancel if the matching file already exists on the destination path.
As is now it can make a new file or overwite it, but it can't cancel the operation.
Many thanks for a great program!
Johnny

Hi, yes that would definitely make sense. Only thing I'm struggling with at the moment is how to introduce this without breaking old config files. I added it to my todo list 👍

Awesome! I really looking forward to the feature.
Many thanks for the time and effort you put down in the development of this awesome program.
Best regards,
Johnny

Hey Johnny, this is now built in!
Have a look at the move action where you can now define what to do in case of a conflict. Options are skip, overwrite, trash, rename_new and rename_existing. So skip would work for you 👍


Editor does not grab focus from fullscreen game upon script error

Godot version
4.0.3 stable [5222a99f5]
System information
Windows 10 - Godot v4.0.3.stable - Vulkan (Forward+)
Issue description
The bug results in your debug window not rendering anything and appearing to crash. It also messes with alt-tabbing; you can still alt-tab, but the window will still be over whatever window you've alt tabbed to. The crash also takes a while to close by itself and doesn't go white like crashes usually do. Important note! This only works when launch in fullscreen mode is set. It should just take you to the code editor like it does when not in fullscreen mode
Video of Bug:
https://github.com/godotengine/godot/assets/136120846/7b5f9d23-0986-4629-b4a3-d0ff95107312
A few notes:
I recorded this using OBS display capture. It recorded slightly differently then what i was seeing. I was seeing the default grey colour you get from empty scenes. but OBS recorded black.
When you see the cursor switch from the loading symbol to the typing cursor that's me Alt-Tabbing in and out of the window. It stays grey in my view, and stays black in OBS's.
There are no reports from Godot whatsoever; besides it saying debug closed.
Steps to reproduce
The easiest way to do it is:

go to "Project > Project Settings > Display > Window > Size" and set mode to Fullscreen
make a scene (any node)
give the node a script
copy paste this in there then hit play:

extends Node

func _ready():
	var variable = Vector2.ZERO
	process_mode = variable

Minimal reproduction project
N/A

I can confirm this on 4.1.dev 06ccbfe8e (Linux).
Even if the window isn't set to the fullscreen, the editor window doesn't get focus anymore on script errors. This also indirectly resolves https://github.com/godotengine/godot/issues/21812, but I guess it goes too far in the other direction :slightly_smiling_face:

Important note! This only works when launch in fullscreen mode is set.


Consider implementing Debug for Client

Is your feature request related to a problem? Please describe.
When trying to install a global configcat::Client inside a std::sync::OnceLock like this:
use std::sync::OnceLock;

static CONFIGCAT: OnceLock<configcat::Client> = OnceLock::new();

fn main() {
    let client = todo!();
    CONFIGCAT.set(client).unwrap();
}

I ran into the following issue:
error[E0277]: `Client` doesn't implement `Debug`
    --> src/main.rs:7:27
     |
7    |     CONFIGCAT.set(client).unwrap();
     |                           ^^^^^^ `Client` cannot be formatted using `{:?}` because it doesn't implement `Debug`
     |
     = help: the trait `Debug` is not implemented for `Client`
note: required by a bound in `Result::<T, E>::unwrap`
    --> /Users/cstyles/.rustup/toolchains/stable-aarch64-apple-darwin/lib/rustlib/src/rust/library/core/src/result.rs:1073:12
     |
1071 |     pub fn unwrap(self) -> T
     |            ------ required by a bound in this associated function
1072 |     where
1073 |         E: fmt::Debug,
     |            ^^^^^^^^^^ required by this bound in `Result::<T, E>::unwrap`

Essentially, OnceLock::<T>::set returns a Result<(), T> and Result::<T, E>::unwrap requires E: Debug.
Describe the solution you'd like
If configcat::Client provided an implementation of Debug this would compile. Even if it's just f.write_str("<Client>").
Describe alternatives you've considered
The lack of a Debug implementation can be worked around by mapping the error into something else:
CONFIGCAT
    .set(client)
    .map_err(|_| "error installing Configcat client")
    .unwrap();

Additional context
Thanks for this library! ❤

Hi @cstyles, thank you for reporting this issue! I've released a new version (0.1.2), which now contains a Debug implementation for Client.

Thanks for the quick response!


boltbrowser: init at 2.0

Motivation for this change
A CLI Browser for BoltDB Files
https://github.com/br0xen/boltbrowser
Things done

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

[x] NixOS
[ ] macOS
[ ] other Linux distributions

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

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

1 package built:

boltbrowser


add retry logic to client and add support for ruby 3 and single vector search

Add logic, all specs pass

also it brings to milvus 2 i guess compat. thats the version im using

@washu Any chance that we can separate this PR into separate ones? For example dropping Ruby 2.7 compatibility could be a separate PR.

Is there an update for this?

@washu @bendangelo I switched the gem to use v2 API. If you're running Milvus yourself you should be using port: 19530


Added PROPFIND to valid methods

This issue has been moved from the zendframework repository as part of the bug migration program as outlined here - http://framework.zend.com/blog/2016-04-11-issue-closures.html

Original Issue: https://api.github.com/repos/zendframework/zendframework/issues/6793
User: @lkneschke
Created On: 2014-10-23T07:04:06Z
Updated At: 2015-09-14T15:54:52Z
Body

Comment
User: @lkneschke
Created On: 2014-10-23T07:07:16Z
Updated At: 2014-10-23T07:07:16Z
Body
Would be nice, if this problem can be fixed in the next 2.2.x release too.

Comment
User: @macnibblet
Created On: 2014-10-31T14:32:21Z
Updated At: 2014-10-31T14:32:21Z
Body
Can you add a test case ?

Comment
User: @lkneschke
Created On: 2014-10-31T14:35:19Z
Updated At: 2014-10-31T14:35:19Z
Body
I can. I'll add a test during the next days.

Comment
User: @weierophinney
Created On: 2015-02-19T20:04:58Z
Updated At: 2015-02-19T20:04:58Z
Body
@lkneschke Still need test cases...

Comment
User: @corneliusweiss
Created On: 2015-09-14T15:54:52Z
Updated At: 2015-09-14T15:54:52Z
Body
with fromString($string, $allowCustomMethods = true) this change might be obsolete

I'm closing this issue due to inactivity.


feat(backend): Add URL validation for Repositorys

Check individual commit messages for details.

The database migration script exceeds my SQL fu. Has it been tested on a real database containing real ORT run data?


DataLoader wrapping, re-instantiation and patching

🐛 Bug
Recent changes to the Lite DataLoader in #10279 have not been properly reviewed and tested. Several clean ups need to be done and better tests need to be written.
The following issue exists:
To Reproduce
def test_setup_custom_dataloaders_new():

    class DataLoaderSubclass1(DataLoader):
        def __init__(self, attribute1, *args, **kwargs):
            # UNCOMMENT THE LINE BELOW, NO DIFFERENCE (EXPECTED)
            # self.attribute1 = attribute1
            super().__init__(*args, **kwargs)

    class DataLoaderSubclass2(DataLoaderSubclass1):
        def __init__(self, attribute1, attribute2, *args, **kwargs):
            # UNCOMMENT THE LINE BELOW, TEST PASSES
            # self.attribute2 = attribute2
            super().__init__(attribute1, *args, **kwargs)

    class LiteWithCustomDataLoader(LightningLite):
        def run(self):
            dataloader = DataLoaderSubclass2("attribute1", "attribute2", dataset=range(4), batch_size=2)
            lite_dataloader = self.setup_dataloaders(dataloader)

            # THIS ASSERTION PASSES
            assert lite_dataloader._dataloader.attribute1 == "attribute1"

            # THIS ASSERTION FAILS
            assert lite_dataloader._dataloader.attribute2 == "attribute2"

    LiteWithCustomDataLoader().run()

Expected behavior
According to the changes in the PR #10279, this is supposed to work and was the main motivation of it. But it does not because the tests were insufficient. This issue was found by me in #10297 while trying to clean up the code.
Environment
Lightning v1.5.0!

The issue exists because this wrapper here
https://github.com/PyTorchLightning/pytorch-lightning/blob/93caa7cda9277f86f17b1eab0793019aee8d40c2/pytorch_lightning/lite/wrappers.py#L106-L116
pops args and kwargs.
When we get to this line here
https://github.com/PyTorchLightning/pytorch-lightning/blob/93caa7cda9277f86f17b1eab0793019aee8d40c2/pytorch_lightning/lite/lite.py#L242-L244
We are re-instantiating (with the new init) but passing arguments by kwargs, strictly! This means the for loop here runs empty!
https://github.com/PyTorchLightning/pytorch-lightning/blob/93caa7cda9277f86f17b1eab0793019aee8d40c2/pytorch_lightning/lite/wrappers.py#L110-L114
What is the design flaw? As always, it is the fact that we are making too strong assumptions without documenting them and without testing for them.

A few things that need to be addressed (discussed with @tchaton and @carmocca):

[ ] Bugfix PR #10334
[ ] Revert dataset TypeError, properly handle it and raise a MisconfigurationError @carmocca
[ ] Also use the the context manager around dataloader methods in Lightning
https://github.com/PyTorchLightning/pytorch-lightning/blob/1c28f361d47257feb0bd34cbc8d41642a1f0a2ab/pytorch_lightning/lite/lite.py#L408-L410
[ ] Open PR to PyTorch Geometric to improve support for their DataLoaders

One minor limitation of the patching is that the dataloader needs to allow write access to it's dict, i.e. something like this would be a valid DataLoader but our patching would fail:
class MyDataLoader(DataLoader):

    def __init__(self, whatever, *args, **kwargs):
        # Lightning will not be allowed to set the attribute self.whatever here (read-only)
        super().__init__(*args, **kwargs)

    @property
    def whatever(self):
        return "foo"

We may want to handle this error case.

In practice, it would be instead:
class MyDataLoader(DataLoader):

    def __init__(self, whatever, *args, **kwargs):
        self._whatever = whatever
        super().__init__(*args, **kwargs)

    @property
    def whatever(self):
        return self._whatever

@carmocca @tchaton Do you remember what we wanted to do with "Open PR to PyTorch Geometric to improve support for their DataLoaders"?
Was it about removing these lines?
https://github.com/pyg-team/pytorch_geometric/blob/4bd1821b72039cb6beb96041bc9f18b2c4af36bb/torch_geometric/loader/dataloader.py#L74-L76
But we need to wait until at least 1.6 is out.

No, it was about addressing the failure which made @tchaton add the TypeError catch. But I'm not sure which implementation caused the error, most already drop the arguments from kwargs.

Hey @awaelchli,
I wonder if it would be this one: https://github.com/pyg-team/pytorch_geometric/blob/4bd1821b72039cb6beb96041bc9f18b2c4af36bb/torch_geometric/loader/graph_saint.py#L50 where the dataset isn't being poped from the kwargs, but this is maybe expected.


Added NVIDIA PyData Open Source posting

Thanks Mike! The post is on the board and also at permalink https://jobs.numfocus.org/job/2022-03-01_nvidia

Great! Thanks Stefan!


Add isdevice

Description:
add new validate function isdevice they can handle the Trouble from #3431.
Any change on isfile will brake that guaranty that the file exists.
Related issue (if applicable): fixes #3431
If the code does not interact with devices:

[x] Local tests with tox run successfully. Your PR cannot be merged unless tests pass
[x] Tests have been added to verify that the new code works.

A Symlink can be exist without a file behind. So it's give a true but we can open a file and fails... We can add this thing to platform and don't make it to cobfig validation, thas is my meaning.


[msal-browser][msal-common] Instance Aware Authentication (updated)

This PR implements the instance_aware feature for msal-browser. Specifically, it adds the authority object to the response and handles all parameters which come back in the fragment.
It also adds TokenResponse to the browser exports (as per PR #1551).
Replaces PR #1584.

@pkanher617 Status on this PR?

@pkanher617 Status on this PR?

Need to update this to latest dev and follow up with proper caching schema. Will update in the next week.

Generally implementation looks ok, a few comments and would like to see unit tests.
Unit tests are updated, and I am adding more for the scenarios around instance_aware. Also going to add e2e tests if I can.

Also curious if the msal-common additions are type safe, given stricter type checking is imminent.
Would like to merge the strictNullChecks PR before I fully confirm this, but I think we should be fine.

Coverage decreased (-0.7%) to 81.278% when pulling b921279b4f285d2984b8458f27348d3f734b0d67 on instance-aware-2.0 into 7252087b80db84ffafe1cf76d48dd767277a2ae5 on dev.


请问有Tongyi Qwen的对接计划吗？

Please write issue titles and descriptions in english please. Please consider adding an example showing use.

I'm going to close this as the linked issue overlaps, please feel free to pick up the conversation as you see fit.

DashScope 模型服务灵积 https://dashscope.console.aliyun.com/

Tanks.


Generic type/scanner?

I'm trying to connect to my Di2 system on my bike and see when the hood-buttons are pressed but Iv'e heard they use a pretty custom profile
Is there a way to just connect to all/any Ant+ devices and spit out any raw data that may come from them?

I got it to work with my Di2 thanks to @kwakeham
If you set the
NetworkKey to 0xA9, 0xAD, 0x32, 0x68, 0x3D, 0x76, 0xC7, 0x4D
Device Type to 0x1
Channel Period to 8198
You can read out the button presses


The X-Files PS1 with BlueRetro & 8BitDo Pro 2 no IGR

Kills my reset combos for IGR
Only game I know of.
If you know how to enable rumble (in-game) also, let me know :)
I get a rumble to let me know the controllers synced, but nothing in game. I have the controller set to "X" on the rear switch.

Ok, I read up and saw the OTA updates (only one of my BlueRetro dongles has USB) and although I lost my BT dongle for the desktop and i CBF setting up my laptop that's being unused since moving a few months back - I used my phone. It only took a couple minutes to update using Chrome on android, which was good. One dongle was even faster!
So I'm on 1.8.2 HW1 now and used the BlueRetro_playstation.bin (they both came with 1.7.1 psx_external)
Rumble enabled in the settings now-  though still no rumble, which is a little disappointing..
As for the IGR, the X-files seems to enable analogue mode on the controllers once it gets past the intro video/FMV.
Prior to this you can reset with the combos.
Off memory the IGR is simple and obviously with a product/ project like this, you'd know all about it
it's a few constant bytes and the 2 variables for button input
1: FFh   2: 41h (digital)/73h (analogue) 3: 5Ah  4: Inputs  5 Inputs
So
Long story short, i disable analogue and the combos work.
I tried in another game, analogue mode broken on that too(weird, i tested both modes before finalising the tiny bit of code and re-assembling the console).
I will debug it eventually... but i guess this is case closed. Really confused about this one!
If you get rumble working on 8BitDo Pro 2 that would be awesome :)

Ok and I guess the IGR do work with a real DS1 in analog mode?

No, but AFAIK analogue wasn't automatically switched on using a genuine controller on that game.
I'll have a proper check tomorrow.
It was definitely working prior to reassembly so a little confused here. Was certain both modes were tested after assembly too...oh well
Get Outlook for Androidhttps://aka.ms/AAb9ysg

From: Jacques Gagnon @.>
Sent: Friday, January 20, 2023 4:00:35 AM
To: darthcloud/BlueRetro @.>
Cc: VajskiDs @.>; Author @.>
Subject: Re: [darthcloud/BlueRetro] The X-Files PS1 with BlueRetro & 8BitDo Pro 2 no IGR (Issue #566)
Ok and I guess the IGR do work with a real DS1 in analog mode?
—
Reply to this email directly, view it on GitHubhttps://github.com/darthcloud/BlueRetro/issues/566#issuecomment-1397307751, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AUYALNDE3Y4AHASLYHND3ODWTFXLHANCNFSM6AAAAAAUACBPCU.
You are receiving this because you authored the thread.Message ID: @.***>


Compiler incorrectly infers the type of literal Hashes containing classes

Bug Report
I ran into what appears to be incorrect behavior when attempting to define a mapping of numeric bases (16, 10, 8, 2) to their corresponding classes. However, the compiler seems to be inferring the type of the literal Hash to be Hash(Int32,Base.class). Even when I explicitly define the type of the Hash using of Int32 => Hexadecimal | ..., it still infers the type to be Hash(Int32,Base.class). This causes problems if you happen to have other classes which inherit from Base but are not defined in BASES; which the compiler will incorrectly error on, thinking you are initializing the Base class directly, or sometimes assumes you could possibly initialize CharOrInt because it inherits from Base but is not listed in BASES.
Example
abstract class Base
  def initialize(@fmt : String)
  end
end

class Hexadecimal < Base
  def initialize
    super("%.x")
  end
end

class Decimal < Base
  def initialize
    super("%.d")
  end
end

class Octal < Base
  def initialize
    super("%.o")
  end
end

class Binary < Base
  def initialize
    super("%.b")
  end
end

class CharOrInt < Base
  def initialize(@base : Base)
    super("%s")
  end
end

class Test

  BASES = {
    16 => Hexadecimal,
    10 => Decimal,
    8  => Octal,
    2  => Binary
  }

  getter base : Hexadecimal | Decimal | Octal | Binary | CharOrInt

  def initialize(base : Int32 = 16, char_or_int : Bool = false)
    @base = BASES[base].new

    if char_or_int
      @base = CharOrInt.new(@base)
    end
  end

end

p Test.new.base

Error
In test.cr:50:25

 50 | @base = BASES[base].new
                          ^--
Error: wrong number of arguments for 'Base#initialize' (given 0, expected 1)

BASES With Explicit Type
  BASES = {
    16 => Hexadecimal,
    10 => Decimal,
    8  => Octal,
    2  => Binary
  } of Int32 => Hexadecimal.class | Decimal.class | Octal.class | Binary.class

In test.cr:50:25

50 | @base = BASES[base].new
                        ^--
Error: wrong number of arguments for 'Base#initialize' (given 0, expected 1)

Inspecting BASES
class Test

  BASES = {
    16 => Hexadecimal,
    10 => Decimal,
    8  => Octal,
    2  => Binary
  } of Int32 => Hexadecimal.class | Decimal.class | Octal.class | Binary.class

end

p typeof(Test::BASES)

Hash(Int32, Base.class)

This is expected behaviour because class metaclasses unify in the same way as normal classes in a hierarchy.

@HertzDevil I could use a module for common methods, although then I'm having to change my program to workaround a limitation of the type system.

What did surprise me a bit is the error that is emitted if
abstract class Base
  abstract def initialize
end

is added. Then you get
In foo.cr:3:16

 3 | abstract def initialize
                  ^---------
Error: this 'initialize' doesn't explicitly initialize instance variable '@fmt' of Base, rendering it nilable

which seems pretty strange, considering that abstract methods cannot initialize anything as they defer their definition to any subtypes.. I'd expect the implementations in the subclasses to error in that case if they didn't fulfill the requirement.


High Cpu caused by supervisord

Context

Linux 5.4.0-29-generic
Ubuntu 20.04 LTS
The mailserver is working fine, except for high cpu usage caused by supervisord
i've tried to set all stderr_logfile and stdout_logfile to NONE but nothing has changed
it usage about 15% of cpu on a xeon and this isn't good
How do we replicate the issue?

I don't know how, this is a normal installation
Actual Behavior

High Cpu usage by supervisord
Expected behavior (i.e. solution)

Not use so much cpu
Your Environment

Amount of RAM available: 64GB
Mailserver version used: tvial/docker-mailserver:latest
Docker version used: Docker version 19.03.8, build afacb8b7f0
Environment settings relevant to the config:
Any relevant stack traces ("Full trace" preferred):

Dockerfile
FROM tvial/docker-mailserver:latest
RUN sed -i -r 's/stdout_logfile=.*/stdout_logfile=NONE/' /etc/supervisor/conf.d/supervisor-app.conf
RUN sed -i -r 's/stderr_logfile=.*/stderr_logfile=NONE/' /etc/supervisor/conf.d/supervisor-app.conf

docker-compose.yml
version: '3.3'
services:
  mail:
    #image: tvial/docker-mailserver:latest
    image: mailserver
    build:
      context: /hangar/mail/
      dockerfile: Dockerfile
    hostname: ${HOSTNAME}
    domainname: ${DOMAINNAME}
    container_name: ${CONTAINER_NAME}
    restart: unless-stopped
    #network_mode: host
    ports:
    - "25:25"
    - "143:143"
    - "587:587"
    - "993:993"
    volumes:
    - maildata:/var/mail
    - mailstate:/var/mail-state
    - maillogs:/var/log/mail
    - ./config/:/tmp/docker-mailserver/
    - /etc/localtime:/etc/localtime:ro
    env_file:
    - .env
    - env-mailserver
    cap_add:
    - NET_ADMIN
    - SYS_PTRACE
    restart: unless-stopped
volumes:
  maildata:
    driver: local
  mailstate:
    driver: local
  maillogs:
    driver: local

Hi,
Do you have any error in your logs? Please check /var/log/supervisor/*.log inside the container too.
It could be related to https://github.com/Supervisor/supervisor/issues/807

Hi,
Do you have any error in your logs? Please check /var/log/supervisor/*.log inside the container too.
It could be related to Supervisor/supervisor#807

hi,
the only error are in
/var/log/supervisor/rsyslog.log
rsyslogd: imklog: cannot open kernel log (/proc/kmsg): Operation not permitted. rsyslogd: activation of module imklog failed [v8.1901.0 try https://www.rsyslog.com/e/2145 ]
and in
/var/log/supervisor/fetchmail.log ->the file is full of the line below. maybe this is causing the cpu usage?
fetchmail: no mailservers have been specified.

rsyslogd: imklog: cannot open kernel log (/proc/kmsg): Operation not permitted. rsyslogd: activation of module imklog failed [v8.1901.0 try https://www.rsyslog.com/e/2145 ]

Interesting! you found a bug which happens because the container is not privileged.
See https://stackoverflow.com/questions/56609182/openthread-environment-docker-rsyslogd-imklog-cannot-open-kernel-log-proc-km/60265997#60265997
We should patch it but I doubt it's the cause of hight cpu usage.

the file is full of the line below. maybe this is causing the cpu usage?

Seen the fetchmail poll is done every 300s; I don't think so. (https://github.com/tomav/docker-mailserver#fetchmail_poll)

Interesting! you found a bug which happens because the container is not privileged.
See https://stackoverflow.com/questions/56609182/openthread-environment-docker-rsyslogd-imklog-cannot-open-kernel-log-proc-km/60265997#60265997

with privileged:true in docker-compose this error is resolved. the cpu usage still high

Seen the fetchmail poll is done every 300s; I don't think so. (https://github.com/tomav/docker-mailserver#fetchmail_poll)

my setting in the env file is 300s
so if that isn't the problem what could be?

The bug when container is not privileged is old and as far as I understand harmless. Fetchmail should also be harmless. Perhaps you can trace supervisord (see for example https://nixcp.com/trace-linux-process/) and see what it is doing?
Plus please remove the sed commands that disables logging. If you get the supervisor logs back perhaps they can tell you something!

Plus please remove the sed commands that disables logging. If you get the supervisor logs back perhaps they can tell you something!

Disabled the sed command, the log output still only for fetchmail
root@mail:/# tail -n 10 /var/log/supervisor/*
==> /var/log/supervisor/amavis.log <==

==> /var/log/supervisor/changedetector.log <==
2020-05-19 12:10:34  Start check-for-changes script.
2020-05-19 12:10:34  Using postmaster address [omitted]

==> /var/log/supervisor/cron.log <==

==> /var/log/supervisor/dovecot.log <==

==> /var/log/supervisor/fetchmail.log <==
fetchmail: no mailservers have been specified.
fetchmail: no mailservers have been specified.
fetchmail: no mailservers have been specified.
fetchmail: no mailservers have been specified.
fetchmail: no mailservers have been specified.
fetchmail: no mailservers have been specified.
fetchmail: no mailservers have been specified.
fetchmail: no mailservers have been specified.
fetchmail: no mailservers have been specified.
fetchmail: no mailservers have been specified.

==> /var/log/supervisor/opendkim.log <==

==> /var/log/supervisor/opendmarc.log <==

==> /var/log/supervisor/postfix.log <==
Starting Postfix Mail Transport Agent: postfix.

==> /var/log/supervisor/rsyslog.log <==

==> /var/log/supervisor/supervisord.log <==
2020-05-19 12:19:51,229 INFO spawned: 'fetchmail' with pid 114036
2020-05-19 12:19:51,233 INFO success: fetchmail entered RUNNING state, process has stayed up for > than 0 seconds (startsecs)
2020-05-19 12:19:51,233 INFO exited: fetchmail (exit status 5; not expected)
2020-05-19 12:19:51,234 INFO spawned: 'fetchmail' with pid 114037
2020-05-19 12:19:51,238 INFO success: fetchmail entered RUNNING state, process has stayed up for > than 0 seconds (startsecs)
2020-05-19 12:19:51,238 INFO exited: fetchmail (exit status 5; not expected)
2020-05-19 12:19:51,239 INFO spawned: 'fetchmail' with pid 114038
2020-05-19 12:19:51,243 INFO success: fetchmail entered RUNNING state, process has stayed up for > than 0 seconds (startsecs)
2020-05-19 12:19:51,244 INFO exited: fetchmail (exit status 5; not expected)
2020-05-19 12:19:51,244 INFO spawned: 'fetchmail' with pid 114039

The bug when container is not privileged is old and as far as I understand harmless. Fetchmail should also be harmless. Perhaps you can trace supervisord (see for example https://nixcp.com/trace-linux-process/) and see what it is doing?

i've tried with strace -e trace=open,read -p 2958004 -s 80 -o ./supervisord-trace.txt
the file only contains this(22mb):
--- SIGCHLD {si_signo=SIGCHLD, si_code=CLD_EXITED, si_pid=48863, si_uid=106, si_status=5, si_utime=0, si_stime=0} ---
read(50, "", 131072)                    = 0
read(45, "", 131072)                    = 0
read(50, "fetchmail: no mailservers have been specified.\n", 131072) = 47
--- SIGCHLD {si_signo=SIGCHLD, si_code=CLD_EXITED, si_pid=48864, si_uid=106, si_status=5, si_utime=0, si_stime=0} ---
read(50, "", 131072)                    = 0
read(45, "", 131072)                    = 0
--- SIGCHLD {si_signo=SIGCHLD, si_code=CLD_EXITED, si_pid=48865, si_uid=106, si_status=5, si_utime=0, si_stime=0} ---
read(50, "fetchmail: no mailservers have been specified.\n", 131072) = 47
read(50, "", 131072)                    = 0
read(45, "", 131072)                    = 0
read(50, "fetchmail: no mailservers have been specified.\n", 131072) = 47

Well, fetchmail is the problem, it shouldn't crash. However, since you haven't given any mail servers I guess it should be off? Can you set ENABLE_FETCHMAIL=0?

Well, fetchmail is the problem, it shouldn't crash. However, since you haven't given any mail servers I guess it should be off? Can you set ENABLE_FETCHMAIL=0?

setting this to 0 solve the problem
thanks for the support

No problem, great that we found a solution!


Add support for InMemoryRelation

What is the problem the feature request solves?
Currently Comet cannot be triggered if Spark users read data from cached RDD. To support this use case, we'll need to add support for Spark's InMemoryRelation.
It looks like we may need to implement Arrow for CachedBatchSerializer.
Describe the potential solution
Add Comet support for InMemoryRelation, so that Spark query starts from cached RDD can also use Comet native execution.
Additional context
It is not a priority as of now, but will be something good to have in future.

@advancedxy Yea, CometRowToColumnarExec could be a more general solution, not only for InMemoryRelation, but also for other types of data sources like CSV, JSON, etc. The advantage of implementing Arrow for CachedBatchSerializer here is that we can avoid the extra cost from row to columnar conversion, and potentially be more space efficient because of better compression.


Adds a new Config Abstract Factory that allows service creation by config

As per issue #128.
This PR fixes everything suggested in PR #143 but opened a new PR for clarity of new comments.
Ping @weierophinney @Ocramius.

Travis failing is fully expected as this is 5.6+ only.

Added documentation, I'm actually happy with this now @weierophinney.

For documentation with code example: 👍 👍 👍

Travis failing is fully expected as this is 5.6+ only.

Since this is for the next minor release, feel free to bump the minimum supported PHP version to 5.6. :smile:

Looks good with the exception of a few documentation changes I've requested. Ping me when done, and I'll merge!
Thanks, @GeeH!

Ping @weierophinney.

Okay, one last comment, and then we can wrap up: It might be a good idea to demonstrate in the documentation a migration path from this abstract factory to an explicit factory. One of the frequent comments I've seen from folks about the deprecation of ServiceLocatorAware is that developers are unaware of how to write factories. Including some explicit steps here will help them when they have the question of, "I need to do something that falls outside the scope of this abstract factory, but I don't know how."
(And yes, I'm aware you're planning on writing a tool for auto-creation of factories from this configuration and/or to create the configuration, but having docs now is never a bad thing.)

I really don't want to look like I'm ducking doing anything more because both points you have made are valid, but I think these points outline an overarching problem with the documentation in general. For me, the component documentation should be dry instructions with examples on how to use this component either stand-alone or in the context of other components such as the MVC. But of course, we also need more "tutorial" style documentation that says "if you outgrow this, you can go here next" or "this is the pros and cons of using abstract factories". I'm 100% onboard that this extra information is needed, but I'm not convinced that this document is the right place for them.
@froschdesign @weierophinney I would love to hear your thoughts on this.

@GeeH I'll go ahead and merge, and then see if I can figure out good locations for that information. In some of the other components, we have "cookbook" sections, and adding such a section might be the path we should take here. If so, it is beyond the scope of the feature, as you note, so we can do it separately.

I'll make sure there are issues created to capture this.

@GeeH and @weierophinney
The "tutorial style documentation" is the "cookbook"! So the documentation is the right and only place.

Awesome feature, @GeeH! Merged to develop for the 3.2.0 release.


Boad support layer for esc-v0

Add a simple board-support package
We need board support packages defining pin mappings and other useful utilities for every board we design.
This boardsupport layer will act as a template for future bsp crates in the future.

https://github.com/Ltu-Bike-trailer/ESC/blob/bsp/bsp/src/lib.rs

This is done, it will be slightly modified as versions change but the pin map is done.


Implemented rust extension

The Rust extension deals with images in tiles instead of all at once. It implements a slightly different algorithm than the Cython version. It runs somewhat slower than the cython version on the same same sized image but easily scales to images of arbitrarily large sizes.
Small images (<500MB in size) are still handled by Cython while larger images are now handled by Rust.

I've moved this work into the reorganized folder and am reopening the PR against dev instead of master.


Error while installing

When I execute vita-makepkg I get an error at the end, so I looked at what went wrong a saw this:
In file included from core.c:133: rlgl.h:693:14: fatal error: GLES2/gl2.h: No such file or directory   693 |     #include <GLES2/gl2.h>              // OpenGL ES 2.0 library       |              ^~~~~~~~~~~~~ compilation terminated.
I checked if /usr/include/GLES2/gls2.h exists and it does. I had this result on two seperate machines, one running pop os, the other one arch.

try to install https://github.com/SonicMastr/Pigs-In-A-Blanket for vitasdk you will need this installed first.

Thanks, that fixed the issue.


Is forward reference in SVD file allowed?

Hi,
during work with svd file in ST ecosystem and outside i found patched SVD files and when I try used it there's error:

Then I used svdconv to check and it return a lot of errors something like "*** ERROR M206: stm32f7x6.svd.patched (Line 46729) DerivedFrom not found: 'USART1'"
The solution was change the ordering enumeratedValue derivedFrom=, for example:
<peripheral derivedFrom="USART1">
      <name>USART6</name>
      <baseAddress>0x40011400</baseAddress>
      <interrupt>
        <name>USART6</name>
        <description>USART6 global interrupt</description>
        <value>71</value>
      </interrupt>
    </peripheral>
    <peripheral>
      <name>USART1</name>
      <description>Universal synchronous asynchronous receiver       transmitter</description>
      <groupName>USART</groupName>
      <baseAddress>0x40011000</baseAddress>

to
<peripheral>
      <name>USART1</name>
      <description>Universal synchronous asynchronous receiver       transmitter</description>
      <groupName>USART</groupName>
      <baseAddress>0x40011000</baseAddress>
......
<peripheral derivedFrom="USART1">
      <name>USART6</name>
      <baseAddress>0x40011400</baseAddress>
      <interrupt>
        <name>USART6</name>
        <description>USART6 global interrupt</description>
        <value>71</value>
      </interrupt>
    </peripheral>

My question is: The forward references are allowed?
If yes: svdconv should handle it usecase
If no: please point me reference to the restriction in specification
Simillar discussion in below issues:
https://github.com/Marus/cortex-debug/issues/513
https://github.com/stm32-rs/stm32-rs/issues/824

Thanks. I understand that  "forward reference" feature is not going to implement?

@KamilDuljas, there is currently no plans to implement support for forward references.
Since this is an open source project, we welcome contributions that help to improve the specification and tools within the scope of intended use cases.

Hi. Could you attach link to updated documentation related to issue?

Sorry, overlooked that we need to add information to the documentation.

Forward references are currently not supported by SVDConv. This is because in SVD you can deriveFrom other derived, dim-ed or mixed sections, e.g. derive from "TIMER1" in a dimed TIMER { 0..3 }.


File renaming is inaccessible

Issue Type: Bug

Press f2 to rename a file.

I expect NVDA to read it like it would any edit field, but arrowing around results in no feedback at all.
VS Code version: Code - Insiders 1.28.0-insider (431ef9da3cf88a7e164f9d33bf62695e07c6c2a9, 2018-10-03T12:44:29.343Z)
OS version: Windows_NT x64 10.0.17134

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-8550U CPU @ 1.80GHz (8 x 1992)

GPU Status
2d_canvas: enabledchecker_imaging: disabled_offflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onnative_gpu_memory_buffers: disabled_softwarerasterization: enabledvideo_decode: enabledvideo_encode: enabledwebgl: enabledwebgl2: enabled

Memory (System)
15.86GB (10.04GB free)

Process Argv

Screen Reader
yes

VM
40%

Extensions (6)

Extension
Author (truncated)
Version

vscode-markdownlint
Dav
0.20.0

vscode-eslint
dba
1.6.0

python
ms-
2018.8.0

cpptools
ms-
0.19.0

rust
rus
0.4.10

markdown-preview-enhanced
shd
0.3.8

@Neurrone I just tried this and I can repro.
However after trying it with our latest VSCode which uses Electron 3.0 version I see this is fixed. So those are great news.
The issue here is how we are doing accessibility for our trees inside vscode, we do not use tabindex inside the tree such that the user would not be lost once he is inside a large tree, but we use the aria-active descendent. And there seemed to be an issue that no all details are read out for active descendent items (since they do not have the tabindex property). New chrome seems to fix this and the input box is properly respected and read out if it is the active descendent in a tree.
In case you want this fixed already and not wait for us to update to electron 3.0 you can use our experimental build:

Windows: Download

@bpasero fyi

@isidorn thanks.


PERF-483 Correlation ID propagation and configuration

Description
This is a DRAFT for feedback.
Checklist
Please make sure these boxes are checked before submitting your pull request - thanks!

[ ] Write the commit message as per https://github.com/apache/fineract/#pull-requests

[ ] Acknowledge that we will not review PRs that are not passing the build ("green") - it is your responsibility to get a proposed PR to pass the build, not primarily the project's maintainers.

[ ] Create/update unit or integration tests for verifying the changes made.

[ ] Follow coding conventions at https://cwiki.apache.org/confluence/display/FINERACT/Coding+Conventions.

[ ] Add required Swagger annotation and update API documentation at fineract-provider/src/main/resources/static/legacy-docs/apiLive.htm with details of any API changes

[ ] Submission is not a "code dump".  (Large changes can be made "in repository" via a branch.  Ask on the developer mailing list for guidance, if required.)

FYI our guidelines for code reviews are at https://cwiki.apache.org/confluence/display/FINERACT/Code+Review+Guide.

@galovics I am working on the Tests, ASAP I will send  them

Closing in favor of https://github.com/apache/fineract/pull/2427


Host Leaderboard: Adjust crown placement, make top players stand out more

https://github.com/hto-projects/friendpardy/pull/170


Prevent assignment of values to undefined 'tempState' property

@roryabraham @parasharrajat @aimane-chnaif
Details

We faced a problem after updating to react-native-web 0.19.7, where an attempt to assign a value to a property results in a console error because tempState is undefined.
this.tempState[statePropertyName] = val;

Uncaught (in promise) TypeError: Cannot set properties of undefined (setting statePropertyName)

This simple solution will stop this error from happening.
Related Issues

GH_LINK
https://github.com/Expensify/App/issues/16660
Automated Tests

Linked PRs

https://github.com/Expensify/App/pull/24482

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

recheck

I also don't have full context about all the recent changes in Onyx, going to need to spend more time with this PR

@parasharrajat added more details to help you understand the issue.

I also don't have full context about all the recent changes in Onyx, going to need to spend more time with this PR

@roryabraham I don't think this change will make a big difference/have other side-effects because if this.tempState is undefined, we shouldn't be trying to set a value or access a key from it.
I am still investigating why this.state.loading is true as it should always be false when this.tempState is undefined beforehand. should we tag someone who is involved in recent changes?

This change is already made https://github.com/Expensify/react-native-onyx/pull/376


Toggle position

Hello PA @anuragverma108
Issue resolved- #1113

@Ridhima10 please resolve the conflict

@RitiChandak its done

Conflicting files
index.html
Please see the conflicts highlighted in the image above. Resolve the conflicts in the mentioned files. Ensure you pull all the latest changes before pushing your feature/bug fixes to the main branch. After resolving the conflicts, please open a new PR for review.
Thank you!


neovide: add missing cctools.libtool dependency

This targets staging-next, because the fix does not work on master (some header files are not found). staging-next should be merged around Nov 12 according to https://github.com/NixOS/nixpkgs/issues/339153.
Fixes https://github.com/NixOS/nixpkgs/issues/354889.
Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[x] Fits CONTRIBUTING.md.

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

@emilazy It works without xcodebuild, cool!

The time between PR and merge is less than one day! How can I as maintainer do the review?

This is a simple Zero Hydra Failures fix to address a regression caused by changes in the Darwin build environment, which I’m one of the maintainers of. I felt confident that the fix is correct and updates the package to not be broken by the new Darwin build environment. If you have concerns about this PR, I’m sorry and let me know if there is anything I can do – note though that maintainership is not ownership and that it’s normal for packages across the tree to get updated quickly to fix issues caused by changes to the standard build environment.

Ugh, but you’re right that this broke eval on Linux and I should have caught that – I didn’t see as I wasn’t pinged on the subsequent issue. I will send a PR to fix it now.

Sorry guys, I forgot to scope the dependency to Darwin.

How come this wasn't caught by CI? Wasn't CI green for this PR? It shouldn't have been if x86_64-linux failed to build, no?

ofborg is very slow these days, and for Darwin it’s basically unusable, so I’m used to manually testing builds for Darwin‐related PRs rather than waiting for it – but of course that meant that Linux didn’t get checked, which is my fault. (Hopefully whatever form the ofborg redesign takes will help address these issues so that we can rely on ofborg results for all platforms.)
Since this was on the staging-next branch where we buffer up fixes for changes that cause mass rebuilds and monitor the results, there is a natural buffer between any commit being merged and it hitting the master branch, and we tend to move on fixes for regressions more quickly. This PR followed the common pattern of fixes for the new Darwin scheme; I’ve reviewed and merged a bunch of these as we work to fix the regressions before the release, and didn’t give this one any more thought than others. However, in this case the new regression being introduced slipped through the cracks and staging-next was merged into master a few days later.
All of that is on me and not an excuse for my sloppiness here, and I’m sorry that the issue hit the channels. The channel should bump with the fix in the next couple days.

Awesome, thanks for the super clear answer! No problem, thank you for your work on fixing neovide for aarch64-darwin, and your work on nixpkgs in general! :)


Use fedora-distro-aliases

The package fedora-distro-aliases have landed in Fedora. We can migrate relevant parts of our code to use this library.
https://src.fedoraproject.org/rpms/fedora-distro-aliases
https://github.com/rpm-software-management/fedora-distro-aliases

We will plan this once https://github.com/rpm-software-management/fedora-distro-aliases/pull/13 is merged.


making navbar sticky GSSoC'2023

Description
The main part of the page is also moving through the navigation bar, I will make the navbar sticky, so that main part does not insert in navbar .
Screenshots

Additional information
No response

Hi pls assign this issue to me for gssoc 23

@chaursiyasanjeet Do it using inspect, then share screen record in comments , if looks good , then will tell you to create a PR

https://github.com/apoorvaron/Shorty/assets/111435565/19308bde-ef03-446b-86d1-45dfaeb539a5
I have done using inspect, you can see now, how the main content is hiding when scrolling throughout the page.
can I raise the PR now?

@chaursiyasanjeet Sure , Create a PR.


Force use of UTF-8 encoding for BlocklyTranslationGenerator

Change-Id: I0d64b490e12683f40f33acec75d514e56b039ac9

General items:

[ ] I have updated the relevant documentation files under docs/
[ ] My code follows the:

[ ] Google Java style guide (for .java files)
[ ] Google JavaScript style guide (for .js files)

[x] ant tests passes on my machine

If your code changes how something works on the device (i.e., it affects the companion):

[ ] I branched from ucr
[ ] My pull request has ucr as the base

Further, if you've changed the blocks language or another user-facing designer/blocks API (added a SimpleProperty, etc.):

[ ] I have updated the corresponding version number in YaVersion.java
[ ] I have updated the corresponding upgrader in YoungAndroidFormUpgrader.java (components only)
[ ] I have updated the corresponding entries in versioning.js

For all other changes:

[x] I branched from master
[x] My pull request has master as the base

What does this PR accomplish?
This PR makes the invocation of BlocklyTranslationGenerator always use UTF-8. On Linux and macOS this isn't an issue since UTF-8 the default charset. Windows is another story. By forcing the file.encoding property to be UTF-8 it ensures that translations are appropriately managed.
Fixes #3155

@mit-cml/core-reviewers I've added this to the nb199 milestone. It's a build only change that affects people building on Windows. It shouldn't affect core developers since Java defaults to UTF-8 encoding by default on macOS and Linux.


An open invitation to collaborate!

Hey GitHub and OV users!
I wanted to invite those who are interested in OV specifically and AI music generation in general, to colab and help to develop this amazing technology! :)
Any help and any feedback/suggestions/constructive criticism are very welcome and much needed to improve OV. So please do not be shy or intimidated to open an issue or to start a discussion. All are welcome!!!
Another help that is much needed is the OV model training. So if any of you with good and available local GPU(s) can help to train large OV models, it will be greatly appreciated as well.
Otherwise, I hope that you guys are enjoying the project and the music :)
Thank you.
Sincerely,
Alex

Hello
I'm Harry. It's a great honor to see your project. I'm also a music lover. I want to participate in your project. I don't know how to contact you. My email is<EMAIL_ADDRESS>I hope you can contact me when you see it.
Thank you

Hi Harry! @Mr-Harry
If you want to contact me in private, please use my GitHub email<EMAIL_ADDRESS>Otherwise, we can chat here or in the Discussions section of the repo.
I am happy to hear that you love music and computers too. :)
How would you like to contribute or what would you like to discuss? I have numerous small tasks that I could really use help with so if you can do any of it, I would be happy to tell you all about what I need.
Let me know.
Sincerely,
Alex

I am interested in to developing this with you, since I have a couple of good usecases for it so I'll be glad to help
thanks


fix: enum keys cannot have a number as first char in most languages

Description
As the title states, enum keys cannot have a number as the first character. This happens for enums that have the value "1" as it is seen as a string (so Number_1 is not prepended).
If the enum generations encounter such a case it prepends string_${number} to the key.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

15 of 15 (100.0%)  changed or added relevant lines in 3 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.03%) to 93.503%

Totals

Change from base Build<PHONE_NUMBER>:
0.03%

Covered Lines:
1916

Relevant Lines:
1955

💛  - Coveralls


Missing offset method

There doesn't appear to be a way to use the OFFSET SQL statement?

Never mind, I can use the range function


Rename PDF button for publications to URL

Please apologise if this already has a fix or discussion somewhere. I am still very new to hugo-academic...
I would like to suggest to rename the PDF button for publications to URL in cases it is really a URL. That means if no PDF file has been uploaded into the publication folder and the url from the bibtex file is used.
Because in many instances (at least in my case) the URL links to pages where the e.g. project report is hosted, etc.
Of course it would make sense to continue using PDF in cases where a PDF has been uploaded.

See https://sourcethemes.com/academic/docs/language/

Again – not sure why this was closed. This is a feature request. Also, I am confused what the documentation page on "Language and translation" has to do with this issue. Cheers.