Hero Section

Hi there @anuragverma108!
I noticed in the mobile version of WildGuard that it looses the hero image. I would love to work on this issue. Additionally, I would love to redesign the 4 elements in the section so they fit with nice spacing. I am UX designer, web developer, and front end developer. Please see the referred sections below.
Thank you!

Thanks for creating the issue in WildGuard! The issue has been assigned to you.
Before you start working on your PR, please make sure to:

⭐ Star the repository if you haven't already.
Pull the latest changes to avoid any merge conflicts.
Attach before & after screenshots in your PR for clarity.
Include the issue number in your PR description for better tracking.

Don't forget to follow @anuragverma108 – Project Admin & @PriyaGhosal-Project Collaborator  – for more updates!
We're excited to see your contribution as part of Hacktoberfest 2024! 🎉
Happy open-source contributing!

@PriyaGhosal - Is there anyway you can link this to my PR #282? I forgot to add it within my PR and isn't showing for Hacktoberfest approval.

Hello, is anyone working on this issue?


Can't access ACD (failed to create file system)

Ubuntu 14.04 using rclone-v1.20-linux-amd64.zip
After configuring rclone, all seemed to work (I named the remote: "remote") good.
30min later, impossible to access my ACD. Any command returns a:
"Failed to create file system for "remote:": Failed to get endpoints: HTTP code 503: "503 Service Unavailable", reponse body: Http/1.1 Service Unavailable "
Not sure what's wrong.

Nevermind, it seems ACD is actually having issues with one of their data centers.
So the service is actually down.

@ceptonit Thanks for the update.  Other people have also told me they have been having trouble with ACD being down too.


Midea AC does not receive or delay commands from the Home Assistant

After update to 278f021,my midea ac devices does not receive commands when i do in Home Assistant,or receive commands after 15-20 minutes.If I give commands immediately after I restart the Home Assistant, the commands
are received immediately by the devices. After about 30 minutes, if I give other commands, the devices receive them only after 10-15 minutes or maybe not at all. The same thing happens if I reinstall Midea AC integration.
Versions (版本信息)

Home Assistant version: 2021.8.8
Configuration - Home Assistant.pdf

Midea msmart version: 278f021

I have the same problem. The commands are executed after 20-30 seconds.

You're lucky, they react to me after 10-15 minutes.

lan.py on line 70, add code, like:
            _LOGGER.error("Send {} Error: {}".format(self.get_socket_info(), error))
            return bytearray(0), True

it will be fix in the next version

when send error,  forgot to return 😭

update new version 37d64c8

If there is still a problem，please add to configs and give me home-assistant.log
logger:
  default: info
  logs:
    custom_components.midea_ac: debug
    msmart: debug

I made the update and I will keep you updated. Thank you.

I made the update and I will keep you updated. Thank you.

any errors now?

Hello,only now, everything works without any mistakes. Congratulations and hopefully it will never go wrong again. I tested everything heavy,that is the cause for my late answer.Thank you very much!

I confirm, no problems.


No map refresh after pan

I just recognize an issue after updateing to the newest version.
If I just pan my map, features (vector or also mbtiles sources) are not shown/refreshed if they are outside the actual viewport. If I pan the map and then zoom in or out, they occur. It seems like the map is only renderd if you use the zoom function.
I'm using version 2.0.3 in an UWP app.
Has anyone an idea?

I miss something. If they are outside the actual viewport they can not be seen. Or are they partially outside of the viewport? Or do I not understand the problem?

To reproduce it:
Bind a mbtiles Source.
Then zoom in. -> Tiles will be loaded an renderd.
Then pan. -> "New" tiles from outside the viewport will be loaded, but blurred.

But there are tiles for this zoom level in the MBTiles file? If not, a tile of a lower zoom level is used and zoomed as replacement of the missing tile.

Yes. If I zoom out and then in again, the tiles will be viewed correctly.

In a previous version, all worked well.

ah thanks, I needed that clarification. Seems like a bug. We should test if it exists in current master.

This is what I see in current master in WPF. Looks correct.

And this in UWP. No refresh at all while dragging. This is in current master.

@sgerst Should be fixed in the next release.


[Feature Request] Tabs component center position

Existing Component
Yes
Component Name
Tabs
Description
It could include a tab-position value of "center" in order to place the tabs in the middle of the element
Example: https://imgur.com/PVZmJEr

Sorry, we have no plan to support this.


Include C source files in pip package

Commit message
Include all nanopb C source files as part of the Pip package to make updating nanopb more convenient for pip users.
Before this commit, updating nanopb took two steps: update the nanopb pip package, then update the nanopb C source files. Now updating nanopb takes only one step (i.e., update the pip package), which avoids the possibility of users updating the generator and forgetting to update the source files.
The source files show up in the root of the package:
(.venv) ~/.../extra/poetry$ python -m zipfile --list dist/nanopb-0.4.9.dev1465-py2.py3-none-any.whl
File Name                                             Modified             Size
nanopb/__init__.py                             1980-01-01 00:00:00            0
nanopb/generator/__init__.py                   1980-01-01 00:00:00            0
nanopb/generator/nanopb_generator              1980-01-01 00:00:00          235
nanopb/generator/nanopb_generator.bat          1980-01-01 00:00:00          206
nanopb/generator/nanopb_generator.py           1980-01-01 00:00:00       112055
nanopb/generator/nanopb_generator.py2          1980-01-01 00:00:00          460
nanopb/generator/platformio_generator.py       1980-01-01 00:00:00         5839
nanopb/generator/proto/Makefile                1980-01-01 00:00:00          126
nanopb/generator/proto/__init__.py             1980-01-01 00:00:00         4851
nanopb/generator/proto/_utils.py               1980-01-01 00:00:00         2790
nanopb/generator/proto/google/protobuf/descriptor.proto 1980-01-01 00:00:00        36277
nanopb/generator/proto/nanopb.proto            1980-01-01 00:00:00         7139
nanopb/generator/proto/nanopb_pb2.py           1980-01-01 00:00:00         4443
nanopb/generator/protoc                        1980-01-01 00:00:00         1577
nanopb/generator/protoc-gen-nanopb             1980-01-01 00:00:00          374
nanopb/generator/protoc-gen-nanopb-py2         1980-01-01 00:00:00          554
nanopb/generator/protoc-gen-nanopb.bat         1980-01-01 00:00:00          449
nanopb/generator/protoc.bat                    1980-01-01 00:00:00          302
pb.h                                           1980-01-01 00:00:00        44076
pb_common.c                                    1980-01-01 00:00:00        12141
pb_common.h                                    1980-01-01 00:00:00         1677
pb_decode.c                                    1980-01-01 00:00:00        53759
pb_decode.h                                    1980-01-01 00:00:00         7378
pb_encode.c                                    1980-01-01 00:00:00        30339
pb_encode.h                                    1980-01-01 00:00:00         6958
nanopb-0.4.9.dev1465.dist-info/METADATA        1980-01-01 00:00:00         5848
nanopb-0.4.9.dev1465.dist-info/WHEEL           1980-01-01 00:00:00           92
nanopb-0.4.9.dev1465.dist-info/entry_points.txt 1980-01-01 00:00:00          143
nanopb-0.4.9.dev1465.dist-info/RECORD          2016-01-01 00:00:00         2430

Hmm, I'm unsure whether this makes sense.
Are there precedents of Python packages including C source files that should be compiled by user?
Is there a reasonable way to include those files in a build, considering they'll end up somewhere inside Python's library folders?
Nanopb generator is designed to have good compatibility for generated files, so having mismatched generator and source code versions is not necessarily a problem. But I agree that most users will want to update them at the same time. However, I would think it would make more sense in that case to get the full nanopb package (either as tar or as git submodule), which also includes the generator.

@theunkn0wn1 If you have time, do you have any thoughts on this? Btw. thanks for the initial pypi support back in #460, it has reduced installation problems for many people.

do you have any thoughts on this?

So my thoughts here are somewhat mixed.
either way, the user needs access to the nanopb sources in order to compile the code this package generates.
When i was using this project, it was as simple as adding the nanopb package from platformio.
I am hesitant to commit generated files to VCS, but true source files i have no real objections to.
Bundling the library sources into the pip package should be fine I think.
Emitting them into the generated directory, however, should be behind an opt-in flag IMHO.
I would rather compilers not get confused by the presence of duplicate header files.
Bottom line: I think the idea of shipping the nanopb sources as part of the pip wheel as long as these assets are not themselves generated is fine. they can essentially be treated as resources.
We should make emitting these embedde sources optional, to retain compatibility with alternate sources of these files.

thanks for the initial pypi support back in https://github.com/nanopb/nanopb/pull/460, it has reduced installation problems for many people.

:heart:
NP. I recall the install process before my PR was driving me crazy :rofl:

Ok, so seems like this would be a reasonable approach.
@mreed348 Can you check how importlib.resources would best work for this, and then add some kind of comment or example on how it would be used? I'm not exactly sure where this should go though, as currently there isn't any documentation specific to the Python package.


Use generic font families

I was thinking that maybe we should just use monospace everywhere, for a couple reasons:

On my machine, the font in code blocks in rendered markdown and the font in our directory listings are different.
I read an article (I wish I could find it now) that suggested that you should just use generic font families (sans-serif, serif, etc) because those are usually the most optimized fonts on the system, and it's hard to do better with a custom font stack.

However, monospace is weird, and there's an argument that we should use font-family: monospace, monospace;, because if you just use font-family: monospace;, it's smaller: https://stackoverflow.com/questions/38781089/font-family-monospace-monospace.
We could use font-family: monospace, monospace; everywhere, and pull in normalize.css which also makes all elements like tt and code use monospace, monospace. Which would also be fine with me.
What do you think?

Yeah, we should probably use normalize.css. Note that the generated html from the .index.md files uses <code> tags.


Determine which kind of app wrapping system we'll use

based on

cost
penitential automation
mobile OS legacy version support
we don't have to recode everything

Acceptance criteria

[x] we have reviewed some options
[x] we have a recommendable service
[ ] we have a service signed off to progress with

Plan

[x] enumerate the current app wrapper systems
[x] examine them with the above criteria in mind
[x] pick one
[ ] get it signed off

Adobe Phonegap Build

is free for one app (and, oddly, can't find prices...), which is all we need
can be automated, thought the docs are... less than clear
we have past experience with it, but still requires configuration
supports google/apple suggested minimums

Apache Cordova
https://cordova.apache.org/

it's free
not easily automated
it's just like build
we have more control
it's build, but with all the self hosting hardware pain

Options that would require significant redevelopment work

Flutter https://flutter.dev/docs/get-started/web - "not ready for production"; their words
Framework7 https://framework7.io/docs/custom-build.html - it's a whole new framework
Ionic https://ionicframework.com - it's a whole new framework
React native https://facebook.github.io/react-native - requires rewriting a lot
Xamarin https://dotnet.microsoft.com/apps/xamarin/cross-platform - wants to eat C++/C# rather than html5/css/js

@marxian Looks like adobe phonegap is the best "fit" for now

Not seeing any other options at this stage, will proceed with phonegapp


[WIP] DO NOT MERGE: test upstream PR#2579

https://github.com/kubernetes-sigs/external-dns/pull/2579

Upstream PR was merged


[FLINK-17810][doc] Add document for K8s application mode

Add document for K8s application mode.

How to start a Flink application
How to stop a Flink application

@sjwiesman Thanks a lot for your review, especially for the text. I have integrated your suggestions and make some small change. Adding a line RUN mkdir -p $FLINK_HOME/usrlib in Dockerfile since in the current Flink release image it does not exist.
I also create a ticket for the translation FLINK-17814. I will do it in the following days.
Please have another look.

Fixed in 7723774a0402e10bc914b1fa6128e3c80678dafe


BrButtonModel registers new BlKeyDownEvent event handlers on every focus

BrButtonModel registers new BlKeyDownEvent event handlers on every focus. As a result, on return key the click action might be evaluated several times.


use a distributed redis lock to hide job promotion from user

call promotion logic inside kue, user can configure promotion parameters if desired.

will this and #522 mean that promote() will work out of the box in multi server deployments of Kue? What about watchStuckJobs()?

Exactly @andrewtamura
.promote method will be emptied and deprecated in future, job promotion should work out of the box in Kue without user involvement :)


Issue displaying hintText for FormBuilderSearchableDropdown

Hi Guys,
I tried using the FormBuilderSearchableDropdown widget but am having issue displaying he hintText on the dropdown field
My code

This is outout below

Hi!
I tried reproduce this error with example code and I couldn't.
Maybe do you have a empty option in your items list?
Screencast from 22-01-23 01:08:55.webm

Okay thanks @deandreamatias will try out this solution from your code


Not clear how to add the client secret for AzureSSO in Docker deployment.

Is some content missing, wrong or not clear?
https://docs.open-metadata.org/latest/deployment/security/azure/docker
If we choose Web instead of SPA, what is the env variable to paste the Client Secret?
Describe the solution you'd like
I would assume that it would be found with the rest of the Azure SSO env.
Additional context
I registered the APP in Azure, created my client secret as per the documentation.

As far as I can see, it should be OIDC_CLIENT_SECRET.
For example, in docker/docker-compose-quickstart/docker-compose.yml you have this variable:
      #For OIDC Authentication, when client is confidential
      ...
      OIDC_CLIENT_SECRET: ${OIDC_CLIENT_SECRET:-""}
      ...


Add support for .crypto (UnstoppableDomains) domain lookup

Support New Blockchain name lookup service: https://unstoppabledomains.com/
Lookup is quite straightforward. When presented with a name ending in .crypto:

fetch resolver contract, call 'resolverOf(uint256 nameHash)' on contract 0xD1E5b0FF1287aA9f9A268759062E4Ab08b9Dacbe

resolver = function resolverOf(uint256 nameHash) returns Address
eg nameHash of 'brad.crypto' is 0x756e4e998dbffd803c21d23b06cd855cdc7a4b57706c95964a37e24b47c10fc9
resolverOf(0x756e4e998dbffd803c21d23b06cd855cdc7a4b57706c95964a37e24b47c10fc9)
result:
0xb66DcE2DA6afAAa98F2013446dBCB0f4B0ab2842

lookup name address using the resolver contract found in step 1.

function get(string key, uint256 nameHash) returns string
where key is always crypto.ETH.address for ethereum lookup, eg:
get('crypto.ETH.address', 0x756e4e998dbffd803c21d23b06cd855cdc7a4b57706c95964a37e24b47c10fc9)
result:
0x8aaD44321A86b170879d7A244c1e8d360c99DdA8
BEWARE! look at the function prototypes, they are different from ENS.

@James-Sangalli do you want to take on this?

@hboon its not really clear what do i need to do here, can u provide more details?

It’s a name service like ENS. So we should be able to add support for like we do for ENS

@hboon i need to replace calling GetENSAddressCoordinator(server: serverToResolveEns).getENSAddressFromResolver(for: value) with calling this new service?


Multi-agent ochestration

Manifest

ToolAgent, or functional calling basement

rool rendering, invocation command parser
function tool and function toolkits protocol
with local memory and checkpoint

Multi-agent ochestration

control flow: loop/if-else/switch
human in the loop

distributed agent protocol. (Agent Protocol based implementaion?)

agent state checkpoint
disributed memory, tool server

built-in tools

external retriever: Google Search, Wikipedia, SERP API
calculator
LLM
python interpreter ?

built-in agents

ReACT Tool Execution Agent
Plan & Execute
LLMCompiler

Project plans
Stage 0 - POC - for one week
I'm still getting too many questions about implementation.  Let's do some minimum implementation for inspirations.

[x] ToolAgent
[x] Built-in toolkits: Search (one of Google, Tavily, duc-duck-go or SERP API) , Calculator, Python Interpreter
[ ] Human in the loop: pause, resume

Stage 1 - Assistant API Server for v0.1.2
#19
First version of mini-assistant

Sync api calls for Assistant, File, Run, RunStep, Message, Thread
Function call support, using ReACT

Future developments

Sprint
Features

v0.1.3
* file search tool* paralled call using LLMCompiler ?

v0.1.4
* code interpreter

v0.1.5
* stream support* scaliblity on cloud: PGSQL, Kafka, …

AutoGen: Enabling Next-Gen LLM Applications via Multi-Agent Conversation

https://github.com/langchain-ai/langgraph/tree/main/examples/multi_agent

Tool Agent
langchian tool invocation without function calling API
https://python.langchain.com/docs/use_cases/tool_use/prompting/
Tool description rendering
def render_text_description(tools: List[BaseTool]) -> str:
    """Render the tool name and description in plain text.

    Output will be in the format of:

    .. code-block:: markdown

        search: This tool is used for search
        calculator: This tool is used for math
    """
    return "\n".join([f"{tool.name}: {tool.description}" for tool in tools])

def render_text_description_and_args(tools: List[BaseTool]) -> str:
    """Render the tool name, description, and args in plain text.

    Output will be in the format of:

    .. code-block:: markdown

        search: This tool is used for search, args: {"query": {"type": "string"}}
        calculator: This tool is used for math, \
args: {"expression": {"type": "string"}}
    """
    tool_strings = []
    for tool in tools:
        args_schema = str(tool.args)
        tool_strings.append(f"{tool.name}: {tool.description}, args: {args_schema}")
    return "\n".join(tool_strings)

it should be closed with release of v0.1.3.


make sysvinit default for rhel <= 6

Signed-off-by: Jeremy J. Miller<EMAIL_ADDRESS>Description
Make sysvinit style service default for RHEL/CentOS <= 6
The reality is that there are still some customers who are paying for extended life support of RHEL 5.
This change is a minimum compromise to allow those to use push-jobs cookbook with SysVinit style service.
Issues Resolved
Check List

[ ] All tests pass. See https://github.com/chef-cookbooks/community_cookbook_documentation/blob/master/TESTING.MD
[ ] New functionality includes testing.
[ ] New functionality has been documented in the README if applicable
[ ] All commits have been signed for the Developer Certificate of Origin. See https://github.com/chef-cookbooks/community_cookbook_documentation/blob/master/CONTRIBUTING.MD

This won't hurt anything and if we need to do more here then I think that's reasonable.


displaying user profiles fails when the user is not logged

Description
The user profile page displays correctly when the user is logged in, but it fails when the user is not.
how to reproduce
1 - access the site app.telos.net and log out
2 - access this link to my profie page
3 - See how it fails to load and shoy my profile data
4 - Now loggin as viterbotelos@active using cleos-authenticator
5 - access again this link to my profie page
6 - how it works perfectly

logged
not logged

We won't address this at this point
https://github.com/telosnetwork/epics/issues/9


Uncaught TypeError: Cannot read property 'crypto' of undefined

I created JS file using make command. Getting above error. Please advice. Thanks!

Can you provide some more information about your issue? Maybe some of your code? It's tough to diagnose without context :-)

When I include the js file in index.html I get the errors below. (Clarification - there is just one file to create for streams correct?  The readme references two different js name conventions).

Runtime Error
Uncaught (in promise): TypeError: Cannot read property 'DocumentClient' of undefined TypeError: Cannot read property 'DocumentClient' of undefined at new DynamoDB
Ionic Framework: 3.4.2
Ionic App Scripts: 1.3.7
Angular Core: 4.1.3
Angular Compiler CLI: 4.1.3
Node: 6.11.0
OS Platform: macOS Sierra
Navigator Platform: MacIntel
User Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10_12_5) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/59.0.3071.86 Safari/537.36

I know this is an old issue, but I got this error when I tried to run the amazon connect JS through Babel.  You need to exclude sending it through babel with something like the below:
{
    test: /\.(js)$/,
    exclude: /(node_modules|src\/amazon-connect-v1\.1\.0\.js)/,
    use: [
      'babel-loader'
    ]
}

Please feel free to reopen if the issue persists.


Refactor chromatic workflows

What are you changing?

move github pages storybook deployment to its own workflow (triggered by successful CI workflow run, only on main)
run the chromatic tests in a matrix

Why?

simplifies the workflow files
the chromatic jobs can run in parallel

this changes the workflow run names, so the UI Tests: source-react-components-style required checks for PRs won't ever run on this branch.
they'll need to be updated to the new CI / chromatic (source-foundations) format if this is merged.

also, .github/workflows/storybook.yml wont run in a PR, so we'll need to watch main actions to run to ensure it works

LGTM, have updated checks to reflect the new naming


[Drawer] support nzVisibleChange

What problem does this feature solve?
[(nzVisible)] two-way binding, like Modal
What does the proposed API look like?
@Output() nzVisibleChange = new EventEmitter<boolean>()

I would like to solve this issue, if no one already on it.


Added test case for expected use when iterating using a rowmapper

Hi Nate,
I've added a test case that demonstrates the functionality I was talking about earlier with row mapper and Iterable.  There are also a few tests for Dynamic Composites that you'll probably want to include that test a couple of serialization bugs Ed fixed a while ago.
Todd

what issue is this related to?

I spoke to Nate via Email on Friday.  Basically it address 2 issues.

Coming from a spring JDBC background, I think most uses would expect the result of a query that uses a rowmapper to be iterable.  This way you can iterate over the object that is produced by mapping each row.  It makes it much easier in the caller to gather results.

There wasn't really any example of doing inserts and iteration on results.  This test also adds an example.

Thanks Todd - I'll take a look at this later this afternoon or evening
and get it in as soon as I can.
On Mon, Oct 17, 2011 at 11:14 PM, Todd
<EMAIL_ADDRESS>wrote:

Hi Nate,
 I've added a test case that demonstrates the functionality I was talking about earlier with row mapper and Iterable.  There are also a few tests for Dynamic Composites that you'll probably want to include that test a couple of serialization bugs Ed fixed a while ago.
Todd
You can merge this Pull Request by running:
git pull https://github.com/Spidertracks/hector master
Or you can view, comment on it, or merge it online at:
https://github.com/rantav/hector/pull/307
-- Commit Summary --

Added test demonstrating incorrect compareTo logic
Merge remote-tracking branch 'rantav/master'
Merge remote-tracking branch 'rantav/master'
Updated tests and row mapper to reflect expected functionality
Merge remote-tracking branch 'rantav/master'

-- File Changes --
M core/src/main/java/me/prettyprint/cassandra/service/template/MappedColumnFamilyResult.java (3)
M core/src/main/java/me/prettyprint/cassandra/service/template/MappedColumnFamilyResultWrapper.java (8)
M core/src/test/java/me/prettyprint/cassandra/service/template/ColumnFamilyTemplateTest.java (210)
M core/src/test/java/me/prettyprint/hector/api/beans/DynamicCompositeTest.java (23)
-- Patch Links --
https://github.com/rantav/hector/pull/307.patch
 https://github.com/rantav/hector/pull/307.diff
--
Reply to this email directly or view it on GitHub:
https://github.com/rantav/hector/pull/307

Todd, mind rebasing this PR ?
I merged it but those two tests are failing.

I reverted the change for now.

Hi Patricio,
Which tests were failing?  The tests for Iterable don't have an implementation yet.

ColumnFamilyTemplateTest with:
java.lang.NullPointerException at
me.prettyprint.cassandra.service.template.ColumnFamilyTemplateTest.testQueryMultigetMapper(ColumnFamilyTemplateTest.java:183)
at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
at
sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:39)
at
sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:25)
and
DynamicCompositeTest with:
java.lang.AssertionError:
at org.junit.Assert.fail(Assert.java:91)
at org.junit.Assert.assertTrue(Assert.java:43)
at org.junit.Assert.assertTrue(Assert.java:54)
at
me.prettyprint.hector.api.beans.DynamicCompositeTest.correctComparatorsUsed(DynamicCompositeTest.java:104)
On Thu, Oct 27, 2011 at 1:12 PM, Todd <
<EMAIL_ADDRESS>
Hi Patricio,
Which tests were failing?  The tests for Iterable don't have an
implementation yet.
--
Reply to this email directly or view it on GitHub:
https://github.com/rantav/hector/pull/307#issuecomment-2548661


Module javafx.controls not found

hi!
i have build a project on raspberry aarch64 raspbian with:
mvn package
but when i start run.sh i see:
Error occurred during initialization of boot layer
java.lang.module.FindException: Module javafx.controls not found

So far i understand i need javafx-sdk.
i have downloaded and moved it as recommended:
wget https://download2.gluonhq.com/openjfx/20/openjfx-20_linux-aarch64_bin-sdk.zip
unzip openjfx-20_linux-aarch64_bin-sdk.zip
sudo mv javafx-sdk-20/ /opt/javafx-sdk-20/

so whats next ? how to link it to the project ?

ok. fixed it. just changed in run.sh:
#!/usr/bin/env bash
java \
  -Dglass.platform=gtk \
  -Djava.library.path=/opt/javafx-sdk-20/lib \
  -Dmonocle.platform.traceConfig=false \
  -Dprism.verbose=false \
  -Djavafx.verbose=false \
  --module-path .:/opt/javafx-sdk-20/lib \
  --add-modules javafx.controls \
  --module com.pi4j.example/com.pi4j.example.JavaFxExample $@

but still not working.
if i start it without sudo i see GUI but have no access to gpio.
if i start it with sudo:
sudo ./run.sh i see:
Error occurred during initialization of boot layer
java.lang.module.FindException: Error reading module: /opt/javafx-sdk-20/lib/javafx.swing.jar
Caused by: java.lang.module.InvalidModuleDescriptorException: Unsupported major.minor version 61.0

ok. fixed it. the problem was that it doesn`t work with SDK 20. i downloaded SDK 17.0.6 (https://gluonhq.com/products/javafx/) and it works.


In the case of multiple admins, show which admin made a financial contribution from the collective/project/organization's budget

Who is your user?
Administrators of organizations and collectives with multiple administrators.
What are they trying to achieve?
When an administrator makes a contribution via an organization/collective budget, I want to have transparency into who made that contribution, so I can follow up with the admin who made the contribution if I have questions about the transaction and so we ensure that there's no conflict of interest (an admin should not be moving money into their own personal account).
How are they currently doing this?
We include in our meeting notes that we've approved the transfer of a certain amount of money to a specific project/person. We would also want to include the name of the admin in the description of the Add Funds window but that's easy to forget to do.

It has also been requested if we could enable this feature to only be available to specified admins.


Rename report to app in Docs

Thank you so much for contributing to Datapane, please help us by filing out this PR template and ensure you have read the CONTRIBUTING.md.
It may take a few days to review your PR and merge it - please bear with us!
Thanks!
Prerequsites

[x] Has been tested locally
[ ] If a bugfix, have included a breaking test if possible

Proposed Changes

Switching out occurrences of "report" to "app" in text and code.
Renaming files.
Renaming pages and setting up redirects from the old ones. The mkdocs-redirects plugin only accepts old pages that end in .md, however, the redirects work regardless. E.g. "tutorials/basics/create-a-report.md": "tutorials/basics/create-an-app.ipynb".
Add PostHog snippet to every page.
Rebuild all pages to update metadata. Note: the upcoming series of docs improvements will include a pre-commit hook that clears all outputs.

Thanks @shahinrostami! I also picked up the piece of work around adding PostHog to the docs, but with a slightly different approach (it seems they recommend adding it as a custom analytics provider, which I've done in my branch.)
I also have some styling changes to fix inconsistencies and add our default colour scheme, and plan to do a general clean-up of the docs (i.e. to remove old enterprise stuff that doesn't make sense anymore.)
I'm wondering if I should branch off of this, in that case (my changes right now won't clash with yours, but if I make changes to the actual content it could be a bit of a hairy rebase)

Thanks @shahinrostami! I also picked up the piece of work around adding PostHog to the docs, but with a slightly different approach (it seems they recommend adding it as a custom analytics provider, which I've done in my branch.)
I also have some styling changes to fix inconsistencies and add our default colour scheme, and plan to do a general clean-up of the docs (i.e. to remove old enterprise stuff that doesn't make sense anymore.)
I'm wondering if I should branch off of this, in that case (my changes right now won't clash with yours, but if I make changes to the actual content it could be a bit of a hairy rebase)

Sounds great - Looking forward to the changes!
I think that's a good idea - please branch off this one so we can avoid the mess 😅. Switching out my inclusion of the PostHog snippet [0] sent by James should be easy enough!
I've also removed all the old "Apps" content as mentioned in the to-do.
If the changes look alright to you I can merge this in now so I don't hold you up 👍
[0] https://squidfunk.github.io/mkdocs-material/customization/#additional-javascript

@shahinrostami, sounds good - might be easier if you remove your PH snippet first in that case, and I'll rebase on master.

No problem @lanthias ! I've just pushed the commit that removes it. I've kept extra.js in-place for future JS snippets.


why the code is not consistent with the paper

the code update the critic in SQL is  'sp_term = (q-v) / alpha + 0.5',
but according to  the paper, it should be 'sp_term = (q-v) / (2*alpha )+ 1 '.
And the policy improvement is also not consistent with the paper, see actor.py from  line 11 to line 17
‘’‘
v = value(batch.observations)
q1, q2 = critic(batch.observations, batch.actions)
q = jnp.minimum(q1, q2)
weight = q - v
weight = jnp.maximum(weight, 0)
weight = jnp.clip(weight, 0, 100.)

’‘’

the same question here

the code update the critic in SQL is 'sp_term = (q-v) / alpha + 0.5', but according to the paper, it should be 'sp_term = (q-v) / (2*alpha )+ 1 '. And the policy improvement is also not consistent with the paper, see actor.py from line 11 to line 17
‘’‘ v = value(batch.observations) q1, q2 = critic(batch.observations, batch.actions) q = jnp.minimum(q1, q2)
weight = q - v
weight = jnp.maximum(weight, 0)
weight = jnp.clip(weight, 0, 100.)

’‘’

the same question here

Thanks for pointing out that!
About the code to update the critic in SQL:
Yes, it should be 'sp_term = (q-v) / alpha + 0.5'. We have updated our code to revise the bug.
About the code to update the policy:
SQL and EQL extract the policy by Eq.(14) and Eq.(17), respectively. However, in practice, we found the scale of $\alpha$ does affect the performance as the behavior cloning weight involves $(Q-V) / \alpha$. To make the residual less affected by the scale of $\alpha$, we multiply a constant value to the residual, this can be seen as a kind of reward normalization and doesn't change the optimal policy. In EQL, we multiply 10 to the residual term and extract the policy by $\max_\pi \ E_{(s, a) \sim \mathcal{D}} \left[ \exp \left(10 \cdot \left(Q(s, a) - V(s) \right) / \alpha \right) \log \pi(a|s) \right]$. In SQL, we remove the "1+" term in Eq.(14) (this amounts to multiple a large value to the residual term) and extract the policy by $\max_{\pi} \ E_{(s, a) \sim \mathcal{D}}\left[1 \left(Q(s, a) - V(s) >0 \right) \left(Q(s, a) - V(s) \right) \log \pi(a|s)\right]$. Note that XQL also applies a similar trick, it extracts the policy by using temperature $\beta$ (3.0 for MuJoCo tasks and 10.0 for AntMaze tasks) instead of $\alpha$.
We have updated the results and added the above clarification to our newest paper, it should be available on Arxiv in a few days. Please check out the paper and code for details.

I will close this issue but feel free to re-open it if you have further questions!


Pull code from tutorial

This looks like a great start. We need some docs on using this, and a PR that shows this change in https://github.com/shakacode/react_on_rails_tutorial_with_ssr_and_hmr_fast_refresh

Looking good. We need some docs.

And then see how this integrates into /spec/dummy on React on Rails.
Since that example has a bit more, that will be good for guidance on the docs.


Update issue templates, add .gitignore, update README.md

Added issue and feature request templates
.gitignore would be nice too
Updated README.md with "expected behavior"
Glad to contribute to LibertyOS kernel!
Feel free to criticize, edit or request additional features/corrections to this PR
I will update my issue to match new issue template after your approval

Thank you for the work you have done this far! I am merging this into the main branch. :)


Remove link to Hack Camp from nav bar

Removing the link because we don't need to drive signups anymore.


fix lint issues

open eslint.config.js
delete lines with rules that are turned 'off'. After deleting each line, run npm run lint, fix all warnings, and commit the changes.
It's recommended to make multiple pull requests, in order to break down this task,
this task can have multiple assignees.

[x] '@typescript-eslint/no-unsafe-call': 'off',
[ ] '@typescript-eslint/no-unsafe-member-access': 'off',
[ ] '@typescript-eslint/no-unsafe-return': 'off',
[ ] '@typescript-eslint/no-floating-promises': 'off',
[ ] '@typescript-eslint/restrict-template-expressions': 'off',
[ ] '@typescript-eslint/no-base-to-string': 'off',
[ ] '@typescript-eslint/no-unsafe-assignment': 'off',
[ ] '@typescript-eslint/no-unsafe-argument': 'off',

also, deleting eslint-disable comments would be amazing.

@NoamGaash can you assign this issue to me?

@LightDDark sure!

I've encountered a problem in utils.ts file, the original code:
export const test = baseTest.extend({
  context: async ({ context }, use) => {
    await context.addInitScript(() =>
      window.addEventListener('beforeunload', () =>
        (window as any).collectIstanbulCoverage(JSON.stringify((window as any).__coverage__)),
      ),
    )
    // more code
  },
})

Naturally after deleting  '@typescript-eslint/no-unsafe-call': 'off'  it'll cause a linting error,
However changing any to Window doesn't fix it (after adding the necessary attributes for it to work), nor defining a new interface inside the function.
It seems window is object becomes any regardless of anything, after some research I think it's related to the issue described over here .

@LightDDark that code piece is an anomaly - I  think adding eslint-ignore-line is justified for this one

sure, thanks!

@yoavsbg  @LightDDark נעים להכיר :-)
כיף להצטרף לפרויקט.


Replace references to Ethereum-related chains

Currently we refer to Ethereum mainnet as mainnet and Ethereum testnets with goerli and sepolia. This may be confusing, we may want to switch to ethereum, ethereum-goerli, ethereum-sepolia, respectively.

Closing as complete. The transition to using @api3/chains in https://github.com/api3dao/airnode/pull/1739 changed mainnet to ethereum, goerli to ethereum-goerli-testnet and sepolia to ethereum-sepolia-testnet.


April CPU - Change Spec Files etc to ship JDK22 s390x Patched Packages

Due to the nature of our RPM spec files, I recommend this PR not be merged, Im only opening it to allow the packages to be published..
The debian changes are all ok, and as standard, but due to the implementation methods in the rpm spec files and build script, I recommend this PR not be merged, and will simply be closed to serve as a reminder detailing how this had to be achieved, where only a single architecture receives a patch.

Closed so as to prevent accidental merging.


Eye Icon Design Mismatch in Password Fields

Describe the bug
The eye icon used for toggling password visibility looks and behaves differently
To Reproduce
Steps to reproduce the behavior:

Navigate to login screen (can be seen on multiple pages)
Look for the password input field
Notice the eye icon on the right side of the password field

Expected behavior
A clear and concise description of what you expected to happen.
Screenshots

Desktop:

OS: [e.g. iOS]
Browser [e.g. chrome, safari]
Version [e.g. 22]

@rithviknishad @Jacobjeevan Can i Work on this issue

@rithviknishad @Jacobjeevan Can I work on this issue ?

This page was also reworked in form-field-v1. We aren't displaying the eye icon right now (maybe we will add it in the future), but closing this.


Static mods, is C+57.02146 a default?

When running CONGA.py without arguments, I see this for static mods:
    --static_mods <string>      Of the form X:[+-]A where X is the amino acid,
                                or rather "cterm" or "nterm" if it is a
                                modification on the C-terminal or N-terminal.
                                A is the absolute mass shift in Daltons.
                                [+-] indicates whether the mass shift is
                                positive or negative. C+57.02146 is always
                                included by default. Variable modifications
                                do not need specification (they are accounted
                                for via the search files). List mods in comma
                                separated format, e.g.
                                nterm:10,cterm:-20,L:50.
                                Default = None.

But when reading through CONGA.py it looks like C+57.02146 might be a default, but I could be wrong about that. I just wanted to confirm as it does affect the Limelight XML conversion to know what static mods are being used.

Nevermind I see now that  C+57.02146 is always included by default.


Add flake8 testing only to Python 3.6

Let's point this at just src/ and setup.py: running it on . will get all of the .tox/ virtualenvs too, which will take forever and not add anythng useful. Travis was complaining because the files in misc/ weren't py3-safe, but I'm inclined to skip everything in misc/ for now and maybe add them later after the real code is flake8-clean.
Also, how about we just add it to tox, instead of travis? It will be easier to fix the warnings that it discovers if we can run the same test locally. How about we use tox -e flake8 for now?

@warner Can you please and accept this PR as is before I open a separate PR to attack the tox piece?  I find tox tricky to configure correctly but if you are a tox master then I would be happy to learn from your implementation of this piece.  In the end, flake8 should be run under both Py2 and Py3 as it will return different results under the two different versions.

The changes look good.. could you do two things first?:

squash this down into two patches: one for misc/demo-journal.py, the other for .travis.yml
rebase to current master

The appveyor failure should be fixed on master (it was a windows-specific problem that I introduced several weeks ago, and fixed yesterday).
Then force-push to this same "patch-2" branch that you're using, and travis+appveyor will pick it up. Once CI is green, I'll land it. And then after it's landed, we can figure out how to extend it to run under both py2 and py3, hopefully via tox somehow. Thanks!

Closed in favor of #229


Delete throws an error when file not found

If we don't have the file or a specific value, we should ignore the error and return a success, since the file does not exists and that's the point of deleting it 😁
I/flutter (25796): 11:20:23.674 FileSystemException: Cannot delete file, path = '/data/user/0/package_name/app_flutter/cmatc3RlXaNvb4ZpZ19vdmVycmlkZQ==.enc' (OS Error: No such file or directory, errno = 2)

From what I can see in the sources, this has been fixed already. Closing


bump ansible to 1.8.4

Between this change and [#10] I've updated all the troublesome pieces that made ansible 1.8.4 unable to run the playbook.

PLAY RECAP ********************************************************************
<IP_ADDRESS>                  : ok=88   changed=13   unreachable=0    failed=0


[Admin] When a product is edited through the admin, the relation product_image disappears

Steps to reproduce:

Edit a product in the admin
The relation product_image disappears

EDIT: In Firefox browser, Chrome is ok

Duplicated #223


Unable to run web server

Hello,
I am able to run my webpage unfortunately with a blank webpage.
For Step 3,
I have created a user, home directory and a group.
Copied init.d into my /etc directory.
Copied "scripts" from your repo to /home/motion directory.

changed the IP address in index.html
did a npm install in script however i got a warning of
npm WARN package.json<EMAIL_ADDRESS>No repository field.

Did i do something wrongly here?
Can you guide me on what are the necessary node.js modules for web server?

I'll have more details in the evening today, but generally, the requirement is only Express framework, npm should have installed it for you (it's listed in package.json).
Are you able to connect to Motion stream directly (port is specified in motion.conf)?

In addition to the above - it's not stated explicitly in the readme, but you have to start the server after installing necessary modules via npm. You do this by 'node server.js'. Did you do this? If the server is started, you to at least get the web page with static text even if Motion is not started.

Do you mean i have to modify motion.conf, motion to accommodate my port?
After running node server.js, my console only gave the following output:
server listening on port 10080
Also, after starting my webpage of http://:10080 , I am able to run my webpage and able to only see a text of "Webcam Stream" with a blank image below it.
Seems like my image is unable to flash, any idea why is this happening?
Thank you in advance.

This sounds like an incorrect motion.conf to me. Try the one from this repo,without any modifications first.
I think the main error there is motion's inability to fetch the image from camera, check your driver and that the camera supports MJPEG format.

I stick to the default motion.conf file from your repo at first and it gave me this error
[0] Processing thread 0 - config file /etc/motion.conf
[0] Unknown config option "ffmpeg_cap_new"
[0] Unknown config option "ffmpeg_cap_motion"
[0] Unknown config option "ffmpeg_timelapse"
[0] Unknown config option "ffmpeg_timelapse_mode"
[0] Unknown config option "ffmpeg_bps"
[0] Unknown config option "ffmpeg_variable_bitrate"
[0] Unknown config option "ffmpeg_video_codec"
[0] Unknown config option "ffmpeg_deinterlace"
[0] Unknown config option "movie_filename"
[0] Unknown config option "timelapse_filename"
[0] Motion 8.4.0 Started
[0] Thread 1 is from /etc/motion.conf
[0] motion-httpd/8.4.0 running, accepting connections
[1] Thread 1 started
[0] motion-httpd: waiting for data on port TCP 8080
[1] cap.driver: "uvcvideo"
[1] cap.card: "UVC Camera (046d:0825)"
[1] cap.bus_info: "usb-dwc3-host.2-1"
[1] cap.capabilities=0x84000001
[1] - VIDEO_CAPTURE
[1] - STREAMING
[1] Test palette MJPG (640x480)
[1] Using palette MJPG (640x480) bytesperlines 0 sizeimage 341333 colorspace 00000008
[1] found control 0x00980900, "Brightness", range 0,255
[1]     "Brightness", default 128, current 128
[1] found control 0x00980901, "Contrast", range 0,255
[1]     "Contrast", default 32, current 32
[1] found control 0x00980902, "Saturation", range 0,255
[1]     "Saturation", default 32, current 32
[1] found control 0x00980913, "Gain", range 0,255
[1]     "Gain", default 64, current 64
[1] mmap information:
[1] frames=4
[1] 0 length=341333
[1] 1 length=341333
[1] 2 length=341333
[1] 3 length=341333
[1] Using V4L2
[1] Resizing pre_capture buffer to 1 items
Corrupt JPEG data: 1 extraneous bytes before marker 0xd1
Corrupt JPEG data: 5 extraneous bytes before marker 0xd7
Corrupt JPEG data: 1 extraneous bytes before marker 0xd2
Corrupt JPEG data: 1 extraneous bytes before marker 0xd5
Corrupt JPEG data: 3 extraneous bytes before marker 0xd0
[1] Error capturing first image
[1] Started stream webcam server in port 8081
[1] Resizing pre_capture buffer to 3 items
Corrupt JPEG data: 1 extraneous bytes before marker 0xd1
Corrupt JPEG data: 1 extraneous bytes before marker 0xd0
Corrupt JPEG data: 1 extraneous bytes before marker 0xd5
Corrupt JPEG data: 1 extraneous bytes before marker 0xd2

It is a continuous script that runs unless i Ctrl+C which outputs this error eventually
module.js:340
    throw err;
          ^
Error: Cannot find module 'temboo/core/temboosession'
    at Function.Module._resolveFilename (module.js:338:15)
    at Function.Module._load (module.js:280:25)
    at Module.require (module.js:364:17)
    at require (module.js:380:17)
    at Object.<anonymous> (/home/motion/scripts/xively_post.js:37:16)
    at Module._compile (module.js:456:26)
    at Object.Module._extensions..js (module.js:474:10)
    at Module.load (module.js:356:32)
    at Function.Module._load (module.js:312:12)
    at Function.Module.runMain (module.js:497:10)
[0] httpd - Finishing
[0] httpd Closing
[0] httpd thread exit
[0] Motion terminating

After doing some research i found out that there is extraneous bytes in the image therefore i tried reducing the resolution which gave me the error stated in the previous post.
I'm not sure what went wrong..

Okay, that sounds like a camera problem then-maybe its mjpeg support is broken. Or broken at that exact resolution. I'd suggest reading up about your cam, its uvcvideo driver compatibility and mjpeg support quality.

I edited the following parameters and managed to run my motion without errors:
daemon on
stream_localhost off
webcontrol_localhost off
Am i getting the correct output after entering the command motion ?
[0] Processing thread 0 - config file /etc/motion.conf
[0] Unknown config option "ffmpeg_cap_new"
[0] Unknown config option "ffmpeg_cap_motion"
[0] Unknown config option "ffmpeg_timelapse"
[0] Unknown config option "ffmpeg_timelapse_mode"
[0] Unknown config option "ffmpeg_bps"
[0] Unknown config option "ffmpeg_variable_bitrate"
[0] Unknown config option "ffmpeg_video_codec"
[0] Unknown config option "ffmpeg_deinterlace"
[0] Unknown config option "movie_filename"
[0] Unknown config option "timelapse_filename"
[0] Motion 8.4.0 Started
[0] Motion going to daemon mode

However i can't seem to run my server now.
After creating the groups, copying the scripts and editing index.html,
I run a node server.jsin the /home/motion/scripts directory and went into my web server however nothing is being displayed.
Any idea what did i do wrong this time?
Is there any specific parameters i need to change?

Well, these options have in general nothing to do with errors you had in your logs before. both _localhost ones simply allow Motion to bind to addresses other than <IP_ADDRESS> (i.e. Motion's stream will be available for clients connecting from other hosts) and the daemon one is for having it run in the background - just like it says in the log file.
I think the errors are actually there, just not visible due to the daemon mode.
So let's split two things

server: to check this one, just run node server.js after running npm install in the respective dir - per readme instructions. After that you should see at least an empty page with "Webcam stream" text in it. If you don't - check that the web server is running by e.g. running netstat on the board and checking that node process is bound to the respective IP and port. If you need my help - console outputs or maybe screenshots would help.
Motion not being able to capture the stream from your webcam: the recommendation on this stays the same - just check that your webcam is actually supported by Edison's version of the uvcvideo driver (version is tied to the OS kernel version, which is 3.10 for Edison SW Release 2.1)

Is this still actual?

Closing for no response and a solution provided


[Unity plugin] Reuse vertex indices when importing STL meshes

This PR makes modifications to the STL IO of the Unity plugin. Changes:

Writes only the new bytes of the buffer to the file, which avoids having trailing nulls in the imported and scaled STL. Blender and other STL viewing software treated the files with trailing nulls as corrupted.
Using a dictionary, indices of shared vertices are reused during the import. This allows loading larger meshes with the same index resolution, and the resulting files are smaller and render faster (~20%-40% smaller asset files in the MPL hand for example).
Switch to 32bit indexing if 16 bits are not sufficient even with index reusing.

Tested with the MuJoCo mug, and the MPL hand.
It was unclear to me why was it necessary to perform axis swizzling in the STL writer part as well, and not just the parser. As I was working on this PR, meshes imported 90 degrees rotated sometimes. Removing the swizzling from there resolved that. In case I missed a case where that was key for the proper import, let me know @erez-tom.
@vidurvij-apptronik
Fixes #1244, replaces #1254

The decision to use UInt32 only when the vertex count exceed the limit is for memory optimizing reasons?

Yeah, using 16 bit will result in smaller assets, and faster loading scenes (the impact would only be noticeable on mesh heavy scenes). With the index reusing implemented, more meshes will be able to use 16 bit. Please let us know if the new implementation works for you!
Since the 3.1.2 dll is not released yet, and this PR is for the upcoming 3.1.2 compatible version of the plugin, if you want to try it now you'd need to backport the parser to your fork (just copy and paste the stl parser from this PR)

@Balint-H I tested this with my use case and it works perfectly.


New command: allow uploading files using Microsoft Graph

Description
Looks like m365 still interact with sharepoint API for any operations. Is there a way to interact with Graph API version? Because we have sites.selected enabled
Steps to reproduce

login to m365
m365 spo site list list --weburl

Expected result
A list of sites
Actual result
401 Unauthorized because invalid_client
Environment
m365 version v3.8

For some commands we use Graph, for other, we use SharePoint APIs as typically they offer more features. This is by design. We can reconsider the implementation when the Graph API reaches feature parity with SharePoint.
What scenario are you working on?

File upload. The problem is that client doesn't want to grant app ability to read all sites. They enabled the new sites.selected permission and grant application to access a subset of sites. This works fine with graph API since it got the sites.selected scope. But doesn't work with sharepoint online API. Which returns 401 when trying to upload files after login

Understood. Thank you for the additional info. To stay backwards compatible, I think it would make the most sense for us to introduce a new command for us like file add that would use Microsoft Graph to upload files. Would that unblock you?

Yes, please. We just need file list and upload

@melcloud, I've updated the title of this issue to reflect what you'd like us to do. For the first version, would it be sufficient if we supported specifying the location where the file should be uploaded and the file to upload or are there other requirements that you'd need as well?

@waldekmastykarz Yeah, that sounds great. Can we also add file list? As long as I can get the list of file names back, it is good.

Definitely! That would be a separate command. Could you please create a separate issue for it so that we can track the work independently? Thank you!

Updated the issue to the spec for the command to implement


test site build fails, but publishing on staging site runs fine

Something is odd, this github action fails with my newest blogpost commit while the publishing github action still runs through: https://github.com/monal-im/monal-im.org/actions/runs/8494401147/job/23269572965

Yes, I got a notification via E-mail. I will check. 👍🏻

oh wow, a lot of problems :(
do you need any help from me regarding these?

I have a great idea if we want to solve the workflow problem first and then fix the remaining parts one by one.
If I give the --baseURL parameter to the command in this section, it will not give any error output. At the same time, it will not cause a compatibility problem with the serve parameter.

we can squash all commits into one while merging the pr, does that help? :)

Normally that would make sense. But after that we need to still delete ExpectedCommaWorkflow.yml (created for test) and change the parameter in your buildSite.yml file. That's why I proposed to change the code in place instead of squashing commits.
Should I use
https://monal-im.org or
https://monal-im.github.io/monal-im.org as --baseURL parameter in workflows?

You could add even one commit more deleting these files again, but sure, you can commit it directly, too :)
Since the deployment on the main website is run on directly on these servers, the workflow should use https://monal-im.github.io/monal-im.org
@FriedrichAltheide could you please change that on our two servers as well?


formatter should change f strings using single quotes with nested double quotes to use double quotes when target-version is py312

on older python versions, the following code is a syntax error:
f"{""}"

which is why the formatter does not convert the following f string to use double quotes:
f'{""}'

but on python 3.12, it's allowed, so the formatter should always convert them to double quotes. playground

This is available under preview style (https://github.com/astral-sh/ruff/discussions/9785) although it seems like the formatter still doesn't change the outer quote. For example, if the quotes are inverted (f"{''}"), the formatter changes the inner quote to use double quotes. This case (f'{""}') seems like a bug.


Simplify dwg2dxf

Signed-off-by: Uilian Ries<EMAIL_ADDRESS>Specify library name and version:  lib/1.0

[ ] I've read the guidelines for contributing.
[ ] I've followed the PEP8 style guides for Python code in the recipes.
[ ] I've used the latest Conan client version.
[ ] I've tried at least one configuration locally with the
conan-center hook activated.

Working on Fedora 31 and Windows 10. Thanks for the PR!


Item ID list is missing some items

The save editor is missing some Item IDs, mainly wall variants of songs and clothing, and art.
The Custom Pattern Item IDs and also Item Sprites are also missing. Also the cedar with an egg (just before xmas/festive lights cedar) is also missing.
Here's a list of alot of the missing Item IDs (Or here's the full file).
Also, for consistency with the list aswell as save editors for previous AC games and to also make it easier for people to name them, the rocks and weeds should probably use letters beside their names rather than numbers.
e.g.
"weed"      ->  "weed (A)"
"weed (2)"  ->  "weed (B)"
"weed (3)"  ->  "weed (C)"
"weed (4)"  ->  "weed (D)"

Thank you for posting this.
I'll take a look at this when possible. Not a priority right now.


Allow storage objects to be JSON data other than objects

Description
The method for write objects in database are validating wrong the json format in the value parameter.
This value works OK: { "carName": "Demon", "selected": 1 }
This value NOT works: [{ "carName": "Demon", "selected": 1 }]
Both formats are valid. The “[]” in json is for represent an array, and it’s valid.
Steps to Reproduce
Write an object that contains a value with a json array.
Expected Result
It should be valid. In fact, i tested insert manually a json with array in storage table, and i can. So, it sounds like it's a wrong validation in the method.
Actual Result
The error msg is: Object value must be JSON.
Context
[ x ] Cocos2d-x c++ 3.17
Your Environment

Nakama: Server 2.4.0 and C++ SDK (in development)
Cockroach: Build Tag 2.1.3
Operating System and version: macOS Mojave 10.14.3

The server currently requires all storage values to be JSON objects, meaning top-level {}. This constraint may be loosened later but for now this is intended behaviour.
I've pushed an update to clear up the error message in https://github.com/heroiclabs/nakama/commit/e588901dfc7092095731e8452c469e7139b2cfea and will leave this open for discussion on allowing other top-level JSON types.

You can easily get around this restriction by wrapping the array in a field, I recommend this issue be closed, ex:
{
    cars: [{ "carName": "Demon", "selected": 1 }]
}

Hey @bhgsbatista. Thanks for the feedback. So far we've left this open as an improvement in the future because there's actually no limiting reason that we couldn't allow storage object values to be JSON array literals or even primitives.
I think it was just a historic decision that we can improve on 😄

I'm not sure what the intended usage is, but given the collection and key name columns, I assumed each object was supposed to represent one value e.g. settings/game_volume_percentage = 60.
We can definitely wrap the value in an object but it would be cool to avoid the overhead of serializing { value: <value> }.

@bryanmylee It's usually best for storage object values to contain a bit more than a single field like that. Using your example, settings should be a complex object with multiple keys in the value:
{
  "game_volume_percentage": 60,
  "music_enabled": true,
  ...
}

Reducing the number of database operations is preferred to reducing serialization overhead, even accounting for reading all settings at once even if you only need to check some or updating the whole object value when changing one.


Graph edge property formatted as datetime causes error and breaks graph

Summary
Attempting to create an edge with a string property containing a date returns an error, but creates a corrupted edge which breaks subsequent queries which would contain that edge.
Reproduction
These steps show how to reproduce using TinkerPop Gremlin Console.
Setup
gremlin> :> g.addV('person').property('id', 'ted')
==>[id:ted,label:person,type:vertex]
gremlin> :> g.addV('person').property('id', 'bob')
==>[id:bob,label:person,type:vertex]

Create the edge
gremlin> :> g.V('ted').addE('knows').property('since', '2017-06-09T15:16:18.623Z').to(g.V('bob'))
Unexpected character ('/' (code 47)): maybe a (non-standard) comment? (not recognized as one since Feature 'ALLOW_COMMENTS' not enabled for parser)
 at [Source: [B@2604b954; line: 1, column: 303]

Query all edges
gremlin> :> g.E()
Unexpected character ('/' (code 47)): maybe a (non-standard) comment? (not recognized as one since Feature 'ALLOW_COMMENTS' not enabled for parser)
 at [Source: [B@4a10fb0d; line: 1, column: 1625]

Additional Info

It is possible to drop the broken edge, which then restores the ability to query all edges.
Putting a date/time string in a vertex property seems to work (but wasn't tested extensively).
Changing the "-" characters to "/" makes the expression work.
We have reproduced this issue also in C# using the Microsoft.Azure.Graphs Nuget package; it throws the following exception:

JsonReaderException: Error parsing comment. Expected: *, got 9. Path '[0].properties.since', line 1, position 182.

Hi @ChrisMiskowiec - thanks for catching this and we are in the process of fixing this.  We should have this resolved within the next week or so.  Please do not hesitate to ping me directly if you have any questions.

@ChrisMiskowiec , the issue is fixed. Can you please verify at your end and confirm.


Plugin do not recover after correcting an erratic .eslintrc

When I tweak .eslintrc and left .eslintrc in an erratic state (e.g. syntax error), this plugin will not recover after I correct the .eslintrc. Instead, a reload is needed to get the plugin working. I would expect the plugin to watch .eslintrc and recover when possible.

This works for me using a .eslintrc.json file without problem. Can you please provide me with a GitHub repository I can clone that demos what you are experiencing.

I cannot reproduce this. Probably a mistake by me. Sorry.


need to add tag path to estimateSpwnLoc function

in addition to knowing the final spawn location site and node for each tag, it would be nice to include a field that shows in a character string all the nodes that fish was seen at.

Done!


Bring main to int (And the drafted NTS changes are coming back)

Bringing this and a whole lotta main to int
https://github.com/transcom/mymove/pull/13466

Happo approved


Allow access to calculated query complexity

Early return when complexity is disabled
This allows other validation rules to depend on the QueryComplexity rule, and disable the query complexity completely on some condition.
$rules[QueryComplexity::class] = $queryComplexity = new QueryComplexity($this->maxQueryComplexity);

// Always enable the query complexity rule, but ignore for introspection queries
$rules[WhenIntrospection::class] = new WhenIntrospection(function () use ($queryComplexity) : void {
    $queryComplexity->setMaxQueryComplexity(QueryComplexity::DISABLED);
});

Allow access to calculated query complexity
This allows you to read the calculated query complexity. A use case could be: add a rule that logs when queries are detected with a complexity higher than X.
This helps with #1204
$rules[QueryComplexity::class] = $queryComplexity = new QueryComplexity($this->maxQueryComplexity);

$rules[WhenLeaveOperation::class] = new WhenLeaveOperation(function () use ($queryComplexity) : void {
    if ($queryComplexity->getQueryComplexity() >= 400) {
        $this->logger->info('Found a query with high complexity');
    }
});

@spawnia this one is ready for review, whenever you have time.


Add an easy way to change validation errors

There are some closed issues, but we really need an easy way, how to change the errors. Traversing is not a good option. We definitely need something better, this is a very common task.

I would like to know while traversing is not enough, as it works perfectly with gettext. Can you please give more information?

@josevalim actually, I just need to rewrite the error messages, I don't need to translate to any other languages.
Except traversing, the only option is gettext? Maybe there is a tutorial / repo / gist on how to do this?

@VladShcherbin most of the helpers accept a :message thing. The only ones that can be configured are "is invalid" and "can't be blank" which you can easily match on traverse_errors.
Ecto.Changeset.traverse_errors changeset, fn "can't be blank" -> "must not be blank"; x -> x end

@josevalim so, I still need to traverse the errors.. I'm using React and send validation errors via websocket. So, currently, I need to traverse all the errors with string.replace to insert values into the error message (like this one). Plus, I will need to change the message.
Don't you think, it is a bit hard to do this things (they are common for many apps)? I used Laravel and Meteor, handling errors is much easier there (via configs).
Maybe we can also have something more simple? Like an array of default errors and a way to change this array? Plus, a standard method to convert errors to json format so we don't need to reinvent and google the method for it?

@VladShcherbin Right. We know it is a bit painful but that's because we decided to keep things simple in Ecto for others to build on top. The only way to build those things so it works in multiple languages is by keeping strings and parameters apart.
Phoenix v1.1 has finally closed this gap by providing I18n. You now do have a list of all error messages, you can simply change the errors.po for english. The performance cost is very low: http://sevenseacat.net/2015/12/20/i18n-in-phoenix-apps.html

@josevalim great, thank you! I'll switch to this solution then.
So, probably my issue is that it's little hard to find this solution, a line in the docs somewhere would be great ;)

Thanks for the feedback, @VladShcherbin. To clarify, we were aware it was an issue and we are trying to improve it now, so we will keep an eye open to more complaints such as yours (which btw we love) and improve accordingly.

@josevalim yes, I've updated to 1.1, but skipped the gettext part as I thought it is only for internalisation.  I really hope, I am not too annoying with my questions and issues as I'm still a newcomer.
Thank you for your awesome work, I really-really love it!


is there a way to disable/enale offScope days rendering ?

Is your feature request related to a problem? Please describe.

On this image, dates of previous month are rendered, I want to decide to render it or not

@loicgeek thanks for helping out improving this package, I really appreciate it!
I would then probably add a config option month.strictScope, and hide days from previous as well as next month when set to true. If this doesn't sound right to you, please let me know

@tomosterlund  that would works.
Letting the abitily to hide previous and next. days of  month is also an option.
@tomosterlund

@loicgeek that's great. I think this could also be a great feature to start with, if you want to open a pull request. Do you want to give it a try?
You could fork this repo, and then instructions for how contribute are found here: https://github.com/tomosterlund/qalendar/blob/master/.github/contributing.md
Let me know at any time if you need help with setting the repo up. Also if you want a little "guided tour" of the source code we could set up a zoom call.

@tomosterlund sure I forked  the repo already, a code tour will help...
Let's setup a call for tomorrow, we will arrange the time by email, mine is<EMAIL_ADDRESS>


Add missing mapstructure annotations to config

Otherwise it wont unmarshall the config properly
Signed-off-by: Itxaka<EMAIL_ADDRESS>

reported by @kkaempf confirmed by me, missing the mapstructure means that viper wont be able to unmarshall it properly as it uses the mapstructure annotations to marshall those values properly. Dont ask me why, I no longer look at viper code as I want to stay sane.

errr... I was gonna add tests for this so we dont drop the mapstructure anymore :D


snet: Add extension support

Extension support has been in since https://github.com/scionproto/scion/pull/2411.


Flattened tooltips on fullscreen apps

Whenever your application is displayed under statusBar, tooltips are flattened.

Hi @uellee can you add a screenshot so I can be sure what you're seeing?

Closing this issue. Feel free to comment with further information.


pythonPackages.mwparserfromhell: init at 0.5.4

Motivation for this change
A Python parser for MediaWiki wikicode. mwparserfromhell is listed as a pip dependency for the existing textacy package.
Things done

Created new package for mwparserfromhell
Added self as maintainer

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

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

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

@GrahamcOfBorg build python.pkgs.mwparserfromhell python3.pkgs.mwparserfromhell


Python 3.6 incompatibility

Receiving error messages when using the latest version of python

Should work now. Let me know...


[VS Code] Starting collaboration session: Could not connect to the server. Live Share requires openssl 1.0 for authentication, please verify you have the dependency installed.

%0A%0A%23%23 Error:%0AStarting collaboration session: Could not connect to the server. Live Share requires openssl 1.0 for authentication, please verify you have the dependency installed.%0A%0A%23%23 Steps to Reproduce:%0A1.%0A2.%0A%0A||Version Data|%0A|-:|:-|%0A|**extensionName**|VSLS|%0A|**extensionVersion**|1.0.125|%0A|**protocolVersion**|2.2|%0A|**applicationName**|VSCode|%0A|**applicationVersion**|1.34.0-insider|%0A|**platformName**|Linux|%0A|**platformVersion**|4.19.0-4-amd64|

The formating has come out fairly poorly :\

@gcarothers Are you running on Ubuntu? If yes, your issue might be a dupe of #2030 . Please reopen if that is not the case. Thanks for using Live Share and please keep the feedback coming!

LiveShare version 1.0.181 was just released to the marketplace and contains a fix for this issue.


{Packaging} Bump azure-core to 1.24.0

Related command

Description
Some modules (Synapse) and extensions (AML) which use new SDK need newer version of azure-core, so we need upgrade it to the latest version (1.24.0).
Testing Guide

History Notes

[Component Name 1] BREAKING CHANGE: az command a: Make some customer-facing breaking change
[Component Name 2] az command b: Add some customer-facing feature

This checklist is used to make sure that common guidelines for a pull request are followed.

[ ] The PR title and description has followed the guideline in Submitting Pull Requests.

[ ] I adhere to the Command Guidelines.

[ ] I adhere to the Error Handling Guidelines.

Close this duplicated PR as https://github.com/Azure/azure-cli/pull/22676 does the same work.


Revert infinite specified as zero

Revert pull request #30.

No, I was wrong. It was the test that was broken.


UI fix Theme text color not bind navTextColorPick

ISSUE TYPE

Bug Report

COMPONENT NAME
UI Theme

CLOUDSTACK VERSION
<IP_ADDRESS> and 4.17 branch

CONFIGURATION
UI
OS / ENVIRONMENT
CentOS 8
SUMMARY
UI choose Theme , text color not bind correctly to navTextColorPick.
click "reset to default" button, will display last choosed color.
SOLUTION
vi ui/src/components/view/Setting.vue
 :73
change navBgColorPick to navTextColorPick
:257
add
this.uiSettings = {}

Fixed with #6271


kde: LookAndFeel never works on Plasma 6

Hey, I've been noticing since I started using stylix on KDE Plasma 6, the LookandFeel portion of the theme never actually gets applied. I look in the logs for Home-Manager and it says this:
Activating stylixLookAndFeel
Successfully set the wallpaper for all desktops to the KPackage based /nix/store/sw672rfici0x108fwgvmbld6mrp0xxjy-stylix-kde-theme/share/wallpapers/stylix
KPackageStructure of KPluginMetaData(pluginId:"stylix", fileName: "/home/novaviper/.nix-profile/share/plasma/look-and-feel/stylix/metadata.json") does not match requested format "Plasma/LookAndFeel"

Nix Information:
❯ nix-shell -p nix-info --run "nix-info -m"
 - system: `"x86_64-linux"`
 - host os: `Linux 6.8.6, NixOS, 24.05 (Uakari), 24.05.20240415.b941d52`
 - multi-user?: `yes`
 - sandbox: `yes`
 - version: `nix-env (Nix) 2.18.2`
 - nixpkgs: `/nix/store/panhvrac6ra4wpj9hrg19v4gn189hrqw-source`

Also I just noticed in the journalctl logs, the system is saying this
May 25 23:14:41 ryzennova plasma-apply-lookandfeel[159060]: "applications.menu"  not found in  QList("/home/novaviper/.config/menus", "/nix/store/6a4mg7j97949zmiid2z49bi4a0cqb1sk-plasma-workspace-6.0.5/etc/xdg/menus", "/etc/profiles/per-user/novaviper/etc/xdg/menus", "/run/current-system/sw/etc/xdg/menus")
May 25 23:14:41 ryzennova plasma-apply-lookandfeel[159060]: KPackageStructure of KPluginMetaData(pluginId:"stylix", fileName: "/home/novaviper/.nix-profile/share/plasma/look-and-feel/stylix/metadata.json") does not match requested format "Plasma/LookAndFeel"


Handle overlapping liquidation intervals for quanto curve

This is an edge case that most likely only appears in proptests. We already saw this in CI: https://github.com/itchysats/itchysats/runs/7685221192?check_suite_focus=true. But it's hard to avoid, so we either need to handle it or explicitly
reject the errors like we do with {Long,Short}OwesTooMuch.
I chose to handle it because it's less code. I chose to increase the lower bound of the short liquidation threshold rather than to
decrease the upper bound of the long liquidation threshold to avoid possibly getting a -1 as a result, which would panic.

I think a scenario where the liquidation intervals overlap actually does not make sense for our usecase and it would be fair to just reject it rather than doing the shift.
If it makes it easier to roll with this solution feel free to go ahead, I just feel it might be hard to understand this complexity when looking at it again after some time.

The problem is that the overlap unfortunately can happen with the code I wrote, so I have to do something about it if we want to keep the proptests (I think we definitely do).
I am also unhappy about the complexity, but the alternative is more complicated for me:

Checking if the overlap occurs (we already do this in this PR) and explicitly modelling a LiquidationIntervalsOverlap error variant to be returned instead of tweaking the intervals.
In the proptests, handling the new error variant to emit a proptest::TestCaseError::Reject so that the proptests don't fail when we encounter this.

bors r+
Going ahead with the merge, but we can reconsider this and change it afterwards.


[P2P, Runtime] Update P2P & base config (part 2)

Description
This is another of a series of PRs split out from https://github.com/pokt-network/pocket/pull/500. Here we update the base and P2P configs in preparation for the addition of the libp2p module.
Issue
#347
Type of change
Please mark the relevant option(s):

[x] New feature, functionality or library
[ ] Bug fix
[ ] Code health or cleanup
[ ] Major breaking change
[ ] Documentation
[ ] Other

List of changes

Replace consensus_port with port in P2P config.
Update default P2P config port to from 8080 to 42069.
Add use_libp2p field to base config.
Add hostname field to P2P config.
Update state hash test after modifying genesis (updated port numbers).

Testing

[x] make develop_test
[x] LocalNet w/ all of the steps outlined in the README

Required Checklist

[x] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have tested my changes using the available tooling
[x] I have updated the corresponding CHANGELOG

If Applicable Checklist

[ ] I have updated the corresponding README(s); local and/or global
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] I have added, or updated, mermaid.js diagrams in the corresponding README(s)
[ ] I have added, or updated, documentation and mermaid.js diagrams in shared/docs/* if I updated shared/*README(s)

Still have to add hostname to the k8s P2P config but I'm not sure how to do that just yet and won't have time in this sitting. Perhaps we can tag-team?
(cc @okdas @Olshansk)

Similar to the configs you updated in build/config/config1.json, you would do the same in build/localnet/manifests/configs.yaml and some of the other manifests. It seems like you already renamed the port from consensus_port but didn't add the hostname there.
Regarding Localnet and Tiltfile, a good starting point would be to look at build/localnet/templates/v1-validator-template.yaml.tpl which is used as a template by build/localnet/templates/v1-validator-template.sh. Since the naming convention for the scaling k8s validators is v1-validatorXXXX, that will likely need to be the hostname (don't quote me on this) for the cluster to resolve each container separately.
Hopefully that's a good enough starting point. Seeing how I'm OOO for ETH Denver next week, hopefully @okdas can help out with some pointers as well.


PPC emulation and analysis improvements

PowerPC analysis, emulation improvements.

please merge in from envi_ppc and resolve conflicts

sorry, i was a jerk and merged a change into master.  please resolve.


Feature: add support for Color Variables (Swatches)

I'm currently working on an implementation of this, but would appreciate any feedback before I go too far down the rabbit hole :)
Example usage:
const red = new Swatch({
    name: 'Primary/Red',
    color: new Color('#c00')
});
sketch.addSwatch(red);

Then use the Swatch anywhere you'd normally use a Color:
artboard.addLayer(new Rectangle({
    width: 100,
    height: 100,
    x: 10,
    y: 10,
    name: 'Red Box',
    style: {
        fills: [
            {
                color: red,
            },
        ],
    },
}));

:tada: This issue has been resolved in version 1.25.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Fix nodejs version issue

Fix for nodejs version issue using yarn

@azizali Any chance you can push out a v0.3.3-beta.1 with this fix?

Sure will do over the weekend
On Tue, Mar 23, 2021, 6:17 PM Aman Gupta Karmani @.***>
wrote:

@azizali https://github.com/azizali Any chance you can push out a
v0.3.3-beta.1 with this fix?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/azizali/react-super-treeview/pull/33#issuecomment-805343402,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAT6QLC7OUR5LJGK3X65OGTTFEOSDANCNFSM4ILAEDAQ
.

@azizali Friendly ping re: new release

Hi, really need a new release so all my yarn invocations don't keep failing as so:
error<EMAIL_ADDRESS>The engine "node" is incompatible with this module. Expected version "^6.7.0". Got "16.6.1"
error Found incompatible module.


Browser contents offset depending on where on the monitor the website is loaded

What version are you using? (exact version please e.g. 75.1.143, not Latest version as that's relative)

CefSharp.WinForms v73.1.130

Are you using WPF/WinForms/OffScreen?

WinForms

Are you targeting .Net Core? If yes which version?

No

I have a bit of an odd issue, just wondering if anyone's seen something like this before. It seems that on one monitor on my computer (just one - not both) the browser contents will be offset depending on where the application is located (or - maybe more applicable - where the CefSharp control is located) on the monitor. The mouse location still seems to trigger the same controls as if the contents were not offset.
Here's a screen recording: https://youtu.be/oo1UTtblhUs
A couple quick notes on my application:

The ChromiumWebBrowser control is set to Dock.Fill inside the "Player" tab
For now I've just overridden the "Add Source" button to call ChromiumWebBrowser.Load("https://www.youtube.com/embed/Y_4i2Oc2zAs"); for testing this issue

It seems to depend on where the browser is "shown". If I click the "Player" tab to show the browser control on the right monitor, then move it to the left monitor, then click the button the content offset can also be seen on the left monitor.
I'm aware I'm on an old version of CefSharp - just wondering if this is some sort of known issue or if things are just super weird on my computer

Do both monitors have the same DPI? Is your application DPI Aware?


test-branch/execute-job/quoted-authorization/69948411-0213-4099-9c69-17aedbd4188a

PR for testing ae4682d8061f010ef69505e1ba4af8127d47164e

/canonical/self-hosted-runners/run-workflows ae4682d8061f010ef69505e1ba4af8127d47164e


Drawing error when border size set to zero.

Describe the bug
There appears to be a minor issue with the borders if the border size is set to zero.
To Reproduce
Steps to reproduce the behavior:

Using the sample, go to 'Settings'
Set the 'Border Thickness' to zero (Click on the '-" button until it darkens).
Close the settings and observe the vertical border between different columns.  Resizing the window may be necessary to see the issue.
Resizing the window causes the vertical border issue to appear, disappear, or move.  There may be zero, one, or more than one vertical border.

Expected behavior
When the border size is set to zero, no vertical borders should be visible.
Screenshots
First example:

After changing the size of the window, the bar has moved:

Desktop (please complete the following information):

OS: Windows
Version 10.0.19044
Maui.DataGrid version 4.0.3

I think this is a windows-only bug in upstream MAUI or WinUI but I think it is fixed when you are using net8.0-windows10.0.22621.0
We use a combination of background color and padding at the DataGridCell level to produce a border effect. I had tried using ColumnSpacing at the DataGridRow level but I think it had some problems.
In any event, there are some simplifications at the cell level that I could try and you could see if it helps

Does this commit help?
https://github.com/akgulebubekir/Maui.DataGrid/commit/791401d894defe45ac4d611a63fb28b3246b9735

Just showing you that everything is working for me:

Updating WinUI on your system may have an effect?
https://www.microsoft.com/store/productId/9P4839KW4RH6?ocid=pdpshare

Wait, I can reproduce it but only if I resize to very specific widths.

Just tried it. Using ColumnSpacing works but has the same issue.


Emit a warning when no SDK is provided. We set sdk_location to empty …

…only in unit tests.
Be sure to do all of the following to help us incorporate your contribution
quickly and easily:

[ ] Make sure the PR title is formatted like:
[BEAM-<Jira issue #>] Description of pull request
[ ] Make sure tests pass via mvn clean verify. (Even better, enable
Travis-CI on your fork and ensure the whole test matrix passes).
[ ] Replace <Jira issue #> in the title with the actual Jira issue
number, if there is one.
[ ] If this contribution is large, please file an Apache
Individual Contributor License Agreement.

r: @charlesccychen

Changes Unknown when pulling ba47d199ce922225b450f26f8b0344fdf0b88dd6 on tvalentyn:add-warning-when-sdk-location-not-provided into ** on apache:master**.

Coverage remained the same at 69.148% when pulling daef4693e801b1f158e124010e971952f41e21bb on tvalentyn:add-warning-when-sdk-location-not-provided into c8dfb851cdcb24a9afb01416598d2881e449f079 on apache:master.

Coverage decreased (-0.007%) to 69.141% when pulling c0ac3b17870ab98013328bec452dc99f922a8d52 on tvalentyn:add-warning-when-sdk-location-not-provided into c8dfb851cdcb24a9afb01416598d2881e449f079 on apache:master.

Coverage decreased (-0.007%) to 69.141% when pulling fa87ab50964a29a90f6897561088eef2565b92e3 on tvalentyn:add-warning-when-sdk-location-not-provided into c8dfb851cdcb24a9afb01416598d2881e449f079 on apache:master.

Coverage decreased (-0.007%) to 69.141% when pulling fa87ab50964a29a90f6897561088eef2565b92e3 on tvalentyn:add-warning-when-sdk-location-not-provided into c8dfb851cdcb24a9afb01416598d2881e449f079 on apache:master.

Coverage decreased (-0.01%) to 69.134% when pulling fa87ab50964a29a90f6897561088eef2565b92e3 on tvalentyn:add-warning-when-sdk-location-not-provided into c8dfb851cdcb24a9afb01416598d2881e449f079 on apache:master.

LGTM


Js treehouse edits

The following are my proposed changes after an initial perusal of the new curriculum. Most are nitpicky grammar and clarity tweaks. Two larger changes are:

Refactoring the git introduction to eliminate the use of a deployable github.io project as the example code/video. This serves two purposes: a) To purge the final references to the personal site project b) to make the git intro more user-friendly and universally applicable.
Updating the JSON server introduction to remove references to local storage as a precursor to this more robust data persistence method, and making this chapter their first experience with manipulating data outside the boundaries of the application code.

My goal was to also update the segment on CSS BEM methodology, but I have struggled with making a clear and concise example that gives the student something that makes sense to them at a very early stage in the course, but also isn't so complicated that it scares them off. My feeling was that the current example was not enough content to create a clear picture of why/how to use it. But I've found that trying to sum it up in a single code example doesn't convey any sense of how to use it. I understand the goal is not absolute clarity and comprehension on the first iteration, but the current version feels a bit like like reading a pamphlet on using a stick shift while you're learning to drive an automatic for the first time. One just makes the other more likely to end badly.
I hope to collaborate with the team to come to a better solution than I've been able to come to on my own with this particular task.

@JoeShep :shipit:


locust task ratios can only be integers

Before this commit, the LMS loadtest would just crash on my machine.
Locust does not appear to tolerate float ratios.

@dianakhuang

👍


Variable names from included function prototypes lost in bndb

If I write an include file for binja to type imported functions, it applies the names from those function prototypes to argument variables. If I save and re-open the bndb, those names disappear.

CONGRATULATIONS ON ISSUE NUMBER 1000

Oh, it looks like argument types are lost too.
Interestingly, I think I can see them for an instant after visiting the function for the first time ... then they disappear.

Fixed in 1.1.1539-dev.