Better documentation support

Documentation handling in the schema could be better.

[ ] Actually validate all the symbol names referenced in the documentations. It is currently a TODO in the code: https://github.com/grid-io/oo_bindgen/blob/5149d529e383466b560fdca2f7a489ef9eb04af5/oo-bindgen/src/lib.rs#L466
[ ] Offer to link to a method of a class and validate it. In the C generator, it can easily find the native function associated with that method.
[ ] Offer to link with a struct element and validate it.
[ ] Instead of relying on the DocBuilder, the From<&str> implementation could actually parse the string and resolve the links, warnings, etc.
[ ] Make a difference between the @brief and the detailed description
[ ] Implement proper linking in .NET and Java.

Another thing that seems to come up as I write docs is what to name things...
For example, what name do you chose for an iterator argument?
In C, it's best to call it an iterator in the docs, but this doesn't translate well to Java/C#. It might be reasonable to have some "short codes" that lets the back-end substitute and appropriate name:
{iterator}:
C: iterator
Java: List
C#: ICollection


Allow blank lines?

The ndjson specification suggests that parsers should have an option for allowing (and ignoring) blank lines. It doesn’t explicitly specify the definition of an “empty line” but I’m guessing an all whitespace line is also ignored.

alternatively, grep -vE '^\s?$' file | ndjson-*

thank you for this great tool (and idea!), it would make data processing easier.  But, could you please at least spell out a more user friendly error when newline is encountered?  I had some semi-human readable json that failed to convert, and it took me a long long time to find that i have to filter thru json5 to rid the white space before ndjson-split would work.  Thanks!


Add a configurable BASE_URL to allow Amundsen to be served outside of the root URL

Summary of Changes
Full description of the issue: https://github.com/lyft/amundsen/issues/424
On the Python (Flask) side, I leveraged url_for in index.html, a BASE_URL configuration variable and a middleware class. I also took the routes defined in amundsen_application/api/__init__.py and standardized them by using the same Blueprints mechanism as the rest was using.
On the Javascript side, I added an importable axios instance to inject the BASE_URL into the calls. Note that I have very little experience on front-end, I got some help from @rebeccarich at Earnest but I realized there's many things I don't understand there. For example I try to get the environment variable here, but  I imagine it's not getting picked up because it's client-side ? Any feedback on which way we should do this?
On the scss side, I'm injecting the variable, but in this case it's done at compile time as far as I understand, which is undesirable  (we probably don't want people building Amundsen themselves everytime they need BASE_URL to be set), also not sure about the right way of doing it
Happy to get initial feedback
Tests
Updates the tests to use the new axios
Documentation
TODO: add docs
CheckList
Make sure you have checked all steps below to ensure a timely review.

[ ] PR title addresses the issue accurately and concisely. Example: "Updates the version of Flask to v1.0.2"

In case you are adding a dependency, check if the license complies with the ASF 3rd Party License Policy.

[ ] PR includes a summary of changes, including screenshots of any UI changes.
[ ] PR adds unit tests, updates existing unit tests, OR documents why no test additions or modifications are needed.
[ ] In case of new functionality, my PR adds documentation that describes how to use it.

All the public python functions and the classes in the PR contain docstrings that explain what it does

[ ] PR passes all tests documented in the developer guide

hey @ttannis I'm going to close this PR for now


Documentation?

Hello,
Would you mind adding comments for explaining all the details of you command line?
Are the scripts still valid today?
Olivier

@olcc

Would you mind adding comments for explaining all the details of you command line?

Could you please be more specific on what exactly is not clear? Check out the Quick Start and look inside of the scripts (they are very small). For more details on the QEMU arguments see their documentation.

Are the scripts still valid today?

I don't know I haven't tried them for awhile. Have you? If something doesn't work please create a separate issue and we can discuss it there.

Questions from me would be:

Why do you forward the 9999 udp port?
How to provide an ADC port for the android-x86 VM?
Do the scripts need to be run as root? (I did so, with success, but I'd prefer avoiding running a VM as root).

@olcc good questions! Thank you!

Why do you forward the 9999 udp port?

You can remove the forwarding. I wrote those scripts when I was working on a project that needed Android x86 and I decided to publish them cause I thought they could be useful for somebody else. The forwarding is just leftovers from the project. I will remove them a little bit later.

How to provide an ADC port for the android-x86 VM?

I have no idea, sorry! I guess this is something that has to be done via QEMU itself.

Do the scripts need to be run as root?

I don't remember. I'll check it a little bit later and update the README accordingly.

Your scripts were useful to me, because I could run the android VM without spending to much time on learning how qemu works.
According to https://www.android-x86.org/documentation/debug.html, an ADC port can be provided with a parameter like kvm -net user,hostfwd=tcp::4444-:5555, but I had no success:
$ adb connect localhost:4444
$ adb devices
List of devices attached
localhost:4444  offline

@olcc this is very strange. I have no idea why it doesn't work tho. If you figure out why, feel free to submit a Pull Request to make that knowledge available for everybody.
Thank you for submitting your questions!


Chaining promises inside for

I'm trying to chain a number of promises that need to be resolved before returning.
In my case, for each element of databaseResult I need to fetch some data with a method that returns a promise.
Once I've fetched the data for every single element of the array ( so all the promises have returned), toReturn is complete and I'm happy
static func fetchList() -> Promise<[MatchModel]> {

var toReturn = [MatchModel]()
//get my array of data
let databaseResults = [ArrayWithMyData]

//not sure what I'm doing
var promise = dispatch_promise{ 0 }
if(databaseResults.count > 0) {

    return Promise { fulfill, reject in

        for index in 0..<databaseResults.count {
            promise = promise.then { y -> Promise<Int> in
                //Fetch the data I need ...
                DataProvider.fetch((databaseResults[y].id)!)
                    .then {(model) in {
                        //and use it to create the data I need to return
                        toReturn.append(MatchModel(
                            databaseResults[y], partnerProfile: model))

                        }
                }
                return dispatch_promise { index }
            }
        }
        //Once all the promises are fulfilled, toReturn contains the data I need and I can return it
        promise.then{ x in {
            fulfill(toReturn)
            }
        }
    }
}

If I run this I get
PromiseKit: Pending Promise deallocated! This is usually a bug
I've looked at some exapmles but I'm new to both swift and promisekit so I'm having a hard time understanding how to solve this problem.
Can I please get some help?

Hey there. Have you checked out when? Like you, I am not strong in Swift, so you'll have to forgive me if I haven't discerned all of your requirements. But I've dealt with the scenario you describe multiple times by passing when an array of promises that represent each of the tasks I need completed. If this doesn't meet your needs or if you would like additional examples let me know.

That's exaclty what I was looking for, I dont' know how I missed it.
I manage to get it work by pushing all promises into an array and put that array into a when.
(Seems to) work like a charm.
Thanks!

Here’s the minimal version:
static func fetchList() -> Promise<[MatchModel]> {
    return when(databaseResults.map{ dbresult in
        DataProvider.fetch(dbresult!).then { model in
            return MatchModel(dbresult, partnerProfile: model))
        }
    })
}


Allow for specifying resource constraints per pipeline step

Describe the issue
When submitting a pipeline to kubeflow I get a resource error
To Reproduce
clone the project and run the pipeline
https://github.com/romeokienzler/elyra-classification/blob/main/train-trusted-ai.pipeline
Screenshots or log output
This step is in Failed state with this message: The node was low on resource: memory. Container wait was using 15980Ki, which exceeds its request of 0. Container main was using 2519384Ki, which exceeds its request of 0.

Pipeline runtime environment
KFP 1.12 on IKS

Which version of Elyra did you use? Version 2.0.x does allow for the optional definition of minimum container resource constraints. However, since this error seems to indicate that your Kubernetes system is out of resources, I doubt this will help.

Which version of Elyra did you use? Version 2.0.x does allow for the optional definition of minimum container resource constraints. However, since this error seems to indicate that your Kubernetes system is out of resources, I doubt this will help.


Create a color-analyzer core function

Input: a string of RGB css code
Outputs:

luminance: Contrast ratio against pure black (1 to 21, up to 2 decimal digits)
saturation: Share of pure hue (as oppose to neutral color) in percentage (0 to 100)
hue: Pure hue (0 to 359) that is mixed with neutralColor to produce the input color (0 to 359)
neutralColor: Neutral color that is mixed with hue to produce the input color (0 to 255)

To parse the input string, we use parse-color package.

Weekly downloads: 327,696 (as of 10 Dec, 2020)
No issue pending
used by get-relative-luminance library

To convert a user-selected color into luminance, we use our own converter getContrastRatio.js.

This function rounds the calculated value up to 2 decimal places.
What's available out there uses the Math.floor(), which makes sense to check whether a given contrast ratio satisfies the WGAC criteria of 3:1 or 4.5:1.
For our purposes, however, we need the actual level of luminance. We don't want to show the contrast ratio of, say, 4.449 as 4.44.

Our test code verifies the conversion by aggregating outputs back into the input.
However, the rounding error is unavoidable. An error of plus/minus 1 is allowed.


CLI Flag Specifying Alternative Config Path

Would you consider adding an additional CLI flag that allows users to specify a custom config path for that specific instance of warpd? This would be quite helpful especially when dealing with various oneshot binds. Thanks!

Can you elaborate on your use case? The last commit made warpd XDG
compliant so you can do XDG_CONFIG_HOME=/dir/ provided the config is
located at /dir/warpd/config.
On Wed, May 4, 2022 at 7:10 PM ? @.***> wrote:

Would you consider adding an additional CLI flag that allows users to
specify a custom config path for that specific instance of warpd? This
would be quite helpful especially when dealing with various oneshot binds.
Thanks!
—
Reply to this email directly, view it on GitHub
https://github.com/rvaiya/warpd/issues/109, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAOP7GNOEAZUSTVP457M7M3VIL7WFANCNFSM5VDOIKTA
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

Sure, so the goal is to enable different oneshot instances of warpd to have different bindings (for example a oneshot instance launched with a certain keybind may have its exit bound to "a" while a oneshot instance launced with a alternative keybind may have its exit bound to "b"). This can also be accomplished by specifying XDG_CONFIG_HOME=/path/to/config warpd but the addition of a CLI flag will make it much easier (and not require that the config file is named "config")

I've implemented this in the latest commit

Thanks!


Allow to pass links to notice message

Describe the problem feature solves
It's no way to pass to notice message content like
`Created new pages at <a href=${LINK} target='_blank' rel='nofollow noopener'>${LINK_HEADER}</a>`

Describe the solution you'd like
Change message content to ReactNode
Describe alternatives you've considered
Allow to pass children objects to notice -> messsagebox or implement this feature in some other way
Acceptance criteria

[ ] Allowance to pass links or components like button to notice

Change message content to ReactNode

This might be a bit problematic if we want to support URLs in server generated notifications but it's worth investigating

Hi @LikoIlya.
Did you find a solution for this case? If yes can you share your experience, please?


seek and volume icon bugged

i just install bloom from marketplace. but when i install with automated powershell cmdline this bug / icon are not buggy

oh. it was fixed. i just reinstall all spicetify and was fixed. but not the default theme

the seek and volume are buggy again

can you kindly send us a screenshot of your xpui file
We will tell you how to get it fixed @TukangM

[Setting]
color_scheme            = dark
replace_colors          = 1
overwrite_assets        = 1
spotify_launch_flags    =
check_spicetify_upgrade = 0
spotify_path            = C:\Users\lenovo\AppData\Roaming\Spotify
prefs_path              = C:\Users\lenovo\AppData\Roaming\Spotify\prefs
current_theme           = marketplace
inject_css              = 1

[Preprocesses]
disable_ui_logging    = 1
remove_rtl_rule       = 1
expose_apis           = 1
disable_upgrade_check = 1
disable_sentry        = 1

[AdditionalOptions]
home_config           = 1
experimental_features = 1
extensions            = bloom.js
custom_apps           = marketplace
sidebar_config        = 1

[Patch]
xpui.js_find_8008 = ,(\w+=)32,
xpui.js_repl_8008 = ,${1}56,

; DO NOT CHANGE!
[Backup]
version = <IP_ADDRESS>2.g949d5fd0
with    = 2.16.1

umm. the marketplace updated to 0.8.5. and the seek and volume became buggy again. but i spicetify apply again that was fixable

@TukangM These kind of Issue ussual occurs when you use marketplace version. With each crash or update marketplace automatically reapply theme and sometimes it cause theme to apply twice and then these type of things happens. My personal recommendation for you is to completely remove marketplace version and apply script version instead. It will help you to keep blooming without any confusions
Thanks
Enjoy bloom

@rashil2000 well in this case you have moved from fluent recently

Removing highlighted part (only fluent.js)
Will fix the issue
After removing tun spicetify apply command
You might good to go

@rashil2000 I meant this highlighted part

Thank you, that did it!


It would be convenient to be able to save the conversation as files

The current UI does not offer the opportunity to download the conversations and save them as files.

Resolved in PR #9


[Fabric] - Android - App is flashing when selecting Fub menu and than tapping outside of the field.

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

Action Performed:

Launch the special build
Log in with any account
Tap on Fub menu
Tap on outside of the field

Expected Result:
App is not flashing when selecting Fub menu and than tapping outside of the field.
Actual Result:
App is flashing when selecting Fub menu and than tapping outside of the field.
Workaround:
Unknown
Platforms:

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

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

Version Number: <IP_ADDRESS>
Reproducible in staging?: No
Reproducible in production?: No
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
https://github.com/Expensify/App/assets/93399543/6c735676-3b4f-4a36-b583-6a83b3404fa6
Expensify/Expensify Issue URL:
Issue reported by: Applause - Internal Team
Slack conversation:
View all open jobs on GitHub

I'll be working on this issue.

@WoLewicki will also be working on this issue, but he's OOO so I'm tagging him.

Fixed in https://github.com/WoLewicki/App/commit/22a657bf154a46631795147e4b8d1089fd33f9ef.

On app build from current(https://github.com/WoLewicki/App/commit/70992f24e718ec6b1ee909358dbeb44e9318b415) andrew-fabric branch:
https://github.com/Expensify/App/assets/21055725/8495b10f-2461-456d-9027-38d322215436


this globs the copilot gem again

please talk to cf-routing before reverting
this or see our slack discussion about it

Co-authored-by: Teal Stannard<EMAIL_ADDRESS>Thanks for contributing to cloud_controller_ng. To speed up the process of reviewing your pull request please provide us with:

A short explanation of the proposed change:

reglobbing the ruby gem. See context here: https://cloudfoundry.slack.com/archives/C07C04W4Q/p1543883000033800

An explanation of the use cases your change solves

This allows the routing team to develop without having to make a PR every time we change something

Links to any other associated PRs

[X] I have reviewed the contributing guide

[X] I have viewed, signed, and submitted the Contributor License Agreement

[X] I have made this pull request to the master branch

[ ] I have run all the unit tests using bundle exec rake

[ ] I have run CF Acceptance Tests

cc @charleshansen should be good to go!

Thanks @tstannard and @zachgersh, merged.
Our pipeline is currently held up due to some unrelated CATS failures (see cloudfoundry/cf-acceptance-tests/pull/347), but you should see this on a ci-passed branch of capi-release near you soon.


[SCM] git is missing in last few insider builds

VSCode Version: 1.11.0-insider (2017-04-03T05:31:39)
OS Version: Windows 10

I know many changes at the moment with SCM but I'm struggling with the problem that git is not working any more form vscode for last few days.
This happen on my two Windows 10 machines that are both in insider build.
I looked for issue and docs and it seems that git extenstion should be packed with distribution and should work without any further steps.
Some screens that might help.

@a8775 Did you open VSCode on a folder or without?

I took the time now to explore more of this issue.
I set the setting to git.enabled ': false by mistake.
I would noticed this problem sooner if the SCM icon was not visible just as before introduction of a new SCM service. Would it be good to obsolete this setting as now it should be possible to select one of the many SCM extensions?

The setting is still useful for some people.


Non-static method Lakshmaji\Thumbnail\Thumbnail::getThumbnail() should not be called staticall

I have follow each and every step while installing this package. i have also run
php artisan vendor:publish
it generated the "thumbnail.php" file in app/config.
i have also inlcuded FFMPEG configuration in .env file

FFMPEG_BINARIES=true
FFMPEG=/usr/bin/ffmpeg
FFPROBE=/usr/bin/ffprobe
FFMPEG_TIMEOUT=3600
FFMPEG_THREADS=12

by doing all these , getThumbanil() method isn't working. can you please guide me ?
i am using:
"php": ">=5.6.4",
"laravel/framework": "5.3.*",

Can you paste / give us your source code which includes at least the usage of this library in your app / some POC?


Adding arbitrary agents into the framework?

As far as I can see, CrewAI's main strengths are the concepts of tasks (which can reference each other) and of delegation between agents, while the agent implementation itself looks reasonable but fairly traditional.
How hard would it be to provide a way to plug in any kind of agents (Tree of Thought, AutoGPT, you name it) as individual agents, while taking advantage of the task and delegation infrastructure of CrewAI? You'd really just need to expose an agent interface (abstract class?) that a new agent plugin has to implement, and that's it, right?
Happy to help implement and test if you're interested ;)

Curious to hear more on how you see that working, is that idea that an eng would wrap it's external agent into this class so it can be integrate within a crew? If that is the case it could be straightforward to implement but I'm curious to hear more and also any examples you might have :)

That's exactly the idea, so one would only need to provide a minimal wrapper (and for the major APIs, such as LangChain agents, we could just include one out of the box) that translates between a CrewAI agent API and that particular agent's (eg other crew members who can be delegated to could eg be wrapped as tools to that agent).
Just thinking that it makes no sense to reinvent the wheel, and the individual agent implementations out there are many and fun, so combining them with crewai's task/comms infrastructure would give the best of both worlds.
Will spend tomorrow morning reading crewai code and playing around, to hopefully get more specific.

Something like this perhaps?
from typing import Any, Optional, List
from abc import ABC, abstractmethod

from pydantic import BaseModel, Field, PrivateAttr
from langchain.agents import AgentExecutor, create_openai_tools_agent
from langchain_core.messages import AIMessage, HumanMessage
from langchain_community.chat_models import ChatOpenAI
from langchain_core.prompts import ChatPromptTemplate

from crewai.utilities import I18N
from crewai.agent import Agent

class AgentParent(ABC, BaseModel):
    _i18n: I18N = PrivateAttr(default=I18N())

    @property
    def i18n(self) -> I18N:
        if hasattr(self._agent, "i18n"):
            return self._agent.i18n
        else:
            return self._i18n

    @i18n.setter
    def i18n(self, value: I18N) -> None:
        if hasattr(self._agent, "i18n"):
            self._agent.i18n = value
        else:
            self._i18n = value

    @abstractmethod
    def execute_task(
        self,
        task: str,
        context: Optional[str] = None,
        tools: Optional[List[Any]] = None,
    ) -> str:
        pass

    @property
    @abstractmethod
    def allow_delegation(self) -> bool:
        pass

    @property
    @abstractmethod
    def role(self) -> str:
        pass

    def set_cache_handler(self, cache_handler: Any) -> None:
        pass

    def set_rpm_controller(self, rpm_controller: Any) -> None:
        pass

# Implemented this one just to figure out the interface
# The example at https://github.com/joaomdmoura/crewAI runs fine when substituting
# AgentWrapper(researcher) for researcher, after fixing the types in Task, Crew and AgentTools

class AgentWrapper(AgentParent):
    _agent: Agent = PrivateAttr()

    def __init__(self, agent: Any, **data: Any):
        super().__init__(**data)  # Call the parent's __init__ method
        self._agent = agent  # Set _agent with the value passed to the constructor

    def execute_task(
        self,
        task: str,
        context: Optional[str] = None,
        tools: Optional[List[Any]] = None,
    ) -> str:
        return self._agent.execute_task(task, context, tools)

    @property
    def allow_delegation(self) -> bool:
        return self._agent.allow_delegation

    @property
    def role(self) -> str:
        return self._agent.role

    def set_cache_handler(self, cache_handler: Any) -> None:
        return self._agent.set_cache_handler(cache_handler)

    def set_rpm_controller(self, rpm_controller: Any) -> None:
        return self._agent.set_rpm_controller(rpm_controller)

# NOTE: running into a Pydantic issue when trying to instantiate OpenAIToolsAgent
class OpenAIToolsAgent(AgentParent):
    llm: ChatOpenAI = Field(description="The language model to be used.")
    prompt: ChatPromptTemplate = Field(description="The prompt to be used.")
    tools: List[Any] = Field(description="The tools to be used.")
    _role: str = PrivateAttr(default="")
    _allow_delegation: bool = PrivateAttr(default=False)
    _agent_executor: Any = PrivateAttr(default=None)

    def __init__(self, role: str, allow_delegation: bool = False, **data: Any):
        super().__init__(**data)
        self._role = role
        self._allow_delegation = allow_delegation
        self.init(self.tools)

    def execute_task(
        self,
        task: str,
        context: Optional[str] = None,
        tools: Optional[List[Any]] = None,
    ) -> str:
        # Most agents require their tools list to be known at creation time,
        # so might need to re-create the agent if there are new tools added
        # TODO: compare whether they're actually the same tools!
        if len(tools) != len(self.tools):
            self.init(tools)

        # TODO: better wrap the context as a sequence of messages
        return self._agent_executor.invoke(
            {"input": task, "chat_history": [HumanMessage(content=context)]}
        )["output"]

    def init(self, tools: List[Any]) -> None:
        agent = create_openai_tools_agent(self.llm, tools, self.prompt)

        # Create an agent executor by passing in the agent and tools
        self._agent_executor = AgentExecutor(agent=agent, tools=tools, verbose=True)

    @property
    def allow_delegation(self) -> bool:
        return self._allow_delegation

    @property
    def role(self) -> str:
        return self._role

if __name__ == "__main__":
    # this is how you would use the OpenAIToolsAgent

    from langchain_community.tools import DuckDuckGoSearchRun
    from langchain_openai import ChatOpenAI
    from langchain import hub

    search_tool = DuckDuckGoSearchRun()
    prompt = hub.pull("hwchase17/openai-tools-agent")
    llm = ChatOpenAI(model="gpt-4-0125-preview", temperature=0)

    researcher = OpenAIToolsAgent(
        llm=llm, prompt=prompt, tools=[search_tool], role="Senior Research Analyst"
    )

    # This should then be substituted for crewAI agents

Looks promising!

This would be a super helpful addition, given I had a similar usecase with langchain sql agent, https://github.com/joaomdmoura/crewAI/issues/341#issuecomment-2032616661
Unfortunately this doesn't work well as of now.
I'll try to test this with PR https://github.com/joaomdmoura/crewAI/pull/246

This would be a super helpful addition, given I had a similar usecase with langchain sql agent, #341 (comment)
Unfortunately this doesn't work well as of now.
I'll try to test this with PR #246

@sreecodeslayer did the PR #246 work for you?


Links don't work when $wgServer is set for protocol relative

$wgServer = "//site.org";
$wgCanonicalServer = "https://site.org";

Links don't get sent properly. On desktop it just appears as plaintext, but on mobile it shows as text links without the http(s);.

Thank you for reporting this! I will merge the above patch which hopefully fix the issue.

This appears to have fixed it, thanks!


PermissionError

When I run the texteditor.open("Some text") I get the following error:
PermissionError: [Errno 13] Permission denied: 'C:\Users\...\AppData\Local\Temp\tmpp_f8wv34.txt'
How to solve it?

Thank you for reporting this bug. I have just pushed a fix it and I will publish a new version in a moment.


Npm website sign up

I have deleted the original user name, why the page shows that this user name has been used when I re-register with the previous user name, I want to know how to solve this problem, please help me.

Are you sure no one else took it already?
Whats the username?

This repo is for bugs in the docs site, for general support please reach out to https://www.npmjs.com/support


Word Scramble Game

I want to add a word scramble game based on marvel movie universe

sure assign to u !! start working and complete it in 10 days
follow contributing.md guide for successful PR merge.
happy coding !!

@Dharmesh177 done
Check my PR


Issue when using Bootstrap fade class

Cool solution to a common problem. Thanks for putting it out here.
I ran into an issue utilizing the fade class with tabs that I thought I’d report.
Using your example code I can reproduce the issue.

Add the "fade" class to divs with tab-pane.
On initial load of the page when in panel mode, click “Key Features” or “Source” panel.

“How To” will stay open in addition to the other panel opening.  It works correctly with subsequent collapses on any panel. I've tested with Bootstrap 3.2.0 and the problem still exists.
Removing fade from my tab-panes everything works as expected. I’m not exactly sure of the root cause, something to do with the transition and timing probably, but it may be worth documenting unless a solution is found.
Here is a jsfiddle that demos the issue with the same setup as your example page.
http://jsfiddle.net/r429t/

The unrelated note should have been addressed with #39.


Cannot open file/folder

VSCode Version: 1.33.1
OS Version: Linux x64 5.0.6-arch1-1-ARCH

xdg-mime query default inode/directory -> pcmanfm.desktop
cat usr/share/applications/mimeinfo.cache | grep inode/directory -> inode/directory=pcmanfm.desktop;ranger.desktop;
System:

5.0.6-arch1-1-ARCH
GDB 8.2.1
gcc 8.2.1 20181127
pcmanfm 1.3.1-1
ranger 1.9.2-1
i3-gaps 4.16.1-1

Visual Studio Code:

VSCode Version: 1.33.1
Date: 2019-04-12T08:59:47.459Z
Electron: 4.1.4
Chrome: 69.0.3497.128
Node.js: 10.11.0
V8: 6.9.427.30-electron.0

Steps to Reproduce:

Click on File.
Then click on Open File, Open Folder or Open Workspace.
See the white window.

Does this issue occur when all extensions are disabled?: Yes

We are building exploration builds that use a much newer version of our UI framework (Electron version 4.0.x). I wonder if this issue reproduces with one of these builds, could you try? Download:

macOS: Download
Linux: Download
Windows: Download

We are building exploration builds that use a much newer version of our UI framework (Electron version 4.0.x). I wonder if this issue reproduces with one of these builds, could you try? Download:
* macOS: [Download](https://az764295.vo.msecnd.net/exploration/14236f9f44cc7c94055bb8e65ebda29a4a91ddc7/VSCode-darwin-exploration.zip)

* Linux: [Download](https://az764295.vo.msecnd.net/exploration/14236f9f44cc7c94055bb8e65ebda29a4a91ddc7/code-exploration_1.32.0-1550490730_amd64.deb)

* Windows: [Download](https://az764295.vo.msecnd.net/exploration/14236f9f44cc7c94055bb8e65ebda29a4a91ddc7/VSCodeUserSetup-x64-1.32.0-exploration.exe)

Isn't .deb for Debian based Linux systems as I'm using Arch Linux.

Ok, I downloaded code-git-1.16.0.r25594.ga31dcc38f2-1 from AUR today and the problem still persists.

@MinecraftPlaye here is a ZIP: https://az764295.vo.msecnd.net/exploration/d3efc180606a1eaa4e8d51e5a9099013ce31518f/code-exploration-1555986884.tar.gz

@MinecraftPlaye here is a ZIP: https://az764295.vo.msecnd.net/exploration/d3efc180606a1eaa4e8d51e5a9099013ce31518f/code-exploration-1555986884.tar.gz

Sadly this did not fix it for me. Thought I noticed that this problem also exists in Firefox.

locate filechooser
/usr/include/gtk-2.0/gtk/gtkfilechooser.h
/usr/include/gtk-2.0/gtk/gtkfilechooserbutton.h
/usr/include/gtk-2.0/gtk/gtkfilechooserdialog.h
/usr/include/gtk-2.0/gtk/gtkfilechooserwidget.h
/usr/include/gtk-3.0/gtk/gtkfilechooser.h
/usr/include/gtk-3.0/gtk/gtkfilechooserbutton.h
/usr/include/gtk-3.0/gtk/gtkfilechooserdialog.h
/usr/include/gtk-3.0/gtk/gtkfilechoosernative.h
/usr/include/gtk-3.0/gtk/gtkfilechooserwidget.h
/usr/share/gtk-doc/html/gtk3/filechooser.png

Ok, I figured out that the problem is caused by scaling my desktop (as I'm using an HiDPI display).
This problem also affected only GTK and thus applications like Firefox. Qt application were not affected and worked fine with scaling.


Add iceberg support to EMR serverless jobs.

Description
This commit adds support for query workbench to access Iceberg tables. Iceberg is a commonly used table format for data stored on S3. Specifically, security lake data is stored in Iceberg format. This change enables security lake data to be queried using query workbench.
Issues Resolved
[List any issues this PR will resolve]
Check List

[ ] New functionality includes testing.

[x] All tests pass, including unit test, integration test and doctest

[ ] New functionality has been documented.

[ ] New functionality has javadoc added
[ ] New functionality has user manual doc added

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

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

@asuresh8  can we add more details in description. What is iceberg and why are we adding this?

Add more details to description

Could you please add screeen shots or alteast mention the scenarios that we have tested.


Appointment card

Examples

Audit

In context

Related to
Panel link component
Card component

Closing as it's a duplication of https://github.com/nhsuk/nhs-app-design-styles-components-patterns/issues/26


Lf asm reenable

This set of patches is the sum set of changes involved in re-enabling use of the existing assembly for CDEF and the other loop filters.
The ASM fell unused when joint RDO of loop filters in rdo.rs went to a fully Pixel:u16 pipeline internally, partly because CDEF needed edge information passed in as part of the pixel data (via out of range flag values).  We only have assembly for the LBD case right now, so the u16 piepline rendered the assembly useless and caused a decent-sized speed regression.
Refactoring loop filter RDO to a Pixel:T pipeline (which will be u8 for LBD input) required refactoring CDEF to a more dav1d-like abstraction, which is a useful long-term goal anyway.  The refactor also eliminated a few large temporary buffer and copying; more copying will fall away once we adopt the actual dav1d assembly (which will require one change to dav1d:  separate src/dest arguments to the ASM routines)
The first commit in this patch set is a functional change to the clipping behavior of CDEF as used in LF RDO; the metrics show no effective change across objective-1-fast.  Subsequent patches are bit-exact.

https://github.com/xiph/rav1e/commit/ff4b17b57d18f4e0d81811f6281acfdda485adfa should fix some of the crashes, I opened a PR on your repo already :)

Coverage decreased (-0.4%) to 82.844% when pulling cdc57e5cd04f6f125b15ba67768eb41c1ec7cf51 on xiphmont:lf_asm_reenable into a811e9ef43e36ca9334146ca3a993a964ae45168 on xiph:master.


FR: Only bust related cache items

Hey - just curious what this would take - I thought this was the default but it busts the entire cache - that kind of reduces the point of it :)
👍

Found this work that shows how to find related Craft elements: https://github.com/aelvan/VarnishPurge-Craft/blob/master/varnishpurge/services/VarnishpurgeService.php


Alpine 3.15 is not supported by sass-embedded. Please upgrade to Alpine 3.18

Liquid Exception: end of file reached in /srv/jekyll/_layouts/base.liquid
/usr/gem/gems/sass-embedded-1.63.6-x86_64-linux-musl/lib/sass/embedded/varint.rb:20:in `readbyte': end of file reached (EOFError)
Requested labels: ubuntu-latest
Job defined at: ****/www/.github/workflows/jekyll-docker-gh-pages.yml@refs/heads/main
Waiting for a runner to pick up this job...
Job is waiting for a hosted runner to come online.
Job is about to start running on the hosted runner: GitHub Actions 4 (hosted)
Current runner version: '2.305.0'
##[group]Operating System
Ubuntu
22.04.2
LTS
##[endgroup]
##[group]Runner Image
Image: ubuntu-22.04
Version: 20230619.1.0
Included Software: https://github.com/actions/runner-images/blob/ubuntu22/20230619.1.0/images/linux/Ubuntu2204-Readme.md
Image Release: https://github.com/actions/runner-images/releases/tag/ubuntu22%2F20230619.1.0
##[endgroup]
##[group]Runner Image Provisioner
<IP_ADDRESS>
##[endgroup]
##[group]GITHUB_TOKEN Permissions
Contents: read
Metadata: read
Pages: write
##[endgroup]
Secret source: Actions
Prepare workflow directory
Prepare all required actions
Getting action download info
Download action repository 'actions/checkout@v3' (SHA:c85c95e3d7251135ab7dc9ce3241c5835cc595a9)
Download action repository 'actions/configure-pages@v3' (SHA:f156874f8191504dae5b037505266ed5dda6c382)
Download action repository 'actions/upload-pages-artifact@v1' (SHA:66b63f4a7de003f4f00cc8e9af4b83b8f2abdb96)
Getting action download info
Download action repository 'actions/upload-artifact@v3' (SHA:0b7f8abb1508181956e8e162db84b466c27e18ce)
Complete job name: build
##[group]Run actions/checkout@v3
with:
  repository: ****/www
  token: ***
  ssh-strict: true
  persist-credentials: true
  clean: true
  sparse-checkout-cone-mode: true
  fetch-depth: 1
  lfs: false
  submodules: false
  set-safe-directory: true
##[endgroup]
Syncing repository: ****/www
##[group]Getting Git version info
Working directory is '/home/runner/work/www/www'
[command]/usr/bin/git version
git version 2.41.0
##[endgroup]
Temporarily overriding HOME='/home/runner/work/_temp/54b6891b-6787-432e-a6c5-c22ee3dbc8f1' before making global git config changes
Adding repository directory to the temporary git global config as a safe directory
[command]/usr/bin/git config --global --add safe.directory /home/runner/work/www/www
Deleting the contents of '/home/runner/work/www/www'
##[group]Initializing the repository
[command]/usr/bin/git init /home/runner/work/www/www
hint: Using 'master' as the name for the initial branch. This default branch name
hint: is subject to change. To configure the initial branch name to use in all
hint: of your new repositories, which will suppress this warning, call:
hint:
hint: 	git config --global init.defaultBranch <name>
hint:
hint: Names commonly chosen instead of 'master' are 'main', 'trunk' and
hint: 'development'.

 Installing Bundler 2.4.10 and restarting using that version.
Fetching gem metadata from https://rubygems.org/.
Fetching bundler 2.4.10
Installing bundler 2.4.10
Fetching gem metadata from https://rubygems.org/.........
Resolving dependencies...
Fetching concurrent-ruby 1.2.2
Fetching minitest 5.18.1
Installing minitest 5.18.1
Installing concurrent-ruby 1.2.2
Fetching public_suffix 5.0.1
Using bundler 2.4.10
Using colorator 1.1.0
Fetching duktape <IP_ADDRESS>
Installing public_suffix 5.0.1
Using eventmachine 1.2.7
Using http_parser.rb 0.8.0
Using execjs 2.8.1
Fetching faraday-net_http 3.0.2
Installing faraday-net_http 3.0.2
Using ruby2_keywords 0.0.5
Using ffi 1.15.5
Using forwardable-extended 2.6.0
Using gemoji 3.0.1
Fetching google-protobuf 3.23.3 (x86_64-linux)
Installing duktape <IP_ADDRESS> with native extensions
Installing google-protobuf 3.23.3 (x86_64-linux)
Fetching racc 1.7.1
Installing racc 1.7.1 with native extensions
Using rb-fsevent 0.11.2
Using rexml 3.2.5
Fetching liquid 4.0.4
Installing liquid 4.0.4
Using mercenary 0.4.0
Fetching rouge 4.1.2
Installing rouge 4.1.2
Using safe_yaml 1.0.5
Fetching unicode-display_width 2.4.2
Installing unicode-display_width 2.4.2
Fetching webrick 1.8.1
Installing webrick 1.8.1
Fetching posix-spawn 0.3.15
Installing posix-spawn 0.3.15 with native extensions
Using jekyll-paginate 1.1.0
Fetching i18n 1.14.1
Installing i18n 1.14.1
Fetching tzinfo 2.0.6
Installing tzinfo 2.0.6
Fetching addressable 2.8.4
Installing addressable 2.8.4
Using em-websocket 0.5.3
Fetching katex 0.10.0
Installing katex 0.10.0
Fetching faraday 2.7.7
Installing faraday 2.7.7
Using rb-inotify 0.10.1
Using pathutil 0.16.2
Fetching sass-embedded 1.63.6 (x86_64-linux-musl)
Installing sass-embedded 1.63.6 (x86_64-linux-musl)
Fetching nokogiri 1.15.2 (x86_64-linux)
Installing nokogiri 1.15.2 (x86_64-linux)
Using kramdown 2.4.0
Fetching terminal-table 3.0.2
Installing terminal-table 3.0.2
Fetching activesupport 7.0.5
Installing activesupport 7.0.5
Fetching faraday-retry 2.2.0
Installing faraday-retry 2.2.0
Fetching listen 3.8.0
Installing listen 3.8.0
Fetching jekyll-sass-converter 3.0.0
Installing jekyll-sass-converter 3.0.0
Using kramdown-parser-gfm 1.1.0
Fetching kramdown-math-katex 1.0.1
Installing kramdown-math-katex 1.0.1
Using html-pipeline 2.14.3
Using jekyll-watch 2.2.1
Fetching jekyll 4.3.2
Installing jekyll 4.3.2
Fetching jekyll-avatar 0.7.0
Installing jekyll-avatar 0.7.0
Using jekyll-compose 0.12.0
Fetching jekyll-default-layout 0.1.5
Installing jekyll-default-layout 0.1.5
Using jekyll-feed 0.17.0
Fetching jekyll-include-cache 0.2.1
Installing jekyll-include-cache 0.2.1
Fetching jekyll-last-modified-at 1.3.0
Installing jekyll-last-modified-at 1.3.0
Using jekyll-mentions 1.6.0
Fetching jekyll-optional-front-matter 0.3.2
Installing jekyll-optional-front-matter 0.3.2
Fetching jekyll-readme-index 0.3.0
Installing jekyll-readme-index 0.3.0
Using jekyll-redirect-from 0.16.0
Fetching jekyll-relative-links 0.7.0
Installing jekyll-relative-links 0.7.0
Using jekyll-seo-tag 2.8.0
Using jekyll-sitemap 1.4.0
Fetching jekyll-titles-from-headings 0.5.3
Installing jekyll-titles-from-headings 0.5.3
Using jemoji 0.12.0
Bundle complete! 23 Gemfile dependencies, 62 gems now installed.
Use `bundle info [gemname]` to see where a bundled gem is installed.
ruby 3.1.1p18 (2022-02-18 revision 53f5fc4236) [x86_64-linux-musl]
Configuration file: /srv/jekyll/_config.yml
            Source: /srv/jekyll
       Destination: /srv/jekyll/_site
 Incremental build: disabled. Enable with --incremental
      Generating...
       Jekyll Feed: Generating feed for posts
       Jekyll Feed: Generating feed for posts tagged php
       Jekyll Feed: Generating feed for posts tagged javascript
       Jekyll Feed: Generating feed for posts tagged python
       Jekyll Feed: Generating feed for posts tagged git
       Jekyll Feed: Generating feed for posts tagged azure
       Jekyll Feed: Generating feed for posts tagged cloud
       Jekyll Feed: Generating feed for posts tagged postgresql
       Jekyll Feed: Generating feed for posts tagged django
       Jekyll Feed: Generating feed for posts tagged bash
       Jekyll Feed: Generating feed for posts tagged jekyll
       Jekyll Feed: Generating feed for posts tagged security
  Liquid Exception: end of file reached in /srv/jekyll/_layouts/base.liquid
/usr/gem/gems/sass-embedded-1.63.6-x86_64-linux-musl/lib/sass/embedded/varint.rb:20:in `readbyte': end of file reached (EOFError)
	from /usr/gem/gems/sass-embedded-1.63.6-x86_64-linux-musl/lib/sass/embedded/varint.rb:20:in `block in read'
	from /usr/gem/gems/sass-embedded-1.63.6-x86_64-linux-musl/lib/sass/embedded/varint.rb:19:in `loop'
	from /usr/gem/gems/sass-embedded-1.63.6-x86_64-linux-musl/lib/sass/embedded/varint.rb:19:in `read'
	from /usr/gem/gems/sass-embedded-1.63.6-x86_64-linux-musl/lib/sass/embedded/connection.rb:62:in `block in read'
	from /usr/gem/gems/sass-embedded-1.63.6-x86_64-linux-musl/lib/sass/embedded/connection.rb:61:in `synchronize'
	from /usr/gem/gems/sass-embedded-1.63.6-x86_64-linux-musl/lib/sass/embedded/connection.rb:61:in `read'
	from /usr/gem/gems/sass-embedded-1.63.6-x86_64-linux-musl/lib/sass/embedded/dispatcher.rb:76:in `receive_proto'
	from /usr/gem/gems/sass-embedded-1.63.6-x86_64-linux-musl/lib/sass/embedded/dispatcher.rb:19:in `block (2 levels) in initialize'
	from /usr/gem/gems/sass-embedded-1.63.6-x86_64-linux-musl/lib/sass/embedded/dispatcher.rb:18:in `loop'
	from /usr/gem/gems/sass-embedded-1.63.6-x86_64-linux-musl/lib/sass/embedded/dispatcher.rb:18:in `block in initialize'
##[error]Process completed with exit code 1.
Post job cleanup.
[command]/usr/bin/git version
git version 2.41.0
Temporarily overriding HOME='/home/runner/work/_temp/b434c8c3-98cf-422e-b3d0-145d6efa1300' before making global git config changes
Adding repository directory to the temporary git global config as a safe directory
[command]/usr/bin/git config --global --add safe.directory /home/runner/work/www/www
[command]/usr/bin/git config --local --name-only --get-regexp core\.sshCommand
[command]/usr/bin/git submodule foreach --recursive sh -c "git config --local --name-only --get-regexp 'core\.sshCommand' && git config --local --unset-all 'core.sshCommand' || :"
[command]/usr/bin/git config --local --name-only --get-regexp http\.https\:\/\/github\.com\/\.extraheader
http.https://github.com/.extraheader
[command]/usr/bin/git config --local --unset-all http.https://github.com/.extraheader
[command]/usr/bin/git submodule foreach --recursive sh -c "git config --local --name-only --get-regexp 'http\.https\:\/\/github\.com\/\.extraheader' && git config --local --unset-all 'http.https://github.com/.extraheader' || :"
Cleaning up orphan processes

Hi 👋
While waiting for @envygeeks to update this repo, I've pushed to Docker Hub an updated version for jekyll/builder image:
https://hub.docker.com/r/jvconseil/jekyll-docker
See also PR https://github.com/envygeeks/jekyll-docker/pull/369
Best,
@JV-conseil


Response prefix not taken into account if last communication doesn't expect answers

When receiving a response and last communication doesn't require an answer, it "absorbs" all answers including prefixed responses to previous communications.
As long as the prefix exists (communication still active), the answer should stick to it and not to the last communication.

Not reproduced


Donet 9.0 android lottianimation not playing in bottomsheet

Lottianimation (using SkiaSharp SkLottiView)  in a bottomsheet does not start playing.

Has this happened to anyone else?

@Th3L0x Can you provide a small repro for me? Does this happen also with .net 8?

@Th3L0x thanks, but your sample isn't working. Could you provide me something working?

@Th3L0x found the reason. SKLottieView.Window is null and therefore the animation does not start playing.
I'm not sure if I want to release a new bug fix release since im working on a new major release.
Is it possible that you set the Window until the new version is released?
Either window.AddLogicalChild(lottieView) or set the window on the top view of your content.

Thank you, I'm not sure if this is what you meant for but this way its working:
 //BOTTOMSHEET BUG, will fixed in next release
 private ScrollView? _scrollView;
 private void ScrollView_BindingContextChanged(object sender, EventArgs e)
 {
     try
     {
         if (sender is ScrollView view && Shell.Current.CurrentPage?.Window != null)
         {
             _scrollView = view;
             var lottiPrompt = view.FindByName<SKLottieView>("lottiPrompt");
             var lottiLoad = view.FindByName<SKLottieView>("lottiLoad");
             if (lottiPrompt != null)
             {
                 Shell.Current.CurrentPage?.Window.AddLogicalChild(lottiPrompt);
                 lottiPrompt.BindingContext = this.BindingContext;
             }
             if (lottiLoad != null)
             {
                 Shell.Current.CurrentPage?.Window.AddLogicalChild(lottiLoad);
                 lottiLoad.BindingContext = this.BindingContext;
             }
         }
     }
     catch (Exception ex)
     {
         System.Diagnostics.Debug.WriteLine(ex);
     }
 }

 private void BottomSheet_Closed(object sender, EventArgs e)
 {
     try
     {
         if (_scrollView != null && Shell.Current.CurrentPage?.Window != null)
         {
             var lottiPrompt = _scrollView.FindByName<SKLottieView>("lottiPrompt");
             var lottiLoad = _scrollView.FindByName<SKLottieView>("lottiLoad");
             if (lottiPrompt != null)
             {
                 Shell.Current.CurrentPage?.Window.RemoveLogicalChild(lottiPrompt);
             }
             if (lottiLoad != null)
             {
                 Shell.Current.CurrentPage?.Window.RemoveLogicalChild(lottiLoad);
             }
             _scrollView = null;
         }
     }
     catch (Exception ex)
     {
         System.Diagnostics.Debug.WriteLine(ex);
     }
 }
 //BOTTOMSHEET BUG, will fixed in next release


Fix error related to missing messageid

Was getting this error with bot.py before putting this fix in. Afterwards the bot worked as expected.
Jul 02 20:23:01 MLDevBasic bash[600608]:   File "/opt/dd-discord-bot/bot.py", line 197, in task_loop
Jul 02 20:23:01 MLDevBasic bash[600608]:     await agent_status()
Jul 02 20:23:01 MLDevBasic bash[600608]:   File "/opt/dd-discord-bot/bot.py", line 1379, in agent_status
Jul 02 20:23:01 MLDevBasic bash[600608]:     if messageid != None:
Jul 02 20:23:01 MLDevBasic bash[600608]: UnboundLocalError: local variable 'messageid' referenced before assignment

Thanks!


add logo and update links to new name

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

Project coverage is 82.50%. Comparing base (2dcd633) to head (bfc0f62).

Additional details and impacted files
@@           Coverage Diff           @@
##           master      #57   +/-   ##
=======================================
  Coverage   82.50%   82.50%
=======================================
  Files          10       10
  Lines         983      983
  Branches      182      182
=======================================
  Hits          811      811
  Misses        134      134
  Partials       38       38

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


Theming: Replace all instances of "Grey" spelling with "Gray"

Bug type
Other
Component name
No response
What happened?
"Grey" and "Gray" are used inconsistently throughout the project,
e.g. BackgroundGrey exists alongside GrayDefault (See Customization / Default Theme).
Expected behavior
Reported: #4931, #7514, #8217.
I propose making the breaking change of replacing all spellings of "Grey" with "Gray".
This will impact things like MudColor.Colors.BlueGrey, --mud-palette-grey-light, Palette.BackgroundGrey.
The preference has been discussed in #4939 and https://github.com/w3c/csswg-drafts/issues/3298.
The W3C prefers U.S. English as does C#.
Aliases are also possible to tide over until the next major version but could add more confusion with situations like one being overridden and another referenced internally.
Reproduction link
https://www.mudblazor.com/customization/default-theme
Reproduction steps
Search for "grey" in the codebase
Relevant log output
No response
Version (bug)
6.19.1
Version (working)
No response
What browsers are you seeing the problem on?
Microsoft Edge
On what operating system are you experiencing the issue?
Windows
Pull Request

[x] I would like to do a Pull Request

Code of Conduct

[X] I agree to follow this project's Code of Conduct

@henon Should I do this now?

Yes, let's do it.


Files get copied as root, cannot chown/chmod after copy

Version: 0.15.2
Since the user is micromamba, but the default user executing COPY/ADD in Docker is root, files that get copied over cannot be changed or modified by user micromamba.
Failing example:
ADD script.sh /home/micromamba/script.sh
RUN chmod +x /home/micromamba/script.sh

Error:  changing permissions of '/home/micromamba/script.sh': Operation not permitted
Explanation: script.sh is owned by root, user micromamba cannot change root files.
The problem and a solution is detailed here: https://github.com/moby/moby/issues/6119#issuecomment-338920866
Solution
ADD --chown=micromamba:micromamba script.sh /home/micromamba/script.sh

I am leaving this here for others to see, took me a while to figure out.
It might be good to add env.yaml using chown, so users of this image have a example to go with.

Hi @nstrelow - thank you for your interest in micromamba-docker. I have updated the examples to use COPY --chown=micromamba:micromamba ... in 1ec7754eb1e9c606bc1a3bc263c1eea84ae6451b as COPY is preferred over ADD (Dockerfile best practices).

Good to know the COPY thing, always wondered what was better ^^


Package for python version 3.13

Trying to install linak-controller with the following command:
python -m pip install linak-controller

gives me this error:
ERROR: Ignored the following versions that require a different python version: 1.0.0 Requires-Python >=3.8,<3.13; 1.0.1 Requires-Python >=3.8,<3.13; 1.0.2 Requires-Python >=3.8,<3.13; 1.0.3 Requires-Python >=3.8,<3.13; 1.1.0 Requires-Python >=3.8,<3.13; 1.1.1 Requires-Python >=3.8,<3.13; 1.1.1.dev0 Requires-Python >=3.8,<3.13; 1.2.0.dev0 Requires-Python >=3.8,<3.13
ERROR: Could not find a version that satisfies the requirement linak-controller (from versions: none)
ERROR: No matching distribution found for linak-controller

My OS hast python 3.13.0 installed:
python --version
Python 3.13.0

and it seems like this version is not supported yet.

Thanks for reporting. I will update this but I will not be able to for a few weeks. I would accept a PR in the meantime though.

@rhyst I'm very inexperienced with python unfortunately and don’t have a development setup for it yet :-/

I have just released 1.2.0 which should add support for Python 3.13. Please let me know if this works for you 🙂

Works again! Thx! (system: Bluefin 41 (from Fedora Silverblue))

Works perfectly on my system @rhyst thank you very much :)


Missing bindings for v3.8.0

Hi,
Would you please provide https://github.com/sass/node-sass-binaries/archive/v3.8.0.tar.gz?
Thanks.

Done https://github.com/sass/node-sass-binaries/releases/tag/v3.8.0


xSQLServerSetup: Can only install Analysis Services in the default (Multi-Dimensional Server Mode)

Unable to automate the installation of SSAS in TABULAR mode via Powershell DSC
The DSC configuration that is using the resource (as detailed as possible):
Version of the Operating System, SQL Server and PowerShell the DSC Target Node is running:
Windows 2016 Data Center: Version 1607 (OS Build 14393.729)
Powershell: 5.1.14393.693

Version of the DSC module you're using, or 'dev' if you're using current dev branch:
<IP_ADDRESS>
Notes:
The command-line install has a parameter available for
/ASServerMode MULTIDIMENSIONAL|TABULAR|POWERPIVOT
See: MSDN
This option does not appear to be available from the DSC Resource.

We need to add that support as soon as possible. If anyone feel they can take this on, then feel free. :)

Almost done

@craigofnz Awesome! Thank you! I label this one as in progress then. :smile:


Fix URL quote in the update_path method

Maybe we need normalize_path function which will convert

ALPHA
(%41-%5A and %61-%7A), DIGIT (%30-%39), hyphen (%2D), period (%2E),
underscore (%5F), or tilde (%7E)
to standard form?
As well ./ and ../ should be converted according to "Reference Resolution" chapter of rfc 3986?


Soft fail upload of packages docker images

Description
Packages upload of docker images isn't on the critical path until we build custom domains, if any errors occur we shouldn't block agent deploys. When agents are served by buildkite packages soft_fail should be removed. Other steps are already soft fail but this needs an update on the docker to packages specific variation on the matrix.
When this build failed https://buildkite.com/buildkite/agent-release-stable/builds/149#01930ac6-f742-4aca-a10e-74aeb7456373 - it was on the upload to docker step for packages and actively blocked a critical release for Shopify.

Context
https://linear.app/buildkite/issue/PKG-7830/change-agent-pipeline-to-soft-fail-on-packages-uploads
Changes
Changes docker deploy step to make packages.buildkite.com step to be soft_fail
Pretty sure this is the right way to do it, but I've not done anything with matrix step before:
https://buildkite.com/docs/pipelines/build-matrix#adding-combinations-to-the-build-matrix

Thanks!


Changed struct-related functions to take a StructType

r? @jyn514

Codecov Report

Merging #248 into master will increase coverage by 0.6%.
The diff coverage is 100%.

@@            Coverage Diff            @@
##           master     #248     +/-   ##
=========================================
+ Coverage   77.15%   77.75%   +0.6%
=========================================
  Files          25       26      +1
  Lines        6159     6133     -26
=========================================
+ Hits         4752     4769     +17
+ Misses       1407     1364     -43

Impacted Files
Coverage Δ

src/arch/mod.rs
84% <100%> (-1.81%)
:arrow_down:

src/data/types.rs
85.87% <100%> (+0.84%)
:arrow_up:

src/parse/mod.rs
86.25% <0%> (-2.19%)
:arrow_down:

tests/stack-overflow.rs
89.47% <0%> (ø)

src/parse/expr.rs
78.73% <0%> (+0.02%)
:arrow_up:

src/parse/decl.rs
83.01% <0%> (+0.02%)
:arrow_up:

src/ir/stmt.rs
87.37% <0%> (+0.74%)
:arrow_up:

src/data/mod.rs
38.53% <0%> (+4.46%)
:arrow_up:

src/parse/stmt.rs
80.93% <0%> (+6.65%)
:arrow_up:

Continue to review full report at Codecov.

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


Add samples

So consumers of this library have a sample game to work from

The examples/ConsoleGame.js fills this requirement


GPS Integration with GLIM

Hi,
I was just wondering if there has been any developments on integrating GPS data with SLAM algorithm?
Would really appreciate if someone can guide me as to how I can integrate GPS data as well.
Regards
Shaswat

I agree, it would be incredible if GLIM integrated GPS as an option for input.


Remove zbox as dependency

Matt Davis's zbox package is no longer necessary, as the toolz packages comes with a package called tlz that provides the same functionality.
We should remember to bump the required version of toolz to v0.8.1, as this is the first version that provides this functionality.

Resolved in PR #225.


Bluetooth: Mesh: Enable Segmented Control Messages

Adding segmentation support to bt_mesh_ctl_send.

@AhmedHussein751 Thank you :)
Whenever you fix changes in the commits in your PR you should amend the commits.
The checkpatch commit should be squashed together with the first commit.
Also you should use git commit --signoff (or just -s) to create the signoff line.

Hi Joakim,
Thanks for the info.
Get Outlook for Androidhttps://aka.ms/ghei36

From: Joakim Andersson<EMAIL_ADDRESS>Sent: Thursday, December 19, 2019 12:44:31 PM
To: zephyrproject-rtos/zephyr<EMAIL_ADDRESS>Cc: AhmedHussein751<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [zephyrproject-rtos/zephyr] Bluetooth: Mesh: Enable Segmented Control Messages (#21485)
@AhmedHussein751https://github.com/AhmedHussein751 Thank you :)
Whenever you fix changes in the commits in your PR you should amend the commits.
The checkpatch commit should be squashed together with the first commit.
Also you should use git commit --signoff (or just -s) to create the signoff line.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/zephyrproject-rtos/zephyr/pull/21485?email_source=notifications&email_token=AOCIZNGKOIW2HQWISA3KROTQZNGA7A5CNFSM4J4K24XKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEHJGO7Y#issuecomment-567437183, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AOCIZNH3S2WXM3DWMJFY34TQZNGA7ANCNFSM4J4K24XA.

@AhmedHussein751 You could use interactive rebase.
git rebase -i HEAD~2
Mark your first commit as pick and the second one as squash
You should end up with one commit then. This one you can append the signoff with git commit --amend -s

Sorry about that it seems there are some copy issues. I will fix them soon
Get Outlook for Androidhttps://aka.ms/ghei36

From: WilliamGFish<EMAIL_ADDRESS>Sent: Thursday, December 19, 2019 1:46:51 PM
To: zephyrproject-rtos/zephyr<EMAIL_ADDRESS>Cc: AhmedHussein751<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [zephyrproject-rtos/zephyr] Bluetooth: Mesh: Enable Segmented Control Messages (#21485)
@WilliamGFish requested changes on this pull request.
I haven't checked your logic but there are a number of errors that need resolving just a allow the code to compile.

In subsys/bluetooth/mesh/transport.chttps://github.com/zephyrproject-rtos/zephyr/pull/21485#discussion_r359817291:

                if (bt_mesh_friend_enqueue_tx(tx, BT_MESH_FRIEND_PDU_SINGLE,

                          seq_auth, &buf->b) && BT_MESH_ADDR_IS_UNICAST(tx->ctx->addr)) {

The segment count parameter is missing..Should it be 1?
tx doesn't have a seg_n (tx->seg_n)

In subsys/bluetooth/mesh/transport.chttps://github.com/zephyrproject-rtos/zephyr/pull/21485#discussion_r359817456:

u16_t seq_zero;

  u16_t seq_zero;

variable declared twice

In subsys/bluetooth/mesh/transport.chttps://github.com/zephyrproject-rtos/zephyr/pull/21485#discussion_r359818106:

  }

  net_buf_reserve(buf, BT_MESH_NET_HDR_LEN);

  /* Segmentation Required */

  u8_t seg_o;

  u16_t seq_zero;

  u16_t seq_zero;

  struct seg_tx *tx_seg;

  u8_t segment_count = (data_len - 1) / 8; /* Number of segments-1 */

u8_t segment_count defined but never used

In subsys/bluetooth/mesh/transport.chttps://github.com/zephyrproject-rtos/zephyr/pull/21485#discussion_r359818276:

  }

  net_buf_reserve(buf, BT_MESH_NET_HDR_LEN);

  /* Segmentation Required */

  u8_t seg_o;

defined but never used

In subsys/bluetooth/mesh/transport.chttps://github.com/zephyrproject-rtos/zephyr/pull/21485#discussion_r359818852:

  net_buf_add_mem(buf, data, data_len);

  for (tx_seg = NULL, i = 0; i < ARRAY_SIZE(seg_tx); i++) {

          if (!seg_tx[i].nack_count) {

                  tx_seg = &seg_tx[i];

                  break;

          }               }

extra stray } which closes unopened block. Please remove.

In subsys/bluetooth/mesh/transport.chttps://github.com/zephyrproject-rtos/zephyr/pull/21485#discussion_r359820034:

        int err;

          seg = bt_mesh_adv_create(BT_MESH_ADV_DATA, tx->xmit, BUF_TIMEOUT);

          if (!seg) {

                  BT_ERR("Out of segment buffers");

                  seg_tx_reset(tx_seg);

                  return -ENOBUFS;

          }

          BT_MESH_ADV(seg)->seg.attempts = SEG_RETRANSMIT_ATTEMPTS;

          BT_MESH_ADV(seg)->seg.attempts = SEG_RETRANSMIT_ATTEMPTS;

          net_buf_reserve(seg, BT_MESH_NET_HDR_LEN);

          net_buf_add_u8(seg, TRANS_CTL_HDR(ctl_op, 1));

          net_buf_add_u8(seg, (tx->aszmic << 7) | seq_zero >> 6);

          net_buf_add_u8(seg, (((seq_zero & 0x3f) << 2) | (seg_o >> 3)));

          net_buf_add_u8(seg, ((seg_o & 0x07) << 5) | tx_seg->seg_n);

          len = min(unsend, 8);

Do you mean MIN rather than min?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/zephyrproject-rtos/zephyr/pull/21485?email_source=notifications&email_token=AOCIZNGHTMKCDDJHHYII3I3QZNNKXA5CNFSM4J4K24XKYY3PNVWWK3TUL52HS4DFWFIHK3DMKJSXC5LFON2FEZLWNFSXPKTDN5WW2ZLOORPWSZGOCPYTNMQ#pullrequestreview-334575282, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AOCIZNA5GZSDJBNPSTEVFF3QZNNKXANCNFSM4J4K24XA.

@AhmedHussein751 You can always join the slack channel to get help with git if you need.

@jhedberg and @trond-snekvik can you please review this?

@AhmedHussein751 I went and did some cleanups myself and did a push to your branch. Some things like indentation and line lengths are not that easy to see in the web interface. I also removed the inline declarations since it's better to leave that to the compiler (it will almost certainly inline any longer functions that are only called from a single place)

Thanks, it is my first pull request to zephyr project and I had to fix a lot of issues in this commit. Great to get knowledge about the contribution process.

Thanks, it is my first pull request to zephyr project and I had to fix a lot of issues in this commit.
Great to get knowledge about the contribution process.

No problem - I hope it won't be the last contribution from you ;)
Btw, a couple of questions:

Did you test that things still work for unsegmented messages (i.e. nothing broke there)?
What do you need this for? Some future spec version which introduces long control messages?

Thanks, it is my first pull request to zephyr project and I had to fix a lot of issues in this commit.
Great to get knowledge about the contribution process.

No problem - I hope it won't be the last contribution from you ;)
Btw, a couple of questions:
1. Did you test that things still work for unsegmented messages (i.e. nothing broke there)?

2. What do you need this for? Some future spec version which introduces long control messages?

The first version merged was tested but unfortunately I don't have enough nodes to test the changes made during the pull request.

It was used before for mesh routing that violates the standard by using the restricted control messages opcodes. I decided to merge this part as it complies and maybe try to tweak the routing design in the future to fix the compliance issues.


Adds RollPitchYaw and Rotation bindings and Rust API

🎉 New feature
Goes on top of #43
Related to #11 #14 #17 #28
Checklist

[x] Signed all commits for DCO
[ ] Added tests
[ ] Added example and/or tutorial
[ ] Updated documentation (as needed)
[ ] Updated migration guide (as needed)
[ ] Consider updating Python bindings (if it affects the public API)

I will proceed with this and merge it however we can tackle this discussion F2F.


merge: dev to stage

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


deparam.querystring() returns abnormal results for URLs without a query string

Normally $.deparam.querystring() will return an object containing only the query string values. If called on a URL which does not contain ? it will return an object containing the URL itself, e.g.
$.deparam.querystring("http://google.com/")
Object
    http://google.com/: ""
    __proto__: Object

This means that anyone intending to pass the result back into a function like $.param.querystring has to remember to include a check for this condition if the URL lacks a querystring altogether

Good point, thanks. I'll look into fixing this.


Mouse Highlighter: Allow fade delay/duration lower than 100ms

Description of the new feature / enhancement
Ability to set the Mouse Highlighter fade down to 0ms, to either instantly start fading, or make the highlight dissapear instantly.
currently both are locked to 100ms for a minimum of 200ms until the highlight is gone.
Scenario when this would be used?
When clicking very fast, as you do when double/triple clicking, the 200ms minimum time means that the second click is barely distinguishable from the the first.
Or in cases where someone just prefers the highlighting to be shown for a shorter time.
Supporting information
related issue: #14711

the difference is that this is really targeted at the 100ms minimum for the settings, with the fast/double clicking only being a example usecase.
i would like to have the delays shorter for single clicks too, but thats harder to give as a example.

I'm thinking maybe the mouse highlighter should react to "down" and "up" events.

I'm thinking that's a separate feature request?
i just want the settings limit removed.

https://github.com/microsoft/PowerToys/pull/29352
I'm working on another issue regarding MouseUtil Settings, so I figured I might as fix this one while I'm at it. Made the PR a draft because of my "hacky" solution for handling values of 0 (changing them to 1ms)

The work associated with this issue has been released as part of the 0.76 sprint. Please update PowerToys to the latest. https://github.com/microsoft/PowerToys/releases


Update yarn.lock

Thanks, it looks like something went wrong when merging d669497a5fc7c74e607229ac88d0120c8361d321 - we didn't get the yarn.lock file copied too.
I'll try to keep an eye out for that next time.
/cc @bowenni


Supported Data Sources and Services - Out of Date?

This page currently lists these services:

However, according to https://learn.microsoft.com/en-us/azure/data-factory/connector-overview many other connectors are supported:

Can you please help us reconcile this discrepancy? Does the list of supported connectors on the former page need to be updated?

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

ID: 0b7d5707-9f14-db2b-00fb-5edd7bbcd06f
Version Independent ID: 2a0f8297-bdf7-3056-121e-963140ec0703
Content: Managed virtual network and managed private endpoints - Azure Data Factory
Content Source: articles/data-factory/managed-virtual-network-private-endpoint.md
Service: data-factory
Sub-service: integration-runtime
GitHub Login: @lrtoyou1223
Microsoft Alias: lle

@VitalyMCT
Thanks for your feedback! We will investigate and update as appropriate.

@lrtoyou1223
Could you please review add comments on this, update as appropriate.

@VitalyMCT
Thank you for bringing this to our attention.
I've delegated this to content author @lrtoyou1223, who will review it and offer their insightful opinions.

In this section, we list all supported data service. And you can see in the next part, we mention that you can go to the connector overview page to see all supported data souces.

Thanks for your dedication to our documentation. We are closing this issue for now, but feel free to comment here as necessary. #please-close

I am not sure why this ticket was closed. The documentation issue is still present, as before.
Regarding

In this section, we list all supported data service.

That's incorrect. Please reference the grid provided in my second screenshot.
Example of missing data source is "Cosmos DB for NoSQL". There are many others.
Compare the 1st screenshot and the 2nd one to see all the missing items. A simple fix may be to remove the first list and just refer to the 2nd, as keeping them in sync may be problematic.

@lrtoyou1223


how to set the thread name and priority?

http://droidyue.com/blog/2015/09/05/android-process-and-thread-schedule-nice/
http://developer.android.com/reference/java/lang/Thread.html
http://developer.android.com/reference/android/os/Process.html
https://github.com/d0f/the-saf/blob/ee51143fe1cb9ddad2a877a851333e783f39af63/SAF/src/cn/salesuite/saf/executor/concurrent/BackgroundPriorityThreadFactory.java
http://geyubin.iteye.com/blog/1530009
http://www.yiibai.com/javaexamples/thread_name.html

http://droidyue.com/blog/2015/09/05/android-process-and-thread-schedule-nice/
http://developer.android.com/reference/java/lang/Thread.html
http://developer.android.com/reference/android/os/Process.html
https://github.com/d0f/the-saf/blob/ee51143fe1cb9ddad2a877a851333e783f39af63/SAF/src/cn/salesuite/saf/executor/concurrent/BackgroundPriorityThreadFactory.java
http://geyubin.iteye.com/blog/1530009
http://www.yiibai.com/javaexamples/thread_name.html


Add design and roadmap documentation

Motivation
Closes #1176
Add a document describing the Ruby LSP's design and long term plans. The goal is for the document to clarify the mental models and opinions we use to make design decisions and also bring visibility into what we have planned for the LSP.

Yeah I still prefer it for 2 reasons:

I personally rarely used that feature, and I'd always prefer clicking the ToC in the doc over that button (one click less).
For this particular doc, having a ToC serving as an outline (without the need to click) is especially helpful for grasping the main ideas behind the project. Even if people don't finish the doc, they can get the basic principles at the first glance.

I also don't think it'd create much maintenance cost as this doc should be relatively static, at least structure-wise?

Just for future ref, there's this GitHub Action automatically maintainthe table of contents:
https://github.com/marketplace/actions/toc-generator


Added support for multiple syslog channels.

Summary
Adds support for multiple syslog channels, logging simultaneously.
Impact
None on existing configurations. It defaults to 1 channel, as it was previously.
Testing
Tested on a custom STM32F4 board, logging simultaneously on serial and in an SD file.

Very nice @fjpanag ! If you can, please supply a board example and how to use it.

@acassis Do you mean a new board config file? As a PR?

@acassis Do you mean a new board config file? As a PR?

Yep!

@acassis Do you mean a new board config file? As a PR?

Yep! Also include in the README.txt of the board an instruction to use the example

I would like to provide an example, but please not in another board config.
Every time that I need to use a new (for me) feature, I have to check the source code of a hideous amount of arch'es, boards and configs, find what is useful and/or applicable for my port etc etc... I just feel this is getting bad.
Wouldn't it better to properly document the features in Kconfig help entries, or in the "proper" NuttX documentation?
Maybe I should document it in nuttx/Documentation/components/drivers/special/syslog.rst?

@acassis Do you mean a new board config file? As a PR?

Yep! Also include in the README.txt of the board an instruction to use the example

I would like to provide an example, but please not in another board config.
Every time that I need to use a new (for me) feature, I have to check the source code of a hideous amount of arch'es, boards and configs, find what is useful and/or applicable for my port etc etc... I just feel this is getting bad.
Wouldn't it better to properly document the features in Kconfig help entries, or in the "proper" NuttX documentation?
Maybe I should document it in nuttx/Documentation/components/drivers/special/syslog.rst?

I agree, documentation it is better than proliferating a new board config. In the other hand, have a board config is very useful for testing, people don't need to go through all the steps and miss something.

Wouldn't it better to properly document the features in Kconfig help entries, or in the "proper" NuttX documentation?
Maybe I should document it in nuttx/Documentation/components/drivers/special/syslog.rst?

Yes, please. This should actually be the norm when adding features/changing behavior.
BTW, I'm not following how this interacts with Kconfig. Right now there are some options that indicate where to send syslog via a choice and other options. If this enables multiple outputs, shouldn't that be changed to a multiple choice? Kconfig options for syslog are also a bit messy, so any improvement in that sense would also be welcome.

BTW, I'm not following how this interacts with Kconfig. Right now there are some options that indicate where to send syslog via a choice and other options. If this enables multiple outputs, shouldn't that be changed to a multiple choice? Kconfig options for syslog are also a bit messy, so any improvement in that sense would also be welcome.

I mentioned Kconfig in the broader sense, compared to providing just a another board config.
Generally, for me, Kconfig help entries are the first stop in looking for information for a feature.
Let's finalize the interface of the new functionality first, and then I will update the documentation. And possibly improve syslog Kconfig in general...

BTW, I'm not following how this interacts with Kconfig. Right now there are some options that indicate where to send syslog via a choice and other options. If this enables multiple outputs, shouldn't that be changed to a multiple choice? Kconfig options for syslog are also a bit messy, so any improvement in that sense would also be welcome.

I mentioned Kconfig in the broader sense, compared to providing just a another board config.
Generally, for me, Kconfig help entries are the first stop in looking for information for a feature.

I understand, I was simply mentioning that I didn't quite get how the change introduced in this PR works in conjunction with existing options as I see those are not affected by this PR, as it is not clear from the PR description.

Let's finalize the interface of the new functionality first, and then I will update the documentation. And possibly improve syslog Kconfig in general...

I personally think that the best time to update docs to account for changes in a PR is as part of the PR itself. Changing the docs is just changing a text file. Most of the time the subsequent doc update PR never happens (not saying this is your case, just an observation of past PRs). If you want to make the change to the docs in another PR, go ahead. I just think it would be a good practice to adopt.

I personally think that the best time to update docs to account for changes in a PR is as part of the PR itself. Changing the docs is just changing a text file. Most of the time the subsequent doc update PR never happens (not saying this is your case, just an observation of past PRs). If you want to make the change to the docs in another PR, go ahead. I just think it would be a good practice to adopt.

No, you misunderstood me. I will do it in the same PR. Just at a later time. After deciding on @xiaoxiang781216 's comments above.

I personally think that the best time to update docs to account for changes in a PR is as part of the PR itself. Changing the docs is just changing a text file. Most of the time the subsequent doc update PR never happens (not saying this is your case, just an observation of past PRs). If you want to make the change to the docs in another PR, go ahead. I just think it would be a good practice to adopt.

No, you misunderstood me. I will do it in the same PR. Just at a later time. After deciding on @xiaoxiang781216 's comments above.

Ah, sorry, cool then.

There is something that I do not really understand, regarding removing syslog channels.
In case of syslog_channel_remove, or even previously when syslog_channel was overwritting the existing channel.
How do the resources of the old channel were deallocated?
For example, when  syslog_file_channel is used, and then this channel is removed, how the underlying  file will be closed?
Maybe there is more to be done in this PR regarding this aspect?

There is something that I do not really understand, regarding removing syslog channels.
In case of syslog_channel_remove, or even previously when syslog_channel was overwritting the existing channel.
How do the resources of the old channel were deallocated?

Yes, your concern is right. syslog_channel is designed to switch the default channel to the new one during the initialization, not for switch in the runtime dynamically. So the deallocation isn't a big issue.

For example, when syslog_file_channel is used, and then this channel is removed, how the underlying file will be closed?
Maybe there is more to be done in this PR regarding this aspect?

Yes, we need to extedn syslog_channel_s(e.g. unlink or unregister) to notify the channel.

@xiaoxiang781216 I see this is merged already.
I guess the rest of the work to be done should be in separate PRs?

Why was this merged? The discussion seemed to be ongoing.

Can we revert it until we are sure things work as expected? Also, we discussed adding Documentation once implementation was settled, which was now left out. @fjpanag I'll leave it to you to decide.

@v01d I don't know why this is merged, but I think this part of the work is finished and works correctly.
Since there are a lot to be done, I would personally prefer to have them separated, to better manage them instead of a huge mega-PR.
But I don't actually care. You may revert it if you feel doing so.

Sure, let's keep it then.

Why was this merged? The discussion seemed to be ongoing.

The implementation is done if we ignore the add/remove channel dynamically at the runtime.  What need to improve include:

Document the new API
Support the runtime modification

Both could be done by separated PR.


request help: failed to create stream_route: unexpected status code 400; error message: {"error_msg":"unknown plugin [key-auth]"}

Issue description
Error while creating ApisixRoute with parameter plugins .
Parameters as follows:

error logs:

Does stream_route not support this plugin? What plugins does it support?
Environment

your apisix-ingress-controller version (output of apisix-ingress-controller version --long): 1.7.0
your Kubernetes cluster version (output of kubectl version): 1.22.15
if you run apisix-ingress-controller in Bare-metal environment, also show your OS version (uname -a):

There was no such issue when creating a layer seven route, but it reported the following error when I accessed it:

Does it mean I need to create a consumer manually？


KAFKA-16475 add TopicImageNodeTest

add unit tests for add TopicImageNode

@mannoopj thanks for this patch. There is already a patch (#15720) to fix the same issue. It would be great if you can be a reviewer there.

Thanks, @mannoopj. Since this adds some new test cases beyond what was in #15720,  can you rebase your PR on trunk?


ValidateCAA test function is flaky

Describe the bug:
The TestValidateCAA test function in pkg/issuer/acme/dns/util/wait_test.go performs DNS queries to daniel.homebrew.ca. Perhaps a unit test should not be making external calls like this.
Expected behaviour:

Steps to reproduce the bug:

Anything else we need to know?:
;) @SgtCoDFish
Environment details::

Kubernetes version:
Cloud-provider/provisioner:
cert-manager version:
Install method: e.g. helm/static manifests

/kind bug

/remove-lifecycle stale
I think it would be a big win to have this be done, so want the issue to stay open

/remove-lifecycle rotten
It's important that we reconsider these tests!

/remove-lifecycle stale

/remove-lifecycle rotten

This test is failing for me locally.

/remove-lifecycle rotten


Why did my objectdetectiongpu.apk quit on my phone when I started it?

Why did my objectdetectiongpu.apk quit on my phone when I started it?

Can you check the output of adb logcat ?


Fix tracks/samples not updating running state immediately

Closes https://github.com/ppy/osu-framework/issues/4396
Resolves https://github.com/ppy/osu/discussions/17361

SampleChannelBass is refactored to work in a similar manner to TrackBass, for ensuring that the zero frequency handler doesn't affect the state of playing when manipulating that during a stop call.
It has been mentioned in https://github.com/ppy/osu-framework/pull/4395#issue-865952658 that this is a bit involved, so I'm slightly wondering if I'm missing something with the current solution (cc @smoogipoo).

for what it's worth i don't see any issue with the actual primary fix (f51863b) but i may be missing something.

@smoogipoo still requesting your review on this to be triple-sure there's no issue.
This branch may help in reviewing the changes in SampleChannelBass (only contains reordering to minimise the diff): https://github.com/ppy/osu-framework/compare/master...peppy:play-stop-update-state-better-diff?expand=1

@peppy Thanks, that branch does help a lot. This looks fine to me.


Cannot zoom editor window after opening issue reporter

Issue Type: Bug
Steps

Open issue reporter
Click back on main vscode window
Press cmd + to zoom in or cmd - to zoom out

Bug
No zoom
Seems to start working again after switching back to issue reporter and then back to editor window again
VS Code version: Code - Insiders 1.22.0-insider (5fbcfff69a9af92efd823f18e4c9f4b8c4dbdf2a, 2018-03-27T20:55:46.655Z)
OS version: Darwin x64 17.4.0

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-4770HQ CPU @ 2.20GHz (8 x 2200)

Load (avg)
2, 3, 3

Memory (System)
16.00GB (1.17GB free)

Process Argv
/private/var/folders/_0/611900jn0yb43f1rc47_2wmh0000gp/T/AppTranslocation/03D24729-1C16-49B5-801F-6914290FC331/d/Visual Studio Code - Insiders.app/Contents/MacOS/Electron

Screen Reader
no

VM
20%

I cant repro this in 1.22.2


Improved Readme

I have corrected some spelling errors.

I disagree that these are improvements. The change to the cloudfront action identifier is an error. "Licence" is british spelling. The heading, I don't know, maybe your suggestion is better but it's a sloppy edit to not change the markdown in alignment. Since it's October I'm going to assume that this is just Hacktober spam.


feat: make use of fetchByFilter's customFilter

This is just an example implementation of the new customFilter that doesn't make any real world sense.
The problem is that the function's explicit logic is highly database depending.
If anyone knows a good general use for demo purposes , feel free to leave a comment.

The way that filter is set in fsxa-pwa has been reworked.

A more useful example for filtering contents has been implemented based on a new field tt_publish_at.

Changes are part of PR https://github.com/e-Spirit/fsxa-pwa/pull/76 => closed


Updates to Multiple Scripts

Updated TokenMod
Updated UniversalVTTImporter
Fixed typo in UDLPasswall script.json

@shdwjk Would you mind removing the extra binary file Dealer/.Dealer.js.swp from this PR, please? Once you do, I'll make sure this gets merged! Thanks!


"response to" terms

We should take a good look what we are really using these  terms to annotate.
Many appear to represent conditions. Some are phenotypes. Some are the target of a drug. Some are requested to construct logical definitions  of phenotypes and should not be used in annotation (maybe we could do this differently?). It often isn't clear which BP  the response  is  related to (when it usually  could be). etc. You can throw any chemical at an organism and most times there will be some "response"...what does it mean?
Some examples:
cellular response to topoisomerase inhibitor? (drug target?)
cellular response to gold(3+)????
cellular response to alkane?
cellular response to boron-containing substance levels?
GO:1901593 response to GW 7647 ?????
GO:0072757 cellular response to camptothecin (camptothecin is A topoisomerase inhibitor see above)
GO:1904842 response to nitroglycerin? hmmm....
GO:0070301 cellular response to hydrogen peroxide?? isn't that just the oxidative stress response? It's a condition used to produce oxidative stress (response to oxidative stress is a useful one, if qualified).
It would be better if these  terms were only used   as refinements to true process annotations:
"signalling pathway x" occurs_during "response to y"
or "cellular detoxification" in response to y.
It is difficult to assess what is "unknown" with so many uninformative "response to x " annotations...

Comment from @mah11
Re "response to" terms: quite a few of the strange-looking ones are there to allow other ontologies (including FYPO) to use them in logical definitions or other cross-references. Any such terms should be flagged as not for gene product annotation (I thought they were, but it's certainly possible that some were missed).
https://mailman.stanford.edu/pipermail/go-ontology/2016-April/007432.html

I deleted the  discussion about "response to" from
https://github.com/geneontology/go-annotation/issues/1807
this is something for the future...
@RLovering
@rachhuntley
@pgaudet

@ValWood
Can you move these comments to #14303?
I've added a new label, Request for proposal, to that ticket.
We need a concrete proposal for creating ontology terms as well as annotation usage for 'response to' terms.  This may require a working group, but there are likely several people with a stake in this area of the ontology.
Thx.

Moved comments to #14303.
Closing.


Redis.OM.Analyzer commands file path

Hi @slorello89
I am not an expert in .NET and C# but seems to me this must be a relative path ?:)
https://github.com/redis/redis-om-dotnet/blob/2b26041d1ceccf1326dc95b5f92b36d8b6cbb57d/src/Redis.OM.Analyzer/RedisCommandsAnalyzer.cs#L28
Might something like Environment.CurrentDirectory or AppDomain.CurrentDomain.BaseDirectory be used.

Hi @zhiburt Thanks for opening this issue - yep that definitely shouldn't be in there, fortunately, it's not shipped as part of Redis.OM (it's more like a POC stuff of Redis.OM) - It might need a slightly different way to access the path because it's a code analyzer not certain, will look when I get back.

going to close this out as it really doesn't do anything - probably should just delete the old POC stuff at some point.


Q: Config.yml not being read.What am I doing wrong?

I have the following entry in my config.yml file, but it's not being read. I don't see any errors in the log. My registry is behind nginx with Lets Encrypt certs.
registries:
myregistry:
url: https://docker-registry.net
username: username
password: password
refresh-rate: "5m"
skip-tls: false
I'm running via docker-compose.

Hi,
This may be related to #114 . Are you using the encrypted password in the config.yml? I'm trying to narrow this down.
To attempted to reproduce with a registry using the below commands
htpasswd -cbB ./auth/htpasswd test test

docker run -d   -p 5000:5000   --restart=always   --name registry   -v `pwd`/auth:/auth   -e "REGISTRY_AUTH=htpasswd"   -e "REGISTRY_AUTH_HTPASSWD_REALM=Registry Realm"   -e REGISTRY_AUTH_HTPASSWD_PATH=/auth/htpasswd registry:2

then in the config use test, test as the username and password

Password is in plain text.

Does the connection to the registry work when you did the hostname as you mentioned in #117 ?

Yes, adding my registry in the UI with these values does connect to the registry:
host<EMAIL_ADDRESS>port: 443
HTTPS: true
There is a bug though. You must click the Registries menu item, then click your registry to right to access the registries. Clicking on Repositories returns a list of repos, but returns Page Not Found error when you click on one.

k, interesting, that helps. Thanks for the extra info

Could you paste the registry (from the registry itself, docker logs -f $REGISTRY_NAME) logs where it errors?

Are you requiring authentication in both nginx and the registry? Or is authentication managed only by nginx, and then unauthenticated is allowed to hit the registry itself?

Authentication is only handled by nginx. Here is the https://www.digitalocean.com/community/tutorials/how-to-set-up-a-private-docker-registry-on-ubuntu-14-04.
I'm unclear what you are asking for regarding docker logs -f $REGISTRY_NAME).

Logs for what scenario?

Errors for the invalid authentication, is nginx not even getting the passed user information? Does it make it through nginx and get denied at the registry itself? Those types of things.
I'll have some time tonight to setup a an environment using the guide you referenced. Everything seems to work fine in my testing using authentication in the registry itself. We can also take this to slack if you would be up to testing fixes

To access the registry using this setup you must provide the username and password:
curl https://<YOUR_USERNAME>:@[YOUR-DOMAIN]/v2/
So when I leave this off the username:password from the host in your UI, it's the same as:
curl https://YOUR-DOMAIN/v2/
I get a 401 error. I verified in the nginx logs.

The config.yml is currently the only supported way to set username/password, as posting the html form with user credentials in plain text (as you have seen) without tls seemed like a poor choice.
Basic authentication is sent in the header of each request when using the config.yml. When you supply credentials in that file it will automatically add the basic auth headers to each request. I'm trying to determine whether those headers are being set incorrectly, nginx is not accepting them, or if the registry itself is denying access.

After reading through this I think there was some confusion. I thought you managed to get the config.yml correctly passed through.
The config.yml is not getting passed correctly in via docker-compose, I'll push some fixes out tomorrow to fix this. As a temporary measure, if you update your docker-compose file to be
version: '2'

services:
  docker-registry-manager:
    container_name: docker-registry-manager
    image: snagles/docker-registry-manager
    build: .
    ports:
      - "8080:8080"
    volumes:
      - "./config.yml:/opt/docker-registry-manager/config.yml"

and then run docker stop docker-registry-manager && docker rm docker-registry-manager && docker rmi snagles/docker-registry-manager and then docker-compose up -d --build it should parse the config.

Working now. Thanks. I did have the following issues though:
Got the following per your instructions above:
WARNING: Image for service docker-registry-manager was built because it did not already exist. To rebuild this image you must use docker-compose build or docker-compose up --build.
After running docker-compose up --build I got the following error:
docker-registry-manager    | time="2017-12-13T03:38:52Z" level=fatal msg="Failed to add registry (https://docker.eislan.net), invalid port: strconv.Atoi: parsing "": invalid syntax"
I had to add port 443 to the end of my host in config.yml.
url: https://docker.eislan.net:443
Would be nice if you could assume 443 for https. Otherwise seems to be working.

@matsch01 My registry is also behind nginx with Lets Encrypt certs. But i always get 401. Any idea?

@kelvinji2009 Did you make the changes that snagles provided in his last comment?

@matsch01 Not yet. I use it with many other services. I will try it later. Thanks very much.

@snagles
In default config tree,  the highest priority dir is $REPO_HOME and it maybe override the opt config dir.
	// If the config path is not passed use the default project dir
	if configPath != "" {
		v.AddConfigPath(path.Dir(configPath))
		base := path.Base(configPath)
		ext := path.Ext(configPath)
		v.SetConfigName(base[0 : len(base)-len(ext)])
	} else {
		// use the default tree
		v.SetConfigName("config")
		v.AddConfigPath(conf.GOPATH + "/src/github.com/snagles/docker-registry-manager")
		v.AddConfigPath("../")
		v.AddConfigPath("/opt/docker-registry-manager")
	}

@matsch01 @snagles It works now after i overwrite the /go/src/github.com/snagles/docker-registry-manager/app/config.yml in my container.

This is now fixed. Do a fresh git clone and a docker-compose pull and everything should work as expected.


Update project to be compatible with django-1.9

Most of the changes are updates to the requirement files. The url structure was change to avoid the usage of patterns() which were deprecated.

-1


Feature: swapping words

For example, replacing ' -> " and " -> ' simultaneously. This is proposed by __alpha__!!

would be nice to have the option to make multiple substitutions - for example " -> ' and ' -> " in one go, basically to invert the ' with ". Also would be nice to have the option for "whole word" substitution to to true -> false but not when part of another word.

https://www.reddit.com/r/neovim/comments/17t1np4/comment/k8u8tx4

Done a3e9541f11252eb5a5a54e4c055af43eccc4c556


Added Luna guide button

Closes #17

(Hey, do you want push access, so you don't have to do PRs?)


Add is_complete() function to query if all required and optional fields are set

I trying to communicate with another protocol that declares all fields as optional, but actually requires all of them to be set.
What do you think about this?

I'm not sure. Original protobuf does not have such operation.
Such operation could be implemented with reflection in user code in generic way, somewhat like that:
fn is_complete(m: &Message) -> bool {
    m.descriptor().fields().all(|f| f.has_field(m))
}

Although it will be obviously much slower, because rust has no compile-time metaprogramming yet. Would't that solve your problem?

... moreover, there's no field presence bit in protobuf 3, and for example, no value and zero are indistinguishable. So such method would probably be confusing protobuf 3.

After using this for a while I came to the conclusion that this solution isn't as useful as I initially thought it was. Closing. :disappointed:


updated spelling in quickstart.md

Closing for now til we get Rebekah's permissions set up properly.


Drop all Internet Explorer support into a pit of flames! 🔥 IE 🔥

BREAKING CHANGE: ajax no longer supports IE
Related issue (if exists): #6367

Thanks, @demensky


chore(main): release 0.12.0

:robot: I have created a release beep boop
0.12.0 (2023-06-27)
Features

add openbabel 3d depiction config (09fb49e)
Add sugar removal utility #231 (7d9d8cf)

Bug Fixes

Documentation, CIP, 3D Mol #236,#237,#238 and #239 (b637d0e)
error fix in zero-time deployment script. (283be2d)
grafana typo (bb2d419)
typo (a7f83f0)
typo descriptors.py (fb26d7c)
typo nplikeness (a558776)
update image pull policy (a895455)
update project title in swagger docs. (a3d17fa)

Documentation

add Docker documentation (d4abe1d)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/Steinbeck-Lab/cheminformatics-python-microservice/releases/tag/v0.12.0 :sunflower:


Support for zotero

Hi
thank you for your work.
Could you please make a compatible addon for zotero (xpi) like
https://github.com/ethanwillis/zotero-scihub

I have been considering it.

follow at https://github.com/aokellermann/zotero-nexus


fix: broken python lsp option

python lsp option is written like as treesitter grammar

Sorry for the delayed response.
One would assume I wouldn't miss anything of such, but since there are not comments or notes I'm going to assume I did not do this on purpose.
Thanks for catching it!


Allow different Affinity / NodeSelector / Tolerations for backup jobs

First of all, thank you for the hard work on the operator.
We encountered a situation I believe may not be unique to us. Some of the databases we run are very limited in compute resources. Unfortunately, with the current setup: https://github.com/presslabs/mysql-operator/blob/818452a2b1d4e8bb05521f71997d332bee1709fa/pkg/controller/mysqlbackup/internal/syncer/job.go#L141 backup pods are placed on the same nodes as MySQL itself without options to somehow distant them from mysqld. This results in poor performance on nodes that are resource-compromised, as xtrabackup is relatively CPU-intensive.
There are things we do to overcome this, using mutations on admission controller side, but it would be much easier if there were separate options in place for backup jobs.
Also, I think that would be useful in other use-cases, for example to speed up backups running them with higher thread count on separate nodes (https://github.com/presslabs/mysql-operator/pull/486).
Please let me know what you think about that, thank you!

Thanks for the idea, @rgershkovich! https://github.com/presslabs/mysql-operator/pull/552 should take care of this. Let me know if that's what you had in mind.


17_1 - Create Quiz Taking Component

Create QuizTaking.jsx component to:

Fetch and display quiz questions using existing REST API endpoint (/api/quizzes/{quizId}/questions)
Style the quiz interface following existing Material UI patterns
Implement question display with answer options
Add proper loading and error states

Relates to #65
Labels: frontend
Milestone: Sprint 2

Created QuizTaking functionality within QuestionList

Created QuizTaking functionality within QuestionList